Bug #57082
Cellule Contenu d'une fiche // Ajout de champ lors de la personnalisation ne fonctionne pas
0%
Description
Sur cette page : https://agents-venissieux-test.demarches.sitiv.fr/manage/pages/304/
pour la cellule contenu d'une fiche (slug = test-pour-debug)
l'option "Personnaliser l’affichage" ne fonctionne pas.
L'action sur le bouton "Ajouter" ne provoque l'ouverture de la fenêtre permettant d'ajouter un champ.
Ce problème apparaît aussi sur d'autres pages / d'autres cellules du même type.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
Uncaught TypeError: _self.field_with_varname(...) is undefined grid_cell__set https://agents-venissieux-test.demarches.sitiv.fr/static/js/combo.manager.js:534 grid_cell__add https://agents-venissieux-test.demarches.sitiv.fr/static/js/combo.manager.js:546 grid_cell__init https://agents-venissieux-test.demarches.sitiv.fr/static/js/combo.manager.js:593 grid_cell__init https://agents-venissieux-test.demarches.sitiv.fr/static/js/combo.manager.js:592
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-manager-pass-over-deleted-fields-in-custom-cell-layo.patch 0001-manager-pass-over-deleted-fields-in-custom-cell-layo.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Frédéric Péters
- Patch proposed changé de Non à Oui
Il y a dans la cellule "informations_professionnelles" référence à un champ "metier" qui n'existe pas/plus dans le type de fiche associé. (et ça essaie de trouver l'info et ça échoue et ça fait planter la suite de la page).
(j'ai appliqué le patch sur l'instance pour le valider)
Mis à jour par Thomas Jund il y a plus de 2 ans
Tatillon côté sémantique
field_with_varname() va récupérer le field au niveau du shéma de la fiche (cardShema).
et stocker cette valeur dans une variable `cell_field` est confus je trouve.
`cell_field` sémantiquement paut faire référence aux champs json qui décrivent la cellule (provenant de `gridShema`) plutôt des champs qui décrivent l'élément de la fiche. (je ne pense pas être très clair dans mes propos)
Mais je préfèrement conserver la sémantique existante avec le prefix `cardShema`
- let cell_field = _self.field_with_varname(schema_cell.varname); - let cardShema_field = _self.field_with_varname(schema_cell.varname);
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-manager-pass-over-deleted-fields-in-custom-cell-layo.patch 0001-manager-pass-over-deleted-fields-in-custom-cell-layo.patch ajouté
et stocker cette valeur dans une variable `cell_field` est confus je trouve.
Patch modifié pour taper ça dans une variable nommée schema_field.
Mis à jour par Thomas Jund il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f4cafbc8bec6e03e7300422d12fad318681e5382 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun Sep 19 18:25:46 2021 +0200 manager: pass over deleted fields in custom cell layout js (#57082)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: pass over deleted fields in custom cell layout js (#57082)