diff --git a/addons/account_asset/account_asset.py b/addons/account_asset/account_asset.py
index 48fd7567dbb..d8c8581648f 100644
--- a/addons/account_asset/account_asset.py
+++ b/addons/account_asset/account_asset.py
@@ -223,10 +223,9 @@ class account_asset_asset(osv.osv):
                 val['currency_id'] = company.currency_id.id
         return {'value': val}
     
-    def onchange_value_residual(self, cr, uid, ids, purchase_value, salvage_value=False, context=None):
+    def onchange_value_residual(self, cr, uid, ids, purchase_value, salvage_value, context=None):
         val = {}
-        asset = self.browse(cr, uid, ids, context=context)
-        for i in asset:
+        for asset in self.browse(cr, uid, ids, context=context):
             if purchase_value:
                 val['value_residual'] = purchase_value - salvage_value
             if salvage_value:
diff --git a/addons/account_asset/account_asset_view.xml b/addons/account_asset/account_asset_view.xml
index 5d611fce7c6..82daa011c87 100644
--- a/addons/account_asset/account_asset_view.xml
+++ b/addons/account_asset/account_asset_view.xml
@@ -108,8 +108,8 @@
                         <page string="General">
                             <group>
                                 <group>
-                                    <field name="purchase_value" widget="monetary" options="{'currency_field': 'currency_id'}" on_change="onchange_value_residual(purchase_value)"/>
-                                    <field name="salvage_value" widget="monetary" options="{'currency_field': 'currency_id'}" on_change="onchange_value_residual(purchase_value,salvage_value)"/>
+                                    <field name="purchase_value" widget="monetary" options="{'currency_field': 'currency_id'}" on_change="onchange_value_residual(purchase_value, salvage_value)"/>
+                                    <field name="salvage_value" widget="monetary" options="{'currency_field': 'currency_id'}" on_change="onchange_value_residual(purchase_value, salvage_value)"/>
                                     <field name="value_residual" widget="monetary" options="{'currency_field': 'currency_id'}"/>
                                     <field name="partner_id"/>
                                 </group>