Saturday, August 29, 2026
29 changes · master
New functionality added to Odoo
Adds a new option that lets users temporarily hide the chatter panel on forms and bring it back when needed. This gives users more horizontal space for complex forms while keeping chatter access available through a small side toggle.
Original PR description
Created as an example for a support speeddate session. Description of the issue/feature this PR addresses: Forms with a chatter panel can lose a significant amount of horizontal space to the chatter,…
Created as an example for a support speeddate session. Description of the issue/feature this PR addresses: Forms with a chatter panel can lose a significant amount of horizontal space to the chatter, even when the user does not need to interact with it. This can make forms with a lot of content more difficult to use. This PR adds a lightweight global toggle to hide and show the chatter on demand. The toggle is displayed as a small floating tab on the right edge of the viewport and is only shown on views where a chatter is present. Current behavior before PR: The chatter panel is always visible and permanently takes up horizontal space on forms where it is available, even when the user does not need it. Desired behavior after PR is merged: Users can hide the chatter when they need more horizontal space and reopen it when needed using the floating toggle on the right edge of the viewport. The toggle is only displayed on views that contain a chatter. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr (no)
Enhancements to existing features
French B2G invoices can now be identified and routed through Chorus Pro via the approved platform instead of Peppol. This improves compliance for invoices sent to French public sector customers, with added status tracking for sent, suspended, and completed cases.
Original PR description
B2G invoices are not sent via Peppol but to Chorus Pro (via the approved platform). Chorus Pro is the platform with ID 9999 on the annuaire. Any invoice to partner associated with that platform on…
B2G invoices are not sent via Peppol but to Chorus Pro (via the approved platform). Chorus Pro is the platform with ID 9999 on the annuaire. Any invoice to partner associated with that platform on the annuaire is considered B2G (business to government). From a user point of view nothing much changes, except that they have to set some additional fields for which we rely on the existing module `l10n_fr_facturx_chorus_pro`. From a technical PoV we store the information whether a partner is behind Chorus Pro in the `peppol_supported_documents` field. (By putting the special document identifier for Chorus Pro invoices there.) We retrieve the information whether a partner is behind Chorus Pro / B2G from the annuaire lookup. The following new lifecycle statuses have been added. They are required for the functional tests for the Chorus Pro connection. - Sent (sent by the platform) - Suspended - Completed (to "resume" the "Suspended" state) See the related IAP PR: https://github.com/odoo/iap-apps/pull/1804 task-6278159 Forward-Port-Of: odoo/odoo#284677
Resolved issues and error corrections
When an X (Twitter) account token becomes invalid, Odoo now disconnects the account instead of showing a generic error while refreshing the feed. This reduces confusion for social media users and helps them reconnect the account to restore the feed.
Original PR description
Bug === If the token because invalid, then an UserError is raised instead of disconnecting the account. This is because we "blind raise" all errors we get from X, instead of filtering the error linked to the stream configuration. Task-6499283 Forward-Port-Of: odoo/enterprise#129560 Forward-Port-Of: odoo/enterprise#129110
Accounting now uses a more targeted database lookup when finding recent journal entries by date. This can dramatically reduce wait times in affected cases, improving responsiveness for users working with accounting records.
Original PR description
This commit adds an index on `(journal_id, date)` to speed up `_get_last_sequence_domain`. The slow part is `.search(domain, order='date [desc|asc]', limit=1)` Because of the order by date,…
This commit adds an index on `(journal_id, date)` to speed up `_get_last_sequence_domain`. The slow part is `.search(domain, order='date [desc|asc]', limit=1)` Because of the order by date, postgresql scans the index on `date`, assuming a row will be quickly matched. If this assumption is wrong though, it'll scan a large part of the index or all of it, which is slow. This is the case with account move 17102258 on odoo.com at the time of writing this. - before - 1st query https://explain.dalibo.com/plan/6b8ff62b1572ah1a - 2nd query https://explain.dalibo.com/plan/41243cgccg5798gb - after - 1st query https://explain.dalibo.com/plan/4fd2c7ed28g7b138 - 2nd query https://explain.dalibo.com/plan/2heh6c6965c88h71 - ~~1st query https://explain.dalibo.com/plan/f874a31f4b07hdeb~~ - ~~2nd query https://explain.dalibo.com/plan/e8h09725gfh4c2e9~~ full `web_read` - before ~1min 10s - after ~650ms task-6481393 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#283159
Self-billing bills now keep separate numbering per partner, improving traceability and reducing confusion in accounting records. Self-billing invoices can also be imported into a dedicated sales journal, preventing regular sales invoices from accidentally using self-billing numbering patterns.
Original PR description
This PR handles 2 cases : ===== PART 1 ===== Self-billing bill sequences should be unique per partner, as implemented in v19+. This PR backports that behavior to 17.0. ===== PART 2 ===== Previously, the `is_self_billing` option on `account.journal` was available only for purchase journals. This caused an issue when importing a self-billing invoice into a regular sales journal with quick edit mode (accounting firm) enabled. In such cases, the newly created invoices would use the self-billing sequence pattern, leading to traceability issues. This PR allows the creation of self-billing sales journals to prevent this issue. task-6103142 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#282642 Forward-Port-Of: odoo/odoo#259935
Companies in the same Odoo database that share the same French SIREN can now benefit from an already successful PDP registration verification. This reduces repeated manual KYC work when many branches or related companies need to be registered.
Original PR description
We have some clients that have several hundreds of companies/branches on the same db, with the same SIREN (incubateur or the like). They will need to do the kyc (that will be identical, as it's the same SIREN) for all the companies. It's especially cumbersome if it needs manual intervention So, if one company on that database, with the same SIREN, managed to register, then it means it has succeded the kyc. Meaning we can bypass the kyc for the other identifiers as well. task-6515315 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#285236
Batch transfer functionality is now built directly into Inventory instead of requiring a separate add-on and several connector modules. This simplifies setup and maintenance for businesses that use warehouse batching, while keeping the feature controlled by a setting.
Original PR description
This main PR's purpose is to move the `stock.picking.batch` model from `stock_picking_batch` to `stock` module, which let us remove the `stock_picking_batch` and a bunch of bridge modules, and replace them by a setting. Other changes: - Move `StockPickingType` code from `stock_picking.py` file into its own file (`stock_picking_type.py`); - Remove `show_operations` field because this field is unused and we forgot to remove it in master. See commits' message for more information. **Enterprise PR**: odoo/enterprise#128175 **Upgrade PR**: odoo/upgrade#11054 task-6154300
The time off dashboard now shows the number of pending allocation requests, rather than adding up the days or hours in those requests. This makes it easier for managers and employees to understand how many items need attention.
Original PR description
change the number of pending allocation to represent how many pending allocation instead of the number of the days or the hours in the pending allocations Task Id: 6482732
The website countdown block now offers a clearer set of layout choices, combining previous layout and template settings into one simpler option. Editors also get more focused text formatting controls and a warning when a countdown may be outdated, helping prevent display issues on published pages.
Original PR description
[IMP] website: merge countdown snippet layout and template options Following [commit](https://github.com//odoo/odoo/commit/74a9e5bb734708497456ed2bc91465f95505b2d7), we'd like to improve the…
[IMP] website: merge countdown snippet layout and template options Following [commit](https://github.com//odoo/odoo/commit/74a9e5bb734708497456ed2bc91465f95505b2d7), we'd like to improve the countdown snippet. Key changes: 1. Merge Layout and Template into a single layout option with the following layouts: - Circle - Text Inline - Dominoes (replaces Boxes) - Wrapped Numbers - Text - Big Numbers (replaces Clean) 2. Enable toolbar for both metrics and indicators 3. Remove Edge Spacing 4. Convert Circle layout to Text Inline format: display the countdown on the left with reduced size. 6. Retain Circle-specific options plus all Text Inline options when possible. task-5242156 --- [IMP] website, html_builder: show notification if countdown is outdated We would like to show the notification to user saying that the countdown is outdated if it should be updated. This commit also changes wording in that notification to indicate that it may cause problems. task-5242156 --- [IMP] website, html_editor: add countdown toolbar namespace For metrics in countdown, we don't want to show all toolbar groups, but rather just font and decoration groups buttons. This commit adds a new namespace for that, and adds that namespace to buttons we want to display. task-5242156
Hong Kong payroll now combines severance pay, long service payment, and payment in lieu of notice into one termination payslip. This reduces HR administration and makes final employee payments clearer and simpler to process.
Original PR description
Up until now, we have been maintaining separate structures for all termination pays (severance pay, long service payment, payment in lieu of notice). While this is one way of doing this, it is not necessarily the best. It asks the HR team to issue up to three payslip on the last month of an employee to cover all statutory payments; and it also makes paying the employee more complex. According to feedbacks, it is actually preferable to issue a single merged payslip that includes all payments, both for clarity and simplicity, which is what we will now do. task-6174257
Inventory batch picking support has been moved into the main stock-related apps, removing several separate bridge modules. This simplifies maintenance while keeping barcode, quality control, and fleet workflows aligned with the updated inventory structure.
Original PR description
*: quality_control_picking_batch, stock_barcode, stock_barcode_picking_batch, stock_barcode_quality_control, stock_barcode_quality_control_picking_batch, stock_fleet_enterprise The `stock.picking.batch` model were moved from `stock_picking_batch` to `stock`, and the module `stock_picking_batch` doesn't exist anymore. This commit adapts the code to this change in the enterprise modules. Since we had some brigde modules, some modules are merged: - `quality_control_picking_batch` is merged into `quality` and `quality_control`; - `stock_barcode_picking_batch` is merged into `stock_barcode`; - `stock_barcode_quality_control_picking_batch` is merged into `stock_barcode_quality_control`. In summary, removed modules are: - `quality_control_picking_batch`; - `stock_barcode_picking_batch`; - `stock_barcode_quality_control_picking_batch`. **Community PR**: odoo/odoo#282884 **Upgrade PR**: odoo/upgrade#11054 [task-6154300](https://www.odoo.com/odoo/966/tasks/6154300)
The app switcher background has been adjusted to look more consistent across light and dark modes, Studio views, and custom image backgrounds. This improves readability and visual polish for users while preserving compatibility with other areas that reuse the same background component.
Original PR description
* web_studio This commit applied general adaptions related to the background app switcher. - Align the design across all cases: light/dark mode, Studio open/ closed, and custom image backgrounds - Optimize contrast with app icons - Improve rendering when Studio is enabled - Adapt the background implementation to restore compatibility with other apps where o_home_menu_background is used (e.g. event registration). task-6511279
The Belgian payroll Dimona flow is clearer and safer for HR users. Dimona statuses are now read-only, manual declaration screens show the correct action and guidance, and certificate setup explains how it supports automated submissions.
Original PR description
Feedback from the Dimona UX review: - The Dimona badge on the employee form sat before the review state pill and was a dropdown, so the state could be set by hand. That state reflects what was…
Feedback from the Dimona UX review: - The Dimona badge on the employee form sat before the review state pill and was a dropdown, so the state could be set by hand. That state reflects what was declared to the ONSS, not a user choice: it becomes a read-only badge, placed after the review pill, and "Send Dimona" stays the only way to act on it. dropdown_selection_badge honors a readonly attribute on the field node for this; props.readonly could not be used, as list and kanban records are never in edition, which would freeze the review state badge as well. - The manual declaration wizard always showed "Dimona IN" with the text of an entrance declaration, whatever had to be declared. Its title and description now follow the next action (in, out, update, cancel or issue), and the ONSS link points to the Dimona portal instead of the DmfA instructions. - The ONSS certificate setting only said "Configure ONSS certificate". It now states what the certificate is for, automating Dimona, DMFA and DRS submissions via the API, and links to the ONSS page explaining how to obtain the credentials. - The Dimona declaration fields on the employee offered to create a declaration from the autocomplete, which would record a declaration that was never sent to the ONSS. task-6470879
Checkout now more reliably requires Chilean and Mexican invoicing information before customers proceed to payment. This prevents shoppers from accidentally bypassing required billing details and avoids checkout steps being incorrectly shown or hidden based on another visitor's session.
Original PR description
Commit odoo/odoo@1facbc0646bd made the checkout validate every `website.checkout.step` through a single flow, but the localization invoicing steps were left out. These steps are shown or hidden by…
Commit odoo/odoo@1facbc0646bd made the checkout validate every `website.checkout.step` through a single flow, but the localization invoicing steps were left out. These steps are shown or hidden by publishing or unpublishing a shared step record on each checkout. This is unreliable: the cart and payment pages don't refresh it, so the step can be left in the state set by another visitor. The step is also only validated when the visitor goes through it, so opening `/shop/payment` directly skips it. An override also forces `try_skip_step` off so the checkout page cannot jump past the step. This commit filters the Chilean and Mexican step out of the breadcrumb steps domain per request when it is not needed, instead of toggling the shared record, and checks it in the validation flow so opening `/shop/payment` redirects back to it while incomplete. Forcing `try_skip_step` off is no longer needed. Chile stores the taxpayer type on the invoicing customer so the check also works when the billing contact is not the ordering one. Mexico uses a best-effort check, since a customer who wants no invoice looks the same as one who skipped it. task-6045788 See also: - https://github.com/odoo/odoo/pull/276844 - https://github.com/odoo/upgrade/pull/10806
Grouped list views now automatically hide sections that no longer contain records when those empty sections are not intentionally preserved. This reduces clutter and helps users focus on relevant information after actions change the contents of a group.
Original PR description
Having empty groups in list view can add unnecessary info for the user, especially when an action empties a group without triggering a full rendering (see Accountant > Journal Audit Review). This commit add a new filtering step to the `RelationalModel` group reading post process which filters out empty groups when the view is not grouped by a default groupBy nor the grouping field has the `group_expand` property. task-6465612
This update prevents barcode EPC generation from failing when tracked and untracked stock move lines are processed together. It keeps valid tracked items aligned correctly and gives untracked items a clear error instead of causing broader list-view failures.
Original PR description
Problem: `_compute_electronic_product_code` built `tracking_number_list` by filtering out move lines without a lot_id/lot_name, but kept iterating over the full, unfiltered `move_line_ids`. As soon…
Problem: `_compute_electronic_product_code` built `tracking_number_list` by filtering out move lines without a lot_id/lot_name, but kept iterating over the full, unfiltered `move_line_ids`. As soon as a tracked product had an untracked move line mixed in with tracked ones (e.g. a manufacturing byproduct move line with no lot), the two lists fell out of sync: at best tracking numbers got assigned to the wrong move line, at worst `tracking_number_list[i]` went out of range and raised an IndexError. Solution: Exclude untracked move lines from `move_line_ids` before building `tracking_number_list`, so both stay the same length and index- aligned. Untracked lines get their own explicit "no tracking number" error instead of breaking the alignment for the rest. Steps to reproduce: Open runbot V19 -> go to moves history (Inventory) -> add `electronic_product_code` to list view using studio -> remove filter/select all records -> https://anotepad.com/notes/jwxyskc2 Forward-Port-Of: odoo/enterprise#128521 Forward-Port-Of: odoo/enterprise#123859
Website helpdesk forms now keep the translations from the original form template. This ensures customers see the form in the website or visitor language, instead of the language used by the employee who created the helpdesk team.
Original PR description
When a helpdesk team has its website form enabled, a dedicated qweb view is generated from the `ticket_submit_form` template. The arch was read in the language of the user creating or editing the…
When a helpdesk team has its website form enabled, a dedicated qweb view is generated from the `ticket_submit_form` template. The arch was read in the language of the user creating or editing the team, so a team set up by an English user language produced an English form even when the website served another language. Steps to reproduce ================== 1. Set a language other than English as the website default language. 2. While your user language is English, create a helpdesk team with the website form enabled. 3. Open the team form on the website. => The form is rendered in English instead of the website language. Root cause ========== `_ensure_submit_form_view` read the template arch without forcing a language, so it used the current user's language and stored only that value on the generated per-team view. Fix === Read the template arch in the default language of the team's website, so the generated form matches the website language regardless of the user's own language. opw-6303903 Forward-Port-Of: odoo/enterprise#129265 Forward-Port-Of: odoo/enterprise#121164
This fixes an error that could block website editors from saving SEO cover images when unrelated expense attachments existed in the system. The change keeps attachment checks focused on the selected file, avoiding unnecessary access checks on records the user should not see.
Original PR description
### Issue: A user with Website Editor rights but no Accounting or Expense access gets an `AccessError` when using Optimize SEO to add a cover image, if another user has an expense with an attachment…
### Issue:
A user with Website Editor rights but no Accounting or Expense access gets an `AccessError` when using Optimize SEO to add a cover image, if another user has an expense with an attachment
### Steps to reproduce:
- Install `ai` and `hr_expense`
- With Admin, upload an image as an expense
- Update Demo user rights (Accounting: No, Expenses: No, Website: Editor and Designer)
- Log in as Demo
- Go to Website > Site > This page > Optimize SEO
- In Cover Image, add any image, select it and save
Before the fix, an `AccessError` is raised
### Cause:
When Save is triggered, `ai.attachment.vacuum.mark_attachments_used()` is called with the outer domain:
`[('attachment_id', 'in', [3868])]`
The `ai.attachment.vacuum` security rule uses:
`domain_force = [('attachment_id.res_access_write', '=', True)]` https://github.com/odoo/enterprise/blob/474ee25f82a3980530aae4c3450c69d925b6076e/ai/security/security.xml#L1-L24
During optimization, the dotted path is decomposed into an `any` condition:
`('attachment_id', 'any', [('res_access_write', '=', True)])` https://github.com/odoo/odoo/blob/dcb9ad42ba302a5ed0ec334daf4c8a4cb5ec0931/odoo/orm/domains.py#L999-L1004
`_optimize_any_domain_at_level` tries to narrow the comodel scan by reusing the outer constraint on `attachment_id` via `search_domain` in context
It finds `c` where `c.field_expr == 'attachment_id'`, but `c.value` is `{3868}` — a plain set of ids, not a `Domain`
Since the code only recognized `Domain` values, it treated this as unknown and fell back to `comodel_domain = Domain.TRUE`
`_search_res_access` for `ir.attachment` then scanned every attachment using:
`['&', ('res_model', '!=', False),
'|', ('res_id', '!=', False), ('create_uid', '!=', 5)]`
This matches a lot of attachments because the initial `attachment_id` constraint is missing
This scan included an attachment linked to an expense the current user cannot read
`_inaccessible_comodel_records` in `hr_expense` then checked all expenses linked to the found attachments and raised the `AccessError`
This fallback also triggers a second failure when the database contains more than 10 000 attachments: `_search_res_access` caps its unbounded scan at `MAX_SEARCH_LIMIT` and raises `ValueError("Cannot search, too many attachments")`
Confirmed by seeding ~10 500 attachments and reproducing the error
### Notes:
A new test in `test_any_domain_search_context.py` verifies in isolation that unrelated attachments are never scanned when a plain `in` condition narrows the search
opw-6467087
Forward-Port-Of: odoo/odoo#285384
Forward-Port-Of: odoo/odoo#284678This fixes an error that prevented sale timesheet reports from being printed when using German or other translated languages. The report now identifies the description column in a language-independent way, making printing more reliable for multilingual users.
Original PR description
Currently a traceback is occurring when the user tries to print a sale timesheet report in the German language. **<h3>To reproduce the issue:</h3>** 1) Install the `sale_timesheet` module. 2) Create…
Currently a traceback is occurring when the user tries to print a sale timesheet report in the German language. **<h3>To reproduce the issue:</h3>** 1) Install the `sale_timesheet` module. 2) Create a `service` product configured to create a `project and task` on the order. 3) Create a confirmed SO with that product 4) Add a timesheet line to the SO from the `Recorded` stat button 5) Switch to the German language 6) Print `Timesheet(Zeiterfassung)` report 7) A traceback occurs **<h3>Error:</h3>** ``` ValueError: Element „<xpath expr="//th/span[text()='Description']">“ kann nicht in der übergeordneten Ansicht lokalisiert werden ``` **<h3>Cause:</h3>** The xpath relies on the plain text `Description` to identify the `<th>` element. https://github.com/odoo/odoo/blob/5f63fb1af418cc75ff2e382b0455002e7a798071/addons/sale_timesheet/report/report_timesheet_templates.xml#L3-L4 The xpath cannot find the Description header when the language is changed because the text is translated in the base template. As a result, the xpath fails due to the missing target. **<h3>Fix:</h3>** Use a language-independent `name` attribute as the `xpath` anchor. So the `Description` column can be reliably matched regardless of the active language. **<h3> Note:</h3>** This fix requires both modules update, which is generally risky in stable branches. However, this template was recently introduced in [saas-19.4](https://github.com/odoo/odoo/pull/191969/changes#diff-9f4f0d28ffb3aa9fcc4a5f1037bdf532e1e71c4e9703e13b6856ddfd79e15e69R4). So there should be no existing customers using it yet, except saas customers coming from a new database. Therefore, applying this fix in 19.4 is less risky. opw- 6475623 Forward-Port-Of: odoo/odoo#284096
Tax returns will no longer be incorrectly marked as paid when someone replies to or sends a normal chatter message. Payment finalization now only happens from the intended tax payment instructions flow, preventing accidental status changes and reducing accounting confusion.
Original PR description
Before this fix: Replying to or sending a message from the chatter of a tax return could incorrectly change its state to Paid. This happened because action_send_mail() automatically called _action_finalize_payment() for account.return records. After this fix: Payment finalization only happens when the composer is opened from the tax payment instructions flow. Normal chatter messages and replies will no longer change the tax return state to Paid. task-6469536 Forward-Port-Of: odoo/enterprise#129161
Point of Sale settings now keep all employees added to Advanced rights when saving configurations that also use employee login and delivery integrations. This prevents selected staff from disappearing after save, so access permissions remain as intended.
Original PR description
Steps to reproduce ------------------ 1. Go to the Point of Sale settings and select a shop. 2. Enable UrbanPiper and set a delivery provider. 3. Enable "Log in with Employees". 4. Add two employees…
Steps to reproduce ------------------ 1. Go to the Point of Sale settings and select a shop. 2. Enable UrbanPiper and set a delivery provider. 3. Enable "Log in with Employees". 4. Add two employees to the "Advanced rights" field. 5. Save. Observation -> Only the employee of the PoS manager is left, the two employees are gone. Why it's happening ------------------ When saving from the settings, `pos.config` is written with the context key `from_settings_view`, which tells `_preprocess_x2many_vals_from_settings_view` that the commands received for an `x2many` field are the complete new list, so every record not in these commands is unlinked. The `write` of `pos_hr` always puts `advanced_employee_ids` in the values, even when the caller does not write this field, to keep the employees of the PoS managers in the list (1766d6d40a45). Since b48ddcce7bf in enterprise, UrbanPiper writes on the config a second time during the same save, still with `from_settings_view`. That write does not touch the employees, so `pos_hr` fills `advanced_employee_ids` with the managers only (cf 1766d6d40a45), and the ones that were just saved are unlinked. The fix ------- When `pos_hr` adds the field by itself, keep the employees already set on the config instead of starting from an empty list. opw-6500207 Forward-Port-Of: odoo/odoo#284961
New onsite learning events created from the Onsite view or an employee resume now appear immediately after creation. The update relaxes the visibility rules and automatically links the current user's employee record, reducing confusion and duplicate event creation.
Original PR description
Onsite events created from the "Onsite" view or the employee resume selector do not appear immediatlely after creation This occurs because currently the domain for onsite events requires that the…
Onsite events created from the "Onsite" view or the employee resume selector do not appear immediatlely after creation This occurs because currently the domain for onsite events requires that the event to have multiple slots as well as to have at least one employee registered to it. Therefore, newly created records often fail these criteria and remain hidden. In further versions, this pr: https://github.com/odoo/odoo/pull/246285/ changes the domain of the event selector in the employee resume by removing the dependency on the multiple slots and filtering by the specific employee for registration. This change is not stable to backport as it indroduces the `employee_id` field as an invisible field in the xml to be able to compare in the domain. This commit partly changes both domains to not require the multiple slots anymore, while still showing all events for which an employee is registered. This commit also ensures that when an event is created from the Onsite view or selector, the current user's employee will be registered to it. Steps to reproduce - Go to employees->Learning->Onsite - Select New and create an event - Go back to Onsite Courses - You will not see the created event (unless it is multi_slot and an employee was registered) opw-5915686 Forward-Port-Of: odoo/odoo#284200 Forward-Port-Of: odoo/odoo#258952
Fixes an issue where confirming a sales order again after cancellation could fail to create the expected event registration. This helps ensure customers who re-confirm event orders are properly registered, including through portal or preview flows.
Original PR description
**Steps to reproduce:** - Create a SO and add the Event Registration - Standard product and specify an event - Confirm the SO then select "Create/Update registrations" - Cancel the SO, then "Set to…
**Steps to reproduce:** - Create a SO and add the Event Registration - Standard product and specify an event - Confirm the SO then select "Create/Update registrations" - Cancel the SO, then "Set to Quotation" - Select "Preview" and confirm the Sale Order again - There will not be any new registration created when there should be one. **Behavior:** Usually when a sale order is confirmed the `action_sale_order_event_registration` form will be opened which when filled correctly creates registrations. However in certain cases: confirming from the customer portal, or simply closing the form when it is opened, will not trigger `action_make_registration` which creates registrations if it is not already the case `action_confirm()` should be creating the registrations correctly on its own anyway by calling ´init_registrations()´ : https://github.com/odoo/odoo/blob/beed378cde592bc96c1e79a976ac775264b843ed/addons/event_sale/models/sale_order_line.py#L49-L66 This function tries to create each missing registrations by looking at the amount in the so_line and deducting the already created registrations, however since some of them can be cancelled, this computation is wrong. And leads to registration not being created when they should. opw-6444127 Forward-Port-Of: odoo/odoo#280426
This fixes an issue where manually adjusted prices on optional quotation items could be overwritten when a customer changed the quantity in the portal preview. Sales teams can now trust that custom prices remain as intended while still allowing standard pricelist updates when no manual price was set.
Original PR description
When a user manually sets a price on an optional line (overriding the pricelist), and then changes the quantity in the portal preview, the manual price is lost and gets reset to the pricelist price.…
When a user manually sets a price on an optional line (overriding the pricelist), and then changes the quantity in the portal preview, the manual price is lost and gets reset to the pricelist price. Steps to reproduce: --- - Install Sales module and enable Pricelists. - Create a product with qty-based pricelist rules: - min qty: 1 → price: 100 - min qty: 10 → price: 80 - Create a quotation with an optional section containing this product. - Manually change the product price to 150 (overriding pricelist). - Mark the section as optional and preview the quotation. - Change the quantity to 10 in portal preview. Issue: --- - The manually set price (150) is incorrectly reset to the pricelist price (80). Root cause: --- - After [commit], if there is no config parameter set and if there is an active pricelist, we simply call `_reset_price_unit()` without checking whether the price was manually set or not. - Additionally, `_reset_price_unit()` calls `update()` which writes `price_unit` and `technical_price_unit` one by one as separate `write()` calls. The `write()` method has a guard([1]) that strips a lone `technical_price_unit` write unless `sale_write_from_compute` is set in context. Without this flag, `technical_price_unit` is silently discarded, causing it to drift from `price_unit`. On the next qty change, this mismatch is detected as a manual price, permanently blocking further pricelist updates. Solution: --- - Check whether the price was manually set before calling `_reset_price_unit()`, and pass `sale_write_from_compute=True` in context so both `price_unit` and `technical_price_unit` are written correctly. [commit]: https://github.com/odoo/odoo/commit/93b6bdd6a4909bc0b45b90ab6a2d0734a218292d [1]https://github.com/odoo/odoo/blob/fffd987cc98d1ea0cd04e24dda2ed8b64a219cdc/addons/sale/models/sale_order_line.py#L1391-L1399 opw-6426634 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#285212 Forward-Port-Of: odoo/odoo#281107
This fixes a problem that could stop PDF quotes from being generated when Quote Builder documents included dynamic fields. Sales teams can now print these quotes without encountering an error in affected Python and PDF library setups.
Original PR description
Issue: --- Due to this issue, generating PDF Quote using Quote Builder with dynamic fields leads to a traceback. This was partially fixed by: e16edc7b0d9f56cb7769068ecf7d64ff8b0f6359 Steps: --- 1-…
Issue: --- Due to this issue, generating PDF Quote using Quote Builder with dynamic fields leads to a traceback. This was partially fixed by: e16edc7b0d9f56cb7769068ecf7d64ff8b0f6359 Steps: --- 1- Using a python 3.13 env, install requirements.txt. (You could instead uninstall pypdf2 and install pypdf==5.4.0) 2- Enable Quote Builder. 3- Create a SO and in quote builder tab, select a document. This document should have dynamic fields. e.g. you could use`Office Furnitures Header` document. 4- Print -> PDF Quote. Cause: --- In previous fix, we fixed the traceback when no dynamic field is set. However, if you have a dynamic field, then inside `PdfWriter._update_field_annotation()`, the font is get from `DR` dict inside acroform: https://github.com/py-pdf/pypdf/blob/f20954f2241640feb484800e191373f8fbdfa44b/pypdf/_writer.py#L917-L929 Even if we are not setting a font, we need to have an empty `DR` dict inside acroform, in order to avoid calling `get` on a none object, which is leading to the traceback. Also in previous fix we were losing `NeedAppearances` inside `AcroForm` by creating new dict which wasn't right. opw-6392001 Forward-Port-Of: odoo/odoo#278881
Checkout now checks whether an invoicing step is required for each shopper individually instead of relying on a shared published state. This prevents customers from skipping required Taiwanese e-invoice details by going directly to payment and avoids one visitor's checkout state affecting another's.
Original PR description
Commit 1facbc0646bd made the checkout validate every `website.checkout.step` through a single flow, but the localization invoicing steps were left out. These steps are shown or hidden by publishing…
Commit 1facbc0646bd made the checkout validate every `website.checkout.step` through a single flow, but the localization invoicing steps were left out. These steps are shown or hidden by publishing or unpublishing a shared step record on each checkout. This is unreliable: the cart and payment pages don't refresh it, so the step can be left in the state set by another visitor. The step is also only validated when the visitor goes through it, so opening `/shop/payment` directly skips it. An override also forces `try_skip_step` off so the checkout page cannot jump past the step. This commit filters the invoicing step out of the breadcrumb steps domain per request when it is not needed, instead of toggling the shared record. The cart is passed to the breadcrumb helpers and their cache is removed so the filter runs for each visitor. The Taiwanese step is now checked by the validation flow, so opening `/shop/payment` redirects back to it while incomplete. The checkout page now jumps to the next step, which is the invoicing step when required, so forcing `try_skip_step` off is no longer needed. task-6045788 See also: - https://github.com/odoo/enterprise/pull/124616 - https://github.com/odoo/upgrade/pull/10806
The attendance Gantt view now correctly displays progress bars for employees who have no attendance records, making empty schedules easier to understand. It also improves the warning display so managers can better distinguish attendance-based employees with missing entries.
Original PR description
[FIX] hr_attendance_gantt: missing progress bars for empty employees On the attendance gantt view, employees without any attendances would have their progress bar hidden, and their label being something like `0h / 8h (+-8h)` in orange. This PR changes it to get the expected behavior: we should also show the progress bar for employees without attendances, and, instead of showing it in yellow all the time, show it when attendance-based employees have no attendances. Since we need to `patch` a renderer class from `hr_attendance_gantt` in `hr_work_entry_attendance`, we need to make the link between those two modules explicit in the manifest file (otherwise hoot tests will fail) This should not change much, as `hr_work_entry_attendance` already installs `hr_attendance_gantt` through an auto_install chain. See this [Discord discussion](https://discord.com/channels/678381219515465750/687338039717920792/1537826154814378167) (in french, sorry) task-6453757
The social media comments window now shows the correct like icon and supports liking Tweets from the comments view. Users also get clearer feedback when creating leads from social posts, and social users have more consistent access to live post information.
Original PR description
Bug === Since https://github.com/odoo/enterprise/commit/86659741990de2ba9c8bf738207edf0e8a0ba8c4 the like icon in the comment in the modal view is broken. We added a new getter `likesIcon` but we didn't use it... Improvements ==== Show create message when creating lead. Make the access of `social.live.post` consistent with the access of `social.post`. Task-6425391
Employer-initiated departure reasons in Belgian payroll are now handled like dismissals. This ensures notice details, 13th month eligibility, and outplacement information are calculated and shown correctly during the end-of-collaboration process.
Original PR description
New departure reasons were added previously but their computation for the end of collaboration process was missing. Selecting employer-initiated reasons (codes `358` and `359`) didn't treat them as "Fired" so the notice duration, notice start date, and 13th month eligibility were not calculated and the `Outplacement` field stayed hidden. This treats these reasons as "Fired" in the departure model and updates the view to display the `Outplacement` field when needed. Task-6499788