Tryton - Issues

 

Issue7434

Title Tree domain clause added multiple times
Priority bug Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type crash Components trytond
Assigned To ced Keywords review
Reviews 47421002
View: 47421002

Created on 2018-05-16.15:36:22 by ced, last changed by roundup-bot.

Messages
New changeset 6836081bd4c4 by C├ędric Krier in branch 'default':
Add tree name clause only if not yet present
https://hg.tryton.org/trytond/rev/6836081bd4c4
review47421002 updated at https://codereview.tryton.org/47421002/#ps1
msg40698 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-05-16.15:36:21
The clause on the name tree is added multiple times if the mixin is defined on the initial class. This is because the field modified is the reference field that is copied. So the domain is added on each __setup__ call which may lead to a very big domain (like when running the tests for all modules).
History
Date User Action Args
2018-05-17 18:26:57roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg40746
2018-05-16 15:47:09reviewbotsetnosy: + reviewbot
messages: + msg40700
2018-05-16 15:37:58cedsetstatus: in-progress -> testing
reviews: 47421002
keyword: + review
2018-05-16 15:36:22cedcreate

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