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

multi word searches only searched as exact string, not multiple strings #644

Closed
Archaeopteryx opened this issue Feb 23, 2016 · 2 comments
Closed
Assignees

Comments

@Archaeopteryx
Copy link

@Archaeopteryx Archaeopteryx commented Feb 23, 2016

Searches for multiple words only work if they are one exact match. E.g.
The search for 'restart disabled' returns nothing but there are several strings which contain these words.

If I remember correct, this worked in the past (and yielded many uninteresting matches if one had to search for common words).

@pascalchevrel
Copy link
Member

@pascalchevrel pascalchevrel commented Feb 23, 2016

git bisect gives me this:
87d3405 is the first bad commit
commit 87d3405
Author: Pascal Chevrel pascal.chevrel@free.fr
Date: Sun Nov 15 19:00:48 2015 +0100

Issues #320: move all our regex logic into a unit tested class
:040000 040000 ecfb0046a95aa33e7791649be2ed55428d390169 31132c9142c3a2ec20238e376b6e18735ef17f2e M  app
:040000 040000 8a401d63d718c7e00d0ef0b880ea133c065cf5a6 786a4f0f94f2527c83181c4a9cfca6a507f5996e M  tests
@pascalchevrel pascalchevrel self-assigned this Feb 23, 2016
pascalchevrel added a commit to pascalchevrel/transvision that referenced this issue Feb 23, 2016
pascalchevrel added a commit that referenced this issue Feb 23, 2016
…ing_search

Issue #644: fix multiword search + fix a call to getRegex()
@pascalchevrel
Copy link
Member

@pascalchevrel pascalchevrel commented Feb 23, 2016

This is fixed on master and the beta sites yields results:
https://transvision-beta.mozfr.org/?recherche=restart+disabled&repo=aurora&sourcelocale=en-US&locale=de&search_type=strings

Note though that the beta site doesn't get its data updated automatically as production, so you will need to wait for the next release to get that fix on production.

Thanks for reporting the regression.

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.