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

[mis_builder] extended account selectors #345

Closed
sbidoul opened this issue Sep 3, 2017 · 3 comments
Closed

[mis_builder] extended account selectors #345

sbidoul opened this issue Sep 3, 2017 · 3 comments

Comments

@sbidoul
Copy link
Member

sbidoul commented Sep 3, 2017

Accounting expressions in mis_builder have the form

(bal|crd|deb)(p|i|e)[account selector]([optional move line domain])?

The account selector is currently based on account codes (eg 70%).

Further account filtering can be added in the move line domain, eg ('account_id.type', '=', ...). But that is slow because additional database queries must be done, whereas the account selector boils done to one read_group per period and move line domain.

So it is desirable to avoid using the move line domains to filter on accounts.

To implement this, the account selector can be extended to support any domain on gl accounts.

@JordiBForgeFlow
Copy link
Sponsor Member

JordiBForgeFlow commented Sep 3, 2017 via email

@sbidoul
Copy link
Member Author

sbidoul commented Sep 3, 2017

Would be great that tags was in the selecion options

Absolutely! So you will be able to write balp[('tag_ids.name', '=', '...')].

On a related topic, optimizing selection on taxes would be interesting too, so you could write an efficient tax report with mis_builder.

@sbidoul
Copy link
Member Author

sbidoul commented Sep 30, 2017

This issue was moved to OCA/mis-builder#4

@sbidoul sbidoul closed this as completed Sep 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants