openobject-italia-core-devs team mailing list archive

["Antonio M. - Zeroincombenze" <info@xxxxxxxxxx>]

Antonio M. - Zeroincombenze has proposed merging lp:~zeroincombenze/openobject-italia/7.0 into lp:openobject-italia/7.0.

Requested reviews:
  OpenERP Italia core devs (openobject-italia-core-devs)

For more details, see:
https://code.launchpad.net/~zeroincombenze/openobject-italia/7.0/+merge/225138
-- 
https://code.launchpad.net/~zeroincombenze/openobject-italia/7.0/+merge/225138
Your team OpenERP Italia core devs is requested to review the proposed merge of lp:~zeroincombenze/openobject-italia/7.0 into lp:openobject-italia/7.0.

=== modified file 'l10n_it_partially_deductible_vat/AUTHORS.txt'
--- l10n_it_partially_deductible_vat/AUTHORS.txt	2013-01-08 22:58:30 +0000
+++ l10n_it_partially_deductible_vat/AUTHORS.txt	2014-07-01 11:44:22 +0000
@@ -1,2 +1,3 @@
-Lorenzo Battistini <lorenzo.battistini@xxxxxxxxxxx>
-Franco Tampieri <franco.tampieri@xxxxxxxxxxx>
+Lorenzo Battistini <lorenzo.battistini@xxxxxxxxxxx>
+Franco Tampieri <franco.tampieri@xxxxxxxxxxx>
+Antonio M. Vigliotti <info@xxxxxxxxxx>
\ No newline at end of file

=== modified file 'l10n_it_partially_deductible_vat/__openerp__.py'
--- l10n_it_partially_deductible_vat/__openerp__.py	2014-06-23 09:55:36 +0000
+++ l10n_it_partially_deductible_vat/__openerp__.py	2014-07-01 11:44:22 +0000
@@ -25,7 +25,7 @@
 
 {
     "name" : "Italy - Partially Deductible VAT",
-    "version" : "0.1",
+    "version" : "0.2",
     "depends" : ['account'],
     "author" : "OpenERP Italian Community",
     "description": """

=== modified file 'l10n_it_partially_deductible_vat/account.py'
--- l10n_it_partially_deductible_vat/account.py	2013-05-23 16:55:09 +0000
+++ l10n_it_partially_deductible_vat/account.py	2014-07-01 11:44:22 +0000
@@ -32,15 +32,19 @@
 
     _inherit = 'account.tax'
 
+# See Bug #1335867
+# [antoniov: 2014-06-30] if account_tax.type is 'balance', 'amount' value is unpredictable. Random bug!
     def _have_same_rate(self, account_taxes):
         rate = None
         for account_tax in account_taxes:
-            if rate is None:
-                rate = account_tax.amount
-            elif rate != account_tax.amount:
-                return False
+            if account_tax.type != 'balance':
+                if rate is None:
+                    rate = account_tax.amount
+                elif rate != account_tax.amount:
+                    return False
         return True
 
+    
     def get_main_tax(self, tax):
         if not tax.parent_id:
             return tax