openerp-community-reviewer team mailing list archive

["Leonardo Pistone @ camptocamp" <leonardo.pistone@xxxxxxxxxxxxxx>]

Leonardo Pistone @ camptocamp has proposed merging lp:~camptocamp/ocb-addons/7.0-fix-1281492-lep into lp:ocb-addons.

Requested reviews:
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #1281492 in OpenERP Addons: "account: cash statement: onchange_journal_id changes the field last_closing_balance which is not in the bank statement view -> breaks YAML tests"
  https://bugs.launchpad.net/openobject-addons/+bug/1281492

For more details, see:
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1281492-lep/+merge/206883
-- 
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1281492-lep/+merge/206883
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~camptocamp/ocb-addons/7.0-fix-1281492-lep into lp:ocb-addons.

=== modified file 'account/account_view.xml'
--- account/account_view.xml	2014-02-05 13:20:57 +0000
+++ account/account_view.xml	2014-02-18 10:32:59 +0000
@@ -566,6 +566,8 @@
                     <h1><field name="name"/></h1>
                     <group>
                         <group>
+                            <!-- last_closing_balance must be here because it is returned by onchange_journal_id -->
+                            <field name="last_closing_balance" invisible="1"/>
                             <field name="journal_id" domain="[('type', '=', 'bank')]" on_change="onchange_journal_id(journal_id)" widget="selection"/>
                             <label for="date" string="Date / Period"/>
                             <div>