[14.0][IMP] l10n_it_fatturapa_in: intermediary contact creation depending by vat #4621

Chionne27 · 2025-02-17T14:42:39Z

No description provided.

francesco-ooops

Errata approvazione

francesco-ooops

Functional ok

sergiocorato · 2025-02-19T17:53:32Z

l10n_it_fatturapa_in/security/ir.model.access.csv

@@ -6,7 +6,7 @@ access_einvoice_line_other_data,access_einvoice_line_other_data,model_einvoice_l
 access_wizard_import_fatturapa,access_wizard_import_fatturapa,model_wizard_import_fatturapa,account.group_account_invoice,1,1,1,1
 access_wizard_link_to_invoice,access_wizard_link_to_invoice,model_wizard_link_to_invoice,account.group_account_invoice,1,1,1,1
 access_wizard_link_to_invoice_line,access_wizard_link_to_invoice_line,model_wizard_link_to_invoice_line,account.group_account_invoice,1,1,1,1
-access_wizard_check_intermediary_all,access_wizard_check_intermediary_all,model_wizard_check_intermediary,,1,1,1,1
+access_wizard_check_intermediary_all,access_wizard_check_intermediary_all,model_wizard_check_intermediary,account.group_account_invoice,1,1,1,1


Credo che questa modifica la volessi mettere nell'altro commit

Ho modificato i commit. Ti ringrazio per avermelo fatto natare

sergiocorato · 2025-02-19T17:57:37Z

l10n_it_fatturapa_in/wizard/wizard_check_intermediary_vat.py

+    _name = "wizard.check.intermediary"
+    _description = "Controllo sull'intermediario"
+
+    message = fields.Text(string="message", readonly=True)


Di norma le stringe vanno con tutte le iniziali maiuscole ed in inglese, verranno poi tradotte, vedi https://github.com/OCA/currency/blob/5621d4578f79a1be38bacd597470255943f4298c/account_cryptocurrency/models/res_currency_move.py#L62

Sistemato. Ti ringrazio

GSLabIt · 2025-02-20T09:02:25Z

l10n_it_fatturapa_in/wizard/wizard_check_intermediary_vat.xml

+                    </group>
+                    <footer>
+                        <button
+                            string="Sì"


Suggested change

string="Sì"

string="Confirm"

GSLabIt · 2025-02-20T09:02:29Z

l10n_it_fatturapa_in/wizard/wizard_check_intermediary_vat.xml

+                            class="btn-primary"
+                        />
+                        <button
+                            string="No"


Suggested change

string="No"

string="Cancel"

GSLabIt · 2025-02-20T09:10:10Z

l10n_it_fatturapa_in/wizard/wizard_import_fatturapa.py

+                    if vat_check:
+                        invoice.write({"intermediary": Intermediary_id})
+                    else:
+                        name = (


Quando arriva qui il partner è stato già creato, o sbaglio?

Se intendi il cliente della fattura si esiste già in quanto dalla fattura elettronica il cliente si associa mediante la partita IVA.

@Chionne27 stiamo parlando dell'intermediario, che è comunque un partner (aka contatto)

@GSLabIt Hai ragione. l'implementazione creava il partner prima dell'apertura del wizard. Pertanto ho effettuato una leggera modifica per cui se l'utente non lo vuole creare, il partner creato prima viene eliminato. Mi sembra un metodo abbastanza furbo poi se si può migliorare sono a disposizione per un confronto.

Evitare di crearlo? Penso sarebbe meglio, non trovi?

return {
"type": "ir.actions.act_window",
"name": "Confirm action",
"res_model": "wizard.check.intermediary",
"view_mode": "form",
"target": "new",
"context": {
"default_message": f'The contact intermediary "{name}" '
"does not exists in the system. Do you want to create it?",
"intermediary_id": Intermediary.DatiAnagrafici,
"invoice_id": invoice.id,
},
}

È vero. Infatti volevo chiamare la funzione getPartnerBase nella funzione di conferma del wizard. Il punto è che con il context non viene passato correttamente Intermediary.DatiAnagrafici come ho fatto nell'esempio sopra perché invece di un oggetto passa una stringa e i metodo getPartnerBase fallisce. Quindi per il momento ho fatto questa implementazione. Se hai un idea per fare come detto poi la implemento.

Sta cosa è un pò tricky, perché può capitare che il partner sia stato già creato precedentemente e non è corretto eliminarlo.

Qual'è il problema esatto che si vuole risolvere?

Qualsiasi volta il sistema cerca di creare un partner chiedere conferma all'utente

Per come è fatto ora non mi pare fattibile, bisognerebbe andare a chiedere all'utente prima della creazione del partner e se è presente l'intermediario. A questo punto probabilmente è meglio aggiungere un flag nella wizard principale; alternativa è rivedere il flusso attuale

primes2h · 2025-02-20T09:40:24Z

l10n_it_fatturapa_in/wizard/wizard_check_intermediary_vat.py

+    intermediary_id = fields.Integer(string="Intermediario", readonly=True)
+    invoice_id = fields.Many2one("account.move", string="Fattura", readonly=True)


Queste andrebbero in inglese.

primes2h · 2025-02-20T09:41:01Z

l10n_it_fatturapa_in/wizard/wizard_check_intermediary_vat.xml

+        <field name="name">check.intermediary.vat.form</field>
+        <field name="model">wizard.check.intermediary</field>
+        <field name="arch" type="xml">
+            <form string="Verifica intermediario">


Anche questa.

primes2h · 2025-02-20T09:41:55Z

l10n_it_fatturapa_in/wizard/wizard_import_fatturapa.py

+                                "default_message": f'Il contatto intermediario "{name}" '
+                                "non esiste nel sistema. Confermi di volerlo creare?",


Come sopra.

…y vat

francesco-ooops approved these changes Feb 17, 2025

View reviewed changes

francesco-ooops self-requested a review February 17, 2025 15:34

francesco-ooops mentioned this pull request Feb 17, 2025

l10n_it_fatturapa_in - scelta creazione contatto intermediario #4622

Open

2 tasks

francesco-ooops linked an issue Feb 17, 2025 that may be closed by this pull request

l10n_it_fatturapa_in - scelta creazione contatto intermediario #4622

Open

2 tasks

Chionne27 force-pushed the 14.0-IMP-l10n_it_fatturapa_in branch 4 times, most recently from ed30f19 to c28b517 Compare February 19, 2025 08:15

francesco-ooops approved these changes Feb 19, 2025

View reviewed changes

sergiocorato requested changes Feb 19, 2025

View reviewed changes

Chionne27 force-pushed the 14.0-IMP-l10n_it_fatturapa_in branch 3 times, most recently from 37f7831 to f6b31d2 Compare February 20, 2025 08:31

GSLabIt reviewed Feb 20, 2025

View reviewed changes

primes2h requested changes Feb 20, 2025

View reviewed changes

Chionne27 force-pushed the 14.0-IMP-l10n_it_fatturapa_in branch from f6b31d2 to 1e7be0b Compare February 24, 2025 15:31

Chionne27 added 2 commits February 25, 2025 14:31

[IMP] l10n_it_fatturapa_in: intermediary contact creation depending b…

940238a

…y vat

[IMP] l10n_it_fatturapa_in_purchase: test with intermediary wizard check

4c3b6da

Chionne27 force-pushed the 14.0-IMP-l10n_it_fatturapa_in branch from 1e7be0b to 4c3b6da Compare February 25, 2025 13:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[14.0][IMP] l10n_it_fatturapa_in: intermediary contact creation depending by vat #4621

[14.0][IMP] l10n_it_fatturapa_in: intermediary contact creation depending by vat #4621

Chionne27 commented Feb 17, 2025

francesco-ooops left a comment •

edited

Loading

francesco-ooops left a comment

sergiocorato Feb 19, 2025

Chionne27 Feb 20, 2025

sergiocorato Feb 19, 2025

Chionne27 Feb 20, 2025

GSLabIt Feb 20, 2025

GSLabIt Feb 20, 2025

GSLabIt Feb 20, 2025

Chionne27 Feb 24, 2025 •

edited

Loading

francesco-ooops Feb 24, 2025 •

edited

Loading

Chionne27 Feb 25, 2025 •

edited

Loading

GSLabIt Feb 25, 2025

Chionne27 Feb 25, 2025

GSLabIt Feb 25, 2025

francesco-ooops Feb 25, 2025

GSLabIt Feb 25, 2025

primes2h Feb 20, 2025

primes2h Feb 20, 2025

primes2h Feb 20, 2025

		intermediary_id = fields.Integer(string="Intermediario", readonly=True)
		invoice_id = fields.Many2one("account.move", string="Fattura", readonly=True)

		"default_message": f'Il contatto intermediario "{name}" '
		"non esiste nel sistema. Confermi di volerlo creare?",

[14.0][IMP] l10n_it_fatturapa_in: intermediary contact creation depending by vat #4621

Are you sure you want to change the base?

[14.0][IMP] l10n_it_fatturapa_in: intermediary contact creation depending by vat #4621

Conversation

Chionne27 commented Feb 17, 2025

francesco-ooops left a comment • edited Loading

Choose a reason for hiding this comment

francesco-ooops 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

Chionne27 Feb 24, 2025 • edited Loading

Choose a reason for hiding this comment

francesco-ooops Feb 24, 2025 • edited Loading

Choose a reason for hiding this comment

Chionne27 Feb 25, 2025 • edited Loading

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

francesco-ooops left a comment •

edited

Loading

Chionne27 Feb 24, 2025 •

edited

Loading

francesco-ooops Feb 24, 2025 •

edited

Loading

Chionne27 Feb 25, 2025 •

edited

Loading