Development #60636
crash quand on affecte une valeur invalide à un champ carte
Début:
13 janvier 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
1/ on lève une exception quelconque
def set_value(self, data, value, raise_on_error=False):
if value and ';' not in value:
raise Exception('invalid coordinates %r (missing ;)' % value)
super().set_value(data, value)
puis 2/ dans wf/backoffice_fields.py, on n'imagine pas du tout qu'une exception puisse arriver,
formdef_field.set_value(formdata.data, new_value)
On pourrait 1/ lever une SetValueError, 2/ l'attraper et faire un record_error.
Ou ajouter un convert_value_from_anything sur le MapField, qui retourne lève ValueError, pour attraper l'erreur un peu plus haut.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-workflows-don-t-crash-on-backoffice-map-field-gettin.patch 0001-workflows-don-t-crash-on-backoffice-map-field-gettin.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Lié à Bug #55802: action backoffice-fields : affecter n'importe quoi dans un champ carte ne devrait pas produire de 500 ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 2 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 2bbc024cb0afc9ee39e569ed02bfad4b38a0d884
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Jan 13 19:18:06 2022 +0100
workflows: don't crash on backoffice map field getting an invalid value (#60636)
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
workflows: don't crash on backoffice map field getting an invalid value (#60636)