#2109035: Make access checkers (much) easier to find replaced StaticAccessCheckInterface

Patch attached which renames to use AccessInterface

CommentFileSizeAuthor
devel-switchacess.patch852 bytesbenjy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

benjy’s picture

Status: Active » Needs review
benjy’s picture

Title: Static Access Check Interface replaced in core » StaticAccessCheckInterface replaced in core

removed spaces from title.

benjy’s picture

Priority: Normal » Critical

Bumping to critical because this stops devel working with D8 right now.

rjacobs’s picture

Yeah. I was able to install devel fine in D8 alpha-7, but I just tried recently to do it on another test environment based on D8 HEAD, and got a WSOD trying to enable devel (PHP Fatal error: Interface 'Drupal\Core\Access\StaticAccessCheckInterface' not found). The patch addresses this.

Seems simple enough of a fix, though I'm still new enough to D8 to not fully comprehend all the changes from #2109035. Anyway, consider this a confirming review.

h3rj4n’s picture

Status: Needs review » Reviewed & tested by the community

The patch works for me.

pcambra’s picture

Category: Bug report » Task
Priority: Critical » Normal
Status: Reviewed & tested by the community » Fixed

Thanks a lot for the fix, pushed to the 8.x branch.

Maybe is just me, but I think we shouldn't open "critical bugs" against something is not a stable version for drupal or devel. Keeping up to date with 8.x is hard enough :P

Status: Fixed » Closed (fixed)

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