You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by hs...@apache.org on 2012/07/11 01:12:26 UTC

svn commit: r1359951 - /shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java

Author: hsaputra
Date: Tue Jul 10 23:12:25 2012
New Revision: 1359951

URL: http://svn.apache.org/viewvc?rev=1359951&view=rev
Log:
Always add Collection metadata for JSON-RPC servlet to match up behavior with the REST counterpart and the updated OpenSocial 2.5.0 spec proposal: http://code.google.com/p/opensocial-resources/issues/detail?id=1323.

Modified:
    shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java

Modified: shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java?rev=1359951&r1=1359950&r2=1359951&view=diff
==============================================================================
--- shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java (original)
+++ shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java Tue Jul 10 23:12:25 2012
@@ -282,14 +282,10 @@ public class JsonRpcServlet extends ApiS
         // always put in totalResults
         map.put("totalResults", collection.getTotalResults());
 
-        if (!collection.isFiltered())
-          map.put("filtered", collection.isFiltered());
-
-        if (!collection.isUpdatedSince())
-          map.put("updatedSince", collection.isUpdatedSince());
-
-        if (!collection.isSorted())
-          map.put("sorted", collection.isSorted());
+        // always add metadata for collections
+        map.put("filtered", collection.isFiltered());
+        map.put("updatedSince", collection.isUpdatedSince());
+        map.put("sorted", collection.isSorted());
 
         map.put("list", collection.getList());
         addResult(result, map);