Commits

christiansalazar committed 0941aa5

FIX ISSUE #51

  • Participants
  • Parent commits d4b84e4

Comments (0)

Files changed (1)

components/CrugeUserManager.php

             $user->username =
                 $this->generateNewUsername($user->email);
         }
-
         // la establece como "Activada"
         $this->activateAccount($user);
         // ahora a ponerle una clave
         $this->changePassword($user, $password);
-        // guarda usando el API, lo que lo hace pasar por el sistema de filtros.
+		// revisa que no duplique
+        $_prev = CrugeFactory::get()->getICrugeStoredUser($user->username);
+		if($_prev != null)
+            throw new CrugeException("nombre usuario duplicado.");
+        $_prev = CrugeFactory::get()->getICrugeStoredUser($user->email);
+		if($_prev != null)
+            throw new CrugeException("correo duplicado.");
+        // guarda usando el API. No pasa por filtros. se reparara a futuro.
         if ($this->save($user)) {
             foreach ($mapped_values as $fieldname => $value) {
                 if (($fieldname == 'username') || ($fieldname == 'email')) {