Fix [NonSerialized] is ignored in some platform.

yfakariya · yfakariya · commit c210cd6f79af · 2018-04-30T23:21:14.000+09:00
This fix change [NonSerialized] attribute check from CustomAttributeData based
to FieldAttributes based.
This is required because nonserialized attribute is NOT custom attribute,
it is IL attribute, so some implementation does not return CustomAttributeData
for such "non-custom" attributes.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -745,3 +745,9 @@ Release 1.0.0-rc1 2017-12-17
   * Fix MessagePackObject.UnderlyingType reports wrong type for ext types. Part of #269.
     This bug also caused misleading error message for incompatible type conversion.
   * Fix exceptions thrown by MessagePackObject.AsBinary()/AsString() reports internal type name. Part of #269.
+
+Release 1.0.0
+
+  BUG FIXES
+  * [NonSerialized] attribute does not effect in Mono based platform including Unity.
+
diff --git a/src/MsgPack/Serialization/SerializationTarget.cs b/src/MsgPack/Serialization/SerializationTarget.cs
@@ -466,12 +466,23 @@ private static IEnumerable<SerializingMember> GetPublicUnpreventedMembers( Membe
 						.Select( data => data.GetAttributeType().FullName )
 						.Any( attr =>
 							attr == "MsgPack.Serialization.MessagePackIgnoreAttribute"
-							|| attr == "System.NonSerializedAttribute"
 							|| attr == "System.Runtime.Serialization.IgnoreDataMemberAttribute"
 						)
+					&& !IsNonSerializedField( member )
 				).Select( member => new SerializingMember( member, new DataMemberContract( member ) ) );
 		}
 
+		private static bool IsNonSerializedField( MemberInfo member )
+		{
+			var asField = member as FieldInfo;
+			if( asField == null )
+			{
+				return false;
+			}
+
+			return ( asField.Attributes & FieldAttributes.NotSerialized ) != 0;
+		}
+
 		private static ConstructorInfo FindDeserializationConstructor( SerializationContext context, Type targetType, out ConstructorKind constructorKind )
 		{
 			var constructors = targetType.GetConstructors().ToArray();
