Changeset 3010 for trunk/sources/HeuristicLab.Persistence/3.3/Default/CompositeSerializers/Storable

Timestamp:

03/12/10 10:43:08 (15 years ago)

Author:

epitzer

Message:

invoke storable hooks in more predictable orders, call hooks defined on base classes first. (#548)

File:

• trunk/sources/HeuristicLab.Persistence/3.3/Default/CompositeSerializers/Storable/StorableHookAttribute.cs (modified) (2 diffs)

trunk/sources/HeuristicLab.Persistence/3.3/Default/CompositeSerializers/Storable/StorableHookAttribute.cs

@@ -46 +46 @@
     }
 
-    private static readonly BindingFlags instanceMembers =
-      BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public;
+    private static readonly BindingFlags declaredInstanceMembers =
+      BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public | BindingFlags.DeclaredOnly;
 
     private static readonly object[] emptyArgs = new object[] { };
@@ -79 +79 @@
     }
 
-    private static IEnumerable<MethodInfo> CollectHooks(HookType hookType, Type type) {      
-      foreach (MemberInfo memberInfo in type.GetMembers(instanceMembers)) {
+    private static IEnumerable<MethodInfo> CollectHooks(HookType hookType, Type type) {  
+      if (type.BaseType != null)
+        foreach (var mi in CollectHooks(hookType, type.BaseType))
+          yield return mi;
+      foreach (MemberInfo memberInfo in type.GetMembers(declaredInstanceMembers)) {
         foreach (StorableHookAttribute hook in memberInfo.GetCustomAttributes(typeof(StorableHookAttribute), false)) {
           if (hook != null && hook.HookType == hookType) {