Closed (fixed)
Project:
Workflow
Version:
7.x-1.1
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Apr 2013 at 12:25 UTC
Updated:
1 Jul 2014 at 07:27 UTC
Jump to comment: Most recent
In workflow.module, function workflow_node_load line 169, the condition seems to be inverted :
if (array_key_exists($node->type, $workflow_types)) {
return;
}
Therefore the $node->workflow property is not set on nodes, which is annoying for modules depending on this property.
Comments
Comment #1
nancydruI wonder how that survived all my testing, but it did. I committed the change and attributed it to you.
Comment #2
nancydruAnother symptom: #1961098: Workflow VBO module doesn't seem to change workflow to next state
Comment #3
pico34 commentedPlease what is the right patch to apply to recover the workflow property ?
Comment #4
pico34 commentedIn workflow.module, function workflow_node_load line 169
replace
with
Comment #5
nancydru@pico34: The change is in the current dev release.
Comment #6
nhck commentedKindly request to roll a new release as this also causes #1971508: Workflow Scheduler does not move states
Comment #7
nancydruThere are a few outstanding patches to be reviewed. If you would test them, it would help.
Comment #9
sverma commentedI have Workflow VBO (7.x-2.3) and Views Bulk Operations (7.x-3.2) installed. I have created a workflow in which a document moves from A to B then to C.
When B changes the state of individual node to next state, it works properly. However, after installing VBO and adding a button in view (Change Workflow State of Post to Next state) , when multiple nodes are selected and executed the message given is
You selected the following 0 items:
After confirm is pressed it says processed 2 out of 2 (if there are 2 nodes) however, the state of node does not change.
I checked for the error as pico34 has suggested but that error does not exist in the code.
Can someone help please?
Comment #10
johnv@sverma, please open a new issue for your problem.