diff options
| author | nie <nie@odoo.com> | 2021-02-08 14:54:09 +0000 |
|---|---|---|
| committer | nie <nie@odoo.com> | 2021-02-09 14:01:02 +0000 |
| commit | a8680ac8815575762b702917940c0b2d4dc0c4ea (patch) | |
| tree | 5852fbc0fb677060d429d48dabcc0081498f858a | |
| parent | 6ac86b426352fa1a2260372b28fe3bd26b25f362 (diff) | |
[FIX] web: fields info for a specific view can be undefined
Steps:
- Install mrp
- Go to Manufacturing > Master Data > Bills of Materials
- Create a BoM
- Components:
- Add a component
- Save
- Click the component
- Close the modal window
- Click the component once more
Bug:
Traceback here:
https://github.com/odoo/odoo/blob/55a6642a9621fa9683895d5d45a015bb04c3017b/addons/web/static/src/js/views/basic/basic_view.js#L147
Error: can't convert undefined to object
Explanation:
As seen on the line just above:
https://github.com/odoo/odoo/blob/55a6642a9621fa9683895d5d45a015bb04c3017b/addons/web/static/src/js/views/basic/basic_view.js#L146
`fieldsInfo` might not have the view we are looking for.
opw:2452142
closes odoo/odoo#65805
X-original-commit: b27c0698d95cbccaa6c35a1fd3faa852a71ac2c2
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
Signed-off-by: backspac <backspac@users.noreply.github.com>
| -rw-r--r-- | addons/web/static/tests/fields/relational_fields/field_one2many_tests.js | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/addons/web/static/tests/fields/relational_fields/field_one2many_tests.js b/addons/web/static/tests/fields/relational_fields/field_one2many_tests.js index 2f74ee4670b..f4d9fba9eec 100644 --- a/addons/web/static/tests/fields/relational_fields/field_one2many_tests.js +++ b/addons/web/static/tests/fields/relational_fields/field_one2many_tests.js @@ -7285,7 +7285,7 @@ QUnit.module('fields', {}, function () { }); QUnit.test('nested one2manys with no widget in list and as invisible list in form', async function (assert) { - assert.expect(4); + assert.expect(6); this.data.partner.records[0].p = [1]; @@ -7311,6 +7311,13 @@ QUnit.module('fields', {}, function () { assert.containsOnce(document.body, '.modal .o_form_view'); assert.containsNone(document.body, '.modal .o_form_view .o_field_one2many'); + // Test possible caching issues + await testUtils.dom.click($('.modal .o_form_button_cancel')); + await testUtils.dom.click(form.$('.o_data_row')); + + assert.containsOnce(document.body, '.modal .o_form_view'); + assert.containsNone(document.body, '.modal .o_form_view .o_field_one2many'); + form.destroy(); }); |
