Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Dec 2012 at 21:49 UTC
Updated:
29 Jul 2014 at 21:38 UTC
Jump to comment: Most recent file
Comments
Comment #1
larowlanstraight forward
Comment #3
dawehnerActually it should be
Shouldn't it be even\Drupal\Core\Entity::EntityAccessControllerInterface()
Comment #4
larowlansure!
Comment #5
dawehnerThis looks great!
Comment #6
fubhy commentedActually I am pretty sure that we agreed not to have full namespaces there. Which is why we removed them at some point in the other issue. @see http://drupal.org/coding-standards/docs#namespaces
Comment #7
larowlanthe method names are wrong eg they're ::viewAccess() not ::view()
Comment #8
larowlanwhich is patch 1
Comment #9
larowlan#1: test-access-1864218.1.patch queued for re-testing.
Comment #10
larowlanPlease review patch 1
Comment #11
dawehnerI'm not sure as all new patches follow that standard at the moment (at least from my perspective).
Comment #12
berdirThis is still being discussed back and forth in #1487760: [policy, no patch] Decide on documentation standards for namespaced items. Most patches that I've seen recently don't use the full name if it's in the same namespace. I currently don't really care about that standard, it has been changed a bunch times already and we have dozens of wrong classes that will need to be fixed anyway, doesn't matter what we decide to do now :) Using non-namespaces versions for things that are in the same namespace or use'd sounds fine to me.
Comment #13
dawehnerHe, so all entity related patches work different to all CMI/VDC related patches :)
Let's RTBC patch #1
Comment #14
fubhy commentedOh... Yeah +1 rtbc. I read the issue too quickly sorry. Patch #1 is okay!
Comment #15
dries commentedCommitted to 8.x. Thanks.