Salut,
Ticket créé.
https://forge.indepnet.net/issues/2635
On doit pouvoir faire en sorte de garder la compatibilité existante en
ajoutant cette gestion sous forme de tableau.
++
Julien
Le 28/01/2011 15:36, François LEGASTELOIS a écrit :
Posted by *Tsmr* on January 27, 2011 - 13:06:
On Thu, 27 Jan 2011 12:55:19 +0100 (CET), François LEGASTELOIS
<flegastelois@xxxxxxxxxx> wrote:
Bonjour à tous,
En bossant sur les redirections via les notifications je me suis
aperçu d'une chose :
- on peut rediriger une personne sur une page du style
/plugins/order/front/order.form.php
- on ne peut pas rediriger une personne sur une page du style
/plugins/order/front/reference.form.php
Ce qui implique qu'on peut faire qu'une seule redirection pour un
plugin :
- et donc une seule entrée $PLUGIN_HOOKS['redirect_page']['order']
Je propose le patch en pièce jointe pour améliorer ceci :
- l'entrée dans le setup du plugin est maintenant (par ex)
: $PLUGIN_HOOKS['redirect_page']['order-reference']
= "front/reference.form.php";
- l'url affichée dans le mail est la suivante (par ex) :
http://localhost/index.php?redirect=plugin_order-reference_1
- l'utilisateur est redirigé directement vers
: http://localhost/plugins/order/front/reference.form.php?id=1&forcetab=1
Qu'en penses-vous ?
++
François
--
François Legastelois (flegastelois@xxxxxxxxxx)
teclib' - Consultant Applications OpenSource - http://www.teclib.com
tel : 06 84 59 42 62 / 01 79 97 02 78
Ca ne serait pas plus propre avec un sous tableau ?
Type :
$PLUGIN_HOOKS['redirect_page']['order']['order']
$PLUGIN_HOOKS['redirect_page']['order']['reference']
?
Salut Xav,
Si effectivement ça serait plus propre, mais qui de la compatible avec
les anciennes versions ?
Si on rajoute une dimension au tableau, tu penses gérer l'appel de
l'url comme ça : plugin_order_reference_3 ?
Dans ce cas, le "explode" réalisé dans common.function doit être
modifié, et tous les plugins qui se servent de cela aussi...non ?
++
--
François Legastelois (flegastel...@teclib.com)
teclib' - Consultant Applications OpenSource - http://www.teclib.com
tel : 06 84 59 42 62 / 01 79 97 02 78
_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev
_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev