diff options
| author | Xavier Morel <xmo@openerp.com> | 2012-05-22 16:16:49 +0200 |
|---|---|---|
| committer | Xavier Morel <xmo@openerp.com> | 2012-05-22 16:16:49 +0200 |
| commit | 09bc1b0d3854d77246e9aab94237cb0b4ecdf4c2 (patch) | |
| tree | 98b8e5525bbb09dbdd476cb33920401ca41604ec /addons/web/static/src | |
| parent | 79c79e1d17964a65d20bd8ac1175c602d1095f8d (diff) | |
[IMP] remove unneeded/redundant 'switch' cases, rename variable (remove unneeded trailing underscore)
bzr revid: xmo@openerp.com-20120522141649-e87xncspejpq3tgi
Diffstat (limited to 'addons/web/static/src')
| -rw-r--r-- | addons/web/static/src/js/view_form.js | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js index 3c55579df6f..3430f1dd703 100644 --- a/addons/web/static/src/js/view_form.js +++ b/addons/web/static/src/js/view_form.js @@ -794,34 +794,29 @@ instance.web.FormView = instance.web.View.extend(_.extend({}, instance.web.form. var self = this; var fields = _.chain(this.fields) .map(function (field, name) { - var value_ = field.get_value(); + var value = field.get_value(); // ignore fields which are empty, invisible, readonly, o2m // or m2m - if (!value_ + if (!value || field.get('invisible') || field.get("readonly") || field.field.type === 'one2many' || field.field.type === 'many2many') { return false; } - var displayed; - switch(field.field.type) { + var displayed = value; + switch (field.field.type) { case 'selection': displayed = _(field.values).find(function (option) { - return option[0] === value_; + return option[0] === value; })[1]; break; - case 'many2one': - displayed = value_; - break; - default: - displayed = value_; } return { name: name, string: field.node.attrs.string || field.field.string, - value: value_, + value: value, displayed: displayed, // convert undefined to false change_default: !!field.field.change_default |