|
From: Roberto C. <ro...@in...> - 2004-11-19 22:32:07
|
Yes,
a) I would refactor the method renderMergedTemplateModel extracting the c=
ode
that puts in the Context the different formatters and tools.
Of course, I'd make it protected in order to be overridden.
b) mergeTemplate should not take the Template argument. Template is an
instance variable, so it doesn't need to be passed in.
It could be renamed into "mergeContext" instead. In this way one coul=
d
use the context the merge more than one templates as in my case nicely.
Hope I made clear my ideas and that they make sense to you all.
/robcos
----- Original Message -----
From: "j=FCrgen h=F6ller [werk3AT]" <jue...@we...>
To: <spr...@li...>
Sent: Friday, November 19, 2004 4:25 PM
Subject: Re: [Springframework-developer] Subclassing VelocityView
OK, I see. I'll add an "isCacheTemplate" getter. Do you have any other
suggestions on how to improve VelocityView in terms of easy subclassing?
Juergen
|