You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2010/12/01 16:52:21 UTC

svn commit: r1041064 - /myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java

Author: lu4242
Date: Wed Dec  1 15:52:21 2010
New Revision: 1041064

URL: http://svn.apache.org/viewvc?rev=1041064&view=rev
Log:
MYFACES-2927 Invalid handling of null values in listboxes and menus (Thanks to Christian Kaltepoth for provide this patch)

Modified:
    myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java

Modified: myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java?rev=1041064&r1=1041063&r2=1041064&view=diff
==============================================================================
--- myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java (original)
+++ myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java Wed Dec  1 15:52:21 2010
@@ -539,6 +539,11 @@ public final class HtmlRendererUtils {
                 if (itemStrValue != null) {
                     writer.writeAttribute(HTML.VALUE_ATTR, itemStrValue, null);
                 }
+                else 
+                {
+                    writer.writeAttribute(HTML.VALUE_ATTR, "", null);
+                }
+                
 
                 if (selected) {
                     writer.writeAttribute(HTML.SELECTED_ATTR,