1. merge intra community taxes (one taxe with two report lines instead of two taxes with one report line each)
2. fix "tva à l'encaissement" fiscal position mapping (two rates were using the same xml_id)
3. add outside EU import taxes

closes odoo/odoo#49186

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

after this commit, cost analysis report will
be there even if bom product qty is 0.

Fixes-2224779

closes odoo/odoo#49512

X-original-commit: c1ed66f8
Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>

Following https://github.com/odoo/odoo/pull/44393



Delete the qweb template `web.styles_company_report`
Recompute company specific style by going into
Settings > Configure Document Layout > change stuff and save
Print a report, in HTML to get an human readable error
(PDF rendering would just ignore the error)

Before this commit, there was an error "could not get asset content"
This was because the css asset created in db had a value of type string
whereas it should have been the same type as b64encode (which is byte-like)

After this commit, there is no error at rendering time

closes #49456

closes odoo/odoo#49502

Signed-off-by: Lucas Perais (lpe) <lpe@odoo.com>

We should only check the consistency between the vat number and country code in
case we have one country involved in the recordset.

closes odoo/odoo#49428

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

For all the purchase cash basis taxes:

The account 445800 must be used as cash_basis_transition_account_id

The account 445660 must be used in distribution of the tax

Wrong minus_report_line_ids and plus_report_line_ids were set for these taxes

opw:2230288

closes odoo/odoo#49338

Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>

Bug
===
We can not share a text with the "share content popover" widget
if the text contains a single quote (').

Task-2223641

closes odoo/odoo#49391

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

closes odoo/odoo#49467

X-original-commit: 8ba95070
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>

A long time ago, commit [1] increased the limit of displayed badges
inside a many2many_tags widget, from 40 to 1000 (a large limit that
should never been reached with such a widget).

However, this solution was specific to the many2many_tags widget,
and didn't apply to its extensions (like the many2many_tags_email
widget). For instance, open the chatter full composer, add 40
partners in the recipients field and try to add one more: nothing
happens.

This commit sets the limit on the widget itself. That way,
extensions automatically have the same limit, and can override it
if needed.

[1] d4cf4374

Task : 2091027

closes odoo/odoo#49466

X-original-commit: 02e41f1131b4ea3ad977cc241a3d447c37195f6e
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>

Doing the check at the beginning seemed to cause issues in upper version
(see #48745). Move it at the end of the process to avoid potential
issues.

closes odoo/odoo#49465

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

Some bits of code still use old variable.

Task ID 2237300
Closes PR #49462

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

When computing `quantity_done` in an onchange, `move.id` is a `NewId`,
not an `int`. However, the `read_group` call returns regular integers
for `move_id`.

Consequently, `rec.get(move.id, [])` is an empty list.

closes odoo/odoo#49322

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

- Create a product P using one component C
- The BOM is flexible
- Create a MO for 10 units
- Product 9 units, but use all components
- Post the inventory

The MO is considered as done, while it should remain in progress.

This happens because the MO state is computed based on the move raw
state. In this case, it should compare the quantity produced and the
quantity to produce.

The material availability is set to `False`.

opw-2220855

closes odoo/odoo#49247

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

Avoid crashing if no relevant move is found.

closes odoo/odoo#49439

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

refactoring

this fixes bugs introduced by
https://github.com/odoo/odoo/commit/beaa30a3d1843de43a45f419bfbc1bfa7613a920



closes odoo/odoo#48997

Signed-off-by: Csaba Tóth <csaba.toth@i3rendszerhaz.hu>
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>

Steps to reproduce:
- install timesheet
- go to general settings > set documents layout to use 'external_layout_boxed'
- go to timesheets > list view > select all > print > timesheet entries

Previous behavior:
the first line of the report's table is missing a border

Current behavior:
borders are consistent

opw-2230710

closes odoo/odoo#49426

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

odoo/odoo#46024 improved the serialisation of arrays being logged (in
order to get more relevant data than just `Array(5)`.

However, chrome apparently serialises *argument* objects as array-like
with a few nits, namely that arguments have non-numeric properties
which don't necessarily have a value associated with them.

The array formatter / converter assumed all properties had a value,
resulting in the process crashing rather dramatically.

Filter out non-numeric properties on arrays.

closes odoo/odoo#49385

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

1. Install PoS and Accounting
2. In a product category activate the Inventory Valuation automated
3. Sell a quantity 0 of such product via POS

Traceback will occur when confirming the sale because the inventory
valuation do not handle zero quantity

opw-2206625

closes odoo/odoo#49376

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

At step of selecting a picture, the trigger pointer was wrong:
it was pointing to a missing picture (image with title "sell.jpg").

**BEFORE FIX**
Selecting an non-existing pointer make the tour break.

**AFTER FIX**
Now, the pointer is on picture with title :
"s_company_team_image_4.png", who exist in the list.

Task ID 2228922

closes odoo/odoo#49147

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

Currently, when we add many2many field relation with purchase.order in view
where view does not have state field, it throws error, it is because
purchase.order list view has attrs for column_invisible which parent.state
which is wrong, purchase.order will not have parent.

With this commit, remove wrongly added column_invisible attrs on invoice_status
field.

task-2195175

closes odoo/odoo#48926

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

1. Install PoS and Accounting
2. In the PoS setting activate the Invoicing option
3. Sell an item X with Invoice toggle to client Y and pay it
4. Sell the same item X with Invoice toggle to the same client Y with
quantity -1
5. Close the PoS session

Traceback wil occur when trying to Validate Closing & Post Entries
because in '_create_invoice_receivable_lines' already reconciled lines
are already filtered out so invoice_receivable_lines is empty

opw-2206625

closes odoo/odoo#49377

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

The generation of SN is incorrect with the following initial sequence:
`BAV023B00001S00001`.
Indeed, the sequence generated is: `BAV023B00001S`, `BAV023B00002S`.

This occurs because the same digits `00001` appear more than once.

opw-2230913

closes odoo/odoo#49373

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

Before this commit, we had an error when writing on account_opening_move_id, fiscalyear_last_day, fiscalyear_last_month on multiple company at once

closes odoo/odoo#49208

Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>

closes odoo/odoo#49370

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

Fix regression introduced with e0ed7b12

Issue without current commit:

When doing `abort` next updates from the bus are only received after the normal
timeout, which makes the interface unresponsive to updates during that amount of
time.

`abort` is for example called during `addChannel`, where it is specifically
documented that new updates are to be received immediately.

closes odoo/odoo#49348

X-original-commit: 888610e5e07794e000ef27edcb075cb45c1e2ccc
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>

This number is intended to be used on payment to match them with the invoice they were made for, so it's totally sound to use it in that field.

[IMP] l10n_ch: make QR-code generation compliant with the current version of the spec

Backport of the l10n_ch part of https://github.com/odoo-dev/odoo/commit/89d87c31801507df8e055c19bcac75c05599bd4f

Forward port of https://github.com/odoo/odoo/pull/48901



closes odoo/odoo#49318

Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>

- Go to Inventory > Configuration > Operation Types
- In Receipt, activate 'Show Detailed Operations'
- Go Inventory > Overview, click on 'Receipts'
- Create a picking, and in the 'Operations' tab add 5 units of a product
  tracked by unique S/N
- Click on the + sign
- Set the First SN (e.g. TEST001) and Number of SN to 5, then Assign
  Serial Numbers

Nothing appear in the Detailed Operations, and it's impossible to
validate the picking.

The stock move lines are created, but not linked with the picking.

opw-2230913

closes odoo/odoo#49339

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

SIPS sometimes send empty notifications. I have been unable to find out
why or to reproduce the issue; it seems to be linked to a delay on their
end since these notifications usually arrive after a customer is
redirected back to Odoo (at least on their test account).

I'd rather log a warning for those cases (since nothing is wrong on
Odoo's side, there's just no info to act upon) instead of a full
traceback.

closes odoo/odoo#49324

X-original-commit: 100ef1bd
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

Render the payment form and process feedback from the provider.

X-original-commit: 60445af2

SIPS date format can be somewhat variable, some sanitation is required
before using the data raw for the ORM.

opw-2224926

X-original-commit: 620e987f
Co-authored-by: Nicolas Martinelli <nim@odoo.com>

Install point_of_sale,l10n_fr_pos_cert,contacts,pos_loyalty
Create French company, add french chart of account
Create DEMO customer loaded with reward points
Enable loyalty program on POS, change the default reward to be:
* Reward Type: Discount(in value)
* Discount Product: create one (all default is ok)

Open POS, add a product, add the customer and the reward.

When you try to use the reward it is positive. You also get a "Decrease
the quantity by" pop-up when trying to remove it.
Since the reward setup conflict with the pos modification the fix is to
add an exception for the rewards

opw-2224086

closes odoo/odoo#49130

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

Issue

	- Install Projects
	- Change language
	- Check projects

	There is a word "X Tasks" which is not
	translated.

Cause

	This is a model attribute which is not
	translatable. It can have any value
	"Tasks" is not mandatory.

Solution

	Make it translatable

OPW-2233084

closes odoo/odoo#49328

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

- Create a new survey 'Test Survey';
- Add a section 'S1';
- Add a multiple lines text box question 'Q1';
- Add a section 'S2';
- Add a section 'S3';
- Don't move any of the section or question to avoid changing the
    sequences, for the moment all the sequences are equal to 10;
- Change the survey layout to 'One page per section';
- Save the survey;
- Test the survey;
- Fill the 'Q1';
- Go to the last page and Submit the survey;
- Review your answers.

Before this commit, the question is empty, this occurs because the
question page_id is not set, in _compute_page_id, the sequence of the
question should be bigger than the question of the page (in this case
the section).

Now, the _compute_page_id was change to take into account the case when
the question has the same sequence as the page (section).

opw-2222045

closes odoo/odoo#49315

Signed-off-by: Jorge Pinna Puissant (jpp) <jpp@odoo.com>
Co-authored-by: Nicolas Lempereur <nle@odoo.com>

Issue

	- Install eCommerce
	- Create a SO
	- Add a section
	- Add a note with big text
	- Customer Preview

	The word are not broken correctly

Cause

	We added a break-all to prevent
	long URL/words overflow.

Solution

	Use work-break: break-word which
	will break the word correctly.

	Use overflow-wrap/word-wrap
	(same usage but we need both
	to ensure browsers compatibility)
	which makes sure the long string
	will wrap and not bust out of the
	container

OPW-2222814

closes odoo/odoo#49312

X-original-commit: 0754590a
Signed-off-by: Jason Van Malder (jvm) <jvm@odoo.com>

Purpose
=======

Avoid to show to much information to leave responsibles.

Specification
=============

- Dynamically add/remove the leave_manager_ids on the group
  group_hr_holidays_responsible, on employee creation/modification
- Retrict the available employees when creating a leave. For
  manager/users : Everyone. For responsibles: The employee for
  who the user is responsible. Otherwise: Only me.
- Don't obfuscate the leave name on employees for who I'm the
  responsible.
- Add a new SQL view for "Everyone" report. To select only the
  information we want to share instead of showing the hr.leave
  model directly.

closes odoo/odoo#47659

Taskid: 2206862
Related: odoo/enterprise#9239
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>

* Outdated xml

See https://stackoverflow.com/questions/53160591/extending-default-template-odoo-12/53162439#53162439

Courtesy of EvgenijjKum from #41970.

* Outdated javascript

See https://stackoverflow.com/questions/41061604/odoo-10-how-to-create-snippets-javascript-option#41062045



closes odoo/odoo#49295

X-original-commit: bdf8ef75
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>

X-original-commit: 45c0f781

Create a product with UoM Dozens
Sold 5x via POS
Go to the product page

The smart button does not reflect the correct quantity
(0.42 instead of 5).

Fixing by editing the SQL query, the POS already provide
the correct quantity of sold goods as it is not possible
to change the UoM. This aim to match the behavior of sale (where
is necessary to account for possible differences between the uom
reported on the sale order line and the one specified in the product
page)

opw-2227482

closes odoo/odoo#49302

X-original-commit: 7af831fe
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

- Create a product P with a Sales Price of 5000 and a Cost of 3000.
- Create a bill for 10 units @ 3000, post
- Create an invoice for 10 units @ 5000, post
- Create a customer credit note for 5 units
- Open the Product Margin report

The # Purchased is 15 while the # Invoiced in Sale is 10.

The query incorrectly sums the invoice lines based on their type. We
should group customer invoices with customer refunds, and group vendor
bills with vendor refunds. When grouping we should subtract invoices
and refunds.

Note that the `avg_unit_price` is not modified since there is no reason
to refund a product at a different unit price.

opw-2211636

closes odoo/odoo#49280

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>

Update information in JS doc for 13.0 (replace control panel mixin with the hasControlPanel attribute).

closes odoo/odoo#48650

Signed-off-by: Denis Ledoux (dle) <dle@odoo.com>