Re,

Autre solution gérer 2 noms de fonctions : _addWhere et _addSpecificWhere par exemple pour gérer la transition en douceur (et donc temporairement appel des 2).

D'ailleurs il y a 2 appels à _addWhere dans search.class.php

++

Julien


Le 26/05/2014 17:20, David DURIEUX a écrit :
Le Mon, 26 May 2014 17:02:00 +0200
MoYo <m...@indepnet.net> a écrit:

Bonjour,

Je suppose que tu parles de la classe search et non ticket...
Oulà oui, la classe Search (moi fatigué :p)

Effectivement il manque quelquechose.
Il faut l'ajouter en essayant de faire en sorte que ca soit
transparent pour les plugins qui définissent cette fonction (pas
gagné..)
Tu as raison, ça va péter les plugins

* Pour la 0.84 je propose :

Une des solutions serait de vérifier le nombre d'arguments que prend la
fonction du plugin qu'on va appeler avec :

$functotest = new ReflectionFunction('plugin_xxxx_addwhere');
$num = $functotest->getNumberOfParameters();


C'est moche, mais ça permet de voir si la fonction attend l'argument
'searchtype' et appeler soit avec, soit sans.

* Pour la 0.85, je propose :

On appelle directement avec le searchtype et on demande aux mainteneurs
de plugins de rajouter ce paramètre dans leur function de hook addwhere


David
++

Julien


Le 26/05/2014 16:43, David DURIEUX a écrit :
Bonjour,

En 0.84.5, dans la classe ticket, ligne 3132, on appelle le hook
addwhere du plugin mais on ne passe pas le $searchtype. Du coup,
impossible de savoir si c'est un EST, CONTIENT...

Est-ce que quelqu'un voit un problème à rajouter cette variable à
l'appel de cette fonction ?

merci ;)

Cordialement,
--
David DURIEUX
Tel : +33 (0)4.82.53.30.53
Mail : d.duri...@siprossii.com
Site Web : http://www.siprossii.com/

SIPROSSII
Rue des jardins
69860 Monsols
FRANCE

_______________________________________________
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


_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Reply via email to