Daily updates from Odoo
Navigate
Branch
Sunday, March 17, 2024
5 changes
2 changes
Miscellaneous changes
…ther This commit only improve the test according to change done in odoo/odoo. Community PR: odoo/odoo#155986 Forward-Port-Of: odoo/enterprise#58707 Forward-Port-Of: odoo/enterprise#57846
Original PR description
…ther This commit only improve the test according to change done in odoo/odoo. Community PR: odoo/odoo#155986 Forward-Port-Of: odoo/enterprise#58707 Forward-Port-Of: odoo/enterprise#57846
The test added in PR #51275 was never run since the method name did not start with the correct prefix. This commit fixes te commit name as well as the test content that did not test the introduced behaviour. Forward-Port-Of: odoo/enterprise#58436
Original PR description
The test added in PR #51275 was never run since the method name did not start with the correct prefix. This commit fixes te commit name as well as the test content that did not test the introduced behaviour. Forward-Port-Of: odoo/enterprise#58436
3 changes
Resolved issues and error corrections
This fix resolves a crash that occurred when users deleted a spreadsheet sheet containing Odoo charts. Previously, the system would retain references to charts from deleted sheets, causing the spreadsheet to fail when reopened. The fix ensures that charts are properly removed when their parent sheet is deleted, preventing the application from crashing.
Original PR description
__Current behavior before commit:__ When a sheet is deleted, `charts` from `OdooChartCorePlugin` are not being updated. Therefore, since the commit [`905d856`][1], `getOdooChartIds` returns some chart ids that do not exist on any sheet anymore. This induces a crash when opening a spreadsheet that contains such charts. __Description of the fix:__ Handle `DELETE_SHEET` event in `OdooChartCorePlugin` by removing Odoo charts that don't belong to any sheet. __Steps to reproduce the issue on runbot:__ - Insert a Odoo graph inside a spreadsheet (starting from any app) - Add a sheet to it to the new spreadsheet - Delete the sheet that contains the chart - Leave the spreadsheet - Go to Documents app and try to open the spreadsheet -> Traceback opw-3783745 [1]: https://github.com/odoo/odoo/commit/905d8565ad2fae3f7ff96606352ee3ab86ee78cb Forward-Port-Of: odoo/odoo#157728
This fix resolves a crash in the mail system that occurred when custom models inherited from mail.thread but defined a user_id field as a text field instead of a relationship field. The system now properly checks the field type before processing, preventing errors in the message recipient suggestion feature.
Original PR description
-Step to reproduce: any custom model that inherit from mail.thread then define a field user_id but with type is char and boom error happen at method '_message_get_suggested_recipients' because it always expect 'user_id' to be a many2one field -Solution: need to check the type of 'user_id' also Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#157350 Forward-Port-Of: odoo/odoo#155149
Documentation and clarification updates
This pull request adds a Contributor License Agreement (CLA) for Third Ocean Private Limited, a company based in Maldives. This legal document authorizes the company to contribute to the Odoo project and ensures proper intellectual property rights management for Odoo's codebase.
Original PR description
CLA for Odoo Project contribution by Third Ocean Private Limited, Maldives. Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr