diff options
| author | Raphael Collet <rco@openerp.com> | 2015-02-23 11:29:03 +0100 |
|---|---|---|
| committer | Raphael Collet <rco@openerp.com> | 2015-02-23 15:50:09 +0100 |
| commit | 5ec8596f67990a52fb3427f31d86a5f2f36bc801 (patch) | |
| tree | cc61fdb9a38789808da110f5fb35f10964b6200f /addons/web/static/lib/underscore/underscore.js | |
| parent | f23e47539e6a61e0f410787eb20a9adf2d186e57 (diff) | |
[FIX] fields: fix `_column.new()` by relying on `to_field_args()`
The implementation was based on the ill-defined method `same_parameters()` that
compares arguments based on a heuristic. Instead, we now create a new column
and check whether it is equivalent to `self` by comparing the arguments
returned by `to_field_args()`. If that is the case, `self` is reused instead
of the new column.
The code refactoring also fixes the column reuse which was broken by the
introduction of the parameter `compute` in commit 9333c62. Indeed, with that
parameter, `same_parameters()` always returned False, since old-api columns do
not have that parameter by default. The parameter has been renamed to
`_computed_field`, and is no longer passed for creating columns.
Diffstat (limited to 'addons/web/static/lib/underscore/underscore.js')
0 files changed, 0 insertions, 0 deletions
