From cb7cb1420be0ffbc06fd0ebbbdbfa2a0786b6964 Mon Sep 17 00:00:00 2001
From: "vta vta@openerp.com" <>
Date: Thu, 8 Nov 2012 11:36:34 +0100
Subject: [PATCH] [FIX] Now the action is type object, method return the same
 action in order to have the right context.

bzr revid: vta@openerp.com-20121108103634-acxd3tyzd13b09ck
---
 addons/account/account_bank_statement.py | 19 +++++++++++++++++++
 addons/account/account_view.xml          | 16 +++-------------
 2 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/addons/account/account_bank_statement.py b/addons/account/account_bank_statement.py
index 48082451f35..93d9dec21b3 100644
--- a/addons/account/account_bank_statement.py
+++ b/addons/account/account_bank_statement.py
@@ -486,6 +486,25 @@ class account_bank_statement(osv.osv):
         default['move_line_ids'] = []
         return super(account_bank_statement, self).copy(cr, uid, id, default, context=context)
 
+    def button_journal_entries(self, cr, uid, ids, context=None):      
+      mod_obj = self.pool.get('ir.model.data')
+      res = mod_obj.get_object_reference(cr, uid, 'account', 'view_move_line_tree')
+      res_id = res and res[1] or False
+      ctx = dict(context)
+      ctx.update({
+        'active_model': 'account.bank.statement',
+        'active_id': ids[0],
+      })
+      return {
+        'view_type':'tree',
+        'view_mode':'tree',
+        'res_model':'account.move.line',
+        'view_id':res_id,
+        'type':'ir.actions.act_window',
+        'domain':[('statement_id','=',ctx.get('active_id'))],
+        'context':ctx,
+      }
+
 account_bank_statement()
 
 class account_bank_statement_line(osv.osv):
diff --git a/addons/account/account_view.xml b/addons/account/account_view.xml
index a946d04d2ff..059c96da4a4 100644
--- a/addons/account/account_view.xml
+++ b/addons/account/account_view.xml
@@ -680,25 +680,15 @@
             </field>
         </record>
 
-        <record id="action_view_bank_statement_journal_entries" model="ir.actions.act_window">
-            <field name="name">Journal Items</field>
-            <field name="res_model">account.move.line</field>
-            <field name="view_type">tree</field>
-            <field name="view_mode">tree</field>
-            <field name="view_id" ref="view_move_line_tree"/>
-            <field name="domain">[('statement_id', '=', context.get('active_ids'))]</field>
-        </record>
-
         <record id="view_bank_statement_form_journal_items" model="ir.ui.view">
             <field name="name">account.bank.statement.journal.items.form.inherit</field>
             <field name="model">account.bank.statement</field>
             <field name="inherit_id" ref="view_bank_statement_form"/>
             <field name="arch" type="xml">
                 <xpath expr="//div[@name='import_buttons']" position="inside">
-                    <button name="%(action_view_bank_statement_journal_entries)d"
-                            string="Journal Items" type="action"
-                            attrs="{'invisible':[('state','!=','confirm')]}"
-                            context="{'journal_id':'journal_id'}"/>
+                    <button name="button_journal_entries"
+                            string="Journal Items" type="object"
+                            attrs="{'invisible':[('state','!=','confirm')]}"/>
                 </xpath>
             </field>
         </record>