[MIG][10.0] web_readonly_bypass rewrite JS and tests

web_readonly_bypass/README.rst

@@ -63,6 +63,9 @@ Contributors
 * Jonathan Nemry <jonathan.nemry@acsone.eu>
 * Laetitia Gangloff <laetitia.gangloff@acsone.eu>
 * Pierre Verkest <pverkest@anybox.fr>
+* Kalpana Hemnani <???>
+* Simone Orsi <simone.orsi@camptocamp.com>
+
 
 Maintainer
 ----------

web_readonly_bypass/__manifest__.py

@@ -5,7 +5,7 @@
 {
     'name': 'Read Only ByPass',
     'version': '10.0.1.0.0',
-    "author": "ACSONE SA/NV,Kalpana Hemnani,Odoo Community Association (OCA)",
+    "author": "ACSONE SA/NV,Odoo Community Association (OCA)",
     "website": "http://www.acsone.eu",
     "license": "LGPL-3",
     'category': 'Technical Settings',

web_readonly_bypass/static/src/js/readonly_bypass.js

@@ -1,9 +1,10 @@
-"use strict";
-(function(){
-    var instance = openerp;
-    var QWeb = instance.web.qweb, _t = instance.web._t;
+odoo.define('web_readonly_bypass', function(require) {
+    'use strict';
 
-    instance.web_readonly_bypass = {
+    var data = require('web.data'),
+        pyeval = require('web.pyeval');
+
+    var readonly_bypass = {
         /**
          * ignore readonly: place options['readonly_fields'] into the data
          * if nothing is specified into the context
@@ -26,7 +27,7 @@
                     }
                 });
             }
-            data = $.extend(data,readonly_by_pass_fields);
+            data = $.extend(data, readonly_by_pass_fields);
         },
 
         /**
@@ -59,15 +60,13 @@
         },
     };
 
-    var readonly_bypass = instance.web_readonly_bypass;
-
-    instance.web.BufferedDataSet.include({
+    data.BufferedDataSet.include({
 
         init : function() {
             this._super.apply(this, arguments);
         },
         /**
-         * Creates Overriding
+         * Create Overriding
          *
          * @param {Object} data field values to set on the new record
          * @param {Object} options Dictionary that can contain the following keys:
@@ -77,13 +76,12 @@
          */
         create : function(data, options) {
             var self = this;
-            var context = instance.web.pyeval.eval('contexts',
-                                                   self.context.__eval_context);
+            var context = pyeval.eval('contexts', self.context.get_eval_context());
             readonly_bypass.ignore_readonly(data, options, true, context);
             return self._super(data,options);
         },
         /**
-         * Creates Overriding
+         * Write Overriding
          *
          * @param {Object} data field values to set on the new record
          * @param {Object} options Dictionary that can contain the following keys:
@@ -93,15 +91,14 @@
          */
         write : function(id, data, options) {
             var self = this;
-            var context = instance.web.pyeval.eval('contexts',
-                                                   self.context.__eval_context);
+            var context = pyeval.eval('contexts', self.context.get_eval_context());
             readonly_bypass.ignore_readonly(data, options, false, context);
             return self._super(id,data,options);
         },
 
     });
 
-    instance.web.DataSet.include({
+    data.DataSet.include({
         /*
         BufferedDataSet: case of 'add an item' into a form view
         */
@@ -138,4 +135,7 @@
         },
 
     });
-})();
+
+    return readonly_bypass;
+
+});

web_readonly_bypass/static/test/web_readonly_bypass.js

@@ -1,22 +1,22 @@
-openerp.testing.section( 'web_readonly_bypass', {},
-function(test){
-        test('ignore_readonly', function(instance){
+odoo.define_section('web_readonly_bypass', ['web_readonly_bypass'], function(test) {
+"use strict";
+
+
+    test('ignore_readonly', function(assert, ro_bypass){
         var data = {};
         var mode_create = true;
         var options = {};
         var context = {};
-            instance.web_readonly_bypass.ignore_readonly(data, options,
-                mode_create, context);
-            deepEqual(data,
+        ro_bypass.ignore_readonly(data, options, mode_create, context);
+        assert.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,
+        ro_bypass.ignore_readonly(data, options, mode_create, context);
+        assert.deepEqual(data,
             {},
             "Empty context and options mode write"
         );
@@ -27,9 +27,9 @@ function(test){
         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,
+        ro_bypass.ignore_readonly(data, options, mode_create, context);
+
+        assert.deepEqual(data,
             {'field_1': 'va1-1', 'field_2': false, 'field_3': 'val-3'},
             "all fields mode write"
         );
@@ -40,9 +40,9 @@ function(test){
         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,
+        ro_bypass.ignore_readonly(data, options, mode_create, context);
+
+        assert.deepEqual(data,
             {'field_1': 'va1-1', 'field_3': 'val-3'},
             "all fields mode create (false value are escaped)"
         );
@@ -53,61 +53,61 @@ function(test){
         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,
+        ro_bypass.ignore_readonly(data, options, mode_create, context);
+
+        assert.deepEqual(data,
             {},
             "without context, default, we won't save readonly fields"
         );
     });
 
-        test('retrieve_readonly_by_pass_fields', function(instance){
+    test('retrieve_readonly_by_pass_fields', ['web_readonly_bypass'], function(assert, ro_bypass){
         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(
+        assert.deepEqual(
+            ro_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(
+        assert.deepEqual(
+            ro_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(
+        assert.deepEqual(
+            ro_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(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "Empty context field"
         );
 
         context = null;
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "Null context"
         );
 
         context = false;
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "false context"
@@ -115,8 +115,8 @@ function(test){
 
         context = {'readonly_by_pass': true}
         options = {'readonly_fields': {'field_1': 'va1-1'}};
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {'field_1': 'va1-1'},
             "Only one option"
@@ -124,40 +124,40 @@ function(test){
 
 
         options = {'readonly_fields': {}};
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "Empty readonly_fields option"
         );
 
         options = {};
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "Empty option"
         );
 
         options = null;
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "null option"
         );
 
         options = false;
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "false option"
         );
 
         context = false;
-            deepEqual(
-                instance.web_readonly_bypass.retrieve_readonly_by_pass_fields(
+        assert.deepEqual(
+            ro_bypass.retrieve_readonly_by_pass_fields(
                 options, context),
             {},
             "false option and false context"

web_readonly_bypass/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_ui

web_readonly_bypass/tests/test_ui.py

@@ -0,0 +1,15 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2017 Simone Orsi <simone.orsi@camptocamp.com>
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
+
+from odoo.tests import HttpCase
+
+
+class TestUi(HttpCase):
+
+    def test_ui_web(self):
+        self.phantom_js(
+            "/web/tests?module=web_readonly_bypass",
+            "",
+            login="admin",
+        )

web_readonly_bypass/views/readonly_bypass.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<openerp>
-    <data>
+<odoo>
     <template id="assets_backend" name="web_readonly_bypass" inherit_id="web.assets_backend">
         <xpath expr="." position="inside">
             <script type="text/javascript" src="/web_readonly_bypass/static/src/js/readonly_bypass.js"></script>
@@ -12,5 +11,4 @@
             <script type="text/javascript" src="/web_readonly_bypass/static/test/web_readonly_bypass.js"></script>
         </xpath>
     </template>
-    </data>
-</openerp>
+</odoo>