Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

ValidationMessageFor together with AddModelError(key, message). What's the key?

I am developing a client-side and server-side validation for a certain viewModel property.

In the .cshtml file I put this:

@Html.DropDownListFor(model => model.EntityType.ParentId, Model.ParentTypeList, "")
@Html.ValidationMessageFor(model => model.EntityType.ParentId)

In the Controller for the business validation

catch (BusinessException e)
{
    ModelState.AddModelError("EntityType.ParentId", Messages.CircularReference);
}

The above works as expected: if an exception is caught, the message appears next to the dropdownlist.

However, I find that this way is not very elegant. In the cshtml, I use a method to generate all the required information about the validation. In the controller, I must know the exact Key string and use it.

Isn't there a better way of doing this?

like image 695
Carlos Magno Rosa Avatar asked Oct 02 '12 09:10

Carlos Magno Rosa


2 Answers

You could write an extension method that will take a lambda expression for the key instead of a string:

public static class ModelStateExtensions
{
    public static void AddModelError<TModel, TProperty>(
        this ModelStateDictionary modelState, 
        Expression<Func<TModel, TProperty>> ex, 
        string message
    )
    {
        var key = ExpressionHelper.GetExpressionText(ex);
        modelState.AddModelError(key, message);
    }
}

and then use this method:

catch (BusinessException e)
{
    ModelState.AddModelError<MyViewModel, int>(
        x => x.EntityType.ParentId, 
        Messages.CircularReference
    );
}
like image 122
Darin Dimitrov Avatar answered Nov 16 '22 02:11

Darin Dimitrov


I follow @Darin Dimitrov solution but i want to avoid <MyViewModel, int> so I used some different way but for that you need MyViewModel object variable.

public static class ModelStateExtensions
{
    public static void AddModelError<TModel, TProperty>(this TModel source,        
                                                    Expression<Func<TModel, TProperty>> ex, 
                                                    string message,
                                                    ModelStateDictionary modelState)
    {
        var key = System.Web.Mvc.ExpressionHelper.GetExpressionText(ex);
        modelState.AddModelError(key, message);
    }
}

How to Use:

catch (BusinessException e)
{
    objMyViewModel.AddModelError(x => x.EntityType.ParentId, 
                                 Messages.CircularReference,
                                 ModelState);
}
like image 30
Rikin Patel Avatar answered Nov 16 '22 02:11

Rikin Patel