Search Does Not Find Words Inside Guillemets quotes (« »)

[KocheshkovAA]

Describe the Bug

The search does not find a word in Guillemets quotes (« ») when the quoted text contains only one word, and there is no space after the first opening quote.

Steps to Reproduce

1. Create a page with text that includes a single word enclosed in Guillemets quotes, without a space after the first opening quote, for example: «Guillemets».
2. Perform a search without using exact search: Guillemets.
3. Observe that the search does not return the expected result when the word is enclosed in Guillemets quotes.

Expected Behaviour

The search should return the expected result. In our example: Guillemets

Screenshots or Additional Context

Case 1: Search works correctly if there is a space after the opening quote
Text: « Guillemets», Search: Guillemets → Finds the word.

Case 2: Search works correctly when there are multiple words inside the quotes
Text: «Guillemets test», Search: Guillemets test → Finds the word.

Browser Details

No response

Exact BookStack Version

v24.12.1

[thegatesdev]

Just wanted to add that in Case 2 the search still fails for the first word, the result only gets shown because it finds the word 'test'.
Looks like it doesn't split words on that character...

[watschi]

This should be related to #2095, my workaround described there should also work for « and »

Facing the same issue with hyphenated words, which are pretty common in german text.

Quick and dirty solution (needs to be applied after any update):

• Edit app/Search/SearchIndex.php, add a hyphen (-) to $delimiters (at [Link](

https://github.com/BookStackApp/BookStack/blob/33b46882f39106697a704dadd25f221f0dc2ff53/app/Search/SearchIndex.php#L19C1-L19C66 ))

• Run php artisan bookstack:regenerate-search

• For the word Test-Word, Test, Word and Test-Word will return the desired content

@ssddanbrown Any reason to exclude - from the delimiters? Feels like this should be included by default, maybe it's an oversight, maybe I'm missing something 🙂

Originally posted by @watschi in #2095

[ssddanbrown]

Thanks @KocheshkovAA for raising and thanks @thegatesdev @watschi for your input.
@inv-hareesh provided a PR via #5475 which has now been merged for next release.