Anomalie #1264: erreur javascript dans recherche avancée (advancedsearch.tpl) - Galette - Galette bugs & features

Actions

Copier le lien

Anomalie #1264

fermé

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

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

Statut:

Fermé

Priorité:

Normal

Assigné à:

Johan Cwiklinski

Catégorie:

IHM

Version cible:

0.9.3

Début:

12/03/2019

Echéance:

% réalisé:

100%

Temps estimé:

Version utilisée:

0.9.2.1

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

Fichiers

Télécharger tous les 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 ouverte — 1 fermée)

Actions

Copier le lien

Mis à jour par Alain Castera il y a plus de 5 ans

Fichier galette912_initial.png galette912_initial.png ajouté

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

Copier le lien Télécharger tous les fichiers

Mis à jour par Alain Castera il y a plus de 5 ans

Fichier galette912_click.png galette912_click.png ajouté
Fichier galette921_initial.png galette921_initial.png ajouté
Fichier galette921_click.png galette921_click.png ajouté

Oups : les autres images ...

Actions

Copier le lien

Mis à jour par Alain Castera il y a plus de 5 ans

Fichier advanced_search.tpl.patchduplication advanced_search.tpl.patchduplication ajouté

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

Copier le lien

Mis à jour par Alain Castera il y a plus de 5 ans

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

Actions

Copier le lien

Mis à jour par Johan Cwiklinski il y a plus de 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).

Actions

Copier le lien