Tryton - Issues



Title One2Many display fails if field is null
Priority bug Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type crash Components sao, tryton
Assigned To ced Keywords review
Reviews 255091005,271071003
View: 255091005, 271071003

Created on 2019-03-08.18:31:57 by ced, last changed by roundup-bot.

New changeset 711e134917e1 by Cédric Krier in branch 'default':
Test not field
New changeset 30641039eee2 by Cédric Krier in branch 'default':
Test not field instead of None
New changeset 7b63227eeeb8 by Cédric Krier in branch 'default':
Test not field instead of undefined
review271071003 updated at
review255091005 updated at
msg47482 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-03-08.18:35:44
Here is review255091005 and I added review271071003 for tryton to keep in sync.
msg47481 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-03-08.18:31:56

Form.display may call Widget.display with field = null but some widgets test only for undefined. This is no more true since r 5faa0ac6f80c.
But I still think we should use a better test to backport on older version and to be more consistent.
Date User Action Args
2019-03-15 11:20:34roundup-botsetmessages: + msg47601
2019-03-15 11:20:32roundup-botsetmessages: + msg47600
2019-03-15 11:20:31roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg47599
2019-03-08 18:38:11reviewbotsetmessages: + msg47484
2019-03-08 18:38:00reviewbotsetnosy: + reviewbot
messages: + msg47483
2019-03-08 18:35:45cedsetstatus: in-progress -> testing
reviews: 255091005,271071003
component: + tryton
messages: + msg47482
keyword: + review
2019-03-08 18:31:57cedcreate

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