Fixed #241 and #217

ninject · Sep 11, 2017 · 68a8ca4 · 68a8ca4
1 parent c10ccf6
commit 68a8ca4
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/src/Ninject.Test/Integration/PropertyInjectionTests.cs b/src/Ninject.Test/Integration/PropertyInjectionTests.cs
@@ -149,7 +149,21 @@ public void ParentPropertiesAreInjected()
             warrior.SecretWeaponAccessor.Should().NotBeNull();
             warrior.VerySecretWeaponAccessor.Should().NotBeNull();
         }
-
+
+        [Fact]
+        public void GrandParentPropertiesAreInjected()
+        {
+            this.kernel.Settings.InjectNonPublic = true;
+            this.kernel.Settings.InjectParentPrivateProperties = true;
+            var warrior = this.kernel.Get<GrandFatherStyleNinja>();
+
+            warrior.Should().NotBeNull();
+            warrior.OffHandWeapon.Should().NotBeNull();
+            warrior.SecondaryWeapon.Should().NotBeNull();
+            warrior.SecretWeaponAccessor.Should().NotBeNull();
+            warrior.VerySecretWeaponAccessor.Should().NotBeNull();
+        }
+
         private class OwnStyleNinja : Ninja
         {
             public OwnStyleNinja(IWeapon weapon)
@@ -171,6 +185,14 @@ public FatherStyleNinja(IWeapon weapon)
             {
             }
         }
+
+        private class GrandFatherStyleNinja : FatherStyleNinja
+        {
+            public GrandFatherStyleNinja(IWeapon weapon)
+                : base(weapon)
+            {
+            }
+        }
 #endif //!SILVERLIGHT
     }
 

diff --git a/src/Ninject/Selection/Selector.cs b/src/Ninject/Selection/Selector.cs
@@ -108,7 +108,7 @@ public virtual IEnumerable<PropertyInfo> SelectPropertiesForInjection(Type type)
             {
                 for (Type parentType = type.GetTypeInfo().BaseType; parentType != null; parentType = parentType.GetTypeInfo().BaseType)
                 {
-                    properties.AddRange(this.GetPrivateProperties(type.GetTypeInfo().BaseType));
+                    properties.AddRange(this.GetPrivateProperties(parentType));
                 }
             }
-Original file line number
+Diff line change
@@ Expand Up @@
                 {
                     for (Type parentType = type.GetTypeInfo().BaseType; parentType != null; parentType = parentType.GetTypeInfo().BaseType)
                     {
-                        properties.AddRange(this.GetPrivateProperties(type.GetTypeInfo().BaseType));
+                        properties.AddRange(this.GetPrivateProperties(parentType));
                     }
                 }
@@ Expand Down @@