Friday, May 3, 2024
11 changes
1 change
Resolved issues and error corrections
A visual flicker issue in the mass mailing template has been resolved. This change ensures the sidebar loads smoothly and correctly, eliminating a distracting visual glitch for users creating and managing email marketing campaigns. This improves the overall user experience.
Original PR description
Issue: ====== mass mailing template flicker while loading. Steps to reproduce the issue: ============================= - Go to email marketing - Create a new mass mailing Origin of the issue: ==================== The sidebar with id `oe_snippets` is being loaded but transformed with translateX to be outside the html_field but there is some space left between the field and the chatter so a part of it will be shown. Solution: ========= Force the display attribue of the sidebar to be `none` at the start and remove it after being completely loaded. task-3792995
1 change
Resolved issues and error corrections
9 changes
Resolved issues and error corrections
This update fixes automated tests in the Appointment module to work with a new way that links are formatted in the system. The tests were adapted to match the updated link implementation, ensuring the appointment booking system continues to function correctly.
Original PR description
Community PR: https://github.com/odoo/odoo/pull/159995 task-3604728 Forward-Port-Of: odoo/enterprise#61558 Forward-Port-Of: odoo/enterprise#60751
This update fixes invalid text escape patterns across multiple Odoo Enterprise modules to avoid warnings in newer Python versions. It helps keep the system compatible with Python 3.12 and future releases without changing business features or workflows.
Original PR description
Since python3.12 invalid escape sequences are deprecated and this will be removed in future versions. This commit fixes all the missing escape automatically using All checks passed!. Some will be escaped, other may be replaced by r string. Another solution would be to hide the deprecation warning but this would maybe need to adapt some of the versions later leading to another forwardport hell, lets fix it one for all once we are at it. Note that ruff check will forbid to introduce new ones in any cases. Manual forward port of #61571
This update corrects a formatting issue in the error message that appears when social media posts fail to publish. The fix ensures that error messages display correctly to users, improving the clarity of post failure notifications.
Original PR description
string format was broken on post fail message, this commit fix it. [Task-3775424](https://www.odoo.com/web#id=3775424&cids=1&menu_id=6478&action=4043&model=project.task&view_type=form) Forward-Port-Of: odoo/enterprise#61857
This update fixes automated tests in the Peppol accounting module by ensuring all external service responses are properly simulated during testing. The tests were failing because they weren't mocking a required participant verification check that occurs when importing supplier documents. This fix ensures tests run reliably without depending on external services.
Original PR description
Responses should be mocked according to requests that are made during tests. The tests use a sample Peppol file, where the supplier is `0198:dk16356706`. When the document is imported and its content is extracted, the corresponding partner is created with these `peppol_eas` and `peppol_endpoint` values. That triggers a participant check but the response is not mocked for this endpoint. This commit adds a mock response for the endpoint that matches the test file. no task, reported by gawa --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
A syntax error in the CRM module was corrected by removing an extra closing curly brace that was not properly matched. This fix ensures the code is syntactically correct and functions as intended.
Original PR description
WHY: There is one extra "}" curly brace which is not needed to close the opening one. I think it is a syntax error. WHAT: I removed the curly brace, it also needs testing to make sure, but it is very obvious to observe. This is my first commit in the code, forgive me for not following the required instructions, but I was happy to commit in the documentation repository and I hope you help me with that in this repository too. Thanks, --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
A test for email template scheduling was failing intermittently due to time changes occurring during test execution. This fix freezes the system clock during the test to ensure consistent, reliable results. The change ensures that email template dynamic date calculations work correctly without being affected by real-world time passing during testing.
Original PR description
A test exists on mail.template to test dynamic evaluation of scheduled_date field, based on datetime. However it currently runs on non-mocked datetime and it sometimes fails due to a minute-switch between test begin and end. In this commit we freeze the time to ensure test is fixed. However we also have to somehow hack "safe_eval.datetime" usage as it is not covered by standard usage of "freeze_time", probably because it is wrapped. Simplest solution is to mock it directly, assuming safe_eval itself is working as intended (breaking safe_eval itself will probably break other tests; purpose of mail.template test is to check its dynamic rendering is effectively called and taken into account when sending emails based on templates). Task-3872732 Runbot-54946 Forward-Port-Of: odoo/odoo#163943
A field label for the "Register of Legal Entities" was not displaying in the Norwegian localization when creating company contacts. This fix restores the proper display of the field name, ensuring users can see what information they're entering when setting up Norwegian business contacts.
Original PR description
Problem: For the Norwegian localization, the field name "Register of Legal Entities (Brønnøysund Register Center)" is not displayed Steps to reproduce: - Install "Contacts" app and "l10n_no" module - Create a new Norwegian contact as company, the field below "Tax ID" has no name Cause: Probably a change in the framework making the behavior different compared to previous versions opw-3863407 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#162836
A test for the website media dialog feature was not running because it was missing the required "test_" prefix in the Python method name. This fix corrects the naming issue and updates the test to work properly with the current website preview system, ensuring the media dialog functionality is properly validated.
Original PR description
The test `website_media_dialog_undraw` (introduced in [this commit]) wasn't launched because the python method was not prefixed by `test_`. [this commit]: https://github.com/odoo/odoo/commit/d6ca59ed9bf6877ee6b1b311223472adfa4db549 Forward-Port-Of: odoo/odoo#163616 Forward-Port-Of: odoo/odoo#163555
A syntax error in the CRM module's team views configuration file has been corrected by removing an extra closing curly brace. This fix ensures the configuration file is properly formatted and functions as intended without any syntax issues.
Original PR description
WHY: There is one extra "}" curly brace which is not needed to close the opening one. I think it is a syntax error. WHAT: I removed the curly brace, it also needs testing to make sure, but it is very obvious to observe. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This update fixes inconsistent test behavior in the Website Sales module by applying proven improvements from newer versions. The change makes automated tests run more reliably and predictably, reducing false failures and improving overall system stability.
Original PR description
Since this issue does not happen in more recent versions, it has probably been fixed by the recent improvements in the tests setups of website_sale. This commits backports those changes to hopefully make the tour more deterministic and solve the issue. runbot issue 47213 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr