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: Closed
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

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

>Comment By: Dmitri Colebatch (dimc)
Date: 2002-02-13 04:58

Message:
Logged In: YES 
user_id=204154

It looks like this has already been done.

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

Comment By: Dmitri Colebatch (dimc)
Date: 2001-12-08 06:11

Message:
Logged In: YES 
user_id=204154

The form name is prefixed with the ejb name for namespace 
issues isn't it?  I would have thought that by removing the 
prefix, you then run the risk of (using defaults) 
generating name clashes.

Could I request some other input into this so we can close 
it one way or the other?

cheers
dim

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

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