Skip to content

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

Merged
alanorth merged 2 commits intoDSpace:mainfrom
saschaszott:patch-17
Feb 22, 2024
Merged

site search should not adopt existing URL query string parameters#2818
alanorth merged 2 commits intoDSpace:mainfrom
saschaszott:patch-17

Conversation

@saschaszott
Copy link
Copy Markdown
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
Copy Markdown
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
@dspace-bot
Copy link
Copy Markdown
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