Development #19005
partager les get_date_value et get_file_value de wf/backoffice_fields.py
0%
Description
ces deux fonctions qui prennent "des trucs" et tentent de les transformer soit en date, soit en fichier (PicklableUpload), ça sera utile dans #8274 et ailleurs sans doute.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-workflows-factorize-get_date_value-and-get_file_valu.patch 0001-workflows-factorize-get_date_value-and-get_file_valu.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Pourquoi les attacher à WorkflowStatusItem et ne pas juste les définir comme de simples fonctions ? Entrevois-tu un besoin de modifier leur comportement un jour par surcharge ?
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-workflows-factorize-get_date_value-and-get_file_valu.patch 0001-workflows-factorize-get_date_value-and-get_file_valu.patch ajouté
À vrai dire ... j'en sais rien ;-) Je me suis dit que c'était bien de coller ça à côté de "compute"... en attendant un jour de mettre tout ça dans un autre coin propre (tu avais commencé pour franceconnect et finalement non, on avait choisi de ne pas compliquer l'affaire).
D'ailleurs je me rends compte que le changement de self.get_file/date_value en WorkflowStatusItem.get_file/date_value était inutile dans SetBackofficeFieldsWorkflowStatusItem.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Pour compute c'était différent, il était déjà là, là en plus c'est pas super nommé. Mais soit, ack.
Mis à jour par Thomas Noël il y a plus de 6 ans
Pour le nommage comme pour l'emplacement je suis d'accord, hein... j'ai juste pas de meilleur feeling là maintenant, je suis ouvert à toute idée.
Par exemple mettre ça dans wcs/fields.py, puisque c'est un peu lié à des valeurs qu'on veut pouvoir mettre telles quelle dans des fields). Genre DateField.build_value(value) et FileField.build_value(value) ?
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Oui mettre ça sur les classes Field je commencerai à trouver ça un peu plus logique.
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-add-generic-value-convertor-for-fields-value-19005.patch 0001-add-generic-value-convertor-for-fields-value-19005.patch ajouté
Voici Field::convert_value_from_anything, sur le modèle des convert_value_from_str qui existent déjà à côté.
La couverture par les tests est déjà assurée par les test_set_backoffice_field_* de tests/test_workflows.py, youpi.
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-add-generic-value-convertor-for-fields-value-19005.patch 0001-add-generic-value-convertor-for-fields-value-19005.patch ajouté
Et en fait, il faut déjà prévoir que ça puisse être utilisé comme méthode de classe (lors du calcul des attachements, qui ne sont pas des champs, sur #8274). J'ajoute donc des @classmethod.
Mis à jour par Thomas Noël il y a plus de 6 ans
- Précède Development #8274: Permettre l'envoi de fichiers joints lors de l'action "envoyer un mail" ajouté
Mis à jour par Thomas Noël il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit fd0ba0d9667c82f6837c0450a67974e41c450c8e
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Sat Sep 30 12:24:38 2017 +0200
add generic value convertor for fields value (#19005)
factorized from set backoffice fields action.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add generic value convertor for fields value (#19005)
factorized from set backoffice fields action.