[14.0][MIG] l10n_it_dichiarazione_intento

[SimoRubi]

Migrazione da v12 (https://github.com/OCA/l10n-italy/tree/f8ffdba23af2a8dfd863544d8bb311c555079bf1/l10n_it_dichiarazione_intento)

cose da fare:

• test automatici
• rebase
• script di migrazione v13 (account.invoice* -> account.move*)
  Fatti direttamente verso i nuovi nomi in inglese
• rinominare il modulo in l10n_it_declaration_of_intent + migrazione con update_module_names

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

[andreampiovesana]

forse bisogna rinominare il modulo in inglese?
l10n_it_declaration_of_intent

[SimoRubi]

forse bisogna rinominare il modulo in inglese?
l10n_it_declaration_of_intent

Sì è una delle cose che devo fare.
Prima punto a farlo funzionare, poi lo rinomino con script di migrazione

[eLBati]

#1938 (comment)

[SimoRubi]

#1938 (comment)

sì grazie, come ho scritto sopra vorrei prima arrivare a far funzionare il modulo così com'è, poi lo rinomino.
Ho aggiunto in descrizione le cose che rimangono da fare

[TheMule71]

Ciao, non ti faccio la PR perché sono due modifiche banali:

probabilmente sia la valuta che il plafond corretti sono quelli della company corrent (scelta dal dropdown) e non quelli della company di default dell'utente corrente.

diff --git a/l10n_it_declaration_of_intent/models/declaration.py b/l10n_it_declaration_of_intent/models/declaration.py
index 8e7e20f5c6..d59d852f2c 100644
--- a/l10n_it_declaration_of_intent/models/declaration.py
+++ b/l10n_it_declaration_of_intent/models/declaration.py
@@ -41,7 +41,7 @@ class DeclarationOfIntent(models.Model):
 
     @api.model
     def _default_currency(self):
-        return self.env.user.company_id.currency_id
+        return self.env.company.currency_id
 
     number = fields.Char(copy=False)
     date = fields.Date(required=True)
@@ -94,7 +94,7 @@ class DeclarationOfIntent(models.Model):
         # Declaration issued by company are "IN"
         if values.get("type", False) == "in":
             year = fields.Date.to_date(values["date_start"]).strftime("%Y")
-            plafond = self.env.user.company_id.declaration_yearly_limit_ids.filtered(
+            plafond = self.env.company.declaration_yearly_limit_ids.filtered(
                 lambda r: r.year == year
             )
             if not plafond:
diff --git a/l10n_it_declaration_of_intent/tests/test_declaration_of_intent.py b/l10n_it_declaration_of_intent/tests/test_declaration_of_intent.py
index aa37cfc1a3..f4ce368202 100644
--- a/l10n_it_declaration_of_intent/tests/test_declaration_of_intent.py
+++ b/l10n_it_declaration_of_intent/tests/test_declaration_of_intent.py
@@ -187,7 +187,7 @@ class TestDeclarationOfIntent(SavepointCase):
             {
                 "year": self.today_date.year,
                 "limit_amount": 50000.0,
-                "company_id": self.env.user.company_id.id,
+                "company_id": self.env.company.id,
             }
         )
         self.declaration4 = self._create_declaration(self.partner4, "in")

[SimoRubi]

Grazie, ho corretto.
A meno di casi eccezionali dove vengono passate cose nel contesto (vedi https://github.com/odoo/odoo/blob/9b7a4e1815de580575bed6657f9f8915907ce534/odoo/api.py#L555) i due valori coincidono, ma concordo che è più corretto prendere la company direttamente dal environment

[TheMule71]

Grazie, ho corretto.
A meno di casi eccezionali dove vengono passate cose nel contesto (vedi https://github.com/odoo/odoo/blob/9b7a4e1815de580575bed6657f9f8915907ce534/odoo/api.py#L555) i due valori coincidono, ma concordo che è più corretto prendere la company direttamente dal environment

Non credo sia così raro nella 14. Se quello che dice il commento è vero:
https://github.com/odoo/odoo/blob/9b7a4e1815de580575bed6657f9f8915907ce534/odoo/api.py#L595-L597
avere le company specificate nel contesto è la norma...

[TheMule71]

LGTM

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[SimoRubi]

@OCA/local-italy-maintainers che dite si può mergiare?
Ho provato a fare rebase per includere le nuove modifiche di v12 ma viene un macello visto anche il renaming fatto in questa PR, se qualcuno vuole cimentarsi comunque può fare una PR al branch di questa PR.

Le issue da portare al momento sono:

• l10n_it_dichiarazione_intento: Il numero del documento del partner non è più obbligatorio #2329
• l10n_it_dichiarazione_intento: poter creare più dichiarazioni d'intento oltre il plafond disponibile #2404
• l10n_it_dichiarazione_intento: duplicazione dichiarazione di intento di tipo in #2415

[TheMule71]

/ocabot merge nobump

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-2043-by-TheMule71-bump-nobump, awaiting test results.

[OCA-git-bot]

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