Skip to content

Conversation

@ThiagoMForgeFlow
Copy link
Contributor

Supersede: #793

@ForgeFlow

yostashiro and others added 14 commits June 10, 2025 14:43
Currently translated at 100.0% (4 of 4 strings)

Translation: account-analytic-16.0/account-analytic-16.0-mrp_stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-mrp_stock_analytic/it/
* analytic_distribution is constrained to be valid on the MO, so we don't need to validate
  on confirmation
* The `states`-attribute on ORM-fields is deprecated, and should be moved to views
* mrp_account adds the distribution to the views we want, so we can just add attributes instead of re-adding the field
* mrp.production already has mixin.analytic, no need to re-add it
* analytic_distribution is already in the MO list view, no need to change its default visibility
Switched to using shared base class instead
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-17.0/account-analytic-17.0-mrp_stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-mrp_stock_analytic/
Currently translated at 100.0% (2 of 2 strings)

Translation: account-analytic-17.0/account-analytic-17.0-mrp_stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-mrp_stock_analytic/it/
@ThiagoMForgeFlow
Copy link
Contributor Author

ThiagoMForgeFlow commented Nov 27, 2025

Hi @anajuaristi, @luisDIXMIT what do you think about this PR?

Copy link

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on runboat and code review, LGTM!

@ThiagoMForgeFlow
Copy link
Contributor Author

@AaronHForgeFlow What do you think about it?

@anajuaristi
Copy link

anajuaristi commented Dec 22, 2025

Hi @ThiagoMForgeFlow
I have been testing in a demo database and I have got several concerns.

In production order there is a field analytic distribution many2many. I tried several ways (with a single distribution, adding several of them, consuming everything, making partials... ) I could not obtain any single analytic line created.

Maybe I'm missing something but It seems it doesn't work for me.

On the other side, there is another module project_mrp, directly included on Odoo V18 core that allows to set a project_id on the manufacturing order. On this case, if you set the project, system will create the cost analytic lines against the analytic account of the project so I don't know if both can be compatible or not the way it is.
Just to tell you that there could be a kind of inconsistence here if someone is installing both modules. I don't know.

Please let me know.
Thank you!

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.

10 participants