Daily updates from Odoo
Wednesday, September 1, 2021
6 changes · master
Enhancements to existing features
The Documents spreadsheet view now uses the same built-in styling approach as the main spreadsheet tool. This removes obsolete custom styling and helps keep icons and layout details visually consistent across spreadsheet experiences.
Original PR description
Up until the introduction of bootstrap in this repository, we have had to make specific style rules inside Odoo for both versions of spreadsheet to look the same. Concretely, we had to override o-spreadsheet rules inside documents_spreadsheet because the former did not have default css rules introduced by bootstrap (e.g. vertical-align: middle on icons). Now that bootstrap is part of o_spreadsheet, the old o-spreadsheet rules have become obsolete and as we removed them, we did not correctly adapt documents_spreadsheet. With bootstrap, we can however define rules directly in the components definition as those will applied in both o-spreadsheet and documents_spreadsheet. Co-authored-by: Pierre Rousseau <pro@odoo.com> Task 2531049
Resolved issues and error corrections
Opening list properties from a spreadsheet cell now works without causing an error. This prevents an interruption when users manage embedded lists in Documents spreadsheets, making the feature more reliable.
Original PR description
Steps to reproduce: - Insert a list in o-spreadsheet - Right click on a cell - Click on "List properties" => Traceback
Fixed the default appraisal feedback templates so numbered lists can be created and edited properly. This prevents whole paragraphs from being selected unexpectedly, making employee and manager feedback forms easier to format.
Original PR description
The way default employee and manager fields are configured now, does not allow the html list (/num) to be used correctly inside it. Replicate an issue: inside dafault form use list (type /num), the whole paragraph will be selected. After this change listing should work properly in default feedbacks. task - 2632811
This fix prevents custom appraisal survey views from unintentionally becoming the default view. It keeps the appraisal survey experience consistent after related widget behavior changes, reducing confusion for users opening surveys or responses.
Original PR description
The behavior of the float_time widget changed. This needed to be updated to match the new options. task-2388785
Miscellaneous changes
Issue : When exporting the ecdf declaration in Balance Sheet (abridged) for Luxembourg accounting, there is a traceback `KeyError: 'no_format_name'` Steps to reproduce: 1) Set your company country to Luxembourg 2) Install Accounting Module 3) Settings > User & Companies > Companies add a Matr Number in Your Company (ex : 12345678901) 4) Settings > Accounting Enable `Add totals below sections`, Save 5) Accounting > Reporting > Balance Sheet (abridged) 6) Click Export ECDF Declarati
Original PR description
Issue : When exporting the ecdf declaration in Balance Sheet (abridged) for Luxembourg accounting, there is a traceback `KeyError: 'no_format_name'` Steps to reproduce: 1) Set your company country to Luxembourg 2) Install Accounting Module 3) Settings > User & Companies > Companies add a Matr Number in Your Company (ex : 12345678901) 4) Settings > Accounting Enable `Add totals below sections`, Save 5) Accounting > Reporting > Balance Sheet (abridged) 6) Click Export ECDF Declaration, then Export XML -> Bug, traceback appears Why is that a bug: There should not be an issue triggering a traceback at that step opw-2581605 Forward-Port-Of: odoo/enterprise#20512 Forward-Port-Of: odoo/enterprise#20163
### [FIX] l10n_mx_edi: skip the header of csv file When copying the csv file into a postgres table whe should skip the header, otherwise ``` test_12=> select * from l10n_mx_edi_product_sat_code limit 1 +------+--------+--------+--------------+----------+--------------+---------------+-------------+--------------+ | id | code | name | applies_to | active | create_uid | create_date | write_uid | write_date | |------+--------+--------+--------------+----------+------------
Original PR description
### [FIX] l10n_mx_edi: skip the header of csv file When copying the csv file into a postgres table whe should skip the header, otherwise ``` test_12=> select * from l10n_mx_edi_product_sat_code limit…
### [FIX] l10n_mx_edi: skip the header of csv file When copying the csv file into a postgres table whe should skip the header, otherwise ``` test_12=> select * from l10n_mx_edi_product_sat_code limit 1 +------+--------+--------+--------------+----------+--------------+---------------+-------------+--------------+ | id | code | name | applies_to | active | create_uid | create_date | write_uid | write_date | |------+--------+--------+--------------+----------+--------------+---------------+-------------+--------------| | 1 | code | name | applies_to | True | <null> | <null> | <null> | <null> | +------+--------+--------+--------------+----------+--------------+---------------+-------------+--------------+ ``` ### [REF] l10n_mx_edi: properly open file and minor refs Opening a file should be done through `tools.file_open`, better also with a context manager. Forward-Port-Of: odoo/enterprise#20495 Forward-Port-Of: odoo/enterprise#20462