ASP.NET Web API method model null

Question

So, first of all I best explain that I am calling this method via AngularJS. I have many other controllers, but what makes this one unique is that the model only has one property:

public class Subscriber
{
    public string Email { get; set; }
}

And so my controller method (which is very simple) just looks like this:

[HttpPost]
[Route("")]
public async Task<IHttpActionResult> Create(Subscriber model)
{

    // Try to find the subscriber
    var subscriber = await this.service.GetAsync(model.Email);

    // If the subscriber does not exist
    if (subscriber != null) {

        // Create the subscriber
        this.service.Create(model);

        // Save the datbase changes
        await this.unitOfWork.SaveChangesAsync();
    }

    return Ok();
}

I didn't create a RequestModel although I might just do that to stop my method being invoked when the model is null.

So, the issue is just that, the model is always null. I have my AngularJS service which looks like this:

.service('SubscriberService', ['Api', function (api) {
    var apiUrl = 'api/subscribers';

    // Subscribe
    this.subscribe = function (email) {

        // Create our model
        var model = {
            email: email
        };

        console.log(model);

        // Create our subscriber
        return $http.post(apiUrl, model);
    };
}])

in that, the console log shows the model with the email value. In my web config I have this line:

serializerSettings.ContractResolver = new CamelCasePropertyNamesContractResolver();

which handles the camel case, although I have also tried with Pascal case to no avail.

Does anyone know why it isn't working? I have spent too much time staring at it.

Did you inspect if the call to the action is actually made? The rout is correct? — Taher Rahgooy
– Taher Rahgooy, Commented Aug 14, 2015 at 15:42
This code looks fine. It should work. Model binding does not care about casing.Both PascalCasing and camelCasing will work as long as the property names match. — Shyju
– Shyju, Commented Aug 14, 2015 at 15:44
yeah I used fiddler, I can see that the model is being posted. I put a breakpoint on the actual method and it hits it, just that there is no model.... — r3plica
– r3plica, Commented Aug 14, 2015 at 15:47
@vidalsasoon has the correct answer below. You need to serialize your js model to json and you need to add the [FromBody] attribute to your method signature. — Stephen Brickner
– Stephen Brickner, Commented Aug 14, 2015 at 17:26

Miguel A. Arilla · Accepted Answer · 2015-08-14 15:44:48Z

2

In your second block of code you are using the following:

return Ok();

By doing so you are only answering your api call with an empty HTTP 200 response. You should be answering with the model loaded from the database.

I'd suggest trying:

return Ok(subscriber);

Hope this helps.

answered Aug 14, 2015 at 15:44

Miguel A. Arilla

5,3821 gold badge17 silver badges29 bronze badges

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

1 Comment

r3plica Over a year ago

I want to return an empty 200 response, it's a subscriber method, so nothing should be returned apart from the status code.

vidalsasoon · Accepted Answer · 2015-08-14 16:01:41Z

1

try adding this: "[FromBody]"

public async Task<IHttpActionResult> Create([FromBody]Subscriber model)

Also worth trying is:

return $http.post(apiUrl, angular.toJson(model));

answered Aug 14, 2015 at 16:01

vidalsasoon

4,4391 gold badge35 silver badges43 bronze badges

1 Comment

r3plica Over a year ago

Hi, neither of these worked. I don't do this anywhere else in my project so I would have been surprised if it had of :o I am starting the think that I am doing something stupid somewhere but can't see it.

Stephen Brickner · Accepted Answer · 2015-08-14 17:23:52Z

0

Have you tried stringifying your model so it can be serialized as json in your api method?

return $http.post(apiUrl, JSON.stringify(model));

answered Aug 14, 2015 at 17:23

Stephen Brickner

2,6221 gold badge13 silver badges19 bronze badges

Comments

r3plica · Accepted Answer · 2015-08-14 18:20:47Z

0

I am fairly certain that this won't help anyone else, but the issue was with the controller itself. I had copied it from one of my other controllers because the methods in it were nearly the same and one thing that I copied along with it was the Authorize annotation. I am using code first EF and I had just deleted my database and recreated it, but the angularJS still had a token for me so figured I was authenticated and the new controller allowed me to get into it but didn't return a 401 error because nothing was getting passed to it.

Anyway, removing Authrozie from the controller fixed the issue.

answered Aug 14, 2015 at 18:20

r3plica

13.5k24 gold badges152 silver badges323 bronze badges

Collectives™ on Stack Overflow

ASP.NET Web API method model null

4 Answers 4

1 Comment

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related