diff options
| author | Hardik Prajapati <hap@odoo.com> | 2019-09-05 08:28:14 +0000 |
|---|---|---|
| committer | Mohammed Shekha <msh@odoo.com> | 2019-09-12 10:01:47 +0000 |
| commit | 3998fa8dc18ac36cf99f273d96c4b5f536ff7a9f (patch) | |
| tree | d6f18308022b81802b51ef73cc04faf2aade7d42 /addons/web | |
| parent | 02327d8a6b5bd2b5c6b424005bb46d95ee39ff94 (diff) | |
[IMP] web: removed extra space in export field list
before this commit:
when manually adding field in export wizard, string has extra space in
begining.
after this commit:
manually added field in export field list doesn't contain any extra
spaces.
task-2065433
closes odoo/odoo#36759
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'addons/web')
| -rw-r--r-- | addons/web/static/src/js/widgets/data_export.js | 2 | ||||
| -rw-r--r-- | addons/web/static/tests/widgets/data_export_tests.js | 4 |
2 files changed, 4 insertions, 2 deletions
diff --git a/addons/web/static/src/js/widgets/data_export.js b/addons/web/static/src/js/widgets/data_export.js index 97ec75e6b47..87900f9f0aa 100644 --- a/addons/web/static/src/js/widgets/data_export.js +++ b/addons/web/static/src/js/widgets/data_export.js @@ -139,7 +139,7 @@ var DataExport = Dialog.extend({ $fieldList.append( $('<li>', {'class': 'o_export_field', 'data-field_id': fieldID}).append( $('<span>', {'class': "fa fa-arrows o_short_field mx-1"}), - label, + label.trim(), $('<span>', {'class': 'fa fa-trash m-1 pull-right o_remove_field', 'title': _t("Remove field")}) ) ); diff --git a/addons/web/static/tests/widgets/data_export_tests.js b/addons/web/static/tests/widgets/data_export_tests.js index 2d3488f5891..7abc3e93f6a 100644 --- a/addons/web/static/tests/widgets/data_export_tests.js +++ b/addons/web/static/tests/widgets/data_export_tests.js @@ -212,7 +212,7 @@ QUnit.module('widgets', { }); QUnit.test('Export dialog UI test', async function (assert) { - assert.expect(4); + assert.expect(5); var list = await createView({ View: ListView, model: 'partner', @@ -269,6 +269,8 @@ QUnit.module('widgets', { // Add field await testUtils.dom.click($('.modal div:contains(Activities) .o_add_field')); assert.strictEqual($('.modal .o_fields_list li').length, 2, "There should be two fields in export field list."); + assert.strictEqual($('.modal .o_fields_list li:eq(1)').text(), "Activities", + "string of second field in export list should be 'Activities'"); // Remove field await testUtils.dom.click($('.modal .o_fields_list li:first .o_remove_field')); assert.strictEqual($('.modal .o_fields_list li').length, 1, "There should be only one field in list"); |