search

jadell / neo4jphp

PHP wrapper of the Neo4j REST interface
Other
532 stars 137 forks source link

Exception on rootNode->getLabels() #110

Closed selimachour closed 10 years ago

selimachour commented 10 years ago

In FILE Command/GetLabels.php LINE 61

if (!$id) { throw new \InvalidArgumentException("Node given with no id"); }

I don't know why there's a test, but changing it to !is_integer($id) enables reading labels on rootNode

jadell commented 10 years ago

Closed by #114

pegamdiwanee commented 10 years ago

This issue is closed, but no correction was made to FILE Command/GetLabels.php LINE 61.

jadell commented 10 years ago

Yeah, you're right. The fix for this is the same as the fix for #114. The test should be is_numeric instead of is_integer because numeric strings are also acceptable.