Mailing-List: contact dev-help@struts.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Struts Developers List" <dev@struts.apache.org>
MIME-Version: 1.0
In-Reply-To: 
 <CAMopvkP=yikUWBNtQd737NwzyF+DjfsH1o_s074gtpxneHPPKg@mail.gmail.com>
References: 
 <CAMopvkMBMcqQYBfnO9uHCLD9ehUo=qJ9QpjFKSHOXbuL9tBtzg@mail.gmail.com>
	<CAJi=3dUnJvnuT7crLnif+ZxEwY_=gWB1POnk87JpLPRrnNdQAA@mail.gmail.com>
	<CAMopvkN=HFeGBO0eutPvZpZ9Ez6E-brpse8c7EzWvGgRm=kjVg@mail.gmail.com>
	<CAJi=3dV6hAOs1aMZtkXP5ViY6GwxT3X1s9oOVW-HqJ9q7wQz_g@mail.gmail.com>
	<CAMopvkOT_1jyuipsg_u4QpWPKqp810=J9Ss=ZJyN1_1jgQwcSA@mail.gmail.com>
	<CAJi=3dUxAu908tUmx6uqT9auVg_k=W8fUq5ZHZ4voVZps=uYww@mail.gmail.com>
	<CAMopvkMBfNA149jidk30fpGuUEh+nN5XPVNK9uhT9t-WWSXRJQ@mail.gmail.com>
	<CAJi=3dVJ2DtQB7wHDUuF2Q=PxyEtSbZywytiXjBtD7Yfz0w5Pg@mail.gmail.com>
	<CAMopvkP=yikUWBNtQd737NwzyF+DjfsH1o_s074gtpxneHPPKg@mail.gmail.com>
Date: Fri, 18 Mar 2016 15:38:40 +0000
Message-ID: 
 <CAJi=3dU0Co5wa5U9mmHUrhPnRvac2aJ9guo8uskh_hyR1gRFCQ@mail.gmail.com>
Subject: Re: [VOTE] Struts 2.3.27
From: Greg Huber <gregh3269@gmail.com>
To: Struts Developers List <dev@struts.apache.org>
Content-Type: multipart/alternative; boundary=047d7b342bac8f0de1052e548ad3

--047d7b342bac8f0de1052e548ad3
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

The reason why its not working it needs to know whether it is an
instanceof ServletRedirectResult in MessageStorePreResultListener.

Rather than use the result (which do do not have) a possible solution is to
construct what its looking for from the invocation and the use an equals.


Map<String, ResultConfig> results =3D
((DefaultActionInvocation)invocation).getProxy().getConfig().getResults();

        ResultConfig resultConfig =3D null;

        try {
            resultConfig =3D results.get(resultCode);
        } catch (NullPointerException e) {
            LOG.debug("Got NPE trying to read result configuration for
resultCode [{}]", resultCode);
        }

        boolean isRedirect =3D false;
        try {
            //isRedirect =3D invocation.getResult() instanceof
ServletRedirectResult;
            isRedirect =3D
"org.apache.struts2.result.ServletRedirectResult".equals(resultConfig.getCl=
assName());
        } catch (Exception e) {
            LOG.warn("Cannot read result!", e);
        }


On 18 March 2016 at 14:04, Lukasz Lenart <lukaszlenart@apache.org> wrote:

> I see no other way just revert that change and change that was
> introduced to fix the original issue
>
>
> Regards
> --
> =C5=81ukasz
> + 48 606 323 122 http://www.lenart.org.pl/
>
> 2016-03-18 14:26 GMT+01:00 Greg Huber <gregh3269@gmail.com>:
> > Sorry forget the last email, its rubbish.  Won't work. Thought the code
> was
> > part of the mod, which it is not.
> >
> > On 18 March 2016 at 11:45, Lukasz Lenart <lukaszlenart@apache.org>
> wrote:
> >
> >> 2016-03-18 12:29 GMT+01:00 Greg Huber <gregh3269@gmail.com>:
> >> > I have tested it without the change (to DefaultActionInvocation) and
> the
> >> > messages work on the redirects. Unless it is confirmed that it is
> >> required
> >> > ie it does not work in its original position, its best not to change
> >> such a
> >> > key program.  In my opinion.
> >>
> >> How do you use MessageStoreInterceptor with redirects? In AUTOMATIC or
> >> STORE mode?
> >> This change is needed to allow AUTOMATIC mode to work with redirects
> >>
> >>
> >>
> https://github.com/apache/struts/blob/master/core/src/main/java/org/apach=
e/struts2/interceptor/MessageStorePreResultListener.java#L72
> >>
> >>
> >> Regards
> >> --
> >> =C5=81ukasz
> >> + 48 606 323 122 http://www.lenart.org.pl/
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> >> For additional commands, e-mail: dev-help@struts.apache.org
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>

--047d7b342bac8f0de1052e548ad3--