You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by gn...@apache.org on 2009/04/27 21:17:33 UTC

svn commit: r769111 - /geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/Parser.java

Author: gnodet
Date: Mon Apr 27 19:17:32 2009
New Revision: 769111

URL: http://svn.apache.org/viewvc?rev=769111&view=rev
Log:
Make sure key / value for properties are specified

Modified:
    geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/Parser.java

Modified: geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/Parser.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/Parser.java?rev=769111&r1=769110&r2=769111&view=diff
==============================================================================
--- geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/Parser.java (original)
+++ geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/context/Parser.java Mon Apr 27 19:17:32 2009
@@ -587,9 +587,16 @@
 
     private MapEntry parseProperty(Element element) {
         // Parse attributes
+        if (!element.hasAttribute(KEY_ATTRIBUTE)) {
+            throw new ComponentDefinitionException(KEY_ATTRIBUTE + " attribute is required");
+        }
+        if (!element.hasAttribute(VALUE_ATTRIBUTE)) {
+            throw new ComponentDefinitionException(VALUE_ATTRIBUTE + " attribute is required");
+        }
         String key = element.getAttribute(KEY_ATTRIBUTE);
-        String value = element.hasAttribute(VALUE_ATTRIBUTE) ? element.getAttribute(VALUE_ATTRIBUTE) : null;
-        return new MapEntryImpl(new ValueMetadataImpl(key), value != null ? new ValueMetadataImpl(value) : NullMetadata.NULL);
+        String value = element.getAttribute(VALUE_ATTRIBUTE);
+        return new MapEntryImpl(new ValueMetadataImpl(key, String.class.getName()),
+                                new ValueMetadataImpl(value, String.class.getName()));
     }
 
     public MapMetadata parseMap(Element element, ComponentMetadata enclosingComponent) {