Mailing-List: contact user-help@struts.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Struts Users Mailing List" <user@struts.apache.org>
MIME-Version: 1.0
In-Reply-To: <CAMopvkM6398xJUfJKmGPtEZVvqJdwXdxzb18bQrcoe37LiYNQg@mail.gmail.com>
References: <CAK8hL1SLOV46gmk0Gw1rTwp8Z+ZRbtabTh27DNJcAjeDK6zgzg@mail.gmail.com>
 <CAK8hL1SMs+GPdGTMbgMKVXFgT4fR=b=BFZR71JyofqS6aHSbGw@mail.gmail.com>
 <DB5PR08MB1062AF6438C4E80236586B1C92470@DB5PR08MB1062.eurprd08.prod.outlook.com>
 <CAK8hL1QAiTbxaswgCaUimC+oyot9BE1DLRu3NFVoLfCGLG7hHw@mail.gmail.com>
 <DB5PR08MB1062F62835F89F797DC2A30292470@DB5PR08MB1062.eurprd08.prod.outlook.com>
 <CAK8hL1RJATmi__YfunBN-yGHVfY7kmRAv-qbj-+a12r7FVd_rQ@mail.gmail.com>
 <DB5PR08MB1062F736F03D1FB984E8727692440@DB5PR08MB1062.eurprd08.prod.outlook.com>
 <CAK8hL1S=nhr+Dh5TXM4FbdUeb0e3a2d-bw84tH-s5O9-t9sWBg@mail.gmail.com>
 <DB5PR08MB10625937BC931F608C80832692440@DB5PR08MB1062.eurprd08.prod.outlook.com>
 <CAK8hL1Q6PpY8uWhv8jhQ3V+eVDGf057J54rwJNULYVctn1MSjQ@mail.gmail.com>
 <CAK8hL1TXSGgpdtEbirurk9DWTyEhMaZsGSAK29PdLXCigQE1HA@mail.gmail.com>
 <CAMopvkPdS1S=eF+=pFeKoMuGdjhytwCh9nAkzcZbUtWLi-YybQ@mail.gmail.com>
 <CAK8hL1S8Lp_niztAq6UptjJhp0z2wvzw0kPQ7=kmpJS88b38rA@mail.gmail.com>
 <DB5PR08MB106245333C5C473B8773BD7C92450@DB5PR08MB1062.eurprd08.prod.outlook.com>
 <CAK8hL1T=kvcnw_Shr0GmiibE1N_ujjoQrWU6aOWXG0sZVWbWug@mail.gmail.com>
 <CAMopvkONfxsnOBFxdaVA+HG4i9mpZWmjENPE03bmHSExZw7QCA@mail.gmail.com>
 <CAK8hL1R+Lu+zret-SrNRJZa0a_r8ygoh3izPRVNDH8N4uKf7HA@mail.gmail.com>
 <CAMopvkNSscy_z_ZDcOgLyCxBEhYE1OFoFHLs9EC1f2RHAz-TjQ@mail.gmail.com>
 <CAK8hL1Ta2RUFE6adns7vW2SBDzq+sg_k-iyKXm0kj7E8k2g5xw@mail.gmail.com> <CAMopvkM6398xJUfJKmGPtEZVvqJdwXdxzb18bQrcoe37LiYNQg@mail.gmail.com>
From: Ej Magdaluyo <ejmagdaluyo@gmail.com>
Date: Fri, 3 Nov 2017 20:38:05 +0800
Message-ID: <CAK8hL1QCFkn96nGsGR_iVPW-o-1MSMBbwcnnxpG8zCwcEqqxpQ@mail.gmail.com>
Subject: Re: Upgrading Struts 2.2.3 to 2.3.34
To: Struts Users Mailing List <user@struts.apache.org>
Content-Type: multipart/alternative; boundary="001a11c020f86d1126055d136089"
archived-at: Fri, 03 Nov 2017 12:38:14 -0000

--001a11c020f86d1126055d136089
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Lukasz, Yasser,

Sorry for the late reply. Yes, the tiles jsp jar is on the classpath.

After weeks of facing this issue, one of my colleagues was able to solve
the issue. We imported the following in the java file:

import org.apache.struts2.dispatcher.*;


and added the following in the viewMyRequest method:

RequestMap rm =3D (RequestMap)ActionContext.getContext().get("request");
rm.put("REQ", ViewReqDetails);


Then in the jsp, we changed the way values were being called:

ex: <s:property value=3D"#request.REQ.reqType"/>

<%
ViewReqDetailsVO ViewReqDetails =3D
(ViewReqDetailsVO)request.getAttribute("ViewReqDetails");
java.util.List classficationList =3D ViewReqDetails.getClassficationList();


This seemed to do the trick and we are now able to fetch all the values
correctly.

Thank you Yasser and Lukasz for all your help. This would not have been
possible without your assistance.

Regards,
EJ Magdaluyo

On Fri, Nov 3, 2017 at 8:05 PM, Lukasz Lenart <lukaszlenart@apache.org>
wrote:

> Do you have the Tiles JSP jar on the classpath?
>
> https://tiles.apache.org/framework/tiles-jsp/index.html
>
>
> Regards
> --
> =C5=81ukasz
> + 48 606 323 122 http://www.lenart.org.pl/
>
> 2017-10-30 16:36 GMT+01:00 Ej Magdaluyo <ejmagdaluyo@gmail.com>:
> > Hi Lukasz,
> >
> > Unfortunately, the coding conventions in our application are not up to
> > today's standards. We were given this old application to work on hence
> some
> > of the conventions (such as scriplets) are not ideal.
> >
> > This is the supposed flow:
> >
> > struts.xml:
> >
> > <action name=3D"viewMyRequest" method=3D"viewMyRequest"
> > class=3D"com.MyRequestAction">
> > <result name=3D"success">/jsp/viewMyRequest.jsp</result>
> > </action>
> >
> >
> > method viewMyRequest (we had tested the java file itself, and the reque=
st
> > is setting corerctly):
> >
> > public String viewMyRequest() throws Exception {
> > SearchCriteriaVO searchCriteriaVO=3D constructSearchCriteria();
> > MyRequestDelegate myRequestDelegate =3D new MyRequestDelegate();
> > ViewReqDetails =3D myRequestDelegate.getRequestViewDetails(
> searchCriteriaVO);
> > ViewReqDetails.setEditApprover(false);
> > String searchEnabled=3Drequest.getParameter("searchEnabled");
> > if(searchEnabled!=3Dnull&&searchEnabled.equalsIgnoreCase("true")){
> > ViewReqDetails.setSearchEnabled(true);
> > }
> > setViewReqDetails(ViewReqDetails);
> > request.setAttribute("ViewReqDetails", ViewReqDetails);
> > return  SUCCESS;
> > }
> >
> >
> > SUCCESS here should mean that the result will ensure that page will get
> > routed to /jsp/viewMyRequest.jsp.
> >
> > Problem now is that even though ViewReqDetails attribute was set, the
> value
> > is not transferring over once we try to retrieve this value in the jsp
> file:
> >
> > viewMyRequest.jsp:
> >
> > ViewReqDetailsVO ViewReqDetails =3D
> > (ViewReqDetailsVO)request.getAttribute("ViewReqDetails");
> > java.util.List classficationList =3D ViewReqDetails.
> getClassficationList();
> >
> > A null value is being returned here, which is most likely why we get 40=
3
> > forbidden also.
> >
> > I am not sure where in struts/tiles update caused the passing of reques=
t
> > attributes to change though, but I am guessing that is what caused this=
.
> >
> > Thanks!
> >
> >
> > On Fri, Oct 27, 2017 at 9:47 PM, Lukasz Lenart <lukaszlenart@apache.org=
>
> > wrote:
> >
> >> 2017-10-27 14:34 GMT+02:00 Ej Magdaluyo <ejmagdaluyo@gmail.com>:
> >> > This is the java code:
> >> > public String viewMyRequest() throws Exception {
> >> > SearchCriteriaVO searchCriteriaVO=3D constructSearchCriteria();
> >> > MyRequestDelegate myRequestDelegate =3D new MyRequestDelegate();
> >> > ViewReqDetails =3D myRequestDelegate.getRequestViewDetails(
> >> searchCriteriaVO);
> >> > ViewReqDetails.setEditApprover(false);
> >> > String searchEnabled=3Drequest.getParameter("searchEnabled");
> >> > if(searchEnabled!=3Dnull&&searchEnabled.equalsIgnoreCase("true")){
> >> > ViewReqDetails.setSearchEnabled(true);
> >> > }
> >> > setViewReqDetails(ViewReqDetails);
> >> > request.setAttribute("ViewReqDetails", ViewReqDetails);
> >> > return  SUCCESS;
> >> > }
> >> >
> >> >
> >> > If we try to run tests within this java file, we are able to get the
> >> values
> >> > without any issues. It is just when we try to call this attribute
> where
> >> the
> >> > problem begins.
> >>
> >> Now I'm confused ... you have this code in your action but still
> >> transferring it over request to a scriplet?
> >>
> >> How defintion of SUCCESS result looks like?
> >>
> >>
> >> Regards
> >> --
> >> =C5=81ukasz
> >> + 48 606 323 122 http://www.lenart.org.pl/
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: user-unsubscribe@struts.apache.org
> >> For additional commands, e-mail: user-help@struts.apache.org
> >>
> >>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: user-unsubscribe@struts.apache.org
> For additional commands, e-mail: user-help@struts.apache.org
>
>

--001a11c020f86d1126055d136089--