Fix bug when preventCallFetch and depends have conflict

Author: ilyalesik

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-fetch-hook",
-  "version": "1.3.4",
+  "version": "1.3.5",
   "description": "React fetch hook",
   "main": "./dist/index.js",
   "scripts": {

src/__tests__/useFetch.test.js

@@ -103,6 +103,29 @@ describe("useFetch", () => {
         });
     });
 
+    it("call with url, options with preventCallFetch and depends", async () => {
+        fetch.mockResponse(JSON.stringify({ data: "12345" }));
+        const options = {
+            headers: {
+                Accept: "application/json, application/xml, text/plain, text/html, *.*",
+                "Content-Type": "application/json; charset=utf-8"
+            }
+        };
+
+        const Component = () => {
+            const result = useFetch("https://google.com", { ...options, depends: [true], preventCallFetch: true });
+            return <div>{result.data && result.data.data}</div>;
+        };
+
+        const { container, rerender } = render(<Component />);
+
+        await wait(() => {
+            rerender(<Component />);
+
+            expect(fetch.mock.calls.length).toEqual(0);
+        });
+    });
+
     it("call with url, options with depends", async () => {
         fetch.mockResponse(JSON.stringify({ data: "12345" }));
         const options = {

src/useFetch.js

@@ -28,9 +28,9 @@ export function useFetch<T>(
     const fetchInstance = formatter => (path, options, specialOptions) => {
         const { depends, preventCallFetch, ...otherOptions } = options || {};
         const _depends = (specialOptions && specialOptions.depends) || depends;
-        const _preventCallFetch = _depends
-            ? _depends.reduce((accumulator, currentValue) => accumulator || !currentValue, false)
-            : preventCallFetch;
+        const _preventCallFetch =
+            (_depends && _depends.reduce((accumulator, currentValue) => accumulator || !currentValue, false)) ||
+            preventCallFetch;
         if (_preventCallFetch) {
             return Promise.resolve();
         }