0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

sarabanjina · 2011-06-07 14:36:36

Bonjour,

Il semble que les marques-pages n'enregistrent pas le tri sur colonnes dans les tickets UNIQUEMENT quand un autre critère est sélectionné :
- si je fais une recherche simple (raz), que je trie sur la colonne "priorité" et que je sauvegarde un marque-page, ça fonctionne
- si je fais une recherche avec des critères, que je trie sur la colonne "priorité" et que je sauvegarde un marque-page, le tri sur colonne est perdu au rechargement du marque-page.

En regardant la table glpi_bookmarks, je vois qu'il n'enregistre pas les paramètres "sort=3&order=DESC" dans la requête, ce qui semble normal vu que ces paramètres ne sont plus dans l'url de la page.

En analysant un peu le code, je me suis rendu compte que le bouton "Rechercher" ne prenait pas en compte l'ordre de tri des colonnes dans l'url.

Voici un patch qui corrige le problème, mais je ne suis pas sûr que ce soit la bonne solution :

--- inc/search.class.php.orig   2011-06-07 14:28:06.000000000 +0200
+++ inc/search.class.php        2011-06-07 14:28:18.000000000 +0200
@@ -1495,6 +1495,10 @@
       echo "</td></tr>";
       echo "</table>\n";
 
+      // For order
+      echo "<input type='hidden' name='sort' value='".$p['sort']."'>";
+      echo "<input type='hidden' name='order' value='".$p['order']."'>";
+
       // For dropdown
       echo "<input type='hidden' name='itemtype' value='$itemtype'>";

Nicolas

yllen · 2011-06-07 16:10:17

Il faudrait que vous passiez en 0.78.5, car ce problème a été corrigé pour moi en r14343

sarabanjina · 2011-06-07 17:39:27

Bonjour,

Comme le titre l'indique (et pas ma signature), le problème a été constaté sur un GLPI 0.78.5 chez un client.
Si vous faites référence au bug signalé ici http://www.glpi-project.org/forum/viewt … p?id=23966, je l'avais déjà lu et je ne pense pas que ce soit le même problème : je n'ai aucun message d'erreur.

Pouvez-vous essayer de créer un marque-page avec au moins 2 critères et un tri de colonne sur une version 0.78.5?
Pour moi, ça ne peut pas fonctionner, car les variables "sort" et "order" ne font pas partie de l'url que le marque-page enregistre.

Je parviens également à reproduire ce bug sur la version 0.80 de démo (http://demo.glpi-project.org).

Nicolas

yllen · 2011-06-08 09:49:15

Le problème que vous indiquez ne se produit que si vous avez le second critère en type Est
Si vous faites un test avec 2 critères Contient, tout marche OK

remi · 2011-06-08 10:00:00

à tester:
https://forge.indepnet.net/projects/glp … 14664/diff

sarabanjina · 2011-06-08 10:02:06

Bonjour,

Je pense que ça dépend de l'ordre dans lequel vous effectuez les opérations :
- si vous ajoutez des critéres, puis classez les colonnes, puis sauvegardez sans pousser sur le bouton "Rechercher", le tri sur colonne sera effectivement sauvegardé, mais les critères de recherche sont transformés en "contient".
- si vous triez sur une colonne, puis ajoutez des critères, puis sauvegardez, le tri sur colonne est perdu, mais les critères de recherche sont préservés ("est" reste "est").

Nicolas

sarabanjina · 2011-06-08 10:14:31

Le patch semble fonctionner dans le premier cas et corrige la transformation du "est" en "contient", mais j'obtiens un message "Chargement partiel du marque-page".
Dans le deuxième cas, le tri sur colonne est toujours perdu.

Nicolas

remi · 2011-06-08 10:27:19

Est-ce que le patch pointé par yllen est appliqué ?

Et il faut aussi recréer les marque-pages

P.S. : https://forge.indepnet.net/issues/2845

sarabanjina · 2011-06-08 13:37:05

Bonjour,

Désolé pour mon dernier test, j'ai effectivement testé sur une version 0.78.2 sans appliquer le patch pointé par yllen.
Avec ce patch, plus de message "Chargement partiel du marque-page".
J'ai effacé tous les marque-pages et en ai recréé un et ça semble fonctionner comme prévu : on peut enregistrer le tri sur la colonne juste après l'avoir sélectionnée.
Par contre, si on ajoute un critère après avoir sélectionné la colonne, l'affichage reste trié, mais le marque-page n'enregistre pas le tri de la colonne.
Ca semble être le fonctionnement normal, mais ce n'est pas très intuitif pour l'utilisateur.

Pour moi, ce bug peut-être marqué comme résolu, car il est maintenant possible d'enregistrer une recherche multi-critères et un tri sur colonne dans un marque-page quand on sait dans quel ordre le faire (plus qu'à le documenter quelque part pour les utilisateurs).

Nicolas

Forum GLPI-Project

Announcement

#1 2011-06-07 14:36:36

0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#2 2011-06-07 16:10:17

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#3 2011-06-07 17:39:27

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#4 2011-06-08 09:49:15

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#5 2011-06-08 10:00:00

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#6 2011-06-08 10:02:06

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#7 2011-06-08 10:14:31

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#8 2011-06-08 10:27:19

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

#9 2011-06-08 13:37:05

Re: 0.78.x/0.80 Enregistrement du tri des colonnes dans les marques-pages

Board footer