Split settings for publishing and unpublishing for node types [#660612]

I think there should be different permission for schedule publish and unpublish (or one check for enable/disable the scheduled publish and another one for scheduled unpublish by content type)

In almost all the websites I develop the only feature the user needs is scheduling the publication of a node, that node never will be unpublished.

Having the unpublished fields on the node edit form add noise (UX) to the form and could confuse the user.

Thank you in advance

Comment	File	Size	Author
#10	scheduler_split.txt	10.81 KB	Eric-Alexander Schaefer
#9	scheduler_split_install.txt	1.05 KB	Eric-Alexander Schaefer
#8	scheduler_split.txt	9.8 KB	Eric-Alexander Schaefer
#1	split.diff	6.68 KB	rbarlow
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rbarlow CreditAttribution: rbarlow commented 18 December 2009 at 13:51

File	Size
split.diff	6.68 KB

I had a similar requirement a few months ago, I needed to show only the un-publish option.
The attached patch adds an extra checkbox to the node settings page allowing publishing and un-publishing of that node type to be controlled independently.
The patch is for the 6.x-1.6 release, first patch I've created for a drupal module so I don't know if it's normal to provide a patch for a released version or CVS head.

Comment #2

Anonymous (not verified) CreditAttribution: Anonymous commented 18 December 2009 at 15:43

Thank you, I'll take a look on this :)

Comment #3

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 22 December 2009 at 14:38

I think splitting the permissions could be useful. But it would have to be a non breaking change. That means, if somebody deploys a new version of scheduler with the split permissions, their permission setup should still work. (e.g. if somebody had the permission to use scheduler, he/she should have both permissions after the upgrade)

Comment #4

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 4 January 2010 at 14:24

If you think about it, maybe it should not be a permission but a configuration option?

Comment #5

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 26 June 2010 at 21:31

What is the status of this issue. The patch looks rather promising. Does it work?

Comment #6

SpikeX CreditAttribution: SpikeX commented 5 July 2010 at 05:59

Subscribing. These permissions should be separate, it really makes the most sense that way. When I saw they were combined the first thing I did was come here to see if there was already an issue about it (and there is). Hopefully this patch will make it into a stable version of the module so I can use it.

Comment #7

codycraven CreditAttribution: codycraven commented 7 July 2010 at 21:50

If this is done as a permission you can make a hook_update_N() implementation that will apply the new split permissions to all roles that currently have the permission.

Comment #8

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 7 November 2010 at 21:46

File	Size
scheduler_split.txt	9.8 KB

Please checkout the attached patch. It splits the node type settings as the original patch in #1. Additionally it uses a separate fieldset for scheduler settings (as opposed to be part of "workflow settings") and nested fieldsets for publishing and unpublishing. This might seem like overkill but it will be used for other options for publishing and unpublishing as well as additional actions (e.g. promote/demote/sticky).

The patch is rolled against the current CVS-HEAD (or 6.x-1.x-dev 2010-Oct-07).
Please test! This will be the base for lots of new features...

Comment #9

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 7 November 2010 at 21:46

Status:

Active

» Needs review

File	Size
scheduler_split_install.txt	1.05 KB

Here comes a patch for scheduler.install to convert the old configuration values to the new ones.

Comment #10

Eric-Alexander Schaefer CreditAttribution: Eric-Alexander Schaefer commented 10 November 2010 at 12:04

Title:

Split schedule (un)publishing of nodes permissions

» Split settings for publishing and unpublishing for node types

File	Size
scheduler_split.txt	10.81 KB

Ignore the last patch please. Let's say it never happened...

Attached patch includes the changes to scheduler.module (which are based on rbarlows patch, thanks) and scheduler.install. Since nobody is listening anyways I will commit the patch right away. I tested it now for quite a while and it seems to work well.

I also changed the title of this issue because it was never really about permissions, but about hiding unused settings. It might be worth a second thought if a separate permission per setting could be useful. But that would be another issue.