Skip to content

Conversation

@arnaudlayec
Copy link

@arnaudlayec arnaudlayec commented Sep 13, 2025

analytic_mixin_analytic_account

Make it compatible with other module account_financial_report of OCA/account-financial-reporting
Just add compute_sudo=True to both fields analytic_account_ids and analytic_account_names, so that the console does not throw the error account.move.line: inconsistent 'compute_sudo' for computed fields: analytic_account_ids, analytic_account_names
Indeed this last module also computes a field analytic_account_ids with `store=True

stock_analytic

Add a _compute_analytic_distribution method, quite generic, to apply by default the analytic distribution model at stock.move creation, like on any other object using analytic (purchase order, invoice, ...).

@arnaudlayec
Copy link
Author

Sorry for the 2 commits of different modules in the same PR, it was not intentional and I did not find how to easily correct this

@arnaudlayec arnaudlayec changed the title [16.0][IMP][analytic_mixin_analytic_account] Compatibility with account_financial_report of OCA/account-financial-reporting [16.0][IMP]analytic_mixin_analytic_account and stock_analytic Sep 14, 2025
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant