[picocontainer-scm] [5751] java/2.x/trunk/pico/container/src/java/org/picocontainer/lifecycle/ReflectionLifecycleStrategy.java: PICO-379 (null method enhancement for lifecyclestrategy) applied for Pico2 (Pico3 already done)

Revision 5751
Author paul
Date 2011-02-02 07:12:06 -0600 (Wed, 02 Feb 2011)

Log Message

PICO-379 (null method enhancement for lifecyclestrategy) applied for Pico2 (Pico3 already done)

Modified Paths

- java/2.x/trunk/pico/container/src/java/org/picocontainer/lifecycle/ReflectionLifecycleStrategy.java

Diff

Modified: java/2.x/trunk/pico/container/src/java/org/picocontainer/lifecycle/ReflectionLifecycleStrategy.java (5750 => 5751)

--- java/2.x/trunk/pico/container/src/java/org/picocontainer/lifecycle/ReflectionLifecycleStrategy.java 2011-01-08 16:17:06 UTC (rev 5750)
+++ java/2.x/trunk/pico/container/src/java/org/picocontainer/lifecycle/ReflectionLifecycleStrategy.java 2011-02-02 13:12:06 UTC (rev 5751)
@@ -7,13 +7,13 @@
*****************************************************************************/
package org.picocontainer.lifecycle;

+import org.picocontainer.ComponentMonitor;
+
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import java.util.HashMap;
import java.util.Map;

-import org.picocontainer.ComponentMonitor;
-
/**
* Reflection lifecycle strategy. Starts, stops, disposes of component if appropriate methods are
* present. The component may implement only one of the three methods.
@@ -158,7 +158,12 @@
methods = new Method[methodNames.length];
for (int i = 0; i < methods.length; i++) {
try {
- methods[i] = type.getMethod(methodNames[i]);
+ final String methodName = methodNames[i];
+ if (methodName == null) {
+ // skipping, we're not interested in this lifecycle method.
+ continue;
+ }
+ methods[i] = type.getMethod(methodName);
} catch (NoSuchMethodException e) {
continue;
}

----------

To unsubscribe from this list please visit:

http://xircles.codehaus.org/manage_email