You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jb...@apache.org on 2005/01/27 07:19:20 UTC
svn commit: r126589 - /geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java
Author: jboynes
Date: Wed Jan 26 22:19:19 2005
New Revision: 126589
URL: http://svn.apache.org/viewcvs?view=rev&rev=126589
Log:
better diags if plan is wrong
Modified:
geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java
Modified: geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java
Url: http://svn.apache.org/viewcvs/geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java?view=diff&rev=126589&p1=geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java&r1=126588&p2=geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java&r2=126589
==============================================================================
--- geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java (original)
+++ geronimo/trunk/modules/service-builder/src/java/org/apache/geronimo/deployment/service/GBeanBuilder.java Wed Jan 26 22:19:19 2005
@@ -52,6 +52,9 @@
// @todo we should not need all of common just for this
if (type == null) {
GAttributeInfo attribute = gbean.getGBeanInfo().getAttribute(name);
+ if (attribute == null) {
+ throw new DeploymentException("Unknown attribute " + name + " on " + gbean.getName());
+ }
type = attribute.getType();
}