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;
         }
     }
 


Reply via email to