[REF] web,*: improve the way field_utils are done

We replace a bunch of formatting and parsing functions by an object
(parse or format) which can be used to dispatch on the correct type.  It
allows us to remove the format_field and parse_field functions.

Also, we camelcase our new code and add some comments.

1 files changed, 4 insertions, 3 deletions

diff --git a/addons/web/static/src/js/widgets/domain_selector.js b/addons/web/static/src/js/widgets/domain_selector.js
index ba1ac9e77c9..61b43c46ae7 100644
--- a/addons/web/static/src/js/widgets/domain_selector.js
+++ b/addons/web/static/src/js/widgets/domain_selector.js
@@ -384,7 +384,7 @@ var DomainLeaf = DomainNode.extend({
                 try {
                     var f = this.fieldSelector.selectedField;
                     if (!f.relation) { // TODO in this case, the value should be m2o input, etc...
-                        this.displayValue = field_utils.format_field(this.value, this.fieldSelector.selectedField);
+                        this.displayValue = field_utils.format[f.type](this.value, f);
                     }
                 } catch (err) {/**/}
                 this.displayOperator = this.operator;
@@ -470,9 +470,10 @@ var DomainLeaf = DomainNode.extend({
     /// @param silent - true if the method call should not trigger_up a domain_changed event
     /// @trigger_up domain_changed event to ask for a re-rendering
     onValueChange: function (value, silent) {
+        var field = this.fieldSelector.selectedField;
         var couldNotParse = false;
         try {
-            this.value = field_utils.parse_field(value, this.fieldSelector.selectedField);
+            this.value = field_utils.parse[field.type](value, field);
         } catch (err) {
             this.value = value;
             couldNotParse = true;
@@ -488,7 +489,7 @@ var DomainLeaf = DomainNode.extend({
             }
         } else if (_.contains(["date", "datetime"], this.fieldSelector.selectedField.type)) {
             if (couldNotParse || _.isBoolean(this.value)) {
-                this.value = field_utils.parse_field(field_utils.format_field(Date.now(), this.fieldSelector.selectedField), this.fieldSelector.selectedField);
+                this.value = field_utils.parse[field.type](field_utils.format[field.type](Date.now(), field), field);
             }
         } else {
             if (_.isBoolean(this.value)) { // Never display "true" or "false" strings from boolean value