Author: mcombellack
Date: Fri Apr 11 03:27:57 2008
New Revision: 647105

URL: http://svn.apache.org/viewvc?rev=647105&view=rev
Log:
Updated exception handling so that it throws the JavaDoc documented 
ObjectCreationException rather than AssertionError when the method/field that 
you are injecting into is not accessible

Modified:
    
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/FieldInjector.java
    
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/MethodInjector.java

Modified: 
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/FieldInjector.java
URL: 
http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/FieldInjector.java?rev=647105&r1=647104&r2=647105&view=diff
==============================================================================
--- 
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/FieldInjector.java
 (original)
+++ 
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/FieldInjector.java
 Fri Apr 11 03:27:57 2008
@@ -51,7 +51,7 @@
         try {
             field.set(instance, objectFactory.getInstance());
         } catch (IllegalAccessException e) {
-            throw new AssertionError("Field is not accessible [" + field + 
"]");
+            throw new ObjectCreationException("Field is not accessible [" + 
field + "]", e);
         }
     }
 }

Modified: 
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/MethodInjector.java
URL: 
http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/MethodInjector.java?rev=647105&r1=647104&r2=647105&view=diff
==============================================================================
--- 
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/MethodInjector.java
 (original)
+++ 
incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection/MethodInjector.java
 Fri Apr 11 03:27:57 2008
@@ -54,7 +54,7 @@
         try {
             method.invoke(instance, objectFactory.getInstance());
         } catch (IllegalAccessException e) {
-            throw new AssertionError("Method is not accessible [" + method + 
"]");
+            throw new ObjectCreationException("Method is not accessible [" + 
method + "]", e);
         } catch (IllegalArgumentException e) {
             throw new ObjectCreationException("Exception thrown by setter: " + 
method.getName(), e);
         } catch (InvocationTargetException e) {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to