UrlHelper.Action includes undesired additional parameters

Question

I have a method in the controller ApplicationsController, in which I need to get the base URL for an action method:

public ActionResult MyAction(string id)
{
    var url = Url.Action("MyAction", "Applications");
    ...
}

The problem is that this includes the string id from the current route data, when I need the URL without (the URL is used to fetch content from a CMS on a URL-based lookup).

I have tried passing null and new { } as the routeValues parameter to no avail.

The matching route is as follows (above all other routes):

routes.MapLowercaseRoute(
    name: "Applications",
    url: "applications/{action}/{id}",
    defaults: new { controller = "Applications",
                    action = "Index", id = UrlParameter.Optional });

I've seen a couple of other questions touch on this but none of them seem to have a viable solution. At present, I am resorting to hardcoding the path in the controller; however, I'd like to be able to abstract this into an action filter, so I need to be able to generate the URL.

Is there a clean/conventional way to prevent this behaviour?

Answer 1

Ended up getting around this with a different approach. The only way I could come up with to prevent arbitrarily-named route values from being inserted into the generated URL was to temporarily remove them from RouteData when calling Url.Action. I've written a couple of extension methods to facilitate this:

public static string NonContextualAction(this UrlHelper helper, string action)
{
    return helper.NonContextualAction(action,
        helper.RequestContext.RouteData.Values["controller"].ToString());
}

public static string NonContextualAction(this UrlHelper helper, string action,
                                         string controller)
{
    var routeValues = helper.RequestContext.RouteData.Values;
    var routeValueKeys = routeValues.Keys.Where(o => o != "controller"
                         && o != "action").ToList();

    // Temporarily remove routevalues
    var oldRouteValues = new Dictionary<string, object>();
    foreach (var key in routeValueKeys)
    {
        oldRouteValues[key] = routeValues[key];
        routeValues.Remove(key);
    }

    // Generate URL
    string url = helper.Action(routeValues["Action"].ToString(),
                               routeValues["Controller"].ToString());

    // Reinsert routevalues
    foreach (var kvp in oldRouteValues)
    {
        routeValues.Add(kvp.Key, kvp.Value);
    }

    return url;
}

This allows me to do this in an action filter where I won't necessarily know what the parameter names for the action are (and therefore can't just pass an anonymous object as in the other answers).

Still very much interested to know if someone has a more elegant solution, however.

Answer 2

Ok, I see the problem. It's something called "Segment variable reuse". When generating the routes for outbound URLs, and trying to find values for each of the segment variables in a route’s URL pattern, the routing system will look at the values from the current request. This is a behavior that confuses many programmers and can lead to a lengthy debugging session. The routing system is keen to make a match against a route, to the extent that it will reuse segment variable values from the incoming URL. So I think you have to override the value like Julien suggested :

var url = Url.Action("MyAction", "Applications", new { id = "" })

Answer 3

Use a null or empty value for id to prevent Url.Action from using the current one:

var url = Url.Action("MyAction", "Applications", new { id = "" })

Answer 4

I was not entirely comfortable with the altering, transient or otherwise, of the RouterData in @AntP's otherwise fine solution. Since my code for creating the links was already centralized, I borrowed @Tomasz Jaskuλa and @AntP to augment the ExpandoObject, I was already using.

IDictionary<string,object> p = new ExpandoObject();

// Add the values I want in the route
foreach (var (key, value) in linkAttribute.ParamMap)
{
    var v = GetPropertyValue(origin, value);                    
    p.Add(key, v); 
}

// Ideas borrowed from https://stackoverflow.com/questions/20349681/urlhelper-action-includes-undesired-additional-parameters
// Null out values that I don't want, but are already in the RouteData
foreach (var key in _urlHelper.ActionContext.RouteData.Values.Keys)
{
    if (p.ContainsKey(key))
        continue;

    p.Add(key, null); 
}

var href = _urlHelper.Action("Get", linkAttribute.HRefControllerName, p);