Development #60696
amélioration de l'erreur "champs manquants" de l'action de création d'une demande/fiche
0%
Description
Champ manquant : ['3']
Je ne comprends de quoi il s'agit, de quel champ il parle.
On gagnerait à afficher une liste de libellés (éventuellement complétés du varname, éventuellement bis de l'identifiant interne).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-workflows-report-labels-of-deleted-fields-in-create-.patch 0001-workflows-report-labels-of-deleted-fields-in-create-.patch ajouté
- Fichier Screenshot 2022-01-16 at 11-59-20 Backoffice de Auquo - Workflow - nouvelle demande.png Screenshot 2022-01-16 at 11-59-20 Backoffice de Auquo - Workflow - nouvelle demande.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà pour conserver un cache des libellés des champs.
Aussi, à l'édition quand un champ n'existait plus ça affichait "---" vu l'absence d'option associée à l'id en question; je modifie ça pour afficher "! ancien libellé (champ supprimé)". (et je pense qu'afficher une option avec grosso modo "erreur" en libellé pourrait être quelque chose à généraliser, pour éviter les moments où on regarde le paramétrage d'une action et où on ne voit pas que derrière une option vide "---" il y a en fait une valeur qui est présente).
(on peut aussi discuter de l'emploi d'un point d'exclamation emoji pour distinguer)
Mis à jour par Thomas Noël il y a presque 2 ans
(relecture bien tardive... sorry)
Dans cette partie de code :
super().__init__(
name,
element_type=MappingWidget,
element_kwargs={
'to_formdef': to_formdef,
'accept_empty_value': accept_empty_value,
+ 'cached_field_labels': kwargs.pop('cached_field_labels', None),
},
**kwargs,
)
je ne suis pas certain que le kwargs envoyé au final soit bien diminué par le pop au dessus. Dans ce doute on pourrait avoir un cached_field_labels = kwargs.pop('cached_field_labels', None) explicitement avant cet appel à super().__init__(...)
Sinon en tant que grand spécialiste de l'UX je n'ai absolument aucun avis sur l'usage de «❗» (je le trouve discret, c'est bien ou pas, je ne sais pas).
Mis à jour par Frédéric Péters il y a presque 2 ans
- Fichier 0001-workflows-report-labels-of-deleted-fields-in-create-.patch 0001-workflows-report-labels-of-deleted-fields-in-create-.patch ajouté
Voilà avec le pop déplacé.
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3232e599b29bbbe9bf9c5346afe29960fd17f351
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat Jan 15 14:04:19 2022 +0100
workflows: report labels of deleted fields in create form/card action (#60696)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: report labels of deleted fields in create form/card action (#60696)