Skip to content

fix: quote nullable Guid values in IN filter#23

Merged
lukaferlez merged 2 commits into
lukaferlez:masterfrom
filipklaric:fix/nullable-guid-in-filter
Jul 24, 2026
Merged

fix: quote nullable Guid values in IN filter#23
lukaferlez merged 2 commits into
lukaferlez:masterfrom
filipklaric:fix/nullable-guid-in-filter

Conversation

@filipklaric

Copy link
Copy Markdown
Contributor

Selector.In generic fallback joined values via plain ToString, so IEnumerable<Guid?> (unlike IEnumerable) never got the ' ' quoting and produced invalid SQL: IN (guid1,guid2). Unwrap Nullable before picking the format.

Selector.In<T> generic fallback joined values via plain ToString, so
IEnumerable<Guid?> (unlike IEnumerable<Guid>) never got the ' ' quoting
and produced invalid SQL: IN (guid1,guid2). Unwrap Nullable<T> before
picking the format.
Verifies the Selector.In<T> fix against real SQL Server: filters
DataTypeNullable via Entity.ListAsync with IEnumerable<Guid?>, incl. a
null-containing list (skipped). Filter property named Guids so it
doesn't collide with the model's Guid column and skip base auto-filter
handling without needing a GetFilterConditions override.
@lukaferlez
lukaferlez merged commit dc840d7 into lukaferlez:master Jul 24, 2026
1 check passed
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.

3 participants