improved the UserRepository::refreshUser() method as in EntityUserProvider class.

Hugo Hamon · weaverryan · commit b85c98b62d46 · 2011-12-24T12:07:55.000-06:00
diff --git a/cookbook/security/entity_provider.rst b/cookbook/security/entity_provider.rst
@@ -344,15 +344,12 @@ The code below shows the implementation of the
 
         public function refreshUser(UserInterface $user)
         {
-            $username = $user->getUsername();
-
-            try {
-                $user = $this->loadUserByUsername($username);
-            } catch (UsernameNotFoundException $e) {
-                throw new UnsupportedUserException(sprintf('Unable to refresh active admin AcmeUserBundle:User object identified by "%s".', $username), null, 0, $e);
+            $class = get_class($user);
+            if (!$this->supportsClass($class)) {
+                throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', $class));
             }
 
-            return $user;
+            return $this->loadUserByUsername($user->getUsername());
         }
 
         public function supportsClass($class)

Original file line number	Diff line number	Diff line change
`@@ -344,15 +344,12 @@ The code below shows the implementation of the`
`344`	`344`
`345`	`345`	`public function refreshUser(UserInterface $user)`
`346`	`346`	`{`
`347`		`- $username = $user->getUsername();`
`348`		`-`
`349`		`- try {`
`350`		`- $user = $this->loadUserByUsername($username);`
`351`		`- } catch (UsernameNotFoundException $e) {`
`352`		`- throw new UnsupportedUserException(sprintf('Unable to refresh active admin AcmeUserBundle:User object identified by "%s".', $username), null, 0, $e);`
	`347`	`+ $class = get_class($user);`
	`348`	`+ if (!$this->supportsClass($class)) {`
	`349`	`+ throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', $class));`
`353`	`350`	`}`
`354`	`351`
`355`		`- return $user;`
	`352`	`+ return $this->loadUserByUsername($user->getUsername());`
`356`	`353`	`}`
`357`	`354`
`358`	`355`	`public function supportsClass($class)`