[FIX] {sale_}purchase{_stock}: select sellers depending on date

Commit 72a17baf force the PO `date_order` to be >= at ` today() to
ensure the select_seller method take the right seller price. This
introduces an error in the MPS module that may need to create PO in the
past.

Instead, this commit patch the call to `_select_seller` to take at least
`today()`

closes odoo/odoo#133759

Opw: 3167094
Related: odoo/enterprise#46623
Signed-off-by: Arnold Moyaux (arm) <arm@odoo.com>

addons/purchase/models/purchase.py

@@ -1223,7 +1223,7 @@ class PurchaseOrderLine(models.Model):
         seller = product_id.with_company(company_id)._select_seller(
             partner_id=partner,
             quantity=uom_po_qty,
-            date=po.date_order and po.date_order.date(),
+            date=max(po.date_order and po.date_order.date(), fields.Date.today()),
             uom_id=product_id.uom_po_id)
 
         taxes = product_id.supplier_taxes_id

addons/purchase_stock/models/stock_rule.py

@@ -60,7 +60,7 @@ class StockRule(models.Model):
                 supplier = procurement.product_id.with_company(procurement.company_id.id)._select_seller(
                     partner_id=procurement.values.get("supplierinfo_name"),
                     quantity=procurement.product_qty,
-                    date=schedule_date.date(),
+                    date=max(schedule_date.date(), fields.Date.today()),
                     uom_id=procurement.product_uom)
 
             # Fall back on a supplier for which no price may be defined. Not ideal, but better than
@@ -259,17 +259,16 @@ class StockRule(models.Model):
             res['orderpoint_id'] = orderpoint_id.id
         return res
 
-    def _get_po_date(self, company_id, values):
-        purchase_date = min([fields.Datetime.from_string(value['date_planned']) - relativedelta(days=int(value['supplier'].delay)) for value in values])
-        return purchase_date - relativedelta(days=company_id.po_lead)
-
     def _prepare_purchase_order(self, company_id, origins, values):
         """ Create a purchase order for procuremets that share the same domain
         returned by _make_po_get_domain.
         params values: values of procurements
         params origins: procuremets origins to write on the PO
         """
-        purchase_date = max(self._get_po_date(company_id, values), fields.Datetime.now())
+        purchase_date = min([fields.Datetime.from_string(value['date_planned']) - relativedelta(days=int(value['supplier'].delay)) for value in values])
+
+        purchase_date = (purchase_date - relativedelta(days=company_id.po_lead))
+
 
         # Since the procurements are grouped if they share the same domain for
         # PO but the PO does not exist. In this case it will create the PO from

addons/purchase_stock/tests/test_purchase_lead_time.py

@@ -361,5 +361,5 @@ class TestPurchaseLeadTime(PurchaseTestCommon):
         purchase_order = self.env['purchase.order'].search([('partner_id', '=', self.partner_1.id)])
 
         today = fields.Datetime.start_of(fields.Datetime.now(), 'day')
-        self.assertEqual(fields.Datetime.start_of(purchase_order.date_order, 'day'), today)
+        self.assertEqual(purchase_order.date_order, today)
         self.assertEqual(fields.Datetime.start_of(purchase_order.date_planned, 'day'), today + timedelta(days=7))

addons/sale_purchase_stock/tests/test_lead_time.py

@@ -29,18 +29,19 @@ class TestLeadTime(TestCommonSalePurchaseNoChart):
     def test_supplier_lead_time(self):
         """ Basic stock configuration and a supplier with a minimum qty and a lead time """
 
+        self.env.user.company_id.po_lead = 7
         seller = self.env['product.supplierinfo'].create({
             'name': self.vendor.id,
             'min_qty': 1,
-            'price': 1,
-            'delay': 7,
+            'price': 10,
+            'date_start': fields.Date.today() - timedelta(days=1),
         })
 
         product = self.env['product.product'].create({
             'name': 'corpse starch',
             'type': 'product',
             'seller_ids': [(6, 0, seller.ids)],
-            'route_ids': [(6, 0, (self.mto_route + self.buy_route).ids)]
+            'route_ids': [(6, 0, (self.mto_route + self.buy_route).ids)],
         })
 
         so = self.env['sale.order'].with_user(self.user_salesperson).create({
@@ -59,9 +60,4 @@ class TestLeadTime(TestCommonSalePurchaseNoChart):
         so.action_confirm()
 
         po = self.env['purchase.order'].search([('partner_id', '=', self.vendor.id)])
-
-        start_of_day = lambda x: fields.Datetime.start_of(x, 'day')
-        today = start_of_day(fields.Datetime.now())
-
-        self.assertEqual(start_of_day(po.date_order), today)
-        self.assertEqual(start_of_day(po.date_planned), today + timedelta(days=7))
+        self.assertEqual(po.order_line.price_unit, seller.price)