Friday, September 4, 2020
6 changes · master
Enhancements to existing features
This update removes an unused reference from the Sales test code. It does not change how the Sales app works, but keeps the codebase cleaner and easier to maintain.
Original PR description
Description of the issue/feature this PR addresses: Remove an unused import Current behavior before PR: `Form` is imported but not used anywhere. Desired behavior after PR is merged: The `Form` class is not imported. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
The payment app removes an outdated internal customization that no longer connects to the main invoicing process. This keeps the codebase cleaner and reduces maintenance risk without changing how users process payments.
Original PR description
The method action_validate_invoice_payment() has been removed from the base model but is still overriden in the payment module. This commit will remove the override. Task id #2285897 -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Odoo can now skip automatic sorting for searches and grouped reads when a specific order is not required. This can improve performance for queries where result order does not matter, reducing unnecessary database work.
Original PR description
Adds a parameter to be able to run queries without forcing an "order by"
clause. It is useful if you don't need any specific order and you'd like
a faster query.
Currently, it is not possible to remove the 'order by' clause, unless you
run the following ugly code:
self._order = None
self.search(...)
This commit supports passing `False` to "order by"
as the `order` argument of the method `search`:
self.search(..., order=False)
Since it doesn't fetch results in any specific order and without an extra
processing step for queries, it should be faster.
cc @rco-odoo @odony @xmo-odoo
If it is approved
- [x] Apply the same for `read_group()`This update removes outdated internal code from the SEPA Direct Debit area after the related payment logic was moved elsewhere. It should not change day-to-day behavior, but it reduces maintenance risk and keeps the accounting payment code aligned with the current system design.
Original PR description
The method get_batch_payment_methods_list() has been removed from the base model but is still overriden in the account_sepa_direct_debit module account_journal inherit. It seems like this functionnality is now implemented in the account_batch_payment model. This commit will remove the unused override. Task id #2285915
The Belgian reports module has been cleaned up by removing an outdated customization that no longer connects to the main reporting system. This reduces maintenance risk and helps keep the module aligned with the latest platform behavior, with no expected change for end users.
Original PR description
The method _get_vat_report_action_to_open() has been removed from the base model but is still overriden in the l10n_be_reports module. This commit will remove the override. Task id #2285933
An unused follow-up processing method was removed from the snail mail account follow-up area because the underlying account follow-up model no longer uses it. This is a minor cleanup that reduces maintenance risk without changing expected user-facing behavior.
Original PR description
The method _execute_followup_partner() has been removed from the base account_followup_report model but is still redefined in the one in snailmail_account_followup. This commit will thus remove it. Task id #2285947