dmkarr 2002/10/02 22:02:04
Modified: contrib/struts-el/src/share/org/apache/strutsel/taglib/html
ELOptionsTag.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/ELOptionsTag.java
Index: ELOptionsTag.java
===================================================================
RCS file:
/home/cvs/jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html/ELOptionsTag.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- ELOptionsTag.java 1 Oct 2002 04:25:50 -0000 1.3
+++ ELOptionsTag.java 3 Oct 2002 05:02:04 -0000 1.4
@@ -180,10 +180,11 @@
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, "Options" does not
+ // support this attribute.
}
}
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
