From d0aa89caff4d52984c72e5d5f440c316e4e06259 Mon Sep 17 00:00:00 2001
From: Arthur Maniet <arthurmaniet@me.com>
Date: Wed, 19 Aug 2015 13:55:44 +0200
Subject: [PATCH] [FIX] account: bank statement usability

- Don't show the button 'Validate' of the statement form when there is no statement line
- In reconciliation widget, use term 'Validate' instead of 'Register Payment' for the use case where the transaction is not reconciled on the spot
---
 addons/account/static/src/js/account_reconciliation_widgets.js | 2 +-
 addons/account/views/account_view.xml                          | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/addons/account/static/src/js/account_reconciliation_widgets.js b/addons/account/static/src/js/account_reconciliation_widgets.js
index 9efc7194fee..b3ad4ee874d 100644
--- a/addons/account/static/src/js/account_reconciliation_widgets.js
+++ b/addons/account/static/src/js/account_reconciliation_widgets.js
@@ -1868,7 +1868,7 @@ var bankStatementReconciliationLine = abstractReconciliationLine.extend({
                 if (self.st_line.has_no_partner) {
                     createOpenBalance(_t("Choose counterpart"));
                 } else {
-                    displayValidState(false, _t("Register Payment"));
+                    displayValidState(false, _t("Validate"));
                     createOpenBalance(_t("Open balance"));
                 }
             }
diff --git a/addons/account/views/account_view.xml b/addons/account/views/account_view.xml
index f000c0b4e88..65d1f101eed 100644
--- a/addons/account/views/account_view.xml
+++ b/addons/account/views/account_view.xml
@@ -428,7 +428,7 @@
                 <header>
                     <field name="all_lines_reconciled" invisible="1" />
                     <button name="%(action_bank_reconcile_bank_statements)d" string="Reconcile" type="action" class="oe_highlight" attrs="{'invisible':['|','|',('all_lines_reconciled','=',True),('line_ids','=',[]),('state', '!=', 'open')]}"/>
-                    <button name="check_confirm_bank" string="Validate" type="object" class="oe_highlight" attrs="{'invisible':['|',('all_lines_reconciled','=',False),('state', '!=', 'open')]}"/>
+                    <button name="check_confirm_bank" string="Validate" type="object" class="oe_highlight" attrs="{'invisible':['|','|',('all_lines_reconciled','=',False),('line_ids','=',[]),('state', '!=', 'open')]}"/>
                     <button name="button_cancel" states="confirm" string="Cancel Statement" type="object"/>
                     <field name="state" widget="statusbar" statusbar_visible="open,confirm"/>
                 </header>