Merge branch 'bugfix/AttributeDescriptor.DefaultValue' into develop

JeremyCaney · JeremyCaney · commit 8d460ae92885 · 2021-04-01T14:35:08.000-07:00
Ensure that the `DefaultValue` defaults to null, not empty, if it is undefined. This makes it easier for calling code to determine if there is a value associated with it that should be utilized.
diff --git a/OnTopic/Metadata/AttributeDescriptor.cs b/OnTopic/Metadata/AttributeDescriptor.cs
@@ -168,7 +168,7 @@ public bool IsRequired {
     /// </remarks>
     [ExcludeFromCodeCoverage]
     public string? DefaultValue {
-      get => Attributes.GetValue("DefaultValue", "");
+      get => Attributes.GetValue("DefaultValue", null);
       set => SetAttributeValue("DefaultValue", value);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ public bool IsRequired {`
`168`	`168`	`/// </remarks>`
`169`	`169`	`[ExcludeFromCodeCoverage]`
`170`	`170`	`public string? DefaultValue {`
`171`		`- get => Attributes.GetValue("DefaultValue", "");`
	`171`	`+ get => Attributes.GetValue("DefaultValue", null);`
`172`	`172`	`set => SetAttributeValue("DefaultValue", value);`
`173`	`173`	`}`
`174`	`174`