[ADD] l10n_ar_edi_ux: mejora de rendimiento en actualizacion masiva de partners

closes #358

Task: 37730
Signed-off-by: Katherine Zaoral <kz@adhoc.com.ar>

Author: mem-adhoc

l10n_ar_edi_ux/__manifest__.py

@@ -1,6 +1,6 @@
 {
     'name': 'Argentinian Electronic Invoicing UX',
-    'version': "16.0.1.9.0",
+    'version': "16.0.1.10.0",
     'category': 'Localization/Argentina',
     'sequence': 14,
     'author': 'ADHOC SA',

l10n_ar_edi_ux/models/res_partner.py

@@ -8,6 +8,11 @@
 class ResPartner(models.Model):
     _inherit = 'res.partner'
 
+
+    failed_to_update = fields.Boolean(
+        string="Falló actualización AFIP",
+        tracking=True)
+
     def update_partner_data_from_afip(self):
         """ Funcion que llama al wizard para actualizar data de partners desde afip sin abrir wizard.
         Podríamos mejorar  pasando un argumento para sobreescribir o no valores que esten o no definidos

l10n_ar_edi_ux/views/res_partner_view.xml

@@ -12,4 +12,17 @@
         </field>
     </record>
 
+    <record id="res_partner_view_search" model="ir.ui.view">
+        <field name="name">res.partner.search.inherit</field>
+        <field name="model">res.partner</field>
+        <field name="inherit_id" ref="base.view_res_partner_filter"/>
+        <field name="arch" type="xml">
+            <xpath expr="//filter[@name='l10n_ar_afip_responsibility_type_id_filter']" position="after">
+                <group>
+                    <filter string="Falló actualización AFIP" name="failed_to_update" domain="[('failed_to_update', '=', True), ('country_code', '=', 'AR')]"/>
+                </group>
+            </xpath>
+            <separator/>
+        </field>
+    </record>
 </odoo>

l10n_ar_edi_ux/wizards/res_partner_update_from_padron_wizard.py

@@ -1,3 +1,5 @@
+import re
+
 from odoo import models, api, fields, _
 from ast import literal_eval
 from odoo.exceptions import UserError
@@ -117,6 +119,8 @@ def get_fields(self):
         required=True,
     )
 
+    wizard_error = fields.Html()
+
     @api.onchange('partner_id')
     def change_partner(self):
         self.ensure_one()
@@ -169,8 +173,13 @@ def _update(self):
     def automatic_process_cb(self):
         for partner in self.partner_ids:
             self.partner_id = partner.id
-            self.change_partner()
-            self._update()
+            try:
+                self.change_partner()
+                self._update()
+                self.partner_id.failed_to_update = False
+            except:
+                self.partner_id.failed_to_update = True
+                continue
         self.write({'state': 'finished'})
         return {
             'type': 'ir.actions.act_window',
@@ -209,16 +218,33 @@ def _next_screen(self):
             values.update({
                 'partner_id': partner.id,
                 'state': 'selection',
+                'wizard_error': False,
             })
         else:
             values.update({
                 'state': 'finished',
+                'wizard_error': False,
+                'partner_id': False,
             })
 
         self.write(values)
         # because field is not changed, view is distroyed and reopen, on change
         # is not called an we call it manually
-        self.change_partner()
+        try:
+            self.change_partner()
+            self.partner_id.failed_to_update = False
+        except (UserError) as exp:
+            self.partner_id.failed_to_update = True
+
+            #Procesamos el error que nos devuelve afip
+            patron = re.compile(r"'error': \['(.*?)'\]")
+            resultado = patron.search(str(exp))
+            if resultado:
+                error = resultado.group(1)
+            else:
+                error = str(exp)
+            self.wizard_error = '<div class= "alert alert-warning" role="alert" style="margin-bottom:0px;" >' + error + '</div>'
+
         return {
             'type': 'ir.actions.act_window',
             'res_model': self._name,

l10n_ar_edi_ux/wizards/res_partner_update_from_padron_wizard_view.xml

@@ -49,6 +49,7 @@
                 </header>
                 <sheet>
                     <h2 attrs="{'invisible': [('state', '!=', 'finished')]}">There are no more partners to update for this request...</h2>
+                    <field name="wizard_error" readonly="1" attrs="{'invisible': not wizard_error}"/>
                     <p class="oe_grey" attrs="{'invisible': [('state', '!=', ('option'))]}">
                         Only Partners with cuit are going to be updated.<br/>
                         Select the list of fields you want to update.