From 18cca232956e2bbd71368840e1e676018e958255 Mon Sep 17 00:00:00 2001
From: docker-odoo <mem@adhoc.com.ar>
Date: Fri, 10 May 2024 17:21:28 +0000
Subject: [PATCH] [FIX] l10n_ar_ux: Recompute the commercial partner for edit
 de contact form in portal

Ticekt: 73707
---
 l10n_ar_ux/__manifest__.py            |  2 +-
 l10n_ar_ux/controllers/portal.py      | 15 ++++++++++++++-
 l10n_ar_ux/views/portal_templates.xml | 10 +++++-----
 3 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/l10n_ar_ux/__manifest__.py b/l10n_ar_ux/__manifest__.py
index 8711eb7b4..fcc569e49 100644
--- a/l10n_ar_ux/__manifest__.py
+++ b/l10n_ar_ux/__manifest__.py
@@ -1,6 +1,6 @@
 {
     'name': 'Argentinian Accounting UX',
-    'version': "16.0.1.11.0",
+    'version': "16.0.1.12.0",
     'category': 'Localization/Argentina',
     'sequence': 14,
     'author': 'ADHOC SA',
diff --git a/l10n_ar_ux/controllers/portal.py b/l10n_ar_ux/controllers/portal.py
index 1a0038bfd..e55588867 100644
--- a/l10n_ar_ux/controllers/portal.py
+++ b/l10n_ar_ux/controllers/portal.py
@@ -48,8 +48,21 @@ def account(self, redirect=None, **post):
         uid = request.session.uid
         partner = request.env['res.users'].browse(uid).partner_id if uid else request.env['res.partner']
         partner = partner.with_context(show_address=1).sudo()
+        #recomputamos el commercial partner porque al editar el fomrulario perdemos ese dato
+        partner._compute_commercial_partner()
+        #Al guardar por primera vez los datos del contacto, se pierden la responsabilidad y el tipo de identificacion
+        # if not partner.l10n_ar_afip_responsibility_type_id and post.get('l10n_ar_afip_responsibility_type_id'):
+        #     resp = int(post.get('l10n_ar_afip_responsibility_type_id'))
+        #     partner.l10n_ar_afip_responsibility_type_id = request.env['l10n_ar.afip.responsibility.type'].browse(resp)
+        # if not partner.l10n_latam_identification_type_id and post.get('l10n_latam_identification_type_id'):
+        #     ident = int(post.get('l10n_latam_identification_type_id'))
+        #     partner.l10n_latam_identification_type_id = request.env['l10n_latam.identification.type'].browse(ident)
+
         response.qcontext.update({
+            'l10n_ar_afip_responsibility_type_id': post.get('l10n_ar_afip_responsibility_type_id'),
+            'l10n_latam_identification_type_id': post.get('l10n_latam_identification_type_id'),
             'identification_types': identification_types,
             'afip_responsibilities': afip_responsibilities,
-            'partner': partner})
+            'partner': partner,
+            'partner_can_edit_vat': partner.can_edit_vat()})
         return response
diff --git a/l10n_ar_ux/views/portal_templates.xml b/l10n_ar_ux/views/portal_templates.xml
index b9c52b57a..ed9b12298 100644
--- a/l10n_ar_ux/views/portal_templates.xml
+++ b/l10n_ar_ux/views/portal_templates.xml
@@ -19,7 +19,7 @@
                 <label class="control-label" for="l10n_latam_identification_type_id">Document Type</label>
                 <input name="commercial_partner_id" class="form-control" t-att-value="partner.commercial_partner_id.id" type='hidden'/>
                 <t t-set="document_type" t-value="partner.commercial_partner_id.l10n_latam_identification_type_id"/>
-                <t t-if="document_type">
+                <t t-if="not partner_can_edit_vat">
                     <p class="form-control" t-out="document_type.name" readonly="1" title="Changing your document type is not allowed. Please contact us directly for this operation."/>
                     <input name="l10n_latam_identification_type_id" class="form-control" t-att-value="document_type.id" type='hidden'/>
                 </t>
@@ -27,7 +27,7 @@
                     <select name="l10n_latam_identification_type_id" class="form-control">
                         <option value="">Document Type...</option>
                         <t t-foreach="identification_types or []" t-as="document_identification">
-                            <option t-att-value="document_identification.id" t-att-selected="str(document_identification.id) == l10n_latam_identification_type_id">
+                            <option t-att-value="document_identification.id" t-att-selected="document_identification.id == int(l10n_latam_identification_type_id) if l10n_latam_identification_type_id else document_identification.id == partner.l10n_latam_identification_type_id.id">
                                 <t t-out="document_identification.name"/>
                             </option>
                         </t>
@@ -38,14 +38,14 @@
             <!-- Add document number -->
             <div t-attf-class="form-group #{error.get('vat') and 'o_has_error' or ''} col-xl-6">
                 <label class="col-form-label" for="vat">Document Number</label>
-                <input type="text" name="vat" t-attf-class="form-control #{error.get('vat') and 'is-invalid' or ''}" t-att-value="partner.commercial_partner_id.vat or vat" t-att-readonly="'1' if partner.commercial_partner_id.vat else None" t-att-title="'Changing your document number is not allowed. Please contact us directly for this operation.' if partner.commercial_partner_id.vat else None"/>
+                <input type="text" name="vat" t-attf-class="form-control #{error.get('vat') and 'is-invalid' or ''}" t-att-value="partner.commercial_partner_id.vat or vat" t-att-readonly="'1' if not partner_can_edit_vat else None" t-att-title="'Changing your document number is not allowed. Please contact us directly for this operation.' if partner.commercial_partner_id.vat else None"/>
             </div>
 
             <!-- Add AFIP Responsibility -->
             <div t-attf-class="form-group #{error.get('l10n_ar_afip_responsibility_type_id') and 'has-error' or ''} col-xl-6">
                 <label class="control-label" for="l10n_ar_afip_responsibility_type_id">AFIP Responsibility</label>
                 <t t-set="afip_responsibility" t-value="partner.commercial_partner_id.l10n_ar_afip_responsibility_type_id"/>
-                <t t-if="afip_responsibility">
+                <t t-if="not partner_can_edit_vat">
                     <p class="form-control" t-out="afip_responsibility.name" readonly="1" title="Changing your document type is not allowed. Please contact us directly for this operation."/>
                     <input name="l10n_ar_afip_responsibility_type_id" class="form-control" t-att-value="afip_responsibility.id" type='hidden'/>
                 </t>
@@ -53,7 +53,7 @@
                     <select name="l10n_ar_afip_responsibility_type_id" class="form-control">
                         <option value="">AFIP Responsibility...</option>
                         <t t-foreach="afip_responsibilities or []" t-as="afip_res">
-                            <option t-att-value="afip_res.id" t-att-selected="str(afip_res.id) == l10n_ar_afip_responsibility_type_id">
+                            <option t-att-value="afip_res.id" t-att-selected="afip_res.id == int(l10n_ar_afip_responsibility_type_id) if l10n_ar_afip_responsibility_type_id else afip_res.id == partner.l10n_ar_afip_responsibility_type_id.id">
                                 <t t-out="afip_res.name"/>
                             </option>
                         </t>