Merge pull request #5 from arcanys/master

address issue where new attributes are not being stored in the User E…
djaney · Nov 16, 2015 · 17c263b · 17c263b
2 parents 5538031 + 92edf25
commit 17c263b
Showing 1 changed file with 14 additions and 15 deletions.
diff --git a/Security/User/SSOUserProvider.php b/Security/User/SSOUserProvider.php
@@ -30,22 +30,21 @@ public function loadUserByUsername($userData)
             $user = new $this->class();
             $new = true;
         }
-        if(isset($userData['username']))
-            $user->setUsername($userData['username']);
-        if(isset($userData['email']))
-            $user->setEmail($userData['email']);
-        if(isset($userData['firstname']))
-            $user->setFirstname($userData['firstname']);
-        if(isset($userData['lastname']))
-            $user->setLastname($userData['lastname']);
-        if(isset($userData['roles']))
-            $user->setRoles($userData['roles']);
-        if(isset($userData['token']))
-            $user->setToken($userData['token']);
 
-       if($new){
-           $this->doctrine->getManager()->persist($user);
-       }
+        foreach ($userData as $name => $value) {
+            $set_method = 'set'.ucfirst($name);
+            $get_method = 'get'.ucfirst($name);            
+            if (method_exists($user, $set_method) && method_exists($user, $get_method)) {
+                if($user->$get_method() != $value) {
+                    $user->$set_method($value);
+                    $new = true;
+                }
+            }            
+        }
+
+        if($new){
+            $this->doctrine->getManager()->persist($user);
+        }
 
         $this->doctrine->getManager()->flush();