Aspnetcore: Allow the full controller/action name in RedirectToAction

Created on 4 Jul 2020 · 4Comments · Source: dotnet/aspnetcore

The overloads of RedirectToAction method which take a controller string only works when the controller name is sent without the "Controller" suffix (works with "Home" but not with "HomeController"). It'd be neat if it works both cases to allow the usage of nameof operator.

Same goes with action names but with the "Async" suffix".

affected-few area-mvc enhancement severity-minor

Source

Youssef1313

👍2

Most helpful comment

@MCCshreyas Although the "Controller" suffix is redundant, but allowing its existence has benefit of the ability of using the nameof operator as I mentioned.

If the issue is addressed, then the developer can pass either "Home" as literal string (which you seem to prefer), or nameof(HomeController) which is great IMO (for renaming and navigation).

Youssef1313 on 4 Jul 2020

👍3

All 4 comments

I mean it's way of thinking that the name of class which gets passed to its will be a controller, so mentioning of HomeController with 'controller' postfix again will be redundant in this case!

MCCshreyas on 4 Jul 2020

@MCCshreyas Although the "Controller" suffix is redundant, but allowing its existence has benefit of the ability of using the nameof operator as I mentioned.

If the issue is addressed, then the developer can pass either "Home" as literal string (which you seem to prefer), or nameof(HomeController) which is great IMO (for renaming and navigation).

Youssef1313 on 4 Jul 2020

👍3

@Youssef1313 ohh. Got your point!

MCCshreyas on 5 Jul 2020

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.