[jboss-svn-commits] JBL Code SVN: r6890 - labs/jbossesb/workspace/eschifman/trunk/product/core/listeners/src/org/jboss/soa/esb/message/listeners

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

Author: estebanschifman
Date: 2006-10-18 10:54:10 -0400 (Wed, 18 Oct 2006)
New Revision: 6890

Modified:
   labs/jbossesb/workspace/eschifman/trunk/product/core/listeners/src/org/jboss/soa/esb/message/listeners/ActionProcessingPipeline.java
Log:
Remove notification of end of chain.
Users should use dummy action class at the end to notify end of action chain

Modified: labs/jbossesb/workspace/eschifman/trunk/product/core/listeners/src/org/jboss/soa/esb/message/listeners/ActionProcessingPipeline.java
===================================================================
--- labs/jbossesb/workspace/eschifman/trunk/product/core/listeners/src/org/jboss/soa/esb/message/listeners/ActionProcessingPipeline.java	2006-10-18 14:51:30 UTC (rev 6889)
+++ labs/jbossesb/workspace/eschifman/trunk/product/core/listeners/src/org/jboss/soa/esb/message/listeners/ActionProcessingPipeline.java	2006-10-18 14:54:10 UTC (rev 6890)
@@ -105,7 +105,9 @@
                     return;
                 }
             }
-            actionChainFinishedOk();
+            // notification of action chain end can be done with a no-operation action class
+            // with a proper ListenerPropertyNames.ACTION_NORMAL_COMPLETION_CALLBACK method
+            // Same idea when some interaction needed with the 'esb unaware' world
         } 
         catch(ClassNotFoundException e)
     	{	_logger.error(prematureTermination("action class is not in path"),e); } 
@@ -170,11 +172,6 @@
 	    	catch (IllegalAccessException e) 	{_logger.error(e); }
     }
 
-    //TODO  What do we do here ?  Is this really necessary ?  Or do we do it in the last action class ?
-    protected void actionChainFinishedOk()
-    {
-    }
-
     protected String[] getActionNames()
     {
     	String[] sa = new String[_actionList.length];