User -> UserInterface

ikovalyov · ikovalyov · commit 9bb86ae1314a · 2017-08-01T15:58:27.000+02:00
diff --git a/src/Service/UserManager.php b/src/Service/UserManager.php
@@ -3,6 +3,7 @@
 
 use ProspectOne\UserModule\Entity\Role;
 use ProspectOne\UserModule\Entity\User;
+use ProspectOne\UserModule\Interfaces\UserInterface;
 use Zend\Crypt\Password\Bcrypt;
 use Zend\Math\Rand;
 use Doctrine\ORM\EntityManager;
@@ -98,28 +99,32 @@ public function addUser($data)
 
     /**
      * This method updates data of an existing user.
-     * @param User $user
+     * @param UserInterface $user
      * @param $data
      * @return bool
      * @throws \Exception
      */
-    public function updateUser(User $user, $data)
+    public function updateUser(UserInterface $user, $data)
     {
         // Do not allow to change user email if another user with such email already exits.
         if($user->getEmail()!=$data['email'] && $this->checkUserExists($data['email'])) {
             throw new \Exception("Another user with email address " . $data['email'] . " already exists");
         }
-        
+
+        if (!($user instanceof User)) {
+            throw new \LogicException("Only instances of " . User::class . " should be passed");
+        }
+
         $user->setEmail($data['email']);
-        $user->setFullName($data['full_name']);        
+        $user->setFullName($data['full_name']);
         $user->setStatus($data['status']);
 
         // Get role object based on role Id from form
         /** @var Role $role */
         $role = $this->entityManager->find(Role::class, $data['role']);
         // Set role to user
         $user->addRole($role);
-        
+
         // Apply changes to database.
         $this->entityManager->flush();
 
