|
From: David H. <dh...@ob...> - 2005-03-23 09:30:32
|
I=20have=20a=20couple=20of=20comments=20about=20Webflow=20based=20on=20my=20= usage=20in=20a=20current=20application. =20 Firstly,=20FlowController=20is=20responsible=20for=20creating=20its=20own=20= HttpFlowExecutionManager=20-=20I=20guess=20I=20don't=20understand=20why=20= this=20isn't=20a=20dependency=20that's=20injected=20in=20the=20traditional= =20way.=20I=20had=20to=20subclass=20the=20existing=20flow=20execution=20ma= nager=20to=20get=20some=20behaviour=20I=20desired=20(storing=20flow=20exec= utions=20in=20the=20servlet=20context,=20rather=20than=20the=20session),=20= and=20the=20current=20structure=20meant=20I=20also=20had=20to=20subclass=20= FlowController.=20 =20 Secondly,=20it=20would=20be=20nice=20if=20the=20exceptions=20thrown=20to=20= indicate=20missing=20state=20or=20actions=20actually=20provided=20accessor= s=20to=20expose=20which=20state/action=20was=20missing.=20Currently=20I=20= have=20immplemented=20fallback=20logic=20such=20that=20if=20an=20action=20= is=20not=20found=20in=20the=20current=20flow,=20I=20can=20check=20in=20the= =20parent=20flow(s)=20and=20handle=20back=20button=20issues=20with=20subfl= ows=20quite=20elegantly.=20My=20logic=20would=20have=20been=20simpler=20if= =20the=20caught=20exception=20indicated=20the=20state/action=20that=20was=20= the=20problem. =20 Thanks, -Dave=20 ________________________________ From:=20s...@li...=20on=20behal= f=20of=20Erwin=20Vervaet Sent:=20Tue=2022/03/2005=2020:22 To:=20s...@li... Subject:=20[Springframework-developer]=20Web=20Flow=20progress All, We've=20been=20making=20very=20good=20progress=20on=20the=20web=20flow=20s= ystem=20the=20last=20few days.=20Things=20are=20shaping=20up=20for=20a=20preview=20release=20that=20= might=20come=20as=20soon=20as next=20weekend! The=20code=20is=20in=20good=20shape=20again=20and=20all=20JavaDoc=20is=20u= p-to-date.=20All=20samples are=20also=20working=20again. For=20those=20with=20an=20interest=20in=20web=20flows,=20I=20fully=20recom= mend=20that=20you=20take=20a look=20at=20the=20code=20that=20is=20in=20CVS=20right=20now (http://cvs.sourceforge.net/viewcvs.py/springframework/spring/sandbox/src/= org/springframework/web/flow/) and=20let=20us=20know=20what=20you=20think!=20Any=20input=20would=20be=20a= ppreciated! Issues=20left=20to=20work=20out=20(not=20necessarily=20for=20the=20preview= =20release=20:): *=20Conditional=20transitions:=20basic=20support=20is=20there=20(via=20Tra= nsitionCriteria interface=20and=20a=20simple=20OgnlTransitionCriteria=20implementation),=20= but=20details have=20to=20be=20worked=20out: =20=20=20=20-=20how=20to=20integrate=20this=20into=20XML=20flow=20definiti= ons? =20=20=20=20-=20scope=20precedence?=20or=20should=20all=20scopes=20be=20me= rged=20before=20being evaluated=20by=20the=20expression? *=20Token=20based=20transitions=20(e.g.=20<transition=20on=3D"countrySelec= ted" to=3D"register.${request.country}"/>):=20do=20we=20need=20this?=20There=20= is=20no=20support for=20this=20at=20the=20moment. *=20Does=20the=20RequestContext=20need=20to=20keep=20track=20of=20a=20hist= ory=20of=20events=20signaled during=20its=20lifecycle?=20It's=20not=20the=20case=20for=20the=20moment=20= but=20would=20be=20simple to=20implement.=20Also,=20do=20we=20need=20a=20query=20interface=20to=20fi= nd=20an=20event=20parameter across=20all=20signaled=20events=20for=20a=20particular=20request?=20Are=20= there=20any=20good use-cases=20for=20this? *=20Do=20we=20need=20some=20convenience=20accessor=20on=20the=20RequestCon= text=20to=20find attributes=20in=20any=20scope:=20try=20request=20scope=20first,=20then=20f= all=20back=20on=20flow scope?=20This=20would=20be=20simple=20to=20add. Erwin=20Vervaet erw...@er... ------------------------------------------------------- This=20SF.net=20email=20is=20sponsored=20by:=202005=20Windows=20Mobile=20A= pplication=20Contest Submit=20applications=20for=20Windows=20Mobile(tm)-based=20Pocket=20PCs=20= or=20Smartphones for=20the=20chance=20to=20win=20$25,000=20and=20application=20distribution= .=20Enter=20today=20at http://ads.osdn.com/?ad_id=3D6882&alloc_id=3D15148&op=3Dclick _______________________________________________ Springframework-developer=20mailing=20list Spr...@li... https://lists.sourceforge.net/lists/listinfo/springframework-developer ________________________________________________________________________ This=20e-mail=20has=20been=20scanned=20for=20all=20viruses=20by=20Star.=20= The service=20is=20powered=20by=20MessageLabs.=20For=20more=20information=20on= =20a=20proactive anti-virus=20service=20working=20around=20the=20clock,=20around=20the=20gl= obe,=20visit: http://www.star.net.uk ________________________________________________________________________ ________________________________________________________________________ This=20e-mail=20has=20been=20scanned=20for=20all=20viruses=20by=20Star.=20= The service=20is=20powered=20by=20MessageLabs.=20For=20more=20information=20on= =20a=20proactive anti-virus=20service=20working=20around=20the=20clock,=20around=20the=20gl= obe,=20visit: http://www.star.net.uk ________________________________________________________________________ |