Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Just putting finishing touches to my workflow and I noticed that whilst I can see the workflow state change form on published content it does not appear on the unpublished states in my workflow. This means the user experience is different for different stages of my workflow. Is it possible to display the workflow state change form on unpublished nodes?
Comments
Comment #1
johnvYep, this seems to be a feature. Need to find out the initial requirement, before this can be removed/replaced.
Apparently, you cannot see the value of the state, too.
You might want to patch the code, and remove the check on '$node->status' (or $entity->status in 7.x-2.x).
Comment #2
mchaplin CreditAttribution: mchaplin commentedThis would be desirable. We are hoping to use workflow like this:
1) Private content created by the end user (unpublished).
2) Approved by the user's line manager, without the possibility of editing ie approved by viewing the node.
Comment #3
johnvStill need to find the reasoning behind this.
some previous discussions:
#786500: Do Workflow permissions override Drupal core permissions?
#438808: Without node permissions Author/Content Admin can't edit unpublished content
#156028: Workflow nodes must always be published for access controls to apply
Most discussions consider the usage of the workflow_access module.
Perhaps is is obsolete, with new permissions in D7:
- View (own) unpublished content
- Create content
- Edit own/any content
Haven't found the commit adding the $node->status check.
[EDIT] found this commit from March 10, 2013: #1380954-38: Triggers Missing
http://drupalcode.org/project/workflow.git/commitdiff/fcb543a3dfaf9b2bcd...
I consider creating a checkbox that enables/disables this check.
Comment #5
johnvAccording to Nancy: "It may have had something to do with the Revisioning module. But I do not recall exactly."
Comment #7
johnvAbove commit removes this. Found no reason why this is introduced 1 year ago.