Anomalie #1265
closed
recherche libre ne fonctionne pas
100%
Description
La recherche libre (dernière partie de la recherche avancée semble ne pas fonctionner, au moins chez moi. :
La modification du sélecteur "free_field" entraîne une copie parasite du sélecteur suivant par dessus le sélecteur "free field".
Toutefois si,en sélectionnant à nouveau le champ de la recherche (grâce au fait que les sélecteurs superposés ont des tailles différentes) on arrive à rentrer un filtre, celui-ci n'est pas pris en compte.
Exemple, en rentrant "et" "nom" "contient" "blabla", la requête émise est :
SELECT DISTINCT `a`.* FROM `adherents_adherents` AS `a` INNER JOIN `adherents_statuts` AS `p` ON `a`.`id_statut`=`p`.`id_statut` LEFT JOIN `adherents_cotisations` AS `ct` ON `ct`.`id_adh`=`a`.`id_adh` ORDER BY `nom_adh` ASC, `prenom_adh` ASC LIMIT 50 OFFSET 0
Lorsqu'on essaie de rentrer un nouveau filtre à l'aide du bouton "+" - après avoir appliqué le patch donné pour le bug #1264 (https://bugs.galette.eu/issues/1264), la valeur initiale des sélecteur est vide. L'usage de ce nouveau filtre provoque également une superposition ... sur le filtre de la première ligne !
Configuration : Firefox 65.0.1 (64 bits) et chromium Version 65.0.3325.181 (Build officiel) sur Ubuntu 14.04
Je poursuis les investigations...
Updated by Johan Cwiklinski about 4 years ago
- Status changed from Nouveau to Fermé
- Assignee set to Johan Cwiklinski
- Target version set to 0.9.3
- % Done changed from 0 to 100
Je pense que ce problème est plus ou moins directement lié à #1264 ; le point commun étant la bibliothèque javascript utilisée depuis peu (chosen).
N'étant pas satisfait de "chosen", je l'ai remplacé pour la 0.9.3 par "selectize" qui semble poser beaucoup moins de problèmes finalement. Je viens de tester la recherche libre avancée, de base ça fonctionne.
Pour moi, ce problème est dores et déjà résolu (je vais essayer de ne pas trop tarder pour la 0.9.3, elle corrige plusieurs bugs).
Updated by Johan Cwiklinski about 4 years ago
Le patch 207443b99e3e5d86af5c1b1efc156ce5bd362b5a devrait corriger le problème dans la version actuelle.
Updated by Alain Castera about 4 years ago
Presque : ça corrige bien le défaut de fonctionnement du filtre (affichage et création de la recherche), mais pas l'erreur lors de la duplication (#1264->https://bugs.galette.eu/issues/1264). Il faudrait donc appliquer les deux patches... Mais si on le fait, la modification du format de recherche en fonction d champ ne marche plus !
Il faudrait donc pouvoir conserver le "span" pour la modification du format de recherche, tout en le neutralisant pour la duplication du filtre.
Je continuerai à chercher ...
Updated by Johan Cwiklinski about 4 years ago
Je viens de tester de nouveau, la recherche libre est pleinement fonctionnelle en version de développement et en version stable avec le seul patch fourni ici ; ajout et suppression de lignes inclus.
En revanche, si tu as tenté de faire d'autres modifs, le comportement pourrait être impacté.
Updated by Alain Castera about 4 years ago
Oups, pardon j'ai dû me prendre les pieds dans le tapis entre les différentes config de test.
Effectivement, ça marche : génial ! (comme toujours ;-) )
Il reste toutefois (sauf erreur de ma part) un petit bug, pas forcément grave : lorsqu'on sélectionne un champ date pour faire une recherche libre, les critères de recherche sont bien modifiés. Mais lrosqu'on revenient sur la recherche (par "modifier les critères de recherche"), ce sont à nouveau des critères de type "texte" qui sont affichés. Il faut sélectionner un autre champ, puis revenir au champ "date", et saisir à nouveau les critères souhaités.
Rien de bloquant donc... Si ça marche en 0.9.2.3, pas la peine de se prendre la tête, je pense.
Je porte la modif ("no chosen" dans mon patch pour la recherche sur plusieurs groupes https://bugs.galette.eu/issues/907).
Updated by Johan Cwiklinski about 4 years ago
Salut,
Pour le bug des champs date, je confirme en effet, mais ça ne semble pas récent non plus :/ Il faudrait créer un autre ticket, ce n'est pas en relation avec le problème d'origine.
Updated by Johan Cwiklinski about 4 years ago
Fait et corrigé en fait https://bugs.galette.eu/issues/1267 ;)