[GitHub] struts issue #108: ConversionErrorInterceptor to extend MethodFilterIntercep...

2016-08-04 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/108 Still two tests are failing ... I think it would be better to setup `excludeMethods`/`includeMethods` in test and call`intercept()` --- If your project is set up for it, you can reply to this

Jenkins build is still unstable: Struts-JDK7-pull-request #210

2016-08-04 Thread Apache Jenkins Server
See - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org

[GitHub] struts issue #108: ConversionErrorInterceptor to extend MethodFilterIntercep...

2016-08-04 Thread gregh3269
Github user gregh3269 commented on the issue: https://github.com/apache/struts/pull/108 OK, looks like it needs a doIntercept(..) rather than in Intercept(). On 4 August 2016 at 09:22, Lukasz Lenart wrote: > Some tests failed >

[GitHub] struts issue #108: ConversionErrorInterceptor to extend MethodFilterIntercep...

2016-08-04 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/108 Some tests failed https://builds.apache.org/job/Struts-JDK7-pull-request/209/org.apache.struts$struts2-core/testReport/ --- If your project is set up for it, you can reply to this email and

Re: ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-04 Thread Lukasz Lenart
2016-08-04 9:40 GMT+02:00 Greg Huber : > OK I will read up on adding another upstream(1) for > https://git-wip-us.apache.org/repos/asf/struts.git for direct commits and > the egit logic for keeping the stuff in sync (upstream/upstream(1)/origin). I'm using such schema

Jenkins build became unstable: Struts-JDK7-pull-request #209

2016-08-04 Thread Apache Jenkins Server
See - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org

Re: ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-04 Thread Greg Huber
OK I will read up on adding another upstream(1) for https://git-wip-us.apache.org/repos/asf/struts.git for direct commits and the egit logic for keeping the stuff in sync (upstream/upstream(1)/origin). On 3 August 2016 at 11:28, Lukasz Lenart wrote: > 2016-08-03 12:15

[GitHub] struts pull request #108: ConversionErrorInterceptor to extend MethodFilterI...

2016-08-04 Thread gregh3269
GitHub user gregh3269 opened a pull request: https://github.com/apache/struts/pull/108 ConversionErrorInterceptor to extend MethodFilterIntercept See WW-4676 Cheers You can merge this pull request into a Git repository by running: $ git pull