summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/fields
diff options
context:
space:
mode:
authorMartin Geubelle <mge@odoo.com>2017-09-15 14:18:57 +0200
committerMartin Geubelle <mge@odoo.com>2017-09-15 16:36:40 +0200
commitcb540c48eb575902018da97f2f541a813bda4d6b (patch)
tree3a0fff1ed0f2ece5cc960028e0fc34182c74e486 /addons/web/static/src/js/fields
parentae7f4c0485590623f2cd66cfa4aeb6b5c931047c (diff)
[FIX] web: correctly display the enterprise label
The `upgrade_boolean` widget should display an `Enterprise` label next to it. There is however an issue when a `label` is specified for this field. In this case, the `Enterprise` label should be placed next to the field label (and not the field itself). The easiest solution would have been to instantiate the field widget with its label, but this can't work because the field widget is instantiated before its label. We thus re-render the widget after its label creation.
Diffstat (limited to 'addons/web/static/src/js/fields')
-rw-r--r--addons/web/static/src/js/fields/upgrade_fields.js22
1 files changed, 16 insertions, 6 deletions
diff --git a/addons/web/static/src/js/fields/upgrade_fields.js b/addons/web/static/src/js/fields/upgrade_fields.js
index ae8cae56ef3..c2c30760abf 100644
--- a/addons/web/static/src/js/fields/upgrade_fields.js
+++ b/addons/web/static/src/js/fields/upgrade_fields.js
@@ -52,9 +52,9 @@ var AbstractFieldUpgrade = {
*
* @abstract
* @private
- * @param {JQuery} the 'Enterprise' label to insert
+ * @param {jQuery} $enterpriseLabel the 'Enterprise' label to insert
*/
- _insertEnterpriseLabel: function ($enterprise_label) {},
+ _insertEnterpriseLabel: function ($enterpriseLabel) {},
/**
* Opens the Upgrade dialog.
*
@@ -127,6 +127,15 @@ var UpgradeBoolean = FieldBoolean.extend(AbstractFieldUpgrade, {
events: _.extend({}, AbstractField.prototype.events, {
'click input': '_onInputClicked',
}),
+ /**
+ * Re-renders the widget with the label
+ *
+ * @param {jQuery} $label
+ */
+ renderWithLabel: function ($label) {
+ this.$label = $label;
+ this._render();
+ },
//--------------------------------------------------------------------------
// Private
@@ -136,8 +145,9 @@ var UpgradeBoolean = FieldBoolean.extend(AbstractFieldUpgrade, {
* @override
* @private
*/
- _insertEnterpriseLabel: function ($enterprise_label) {
- this.$el.append('&nbsp;').append($enterprise_label);
+ _insertEnterpriseLabel: function ($enterpriseLabel) {
+ var $el = this.$label || this.$el;
+ $el.append('&nbsp;').append($enterpriseLabel);
},
/**
* @override
@@ -170,8 +180,8 @@ var UpgradeRadio = FieldRadio.extend(AbstractFieldUpgrade, {
* @override
* @private
*/
- _insertEnterpriseLabel: function ($enterprise_label) {
- this.$('label').last().append('&nbsp;').append($enterprise_label);
+ _insertEnterpriseLabel: function ($enterpriseLabel) {
+ this.$('label').last().append('&nbsp;').append($enterpriseLabel);
},
/**
* @override