Fix `Nullable<T>.ToString()`

Open ranma42 opened this issue 1 year ago • 1 comments

This PR makes most Nullable<T> types have a string representation that is consistent with C#.

Contributes to #33942.

Jun 17 '24 20:06 ranma42

This does not tackle the issue for bool? (handled in https://github.com/dotnet/efcore/pull/33940) This relies on the improvements to COALESCE optimization (specifically https://github.com/dotnet/efcore/pull/33938) to avoid regressions.

Jun 17 '24 20:06 ranma42

Rebased to fix merge conflicts and to make commits more readable now that #33938 is merged.

Jul 02 '24 20:07 ranma42

Thanks @ranma42 !

Jul 16 '24 01:07 maumar