Cannot pass object with array property to MVC controller method

Question

I have the following controller method signature:

public void DiscoveryMailboxAnnotate(string note, List<string> ids)

And I am trying to post the following javascript object to this method:

  var data = {
                note: 'note',
                ids: ['1','2']
            };
 $.ajax({
                    type: "POST",
                    url: '/Supervise/DiscoveryMailboxAnnotate',
                    data: data,
                    error: function (xhr, status, error) {
                    },
                    success: function (response) {

                    }
                });

The note parameter in my controller method is populated but not the List<string> id parameter. What am I doing wrong?

I have ids every where and am not using id anywhere

Brian Ogden
– Brian Ogden

2014-04-12 05:52:34 +00:00
Commented Apr 12, 2014 at 5:52 — Brian Ogden
– Brian Ogden, Commented Apr 12, 2014 at 5:52

score 1 · Accepted Answer · 2014-04-12 06:13:56Z

1

Think you've got a model binding problem, try changing to a List<int> on your method signature and think you'll find that works.

Edit: But it was a model biding issue. Properties are private by default, so change your view model so they are public:

        public class DiscoveryMailboxAnnotateViewModel
    {
        public string Note { get; set; }
        public List<string> Ids { get; set; }
    }

Then fix your controller:

    [HttpPost]
    public void DiscoveryMailboxAnnotate(DiscoveryMailboxAnnotateViewModel model) 
    {
        if (!ModelState.IsValid)
        { 
        }   
    }

Then JQuery

<script>
var model = {
    Note: 'note',
    Ids: ['11', '20', '30']
};
$.ajax({
    type: "POST",
    url: '/Home/DiscoveryMailboxAnnotate',
    data: JSON.stringify(model),
    dataType: 'json',
    contentType: 'application/json',
    error: function (xhr, status, error) {
    },
    success: function (response) {

    }
});

This article will explain more, binding a List didn't do work exactly how I expected: http://haacked.com/archive/2008/10/23/model-binding-to-a-list.aspx/

edited Apr 12, 2014 at 6:13

answered Apr 12, 2014 at 5:24

anon

Sign up to request clarification or add additional context in comments.

5 Comments

Brian Ogden Over a year ago

But look at the array of ids, they are strings

Brian Ogden Over a year ago

Thanks this helped as it is so late at night and this is driving so crazy that I didn't realize the properties where private so public was a good tip. But the parsing is exactly the same where with the view model as with declaring the parameters as part of the controller signature. Note is populated but the List<string> or List<int> Ids is null, I tried both types.

anon Over a year ago

The above is working on a scrach project, note I did change the property names to match C# convention and the JS too. Pastebined the code here: pastebin.com/Sq0u2ZPC

Brian Ogden Over a year ago

After adding contentType: 'application/json', it worked, what a nightmare, thanks!

anon Over a year ago

Np, we've all been there late at night! :)

Collectives™ on Stack Overflow

Cannot pass object with array property to MVC controller method

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related