cleanup

Author: LachlanNewman

src/api.js

@@ -222,67 +222,67 @@ class DICOMwebClient {
   _httpRequest(url, method, headers = {}, request = {}) {
     const { errorInterceptor, requestHooks } = this;
     return new Promise((resolve, reject) => {
-      let instance = request.instance ? request.instance : new XMLHttpRequest();
+      let requestInstance = request.instance ? request.instance : new XMLHttpRequest();
 
-      instance.open(method, url, true);
+      requestInstance.open(method, url, true);
       if ('responseType' in request) {
-        instance.responseType = request.responseType;
+        requestInstance.responseType = request.responseType;
       }
 
       if (typeof headers === 'object') {
         Object.keys(headers).forEach(key => {
-          instance.setRequestHeader(key, headers[key]);
+          requestInstance.setRequestHeader(key, headers[key]);
         });
       }
 
       // now add custom headers from the user
       // (e.g. access tokens)
       const userHeaders = this.headers;
       Object.keys(userHeaders).forEach(key => {
-        instance.setRequestHeader(key, userHeaders[key]);
+        requestInstance.setRequestHeader(key, userHeaders[key]);
       });
 
       // Event triggered when upload starts
-      instance.onloadstart = function onloadstart() {
+      requestInstance.onloadstart = function onloadstart() {
         debugLog('upload started: ', url)
       };
 
       // Event triggered when upload ends
-      instance.onloadend = function onloadend() {
+      requestInstance.onloadend = function onloadend() {
         debugLog('upload finished')
       };
 
       // Handle response message
-      instance.onreadystatechange = () => {
-        if (instance.readyState === 4) {
-          if (instance.status === 200) {
-            const contentType = instance.getResponseHeader('Content-Type');
+      requestInstance.onreadystatechange = () => {
+        if (requestInstance.readyState === 4) {
+          if (requestInstance.status === 200) {
+            const contentType = requestInstance.getResponseHeader('Content-Type');
             // Automatically distinguishes between multipart and singlepart in an array buffer, and
             // converts them into a consistent type.
             if (contentType && contentType.indexOf('multipart') !== -1) {
-              resolve(multipartDecode(instance.response));
-            } else if (instance.responseType === 'arraybuffer') {
-              resolve([instance.response]);
+              resolve(multipartDecode(requestInstance.response));
+            } else if (requestInstance.responseType === 'arraybuffer') {
+              resolve([requestInstance.response]);
             } else {
-              resolve(instance.response);
+              resolve(requestInstance.response);
             }
-          } else if (instance.status === 202) {
+          } else if (requestInstance.status === 202) {
             if (this.verbose) {
-              console.warn('some resources already existed: ', instance);
+              console.warn('some resources already existed: ', requestInstance);
             }
-            resolve(instance.response);
-          } else if (instance.status === 204) {
+            resolve(requestInstance.response);
+          } else if (requestInstance.status === 204) {
             if (this.verbose) {
-              console.warn('empty response for request: ', instance);
+              console.warn('empty response for request: ', requestInstance);
             }
             resolve([]);
           } else {
             const error = new Error('request failed');
-            error.request = instance;
-            error.response = instance.response;
-            error.status = instance.status;
+            error.request = requestInstance;
+            error.response = requestInstance.response;
+            error.status = requestInstance.status;
             if (this.verbose) {
-              console.error('request failed: ', instance);
+              console.error('request failed: ', requestInstance);
               console.error(error);
               console.error(error.response);
             }
@@ -297,7 +297,7 @@ class DICOMwebClient {
       // Event triggered while download progresses
       if ('progressCallback' in request) {
         if (typeof request.progressCallback === 'function') {
-          instance.onprogress = request.progressCallback;
+          requestInstance.onprogress = request.progressCallback;
         }
       }
 
@@ -307,20 +307,20 @@ class DICOMwebClient {
         const pipeRequestHooks = functions => args =>
           functions.reduce((props, fn) => fn(props, metadata), args);
         const pipedRequest = pipeRequestHooks(requestHooks);
-        instance = pipedRequest(instance);
+        requestInstance = pipedRequest(requestInstance);
       }
 
       // Add withCredentials to request if needed
       if ('withCredentials' in request) {
         if (request.withCredentials) {
-          instance.withCredentials = true;
+          requestInstance.withCredentials = true;
         }
       }
 
       if ('data' in request) {
-        instance.send(request.data);
+        requestInstance.send(request.data);
       } else {
-        instance.send();
+        requestInstance.send();
       }
     });
   }