[WICKET-2368] Page.checkRendering fails after setting BorderBodyContainer visiblity to false

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.4-RC5
• Fix Version/s: 1.4.0
• Component/s: wicket
• Environment: <groupId>org.apache.wicket</groupId>; <artifactId>wicket</artifactId>; <version>1.4-rc5</version>
• Assignee: Juegen Donnerstag
• Reporter: Michael Frankerl
• Created: 14/Jul/09 13:14
• Updated: 14/Jul/09 15:16
• Resolved: 14/Jul/09 15:16

Description

After toggling visibility of the BorderBodyContainer to false the Page.checkRendering method fails in line 1157, claiming an iterator IllegalStateException. This happens because iterator.remove() is called twice for a child component in the border component, if the body is not visible.

My Code:

public class TogglePanel extends Border {
private boolean expanded = true;

public TogglePanel(String id, IModel<String> titleModel) {
super(id, titleModel);

Link link = new Link("title") {

@Override
public void onClick()

{ expanded = !expanded; getBodyContainer().setVisible(expanded); }

};
link.add(new Label("titleLabel", titleModel));

add(link);
}

}

Markup:

<wicket:border>
<h3 class="collapse" wicket:id="title">
<span class="label" wicket:id="titleLabel">Panel Title</span>
<a class="foldicon"> </a>
</h3>
<wicket:body />
</wicket:border>