Two more comments:

In the second tutorial, you are preparing the employeeForm using the
SetUpEmployeeAction.  In doing so, you run

employeeForm.setDepartment("2");

Shouldn't you instead set the default department selection in the
form reset() method?  Naturally the reset() method is the incorrect
place to populate the options, but it is a good place to define
which one will be selected by default.

In regards to the discussion on using Tokens for the forward mapping
keys (such as "success" and "continue") you may also want to note
that doing so allows you to have two tokens point to the same
definition.  In short

Tokens.SUCCESS_KEY = "success";

Tokens.CONTINUE_KEY = "success";

So you can really play with the means of your Action class without
having to bother the Action class.  In short the action class
shouldn't really care what view is choosen or its meaning, it just
deals with the general workflow, such as

success
failure
continue
redo

Dan

-- 
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
Daniel Allen, <[EMAIL PROTECTED]>
http://www.mojavelinux.com/
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
"If you still don't like it, that's ok: that's why I'm boss. I 
simply know better than you do." 
 -- Linus Torvalds
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to