Author: nbelaevski Date: 2010-04-08 14:39:46 -0400 (Thu, 08 Apr 2010) New Revision: 16750 Modified: root/ui/trunk/components/core/src/main/java/org/richfaces/renderkit/html/RepeatRenderer.java Log: RepeatRenderer: minor performance optimization Modified: root/ui/trunk/components/core/src/main/java/org/richfaces/renderkit/html/RepeatRenderer.java =================================================================== --- root/ui/trunk/components/core/src/main/java/org/richfaces/renderkit/html/RepeatRenderer.java 2010-04-08 18:22:59 UTC (rev 16749) +++ root/ui/trunk/components/core/src/main/java/org/richfaces/renderkit/html/RepeatRenderer.java 2010-04-08 18:39:46 UTC (rev 16750) @@ -21,18 +21,19 @@ */ package org.richfaces.renderkit.html; -import org.ajax4jsf.model.DataVisitResult; -import org.ajax4jsf.model.DataVisitor; -import org.richfaces.component.UIRepeat; +import java.io.IOException; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; import javax.faces.render.Renderer; -import java.io.IOException; +import org.ajax4jsf.model.DataVisitResult; +import org.ajax4jsf.model.DataVisitor; +import org.richfaces.component.UIRepeat; + /** * @author Nick Belaevski - * + * */ //@JsfRenderer(type = "org.richfaces.RepeatRenderer") public class RepeatRenderer extends Renderer { @@ -48,17 +49,17 @@ DataVisitor visitor = new DataVisitor() { public DataVisitResult process(FacesContext context, Object rowKey, Object argument) { - repeater.setRowKey(rowKey); - + repeater.setRowKey(context, rowKey); + if (repeater.isRowAvailable()) { if (repeater.getChildCount() > 0) { - for (UIComponent child : repeater.getChildren()) { - try { + try { + for (UIComponent child : repeater.getChildren()) { child.encodeAll(context); - } catch (IOException e) { - // TODO Auto-generated catch block - e.printStackTrace(); } + } catch (IOException e) { + // TODO Auto-generated catch block + e.printStackTrace(); } } } @@ -69,7 +70,7 @@ repeater.walk(context, visitor, null); } finally { - repeater.setRowKey(null); + repeater.setRowKey(context, null); } }