mis_builder - extend AccountingExpressionProcessor about account-financial-reporting HOT 3 CLOSED

We could make AEP and AbstractModel, but I think class is not really meant to be extended.

I wrote some thoughts about this in the README roadmap section: A mechanism to have a global move line filter at the level of the report template, report instance, or even column. Such a domain filter would be ANDed with the other filters and would allow to easily create reports filtered on analytic axis or business unit. To be complete such a mechanism should allow implementing similar filters on non accounting queries.

With mis_builder as is, you can already write expressions such as balp[60%][('operating_unit_id', '=', ...)]. But you know that, since you did reports for projects.

What I'd suggest is start with the following:

• add an "Additional Domain" field on mis.report and mis.report.instance
• AND these two domains and pass the result as an additional parameter to aep.do_queries()
• append it here: https://github.com/OCA/account-financial-reporting/blob/9.0/mis_builder/models/aep.py#L349

Then it's just a matter of extending the mis.report.instance form in your operational unit module to add the OU field and update the new domains in the on-change.

I think it would be a good start and I'm totally willing to review such a PR.

from account-financial-reporting.

@jbeficent please have a look at #128. This should provides the hooks you need.

from account-financial-reporting.

This is now implemented. Closing.

from account-financial-reporting.