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

[Bug]: Fulltext search result links are wrong #5514

Closed
5 of 8 tasks
arnowelzel opened this issue Mar 10, 2023 · 3 comments
Closed
5 of 8 tasks

[Bug]: Fulltext search result links are wrong #5514

arnowelzel opened this issue Mar 10, 2023 · 3 comments

Comments

@arnowelzel
Copy link

⚠️ Before submitting, please verify the following: ⚠️

Bug description

When doing a fulltext search in the client, the results are linked but the links are wrong. They do not contain the directory of the file, eventhough the search result displays correct directory.

Steps to reproduce

  1. Do a full text search in the client which will find files in sub folders
  2. Try to open the search result
  3. The browser will now only open the main file list and not the file referred in the result since the link does not contain the sub folder of the file but only /: `https://server.example/apps/files/?dir=/&fileid=1234``

Expected behavior

When search result is clicked where the file is inside the folder /foo/bar it should be like this:

https://server.example/apps/files/?dir=/foo/bar&fileid=1234

Which files are affected by this bug

N/A

Operating system

Windows

Which version of the operating system you are running.

Windows 10

Package

Other

Nextcloud Server version

25.0.4

Nextcloud Desktop Client version

3.7.4

Is this bug present after an update or on a fresh install?

Fresh desktop client install

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

Are you using an external user-backend?

  • Default internal user-backend
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Nextcloud Server logs

too big and not relevant for this bug.

Additional info

There was already a pull request (#5279) to introduce linking for fulltext search results. I don't know if it worked better with older versions of Nextcloud.

@ostasevych
Copy link

Duplicate #5511

@claucambra
Copy link
Collaborator

Hi, we get these URLs from the Full Text Search component on the server side, so this bug originates from the server module. I opened an issue here: nextcloud/fulltextsearch#729

Feel free to add your report here too.

Since this is an upstream issue I will close the issue here

@arnowelzel
Copy link
Author

Maybe you should add a tag for "upstream" so it is clear that the problem is not fixed but only the issue got closed because the upstream needs to fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants