From 1572f41acf56f264e54a808dfc2cbe4a15dbb2b0 Mon Sep 17 00:00:00 2001
From: Denis Vermylen <dve@odoo.com>
Date: Tue, 14 Mar 2017 18:04:45 +0100
Subject: [PATCH] [FIX] account: set conditional require attribute

When registering a payment on an invoice, the pop-up kept asking you to
enter a writeoff_account_id even when no reconciliation was asked.
The field should of course not be required in that case.
---
 addons/account/views/account_payment_view.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/addons/account/views/account_payment_view.xml b/addons/account/views/account_payment_view.xml
index e6298362513..620d6638e65 100644
--- a/addons/account/views/account_payment_view.xml
+++ b/addons/account/views/account_payment_view.xml
@@ -210,9 +210,9 @@
                                     <field name="payment_difference_handling" widget="radio" nolabel="1"/>
                                     <div attrs="{'invisible': [('payment_difference_handling','=','open')]}">
                                         <label for="writeoff_account_id" class="oe_edit_only" string="Post Difference In"/>
-                                        <field name="writeoff_account_id" string="Post Difference In" required="1"/>
+                                        <field name="writeoff_account_id" string="Post Difference In" attrs="{'required': [('payment_difference_handling', '=', 'reconcile')]}"/>
                                         <label for="writeoff_label" class="oe_edit_only" string="Label"/>
-                                        <field name="writeoff_label" required="1"/>
+                                        <field name="writeoff_label" attrs="{'required': [('payment_difference_handling', '=', 'reconcile')]}"/>
                                     </div>
                                 </div>
                             </group>