philippkueng / node-neo4j

Neo4j REST API wrapper for Node.js
MIT License
211 stars 44 forks source link

A better way of getting _id at the end of node.self. #25

Closed oskarhane closed 10 years ago

oskarhane commented 10 years ago

I'm getting {name: 'Testing Testsson', _id:NaN} when calling db.insertNode({name: 'Testing Testsson'}, function(err, res) { console.log(res) }).

I'm running node-neo4j v2.0.0-RC1 and neo4j v1.9.5.

This way, just getting all the digits from the end of the node.self seems like a better idea.

Would you agree?

philippkueng commented 10 years ago

I completely agree. Thanks a lot @oskarhane for the PR.

Btw. you might stumble upon more bugs when using Neo4j 1.9.5 and our newest release, i'll be debugging some of them over the coming weekend.

Cheers

oskarhane commented 10 years ago

No problem @philippkueng , I'm glad to help. I'll keep an eye open for more bugs. :)