Merge pull request MessagePack-CSharp#1462 from gllebede/user/gllebede/skip_properties_with_MessagePackFormatterAttribute

AArnott · web-flow · commit 8fe5fc94669f · 2022-07-23T07:39:59.000-06:00
Skip type collection if property has MessagePackFormatterAttribute
diff --git a/src/MessagePack.GeneratorCore/CodeAnalysis/TypeCollector.cs b/src/MessagePack.GeneratorCore/CodeAnalysis/TypeCollector.cs
@@ -720,7 +720,12 @@ private ObjectSerializationInfo GetObjectInfo(INamedTypeSymbol type)
                         stringMembers.Add(member.StringKey, member);
                     }
 
-                    this.CollectCore(item.Type); // recursive collect
+                    var messagePackFormatter = item.GetAttributes().FirstOrDefault(x => x.AttributeClass.ApproximatelyEqual(this.typeReferences.MessagePackFormatterAttribute))?.ConstructorArguments[0];
+
+                    if (messagePackFormatter == null)
+                    {
+                        this.CollectCore(item.Type); // recursive collect
+                    }
                 }
 
                 foreach (IFieldSymbol item in type.GetAllMembers().OfType<IFieldSymbol>())

Original file line number	Diff line number	Diff line change
`@@ -720,7 +720,12 @@ private ObjectSerializationInfo GetObjectInfo(INamedTypeSymbol type)`
`720`	`720`	`stringMembers.Add(member.StringKey, member);`
`721`	`721`	`}`
`722`	`722`
`723`		`- this.CollectCore(item.Type); // recursive collect`
	`723`	`+ var messagePackFormatter = item.GetAttributes().FirstOrDefault(x => x.AttributeClass.ApproximatelyEqual(this.typeReferences.MessagePackFormatterAttribute))?.ConstructorArguments[0];`
	`724`	`+`
	`725`	`+ if (messagePackFormatter == null)`
	`726`	`+ {`
	`727`	`+ this.CollectCore(item.Type); // recursive collect`
	`728`	`+ }`
`724`	`729`	`}`
`725`	`730`
`726`	`731`	`foreach (IFieldSymbol item in type.GetAllMembers().OfType<IFieldSymbol>())`