You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@synapse.apache.org by su...@apache.org on 2011/04/25 10:50:58 UTC

svn commit: r1096431 - /synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/xml/endpoints/TemplateFactory.java

Author: supun
Date: Mon Apr 25 08:50:58 2011
New Revision: 1096431

URL: http://svn.apache.org/viewvc?rev=1096431&view=rev
Log:
applying patch synapse_SYNAPSE-753_update.patch, thanks Udayanga for the contribution

Modified:
    synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/xml/endpoints/TemplateFactory.java

Modified: synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/xml/endpoints/TemplateFactory.java
URL: http://svn.apache.org/viewvc/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/xml/endpoints/TemplateFactory.java?rev=1096431&r1=1096430&r2=1096431&view=diff
==============================================================================
--- synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/xml/endpoints/TemplateFactory.java (original)
+++ synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/xml/endpoints/TemplateFactory.java Mon Apr 25 08:50:58 2011
@@ -41,8 +41,6 @@ public class TemplateFactory {
         OMAttribute nameAttribute = element.getAttribute(
                 new QName(XMLConfigConstants.NULL_NAMESPACE, "name"));
 
-        template.addParameter("name");
-        template.addParameter("uri");
 
         if (nameAttribute != null) {
             template.setName(nameAttribute.getAttributeValue());
@@ -64,7 +62,14 @@ public class TemplateFactory {
 
             assert paramName != null;
 
-            template.addParameter(paramName.getAttributeValue());
+            template.addParameter(paramName.getAttributeValue().trim());
+        }
+
+        if (!template.getParameters().contains("name")) {
+            template.addParameter("name");
+        }
+        if (!template.getParameters().contains("uri")) {
+            template.addParameter("uri");
         }
 
         OMElement endpointElement = element.getFirstChildWithName(