Wednesday, May 15, 2024
23 changes
1 change
Miscellaneous changes
The goal of this PR is to simply disable the "Text Animation" button from the editor's toolbar in translate mode. An adaptation of code in stable `17.0` will be added to allow the use of text options (text animations & text highlights) in the translation mode, mainly by allowing the creation of snippet editors if the target is a text option snippet. opw-3686777 Forward-Port-Of: odoo/odoo#157726 Forward-Port-Of: odoo/odoo#157087
Original PR description
The goal of this PR is to simply disable the "Text Animation" button from the editor's toolbar in translate mode. An adaptation of code in stable `17.0` will be added to allow the use of text options (text animations & text highlights) in the translation mode, mainly by allowing the creation of snippet editors if the target is a text option snippet. opw-3686777 Forward-Port-Of: odoo/odoo#157726 Forward-Port-Of: odoo/odoo#157087
18 changes
Enhancements to existing features
Additional automated checks were added to help prevent timers from incorrectly starting at a negative time and to avoid test errors when the Timer module is installed. This improves confidence that time tracking starts cleanly and remains reliable across related timesheet workflows.
Original PR description
[IMP] timer: Test start formatted time Before this commit: - Multiple instances of the timer were observed starting at -00:00:59 instead of 00:00:00. Unfortunately, it seems quite random and reproducing the issue exactly wasn't possible. - An error is raised when installing Timer module then starting JS tests. In this commit: - JS test added to try to reproduce the issue. - The file is inside timer but using hr_timesheet_common_tests from hr_timesheet. It also concerns hr_timesheet context. File moved to timesheet_grid and another test created in timer to simply test timer_start widget. task-3741938
4 changes
Resolved issues and error corrections
This update fixes failing tests in the Dutch tax reporting module (l10n_nl_reports_sbr_icp). The issue was caused by an outdated reference that wasn't properly updated during a code migration. By correcting this reference and adding the necessary tag, the tests can now run properly without being blocked.
Original PR description
Changed the chart_template_ref that wasn't adapted for a fw-port. Also added the tag, so the test's requests are not blocked in 17+ runbot-64732 runbot-62320 Forward-Port-Of: odoo/enterprise#62375
Available planning shifts are now shown in chronological order on the front-end. This makes the schedule easier to read and keeps the shift list consistent with the rest of the planning view.
Original PR description
It makes more sense to display the shifts in the chronological order from the front-end. It also allows to form a coherent whole with the rest of the view. task-3853862
The point of sale rental workflow tests were updated to match a newer standard order selection screen. This helps keep automated checks aligned with the current interface, reducing the risk of future changes breaking rental order settlement unnoticed.
Original PR description
In the corresponding community commit we replace the custom list view for selecting a `sale.order` record with the default `view` component from `web`. In this commit we adapt the tests. Task: 3859301 https://github.com/odoo/odoo/pull/161066
The Knowledge app was updated to stay aligned with related messaging changes in Odoo. This helps keep article discussions and message formatting working consistently after the underlying platform update.
Original PR description
@see https://github.com/odoo/odoo/pull/162065
Resolved issues and error corrections
The Chilean electronic invoicing point-of-sale setup now skips an unnecessary database lookup when there are no records to load. This reduces wasted processing during POS data loading and helps keep startup behavior efficient without changing user-facing workflows.
Original PR description
Previously, to avoid loading records when loading data into point_of_sale we used a domain with a falsy value `[('id', '=', False)]` but this required a query to the database.
Now we return False for the domain and if the domain is false we don't search for records. no record search.Code cleanup and technical improvements
Restaurant point-of-sale related test setup has been streamlined to reduce duplicated maintenance work across enterprise modules. This internal cleanup helps keep future changes more consistent and lowers the risk of test differences between related restaurant workflows.
Original PR description
The test assets that we use in pos are duplicated across different classes. In the corresponding commit we take the first steps in unifying them, by having the class in `pos_restaurant/tests/test_frontent.py` inherit from `point_of_sale`. In this commit we make the necessary changes to the enterprise modules. Task 3895255 https://github.com/odoo/odoo/pull/155540
Miscellaneous changes
Since a055dfeb84ab, the field `current_revision_uuid` is in the documents kanban view. Which means it can be computed for many documents. The compute method loads all the revisions of spreadsheet documents. Revisions can hold a lot of data (field `commands`), several hundreds of Mb for large spreadsheets. With this commit, we pre-fetch the only field that we need to prevent the ORM to prefetch all fields that we don't need (and specifically avoid loading the field `commands`) Task: 3918
Original PR description
Since a055dfeb84ab, the field `current_revision_uuid` is in the documents kanban view. Which means it can be computed for many documents. The compute method loads all the revisions of spreadsheet documents. Revisions can hold a lot of data (field `commands`), several hundreds of Mb for large spreadsheets. With this commit, we pre-fetch the only field that we need to prevent the ORM to prefetch all fields that we don't need (and specifically avoid loading the field `commands`) Task: 3918564 Forward-Port-Of: odoo/enterprise#62173
Changed the chart_template_ref that wasn't adapted for a fw-port. Also added the tag, so the test's requests are not blocked in 17+ runbot-64732 runbot-62320 Forward-Port-Of: odoo/enterprise#62375
Original PR description
Changed the chart_template_ref that wasn't adapted for a fw-port. Also added the tag, so the test's requests are not blocked in 17+ runbot-64732 runbot-62320 Forward-Port-Of: odoo/enterprise#62375
the nssf number restriction should not be apply given the real world scenario where the number can be in other format (eg 000000000x) task-3908312 Forward-Port-Of: odoo/enterprise#61947
Original PR description
the nssf number restriction should not be apply given the real world scenario where the number can be in other format (eg 000000000x) task-3908312 Forward-Port-Of: odoo/enterprise#61947
In the BE loca, we were flexible on the notion of complete month. This was useful for odoo but did not make sense for the localization. A complete month of work is if you work from the first work day of the month to the last. Non-working days and public holidays are excluded. Forward-Port-Of: odoo/enterprise#62258
Original PR description
In the BE loca, we were flexible on the notion of complete month. This was useful for odoo but did not make sense for the localization. A complete month of work is if you work from the first work day of the month to the last. Non-working days and public holidays are excluded. Forward-Port-Of: odoo/enterprise#62258
task-3908271 Forward-Port-Of: odoo/enterprise#61944
Original PR description
task-3908271 Forward-Port-Of: odoo/enterprise#61944
Quality check wizard shows '- False' instead of Serial number in title if no lot_name opw-3839794 Forward-Port-Of: odoo/enterprise#62260
Original PR description
Quality check wizard shows '- False' instead of Serial number in title if no lot_name opw-3839794 Forward-Port-Of: odoo/enterprise#62260
Before this commit, the upsell lines were not translated into the customer lang. taskid: 3911032 Forward-Port-Of: odoo/enterprise#62394 Forward-Port-Of: odoo/enterprise#62086
Original PR description
Before this commit, the upsell lines were not translated into the customer lang. taskid: 3911032 Forward-Port-Of: odoo/enterprise#62394 Forward-Port-Of: odoo/enterprise#62086
**Steps:** - Install indian payroll(l10n_in_hr_payroll) - Create an employee. - Create contract on that employee. - Generate payslip. - Go to employee app > Report . - Print the `Yearly Salary by Head` or `Yearly Salary by Employee`. **Description of the issue/feature this PR addresses:** In the payroll module, - when we print employee salary report it will give traceback. - The `yearly salary by employee` report was generating duplicate lines for the net salary category. **Cause
Original PR description
**Steps:** - Install indian payroll(l10n_in_hr_payroll) - Create an employee. - Create contract on that employee. - Generate payslip. - Go to employee app > Report . - Print the `Yearly Salary by…
**Steps:** - Install indian payroll(l10n_in_hr_payroll) - Create an employee. - Create contract on that employee. - Generate payslip. - Go to employee app > Report . - Print the `Yearly Salary by Head` or `Yearly Salary by Employee`. **Description of the issue/feature this PR addresses:** In the payroll module, - when we print employee salary report it will give traceback. - The `yearly salary by employee` report was generating duplicate lines for the net salary category. **Cause** printing the employee salary report resulted in a traceback error. This was caused by an update (reference: https://github.com/odoo/enterprise/pull/53816) that made category_id in the payslip_line module unstored. As a result, the SQL query couldn't find the category_id and failed. **Fix:** This PR fix issues by modifying the SQL query used in the reports. Instead of relying on the unstored category_id from payslip_line, the query now retrieves the category ID directly from the hr_salary_rule_category table. task-3791974 Forward-Port-Of: odoo/enterprise#58933
part of task-3818666 community: https://github.com/odoo/odoo/pull/165340 Forward-Port-Of: odoo/enterprise#62419
Original PR description
part of task-3818666 community: https://github.com/odoo/odoo/pull/165340 Forward-Port-Of: odoo/enterprise#62419
This commit aims to convert QUnit tests which rely on `mail/test_utils` to hoot. Part of task-3818666 Forward-Port-Of: odoo/enterprise#62367
Original PR description
This commit aims to convert QUnit tests which rely on `mail/test_utils` to hoot. Part of task-3818666 Forward-Port-Of: odoo/enterprise#62367
Child asset has an `asset_lifetime_days` that depends on it's parent's. So, we need to add `recursive=True` to the field. Forward-Port-Of: odoo/enterprise#62351
Original PR description
Child asset has an `asset_lifetime_days` that depends on it's parent's. So, we need to add `recursive=True` to the field. Forward-Port-Of: odoo/enterprise#62351
Before this PR: - IGST is not applicable for intra-state transactions. - Transactions within special economic zones were not properly handled. After this PR: - Enhanced condition to exclude IGST for intra-state transactions. - Improved handling of transactions within special economic zones for accurate GST treatment. Forward-Port-Of: odoo/enterprise#62021
Original PR description
Before this PR: - IGST is not applicable for intra-state transactions. - Transactions within special economic zones were not properly handled. After this PR: - Enhanced condition to exclude IGST for intra-state transactions. - Improved handling of transactions within special economic zones for accurate GST treatment. Forward-Port-Of: odoo/enterprise#62021
This fix corrects where the automatic deletion rule was applied in the HR contract documents module. The ondelete setting was incorrectly placed in one model and has been moved to the correct location, ensuring data is properly managed when records are deleted.
Original PR description
The ondelete was not in the right model. task-3928257
A faulty test in the Google Calendar module has been removed. This test was incorrectly assuming that event organizers would always appear in a specific position in the attendees list, which caused unreliable test results. Removing this test improves the reliability of the Google Calendar synchronization testing.
Original PR description
This commit removes the dead and undeterministic test added by mistake after an unsucessful merge. This test assumes that the organizer of the event might always be on the position 0 of the attendees list but it is not always the case. Issue from: odoo/odoo#164408 Forward-Port-Of: odoo/odoo#165346
This update fixes an issue where automated tests for the Spanish EDI TBAI module were being skipped during testing. The tests were missing a required configuration tag that tells the system to run them. This ensures the module's functionality is properly validated.
Original PR description
The class doesn't have the post_install tag, and so, is skipped. runbot-25903 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#165240