Le Sat, 31 Mar 2012 23:14:18 +0200 Julien Dombre <m...@indepnet.net> a écrit:
>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); >} Ah oui en effet, on peut faire comme ça ;) >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. Je me disait que renvoyer le $result serait plus simple que de renvoyer un gros tableau (traitement pour renvoyer un array() au lieu de renvoyer le $result et faire un foreach derrière). Dis moi ce qui te semble le mieux : tableau de donnée ou $result. David ++ >++ > >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