Steps to reproduce:
 - Install website_sale_coupon
 - Load demo data
 - Create a promotion program that applies on the order
 - Go to the webshop and add 2 different products to the cart

Bug:
The line of promotion is in second place, instead of at the end.

Fix:
move the old sale order line to the end if it applies on all the order

opw-2985632

closes odoo/odoo#102700

Signed-off-by: William Braeckman (wbr) <wbr@odoo.com>

Since the provider only accepts A4 letters, prevent the user from using
non A4 formats by giving them an error when clicking the Send button.

closes odoo/odoo#105058

Signed-off-by: Florian Daloze (fda) <fda@odoo.com>

The valuation field of the product_id is company-dependent.
If the user validate the picking in multi-company context with his current company != picking company, then the checks may fail, and the account move can not be created.

closes odoo/odoo#104895

Signed-off-by: William Henrotin (whe) <whe@odoo.com>

Pingen does not support localized country names, as such we need to force English
locale.

closes odoo/odoo#104852

Signed-off-by: Florian Daloze (fda) <fda@odoo.com>

EXPLAIN ANALYZE SELECT "stock_move".id
FROM "stock_move"
WHERE ("stock_move"."created_purchase_line_id" in (24065, 24066, 24081, 24082, 24083, 24084, 24085, 24086, 24087, 24088, 24089, 24090, 24091, 24092, 24093, 26121, 26487)) and ("stock_move"."company_id" in (1))
ORDER BY "stock_move"."sequence" ,"stock_move"."id" ;
Goes from 2231.021 ms to 1ms on a database with 2.800.000 stock_move.

closes odoo/odoo#100035

Signed-off-by: Arnold Moyaux (arm) <arm@odoo.com>

This commit aims at preventing the deactivation of a company currency.
Was the issue on 2852452 support ticket (v14). But it seems appropriate to merge it in 13.0 as it is probably a good idea that a company currency always stays active.

How to reproduce bug:
In 13.0:
Install accounting with demo data > activate multi currency in settings > deactivate usd > create new invoice > select eur currency > cannot set usd currency back on invoice

Reconcile JS traceback in 14.0:
Install accounting with demo data > activate multi currency in settings > deactivate usd > go to accounting dashboard > click on reconcile 7 items on Bank journal > click on any “customer/vendor matching” line.

closes odoo/odoo#91470

Task: 2852452
Signed-off-by: Quentin De Paoli <qdp@odoo.com>

When there are many `calendar.event`s (700K+), MemoryError happens when they are tried to be sorted
before returning them in `get_recurrent_ids` method.

[upg-377987](https://upgrade.odoo.com/web#action=150&cids=1&id=377987&menu_id=107&model=upgrade.request&view_type=form)
Traceback from the upgrade request:
```
Traceback (most recent call last):
  File "/home/odoo/src/odoo/12.0/odoo/addons/base/maintenance/migrations/base/tests/test_mock_crawl.py", line 220, in crawl_menu
    self.mock_action(action_vals)
  File "/home/odoo/src/odoo/12.0/odoo/addons/base/maintenance/migrations/base/tests/test_mock_crawl.py", line 347, in mock_action
    mock_method(model, view, fields_list, domain, group_by)
  File "/home/odoo/src/odoo/12.0/odoo/addons/base/maintenance/migrations/base/tests/test_mock_crawl.py", line 368, in mock_view_form
    records = model.search(domain, limit=3)
  File "/home/odoo/src/odoo/12.0/odoo/models.py", line 1581, in search
    res = self._search(args, offset=offset, limit=limit, order=order, count=count)
  File "/home/odoo/src/odoo/12.0/addons/calendar/models/calendar.py", line 1802, in _search
    events = self.browse(events.get_recurrent_ids(args, order=order))
  File "/home/odoo/src/odoo/12.0/addons/calendar/models/calendar.py", line 1261, in get_recurrent_ids
    return [r['id'] for r in sorted(result_data, key=key)]
  File "/home/odoo/src/odoo/12.0/addons/calendar/models/calendar.py", line 1259, in key
    for v, desc in vals_spec
  File "/home/odoo/src/odoo/12.0/addons/calendar/models/calendar.py", line 1259, in <listcomp>
    for v, desc in vals_spec
MemoryError

The issue is happening in the `key` function when data is being sorted before returning it. In this function to compare events for every key a list is returned.
Returning a list for every element leads to a memory error because lists over allocate memory when they are created to make Time complexity of [appending to a list O(1) in amortized time](https://stackoverflow.com/questions/46664007/why-do-tuples-take-less-space-in-memory-than-lists

)
Over allocating memory a few times would not be a problem but given that this db has 700K+ `calendar.events`, over allocation causes a memory error. That's why it is better
to return tuples instead of lists, which do not over allocate because they are immutable, which will resolve the MemoryError.

closes odoo/odoo#103182

X-original-commit: c53081f1
Signed-off-by: Christophe Simonis <chs@odoo.com>

When executing the product2bom method it is possible that an active_test
context is present in self. However, the method should not consider
archived BoMs.

closes odoo/odoo#100771

Signed-off-by: William Henrotin (whe) <whe@odoo.com>

# Description
The absence of those fields can generate a log note spam & performance issue when updating an order line value.

# HOW TO REPRODUCE
- Create PO with 40+ order lines (50 recommend) (* SA to reproduce would be nice)
- Confirm PO (state needs to be "purchase")
- Update the Ordered Quantity of the first line.
- Save
=> PO Lines from page 2 and beyond have each created the following log message:
"The ordered quantity has been updated."

# Explanation
The purchase.order.line onchange will not save any update on 'price_total' because it is not on the onchangeSpec.
Then a purchase.order onchange is done, and this one does have 'order_line.price_total' on the onchangeSpec, forcing the onchange method to return the field 'order_line' (aka ALL the order lines) as updated.
When saving the changes, the JS framework, who does not have the data for the order_lines on the 2nd page, will not correctly filter the fields with real changes, and add the 'product_qty' field (and more) to the "write" call for every order_line not in the 1st page.
Then the write method, without checking if there is any real change on the product_qty, for each lines that has product_qty in the Write's values, will write in the log note and call a performance intensive method: _create_or_update_picking()

OPW-2982004

closes odoo/odoo#100064

Signed-off-by: William Henrotin (whe) <whe@odoo.com>

closes odoo/odoo#104196

Signed-off-by: Louis Baudoux (lba) <lba@odoo.com>

Before this commit, following these steps:
- Go to /blog
- Activate the option Customize > Top banner - Name / Latest Post
- Disable the option Customize > Full Width Cover
- Click on the "guides" tag of the "Buying A Telescope" blog

No blog is displayed. This was because the top banner post is not shown
when the posts are filtered. Basically, once filtered, a blog post was
always missing.

Related to opw-2882492

closes odoo/odoo#93680

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>

Before this commit, following these steps:
- Go to /blog
- Activate the option Customize > Top banner - Name / Latest Post
- Disable the option Customize > Full Width Cover

The URL to which we are redirected when we click on the blog of the
post presented at the top of the page leads to an error.

Indeed the URL of that link is something like "/blog?blog=blog.blog(2,)"
because of the template wrongly using the 'blog_url' QueryURL which is
defined in the case where we are rendering a blog page where no specific
blog is selected. We define that as `QueryURL('/blog', ['tag'], ...)`
but then parts of the template used it like this: `blog_url(blog=X)`
thus generating an URL like "/blog?blog=blog.blog(2,)". Adding "blog" to
the list of params would not be right as would create "/blog/blog/2"
which is still wrong as we want "/blog/2". And of course the "/blog"
prefix in the QueryURL definition is needed in case we only specify a
tag via `blog_url(tag=X)` (we expect /blog/tag/X). Patching QueryURL or
making blog_url a custom function instead of a QueryURL instance could
be a solution but it was judged not stable enough. We'll do that in
master. Here we only support "/blog?blog=blog.blog(2,)" URLs.

Note that many parts of those templates do not use the `blog_url`
function and simply build the URL by hand, which is why the bug only
occurred for a very specific set of blog options.

opw-2882492

Part-of: odoo/odoo#93680
Co-authored-by: Guillaume (gdi) <gdi@odoo.com>

Backport of #83422

closes odoo/odoo#104094

Signed-off-by: Vranckx Florian (flvr) <flvr@odoo.com>

Have the following taxes:
- Tax A: 15%, included in price
- Tax B: 21%, included in price

Create a product P with list price 115 and tax A

Have a fiscal positions FPOS configured with:
- Detect Automatically checked
- Country: Belgium
- Map tax on product: A -> B

Open a web shop session as guest, add P to cart
Go to checkout: Price total will be 115
Fill address details with Belgium address
Go to confirmation page

Error:
Price total will be 100. In the application of fiscal position the wrong
unit price is used

opw-2973879

closes odoo/odoo#101364

Signed-off-by: William Braeckman (wbr) <wbr@odoo.com>

Pingen's v2 API does not offer an endpoint to generate a Cover Page automatically with an API call anymore, thus the need to generate it and append it to the invoice from the client-side beforehand.
Also make the Cover Page option mandatory due to the invoice being near
impossible to format for Pingen's validation needs. (Odoo 13 Only)

closes odoo/odoo#103385

Signed-off-by: Louis Baudoux (lba) <lba@odoo.com>

Because of an issue with converting bool values, read_progress_bar didn't work
on grouping by bool fields (e.g. Active).

Related tests worked fine because mocked server responses were different from
real server responsed. So, we need to adjust mocked server too.

opw-2870937

closes odoo/odoo#95654

Signed-off-by: Julien Mougenot (jum) <jum@odoo.com>

Before this commit: the "no-store" was overridden in developer mode.

'no-cache' does not require that the response must not be stored in the
cache. It only specifies that the cached response must not be reused to
serve a subsequent request without re-validating.
It is up to the browser what to qualify as a subsequent request. In
Chrome, using the back button and duplicating the tab does not. So I
agree that it's better to use 'no-store' in dev mode.

The solution is to prevent changing the "Cache-Control" if it's "no-store".

opw-2855802

closes odoo/odoo#101590

Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>

We are spammed by some accounts that don't have credits anymore.

With this commit, when we try to send letters from the cron, whenever we
meet the CREDIT_ERROR error_code, we stop trying to send the letter.

task-2930455

closes odoo/odoo#103264

Signed-off-by: Florian Daloze (fda) <fda@odoo.com>

Before this commit, customers might see a "transaction not found" page
when returning from Adyen Pay By Link Payment Page.

Now, they'll see a "waiting for payment" page if they returned before
the webhook notification or the confirmation of payment if they
returned after the webhook notification.

opw-3006401

closes odoo/odoo#102999

Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>

The previous link was dead

closes odoo/odoo#101911

Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>

Fixup of 706db8a7

closes odoo/odoo#102558

Signed-off-by: Florian Gilbert (flg) <flg@odoo.com>

The implementation of _find_mail_template allow to call it only
on one record at a time.

This was not the case with in _send_order_confirmation_mail.

- Add excplicit constraint in _find_mail_template
- Make sure _send_order_confirmation_mail call the method once per
  record

closes odoo/odoo#102135

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>

Customer display does not show the appropriate logo when multi-company is
enabled. The logo shown is always the logo of the default company.

Step to reproduce the issue:
1) Install Point of Sale and set up a second company
2) Put a logo on both companies (different ones)
3) Create a POS session on the newest company (not the default one)
4) In this session, activate Customer Display
5) Launch the session and open the Customer Display
The logo shown is the logo of the default company.

Solution: The issue is that when fetching the logo, we don't include
information about the current company. Thus, we include the logo of the default
company (at url /logo). We can easily specifiy which logo we need via the url
/logo?company={company_id}. As dynamic information cannot be included into a
CSS, we include this into the XML as it is done with other images rendered.
In our case, we don't need to retrieve the logo and map it to base 64 (for
ressources requiring to be logged in) because the logo is a resource available
to anyone.

opw-2745014

closes odoo/odoo#93638

Signed-off-by: Trinh Jacky (trj) <trj@odoo.com>

closes odoo/odoo#98021

Signed-off-by: Josse Colpaert <jco@odoo.com>

Part-of: odoo/odoo#98021

This PR https://github.com/odoo/odoo/pull/97805

 wrongly assumed that
self contained a single workorder. This would raise an error if there
were multiple records in self.

Solution:
Modify calls on `self` to `workorder` (as we already iterate through
them)

closes odoo/odoo#101580

Signed-off-by: William Henrotin (whe) <whe@odoo.com>

Steps to reproduce:

1. change system and admin's time zone to one with a different
date than UTC
2. enable multi-currency in the accounting module
3. add rates for the current date in local time zone and UTC
4. go to settings > currencies
5. the rate for the UTC date is displayed

To fix this, we should use a time zone aware field in `res_currency`.

opw-2945108

closes odoo/odoo#100123

Signed-off-by: William André (wan) <wan@odoo.com>

closes odoo/odoo#101380

Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>

- Go to accounting/Report/Invoices Analysis
- Open studio
- enable list view
- empty view

This code bellow compute count_field as `__count` because group_by_no_leaf
is in the context.

But in the basic_model we don't consider this case.

https://github.com/odoo/odoo/blob/5797fd80a63309269f15bcbe4948d4429a53eec2/odoo/models.py#L2293-L2297



opw-2980923

closes odoo/odoo#100869

Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>

Those constraints are an incorrect leftover from 0a42a9de which removed the size constraints on a
number of core tables including 'name' and 'model' of ir_model_data.

Temporary table tmp_ir_translation_import limits columns size (imd_name and imd_model).
Those columns are only used to match columns 'model' and 'name' in ir_model_data.

But in ir_model_data, columns 'model' and 'name' are VARCHAR, no reasons to restrict the temporary table.

opw-2983422

closes odoo/odoo#101144

Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>

Change 'Invoice date' label on report according move_type

closes odoo/odoo#100851

Signed-off-by: William André (wan) <wan@odoo.com>
Co-authored-by: wan <wan@odoo.com>

The default value of max_time_between_keys_in_ms in barcode is 55 ms.
The default value of 55 ms is too low and creates an issue for RFID readers.

The default value of max_time_between_keys_in_ms is increased to 100 ms.

task-2996072

closes odoo/odoo#101128

Signed-off-by: Kevin Baptiste <kba@odoo.com>

Steps to reproduce:
- install sale_management
- enable variant in the setting of sales
- create a second company
- create a product with at least a variant and set the company
  to the active one
- switch to the second company
- create another product and select the same variant
- error on creation

Cause:

 The stored compute is executed as sudo, therefore a value that the user
 has no access may stays in the cache. In subsequent flush we might read
 value from cache when computing some other fields that contain an
 unreadable value for current user.

Solution:

 Flush and invalidate so we don't have unaccessible value in cache in
 this use case.

note: this is not happening in 14.0 and over

opw-2898457
opw-2950290

closes odoo/odoo#96023

Signed-off-by: Nicolas Lempereur (nle) <nle@odoo.com>

A "thematic break"[1] is composed of 3 or more characters (-, _,
or *). The current separator using only 2 characters between the main
body of the PR and the footer / addendum makes it more difficult to
interpret the message.

Update to 3 so e.g. the mergebot can strip the footer.

[1]: https://spec.commonmark.org/0.30/#thematic-breaks



closes odoo/odoo#101072

Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>