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

Search filter "in here" only applies together with search term #9970

Closed
kulmann opened this issue Nov 16, 2023 · 4 comments · Fixed by #10092
Closed

Search filter "in here" only applies together with search term #9970

kulmann opened this issue Nov 16, 2023 · 4 comments · Fixed by #10092
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@kulmann
Copy link
Contributor

kulmann commented Nov 16, 2023

Describe the bug

The search filter "in here" from the search input field only applies together with a search term, not with empty search term but other filters.

Steps to reproduce

  1. Have two "test.md" files, one in your personal space and one in a project space
  2. Navigate into the project space
  3. Select "in here" in the search input field and trigger the search (enter / click on the magnifier icon)
  4. Select a media type filter for "Documents" (which includes markdown)

Expected behavior

Search result only shows documents from the project space

Actual behavior

Search result also shows the "test.md" file from your personal space. Adding "test" as a search term successfully adds the "in here" location filter as well.

@kulmann kulmann added Type:Bug Something isn't working Priority:p3-medium Normal priority labels Nov 16, 2023
@kulmann kulmann moved this from Qualification to Prio 3 or less in Infinite Scale Team Board Nov 16, 2023
@jacob-nv
Copy link
Contributor

Any ideas how to go about this?
Having empty query removes "scope", which is why empty search with "in here" searches everywhere.
Tried forcing the search with empty query, but that throws error from WebDAV

@lookacat
Copy link
Contributor

lookacat commented Nov 27, 2023

@kulmann i think then we either can't fix it or we need a backend ticket
(and not sure if that would be doable in this sprint for backend)

@kulmann
Copy link
Contributor Author

kulmann commented Nov 27, 2023

Correct, I'd say this is a backend bug. The only thing to do in the frontend would be to remove this line:

But then this backend issue happens:
owncloud/ocis#7812

@kulmann
Copy link
Contributor Author

kulmann commented Nov 27, 2023

@jacob-nv could you please try the solution posted in owncloud/ocis#7812 (comment) ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants