Skip to content

Commit c43f024

Browse files
Merge pull request #978 from microsoft/ma/fix-anyOf-oneOf-v2-serialization
Fix anyOf and oneOf v2 serialization
2 parents 06b608c + 19d9edc commit c43f024

File tree

3 files changed

+3
-27
lines changed

3 files changed

+3
-27
lines changed

src/Microsoft.OpenApi.Readers/V2/OpenApiOperationDeserializer.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ internal static partial class OpenApiV2Deserializer
6060
"consumes", (o, n) => {
6161
var consumes = n.CreateSimpleList(s => s.GetScalarValue());
6262
if (consumes.Count > 0) {
63-
n.Context.SetTempStorage(TempStorageKeys.OperationConsumes,consumes);
63+
n.Context.SetTempStorage(TempStorageKeys.OperationConsumes,consumes);
6464
}
6565
}
6666
},

src/Microsoft.OpenApi.Readers/V2/OpenApiV2VersionService.cs

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -107,30 +107,6 @@ private static ReferenceType ParseReferenceType(string referenceTypeName)
107107
}
108108
}
109109

110-
private static string GetReferenceTypeV2Name(ReferenceType referenceType)
111-
{
112-
switch (referenceType)
113-
{
114-
case ReferenceType.Schema:
115-
return "definitions";
116-
117-
case ReferenceType.Parameter:
118-
return "parameters";
119-
120-
case ReferenceType.Response:
121-
return "responses";
122-
123-
case ReferenceType.Tag:
124-
return "tags";
125-
126-
case ReferenceType.SecurityScheme:
127-
return "securityDefinitions";
128-
129-
default:
130-
throw new ArgumentException();
131-
}
132-
}
133-
134110
private static ReferenceType GetReferenceTypeV2FromName(string referenceType)
135111
{
136112
switch (referenceType)

src/Microsoft.OpenApi/Models/OpenApiSchema.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -700,12 +700,12 @@ internal void WriteAsSchemaProperties(
700700
if (AllOf == null || AllOf.Count == 0)
701701
{
702702
// anyOf (Not Supported in V2) - Write the first schema only as an allOf.
703-
writer.WriteOptionalCollection(OpenApiConstants.AllOf, AnyOf.Take(1), (w, s) => s.SerializeAsV2(w));
703+
writer.WriteOptionalCollection(OpenApiConstants.AllOf, AnyOf?.Take(1), (w, s) => s.SerializeAsV2(w));
704704

705705
if (AnyOf == null || AnyOf.Count == 0)
706706
{
707707
// oneOf (Not Supported in V2) - Write the first schema only as an allOf.
708-
writer.WriteOptionalCollection(OpenApiConstants.AllOf, OneOf.Take(1), (w, s) => s.SerializeAsV2(w));
708+
writer.WriteOptionalCollection(OpenApiConstants.AllOf, OneOf?.Take(1), (w, s) => s.SerializeAsV2(w));
709709
}
710710
}
711711

0 commit comments

Comments
 (0)