index.get/search fails with string properties with embedded spaces.

[neo4j-contrib]

index.get/search fails with string properties with embedded spaces.

[jekirl]

I had a similar problem with ampersands, I was able to fix this by reverting from that nasty regular expression back to the Lucene query builder that properly escapes, for example, using Q:

from lucenequerybuilder import Q
str(reduce(lambda x, y: x & y, [Q(k, v) for k, v in {"props":"Te st"}.iteritems()]))
'props:("Te st")'

Using the regex:

import re
ESCAPE_CHARS_RE = re.compile(r'(?<!\\)(?P<char>[&|+\-!(){}[\]^"~*?:])')
lucene_esc = lambda v: ESCAPE_CHARS_RE.sub(r'\\\g<char>', unicode(v))
','.join([k + ':' + lucene_esc(v) for k, v in {"props" : "Te st"}.items()])
u'props:Te st'

The first query using Q will work, while the second will yield the NullPointerException. I made the changes in my own fork, but I am hesitant to send the pull request due to the re-addition of lucenequerybuilder as a dependency. It would be nice if there were a fix for the regular expression, but in the mean time using Q is a reasonable workaround.

[art-solopov]

I confirm this on Python 2.7.
I think this issue should be marked as a bug.

[robinedwards]

Thanks guys, perhaps I need to re-introduce lucenequerybuilder I initially pulled it out as it's broken in python 3.

I will look try and look into this asap sorry been a little bit short of time

[robinedwards]

Update master now uses lucenquery-builder again. @bluepeppers++ patched lucenequery-builder to support python3 I am just waiting on a release of lucenequery-builder to pypi so I can bump the version number before releasing neomodel.

I tried pinning the github version in the setup.py to no avail so in the mean time I suggest installing lucenequery-builder from git.

[lclarkmichalek]

Yeah, I was going to ask for a new release before updating this issue, but never got around to it. I'll ask on the pull request for a version bump.

[lclarkmichalek]

The lucene query builder version has been bumped to 0.2, so it should be possible to fix the dependency version now.

[robinedwards]

0.3.6 has just been uploaded to pypi with the fix included, sorry for the delay