From 7df4af2deb8e491f8bc302c5dab3c5abd40ec494 Mon Sep 17 00:00:00 2001
From: Lukasz Lenart <lukaszlenart@apache.org>
Date: Mon, 20 May 2013 12:59:25 +0000
Subject: [PATCH] WW-3676 Uses simple toString() for non-null objects to
 support other types than String and Integer

git-svn-id: https://svn.apache.org/repos/asf/struts/struts2/trunk@1484461 13f79535-47bb-0310-9956-ffa450edef68
---
 .../views/java/simple/RadioHandler.java       | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/RadioHandler.java b/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/RadioHandler.java
index 8552f8bbb..9ee30107e 100644
--- a/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/RadioHandler.java
+++ b/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/RadioHandler.java
@@ -33,8 +33,8 @@ import java.util.Iterator;
 import java.util.Map;
 
 public class RadioHandler extends AbstractTagHandler implements TagGenerator {
-    public void generate() throws IOException {
 
+    public void generate() throws IOException {
         Map<String, Object> params = context.getParameters();
 
         Object listObj = params.get("list");
@@ -59,19 +59,12 @@ public class RadioHandler extends AbstractTagHandler implements TagGenerator {
                 Object itemValue = findValue(listValue != null ? listValue : "top");
                 String itemValueStr = StringUtils.defaultString(itemValue == null ? null : itemValue.toString());
 
-                //Namevalue needs to cast to a string from object. It's object because the 
-                //Property can be defined as String or as Integer
-                String itemNameValueStr;
-                if (nameValue instanceof java.lang.Integer || nameValue instanceof java.lang.String)
-                    itemNameValueStr = nameValue.toString();
-                else
-                    itemNameValueStr = null;
+                // nameValue needs to cast to a string from object
+                String itemNameValueStr = (nameValue == null ? null : nameValue.toString());
 
                 //Checked value.  It's set to true if the nameValue (the value associated with the name which is typically set in 
                 //the action is equal to the current key value.
-                Boolean checked = itemKeyStr != null &&
-                        itemNameValueStr != null &&
-                        itemNameValueStr.equals(itemKeyStr);
+                Boolean checked = itemKeyStr != null && itemNameValueStr != null && itemNameValueStr.equals(itemKeyStr);
 
                 //Radio button section
                 String id = params.get("id") + Integer.toString(cnt++);
@@ -93,11 +86,13 @@ public class RadioHandler extends AbstractTagHandler implements TagGenerator {
                         .addIfExists("style", params.get("cssStyle"))
                         .addIfExists("title", params.get("title"));
                 super.start("label", a);
-                if (StringUtils.isNotEmpty(itemValueStr))
+                if (StringUtils.isNotEmpty(itemValueStr)) {
                     characters(itemValueStr);
+                }
                 super.end("label");
                 stack.pop();
             }
         }
     }
+
 }