Projet

Général

Profil

Actions

Anomalie #1264

fermé

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

Ajouté par Alain Castera il y a environ 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
IHM
Version cible:
Début:
12/03/2019
Echéance:
% réalisé:

100%

Temps estimé:
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


Fichiers

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

Demandes liées 1 (0 ouverte1 fermée)

Dupliqué par Galette - Anomalie #1268: Recherche avancéeRejeté27/03/2019

Actions

Mis à jour par Alain Castera il y a environ 5 ans

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

Mis à jour par Alain Castera il y a environ 5 ans

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.

Mis à jour par Alain Castera il y a environ 5 ans

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Catégorie mis à IHM
  • Assigné à mis à Johan Cwiklinski
  • Version cible mis à 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).

Mis à jour par Johan Cwiklinski il y a presque 5 ans

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Résolu à Fermé
Actions

Formats disponibles : Atom PDF