From 5cc71e1bababf7ea1d2bc9d8ccdd33ae8af74711 Mon Sep 17 00:00:00 2001
From: Vo Minh Thu <vmt@openerp.com>
Date: Tue, 26 Mar 2013 15:39:37 +0100
Subject: [PATCH] [FIX] workitem: the early return in the case workitem.state
 == running was seemingly wrong as workitem.state is mutated in some other
 IFs.

bzr revid: vmt@openerp.com-20130326143937-lf4mswer4ablj46i
---
 openerp/workflow/workitem.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/openerp/workflow/workitem.py b/openerp/workflow/workitem.py
index 88a786e6c67..a5639fa0c7c 100644
--- a/openerp/workflow/workitem.py
+++ b/openerp/workflow/workitem.py
@@ -44,12 +44,10 @@ def create(cr, act_datas, inst_id, ident, stack):
 
 def process(cr, workitem, ident, signal=None, force_running=False, stack=None):
     assert stack is not None
+
     cr.execute('select * from wkf_activity where id=%s', (workitem['act_id'],))
     activity = cr.dictfetchone()
 
-    if workitem['state'] == 'running':
-        return True
-
     triggers = False
     if workitem['state'] == 'active':
         triggers = True