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

Node#create with nil in one property doesn't create the node #23

Closed
elado opened this issue Jan 10, 2012 · 2 comments
Closed

Node#create with nil in one property doesn't create the node #23

elado opened this issue Jan 10, 2012 · 2 comments

Comments

@elado
Copy link
Contributor

elado commented Jan 10, 2012

Neography::Node.create(x: nil, y: 1).neo_id # => nil

Neography::Node.create(x: 0, y: 1).neo_id # => "1"

Should reject nils from the hash if that's a problem.

data.reject { |k, v| v.nil? }

Anyway if there's an error it should be raised.

@elado
Copy link
Contributor Author

elado commented Jan 10, 2012

Update:

Looks like Neo4j server returns 500:

RestClient.post('http://localhost:7474/db/data/node/', {x:nil})

and HTTParty's post returns nil. I checked their docs and it should return a response object with .code. Any idea why?

@maxdemarzi
Copy link
Owner

In evaluate_response I log errors but return nil.

willkessler pushed a commit to willkessler/neography that referenced this issue Apr 21, 2014
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

2 participants