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]