Posting array of strings to .net-core mvc

Question

Problem:

I am trying to send an array of strings via jquery post but they dont get parsed correctly, all I get is null in the list.

javascript:

var array = [];
array.push("test")
array.push("test2")
array.push("tes3")

$.post("Admin/FilteredKeys", $.param(JSON.stringify({ Ids: array, OnlyActive: true }, true)));

C# Model:

public class MySearch
{
    public bool OnlyActive { get; set; } = true;
    public List<string> Ids { get; set; }
}

action in controller:

public async Task<IActionResult> FilteredKeys(MySearch filter)
{
    var data = await _service.GetFilteredKeyTypes(filter);
    return View();
}

I've googled and found that the traditional property need to be set to true but it remains the same, i tried this snippet as well:

$.ajax({
    type: "POST",
    url: "Admin/FilteredKeys",
    data: postData,
    success: function(data){
        alert(data.Result);
    },
    dataType: "json",
    traditional: true
});

This is a .net core project do additional I need to change additional paramaters somewhere?

EDIT:

forgott to add that my original attempt was like this:

$.Admin.worker.postJson("Admin/FilteredKeys", JSON.stringify({ Ids: array, OnlyActive: true }), function (data) {
    var t = "";
});

postJson: function (url, data, callback) {
    $.LoadingOverlay("show");
    $.ajax({
        url: url,
        type: 'POST',
        data: data,
        contentType: 'application/json; charset=utf-8',
        success: function (data) {
            if (callback)
                callback(data);
            $.LoadingOverlay("hide");
        },
        error: function (event, jqxhr, settings, thrownError) {
            //$.helpers.errorHandler($("#fileDialogErrors"), event.responseText);
            $.LoadingOverlay("hide");
        }
    });
}

Didn't give it a test but I would suggest using data: { filter : postData } in ajax since I faced and solved similar problems. — Hikarunomemory
– Hikarunomemory, Commented Apr 10, 2018 at 13:24
My sneaking suspicion is something to do with where you're parameterizing a stringified object: $.param(JSON.stringify({ Ids: array, OnlyActive: true }. Have you simply tried posting the object itself as the post data? e.g. { Ids: array, OnlyActive: true } — Geoff James
– Geoff James, Commented Apr 10, 2018 at 13:25

Evk · Accepted Answer · 2018-04-10 14:31:08Z

3

Since I believe accepted answer is not correct - here is alternative one (similar, but not the same as other answers here). First you need to decorate model with FromBody attribute:

public async Task<IActionResult> FilteredKeys([FromBody] MySearch filter)
{
    var data = await _service.GetFilteredKeyTypes(filter);
    return View();
}

And ajax call should look like this:

var array = [];
array.push("test")
array.push("test2")
array.push("test3")
 $.ajax({
    type: "POST",
    url: "Admin/FilteredKeys",
    data: JSON.stringify({ Ids: array, OnlyActive: true}),
    contentType: "application/json; charset=utf-8",
    success: function(data){
        // ...
    },
    failure: function(errMsg) {
        // ...
    }
});

answered Apr 10, 2018 at 14:31

Evk

102k8 gold badges156 silver badges204 bronze badges

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

Comments

SᴇM · Accepted Answer · 2018-04-11 08:46:23Z

1

Why not just this:

  $.post("Admin/FilteredKeys",{ Ids: JSON.stringify(array), OnlyActive: true });

edited Apr 11, 2018 at 8:46

SᴇM

7,2433 gold badges28 silver badges43 bronze badges

answered Apr 10, 2018 at 13:24

Manoz

6,62713 gold badges72 silver badges117 bronze badges

7 Comments

Geoff James Over a year ago

I thought the same, Manoj :) Only thing I would say is it might benefit everyone if you explain your answer, a little.

Geoff James Over a year ago

@Keith - I know that. Perhaps I wasn't clear. What I meant was if it was explained why the existing code doesn't work, and how this differs, this would make the answer much more complete and understandable to all users.

SᴇM Over a year ago

It will stringify whole array into one string. So, there is no need to JSON.stringify(array), only Ids: array.

SᴇM Over a year ago

@Manoj I've downvoted, because your solution will return to controller non empty object, but for Ids list it will get one item, with json serialized string of an actual array and I don't think that this is a right solution.

Geoff James Over a year ago

@SeM - same reason I've not upvoted yet. I think it would be unfair to downvote - the answer is on the right track, at least. But, I agree: This answer needs to simply pass the array as the value of Ids, rather than serializing it, which renders it not entirely correct, yet.

|

SᴇM · Accepted Answer · 2018-04-11 08:50:13Z

1

This answer is for the sake of the alternative (if someone will prefer this solution):

$.post("Admin/FilteredKeys", { Ids: array, OnlyActive: true });

answered Apr 11, 2018 at 8:50

SᴇM

7,2433 gold badges28 silver badges43 bronze badges

Comments

J.Loscos · Accepted Answer · 2018-04-10 13:29:10Z

0

Too be sure that the controller tries to parse the model from the body you could add the FromBody attribute:

public async Task<IActionResult> FilteredKeys([FromBody] MySearch filter)
{
...
}

Also you should add a application/json content type to your request :

$.ajax({
    type: "POST",
    url: ...,
    data: ...,
    success: ...
    contentType: "application/json"
});

answered Apr 10, 2018 at 13:29

J.Loscos

2,9571 gold badge15 silver badges23 bronze badges

Comments

Emir Cangır · Accepted Answer · 2018-04-10 13:38:34Z

0

 $.ajax({
        type: "POST",
        url: "Admin/FilteredKeys",
        data: { filter: postData},
        success: function(data){
            alert(data.Result);
        },
        dataType: "json",
        traditional: true
    });

answered Apr 10, 2018 at 13:38

Emir Cangır

231 silver badge4 bronze badges

2 Comments

Georg Plaz Over a year ago

Generally, answers are much more helpful if they include an explanation of what the code is intended to do, and why that solves the problem without introducing others.

Emir Cangır Over a year ago

@LonelyNeuron Thank you for your advise.

Collectives™ on Stack Overflow

Posting array of strings to .net-core mvc

5 Answers 5

Comments

7 Comments

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

7 Comments

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related