I decided last night that the SessionValidators for the Template systems
(WebMacro/FreeTrade/...) should be restructured a bit.
If an app is set up to use template defaults for things such as homepage,
login, etc., the SessionValidators should use those values as opposed to the
default Screens. So now the Template*SessionValidators only set Screen
defaults if a template default is not found.
The other thing I did was to remove the FreeMarker/WebMacro specific named
SessionValidators since there was nothing specific to the template engine in
the body of the classes. This is going to cause some errors to pop up for
those who extended the now absent classes or those who miss the update to
their properties file. But I think it is a good idea to use a common name
as we add the Velocity and JSP services.
I also touched Turbine.java. I commented out some code that was setting the
Screen parameter to the homepage before the redirect. The SessionValidators
have had the responsibility in the past of setting the default response if
none is specified. So it appeared to me that this code mainly worked to add
a screen pathinfo variable to the url, though if it is not there the
SessionValidator would set it anyway.
It is not relevant in many cases now where the homepage is a template, so I
am proposing that we drop it. Though if there is a reason it needs to be
there I guess we can just make sure to change the screen.homepage property
to return null by default.
John McNally
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?: [EMAIL PROTECTED]
