Tryton - Issues

 

Issue8565

Title Better error message when context is missing
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components trytond
Assigned To ced Keywords review
Reviews 257921003
View: 257921003

Created on 2019-08-09.18:16:51 by ced, last changed by roundup-bot.

Messages
New changeset beb9de414b69 by Cédric Krier in branch 'default':
Raise proper exceptions when context is missing in RPC
https://hg.tryton.org/tryton-env/rev/beb9de414b69
New changeset 5a71d5cc9f30 by Cédric Krier in branch 'default':
Raise proper exceptions when context is missing in RPC
https://hg.tryton.org/trytond/rev/5a71d5cc9f30
review257921003 updated at https://codereview.tryton.org/257921003/#ps287921002
msg51224 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-08-09.18:16:50
A common mistake when using proteus is to call .search instead of .find. In such case most of the time, the error is dues to the missing context. But the error message is not very clear as it says usually: "AttributeError: 'list' object has no attribute 'keys'" (because the domain is used as context).
So I propose to have more meaningful error message that will guide user to the right direction.
History
Date User Action Args
2019-08-22 22:48:37roundup-botsetmessages: + msg51432
2019-08-22 22:48:33roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg51431
2019-08-09 18:45:26reviewbotsetnosy: + reviewbot
messages: + msg51226
2019-08-09 18:19:59cedsetstatus: in-progress -> testing
reviews: 257921003
keyword: + review
2019-08-09 18:16:51cedcreate

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