|
From: =?iso-8859-1?Q?<jp....@ti...> - 2003-07-23 13:57:50
|
+1 for me,=0D=0A=0D=0AJean-Pierre=0D=0A=0D=0A---------- Initial Header --=
---------=0D=0A=0D=0AFrom : j=FCrgen h=F6ller [werk3AT] <juergen.hoe=
ll...@we...>=0D=0ATo : <jp....@ti...>,"springframew=
ork-developer" <spr...@li...>=0D=0ACc =
: =0D=0ADate : Wed, 23 Jul 2003 13:15:59 +0200=0D=0ASubject=
: Re: [Springframework-developer] Summary of pending points=0D=0A=0D=0AJ=
ean-Pierre, Thomas, Rod, everybody,=0D=0A=0D=0AI suggest to address the i=
ssue by offering overloaded execute and mapRow methods with a "Map contex=
t" parameter (instead of "Object context"). Additionally, I suggest to of=
fer just one single MappingSqlQuery class with 3 mapRow hooks:=0D=0A=0D=0A=
- mapRow(ResultSet rs, int rowNum, Map context, Object[] parameters)=0D=0A=
- mapRow(ResultSet rs, int rowNum, Map context)=0D=0A- mapRow(ResultSet r=
s, int rowNum)=0D=0A=0D=0AThe default implementations of the former two c=
ould simply cascade towards the latter. A subclass can choose to override=
any of those. This mechanism is inspired by our SimpleFormController tha=
t offers similar cascading methods for onSubmit.=0D=0A=0D=0AThis would ef=
fectively mean dissolving MappingSqlQueryWithParameters and merging it wi=
th MappingSqlQuery. As I assume that most people use MappingSqlQuery anyw=
ay, this shouldn't pose a problem. All former MappingSqlQueryWithParamete=
rs subclasses would simply have to extend the merged MappingSqlQuery then=
.=0D=0A=0D=0AAs I consider both a context map and a consolidation of all =
overloaded mapRow methods into MappingSqlQuery sensible and straightforwa=
rd changes, I suggest to implement them promptly. I'm even willing to inc=
lude this in 0.9.1, to reflect the merging of MappingSqlQueryWithParamete=
rs and MappingSqlQuery as early as possible.=0D=0A=0D=0AWhat do you think=
?=0D=0A=0D=0AJuergen=0D=0A=0A=0A********** L'ADSL A 20 EUR/MOIS**********=
=0ATiscali propose l'ADSL le moins cher du march=E9 : 20 EUR/mois et le m=
odem ADSL offert ! =0APour profiter de cette offre exceptionnelle, clique=
z ici : http://register.tiscali.fr/adsl/=0AOffre soumise =E0 conditions.=0A=
|