Saturday, July 26, 2025
5 changes · saas-18.3
Resolved issues and error corrections
Fixed a display issue where dragged items with certain forced percentage sizing could appear at the wrong size. This improves visual consistency when moving items, including in automation rule kanban views.
Original PR description
This commit fixes an issue in draggable_hook_builder where elements with !important percentage-based width or height styles would be incorrectly sized when dragged. The sizing would be computed relative to the document rather than the style computed by the hook. To address this, the hook now applies max-width and max-height styles explicitly. This helps enforce correct sizing and prevents !important rules from interfering with the visual consistency of dragged elements. The issue was first spotted in the actions kanban view present in the automation rules form view. task-4891808 Forward-Port-Of: odoo/odoo#219246
Users can now upload a non-PDF product document and choose quote PDF visibility without immediately triggering an error. The system skips PDF-specific processing for non-PDF files and relies on the existing save-time validation, making the product document workflow more stable.
Original PR description
Currently an error occurs when user uploads a non-pdf file on product documents. Steps to replicate: - Install `sale_management` and go to products. - Open any product's form view and click on the…
Currently an error occurs when user uploads a non-pdf file on product documents. Steps to replicate: - Install `sale_management` and go to products. - Open any product's form view and click on the `Documents` smart button. - Click new and upload any non-pdf file. - On the field `Sale : Visible at`, select the value `inside quote pdf` and you will get the error. Error: `PdfReadError: EOF marker not found` The error occurs because at the line [1] the code requires a pdf file, and as we have passed a non-pdf file the error occurs. [1] - https://github.com/odoo/odoo/blob/e750244c3125a48e2ca030160b977bb0344609db/addons/sale_pdf_quote_builder/models/sale_pdf_form_field.py#L206 There is already a constraint made for this particular thing [2], but the problem is that the error is due the compute [3] (Because constraints are checked at the time of form saving, and compute runs when a field is changed so even before the constraint is checked the error will be triggered). [2] - https://github.com/odoo/odoo/blob/e750244c3125a48e2ca030160b977bb0344609db/addons/sale_pdf_quote_builder/models/product_document.py#L44-L45 [3] - https://github.com/odoo/odoo/blob/e750244c3125a48e2ca030160b977bb0344609db/addons/sale_pdf_quote_builder/models/product_document.py#L59-L61 This commit resolves this issue by skipping the pdf extraction if the file is not a pdf type, because we already have a constraint [2] that will trigger at save. sentry-6161120972 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#220574 Forward-Port-Of: odoo/odoo#215844
This fix makes the automated check for resizing embedded calendar events in Knowledge more reliable. It prevents occasional false failures caused by the resize control disappearing during the test, helping keep release validation stable without changing user-facing behavior.
Original PR description
This commit fixes an indeterministic error in the knowledge_calendar_command_tour when a calendar view is embedded and an event is resized. The resize handler is only visible when hovering the event, which is required to actually be able to resize it. This commit merges both the "make resizer visible" step and the "resize the item" step into a single one as those actions should be done in one go, instead of separate steps (cf. the hovering state is kind of reset in-between the steps). Note: this matches the behavior of the CalendarView's unit tests helpers (`resizeEventToTime()`). runbot-error-163015 Forward-Port-Of: odoo/enterprise#91012
Printed Swiss payslips now show rates in a cleaner format without unnecessary trailing zeros after the decimal separator. This makes payroll documents easier to read and more professional for employees and payroll teams.
Original PR description
…d payslip In this PR we improve the rate display, which could include a lot of 0's after the coma. Forward-Port-Of: odoo/enterprise#90892
Miscellaneous changes
## Issue When a client has a lot of pricelist rules, the PoS can be slow (e.g. when adding a product to the cart). ### Analysis This is due to the filtering of pricelist rules, which is recomputed on each rendering. ### Solution To solve this, we cache them per pricelist so we only compute them once. ## Benchmarks Profiling `addLineToCurrentOrder` when adding a product to a cart: | # pricelist.item_ids | Before | After | Percentage | ---------------------- | --------- | -----
Original PR description
## Issue When a client has a lot of pricelist rules, the PoS can be slow (e.g. when adding a product to the cart). ### Analysis This is due to the filtering of pricelist rules, which is recomputed on each rendering. ### Solution To solve this, we cache them per pricelist so we only compute them once. ## Benchmarks Profiling `addLineToCurrentOrder` when adding a product to a cart: | # pricelist.item_ids | Before | After | Percentage | ---------------------- | --------- | --------|------------------ | | 21650 | 2350 ms | 6 ms | 99% | ### References opw-4611027 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#203949 Forward-Port-Of: odoo/odoo#200399