PermissionAccessCheck uses user_access() to grant access, which means it should be part of User.module, rather than core in general.

CommentFileSizeAuthor
#3 drupal_1911260_01.patch3.97 KBXano
#1 drupal_1911260_00.patch3.93 KBXano
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xano’s picture

Status: Active » Needs review
FileSize
3.93 KB

Status: Needs review » Needs work

The last submitted patch, drupal_1911260_00.patch, failed testing.

Xano’s picture

Status: Needs work » Needs review
FileSize
3.97 KB

Status: Needs review » Needs work

The last submitted patch, drupal_1911260_01.patch, failed testing.

Xano’s picture

Status: Needs work » Needs review

#3: drupal_1911260_01.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal_1911260_01.patch, failed testing.

ParisLiakos’s picture

Status: Needs work » Needs review

#3: drupal_1911260_01.patch queued for re-testing.

Crell’s picture

Status: Needs review » Reviewed & tested by the community

I do believe Xano is correct.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Makes sense. Committed/pushed to 8.x.

Status: Fixed » Closed (fixed)

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

Xano’s picture

Assigned: Xano » Unassigned