Re: FW: [jira] [Comment Edited] (WW-4815) Migrating Struts 2.3.16.3 to 2.3.32

2017-08-07 Thread Yasser Zamani
Ouch... I now remembered same issue and what the problem was. Please try following which is documented at [1]. Field names If you have field names which starts with single lower case letter, for example: private String sTrng; public String getSTrng() {...} public void setSTrng(String str)

RE: FW: [jira] [Comment Edited] (WW-4815) Migrating Struts 2.3.16.3 to 2.3.32

2017-08-07 Thread Deborah White
It does reach that line and for some reason, it gets a value of 0 instead of 1 passed in from the .jsp, so then it does not do the return. I can't see anything obvious at all. Works with 2.3.16.3. Not finding renewSave1 or 2 in the .java file, strictly in struts.xml. -Original

[GitHub] struts issue #153: WW-4827 Not fully initialized ObjectFactory tries to crea...

2017-08-07 Thread yasserzamani
Github user yasserzamani commented on the issue: https://github.com/apache/struts/pull/153 > Thanks a lot :) I see now what's going on but have no idea why :\ @lukaszlenart , as I debug, the trace is `object factory creation` then `set1 (inject it's 1st property)` then `set2

Build failed in Jenkins: Struts-support-2-3-JDK6 #8

2017-08-07 Thread Apache Jenkins Server
See Changes: [stefaan.dutry] WW-4834 Improve RegEx used to validate URLs -- [...truncated 2.49 MB...] Uploading:

[GitHub] struts pull request #159: WW-4834 Improve RegEx used to validate URLs

2017-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/159 --- 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, or if the feature is

Jenkins build is back to normal : Struts-master-JDK8 #17

2017-08-07 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 pull request #160: WW-4839 JakartaStreamMultiPartRequest Should Honor...

2017-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/160 --- 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, or if the feature is

[GitHub] struts issue #159: WW-4834 Improve RegEx used to validate URLs

2017-08-07 Thread sdutry
Github user sdutry commented on the issue: https://github.com/apache/struts/pull/159 @lukaszlenart i copied the regex after the changes from #157 . So it should be the same. Feel free to check. --- If your project is set up for it, you can reply to this email and have your

[GitHub] struts pull request #161: WW-4840 Build Fails Due to Unused com.sun Import

2017-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/161 --- 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, or if the feature is

Build failed in Jenkins: Struts-master-JDK8 #16

2017-08-07 Thread Apache Jenkins Server
See Changes: [stefaan.dutry] WW-4834 fixed faulty regex [stefaan.dutry] further fixed faulty RegEx: missing grouping for last number in [stefaan.dutry] added tests for valid urls with default regex consisting

[GitHub] struts issue #159: WW-4834 Improve RegEx used to validate URLs

2017-08-07 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/159 Is it ready to merge? Does it contain the latest changes from #157? --- 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