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

Filter hidden files #5018

Merged
merged 5 commits into from
Nov 10, 2022
Merged

Filter hidden files #5018

merged 5 commits into from
Nov 10, 2022

Conversation

aduffeck
Copy link
Contributor

@aduffeck aduffeck commented Nov 9, 2022

With this PR hidden files (dotfiles) will no longer be part of the search results.

@ownclouders
Copy link
Contributor

ownclouders commented Nov 9, 2022

💥 Acceptance test Core-API-Tests-ocis-storage-1 failed. Further test are cancelled...

@mmattel
Copy link
Contributor

mmattel commented Nov 9, 2022

Just a question, should this be set by default or maybe defined via a env var? Because you do not restrict searching but filtering the results?

@aduffeck
Copy link
Contributor Author

aduffeck commented Nov 9, 2022

@mmattel the e2e test suite also wasn't happy about filtering hidden files by default. We'll now index the hidden state so it's available for future changes but we don't filter by default. So the search behaves exactly the same as before.

@sonarcloud
Copy link

sonarcloud bot commented Nov 10, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@butonic butonic merged commit a0762e2 into owncloud:master Nov 10, 2022
ownclouders pushed a commit that referenced this pull request Nov 10, 2022
Author: Andre Duffeck <aduffeck@users.noreply.github.com>
Date:   Thu Nov 10 17:05:43 2022 +0100

    Filter hidden files (#5018)

    * Filter hidden files from search results

    * Add changelog

    * Do not filter hidden files by default

    * Set the hidden fields when converting fields to an entity

    * Fix test
@aduffeck aduffeck deleted the filter-hidden-files branch November 11, 2022 07:30
@micbar micbar mentioned this pull request Nov 25, 2022
73 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants