NotificationsYou must be signed in to change notification settings
Fork9.8k
Star30.9k

Commit96c4486

committed

minor#9374 Change of scope (djoos)

This PR was submitted for the 2.3 branch but it was merged into the 2.5-dev branch instead (closes#9374).Discussion----------Change of scopeHi,when overriding the Symfony RoleHierarchy it would be great to be able to get access to the buildRoleMap-method and map-variable for more advanced usage.Don't hesitate to ask for further clarification if necessary - thanks in advance for merging the PR!Kind regards,DavidCommits-------621aaf1 Change of scope

2 parents5079f34 +335bee2 commit96c4486Copy full SHA for 96c4486

File tree

1 file changed

-2

lines changed

src/Symfony/Component/Security/Core/Role
- RoleHierarchy.php

1 file changed

-2

lines changed

`‎src/Symfony/Component/Security/Core/Role/RoleHierarchy.php‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@`
`19`	`19`	`class RoleHierarchyimplements RoleHierarchyInterface`
`20`	`20`	`{`
`21`	`21`	`private$hierarchy;`
`22`		`-private$map;`
	`22`	`+protected$map;`
`23`	`23`
`24`	`24`	`/**`
`25`	`25`	`* Constructor.`
`@@ -56,7 +56,7 @@ public function getReachableRoles(array $roles)`
`56`	`56`	`return$reachableRoles;`
`57`	`57`	`}`
`58`	`58`
`59`		`-privatefunctionbuildRoleMap()`
	`59`	`+protectedfunctionbuildRoleMap()`
`60`	`60`	`{`
`61`	`61`	`$this->map =array();`
`62`	`62`	`foreach ($this->hierarchyas$main =>$roles) {`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit96c4486

File tree

1 file changed

1 file changed

`‎src/Symfony/Component/Security/Core/Role/RoleHierarchy.php‎`

0 commit comments