diff options
| author | Raphael Collet <rco@openerp.com> | 2015-05-07 10:08:19 +0200 |
|---|---|---|
| committer | Raphael Collet <rco@openerp.com> | 2015-05-07 11:29:20 +0200 |
| commit | 926aa5bcf9217e9fe687602ce8e5281cfc631a73 (patch) | |
| tree | 72f709d10dd413f70a05ab718ff4acf1ce7b4cf2 /addons/web/static/src/css/base.sass | |
| parent | 0a83ee4c34c32d5c83ab55846c5dea953e554131 (diff) | |
[IMP] fields: when computing a field, do not recompute other stored computed fields
In the process of determining a stored computed field's value, the field could
not be read from the database if the environment was in "draft" mode. The idea
behind this condition was that during an onchange, no computed field should be
read from the database. We fix the condition by checking whether we are in
"onchange" mode instead of "draft" mode.
The change may be critical for performance: one of the purposes of storing a
computed field is to avoid recomputing it when not necessary. Now a computed
field can read the value of a stored computed field from the database.
This issue was spotted by Cédric Snauwaert (csn), who came with a concrete
example were a stored computed field was recomputed without apparent reason.
Diffstat (limited to 'addons/web/static/src/css/base.sass')
0 files changed, 0 insertions, 0 deletions
