Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

System.Web.Mvc.Html.SelectExtensions #12

Closed
eramires opened this issue Dec 28, 2014 · 5 comments
Closed

System.Web.Mvc.Html.SelectExtensions #12

eramires opened this issue Dec 28, 2014 · 5 comments
Assignees

Comments

@eramires
Copy link

Hey guys, i had to recompile the DLL to change the namespace of the DropDownListFor

For some reason it is in the wrong class.

in the original DLL it is in DropDownListExtensions

My framework is trying to find it in the SelectExtensions instead.

And place it into the System.Web.Mvc.Html namespace.

It works for me now that i recompiled it.

@johnnyreilly
Copy link
Owner

Hey @eramires,

Thanks for reporting. Which DLL did you have the issue with please? MVC 5?

@eramires
Copy link
Author

Hey, thanks for replying.

It was in the jQuery.Validation.Unobtrusive.Native itself.

Yes MVC 5.
In the DropDownListExtensions.cs file.

What i did was:

  1. Change the namespace to System.Web.Mvc.Html
  2. Change the class name to SelectExtensions.

That's all.

@johnnyreilly
Copy link
Owner

Thanks chap. I've just taken a quick look and it looks like the namespaces in MVC have all shifted to System.Web.Mvc.Html from System.Web.Mvc at some point (I'm sure my namespaces were correct once!)

Will take a look into this and try to ship a new version of the nuget package.

@eramires
Copy link
Author

Oh! I am sure it was correct John, it was not my intention to blame or anything :)

I think it could worth wait for the release of the new asp.net, I think its on preview yet :)

I appreciate your prompt reply on this!

Will certainly get the updated version when released in nuget.

Thanks man!

Happy new year!! ^_^

@johnnyreilly
Copy link
Owner

And you - and no blame taken. I think you've genuinely spotted a minor issue on the namespace front. It probably doesn't affect many people as they probably have both namespaces in their web.configs anyway. But it'd be good to sort it.

Happy new year to you too!

@johnnyreilly johnnyreilly self-assigned this Jan 7, 2015
johnnyreilly added a commit that referenced this issue Jan 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants