feature #10792 [Security] Allow overloading ContextListener::refreshUser() (lstrojny)

fabpot · fabpot · commit e7898218696c · 2014-06-03T22:56:38.000+02:00
This PR was merged into the 2.6-dev branch.

Discussion
----------

[Security] Allow overloading ContextListener::refreshUser()

Allow overloading refreshUser() for the use case of doing something special with user providers.

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n.A.
| License       | MIT
| Doc PR        | n.A.

Commits
-------

b67ed43 Allow overloading ContextListener::refreshUser()
diff --git a/Firewall/ContextListener.php b/Firewall/ContextListener.php
@@ -142,7 +142,7 @@ public function onKernelResponse(FilterResponseEvent $event)
      *
      * @throws \RuntimeException
      */
-    private function refreshUser(TokenInterface $token)
+    protected function refreshUser(TokenInterface $token)
     {
         $user = $token->getUser();
         if (!$user instanceof UserInterface) {

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ public function onKernelResponse(FilterResponseEvent $event)`
`142`	`142`	`*`
`143`	`143`	`* @throws \RuntimeException`
`144`	`144`	`*/`
`145`		`- private function refreshUser(TokenInterface $token)`
	`145`	`+ protected function refreshUser(TokenInterface $token)`
`146`	`146`	`{`
`147`	`147`	`$user = $token->getUser();`
`148`	`148`	`if (!$user instanceof UserInterface) {`