Le 31/03/2012 22:44, David DURIEUX a écrit :
Voici le patch pour le ticket #2476


Voici ce que j'ai fait :

* Rajouté des options à la fonction 'Search::showGenericSearch' afin de
   pouvoir l'intégrer dans un formulaire (cf le screenshot)

* Scindé lea fonction 'Search::showList' en 2 afin de pouvoir avoir la
   requete SQL d'un coté, et l'affichage de l'autre. On a toujours la
   fonction 'Search::showList' mais j'ai rajouté la fonction
   'Search::constructSQL' qui construit la requête.

Salut,


merci pour le contribution.

Une remarque rapide, on en rediscute en direct si tu veux :
scinder la fonction en 2 pour moi ca voulait dire que la fonction même ne changeait pas mais qu'on appelait 2 fonctions dans cette fonction afin de ne pas changer le prototypage de celle-ci...
function showList(...) {
$datas = getDatas();
displayDatas($datas);
}

par exemple.
Après la grande question sur le sujet c'est est-ce que le retour du $result SQL est satisfaisant. Sur le ticket, il est indiqué une fonction qui renvoie un tableau des données et une autre qui affiche les données.

++

Julien






J'ai testé avec le plugin Monitoring sur la 0.84 et ça permet de faire
ce qu'on veut avec le moteur de recherche, cf la capture d'écran pour
une 'règle monitoring' se basant sur le moteur de recherche. Celà
fonctionne aussi dans ce même plugin avec uniquement l'utilisation de
la fonction 'Search::constructSQL' qui se charge de faire la requete,
laquelle que j'exploite derrière pour récupérer la liste des items.


J'ai fait pas mal de tests, je n'ai pas vu de régression.

David
++


_______________________________________________
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