F#T63953 - [Otsokop] A tool to check accounts balances

Author: nguyenminhchien

coop_account/__init__.py

@@ -2,3 +2,4 @@
 from . import models
 from . import wizard
 from . import report
+from .init_hook import pre_init_hook

coop_account/__manifest__.py

@@ -29,4 +29,5 @@
         "view/account_journal_view.xml"
     ],
     'installable': True,
+    'pre_init_hook': 'pre_init_hook',
 }

coop_account/init_hook.py

@@ -0,0 +1,33 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
+
+import logging
+logger = logging.getLogger(__name__)
+
+
+def pre_init_hook(cr):
+    store_field_stored_other_balance(cr)
+
+
+def store_field_stored_other_balance(cr):
+    """Store other_balance in the account move line."""
+    cr.execute("""SELECT column_name
+    FROM information_schema.columns
+    WHERE table_name='account_move_line' AND
+    column_name='other_balance'""")
+    if not cr.fetchone():
+        cr.execute(
+            """
+            ALTER TABLE account_move_line
+            ADD COLUMN other_balance float;
+            COMMENT ON COLUMN account_move_line.other_balance
+            IS 'Other Balance';
+            """)
+
+    logger.info('Computing field other_balance on account.move.line')
+
+    cr.execute(
+        """
+        UPDATE account_move_line
+        SET other_balance = credit - debit
+        """
+    )

coop_account/models/account_move_line.py

@@ -15,7 +15,9 @@ class AccountMoveLine(models.Model):
     full_reconcile_id = fields.Many2one(index=True)
     other_balance = fields.Monetary(
         string='Other Balance',
-        default=0.0)
+        compute="_compute_other_balance",
+        store=True
+    )
     search_year = fields.Char(
         string='Year (Search)', compute='_compute_date_search',
         multi='_date_search', store=True, index=True)
@@ -26,30 +28,11 @@ class AccountMoveLine(models.Model):
         string='Day (Search)', compute='_compute_date_search',
         multi='_date_search', store=True, index=True)
 
-    @api.model
-    def create(self, vals):
-        debit = vals.get('debit', 0)
-        credit = vals.get('credit', 0)
-        vals.update({
-            'other_balance': credit - debit,
-        })
-        return super(AccountMoveLine, self).create(vals)
-
-    @api.multi
-    def write(self, vals):
-        self.calculate_orther_balance(vals)
-        return super(AccountMoveLine, self).write(vals)
-
     @api.multi
-    def calculate_orther_balance(self, vals):
+    @api.depends("credit", "debit")
+    def _compute_other_balance(self):
         for record in self:
-            if 'debit' in vals or 'credit' in vals:
-                debit = vals.get('debit', record.debit)
-                credit = vals.get('credit', record.credit)
-                vals.update({
-                    'other_balance': credit - debit,
-                })
-        return True
+            record.other_balance = record.credit - record.debit
 
     @api.multi
     @api.constrains('move_id', 'account_id')