From 52f5dfafde95f660601170ae96d56b38796e6e8b Mon Sep 17 00:00:00 2001 From: Shay Rojansky Date: Mon, 13 Dec 2021 19:43:41 +0100 Subject: [PATCH] Don't block querying over DateOnly/TimeOnly in JSON Though until System.Text.Json supports DateOnly/TimeOnly, converters must be used. Fixes #2148 --- .../Internal/NpgsqlJsonPocoTranslator.cs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/EFCore.PG/Query/ExpressionTranslators/Internal/NpgsqlJsonPocoTranslator.cs b/src/EFCore.PG/Query/ExpressionTranslators/Internal/NpgsqlJsonPocoTranslator.cs index ac06fe093..4814ff988 100644 --- a/src/EFCore.PG/Query/ExpressionTranslators/Internal/NpgsqlJsonPocoTranslator.cs +++ b/src/EFCore.PG/Query/ExpressionTranslators/Internal/NpgsqlJsonPocoTranslator.cs @@ -123,7 +123,10 @@ private SqlExpression ConvertFromText(SqlExpression expression, Type returnType) return _sqlExpressionFactory.Convert(expression, returnType, _typeMappingSource.FindMapping(returnType, _model)); } - if (unwrappedReturnType == typeof(Guid) || unwrappedReturnType == typeof(DateTimeOffset)) + if (unwrappedReturnType == typeof(Guid) + || unwrappedReturnType == typeof(DateTimeOffset) + || unwrappedReturnType == typeof(DateOnly) + || unwrappedReturnType == typeof(TimeOnly)) { return _sqlExpressionFactory.Convert(expression, returnType, _typeMappingSource.FindMapping(returnType, _model)); }