You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jd...@apache.org on 2006/11/11 12:28:16 UTC

svn commit: r473692 - /incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/list/Loop.java

Author: jdonnerstag
Date: Sat Nov 11 03:28:13 2006
New Revision: 473692

URL: http://svn.apache.org/viewvc?view=rev&rev=473692
Log:
ListView/Item and Loop/LoopItem are now based on MarkupFragments as a small step forward towards
MarkupFragment based rendering

Modified:
    incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/list/Loop.java

Modified: incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/list/Loop.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/list/Loop.java?view=diff&rev=473692&r1=473691&r2=473692
==============================================================================
--- incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/list/Loop.java (original)
+++ incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/list/Loop.java Sat Nov 11 03:28:13 2006
@@ -157,9 +157,6 @@
 	@Override
 	protected final void onRender(final MarkupStream markupStream)
 	{
-		// Save position in markup stream
-		final int markupStart = markupStream.getCurrentIndex();
-
 		// Get number of iterations
 		final int iterations = getIterations();
 		if (iterations > 0)
@@ -177,17 +174,11 @@
 							"Loop item is null.  Probably the number of loop iterations were changed between onBeginRequest and render time.");
 				}
 
-				// Rewind to start of markup for kids
-				markupStream.setCurrentIndex(markupStart);
-
 				// Render iteration
 				renderItem(item);
 			}
 		}
-		else
-		{
-			markupStream.skipComponent();
-		}
+		markupStream.skipComponent();
 	}
 
 	/**