Merge lp:~therp-nl/banking-addons/ba61-lp1073521_force_alternative_move_line_tree_view into lp:banking-addons/6.1

[Stefan Rijnhart (Opener)]

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Shouldn't it be 'view_mode': 'tree'?

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Ok, I understood by reading the fields_view_get override in account/account_move_line.py

1009 if context.get('view_mode', False):
1010 return result

Whatever context['view_mode'] contains, it will return early and won't replace your view by the hardcoded one.

That's a bit disturbing but should work to workaround the bug.

review: Approve

[Stefan Rijnhart (Opener) (stefan-opener)]

Yes, I could have annotated that I used a nonsense value to reflect the nonsense of the code that it needs to interact with. Thank you for finding that out yourself.

=== 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"