Project

General

Profile

Development #60636

crash quand on affecte une valeur invalide à un champ carte

Added by Frédéric Péters 13 days ago. Updated 11 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
13 Jan 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files


Related issues

Related to w.c.s. - Bug #55802: action backoffice-fields : affecter n'importe quoi dans un champ carte ne devrait pas produire de 500Fermé23 Jul 2021

Actions

Associated revisions

Revision 2bbc024c (diff)
Added by Frédéric Péters 13 days ago

workflows: don't crash on backoffice map field getting an invalid value (#60636)

History

#2

Updated by Frédéric Péters 13 days ago

  • Assignee set to Frédéric Péters
#3

Updated by Frédéric Péters 13 days ago

#4

Updated by Benjamin Dauvergne 13 days ago

  • Related to Bug #55802: action backoffice-fields : affecter n'importe quoi dans un champ carte ne devrait pas produire de 500 added
#5

Updated by Benjamin Dauvergne 13 days ago

  • Status changed from Solution proposée to Solution validée
#6

Updated by Frédéric Péters 13 days ago

  • Status changed from Solution validée to 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)
#7

Updated by Frédéric Péters 11 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF