Author: chabotc
Date: Fri May  9 09:26:59 2008
New Revision: 654875

URL: http://svn.apache.org/viewvc?rev=654875&view=rev
Log:
When applying [EMAIL PROTECTED]'s sugested change from the orignal patch
from:
 else {
   // Straight into the Feed object - override usual resp structure
   resp = data.body;
too:
} else {
 resp.data = data.body;
}

We missed changing the gadgets.json.parse bit, which broke the feed support.
After this change it's working correctly again for the php part, and i fully
expect the java side's feed support patch requires this change too to function.


Modified:
    incubator/shindig/trunk/features/core.io/io.js

Modified: incubator/shindig/trunk/features/core.io/io.js
URL: 
http://svn.apache.org/viewvc/incubator/shindig/trunk/features/core.io/io.js?rev=654875&r1=654874&r2=654875&view=diff
==============================================================================
--- incubator/shindig/trunk/features/core.io/io.js (original)
+++ incubator/shindig/trunk/features/core.io/io.js Fri May  9 09:26:59 2008
@@ -133,21 +133,13 @@
     };
     switch (params.CONTENT_TYPE) {
       case "JSON":
-        // Same as before, but specific to JSON (not FEED)
+      case "FEED":
         resp.data = gadgets.json.parse(resp.text);
         if (!resp.data) {
           resp.errors.push("failed to parse JSON");
           resp.data = null;
         }
         break;
-      case "FEED":
-        if (!data.body) {
-          resp.errors.push("failed to parse JSON");
-          resp.data = null;
-        } else {
-          resp.data = data.body;
-        }
-        break;
       case "DOM":
         var dom;
         if (window.ActiveXObject) {


Reply via email to