@selkirk-systems/fetch 0.1.3 → 0.1.5

package/dist/index.js

@@ -9,74 +9,72 @@ require('isomorphic-fetch');
 var _networkTimer = null;
 
 function _checkFetchStatus(response) {
-    if (response.status >= 200 && response.status < 300) {
-        return response.json();
-    } else {
-        var error = new Error(response.statusText);
-        return response.json().then(function (result) {
-            error.response = result;
-            throw error;
-        });
-    }
-}
-
-function _parseJSON(response) {
-    return response.text().then(function (text) {
-        return text ? JSON.parse(text) : {};
-    });
+   if (response.status >= 200 && response.status < 300) {
+      return response.text().then(function (text) {
+         return text ? JSON.parse(text) : {};
+      });
+   } else {
+      var error = new Error(response.statusText);
+      return response.text().then(function (text) {
+         if (text) {
+            error.response = JSON.parse(text);
+         }
+         throw error;
+      });
+   }
 }
 
 function _invokeFetch(url, props) {
-    if (!url) {
-        throw new Error("fetch:Missing Parameter - URL");
-    }
-
-    var options = {
-        method: "GET",
-        credentials: 'same-origin',
-        headers: {
-            'Accept': 'application/json',
-            'Content-Type': 'application/json'
-        }
-    };
-
-    options = Assign(options, props);
-
-    if (_networkTimer) {
-        return new Promise(function (resolve, reject) {
-            setTimeout(resolve, _networkTimer);
-        });
-    }
-
-    if (options.headers['Content-Type'] === "application/json" && options.body) options.body = JSON.stringify(options.body);
-
-    return fetch(url, options).then(_checkFetchStatus);
+   if (!url) {
+      throw new Error("fetch:Missing Parameter - URL");
+   }
+
+   var options = {
+      method: "GET",
+      credentials: 'same-origin',
+      headers: {
+         'Accept': 'application/json',
+         'Content-Type': 'application/json'
+      }
+   };
+
+   options = Assign(options, props);
+
+   if (_networkTimer) {
+      return new Promise(function (resolve, reject) {
+         setTimeout(resolve, _networkTimer);
+      });
+   }
+
+   if (options.headers['Content-Type'] === "application/json" && options.body) options.body = JSON.stringify(options.body);
+
+   return fetch(url, options).then(_checkFetchStatus);
 }
 module.exports = function (dispatcher, params) {
-    if (!dispatcher) return _invokeFetch;
-
-    var options = Assign({
-        lastErrorEvent: "LAST_ERROR_EVENT",
-        errorID: null
-    }, params);
-
-    return function (url, props, responseEvent) {
-        return _invokeFetch(url, props).then(function (response) {
-
-            if (!responseEvent) return null;
-
-            dispatcher.handleViewAction({
-                actionType: responseEvent,
-                item: response
-            });
-
-            return response;
-        }).catch(function (err) {
-            console.error(new Error("Fetch - " + options.errorID || ""), err);
-            dispatcher.handleViewAction({
-                actionType: options.lastErrorEvent,
-                item: { error: err, id: options.errorID }
-            });
-        });
-    };
+   if (!dispatcher) return _invokeFetch;
+
+   var options = Assign({
+      lastErrorEvent: "LAST_ERROR_EVENT",
+      errorID: null
+   }, params);
+
+   return function (url, props, responseEvent) {
+      return _invokeFetch(url, props).then(function (response) {
+
+         if (!responseEvent) return null;
+
+         dispatcher.handleViewAction({
+            actionType: responseEvent,
+            item: response
+         });
+
+         return response;
+      }).catch(function (err) {
+         console.error(new Error("Fetch - " + options.errorID || ""), err);
+         dispatcher.handleViewAction({
+            actionType: options.lastErrorEvent,
+            item: { error: err, id: options.errorID }
+         });
+      });
+   };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@selkirk-systems/fetch",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "A wrapper on fetch-polyfill that will work for server and client environments",
   "selkirk": {
     "bambooDeploy": "npm",

package/src/index.js

@@ -8,22 +8,20 @@ const _networkTimer = null;
 
 function _checkFetchStatus(response) {
    if (response.status >= 200 && response.status < 300) {
-      return response.json()
+      return response.text().then(function(text) {
+  	  		return text ? JSON.parse(text) : {}
+      });
    } else {
       var error = new Error(response.statusText);
-       return response.json().then((result)=>{
-           error.response = result;
+       return response.text().then(function(text) {
+    	   if (text) {
+	           error.response = JSON.parse(text);
+    	   }
            throw error;
        });
    }
 }
 
-function _parseJSON(response) {
-	  return response.text().then(function(text) {
-	    return text ? JSON.parse(text) : {}
-	  });
-}
-
 function _invokeFetch(url,props) {
    if(!url) {
       throw new Error("fetch:Missing Parameter - URL");