[MYFACES-3256] CommonPropertyUtils assumes all its managed HTML attributes hold string values

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 2.0.8-SNAPSHOT, 2.1.2-SNAPSHOT
• Fix Version/s: 2.0.8, 2.1.2
• Assignee: Leonardo Uribe
• Reporter: Matthew Jason Benson
• Created: 02/Aug/11 16:36
• Updated: 23/Aug/11 02:36
• Resolved: 02/Aug/11 17:32

Description

I encountered the ClassCastException due to Integer vs. String on h:input @maxlength, which is defined at http://download.oracle.com/docs/cd/E17802_01/j2ee/javaee/javaserverfaces/2.0/docs/pdldocs/facelets/h/inputText.html as (must evaluate to int). I would presume the same for @size (int) and @readonly (boolean). Perhaps naively, it would seem that simply removing the String cast and using Object in CommonPropertyUtils#renderHtmlStringAttribute() (potentially renaming this method in the process) would solve the issue.

Issue Links

• blocks: MYFACES-3237 [PERF] Renderers for components like h:outputText and others do many unecessary getAttributes().get() calls