You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by an...@apache.org on 2008/02/14 12:43:17 UTC

svn commit: r627713 - /incubator/tuscany/java/sca/modules/host-jms-activemq/src/main/java/org/apache/tuscany/sca/host/jms/activemq/JMSResourceFactoryImpl.java

Author: antelder
Date: Thu Feb 14 03:43:08 2008
New Revision: 627713

URL: http://svn.apache.org/viewvc?rev=627713&view=rev
Log:
Only set jndi context properties if they've been defined by the application

Modified:
    incubator/tuscany/java/sca/modules/host-jms-activemq/src/main/java/org/apache/tuscany/sca/host/jms/activemq/JMSResourceFactoryImpl.java

Modified: incubator/tuscany/java/sca/modules/host-jms-activemq/src/main/java/org/apache/tuscany/sca/host/jms/activemq/JMSResourceFactoryImpl.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/host-jms-activemq/src/main/java/org/apache/tuscany/sca/host/jms/activemq/JMSResourceFactoryImpl.java?rev=627713&r1=627712&r2=627713&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/host-jms-activemq/src/main/java/org/apache/tuscany/sca/host/jms/activemq/JMSResourceFactoryImpl.java (original)
+++ incubator/tuscany/java/sca/modules/host-jms-activemq/src/main/java/org/apache/tuscany/sca/host/jms/activemq/JMSResourceFactoryImpl.java Thu Feb 14 03:43:08 2008
@@ -36,20 +36,19 @@
  */
 public class JMSResourceFactoryImpl implements JMSResourceFactory {
 
-    private static final String DEFAULT_ICFN = "org.apache.activemq.jndi.ActiveMQInitialContextFactory";
-    private String initialContextFactoryName = DEFAULT_ICFN;
+    private String initialContextFactoryName;
     private String connectionFactoryName = "ConnectionFactory";
-    private String jndiURL = ActiveMQBroker.CONNECTOR_URL;
+    private String jndiURL;
 
     private Connection connection;
     private Context context;
     private boolean isConnectionStarted;
 
     public JMSResourceFactoryImpl(String connectionFactoryName, String initialContextFactoryName, String jndiURL) {
-        if (connectionFactoryName != null) {
+        if (connectionFactoryName != null && connectionFactoryName.trim().length() > 0) {
             this.connectionFactoryName = connectionFactoryName.trim();
         }
-        if (initialContextFactoryName != null) {
+        if (initialContextFactoryName != null && initialContextFactoryName.trim().length() > 0) {
             this.initialContextFactoryName = initialContextFactoryName.trim();
         }
         if (jndiURL != null) {
@@ -128,7 +127,6 @@
         } catch (Exception e) {
             return true;
         }
-        // return DEFAULT_ICFN.equals(initialContextFactoryName) && ActiveMQBroker.CONNECTOR_URL.equals(jndiURL);
     }
 
     private void createConnection() throws NamingException, JMSException {
@@ -139,9 +137,12 @@
     private synchronized Context getInitialContext() throws NamingException {
         if (context == null) {
             Properties props = new Properties();
-            props.setProperty(Context.INITIAL_CONTEXT_FACTORY, initialContextFactoryName);
-            props.setProperty(Context.PROVIDER_URL, jndiURL);
-
+            if (initialContextFactoryName != null) {
+                props.setProperty(Context.INITIAL_CONTEXT_FACTORY, initialContextFactoryName);
+            }
+            if (jndiURL != null) {
+                props.setProperty(Context.PROVIDER_URL, jndiURL);
+            }
             context = new InitialContext(props);
         }
         return context;



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org