Author: veithen
Date: Sun Dec 13 12:11:04 2009
New Revision: 890055

URL: http://svn.apache.org/viewvc?rev=890055&view=rev
Log:
The ListenerManager#addListener method now may have the side effect of starting 
the listener manager. This may cause unexpected behavior in 
UtilsTransportServer. Changed UtilsTransportServer so that the listener manager 
is always started in UtilsTransportServer#start.

Modified:
    
webservices/commons/trunk/modules/transport/modules/testkit/src/main/java/org/apache/axis2/transport/UtilsTransportServer.java

Modified: 
webservices/commons/trunk/modules/transport/modules/testkit/src/main/java/org/apache/axis2/transport/UtilsTransportServer.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/transport/modules/testkit/src/main/java/org/apache/axis2/transport/UtilsTransportServer.java?rev=890055&r1=890054&r2=890055&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/transport/modules/testkit/src/main/java/org/apache/axis2/transport/UtilsTransportServer.java
 (original)
+++ 
webservices/commons/trunk/modules/transport/modules/testkit/src/main/java/org/apache/axis2/transport/UtilsTransportServer.java
 Sun Dec 13 12:11:04 2009
@@ -53,17 +53,16 @@
 
         // create listener manager
         listnMgr = new ListenerManager();
-        listnMgr.init(cfgCtx);
         cfgCtx.setTransportManager(listnMgr);
     }
     
     public void addTransport(TransportInDescription trpInDesc,
                              TransportOutDescription trpDescOut) throws 
AxisFault {
+        cfgCtx.getAxisConfiguration().addTransportIn(trpInDesc);
         if (trpDescOut != null) {
             trpDescOut.getSender().init(cfgCtx, trpDescOut);
             cfgCtx.getAxisConfiguration().addTransportOut(trpDescOut);
         }
-        listnMgr.addListener(trpInDesc, false);
     }
 
     public void addTransport(TransportInDescription trpInDesc) throws 
AxisFault {
@@ -71,6 +70,7 @@
     }
 
     public void start() throws Exception {
+        listnMgr.init(cfgCtx);
         listnMgr.start();
     }
     


Reply via email to