Skip to content

Fix search issue for words inside Guillemets (« ») without spaces#5475

Merged
ssddanbrown merged 1 commit intoBookStackApp:developmentfrom
inv-hareesh:development
Feb 14, 2025
Merged

Fix search issue for words inside Guillemets (« ») without spaces#5475
ssddanbrown merged 1 commit intoBookStackApp:developmentfrom
inv-hareesh:development

Conversation

@inv-hareesh
Copy link
Copy Markdown
Contributor

This commit fixes an issue where words enclosed in Guillemets (« ») were not indexed correctly for search when there was no space after the opening quote. The search engine was not recognizing such words as separate terms, leading to failed search results.

The fix involves updating the list of delimiter characters in SearchIndex::$delimiters to include Guillemets (« and »), ensuring that words within them are properly tokenized and indexed.

ssddanbrown added a commit that referenced this pull request Feb 14, 2025
@ssddanbrown ssddanbrown merged commit c291d27 into BookStackApp:development Feb 14, 2025
@ssddanbrown ssddanbrown added this to the Next Feature Release milestone Feb 14, 2025
@ssddanbrown
Copy link
Copy Markdown
Member

Thanks @inv-hareesh, now merged for next feature release.
I followed this up with some tests to cover via f1b8e85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants