Bladeren bron

SimpleInjector now includes all parent interfaces.

Previously it was just interfaces implemented explicitly in the
given class, which wasn't what was intended

Change-Id: I101e504264182b0fe1eed252a648c1ef3d1333d6
Reviewed-on: http://gerrit.dmdirc.com/2377
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>
tags/0.7rc1
Chris Smith 12 jaren geleden
bovenliggende
commit
54800b0ab6
1 gewijzigde bestanden met toevoegingen van 6 en 5 verwijderingen
  1. 6
    5
      src/com/dmdirc/util/SimpleInjector.java

+ 6
- 5
src/com/dmdirc/util/SimpleInjector.java Bestand weergeven

@@ -79,13 +79,14 @@ public class SimpleInjector {
79 79
         Class<?> target = object.getClass();
80 80
         do {
81 81
             addParameter(target, object);
82
+
83
+            // Add all interfaces
84
+            for (Class<?> iface : target.getInterfaces()) {
85
+                addParameter(iface, object);
86
+            }
87
+
82 88
             target = target.getSuperclass();
83 89
         } while (target != null);
84
-
85
-        // Add all interfaces
86
-        for (Class<?> iface : object.getClass().getInterfaces()) {
87
-            addParameter(iface, object);
88
-        }
89 90
     }
90 91
 
91 92
     /**

Laden…
Annuleren
Opslaan