From 0d9e4804b1cd84c729102e39b7a9c9bb430b8454 Mon Sep 17 00:00:00 2001 From: "Adrien Minne (adrm)" Date: Mon, 16 Oct 2023 09:17:10 +0200 Subject: [PATCH] [FIX] spreadsheet: re-insert pivot w/ contextual domain Re-inserting a pivot that had a contextual domain would fail because the data source would try to fetch the data with the domain as a string, but the domain should be a list at this point. Task: 3553122 Part-of: odoo/odoo#138733 --- .../spreadsheet/static/src/pivot/pivot_data_source.js | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/addons/spreadsheet/static/src/pivot/pivot_data_source.js b/addons/spreadsheet/static/src/pivot/pivot_data_source.js index 71a64595735..fef4ffcb235 100644 --- a/addons/spreadsheet/static/src/pivot/pivot_data_source.js +++ b/addons/spreadsheet/static/src/pivot/pivot_data_source.js @@ -3,6 +3,7 @@ import { _t } from "@web/core/l10n/translation"; import { OdooViewsDataSource } from "../data_sources/odoo_views_data_source"; import { SpreadsheetPivotModel } from "./pivot_model"; +import { Domain } from "@web/core/domain"; export class PivotDataSource extends OdooViewsDataSource { /** @@ -47,7 +48,15 @@ export class PivotDataSource extends OdooViewsDataSource { metadataRepository: this._metadataRepository, } ); - await model.load(this._initialSearchParams); + + const userContext = this._orm.user.context; + const domain = new Domain(this._initialSearchParams.domain).toList({ + ...this._initialSearchParams.context, + ...userContext, + }); + + const searchParams = { ...this._initialSearchParams, domain }; + await model.load(searchParams); return model; }