[FIX] report_xlsx: fix generation of pdf reports

the report handler for xlsx added over here is always returning `true` even if the report being
generated is not xlsx. Due to this, the report handler doesn't check for other report types
at b309d3a99f/addons/web/static/src/webclient/actions/action_service.js (L1019)
and thus doesn't generate any other reports. So if the converter is not xlsx, return false instead of true.

Also, updating the docids and context in the report_routes controller only needs to be done if
the converter is xlsx and not in any other case.
This commit is contained in:
Mihran Thalhath
2022-01-18 12:00:20 +05:30
committed by Rodrigo
parent 97e511c388
commit 867a989be6
3 changed files with 12 additions and 11 deletions

View File

@@ -48,6 +48,7 @@ registry
} else if (onClose) {
onClose();
}
return Promise.resolve(true);
}
return Promise.resolve(true);
return Promise.resolve(false);
});