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

[QA] fast editing in the the searchbox fails to update the search results #263

Open
jnweiger opened this issue Feb 25, 2022 · 3 comments
Open

Comments

@jnweiger
Copy link
Contributor

jnweiger commented Feb 25, 2022

Seen with search_elastic 2.1.0 RC1 in core 10.9.1 (unrelated to elastic, can be reproduced with normal search too)

  • use a remote owncloud instance, so that there is some noticable network latency
  • admin enables app search_elastic and configures a server
  • search for the word welcome, we get two results, from example.odt and the ownCloud manual pdf. OK.
  • highlight the word welcome (so that it can be replaced quickly) and type portu fast, so that the search result heading does not refresh while typing.
  • The heading "2 search results in other folders" flashes once afterwards, but the old results remain. BAD
  • Hitting enter afterwards, even multiple times, does not help.

image

Expected behaviour

  • old search results are erased, when search terms are erased.
  • after seeing a refresh of the heading, the user should be able to trust the contents below the heading.

Workaround:

  • use left cursor key to move the cursor back, by one position, the matching file name Portugal.jpg appears.
    image
@jnweiger jnweiger changed the title [QA] fast editing in the the searchbox fails to update the search [QA] fast editing in the the searchbox fails to update the search results Feb 25, 2022
@jnweiger
Copy link
Contributor Author

This also happens with normal search. (tested with setting a group on elastic search, and a user not in that group)

@hodyroff
Copy link
Contributor

Can't reproduce with Windows/Edge. Works as expected.

@jnweiger
Copy link
Contributor Author

Reproduced with oc 10.10.0 and search_elastic 2.2.0-rc1

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

No branches or pull requests

2 participants