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']['<nom_du_plugin>'] = true
et en utilisant la méthode
Plugin::RegisterClass("<classe_du_type>").
Une liste peut ainsi être générée par la fonction
plugin_<nom_du_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
_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev
