Daily updates from Odoo
Tuesday, August 6, 2024
3 changes · saas-17.2
Resolved issues and error corrections
This fix ensures self-ordering always has the required default user set when the feature is enabled. It prevents customers or staff from seeing access errors when opening kiosk or menu ordering pages after the setting was removed.
Original PR description
When self ordering is enabled, default user should always be set. If it is not the case, self ordering pages would return access error Steps to reproduce: ------------------- * Enable self order (kiosk/menu/menu+ordering) * Remove default user from pos config * Try to open the self order page > Observation: you get an access error opw-4076798 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Incoming emails in very long conversations now check only the most recent message references when deciding which thread they belong to. This avoids inefficient database searches on large systems while preserving normal behavior for typical email threads.
Original PR description
We previously took into account all possible Message-IDs found in the References header of incoming messages. We also don't try to match the `In-Reply-To` header separately, since it is supposed to…
We previously took into account all possible Message-IDs found in the References header of incoming messages. We also don't try to match the `In-Reply-To` header separately, since it is supposed to be appended to the `References` header, when present. This is all specified in RFC5322, section 3.6.4: https://datatracker.ietf.org/doc/html/rfc5322#section-3.6.4 However, for very long threads, the `References` field can contain dozens or even hundreds of previous Message-IDs, even though we generally care only about the last one. This can be an issue for databases with a large number of messages, as the query planner could mistakenly choose a suboptimal plan (e.g. a Seq Scan or filtered Index Scan), rather than looping on the `message_id` index, which is the best plan. Here we truncate the list of References to only include the last 32 ones. For most threads this will make no difference, as they have less than 32 referencs. For very long threads it should still pick up the last known Message-ID without trouble. And if the last 32 messages aren't known by the system, it's unlikely that the message should still be considered part of earlier threads - it's been sidetracked elsewhere. PS: also simplify the `order` clause, as no secondary sort criterion is going to work after `ORDER BY id`.
Fixes an issue in Point of Sale where failed kitchen/change printing could leave old content behind and cause it to appear on the customer's receipt. Receipts now print only the intended order information, reducing confusion for restaurant staff and customers.
Original PR description
If you print changes in a restaurant, then print the receipt of the same order. The receipt would contain the changes. Steps to reproduce: ------------------- * Setup a printer to print changes and give it the IP 0.0.0.0 so that it fails * Open a PoS restaurant * Add some product to the order, and send it in preparation * You will have an error * Pay for the order, and print the receipt with web print > Observation: The changes of the order appears on top of the receipt Why the fix: ------------ This was happening because the content of `render-container` was not emptied. So when trying to print the receipt, the content of the receipt was added to the changes already in the `render-container` opw-4027722 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr