You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2014/05/27 15:08:04 UTC
[3/4] git commit: Simplify the code to find the RadioGroup's model
value
Simplify the code to find the RadioGroup's model value
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/80e3c6a2
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/80e3c6a2
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/80e3c6a2
Branch: refs/heads/wicket-6.x
Commit: 80e3c6a209f79de9074894325c07ecd96f6784fd
Parents: f11a75c
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Tue May 27 15:06:53 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Tue May 27 15:06:53 2014 +0200
----------------------------------------------------------------------
.../org/apache/wicket/markup/html/form/RadioGroup.java | 13 ++++---------
1 file changed, 4 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/80e3c6a2/wicket-core/src/main/java/org/apache/wicket/markup/html/form/RadioGroup.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/RadioGroup.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/RadioGroup.java
index b577615..29d0869 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/RadioGroup.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/RadioGroup.java
@@ -92,24 +92,19 @@ public class RadioGroup<T> extends FormComponent<T> implements IOnChangeListener
@Override
protected String getModelValue()
{
- final StringBuilder builder = new StringBuilder();
-
- final T t = getModelObject();
-
- visitChildren(Radio.class, new IVisitor<Radio<T>, Void>()
+ String radioValue = visitChildren(Radio.class, new IVisitor<Radio<T>, String>()
{
@Override
- public void component(Radio<T> radio, IVisit<Void> visit)
+ public void component(Radio<T> radio, IVisit<String> visit)
{
if (getModelComparator().compare(RadioGroup.this, radio.getDefaultModelObject()))
{
- builder.append(radio.getValue());
- visit.stop();
+ visit.stop(radio.getValue());
}
}
});
- return builder.toString();
+ return radioValue;
}
/**