|
From: Matt R. <li...@ra...> - 2004-04-28 08:01:45
|
+1 for adding them to the org.springframework.web.struts package. I use XDoclet for all my Actions in AppFuse and therefore, I'm not using the ContextLoaderPlugIn. I'd like to use it b/c then I can use MockStrutsTestCase to test my actions. This change would make it possible. Matt > -----Original Message----- > From: spr...@li...=20 > [mailto:spr...@li...] > On Behalf Of j=FCrgen h=F6ller [werk3AT] > Sent: Wednesday, April 28, 2004 1:36 AM > To: spr...@li... > Subject: Re: [Springframework-developer] Re: struts=20 > integration - alternate approach >=20 >=20 > I see. > =20 > I wonder whether it's worth adding a=20 > DelegatingRequestProcessor and a=20 > DelegatingTilesRequestProcessor to our=20 > org.springframework.web.struts package then: That approach is=20 > mainly useful when generating struts-config via XDoclet, if I=20 > understand correctly. Of course, quite a lot of people do use=20 > Struts with XDoclet. > =20 > What does everybody think? I've got those RequestProcessor=20 > subclasses lying around already; the question is whether to=20 > include them in Spring 1.0.2. Else, I'll add them to the=20 > sandbox for the time being. Votes, please :-) > =20 > Juergen > =20 >=20 > ________________________________ >=20 > Von: spr...@li... im=20 > Auftrag von Seth Ladd > Gesendet: Mo 26.04.2004 23:54 > An: spr...@li... > Betreff: Re: [Springframework-developer] Re: struts=20 > integration - alternate approach >=20 >=20 >=20 > j=FCrgen h=F6ller [werk3AT] wrote: > > Dan, > >=20 > > I've just done some prototypical tests. As far as I can=20 > tell, the only=20 > > benefit of the RequestProcessor approach is that you can write > >=20 > > <action path=3D"/test"/> > > > > rather than > >=20 > > <action path=3D"/test"=20 > > class=3D"org.springframework.web.struts.DelegatingActionProxy"/> > >=20 > > The disadvantage is that you need to have a special=20 > subclass of every=20 > > RequestProcessor that you might use: at least of the default=20 > > RequestProcessor and of TilesRequestProcessor. Some amount of code=20 > > duplication is inevitable there. And if you already have a custom=20 > > RequestProcessor, you need to subclass it on your own. > >=20 > > So all things considered, I still tend to recommend the=20 > > DelegatingActionProxy approach, which doesn't affect=20 > RequestProcessor=20 > > choice at all. Just saving the "class" attribute above does=20 > not seem=20 > > to be enough benefit to accept the RequestProcessor=20 > subclass hassle.=20 > > Is there anything I miss here? > >=20 > > Juergen > >=20 > > P.S.: > > I'm forwarding this to developer list for further feedback. >=20 > Juergen, >=20 > For our integration here, we ended up subclassing=20 > RequestProcessor. The reason was because we're using xdoclet=20 > to build our struts-config.xml file. So IIRC we couldn't use=20 > DelegatingActionProxy (or the similar > alternatives) because we needed Xdoclet to read into our=20 > action classes. >=20 > Hope that helps, > Seth >=20 >=20 >=20 > ------------------------------------------------------- > This SF.net email is sponsored by: The Robotic Monkeys at=20 > ThinkGeek For a limited time only, get FREE Ground shipping=20 > on all orders of $35 or more. Hurry up and shop folks, this=20 > offer expires April 30th!=20 > http://www.thinkgeek.com/freeshipping/?cpg=3D> 12297 >=20 > _______________________________________________ >=20 > Springframework-developer mailing list=20 > Spr...@li... > https://lists.sourceforge.net/lists/listinfo/springframework-developer >=20 >=20 >=20 >=20 > ------------------------------------------------------- > This SF.Net email is sponsored by: Oracle 10g > Get certified on the hottest thing ever to hit the market...=20 > Oracle 10g.=20 > Take an Oracle 10g class now, and we'll give you the exam FREE.=20 > http://ads.osdn.com/?ad_id149&alloc_id=8166&op=CCk > _______________________________________________ > Springframework-developer mailing list=20 > Spr...@li... > https://lists.sourceforge.net/lists/listinfo/springframework-developer >=20 |