You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2008/06/09 20:50:20 UTC

svn commit: r665824 - /activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsQueueEndpoint.java

Author: davsclaus
Date: Mon Jun  9 11:50:19 2008
New Revision: 665824

URL: http://svn.apache.org/viewvc?rev=665824&view=rev
Log:
CAMEL-597: Faulty constructor. Applied patch with thanks to Dave Brosius.

Modified:
    activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsQueueEndpoint.java

Modified: activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsQueueEndpoint.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsQueueEndpoint.java?rev=665824&r1=665823&r2=665824&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsQueueEndpoint.java (original)
+++ activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsQueueEndpoint.java Mon Jun  9 11:50:19 2008
@@ -36,7 +36,6 @@
     private int maximumBrowseSize = -1;
     private final QueueBrowseStrategy queueBrowseStrategy;
 
-
     public JmsQueueEndpoint(String uri, JmsComponent component, String destination,
             JmsConfiguration configuration) {
         this(uri, component, destination, configuration, null);
@@ -45,17 +44,19 @@
     public JmsQueueEndpoint(String uri, JmsComponent component, String destination,
             JmsConfiguration configuration, QueueBrowseStrategy queueBrowseStrategy) {
         super(uri, component, destination, false, configuration);
-        this.queueBrowseStrategy = queueBrowseStrategy;
         if (queueBrowseStrategy == null) {
-            queueBrowseStrategy = createQueueBrowseStrategy();
+            this.queueBrowseStrategy = createQueueBrowseStrategy();
+        } else {
+            this.queueBrowseStrategy = queueBrowseStrategy;
         }
     }
 
     public JmsQueueEndpoint(String endpointUri, String destination, QueueBrowseStrategy queueBrowseStrategy) {
         super(endpointUri, destination, false);
-        this.queueBrowseStrategy = queueBrowseStrategy;
         if (queueBrowseStrategy == null) {
-            queueBrowseStrategy = createQueueBrowseStrategy();
+            this.queueBrowseStrategy = createQueueBrowseStrategy();
+        } else {
+            this.queueBrowseStrategy = queueBrowseStrategy;
         }
     }