Development #69546
intégrer la redirection /formulaire/ → /categorie/formulaire/
0%
Description
Ce code est actuellement dans au-quotidien (avec ses bugs), le taf devrait se faire dans w.c.s. (sans les bugs).
Files
Associated revisions
misc: redirect form URL to include category slug (#69546)
Revert "misc: remove redirection to /category/formdef, handled by w.c.s. (#69546)"
This reverts commit efe6c53faffafd77527caf89833e6139cae6963b.
Revert "misc: redirect form URL to include category slug (#69546)"
This reverts commit 025d7e468d8f82bd8e01f0eb99161e501324642e.
misc: redirect form{def,data} URL to include category slug (#69546)
misc: remove redirection to /category/formdef, handled by w.c.s. (back) (#69546)
History
Updated by Frédéric Péters almost 2 years ago
- File 0001-misc-redirect-form-URL-to-include-category-slug-6954.patch 0001-misc-redirect-form-URL-to-include-category-slug-6954.patch added
- File 0001-misc-remove-redirection-to-category-formdef-handled-.patch 0001-misc-remove-redirection-to-category-formdef-handled-.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Ça amène quelques bouts moche mais ça doit gérer correctement les cas un peu nuls qu'on rencontrait type catégorie du nom d'une démarche.
(le comportement est plus uniforme mais ça fait quelque chose d'un peu différent d'auquotidien qui parfois affichait le formulaire sur /formdef/ sans redirection)
Updated by Benjamin Dauvergne almost 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Frédéric Péters almost 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 025d7e468d8f82bd8e01f0eb99161e501324642e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Oct 16 15:58:22 2022 +0200
misc: redirect form URL to include category slug (#69546)
Updated by Frédéric Péters almost 2 years ago
- Status changed from Résolu (à déployer) to En cours
Nope; ça oublie qu'il faut aussi les redirections pour /formulaire/<id>/; j'ai reverté :/
Updated by Frédéric Péters almost 2 years ago
- File 0001-misc-redirect-form-def-data-URL-to-include-category-.patch 0001-misc-redirect-form-def-data-URL-to-include-category-.patch added
- Status changed from En cours to Solution proposée
Voilà.
Et pour être honnête par rapport à l'ancien code qui tentait de jouer via _q_traverse, il y a le cas de l'url de catégorie sans / final qu'on gère moins bien (mais perso ça me va).
(aussi, je me suis demandé ici s'il était opportun de modifier les retours de l'API, pour inclure la catégorie dans les URL, mais j'ai continué à laisser de côté).
Updated by Thomas Noël almost 2 years ago
- Status changed from Solution proposée to Solution validée
Good.
Je me demandais pourquoi on ne voulait pas profiter de l'occasion pour supprimer cette redirection, mais le cas de la catérogie qui a le même slug qu'un formulaire suffit à ce que ça reste un bon fonctionnement. Cependant je reste d'accord que l'URL canonique, par exemple celle dans les API, reste sans le slug de catégorie ; ça autorise ainsi de modifier la catégorie d'un formulaire sans accroc (du moins dans une certaine mesure). Et j'arrête mon blabla.
Updated by Frédéric Péters almost 2 years ago
(je laisse de toute façon ça pour le cycle prochain, je ne vais pas le risquer à nouveau)
Updated by Frédéric Péters almost 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 8f43c01beabf4b859665203480926c12186d4470
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Oct 21 13:14:06 2022 +0200
misc: redirect form{def,data} URL to include category slug (#69546)
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: remove redirection to /category/formdef, handled by w.c.s. (#69546)