[FIX] gamification: avoid discarding False or 0 given as result for
authorThibault Delavallée <tde@openerp.com>
Thu, 10 Apr 2014 14:10:49 +0000 (16:10 +0200)
committerThibault Delavallée <tde@openerp.com>
Thu, 10 Apr 2014 14:10:49 +0000 (16:10 +0200)
python computation.

bzr revid: tde@openerp.com-20140410141049-luw63buvf0jsvkoh

addons/gamification/models/goal.py

index 925b05d..5e58e8b 100644 (file)
@@ -251,14 +251,14 @@ class gamification_goal(osv.Model):
                     'cr': cr,
                     'context': dict(context), # copy context to prevent side-effects of eval
                     'uid': uid,
-                    'result': False,
+                    # 'result': False,
                     'date': date, 'datetime': datetime, 'timedelta': timedelta, 'time': time
                 }
                 code = goal.definition_id.compute_code.strip()
                 safe_eval(code, cxt, mode="exec", nocopy=True)
                 # the result of the evaluated codeis put in the 'result' local variable, propagated to the context
-                result = cxt.get('result', False)
-                if result and type(result) in (float, int, long):
+                result = cxt.get('result')
+                if result is not None and type(result) in (float, int, long):
                     if result != goal.current:
                         towrite['current'] = result
                 else: