Issue 9187

Title
Ensure all contextual field are in the cache to use the cache
Priority
bug
Status
resolved
Nosy list
ced, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2020-03-27.15:44:56 by ced, last changed 7 months ago by roundup-bot.

Messages

New changeset ce0d3b682169 by Cédric Krier in branch '5.4':
Do not use only cache if context depends on other fields
https://hg.tryton.org/trytond/rev/ce0d3b682169

New changeset 696f8c2879dd by Cédric Krier in branch '5.2':
Do not use only cache if context depends on other fields
https://hg.tryton.org/trytond/rev/696f8c2879dd

New changeset dbc514429db6 by Cédric Krier in branch '5.0':
Do not use only cache if context depends on other fields
https://hg.tryton.org/trytond/rev/dbc514429db6
New changeset 5ee3a295aea6 by Cédric Krier in branch 'default':
Do not use only cache if context depends on other fields
https://hg.tryton.org/tryton-env/rev/5ee3a295aea6
New changeset a4b513b75f9c by Cédric Krier in branch 'default':
Do not use only cache if context depends on other fields
https://hg.tryton.org/trytond/rev/a4b513b75f9c
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-03-27.15:44:55
Discovered with issue9111, ModelStorage.__getattr__ use the value of the field in the cache if present but indeed if the field requires a context to be instantiated we will miss the values as only the field is in the data. In such case, we need to read all the needed fields.
History
Date User Action Args
2020-04-17 14:55:24roundup-botsetmessages: + msg57229
keyword: - backport
2020-04-10 18:25:30roundup-botsetmessages: + msg56973
2020-04-10 18:25:23roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg56972
2020-03-27 16:03:07reviewbotsetnosy: + reviewbot
messages: + msg56700
2020-03-27 15:53:32cedsetkeyword: + backport
2020-03-27 15:48:02cedlinkissue9111 superseder
2020-03-27 15:47:41cedsetstatus: in-progress -> testing
reviews: 297391002
keyword: + review
2020-03-27 15:44:56cedcreate

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