Project

General

Profile

Anomalie #1264

erreur javascript dans recherche avancée (advancedsearch.tpl)

Added by Alain Castera 9 months ago. Updated 2 months ago.

Status:
Fermé
Priority:
Normal
Category:
IHM
Target version:
Start date:
03/12/2019
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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

galette912_initial.png (119 KB) galette912_initial.png 9.1.2 état initial Alain Castera, 03/13/2019 03:01 PM
galette912_click.png (119 KB) galette912_click.png 9.1.2 après avoir cliqué sur "+" Alain Castera, 03/13/2019 03:02 PM
galette921_initial.png (126 KB) galette921_initial.png 9.2.1 état initial Alain Castera, 03/13/2019 03:02 PM
galette921_click.png (130 KB) galette921_click.png 9.2.1 après avoir cliqué sur "+" Alain Castera, 03/13/2019 03:03 PM
advanced_search.tpl.patchduplication (1.13 KB) advanced_search.tpl.patchduplication Alain Castera, 03/15/2019 11:43 AM

Related issues

Has duplicate Galette - Anomalie #1268: Recherche avancéeRejeté03/27/2019

Actions

Associated revisions

Revision 207443b9 (diff)
Added by Johan Cwiklinski 9 months ago

Remove JS from free advanced search form to prevent issues; refs #1264 and #1265

Revision 1ecfbc81 (diff)
Added by Johan Cwiklinski 2 months ago

Fix freesearch with selectize; closes #1264

History

#1

Updated by Alain Castera 9 months ago

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...

#3

Updated by Alain Castera 9 months 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.

#4

Updated by Alain Castera 9 months ago

Si on tient au "span", il suffit de lui attribuer une classe particulière me semble-t-il...

#5

Updated by Johan Cwiklinski 9 months 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).

#6

Updated by Johan Cwiklinski 9 months ago

#7

Updated by Johan Cwiklinski 2 months ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
#8

Updated by Johan Cwiklinski 2 months ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF