Skip to content

Light Boolean search implementation#1828

Merged
Alkarex merged 3 commits intoFreshRSS:devfrom
Alkarex:boolean_search
Mar 14, 2018
Merged

Light Boolean search implementation#1828
Alkarex merged 3 commits intoFreshRSS:devfrom
Alkarex:boolean_search

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Mar 10, 2018

Hello intitle:World OR date:P1D example OR author:Else intitle:"This or that"
#879

"Hello intitle:World OR date:P1D example"
FreshRSS#879
@Alkarex Alkarex mentioned this pull request Mar 10, 2018
@Alkarex Alkarex added this to the 1.11.0 milestone Mar 10, 2018
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Mar 10, 2018

All parts are implicitly AND (which should not be written), except if OR is stated.
No use of parentheses. Support for quotes to disable the Boolean search, like "This or that".

Copy link
Copy Markdown
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

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

Looks pretty straightforward.

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Mar 10, 2018

It was not the case before this patch either, but we so far do not support quotes for free text:
"Hello World".
We do already support quotes in more precise searches like author:"Jean Dupont"

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Mar 10, 2018

Thanks, fixed :-) 4c767fb

@Alkarex Alkarex merged commit 84d891f into FreshRSS:dev Mar 14, 2018
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Mar 14, 2018

Let's merge to increase the testing

@Alkarex Alkarex deleted the boolean_search branch March 14, 2018 16:20
Alkarex added a commit that referenced this pull request Apr 26, 2018
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this pull request Mar 25, 2021
* Light Boolean search implementation

"Hello intitle:World OR date:P1D example"
FreshRSS#879

* Doc Boolean search

* Doc typos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants