Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 1.8
-
Fix Version/s: 1.8.2
-
Component/s: Web - Struts
-
Labels:None
-
Environment:All
Description
This comment at line 143 in /metadata/templates/struts_form.xdt refers to an incorrect file name for the 'merge file'.
/* To add non XDoclet-generated methods, create a file named
actionform-methods-<XDtClass:classOf><XDtActionForm:actionFormClass/></XDtClass:classOf>.java
containing the additional code and place it in your merge dir.
*/
The prefix 'actionform-methods' should be 'xdoclet-' as below:
/* To add non XDoclet-generated methods, create a file named
xdoclet-<XDtClass:classOf><XDtActionForm:actionFormClass/></XDtClass:classOf>.java
containing the additional code and place it in your merge dir (/metadata/web/).
*/
Also, after any methods are merged in, app gen removes this comment entirely. It would be my preference for the comment to stay in there in case I forget, or when others are trying to figure out my code. I would also prefer to have the actual path for the "merge dir" specified unless there some reason not to.
/* To add non XDoclet-generated methods, create a file named
actionform-methods-<XDtClass:classOf><XDtActionForm:actionFormClass/></XDtClass:classOf>.java
containing the additional code and place it in your merge dir.
*/
The prefix 'actionform-methods' should be 'xdoclet-' as below:
/* To add non XDoclet-generated methods, create a file named
xdoclet-<XDtClass:classOf><XDtActionForm:actionFormClass/></XDtClass:classOf>.java
containing the additional code and place it in your merge dir (/metadata/web/).
*/
Also, after any methods are merged in, app gen removes this comment entirely. It would be my preference for the comment to stay in there in case I forget, or when others are trying to figure out my code. I would also prefer to have the actual path for the "merge dir" specified unless there some reason not to.