ASP.NET MVC UpdateModel with interface

Question

I am trying to get UpdateModel to populate a model that is set as only an interface at compile-time. For example, I have:

// View Model
public class AccountViewModel {
  public string Email { get; set; }
  public IProfile Profile { get; set; }
}

// Interface
public interface IProfile {
  // Empty
}

// Actual profile instance used
public class StandardProfile : IProfile {
  public string FavoriteFood { get; set; }
  public string FavoriteMusic { get; set; }
}

// Controller action
public ActionResult AddAccount(AccountViewModel viewModel) {
  // viewModel is populated already
  UpdateModel(viewModel.Profile, "Profile"); // This isn't working.
}

// Form
<form ... >
  <input name='Email' />
  <input name='Profile.FavoriteFood' />
  <input name='Profile.FavoriteMusic' />
  <button type='submit'></button>
</form>

Also note that I have a custom model binder that inherits from DefaultModelBinder being used that populates IProfile with an instance of StandardProfile in the overriden CreateModel method.

The problem is that FavoriteFood and FavoriteMusic are never populated. Any ideas? Ideally this would all be done in the model binder, but I'm not sure it is possible without writing a completely custom implementation.

Thanks, Brian

Answer 1

I would have to check the ASP.NET MVC code (DefaultModelBinder) but I'm guessing that its reflecting on the type IProfile, and not the instance, StandardProfile.

So it looks for any IProfile members it can try to bind, but its an empty interface, so it considers itself done.

You could try something like updating the BindingContext and changing the ModelType to StandardProfile and then calling

bindingContext.ModelType = typeof(StandardProfile);
IProfile profile = base.BindModel(controllerContext, bindingContext);

Anyways, having an empty Interface is weird~

---

Edit: just want to add that code above is just pseudo code, you would need to check DefaultModelBinder to see exactly what you want to write.

---

Edit#2:

Can you do:

public class ProfileModelBinder : DefaultModelBinder
{
    public override object BindModel(ControllerContext controllerContext, ModelBindingContext bindingContext) {
    {
        bindingContext.ModelType = typeof(StandardProfile);
        return base.BindModel(controllerContext, bindingContext);
    }
}

No need to make a model binder for AccountView, that one works fine.

---

Edit #3

Tested it out, the above binder works, just need to add:

ModelBinders.Binders[typeof(IProfile)] = new ProfileModelBinder();

Your action looks like:

public ActionResult AddAccount(AccountViewModel viewModel) {
    // viewModel is fully populated, including profile, don't call UpdateModel
}

You can use IOC when setting the model binder (have the type constructor injected for instance).

Answer 2

Not inspecting the actual type behind the interface was discussed here: http://forums.asp.net/t/1348233.aspx

That said, I found a hackish way around the problem. Since I already had a custom model binder for this type, I was able to add some code to it to perform the binding for me. Here's what my model binder looks like now:

public class AccountViewModelModelBinder : DefaultModelBinder
{
    private readonly IProfileViewModel profileViewModel;
    private bool profileBound = false;

    public AccountViewModelModelBinder(IProfileViewModel profileViewModel)
    {
        this.profileViewModel = profileViewModel;
    }

    protected override void OnModelUpdated(ControllerContext controllerContext, ModelBindingContext bindingContext)
    {
        // Bind the profile
        if (profileBound)
            return;

        profileBound = true;

        bindingContext.ModelType = profileViewModel.GetType();
        bindingContext.Model = profileViewModel;
        bindingContext.ModelName = "Profile";

        BindModel(controllerContext, bindingContext);
    }

    protected override object CreateModel(ControllerContext controllerContext, ModelBindingContext bindingContext, System.Type modelType)
    {
        var model = new AccountViewModel();
        model.Profile = profileViewModel;

        return model;
    }
}

Basically, when the model binder is "done" binding the main AccountViewModel, I then alter the binding context (as suggested by eyston) and call BindModel once again. This then binds my profile. Note that I called GetType on the profileViewModel (which is supplied by the IOC container in the constructor). Also notice that I include a flag to indicate if the profile model has been bound already. Otherwise there would be an endless loop of OnModelUpdated being called.

I'm not saying this is pretty, but it does work well enough for my needs. I'd still love to hear about other suggestions.