From 18753eadf9b9ea8554fb15c3721883dad6bdf4eb Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Fri, 15 Apr 2016 15:06:57 +0200
Subject: [PATCH] Remove start/end balance in bank statement

---
 .../__openerp__.py                            |  1 +
 .../account_bank_statement_import.py          | 19 ++++++++
 .../account_view.xml                          | 44 +++++++++++++++++++
 3 files changed, 64 insertions(+)
 create mode 100644 account_bank_statement_import_usability/account_view.xml

diff --git a/account_bank_statement_import_usability/__openerp__.py b/account_bank_statement_import_usability/__openerp__.py
index 1e3fb97..82e3b5e 100644
--- a/account_bank_statement_import_usability/__openerp__.py
+++ b/account_bank_statement_import_usability/__openerp__.py
@@ -44,5 +44,6 @@ This module has been written by Alexis de Lattre from Akretion <alexis.delattre@
     'author': 'Akretion',
     'website': 'http://www.akretion.com',
     'depends': ['account_bank_statement_import'],
+    'data': ['account_view.xml'],
     'installable': True,
 }
diff --git a/account_bank_statement_import_usability/account_bank_statement_import.py b/account_bank_statement_import_usability/account_bank_statement_import.py
index da96fa9..c59e254 100644
--- a/account_bank_statement_import_usability/account_bank_statement_import.py
+++ b/account_bank_statement_import_usability/account_bank_statement_import.py
@@ -59,3 +59,22 @@ class AccountBankStatementImport(models.TransientModel):
             if bank_account_ids:
                 bank_account_id = bank_account_ids[0]
         return bank_account_id
+
+
+class AccountBankStatement(models.Model):
+    _inherit = 'account.bank.statement'
+
+    # When we use the import of bank statement via files,
+    # the start/end_balance is usually computed from the lines itself
+    # because we don't have the 'real' information in the file
+    # But, in the module account_bank_statement_import, in the method
+    # _create_bank_statement(), the bank statement lines already present in
+    # Odoo are filtered out, but the start/end balance is not adjusted,
+    # so the user has to manually modifiy it the close the bank statement
+    # I think the solution is just to remove the start/end balance system
+    # on the bank statement when we use the file import
+    # This code is present in the 'account' module, but I override it here
+    # and not in account_usability because the users who don't have
+    # account_bank_statement_import may want to keep start/end balance
+    def balance_check(self, cr, uid, st_id, journal_type='bank', context=None):
+        return True
diff --git a/account_bank_statement_import_usability/account_view.xml b/account_bank_statement_import_usability/account_view.xml
new file mode 100644
index 0000000..a241ff6
--- /dev/null
+++ b/account_bank_statement_import_usability/account_view.xml
@@ -0,0 +1,44 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+   Copyright (C) 2016 Akretion (http://www.akretion.com/)
+   @author: Alexis de Lattre <alexis.delattre@akretion.com>
+   The licence is in the file __openerp__.py
+-->
+
+<openerp>
+<data>
+
+<record id="view_bank_statement_form" model="ir.ui.view">
+    <field name="name">bank_statement_import_usability.account.bank.statement.form</field>
+    <field name="model">account.bank.statement</field>
+    <field name="inherit_id" ref="account.view_bank_statement_form"/>
+    <field name="arch" type="xml">
+        <field name="balance_start" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <field name="balance_end_real" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+    </field>
+</record>
+
+<record id="view_bank_statement_tree" model="ir.ui.view">
+    <field name="name">bank_statement_import_usability.account.bank.statement.tree</field>
+    <field name="model">account.bank.statement</field>
+    <field name="inherit_id" ref="account.view_bank_statement_tree"/>
+    <field name="arch" type="xml">
+        <field name="balance_start" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <field name="balance_end_real" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <tree position="attributes">
+            <attribute name="colors">blue:state=='draft';black:state=='confirm'</attribute>
+        </tree>
+    </field>
+</record>
+
+
+</data>
+</openerp>