search

neoxygen / neo4j-neoclient

Simple PHP HttpClient for the Neo4j ReST API with Multi DB Support
MIT License
121 stars 138 forks source link

Fix issues when using with Symfony #40

Closed remialvado closed 9 years ago

remialvado commented 9 years ago

Hi,

this PR fixes two separate issues : . logger was not defined in the ContainerBuilder so cannot be passed to HA*Manager. Since $this->logger was not used on this 2 classes, I simply removed it. . HAEnterpriseManager was throwing a Fatal Error when the cluster (slaves + master) was done. I simply add a small test to check if it exists or not.

Thanks for the tool which is great BTW ! I only needed 2 days to move from OrientDB to Neo4j using it :)

ikwattro commented 9 years ago

Thanks a lot for your PR, happy you enjoy it !

ikwattro commented 9 years ago

2.1.8 released with your fixes

remialvado commented 9 years ago

Cool. Thanks :)