Merge lp:~ruchir.shukla/banking-addons/banking-addons-payment-order-improvment into lp:banking-addons

[Ruchir Shukla(BizzAppDev)]

return Form view of Payment Orders so record will be reloaded
make account_payment_shortcut installable so default Invoices will be selected in Wizard

[Holger Brunn (Therp) (hbrunn)]

review: Approve (code review)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

review: Approve

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

Hi Ruchir,

sorry I missed this one. Please check if the code needs updating when you activate a module! In this case, it is thankfully only a very small number of changes. Additionally, I think reloading the payment order view is a workaround for https://bugs.launchpad.net/openerp-web/+bug/1155525 so this had better be pointed out in a comment. You can find these small changes here: https://code.launchpad.net/~therp-nl/banking-addons/ba70-fix_payment_order_improvements/+merge/204485.

Could you review my additional changes to this proposal?

Thanks!

=== modified file 'account_banking_payment/model/payment_order_create.py'
--- account_banking_payment/model/payment_order_create.py	2013-09-12 07:05:59 +0000
+++ account_banking_payment/model/payment_order_create.py	2013-12-11 06:44:15 +0000
@@ -194,4 +194,11 @@
                              or line.journal_id.currency.id
                              or line.journal_id.company_id.currency_id.id),
                 }, context=context)
-        return {'type': 'ir.actions.act_window_close'}
+        return {'name': _('Payment Orders'),
+                'context': context,
+                'view_type': 'form',
+                'view_mode': 'form,tree',
+                'res_model': 'payment.order',
+                'res_id': context['active_id'],
+                'type': 'ir.actions.act_window',
+        }

=== modified file 'account_payment_shortcut/__openerp__.py'
--- account_payment_shortcut/__openerp__.py	2013-04-15 14:14:27 +0000
+++ account_payment_shortcut/__openerp__.py	2013-12-11 06:44:15 +0000
@@ -29,5 +29,5 @@
     'description': '''
 When composing a payment order, select all candidates by default (in the second step of the "Select invoices to pay" wizard).
     ''',
-    'installable': False,
+    'installable': True,
 }