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).
Fichiers
Révisions associées
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)
Historique
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-misc-redirect-form-URL-to-include-category-slug-6954.patch 0001-misc-redirect-form-URL-to-include-category-slug-6954.patch ajouté
- Fichier 0001-misc-remove-redirection-to-category-formdef-handled-.patch 0001-misc-remove-redirection-to-category-formdef-handled-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ç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)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à 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)
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Résolu (à déployer) à En cours
Nope; ça oublie qu'il faut aussi les redirections pour /formulaire/<id>/; j'ai reverté :/
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-misc-redirect-form-def-data-URL-to-include-category-.patch 0001-misc-redirect-form-def-data-URL-to-include-category-.patch ajouté
- Statut changé de En cours à 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é).
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution proposée à 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.
Mis à jour par Frédéric Péters il y a plus d'un an
(je laisse de toute façon ça pour le cycle prochain, je ne vais pas le risquer à nouveau)
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à 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)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: remove redirection to /category/formdef, handled by w.c.s. (#69546)