Added PropertyInfo to RelationshipAttribute

Bart Koelman · Bart Koelman · commit 397e43ae8e19 · 2020-04-29T12:41:32.000+02:00
Inlined RelationshipAttribute.InternalRelationshipName
Explicit interface implementation and property rename
Inlined IsHasOne/IsHasMany on RelationshipAttribute
diff --git a/src/JsonApiDotNetCore/Builders/ResourceGraphBuilder.cs b/src/JsonApiDotNetCore/Builders/ResourceGraphBuilder.cs
@@ -128,8 +128,8 @@ protected virtual List<RelationshipAttribute> GetRelationships(Type entityType)
                 var attribute = (RelationshipAttribute)prop.GetCustomAttribute(typeof(RelationshipAttribute));
                 if (attribute == null) continue;
 
+                attribute.PropertyInfo = prop;
                 attribute.PublicRelationshipName ??= FormatPropertyName(prop);
-                attribute.InternalRelationshipName = prop.Name;
                 attribute.RightType = GetRelationshipType(attribute, prop);
                 attribute.LeftType = entityType;
                 attributes.Add(attribute);
@@ -179,7 +179,7 @@ protected virtual List<RelationshipAttribute> GetRelationships(Type entityType)
         }
 
         protected virtual Type GetRelationshipType(RelationshipAttribute relation, PropertyInfo prop) =>
-            relation.IsHasMany ? prop.PropertyType.GetGenericArguments()[0] : prop.PropertyType;
+            relation is HasOneAttribute ? prop.PropertyType : prop.PropertyType.GetGenericArguments()[0];
 
         private List<EagerLoadAttribute> GetEagerLoads(Type entityType, int recursionDepth = 0)
         {
diff --git a/src/JsonApiDotNetCore/Data/DefaultResourceRepository.cs b/src/JsonApiDotNetCore/Data/DefaultResourceRepository.cs
@@ -149,7 +149,7 @@ private void LoadInverseRelationships(object trackedRelationshipValue, Relations
 
         private bool IsHasOneRelationship(string internalRelationshipName, Type type)
         {
-            var relationshipAttr = _resourceGraph.GetRelationships(type).FirstOrDefault(r => r.InternalRelationshipName == internalRelationshipName);
+            var relationshipAttr = _resourceGraph.GetRelationships(type).FirstOrDefault(r => r.PropertyInfo.Name == internalRelationshipName);
             if (relationshipAttr != null)
             {
                 if (relationshipAttr is HasOneAttribute)
@@ -425,7 +425,7 @@ protected void LoadCurrentRelationships(TResource oldEntity, RelationshipAttribu
             }
             else if (relationshipAttribute is HasManyAttribute hasManyAttribute)
             {
-                _context.Entry(oldEntity).Collection(hasManyAttribute.InternalRelationshipName).Load();
+                _context.Entry(oldEntity).Collection(hasManyAttribute.PropertyInfo.Name).Load();
             }
         }
 
diff --git a/src/JsonApiDotNetCore/Extensions/QueryableExtensions.cs b/src/JsonApiDotNetCore/Extensions/QueryableExtensions.cs
@@ -199,7 +199,7 @@ private static IQueryable<TSource> CallGenericWhereContainsMethod<TSource>(IQuer
             MemberExpression member;
             if (filter.IsAttributeOfRelationship)
             {
-                var relation = Expression.PropertyOrField(entity, filter.Relationship.InternalRelationshipName);
+                var relation = Expression.PropertyOrField(entity, filter.Relationship.PropertyInfo.Name);
                 member = Expression.Property(relation, filter.Attribute.PropertyInfo.Name);
             }
             else
@@ -263,16 +263,16 @@ private static IQueryable<TSource> CallGenericWhereMethod<TSource>(IQueryable<TS
             // Is relationship attribute
             if (filter.IsAttributeOfRelationship)
             {
-                var relationProperty = concreteType.GetProperty(filter.Relationship.InternalRelationshipName);
+                var relationProperty = concreteType.GetProperty(filter.Relationship.PropertyInfo.Name);
                 if (relationProperty == null)
-                    throw new ArgumentException($"'{filter.Relationship.InternalRelationshipName}' is not a valid relationship of '{concreteType}'");
+                    throw new ArgumentException($"'{filter.Relationship.PropertyInfo.Name}' is not a valid relationship of '{concreteType}'");
 
                 var relatedType = filter.Relationship.RightType;
                 property = relatedType.GetProperty(filter.Attribute.PropertyInfo.Name);
                 if (property == null)
-                    throw new ArgumentException($"'{filter.Attribute.PropertyInfo.Name}' is not a valid attribute of '{filter.Relationship.InternalRelationshipName}'");
+                    throw new ArgumentException($"'{filter.Attribute.PropertyInfo.Name}' is not a valid attribute of '{filter.Relationship.PropertyInfo.Name}'");
 
-                var leftRelationship = Expression.PropertyOrField(parameter, filter.Relationship.InternalRelationshipName);
+                var leftRelationship = Expression.PropertyOrField(parameter, filter.Relationship.PropertyInfo.Name);
                 // {model.Relationship}
                 left = Expression.PropertyOrField(leftRelationship, property.Name);
             }
diff --git a/src/JsonApiDotNetCore/Hooks/Execution/RelationshipsDictionary.cs b/src/JsonApiDotNetCore/Hooks/Execution/RelationshipsDictionary.cs
@@ -94,7 +94,7 @@ public Dictionary<RelationshipAttribute, HashSet<TResource>> GetByRelationship(T
         public HashSet<TResource> GetAffected(Expression<Func<TResource, object>> navigationAction)
         {
             var property = TypeHelper.ParseNavigationExpression(navigationAction);
-            return this.Where(p => p.Key.InternalRelationshipName == property.Name).Select(p => p.Value).SingleOrDefault();
+            return this.Where(p => p.Key.PropertyInfo.Name == property.Name).Select(p => p.Value).SingleOrDefault();
         }
     }
 }
diff --git a/src/JsonApiDotNetCore/Internal/InverseRelationships.cs b/src/JsonApiDotNetCore/Internal/InverseRelationships.cs
@@ -52,7 +52,7 @@ public void Resolve()
                     foreach (var attr in ce.Relationships)
                     {
                         if (attr is HasManyThroughAttribute) continue;
-                        INavigation inverseNavigation = meta.FindNavigation(attr.InternalRelationshipName)?.FindInverse();
+                        INavigation inverseNavigation = meta.FindNavigation(attr.PropertyInfo.Name)?.FindInverse();
                         attr.InverseNavigation = inverseNavigation?.Name;
                     }
                 }
diff --git a/src/JsonApiDotNetCore/Internal/Query/BaseQueryContext.cs b/src/JsonApiDotNetCore/Internal/Query/BaseQueryContext.cs
@@ -23,7 +23,7 @@ protected BaseQueryContext(TQuery query)
         public string GetPropertyPath()
         {
             if (IsAttributeOfRelationship)
-                return $"{Relationship.InternalRelationshipName}.{Attribute.PropertyInfo.Name}";
+                return $"{Relationship.PropertyInfo.Name}.{Attribute.PropertyInfo.Name}";
 
             return Attribute.PropertyInfo.Name;
         }
diff --git a/src/JsonApiDotNetCore/Internal/ResourceGraph.cs b/src/JsonApiDotNetCore/Internal/ResourceGraph.cs
@@ -66,7 +66,7 @@ public RelationshipAttribute GetInverse(RelationshipAttribute relationship)
             if (relationship.InverseNavigation == null) return null;
             return GetResourceContext(relationship.RightType)
                             .Relationships
-                            .SingleOrDefault(r => r.InternalRelationshipName == relationship.InverseNavigation);
+                            .SingleOrDefault(r => r.PropertyInfo.Name == relationship.InverseNavigation);
         }
 
         private IEnumerable<IResourceField> Getter<T>(Expression<Func<T, dynamic>> selector = null, FieldFilterType type = FieldFilterType.None) where T : IIdentifiable
@@ -88,7 +88,7 @@ private IEnumerable<IResourceField> Getter<T>(Expression<Func<T, dynamic>> selec
             {   // model => model.Field1
                 try
                 {
-                    targeted.Add(available.Single(f => f.ExposedInternalMemberName == memberExpression.Member.Name));
+                    targeted.Add(available.Single(f => f.PropertyName == memberExpression.Member.Name));
                     return targeted;
                 }
                 catch (InvalidOperationException)
@@ -109,7 +109,7 @@ private IEnumerable<IResourceField> Getter<T>(Expression<Func<T, dynamic>> selec
                     foreach (var member in newExpression.Members)
                     {
                         memberName = member.Name;
-                        targeted.Add(available.Single(f => f.ExposedInternalMemberName == memberName));
+                        targeted.Add(available.Single(f => f.PropertyName == memberName));
                     }
                     return targeted;
                 }
diff --git a/src/JsonApiDotNetCore/Models/Annotation/AttrAttribute.cs b/src/JsonApiDotNetCore/Models/Annotation/AttrAttribute.cs
@@ -35,7 +35,7 @@ public AttrAttribute(string publicName = null, bool isImmutable = false, bool is
             IsSortable = isSortable;
         }
 
-        public string ExposedInternalMemberName => PropertyInfo.Name;
+        string IResourceField.PropertyName => PropertyInfo.Name;
 
         /// <summary>
         /// How this attribute is exposed through the API
@@ -62,9 +62,9 @@ public AttrAttribute(string publicName = null, bool isImmutable = false, bool is
         public bool IsSortable { get; }
 
         /// <summary>
-        /// The member property info
+        /// The resource property that this attribute is declared on.
         /// </summary>
-        public PropertyInfo PropertyInfo { get; set; }
+        public PropertyInfo PropertyInfo { get; internal set; }
 
         /// <summary>
         /// Get the value of the attribute for the given object.
diff --git a/src/JsonApiDotNetCore/Models/Annotation/HasManyAttribute.cs b/src/JsonApiDotNetCore/Models/Annotation/HasManyAttribute.cs
@@ -38,11 +38,10 @@ public HasManyAttribute(string publicName = null, Link relationshipLinks = Link.
         public override object GetValue(object entity)
         {
            return entity?.GetType()
-               .GetProperty(InternalRelationshipName)?
+               .GetProperty(PropertyInfo.Name)?
                 .GetValue(entity);
         }
 
-
         /// <summary>
         /// Sets the value of the property identified by this attribute
         /// </summary>
@@ -52,7 +51,7 @@ public override void SetValue(object entity, object newValue)
         {
             var propertyInfo = entity
                 .GetType()
-                .GetProperty(InternalRelationshipName);
+                .GetProperty(PropertyInfo.Name);
 
             propertyInfo.SetValue(entity, newValue);
         }
diff --git a/src/JsonApiDotNetCore/Models/Annotation/HasManyThroughAttribute.cs b/src/JsonApiDotNetCore/Models/Annotation/HasManyThroughAttribute.cs
@@ -94,7 +94,6 @@ public override object GetValue(object entity)
             return rightEntities.Cast(RightType);
         }
 
-
         /// <summary>
         /// Sets the value of the property identified by this attribute
         /// </summary>
@@ -104,7 +103,7 @@ public override void SetValue(object entity, object newValue)
         {
             var propertyInfo = entity
                 .GetType()
-                .GetProperty(InternalRelationshipName);
+                .GetProperty(PropertyInfo.Name);
             propertyInfo.SetValue(entity, newValue);
 
             if (newValue == null)
diff --git a/src/JsonApiDotNetCore/Models/Annotation/HasOneAttribute.cs b/src/JsonApiDotNetCore/Models/Annotation/HasOneAttribute.cs
@@ -41,7 +41,7 @@ public HasOneAttribute(string publicName = null, Link links = Link.NotConfigured
         public override object GetValue(object entity)
         {
             return entity?.GetType()
-                .GetProperty(InternalRelationshipName)?
+                .GetProperty(PropertyInfo.Name)?
                  .GetValue(entity);
         }
 
@@ -51,7 +51,7 @@ public override object GetValue(object entity)
         /// The independent resource identifier.
         /// </summary>
         public string IdentifiablePropertyName => string.IsNullOrWhiteSpace(_explicitIdentifiablePropertyName)
-            ? JsonApiOptions.RelatedIdMapper.GetRelatedIdPropertyName(InternalRelationshipName)
+            ? JsonApiOptions.RelatedIdMapper.GetRelatedIdPropertyName(PropertyInfo.Name)
             : _explicitIdentifiablePropertyName;
 
         /// <summary>
@@ -61,7 +61,7 @@ public override object GetValue(object entity)
         /// <param name="newValue">The new property value</param>
         public override void SetValue(object entity, object newValue)
         {
-            string propertyName = InternalRelationshipName;
+            string propertyName = PropertyInfo.Name;
             // if we're deleting the relationship (setting it to null),
             // we set the foreignKey to null. We could also set the actual property to null,
             // but then we would first need to load the current relationship, which requires an extra query.
diff --git a/src/JsonApiDotNetCore/Models/Annotation/IResourceField.cs b/src/JsonApiDotNetCore/Models/Annotation/IResourceField.cs
@@ -1,7 +1,7 @@
-﻿namespace JsonApiDotNetCore.Models
+namespace JsonApiDotNetCore.Models
 {
     public interface IResourceField
     {
-        string ExposedInternalMemberName { get; }
+        string PropertyName { get; }
     }
-}
+}
diff --git a/src/JsonApiDotNetCore/Models/Annotation/RelationshipAttribute.cs b/src/JsonApiDotNetCore/Models/Annotation/RelationshipAttribute.cs
@@ -1,5 +1,5 @@
 using System;
-using JsonApiDotNetCore.Extensions;
+using System.Reflection;
 using JsonApiDotNetCore.Internal;
 using JsonApiDotNetCore.Models.Links;
 
@@ -17,11 +17,16 @@ protected RelationshipAttribute(string publicName, Link relationshipLinks, bool
             CanInclude = canInclude;
         }
 
-        public string ExposedInternalMemberName => InternalRelationshipName;
+        string IResourceField.PropertyName => PropertyInfo.Name;
+
         public string PublicRelationshipName { get; internal set; }
-        public string InternalRelationshipName { get; internal set; }
         public string InverseNavigation { get; internal set; }
 
+        /// <summary>
+        /// The resource property that this attribute is declared on.
+        /// </summary>
+        public PropertyInfo PropertyInfo { get; internal set; }
+
         /// <summary>
         /// The related entity type. This does not necessarily match the navigation property type.
         /// In the case of a HasMany relationship, this value will be the generic argument type.
@@ -39,9 +44,6 @@ protected RelationshipAttribute(string publicName, Link relationshipLinks, bool
         /// </summary>
         public Type LeftType { get; internal set; }
 
-        public bool IsHasMany => GetType() == typeof(HasManyAttribute) || GetType().Inherits(typeof(HasManyAttribute));
-        public bool IsHasOne => GetType() == typeof(HasOneAttribute);
-
         /// <summary>
         /// Configures which links to show in the <see cref="RelationshipLinks"/>
         /// object for this relationship.
@@ -85,8 +87,8 @@ public override int GetHashCode()
         /// The internal navigation property path to the related entity.
         /// </summary>
         /// <remarks>
-        /// In all cases except the HasManyThrough relationships, this will just be the <see cref="InternalRelationshipName" />.
+        /// In all cases except the HasManyThrough relationships, this will just be the property name.
         /// </remarks>
-        public virtual string RelationshipPath => InternalRelationshipName;
+        public virtual string RelationshipPath => PropertyInfo.Name;
     }
 }
diff --git a/test/UnitTests/Builders/ContextGraphBuilder_Tests.cs b/test/UnitTests/Builders/ContextGraphBuilder_Tests.cs
@@ -93,8 +93,8 @@ public void Relationships_Without_Names_Specified_Will_Use_Configured_Formatter(
 
             // Assert
             var resource = resourceGraph.GetResourceContext(typeof(TestResource));
-            Assert.Equal("relatedResource", resource.Relationships.Single(r => r.IsHasOne).PublicRelationshipName);
-            Assert.Equal("relatedResources", resource.Relationships.Single(r => r.IsHasMany).PublicRelationshipName);
+            Assert.Equal("relatedResource", resource.Relationships.Single(r => r is HasOneAttribute).PublicRelationshipName);
+            Assert.Equal("relatedResources", resource.Relationships.Single(r => !(r is HasOneAttribute)).PublicRelationshipName);
         }
 
         public sealed class TestResource : Identifiable
diff --git a/test/UnitTests/ResourceHooks/AffectedEntitiesHelperTests.cs b/test/UnitTests/ResourceHooks/AffectedEntitiesHelperTests.cs
@@ -42,19 +42,19 @@ public RelationshipDictionaryTests()
             {
                 LeftType = typeof(Dummy),
                 RightType = typeof(ToOne),
-                InternalRelationshipName = "FirstToOne"
+                PropertyInfo = typeof(Dummy).GetProperty(nameof(Dummy.FirstToOne))
             };
             SecondToOneAttr = new HasOneAttribute("secondToOne")
             {
                 LeftType = typeof(Dummy),
                 RightType = typeof(ToOne),
-                InternalRelationshipName = "SecondToOne"
+                PropertyInfo = typeof(Dummy).GetProperty(nameof(Dummy.SecondToOne))
             };
             ToManyAttr = new HasManyAttribute("toManies")
             {
                 LeftType = typeof(Dummy),
                 RightType = typeof(ToMany),
-                InternalRelationshipName = "ToManies"
+                PropertyInfo = typeof(Dummy).GetProperty(nameof(Dummy.ToManies))
             };
             Relationships.Add(FirstToOneAttr, FirstToOnesEntities);
             Relationships.Add(SecondToOneAttr, SecondToOnesEntities);

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ private void LoadInverseRelationships(object trackedRelationshipValue, Relations`
`149`	`149`
`150`	`150`	`private bool IsHasOneRelationship(string internalRelationshipName, Type type)`
`151`	`151`	`{`
`152`		`- var relationshipAttr = _resourceGraph.GetRelationships(type).FirstOrDefault(r => r.InternalRelationshipName == internalRelationshipName);`
	`152`	`+ var relationshipAttr = _resourceGraph.GetRelationships(type).FirstOrDefault(r => r.PropertyInfo.Name == internalRelationshipName);`
`153`	`153`	`if (relationshipAttr != null)`
`154`	`154`	`{`
`155`	`155`	`if (relationshipAttr is HasOneAttribute)`
`@@ -425,7 +425,7 @@ protected void LoadCurrentRelationships(TResource oldEntity, RelationshipAttribu`
`425`	`425`	`}`
`426`	`426`	`else if (relationshipAttribute is HasManyAttribute hasManyAttribute)`
`427`	`427`	`{`
`428`		`- _context.Entry(oldEntity).Collection(hasManyAttribute.InternalRelationshipName).Load();`
	`428`	`+ _context.Entry(oldEntity).Collection(hasManyAttribute.PropertyInfo.Name).Load();`
`429`	`429`	`}`
`430`	`430`	`}`
`431`	`431`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ public Dictionary<RelationshipAttribute, HashSet<TResource>> GetByRelationship(T`
`94`	`94`	`public HashSet<TResource> GetAffected(Expression<Func<TResource, object>> navigationAction)`
`95`	`95`	`{`
`96`	`96`	`var property = TypeHelper.ParseNavigationExpression(navigationAction);`
`97`		`- return this.Where(p => p.Key.InternalRelationshipName == property.Name).Select(p => p.Value).SingleOrDefault();`
	`97`	`+ return this.Where(p => p.Key.PropertyInfo.Name == property.Name).Select(p => p.Value).SingleOrDefault();`
`98`	`98`	`}`
`99`	`99`	`}`
`100`	`100`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public void Resolve()`
`52`	`52`	`foreach (var attr in ce.Relationships)`
`53`	`53`	`{`
`54`	`54`	`if (attr is HasManyThroughAttribute) continue;`
`55`		`- INavigation inverseNavigation = meta.FindNavigation(attr.InternalRelationshipName)?.FindInverse();`
	`55`	`+ INavigation inverseNavigation = meta.FindNavigation(attr.PropertyInfo.Name)?.FindInverse();`
`56`	`56`	`attr.InverseNavigation = inverseNavigation?.Name;`
`57`	`57`	`}`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ protected BaseQueryContext(TQuery query)`
`23`	`23`	`public string GetPropertyPath()`
`24`	`24`	`{`
`25`	`25`	`if (IsAttributeOfRelationship)`
`26`		`- return $"{Relationship.InternalRelationshipName}.{Attribute.PropertyInfo.Name}";`
	`26`	`+ return $"{Relationship.PropertyInfo.Name}.{Attribute.PropertyInfo.Name}";`
`27`	`27`
`28`	`28`	`return Attribute.PropertyInfo.Name;`
`29`	`29`	`}`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public RelationshipAttribute GetInverse(RelationshipAttribute relationship)`
`66`	`66`	`if (relationship.InverseNavigation == null) return null;`
`67`	`67`	`return GetResourceContext(relationship.RightType)`
`68`	`68`	`.Relationships`
`69`		`- .SingleOrDefault(r => r.InternalRelationshipName == relationship.InverseNavigation);`
	`69`	`+ .SingleOrDefault(r => r.PropertyInfo.Name == relationship.InverseNavigation);`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`private IEnumerable<IResourceField> Getter<T>(Expression<Func<T, dynamic>> selector = null, FieldFilterType type = FieldFilterType.None) where T : IIdentifiable`
`@@ -88,7 +88,7 @@ private IEnumerable<IResourceField> Getter<T>(Expression<Func<T, dynamic>> selec`
`88`	`88`	`{ // model => model.Field1`
`89`	`89`	`try`
`90`	`90`	`{`
`91`		`- targeted.Add(available.Single(f => f.ExposedInternalMemberName == memberExpression.Member.Name));`
	`91`	`+ targeted.Add(available.Single(f => f.PropertyName == memberExpression.Member.Name));`
`92`	`92`	`return targeted;`
`93`	`93`	`}`
`94`	`94`	`catch (InvalidOperationException)`
`@@ -109,7 +109,7 @@ private IEnumerable<IResourceField> Getter<T>(Expression<Func<T, dynamic>> selec`
`109`	`109`	`foreach (var member in newExpression.Members)`
`110`	`110`	`{`
`111`	`111`	`memberName = member.Name;`
`112`		`- targeted.Add(available.Single(f => f.ExposedInternalMemberName == memberName));`
	`112`	`+ targeted.Add(available.Single(f => f.PropertyName == memberName));`
`113`	`113`	`}`
`114`	`114`	`return targeted;`
`115`	`115`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,11 +38,10 @@ public HasManyAttribute(string publicName = null, Link relationshipLinks = Link.`
`38`	`38`	`public override object GetValue(object entity)`
`39`	`39`	`{`
`40`	`40`	`return entity?.GetType()`
`41`		`- .GetProperty(InternalRelationshipName)?`
	`41`	`+ .GetProperty(PropertyInfo.Name)?`
`42`	`42`	`.GetValue(entity);`
`43`	`43`	`}`
`44`	`44`
`45`		`-`
`46`	`45`	`/// <summary>`
`47`	`46`	`/// Sets the value of the property identified by this attribute`
`48`	`47`	`/// </summary>`
`@@ -52,7 +51,7 @@ public override void SetValue(object entity, object newValue)`
`52`	`51`	`{`
`53`	`52`	`var propertyInfo = entity`
`54`	`53`	`.GetType()`
`55`		`- .GetProperty(InternalRelationshipName);`
	`54`	`+ .GetProperty(PropertyInfo.Name);`
`56`	`55`
`57`	`56`	`propertyInfo.SetValue(entity, newValue);`
`58`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,6 @@ public override object GetValue(object entity)`
`94`	`94`	`return rightEntities.Cast(RightType);`
`95`	`95`	`}`
`96`	`96`
`97`		`-`
`98`	`97`	`/// <summary>`
`99`	`98`	`/// Sets the value of the property identified by this attribute`
`100`	`99`	`/// </summary>`
`@@ -104,7 +103,7 @@ public override void SetValue(object entity, object newValue)`
`104`	`103`	`{`
`105`	`104`	`var propertyInfo = entity`
`106`	`105`	`.GetType()`
`107`		`- .GetProperty(InternalRelationshipName);`
	`106`	`+ .GetProperty(PropertyInfo.Name);`
`108`	`107`	`propertyInfo.SetValue(entity, newValue);`
`109`	`108`
`110`	`109`	`if (newValue == null)`