Fix for Jira [DS-96] XMLUI Wing-Framework 'Radio' class doesn't support adding radio buttons which are unselected by default.

git-svn-id: http://scm.dspace.org/svn/repo/branches/dspace-1_5_x@3604 9c30dcfa-912a-0410-8fc2-9e0234be79fd

dspace-xmlui/dspace-xmlui-wing/src/main/java/org/dspace/app/xmlui/wing/element/Radio.java

@@ -132,8 +132,11 @@ public class Radio extends Field
      */
     public Option addOption(boolean selected, String returnValue)
             throws WingException
+    {
+        if (selected)
         {
             setOptionSelected(returnValue);
+        }
         return addOption(returnValue);
     }
     
@@ -164,8 +167,11 @@ public class Radio extends Field
      *            (Required) The text to set as the visible option.
      */
     public void addOption(boolean selected,String returnValue, String characters) throws WingException
+    {
+        if (selected)
         {
             setOptionSelected(returnValue);
+        }
         addOption(returnValue,characters);
     }
     
@@ -196,8 +202,11 @@ public class Radio extends Field
      *            (Required) The text to set as the visible option.
      */
     public void addOption(boolean selected, int returnValue, String characters) throws WingException
+    {
+        if (selected)
         {
             setOptionSelected(returnValue);
+        }
         addOption(returnValue,characters);
     }
     
@@ -228,8 +237,11 @@ public class Radio extends Field
      *            (Required) The transalted text to set as the visible option.
      */
     public void addOption(boolean selected, String returnValue, Message message) throws WingException
+    {
+        if (selected)
         {
             setOptionSelected(returnValue);
+        }
         addOption(returnValue,message);
     }
     
@@ -260,8 +272,11 @@ public class Radio extends Field
      *            (Required) The transalted text to set as the visible option.
      */
     public void addOption(boolean selected, int returnValue, Message message) throws WingException
+    {
+        if (selected)
         {
             setOptionSelected(returnValue);
+        }
         addOption(returnValue,message);
     }
     

dspace/CHANGES

@@ -70,6 +70,7 @@
  - Fix for Jira bug [DS-86] XMLUI Feedback form does not include any protection from spamming
  - Fix for Jira [DS-87] XMLUI file download links break in Google search results if file 'sequence' number changes.
  - Fix for Jira [DS-95] XMLUI: Visiting an item page with a trailing slash returns a 404 message
+ - Fix for Jira [DS-96] XMLUI Wing-Framework 'Radio' class doesn't support adding radio buttons which are unselected by default.
 
 (Kim Shepherd)
  - [2385187] Fix for toDate method in DCDate