Re: #FORMULAIRE_RECHERCHE, form_hidden, et params en trop dans le GET

Travailler sur les tests unitaires de la fonction form_hidden qui font office de spec, et éplucher le comment du pourquoi on a fait ça dans les logs pour éviter de casser quelque chose ?

Note bien que
1/ c’est parfaitement fonctionnel, ce que tu demandes n’est pas un bugfix mais une amélioration, donc ça ne peut pas casser le fonctionnel d’un autre cas
2/ form_hidden est la pierre angulaire de tous les CVT, donc achtung
3/ on est quand même dans un cas borderline, et donc il est pas délirant que dans ce cas précis tu doives surcharger/modifier ton formulaire html pour avoir pile poil ce que tu veux

Une piste peut-être, mais je n’ai ouvert aucun fichier pour regarder : ajouter un argument optionnel pour squizzer le decodage de l’URL et ne reinjecter que les arguments explicites de l’URL fournie ?

--
Cédric
Le 14 nov. 2019 à 13:38 +0100, RastaPopoulos <[email protected]>, a écrit :
>
> N'y a-t-il pas un problème avec #FORMULAIRE_RECHERCHE quand on veut
> diriger vers une autre page que celle par défaut ?
>
> On peut depuis longtemps passer une autre URL, sauf que dès que ce n'est
> pas une page=truc, mais un objet SPIP avec son URL libre, ça ajoute
> automatiquement des hidden qui aboutissent à des params en plus dans le
> GET qui n'ont aucune raison d'être.
>
> Si on fait #FORMULAIRE_RECHERCHE{#URL_RUBRIQUE} qui correspond à
> #FORMULAIRE_RECHERCHE{trucmuche}, on veut alors arriver à :
> domaine.com/trucmuche?recherche=prout
>
> Sauf qu'on arrive à :
> domaine.com/trucmuche?id_rubrique=123&page=rubrique&recherche=prout
>
> HORREUR.
>
> C'est |form_hidden qui fait ça, en étant couplé au fait que ça part en
> GET. Quand c'est en POST, ça ne se voit pas mais là pour la recherche on
> veut bien évidemment du GET, et personne ne veut ce résultat !
>
> M'est-avis que c'est un bug, et c'est pas à chacun de trouver une
> bidouille, mais à #FORMULAIRE_RECHERCHE de diriger vers une URL propre
> comme ce que tout le monde attend. Mais comment faire ?
>
> --
> RastaPopoulos
>
> _______________________________________________
> liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> doc: http://www.spip.net/
> dev: http://trac.rezo.net/trac/spip/
> irc://irc.freenode.net/spip