banking-addons-team team mailing list archive

["Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>]

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/banking-addons/ba61-lp1073521_force_alternative_move_line_tree_view into lp:banking-addons.

Requested reviews:
  Banking Addons Team (banking-addons-team)
Related bugs:
  Bug #1073521 in Banking Addons: "[6.1] Cannot select an item if its default tree view is editable"
  https://bugs.launchpad.net/banking-addons/+bug/1073521

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/ba61-lp1073521_force_alternative_move_line_tree_view/+merge/143043
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba61-lp1073521_force_alternative_move_line_tree_view/+merge/143043
Your team Banking Addons Team is requested to review the proposed merge of lp:~therp-nl/banking-addons/ba61-lp1073521_force_alternative_move_line_tree_view into lp:banking-addons.

=== modified file 'account_banking/wizard/banking_transaction_wizard.xml'
--- account_banking/wizard/banking_transaction_wizard.xml	2012-02-25 01:15:48 +0000
+++ account_banking/wizard/banking_transaction_wizard.xml	2013-01-13 14:15:23 +0000
@@ -93,7 +93,21 @@
                             </page>
                             <page string="Manual match">
                                 <field name="manual_invoice_id"/>
-                                <field name="manual_move_line_id"/>
+                                <!-- 
+                                     Specify alternative tree_view_ref as a
+                                     workaround for lp:1073521 in OpenERP 6.1
+                                     Need to also define 'view_mode' to prevent
+                                     an instant editable tree view
+                                     reconstruction by account.move.line's
+                                     fields_view_get().
+                                     Both are not needed in OpenERP 6.0 or 7.0.
+                                -->
+                                <field name="manual_move_line_id"
+                                       context="{
+                                                'tree_view_ref': 'account.view_move_line_tax_tree',
+                                                'view_mode': 'yes'
+                                                }"
+                                       />
                                 <newline/>
                                 <button colspan="1"
                                         name="trigger_write"