There are two separate flags that are required to be set when attempting to skip the workflow.

Unify them to one.

Comments

fmitchell’s picture

fmitchell commented
Status: Active » Closed (fixed)

Fixed in commit http://drupalcode.org/project/state_machine.git/commit/d93d4ac

fmitchell’s picture

fmitchell commented
Status: Closed (fixed) » Active

Backed this fix out. Broke logic. Need to revisit later. http://drupalcode.org/project/state_machine.git/commit/6c14cec

stevector’s picture

stevector
he/him
commented

If a content type should be ignored by state_flow would it be better for the load_state_machine() function to just return FALSE?

fmitchell’s picture

fmitchell commented

There is a lot of logic that assumes an object is returned. That's essentially what Issue #1515314 is trying to fix. I hadn't written an appropriate exception method (which is what probably needs to happen so we don't get WSOD).

The ignore_state is for when the node is re-saved and the skip is for not doing workflow at all.

fmitchell’s picture

fmitchell commented
Status: Active » Closed (won't fix)