Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
configuration entity system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Jan 2014 at 03:10 UTC
Updated:
29 Jul 2014 at 23:16 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
tim.plunkettComment #2
larowlanI think whilst this makes us more consistent, not sure it helps with the 'whoops I forgot to add admin_permission' perhaps we should add a watchdog notice before returning null in checkAccess?
otherwise rtbc
Comment #3
tim.plunkettHmm, access() is called a lot. Though I guess it doesn't fall through to the NULL part of checkAccess that often...
Putting a watchdog call in EntityAccessController feels a bit wrong though.
Comment #4
larowlanSounds like a good use case for #2165549: Reduce number of password hashing iterations in all tests to improve test performance
Comment #5
larowlanmeh thats not the issue, there's an issue about having a dev env.
Comment #6
alexpottCommitted 723e5ea and pushed to 8.x. Thanks!