[GitHub] struts pull request #140: WW-4801 Only one hidden field per checkboxlist.

2017-05-30 Thread gregh3269
GitHub user gregh3269 opened a pull request: https://github.com/apache/struts/pull/140 WW-4801 Only one hidden field per checkboxlist. You can merge this pull request into a Git repository by running: $ git pull https://github.com/gregh3269/struts master Alternatively you

[GitHub] struts issue #130: WW-3952: Credit card validator

2017-04-21 Thread gregh3269
Github user gregh3269 commented on the issue: https://github.com/apache/struts/pull/130 Last time I did some credit card validation, I needed to use a java script version as the payment process obscured the field names by not having the "name" attribute (so it was not ob

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

2016-09-30 Thread gregh3269
Github user gregh3269 commented on the issue: https://github.com/apache/struts/pull/108 Great thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

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

2016-09-30 Thread gregh3269
Github user gregh3269 commented on the issue: https://github.com/apache/struts/pull/108 Revert "WW-4628: proper decoding of parameters in query-string" Sorry, this was a bad by not really understanding git and git making automatic commits, it should not hav

[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 <notificati...@github.com> wrote: > Some tests failed

[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 https://github.com

[GitHub] struts pull request: Allow html5 attributes

2016-02-10 Thread gregh3269
Github user gregh3269 commented on a diff in the pull request: https://github.com/apache/struts/pull/86#discussion_r52437467 --- Diff: plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/TextFieldHandler.java --- @@ -31,8 +31,8 @@ public void

[GitHub] struts pull request: Allow html5 attributes

2016-02-10 Thread gregh3269
GitHub user gregh3269 opened a pull request: https://github.com/apache/struts/pull/86 Allow html5 attributes See WW-4607 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gregh3269/struts master Alternatively you can review