You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by tm jee <tm...@yahoo.co.uk> on 2006/11/01 06:45:15 UTC

[s2][Improvement] ActionContextCleanUp and FilterDispatcher

Hi,

Both filter, ActionContextCleanUp and FilterDispatcher contains some similar logic which I think would be more appropriate if we move them to an abstract superclass, maybe called AbstractFilterDispatcher.

Some of the logics are 
- creation of a Dispatcher  
- populating the created Dispatcher to Dispatcher's ThreadLocal
- using Dispatcher prepare method.

rgds


 
 Send instant messages to your online friends http://uk.messenger.yahoo.com 

Re: [s2][Improvement] ActionContextCleanUp and FilterDispatcher

Posted by tm_jee <fo...@opensymphony.com>.
Resolved issue. 

Please do feel free to review comment the code. Changes details are at the jira issue itself. Thx

Cheers.
---------------------------------------------------------------------
Posted via Jive Forums
http://forums.opensymphony.com/thread.jspa?threadID=48630&messageID=99258#99258


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Re: [s2][Improvement] ActionContextCleanUp and FilterDispatcher

Posted by tm_jee <fo...@opensymphony.com>.
Okie dokie. Created a jira issue for this

https://issues.apache.org/struts/browse/WW-1489
---------------------------------------------------------------------
Posted via Jive Forums
http://forums.opensymphony.com/thread.jspa?threadID=48630&messageID=98894#98894


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Re: [s2][Improvement] ActionContextCleanUp and FilterDispatcher

Posted by Don Brown <mr...@twdata.org>.
Sounds good.

Don

tm jee wrote:
> Hi,
>
> Both filter, ActionContextCleanUp and FilterDispatcher contains some similar logic which I think would be more appropriate if we move them to an abstract superclass, maybe called AbstractFilterDispatcher.
>
> Some of the logics are 
> - creation of a Dispatcher  
> - populating the created Dispatcher to Dispatcher's ThreadLocal
> - using Dispatcher prepare method.
>
> rgds
>
>
>  
>  Send instant messages to your online friends http://uk.messenger.yahoo.com 
>   


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org