Dapper icon indicating copy to clipboard operation
Dapper copied to clipboard
verified publisher icon DapperLib

This PR addresses issue #1573 by modifying the GetDeserializer logic to map nullable decimal values to 0m instead of null.

Open AlexGreatDev opened this issue 8 months ago • 1 comments

  • Added explicit check for decimal? types in GetDeserializer
  • Returns 0m when encountering a NULL value
  • Maintains existing behavior for other types

Closes #1573

AlexGreatDev avatar May 13 '25 15:05 AlexGreatDev

Returning zero for null seems ... just wrong; unless I'm mistaken (and that does happen), we don't usually null-coalesce like this; what is the reason / scenario here?

mgravell avatar May 13 '25 19:05 mgravell