Issue 9989

Title
Failed to format domain of exception with string as selection
Priority
bug
Status
resolved
Nosy list
ced, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2021-01-12.18:31:00 by ced, last changed 10 months ago by roundup-bot.

Messages

New changeset 9562e5d3d406 by Cédric Krier in branch '5.8':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/tryton/rev/9562e5d3d406

New changeset d107b17c7ae5 by Cédric Krier in branch '5.6':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/tryton/rev/d107b17c7ae5

New changeset 7826b7b96561 by Cédric Krier in branch '5.0':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/tryton/rev/7826b7b96561
New changeset 7482d84c69c0 by Cédric Krier in branch '5.8':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/sao/rev/7482d84c69c0

New changeset bae6e84680a2 by Cédric Krier in branch '5.6':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/sao/rev/bae6e84680a2

New changeset 80663838f18b by Cédric Krier in branch '5.0':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/sao/rev/80663838f18b
New changeset 4278ea310f93 by Cédric Krier in branch 'default':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/tryton-env/rev/4278ea310f93
New changeset 9bf28fc36d79 by Cédric Krier in branch 'default':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/tryton/rev/9bf28fc36d79
New changeset ee651d2da251 by Cédric Krier in branch 'default':
Fall-back to internal selection value if selection is a string
https://hg.tryton.org/sao/rev/ee651d2da251
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-01-12.18:30:59

When trytond raises a DomainValidationError with an invalid domain and the field definition, this definition may contain as selection only a string. The DomainParser crashes when trying to format the value to display in the error message because it is expecting a list of selection.
The client at this place (processing an RPC error) can not really make a second call to convert the selection. So I think we should fall-back to the internal value.

History
Date User Action Args
2021-01-28 23:00:35roundup-botsetmessages: + msg64155
2021-01-28 23:00:26roundup-botsetkeyword: - backport
messages: + msg64153
2021-01-22 01:11:35roundup-botsetmessages: + msg64023
2021-01-22 01:11:34roundup-botsetmessages: + msg64022
2021-01-22 01:11:33roundup-botsetmessages: + msg64021
nosy: + roundup-bot
status: testing -> resolved
2021-01-12 18:49:07reviewbotsetmessages: + msg63855
nosy: + reviewbot
2021-01-12 18:33:30cedsetkeyword: + backport, review
reviews: 349271002
status: in-progress -> testing
2021-01-12 18:31:00cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)