Le Thu, 25 Oct 2012 08:26:10 +0200 Julien Dombre <m...@indepnet.net> a écrit:
>Le 24/10/2012 22:08, David DURIEUX a écrit : >> Je suis en train de bosser sur la réécriture de la classe Search >> (ticket 2476) et je sui stombé sur un cas spécial, dans le >> getSearchOptions de la classe Change, le forcegroupby de l'id est à >> true : >> >> $tab[1]['forcegroupby'] = true; > >Salut, > >ce genre de comportement qui n'est pas isolé à Change (8 classes dans >ce cas). >Le but de cette définition est de dire que quoiqu'il arrive pour cet >itemtype il faudra faire un group by. >Exemple : pour les utilisateurs car on lie ses droits à chaque fois ou >les reminder pour la visibilité >Ce n'est pas forcèment la solution idéal pour définir ce comportement >mais pour le moteur de recherche la colonne name étant toujours >affichée cela fonctionnait. > >Par contre pour Problem et Change je ne vois pas l'intérêt. >Je vais faire un passage sur le code pour vérifier tout ca. >Sur ton travail s'il faut définir ce comportement de Group By forcé >globalement pour l'itemtype ailleurs on peut aussi. Oui c'est que j'ai rendu le Search un peu plus générique et en faisant le tour, je suis tombé sur ces 2 cas (car le 'name' sert à avoir le lien vers le form de l'objet). La c'est spécifique au forcegroupby du getSearchOptions pour le champs 'name' de l'itemtype Je peux modifier le comportement mais je me suis posé la question de savoir si c'était requis ou pas. >++ > >Julien > > >> >> Est-ce normal? Aucun autre getSearchOptions portant sur le champs >> 'name' n'a le forcegroupby à true. >> >> La raison c'est que dans le moteur de recherche la requete SQL va >> nous sortir un 'name$$1' au lieu de 'name$$$$1' (1 étant l'id) >> >> 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 _______________________________________________ Glpi-dev mailing list Glpi-dev@gna.org https://mail.gna.org/listinfo/glpi-dev
