Patches item #485601, was opened at 2001-11-26 05:57
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=402706&aid=485601&group_id=31602

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Mathias Bogaert (pathoss)
>Assigned to: Dmitri Colebatch (dimc)
Summary: Struts form name has bad default

Initial Comment:
IMHO the for name should not have the ejbname in front 
of it (for example: registration.Full).

Most of the examples and real uses of Struts use the 
entire decapitalized bean name (ejb name) for the form 
name.

So the line 

return Introspector.decapitalize( ejbName() + "." + 
getParameterValue( DocletUtil.getText( 
currentTag ), "name", 0 ) );

in StrutsFormTagsHandler should be changed to

return Introspector.decapitalize( getParameterValue( 
DocletUtil.getText( currentTag ), "name", 0 ) );

so that the developer is sure that the name will be 
the final name in struts-config.xml.

One could also check if name is not null, and if it 
is, by default use

return Introspector.decapitalize( ejbName() + "Form" );

Thanks,
Mathias

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=402706&aid=485601&group_id=31602

_______________________________________________
Xdoclet-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-devel

Reply via email to