Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Error converting value {null} to type 'System.Double'. Path 'coordinates.latitude' #5

Closed
jdoan3 opened this issue Jul 14, 2017 · 2 comments

Comments

@jdoan3
Copy link

jdoan3 commented Jul 14, 2017

It seems for every YelpFusion api I use I get a similar issue when I am placing a lattitude/longitude value in the autoComplete/GetBusiness/Search functions of the yelp.Api. I really don't know how to get around this. For a test scenario, I used:

request.Term = "Papa Johns Pizza";
request.Latitude = double.Parse("30.0477325");
request.Longitude = double.Parse("-95.18446720000003");

I even used request.Location = "2855 W Lake Houston Pkwy, Kingwood, TX 77339, USA";

From here when I run the GetSearch I run into the error converting value {null}. I tried doing some searching on why this was the case but couldn't find anything.

I also did a test case where I ran autoComplete for this, which gave me a business Id.
request.Term = "Domino's Pizza";
request.Latitude = double.Parse("30.0498865");
request.Longitude = double.Parse("-95.18670780000002");

I even used request.Location = "4319 Kingwood Dr, Kingwood, TX 77339, USA";

From the business Id I try to get the business from the GetBusiness call but I run into the same error.

@Pinski
Copy link
Contributor

Pinski commented Jul 21, 2017

Hello @jdoan3,

I ran into this same issue. I created a pull request to fix this (#6). However, I've tried contacting the owner of this repo and I've created a few pull requests and have heard nothing in return. I started a fork of this library that is compatible with .Net Core Web. I've fixed this problem and added some more features if you are interested.

https://www.nuget.org/packages/Yelp.Api.Web

@madenwala
Copy link
Owner

Hi sorry for the late reply, but the pull request for this fix has been accepted, I tested your search @jdoan3 and it is working now, and I've pushed this update to NuGet as package 1.2.5.0

Thanks again,
Mohammed

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants