svn commit: r705607 - /incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/util/BeanJsonConverter.java

[ie...@apache.org]

Author: ieb
Date: Fri Oct 17 06:26:15 2008
New Revision: 705607

URL: http://svn.apache.org/viewvc?rev=705607&view=rev
Log:
SHINDIG-656

Patch from Rodrigo Gallardo
 "

Included patch implements requesting a json string to be decoded as a list. It assumes the string represents a json array, I simply could not find any reasonable way to handle a different case.
"

Thanks

Modified:
    incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/util/BeanJsonConverter.java

Modified: incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/util/BeanJsonConverter.java
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/util/BeanJsonConverter.java?rev=705607&r1=705606&r2=705607&view=diff
==============================================================================
--- incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/util/BeanJsonConverter.java (original)
+++ incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/util/BeanJsonConverter.java Fri Oct 17 06:26:15 2008
@@ -243,10 +243,10 @@
       }
 
     } else if (pojo instanceof List) {
-      // TODO: process as a JSONArray
-      throw new UnsupportedOperationException("We don't support lists as a "
-          + "base json type yet. You can put it inside a pojo for now.");
-
+      JSONArray array = new JSONArray(json);
+      for (int i = 0; i < array.length(); i++) {
+        ((List<Object>) pojo).add(array.get(i));
+      }
     } else {
       JSONObject jsonObject = new JSONObject(json);
       List<MethodPair> methods;