Monday, July 1, 2024
14 changes · saas-17.2
Resolved issues and error corrections
Invoices with fixed taxes now avoid showing a base amount next to those taxes, because that amount is not used in the fixed tax calculation. This makes printed invoices clearer and prevents customers from seeing potentially misleading tax details.
Original PR description
When printing an invoice including a fixed tax, we don't want the base amount to be displayed after said tax, since the fixed computation doesn't take into account said amount. Added a check to differenciate fixed taxes from other and adapt display accordingly. see https://github.com/odoo/odoo/pull/168638 task-3964942
This fix keeps the header line visible in empty list views, preventing the page from looking visually broken. Users get a clearer, more consistent view even when there are no records to display.
Original PR description
Before this commit: The content of list view is empty, the outline of the `<thead>` tag (the line below the first `<tr>` inside `<thead>`) would disappear, means its opacity reduced. After this commit: This commit fixes the issue by ensuring that the outline of the `<thead>` tag remains visible even when the list view is empty. task - 3834758
Features or functions removed from Odoo
This update removes an unused piece of timesheet-related project logic. It helps keep the application code cleaner and easier to maintain, with no expected change for end users.
Original PR description
task-3754591
Miscellaneous changes
The TicketBAI EDI integration will pre-process invoice lines to make sure that the description in the EDI is properly encoded/striped of non-supported characters. However this pre-processing step does not take into account the fact that the `name` field of an `account.move.line` record is not required and can thus be Falsy. This causes the EDI pre-processing to crash with a traceback for invoices with empty lines. This also prevents the cron from going any further than such an invoice in i
Original PR description
The TicketBAI EDI integration will pre-process invoice lines to make sure that the description in the EDI is properly encoded/striped of non-supported characters. However this pre-processing step does not take into account the fact that the `name` field of an `account.move.line` record is not required and can thus be Falsy. This causes the EDI pre-processing to crash with a traceback for invoices with empty lines. This also prevents the cron from going any further than such an invoice in its queue. This fix solves this issue. Note that the generated XML will not pass the XLD validation, since the presence of at least one alpha-numeric character is required - but at least the problem will now be explained with an actual error flow and not a complete crash. opw-3974117 Forward-Port-Of: odoo/odoo#171181
Steps: - In Studio, add a many2one field with the widget "many2one_avatar_user" in SO form - Create a SO and confirm it - Press Alt+I Actual result: - Alt+I is related to "Create Invoice" in UI - Alt+I open the user choice for the added field Expected result: - Alt+Q for "Create Invoice" - Alt+I open the user choice for the added field `q` used to be aligned with [sale_subscription](https://github.com/odoo/enterprise/blob/17.0/sale_subscription/views/sale_order_views.xml#L28) `q
Original PR description
Steps: - In Studio, add a many2one field with the widget "many2one_avatar_user" in SO form - Create a SO and confirm it - Press Alt+I Actual result: - Alt+I is related to "Create Invoice" in UI - Alt+I open the user choice for the added field Expected result: - Alt+Q for "Create Invoice" - Alt+I open the user choice for the added field `q` used to be aligned with [sale_subscription](https://github.com/odoo/enterprise/blob/17.0/sale_subscription/views/sale_order_views.xml#L28) `q` used for "Confirm" action in the same view, should not be a conflict opw-4004869 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#170997
…xerror ForecastedDetails template inheritance wrong declaration of name (sale_stock, mrp, purchase_stock) Steps to Reproduce customer want to add an additional column to the ForecastedDetails template. customer able to do it to basic template But not for prepared inheritances in sale_stock, mrp, purchase_stock. The problem is that system can’t resolve symbol e.g t-inherit="sale_stock.ForecastedDetails” because somebody declatest in code ```py <t name="sale_stock.ForecastedDe
Original PR description
…xerror ForecastedDetails template inheritance wrong declaration of name (sale_stock, mrp, purchase_stock) Steps to Reproduce customer want to add an additional column to the ForecastedDetails…
…xerror ForecastedDetails template inheritance wrong declaration of name (sale_stock, mrp, purchase_stock) Steps to Reproduce customer want to add an additional column to the ForecastedDetails template. customer able to do it to basic template But not for prepared inheritances in sale_stock, mrp, purchase_stock. The problem is that system can’t resolve symbol e.g t-inherit="sale_stock.ForecastedDetails” because somebody declatest in code ```py <t name="sale_stock.ForecastedDetails" t-inherit="stock.ForecastedDetails" t-inherit-mode="extension"> instead <t t-name="sale_stock.ForecastedDetails" t-inherit="stock.ForecastedDetails" t-inherit-mode="extension"> ``` here is a issue link :- https://github.com/odoo/odoo/issues/167741 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#171238 Forward-Port-Of: odoo/odoo#169804
In this PR fixes the following issue: - This pull request addresses a mobile view issue in the 'customer ratings' section by setting the 'frequency' field width to 'auto.' This adjustment prevents content overflow on mobile devices, enhancing user experience. Your review and feedback are appreciated for seamless integration. - I'm submitting a pull request to address an issue in the 'project.task' kanban card. The problem is the persistent visibility of the 'Assign People' b
Original PR description
In this PR fixes the following issue:
- This pull request addresses a mobile view issue in the 'customer ratings' section by setting
the 'frequency' field width to 'auto.' This adjustment prevents content overflow on mobile devices,
enhancing user experience. Your review and feedback are appreciated for seamless integration.
- I'm submitting a pull request to address an issue in the 'project.task' kanban card. The problem is
the persistent visibility of the 'Assign People' button for sub-tasks, causing UI clutter. My solution
involves displaying the 'Assign People' button only upon hovering over a sub-task line within the
'project.task' kanban card's sub-tasks list. Your review and feedback on this change are appreciated.
task- 3549267
---
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Forward-Port-Of: odoo/odoo#139807In a large database with lots (>1M) account.moves, installing the ecommerce module can lead to a MemoryError. This is because of the computed stored field `website_id`. Computing this field's value for each one of the >1M account.moves overfills the field_cache and make the 2gb worker hit a memory limit. To fix that, this commit init the compute field through an override of the `_auto_init` method. As `sale.order.website_id` is expected to be NULL at the module installation, creating the webs
Original PR description
In a large database with lots (>1M) account.moves, installing the ecommerce module can lead to a MemoryError. This is because of the computed stored field `website_id`. Computing this field's value for each one of the >1M account.moves overfills the field_cache and make the 2gb worker hit a memory limit. To fix that, this commit init the compute field through an override of the `_auto_init` method. As `sale.order.website_id` is expected to be NULL at the module installation, creating the website_id column on the account.move table is enough. opw-3989064 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#169991
When reducing the quantity to produce in the subcontracted manufacturing orders, we use the wrong quantity for the call to `change_prod_qty()`. Instead of `production.product_uom_qty` (which is in the **product** uom), use `production.product_qty` (which is in the **production** uom), itself set from the move's uom. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#171204 Forward-Port-Of: odoo/odoo#171093
Original PR description
When reducing the quantity to produce in the subcontracted manufacturing orders, we use the wrong quantity for the call to `change_prod_qty()`. Instead of `production.product_uom_qty` (which is in the **product** uom), use `production.product_qty` (which is in the **production** uom), itself set from the move's uom. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#171204 Forward-Port-Of: odoo/odoo#171093
Steps to reproduce: 1. Go to website > Edit 2. Drag and drop 'Banner' snippet and 'Items' snippet > Save 3. Repeat the step 2 again 4. Error will be generated in terminal Current behavior before PR: Dropping and saving new snippets immediately after snippets containing links would generate a traceback. This issue occurred because the `callPostSnippetDrop` had not completed execution, and `cleanForSave`, responsible for removing `zwnbsp` was called prematurely. Consequently `historySt
Original PR description
Steps to reproduce: 1. Go to website > Edit 2. Drag and drop 'Banner' snippet and 'Items' snippet > Save 3. Repeat the step 2 again 4. Error will be generated in terminal Current behavior before PR: Dropping and saving new snippets immediately after snippets containing links would generate a traceback. This issue occurred because the `callPostSnippetDrop` had not completed execution, and `cleanForSave`, responsible for removing `zwnbsp` was called prematurely. Consequently `historyStep` after `callPostSnippetDrop` would reintroduce `zwnbsp` to links. Desired behavior after PR is merged: Ensure that all post-snippet drop promises are resolved before saving. task-3954229 Forward-Port-Of: odoo/odoo#171068 Forward-Port-Of: odoo/odoo#167365
When using base.automation a timed condition using a calendar, the check was done for each record to get the datetime with the delay using the calendar. Now we invert the logic instead of adding the delay to the record in a loop datetime field, we substract from the last_run and now once outside of the loop. It goes from this logic: last_run <= record_dt + delay < now to this: last_run - delay <= record_dt < now - delay In a database with 1000 records to check a single calendar: it
Original PR description
When using base.automation a timed condition using a calendar, the check was done for each record to get the datetime with the delay using the calendar. Now we invert the logic instead of adding the delay to the record in a loop datetime field, we substract from the last_run and now once outside of the loop. It goes from this logic: last_run <= record_dt + delay < now to this: last_run - delay <= record_dt < now - delay In a database with 1000 records to check a single calendar: it went from 15s to 20ms opw-3876867 https://github.com/odoo/enterprise/pull/65417 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#171270 Forward-Port-Of: odoo/odoo#167310
**Current behavior before PR:** In discuss guest user view, notification settings button was visible, which used to throw a "Session Expired" error upon being clicked. **Desired behavior after PR is merged:** In discuss guest user view, notification settings button is removed. **task:3975859** --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#171278 Forward-Port-Of: odoo/odoo#169044
Original PR description
**Current behavior before PR:** In discuss guest user view, notification settings button was visible, which used to throw a "Session Expired" error upon being clicked. **Desired behavior after PR is merged:** In discuss guest user view, notification settings button is removed. **task:3975859** --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#171278 Forward-Port-Of: odoo/odoo#169044
Anglo-Saxon accounting is the main system used in Colombia, this commit enables it by default when installing `l10n_co`. This configuration is only accessible in Debug mode in the `Settings` -> `Bank and Cash` section, so during the implementation it doesn't get much visibility. Setting the correct default for the localization helps prevent re-implementations and reduces accounting errors for companies. At present, the impact of this configuration can only be assessed after going live.
Original PR description
Anglo-Saxon accounting is the main system used in Colombia, this commit enables it by default when installing `l10n_co`. This configuration is only accessible in Debug mode in the `Settings` -> `Bank and Cash` section, so during the implementation it doesn't get much visibility. Setting the correct default for the localization helps prevent re-implementations and reduces accounting errors for companies. At present, the impact of this configuration can only be assessed after going live. [Task link](https://www.odoo.com/odoo/project.task/3990709) task-3990709 Forward-Port-Of: odoo/odoo#170732
[FIX] l10n_pk: script to auto trigger try_loading here->https://github.com/odoo/odoo/commit/049f2a2fad240e738b15794763b48e0e79bf50d3 We did a big improvement in the pk package but if the user tries to use the new stuff He won't have them because he needs to manually trigger the reload button in the configuration page solution: bump-up the package version and add a local script to automatically trigger that try_loading functionality task-id#3871597 --- I confirm I have signed the C
Original PR description
[FIX] l10n_pk: script to auto trigger try_loading here->https://github.com/odoo/odoo/commit/049f2a2fad240e738b15794763b48e0e79bf50d3 We did a big improvement in the pk package but if the user tries to use the new stuff He won't have them because he needs to manually trigger the reload button in the configuration page solution: bump-up the package version and add a local script to automatically trigger that try_loading functionality task-id#3871597 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#171361