From 39c1d8241f6ee50c7dc6b2aa8589f45aa05d9974 Mon Sep 17 00:00:00 2001
From: "Purnendu Singh (OpenERP)" <psi@tinyerp.com>
Date: Wed, 13 Jun 2012 12:25:02 +0530
Subject: [PATCH] [IMP] small improvments

bzr revid: psi@tinyerp.com-20120613065502-r1bez7o8e9rj7kyp
---
 addons/account/account_invoice_view.xml   |  3 ++-
 addons/account_voucher/account_voucher.py | 16 ++++++++--------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/addons/account/account_invoice_view.xml b/addons/account/account_invoice_view.xml
index 9599f23fce2..6646ff552ca 100644
--- a/addons/account/account_invoice_view.xml
+++ b/addons/account/account_invoice_view.xml
@@ -148,7 +148,8 @@
             <field name="arch" type="xml">
                 <form version="7.0">
                 <header>
-                    <button name="invoice_open" states="draft,proforma2" string="Validate" class="oe_form_button_hi"/>
+                    <button name="invoice_open" states="draft" string="Validate" class="oe_form_button_hi"/>
+                    <button name="invoice_open" states="proforma2" string="Validate"/>
                     <button name="%(action_account_invoice_refund)d" type='action' string='Ask Refund' states='open,paid'/>
                     <button name="invoice_cancel" states="draft,proforma2,sale,open" string="Cancel" groups="base.group_no_one"/>
                     <button name="action_cancel_draft" states="cancel" string="Set to Draft" type="object"/>
diff --git a/addons/account_voucher/account_voucher.py b/addons/account_voucher/account_voucher.py
index 526eed69d83..c1e6623fc2d 100644
--- a/addons/account_voucher/account_voucher.py
+++ b/addons/account_voucher/account_voucher.py
@@ -335,6 +335,12 @@ class account_voucher(osv.osv):
         'payment_rate_currency_id': _get_payment_rate_currency,
     }
 
+    def create(self, cr, uid, vals, context=None):
+        voucher =  super(account_voucher, self).create(cr, uid, vals, context=context)
+        if voucher:
+            self.create_send_note(cr, uid, [voucher], context=context)
+        return voucher
+
     def compute_tax(self, cr, uid, ids, context=None):
         tax_pool = self.pool.get('account.tax')
         partner_pool = self.pool.get('res.partner')
@@ -1264,12 +1270,6 @@ class account_voucher(osv.osv):
                 self.reconcile_send_note(cr, uid, [voucher.id], context=context)
         return True
 
-    def create(self, cr, uid, vals, context=None):
-        voucher =  super(account_voucher, self).create(cr, uid, vals, context=context)
-        if voucher:
-            self.create_send_note(cr, uid, [voucher], context=context)
-        return voucher
-    
     def copy(self, cr, uid, id, default={}, context=None):
         default.update({
             'state': 'draft',
@@ -1284,7 +1284,7 @@ class account_voucher(osv.osv):
         return super(account_voucher, self).copy(cr, uid, id, default, context)
 
     # --------------------------------------
-    # OpenChatter methods and notifications
+    # OpenChatter notifications and need_action
     # --------------------------------------
     
     def _get_document_type(self, type):
@@ -1294,7 +1294,7 @@ class account_voucher(osv.osv):
                 'sale': 'Sales Receipt',
                 'receipt': 'Customer Payment',
         }
-        return type_dict.get(type, 'payment')
+        return type_dict.get(type, 'Payment')
 
     def create_send_note(self, cr, uid, ids, context=None):
         for obj in self.browse(cr, uid, ids, context=context):