Re: redirectAction: different behaviour depending on the name of the result

2017-06-29 Thread i...@flyingfischer.ch
> This may have other reasons. Is there a value for myParam in every case? > What have you configured for struts.url.includeParams? > > > Regards, > Christoph > > This Email was scanned by Sophos Anti Virus > You are right! It's been an issue with myParam and not the architecture. Shame... It wo

Re: redirectAction: different behaviour depending on the name of the result

2017-06-29 Thread Christoph Nenning
> > No big deal and probably not related to 2.5.11. Is this on purpose: > > redirectAction behaves differently depending on the name of the result. > > > login > ${myParam} > > > Called in action as: return "redirectToLogin"; > > => myParam gets transmitte

redirectAction: different behaviour depending on the name of the result

2017-06-29 Thread i...@flyingfischer.ch
No big deal and probably not related to 2.5.11. Is this on purpose: redirectAction behaves differently depending on the name of the result. login ${myParam} Called in action as: return "redirectToLogin"; => myParam gets transmitted as expected.

Re: Jenkins build is back to normal : Struts-2_3-JDK6 #2

2017-06-29 Thread Christoph Nenning
> > Please wait! Maybe we don't have to convert them because I could compile > > and pass tests of Struts 2.3.33 under JDK8 and maven 3.5.0 so we can > > keep it alive :) > > > > To do this, we should do just two things on Struts 2.3.33: > > > > 1) Adding tools.jar from JDK7 to classpath of the ma

Re: Jenkins build is back to normal : Struts-2_3-JDK6 #2

2017-06-29 Thread Christoph Nenning
> > Hip, hip hurra! Now I can convert the rest of the builds :) > I'm glad that it works :) Regards, Christoph This Email was scanned by Sophos Anti Virus

Re: Jenkins build is back to normal : Struts-2_3-JDK6 #2

2017-06-29 Thread Lukasz Lenart
2017-06-29 10:06 GMT+02:00 Yasser Zamani : > Please wait! Maybe we don't have to convert them because I could compile > and pass tests of Struts 2.3.33 under JDK8 and maven 3.5.0 so we can > keep it alive :) > > To do this, we should do just two things on Struts 2.3.33: > > 1) Adding tools.jar from

Re: Jenkins build is back to normal : Struts-2_3-JDK6 #2

2017-06-29 Thread Yasser Zamani
Please wait! Maybe we don't have to convert them because I could compile and pass tests of Struts 2.3.33 under JDK8 and maven 3.5.0 so we can keep it alive :) To do this, we should do just two things on Struts 2.3.33: 1) Adding tools.jar from JDK7 to classpath of the maven. 2) Correcting four

[GitHub] struts issue #145: WW-4805 Ports some of PR #142 changes to branch support-2...

2017-06-29 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/145 Nope, I mean PRs are a GitHub specific feature, and looks like PR is always merged with the `master` branch so checking out the PR isn't an option to test non-`master` PRs ;-) https://h

[GitHub] struts pull request #145: WW-4805 Ports some of PR #142 changes to branch su...

2017-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/145 --- 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 enab

[GitHub] struts issue #145: WW-4805 Ports some of PR #142 changes to branch support-2...

2017-06-29 Thread yasserzamani
Github user yasserzamani commented on the issue: https://github.com/apache/struts/pull/145 @lukaszlenart , it seems it was occurred just on your local PC during git fetch and there is not any issue with github or original apache repositories. Please revert your local changes and try c

[GitHub] struts issue #145: WW-4805 Ports some of PR #142 changes to branch support-2...

2017-06-29 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/145 Nope, I'm getting compilation errors: ``` [ERROR] chainHistory = new LinkedList<>(); ``` which means when I fetch your PR it was merged against the `master` branch instead of

Re: Jenkins build is back to normal : Struts-2_3-JDK6 #2

2017-06-29 Thread Lukasz Lenart
Hip, hip hurra! Now I can convert the rest of the builds :) 2017-06-29 9:01 GMT+02:00 Apache Jenkins Server : > See > > > - > To unsubscribe, e-mail: dev-unsubscr.

Jenkins build is back to normal : Struts-2_3-JDK6 #2

2017-06-29 Thread Apache Jenkins Server
See - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org