Wednesday, November 20, 2024
22 changes · 17.0
New functionality added to Odoo
This update adds the ability to include additional data columns in the account partner ledger report queries. Specifically, a new function allows for the addition of the 'date' field, providing more detailed information within the report. This improves the reporting capabilities for financial analysis.
Original PR description
### After this PR
You can add new column to the account partner ledger
Example:
```
def _get_additional_column_aml_query_values(self):
res = super()._get_additional_column_aml_query_values()
res +="account_move_line.date as date,"
return res
```Resolved issues and error corrections
Users can now extend text selections with Shift + Arrow keys even when the selection crosses protected editor elements such as banners or clipboards. This makes editing website content smoother and avoids interruptions when selecting mixed editable and non-editable areas.
Original PR description
Description of the issue this PR addresses: Previously, selection using Shift + Arrow keys was not possible across contenteditable=false elements such as banners and clipboards. This commit makes sure now selection be done across these elements using Shift + Arrow keys. task-3502724 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Documentation and clarification updates
This pull request adds an individual contributor license agreement record for ShalithaWanasinghe. It is an administrative legal update that confirms contribution eligibility and does not change any Odoo product functionality.
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
Miscellaneous changes
Since October 2024, the Facebook mobile site (m.facebook.com) uses an 'X-Frame-Options' header set to 'deny', preventing iframe embedding across different domains. This header enforces a security policy restricting cross-origin access to the content. Meta has not provided any communication regarding this change. Until it is resolved, we have no choice but to hide the iframe on mobile to prevent users from seeing a "unavailable content" icon. Steps to reproduce: - In website edit mode. -
Original PR description
Since October 2024, the Facebook mobile site (m.facebook.com) uses an 'X-Frame-Options' header set to 'deny', preventing iframe embedding across different domains. This header enforces a security policy restricting cross-origin access to the content. Meta has not provided any communication regarding this change. Until it is resolved, we have no choice but to hide the iframe on mobile to prevent users from seeing a "unavailable content" icon. Steps to reproduce: - In website edit mode. - Drag and drop the "Facebook" snippet into the footer. - Save the page. - Open the dev tools and enable the mobile preview. - Bug: The "Facebook" snippet crashes. More info about the issue: https://developers.facebook.com/community/threads/550478830783215/ opw-4302563 opw-4309180 opw-4222359 opw-4277045 Forward-Port-Of: odoo/odoo#186813
Fixed an issue where inserting a separator in an empty report section could place it at the beginning instead of where the cursor was positioned. This makes editing internal reports in Studio more predictable and reduces formatting corrections for users.
Original PR description
Issue: ====== sperator isn't inserted in the correct place. Steps to reproduce the issue: ============================= - Go to studio - Create a new internal report - Put the cursor in the div (second line) - Add some content over some lines - Insert seperator at the end of the lines - It gets inserted at the start Origin of the issue: ==================== Since we are writing inside the div directly, inserting the hr will look for the closest block and insert it before it which is the div. Because, new line inside a div is just a br and doesn't split the div. Solution: ========= Add a p element inside an empty div element when we put the selection there. task-4240730
A self-ordering test was moved to the correct point-of-sale preparation display area so it only runs when the needed feature is available. This prevents false test failures and helps keep release validation reliable without changing user-facing behavior.
Original PR description
The test `test_self_order_preparation_disabling_preparation_display` in `test_self_order_common` does not work when the `pos_preparation_display` module is not installed. This is because there is no warning message when `pos_preparation_display` is absent. This commit changes the test location to the module `pos_self_order_preparation_display`.
Call invitation profile pictures now keep a square shape regardless of the user's uploaded image. This prevents distorted or uneven invitation cards, making calls in Discuss look more consistent and polished.
Original PR description
Before this commit, the profile picture of a user would affect the dimensions of the image element. This commit fixes the issue by enforcing an aspect ratio of 1.
The form view’s save, cancel, and settings controls now line up more cleanly with the breadcrumb area. This fixes a small visual inconsistency, making the interface feel more polished and easier to scan.
Original PR description
Prior to this commit, the `FormStatusIndicator` buttons were not properly aligned with the content of the breadcrumb in the control panel. This commit adjusts the alignment of `FormStatusIndicator`…
Prior to this commit, the `FormStatusIndicator` buttons were not properly aligned with the content of the breadcrumb in the control panel. This commit adjusts the alignment of `FormStatusIndicator` correctly according to the content of the breadcrumb. Also, to align the cog menu perfectly with the "save" and "cancel" buttons, we've removed the border around the latter two. task-3874495 | Before | After | |--------|--------| |  |  | |  |  | --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
OCR status check issues are now recorded as warnings instead of errors when they do not require immediate action. This reduces unnecessary alert noise while still keeping the information available for follow-up if needed.
Original PR description
``_try_to_check_ocr_status`` method attempts to check the OCR status for a record, If an exception occurs during this process, It will log an error. Error: ``` Couldn't check OCR status of hr.expense with id 1: 'currency_id' ``` This commit changes the log level from an error to a warning. This helps reduce noise in the logs by preventing excessive error messages. sentry-4597345306
Expense receipt scanning no longer fails when a receipt uses an inactive or unavailable currency, or when some OCR totals are missing. This makes automated expense creation more reliable in companies with limited active currencies and reduces test or processing interruptions.
Original PR description
When extracting data from a ticket expressed in a currency that's inactive in the company, the code crashed with a KeyError, because the 'currency_id' was not in the vals dictionary. This crashed some tests in hr_expense_extract when they were run in a db with only USD as active company.
This update resolves an issue where duplicate keyboard shortcuts were preventing users from accessing key actions within the Subscription module. The fix ensures that different actions have unique shortcuts, improving usability and preventing workflow disruptions. This change focuses on the 'Upsell' and 'Close' buttons.
Original PR description
**Issue:** Two different actions have the same shortcut making one action impossible to reach. **Expected:** Different actions should have different keyboard shortcuts. **Steps to reproduce:** - Activate Subscription app; - Open an existing subscription; - Press `ALT` (`CTRL` on MacOS) to display all shortcuts and look at the `UPSELL` and `CLOSE` buttons. **Cause:** The same `data-hotkey` has been used for both buttons. **Fix:** Change the `UPSELL` button shortcut to `ALT+E` (`CTRL+E` on MacOS) as for Odoo 17. opw-4306179 Forward-Port-Of: odoo/enterprise#73548
This update addresses a test failure within the Web Studio module, which is a key component of Odoo's website customization tools. The fix was triggered by a recent change from the Odoo community, ensuring continued stability and functionality for users building and managing their online storefronts.
Original PR description
This is a fix for failing test after the change introduced in the corrosponding community [PR] task-4240730 [PR]: https://github.com/odoo/odoo/pull/183129
This update resolves an issue where a key test failed when the 'pos_preparation_display' module wasn't present. The test has been moved to the correct module, ensuring consistent and reliable testing of the self-order POS functionality. This improves the overall stability of the system.
Original PR description
The test `test_self_order_preparation_disabling_preparation_display` in `test_self_order_common` does not work when the `pos_preparation_display` module is not installed. This is because there is no warning message when `pos_preparation_display` is absent. This commit changes the test location to the module `pos_self_order_preparation_display`.
This update resolves a technical issue that could cause errors in the generation of financial reports for Belgium. Specifically, it avoids division-by-zero errors when dealing with entries that have zero balances and currency amounts. The fix ensures reports are generated accurately and reliably.
Original PR description
## [FIX] l10n_be_reports: 281.50 and exchange diff The aim of this commit is to prevent a Zero devision error by excluding irrelevant 'zeroed' exchange difference. Context: Some misc entry with a 0 balance and some amount_currency might be present and taken into account for the computation. Such moves have an `amount_total_signed` equal to 0 leading to the zeroDivisionError issue in the query computation. Before the commit: When having such entry considered by the query, creating the 325.50 form leads to a traceback. After the commit: Those irrelevant moves are ignored and the process carry on smoothly. task-id: None - Issue spotted on next ## [FIX] l10n_be_reports: 281.50, no 0-0-0-0-0 form The aim of this commit is to avoid generating a 281.50 form if there isn't any amounts to fill in the form. task-id: None - Dicussed with TSB Forward-Port-Of: odoo/enterprise#73842
This update fixes a technical error that prevented users from configuring TaxCloud within the invoicing system. The issue stemmed from a problem accessing data related to CartItemsResponse, which has now been resolved. This ensures TaxCloud functionality is reliably available for US customers.
Original PR description
**Steps to reproduce:** - Edit \__init__.py file of account_taxcloud module to allow its installation - Install sale_account_taxcloud and industry_fsm_stock - In Invoicing settings, configure TaxCloud credentials - Configure TaxCloud fiscal position: * Country: United States * Detect Automatically: [enabled] - Go to "Field Service" - Create a new Task: * Title: [anything] * Customer: [a US customer] (e.g. Deco Addict) - Click on "Products" smart button - Add a product - Click on the task name in the breadcrumb to go back to the task **Issue:** A traceback is raised while trying to access CartItemResponse property of the None value of CartItemsResponse. opw-4199902
This update adjusts the color contrast of primary buttons in the Odoo Enterprise application. Previously, the focus state wasn't visually distinct enough, making it harder for users to easily identify which buttons were active. This change ensures better usability and accessibility by providing clear visual feedback when buttons receive focus.
Original PR description
This PR aims to increase the contrast between the default and the `focus` state of our `btn-primary`. | 17.0 | This PR | |--------|--------| | <img width="729" alt="image" src="https://github.com/user-attachments/assets/19c29432-4467-46bf-b240-41017d435c09"> | <img width="717" alt="image" src="https://github.com/user-attachments/assets/1efc15cc-6e10-460b-9040-4088cf3a98e0"> | Prior to this PR, the color was not dark enough to grant enough visual feedback on focus. This can be solve quickly by tweaking the value directly. task-3382308
This pull request adds an individual contributor license agreement record for a contributor. It supports Odoo's legal compliance process and does not change product functionality for users.
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
UserWarning: unsupported operand type(s) for "==": 'res.country()' == '233' --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#187821
Original PR description
UserWarning: unsupported operand type(s) for "==": 'res.country()' == '233' --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#187821
Steps to reproduce: 1. Drag and drop a text snippet. 2. Select background video and insert the video in the mediaDialog. 3. Click on the add button. Issue: A traceback occurs due to the tag name not being found for an element. This issue was introduced in commit [1]. Issue link: https://github.com/odoo/odoo/issues/186874 Solution: This PR resolves the traceback by adding a `parentEl` condition in the replace media on save function. [1] : https://github.com/odoo/odoo/commit/36594d04a
Original PR description
Steps to reproduce: 1. Drag and drop a text snippet. 2. Select background video and insert the video in the mediaDialog. 3. Click on the add button. Issue: A traceback occurs due to the tag name not being found for an element. This issue was introduced in commit [1]. Issue link: https://github.com/odoo/odoo/issues/186874 Solution: This PR resolves the traceback by adding a `parentEl` condition in the replace media on save function. [1] : https://github.com/odoo/odoo/commit/36594d04a8909dd40ab6384f387c372ddae62345 Forward-Port-Of: odoo/odoo#186876
This PR fixes an issue about the dark mode switch overlapping its label. To handle the issue, we just need to apply a width utility class. task-4344243 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#187688
Original PR description
This PR fixes an issue about the dark mode switch overlapping its label. To handle the issue, we just need to apply a width utility class. task-4344243 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#187688
The existing FedEx implementation used the old FedEx SOAP API which is no longer in development. This new version makes use of the current REST APIs available at developer.fedex.com task-3759206 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#186787
Original PR description
The existing FedEx implementation used the old FedEx SOAP API which is no longer in development. This new version makes use of the current REST APIs available at developer.fedex.com task-3759206 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#186787
**Steps to reproduce:** - Create a PDF report for the stock.picking model - Use the template stock.label_transfer_template_view_pdf. - When saving the report, the mentioned error occurred. ```py odoo.addons.base.models.ir_qweb.QWebException: Error while render the template AttributeError: 'stock.picking' object has no attribute 'move_lines' Template: stock.label_transfer_template_view_pdf Path: /t/t/div/t[2]/t[2] Node: <t t-foreach="picking.move_ids" t-as="move"/> ``` - [Ref](https:
Original PR description
**Steps to reproduce:** - Create a PDF report for the stock.picking model - Use the template stock.label_transfer_template_view_pdf. - When saving the report, the mentioned error occurred. ```py…
**Steps to reproduce:** - Create a PDF report for the stock.picking model - Use the template stock.label_transfer_template_view_pdf. - When saving the report, the mentioned error occurred. ```py odoo.addons.base.models.ir_qweb.QWebException: Error while render the template AttributeError: 'stock.picking' object has no attribute 'move_lines' Template: stock.label_transfer_template_view_pdf Path: /t/t/div/t[2]/t[2] Node: <t t-foreach="picking.move_ids" t-as="move"/> ``` - [Ref](https://github.com/odoo/odoo/pull/78732) were move_lines has been renamed to move_ids. - [Ref](https://github.com/odoo/odoo/pull/80434) were product_uom_qty has been renamed to reserved_uom_qty. **Desired behavior after PR is merged:** - Can create a report for stock.picking model  OPW - [4239654](https://www.odoo.com/odoo/project/70/tasks/4239654) --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#184046