[jboss-remoting-commits] JBoss Remoting SVN: r4882 - remoting2/branches/2.2/src/main/org/jboss/remoting.

[jboss-remoting-commits at lists.jboss.org]

Author: ron.sigal at jboss.com
Date: 2009-03-20 02:38:41 -0400 (Fri, 20 Mar 2009)
New Revision: 4882

Modified:
   remoting2/branches/2.2/src/main/org/jboss/remoting/Client.java
Log:
JBREM-1082: Corrected unary addConnectionListener().

Modified: remoting2/branches/2.2/src/main/org/jboss/remoting/Client.java
===================================================================
--- remoting2/branches/2.2/src/main/org/jboss/remoting/Client.java	2009-03-20 06:36:01 UTC (rev 4881)
+++ remoting2/branches/2.2/src/main/org/jboss/remoting/Client.java	2009-03-20 06:38:41 UTC (rev 4882)
@@ -346,7 +346,8 @@
    public void addConnectionListener(ConnectionListener listener)
    {
       HashMap metadata = new HashMap();
-      if (configuration.get(ConnectionValidator.VALIDATOR_PING_PERIOD) == null)
+      if (configuration.get(ConnectionValidator.VALIDATOR_PING_PERIOD) == null && 
+          locator.getParameters().get(ConnectionValidator.VALIDATOR_PING_PERIOD) == null)
       {
          String pingPeriod = Long.toString(ConnectionValidator.DEFAULT_PING_PERIOD);
          metadata.put(ConnectionValidator.VALIDATOR_PING_PERIOD, pingPeriod);