Problem/Motivation

Currently we use validation to prevent assigning sensitive perms to anonymous or authenticated users. This works, but it would seem preferable to disable the checkboxes in the first place.

Proposed resolution

  • Set '#disabled' to TRUE on the checkboxes and set value to FALSE.
  • Remove the validation code.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 paranoia-anon-auth-perms-2894993-4.patch2.45 KBnedjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo created an issue. See original summary.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Version: 7.x-1.x-dev » 8.x-1.x-dev
nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Title: Remove checkboxes for security-related permissions » Disable checkboxes for security-related permissions for anonymous and authenticated
Issue summary: View changes

Better UX to disable rather than remove.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Active » Postponed
FileSize
paranoia-anon-auth-perms-2894993-4.patch2.45 KB

Here's a patch that requires the patch on #2894995: Remove permissions rather than suppressing their checkboxes. Postponing until that's applied.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Postponed » Needs review

#2894995 was applied so this is ready for review.

  • nedjo committed 8fa6a7d on 8.x-1.x 
    Issue #2894993 by nedjo: Disable checkboxes for security-related...
nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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