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

Comments

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Active » Needs review
StatusFileSize
new drupal_1911260_00.patch3.93 KB

Status: Needs review » Needs work

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

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Needs work » Needs review
StatusFileSize
new drupal_1911260_01.patch3.97 KB

Status: Needs review » Needs work

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

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
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

ParisLiakos commented
Status: Needs work » Needs review

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

Crell’s picture

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

I do believe Xano is correct.

catch’s picture

catch
he/him
Primary language English
commented
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

xano
he/they
Primary language English
Location Southampton
commented
Assigned: xano » Unassigned