FIX l10n_it_withholding_tax: don't intervene when not our scope (OCA#485

)
eLBati · Nov 8, 2018 · 488c8b1 · 488c8b1
1 parent 8be1ef4
commit 488c8b1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/l10n_it_withholding_tax/__manifest__.py b/l10n_it_withholding_tax/__manifest__.py
@@ -4,7 +4,7 @@
 
 {
     'name': 'Italian Withholding Tax',
-    'version': '10.0.1.2.1',
+    'version': '10.0.1.2.2',
     'category': 'Account',
     'author': 'Openforce, Odoo Italia Network, '
               'Odoo Community Association (OCA)',

diff --git a/l10n_it_withholding_tax/models/account.py b/l10n_it_withholding_tax/models/account.py
@@ -29,10 +29,10 @@ def create(self, vals):
             if invoice:
                 break
         # Limit value of reconciliation
-        if invoice and invoice.amount_net_pay:
+        if invoice and invoice.withholding_tax and invoice.amount_net_pay:
             # We must consider amount in foreign currency, if present
             # Note that this is always executed, for every reconciliation.
-            # Thus, we must not chnage amount when not in withholding tax case
+            # Thus, we must not change amount when not in withholding tax case
             amount = vals.get('amount_currency') or vals.get('amount')
             if amount > invoice.amount_net_pay:
                 vals.update({'amount': invoice.amount_net_pay})