audrius/stock-logistics-reporting
1
0
Fork 1
mirror of https://github.com/OCA/stock-logistics-reporting.git synced 2025-02-16 17:13:21 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] stock_average_daily_sale: Don't rely on tuples as id

Browse Source 
As for x reason, some products appear more than one time in the report,
id generated by the concatenation of product_id and warehouse_id is
irrelevant (as duplicate values possible). Use row_number() instead
This commit is contained in:
Denis Roussel
2023-10-11 16:15:37 +02:00
committed by twalter-c2c
parent 0c2f631a4c
commit fb2cee485b
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
stock_average_daily_sale/models/stock_average_daily_sale.py
View File

@@ -214,7 +214,8 @@ class StockAverageDailySale(models.Model):
averages AS(
SELECT
concat(warehouse_id, product_id)::integer as id,
row_number() over (order by product_id) as id,
concat(warehouse_id, product_id)::integer as window_id,
product_id,
warehouse_id,
(avg(product_uom_qty) FILTER
@@ -295,7 +296,7 @@ class StockAverageDailySale(models.Model):
(cfg.number_days_qty_in_stock * average_qty_by_sale)
) as recommended_qty
FROM averages t
JOIN daily_standard_deviation ds on ds.id= t.id
JOIN daily_standard_deviation ds on ds.id= t.window_id
JOIN stock_average_daily_sale_config cfg on cfg.id = t.config_id
JOIN stock_qty sqty on sqty.pp_id = t.product_id AND t.warehouse_id = sqty.warehouse_id
JOIN product_product pp on pp.id = t.product_id