geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gno...@apache.org
Subject svn commit: r774667 - /geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/ReferenceRecipe.java
Date Thu, 14 May 2009 07:28:07 GMT
Author: gnodet
Date: Thu May 14 07:28:07 2009
New Revision: 774667

URL: http://svn.apache.org/viewvc?rev=774667&view=rev
Log:
Add a TODO

Modified:
    geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/ReferenceRecipe.java

Modified: geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/ReferenceRecipe.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/ReferenceRecipe.java?rev=774667&r1=774666&r2=774667&view=diff
==============================================================================
--- geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/ReferenceRecipe.java
(original)
+++ geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/ReferenceRecipe.java
Thu May 14 07:28:07 2009
@@ -116,6 +116,9 @@
                 Recipe recipe = (Recipe) object;
                 object = recipe.create(expectedType, false);
             } else if (isServiceReferenceInjection(object, expectedType)) {
+                // TODO: I don't think this is sufficient as the reference injection may
be done by
+                // TODO: having a direct dependency on the service and not go through a <ref>
element
+                // TODO: the conversion has to be done by the conversion service imho
                 object = ((ServiceReferenceAccessor) object).getServiceReference();
             }
         }



Mime
View raw message