Monday, March 11, 2024
8 changes · 17.0
Enhancements to existing features
This update improves the clarity and wording of the holiday attestation document in the Belgian payroll module. The changes make the termination holidays report easier to understand for employees and HR staff by refining the language used in the document.
Original PR description
task-3610704
Resolved issues and error corrections
Miscellaneous changes
Before this commit, a field would first check if the field was dirty to reach the model for further validation and update. However, if the following steps are followed: - type a valid value (e.g. 10 in an integer field) - make it invalid (e.g. 10*) - make it valid again (e.g. 10) The whole onChange code from the input fied hook would be bypassed as the field would no be dirty anymore (because of the way it's handle in the implementation). If it's bypassed, the field cannot know it's
Original PR description
Before this commit, a field would first check if the field was dirty to reach the model for further validation and update. However, if the following steps are followed: - type a valid value (e.g. 10 in an integer field) - make it invalid (e.g. 10*) - make it valid again (e.g. 10) The whole onChange code from the input fied hook would be bypassed as the field would no be dirty anymore (because of the way it's handle in the implementation). If it's bypassed, the field cannot know it's no longer invalid. So the field would be underlined red and the record would't be savable. This commit being a fix in stable, we only make sure we always apply the onChange logic of the hook by removing a condition. opw 3680495 Forward-Port-Of: odoo/odoo#151446 Forward-Port-Of: odoo/odoo#150192
This update standardizes the spacing of alert messages displayed above forms across Odoo, ensuring a consistent visual appearance. Following the recent Milk redesign, all alerts now use the same bottom margin as the status bar, removing previous inconsistencies in how alerts were displayed throughout the system.
Original PR description
Since Milk redesign the margin bottom on the alerts displayed above the sheet in a form view should be mb-2 (equivalent of the margin-bottom of the statusbar). Some alerts across Odoo were still inconsistent. This PR adapts the alerts from enterprise to remove the margin customizations on the impacted alerts. Letting the CSS variable introduced in the community counterpart PR handling the margin. task-3577058 Community PR: https://github.com/odoo/odoo/pull/146196 Forward-Port-Of: odoo/enterprise#58168 Forward-Port-Of: odoo/enterprise#52726
This update corrects an incorrect Serbian translation for the 10% VAT tax rate in Odoo's Serbian localization module. The tax group label is now properly translated to "Opšta stopa PDV-a 10%" instead of the previous invalid translation, ensuring accurate tax documentation and compliance for Serbian users.
Original PR description
Description of the issue/feature this PR addresses: This PR fixes translation of the tax group `Specific VAT rate 10%` Current behavior before PR: When using Serbian language tax group `Specific VAT rate 10%` is translated as `ZALIHE MATERIJALA` which is invalid. Desired behavior after PR is merged: When using Serbian language tax group `Specific VAT rate 10%` will be translated as `Opšta stopa PDV-a 10%` which is correct. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This fix resolves a validation warning that appears when modules use the post_load hook in their manifest file. The post_load default value is now aligned with other initialization hooks, eliminating false warnings during module testing and ensuring consistent behavior across Odoo versions.
Original PR description
-Step to reproduce: add a post_load method in the init of any module, specify in the manifest like : 'post_load': 'post_load'. Run test_manifests of the test_lint module and we will get warning 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 Forward-Port-Of: odoo/odoo#156347 Forward-Port-Of: odoo/odoo#154316
This update ensures consistent spacing for alert messages displayed above form sections across Odoo. The change introduces a unified margin standard (matching the status bar spacing) and removes conflicting spacing rules that were causing visual inconsistencies. This improves the overall appearance and user experience when alerts are shown in forms.
Original PR description
Since Milk redesign the margin bottom on the alerts displayed above the sheet in a form view should be mb-2 (equivalent of the margin-bottom of the statusbar). Some alerts across Odoo were still inconsistent. This PR introduces a custom property `--alert-margin-bottom` which allows the customization of the alert's margin bottom depending on its context. This ensure that an alert rendered in the form view above the form_sheet will receive the appropriate margin as long as there is no utility class or override on the alert with this property, which is why this PR removes the mb-x classes or inner styling on the impacted alerts. task-3577058 Enterprise PR: https://github.com/odoo/enterprise/pull/52726 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#156717 Forward-Port-Of: odoo/odoo#146196
Before this commit, defining an always true invisible attrs in a list view crashed, e.g. ```xml <button type="object" name="do_it" attrs="{'invisible': true}"/> ``` For some reasons, we typically don't do that. However, such a usecase has been introduced in [1], which thus completely broke the view (go to the TimeOff app and switch to list view). In form view, this works fine though, so we decided to make it work in list as well. [1] d1f23f32af3f299133c8c3f8931754582e004c8a Iss
Original PR description
Before this commit, defining an always true invisible attrs in a list view crashed, e.g.
```xml
<button type="object" name="do_it" attrs="{'invisible': true}"/>
```
For some reasons, we typically don't do that. However, such a usecase has been introduced in [1], which thus completely broke the view (go to the TimeOff app and switch to list view).
In form view, this works fine though, so we decided to make it work in list as well.
[1] d1f23f32af3f299133c8c3f8931754582e004c8a
Issue spotted by the nightly click all
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
Forward-Port-Of: odoo/odoo#157193The tooltip of leave allocation is changed to be more informative task-3762965 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#155350
Original PR description
The tooltip of leave allocation is changed to be more informative task-3762965 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#155350