fix #1137

jonsequitur · jonsequitur · commit 3264927b51a5 · 2021-05-25T16:22:22.000-07:00
diff --git a/src/System.CommandLine.Tests/Binding/ModelBinderTests.cs b/src/System.CommandLine.Tests/Binding/ModelBinderTests.cs
@@ -5,6 +5,7 @@
 using System.CommandLine.Binding;
 using System.CommandLine.Builder;
 using System.CommandLine.Invocation;
+using System.CommandLine.IO;
 using System.CommandLine.Parsing;
 using System.IO;
 using FluentAssertions;
@@ -769,5 +770,29 @@ public async Task Empty_input_is_bound_correctly_to_list_type_properties()
 
             boundInstance.Should().NotBeNull();
         }
+
+        [Fact]
+        public async Task Decimals_are_bound_correctly_when_no_token_is_matched()
+        {
+            decimal? receivedValue = null;
+
+            var rootCommand = new RootCommand
+            {
+                new Option<decimal>("--opt-decimal")
+            };
+            rootCommand.Handler = CommandHandler.Create((ComplexType options) =>
+            {
+                receivedValue = options.OptDecimal;
+            });
+
+            await rootCommand.InvokeAsync("");
+
+            receivedValue.Should().Be(0);
+        }
+
+        public class ComplexType
+        {
+            public decimal OptDecimal { get; set; }
+        }
     }
 }
diff --git a/src/System.CommandLine/Binding/ModelBinder.cs b/src/System.CommandLine/Binding/ModelBinder.cs
@@ -105,7 +105,8 @@ private bool ShortCutTheBinding()
             var modelType = ModelDescriptor.ModelType;
             return modelType.IsPrimitive ||
                    modelType.IsNullableValueType() ||
-                   modelType == typeof(string);
+                   modelType == typeof(string) ||
+                   modelType == typeof(decimal) ;
         }
 
         private (bool success, object? newInstance, bool anyNonDefaults) GetSimpleModelValue(

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,8 @@ private bool ShortCutTheBinding()`
`105`	`105`	`var modelType = ModelDescriptor.ModelType;`
`106`	`106`	`return modelType.IsPrimitive \|\|`
`107`	`107`	`modelType.IsNullableValueType() \|\|`
`108`		`- modelType == typeof(string);`
	`108`	`+ modelType == typeof(string) \|\|`
	`109`	`+ modelType == typeof(decimal) ;`
`109`	`110`	`}`
`110`	`111`
`111`	`112`	`private (bool success, object? newInstance, bool anyNonDefaults) GetSimpleModelValue(`