Problem/Motivation

At now all users with Drupal Administrator permissions always see "Schedule" form field group, but for most cases it is not needed in interface. Showing this form can be managed via "Schedule state transition" permission, but administrators have all permissions by default, so can't remove this permission from himself.

Also we have cases, when in one workflow same user must have "Schedule" feature, but in other - not, and this also can't be solved via permission settings.

Proposed resolution

Possible solution for this problem can be adding an option into each workflow "Allow schedule state transitions". What do you think about this solution?

CommentFileSizeAuthor
#5 workflow_3173802_5_hide_scheduling.patch3.56 KBjohnv
#2 workflow-hide_schedule_option-3173802-1.patch2.84 KBMurz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Murz created an issue. See original summary.

Murz’s picture

Murz
Primary language English
Location Yerevan, Armenia
CreditAttribution: Murz at Digiterra commented
Status: Active » Needs review
FileSize
workflow-hide_schedule_option-3173802-1.patch2.84 KB

Here is the patch, that adds new option "Show a scheduler in state form" to each workflow form settings (checked by default), which allow to force hide "Schedule" form from needed workflow forms even for administrators, via unchecking this option.

It is not change previous behavior, if this option is missed in current config, it threats them as "TRUE".

Please review my patch - is the implementation right?

Status: Needs review » Needs work

The last submitted patch, 2: workflow-hide_schedule_option-3173802-1.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Title: Add option to force hide "Schedule" form field group for administrator permissions » Add option to hide "Schedule" from administrator permissions
johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Version: 8.x-1.x-dev » 8.x-1.4
Status: Needs work » Needs review
FileSize
workflow_3173802_5_hide_scheduling.patch3.56 KB

Patch updated. I renamed the option to be more in line with other fields. Sorry for that.

Status: Needs review » Needs work

The last submitted patch, 5: workflow_3173802_5_hide_scheduling.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

  • johnv committed 95a72f6 on 8.x-1.x 
    Issue #3173802 by Murz, johnv: Add option to hide "Schedule" from...
johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.