Wednesday, June 2, 2021
7 changes · master
Enhancements to existing features
Several Odoo apps now use the updated avatar naming for user, contact, and employee profile images. This keeps Enterprise modules aligned with the core platform change and helps prevent display or compatibility issues wherever profile pictures appear.
Original PR description
In odoo/odoo#69819 a new mixin is created and used by users, partners and employees. This caused us to rename image_xxxx fields to avatar_xxxx. reference: odoo/odoo#69819 task-2404630
Code cleanup and technical improvements
This change reorganizes batch payment reconciliation assets and tests so they live under the accountant-focused batch payment module. It keeps functionality aligned with the correct app structure, making future maintenance safer without changing day-to-day user workflows.
Original PR description
Should have been done in https://github.com/odoo/enterprise/commit/726879a6ec5eb5775caec4a0461ae5c1795c3d59
Miscellaneous changes
Steps: - Install Accounting - Go to Accounting / Reporting / Management / Budgets Analysis - Switch to Pivot view - Click Insert in Spreadsheet - Confirm - Try to autofill the Total row header Bug: Traceback here: https://github.com/odoo/enterprise/blob/72541b4904adc028e4700ff9e5be93e90ed7907e/documents_spreadsheet/static/src/js/o-spreadsheet/plugins/pivot_plugin.js#L1081 cols is undefined Explanation: `cols` is undefined because the Total header autofill goes through `_autofill
Original PR description
Steps: - Install Accounting - Go to Accounting / Reporting / Management / Budgets Analysis - Switch to Pivot view - Click Insert in Spreadsheet - Confirm - Try to autofill the Total row header Bug: Traceback here: https://github.com/odoo/enterprise/blob/72541b4904adc028e4700ff9e5be93e90ed7907e/documents_spreadsheet/static/src/js/o-spreadsheet/plugins/pivot_plugin.js#L1081 cols is undefined Explanation: `cols` is undefined because the Total header autofill goes through `_autofillPivotColHeader` while autofilling a row header. Total headers don't have arguments. This makes them fit neither in row headers nor in column headers. This commit uses the direction of the autofill to know if the Total header is a row or a column header. opw:2522672 Forward-Port-Of: odoo/enterprise#18672 Forward-Port-Of: odoo/enterprise#18636
* `validate_session` was changing the signature of the original function * The assets were fetching files from another module. Forward-Port-Of: odoo/enterprise#18610
Original PR description
* `validate_session` was changing the signature of the original function * The assets were fetching files from another module. Forward-Port-Of: odoo/enterprise#18610
COM PR: https://github.com/odoo/odoo/pull/71536 task-2494916 Forward-Port-Of: odoo/enterprise#18654
Original PR description
COM PR: https://github.com/odoo/odoo/pull/71536 task-2494916 Forward-Port-Of: odoo/enterprise#18654
Description of the issue/feature this PR addresses: 1: When the tax_type is set to 'Exento', there was an error in the CFDI generated by Odoo as the case was explicitly skipped after the accounting rework between v13 and v14. 2: Additional conditions on displaying traslados and impuestos lines were missing Desired behavior after PR is merged: The same behaviour as in v13, tax exemption is handled and the fields are set to false when it is applicable. Missing conditions were added back.
Original PR description
Description of the issue/feature this PR addresses: 1: When the tax_type is set to 'Exento', there was an error in the CFDI generated by Odoo as the case was explicitly skipped after the accounting rework between v13 and v14. 2: Additional conditions on displaying traslados and impuestos lines were missing Desired behavior after PR is merged: The same behaviour as in v13, tax exemption is handled and the fields are set to false when it is applicable. Missing conditions were added back. OPW: 2503843 -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/enterprise#18542 Forward-Port-Of: odoo/enterprise#18261
This commit updates the twitter route URL used to fetch the tweet HTML. Indeed, the old "oembed" route is deprecated and will be removed completely soon (November 23, 2021). More information here: https://twittercommunity.com/t/consolidating-the-oembed-functionality/154690 In order to use the updated oembed route, we have to pass an additional author_id parameter, already contained in the retrieved tweet information. This implies a modification of the method signature, so the old rout
Original PR description
This commit updates the twitter route URL used to fetch the tweet HTML. Indeed, the old "oembed" route is deprecated and will be removed completely soon (November 23, 2021). More information here: https://twittercommunity.com/t/consolidating-the-oembed-functionality/154690 In order to use the updated oembed route, we have to pass an additional author_id parameter, already contained in the retrieved tweet information. This implies a modification of the method signature, so the old route was kept for retro-compatibility in case the parameter is missing. Task-2544931 Forward-Port-Of: odoo/enterprise#18641 Forward-Port-Of: odoo/enterprise#18593