[MIG] account_move_line_cash_basis_report: Migration to 12.0

Author: kaynnan

account_move_line_cash_basis_report/__manifest__.py

@@ -5,7 +5,7 @@
     "name": "Account Move Line Cash Basis Report",
     "summary": """
         Account Move Line Cash Basis Report""",
-    "version": "14.0.1.0.0",
+    "version": "12.0.1.0.0",
     "license": "AGPL-3",
     "author": "Escodoo",
     "website": "https://github.com/Escodoo/account-addons",

account_move_line_cash_basis_report/models/account_move_line_cash_basis_report.py

@@ -2,7 +2,9 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
 
-from odoo import fields, models
+from psycopg2.extensions import AsIs
+
+from odoo import api, fields, models, tools
 
 
 class AccountMoveLineCashBasisReport(models.Model):
@@ -126,7 +128,6 @@ class AccountMoveLineCashBasisReport(models.Model):
     account_internal_group = fields.Selection(
         selection="_selection_account_internal_group", readonly=True
     )
-    account_root_id = fields.Many2one("account.root", readonly=True)
     state = fields.Selection(
         selection="_selection_parent_state",
     )
@@ -164,17 +165,15 @@ def _selection_account_internal_group(self):
             allfields=["internal_group"]
         )["internal_group"]["selection"]
 
-    @property
-    def _table_query(self):
-        return """
-             WITH moves AS (
+    @api.model_cr
+    def init(self):
+        query = """
+            WITH moves AS (
              SELECT
                  "account_move_line".id as id,
                  "account_move_line".move_id as move_id,
                  "account_move_line".name as name,
                  "account_move_line".ref as ref,
-                 "account_move_line".parent_state as parent_state,
-                 "account_move_line".parent_state as state,
                  "account_move_line".company_id as company_id,
                  "account_move_line".currency_id as currency_id,
                  "account_move_line".journal_id as journal_id,
@@ -192,12 +191,10 @@ def _table_query(self):
                  "account_move_line".reconciled as reconciled,
                  "account_move_line".full_reconcile_id as full_reconcile_id,
                  "account_move_line".statement_id as statement_id,
-                 "account_move_line".account_root_id as account_root_id,
                  "account".user_type_id as user_type_id,
                  "account".group_id as account_group_id,
                  "account_type".type as account_internal_type,
-                 "account_type".internal_group as account_internal_group,
-                 "account_move_line".exclude_from_invoice_tab
+                 "account_type".internal_group as account_internal_group
              FROM ONLY account_move_line
              JOIN ONLY account_account account ON
                 "account_move_line".account_id = account.id
@@ -245,8 +242,6 @@ def _table_query(self):
                  "account_move_line".move_id as move_id,
                  "account_move_line".name as name,
                  "account_move_line".ref as ref,
-                 "account_move_line".parent_state as parent_state,
-                 "account_move_line".parent_state as state,
                  "account_move_line".company_id as company_id,
                  "account_move_line".currency_id as currency_id,
                  "account_move_line".journal_id as journal_id,
@@ -266,12 +261,10 @@ def _table_query(self):
                  "account_move_line".reconciled as reconciled,
                  "account_move_line".full_reconcile_id as full_reconcile_id,
                  "account_move_line".statement_id as statement_id,
-                 "account_move_line".account_root_id as account_root_id,
                  "account".user_type_id as user_type_id,
                  "account".group_id as account_group_id,
                  "account_type".type as account_internal_type,
-                 "account_type".internal_group as account_internal_group,
-                 "account_move_line".exclude_from_invoice_tab
+                 "account_type".internal_group as account_internal_group
              FROM payment_table ref
              JOIN ONLY account_move_line ON
                 "account_move_line".move_id = ref.move_id
@@ -289,4 +282,8 @@ def _table_query(self):
                      WHERE account.internal_type IN ('receivable', 'payable')
                  )
              )
-         """
+        """
+        tools.drop_view_if_exists(self.env.cr, self._table)
+        self._cr.execute(
+            "CREATE OR REPLACE VIEW %s AS %s", (AsIs(self._table), AsIs(query))
+        )

account_move_line_cash_basis_report/security/account_move_line_cash_basis_report.xml

@@ -8,7 +8,7 @@
         <field name="model_id" ref="model_account_move_line_cash_basis_report" />
         <field
             name="domain_force"
-        >['|',('company_id','=',False),('company_id', 'in', company_ids)]</field>
+        >['|',('company_id','=',False),('company_id','child_of',[user.company_id.id])]</field>
     </record>
 
     <record
@@ -34,13 +34,14 @@
             name="name"
         >account.move.line.cash.basis.report access group_account_readonly</field>
         <field name="model_id" ref="model_account_move_line_cash_basis_report" />
-        <field name="group_id" ref="account.group_account_readonly" />
+        <field name="group_id" ref="account.group_account_user" />
         <field name="perm_read" eval="1" />
         <field name="perm_create" eval="0" />
         <field name="perm_write" eval="0" />
         <field name="perm_unlink" eval="0" />
     </record>
 
+
     <record
         model="ir.model.access"
         id="account_move_line_cash_basis_report_access_group_account_manager"

account_move_line_cash_basis_report/views/account_move_line_cash_basis_report.xml

@@ -12,7 +12,7 @@
             <form string="Journal Item" create="false">
                 <sheet>
                     <field name="company_id" invisible="1" />
-                    <field name="parent_state" invisible="1" />
+                    <!-- <field name="parent_state" invisible="1" /> -->
                     <group>
                         <field name="name" />
                         <field
@@ -46,7 +46,7 @@
                                 <group string="Dates">
                                     <field
                                         name="date"
-                                        groups="account.group_account_readonly"
+                                        groups="account.group_account_user"
                                     />
 <!--                                    <field name="date_maturity"/>-->
                                 </group>
@@ -114,7 +114,6 @@
                 />
                 <field name="date" />
                 <field name="account_id" />
-                <field name="account_root_id" />
                 <field name="account_internal_type" />
                 <field name="partner_id" />
                 <field name="journal_id" />
@@ -254,18 +253,18 @@
         <field name="arch" type="xml">
             <tree>
                 <field name="date" />
-                <field name="journal_id" optional="show" />
-                <field name="name" optional="show" />
-                <field name="ref" optional="hidden" />
-                <field name="account_id" optional="show" />
-                <field name="partner_id" optional="show" />
-                <field name="analytic_account_id" optional="hidden" />
-                <field name="debit" optional="show" />
-                <field name="credit" optional="show" />
-                <field name="balance" optional="hidden" />
-                <field name="user_type_id" optional="hidden" />
-                <field name="account_internal_type" optional="hidden" />
-                <field name="account_internal_group" optional="hidden" />
+                <field name="journal_id" />
+                <field name="name" />
+                <field name="ref" invisible="1" />
+                <field name="account_id" />
+                <field name="partner_id" />
+                <field name="analytic_account_id" invisible="1" />
+                <field name="debit" />
+                <field name="credit" />
+                <field name="balance" invisible="1" />
+                <field name="user_type_id" invisible="1" />
+                <field name="account_internal_type" invisible="1" />
+                <field name="account_internal_group" invisible="1" />
             </tree>
         </field>
     </record>
@@ -276,7 +275,7 @@
         >account.move.line.cash.basis.report.pivot (in account_move_line_cash_basis_report)</field>
         <field name="model">account.move.line.cash.basis.report</field>
         <field name="arch" type="xml">
-            <pivot string="Account Move Line Cash Basis Report" sample="1">
+            <pivot string="Account Move Line Cash Basis Report">
                 <field name="user_type_id" type="row" />
                 <field name="account_id" type="row" />
                 <field name="date" interval="month" type="col" />