Skip to content

fix: handle special characters in column fts queries#15

Merged
jguo144 merged 1 commit intomasterfrom
jguo144/2025-12-08/fix-column-search
Dec 8, 2025
Merged

fix: handle special characters in column fts queries#15
jguo144 merged 1 commit intomasterfrom
jguo144/2025-12-08/fix-column-search

Conversation

@jguo144
Copy link
Collaborator

@jguo144 jguo144 commented Dec 8, 2025

Description

Column searches containing apostrophes (e.g. "L'est") or forward slashes (e.g. "Board/Village") were returning no results, even when matching data existed. The search query builder was handling these characters incorrectly by replacing them with the FTS wildcard (_), which caused issues when the special character appeared in the middle of a search term.

Updated the search query logic to:

  • Split search terms on spaces (natural word boundaries)
  • For compound words with special characters, use OR logic to match any part
  • For separate words, use AND logic to require all words to match

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@jguo144 jguo144 force-pushed the jguo144/2025-12-08/fix-column-search branch 3 times, most recently from 02c052a to 400e13e Compare December 8, 2025 20:07
@jguo144 jguo144 force-pushed the jguo144/2025-12-08/fix-column-search branch from 400e13e to d9febe4 Compare December 8, 2025 20:20
@jguo144 jguo144 enabled auto-merge (squash) December 8, 2025 20:54
@jguo144 jguo144 requested a review from peterVorman December 8, 2025 20:55
@jguo144 jguo144 merged commit d080911 into master Dec 8, 2025
9 checks passed
@jguo144 jguo144 deleted the jguo144/2025-12-08/fix-column-search branch December 8, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants