Produits Entr'ouvert » w.c.s.

Ca manque de tests :)

Mais surtout, on aura un soucis après traduction, l'erreur affichée sera du genre « Le fichier reçu n'est pas un formulaire, mais semble être de type "block" » au lieu de « ... de type "bloc de champs" ». Il faudrait quelque chose comme XML_ROOT_NODE_NAME.get(tree.tag, _('%s (unknown)') % tree.tag) à la place de tree.tag.

Ca serait mieux aussi de ne pas changer la ligne « if tree.tag != 'workflow': » : les patches les moins longs sont les meilleurs.

J'avais pensé à utiliser l'attribut de classe 'verbose_name' sinon pour avoir un nom localisé, mais il est défini sur FormDef uniquement, je peux l'ajouter sur BlockDef & Workflow ?

Corentin Séchet a écrit :

J'avais pensé à utiliser l'attribut de classe 'verbose_name' sinon pour avoir un nom localisé, mais il est défini sur FormDef uniquement, je peux l'ajouter sur BlockDef & Workflow ?

Bonne idée !

Discussions en parallèle, tout ça est un peu too much par rapport au but recherché, on pourrait rester sur un message simple :

"Votre fichier XML n'est pas du type attendu, il commence par une balise <%(seen)s> au lieu de <%(expected)s>" % {'seen': tree.tag, 'expected': cls.xml_root_node}

ça donnera <form> ou <workflow>, c'est compréhensible à ce niveau d'utilisation de Publik.

wcs: Improved error reporting when importing a file of the wrong type (#62109)

(sans lire le code)

Sur la forme des messages on ne met pas l'application concernée en préfixe, ici c'est une modif qui relève plutôt du backoffice, on prendrait plutôt ça comme préfixe. (il a pu être simplifié à un moment le choix du préfixe à quelque chose en rapport avec le répertoire où se trouve le fichier, ça n'est pas correct).

Aussi, on essaie de tenir sous 80 caractères, et on n'utilise pas de participe passé.

Ici par exemple on pourrait avoir backoffice: report expected XML node in invalid XML import messages (#.....)

Sur le code, XML doit être en majuscules dans les messages, "doesn't seems" n'est pas correct (et le plus simple "is not" serait correct). Surtout "to be a form definition" va s'afficher aussi pour les modèles de fiches.

J'ai modifié en affichant le tag attendu, pour que ça fonctionne pour les modèles de fiche.

J'arrive pour jouer les pénibles sur deux détails :

Pour factoriser la traduction, tu pourrais utiliser la même "Provided XML file is invalid, it starts with a <%(seen)s> tag instead of <%(expected)s>" au niveau de l'import des workflows.

Ensuite, dans les tests, fait un assert sur le message complet « assert excinfo.value.msg == "Provided ..." ». Aussi, ajouter un test de CardDef.import_from_xml(io.BytesIO(export)) pour ainsi faire de tous les 4 cas possibles (blocs, formulaire, workflow et donc, modèle de fiche).

Avec ça, je pense pouvoir promettre que ça va être un ack :-)

Automatic expiration