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

DS-4271: Replaced brackets with double quotes in SolrServiceImpl. #2543

Merged

Conversation

jonas-atmire
Copy link
Contributor

@jonas-atmire jonas-atmire commented Oct 10, 2019

DSpace 6 versions of the fix provided in:
https://jira.duraspace.org/browse/DS-4271
Fixes #7611

Quite a small fix, but it has been open for a little while now, so took the liberty to create a quick PR
(More information is present in the JIRA ticket itself)

Copy link
Contributor

@AlexanderS AlexanderS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good.

@tdonohue tdonohue added bug component: Discovery Related to Discovery search or browse system port to main This PR needs to be ported to `main` branch for the next major release quick win Pull request is small in size & should be easy to review and/or merge labels Feb 7, 2020
@tdonohue tdonohue added this to the 6.4 milestone Feb 7, 2020
Copy link
Contributor

@nwoodward nwoodward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Tested and confirmed it works.

@alanorth
Copy link
Contributor

Agree this is a good fix for a problem I didn't even realize we had. Tested with a contains filter with spaces on a subject term in our staging repository.

Before, 293 results, of which many are not what the user would expect:

ds-4271-before-fs8

After, 162 results, of which all are exactly matching the search term:

ds-4271-after-fs8

Looks good to me for DSpace 6.4. Merge!

alanorth pushed a commit to ilri/DSpace that referenced this pull request Jun 23, 2021
@J4bbi
Copy link
Contributor

J4bbi commented Jun 24, 2021

I haven't tested this but it's got +3 @kshepherd

@tdonohue tdonohue linked an issue Sep 1, 2021 that may be closed by this pull request
@tdonohue tdonohue closed this Sep 3, 2021
@tdonohue tdonohue reopened this Sep 3, 2021
@alanorth
Copy link
Contributor

@J4bbi I think this is a good one for 6.4. It's at +3 for some months now.

@alanorth
Copy link
Contributor

alanorth commented May 5, 2022

Ported to main for DSpace 7.x here: #8284

tdonohue pushed a commit that referenced this pull request May 6, 2022
Port of GitHub PR #2543 from DSpace 6.x to DSpace 7.x, originally
by @jonas-atmire.

Resolves DS-4271 (GitHub Issue #7611).
@tdonohue tdonohue removed the port to main This PR needs to be ported to `main` branch for the next major release label Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component: Discovery Related to Discovery search or browse system quick win Pull request is small in size & should be easy to review and/or merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DS-4271] Discovery "contains" filter does not perform a phrase query
6 participants