Converting the url to query string

Question

So I am trying to convert this url which handled the POST reqs:

// this works
http://localhost/api/locations/postlocation/16/555/556

to its equavalent query string which is supposed to be:

http://localhost/api/locations/postlocation?id=16&lat=88&lon=88

but when I'm doing this I'm getting this error. Apparently it doesn't recognise one of the parameters:

"Message": "An error has occurred.",
  "ExceptionMessage": "Value cannot be null.\r\nParameter name: entity",
  "ExceptionType": "System.ArgumentNullException",

This is the method that handles this Post request:

[Route("api/locations/postlocation/{id:int}/{lat}/{lon}")]
public IHttpActionResult UpdateUserLocation(string lat, string lon, int id)
{
    if (!ModelState.IsValid)
    {
        return BadRequest(ModelState);
    }
    var user = db.Users.FirstOrDefault(u => u.Id == id);

    if (user == null)
    {
        return NotFound();
    }

    var userId = user.Id;

    var newLocation = new Location
    {
        Latitude = Convert.ToDouble(lat),
        Longitude = Convert.ToDouble(lon),
        User = user,
        UserId = user.Id,
        Time = DateTime.Now
    };

    var postLocation = PostLocation(newLocation);

    return Ok();
}

Any idea what's the problem?

Stinky Towel · Accepted Answer · 2017-05-05 18:15:10Z

2

The controller action doesn't know to look for the query string params. You have to define them explicitly.

[Route("api/locations/postlocation")]
public IHttpActionResult UpdateUserLocation([FromUri] int id, [FromUri] string lat, [FromUri] string lon)

Note this will break your first (RESTful) call example.

answered May 5, 2017 at 18:15

Stinky Towel

7787 silver badges26 bronze badges

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

1 Comment

Tamás Szabó Over a year ago

If you add both Routes, then it won't break the first example.

Collectives™ on Stack Overflow

Converting the url to query string

1 Answer 1

1 Comment

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Related