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

[17.0][MIG] partner_invoicing_mode_at_shipping: Migration to 17.0 #1874

Open
wants to merge 32 commits into
base: 17.0
Choose a base branch
from

Conversation

chaule97
Copy link

@chaule97 chaule97 commented Jan 3, 2025

mmequignon and others added 30 commits December 31, 2024 16:46
the `sale.order._create_invoices() method does not only requires
`create` access to the `account.move` and `account.move.line` models.

Because we're actually sorting lines after the creating, it requires
`read` access to sort, and `write` access to modify their sequence.

This is problematic when interracting with other modules like
`invoice_mode_at_shipping` where stock users are the ones to create
invoices.

This commit adds a few `sudo()` in the code, in order to avoid granting
`read` and `write` access to users that shouldn't be allowed to read and write invoices.
Currently translated at 100.0% (6 of 6 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode_at_shipping
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode_at_shipping/es/
Currently translated at 100.0% (6 of 6 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode_at_shipping
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode_at_shipping/it/
…ne_invoice_per_shipping' fill in / Use draft invoices.
…ing_validation function and add option auto validate invoice on partner
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode_at_shipping
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode_at_shipping/
Currently translated at 58.3% (7 of 12 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode_at_shipping
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode_at_shipping/es/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode_at_shipping
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode_at_shipping/it/
Currently translated at 100.0% (12 of 12 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-partner_invoicing_mode_at_shipping
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-partner_invoicing_mode_at_shipping/fr/
@chaule97 chaule97 force-pushed the 17.0-mig-partner_invoicing_mode_at_shipping branch from de3c6c9 to 150f82d Compare January 3, 2025 15:26
@chaule97 chaule97 force-pushed the 17.0-mig-partner_invoicing_mode_at_shipping branch from 150f82d to d148854 Compare January 3, 2025 15:43
@chaule97
Copy link
Author

chaule97 commented Jan 3, 2025

Sorry everyone, I have just fixed the hook code.

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.