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

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

Author: ningjiang
Date: Tue Nov 11 02:43:19 2008
New Revision: 713013

URL: http://svn.apache.org/viewvc?rev=713013&view=rev
Log:
CAMEL-246 change to throw the IllegalArgumentException instead of CamelException

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

Modified: activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java?rev=713013&r1=713012&r2=713013&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java (original)
+++ activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsComponent.java Tue Nov 11 02:43:19 2008
@@ -415,7 +415,8 @@
             endpoint.getConfiguration().setConnectionFactory(ucfa);
         } else {
             if (username != null || password != null) {
-                throw new CamelException("The JmsComponent's username or password is null");
+                // exclude the the saturation of username and password are all empty
+                throw new IllegalArgumentException("The JmsComponent's username or password is null");
             }
         }
         setProperties(endpoint.getConfiguration(), parameters);