yes, you are correct. fixed in trunk.
-igor
On Tue, Feb 26, 2008 at 3:23 AM, Federico Fanton <[EMAIL PROTECTED]> wrote:
> On Mon, 25 Feb 2008 08:58:11 -0800
> "Igor Vaynberg" <[EMAIL PROTECTED]> wrote:
>
>
> > yeah, the components are recreated. you have to set an item reuse
> > strategy for the gridview, see setItemReuseStrategy ...
>
> I'm sorry, while trying to figure out how reuse strategies work I stumbled
> upon
> GridView$ItemsIterator.. Here's its findNext() method:
>
> private void findNext()
> {
> next = null;
>
> if (cells != null && cells.hasNext())
> {
> next = (Item)cells.next();
> }
>
> while (rows.hasNext())
> {
> MarkupContainer row = (MarkupContainer)rows.next();
> cells = ((MarkupContainer)row.iterator().next()).iterator();
> if (cells.hasNext())
> {
> next = (Item)cells.next();
> break;
> }
> }
> }
>
> Shouldn't there be an "else" or something like that, between the outmost if
> and the while? ?_? Isn't this method supposed to iterate over every cell of
> every row?
> (Wicket 1.3.1, but this class hasn't changed since rev 584925)
>
> Thanks for your attention
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
