Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

escape slash #589

Closed
lalop opened this issue Apr 17, 2014 · 6 comments
Closed

escape slash #589

lalop opened this issue Apr 17, 2014 · 6 comments

Comments

@lalop
Copy link

lalop commented Apr 17, 2014

Since Elasticsearch uses Lucene 4.0, / needs to be escape like \\/not like \/

@ruflin
Copy link
Owner

ruflin commented Apr 19, 2014

Isn't that already the case here?

https://github.com/ruflin/Elastica/blob/master/lib/Elastica/Util.php#L50

Which version of Elastica are you using?

@lalop
Copy link
Author

lalop commented Apr 20, 2014

I've the current master version.
Special character is prefixed by "\\" which is \ since the first slash currently escape the second one.

@ruflin
Copy link
Owner

ruflin commented Apr 20, 2014

Hm, so the right code would be: \\\\ ?

@lalop
Copy link
Author

lalop commented Apr 21, 2014

It seems, yes.
It was the case for me, but that depend on the elasticsearch version.

@ruflin
Copy link
Owner

ruflin commented Apr 21, 2014

The newest version of Elastica only supports the newer version of elasticsearch, so it shouldn't be a problem. Can you open a pull request with it?

@lalop lalop mentioned this issue Apr 21, 2014
ruflin added a commit that referenced this issue Apr 21, 2014
@ruflin
Copy link
Owner

ruflin commented Apr 27, 2014

Closed as merged into maser

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants