Development #54259
cellule fiche: ajouter checkbox "personnaliser" et input (hidden]
0%
Description
Pour permettre de finaliser une version testable de l' IU de personalisation :
Ajouter
- une checkbox "Personnaliser" qui permettra d'afficher de manière contitionnel l'UI de peronnalisation
- un input[hidden] qui stockera le shéma de personalisation.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Lié à Development #52073: composition graphique grille fiche ajouté
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Fichier 0001-wcs-add-fields-to-customize-Card-display-54259.patch 0001-wcs-add-fields-to-customize-Card-display-54259.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Statut changé de Solution proposée à Information nécessaire
J'ai un soucie avec le format JSON retourné dans la value de l'input "*-custom_shema" suite à un enregistrement asynchrone.
Détails du bug
- J'assigne un JSON.string à la value de l'input
{"prenom":"Thomas"}
- J'enregistre (enregistrement asynchrone)
- La cellule se rafraichie et l'input retourne la value avec des single quotes
{'prenom':'Thomas'}
- JSON.parse error
Uniquement en asynchrone
En effet, la value est bien sauvegarder, et si je recharge toute la page, l'input retourne un string correct, avec double quotes.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
Autre problème recontré.
À l'enregistrement, la checkbox "Customize display :" perd sa valeur : si elle est cochée, elle est retournée décochée.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Lié à Development #52073: composition graphique grille fiche supprimé
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Bloque Development #52073: composition graphique grille fiche ajouté
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Fichier 0001-wcs-add-fields-to-customize-Card-display-54259.patch 0001-wcs-add-fields-to-customize-Card-display-54259.patch ajouté
- Statut changé de Information nécessaire à Solution proposée
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Bloque Development #52073: composition graphique grille fiche supprimé
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Lié à Development #52073: composition graphique grille fiche ajouté
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
De mon côté c'est tout bon.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
Le bug du checkbox 'Customize display' qui se recoche après enregistrement me semble toujours d'actualité :
Test :
au départ, checkbox on avec customisation existante; je décoche la checkbox et enregistre; la cellule se rafraichie en ajax et la checkbox se retrouve à nouveau cochée et la grille de customisation présente.
Mis à jour par Frédéric Péters il y a presque 3 ans
De ce que je comprends, l'information n'est pas enregistrée dans le modèle, c'est juste une case à cocher du formulaire; s'il y a schéma spécifié c'est qu'il est à utiliser,
if self.instance.custom_schema: self.initial['customize_display'] = True
Il me semble que ce que tu souhaiterais c'est à côté de l'enregistrement du schéma que se trouve également l'enregistrement d'un booléen disant qu'il faut l'utiliser (ou pas).
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
Il me semble que ce que tu souhaiterais c'est à côté de l'enregistrement du schéma que se trouve également l'enregistrement d'un booléen disant qu'il faut l'utiliser (ou pas).
C'était en effet mon idée.
Après échange avec Lauréline, elle trouvait redondant d'enregistrer la valeur de la checkbox + schema.
Pour elle, l'enregistrement schema devrait suffire.
Moi je suis OK à partir du moment où niveau UI c'est cohérent : quand l'utilisateur décoche la checkbox et enregistre, le schema se retrouve supprimé à l'enregistrement et le formulaire est retourné checkbox décochée.
Mis à jour par Frédéric Péters il y a presque 3 ans
Et donc le propos ici, "la checkbox se retrouve à nouveau cochée et la grille de customisation présente", c'est "alors que comme les deux informations sont liées il faudrait que l'agencement manuel soit éliminé, pour que la case ne soit pas cochée".
J'étais de mon côté plutôt pour avoir deux informations distinctes, qu'il soit possible de désactiver l'agencement manuel mais que le réactiver ne fasse pas repartir de zéro; mais ça me va d'aller sur la suggestion, il sera de toute façon toujours plus facile d'ajouter un champ plus tard.
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Fichier 0001-wcs-add-fields-to-customize-Card-display-54259.patch 0001-wcs-add-fields-to-customize-Card-display-54259.patch ajouté
Comportement au save corrigé
Mis à jour par Frédéric Péters il y a presque 3 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 9761e251c54b717adf74ee708e882bb598946bc3 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu May 27 10:44:51 2021 +0200 wcs: add fields to customize Card display (#54259)
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
wcs: add fields to customize Card display (#54259)