Skip to main content
4 de 4
se añadieron 11 caracteres en el cuerpo
devconcept
  • 12.8k
  • 3
  • 42
  • 57

El problema es que el login es una función asincrónica (utiliza ajax) por lo que estas enviando la petición e intentando recoger los errores resultantes al mismo tiempo. La solución mas sencilla es retornar una promesa y recoger los errores luego de que esta se haya terminado.

Cambia tu función authFactory.logIn a esto

authFactory.logIn = function(user){
    // al retornar la llamada a la función estas devolviendo un promise implicitamente
    return $http({
            method: 'POST',
            skipAuthorization: true,
            url: 'login.php',
            data: { op:'login', username: user.username, password: user.password } ,
            headers: {'Content-Type': 'application/x-www-form-urlencoded'}
    }).then(function successCallback(response) {
        console.log("Entrando al success");
        // manejas primero el error
        if ('error' in response.data) {
            console.log("Login Erroneo");
            // retornas una promesa rechazada
            return $q.reject(response.data);
        }
        // Si no ocurrieron errores la petición es satisfactoria
        console.log("Login Exitoso");
        LocalStore.saveJWT(response.data.jwt);
        
        if(authFactory.isLogIn){
            $location.path("/home");
        }else{
            $location.path("/login");
        }
        // Retornas el token para encadenar un then
        return response.data.jwt;
  }).catch(function errorCallback(response) {
     console.log("Login Fallido jajajajaj");
     // Retornas el contenido del error
     return $q.reject(response.data.error || response.data);
  });
};

y luego en tu controller haces esto

$scope.logIn = function(user){
    authFactory.logIn(user).then(function() {
        // aquí pones lo que quieras
    }).catch(function(error) {
        // aquí manejas el error
        // ya el método authFactory.getError() sobra
        $scope.error = error;
    });
};

También puedes hacer

$scope.logIn = function(user){
    // ya que el factory lo hace todo no es necesario un método then, solo recoger el error
    authFactory.logIn(user).catch(function(error) {
        // aquí manejas el error
        $scope.error = error;
    });
};

Para que esto funcione debes inyectar $q como dependencia porque uso $q.reject en una parte del código del Factory

No uses then(successFn, errorFn) en su lugar usa then(successFn).catch(errorFn), el primero es un anti patrón.

Debes retornar algo (return [objeto]) en el cuerpo del then o el catch para seguir encadenando, de lo contrario JavaScript retornará return undefined; automáticamente por ti.

devconcept
  • 12.8k
  • 3
  • 42
  • 57