Odoo Changelog — 2024-12-20

Friday, December 20, 2024

26 changes · saas-17.4

Bug Fixes(4)

Resolved issues and error corrections

[FIX] hr: prevent error when computing avatar_128 for resource

PR #191170

Coresaas-17.4hr

This fix prevents an error when the system tries to load an avatar for a resource that is not linked to an employee. It improves reliability in HR workflows by safely leaving the avatar empty instead of causing a crash.

Original PR description

Currently, a traceback is occurring when the `_compute_avatar_128` 
method triggers with resource having no employee.

Error:- 
```
IndexError: tuple index out of range
```

Here in resource employee_id is not a required field. 
So when the resource doesn't have any employee_id, it leads to the above traceback.

In the if condition we already checked the presence of employee, but the issue is 
occurring from the else condition.
https://github.com/odoo/odoo/blob/fb4d758ed78211a61ea797759e1fb3b02b51be60/addons/hr/models/resource.py#L30-L34

We can resolve this issue by making the avatar_128 field value False, if there is no employee_id.

sentry-6134007941

#avatar#resource#employee#stability

[DRAFT][FIX] hr_timesheet: resolve error when portal user click time spent on sub-tasks

PR #75395

Enterprisesaas-17.4timesheet_grid

Portal users no longer encounter an error when opening the time spent on sub-tasks from a mobile device. The change avoids showing a view that is not available in the portal experience, keeping shared project access usable without adding unnecessary complexity.

Original PR description

- saas-17.4 ### Steps to reproduce: - Install the hr_timesheet module. - Create a portal user. - Create a project, task, and sub-task (with timesheet entries). - Share the project with the portal…

- saas-17.4

### Steps to reproduce:
- Install the hr_timesheet module.
- Create a portal user.
- Create a project, task, and sub-task (with timesheet entries).
- Share the project with the portal user.
- Log in as the portal user (mobile device).
- Open the shared project and navigate to the task.
- Click on the Time Spent on Sub-tasks action.

### Issue:
Traceback error

### Cause:
On mobile, it attempts to load the grid view.
Technically:
The grid view exists in session.view_info, but the grid view's file is not available in project.webclient, which is used to display view to portal users.

### Solution:
To display the grid view to portal users, we can include the grid view files in project.webclient. However, this would also require adding dependencies for JS classes, widgets, and providing access permissions for certain modules like resource calendar, timer, and public employee, making it a more costly solution.

To maintain similar behavior, we can opt to remove the grid view from the action specifically for portal users.

task-4369891

[FIX] pos_blackbox_be: unnecessary blackbox call

PR #75981

Enterprisesaas-17.4pos_blackbox_be

Belgian Point of Sale syncing now avoids contacting the blackbox device when an order has no line changes. This reduces needless processing during synchronization while preserving the required checks when order contents actually change.

Original PR description

We remove unnecessary blackbox call in preSyncAllOrders method. Before we were calling the blackbox every time we were syncing orders with the server, even if it was not useful when there was no line change. Now we only call the blackbox when there is at least a line change.

#point-of-sale#belgium#sync#performance

[FIX] l10n_co_dian,l10n_co_edi: add missing translations

PR #75913

Enterprisesaas-17.4

Original PR description

**Issue:**
Some terms don't appear in Spanish in DIAN invoice PDF files.
![image](https://github.com/user-attachments/assets/3a76f66c-9a53-473c-a5cd-68ef1e4c3980)

**Expected:**
These terms should be translated in Spanish.

**Steps to reproduce:**
- Activate Accounting app in a Colombian database;
- Configure the DIAN credentials, client, journal and product (https://www.odoo.com/documentation/18.0/applications/finance/fiscal_localizations/colombia.html?highlight=dian);
- Create a customer invoice;
- Send and print with (only) DIAN checkbox checked;
- Open the PDF for the invoice.

**Cause:**
No translations (or erroneous ones) have been found for these keys.

**Fix:**
Add Spanish translations for these terms.
<img width="642" alt="Capture d’écran 2024-12-09 à 11 52 44" src="https://github.com/user-attachments/assets/77f548f4-f87e-4823-9fdc-5b54d0db2a1f">
Backport of Odoo 18 fix: https://github.com/odoo/enterprise/commit/99f332e3211cf17eb7bf0a970544dd0e0e07a9f8

**Testing:**
Testing needs actual credentials and complete configuration (including setting the client's language to LATAM Spanish)
![Capture d’écran 2024-12-09 à 11 34 54](https://github.com/user-attachments/assets/3bbea0c4-2696-4fb9-88d0-38be9a8cae06)


opw-4343184

Other(22)

Miscellaneous changes

[FW][FIX] pos_viva_wallet: add webhook fallback

PR #191241

Coresaas-17.4pos_viva_wallet

The Viva Wallet integration relies on Odoo being called by Viva Wallet via a webhook. There have been many issues of this webhook never being called, and therefore payments getting stuck in Odoo. In this commit, we add a polling fallback. Every 5 seconds we query the Viva Wallet session, and if we get a response (success/failure) then we update the payment state in the PoS. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odo

Original PR description

The Viva Wallet integration relies on Odoo
being called by Viva Wallet via a webhook.
There have been many issues of this webhook
never being called, and therefore payments
getting stuck in Odoo.

In this commit, we add a polling fallback.
Every 5 seconds we query the Viva Wallet
session, and if we get a response (success/failure) then we update the payment state in the PoS.

---
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: odoo/odoo#191156
Forward-Port-Of: odoo/odoo#190213

Currently, a traceback is occurring when the `_compute_avatar_128` method triggers with resource having no employee. Error:- ``` IndexError: tuple index out of range ``` Here in resource employee_id is not a required field. So when the resource doesn't have any employee_id, it leads to the above traceback. In the if condition we already checked the presence of employee, but the issue is occurring from the else condition. https://github.com/odoo/odoo/blob/fb4d758ed78211a61ea797759e1fb3b02b51be60/addons/hr/models/resource.py#L30-L34 We can resolve this issue by making the avatar_128 field value False, if there is no employee_id. sentry-6134007941

- saas-17.4 ### Steps to reproduce: - Install the hr_timesheet module. - Create a portal user. - Create a project, task, and sub-task (with timesheet entries). - Share the project with the portal user. - Log in as the portal user (mobile device). - Open the shared project and navigate to the task. - Click on the Time Spent on Sub-tasks action. ### Issue: Traceback error ### Cause: On mobile, it attempts to load the grid view. Technically: The grid view exists in session.view_info, but the grid view's file is not available in project.webclient, which is used to display view to portal users. ### Solution: To display the grid view to portal users, we can include the grid view files in project.webclient. However, this would also require adding dependencies for JS classes, widgets, and providing access permissions for certain modules like resource calendar, timer, and public employee, making it a more costly solution. To maintain similar behavior, we can opt to remove the grid view from the action specifically for portal users. task-4369891

We remove unnecessary blackbox call in preSyncAllOrders method. Before we were calling the blackbox every time we were syncing orders with the server, even if it was not useful when there was no line change. Now we only call the blackbox when there is at least a line change.

**Issue:** Some terms don't appear in Spanish in DIAN invoice PDF files. ![image](https://github.com/user-attachments/assets/3a76f66c-9a53-473c-a5cd-68ef1e4c3980) **Expected:** These terms should be translated in Spanish. **Steps to reproduce:** - Activate Accounting app in a Colombian database; - Configure the DIAN credentials, client, journal and product (https://www.odoo.com/documentation/18.0/applications/finance/fiscal_localizations/colombia.html?highlight=dian); - Create a customer invoice; - Send and print with (only) DIAN checkbox checked; - Open the PDF for the invoice. **Cause:** No translations (or erroneous ones) have been found for these keys. **Fix:** Add Spanish translations for these terms. <img width="642" alt="Capture d’écran 2024-12-09 à 11 52 44" src="https://github.com/user-attachments/assets/77f548f4-f87e-4823-9fdc-5b54d0db2a1f"> Backport of Odoo 18 fix: https://github.com/odoo/enterprise/commit/99f332e3211cf17eb7bf0a970544dd0e0e07a9f8 **Testing:** Testing needs actual credentials and complete configuration (including setting the client's language to LATAM Spanish) ![Capture d’écran 2024-12-09 à 11 34 54](https://github.com/user-attachments/assets/3bbea0c4-2696-4fb9-88d0-38be9a8cae06) opw-4343184

[FIX] l10n_fr: add tax move lines in tax report In the l10n_fr tax report, the wrong `account_move_line` entries are used in the journal item list view of the tax report in section `TD-VAT Due`. As a result, the balance debit/credit displayed under the `TD-VAT Due` is not the same as the balance in the journal items list view. - Create a customer invoice with a price 1000 and a tax 20% Goods. - Create a vendor bill with a price 100 and a tax 20% Goods. - Open the tax report, under the TD - VAT Due, the balance is 180. - Click on the amount in TD - VAT Due, the debit is 20 but the credit is 1000 (should be 200). The formula for `TD-VAT Due` is defined as `box_16.balance - box_23.balance`. - `box_16` is the sum of box_xx.balance which are defined by `account_tag`. - The balances of these boxes were computed using the base `account_tag`, which calculates the correct amount but causes `_expand_aggregations` in `account_report_expression` to generate a list view based on base move lines instead of the corresponding tax lines. This can be seen the label field of the lines, which are not the tax label. `box_16` needs to be computed using `box_xx_taxe` instead of `box_xx_base` to ensure the correct tax lines are included. For box 08, 09, 9B, 10, 11, T1, T2, T3, T4, T5, T6, P1, P2, I1, I2, I3, I4, I5, I6 we replace balance to balance_from_tags (which uses box_xx_taxe) For box 13, 14, 15, T7, 5B we keep balance (which already uses box_xx_taxe) opw-4335396 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#188640

Bug Fixes(4)

[FIX] hr: prevent error when computing avatar_128 for resource

[DRAFT][FIX] hr_timesheet: resolve error when portal user click time spent on sub-tasks

[FIX] pos_blackbox_be: unnecessary blackbox call

[FIX] l10n_co_dian,l10n_co_edi: add missing translations

Other(22)

[FW][FIX] pos_viva_wallet: add webhook fallback

Bug Fixes(4)

[FIX] hr: prevent error when computing avatar_128 for resource

[DRAFT][FIX] hr_timesheet: resolve error when portal user click time spent on sub-tasks

[FIX] pos_blackbox_be: unnecessary blackbox call

[FIX] l10n_co_dian,l10n_co_edi: add missing translations

Other(22)

[FW][FIX] pos_viva_wallet: add webhook fallback

[FW][FIX] web: MultiRecordSelector should pass domain to SelectCreateDialog

[FW][IMP] l10n_fr: Update COA and account groups

[FW][FIX] account: allows attaching images to draft account move chatter

[FW][FIX] sale_timesheet: protect timesheet product company id

[FW][FIX] mail: hide New button when 'create' is false in the action context

[FW][17.0][FIX] account: Apply sudo to avoid an access error when importing records (e.g. partners).

[FW][FIX] point_of_sale: not show discount on price manually set

[FW][FIX] account_edi_ubl_cii: remove ZIP constraint on facturx

[FW][FIX] im_livechat: assign operator according to lang

[FW][FIX] l10n_ch: force report paper format with snailmail

[FW][FIX] hw_drivers: always use pi as home directory

[FW][ADD] l10n_bh: Add basic package

[FW][FIX] l10n_fr: add tax move lines in tax report

[FW][FIX] iap: handle timeout error while calling JSON-RPC endpoint

[FW][FIX] analytic: enhance message for base analytic plan

[FW][FIX] l10n_ph_check_printing: new check format

[FW][IMP] l10n_fr_reports: Adapt IMB report to the new COA

[FW][FIX] industry_fsm_sale: display SO and SOL button in field service

[FW][FIX] l10n_es_reports: refunds missing in mod349

[FW][FIX] iot: calipers were missing read_once action

[FW][FIX] account_online_synchronization: currency_id not serializable