Tuesday, December 17, 2024
13 changes · 17.0
Resolved issues and error corrections
A new automated test was added to help ensure down payment invoicing works correctly in Point of Sale sales flows. This reduces the risk of regressions in a customer-facing payment and invoicing scenario.
Original PR description
This test is linked to this commit: https://github.com/odoo/odoo/commit/42eb159b6adb668d26cfb31aec2a59a1fd4d3a00 Related PR: https://github.com/odoo/odoo/pull/188204 opw-4352799
This fix updates the Estonian localization upgrade process so an existing tax report is renamed rather than deleted and recreated. It helps preserve users' current report setup during module upgrades and reduces the risk of disruption.
Original PR description
In odoo/odoo#180856, a new version of the Estonian tax report is added with the xmlid tax_report, in order to not overwrite the tax report for users in stable who didn't update their modules. When upgrading the module, to avoid dropping and recreating the report, we add this upgrade script to rename the xmlid of the tax report. no-task --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This update backports fixes to automated tests for the import feature. It helps keep quality checks reliable without changing how users import records in Odoo.
Original PR description
backport test fixes from https://github.com/odoo/odoo/pull/190453 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
The inventory error message now shows the relevant lot name when a lot is linked to another product, instead of incorrectly showing a location name. This makes the message clearer for users and helps them identify the right stock lot when resolving the issue.
Original PR description
Description of the issue/feature this PR addresses: - On the error message when we check the lot and it's linked to another product it displays the location_id instead of the lot_id which isn't correct. Current behavior before PR: - Incorrect displayed value in the error message. Desired behavior after PR is merged: - Replace the location name with the lot name. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This update enhances the planning module's search capabilities by adding a filter for 'Properties'. This allows users to more effectively narrow down their planning searches within the planning.slot and planning.analysis.report modules, improving efficiency and data access.
Original PR description
Before this commit there was no groupby Properties. In this commit we add a filter for groupby properties in planning module. The search view is being used by two modules`planning.slot` and `planning.analysis.report` thus solution resulted in adding the filter in `planning_view_search` as it is a primary inherited view of search view base in planning.slot. task-3801551
A recent update to Luxembourg tax rates in the Odoo Enterprise system caused some tests to fail. This commit adjusts the tests to reflect the new rates, ensuring accurate reporting and financial calculations. This update is part of ongoing improvements to our tax reporting functionality.
Original PR description
Description of the issue this commit addresses: The community commit linked to this one changed the rates of luxembourgian taxes used in the oss fiscal positions. Because of that, some tests are failing. --- Desired behavior after this commit is merged: Said tests are not failing anymore. --- Community PR: https://github.com/odoo/odoo/pull/189444 task-[4369658](https://www.odoo.com/odoo/project/967/tasks/4369658) Forward-Port-Of: odoo/enterprise#75650
This update resolves a visual issue on the Helpdesk dashboard where a sample ribbon was obscured by the main banner. The fix adjusts the loading order to ensure the ribbon displays correctly, providing a cleaner and more functional user experience. This improves the overall appearance of the dashboard.
Original PR description
Steps to Reproduce: - Install the Helpdesk app without demo data. - Navigate to the dashboard. - Observe the `SAMPLE` ribbon in the top-right corner. Cause: - The issue occurs because the SAMPLE ribbon for "Fake Data for Demo" is rendered before the helpdesk banner. Due to this loading order, the banner displays on top of the ribbon, obscuring it. Solution: - By adjusting the loading order so the ribbon loads after the helpdesk banner, we ensure the ribbon displays clearly above the banner without additional adjustments. task: 4209920
This update resolves an issue where comparison cells in the consolidation report were incorrectly marked as auditable, potentially causing confusion. The fix sets the 'auditable' flag to false specifically for the ComparisonBuilder, ensuring compliance with audit rules. As the consolidation module is being decommissioned, this is a low-impact change.
Original PR description
Steps to reproduce: [l10n_be_us_consolidation_demo] - go to consolidation > Test Chart > Consolidated Balance > Comparison > Test Chart - click on a cell to proceed to an audit Issue: Stacktrace…
Steps to reproduce: [l10n_be_us_consolidation_demo] - go to consolidation > Test Chart > Consolidated Balance > Comparison > Test Chart - click on a cell to proceed to an audit Issue: Stacktrace Cause: Journal id is not defined. When there are multiple periods we use the `ComparisonBuilder` https://github.com/odoo/enterprise/blob/4367042c8c7f2190d23019e591c332045b7d4993/account_consolidation/report/trial_balance.py#L139 Which, contrarily to the DefautBuilder, journals is not defined. In the AbstractBuilder (inherited in the two builders), `auditable` is set to True without consideration of the builder's definition https://github.com/odoo/enterprise/blob/3e5ac3554f4fded90fd37c56c7c60723aec59404/account_consolidation/report/builder/abstract.py#L278 On the front side, whenver we click on a auditable cell, we call the action_open_audit but journal_id is defined https://github.com/odoo/enterprise/blob/2d774344109c2f9b55d9ea6160ae99aeca4f0420/account_consolidation/static/src/components/consolidation_report/line_cell/line_cell.js#L19-L38 -> error Solution: Asusming that `auditable` is the rule and the case for the comparison builder is the exception, auditable is set to False only in the ComparisonBuilder Since `consolidation` is meant to be decommissioned, the least impacting fix is being applied. Note: in 16.0, the cells are not auditable when using comparison periods. opw-4265031
This update fixes a display issue where the 'remaining' value on project budgets was incorrectly shown as a surplus. Now, it accurately reflects the difference between the budget's forecast and actual spending. This ensures project managers have a clear and correct view of budget status.
Original PR description
The "remaining" value of a budget should be the difference between the amount spent and the amount forecast, not the other way around. Task-4331872
This update resolves a testing issue within the l10n_pe_edi module related to handling 'NOT_EU_VAT' scenarios in Peru. The fix ensures accurate test results, improving the reliability of the module's functionality for Peruvian businesses. This addresses a technical detail that contributes to the overall stability of the Odoo Enterprise platform.
Original PR description
Community: https://github.com/odoo/odoo/pull/189421 task-4374458
This update resolves an issue where invoices couldn't be confirmed when a country lacked VAT information. The fix ensures that missing VAT country data is handled gracefully, treating it as '0' to allow invoice creation and confirmation. This prevents errors related to incorrect XML formatting.
Original PR description
This fix is to avoid the an error that occurs when a country does not have the VAT information set. Before this fix, when that situation happened the method get_vat_country returned False as those…
This fix is to avoid the an error that occurs when a country does not have the VAT information set. Before this fix, when that situation happened the method get_vat_country returned False as those fields values and therefore the XML could not be processed since it was expected an integer and it got a boolean. After this fix, if a country does not have VAT information, the field will be process as 0 and the XML will be created, so the invoice will be confirmed. We already had this approach in the past, inside the method wsfex_get_cae_request, but it was lost in a refactor made on the module. So what I did was just to recover that approach. https://github.com/odoo/enterprise/commit/33ea78a5bb34e657c16bcbb704eeb71897ae642d#diff-2459e118c605cf039bb94c62561285ad753b6a27c571f10a25547ee9b01aa318L741. Steps to reproduce the error: - Create a test contact based on Uruguay. - Go to Contacts/Configuration/Localization/Countries and delete all VAT information from Uruguay (Natural Person VAT, Legal Entity VAT, Other VAT) . - Create an invoice for the test client created before, add a product and all needed fields and try to confirm the invoice. - An error will pop up, saying that the server was unable to read request because of an error in the XML document. Forward-Port-Of: odoo/enterprise#75501
This update corrects a technical issue preventing the l10n_ee_rounding module from automatically installing when the related l10n_ee_reports module is installed. Removing a specific country designation from the module's configuration ensures proper dependency management and simplifies the installation process.
Original PR description
In odoo/enterprise#70371, the manifest file for l10n_ee_rounding specifies the country 'ee'. However, since the module depends on l10n_ee_reports, adding 'countries' interferes with auto_install, leading to the module l10n_ee_rounding not being auto installed when l10n_ee_reports is installed. This commit removes the country specification from the manifest. no-task
This update corrects a visual issue in the Field Service Gantt View within Dark Mode. Previously, half-day off times displayed with a white background, now they match the background color of the full working day. This ensures a consistent and professional appearance across all views.
Original PR description
**Issue:** In dark mode, for current day, half days off are displayed with white background in the Field Service Gantt View for the working part.…
**Issue:** In dark mode, for current day, half days off are displayed with white background in the Field Service Gantt View for the working part.  **Expected:** In dark mode, worked half days should have same background as entire worked day for current date. **Steps to reproduce:** - Activate Time Off and Field Service apps and activate Dark Mode; - In `Time Off / Management / Time Off` create a new half day off time for any employee under contract (e.g. Mitchell Admin) for the current day; - Approve and validate; - Move to `Field Service / Planning / By User`; - Remove all search filters and display using Gantt View by week or by month; - *(opt) Create a new task for the chose user if does not appear*; - Look at the current day cell of the user. **Cause:** No dark theme background color is set to override the light theme. **Fix:** Setup background color for dark theme by overriding hard coded light theme color: https://github.com/odoo/enterprise/blob/7778a8e45502da89e27910efab401d3556d8c7f7/web_gantt/static/src/gantt_view.scss#L21   opw-4368966