Project

General

Profile

Actions

Anomalie #1264

closed

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

Added by Alain Castera over 5 years ago. Updated about 5 years ago.

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

100%

Estimated time:
Version utilisée:

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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Alain Castera over 5 years 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...

Actions #3

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.

Actions #4

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

Actions #5

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

Actions #6

Updated by Johan Cwiklinski over 5 years ago

Actions #7

Updated by Johan Cwiklinski about 5 years ago

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

Updated by Johan Cwiklinski about 5 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF