You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by mc...@apache.org on 2008/04/11 12:28:30 UTC
svn commit: r647105 - in
/incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/injection:
FieldInjector.java MethodInjector.java
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: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org