Bonjour
Je pense que c'est une bonne amélioration. Il faudra vérifier l'impact sur
SimCards et le patcher en conséquence.
Le 3 septembre 2014 10:32, Tsmr <t...@thetsmr.fr> a écrit :
> Du moment que ceci est appliqué lors d'un passage dans une version
> majeure et que c'est documenté dans la procédure d'upgrade des plugins sur
> le wiki de la forge, pas de souci de mon coté.
>
> Le 03.09.2014 10:03, Julien Dombre a écrit :
>
> Bonjour,
>
> De mémoire c'était pour que la configuration des plugins ne soit faites
> que dans les plugins.
> J'imagine que certains plugins gèrent l'associabilité directement dans la
> conf de leurs plugins.
> Si on applique ton patch ca voudra dire qu'on bascule complètement cette
> conf dans la conf générale et je n'aimerai pas que ca pète les autres
> plugins.
> C'est faisable mais il faut que les autres mainteneurs de plugins qui
> utilisent cette fonctionnalité valide cette bascule.
>
> Qu'ils se manifestent donc où se taisent à jamais :)
>
> ++
>
> Julien
>
>
>
> Le 02/09/2014 17:30, Kevin Roy a écrit :
>
> Bonjour,
>
> Actuellement, les plugins peuvent enregistrer des types d'objets pour
> pouvoir les assigner à un ticket grâce à
> $PLUGIN_HOOKS['assign_to_ticket'][''] = true
> et en utilisant la méthode
> Plugin::RegisterClass("").
>
> Une liste peut ainsi être générée par la fonction
> plugin__AssignToTicket($types)
> qui est utilisée dans la méthode
> CommonITILObject::getAllTypesForHelpdesk().
>
> Jusqu'ici tout va bien :) ... mais les 2 points suivants
> empêchent la pleine intégration de ces types dans GLPI :
>
> 1. Ces types n'apparaissent pas dans la liste des éléments associables
> autorisés pour un profil (ie. onglet Assistance du formulaire d'un
> Profil) à cause d'une condition empêchant les types provenant de
> plugins d'y être répertoriés.
> À mon avis, on peut faire sauter cette condition vu que les plugins
> peuvent déjà utiliser le hook (... à moins qu'il y ait d'autres
> implications que mon grep n'a pas relevé).
>
> 2. Si la correction du point précédent est acceptée, la méthode
> CommonITILObject::getAllTypesForHelpdesk() devrait aussi filtrer
> ces types par la liste des types autorisés pour le profil actif,
> c'est à dire $_SESSION["glpiactiveprofile"]["helpdesk_item_type"].
> Cela peut être fait côté plugin mais comme cette méthode le fait
> déjà pour les types du coeur, je pense qu'elle devrait vérifier
> aussi les types des plugins.
>
> J'ai fait plusieurs tests avec le patch ci-joint en reliant un objet
> et en supprimant de la base et je n'ai pas encore constaté de
> surprise.
>
> --
> Kevin Roy
>
>
>
> _______________________________________________
> Glpi-dev mailing listGlpi-dev@gna.orghttps://mail.gna.org/listinfo/glpi-dev
>
>
> --
>
> Tsmr
> Xavier CAILLAUD
>
>
> _______________________________________________
> 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
