JsonNumberHandling.AllowReadingFromString should support read empty string to null/default
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Start with the provided System.Text.Json.JsonSerializer.Deserialize<decimal?> reproduction and the JsonNumberHandling.AllowReadingFromString option. Trace the number-conversion behavior for nullable decimal values, then verify that an empty JSON string becomes null while quoted numeric strings and existing handling continue to work.
Written by the indexing model from the issue text.
Description
Description
currently , S.T.J allow read "123" to number type like Nullable<int>/Nullable<decimal> , but it doesn't allow "" (eg. empty string) to null
since the number is using string format, so it might be null | "" when there is no number , so null or "" both should be deserialize as null
for example : a html input bind a value of string , when user type a number and delete
Reproduction Steps
var op = new System.Text.Json.JsonSerializerOptions(System.Text.Json.JsonSerializerDefaults.Web)
{ NumberHandling = System.Text.Json.Serialization.JsonNumberHandling.AllowReadingFromString | System.Text.Json.Seria
lization.JsonNumberHandling.AllowNamedFloatingPointLiterals
};
System.Text.Json.JsonSerializer.Deserialize<decimal?>("\"\"", op)
Expected behavior
"" => null (decimal?)
Actual behavior
thow exception : The JSON value could not be converted to System.Nullable`1[System.Decimal]
Regression?
No response
Known Workarounds
No response
Configuration
No response
Other information
No response
- Dominant language
- C#
- Stars
- 18.3k
- Forks
- 5.6k
- PR merge metrics
- PR metrics pending
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from dotnet/runtime
-
agentic-workflows untriaged
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
area-System.Reflection blocking-clean-ci-optional Known Build Error os-mac-os-x untriaged
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
area-CodeGen-coreclr untriaged
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
agentic-workflows untriaged
Difficulty 1/5 Under an hour Newbie friendliness 78/100
-
area-VM-meta-mono untriaged
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
-
:watch: Not Triaged 11.0 fundamentals/subsvc
Difficulty 2/5 1-3 hours Newbie friendliness 92/100
dotnet/AspNetCore.Docs#37699 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
SubtitleEdit/subtitleedit#15108 · 1 comment ·
-
area/docs-content Bug pulumi/docs
Difficulty 1/5 1-3 hours Newbie friendliness 94/100
-
Create parent directories only after the containment check in InstallHelper.TryExtractToDirectory Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
PowerShell/PSResourceGet#2056 ·