Author: clement
Date: Thu Jul  5 08:59:26 2012
New Revision: 1357501

URL: http://svn.apache.org/viewvc?rev=1357501&view=rev
Log:
Fixed FELIX-3576
When we compute the MethodId for constructors skip the instance manager.

Modified:
    
felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/parser/MethodMetadata.java

Modified: 
felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/parser/MethodMetadata.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/parser/MethodMetadata.java?rev=1357501&r1=1357500&r2=1357501&view=diff
==============================================================================
--- 
felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/parser/MethodMetadata.java
 (original)
+++ 
felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/parser/MethodMetadata.java
 Thu Jul  5 08:59:26 2012
@@ -21,6 +21,7 @@ package org.apache.felix.ipojo.parser;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Method;
 
+import org.apache.felix.ipojo.InstanceManager;
 import org.apache.felix.ipojo.metadata.Element;
 
 /**
@@ -151,6 +152,12 @@ public class MethodMetadata {
         StringBuffer identifier = new StringBuffer("$init");
         Class[] args = method.getParameterTypes();
         for (int i = 0; i < args.length; i++) {
+            // If the first argument is the InstanceManager skip it
+            if (i == 0  && InstanceManager.class.equals(args[i])) {
+                // Skip it.
+                continue;
+            }
+
             identifier.append('$'); // Argument separator.
             if (args[i].isArray()) {
                 if (args[i].getComponentType().isPrimitive()) {


Reply via email to