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

nancydru’s picture

Status: Active » Fixed

I wonder how that survived all my testing, but it did. I committed the change and attributed it to you.

nancydru’s picture

pico34’s picture

Please what is the right patch to apply to recover the workflow property ?

pico34’s picture

In workflow.module, function workflow_node_load line 169

replace

if (array_key_exists($node->type, $workflow_types)) {
return;
}

with

if (!array_key_exists($node->type, $workflow_types)) {
return;
}
nancydru’s picture

@pico34: The change is in the current dev release.

nhck’s picture

Kindly request to roll a new release as this also causes #1971508: Workflow Scheduler does not move states

nancydru’s picture

There are a few outstanding patches to be reviewed. If you would test them, it would help.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

sverma’s picture

Issue summary: View changes

I 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?

johnv’s picture

@sverma, please open a new issue for your problem.