workflow_get_states has two conditional branches. In one branch the states are ordered by weight, sid (as they should), in another only by sid. In effect, the filter by sid field in Views is not sorted properly if wid is set for this field.

CommentFileSizeAuthor
#1 workflow_1005056.patch819 bytesjpl-2

Comments

jpl-2’s picture

jpl-2 commented
StatusFileSize
new workflow_1005056.patch819 bytes

This tiny patch fixes the problem.

dawansv’s picture

dawansv commented

I confirm this is a bug and that the above patch fixes it. Thanks.

johnv’s picture

johnv
Primary language Dutch
commented

As a reminder: this is fixed in version 7.x-1.2. The function is renamed to workflow_get_workflow_states().

johnv’s picture

johnv
Primary language Dutch
commented
Title: workflow_get_states doesn't order by weight » workflow_get_states doesn't order by weight (D6)

making this exlicit in the title, too.

johnv’s picture

johnv
Primary language Dutch
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

I wish there was a nicer way to clear the issue queue from 'D6-issues that are fixed in D7' then a "won't fix for D6."