Anomalie #1264
closed
erreur javascript dans recherche avancée (advancedsearch.tpl)
100%
Description
J'ai un souci avec 9.2.1, dans la page recherche avancée : le bouton "+"
(addbutton) qui sert à afficher une ligne de recherche libre supplémentaire
ne fonctionne pas correctement : seul les deux premiers "select" (opérateur
booléen et nom du champ) sont recopiés, et mal remplis (valeur initiale
vide). La ligne ajoutée ne fonctionne pas correctement.
J'ai un comportement similaire avec mes modifications pour permettre la recherche dans plusieurs groupes (que je vais re-soumettre rapidement), et là il n'y a qu'un seul "select". Il est également mal recopié, avec un fonctionnement tout aussi bizarre. ce n'est donc pas lié au nombre de "select" dans le <li>
Aide au déboggage : 9.1.2 marche parfaitement, mais justement ce bouton y
est traité différemment (btnadd_small, class clearfilter).
J'ai le même comportement avec firefox 65.0.1 et Chromium 65.0.3325.181
(Build officiel) Built on Ubuntu , running on Ubuntu 14.04 (64 bits)
La console javascript affiche 2 messages, qui ne semblent pas être en rapport (enfin, je crois) :
Using //@ to indicate sourceMappingURL pragmas is deprecated. Use //# instead[En savoir plus] jquery-1.10.2.min.js:1
Erreur dans les liens source : request failed with status 404
URL de la ressource : https://[...]/webroot/js/jquery/jquery-1.10.2.min.js
URL du lien source : jquery-1.10.2.min.map
Files
Updated by Alain Castera over 5 years ago
- File galette912_initial.png galette912_initial.png added
Rectificatif : il y en en fait 2 bugs.
1 qui fait que les objets créés par le script sont mal remplis et fonctionnent mal : apparaît dans 0.9.2.1
(voir images (9.2.1 : duplication incomplète et options mal remplies, plus comportement aberrant à l'utilisation pas visible ici)
1 qui fait arrêter prématurément la duplication des objets dans la recherche libre (seul les deux premiers opérateurs logiques sont créés) : déjà présente dans 0.9.1.2 !
(voir images 9.1.2 : duplication incomplète mais options bien remplies)
Désolé, focalisé sur les modifs pour la recherche dans plusieurs groupes, ça m'avait échappé.
Du coup, il faudrait que je duplique ce ticket, mais je ne sais pas trop comment m'y prendre pour que ce soit clair...
Updated by Alain Castera over 5 years ago
- File galette912_click.png galette912_click.png added
- File galette921_initial.png galette921_initial.png added
- File galette921_click.png galette921_click.png added
Oups : les autres images ...
Updated by Alain Castera over 5 years ago
Bon, il y a bien deux bugs distinct.
Le premier (mauvaise recopie des champs de la recherche libre), déjà présent dans 0.9.1.2, vient de la présence d'un <span> au milieu de la ligne à dupliquer. Le correctif ci-dessous se contente de supprimer ce span.
L'autre problème n'est présent que dans 0.9.2.1, et je vien de découvrir que, de toutes façons, la recherche libre ne marche pas dans 0.9.2.1 - au moins chez moi- même sans duplication. J'ouvre donc un ticket séparé pour ça.
Updated by Alain Castera over 5 years ago
Si on tient au "span", il suffit de lui attribuer une classe particulière me semble-t-il...
Updated by Johan Cwiklinski over 5 years ago
- Category set to IHM
- Assignee set to Johan Cwiklinski
- Target version set to 0.9.3
Bon, le problème, c'est « juste » la bibliothèque javascript. Elle remplace les éléments select standards par un amas plus complexe de HTML. La fonctionnalité d'ajout de ligne en recherche libre n'a jamais été prévue pour ça, et en conséquence ; ça fait n'importe quoi :/
Le patch 207443b99e3e5d86af5c1b1efc156ce5bd362b5a empêche juste que le JS se préoccupe de ces éléments, et tout refonctionne sans soucis. Je laisse ce ticket ouvert, car il s'agit d'un palliatif, il faudrait que les éléments soient correctement gérés avec le javascript actif (ça permettrait la recherche dans la liste des champs par exemple).
Updated by Johan Cwiklinski over 5 years ago
- Has duplicate Anomalie #1268: Recherche avancée added
Updated by Johan Cwiklinski about 5 years ago
- Status changed from Nouveau to Résolu
- % Done changed from 0 to 100
Appliqué par commit 1ecfbc817908e147167a448b020e1721e2fda72e.
Updated by Johan Cwiklinski about 5 years ago
- Status changed from Résolu to Fermé