[ http://jira.codehaus.org/browse/DISPL-238?page=comments#action_46830 ]
Emmanuel Feller commented on DISPL-238: --------------------------------------- Excuse me from beeing late ! Yes, you're wright on the css order, but in fact i changed the returned string to remove the space betwenn css classe names. Therefor i may have "class1class2" or "class1 class2". So for me the iteration order was very important. So thanks for the commit. Emmanuel > org.displaytag.util.MultipleHtmlAttribute : css class composition is not safe > ------------------------------------------------------------------------------ > > Key: DISPL-238 > URL: http://jira.codehaus.org/browse/DISPL-238 > Project: DisplayTag > Type: Bug > Versions: 1.0 > Fix For: 1.1 > Attachments: MultipleHtmlAttribute.java > > Original Estimate: 2 minutes > Remaining: 2 minutes > > org.displaytag.util.MultipleHtmlAttribue use an HashSet to store the elements > of css class namùe composition. But HashSet is not order safe and the same > application deployed on a websphere 5.0.2 and websphere 5.1 does not have the > same name at the result. > To resolve this bug you have to replace the HashSet use by a LinkedHashSet > (import and line 58). > I may not create a patch since i does not have the rights tools at my office. > Therefore i attach you the replacement class (with only the two modification). -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ------------------------------------------------------- SF.Net email is sponsored by: Tame your development challenges with Apache's Geronimo App Server. Download it for free - -and be entered to win a 42" plasma tv or your very own Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php _______________________________________________ displaytag-devel mailing list displaytag-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/displaytag-devel
