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

All search results filtered by Moodle #47

Open
mattporritt opened this issue May 23, 2019 · 0 comments
Open

All search results filtered by Moodle #47

mattporritt opened this issue May 23, 2019 · 0 comments

Comments

@mattporritt
Copy link
Contributor

We are seeing an edge case where a search term entered in Moodle returns no results to the user. However there are plenty of results being returned by the Elasticsearch backend.

This looks to be an issue with the compile_results method in the engine class. It is still to be 100% confirmed but the root cause seems to be:

  • User searches for a term
  • Results from the Elasticsearch backend are limited to 100 total results.
  • 100 Results are returned from the Elasticsearch backend
  • The user has access to the context the results are in, but not the results themselves (This is the case for messages between users).
  • compile_results filters out the results that the user doesn't have access to. In this case all of the results are filtered.
  • No results are displayed to the end user.

The next steps:

  • Confirm the above is correct
  • Find out why the context filtering is allowing results to be returned that the user doesn't have access to. This might mean that user messages are a special case.
  • 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

1 participant