diff options
| author | Jeremy Kersten <jke@odoo.com> | 2020-11-20 13:34:24 +0000 |
|---|---|---|
| committer | Jeremy Kersten <jke@odoo.com> | 2020-11-20 14:11:23 +0000 |
| commit | 9359be0d4be946d9b20ba85ea26312b45a8c7da3 (patch) | |
| tree | 708f24172d285555237497161d0715a4d13c8fb7 | |
| parent | e18c43d217be89b35213236038e72bc8298061a0 (diff) | |
[FIX] website_sale: don't crash if website.filter.snippet has been deleted
Meanwhile another fix to make these filter more compliant in multi website.
For now, if you delete website 1 (and filter by cascade) and create website 2
and try to drop a caroussel product, it will crash because no filter exists.
closes odoo/odoo#62097
Signed-off-by: Jérémy Kersten (jke) <jke@openerp.com>
| -rw-r--r-- | addons/website_sale/static/src/snippets/s_dynamic_snippet_products/options.js | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/addons/website_sale/static/src/snippets/s_dynamic_snippet_products/options.js b/addons/website_sale/static/src/snippets/s_dynamic_snippet_products/options.js index b5b3bc7018b..5a92baed823 100644 --- a/addons/website_sale/static/src/snippets/s_dynamic_snippet_products/options.js +++ b/addons/website_sale/static/src/snippets/s_dynamic_snippet_products/options.js @@ -28,8 +28,10 @@ const dynamicSnippetProductsOptions = s_dynamic_snippet_carousel_options.extend( fields: ['id', 'res_id'], } }).then((data) => { - this.$target.get(0).dataset.filterId = data[0].res_id; - this.$target.get(0).dataset.numberOfRecords = this.dynamicFilters[data[0].res_id].limit; + if (data) { + this.$target.get(0).dataset.filterId = data[0].res_id; + this.$target.get(0).dataset.numberOfRecords = this.dynamicFilters[data[0].res_id].limit; + } }); }, |