From 749b47ee15dd0c1031d9304529b96eeb7181e847 Mon Sep 17 00:00:00 2001 From: Jonathan Nemry Date: Fri, 11 Apr 2014 14:11:45 +0200 Subject: [PATCH 1/8] [ADD] web_readonly_bypass: allow to save an onchange modifications on readonly fields --- web_readonly_bypass/__init__.py | 28 +++++ web_readonly_bypass/__openerp__.py | 61 ++++++++++ .../static/src/js/readonly_bypass.js | 109 ++++++++++++++++++ web_readonly_bypass/views/readonly_bypass.xml | 12 ++ 4 files changed, 210 insertions(+) create mode 100644 web_readonly_bypass/__init__.py create mode 100644 web_readonly_bypass/__openerp__.py create mode 100644 web_readonly_bypass/static/src/js/readonly_bypass.js create mode 100644 web_readonly_bypass/views/readonly_bypass.xml diff --git a/web_readonly_bypass/__init__.py b/web_readonly_bypass/__init__.py new file mode 100644 index 00000000..5fa4a2b1 --- /dev/null +++ b/web_readonly_bypass/__init__.py @@ -0,0 +1,28 @@ +# -*- coding: utf-8 -*- +############################################################################## +# +# Authors: Nemry Jonathan & Laetitia Gangloff +# Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu) +# All Rights Reserved +# +# WARNING: This program as such is intended to be used by professional +# programmers who take the whole responsibility of assessing all potential +# consequences resulting from its eventual inadequacies and bugs. +# End users who are looking for a ready-to-use solution with commercial +# guarantees and support are strongly advised to contact a Free Software +# Service Company. +# +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU Affero General Public License as +# published by the Free Software Foundation, either version 3 of the +# License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU Affero General Public License for more details. +# +# You should have received a copy of the GNU Affero General Public License +# along with this program. If not, see . +# +############################################################################## diff --git a/web_readonly_bypass/__openerp__.py b/web_readonly_bypass/__openerp__.py new file mode 100644 index 00000000..7f6410e1 --- /dev/null +++ b/web_readonly_bypass/__openerp__.py @@ -0,0 +1,61 @@ +# -*- coding: utf-8 -*- +############################################################################## +# +# Authors: Nemry Jonathan & Laetitia Gangloff +# Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu) +# All Rights Reserved +# +# WARNING: This program as such is intended to be used by professional +# programmers who take the whole responsibility of assessing all potential +# consequences resulting from its eventual inadequacies and bugs. +# End users who are looking for a ready-to-use solution with commercial +# guarantees and support are strongly advised to contact a Free Software +# Service Company. +# +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU Affero General Public License as +# published by the Free Software Foundation, either version 3 of the +# License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU Affero General Public License for more details. +# +# You should have received a copy of the GNU Affero General Public License +# along with this program. If not, see . +# +############################################################################## +{ + 'name': 'Read Only ByPass', + 'version': '1.0', + "author": "ACSONE SA/NV", + "maintainer": "ACSONE SA/NV", + "website": "http://www.acsone.eu", + 'category': 'Technical Settings', + 'depends': [ + 'base', + 'web', + ], + 'description': """ +Read Only ByPass +================ +This Module provides a solution to the problem of the interaction between +'readonly' attribute and 'on_change' attribute when used together + +Behavior: add `readonly_fields` changed by `on_change` into the fields passing +into an update or create. If `filter_out_readonly` is into the context and set +True then apply native behavior. + """, + 'images': [], + 'data': [ + 'views/readonly_bypass.xml', + ], + 'qweb': [ + ], + 'css': [], + 'demo': [], + 'test': [], + 'installable': True, + 'auto_install': False, +} diff --git a/web_readonly_bypass/static/src/js/readonly_bypass.js b/web_readonly_bypass/static/src/js/readonly_bypass.js new file mode 100644 index 00000000..d8581f3e --- /dev/null +++ b/web_readonly_bypass/static/src/js/readonly_bypass.js @@ -0,0 +1,109 @@ +/* + * Allow to bypass readonly fi the value is changed + */ + +openerp.readonly_bypass = function(instance) { + + var QWeb = instance.web.qweb, _t = instance.web._t; + /** + * ignore readonly: place options['readonly_fields'] into the data + * if nothing is specified into the context + * + * create mode: remove read-only keys having a 'false' value + * + * @param boolean mode: True case of create, false case of write + * @param {Object} context->filter_out_readonly + * @param {Object} data field values to possibly be updated + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: Values from readonly fields to merge into the data object + */ + function ignore_readonly(data, options, mode, context){ + if (options){ + if (!('filter_out_readonly' in context && context['filter_out_readonly'] == true + && 'readonly_fields' in options && options['readonly_fields'])) { + if(mode){ + $.each( options.readonly_fields, function( key, value ) { + if(value==false){ + delete(options.readonly_fields[key]); + } + }); + } + data = $.extend(data,options['readonly_fields']) + } + } + }; + + instance.web.BufferedDataSet.include({ + + init : function() { + this._super.apply(this, arguments); + }, + /** + * Creates Overriding + * + * @param {Object} data field values to set on the new record + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: Values from readonly fields that were updated by + * on_changes. Only used by the BufferedDataSet to make the o2m work correctly. + * @returns super {$.Deferred} + */ + create : function(data, options) { + var self = this; + ignore_readonly(data, options, true, self.context); + return self._super(data,options); + }, + /** + * Creates Overriding + * + * @param {Object} data field values to set on the new record + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: Values from readonly fields that were updated by + * on_changes. Only used by the BufferedDataSet to make the o2m work correctly. + * @returns super {$.Deferred} + */ + write : function(id, data, options) { + var self = this; + ignore_readonly(data, options, false, self.context); + return self._super(id,data,options); + }, + + }); + + instance.web.DataSet.include({ + /* + BufferedDataSet: case of 'add an item' into a form view + */ + init : function() { + this._super.apply(this, arguments); + }, + /** + * Creates Overriding + * + * @param {Object} data field values to set on the new record + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: Values from readonly fields that were updated by + * on_changes. Only used by the BufferedDataSet to make the o2m work correctly. + * @returns super {$.Deferred} + */ + create : function(data, options) { + var self = this; + ignore_readonly(data, options, true, self.context); + return self._super(data,options); + }, + /** + * Creates Overriding + * + * @param {Object} data field values to set on the new record + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: Values from readonly fields that were updated by + * on_changes. Only used by the BufferedDataSet to make the o2m work correctly. + * @returns super {$.Deferred} + */ + write : function(id, data, options) { + var self = this; + ignore_readonly(data, options, false, self.context); + return self._super(id,data,options); + }, + + }); +}; diff --git a/web_readonly_bypass/views/readonly_bypass.xml b/web_readonly_bypass/views/readonly_bypass.xml new file mode 100644 index 00000000..2bad7ace --- /dev/null +++ b/web_readonly_bypass/views/readonly_bypass.xml @@ -0,0 +1,12 @@ + + + + + + + \ No newline at end of file From 3260cd269dbabb9e4efbd0fa38b31715de483923 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?St=C3=A9phane=20Bidoul?= Date: Thu, 2 Jul 2015 10:27:05 +0200 Subject: [PATCH 2/8] [IMP] web_readonly_bypass: port to OCA from acsone-addons --- web_readonly_bypass/README.rst | 74 ++++++++++++++++++++++++++++++ web_readonly_bypass/__init__.py | 27 ----------- web_readonly_bypass/__openerp__.py | 28 +---------- 3 files changed, 76 insertions(+), 53 deletions(-) create mode 100644 web_readonly_bypass/README.rst diff --git a/web_readonly_bypass/README.rst b/web_readonly_bypass/README.rst new file mode 100644 index 00000000..2ffbc11d --- /dev/null +++ b/web_readonly_bypass/README.rst @@ -0,0 +1,74 @@ +.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg + :alt: License: AGPL-3 + +Read Only ByPass +================ + +This module provides a solution to the problem of the interaction between +'readonly' attribute and 'on_change' attribute when used together. It allows +saving onchange modifications to readonly fields. + +Behavior: add readonly fields changed by `on_change` methods to the values +passed to write or create. If `filter_out_readonly` is in the context and +True then apply native behavior. + +Installation +============ + +There are no specific installation instructions for this module. + +Configuration +============= + +There is nothing to configure. + +Usage +===== + +This module changes the default behaviour of Odoo by propagating +on_change modifications to readonly fields to the backend create and write +methods. + +To restore the standard behaviour, set `filter_out_readonly` in the context. + +For further information, please visit: + +* https://www.odoo.com/forum/help-1 + +Known issues / Roadmap +====================== + +None + +Bug Tracker +=========== + +Bugs are tracked on `GitHub Issues `_. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed feedback +`here `_. + + +Credits +======= + +Contributors +------------ + +* Jonathan Nemry +* Laetitia Gangloff + +Maintainer +---------- + +.. image:: https://odoo-community.org/logo.png + :alt: Odoo Community Association + :target: https://odoo-community.org + +This module is maintained by the OCA. + +OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use. + +To contribute to this module, please visit http://odoo-community.org. diff --git a/web_readonly_bypass/__init__.py b/web_readonly_bypass/__init__.py index 5fa4a2b1..40a96afc 100644 --- a/web_readonly_bypass/__init__.py +++ b/web_readonly_bypass/__init__.py @@ -1,28 +1 @@ # -*- coding: utf-8 -*- -############################################################################## -# -# Authors: Nemry Jonathan & Laetitia Gangloff -# Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu) -# All Rights Reserved -# -# WARNING: This program as such is intended to be used by professional -# programmers who take the whole responsibility of assessing all potential -# consequences resulting from its eventual inadequacies and bugs. -# End users who are looking for a ready-to-use solution with commercial -# guarantees and support are strongly advised to contact a Free Software -# Service Company. -# -# This program is free software: you can redistribute it and/or modify -# it under the terms of the GNU Affero General Public License as -# published by the Free Software Foundation, either version 3 of the -# License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU Affero General Public License for more details. -# -# You should have received a copy of the GNU Affero General Public License -# along with this program. If not, see . -# -############################################################################## diff --git a/web_readonly_bypass/__openerp__.py b/web_readonly_bypass/__openerp__.py index 7f6410e1..d54969d7 100644 --- a/web_readonly_bypass/__openerp__.py +++ b/web_readonly_bypass/__openerp__.py @@ -3,14 +3,6 @@ # # Authors: Nemry Jonathan & Laetitia Gangloff # Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu) -# All Rights Reserved -# -# WARNING: This program as such is intended to be used by professional -# programmers who take the whole responsibility of assessing all potential -# consequences resulting from its eventual inadequacies and bugs. -# End users who are looking for a ready-to-use solution with commercial -# guarantees and support are strongly advised to contact a Free Software -# Service Company. # # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU Affero General Public License as @@ -30,32 +22,16 @@ 'name': 'Read Only ByPass', 'version': '1.0', "author": "ACSONE SA/NV", - "maintainer": "ACSONE SA/NV", + "maintainer": "ACSONE SA/NV,Odoo Community Association (OCA)", "website": "http://www.acsone.eu", 'category': 'Technical Settings', 'depends': [ - 'base', 'web', ], - 'description': """ -Read Only ByPass -================ -This Module provides a solution to the problem of the interaction between -'readonly' attribute and 'on_change' attribute when used together - -Behavior: add `readonly_fields` changed by `on_change` into the fields passing -into an update or create. If `filter_out_readonly` is into the context and set -True then apply native behavior. - """, - 'images': [], + 'summary': 'Allow to save onchange modifications to readonly fields', 'data': [ 'views/readonly_bypass.xml', ], - 'qweb': [ - ], - 'css': [], - 'demo': [], - 'test': [], 'installable': True, 'auto_install': False, } From 3b984312b6b6178058e00765e6b2e1c7f1ceb3e7 Mon Sep 17 00:00:00 2001 From: "Jonathan Nemry (ACSONE)" Date: Fri, 3 Jul 2015 08:04:45 +0200 Subject: [PATCH 3/8] [CHG] some code review: * Use a better test condition * Fix the email address [FIX] * rename the path and web instance just as the module was --- web_readonly_bypass/README.rst | 2 +- web_readonly_bypass/static/src/js/readonly_bypass.js | 6 +++--- web_readonly_bypass/views/readonly_bypass.xml | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/web_readonly_bypass/README.rst b/web_readonly_bypass/README.rst index 2ffbc11d..12d8ac17 100644 --- a/web_readonly_bypass/README.rst +++ b/web_readonly_bypass/README.rst @@ -56,7 +56,7 @@ Contributors ------------ * Jonathan Nemry -* Laetitia Gangloff +* Laetitia Gangloff Maintainer ---------- diff --git a/web_readonly_bypass/static/src/js/readonly_bypass.js b/web_readonly_bypass/static/src/js/readonly_bypass.js index d8581f3e..0635c74d 100644 --- a/web_readonly_bypass/static/src/js/readonly_bypass.js +++ b/web_readonly_bypass/static/src/js/readonly_bypass.js @@ -2,7 +2,7 @@ * Allow to bypass readonly fi the value is changed */ -openerp.readonly_bypass = function(instance) { +openerp.web_readonly_bypass = function(instance) { var QWeb = instance.web.qweb, _t = instance.web._t; /** @@ -19,8 +19,8 @@ openerp.readonly_bypass = function(instance) { */ function ignore_readonly(data, options, mode, context){ if (options){ - if (!('filter_out_readonly' in context && context['filter_out_readonly'] == true - && 'readonly_fields' in options && options['readonly_fields'])) { + if ('readonly_fields' in options && options['readonly_fields'] && + !('filter_out_readonly' in context && context['filter_out_readonly'] == true )) { if(mode){ $.each( options.readonly_fields, function( key, value ) { if(value==false){ diff --git a/web_readonly_bypass/views/readonly_bypass.xml b/web_readonly_bypass/views/readonly_bypass.xml index 2bad7ace..8aff08f5 100644 --- a/web_readonly_bypass/views/readonly_bypass.xml +++ b/web_readonly_bypass/views/readonly_bypass.xml @@ -5,8 +5,8 @@ - \ No newline at end of file + From c4bc379de7903022c9b7793c0a7ac49fd6d52e04 Mon Sep 17 00:00:00 2001 From: Pierre Verkest Date: Tue, 14 Jul 2015 13:56:09 +0200 Subject: [PATCH 4/8] cherry-pick 790a853 by pass readonly fields only if context is set with readonly_by_pass. This context can either be a bool or field name array --- .../static/src/js/readonly_bypass.js | 86 ++++++--- .../static/test/web_readonly_bypass.js | 168 ++++++++++++++++++ web_readonly_bypass/views/readonly_bypass.xml | 9 +- 3 files changed, 235 insertions(+), 28 deletions(-) create mode 100644 web_readonly_bypass/static/test/web_readonly_bypass.js diff --git a/web_readonly_bypass/static/src/js/readonly_bypass.js b/web_readonly_bypass/static/src/js/readonly_bypass.js index 0635c74d..aa40a0d3 100644 --- a/web_readonly_bypass/static/src/js/readonly_bypass.js +++ b/web_readonly_bypass/static/src/js/readonly_bypass.js @@ -5,34 +5,70 @@ openerp.web_readonly_bypass = function(instance) { var QWeb = instance.web.qweb, _t = instance.web._t; - /** - * ignore readonly: place options['readonly_fields'] into the data - * if nothing is specified into the context - * - * create mode: remove read-only keys having a 'false' value - * - * @param boolean mode: True case of create, false case of write - * @param {Object} context->filter_out_readonly - * @param {Object} data field values to possibly be updated - * @param {Object} options Dictionary that can contain the following keys: - * - readonly_fields: Values from readonly fields to merge into the data object - */ - function ignore_readonly(data, options, mode, context){ - if (options){ - if ('readonly_fields' in options && options['readonly_fields'] && - !('filter_out_readonly' in context && context['filter_out_readonly'] == true )) { - if(mode){ + var instance = instance; + + instance.web_readonly_bypass = { + /** + * ignore readonly: place options['readonly_fields'] into the data + * if nothing is specified into the context + * + * create mode: remove read-only keys having a 'false' value + * + * @param {Object} data field values to possibly be updated + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: Values from readonly fields to merge into the data object + * @param boolean mode: True case of create, false case of write + * @param {Object} context->readonly_by_pass + */ + ignore_readonly: function(data, options, mode, context){ + console.log(options ); + console.log(context ); + var readonly_by_pass_fields = this.retrieve_readonly_by_pass_fields( + options, context); + console.log(readonly_by_pass_fields ); + if(mode){ + $.each( readonly_by_pass_fields, function( key, value ) { + if(value==false){ + delete(readonly_by_pass_fields[key]); + } + }); + } + data = $.extend(data,readonly_by_pass_fields); + console.log(data ); + }, + + /** + * retrieve_readonly_by_pass_fields: retrieve readonly fields to save + * according context. + * + * @param {Object} options Dictionary that can contain the following keys: + * - readonly_fields: all values from readonly fields + * @param {Object} context->readonly_by_pass: Can be true if all + * all readonly fields should be saved or an array of field name to + * save ie: ['readonly_field_1', 'readonly_field_2'] + * @returns {Object}: readonly key/value fields to save according context + */ + retrieve_readonly_by_pass_fields: function(options, context){ + var readonly_by_pass_fields = {}; + if (options && 'readonly_fields' in options && + options['readonly_fields'] && context && + 'readonly_by_pass' in context && context['readonly_by_pass']){ + if (_.isArray(context['readonly_by_pass'])){ $.each( options.readonly_fields, function( key, value ) { - if(value==false){ - delete(options.readonly_fields[key]); + if(_.contains(context['readonly_by_pass'], key)){ + readonly_by_pass_fields[key] = value; } }); + }else{ + readonly_by_pass_fields = options.readonly_fields; } - data = $.extend(data,options['readonly_fields']) } - } + return readonly_by_pass_fields; + }, }; + readonly_bypass = instance.web_readonly_bypass; + instance.web.BufferedDataSet.include({ init : function() { @@ -49,7 +85,7 @@ openerp.web_readonly_bypass = function(instance) { */ create : function(data, options) { var self = this; - ignore_readonly(data, options, true, self.context); + readonly_bypass.ignore_readonly(data, options, true, self.context); return self._super(data,options); }, /** @@ -63,7 +99,7 @@ openerp.web_readonly_bypass = function(instance) { */ write : function(id, data, options) { var self = this; - ignore_readonly(data, options, false, self.context); + readonly_bypass.ignore_readonly(data, options, false, self.context); return self._super(id,data,options); }, @@ -87,7 +123,7 @@ openerp.web_readonly_bypass = function(instance) { */ create : function(data, options) { var self = this; - ignore_readonly(data, options, true, self.context); + readonly_bypass.ignore_readonly(data, options, true, self.context); return self._super(data,options); }, /** @@ -101,7 +137,7 @@ openerp.web_readonly_bypass = function(instance) { */ write : function(id, data, options) { var self = this; - ignore_readonly(data, options, false, self.context); + readonly_bypass.ignore_readonly(data, options, false, self.context); return self._super(id,data,options); }, diff --git a/web_readonly_bypass/static/test/web_readonly_bypass.js b/web_readonly_bypass/static/test/web_readonly_bypass.js new file mode 100644 index 00000000..4aa182a9 --- /dev/null +++ b/web_readonly_bypass/static/test/web_readonly_bypass.js @@ -0,0 +1,168 @@ +openerp.testing.section( 'web_readonly_bypass', {}, +function(test){ + test('ignore_readonly', function(instance){ + openerp.web_readonly_bypass(instance); + var data = {}; + var mode_create = true; + var options = {}; + var context = {}; + instance.web_readonly_bypass.ignore_readonly(data, options, + mode_create, context); + deepEqual(data, + {}, + "Empty context and options mode create" + ); + + mode_create = false; + data = {}; + instance.web_readonly_bypass.ignore_readonly(data, options, + mode_create, context); + deepEqual(data, + {}, + "Empty context and options mode write" + ); + + mode_create = false; + data = {}; + context = {'readonly_by_pass': true}; + options = {'readonly_fields': {'field_1': 'va1-1', + 'field_2': false, + 'field_3': 'val-3'}}; + instance.web_readonly_bypass.ignore_readonly(data, options, + mode_create, context); + deepEqual(data, + {'field_1': 'va1-1', 'field_2': false, 'field_3': 'val-3'}, + "all fields mode write" + ); + + mode_create = true; + data = {}; + context = {'readonly_by_pass': true}; + options = {'readonly_fields': {'field_1': 'va1-1', + 'field_2': false, + 'field_3': 'val-3'}}; + instance.web_readonly_bypass.ignore_readonly(data, options, + mode_create, context); + deepEqual(data, + {'field_1': 'va1-1', 'field_3': 'val-3'}, + "all fields mode create (false value are escaped)" + ); + + mode_create = true; + data = {}; + context = {}; + options = {'readonly_fields': {'field_1': 'va1-1', + 'field_2': false, + 'field_3': 'val-3'}}; + instance.web_readonly_bypass.ignore_readonly(data, options, + mode_create, context); + deepEqual(data, + {}, + "without context, default, we won't save readonly fields" + ); + }); + + test('retrieve_readonly_by_pass_fields', function(instance){ + openerp.web_readonly_bypass(instance); + var context = {'readonly_by_pass': true} + var options = {'readonly_fields': {'field_1': 'va1-1', + 'field_2': 'val-2', + 'field_3': 'val-3'}}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {'field_1': 'va1-1', 'field_2': 'val-2', 'field_3': 'val-3'}, + "All fields should be accepted!" + ); + + context = {'readonly_by_pass': ['field_1', 'field_3']}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {'field_1': 'va1-1','field_3': 'val-3'}, + "two field s1" + ); + + context = {'readonly_by_pass': ['field_1',]}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {'field_1': 'va1-1'}, + "Only field 1" + ); + + context = {'readonly_by_pass': []}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "Empty context field" + ); + + context = null; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "Null context" + ); + + context = false; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "false context" + ); + + context = {'readonly_by_pass': true} + options = {'readonly_fields': {'field_1': 'va1-1'}}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {'field_1': 'va1-1'}, + "Only one option" + ); + + + options = {'readonly_fields': {}}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "Empty readonly_fields option" + ); + + options = {}; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "Empty option" + ); + + options = null; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "null option" + ); + + options = false; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "false option" + ); + + context = false; + deepEqual( + instance.web_readonly_bypass.retrieve_readonly_by_pass_fields( + options, context), + {}, + "false option and false context" + ); + }); +}); \ No newline at end of file diff --git a/web_readonly_bypass/views/readonly_bypass.xml b/web_readonly_bypass/views/readonly_bypass.xml index 8aff08f5..32c584c0 100644 --- a/web_readonly_bypass/views/readonly_bypass.xml +++ b/web_readonly_bypass/views/readonly_bypass.xml @@ -1,12 +1,15 @@ - -