Skip to content

Conversation

@giarve
Copy link

@giarve giarve commented Sep 19, 2025

The currency should be the main company currency for this report instead of the currently enabled company. Otherwise, values are incorrect because currency conversion is never performed.

must be fw ported to 17.0 and 18.0

@pedrobaeza pedrobaeza added this to the 16.0 milestone Nov 3, 2025
@pedrobaeza
Copy link
Member

If the problem is that res_company is not pointing to the proper company (the active one vs the wizard one), shouldn't you change the value of res_company instead?

@giarve
Copy link
Author

giarve commented Nov 4, 2025

If the problem is that res_company is not pointing to the proper company (the active one vs the wizard one), shouldn't you change the value of res_company instead?

Should res_company be pointing to the wizard one this time or always the active company?

You mention it because the report could be generated without the wizard at some point right?

To change it is it only required to to a t-set from the wizard or to modify the data getter for the report?

@pedrobaeza
Copy link
Member

There's no sense res_company to be the active company. For what? You want to have the "report company".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants