ASP.NET MVC: return Redirect y ViewData

Question

Tengo un cuadro de inicio de sesión en mi página maestra. Cuando la información de inicio de sesión no es correcta, valorizo ​​ViewData["loginError"] para mostrar el mensaje de error al usuario.

El inicio de sesión es una acción del UserController, por lo que el formulario que contiene el inicio de sesión tiene action = "/User/Login".

Como un usuario puede intentar iniciar sesión desde cualquier página, en caso de éxito lo redirigiré a su página personal, pero en caso de error quiero que permanezca en la misma página donde intentó iniciar sesión. He encontrado que esto funciona:

return Redirect(Request.UrlReferrer.ToString()); 

pero parece que, ya que no estoy devolviendo visualizar correctamente, los datos sobre ViewData se pierde, así que no puedo mostrar el mensaje de error.

¿Alguna sugerencia sobre cómo solucionar este y problemas similares?

Gracias

Answer 1

Probablemente desee utilizar la propiedad TempData, esto se mantendrá hasta la siguiente solicitud HTTP.

Answer 2

Por qué no manejar la entrada a través de AJAX en vez de un post completo? Puede proporcionar fácilmente el estado, una URL de redireccionamiento y cualquier mensaje de error a través de JSON.

public ActionResult Logon(string username, string password) 
{ 
    ... 

    // Handle master page login 
    if (Request.IsAjaxRequest()) 
    { 
      if (success) 
      { 
       return Json(new { Status = true, Url = Url.Action("Index", "Home") }); 
      } 
      else 
      { 
       return Json(new { Status = false, Message = ... }); 
      } 
    } 
    else // handle login page logon or no javascript 
    { 
      if (success) 
      { 
       return RedirectToAction("Index", "Home"); 
      } 
      else 
      { 
       ViewData["error"] = ... 
       return View("Logon"); 
      } 
     } 
    } 

-lado del cliente

$(function() { 
     $('#loginForm input[type=submit]').click(function() { 
      $('#loginError').html(''); 
      $.ajax({ 
      url: '<%= Url.Action("Logon","Account") %>', 
      dataType: 'json', 
      type: 'post', 
      data: function() { return $('#loginForm').serialize(); }, 
      success: function(data,status) { 
       if (data.Status) { 
        location.href = data.Url; 
       } 
       else { 
        $('#loginError').html(data.Message); 
       } 
      } 
      }); 
      return false; 
     }); 
    }); 

Answer 3

Normalmente, para la mayoría de los sitios web, cuando el usuario deja de autenticar (debido a la contraseña o algo así), se irá a otra página que ayudan al usuario (como la contraseña recuperar, o pedirle al usuario que se registre) que es raro quedarse en la misma página. Creo que puedes volver a considerar que realmente necesitas la navegación que estás usando.

OK, una solución si realmente quiere apegarse a su modelo, es que puede adjuntar el error de inicio de sesión a la URL. Por ejemplo, http://www.example.com/index.aspx?login_error=1 indica se produce ese error, y se puede utilizar BEGIN_REQUEST (o módulo HTTP) para capturar este, y decirle al modelo de estado acerca del error:

ModelState.AddModelError(...); 

Por cierto, añadir el error del modelo es en realidad una forma más adecuada informar a la vista sobre cualquier error en lugar de usar ViewState (esto es similar a lanzar una excepción vs devolver un entero sobre el resultado de la ejecución en días pasados).

Si bien utilizar AJAX para iniciar sesión (como lo sugiere tvanfosson) es perfectamente alcanzable y a veces se destaca en la experiencia del usuario, la publicación completa clásica sigue siendo irremplazable (considérese que algunos usuarios inhabilitarán javascript o incluso en mi volcado el teléfono WM6 que no es compatible con javascript).

Answer 4

Estoy confundido. No

return View(); 

¿Devolver la página actual a usted?

Por lo tanto, en su caso, cuando falla el inicio de sesión, configure su viewdata y llame a return View();

decir

if (!FailedLogin) { 
    //Go to success page 
}else{ 
    //Add error to View Data or use ModelState to add error 
    return View(); 
} 

¿Está utilizando el [Autorizar] decorador? El proceso de inicio de sesión de MVC solicita automáticamente con la página de inicio de sesión y luego lo regresa a la acción del controlador que estaba tratando de ejecutar. Disminuye un montón de redirigir.

Answer 5

El siguiente ejemplo es de esperar que ayudaría a cabo para resolver este problema:

View.aspx

<%= Html.ValidationSummary("Login was unsuccessful. Please correct the errors and try again.") %> 
<% using (Html.BeginForm()) { %> 
    <div> 
     <fieldset> 
      <legend>Account Information</legend> 
      <p> 
       <label for="username">Username:</label> 
       <%= Html.TextBox("username") %> 
       <%= Html.ValidationMessage("username") %> 
      </p> 
      <p> 
       <label for="password">Password:</label> 
       <%= Html.Password("password") %> 
       <%= Html.ValidationMessage("password") %> 
      </p> 
      <p> 
       <%= Html.CheckBox("rememberMe") %> <label class="inline" for="rememberMe">Remember me?</label> 
      </p> 
      <p> 
       <input type="submit" value="Log On" /> 
      </p> 
     </fieldset> 
    </div> 
<% } %> 

AccountController.cs

private bool ValidateLogOn(string userName, string password) 
{ 
    if (String.IsNullOrEmpty(userName)) 
    { 
     ModelState.AddModelError("username", "You must specify a username."); 
    } 
    if (String.IsNullOrEmpty(password)) 
    { 
     ModelState.AddModelError("password", "You must specify a password."); 
    } 
    if (!MembershipService.ValidateUser(userName, password)) 
    { 
     ModelState.AddModelError("_FORM", "The username or password provided is incorrect."); 
    } 
    return ModelState.IsValid; 
} 

Usted no será capaz de capturar la información agregada en ViewData después de la acción Redirigir. por lo tanto, el enfoque correcto es devolver la misma Vista() y usar el Estado del modelo para los errores mencionados también por "xandy".

Espero que esto le de una ventaja con la validación de formulario.