Merge pull request #135 from jamesplease/bug-fixies

Add ESLint rule to catch unused variable use

Author: jamesplease

.eslintrc

@@ -10,6 +10,7 @@
   },
   "rules": {
     "no-unused-vars": "error",
+    "no-use-before-define": "error",
     "react/jsx-uses-react": "error",
     "react/jsx-uses-vars": "error"
   },

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+### v2.0.2 (2018/2/21)
+
+**Bug Fixes**
+
+* Fixes a bug where an Uncaught ReferenceError could be thrown
+
 ### v2.0.1 (2018/2/17)
 
 **Bug Fixes**

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-request",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "Declarative HTTP requests with React.",
   "main": "lib/index.js",
   "module": "es/index.js",

src/fetch.js

@@ -179,6 +179,21 @@ export class Fetch extends React.Component {
     const uppercaseMethod = method.toUpperCase();
     const shouldCacheResponse = this.shouldCacheResponse();
 
+    const init = {
+      body,
+      credentials,
+      headers,
+      method: uppercaseMethod,
+      mode,
+      cache,
+      redirect,
+      referrer,
+      referrerPolicy,
+      integrity,
+      keepalive,
+      signal
+    };
+
     const responseReceivedInfo = {
       url,
       init,
@@ -221,20 +236,6 @@ export class Fetch extends React.Component {
         return Promise.resolve(cacheError);
       }
     }
-    const init = {
-      body,
-      credentials,
-      headers,
-      method: uppercaseMethod,
-      mode,
-      cache,
-      redirect,
-      referrer,
-      referrerPolicy,
-      integrity,
-      keepalive,
-      signal
-    };
 
     this.setState({
       fetching: true,