Odoo Changelog — 2023-09-27

Wednesday, September 27, 2023

4 changes

Improvements(4)

Enhancements to existing features

[IMP] appointment: remove MockServer patch interfering with calendar tests

Enterprisemasterappointment

This change updates how appointment-related test data is mocked so it no longer disrupts calendar tests when multiple modules are installed. It helps keep automated testing stable and reduces the chance of unrelated test failures slowing development.

Original PR description

The appointment module currently extends the CalendarAttendeeController from the calendar module. Since appointment adds some rpc calls in the controller, it was decided to patch the MockServer with the appointment routes to prevent having tests fail because of unimplemented routes. However, patching the MockServer is not advised by the JS team and the current patch calls ORM mock methods (like mockRead, mockSearchRead...) that depend on serverData that is solely defined on the appointment tests. This causes any test done in the calendar module using the calendar_attendee js class to fail when all modules are installed because, even though the route is mocked, it tries to fetch data that is only defined for appointment. This commit changes the approach to use the mock_server registry, defining each route individually. For routes that are called on the onWillStart hook and their mock should be doing a mockRead, it was decided to overwrite the mock inside the appointment tests. This way, it is possible to return an "empty" object for all tests that are not appointment related, and for appointment tests we simply overwrite the mock_server for that route with the original implementation and after the tests are done, we can restore the mock.

#appointments#calendar#testing#stability

[IMP] *: remove legacy rpc

PR #47775

Enterprisemasteraccount_bank_statement_import_csvappointmenthr_contract_salarymail_mobilepayment_sepa_direct_debitpos_blackbox_besocial_push_notificationsvoipweb_studiowebsite_appointmentwebsite_helpdeskwebsite_knowledgewebsite_sale_rentingwebsite_sale_stock_rentingwebsite_sale_upswebsite_twitter_wall

This change updates several Odoo Enterprise apps to work with the newer internal communication services after older components were removed. It helps keep appointment booking, payments, rentals, POS, website tools, and other features compatible with the latest platform foundation without changing their intended business behavior.

[IMP] project_enterprise: move planned dates column after name

PR #47652

Enterprisemasterproject_enterprise

The task list in Project now shows planned dates immediately after the task name. This makes scheduling information easier to spot while reviewing or organizing project tasks.

Original PR description

...in task tree view.

Related: https://github.com/odoo/odoo/pull/136001
task-3514409

#project#tasks#planning#usability

[IMP] core: add SQL wrapper

PR #47759

Enterprisemasteraccount_accountantaccount_reportsaccount_reports_cash_basisdocuments_projectindustry_fsm_salel10n_pe_reports

This update adapts several accounting, reporting, document, and field service areas to a newer shared database query mechanism. It reduces the risk of errors in reports and related workflows while keeping the changes mostly behind the scenes for users.

Original PR description

Companion of https://github.com/odoo/odoo/pull/134677.

#accounting#reporting#documents#field-service

The appointment module currently extends the CalendarAttendeeController from the calendar module. Since appointment adds some rpc calls in the controller, it was decided to patch the MockServer with the appointment routes to prevent having tests fail because of unimplemented routes. However, patching the MockServer is not advised by the JS team and the current patch calls ORM mock methods (like mockRead, mockSearchRead...) that depend on serverData that is solely defined on the appointment tests. This causes any test done in the calendar module using the calendar_attendee js class to fail when all modules are installed because, even though the route is mocked, it tries to fetch data that is only defined for appointment. This commit changes the approach to use the mock_server registry, defining each route individually. For routes that are called on the onWillStart hook and their mock should be doing a mockRead, it was decided to overwrite the mock inside the appointment tests. This way, it is possible to return an "empty" object for all tests that are not appointment related, and for appointment tests we simply overwrite the mock_server for that route with the original implementation and after the tests are done, we can restore the mock.