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.