search

aspnet / live.asp.net

Code for live.asp.net, which hosts the ASP.NET Community Stand-up
https://live.asp.net/
MIT License
289 stars 114 forks source link

Remove ObjectMapper #125

Closed benaadams closed 7 years ago

benaadams commented 7 years ago

Just an experiment to see if Interfaces can be used instead of object mapping.

Full model binding to interface can't be done as while MVC will instantiate from DI; it won't then write the properties from the input stream https://github.com/aspnet/Mvc/issues/6014#issuecomment-288471434

Luckily there is no interface model binding on this occasion

e.g. AdminModel : IAdminViewModel, IAdminInputModel

So that's not a problem here.

dnfclas commented 7 years ago

@benaadams, Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request. Thanks, .NET Foundation Pull Request Bot

benaadams commented 7 years ago

Don't know if it works; are no tests 😉