Fix liquidaciones. Cambia los creates a multi para evitar el warning.

Añade campos is desit y última fecha de liquidación a purchase.order. Se
asegura de linkar la línea de invoice con la línea de purchase order.

stock_picking_deposito/models/purchase_order.py

@@ -2,34 +2,33 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 from odoo import models, fields, api
-from odoo.exceptions import ValidationError, Warning
+from odoo.exceptions import ValidationError
 
-import pdb
-import logging
-_logger = logging.getLogger(__name__)
 
-class PurchaseOrde(models.Model):
+class PurchaseOrderDesposito(models.Model):
     _inherit = 'purchase.order'
 
     is_deposit = fields.Boolean(
         help='Éste proveedor nos deja material a depósito.',
-        string='Es a depósito',
+        string='A depósito',
     )
     deposit_last_liquidation_date = fields.Datetime(
-        string='Fecha última liquidación compras',
-        help="Cuándo se realizó la última liquidación de compras con éste proveedor."
+        string='Última liquidación',
+        help="La fecha en la que se realizó la última liquidación de compras de éste depósito."
     )
 
-    @api.model
-    def create(self, vals):
-        if self.is_deposit:
-            if not self.partner_id.deposit_buy_accept:
-                raise ValidationError("Este proveedor no suministra material en depósito, configúralo antes de crear un depósito para él.")
-            vals['deposit_last_liquidation_date'] = fields.datetime.now()
-        return super(PurchaseOrde, self).create(vals)
+    @api.model_create_multi
+    def create(self, vals_list):
+        pol = super().create(vals_list)
+        for po in pol:
+            if po.is_deposit:
+                if not po.partner_id.deposit_buy_accept:
+                    raise ValidationError("Este proveedor no suministra material en depósito, configúralo antes de crear un depósito para él.")
+            po.deposit_last_liquidation_date = fields.datetime.now()
+        return pol
 
-    def make_liquidation_invoice(self, context = None):
-        invoice_obj = self.env['account.move']
+    def get_products_sold_sales(self):    
+        product_dict = {}
         if self.deposit_last_liquidation_date:
             sale_orders = self.env['sale.order'].search([
                 ('date_order', '>', self.deposit_last_liquidation_date),
@@ -42,7 +41,6 @@ class PurchaseOrde(models.Model):
                 ('state', 'in', ['sale', 'done'])
             ])
         
-        product_dict = {}
         if sale_orders:
             sale_lines = self.env['sale.order.line'].search([
                 ('order_id', 'in', sale_orders.ids),
@@ -62,9 +60,11 @@ class PurchaseOrde(models.Model):
                         'tax_ids': [(6, 0, line.product_id.supplier_taxes_id.ids)],
                         'quantity': line.product_uom_qty
                     }
-        _logger.warning("product_dict %s", product_dict)
-        pdb.set_trace()
-        pos_orders = False
+
+        return product_dict
+
+    def get_products_sold_pos(self):
+        product_dict = {}
         if self.deposit_last_liquidation_date:
             pos_orders = self.env['pos.order'].search([
                 ('date_order', '>', self.deposit_last_liquidation_date),
@@ -76,7 +76,6 @@ class PurchaseOrde(models.Model):
                 ('lines.product_id', 'in', self.order_line.mapped('product_id').ids),
                 ('state', 'in', ['invoiced', 'done'])
             ])
-        
         if pos_orders:
             pos_lines = self.env['pos.order.line'].search([
                 ('order_id', 'in', pos_orders.ids),
@@ -96,6 +95,12 @@ class PurchaseOrde(models.Model):
                         'tax_ids': [(6, 0, line.product_id.supplier_taxes_id.ids)],
                         'quantity': line.qty
                     }
+        return product_dict
+
+    def make_liquidation_invoice(self, context = None):
+        invoice_obj = self.env['account.move']
+        product_dict = self.get_products_sold_sales()
+        product_dict |= self.get_products_sold_pos() # > Python 3.9
         
         if not product_dict:
             msg = "No se ha vendido ningún producto en depósito de éste proveedor desde la última liquidación.\n\n"
@@ -104,7 +109,7 @@ class PurchaseOrde(models.Model):
             else:
                 msg += "Todavía no se ha realizado ninguna liquidación de compras a éste proveedor."
             raise ValidationError(msg)
-        _logger.warning("product_dict %s", product_dict)
+
         invoice_vals = invoice_obj.default_get(invoice_obj._fields.keys())
         invoice_vals.update({
             'ref': "Liquidación Compras " + self.name,
@@ -117,7 +122,19 @@ class PurchaseOrde(models.Model):
             'purchase_id': self.id,
         })
         invoice = invoice_obj.create(invoice_vals)
+        
+        self.update({'invoice_ids': [(4, invoice.id)]})
+        self.invoice_count = len(self.invoice_ids)
         self.deposit_last_liquidation_date = fields.datetime.now()
+
+        for line in self.order_line:
+            if line:
+                invoice_line = invoice.invoice_line_ids.filtered(lambda x: x.product_id.id == line.product_id.id)
+                if invoice_line:
+                    line.invoice_lines = [(4, invoice_line.id)]
+
+        self.message_post(body="Se ha creado la liquidación de compras %s" % invoice.name)
+
         return {
             'type': 'ir.actions.act_window',
             'res_model': 'account.move',