Saturday, June 11, 2022
9 changes · master
New functionality added to Odoo
This change adds a way for Odoo reports and grouped lists to filter results after totals are calculated, similar to filtering on final subtotals. This helps businesses avoid misleading grouped results, such as customers whose transactions cancel out to zero still appearing in reports.
Original PR description
Description of the issue/feature this PR addresses:
Having Clause in read_group Function
Current behavior before PR:
Let's say we have some records:
id | partner_id | balance
1 | 10 | 1000
2 | 10 | -1000
3 | 11 | 500
4 | 11 | 200
Currently, only 'WHERE' clauses are provided. If we call the read_group(domain=[('balance','!=',0)], fields=['partner_id', 'balance'], groupby=['partner_id']) , then we'll get:
partner_id | balance
10 | 0
11 | 700
Desired behavior after PR is merged:
Sometimes we need the read_group function to filter as the 'HAVING' clause in SQL do and expect something like read_group(domain=[], fields=['partner_id', 'balance'], groupby=['partner_id'], having_clause_domain=[('balance','!=',0)]) so we can get:
partner_id | balance
11 | 700
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-prEnhancements to existing features
The mail module has been reorganized so more display-related behavior is handled in shared models rather than individual interface components. This should make future improvements to chat, calls, and messaging screens easier to deliver while keeping the current user experience largely unchanged.
Original PR description
This commit moves code from components to models, by introduce some models and fields. It also cleans a few things in models, like: - use `Timer` model instead of `setTimeout` - replace local variables by model fields Task-2881075 Enterprise: https://github.com/odoo/enterprise/pull/28274
Miscellaneous changes
Forward-Port-Of: odoo/odoo#93344 Forward-Port-Of: odoo/odoo#93277
Original PR description
Forward-Port-Of: odoo/odoo#93344 Forward-Port-Of: odoo/odoo#93277
The VoIP module was adjusted to stay compatible with recent internal messaging changes in Odoo. This helps keep VoIP behavior stable for users after the underlying chatter component lifecycle was updated.
Original PR description
Task-2881075 Community: https://github.com/odoo/odoo/pull/93330
The js content of the odoo-editor is located into the web_editor/static/lib/web-editor directory. Only the js code located into */static/src/* is considered for export of translations. This is done to avoid poluting translations for code not managed by Odoo. The best solution is to move the odoo-editor code inside static/src but as a workaround in stable, we can add the terms manually in the .pot file. This has the drawback of being lost in the next export of translations so should be
Original PR description
The js content of the odoo-editor is located into the web_editor/static/lib/web-editor directory. Only the js code located into */static/src/* is considered for export of translations. This is done to avoid poluting translations for code not managed by Odoo. The best solution is to move the odoo-editor code inside static/src but as a workaround in stable, we can add the terms manually in the .pot file. This has the drawback of being lost in the next export of translations so should be considered as a temporary solution. Fixes odoo/odoo#93258 Forward-Port-Of: odoo/odoo#93337 Forward-Port-Of: odoo/odoo#93272
This is a small fixup of recent commit [1]. It seems some files do include the charges in the Amt element at NtryDtls level, while others don't. The mentioned commit aims to always retrieve the amount at Ntry level in case of charges (as it always includes them) but this is creating side effects in case of batch transactions. We should partially revert it but still manage the case where there is only one transaction and charges are not included. [1]:https://github.com/odoo/enterprise/
Original PR description
This is a small fixup of recent commit [1]. It seems some files do include the charges in the Amt element at NtryDtls level, while others don't. The mentioned commit aims to always retrieve the amount at Ntry level in case of charges (as it always includes them) but this is creating side effects in case of batch transactions. We should partially revert it but still manage the case where there is only one transaction and charges are not included. [1]:https://github.com/odoo/enterprise/commit/b10c498f57b36b1f194f0064539d5fb39bdc27c2 opw-2859377 opw-2860167 Forward-Port-Of: odoo/enterprise#28272 Forward-Port-Of: odoo/enterprise#28111
This is a small fixup to commit [1]. In case the journal currency is set, we should round the computed amount with the rate using the currency's precision before checking if it matches the entry amount. Otherwise, it could create a rounding issue depending on the rate exact value. [1]: https://github.com/odoo/enterprise/commit/400e2bcfcbfd9562db5ba9e3ce5caa634ae6f904 opw-2859377 Forward-Port-Of: odoo/enterprise#28265 Forward-Port-Of: odoo/enterprise#28123
Original PR description
This is a small fixup to commit [1]. In case the journal currency is set, we should round the computed amount with the rate using the currency's precision before checking if it matches the entry amount. Otherwise, it could create a rounding issue depending on the rate exact value. [1]: https://github.com/odoo/enterprise/commit/400e2bcfcbfd9562db5ba9e3ce5caa634ae6f904 opw-2859377 Forward-Port-Of: odoo/enterprise#28265 Forward-Port-Of: odoo/enterprise#28123
The filters were the default one for invoices, while it should have been the filters for entries, so we just add them to the action of the stat button. t-2879022 Forward-Port-Of: odoo/enterprise#28177
Original PR description
The filters were the default one for invoices, while it should have been the filters for entries, so we just add them to the action of the stat button. t-2879022 Forward-Port-Of: odoo/enterprise#28177
…ank reco widget Create a statement line Create a batch with at least 2 payments Open the statement line in the bank reconciliation widget Mount the batch Remove one of the payments Mount the batch a second time => The remaining payment has been duplicated Forward-Port-Of: odoo/enterprise#28130
Original PR description
…ank reco widget Create a statement line Create a batch with at least 2 payments Open the statement line in the bank reconciliation widget Mount the batch Remove one of the payments Mount the batch a second time => The remaining payment has been duplicated Forward-Port-Of: odoo/enterprise#28130