[FIX] web: properly evaluate unset numeric values in domain

Before this commit, when a numeric value was not set (for example, a new
record with no default_get value), then it was not properly evaluated as
0 in a domain.

We also slightly refactored the code so we always use the same method to
compute the evaluation context.

1 files changed, 25 insertions, 0 deletions

diff --git a/addons/web/static/test/views/form_tests.js b/addons/web/static/test/views/form_tests.js
index d630cfd7e01..40ea74f8890 100644
--- a/addons/web/static/test/views/form_tests.js
+++ b/addons/web/static/test/views/form_tests.js
@@ -363,6 +363,31 @@ QUnit.module('Views', {
         form.destroy();
     });
 
+    QUnit.test('invisible attrs with zero value in domain and unset value in data', function (assert) {
+        assert.expect(1);
+
+        this.data.partner.fields.int_field.type = 'monetary';
+
+        var form = createView({
+            View: FormView,
+            model: 'partner',
+            data: this.data,
+            arch: '<form string="Partners">' +
+                    '<sheet>' +
+                        '<field name="foo"/>' +
+                        '<group attrs=\'{"invisible": [["int_field", "=", 0.0]]}\'>' +
+                            '<div class="hello">this should be invisible</div>' +
+                            '<field name="int_field"/>' +
+                        '</group>' +
+                    '</sheet>' +
+                '</form>',
+        });
+
+        assert.notOk(form.$('div.hello').is(':visible'),
+            "attrs invisible should have been computed and applied");
+        form.destroy();
+    });
+
     QUnit.test('rendering stat buttons', function (assert) {
         assert.expect(3);