Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] base: wrong default inherited view in qweb reports #38291

Open
wants to merge 1 commit into
base: 11.0
from

Conversation

@agr-odoo
Copy link
Contributor

commented Oct 9, 2019

In debug mode open Settings>Technical>Reporting>Reporting
Select one report (i.e. Purchase Order), click on "Qweb views", select
one of the shown report (i.e. report_purchaseorder), switch in Edit
mode, go to "Inherited Views" tab, click on "Add an item", a form will
popup partially filled.

The "Inherited View" shown by the popup is not the view in which the
user was, creating confusion and potentially could also be non-existant.

This is because of the xml context passed with the request for default
values which gives an 'active_id' instead of the 'id' of the previous
view.

opw-2080681

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

In debug mode open Settings>Technical>Reporting>Reporting
Select one report (i.e. Purchase Order), click on "Qweb views", select
one of the shown report (i.e. report_purchaseorder), switch in Edit
mode, go to "Inherited Views" tab, click on "Add an item", a form will
popup partially filled.

The "Inherited View" shown by the popup is not the view in which the
user was, creating confusion and potentially could also be non-existant.

This is because of the xml context passed with the request for default
values which gives an 'active_id' instead of the 'id' of the previous
view.

opw-2080681
@robodoo robodoo added the seen 🙂 label Oct 9, 2019
@agr-odoo

This comment has been minimized.

Copy link
Contributor Author

commented Oct 9, 2019

@nim-odoo
Is it possible to do something for the running instances?

@C3POdoo C3POdoo added the OE label Oct 9, 2019
@nim-odoo

This comment has been minimized.

Copy link
Contributor

commented Oct 9, 2019

@agr-odoo for running instance, a manual update is necessary. I'll double check this tomorrow, though

@robodoo robodoo added the CI 🤖 label Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.