Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Issue when filtering by title #1408

Closed
1 task
agbeltran opened this issue Sep 2, 2022 · 1 comment · Fixed by #1416
Closed
1 task

Issue when filtering by title #1408

agbeltran opened this issue Sep 2, 2022 · 1 comment · Fixed by #1416
Labels
bug Something isn't working

Comments

@agbeltran
Copy link
Member

Description:
When navigating experiments and then filtering by title, I seemed to have found a bug.

To reproduce:

  • in DG for ISIS, browse by Experiments
  • choose the GEM instrument
  • in the title filter, enter 'tests'

URL: https://data.isis.stfc.ac.uk/browseStudyHierarchy/instrument/28/study?filters=%7B%22studyInvestigations.investigation.title%22%3A%7B%22value%22%3A%22tests%22%2C%22type%22%3A%22include%22%7D%7D&sort=%7B%22studyInvestigations.investigation.startDate%22%3A%22desc%22%7D

the first 3 results seem OK, but not sure why the fourth result is listed - see screenshot

Screenshot 2022-09-02 at 16 30 09

Now, changing the filter from include to exclude, note that the first result contains 'tests' (and it wasn't given in the previous list)

Screenshot 2022-09-02 at 16 31 37

URL: https://data.isis.stfc.ac.uk/browseStudyHierarchy/instrument/28/study?filters=%7B%22studyInvestigations.investigation.title%22%3A%7B%22value%22%3A%22tests%22%2C%22type%22%3A%22exclude%22%7D%7D&sort=%7B%22studyInvestigations.investigation.startDate%22%3A%22desc%22%7D

Could there be hidding characters that are affecting these filtering results?

Acceptance criteria:

  • Investigate and fix filtering issue
@agbeltran agbeltran added the bug Something isn't working label Sep 2, 2022
@louise-davies
Copy link
Member

@agbeltran we think this may be related to #1390 - which is a problem with the current implementation of the study view

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants