Refactoring des fonctions d'URLs #4811

Merged
cerdic merged 9 commits from issue_3311 into master 8 months ago
cerdic commented 9 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 9 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 {
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 9 months ago
cerdic changed title from [WIP] Refactoring des fonctions d'URLs to Refactoring des fonctions d'URLs 9 months ago
cerdic added 1 commit 9 months ago
marcimat added 2 commits 9 months ago
marcimat approved these changes 8 months ago
JamesRezo approved these changes 8 months ago
b_b approved these changes 8 months ago
cerdic merged commit 831af16059 into master 8 months ago
cerdic deleted branch issue_3311 8 months ago

Reviewers

marcimat approved these changes 8 months ago
JamesRezo approved these changes 8 months ago
b_b approved these changes 8 months ago
The pull request has been merged as 831af16059.
Sign in to join this conversation.
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.