Sunday, January 21, 2024
4 changes · master
Miscellaneous changes
'_redirect_to_record' is used notably when accessing 'mail/view' controller used notably with email notifications ('See Document' link). In order to try to avoid multi company issues a call to '_get_mail_redirect_suggested_company' is performed in order to find the record's preferred company. However it is defined on 'mail.thread' which means the redirect crashes if invoked using a non thread record. This is quite rare but may happen with models using a mailgateway implementation similar to '
Original PR description
'_redirect_to_record' is used notably when accessing 'mail/view' controller used notably with email notifications ('See Document' link). In order to try to avoid multi company issues a call to '_get_mail_redirect_suggested_company' is performed in order to find the record's preferred company.
However it is defined on 'mail.thread' which means the redirect crashes if invoked using a non thread record. This is quite rare but may happen with models using a mailgateway implementation similar to 'mail.thread' without inheriting from the mixin (see 'mail.group' model added in Odoo 16).
This limitation is present since a long time but is now more present since odoo/odoo@f4523fcd9219fd781a984dab11a0b8689e236b77 .
Task-3689459
Forward-Port-Of: odoo/odoo#150086
Forward-Port-Of: odoo/odoo#149794Install module l10n_eu_oss_reports and website_sale Edit a product (i.e. Acoustic Bloc Screens) In Accounting tab add an account tag Try to access the product in webshop as public user User receive an error message 403: Forbidden The page you were looking for could not be authorized. Error message: You are not allowed to access 'Account Tag' (account.account.tag) records. [...] This occurs because during tax computation the system try to access account tags but they cannot be r
Original PR description
Install module l10n_eu_oss_reports and website_sale Edit a product (i.e. Acoustic Bloc Screens) In Accounting tab add an account tag Try to access the product in webshop as public user User receive an error message 403: Forbidden The page you were looking for could not be authorized. Error message: You are not allowed to access 'Account Tag' (account.account.tag) records. [...] This occurs because during tax computation the system try to access account tags but they cannot be read by the public user opw-3455325 Forward-Port-Of: odoo/odoo#145536
Steps to reproduce: Set up a UPS shipping method, enable "Collect on Delivery" in the configuration. Make a sale order with this shipping method, confirm and validate the delivery. Error: ``` Missing or invalid COD monetary value for Package 1. Valid length is 0 to 8 numeric and can hold up to 2 decimal places. ``` Based on UPS documentation, the value for COD needs to be of type string. opw-3666755 Forward-Port-Of: odoo/enterprise#54715 Forward-Port-Of: odoo/enterprise#53896
Original PR description
Steps to reproduce: Set up a UPS shipping method, enable "Collect on Delivery" in the configuration. Make a sale order with this shipping method, confirm and validate the delivery. Error: ``` Missing or invalid COD monetary value for Package 1. Valid length is 0 to 8 numeric and can hold up to 2 decimal places. ``` Based on UPS documentation, the value for COD needs to be of type string. opw-3666755 Forward-Port-Of: odoo/enterprise#54715 Forward-Port-Of: odoo/enterprise#53896
When inheriting from a test case that contains tests methods, the tests are run twice. The only reason to inherit from this case here, was to use the created partner. So let's use the existing common case ans let's create the partner. Forward-Port-Of: odoo/enterprise#54758 Forward-Port-Of: odoo/enterprise#54727
Original PR description
When inheriting from a test case that contains tests methods, the tests are run twice. The only reason to inherit from this case here, was to use the created partner. So let's use the existing common case ans let's create the partner. Forward-Port-Of: odoo/enterprise#54758 Forward-Port-Of: odoo/enterprise#54727