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

Allow searching both title and body at the same time #1265

Merged
merged 2 commits into from
Sep 26, 2023

Conversation

mkanilsson
Copy link
Contributor

This PR adds option Both to Search In, allowing users to search through both title and body of articles at the same time.

This closes #1263

Implementation details:

  1. Added private helper function getSearchSQLForColumn in DatabaseConnectionOrm to reduce code duplication.
  2. Changed getAllItemsIdsForFolderSQLSearch to take a list of columns to make the function more flexible and to minimize code duplication.
  3. Added new string value pref_general_search_in_both

Note: Translations are missing

This closes nextcloud#1263

Signed-off-by: Marcus Nilsson <marcus.nilsson@genarp.com>
Signed-off-by: Marcus Nilsson <marcus.nilsson@genarp.com>
@David-Development
Copy link
Member

Note: Translations are missing

Once this is merged into the master - Transifex will automatically pull/detect the strings and our translators can translate them.

Thank you again for this great addition! I really like the option to be able to search in both, title and body!

@David-Development David-Development merged commit 6ab7aeb into nextcloud:master Sep 26, 2023
6 of 9 checks passed
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.

Enhancement: Allow searching title and body at the same time
2 participants