Merge lp:~camptocamp/banking-addons/bank-statement-reconcile-61-fix-1083517 into lp:banking-addons/bank-statement-reconcile-61

[Alexandre Fayolle - camptocamp]

use an in-database action with a customized domain to get the correct redirection after the wizard has run.

[Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount)]

Hello
In domain can sid be a list? If it is not the case the fix seems ok else the case should be managed.

Regards

Nicolas

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

> Hello
> In domain can sid be a list? If it is not the case the fix seems ok else the
> case should be managed.

The code for statement_import in AccountStatementProfile returns a single statement id.

(and the original code used a very complicated construct to put the single integer in a list)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Seems good to me

[Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount)]

review: Approve

=== modified file 'account_statement_base_import/wizard/import_statement.py'
--- account_statement_base_import/wizard/import_statement.py	2012-08-02 12:46:12 +0000
+++ account_statement_base_import/wizard/import_statement.py	2012-11-27 10:56:20 +0000
@@ -110,11 +110,9 @@
                                             ftype.replace('.',''),
                                             context=context
                                         )
-        return {
-            'domain': "[('id','in', ["+','.join(map(str,[sid]))+"])]",
-            'name': 'Imported Bank Statement',
-            'view_type': 'form',
-            'view_mode': 'tree,form',
-            'res_model': 'account.bank.statement',
-            'type': 'ir.actions.act_window',
-        }
+        model_obj = self.pool.get('ir.model.data')
+        action_obj = self.pool.get('ir.actions.act_window')
+        action_id = model_obj.get_object_reference(cursor, uid, 'account', 'action_bank_statement_tree')[1]
+        res = action_obj.read(cursor, uid, action_id)
+        res['domain'] = res['domain'][:-1] + ",('id', '=', %d)]" % sid
+        return res