[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

@@ -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