Bug #4281
L'action "Créer un document" permet la sélection d'un fichier qui ne sera pas utilisé
0%
Description
Il m'a fallu dix minutes et un passage par pdb pour comprendre que je lisais trop vite l'écran de définition de l'action "Créer un document", que je ne décochais pas la case "Garder le fichier actuel", je me contentais de choisir un nouveau fichier.
Je pense que ce formulaire devrait être amélioré, diverses actions sont possibles, utiliser du js pour désactiver l'upload quand la case est cochée, ou peut-être simplement considérer que le champ fichier non complété signifie garder l'ancien. Je ne sais pas ce qui a justifié le design actuel.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 10 ans
Je ne vois pas de traitement particulier pour obtenir ce comportement dans la méthode add_parameters_widget() ou ailleurs dans la classe, j'ai l'impression que c'est le comportement habituel du widget UploadWidget.
Mis à jour par Frédéric Péters il y a environ 10 ans
Certes mais ce serait peut-être utile de savoir ce qui a justifié le fonctionnement actuel, avant de défaire.
Mis à jour par Benjamin Dauvergne il y a environ 10 ans
Le design est directement inspiré du FileWithPreviewWidget version avant avril-2013. Je m'inquiétais à l'époque du comportement de ce dernier en dehors d'un formulaire w.c.s. je pense:
commit 290221c526888a396c32a115cd922b702680cbaf Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Jun 21 20:08:09 2011 +0000 Add the possibility to keep the actual value to the UploadWidget It looks ever-more like FileWithPreviewWidget, but I'm a bit afraid to touch this one; seems there is a lot of legacy around it that I would not want to break.
Le besoin est simplement de pouvoir conserver le fichier quand on y touche pas et je pense qu'il pourrait être couvert sans la checkbox effectivement. On renomme le ticket ?
Mis à jour par Frédéric Péters il y a environ 10 ans
Pour moi le sujet décrit bien le bug de comportement, mais tu peux le changer si tu trouves qu'autre chose irait mieux.
Pour la résolution, tu vois l'utilisation d'un FileWidget, tout simplement ?
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-workflows-use-a-simple-file-widget-to-set-model-docu.patch 0001-workflows-use-a-simple-file-widget-to-set-model-docu.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Frédéric Péters
- Patch proposed mis à Oui
- Planning mis à Non
Mis à jour par Lauréline Guérin il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 97e8be1ee9f5195e0353744926aab6a24cfe3647 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Nov 4 18:10:30 2022 +0100 workflows: use a simple file widget to set model document (#4281)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: use a simple file widget to set model document (#4281)