This is a follow up to a previous question that I had before about passing an error back to the client, but also pertains to the ModelState.
Has anyone successful used the Nerd Dinner approach, but with Ajax? So Nerd Dinner does an update as so.
[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Edit(int id, FormCollection formValues)
{
Dinner dinner = dinnerRepository.GetDinner(id);
try
{
UpdateModel(dinner);
dinnerRepository.Save();
return RedirectToAction("Details", new { id=dinner.DinnerID });
}
catch
{
foreach (var issue in dinner.GetRuleViolations()) {
ModelState.AddModelError(issue.PropertyName, issue.ErrorMessage);
}
return View(dinner);
}
}
Using jQuery $.ajax
function hijack(form, callback, errorFunction, format) {
$.ajax({
url: form.action,
type: form.method,
dataType: format,
data: $(form).serialize(),
success: callback,
error: function(xhr, textStatus, errorThrown) {
errorFunction(xhr, textStatus, errorThrown);
}
});
}
Ajax, the "try" part of the controller becomes
try
{
UpdateModel(dinner);
dinnerRepository.Save();
return PartialView("PartialDetails", new { id=dinner.DinnerID });
}
, but what do you do about the catch part?
A simple error handling solution to send back an error would be
catch(Exception ex)
{
Response.StatusCode = 500;
return Content("An Error occured.");
//throw ex;
}
, but that doesn't pass through the robust modelstate built into MVC. I thought of a number of options, but I really want 2 things:
Is this possible? If not, what are the best alternatives that you know of?
Many thanks.
Update I haven't marked this as answered yet, because I haven't yet implemented what I think will work best.
I have decided that I don't really like the success => send refreshed list, failure => send error message approach that I was taking. I did this to reduce the number of calls, but a refreshed list is really being set to the page. Trying to do both tightly binds the popup to its overall page.
I am going to add a custom jQuery event refresh the master page list when the dialog closes. In essence, it's the observer pattern. I like the idea that the page says to the popup "tell me when you're done" (aka closed), without having to tell the popup why. It does require an additional call, but I don't see that as a big issue.
I'm still not sure how well that I like/dislike server-side validation and I'm considering going with client-side only validation. While server side validation seems like clean layering, it also has a number of problems, including:
1) It puts quality checks at the end, instead of the beginning. An analogy to manufacturing would be a car that's tested when it arrives at the dealer, instead at the points in the process where it's being built.
2) It violates the intent of Ajax. Ajax isn't just about sending asynchronous events, it's also about sending only what I need and receiving only what I need. Sending back the entire modelstate in order to provide error details doesn't seem to go with Ajax.
What I'm thinking about doing is having client-side only validation, but that server code and a custom viewmodel can be used to tell the client how to dynamically create those validation rules.
I also suspect that a dynamic language like IronRuby or IronPython might offer a more elegant way to solve these problems, but it could be a little longer before I look into that possibility.
When MVC creates the model state for the submitted properties, it also iterates through each property in the view model and validates the property using attributes associated to it. If any errors are found, they are added to the Errors collection in the property's ModelState .
The MVC Framework contains built-in support for unobtrusive Ajax. You can use the helper methods to define your Ajax features without adding a code throughout all the views. This feature in MVC is based on the jQuery features. To enable the unobtrusive AJAX support in the MVC application, open the Web.
Clear() is required to display back your model object. If you are getting your Model from a form and you want to manipulate the data that came from the client form and write it back to a view, you need to call ModelState. Clear() to clean the ModelState values.
ModelState. IsValid property can be used to perform some logical operations based on the values submitted by the User. Note: If you want to learn about Client Side validations in ASP.Net MVC Razor, please refer ASP.Net MVC: Client Side validations using Data Annotation attributes and jQuery.
If I understand what you are trying to do, my first answer will be no, you cannot use the model state as it is through an Ajax request.
You may be able to emulate the ModelState
behavior, to display the errors:
Passing a List<KeyValuePair<string,string>>
(property,message) by JSON (this will require you to pass the modelErrors from the ModelState
to the new structure) and do the HTML construction of a Validation Summary by JS/jQuery (which I think is over killing solution).
If you are going to the server, and there are any errors, just do a render partial of the Html.ValidationSummary()
, pass it through JSON, and prepend it to the form. If everything was OK, just return the PartialDetails
view and replace the actual content. This will require some kind of status parameter so you know what is coming back from the server on the ajax callback.
Edit: This last option sounds good but tricky, because you will need to return a partial view in a string form by JSONResult
. Here is a question and solution about that hack: How to render an ASP.NET MVC view as a string?.
Personally, I don't think that using the error attribute will do any good at all. I just use it in very specific situations like timeout errors and server exceptions, not app exceptions.
Edit:
Using JSON:
[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Edit(int id, FormCollection formValues)
{
Dinner dinner = dinnerRepository.GetDinner(id);
try
{
UpdateModel(dinner);
dinnerRepository.Save();
return Json(new
{
result = "success",
html = this.RenderToString("PartialDetails", dinner)
});
}
catch
{
foreach (var issue in dinner.GetRuleViolations())
{
ModelState.AddModelError(issue.PropertyName, issue.ErrorMessage);
}
return Json(new
{
result = "failed",
html = this.RenderToString("PartialEdit", dinner)
});
}
}
Here the result
parameter will let you know what action to do in each case, just have to check it on the callback.
Adding my prefered approach with MVC3. Using your edit method you could do something like
[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Edit(int id, FormCollection formValues)
{
Dinner dinner = dinnerRepository.GetDinner(id);
try
{
UpdateModel(dinner);
dinnerRepository.Save();
return Json (new { Success = true, Url = Url.Action("DetailsPartial", dinner), Div = "#DivToUpdateId" });
}
catch
{
foreach (var issue in dinner.GetRuleViolations()) {
ModelState.AddModelError(issue.PropertyName, issue.ErrorMessage);
}
//I am replacing this with a partial view which will contain the model state errors. For people using MVC 3 with razor they should be able to use their normal views as partials
return PartialView(dinner);
}
}
and then you can use a success function like
success: function(data) {
if (data.Success) {
$.post(data.Url, function(partial) {
$(data.Div).html(partial);
});
}
else
{
$('#formDiv').html(data)
}
}
so basically, if the result is Json then data.Success is true. It then updates the div specified in the json (data.Div) with the results of the action specified in the Url. If the result is not Json its because the post method failed and the formDiv is updated with the partial form that contains the ModelState Errors. This is the approach i use for dialogs but it works pretty well. Obviously with MVC3 i'd change some of the edit method like using TryUpdateModel etc, but just trying to give a example of my approach. By passing the url and posting the results of the method there is no need to try to render html to a string in order to pass a partial.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With