Re: Showcase app ajax validation check

2017-02-16 Thread Greg Huber
>I don't understand, 400 is a common way to say that the request was >wrong, do you want something else? On firefox, a 400 triggers the error: function(xhr, textStatus, errorThrown) {..} so only displays 'Network or server error!' rather than the validation response. Maybe a 400 on other

Re: Showcase app ajax validation check

2017-02-16 Thread Lukasz Lenart
2017-02-16 9:08 GMT+01:00 Greg Huber : >>I don't understand, 400 is a common way to say that the request was >>wrong, do you want something else? > > On firefox, a 400 triggers the > > error: function(xhr, textStatus, errorThrown) {..} > > so only displays 'Network or server

[GitHub] struts issue #118: [WW-4105] OgnlUtil improved in order to only setting prop...

2017-02-16 Thread yasserzamani
Github user yasserzamani commented on the issue: https://github.com/apache/struts/pull/118 😍 PR re-based with last master and ChainingInterceptor equipped with an optional param (editableClass) which user can set to a custom class (or interface) fully qualified name to restrict

[GitHub] struts issue #118: [WW-4105] OgnlUtil improved in order to only setting prop...

2017-02-16 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/118 only someone with write access can do it, so let's assume that adding :+1: by the author means `Ready for merge` - I will update docs --- If your project is set up for it, you can reply to

[GitHub] struts issue #118: [WW-4105] OgnlUtil improved in order to only setting prop...

2017-02-16 Thread yasserzamani
Github user yasserzamani commented on the issue: https://github.com/apache/struts/pull/118 @lukaszlenart , I am not able to mark this PR with a label `Ready for merge`. I think because I should not have write access to this repository. --- If your project is set up for it, you can

[GitHub] struts issue #118: [WW-4105] OgnlUtil improved in order to only setting prop...

2017-02-16 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/118 ech... will ask Infra --- 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