2014-04-10 17:36 GMT+02:00 MoYo <m...@indepnet.net>: > le correctif proposé change fondamentalement le comportement de la fonction > updateItemOnEventJsCode en ajoutant un déclenchement de l'action à la > définition de la gestion d'un événement. Les effets de bords à d'autres > endroits de GLPI me semble donc plutôt dangereux.
Je me suis fais la même réflexion que toi mais après avoir fais le test sur plusieurs formulaire de GLPI, le seul danger (si on peut appeler ça un danger) que j'ai pu voir en rajoutant le trigger, c'est qu'une requête ajax est faite en lieu et place d'un éventuel chargement initial des données par le PHP (ce qui à l'air de transparaître dans ton dernier commit) et duplique le code normalement déjà en place par cette requête ajax pour charger les données en fonctions des paramètres associés à l'évènement. > Je viens de commiter une correction pour le problème soulevé sans toucher à > cette fonction. Le problème est bien réglé avec ton dernier commit. > A voir par la suite pour reréflechir à une gestion plus simple de tout cela. Une fois mon "chantier" fusioninventory 0.85 terminé, je devrais pouvoir me pencher un peu plus sur le sujet. Cordialement, -- Kevin 'kiniou' Roy _______________________________________________ Glpi-dev mailing list Glpi-dev@gna.org https://mail.gna.org/listinfo/glpi-dev
