I would like to make it so the "preview is required" option can be assigned to roles. To do so, there would be 2 basic steps:

1) Remove the "Preview post: optional/required" option from admin/settings/node.
2) Create a new access class named (say) 'optional post preview'.

It seems to me equally easy to configure and more powerfull. Thoughts?

CommentFileSizeAuthor
#1 optional_post_preview.patch1.58 KBmarkus_petrux
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented
Status: Active » Reviewed & tested by the community
FileSize
optional_post_preview.patch1.58 KB

It end up much easier than I thought. I hope you like it.

Please, see attached patch.

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented
Status: Reviewed & tested by the community » Needs review

Am I alone? ...bee, bee, boo, boo, woaaaaa

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented

is this reasonable or maybe food for a contrib module?

chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Closed (won't fix)

use hook_form_alter rather than a core patch.

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented

Just for reference here's this feature implemented as a contrib module:
http://drupal.org/node/51835

csc4’s picture

csc4 CreditAttribution: csc4 commented

That page is not found - did this get implemented anywhere?

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented

http://www.phpmix.org/projects/drupal/4_7/bypass_forced_preview

Hope that helps