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

This reverts commit bb3bb71.

closes #397

Signed-off-by: Filoquin adhoc <maq@adhoc.com.ar>

Author: pablohmontenegro

l10n_ar_edi_ux/__manifest__.py

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

l10n_ar_edi_ux/models/res_partner.py

@@ -8,11 +8,6 @@
 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,17 +12,4 @@
         </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,5 +1,3 @@
-import re
-
 from odoo import models, api, fields, _
 from ast import literal_eval
 from odoo.exceptions import UserError
@@ -119,8 +117,6 @@ def get_fields(self):
         required=True,
     )
 
-    wizard_error = fields.Html()
-
     @api.onchange('partner_id')
     def change_partner(self):
         self.ensure_one()
@@ -173,13 +169,8 @@ def _update(self):
     def automatic_process_cb(self):
         for partner in self.partner_ids:
             self.partner_id = partner.id
-            try:
-                self.change_partner()
-                self._update()
-                self.partner_id.failed_to_update = False
-            except:
-                self.partner_id.failed_to_update = True
-                continue
+            self.change_partner()
+            self._update()
         self.write({'state': 'finished'})
         return {
             'type': 'ir.actions.act_window',
@@ -218,33 +209,16 @@ 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
-        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>'
-
+        self.change_partner()
         return {
             'type': 'ir.actions.act_window',
             'res_model': self._name,

l10n_ar_edi_ux/wizards/res_partner_update_from_padron_wizard_view.xml

@@ -49,7 +49,6 @@
                 </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.