spip
/
urls_etendues
14
0
Fork 1
Code Issues 8 Pull Requests 1 Releases Wiki Activity
Labels Milestones
New Pull Request

Refactoring des fonctions d'URLs #4811

Merged
cerdic merged 9 commits from issue_3311 into master 4 months ago
Conversation 0 Commits 9 Files Changed 12
cerdic commented 4 months ago
Owner

Suite de spip/spip#5000
Refactoring des fonctions d'urls pour une ecriture plus claire et mieux typee des fonctions avec 2 points d'entree pour decodage/generer l'url

Suite de https://git.spip.net/spip/spip/pulls/5000 Refactoring des fonctions d'urls pour une ecriture plus claire et mieux typee des fonctions avec 2 points d'entree pour decodage/generer l'url
marcimat reviewed 4 months ago
urls/html.php Outdated
* @param string $ancre
* @return string
*/
function urls_html_generer_url_objet_dist(int $id, string $objet, string $args = '', string $ancre = '') : string {
marcimat commented 4 months ago
Poster
Owner

En général il n’y a pas d’espace avant le : du typage du retour, soit donc string $ancre = ''): string {
mais un coup de fixer corrigera plus tard.

En général il n’y a pas d’espace avant le `:` du typage du retour, soit donc `string $ancre = ''): string {` mais un coup de fixer corrigera plus tard.
cerdic force-pushed issue_3311 from 291339592e to 9923ca512c 4 months ago
cerdic changed title from [WIP] Refactoring des fonctions d'URLs to Refactoring des fonctions d'URLs 4 months ago
cerdic added 1 commit 4 months ago
marcimat added 2 commits 4 months ago
marcimat approved these changes 4 months ago
JamesRezo approved these changes 4 months ago
b_b approved these changes 4 months ago
cerdic merged commit 831af16059 into master 4 months ago
cerdic deleted branch issue_3311 4 months ago

Reviewers

marcimat approved these changes 4 months ago
JamesRezo approved these changes 4 months ago
b_b approved these changes 4 months ago
The pull request has been merged as 831af16059.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
marcimat
JamesRezo
b_b
Labels
Apply labels
Clear labels
amélioration
Amélioration, nouvelle fonctionnalité bug
Ca ne fonctionne pas divers documentation
Documentation souhaite doublon
Ce ticket est un doublon invalide
Ticket invalide refusé
Ignoré, c'est comme Ca...
No Label amélioration bug divers documentation doublon invalide refusé
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'issue_3311'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes