porting l10n_it_vat_statement_communication to 14.0

l10n_it_vat_statement_communication/__manifest__.py

@@ -1,18 +1,18 @@
 # Copyright 2017 Alessandro Camilli - Openforce
-# Copyright 2017-2019 Lorenzo Battistini
+# Copyright 2017-2021 Lorenzo Battistini
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 {
     "name": "ITA - Comunicazione liquidazione IVA",
     "summary": "Comunicazione liquidazione IVA ed esportazione file xml"
     "conforme alle specifiche dell'Agenzia delle Entrate",
-    "version": "12.0.1.6.0",
+    "version": "14.0.1.0.0",
     "category": "Account",
-    "author": "Openforce di Camilli Alessandro, " "Odoo Community Association (OCA)",
+    "author": "Openforce di Camilli Alessandro, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/l10n-italy",
     "license": "AGPL-3",
     "depends": [
         "account_vat_period_end_statement",
-        "l10n_it_codici_carica",
+        "l10n_it_appointment_code",
         "l10n_it_fiscalcode",
     ],
     "data": [

l10n_it_vat_statement_communication/migrations/14.0.1.0.0/noupdate_changes.xml

@@ -0,0 +1,8 @@
+<?xml version='1.0' encoding='utf-8' ?>
+<odoo>
+    <record id="comunucazione_liquidazione_iva_multi_company" model="ir.rule">
+        <field
+            name="domain_force"
+        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+    </record>
+</odoo>

l10n_it_vat_statement_communication/migrations/14.0.1.0.0/post-migration.py

@@ -0,0 +1,10 @@
+from openupgradelib import openupgrade  # pylint: disable=W7936
+
+
+@openupgrade.migrate()
+def migrate(env, version):
+    openupgrade.load_data(
+        env.cr,
+        "l10n_it_vat_statement_communication",
+        "migrations/14.0.1.0.0/noupdate_changes.xml",
+    )

l10n_it_vat_statement_communication/models/comunicazione_liquidazione.py

@@ -20,7 +20,7 @@ class ComunicazioneLiquidazione(models.Model):
 
     @api.model
     def _default_company(self):
-        company_id = self._context.get("company_id", self.env.user.company_id.id)
+        company_id = self._context.get("company_id", self.env.company.id)
         return company_id
 
     @api.constrains("identificativo")
@@ -34,7 +34,6 @@ def _check_identificativo(self):
                 )
             )
 
-    @api.multi
     def _compute_name(self):
         for dich in self:
             name = ""
@@ -64,7 +63,7 @@ def _get_identificativo(self):
     )
     identificativo = fields.Integer(string="Identifier", default=_get_identificativo)
     name = fields.Char(string="Name", compute="_compute_name")
-    year = fields.Integer(string="Year", required=True, size=4)
+    year = fields.Integer(string="Year", required=True)
     last_month = fields.Integer(string="Last month")
     liquidazione_del_gruppo = fields.Boolean(string="Group's statement")
     taxpayer_vat = fields.Char(string="Vat", required=True)
@@ -75,7 +74,7 @@ def _get_identificativo(self):
     )
     declarant_fiscalcode = fields.Char(string="Declarant Fiscalcode")
     declarant_fiscalcode_company = fields.Char(string="Fiscalcode company")
-    codice_carica_id = fields.Many2one("codice.carica", string="Role code")
+    codice_carica_id = fields.Many2one("appointment.code", string="Role code")
     declarant_sign = fields.Boolean(string="Declarant sign", default=True)
 
     delegate_fiscalcode = fields.Char(string="Delegate Fiscalcode")
@@ -100,7 +99,6 @@ def create(self, vals):
         comunicazione._validate()
         return comunicazione
 
-    @api.multi
     def write(self, vals):
         super(ComunicazioneLiquidazione, self).write(vals)
         for comunicazione in self:
@@ -468,7 +466,6 @@ class ComunicazioneLiquidazioneVp(models.Model):
     _name = "comunicazione.liquidazione.vp"
     _description = "VAT statement communication - VP table"
 
-    @api.multi
     @api.depends("iva_esigibile", "iva_detratta")
     def _compute_VP6_iva_dovuta_credito(self):
         for quadro in self:
@@ -479,7 +476,6 @@ def _compute_VP6_iva_dovuta_credito(self):
             else:
                 quadro.iva_dovuta_credito = quadro.iva_detratta - quadro.iva_esigibile
 
-    @api.multi
     @api.depends(
         "iva_dovuta_debito",
         "iva_dovuta_credito",
@@ -631,7 +627,6 @@ def _compute_imponibile_operazioni_passive(self, liq, period):
             )
             self.imponibile_operazioni_passive -= tax.base_balance
 
-    @api.multi
     @api.onchange("liquidazioni_ids")
     def compute_from_liquidazioni(self):
 

l10n_it_vat_statement_communication/security/ir.model.access.csv

@@ -1,3 +1,4 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 account_comunicazione_liquidazione_manager,comunicazione.liquidazione,model_comunicazione_liquidazione,account.group_account_manager,1,1,1,1
 account_comunicazione_liquidazione_vp_manager,comunicazione.liquidazione.vp,model_comunicazione_liquidazione_vp,account.group_account_manager,1,1,1,1
+account_comunicazione_liquidazione_export_file_manager,comunicazione.liquidazione.export.file,model_comunicazione_liquidazione_export_file,account.group_account_manager,1,1,1,1

l10n_it_vat_statement_communication/security/security.xml

@@ -7,7 +7,7 @@
         <field name="global" eval="True" />
         <field
             name="domain_force"
-        >['|',('company_id','=',False),('company_id','child_of',[user.company_id.id])]</field>
+        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
     </record>
 
 </data>

l10n_it_vat_statement_communication/wizard/export_file.py

@@ -1,6 +1,6 @@
 import base64
 
-from odoo import _, api, exceptions, fields, models
+from odoo import _, exceptions, fields, models
 
 
 class ComunicazioneLiquidazioneExportFile(models.TransientModel):
@@ -10,7 +10,6 @@ class ComunicazioneLiquidazioneExportFile(models.TransientModel):
     file_export = fields.Binary("File", readonly=True)
     name = fields.Char("File Name", readonly=True, default="liquidazione.xml")
 
-    @api.multi
     def export(self):
 
         comunicazione_ids = self._context.get("active_ids")
@@ -37,7 +36,6 @@ def export(self):
             view_id = view_rec and view_rec[1] or False
 
             return {
-                "view_type": "form",
                 "view_id": [view_id],
                 "view_mode": "form",
                 "res_model": "comunicazione.liquidazione.export.file",

l10n_it_vat_statement_communication/wizard/export_file_view.xml

@@ -45,7 +45,6 @@
     <record id="action_liquidazione_export_file" model="ir.actions.act_window">
         <field name="name">Export VAT statement communication file</field>
         <field name="res_model">comunicazione.liquidazione.export.file</field>
-        <field name="view_type">form</field>
         <field name="view_mode">form</field>
         <field name="view_id" ref="wizard_liquidazione_export_file" />
         <field name="target">new</field>