Find private state properties if in base classes (Fixes #25) (#28)

Author: mrpmorris

Docs/releases.md

@@ -4,6 +4,7 @@
  * Fixed bug that caused exception when using `.ConfigureAwait` in an Effect ([#20](https://github.com/mrpmorris/Fluxor/issues/20))
  * Ensured add/remove on events are thread safe ([#23](https://github.com/mrpmorris/Fluxor/issues/23))
  * Made it easier to find the source of DisposableCallback instances that are not disposed ([#24](https://github.com/mrpmorris/Fluxor/issues/24))
+ * State properties were not discovered if they were declared as private in a base class ([#25](https://github.com/mrpmorris/Fluxor/issues/25))
 
 ### New in 3.1.0
   * Used Newtonsoft entirely for JS interop to ReduxDevTools to prevent serialization errors ([#7](https://github.com/mrpmorris/Fluxor/issues/7))

Source/Fluxor.Blazor.Web/Components/StateSubscriber.cs

@@ -25,7 +25,7 @@ public static IDisposable Subscribe(object subject, Action<IState> callback)
 
 			IEnumerable<GetStateDelegate> getStateDelegates = GetStateDelegatesForType(subject.GetType());
 			var subscriptions = new List<(IState State, EventHandler Handler)>();
-			foreach(GetStateDelegate getState in getStateDelegates)
+			foreach (GetStateDelegate getState in getStateDelegates)
 			{
 				var state = (IState)getState(subject);
 				var handler = new EventHandler((s, a) => callback(state));
@@ -42,18 +42,24 @@ public static IDisposable Subscribe(object subject, Action<IState> callback)
 					});
 		}
 
+		private static IEnumerable<PropertyInfo> GetStateProperties(Type t) =>
+			t == typeof(object)
+			? Enumerable.Empty<PropertyInfo>()
+			: GetStateProperties(t.BaseType)
+				.Union(
+					t.GetProperties(BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.DeclaredOnly)
+						.Where(p => p.PropertyType.IsGenericType)
+						.Where(p => p.PropertyType.GetGenericTypeDefinition() == typeof(IState<>))
+				);
+
 		private static IEnumerable<GetStateDelegate> GetStateDelegatesForType(Type type)
 		{
 			return ValueDelegatesForType.GetOrAdd(type, _ =>
 			{
 				var delegates = new List<GetStateDelegate>();
+				IEnumerable<PropertyInfo> stateProperties = GetStateProperties(type);
 
-				const BindingFlags bindingFlags = BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic;
-				IEnumerable<PropertyInfo> stateProperties = type.GetProperties(bindingFlags)
-					.Where(p => p.PropertyType.IsGenericType)
-					.Where(p => p.PropertyType.GetGenericTypeDefinition() == typeof(IState<>));
-
-				foreach(PropertyInfo currentProperty in stateProperties)
+				foreach (PropertyInfo currentProperty in stateProperties)
 				{
 					Type stateType = currentProperty.PropertyType.GetGenericArguments()[0];
 					Type iStateType = typeof(IState<>).MakeGenericType(stateType);