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
