diff options
| author | Vo Minh Thu <vmt@openerp.com> | 2013-03-26 15:39:37 +0100 |
|---|---|---|
| committer | Vo Minh Thu <vmt@openerp.com> | 2013-03-26 15:39:37 +0100 |
| commit | 5cc71e1bababf7ea1d2bc9d8ccdd33ae8af74711 (patch) | |
| tree | 62a32aaa22bce40def213af2ba5433980a6f1aba /openerp/workflow | |
| parent | d185f17ec68ccdbc0c1f0b414bb6301fce9524b4 (diff) | |
[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
Diffstat (limited to 'openerp/workflow')
| -rw-r--r-- | openerp/workflow/workitem.py | 4 |
1 files changed, 1 insertions, 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 |