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

2017-08-04 Thread Yasser Zamani
you're welcome :) 1. Does gov.ca.doj.sotas.action.renew.ExtRenewSave.execute reach line `return "renewEsignProc";` after `document.regSubmitForm.submit();`? please verify by a breakpoint. 2. What do you find if you search "renewSave1" (including double quotes) in all of your .java files? (also

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

2017-08-04 Thread Deborah White
So I have now updated to 2.3.33 and have a new piece of code that is not acting as expected. You were such a big help last time I thought I would ask. I have this in my jsp: function submitESignature() { $('#StatusMessage').html(""); //document.getElementById("button_co

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

2017-08-04 Thread swiftelan
GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/161 WW-4840 Build Fails Due to Unused com.sun Import [WW-4840](https://issues.apache.org/jira/browse/WW-4840) Build Fails Due to Unused com.sun Import You can merge this pull request into a Git repos

[GitHub] struts pull request #160: WW-4839 JakartaStreamMultiPartRequest Should Honor...

2017-08-04 Thread swiftelan
GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/160 WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.… [WW-4839](https://issues.apache.org/jira/browse/WW-4839) JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSi

Re: Struts 2.5.13

2017-08-04 Thread Sreenivas M
m** On 3 Aug 2017 7:39 pm, "Lukasz Lenart" wrote: > Hi > > We are almost good to go ... there is one issue to be reviewed and I > can start the release process. > https://issues.apache.org/jira/projects/WW/versions/12341116 > > > Regards > -- > Łukasz > + 48 606 323 122 http://www.lenart.org.pl

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

2017-08-04 Thread yasserzamani
Github user yasserzamani commented on the issue: https://github.com/apache/struts/pull/153 :) Yes I know, just a bit worry about backward compatibility between minor versions. I also am studying your sample and @lukaszlenart 's pr to understand what is going on as I think sam

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

2017-08-04 Thread aleksandr-m
Github user aleksandr-m commented on the issue: https://github.com/apache/struts/pull/153 @yasserzamani Looks like a hack :) I would rather leave as it is, then to introduce that. If container is needed then it **should** be injected in the constructor. Take a look at spring object f

[GitHub] struts issue #157: WW-4834 fixed faulty regex

2017-08-04 Thread atcazzual
Github user atcazzual commented on the issue: https://github.com/apache/struts/pull/157 Looks good to me. All matching capabilities from prior to the optimizations seem to be there. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

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

2017-08-04 Thread yasserzamani
Github user yasserzamani commented on the issue: https://github.com/apache/struts/pull/153 @aleksandr-m , I modified `ObjectFactory` as below ```java ... private List myPrivateInjectQueue; ... @Inject public void setContainer(Container container)

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

2017-08-04 Thread sdutry
Github user sdutry commented on the issue: https://github.com/apache/struts/pull/159 see #157 and #156 --- 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

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

2017-08-04 Thread sdutry
GitHub user sdutry opened a pull request: https://github.com/apache/struts/pull/159 WW-4834 Improve RegEx used to validate URLs You can merge this pull request into a Git repository by running: $ git pull https://github.com/sdutry/struts WW-4834-support-2-3 Alternatively you