diff --git a/addons/web_editor/static/src/js/backend/field_html.js b/addons/web_editor/static/src/js/backend/field_html.js index 0fe35887a7b..75548eee4b8 100644 --- a/addons/web_editor/static/src/js/backend/field_html.js +++ b/addons/web_editor/static/src/js/backend/field_html.js @@ -116,7 +116,8 @@ var FieldHtml = basic_fields.DebouncedField.extend(TranslatableFieldMixin, { * @override */ isSet: function () { - return this.value && this.value !== "


" && this.value.match(/\S/); + var value = this.value && this.value.split(' ').join('').replace(/\s/g, ''); // Removing spaces & html spaces + return value && value !== "

" && value !== "


" && value.match(/\S/); }, /** * @override diff --git a/addons/web_editor/static/tests/field_html_tests.js b/addons/web_editor/static/tests/field_html_tests.js index 6c4c3f9c5a3..ff50e0ef809 100644 --- a/addons/web_editor/static/tests/field_html_tests.js +++ b/addons/web_editor/static/tests/field_html_tests.js @@ -22,6 +22,11 @@ QUnit.module('web_editor', {}, function () { string: "Displayed name", type: "char" }, + header: { + string: "Header", + type: "html", + required: true, + }, body: { string: "Message", type: "html" @@ -30,6 +35,7 @@ QUnit.module('web_editor', {}, function () { records: [{ id: 1, display_name: "first record", + header: "

  

", body: "

toto toto toto

tata

", }], }, @@ -126,6 +132,38 @@ QUnit.module('web_editor', {}, function () { form.destroy(); }); + QUnit.test('check if required field is set', async function (assert) { + assert.expect(1); + + var form = await testUtils.createView({ + View: FormView, + model: 'note.note', + data: this.data, + arch: '
' + + '' + + '', + res_id: 1, + }); + + testUtils.mock.intercept(form, 'call_service', function (ev) { + if (ev.data.service === 'notification') { + assert.deepEqual(ev.data.args[0], { + "className": undefined, + "message": "", + "sticky": undefined, + "title": "The following fields are invalid:", + "type": "danger" + }); + } + }, true); + + await testUtils.form.clickEdit(form); + await testUtils.nextTick(); + await testUtils.dom.click(form.$('.o_form_button_save')); + + form.destroy(); + }); + QUnit.test('colorpicker', async function (assert) { assert.expect(6);