commit c959bfe63fb958e3b3c114723971c8a78156a66e
author Jason David Davies <jasondavies@apache.org> Sun Apr 17 18:22:42 2011 +0000
committer Alexander Shorin <kxepal@gmail.com> Sun Apr 17 18:22:42 2011 +0000
tree 8e378b01f114e07d7c3bdb2ff1053f80dbdf96f7
parent 4f6e3433c22ab5340cd066d278bb1a2489957c42

Simplify JSON parsing calls.

Thanks to @brainsatwork for the suggestion.  Instead of carrying around a lump
of legacy jQuery 1.4.x $.httpData code, I've simply called $.parseJSON directly.

git-svn-id: https://svn.apache.org/repos/asf/couchdb/trunk@1094181 13f79535-47bb-0310-9956-ffa450edef68

jquery.couch.js

diff --git a/jquery.couch.js b/jquery.couch.js
index 71ff6e9..3cec9b8 100644
--- a/jquery.couch.js
+++ b/jquery.couch.js
@@ -145,7 +145,7 @@
             xhr.setRequestHeader('Accept', 'application/json');
         },
         complete: function(req) {
-          var resp = httpData(req, "json");
+          var resp = $.parseJSON(req.responseText);
           if (req.status == 200) {
             if (options.success) options.success(resp);
           } else if (options.error) {
@@ -229,7 +229,7 @@
             xhr.setRequestHeader('Accept', 'application/json');
         },
         complete: function(req) {
-          var resp = httpData(req, "json");
+          var resp = $.parseJSON(req.responseText);
           if (req.status == 200) {
             if (options.success) options.success(resp);
           } else if (options.error) {
@@ -257,7 +257,7 @@
             xhr.setRequestHeader('Accept', 'application/json');
         },
         complete: function(req) {
-          var resp = httpData(req, "json");
+          var resp = $.parseJSON(req.responseText);
           if (req.status == 200) {
             if (options.success) options.success(resp);
           } else if (options.error) {
@@ -652,7 +652,7 @@
             dataType: "json", data: toJSON(doc),
             beforeSend : beforeSend,
             complete: function(req) {
-              var resp = httpData(req, "json");
+              var resp = $.parseJSON(req.responseText);
               if (req.status == 200 || req.status == 201 || req.status == 202) {
                 doc._id = resp.id;
                 doc._rev = resp.rev;
@@ -762,7 +762,7 @@
         copyDoc: function(docId, options, ajaxOptions) {
           ajaxOptions = $.extend(ajaxOptions, {
             complete: function(req) {
-              var resp = httpData(req, "json");
+              var resp = $.parseJSON(req.responseText);
               if (req.status == 201) {
                 if (options.success) options.success(resp);
               } else if (options.error) {
@@ -1001,28 +1001,6 @@
     }
   });
 
-  var httpData = $.httpData || function( xhr, type, s ) { // lifted from jq1.4.4
-    var ct = xhr.getResponseHeader("content-type") || "",
-      xml = type === "xml" || !type && ct.indexOf("xml") >= 0,
-      data = xml ? xhr.responseXML : xhr.responseText;
-
-    if ( xml && data.documentElement.nodeName === "parsererror" ) {
-      $.error( "parsererror" );
-    }
-    if ( s && s.dataFilter ) {
-      data = s.dataFilter( data, type );
-    }
-    if ( typeof data === "string" ) {
-      if ( type === "json" || !type && ct.indexOf("json") >= 0 ) {
-        data = $.parseJSON( data );
-      } else if ( type === "script" ||
-                  !type && ct.indexOf("javascript") >= 0 ) {
-        $.globalEval( data );
-      }
-    }
-    return data;
-  };
-
   /**
    * @private
    */
@@ -1041,7 +1019,7 @@
       },
       complete: function(req) {
         try {
-          var resp = httpData(req, "json");
+          var resp = $.parseJSON(req.responseText);
         } catch(e) {
           if (options.error) {
             options.error(req.status, req, e);