From 8b9da6843593be7a0ba0c62a11c4d61658baef34 Mon Sep 17 00:00:00 2001
From: mtr <mtr@mtr>
Date: Tue, 28 Sep 2010 17:16:24 +0530
Subject: [PATCH] [IMP] sale:-wizard in the action of list view and changes in
 the yaml

bzr revid: mtr@mtr-20100928114624-s4zukca2g5ea0dt8
---
 addons/sale/sale.py                       |  7 -----
 addons/sale/sale_view.xml                 | 12 ++++-----
 addons/sale/test/so_make_line_invoice.yml | 32 ++++++++++++++---------
 addons/sale/wizard/sale_line_invoice.xml  | 11 ++++++++
 addons/sale/wizard/sale_make_invoice.xml  | 12 ++++++++-
 5 files changed, 48 insertions(+), 26 deletions(-)

diff --git a/addons/sale/sale.py b/addons/sale/sale.py
index b6606a61215..b1ae33d2c43 100644
--- a/addons/sale/sale.py
+++ b/addons/sale/sale.py
@@ -841,13 +841,6 @@ class sale_order_line(osv.osv):
         'product_packaging': False
     }
 
-    def create_sale_order_line_invoice(self, cr, uid, ids, context=None):
-        if context is None:
-            context = {}
-        context.update({'active_ids' : ids,'active_id' : ids})
-        self.pool.get('sale.order.line.make.invoice').make_invoices(cr, uid, ids, context=context)
-        return True
-
     def invoice_line_create(self, cr, uid, ids, context=None):
         if context is None:
             context = {}
diff --git a/addons/sale/sale_view.xml b/addons/sale/sale_view.xml
index b8a8b64e7ce..1c7bf048415 100644
--- a/addons/sale/sale_view.xml
+++ b/addons/sale/sale_view.xml
@@ -157,10 +157,10 @@
                                                 <field name="invoiced"/>
                                                 <group attrs="{'invisible':[('invoiced','=',True)]}">
                                                 <button colspan="1"
-                                                    name="create_sale_order_line_invoice"
+                                                    name="%(action_view_sale_order_line_make_invoice)d"
                                                     states="confirmed"
-                                                    string="Create Invoice"
-                                                    type="object"
+                                                    string="Make Invoices"
+                                                    type="action"
                                                     icon="terp-document-new" />
                                                  </group>
                                             </group>
@@ -403,9 +403,9 @@ to your configuration: from the sales order, from the pickings, etc.
                             <field name="state"/>
                             <group col="3" colspan="2">
                                 <button colspan="1"
-                                  name="create_sale_order_line_invoice"
-                                  string="Create Invoice"
-                                  type="object"
+                                  name="%(action_view_sale_order_line_make_invoice)d"
+                                  string="Make Invoice"
+                                  type="action"
                                   icon="terp-document-new"
                                   attrs="{'invisible': ['|',('state', 'in', ('draft','cancel')),('invoiced', '=', 1)]}"/>
                                 <button name="button_cancel"
diff --git a/addons/sale/test/so_make_line_invoice.yml b/addons/sale/test/so_make_line_invoice.yml
index 0aa18df1c9f..d5e81c9e96c 100644
--- a/addons/sale/test/so_make_line_invoice.yml
+++ b/addons/sale/test/so_make_line_invoice.yml
@@ -41,15 +41,20 @@
 - 
   !workflow {model: sale.order, action: order_confirm, ref: sale_order_so3}
 -
-  I click on the "Create Invoice" button of sale order line
+  I click on the "Make Invoice" button of sale order line
+-
+  !record {model: sale.order.line.make.invoice, id: sale_order_line_make_invoice_0}:
+    {}
+-
+  I click on the "Create Invoice" button of wizard
 - 
-  !python {model: sale.order.line}: |
+  !python {model: sale.order.line.make.invoice}: |
     sale_order_obj = self.pool.get('sale.order')
     so = sale_order_obj.browse(cr, uid, ref("sale_order_so3"))
     sol = so.order_line[0] 
-    self.create_sale_order_line_invoice(cr, uid, [sol.id], {"lang": "en_US",
-      "tz": False, "active_model": "ir.ui.menu", "active_ids": [ref("sale.menu_invoicing_sales_order_lines")],
-      "search_default_uninvoiced": 1, "active_id": ref("sale.menu_invoicing_sales_order_lines"),
+    self.make_invoices(cr, uid, [ref("sale_order_line_make_invoice_0")], {"lang": "en_US",
+      "tz": False, "active_model": "sale.order.line", "active_ids": [sol.id],
+      "search_default_uninvoiced": 1, "active_id": sol.id,
       })
 -
   I verify that "Invoiced" has been set to True.
@@ -115,17 +120,20 @@
     sale_id=self.browse(cr, uid, ref("sale_order_so3"))
     assert(sale_id.invoiced == True), "Paid has not been set to true"
 -
-  I click on the "Create Invoice" button of sale order and verify that it gives a warning message
+  I create an invoice for another sale order line. I click on the "Make Invoice" button of sale order line
 -
-  I create an invoice for another sale order line. I click on the "Create Invoice" button of sale order line
-- 
-  !python {model: sale.order.line}: |
+  !record {model: sale.order.line.make.invoice, id: sale_order_line_make_invoice_1}:
+    {} 
+-
+  I click on the "Create Invoice" button of wizard
+-
+  !python {model: sale.order.line.make.invoice}: |
     sale_order_obj = self.pool.get('sale.order')
     so = sale_order_obj.browse(cr, uid, ref("sale_order_so3"))
     sol = so.order_line[1]    
-    self.create_sale_order_line_invoice(cr, uid, [sol.id], {"lang": "en_US",
-      "tz": False, "active_model": "ir.ui.menu", "active_ids": [ref("sale.menu_invoicing_sales_order_lines")],
-      "search_default_uninvoiced": 1, "active_id": ref("sale.menu_invoicing_sales_order_lines"),
+    self.make_invoices(cr, uid, [ref("sale_order_line_make_invoice_1")], {"lang": "en_US",
+      "tz": False, "active_model": "sale.order.line", "active_ids": [sol.id],
+      "search_default_uninvoiced": 1, "active_id": sol.id,
       })
 -
   I verify that invoice for sale order line has been created.
diff --git a/addons/sale/wizard/sale_line_invoice.xml b/addons/sale/wizard/sale_line_invoice.xml
index 63a26587d43..b48247f8e58 100644
--- a/addons/sale/wizard/sale_line_invoice.xml
+++ b/addons/sale/wizard/sale_line_invoice.xml
@@ -20,7 +20,18 @@
             <field name="res_model">sale.order.line.make.invoice</field>
             <field name="view_type">form</field>
             <field name="view_mode">form</field>
+            <field name="view_id" ref="view_sale_order_line_make_invoice"/>
             <field name="target">new</field>
         </record>
+        
+        <record model="ir.values" id="sale_order_line_make_invoice">
+            <field name="model_id" ref="sale.model_sale_order_line" />
+            <field name="object" eval="1" />
+            <field name="name">Make Invoices</field>
+            <field name="key2">client_action_multi</field>
+            <field name="value" eval="'ir.actions.act_window,' + str(ref('action_view_sale_order_line_make_invoice'))" />
+            <field name="key">action</field>
+            <field name="model">sale.order.line</field>
+        </record>
     </data>
 </openerp>
\ No newline at end of file
diff --git a/addons/sale/wizard/sale_make_invoice.xml b/addons/sale/wizard/sale_make_invoice.xml
index 2bef6b67513..106e6945207 100644
--- a/addons/sale/wizard/sale_make_invoice.xml
+++ b/addons/sale/wizard/sale_make_invoice.xml
@@ -24,8 +24,18 @@
             <field name="res_model">sale.make.invoice</field>
             <field name="view_type">form</field>
             <field name="view_mode">form</field>
+            <field name="view_id" ref="view_sale_order_make_invoice"/>
             <field name="target">new</field>
         </record>
-
+        
+        <record model="ir.values" id="sale_order_make_invoice">
+            <field name="model_id" ref="sale.model_sale_order" />
+            <field name="object" eval="1" />
+            <field name="name">Make Invoices</field>
+            <field name="key2">client_action_multi</field>
+            <field name="value" eval="'ir.actions.act_window,' + str(ref('action_sale_order_make_invoice'))" />
+            <field name="key">action</field>
+            <field name="model">sale.order</field>
+        </record>
     </data>
 </openerp>
\ No newline at end of file