Updated Branches: refs/heads/master aa30a00e9 -> 79ba3a962
WICKET-4687 ConcurrentModificationException with IFeedback instances that contain other IFeedback instances Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/79ba3a96 Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/79ba3a96 Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/79ba3a96 Branch: refs/heads/master Commit: 79ba3a96252a35286c2d4d14564891e79dccc934 Parents: aa30a00 Author: Martin Tzvetanov Grigorov <[email protected]> Authored: Thu Aug 2 15:04:28 2012 +0300 Committer: Martin Tzvetanov Grigorov <[email protected]> Committed: Thu Aug 2 15:04:28 2012 +0300 ---------------------------------------------------------------------- .../src/main/java/org/apache/wicket/Component.java | 4 +++- .../validation/FormComponentFeedbackBorder.java | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/79ba3a96/wicket-core/src/main/java/org/apache/wicket/Component.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/main/java/org/apache/wicket/Component.java b/wicket-core/src/main/java/org/apache/wicket/Component.java index 60819c5..b599813 100644 --- a/wicket-core/src/main/java/org/apache/wicket/Component.java +++ b/wicket-core/src/main/java/org/apache/wicket/Component.java @@ -2210,7 +2210,9 @@ public abstract class Component List<Component> feedbacks = getRequestCycle().getMetaData(FEEDBACK_LIST); if (feedbacks != null) { - for (Component feedback : feedbacks) + // iterate over a copy because a IFeedback may add more IFeedback children (WICKET-4687) + Component[] feedbacksCopy = feedbacks.toArray(new Component[feedbacks.size()]); + for (Component feedback : feedbacksCopy) { feedback.internalBeforeRender(); } http://git-wip-us.apache.org/repos/asf/wicket/blob/79ba3a96/wicket-core/src/main/java/org/apache/wicket/markup/html/form/validation/FormComponentFeedbackBorder.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/validation/FormComponentFeedbackBorder.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/validation/FormComponentFeedbackBorder.java index aa8ef9f..528e5da 100644 --- a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/validation/FormComponentFeedbackBorder.java +++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/validation/FormComponentFeedbackBorder.java @@ -30,7 +30,7 @@ import org.apache.wicket.markup.html.border.Border; * asterisk. * <p> * <strong>Note: </strong> Since this border checks its children do not use - * setTransparentResolver(true) and add the children directly into the border + * TransparentWebMarkupContainer and add the children directly into the border * * @author Jonathan Locke * @author Eelco Hillenius
