ASP.Net MVC RouteData and arrays

Question

If I have an Action like this:

public ActionResult DoStuff(List<string> stuff)
{
   ...
   ViewData["stuff"] = stuff;
   ...
   return View();
}

I can hit it with the following URL:

http://mymvcapp.com/controller/DoStuff?stuff=hello&stuff=world&stuff=foo&stuff=bar

But in my ViewPage, I have this code:

<%= Html.ActionLink("click here", "DoMoreStuff", "MoreStuffController", new { stuff = ViewData["stuff"] }, null) %>

Unfortunately, MVC is not smart enough to recognize that the action takes an array, and unrolls the list to form the proper url route. instead it just does a .ToString() on the object which just lists the data type in the case of a List.

Is there a way to get Html.ActionLink to generate a proper URL when one of the destination Action's parameters is an array or list?

-- edit --

As Josh pointed out below, ViewData["stuff"] is just an object. I tried to simplify the problem but instead caused an unrelated bug! I'm actually using a dedicated ViewPage<T> so I have a tightly coupled type aware Model. The ActionLink actually looks like:

<%= Html.ActionLink("click here", "DoMoreStuff", "MoreStuffController", new { stuff = ViewData.Model.Stuff }, null) %>

Where ViewData.Model.Stuff is typed as a List

ViewData["stuff"] is just an object. What happens when you pass in a real list, like {Stuff= (List<string>)ViewData["stuff"]} or {Stuff= ViewData["stuff"] as List<string>} or {Stuff= new List<string>(...)}? — Josh Pearce
– Josh Pearce, Commented Nov 18, 2009 at 0:04
same problem... In my actual implementation currently, I use tightly coupled ViewPage<T> so that line looks more like: <%= Html.ActionLink("click here", "DoMoreStuff", "MoreStuffController", new { stuff = ViewData.Model.Stuff }, null) %> where ViewData.Model.Stuff is typed as a List<string> — puffpio
– puffpio, Commented Nov 18, 2009 at 0:13

tvanfosson · Accepted Answer · 2011-06-27 22:57:44Z

I'm thinking that a custom HtmlHelper would be in order.

 public static string ActionLinkWithList( this HtmlHelper helper, string text, string action, string controller, object routeData, object htmlAttributes )
 {
     var urlHelper = new UrlHelper( helper.ViewContext.RequestContext );


     string href = urlHelper.Action( action, controller );

     if (routeData != null)
     {
         RouteValueDictionary rv = new RouteValueDictionary( routeData );
         List<string> urlParameters = new List<string>();
         foreach (var key in rv.Keys)
         {
             object value = rv[key];
             if (value is IEnumerable && !(value is string))
             {
                 int i = 0;
                 foreach (object val in (IEnumerable)value)
                 {
                     urlParameters.Add( string.Format( "{0}[{2}]={1}", key, val, i ));
                     ++i;
                 }
             }
             else if (value != null)
             {
                 urlParameters.Add( string.Format( "{0}={1}", key, value ) );
             }
         }
         string paramString = string.Join( "&", urlParameters.ToArray() ); // ToArray not needed in 4.0
         if (!string.IsNullOrEmpty( paramString ))
         {
            href += "?" + paramString;
         }
     }

     TagBuilder builder = new TagBuilder( "a" );
     builder.Attributes.Add("href",href);
     builder.MergeAttributes( new RouteValueDictionary( htmlAttributes ) );
     builder.SetInnerText( text );
     return builder.ToString( TagRenderMode.Normal );
}

Yeah, this will definitely work. On a more general note, what level of compatibility should be expect between Url generators like UrlHelper.Action and the action parameter binding? I mean, is this a bug or what?
No. Not a bug. I would call it a design decision. The case where you have multiple values for a parameter is pretty rare, especially on a get. I think the expectation that the object is a map of simple value types serves most people well.
Thanks for this, it got my thinking in the right direction. It won't exactly work because a string is an IEnumerable that will enumerate chars. It also makes all the route params querystring params, whereas the original ActionLink takes into account the Routes that are created in global.asax.cs to format route parameters REST style that support it.
I took your idea, and I used LINQ to give me all the routedata items that as IEnumerable<string>..and generated a partial querystring with that. I then took the remaining route data, and add another entry to it like "replacemequerystringname", "replacmequerystringvalue" I then used the regular ActionLink method with the remaining route data. Then I replace that above route entry which has been transformed into a querystring parameter with my aforementioned partial querystring
Hmmm. Maybe check if it implements IList instead of IEnumerable? Or specifically exclude string.

Nerdroid · Accepted Answer · 2017-05-02 01:00:31Z

4

you can suffix your routevalues with an array index like so:

RouteValueDictionary rv = new RouteValueDictionary();
rv.Add("test[0]", val1);
rv.Add("test[1]", val2);

this will result in the querystring containing test=val1&test=val2

that might help ?

edited May 2, 2017 at 1:00

Nerdroid

14.1k5 gold badges62 silver badges72 bronze badges

answered Jan 7, 2010 at 15:17

user245633

2 Comments

JarrettV Over a year ago

thank-you, this is brilliant... oh wait, nevermind, it didn't work as expected

Mathias F Over a year ago

@JarrettV the Url seems to be ok, its just encoded (and should be)

Nerdroid · Accepted Answer · 2017-05-02 01:01:35Z

Combining both methods works nicely.

public static RouteValueDictionary FixListRouteDataValues(RouteValueDictionary routes)
{
    var newRv = new RouteValueDictionary();
    foreach (var key in routes.Keys)
    {
        object value = routes[key];
        if (value is IEnumerable && !(value is string))
        {
            int index = 0;
            foreach (string val in (IEnumerable)value)
            {
                newRv.Add(string.Format("{0}[{1}]", key, index), val);
                index++;
            }
        }
        else
        {
            newRv.Add(key, value);
        }
    }

    return newRv;
}

Then use this method in any extension method that requires routeValues with IEnumerable(s) in it.

Sadly, this workaround seams to be needed in MVC3 too.

Krisztián Balla · Accepted Answer · 2014-05-23 08:57:14Z

1

There is a librarly called Unbinder, which you can use to insert complex objects into routes/urls.

It works like this:

using Unbound;

Unbinder u = new Unbinder();
string url = Url.RouteUrl("routeName", new RouteValueDictionary(u.Unbind(YourComplexObject)));

answered May 23, 2014 at 8:57

Krisztián Balla

20.5k13 gold badges78 silver badges91 bronze badges

Comments

2 revs, 2 users 73% · Accepted Answer · 2017-05-02 01:02:20Z

This will just act as an extension to the UrlHelper and just provide a nice url ready to put anywhere rather than an an entire a tag, also it will preserve most of the other route values for any other specific urls being used... giving you the most friendly specific url you have (minus the IEnumerable values) and then just append the query string values at the end.

public static string ActionWithList(this UrlHelper helper, string action, object routeData)
{

    RouteValueDictionary rv = new RouteValueDictionary(routeData);

    var newRv = new RouteValueDictionary();
    var arrayRv = new RouteValueDictionary();
    foreach (var kvp in rv)
    {
        var nrv = newRv;
        var val = kvp.Value;
        if (val is IEnumerable && !(val is string))
        {
            nrv = arrayRv;
        }

        nrv.Add(kvp.Key, val);

    }


    string href = helper.Action(action, newRv);

    foreach (var kvp in arrayRv)
    {
        IEnumerable lst = kvp.Value as IEnumerable;
        var key = kvp.Key;
        foreach (var val in lst)
        {
            href = href.AddQueryString(key, val);
        }

    }
    return href;
}

public static string AddQueryString(this string url, string name, object value)
{
    url = url ?? "";

    char join = '?';
    if (url.Contains('?'))
        join = '&';

    return string.Concat(url, join, name, "=", HttpUtility.UrlEncode(value.ToString()));
}

Chaddeus · Accepted Answer · 2009-11-18 00:56:27Z

-2

I'm not at my workstation, but how about something like:

<%= Html.ActionLink("click here", "DoMoreStuff", "MoreStuffController", new { stuff = (List<T>)ViewData["stuff"] }, null) %>

or the typed:

<%= Html.ActionLink("click here", "DoMoreStuff", "MoreStuffController", new { stuff = (List<T>)ViewData.Model.Stuff }, null) %>

answered Nov 18, 2009 at 0:56

Chaddeus

13.4k30 gold badges108 silver badges167 bronze badges

1 Comment

Omar Over a year ago

It'll still call ToString() on the object.

Collectives™ on Stack Overflow

ASP.Net MVC RouteData and arrays

6 Answers 6

8 Comments

2 Comments

Comments

Comments

Comments

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

8 Comments

2 Comments

Comments

Comments

Comments

1 Comment

Linked

Related