Skip to content

Conversation

@damien-git
Copy link
Contributor

@damien-git damien-git commented Nov 26, 2025

See VUFIND-1808 - Do not strip fancy quotes

This change fixes issues with fancy quotes when they are part of the title and are included in a phrase (exact) query. Also, normalizing quotes does not seem useful in any way with the current Solr implementation.

TODO

  • Add changelog note about changed behavior when merging

@demiankatz
Copy link
Member

Thanks, @damien-git -- now the question is whether we should treat this as a bug fix for 11.0.1, or if it's a big enough change that we want to wait until 11.1. I'm personally inclined to lean toward 11.0.1 -- but maybe others will feel differently. I'll leave this open for a few days to collect comments; maybe we can discuss at next week's Community Call.

@damien-git
Copy link
Contributor Author

@demiankatz Right, I was also wondering. It's a bug fix, but I can't entirely discard the possibility of it breaking something for someone...

@demiankatz
Copy link
Member

I've put it on the agenda for next week's call so I don't forget to bring it up there!

@meganschanz
Copy link
Contributor

Discussion in the community call: no immediate concerns raised, but we will wait a few more days to allow others to look at this more closely if desired. If no comments or issues are raised in a week we will plan to merge this for the 11.0.1 release as a bug fix.

@demiankatz demiankatz added this to the 11.0.1 milestone Dec 2, 2025
@demiankatz
Copy link
Member

Thanks for capturing the update, @meganschanz. I have added this to the 11.0.1 milestone so it is tracked, and I'll approve and merge it in a week if no objections are raised before then.

Copy link
Member

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

There has been no further discussion since the Community Call, which I will take as a lack of objections/concerns. Thanks, @damien-git, I'm merging this now!

@demiankatz demiankatz merged commit 94e6c4a into vufind-org:dev Dec 9, 2025
6 checks passed
@demiankatz demiankatz deleted the no_fancy_quote_normalization branch December 9, 2025 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants