[ADD] l10n_ar_sale: Tomamos la secuencia de los talonarios

Task: 72835

Author: mem-adhoc

l10n_ar_sale/__manifest__.py

@@ -1,6 +1,6 @@
 {
     'name': 'Argentinian Sale Total Fields',
-    'version': "17.0.1.1.0",
+    'version': "17.0.1.2.0",
     'category': 'Localization/Argentina',
     'sequence': 14,
     'author': 'ADHOC SA',

l10n_ar_sale/models/sale_order.py

@@ -53,16 +53,43 @@ def _get_sale_checkbook(self):
         )
 
     @api.model_create_multi
-    def create(self, vals):
-        for val in vals:
-            if self.env.user.has_group('l10n_ar_sale.use_sale_checkbook') and \
-                val.get('name', _('New')) == _('New') and \
-                    val.get('sale_checkbook_id'):
+    def create(self, vals_list):
+        for value in vals_list:
+            if self.env.user.has_group('l10n_ar_sale.use_sale_checkbook') and value.get("sale_checkbook_id"):
+                sale_checkbook = self.env["sale.checkbook"].browse(value["sale_checkbook_id"])
+                if sale_checkbook.sequence_id:
+                    value["name"] = sale_checkbook.sequence_id._next() or _('New')
+        return super(SaleOrder, self).create(vals_list)
+
+    @api.onchange('partner_id', 'sale_checkbook_id')
+    def set_order_sequence(self):
+        """
+        Este metodo muestra en el momento de crear la OV la sequencia del talonario
+        """
+        if self.env.user.has_group('l10n_ar_sale.use_sale_checkbook') and \
+            self.sale_checkbook_id:  
                 sale_checkbook = self.env['sale.checkbook'].browse(
-                    val.get('sale_checkbook_id'))
-                val['name'] = sale_checkbook.sequence_id and\
-                    sale_checkbook.sequence_id._next() or _('New')
-        return super(SaleOrder, self).create(vals)
+                    self.sale_checkbook_id.id)
+                number_next = sale_checkbook.sequence_id.number_next_actual
+                self.name = sale_checkbook.sequence_id.get_next_char(number_next) and\
+                    sale_checkbook.sequence_id.get_next_char(number_next) or _('New')
+                
+    def write(self, vals):
+        """A sale checkbook could have a different order sequence, so we could
+        need to change it accordingly"""
+        if self.env.user.has_group('l10n_ar_sale.use_sale_checkbook') and self.sale_checkbook_id:
+            sale_checkbook = self.sale_checkbook_id
+            if sale_checkbook.sequence_id:
+                for record in self:
+                    if record.state in {"draft", "sent"} and record.type_id.sequence_id != sale_checkbook.sequence_id:
+                        number_next = sale_checkbook.sequence_id.number_next_actual
+                        new_vals = vals.copy()
+                        new_vals["name"] = sale_checkbook.sequence_id.get_next_char(number_next) or _('New')
+                        super(SaleOrder, record).write(new_vals)
+                    else:
+                        super(SaleOrder, record).write(vals)
+                return True
+        return super().write(vals)
 
     def _compute_tax_totals(self):
         """ Mandamos en contexto el invoice_date para calculo de impuesto con partner aliquot

l10n_ar_sale/views/sale_report_templates.xml

@@ -57,9 +57,9 @@
             <attribute name="t-field">line.report_price_unit</attribute>
         </xpath>
 
-        <t t-set="taxes" position="attributes">
+        <!-- <t t-set="taxes" position="attributes">
             <attribute name="t-value">', '.join([(tax.invoice_label or tax.name) for tax in line.report_tax_id])</attribute>
-        </t>
+        </t> -->
 
         <t t-set="current_subtotal" t-value="current_subtotal + line.price_subtotal" position="attributes">
             <attribute name="t-value">current_subtotal + line.report_price_subtotal</attribute>

l10n_ar_sale/views/sale_view.xml

@@ -17,7 +17,7 @@
                 <!-- <field name="currency_id" invisible="1"/> -->
             </xpath>
             <field name="payment_term_id" position="before">
-                <field name="sale_checkbook_id" domain="[('company_id','in', [company_id, False])]" groups="l10n_ar_sale.use_sale_checkbook" readonly="state not in ['draft', 'sent']"/>
+                <field name="sale_checkbook_id" domain="[('company_id','in', [company_id, False])]" groups="l10n_ar_sale.use_sale_checkbook" readonly="state not in ['draft', 'sent']" force_save="1"/>
             </field>
         </field>
     </record>