[12.0] REF l10n_it_website_portal_fatturapa_sale for higher control #1556

eLBati · 2019-12-07T20:36:39Z

Refactored without nested ajax calls to get more control and handle anonymous requests

Add support for signature flow

Add the possibility to configure "Mandatory e-invoicing for portal customers"

FIX Accessing to online quote with anonymous user:
odoo.exceptions.AccessError: ('The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Contact, Operation: read) - (Records: [7], User: 4)', None)
Error to render compiling AST
AccessError: ('The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Contact, Operation: read) - (Records: [7], User: 4)', None)
Template: payment.payment_tokens_list
Path: /templates/t/form/div[2]/label/input
Node:
I request to receive electronic invoice - - -

Also FIX:

And:

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

mrcast · 2019-12-10T13:33:44Z

@eLBati Passi fatti:

da backend creato un partner Mario Rossi, inseriti i dati anagrafici tranne codice fiscale
fatta offerta
inviata via email
lato portale cliccato su Accetta Paga ho messo flag su "I request to receive electronic invoice"
mi viene chiesto login
dato accesso di portale e seguito link per registrarmi
rientrato su offerta e fatto Accetta Paga e ho messo flag su "I request to receive electronic invoice"
vengo portato alla compilazione dati profilo perchè manca il codice fiscale ma:

CF e partita IVA non sono editabili
mi viene richiesto obbligatoriamente il telefono (è necessario?)

Accessing to online quote with anonymous: odoo.exceptions.AccessError: ('The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Contact, Operation: read) - (Records: [7], User: 4)', None) Error to render compiling AST AccessError: ('The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Contact, Operation: read) - (Records: [7], User: 4)', None) Template: payment.payment_tokens_list Path: /templates/t/form/div[2]/label/input Node: <input type="checkbox" name="electronic_invoice_subjected" t-att-checked="'checked' if request.env['res.partner'].browse(partner_id).electronic_invoice_subjected else None"/> I request to receive electronic invoice - - - Refactored without nested ajax calls to get more control and handle anonymous requests Add support for signature flow

…onfigure "Mandatory e-invoicing for portal customers" Avoid to block TIN and FC editing for sent quotations

eLBati · 2019-12-11T15:54:39Z

@mrcast dovrei aver risolto

mrcast

Funzionalità OK
grazie

eLBati · 2019-12-11T16:59:53Z

@SimoRubi vuoi dare un'occhiata?

eLBati · 2019-12-13T13:12:54Z

Essendo modulo beta, intanto faccio merge.

eLBati · 2019-12-13T13:13:16Z

/ocabot merge minor

OCA-git-bot · 2019-12-13T13:13:28Z

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-1556-by-eLBati-bump-minor, awaiting test results.

Signed-off-by eLBati

OCA-git-bot · 2019-12-13T14:22:31Z

Congratulations, your PR was merged at 3cc9c7a. Thanks a lot for contributing to OCA. ❤️

SimoRubi

Purtroppo sono riuscito a dare un'occhiata solo ora, anche se la PR è già mergiata, ho trovato un paio di cose su cui ho dei dubbi.

SimoRubi · 2019-12-18T15:22:43Z

l10n_it_website_portal_fatturapa_sale/models/res_config_settings.py

+
+    portal_mandatory_e_invoicing = fields.Boolean(
+        related='company_id.portal_mandatory_e_invoicing',
+        string="Mandatory e-invoicing for portal customers", readonly=False)


Meglio rimuovere l'attributo string: viene recuperato automaticamente dal campo related (https://github.com/odoo/odoo/blob/81f4e69c27afa8e0fa55f8870ca4788d87596b37/odoo/fields.py#L238).

SimoRubi · 2019-12-18T16:25:35Z

l10n_it_website_portal_fatturapa_sale/models/res_partner.py

+    SaleOrder = self.env['sale.order']
+    has_so = SaleOrder.search([
+        ('partner_id', 'child_of', self.commercial_partner_id.id),
+        ('state', 'in', ['sale', 'done'])  # removing 'sent'


Forse sbaglio, ma mi sembra che questo metodo non possa mai cambiare il risultato di super..
Il codice qui è lo stesso del metodo in super, l'unica modifica è che recuperi meno ordini (solo ordini sale e done invece di sent, sale e done).
Se il super trova un sale.order in stato sent, sale o done, ritorna False quindi qui non ci arriviamo nemmeno, a cosa serve fare di nuovo questa ricerca su un sottinsieme degli ordini già cercati in super?
Mi potresti chiarire a cosa serve questo metodo in pratica?

SimoRubi · 2019-12-18T16:26:43Z

l10n_it_website_portal_fatturapa_sale/models/res_partner.py

-            })
-        except ValidationError:
-            return False
+ResPartner.can_edit_vat = can_edit_vat


Come mai non usi l'ereditarietà di Odoo?

Proprio per ovviare al problema che descrivi in https://github.com/OCA/l10n-italy/pull/1556/files/497e657ae470f013c01297c3ffb05944cc68c5f9#r359441273

In pratica sto sostituendo la can_edit_vat di from odoo.addons.sale.models.res_partner.ResPartner con la mia; quella del modulo sale, per come è scritta nel modulo sale non viene più eseguita.

Questo perchè, per come è strutturata la can_edit_vat, l'unica alternativa era fare un override senza super (come dici tu, se facessi un override normale, il mio codice non verrebbe mai eseguito).
Allora ho seguito la strada di "patchare" la can_edit_vat di sale

Avevo capito che c'era un problema ma non che c'era anche già una soluzione, beh dai meglio che niente, grazie!

SimoRubi · 2019-12-18T17:05:43Z

l10n_it_website_portal_fatturapa_sale/views/payment_templates.xml

@@ -9,11 +9,23 @@
              name="Add e-invoice subjected in Payment Tokens list"
              inherit_id="payment.payment_tokens_list">
        <xpath expr="//div[hasclass('card')]" position="after">
-            <div class="form-check">
+            <div class="form-check" t-att-style="'display:none;' if (sale_order and sale_order.company_id.portal_mandatory_e_invoicing) else ''">


Se ho ben capito, questo dovrebbe far sì che abilitando portal_mandatory.. non viene visualizzato il check I request to receive.. ma 'sotto-sotto' il check è True.
Se è così, non mi pare funzionare (ho provato nel runbot della PR):

Attenzione che questo modulo opera sugli ordini "portal", quindi i preventivi online, e non sugli ordini dell'e-commerce

Giusto, ho provato su un ordine portale ed è tutto ok, grazie!

eLBati added the needs review label Dec 7, 2019

eLBati force-pushed the 12.0-fix-l10n_it_website_portal_fatturapa_sale branch 8 times, most recently from f768e52 to 8c758e0 Compare December 9, 2019 18:29

eLBati added 4 commits December 11, 2019 16:52

ADD electronic_invoice_data_complete to l10n_it_fatturapa

e715503

IMP l10n_it_website_portal_ipa: auto_install and fix web page layout

5b5b5fb

IMP l10n_it_website_portal_fatturapa_sale adding the possibility to c…

497e657

…onfigure "Mandatory e-invoicing for portal customers" Avoid to block TIN and FC editing for sent quotations

eLBati force-pushed the 12.0-fix-l10n_it_website_portal_fatturapa_sale branch from 8c758e0 to 497e657 Compare December 11, 2019 15:53

mrcast approved these changes Dec 11, 2019

View reviewed changes

OCA-git-bot added a commit that referenced this pull request Dec 13, 2019

Merge PR #1556 into 12.0

5ab6496

Signed-off-by eLBati

OCA-git-bot merged commit 497e657 into OCA:12.0 Dec 13, 2019

OCA-git-bot added the merged 🎉 label Dec 13, 2019

SimoRubi reviewed Dec 18, 2019

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[12.0] REF l10n_it_website_portal_fatturapa_sale for higher control #1556

[12.0] REF l10n_it_website_portal_fatturapa_sale for higher control #1556

eLBati commented Dec 7, 2019 •

edited

Loading

mrcast commented Dec 10, 2019

eLBati commented Dec 11, 2019

mrcast left a comment

eLBati commented Dec 11, 2019

eLBati commented Dec 13, 2019

eLBati commented Dec 13, 2019

OCA-git-bot commented Dec 13, 2019

OCA-git-bot commented Dec 13, 2019

SimoRubi left a comment

SimoRubi Dec 18, 2019

SimoRubi Dec 18, 2019

SimoRubi Dec 18, 2019

eLBati Dec 19, 2019

SimoRubi Dec 20, 2019

SimoRubi Dec 18, 2019

eLBati Dec 19, 2019

SimoRubi Dec 20, 2019

[12.0] REF l10n_it_website_portal_fatturapa_sale for higher control #1556

[12.0] REF l10n_it_website_portal_fatturapa_sale for higher control #1556

Conversation

eLBati commented Dec 7, 2019 • edited Loading

mrcast commented Dec 10, 2019

eLBati commented Dec 11, 2019

mrcast left a comment

Choose a reason for hiding this comment

eLBati commented Dec 11, 2019

eLBati commented Dec 13, 2019

eLBati commented Dec 13, 2019

OCA-git-bot commented Dec 13, 2019

OCA-git-bot commented Dec 13, 2019

SimoRubi left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

eLBati commented Dec 7, 2019 •

edited

Loading