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

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

Re: ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-03 Thread Lukasz Lenart
2016-08-03 12:15 GMT+02:00 Greg Huber : > Not that I understand much, if I try a push to my remote "upstream" it says > > https://github.com/apache/struts: git-receive-pack not permitted > > Remote Fetch Specification:+refs/heads/*:refs/remotes/upstream/* > Remote Fetch

Re: ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-03 Thread Greg Huber
Not that I understand much, if I try a push to my remote "upstream" it says https://github.com/apache/struts: git-receive-pack not permitted Remote Fetch Specification:+refs/heads/*:refs/remotes/upstream/* Remote Fetch URL:https://github.com/apache/struts The way I think I have it set

Re: ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-03 Thread Lukasz Lenart
2016-08-03 10:33 GMT+02:00 Greg Huber : > I will log a request and see if I can do a pull request, but I seem to have > lost the link I was using for this and I cannot get egit to be friendly :( This looks like one-commit-change, no need to prepare a PR - directly hit

Re: ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-03 Thread Lukasz Lenart
+1 2016-08-03 9:32 GMT+02:00 Greg Huber <gregh3...@gmail.com>: > Would it be possible to modify the ConversionErrorInterceptor to extend > MethodFilterInterceptor so I can exclude the validation on certain methods? > > ie > > > cancel* > > > It seems a

ConversionErrorInterceptor to extend MethodFilterInterceptor

2016-08-03 Thread Greg Huber
Would it be possible to modify the ConversionErrorInterceptor to extend MethodFilterInterceptor so I can exclude the validation on certain methods? ie cancel* It seems always to be called (needs to be like the validator/workflow) I have noticed that if I there is a conversion error