Projet

Général

Profil

Bug #85623

cellule fiche(s) : champs listés en double dans le paramétrage personnalisé

Ajouté par Frédéric Péters il y a 4 mois. Mis à jour il y a 3 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
13 janvier 2024
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

https://agents-isere.test.entrouvert.org/manage/pages/67/#open:wcs_wcscardcell-44-display

Dans ce qui est produit par combo ça me semble correct, une seule mention, mais à l'affichage dans le select "Champs de la fiche" ça apparait en double.

J'imagine donc que quelque part bug dans le javascript.

Révisions associées

Révision dab4a2ae (diff)
Ajouté par Frédéric Péters il y a 3 mois

misc: only init card cell custom schema form once (#85623)

Historique

#3

Mis à jour par Frédéric Péters il y a 4 mois

Pété par #79989 (et l'absence de tests js), plus précisément https://git.entrouvert.org/entrouvert/combo/pulls/146#issuecomment-16955

@@ -1097,6 +1114,7 @@ Card_cell_custom.prototype = {
         this.grid_toggle();
     }).change();
     $(this.toggleBtn).on('change', (e) => {
+        this.on();
         this.grid_toggle();
     }).change();
   }
(

ça fait qu'on peut avoir deux fois le on() appelé, et donc deux fois les éléments ajoutés à la liste.

(vu via l'ajout d'un console.trace() en haut de grid_cell__init_form).

Ça semble volontaire de pouvoir appeler plusieurs fois .on(), pour passer du mode "carte" au mode "liste" (en tout cas si on l'empêche la zone de personnalisation est vidée quand on passe en mode liste, avec un juste un trait), si c'est bien le cas la correction pourrait simplement être de vider les select dans grid_cell__init_form.

#4

Mis à jour par Robot Gitea il y a 4 mois

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Robot Gitea il y a 3 mois

  • Statut changé de Solution proposée à Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Robot Gitea il y a 3 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Transition automatique il y a 3 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#8

Mis à jour par Transition automatique il y a environ un mois

Automatic expiration

Formats disponibles : Atom PDF