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

Reply via email to