diff --git a/addons/spreadsheet_account/static/src/index.js b/addons/spreadsheet_account/static/src/index.js index ea0d691fb6f..a0cf5ccf8c4 100644 --- a/addons/spreadsheet_account/static/src/index.js +++ b/addons/spreadsheet_account/static/src/index.js @@ -27,9 +27,14 @@ cellMenuRegistry.add("move_lines_see_records", { codes = toString(codes).split(","); const locale = env.model.getters.getLocale(); const dateRange = parseAccountingDate(date_range, locale); + offset = parseInt(offset) || 0; dateRange.year += offset || 0; - companyId = companyId || null; - includeUnposted = toBoolean(includeUnposted); + companyId = parseInt(companyId) || null; + try { + includeUnposted = toBoolean(includeUnposted); + } catch { + includeUnposted = false; + } const action = await env.services.orm.call( "account.account", diff --git a/addons/spreadsheet_account/static/tests/ui/accounting_drilldown_tests.js b/addons/spreadsheet_account/static/tests/ui/accounting_drilldown_tests.js index 025af20edac..c804805bb09 100644 --- a/addons/spreadsheet_account/static/tests/ui/accounting_drilldown_tests.js +++ b/addons/spreadsheet_account/static/tests/ui/accounting_drilldown_tests.js @@ -66,8 +66,10 @@ QUnit.module("spreadsheet_account > Accounting Drill down", { beforeEach }, () = env.model = model; setCellContent(model, "A1", `=ODOO.BALANCE("100", 2020)`); setCellContent(model, "A2", `=ODOO.BALANCE("100", 0)`); + setCellContent(model, "A3", `=ODOO.BALANCE("100", 2020, , , FALSE)`); + setCellContent(model, "A4", `=ODOO.BALANCE("100", 2020, , , )`); // Does not affect non formula cells - setCellContent(model, "A4", `5`); + setCellContent(model, "A5", `5`); await waitForDataSourcesLoaded(model); selectCell(model, "A1"); const root = cellMenuRegistry @@ -78,7 +80,15 @@ QUnit.module("spreadsheet_account > Accounting Drill down", { beforeEach }, () = assert.verifySteps(["drill down action"]); selectCell(model, "A2"); assert.equal(root.isVisible(env), false); + selectCell(model, "A3"); + assert.equal(root.isVisible(env), true); + await root.execute(env); + assert.verifySteps(["drill down action"]); selectCell(model, "A4"); + assert.equal(root.isVisible(env), true); + await root.execute(env); + assert.verifySteps(["drill down action"]); + selectCell(model, "A5"); assert.equal(root.isVisible(env), false); }); });