summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/fields/relational_fields.js
diff options
context:
space:
mode:
authorAaron Bohy <aab@odoo.com>2019-01-24 08:08:30 +0000
committerMathieu Duckerts-Antoine <dam@odoo.com>2019-02-13 12:23:28 +0000
commit0eaee456591b31e35b56f365cdfbf1e618b17a6c (patch)
treebe3cda93e0d3aa7574ce9527593f0ce35cbbb8c9 /addons/web/static/src/js/fields/relational_fields.js
parentba5a2b47ac098bee75f1346cbd3dfab0202f674e (diff)
[IMP] web: force activeActions to be boolean
The docstring says they are boolean, but they actually aren't (e.g. edit="0" in the arch would lead to activeActions.edit=0). This could cause issue if that value is used to toggle a class with jQuery, as in this case the flag argument *must* be a boolean.
Diffstat (limited to 'addons/web/static/src/js/fields/relational_fields.js')
-rw-r--r--addons/web/static/src/js/fields/relational_fields.js4
1 files changed, 2 insertions, 2 deletions
diff --git a/addons/web/static/src/js/fields/relational_fields.js b/addons/web/static/src/js/fields/relational_fields.js
index 24a1eced358..d30d263fa12 100644
--- a/addons/web/static/src/js/fields/relational_fields.js
+++ b/addons/web/static/src/js/fields/relational_fields.js
@@ -817,10 +817,10 @@ var FieldX2Many = AbstractField.extend({
var arch = this.view && this.view.arch;
if (arch) {
this.activeActions.create = arch.attrs.create ?
- JSON.parse(arch.attrs.create) :
+ !!JSON.parse(arch.attrs.create) :
true;
this.activeActions.delete = arch.attrs.delete ?
- JSON.parse(arch.attrs.delete) :
+ !!JSON.parse(arch.attrs.delete) :
true;
this.editable = arch.attrs.editable;
}