Merge pull request #28 from LykkeCity/bugfix-required-properties

bug fixed
LykkeCity · Nov 16, 2018 · a56852c · a56852c
2 parents 4eb1366 + 9346152
commit a56852c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/Lykke.Common.ApiLibrary/Swagger/XmsEnum/ResponseValueTypesRequiredSchemaFilter.cs b/src/Lykke.Common.ApiLibrary/Swagger/XmsEnum/ResponseValueTypesRequiredSchemaFilter.cs
@@ -20,7 +20,7 @@ public void Apply(Schema schema, SchemaFilterContext context)
                 return;
             }
 
-            var nonNulableValueTypedPropNames = context.SystemType.GetProperties()
+            var nonNullableValueTypedPropNames = context.SystemType.GetProperties()
                 .Where(p =>
                     // is it value type?
                     p.PropertyType.GetTypeInfo().IsValueType &&
@@ -30,9 +30,9 @@ public void Apply(Schema schema, SchemaFilterContext context)
                     p.CanRead && p.CanWrite)
                 .Select(p => p.Name);
 
-            schema.Required = schema.Properties.Keys
-                .Union(nonNulableValueTypedPropNames, StringComparer.OrdinalIgnoreCase)
-                .ToList();
+            schema.Required = schema.Required == null
+                ? nonNullableValueTypedPropNames.ToList()
+                : schema.Required.Union(nonNullableValueTypedPropNames, StringComparer.OrdinalIgnoreCase).ToList();
 
             if (!schema.Required.Any())
             {