Wednesday, July 17, 2024
4 changes · saas-17.3
Miscellaneous changes
The index definition that supports the implementation of `_search_phone_mobile_search` defined in the `PhoneMixin` is created in an `_auto_init` in `crm_lead` of the `crm` module. This means that other models that use the mixin (for ex: `res_partner`) will not get the indexes for it's table, leading to Seq.Scans when searching for a phone number on those models. By hoisting the index definition to an `init` in the mixin, all models that inherit from the mixin will have the supporting index.
Original PR description
The index definition that supports the implementation of `_search_phone_mobile_search` defined in the `PhoneMixin` is created in an `_auto_init` in `crm_lead` of the `crm` module. This means that other models that use the mixin (for ex: `res_partner`) will not get the indexes for it's table, leading to Seq.Scans when searching for a phone number on those models. By hoisting the index definition to an `init` in the mixin, all models that inherit from the mixin will have the supporting index. Also add the same custom `trigram` index to support `like/ilike` operators in domains. task-3942852 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#166698
Clear the EDI document errors and the blocking_level when the action to abandon cancel posted moves is called. Fixed the following case: 1. Stamp a customer invoice (Could you test with a Mexican record) 2. Request EDI cancellation (In this case must be generated a cancel error from the PAC, maybe the customer does not accept the cancellation) 3. Ensure that in step 2, the EDI document has an error message 4. Call the action to abandon the EDI cancellation Without this commit, the EDI
Original PR description
Clear the EDI document errors and the blocking_level when the action to abandon cancel posted moves is called. Fixed the following case: 1. Stamp a customer invoice (Could you test with a Mexican record) 2. Request EDI cancellation (In this case must be generated a cancel error from the PAC, maybe the customer does not accept the cancellation) 3. Ensure that in step 2, the EDI document has an error message 4. Call the action to abandon the EDI cancellation Without this commit, the EDI document returns to `send`, but the record presents the error. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#172169
When importing a module through `base_import_module`, it does not have a path on the file system. Because of this the SVG shapes could not be located from the file system for imported modules. This commit falls back to `ir_attachment` record that contains the imported module file in case the module was an imported one. Steps to reproduce: - Have a module within a zip that contains a block with an image shape. (e.g. zip `website_coconuts` from [1]) - Install `base_import_module`. - G
Original PR description
When importing a module through `base_import_module`, it does not have a path on the file system. Because of this the SVG shapes could not be located from the file system for imported modules. This commit falls back to `ir_attachment` record that contains the imported module file in case the module was an imported one. Steps to reproduce: - Have a module within a zip that contains a block with an image shape. (e.g. zip `website_coconuts` from [1]) - Install `base_import_module`. - Go to 'Apps' in debug mode. - Import the zipped module with `Import Module`. => The image obtained through the `/web_editor/image_shape` route was not retrieved and a "module not found" warning appeared in the logs. [1]: https://github.com/odoo/odoo/commit/43f18e0863c779493f7b0ed82c37acb8344d9ae3 task-2888647 Forward-Port-Of: odoo/odoo#172917 Forward-Port-Of: odoo/odoo#94161
…d warning Steps to reproduce: - Go to Accounting/Reporting/Tax Report; - Make sure there is a negative amount in the tax report; - Click on PDF/EDI VAT/Test Interchange (available in debug) not to send the tax report to the government for real, and "Send VAT Report"; - Go to Reporting/EDI exports > it should have the "Rejected" status because of the negative amount(s); Issue: On the Accounting Dashboard: the error "Export(s) de taxes rejeté(s)" is appearing on several journals Sol
Original PR description
…d warning Steps to reproduce: - Go to Accounting/Reporting/Tax Report; - Make sure there is a negative amount in the tax report; - Click on PDF/EDI VAT/Test Interchange (available in debug) not to send the tax report to the government for real, and "Send VAT Report"; - Go to Reporting/EDI exports > it should have the "Rejected" status because of the negative amount(s); Issue: On the Accounting Dashboard: the error "Export(s) de taxes rejeté(s)" is appearing on several journals Solution: We get the journal defined for the Tax Return Periodicity. If no journal defined; We keep only the first Misc journal that is found This way, we let the user have custom misc journals (fec import or custom coa) opw-4004375 Forward-Port-Of: odoo/enterprise#66346