You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2010/01/28 21:48:24 UTC

svn commit: r904262 - /pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDRadioCollection.java

Author: lehmi
Date: Thu Jan 28 20:48:24 2010
New Revision: 904262

URL: http://svn.apache.org/viewvc?rev=904262&view=rev
Log:
PDFBOX-50: using the right type for the value of a PDRadioCollection. Thanks to Johannes Koch (johannes dot koch at fit dot fraunhofer dot de) for his contribution

Modified:
    pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDRadioCollection.java

Modified: pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDRadioCollection.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDRadioCollection.java?rev=904262&r1=904261&r2=904262&view=diff
==============================================================================
--- pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDRadioCollection.java (original)
+++ pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDRadioCollection.java Thu Jan 28 20:48:24 2010
@@ -92,14 +92,18 @@
         List kids = getKids();
         for (int i = 0; i < kids.size(); i++)
         {
-            PDCheckbox btn = (PDCheckbox)kids.get(i);
-            if( btn.getOnValue().equals(value) )
+            PDField field = (PDField)kids.get(i);
+            if ( field instanceof PDCheckbox )
             {
-                btn.check();
-            }
-            else
-            {
-                btn.unCheck();
+                PDCheckbox btn = (PDCheckbox)field;
+                if( btn.getOnValue().equals(value) )
+                {
+                    btn.check();
+                }
+                else
+                {
+                    btn.unCheck();
+                }
             }
         }
     }
@@ -117,10 +121,14 @@
         List kids = getKids();
         for (int i = 0; i < kids.size(); i++)
         {
-            PDCheckbox btn = (PDCheckbox)kids.get(i);
-            if( btn.isChecked() )
+            PDField kid = (PDField)kids.get(i);
+            if ( kid instanceof PDCheckbox )
             {
-                retval = btn.getOnValue();
+                PDCheckbox btn = (PDCheckbox)kid;
+                if( btn.isChecked() )
+                {
+                    retval = btn.getOnValue();
+                }
             }
         }
         if( retval == null )
@@ -138,6 +146,7 @@
      * @return A list of PDWidget objects.
      * @throws IOException if there is an error while creating the children objects.
      */
+    @SuppressWarnings("unchecked")
     public List getKids() throws IOException
     {
         List retval = null;