[SPIP Zone] [Spip-zone-commit] r37453 - in _plugins_/bandeau: . formulaires

Archives Zone
1

Les redirect n'ont rien à faire là, ni dans charger, et encore moins
en input hidden puisque le formulaire est naturellement posté sur la
page courante.

Par ailleurs, si redirect il doit y avoir :
- il est un bon usage de la passe en argument du formulaire, et non de
la mettre en dur dans charger()
- du coup traiter connait le redirect car il le reçoit en argument aussi
- ce n'est *surtout pas* à l'action de faire un redirect, mais à
traiter de renvoyer un 'redirect' dans ses résultats. La surcharge du
fichier action/converser était volontaire, justifiée, et légitime, je
ne m'amuse pas à surcharger pour le plaisir. J'aurais par contre du
intégrer cette version dans la 2.1, ce que j'ai oublié.

Et si on veut ne pas faire d'ajax, il suffit simplement de ne pas
mettre de classe ajax autour de la balise #FORMULAIRE.

Je te laisse remettre tout ça en ordre de marche.
Cédric

Le 20 avril 2010 15:49, <real3t@gmail.com> a écrit :

Author: real3t@gmail.com
Date: 2010-04-20 15:49:00 +0200 (Tue, 20 Apr 2010)
New Revision: 37453

Removed:
_plugins_/bandeau/action/
Modified:
_plugins_/bandeau/formulaires/configurer_langage.html
_plugins_/bandeau/formulaires/configurer_langage.php
Log:
Pourquoi faire en Ajax quelque chose qui doit changer toute la page ?
De plus, cela évite de forker un fichier de SPIP

Details: Connexion · GitLab

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

2

* Cédric Morin tapuscrivait, le 20/04/2010 16:24:

Les redirect n'ont rien à faire là, ni dans charger, et encore moins
en input hidden puisque le formulaire est naturellement posté sur la
page courante.

Par ailleurs, si redirect il doit y avoir :
- il est un bon usage de la passe en argument du formulaire, et non de
la mettre en dur dans charger()
- du coup traiter connait le redirect car il le reçoit en argument aussi
- ce n'est *surtout pas* à l'action de faire un redirect, mais à
traiter de renvoyer un 'redirect' dans ses résultats. La surcharge du
fichier action/converser était volontaire, justifiée, et légitime, je
ne m'amuse pas à surcharger pour le plaisir. J'aurais par contre du
intégrer cette version dans la 2.1, ce que j'ai oublié.

Et si on veut ne pas faire d'ajax, il suffit simplement de ne pas
mettre de classe ajax autour de la balise #FORMULAIRE.

Je te laisse remettre tout ça en ordre de marche.

OK.
Mais avant de le faire, la raison qui m'a amené à le faire c'est que le plugin Menu Langue Liens se prenait un accès interdit sur converser avec cet appel :
#URL_ACTION_AUTEUR{'converser', #LANG&var_lang=#LANG, #GET{ListeTraductions}|table_valeur{#LANG}|sinon{#SELF}}

Et tout ce que j'ai trouvé pour contourner le fonctionnement du Bandeau (avant d'enlever le Fork), c'est :
#URL_ACTION_AUTEUR{'converser', #LANG, #GET{ListeTraductions}|table_valeur{#LANG}|sinon{#SELF}}|parametre_url{var_lang,#LANG}|parametre_url{arg,''}|parametre_url{hash,''}

Donc, d'enlever des paramètre à l'URL générée par #URL_ACTION_AUTEUR
En l'occurrence, c'est peut-être qu'il manquerait à SPIP une balise #URL_ACTION qui ne passe ni arg, ni hash...

cf : Connexion · GitLab

Donc, je remets Bandeau comme avant ?

-- RealET

3

Le 20 avril 2010 16:38, RealET <real3t@gmail.com> a écrit :

* Cédric Morin tapuscrivait, le 20/04/2010 16:24:

Les redirect n'ont rien à faire là, ni dans charger, et encore moins
en input hidden puisque le formulaire est naturellement posté sur la
page courante.

Par ailleurs, si redirect il doit y avoir :
- il est un bon usage de la passe en argument du formulaire, et non de
la mettre en dur dans charger()
- du coup traiter connait le redirect car il le reçoit en argument aussi
- ce n'est *surtout pas* à l'action de faire un redirect, mais à
traiter de renvoyer un 'redirect' dans ses résultats. La surcharge du
fichier action/converser était volontaire, justifiée, et légitime, je
ne m'amuse pas à surcharger pour le plaisir. J'aurais par contre du
intégrer cette version dans la 2.1, ce que j'ai oublié.

Et si on veut ne pas faire d'ajax, il suffit simplement de ne pas
mettre de classe ajax autour de la balise #FORMULAIRE.

Je te laisse remettre tout ça en ordre de marche.

OK.
Mais avant de le faire, la raison qui m'a amené à le faire c'est que le
plugin Menu Langue Liens se prenait un accès interdit sur converser avec cet
appel :
#URL_ACTION_AUTEUR{'converser', #LANG&var_lang=#LANG,
#GET{ListeTraductions}|table_valeur{#LANG}|sinon{#SELF}}

Bien ça me parait normal, puisque tu envoies une action signée, qui
plus est en mettant n'importe quoi dans le arg (#LANG&var_lang=#LANG).
Ecris la correctement et ça ira mieux.

..>

Donc, je remets Bandeau comme avant ?

oui, stp, et tu peux améliorer en enlevant l'ajax qui n'est
effectivement pas judicieux sur ce formulaire.

Cédric

4

* Cédric Morin tapuscrivait, le 20/04/2010 17:06:

Le 20 avril 2010 16:38, RealET<real3t@gmail.com> a écrit :

* Cédric Morin tapuscrivait, le 20/04/2010 16:24:

Les redirect n'ont rien à faire là, ni dans charger, et encore moins
en input hidden puisque le formulaire est naturellement posté sur la
page courante.

Par ailleurs, si redirect il doit y avoir :
- il est un bon usage de la passe en argument du formulaire, et non de
la mettre en dur dans charger()
- du coup traiter connait le redirect car il le reçoit en argument aussi
- ce n'est *surtout pas* à l'action de faire un redirect, mais à
traiter de renvoyer un 'redirect' dans ses résultats. La surcharge du
fichier action/converser était volontaire, justifiée, et légitime, je
ne m'amuse pas à surcharger pour le plaisir. J'aurais par contre du
intégrer cette version dans la 2.1, ce que j'ai oublié.

Et si on veut ne pas faire d'ajax, il suffit simplement de ne pas
mettre de classe ajax autour de la balise #FORMULAIRE.

Je te laisse remettre tout ça en ordre de marche.

OK.
Mais avant de le faire, la raison qui m'a amené à le faire c'est que le
plugin Menu Langue Liens se prenait un accès interdit sur converser avec cet
appel :
#URL_ACTION_AUTEUR{'converser', #LANG&var_lang=#LANG,
#GET{ListeTraductions}|table_valeur{#LANG}|sinon{#SELF}}

Bien ça me parait normal, puisque tu envoies une action signée, qui
plus est en mettant n'importe quoi dans le arg (#LANG&var_lang=#LANG).
Ecris la correctement et ça ira mieux.

Effectivement, je sais pas si c'est correctement, mais ça va mieux :

...>

Donc, je remets Bandeau comme avant ?

oui, stp, et tu peux améliorer en enlevant l'ajax qui n'est
effectivement pas judicieux sur ce formulaire.

Fait : Connexion · GitLab

-- RealET