diff --git a/currency_rate_update/__openerp__.py b/currency_rate_update/__openerp__.py index 1cce4f384..9d6396341 100755 --- a/currency_rate_update/__openerp__.py +++ b/currency_rate_update/__openerp__.py @@ -75,6 +75,8 @@ If in multi-company mode, the base currency will be the first company's currency found in database. Thanks to main contributors : Grzegorz Grzelak, Alexis de Lattre + + """, "depends" : ["base", "account"], #Added to ensure account security groups are present diff --git a/currency_rate_update/currency_rate_update.py b/currency_rate_update/currency_rate_update.py index 1ad031a28..ad216f43d 100755 --- a/currency_rate_update/currency_rate_update.py +++ b/currency_rate_update/currency_rate_update.py @@ -98,12 +98,11 @@ class Currency_rate_update_service(osv.osv): ] def _check_max_delta_days(self, cr, uid, ids): - for company in self.read(cr, uid, ids, ['max_delta_days']): - if company['max_delta_days'] >= 0: - continue - else: - return False - return True + for i in ids: + value_to_check = self.read(cr, uid, i, ['max_delta_days'])['max_delta_days'] + if value_to_check >= 0: + return True + else: return False _constraints = [ (_check_max_delta_days, "'Max delta days' must be >= 0", ['max_delta_days']),