jControllerDaoCrud - Ajout de la possibilité de modifier l'ordre d'apparition des résultats

[brunto]

Pouvoir classer l'ordre d'apparition des résultats en cliquant sur le titre de chaque colonne. A chaque clique l'ordre de la colonne passe de "asc" à "desc" puis à "null", ainsi de suite. L'ordre des cliques sur les titres est pris en compte.

[brunto]

Icon order

[brunto]

Patch

[laurentj]

le nom du dao n'est pas assez discriminant pour le stockage dans la session. On peut très bien avoir deux crud avec le même dao. il faut je pense plutôt utiliser module~controlleur.

+                if (array_key_exists($lo, $this->propertiesForListOrder)) {
+                    if (isset($_SESSION['CRUD_LISTORDER'][$this->dao][$lo]) && $_SESSION['CRUD_LISTORDER'][$this->dao][$lo] == 'asc')
+                        $_SESSION['CRUD_LISTORDER'][$this->dao][$lo] = 'desc';
+                    elseif (isset($_SESSION['CRUD_LISTORDER'][$this->dao][$lo]) && $_SESSION['CRUD_LISTORDER'][$this->dao][$lo] == 'desc')
+                        unset($_SESSION['CRUD_LISTORDER'][$this->dao][$lo]);
+                    else
+                        $_SESSION['CRUD_LISTORDER'][$this->dao][$lo] = 'asc';
+                }

ça manque de lisibilité. on pourrait passer par des variables temporaires pour éviter d'avoir à manipuler $_SESSION à toutes les lignes.

il y a une notice dans _indexSetConditions. tester l'existence de la variable de session. Il me semble qu'il y a moyen d'optimiser, de faire tout en une seule boucle.

[laurentj]

deuxième version du patch non corrigé, mais qui s'applique sans erreur et avec l'image incluse

[brunto]

• Modification pour le stockage dans la session en module~controlleur:dao utilisation de $this->_getAction($this->dao)

• Variable temporaire pour $_SESSION + allégement du code.

• Notice dans _indexSetConditions() corrigée.

[nesswaw]

Bonjour,

Ce ticket est toujours d'actualité? J'aurais une solution simple à proposer...mais je ne sais pas faire de patch...

[brunto]

Bonjour nesswaw,

Comme tu peux le voir, j'ai proposé un patch que tu peux trouver ci-dessus "jControllerDaoCrud-3.diff" Bien entendu libre à toi de proposer une solution.

Tu peux consulter la page suivante ​Proposer un patch afin de proposer ton patch.

[laurentj]

C'est intégré dans la branche 1.7 ;-)

​https://github.com/jelix/jelix/commit/238be05e46bb71f8524fb33a2e3aa95a424694da