Tuesday, March 5, 2019
9 changes · master
Enhancements to existing features
Belgian payroll demo data is moved into a dedicated test module instead of being loaded during normal module installation. This prevents installation conflicts with accounting data and avoids crashes when other country modules are automatically installed.
Original PR description
Revert the commit in which we added l10n_be as dependency of l10n_be_hr_payroll Move the demo data into a specific test module, to avoid crashes when installing the module, that create payslips on the Belgian company, the auto install l10n_us that tries to unlink the journals and the accounts. 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
Belgian payroll demo data is now kept in a dedicated test module instead of being loaded with the main payroll module. This reduces installation conflicts with other country accounting setups and helps prevent crashes during module setup.
Original PR description
Revert the commit in which we added l10n_be as dependency of l10n_be_hr_payroll Move the demo data into a specific test module, to avoid crashes when installing the module, that create payslips on the Belgian company, the auto install l10n_us that tries to unlink the journals and the accounts.
The Documents app test coverage was updated to reflect that scheduling an activity now opens directly in an inline form instead of a separate pop-up action. This supports a smoother workflow by keeping users in context while planning follow-up activities.
Original PR description
Task:https://www.odoo.com/web#id=37030&view_type=form&model=project.task&action=327&menu_id=4720 Pad: https://pad.odoo.com/p/r.442184ccb044087f22fd8992d1e52b9c
Resolved issues and error corrections
This fix updates website test code to avoid depending on the web editor base component. It helps keep automated website testing more reliable and easier to maintain without changing business-facing website features.
Original PR description
Related to da1899af48902edec7ac6ed8dd5ae2f8b02ec689 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
This fixes an issue where the bus longpolling service did not properly remove its event handlers when it was shut down. The change helps prevent lingering background handlers that could cause unstable behavior or test failures.
Original PR description
`this._id` is overriden by the crosstab_bus, so the event handlers weren't unbound at destroy. Manual forwardport of odoo/odoo#31578, as we need this for another branch, in which it makes tests fail. 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
This change removes unnecessary internal checks and duplicated logic in inventory and purchase order processing. It helps keep stock validation and purchasing flows simpler and less error-prone without changing day-to-day user workflows.
Original PR description
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
Reordering rules now only show relevant stock locations based on the selected warehouse. This helps users avoid choosing an incorrect location, reducing inventory planning mistakes.
Original PR description
The model reordering rules (stock.warehous.orderpoint) has a field waherouse_id and location_id. This commit will restrict the domain on location_id depending on the warehouse set. The user can now choose only locations child of the warehouse view location or location with no warehouse parent. Task : 1933828 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
This fixes a rare issue where pop-up details could remain visible after moving items in the Gantt view. Popovers are now tied to the relevant Gantt row so they are removed properly when the view updates, keeping scheduling screens cleaner and less confusing.
Original PR description
Before this rev. there were still remaining popovers in some rare cases when moving pills (and thus after update). Now, the popovers are placed inside the gantt row (instead of the <body>) so it is correctly removed. Manual forwardport of 379b6740cd as this is needed for the update jquery branch. Task 1945899
This fix repairs broken automated tests for barcode-based stock workflows so they correctly reflect expected behavior. It helps prevent false failures and missed issues in future updates, improving confidence in the stock barcode module.
Original PR description
These tests were completely broken because: 1. The assertPageSummary function cannot take false as argument, and doing it resulted in a crash that prevented the other assertion from running, thus hiding the following two issues. 2. Some assertion functions that tested the presence of a class that is supposed to hide the element were forcing the element to be present in the dom to begin with, which was wrong according to the qweb template. Not being in the dom is now correctly interpreted as being "not visible". 3. Some assertions in the tests were functionally wrong and had to be fixed according to the specs of the module. 4. The for loop was using an undefined variable as well as passing a dom element to assertLineQty instead of a jQuery element.