Skip to content

Better search#1478

Merged
Alkarex merged 5 commits intoFreshRSS:devfrom
Alkarex:better_search
Apr 3, 2017
Merged

Better search#1478
Alkarex merged 5 commits intoFreshRSS:devfrom
Alkarex:better_search

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Apr 1, 2017

@Alkarex Alkarex added this to the 1.7.0 milestone Apr 1, 2017
@Alkarex Alkarex added the Work in progress 🚧 Wait before merging label Apr 1, 2017
FreshRSS#1381
Possibility to exclude authors, titles, tags, urls, words by prefixing
them by ! or - (like Google Search):

* !intitle:unwanted
* -intitle:unwanted
* -author:unwanted
* -#unwanted
* -unwanted

And one can use many of each and combine them with positive searches
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Apr 1, 2017

#1381 @Crupuk
Possibility to exclude authors, titles, tags, URLs, words, by prefixing them by ! or - (like Google Search):

  • !intitle:unwanted
  • -intitle:unwanted
  • -author:unwanted
  • -#unwanted
  • -unwanted

And one can use many of each and combine them with positive searches

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Apr 1, 2017

#511 (comment)
#879
#1189


```sh
cd ./tests/
wget https://phar.phpunit.de/phpunit.phar
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Apr 2, 2017

Documentation done in FreshRSS/documentation@46b5792
@marienfressinaud

Alkarex referenced this pull request Apr 2, 2017
Before, the search was a single value.
Now it is splited in chuncks when separated by spaces.
Except if they are enclosed by single quotes or double quotes.

For some reasons, the unit tests are working for both single and double quotes but the
search box isn't. It is working only with single quotes.
We need to investigate the reason of this behavior.

See #823
@Alkarex Alkarex merged commit 1cace07 into FreshRSS:dev Apr 3, 2017
@Alkarex Alkarex deleted the better_search branch April 3, 2017 16:42
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

Successfully merging this pull request may close these issues.

1 participant