2014-09-10 16:03 GMT+02:00 MoYo <m...@indepnet.net>: > Le 10/09/2014 15:59, Kevin Roy a écrit : > >> 2014-09-10 14:22 GMT+02:00 MoYo <m...@indepnet.net>: >>> >>> est-ce que ce fix n'est pas plus simple : >>> >>> https://forge.indepnet.net/projects/glpi/repository/revisions/23158/diff/trunk/inc/search.class.php >> >> Ça fonctionne mais ça me rajoute 2 fois l'itemtype en paramètre GET : >> >> plugins/genericobject/front/object.php?itemtype=PluginGenericobjectCar&itemtype=PluginGenericobjectCar&sort=1&order=DESC&start=0... >> >> L'idée du patch c'était surtout de réutiliser les fonctions communes >> getSearchURL() comme pour les précédents patchs (c.f. >> Search::showGenericSearch et Search::prepareDatasForSearch). > > > Oui j'ai bien compris mais normalement le $data['search']['target'] doit > déjà provenir de ces fonctions donc je ne vois pas trop l'intérêt de le > recalculer. > Si 2 fois l'itemtype est génant on peut conditionner sont ajout
Ce n'est pas gênant pour faire le tri, ça rallonge juste l'url. Par contre je ne comprends pas l'ajout de l'itemtype à cet endroit si la variable data['search']['target'] contient déjà la base de l'url de recherche à laquelle on colle les paramètres de tri et de recherche. Je viens de faire l'essai en enlevant ce paramètre de l'url de plusieurs type et le tri se fait correctement. -- Kevin Roy _______________________________________________ Glpi-dev mailing list Glpi-dev@gna.org https://mail.gna.org/listinfo/glpi-dev
