Re: Cookie Interceptor

2013-04-03 Thread JOSE L MARTINEZ-AVIAL
Done https://issues.apache.org/jira/browse/WW-4037 2013/4/3 JOSE L MARTINEZ-AVIAL > You are right. Only thing is that due to time constraints I can only > provide the code and some explanations. The code is production ready for > me, but I know for a fact it needs to be modified in order to mak

Re: Cookie Interceptor

2013-04-03 Thread JOSE L MARTINEZ-AVIAL
You are right. Only thing is that due to time constraints I can only provide the code and some explanations. The code is production ready for me, but I know for a fact it needs to be modified in order to make it more struts-like (basically avoiding any dependence of javax.servlet). I will create th

RE: Cookie Interceptor

2013-04-03 Thread Martin Gainty
if you are convinced this Intercetpr and Interface doesnt exist in the codebase.. i am certain the struts devs would like to have these 2 classes incorporated into the codebase why not file JIRA ? https://issues.apache.org/jira/browse/WW Saludos Cordiales desde EEUU Martin

Re: Cookie Interceptor

2013-04-03 Thread JOSE L MARTINEZ-AVIAL
I use CookieAware in Struts 2.2.3, and I don't have any issues for not having the setters. But I do have a bunch of issues with cookies with name MENU-STATUS, since the interceptor tries to parse the cookie name as an OGNL value (ACCEPTED_PATTERN was added later than 2.2.3). My proposal would be to

Cookie Interceptor

2013-04-03 Thread Christian Grobmeier
Hi, the CookieInterceptor currently looks for all available cookies. Lets say there are cookies x, y, z. Now it would call setX, setY, setZ on the action. In addition it creates a cookie-map which is then injected to the action. The cookie interceptor does not make sense at a general level so far

Re: Struts 2.3.14 ready to test

2013-04-03 Thread Lukasz Lenart
2013/4/3 Christian Grobmeier : > So far I could not spot any more issues... I consider my testing app a > pretty "standard" app Cool! Let's wait few more days... Regards -- Ɓukasz + 48 606 323 122 http://www.lenart.org.pl/ - T

Re: Struts 2.3.14 ready to test

2013-04-03 Thread Christian Grobmeier
So far I could not spot any more issues... I consider my testing app a pretty "standard" app Cheers On Fri, Mar 29, 2013 at 12:12 AM, Lukasz Lenart wrote: > Hi, > > Please take a time and test the bits - any help is appreciated. > > Staging Maven repo > https://repository.apache.org/content/grou