Monday, January 29, 2024
5 changes · 17.0
Enhancements to existing features
This update improves the speed of automated tests for inserting lists into spreadsheets by approximately 6.2 seconds. The tests now run more than twice as fast (from 11.6 seconds down to 5.4 seconds on average), which helps development teams get faster feedback during testing cycles without changing any user-facing functionality.
Original PR description
The helper `toggleCogMenuSpreadsheet`is quite slow atm. As much as this should be investigated, there is a quick win by cutting in the functional flow to insert a list in a spreadsheet and rely on the implementation by triggering the bus event related to tje insert of the list in a spreadsheet. This revision speeds up the related concerned tests by ~6.2 seconds on a local installation. Testing the modules `documents_spreadsheet > list view` and `documents_spreadsheet > insert_list_spreadsheet_menu`: Average on 10 runs Before: 11600 ms After: 5400 ms Forward-Port-Of: odoo/enterprise#55219 Forward-Port-Of: odoo/enterprise#55202
Resolved issues and error corrections
Fixed an issue where simply clicking inside a link in a saved document would incorrectly mark the editor as having unsaved changes, causing an unnecessary discard button to appear. The fix ensures that selecting text without actually editing it no longer triggers the dirty state.
Original PR description
Current behaviour before commit: In a saved pad, putting cursor inside a link without making any other changes makes editor dirty. This happens because `sanitize` method fails to sanitize the clone node of root editable element. As result discard button appears on top. Desired behaviour after commit: Now, `sanitize` method sanitizes the clone node of root editable element. Due to this, putting cursor inside a link without editing doesn't make editor dirty and discard button doesn't appear on top. task-3607361 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#143737
Miscellaneous changes
Error sending to TBAI 'IVA 0% Prestación de servicios extracomunitaria' --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#151479
Original PR description
Error sending to TBAI 'IVA 0% Prestación de servicios extracomunitaria' --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#151479
The website publishing feature in server actions was displaying incorrectly after a recent update, showing a checkbox without proper labels and organization. This fix properly organizes these fields so they display clearly and function as intended. This improves the user experience for administrators who manage website publishing automation.
Original PR description
Since the refactoring of base_automation, the 'website-publishing' option of server actions was broken; it did get included in the view but outside of a group node and without labels. The result was a floating checkbox which would trigger a label-less char field upon selection - not the clearest of UIs. This commit properly insert these fields into a group node. Note that since server actions can normally only be reached by going through technical menus, the fact that this section of view is now in a group_no_one-only section should not change anything functionally.
Before this commit, it was not possible to create a new picking on mobile if this picking has multiple moves. How to reproduce: - Open Odoo on mobile (or on desktop but simulate mobile device with devtools); - Inventory > Operations > Transfers; - Create a new transfer (a receipt for example); - In its operations (moves), add a new product, then add a second product. - Try to save or validate the picking -> A warning prevent to do it, saying you're trying to write on the field `product_q
Original PR description
Before this commit, it was not possible to create a new picking on mobile if this picking has multiple moves. How to reproduce: - Open Odoo on mobile (or on desktop but simulate mobile device with devtools); - Inventory > Operations > Transfers; - Create a new transfer (a receipt for example); - In its operations (moves), add a new product, then add a second product. - Try to save or validate the picking -> A warning prevent to do it, saying you're trying to write on the field `product_qty` instead of the field `product_uom_qty`. What's happening: For a unknown reason, the fields from the `stock.move` list view are saved once a second move is created, even if we're using the kanban view in the mobile version. To avoid this issue, we force the `force_save` to be false for the field `product_qty` in the kanban view. That's way, it prevents the issue. OPW-3577844 Forward-Port-Of: odoo/odoo#143776