WICKET-4329 Optimize consequent usages of MarkupContainer# hasAssociatedMarkup() and #getAssociatedMapkup()
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/c54959df Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/c54959df Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/c54959df Branch: refs/heads/wicket-1.5.x Commit: c54959dfcee3d1432a7180e03d2ca8043cc0ef95 Parents: 004a136 Author: martin-g <[email protected]> Authored: Mon Jan 9 11:17:32 2012 +0200 Committer: martin-g <[email protected]> Committed: Mon Jan 9 11:19:16 2012 +0200 ---------------------------------------------------------------------- .../src/main/java/org/apache/wicket/Component.java | 6 ++++-- .../html/panel/FragmentMarkupSourcingStrategy.java | 6 ++++-- 2 files changed, 8 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/c54959df/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 bad6495..5501f14 100644 --- a/wicket-core/src/main/java/org/apache/wicket/Component.java +++ b/wicket-core/src/main/java/org/apache/wicket/Component.java @@ -37,6 +37,7 @@ import org.apache.wicket.feedback.FeedbackMessage; import org.apache.wicket.feedback.IFeedback; import org.apache.wicket.markup.ComponentTag; import org.apache.wicket.markup.IMarkupFragment; +import org.apache.wicket.markup.Markup; import org.apache.wicket.markup.MarkupCache; import org.apache.wicket.markup.MarkupElement; import org.apache.wicket.markup.MarkupException; @@ -718,9 +719,10 @@ public abstract class Component if (this instanceof MarkupContainer) { MarkupContainer container = (MarkupContainer)this; - if (container.hasAssociatedMarkup()) + Markup associatedMarkup = container.getAssociatedMarkup(); + if (associatedMarkup != null) { - markup = container.getAssociatedMarkup(); + markup = associatedMarkup; return markup; } } http://git-wip-us.apache.org/repos/asf/wicket/blob/c54959df/wicket-core/src/main/java/org/apache/wicket/markup/html/panel/FragmentMarkupSourcingStrategy.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/panel/FragmentMarkupSourcingStrategy.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/panel/FragmentMarkupSourcingStrategy.java index 07f4075..8a6cd94 100644 --- a/wicket-core/src/main/java/org/apache/wicket/markup/html/panel/FragmentMarkupSourcingStrategy.java +++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/panel/FragmentMarkupSourcingStrategy.java @@ -20,6 +20,7 @@ import org.apache.wicket.Component; import org.apache.wicket.MarkupContainer; import org.apache.wicket.markup.ComponentTag; import org.apache.wicket.markup.IMarkupFragment; +import org.apache.wicket.markup.Markup; import org.apache.wicket.markup.MarkupException; import org.apache.wicket.markup.MarkupNotFoundException; import org.apache.wicket.markup.MarkupStream; @@ -126,9 +127,10 @@ public class FragmentMarkupSourcingStrategy extends AbstractMarkupSourcingStrate { // There is one more option if the markup provider has associated markup MarkupContainer markupProvider = getMarkupProvider(container); - if (markupProvider.hasAssociatedMarkup()) + Markup associatedMarkup = markupProvider.getAssociatedMarkup(); + if (associatedMarkup != null) { - markup = markupProvider.getAssociatedMarkup(); + markup = associatedMarkup; if (markup != null) { childMarkup = markup.find(markupId);
