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
list
>> Glpi-dev@gna.org
>> https://mail.gna.org/listinfo/glpi-dev
--
Tsmr
Xavier CAILLAUD
_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev
