dmkarr 2002/10/02 22:02:32
Modified: contrib/struts-el/src/share/org/apache/strutsel/taglib/html
ELOptionsCollectionTag.java
Log:
Replaced commented-out handler for "styleId" with a block comment that explains
WHY the base tag (and this tag) does not support a "styleId" attribute (as "id"
has to be unique in the document).
Revision Changes Path
1.4 +10 -9
jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html/ELOptionsCollectionTag.java
Index: ELOptionsCollectionTag.java
===================================================================
RCS file:
/home/cvs/jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html/ELOptionsCollectionTag.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- ELOptionsCollectionTag.java 1 Oct 2002 04:25:50 -0000 1.3
+++ ELOptionsCollectionTag.java 3 Oct 2002 05:02:32 -0000 1.4
@@ -169,11 +169,12 @@
setStyleClass(null);
}
-// try {
-// setStyleId((String) evalAttr("styleId", getStyleId(), String.class));
-// } catch (NullAttributeException ex) {
-// setStyleId(null);
-// }
+ // Note that in contrast to other elements which have "style" and
+ // "styleClass" attributes, this tag does not have a "styleId"
+ // attribute. This is because this produces the "id" attribute, which
+ // has to be unique document-wide, but this tag can generate more than
+ // one "option" element. Thus, the base tag, "OptionsCollectionTag"
+ // does not support this attribute.
try {
setValue((String) evalAttr("value", getValue(), String.class));
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
