Check for RoleInterface instead of Role object

zerrvox · zerrvox · commit 5aae161f035b · 2012-08-20T15:57:18.000+03:00
It should not be necessary to extend the Core Role object when the RoleInterface is implemented.
diff --git a/src/Symfony/Component/Security/Acl/Domain/RoleSecurityIdentity.php b/src/Symfony/Component/Security/Acl/Domain/RoleSecurityIdentity.php
@@ -12,7 +12,7 @@
 namespace Symfony\Component\Security\Acl\Domain;
 
 use Symfony\Component\Security\Acl\Model\SecurityIdentityInterface;
-use Symfony\Component\Security\Core\Role\Role;
+use Symfony\Component\Security\Core\Role\RoleInterface;
 
 /**
  * A SecurityIdentity implementation for roles
@@ -30,7 +30,7 @@ final class RoleSecurityIdentity implements SecurityIdentityInterface
      */
     public function __construct($role)
     {
-        if ($role instanceof Role) {
+        if ($role instanceof RoleInterface) {
             $role = $role->getRole();
         }
 

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`namespace Symfony\Component\Security\Acl\Domain;`
`13`	`13`
`14`	`14`	`use Symfony\Component\Security\Acl\Model\SecurityIdentityInterface;`
`15`		`-use Symfony\Component\Security\Core\Role\Role;`
	`15`	`+use Symfony\Component\Security\Core\Role\RoleInterface;`
`16`	`16`
`17`	`17`	`/**`
`18`	`18`	`* A SecurityIdentity implementation for roles`
`@@ -30,7 +30,7 @@ final class RoleSecurityIdentity implements SecurityIdentityInterface`
`30`	`30`	`*/`
`31`	`31`	`public function __construct($role)`
`32`	`32`	`{`
`33`		`- if ($role instanceof Role) {`
	`33`	`+ if ($role instanceof RoleInterface) {`
`34`	`34`	`$role = $role->getRole();`
`35`	`35`	`}`
`36`	`36`