2010/9/20 Erick Dennis <erick.den...@gmail.com> > Hello there! > > I stumbled upon some rather strange behavior with the jsonAddToIndex method > in the REST server and was wondering whether this was intentional or not. > At > the moment jsonAddToIndex is the only method that does not accept content > of > the type application/json but takes plain old text instead. Actually, it > even barfs if you give it a json encoded string. >
It's intentional, since there really is no point accepting JSON when the only values it will ever receive is URLs for nodes or relationships to index. But for consistency it's probably a good thing... so why not? > In agreement with Peter Neubauer I'm sending a patch to the list that fixes > this. In addition to this change I've fixed the tests which I broke. I've > also added a new test to check for malformed json and have updated the > documentation of the curl examples. > Excellent, however the patch didn't make it to the list since attachments have a hard time getting here. Maybe you can send it to me directly and I'll apply it? > The one thing I'd like to note that is now different with this patch is > that > when attempting to index a node which doesn't exist the stack trace is no > longer propagated to the output. It still returns a 500 as before but you > don't see the org.neo4j.graphdb.NotFoundException. If someone could point > me in the right direction of how to solve this I can gladly include it in > the patch. > Ok, I'll have a look at the patch and see what might be the problem, apply it and then come back to you (and the list) what caused it, allright? > > Cheers, > > Erick > > _______________________________________________ > Neo4j mailing list > User@lists.neo4j.org > https://lists.neo4j.org/mailman/listinfo/user > > -- Mattias Persson, [matt...@neotechnology.com] Hacker, Neo Technology www.neotechnology.com _______________________________________________ Neo4j mailing list User@lists.neo4j.org https://lists.neo4j.org/mailman/listinfo/user
