**Problem:**
For products with variants and differents steps for the BoM depending
on the variant, the Manufacturing Order overview for a given product
with a given variant would display the sum of the cost of all variants
possible for this product. If we had a product "Drawer" with a color
variant "Black" and "White", which cost 10 and 20 euros respectively,
the Manufacturing Order overview of a Black Drawer would display the
cost of the drawer + **30** euros (10+20), instead of being the cost
of the drawer + **10** euros. The display cost is the sum of the price
of every variants for this product, instead of being the price of the
current variant of the product.

**Steps to reproduce:**
- Create a product with variants
- Change the BoM and make some components needed only for certain variants and add a price for them
- Go to the MO overview
- The Cost BoM cost total is the sum of all prices AND variants

**Cause of the issue:**
https://github.com/odoo/odoo/blob/d3ab7b2c3ceddbb4cb8d8e91fac0d60e28e5673a/addons/mrp/report/mrp_report_mo_overview.py#L85
This treats every remaining lines of the BoM as a missing component,
thus adding it's price to the total price. Some lines should only be
applied if a specific variant is selected, but with this code, every
line that has a condition depending on variants will be taken in.

**Fix:**
Before adding the product of a BoM line to the missing components, we
check that the BoM line should be applied to the current variant.
The missing components are still used and can be not empty in the
case where the BoM includes a line that was deleted from the MO,
making the MO cost and the BoM cost different. In this case, the
missing components will not be empty and will still work as
intended without taking the unwanted variants into account.

opw-4557169

Forward-Port-Of: odoo/odoo#199264

**Problem:**
when doing a refund from pos of an avco (or fifo) product, with the option 
"ship later" the cost of the product does not appear on the pos order line

**Steps to reproduce:**
- Enable Settings "Allow Ship Later", "Automatic Accounting"
- Navigate to Sales/Configuration/Product Categories
- Create a new category with Inventory Valuation field as "Automated" 
and the Costing Method field as "Average Cost (AVCO)"
- Open Point of Sale/Product
- Create a new storable product
- In the general information tab set a cost and select your new 
category for the product category field
- In the Sales tab, make sure that "Available in POS" is checked 
and select a category
- Set an on Hand quantity
- Open a point of sale from the Point of Sale Dashboard
- Select your product and click on Payment
- On the top right of the screen select a customer with an adress,
click on Invoice and on Ship later
- Select a payment method and validate
- Navigate to Point of Sale/Orders
- click on the last order created, and click on the "Pickings" smart button
- validate the picking
- come back to the store in point of sale
- select the three horizontal lines on the top right and click on Orders
- filter by paid orders
- select the last order created and click on refund and then on payment
- check Invoice and Ship Later on the top right, select a payment method and validate
- Navigate back to Point of Sale/orders and select the last order created

**Current behavior:**
 The total cost is zero on the pos order line
 Even if the delivery is validated via the Picking smart button
 the total cost is still zero

**Expected behavior:**
 It should reflect the cost of the product (depending on the costing method)

**Cause of the issue:**

When the pos order is created from Point of Sale, the process_saved_order method is called
this method calls (1)create_order_picking and then (2)compute_total_cost.
https://github.com/odoo/odoo/blob/504ef7cdeb1600b86feefbe8668144da9cd5a70b/addons/point_of_sale/models/pos_order.py#L183-L184 

**Scenario A** Inside (1) create_order_picking, if the PosOrder doesn't have 
a shipping_date the if statement is False
https://github.com/odoo/odoo/blob/504ef7cdeb1600b86feefbe8668144da9cd5a70b/addons/point_of_sale/models/pos_order.py#L1017 
and the method _create_picking_from_pos_order_lines is called.
This method then calls _action_done.
https://github.com/odoo/odoo/blob/504ef7cdeb1600b86feefbe8668144da9cd5a70b/addons/point_of_sale/models/stock_picking.py#L70 
Inside the stock_account override of _action_done, _create_in_svl is called.
https://github.com/odoo/odoo/blob/504ef7cdeb1600b86feefbe8668144da9cd5a70b/addons/stock_account/models/stock_move.py#L289
This method sets the value of the stock_valuation_layer_ids attribute
of the stock move to a new stock valuation layer.

**Scenario B** Inside (1) create_order_picking, if the PosOrder has a 
shipping_date (in the case where we selected "Ship Later") the 
method _launch_stock_rule_from_pos_order_lines is called.
https://github.com/odoo/odoo/blob/504ef7cdeb1600b86feefbe8668144da9cd5a70b/addons/point_of_sale/models/pos_order.py#L1017-L1018 
and nowhere inside this method is _action_done called.
This makes sense because the stock move hasn't been validated.

When (2) compute_total_cost is then called by process_saved_oder, if the product category is avco, _compute_average_price will be called. 
If we are in Scenario B there is no stock valuation layer for this stock move.
Consequently there will be no candidates and the price will not be computed.
https://github.com/odoo/odoo/blob/504ef7cdeb1600b86feefbe8668144da9cd5a70b/addons/stock_account/models/product.py#L795-L798

opw-4614503

Forward-Port-Of: odoo/odoo#210868
Forward-Port-Of: odoo/odoo#207482

When a signed document is moved to trash in the documents app, when the cron tries to delete the said document it should raise a foreign key violation error in the terminal not in UI.

**Steps to reproduce:**
* Install document and sign
* Document App> all> click `Employment contract.pdf`.
* Select sign>drag and drop Signature>Sign now
* Sign the document>validate and send completed document
* Documents>certificate of completion or the signed document>actions>trash
* The error will be produced when cron deletes the document.
* You can delete the document from trash to see what error will be displayed by cron.
  
 `ERROR: update or delete on table "ir_attachment" violates foreign key constraint "sign_request_completed_document_rel_ir_attachment_id_fkey" on table "sign_request_completed_document_rel"`
 
**Solution:**
* Modifying the domain for `_gc_clear_bin` via a new function called overridden `_gc_documents_domain` in `documents_sign` this will allow for cron to skip over signed required document during unlink process.
* This will still throw an `Validation Error` when the user tries to delete it from the GUI letting them know it cannot be deleted.

Sentry-6225030131

Forward-Port-Of: odoo/enterprise#83765

**Steps to reproduce:**
- Install Planning app
- Set the type of view to Gantt
- (Create a shift if there is none)
- Click on `Publish` button
- The period end date doesn't take the last day into account (time is 00:00)

**Issue:**
Gantt model default `_buildMetaData` computation behavior was changed multiple times during previous refactoring:

Previously used:
```
export function computeRange(scale, date) {
    let start = date;
    let end = date;

    if (scale === "week") {
        // startOf("week") does not depend on locale and will always give the
        // "Monday" of the week... (ISO standard)
        const { weekStart } = localization;
        const weekday = start.weekday < weekStart ? weekStart - 7 : weekStart;
        start = start.set({ weekday }).startOf("day");
        end = start.plus({ weeks: 1, days: -1 }).endOf("day");
    } else {
        start = start.startOf(scale);
        end = end.endOf(scale);
    }

    return { start, end };
}
```
Which became:
```
export function getRangeFromDate(rangeId, date) {
    const startDate = localStartOf(date, rangeId);
    const stopDate = startDate.plus({ [rangeId]: 1 }).minus({ day: 1 });
    return { focusDate: date, startDate, stopDate, rangeId };
}
```

**Fix:**
Added `localStartOf` function in the context setup of PlanningGanttModel.

To avoid overwriting current changes for other modules (which could be intended), this was done instead of :
```
export function getRangeFromDate(rangeId, date) {
    const startDate = localStartOf(date, rangeId);
    const stopDate = localEndOf(startDate, rangeId);
    return { focusDate: date, startDate, stopDate, rangeId };
}
```

related commits:
https://github.com/odoo/enterprise/commit/d18639785e622d97102227103d62ddb3a2716be6 https://github.com/odoo/enterprise/commit/c175a848369057cc69596737dcd9d21dd24c9d5c

opw-4580458

Forward-Port-Of: odoo/enterprise#85608

<b>Steps to produce :</b>

1) Install 10n_de_reports and switch to the German company 
2) Create a journal entry from accounting with a credit and debit 
3) Now update the credit and debit to a different value. 4) Post the journal
5) Go to "Accounting / Reporting / Audit Reports / General Ledger" 
6) Download "DATEV DATA (ZIP)" and check "EXTF_accounting_entries.csv"

<b>Issue:</b>
The line for the created journal entry has a price_total of 
the previous balance even after updating the credit and debit values.

<b>Cause:</b>
When the user updates the 'debit' and 'credit' values for an `aml`, 
The `price_totals` value remains unchanged as it is not dependent on `balance`.

So, it will show the previous value in the report line 
because we are taking the abs(aml.price_total) from the report data.

<b>Solution:</b>
We can add an extra check of the move type so that it will fall back to the else block, 
where the line_amount will take a value by calculating taxes from `aml.tax_ids.compute_all`.

opw-4707567

Forward-Port-Of: odoo/enterprise#83777