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

site search should not adopt existing URL query string parameters #2818

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

saschaszott
Copy link
Contributor

Description

A site search (issued by the search box in the navigation bar) should not adopt existing URL query parameters. For example, a site search that is issued in the context of MyDSpace will adopt configuration=workspace as a URL query parameter.

The normal user would not expect that a site search considers only a subset of all items in the repository. It is commonly expected that a site search behaves consistent regardless of the context from which the search was started.

@saschaszott saschaszott changed the title site search should not adopt existing query string parameters site search should not adopt existing URL query string parameters Feb 19, 2024
@tdonohue tdonohue added bug component: Discovery related to discovery search or browse system 1 APPROVAL pull request only requires a single approval to merge port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release labels Feb 20, 2024
@alanorth
Copy link
Contributor

alanorth commented Feb 22, 2024

Thanks @saschaszott! I confirmed the bug and the fix on DSpace 7.6.1.

I will squash these two commits into one before merging because the second one fixes a test broken by the first and this can easily be one atomic change.

@alanorth alanorth self-requested a review February 22, 2024 08:06
@alanorth alanorth added this to the 8.0 milestone Feb 22, 2024
@alanorth alanorth merged commit d5aca76 into DSpace:main Feb 22, 2024
13 checks passed
@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-7_x:

@alanorth alanorth removed the port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release label Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge bug component: Discovery related to discovery search or browse system
Projects
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants