Odoo Changelog — 2024-03-29

Friday, March 29, 2024

4 changes · master

Bug Fixes(4)

Resolved issues and error corrections

[FIX] l10n_ch_hr_payroll: inherit correct emp. view

Enterprisemasterl10n_ch_hr_payroll

The Swiss payroll setup now points to the correct employee view after a duplicate payroll section was removed. This prevents related payroll information from being placed incorrectly and helps keep employee payroll screens consistent.

Original PR description

Before this commit, and the commit introduced in odoo/odoo#158508, two group named "payroll_group" were present on the employee view. The group introduced in hr has been removed and the one in payroll kept. This commit changes the xml_id used as inheritance in l10n_ch_hr_payroll to inherit the view defined in payroll rather than the view from hr_contract to allow inserting some content after the group mentionned above.

#switzerland#payroll#employee-view

[FIX] industry_fsm(_report): fix sign modal z-index

PR #55630

Enterprisemasterindustry_fsmindustry_fsm_report

The field service task signing pop-up now appears above the page header as intended. This prevents confusion for workers signing worksheets from the customer portal and removes a visual overlay gap at the top of the page.

Original PR description

Steps:
Install industry_fsm, worksheet and website.
Create a task, set Joel Willis as assignee and add a worksheet.
Log in as Joel Willis, got to that task and click sign.

Issue:
Z-wise, the modal isn't above the header.

Cause:
The modal is at the same level as the sign button, that is in the sidebar, which has `position: sticky;`.
This fixes the `z-index` to the default value (even setting it manually has no impact).
The fact that the modal's z-index is greater than the header's doesn't matter, as its container limits it.

Solution:
Put the modal div in the main content, which will never be sticky.
Also, `mt-5 pt-5` were creating a "non grayed" stripe in the upper part
of the page (where the header is at), so we delete it.

task-3644729

#field-service#portal#signature#worksheet

[FIX] website_helpdesk: rectify irrelevant information in chatter

PR #54611

Enterprisemasterwebsite_helpdesk

Website form submissions to existing helpdesk tickets no longer add confusing HTML tags or irrelevant text in the ticket chatter. This keeps ticket history clearer for support teams reviewing customer requests.

Original PR description

**Steps:**
- Open website and go to contact us
- Using web editor option click on existing form and change its action
- In create task action > select an existing ticket
- Fill and submit the form
- Open Ticket and go to the ticket given in create task action
- Open the ticket created from the website
- Have a glance at the chatter

**Issue:**
- Chatter is showing irrelevant information. (namely html tags)

**Cause:**
- Regrettably, due to last-minute changes preceding the merge, the review process for PR(https://github.com/odoo/enterprise/pull/47278)  was unintentionally skipped. As a result, certain modifications were pushed that do not align with the expected standards.

**Fix:**
- Taking corrective action by adding proper if condition and assigning values correctly which will resolve this issue promptly.

**Task**-3674768

#helpdesk#website forms#chatter

[FIX] l10n_mx_edi_pos: fix runbot error

PR #59549

Enterprisemasterl10n_mx_edi_pos

This fix prevents a system error in Mexican electronic invoicing for Point of Sale orders by checking whether invoice information is available before using it. It helps keep automated validation and related POS invoicing flows stable without changing user-facing behavior.

Original PR description

### Commit 1:

Fix the runbot issue number 60914 by verifying that the field to_invoice exists in the pos.order dict before trying to access it.

Runbot error ID: 60914

#point-of-sale#mexico#invoicing#stability

Before this commit, and the commit introduced in odoo/odoo#158508, two group named "payroll_group" were present on the employee view. The group introduced in hr has been removed and the one in payroll kept. This commit changes the xml_id used as inheritance in l10n_ch_hr_payroll to inherit the view defined in payroll rather than the view from hr_contract to allow inserting some content after the group mentionned above.

Steps: Install industry_fsm, worksheet and website. Create a task, set Joel Willis as assignee and add a worksheet. Log in as Joel Willis, got to that task and click sign. Issue: Z-wise, the modal isn't above the header. Cause: The modal is at the same level as the sign button, that is in the sidebar, which has `position: sticky;`. This fixes the `z-index` to the default value (even setting it manually has no impact). The fact that the modal's z-index is greater than the header's doesn't matter, as its container limits it. Solution: Put the modal div in the main content, which will never be sticky. Also, `mt-5 pt-5` were creating a "non grayed" stripe in the upper part of the page (where the header is at), so we delete it. task-3644729

**Steps:** - Open website and go to contact us - Using web editor option click on existing form and change its action - In create task action > select an existing ticket - Fill and submit the form - Open Ticket and go to the ticket given in create task action - Open the ticket created from the website - Have a glance at the chatter **Issue:** - Chatter is showing irrelevant information. (namely html tags) **Cause:** - Regrettably, due to last-minute changes preceding the merge, the review process for PR(https://github.com/odoo/enterprise/pull/47278) was unintentionally skipped. As a result, certain modifications were pushed that do not align with the expected standards. **Fix:** - Taking corrective action by adding proper if condition and assigning values correctly which will resolve this issue promptly. **Task**-3674768