Programming Jakarta Struts, 2nd edition
by Chuck Cavaness

The unconfirmed error reports are from readers.  They have not yet been 
approved or disproved by the author or editor and represent solely the
opinion of the reader.

Here's a key to the markup:
        [page-number]: serious technical mistake
        {page-number}: minor technical mistake
        <page-number>: important language/formatting problem
        (page-number): language change or minor formatting problem
        ?page-number?: reader question or request for clarification

This page was updated April 1, 2008.


UNCONFIRMED  errors and comments from readers: 


{7}  3rd paragraph.;
In referencing Figure 1-3, the paragraph begins by saying "Notice that there is no
servlet involved in the process".  However, on page 5, it correctly states that JSP
pages end up being nothing more than servlets.    The 1st and 2nd sentences could
probably be combined as "Notice that the client request is sent directly to a JSP
page...".  Then in paragraph 4, the part reading "the client request is first
intercepted by a servlet..." could be changed to something like "the client request
is first intercepted by a front controller servlet...".


{54} (LoginForm) :
        MessageResources resources =
      (MessageResources)req.getAttribute( Action.MESSAGES_KEY );

gives a compiler error. Solution:

        MessageResources resources =
      (MessageResources)req.getAttribute(org.apache.struts.Globals.MESSAGES_KEY  );


[41]  Figure 3-4 Caption;
"There is also a ninth top-level package named config, which consists..."
should read:
"There is also a ninth top-level package named plugins, which consists..."


(48)  Mapping the Actions section, 2nd paragraph;
"...instance of the org.apache.struts.action. ActionMapping..."
should read:
"...instance of the org.apache.struts.action.ActionMapping..."


(57)  2nd paragraph;
"...called org. apache.struts.action.DynaActionForm..."
should read:
"...called org.apache.struts.action.DynaActionForm..."


[123]  action tag example for ForwardAction;
There is an extra closing / at the end of the "validate=" attribute.


?161?  The comment in the logout method.;
The comment reads "Do nothing with right now, but might want to log it for auditing
purposes".  It is referring to the email argument.  This comment could be clarified a
bit.


{174}  example 7-1;
This example doesn't use jsp HTML tags. The source bundle does use jsp tags in
storefront/web/security/signin.jsp but doesn't work because the <html:form> scope
doesn't include the continue button.

Solution is to move closing <html:/form> to approx 93 (after the continue image)


[346] Near the end of sec 14.5;
I've discovered a problem with tiles. In your chapter on tiles you suggest that a tile definition can be referenced in a global forward, like this:

<global-forwards>
<forward name="Super_Success" path="storefront.superuser.main" />
</global-forwards>


However, this doesn't work - you'll get a 404 error saying /<whatever>/search.page can't be found.

This is a known problem - see http://issues.apache.org/bugzilla/show_bug.cgi?id=18799 It still doesn't work in Struts 1.3.8.

For some reason the Struts developers have decided not to fix it. In fact, there is a pretty simple workaround: create an action like this:

<action path="/Super_Success" forward="storefront.superuser.main"></action>