diff --git a/addons/mrp/mrp_view.xml b/addons/mrp/mrp_view.xml
index 83685a7aa36..7eb8466d8a2 100644
--- a/addons/mrp/mrp_view.xml
+++ b/addons/mrp/mrp_view.xml
@@ -492,7 +492,7 @@
 				                    	states="draft,waiting,confirmed,assigned" />
 				                    <button name="%(stock.move_scrap)d"
 				                    	string="Scrap Products" type="action"
-				                    	icon="gtk-justify-fill" context="{'scrap': True}"
+				                    	icon="gtk-convert" context="{'scrap': True}"
 				                    	states="draft,waiting,confirmed,assigned" />				                    
                                 </tree>
                             </field>
@@ -545,7 +545,7 @@
 				                    	states="draft,waiting,confirmed,assigned" />
 				                    <button name="%(stock.move_scrap)d"
 				                    	string="Scrap Products" type="action"
-				                    	icon="gtk-justify-fill" context="{'scrap': True}"
+				                    	icon="gtk-convert" context="{'scrap': True}"
 				                    	states="draft,waiting,confirmed,assigned" />				                    
                                 </tree>
                             </field>
@@ -562,7 +562,7 @@
                            			<button name="%(stock.track_line)d"
                                             string="Split in production lots" type="action" icon="gtk-justify-fill" states="done,cancel"/>
                            			<button name="%(stock.move_scrap)d"
-                           				string="Scrap Products" type="action" icon="gtk-go-forward"
+                           				string="Scrap Products" type="action" icon="gtk-convert"
                            				states="done,cancel" />
                            		</tree>
                            	</field>
diff --git a/addons/stock/stock.py b/addons/stock/stock.py
index c063678c4a0..edccdcd29db 100644
--- a/addons/stock/stock.py
+++ b/addons/stock/stock.py
@@ -1462,7 +1462,7 @@ class stock_move(osv.osv):
         @ param location_id : specify scrap location
         @ param context: context arguments
 
-        @ return: True
+        @ return: Scraped lines
         '''   
         if quantity <= 0:
             raise osv.except_osv(_('Warning!'), _('Please provide Proper Quantity !'))
@@ -1479,7 +1479,7 @@ class stock_move(osv.osv):
             new_move = self.copy(cr, uid, move.id, default_val)
             res += [new_move]  
                   
-        return True
+        return res
 
     def action_split(self, cr, uid, ids, quantity, split_by_qty=1, prefix=False, with_lot=True, context=None):
         '''
@@ -1493,7 +1493,7 @@ class stock_move(osv.osv):
         @ param with_lot : if true, prodcution lot will assign for split line otherwise not.
         @ param context: context arguments
 
-        @ return: True
+        @ return: splited move lines
         '''   
 
         if quantity <= 0:
@@ -1555,7 +1555,7 @@ class stock_move(osv.osv):
         @ param location_id : specify source location         
         @ param context: context arguments
 
-        @ return: True
+        @ return: Consumed lines
         '''   
         if not context:
             context = {}