You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by pi...@apache.org on 2004/11/01 16:02:52 UTC

svn commit: rev 56241 - cocoon/whiteboard/kernel/src/org/apache/cocoon/kernel/description

Author: pier
Date: Mon Nov  1 07:02:52 2004
New Revision: 56241

Modified:
   cocoon/whiteboard/kernel/src/org/apache/cocoon/kernel/description/Block.java
Log:
Blocks must provide classes, and small XML element name bug

Modified: cocoon/whiteboard/kernel/src/org/apache/cocoon/kernel/description/Block.java
==============================================================================
--- cocoon/whiteboard/kernel/src/org/apache/cocoon/kernel/description/Block.java	(original)
+++ cocoon/whiteboard/kernel/src/org/apache/cocoon/kernel/description/Block.java	Mon Nov  1 07:02:52 2004
@@ -57,7 +57,7 @@
         }
         
         /* Implementations */
-        Iterator iterator = configuration.children(NAMESPACE, "extends");
+        Iterator iterator = configuration.children(NAMESPACE, "implements");
         Set list = new HashSet();
         while (iterator.hasNext()) {
             Configuration current = (Configuration) iterator.next();
@@ -75,7 +75,10 @@
         this.clazz = provides.getStringAttribute("component", null);
 
         /* No class? Abstract block, only libraries */
-        if (this.clazz == null) return;
+        if (this.clazz == null) {
+            throw new DeployerException("Block descriptor does not provide component"
+                    + " instance at " + provides.location());
+        }
 
         /* Process initializer, destroyer, and singleton */
         this.initializer = provides.getStringAttribute("initialize", null);