You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2006/01/22 01:21:59 UTC

svn commit: r371147 - /myfaces/commons/trunk/src/main/java/org/apache/myfaces/renderkit/html/HtmlRadioRendererBase.java

Author: mmarinschek
Date: Sat Jan 21 16:21:52 2006
New Revision: 371147

URL: http://svn.apache.org/viewcvs?rev=371147&view=rev
Log:
fix for MYFACES-1046

Modified:
    myfaces/commons/trunk/src/main/java/org/apache/myfaces/renderkit/html/HtmlRadioRendererBase.java

Modified: myfaces/commons/trunk/src/main/java/org/apache/myfaces/renderkit/html/HtmlRadioRendererBase.java
URL: http://svn.apache.org/viewcvs/myfaces/commons/trunk/src/main/java/org/apache/myfaces/renderkit/html/HtmlRadioRendererBase.java?rev=371147&r1=371146&r2=371147&view=diff
==============================================================================
--- myfaces/commons/trunk/src/main/java/org/apache/myfaces/renderkit/html/HtmlRadioRendererBase.java (original)
+++ myfaces/commons/trunk/src/main/java/org/apache/myfaces/renderkit/html/HtmlRadioRendererBase.java Sat Jan 21 16:21:52 2006
@@ -98,8 +98,6 @@
             renderGroupOrItemRadio(facesContext, selectOne,
             		selectItem, currentValue,
             		converter, pageDirectionLayout);
-
-            if (pageDirectionLayout) writer.endElement(HTML.TR_ELEM);
         }
 
         if (!pageDirectionLayout) writer.endElement(HTML.TR_ELEM);
@@ -189,7 +187,7 @@
 
         boolean itemChecked = itemStrValue.equals(currentValue);
 
-        String labelClass = null;
+        String labelClass;
         boolean componentDisabled = isDisabled(facesContext, selectOne);
 
         if (componentDisabled || itemDisabled) {