@dhis2/app-service-data 3.4.1 → 3.4.4

package/build/cjs/links/RestAPILink/queryToRequestOptions/multipartFormDataMatchers.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.isSvgConversion = exports.isAppInstall = exports.isStaticContentUpload = exports.isMessageConversationAttachment = exports.isFileResourceUpload = exports.isDataValue = void 0;
+exports.isAppInstall = exports.isStaticContentUpload = exports.isMessageConversationAttachment = exports.isFileResourceUpload = exports.isDataValue = void 0;
 
 /*
  * Requests that expect a "multipart/form-data" Content-Type have been collected by scanning
@@ -45,13 +45,6 @@ exports.isStaticContentUpload = isStaticContentUpload;
 
 const isAppInstall = (type, {
   resource
-}) => type === 'create' && resource === 'apps'; // POST to convert an SVG file
+}) => type === 'create' && resource === 'apps';
 
-
-exports.isAppInstall = isAppInstall;
-
-const isSvgConversion = (type, {
-  resource
-}) => type === 'create' && (resource === 'svg.png' || resource === 'svg.pdf');
-
-exports.isSvgConversion = isSvgConversion;
+exports.isAppInstall = isAppInstall;

package/build/cjs/links/RestAPILink/queryToRequestOptions/multipartFormDataMatchers.test.js

@@ -71,21 +71,4 @@ describe('isAppInstall', () => {
       resource: 'notApps'
     })).toEqual(false);
   });
-});
-describe('isSvgConversion', () => {
-  it('returns true for a POST to "svg.png"', () => {
-    expect((0, _multipartFormDataMatchers.isSvgConversion)('create', {
-      resource: 'svg.png'
-    })).toEqual(true);
-  });
-  it('returns true for a POST to "svg.pdf"', () => {
-    expect((0, _multipartFormDataMatchers.isSvgConversion)('create', {
-      resource: 'svg.pdf'
-    })).toEqual(true);
-  });
-  it('retuns false for a POST to a different resource', () => {
-    expect((0, _multipartFormDataMatchers.isSvgConversion)('create', {
-      resource: 'notSvg'
-    })).toEqual(false);
-  });
 });

package/build/cjs/links/RestAPILink/queryToRequestOptions/requestContentType.js

@@ -3,12 +3,14 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.requestBodyForContentType = exports.requestHeadersForContentType = exports.requestContentType = exports.FORM_DATA_ERROR_MSG = void 0;
+exports.requestBodyForContentType = exports.requestHeadersForContentType = exports.requestContentType = exports.getConversionErrorMessage = void 0;
 
 var multipartFormDataMatchers = _interopRequireWildcard(require("./multipartFormDataMatchers"));
 
 var textPlainMatchers = _interopRequireWildcard(require("./textPlainMatchers"));
 
+var xWwwFormUrlencodedMatchers = _interopRequireWildcard(require("./xWwwFormUrlencodedMatchers"));
+
 function _getRequireWildcardCache() { if (typeof WeakMap !== "function") return null; var cache = new WeakMap(); _getRequireWildcardCache = function () { return cache; }; return cache; }
 
 function _interopRequireWildcard(obj) { if (obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
@@ -17,20 +19,23 @@ const resourceExpectsTextPlain = (type, query) => Object.values(textPlainMatcher
 
 const resourceExpectsMultipartFormData = (type, query) => Object.values(multipartFormDataMatchers).some(multipartFormDataMatcher => multipartFormDataMatcher(type, query));
 
-const FORM_DATA_ERROR_MSG = 'Could not convert data to FormData: object does not have own enumerable string-keyed properties';
-exports.FORM_DATA_ERROR_MSG = FORM_DATA_ERROR_MSG;
+const resourceExpectsXWwwFormUrlencoded = (type, query) => Object.values(xWwwFormUrlencodedMatchers).some(xWwwFormUrlencodedMatcher => xWwwFormUrlencodedMatcher(type, query));
+
+const getConversionErrorMessage = outputType => "Could not convert data to ".concat(outputType, ": object does not have own enumerable string-keyed properties");
 
-const convertToFormData = data => {
+exports.getConversionErrorMessage = getConversionErrorMessage;
+
+const convertData = (data, initialValue) => {
   const dataEntries = Object.entries(data);
 
   if (dataEntries.length === 0) {
-    throw new Error(FORM_DATA_ERROR_MSG);
+    throw new Error(getConversionErrorMessage(initialValue.constructor.name));
   }
 
-  return dataEntries.reduce((formData, [key, value]) => {
-    formData.append(key, value);
-    return formData;
-  }, new FormData());
+  return dataEntries.reduce((convertedData, [key, value]) => {
+    convertedData.append(key, value);
+    return convertedData;
+  }, initialValue);
 };
 
 const requestContentType = (type, query) => {
@@ -50,6 +55,10 @@ const requestContentType = (type, query) => {
     return 'multipart/form-data';
   }
 
+  if (resourceExpectsXWwwFormUrlencoded(type, query)) {
+    return 'application/x-www-form-urlencoded';
+  }
+
   return 'application/json';
 };
 
@@ -86,7 +95,11 @@ const requestBodyForContentType = (contentType, {
   }
 
   if (contentType === 'multipart/form-data') {
-    return convertToFormData(data);
+    return convertData(data, new FormData());
+  }
+
+  if (contentType === 'application/x-www-form-urlencoded') {
+    return convertData(data, new URLSearchParams());
   } // 'text/plain'
 
 

package/build/cjs/links/RestAPILink/queryToRequestOptions/requestContentType.test.js

@@ -88,7 +88,28 @@ describe('requestBodyForContentType', () => {
           type: 'text/plain'
         })
       });
-    }).toThrow(new Error(_requestContentType.FORM_DATA_ERROR_MSG));
+    }).toThrow(new Error('Could not convert data to FormData: object does not have own enumerable string-keyed properties'));
+  });
+  it('converts to URLSearchParams if contentType is "application/x-www-form-urlencoded"', () => {
+    const data = {
+      a: 'AAA'
+    };
+    const result = (0, _requestContentType.requestBodyForContentType)('application/x-www-form-urlencoded', {
+      resource: 'test',
+      data
+    });
+    expect(result instanceof URLSearchParams).toEqual(true);
+    expect(result.get('a')).toEqual('AAA');
+  });
+  it('throws an error if contentType is "application/x-www-form-urlencoded" and data does have own string-keyd properties', () => {
+    expect(() => {
+      (0, _requestContentType.requestBodyForContentType)('application/x-www-form-urlencoded', {
+        resource: 'test',
+        data: new File(['foo'], 'foo.txt', {
+          type: 'text/plain'
+        })
+      });
+    }).toThrow(new Error('Could not convert data to URLSearchParams: object does not have own enumerable string-keyed properties'));
   });
   it('returns the data as received if contentType is "text/plain"', () => {
     const data = 'Something';

package/build/cjs/links/RestAPILink/queryToRequestOptions/xWwwFormUrlencodedMatchers.js

@@ -0,0 +1,13 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.isSvgConversion = void 0;
+
+// POST to convert an SVG file
+const isSvgConversion = (type, {
+  resource
+}) => type === 'create' && (resource === 'svg.png' || resource === 'svg.pdf');
+
+exports.isSvgConversion = isSvgConversion;

package/build/cjs/links/RestAPILink/queryToRequestOptions/xWwwFormUrlencodedMatchers.test.js

@@ -0,0 +1,21 @@
+"use strict";
+
+var _xWwwFormUrlencodedMatchers = require("./xWwwFormUrlencodedMatchers");
+
+describe('isSvgConversion', () => {
+  it('returns true for a POST to "svg.png"', () => {
+    expect((0, _xWwwFormUrlencodedMatchers.isSvgConversion)('create', {
+      resource: 'svg.png'
+    })).toEqual(true);
+  });
+  it('returns true for a POST to "svg.pdf"', () => {
+    expect((0, _xWwwFormUrlencodedMatchers.isSvgConversion)('create', {
+      resource: 'svg.pdf'
+    })).toEqual(true);
+  });
+  it('retuns false for a POST to a different resource', () => {
+    expect((0, _xWwwFormUrlencodedMatchers.isSvgConversion)('create', {
+      resource: 'notSvg'
+    })).toEqual(false);
+  });
+});

package/build/cjs/links/RestAPILink/queryToResourcePath.js

@@ -64,7 +64,7 @@ const isAction = resource => resource.startsWith(actionPrefix);
 const makeActionPath = resource => (0, _path.joinPath)('dhis-web-commons', "".concat(resource.substr(actionPrefix.length), ".action"));
 
 const skipApiVersion = (resource, config) => {
-  if (resource === 'tracker') {
+  if (resource === 'tracker' || resource.startsWith('tracker/')) {
     var _config$serverVersion, _config$serverVersion2;
 
     if (!((_config$serverVersion = config.serverVersion) !== null && _config$serverVersion !== void 0 && _config$serverVersion.minor) || ((_config$serverVersion2 = config.serverVersion) === null || _config$serverVersion2 === void 0 ? void 0 : _config$serverVersion2.minor) < 38) {

package/build/cjs/links/RestAPILink/queryToResourcePath.test.js

@@ -153,6 +153,12 @@ describe('queryToResourcePath', () => {
     };
     expect((0, _queryToResourcePath.queryToResourcePath)(link, query, 'read')).toBe("".concat(link.unversionedApiPath, "/tracker"));
   });
+  it('should return an unversioned endpoint sub-resources of the new tracker importer (in version 2.37)', () => {
+    const query = {
+      resource: 'tracker/test'
+    };
+    expect((0, _queryToResourcePath.queryToResourcePath)(link, query, 'read')).toBe("".concat(link.unversionedApiPath, "/tracker/test"));
+  });
   it('should return a VERSIONED endpoint for the new tracker importer (in version 2.38)', () => {
     const query = {
       resource: 'tracker'

package/build/cjs/react/hooks/mergeAndCompareVariables.js

@@ -0,0 +1,33 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.mergeAndCompareVariables = void 0;
+
+var _stableVariablesHash = require("./stableVariablesHash");
+
+const mergeAndCompareVariables = (previousVariables, newVariables, previousHash) => {
+  if (!newVariables) {
+    return {
+      identical: true,
+      mergedVariablesHash: previousHash,
+      mergedVariables: previousVariables
+    };
+  } // Use cached hash if it exists
+
+
+  const currentHash = previousHash || (0, _stableVariablesHash.stableVariablesHash)(previousVariables);
+  const mergedVariables = { ...previousVariables,
+    ...newVariables
+  };
+  const mergedVariablesHash = (0, _stableVariablesHash.stableVariablesHash)(mergedVariables);
+  const identical = currentHash === mergedVariablesHash;
+  return {
+    identical,
+    mergedVariablesHash,
+    mergedVariables
+  };
+};
+
+exports.mergeAndCompareVariables = mergeAndCompareVariables;

package/build/cjs/react/hooks/mergeAndCompareVariables.test.js

@@ -0,0 +1,57 @@
+"use strict";
+
+var _mergeAndCompareVariables = require("./mergeAndCompareVariables");
+
+var _stableVariablesHash = require("./stableVariablesHash");
+
+jest.mock('./stableVariablesHash', () => ({
+  stableVariablesHash: object => JSON.stringify(object)
+}));
+const testVariables = {
+  question: 'What do you get when you multiply six by nine?',
+  answer: 42
+};
+const testHash = (0, _stableVariablesHash.stableVariablesHash)(testVariables);
+describe('mergeAndCompareVariables', () => {
+  it('Should return previous variables and hash when no new variables are provided', () => {
+    expect((0, _mergeAndCompareVariables.mergeAndCompareVariables)(testVariables, undefined, undefined)).toMatchObject({
+      identical: true,
+      mergedVariables: testVariables,
+      mergedVariablesHash: undefined
+    });
+  });
+  it('Should return identical: true when merged variables are identical to old variables (without prev hash)', () => {
+    const newVariables = {
+      answer: testVariables.answer
+    };
+    expect((0, _mergeAndCompareVariables.mergeAndCompareVariables)(testVariables, newVariables, undefined)).toMatchObject({
+      identical: true,
+      mergedVariables: testVariables,
+      mergedVariablesHash: testHash
+    });
+  });
+  it('Should return identical: false with incorrect previous hash', () => {
+    const incorrectPreviousHash = 'IAmAHash';
+    const newVariables = {
+      answer: 42
+    };
+    expect((0, _mergeAndCompareVariables.mergeAndCompareVariables)(testVariables, newVariables, incorrectPreviousHash)).toMatchObject({
+      identical: false,
+      mergedVariables: testVariables,
+      mergedVariablesHash: testHash
+    });
+  });
+  it('Should return identical: false when merged variables are different than old variables', () => {
+    const newVariables = {
+      answer: 43
+    };
+    const expectedMergedVariables = { ...testVariables,
+      ...newVariables
+    };
+    expect((0, _mergeAndCompareVariables.mergeAndCompareVariables)(testVariables, newVariables, testHash)).toMatchObject({
+      identical: false,
+      mergedVariables: expectedMergedVariables,
+      mergedVariablesHash: (0, _stableVariablesHash.stableVariablesHash)(expectedMergedVariables)
+    });
+  });
+});

package/build/cjs/react/hooks/useDataQuery.js

@@ -9,7 +9,7 @@ var _react = require("react");
 
 var _reactQuery = require("react-query");
 
-var _stableVariablesHash = require("./stableVariablesHash");
+var _mergeAndCompareVariables = require("./mergeAndCompareVariables");
 
 var _useDataEngine = require("./useDataEngine");
 
@@ -35,24 +35,35 @@ const useDataQuery = (query, {
   variables: initialVariables = {},
   lazy: initialLazy = false
 } = {}) => {
-  const variablesHash = (0, _react.useRef)(null);
-  const [variables, setVariables] = (0, _react.useState)(initialVariables);
-  const [enabled, setEnabled] = (0, _react.useState)(!initialLazy);
   const [staticQuery] = (0, _useStaticInput.useStaticInput)(query, {
     warn: true,
     name: 'query'
   });
+  const [variablesUpdateCount, setVariablesUpdateCount] = (0, _react.useState)(0);
+  const queryState = (0, _react.useRef)({
+    variables: initialVariables,
+    variablesHash: undefined,
+    enabled: !initialLazy,
+    refetchCallback: undefined
+  });
   /**
-   * User callbacks and refetch handling
+   * Display current query state and refetch count in React DevTools
    */
 
-  const refetchCallback = (0, _react.useRef)(null);
+  (0, _react.useDebugValue)({
+    variablesUpdateCount,
+    enabled: queryState.current.enabled,
+    variables: queryState.current.variables
+  }, debugValue => JSON.stringify(debugValue));
+  /**
+   * User callbacks and refetch handling
+   */
 
   const onSuccess = data => {
-    if (refetchCallback.current) {
-      refetchCallback.current(data);
-      refetchCallback.current = null;
-    }
+    var _queryState$current$r, _queryState$current;
+
+    (_queryState$current$r = (_queryState$current = queryState.current).refetchCallback) === null || _queryState$current$r === void 0 ? void 0 : _queryState$current$r.call(_queryState$current, data);
+    queryState.current.refetchCallback = undefined;
 
     if (userOnSuccess) {
       userOnSuccess(data);
@@ -61,9 +72,7 @@ const useDataQuery = (query, {
 
   const onError = error => {
     // If we'd want to reject on errors we'd call the cb with the error here
-    if (refetchCallback.current) {
-      refetchCallback.current = null;
-    }
+    queryState.current.refetchCallback = undefined;
 
     if (userOnError) {
       userOnError(error);
@@ -75,10 +84,10 @@ const useDataQuery = (query, {
 
 
   const engine = (0, _useDataEngine.useDataEngine)();
-  const queryKey = [staticQuery, variables];
+  const queryKey = [staticQuery, queryState.current.variables];
 
   const queryFn = () => engine.query(staticQuery, {
-    variables
+    variables: queryState.current.variables
   });
 
   const {
@@ -89,7 +98,7 @@ const useDataQuery = (query, {
     data,
     refetch: queryRefetch
   } = (0, _reactQuery.useQuery)(queryKey, queryFn, {
-    enabled,
+    enabled: queryState.current.enabled,
     onSuccess,
     onError
   });
@@ -103,11 +112,17 @@ const useDataQuery = (query, {
    */
 
   const refetch = (0, _react.useCallback)(newVariables => {
+    const {
+      identical,
+      mergedVariables,
+      mergedVariablesHash
+    } = (0, _mergeAndCompareVariables.mergeAndCompareVariables)(queryState.current.variables, newVariables, queryState.current.variablesHash);
     /**
      * If there are no updates that will trigger an automatic refetch
      * we'll need to call react-query's refetch directly
      */
-    if (enabled && !newVariables) {
+
+    if (queryState.current.enabled && identical) {
       return queryRefetch({
         cancelRefetch: true,
         throwOnError: false
@@ -116,44 +131,19 @@ const useDataQuery = (query, {
       }) => data);
     }
 
-    if (newVariables) {
-      // Use cached hash if it exists
-      const currentHash = variablesHash.current || (0, _stableVariablesHash.stableVariablesHash)(variables);
-      const mergedVariables = { ...variables,
-        ...newVariables
-      };
-      const mergedHash = (0, _stableVariablesHash.stableVariablesHash)(mergedVariables);
-      const identical = currentHash === mergedHash;
-
-      if (identical && enabled) {
-        /**
-         * If the variables are identical and the query is enabled
-         * we'll need to trigger the refetch manually
-         */
-        return queryRefetch({
-          cancelRefetch: true,
-          throwOnError: false
-        }).then(({
-          data
-        }) => data);
-      } else {
-        variablesHash.current = mergedHash;
-        setVariables(mergedVariables);
-      }
-    } // Enable the query after the variables have been set to prevent extra request
-
-
-    if (!enabled) {
-      setEnabled(true);
-    } // This promise does not currently reject on errors
-
-
-    return new Promise(resolve => {
-      refetchCallback.current = data => {
+    queryState.current.variables = mergedVariables;
+    queryState.current.variablesHash = mergedVariablesHash;
+    queryState.current.enabled = true; // This promise does not currently reject on errors
+
+    const refetchPromise = new Promise(resolve => {
+      queryState.current.refetchCallback = data => {
         resolve(data);
       };
-    });
-  }, [enabled, queryRefetch, variables]);
+    }); // Trigger a react-query refetch by incrementing variablesUpdateCount state
+
+    setVariablesUpdateCount(prevCount => prevCount + 1);
+    return refetchPromise;
+  }, [queryRefetch]);
   /**
    * react-query returns null or an error, but we return undefined
    * or an error, so this ensures consistency with the other types.

package/build/cjs/react/hooks/useDataQuery.test.js

@@ -507,6 +507,65 @@ describe('useDataQuery', () => {
     });
   });
   describe('return values: refetch', () => {
+    it('Should be stable if the query variables change', async () => {
+      let count = 0;
+      const spy = jest.fn(() => {
+        count++;
+        return count;
+      });
+      const data = {
+        answer: spy
+      };
+      const query = {
+        x: {
+          resource: 'answer'
+        }
+      };
+
+      const wrapper = ({
+        children
+      }) => /*#__PURE__*/React.createElement(_CustomDataProvider.CustomDataProvider, {
+        data: data
+      }, children);
+
+      const {
+        result,
+        waitFor
+      } = (0, _reactHooks.renderHook)(() => (0, _useDataQuery.useDataQuery)(query, {
+        lazy: true
+      }), {
+        wrapper
+      });
+      expect(spy).not.toHaveBeenCalled();
+      const initialRefetch = result.current.refetch;
+      (0, _reactHooks.act)(() => {
+        initialRefetch();
+      });
+      await waitFor(() => {
+        expect(result.current).toMatchObject({
+          loading: false,
+          called: true,
+          data: {
+            x: 1
+          }
+        });
+      });
+      expect(spy).toHaveBeenCalledTimes(1);
+      (0, _reactHooks.act)(() => {
+        initialRefetch();
+      });
+      await waitFor(() => {
+        expect(result.current).toMatchObject({
+          loading: false,
+          called: true,
+          data: {
+            x: 2
+          }
+        });
+      });
+      expect(spy).toHaveBeenCalledTimes(2);
+      expect(initialRefetch).toBe(result.current.refetch);
+    });
     it('Should only trigger a single request when refetch is called on a lazy query with new variables', async () => {
       const spy = jest.fn((type, query) => {
         if (query.id === '1') {

package/build/cjs/react/hooks/useStaticInput.js

@@ -13,6 +13,7 @@ const useStaticInput = (staticValue, {
 } = {}) => {
   const originalValue = (0, _react.useRef)(staticValue);
   const [value, setValue] = (0, _react.useState)(() => originalValue.current);
+  (0, _react.useDebugValue)(value, debugValue => "".concat(name, ": ").concat(JSON.stringify(debugValue)));
   (0, _react.useEffect)(() => {
     if (warn && originalValue.current !== staticValue) {
       console.warn("The ".concat(name, " should be static, don't create it within the render loop!"));

package/build/es/links/RestAPILink/queryToRequestOptions/multipartFormDataMatchers.js

@@ -26,8 +26,4 @@ export const isStaticContentUpload = (type, {
 
 export const isAppInstall = (type, {
   resource
-}) => type === 'create' && resource === 'apps'; // POST to convert an SVG file
-
-export const isSvgConversion = (type, {
-  resource
-}) => type === 'create' && (resource === 'svg.png' || resource === 'svg.pdf');
+}) => type === 'create' && resource === 'apps';

package/build/es/links/RestAPILink/queryToRequestOptions/multipartFormDataMatchers.test.js

@@ -1,4 +1,4 @@
-import { isFileResourceUpload, isMessageConversationAttachment, isStaticContentUpload, isAppInstall, isSvgConversion, isDataValue } from './multipartFormDataMatchers';
+import { isFileResourceUpload, isMessageConversationAttachment, isStaticContentUpload, isAppInstall, isDataValue } from './multipartFormDataMatchers';
 describe('isDataValue', () => {
   it('returns true for a POST to "dataValues"', () => {
     expect(isDataValue('create', {
@@ -68,21 +68,4 @@ describe('isAppInstall', () => {
       resource: 'notApps'
     })).toEqual(false);
   });
-});
-describe('isSvgConversion', () => {
-  it('returns true for a POST to "svg.png"', () => {
-    expect(isSvgConversion('create', {
-      resource: 'svg.png'
-    })).toEqual(true);
-  });
-  it('returns true for a POST to "svg.pdf"', () => {
-    expect(isSvgConversion('create', {
-      resource: 'svg.pdf'
-    })).toEqual(true);
-  });
-  it('retuns false for a POST to a different resource', () => {
-    expect(isSvgConversion('create', {
-      resource: 'notSvg'
-    })).toEqual(false);
-  });
 });

package/build/es/links/RestAPILink/queryToRequestOptions/requestContentType.js

@@ -1,23 +1,26 @@
 import * as multipartFormDataMatchers from './multipartFormDataMatchers';
 import * as textPlainMatchers from './textPlainMatchers';
+import * as xWwwFormUrlencodedMatchers from './xWwwFormUrlencodedMatchers';
 
 const resourceExpectsTextPlain = (type, query) => Object.values(textPlainMatchers).some(textPlainMatcher => textPlainMatcher(type, query));
 
 const resourceExpectsMultipartFormData = (type, query) => Object.values(multipartFormDataMatchers).some(multipartFormDataMatcher => multipartFormDataMatcher(type, query));
 
-export const FORM_DATA_ERROR_MSG = 'Could not convert data to FormData: object does not have own enumerable string-keyed properties';
+const resourceExpectsXWwwFormUrlencoded = (type, query) => Object.values(xWwwFormUrlencodedMatchers).some(xWwwFormUrlencodedMatcher => xWwwFormUrlencodedMatcher(type, query));
 
-const convertToFormData = data => {
+export const getConversionErrorMessage = outputType => "Could not convert data to ".concat(outputType, ": object does not have own enumerable string-keyed properties");
+
+const convertData = (data, initialValue) => {
   const dataEntries = Object.entries(data);
 
   if (dataEntries.length === 0) {
-    throw new Error(FORM_DATA_ERROR_MSG);
+    throw new Error(getConversionErrorMessage(initialValue.constructor.name));
   }
 
-  return dataEntries.reduce((formData, [key, value]) => {
-    formData.append(key, value);
-    return formData;
-  }, new FormData());
+  return dataEntries.reduce((convertedData, [key, value]) => {
+    convertedData.append(key, value);
+    return convertedData;
+  }, initialValue);
 };
 
 export const requestContentType = (type, query) => {
@@ -37,6 +40,10 @@ export const requestContentType = (type, query) => {
     return 'multipart/form-data';
   }
 
+  if (resourceExpectsXWwwFormUrlencoded(type, query)) {
+    return 'application/x-www-form-urlencoded';
+  }
+
   return 'application/json';
 };
 export const requestHeadersForContentType = contentType => {
@@ -67,7 +74,11 @@ export const requestBodyForContentType = (contentType, {
   }
 
   if (contentType === 'multipart/form-data') {
-    return convertToFormData(data);
+    return convertData(data, new FormData());
+  }
+
+  if (contentType === 'application/x-www-form-urlencoded') {
+    return convertData(data, new URLSearchParams());
   } // 'text/plain'
 
 

package/build/es/links/RestAPILink/queryToRequestOptions/requestContentType.test.js

@@ -1,4 +1,4 @@
-import { requestContentType, requestHeadersForContentType, requestBodyForContentType, FORM_DATA_ERROR_MSG } from './requestContentType';
+import { requestContentType, requestHeadersForContentType, requestBodyForContentType } from './requestContentType';
 describe('requestContentType', () => {
   it('returns "application/json" for a normal resource', () => {
     expect(requestContentType('create', {
@@ -85,7 +85,28 @@ describe('requestBodyForContentType', () => {
           type: 'text/plain'
         })
       });
-    }).toThrow(new Error(FORM_DATA_ERROR_MSG));
+    }).toThrow(new Error('Could not convert data to FormData: object does not have own enumerable string-keyed properties'));
+  });
+  it('converts to URLSearchParams if contentType is "application/x-www-form-urlencoded"', () => {
+    const data = {
+      a: 'AAA'
+    };
+    const result = requestBodyForContentType('application/x-www-form-urlencoded', {
+      resource: 'test',
+      data
+    });
+    expect(result instanceof URLSearchParams).toEqual(true);
+    expect(result.get('a')).toEqual('AAA');
+  });
+  it('throws an error if contentType is "application/x-www-form-urlencoded" and data does have own string-keyd properties', () => {
+    expect(() => {
+      requestBodyForContentType('application/x-www-form-urlencoded', {
+        resource: 'test',
+        data: new File(['foo'], 'foo.txt', {
+          type: 'text/plain'
+        })
+      });
+    }).toThrow(new Error('Could not convert data to URLSearchParams: object does not have own enumerable string-keyed properties'));
   });
   it('returns the data as received if contentType is "text/plain"', () => {
     const data = 'Something';

package/build/es/links/RestAPILink/queryToRequestOptions/xWwwFormUrlencodedMatchers.js

@@ -0,0 +1,4 @@
+// POST to convert an SVG file
+export const isSvgConversion = (type, {
+  resource
+}) => type === 'create' && (resource === 'svg.png' || resource === 'svg.pdf');

package/build/es/links/RestAPILink/queryToRequestOptions/xWwwFormUrlencodedMatchers.test.js

@@ -0,0 +1,18 @@
+import { isSvgConversion } from './xWwwFormUrlencodedMatchers';
+describe('isSvgConversion', () => {
+  it('returns true for a POST to "svg.png"', () => {
+    expect(isSvgConversion('create', {
+      resource: 'svg.png'
+    })).toEqual(true);
+  });
+  it('returns true for a POST to "svg.pdf"', () => {
+    expect(isSvgConversion('create', {
+      resource: 'svg.pdf'
+    })).toEqual(true);
+  });
+  it('retuns false for a POST to a different resource', () => {
+    expect(isSvgConversion('create', {
+      resource: 'notSvg'
+    })).toEqual(false);
+  });
+});

package/build/es/links/RestAPILink/queryToResourcePath.js

@@ -56,7 +56,7 @@ const isAction = resource => resource.startsWith(actionPrefix);
 const makeActionPath = resource => joinPath('dhis-web-commons', "".concat(resource.substr(actionPrefix.length), ".action"));
 
 const skipApiVersion = (resource, config) => {
-  if (resource === 'tracker') {
+  if (resource === 'tracker' || resource.startsWith('tracker/')) {
     var _config$serverVersion, _config$serverVersion2;
 
     if (!((_config$serverVersion = config.serverVersion) !== null && _config$serverVersion !== void 0 && _config$serverVersion.minor) || ((_config$serverVersion2 = config.serverVersion) === null || _config$serverVersion2 === void 0 ? void 0 : _config$serverVersion2.minor) < 38) {

package/build/es/links/RestAPILink/queryToResourcePath.test.js

@@ -150,6 +150,12 @@ describe('queryToResourcePath', () => {
     };
     expect(queryToResourcePath(link, query, 'read')).toBe("".concat(link.unversionedApiPath, "/tracker"));
   });
+  it('should return an unversioned endpoint sub-resources of the new tracker importer (in version 2.37)', () => {
+    const query = {
+      resource: 'tracker/test'
+    };
+    expect(queryToResourcePath(link, query, 'read')).toBe("".concat(link.unversionedApiPath, "/tracker/test"));
+  });
   it('should return a VERSIONED endpoint for the new tracker importer (in version 2.38)', () => {
     const query = {
       resource: 'tracker'

package/build/es/react/hooks/mergeAndCompareVariables.js

@@ -0,0 +1,23 @@
+import { stableVariablesHash } from './stableVariablesHash';
+export const mergeAndCompareVariables = (previousVariables, newVariables, previousHash) => {
+  if (!newVariables) {
+    return {
+      identical: true,
+      mergedVariablesHash: previousHash,
+      mergedVariables: previousVariables
+    };
+  } // Use cached hash if it exists
+
+
+  const currentHash = previousHash || stableVariablesHash(previousVariables);
+  const mergedVariables = { ...previousVariables,
+    ...newVariables
+  };
+  const mergedVariablesHash = stableVariablesHash(mergedVariables);
+  const identical = currentHash === mergedVariablesHash;
+  return {
+    identical,
+    mergedVariablesHash,
+    mergedVariables
+  };
+};

package/build/es/react/hooks/mergeAndCompareVariables.test.js

@@ -0,0 +1,53 @@
+import { mergeAndCompareVariables } from './mergeAndCompareVariables';
+import { stableVariablesHash } from './stableVariablesHash';
+jest.mock('./stableVariablesHash', () => ({
+  stableVariablesHash: object => JSON.stringify(object)
+}));
+const testVariables = {
+  question: 'What do you get when you multiply six by nine?',
+  answer: 42
+};
+const testHash = stableVariablesHash(testVariables);
+describe('mergeAndCompareVariables', () => {
+  it('Should return previous variables and hash when no new variables are provided', () => {
+    expect(mergeAndCompareVariables(testVariables, undefined, undefined)).toMatchObject({
+      identical: true,
+      mergedVariables: testVariables,
+      mergedVariablesHash: undefined
+    });
+  });
+  it('Should return identical: true when merged variables are identical to old variables (without prev hash)', () => {
+    const newVariables = {
+      answer: testVariables.answer
+    };
+    expect(mergeAndCompareVariables(testVariables, newVariables, undefined)).toMatchObject({
+      identical: true,
+      mergedVariables: testVariables,
+      mergedVariablesHash: testHash
+    });
+  });
+  it('Should return identical: false with incorrect previous hash', () => {
+    const incorrectPreviousHash = 'IAmAHash';
+    const newVariables = {
+      answer: 42
+    };
+    expect(mergeAndCompareVariables(testVariables, newVariables, incorrectPreviousHash)).toMatchObject({
+      identical: false,
+      mergedVariables: testVariables,
+      mergedVariablesHash: testHash
+    });
+  });
+  it('Should return identical: false when merged variables are different than old variables', () => {
+    const newVariables = {
+      answer: 43
+    };
+    const expectedMergedVariables = { ...testVariables,
+      ...newVariables
+    };
+    expect(mergeAndCompareVariables(testVariables, newVariables, testHash)).toMatchObject({
+      identical: false,
+      mergedVariables: expectedMergedVariables,
+      mergedVariablesHash: stableVariablesHash(expectedMergedVariables)
+    });
+  });
+});

package/build/es/react/hooks/useDataQuery.js

@@ -1,6 +1,6 @@
-import { useState, useRef, useCallback } from 'react';
+import { useState, useRef, useCallback, useDebugValue } from 'react';
 import { useQuery, setLogger } from 'react-query';
-import { stableVariablesHash } from './stableVariablesHash';
+import { mergeAndCompareVariables } from './mergeAndCompareVariables';
 import { useDataEngine } from './useDataEngine';
 import { useStaticInput } from './useStaticInput';
 
@@ -23,24 +23,35 @@ export const useDataQuery = (query, {
   variables: initialVariables = {},
   lazy: initialLazy = false
 } = {}) => {
-  const variablesHash = useRef(null);
-  const [variables, setVariables] = useState(initialVariables);
-  const [enabled, setEnabled] = useState(!initialLazy);
   const [staticQuery] = useStaticInput(query, {
     warn: true,
     name: 'query'
   });
+  const [variablesUpdateCount, setVariablesUpdateCount] = useState(0);
+  const queryState = useRef({
+    variables: initialVariables,
+    variablesHash: undefined,
+    enabled: !initialLazy,
+    refetchCallback: undefined
+  });
   /**
-   * User callbacks and refetch handling
+   * Display current query state and refetch count in React DevTools
    */
 
-  const refetchCallback = useRef(null);
+  useDebugValue({
+    variablesUpdateCount,
+    enabled: queryState.current.enabled,
+    variables: queryState.current.variables
+  }, debugValue => JSON.stringify(debugValue));
+  /**
+   * User callbacks and refetch handling
+   */
 
   const onSuccess = data => {
-    if (refetchCallback.current) {
-      refetchCallback.current(data);
-      refetchCallback.current = null;
-    }
+    var _queryState$current$r, _queryState$current;
+
+    (_queryState$current$r = (_queryState$current = queryState.current).refetchCallback) === null || _queryState$current$r === void 0 ? void 0 : _queryState$current$r.call(_queryState$current, data);
+    queryState.current.refetchCallback = undefined;
 
     if (userOnSuccess) {
       userOnSuccess(data);
@@ -49,9 +60,7 @@ export const useDataQuery = (query, {
 
   const onError = error => {
     // If we'd want to reject on errors we'd call the cb with the error here
-    if (refetchCallback.current) {
-      refetchCallback.current = null;
-    }
+    queryState.current.refetchCallback = undefined;
 
     if (userOnError) {
       userOnError(error);
@@ -63,10 +72,10 @@ export const useDataQuery = (query, {
 
 
   const engine = useDataEngine();
-  const queryKey = [staticQuery, variables];
+  const queryKey = [staticQuery, queryState.current.variables];
 
   const queryFn = () => engine.query(staticQuery, {
-    variables
+    variables: queryState.current.variables
   });
 
   const {
@@ -77,7 +86,7 @@ export const useDataQuery = (query, {
     data,
     refetch: queryRefetch
   } = useQuery(queryKey, queryFn, {
-    enabled,
+    enabled: queryState.current.enabled,
     onSuccess,
     onError
   });
@@ -91,11 +100,17 @@ export const useDataQuery = (query, {
    */
 
   const refetch = useCallback(newVariables => {
+    const {
+      identical,
+      mergedVariables,
+      mergedVariablesHash
+    } = mergeAndCompareVariables(queryState.current.variables, newVariables, queryState.current.variablesHash);
     /**
      * If there are no updates that will trigger an automatic refetch
      * we'll need to call react-query's refetch directly
      */
-    if (enabled && !newVariables) {
+
+    if (queryState.current.enabled && identical) {
       return queryRefetch({
         cancelRefetch: true,
         throwOnError: false
@@ -104,44 +119,19 @@ export const useDataQuery = (query, {
       }) => data);
     }
 
-    if (newVariables) {
-      // Use cached hash if it exists
-      const currentHash = variablesHash.current || stableVariablesHash(variables);
-      const mergedVariables = { ...variables,
-        ...newVariables
-      };
-      const mergedHash = stableVariablesHash(mergedVariables);
-      const identical = currentHash === mergedHash;
-
-      if (identical && enabled) {
-        /**
-         * If the variables are identical and the query is enabled
-         * we'll need to trigger the refetch manually
-         */
-        return queryRefetch({
-          cancelRefetch: true,
-          throwOnError: false
-        }).then(({
-          data
-        }) => data);
-      } else {
-        variablesHash.current = mergedHash;
-        setVariables(mergedVariables);
-      }
-    } // Enable the query after the variables have been set to prevent extra request
-
-
-    if (!enabled) {
-      setEnabled(true);
-    } // This promise does not currently reject on errors
-
-
-    return new Promise(resolve => {
-      refetchCallback.current = data => {
+    queryState.current.variables = mergedVariables;
+    queryState.current.variablesHash = mergedVariablesHash;
+    queryState.current.enabled = true; // This promise does not currently reject on errors
+
+    const refetchPromise = new Promise(resolve => {
+      queryState.current.refetchCallback = data => {
         resolve(data);
       };
-    });
-  }, [enabled, queryRefetch, variables]);
+    }); // Trigger a react-query refetch by incrementing variablesUpdateCount state
+
+    setVariablesUpdateCount(prevCount => prevCount + 1);
+    return refetchPromise;
+  }, [queryRefetch]);
   /**
    * react-query returns null or an error, but we return undefined
    * or an error, so this ensures consistency with the other types.

package/build/es/react/hooks/useDataQuery.test.js

@@ -497,6 +497,65 @@ describe('useDataQuery', () => {
     });
   });
   describe('return values: refetch', () => {
+    it('Should be stable if the query variables change', async () => {
+      let count = 0;
+      const spy = jest.fn(() => {
+        count++;
+        return count;
+      });
+      const data = {
+        answer: spy
+      };
+      const query = {
+        x: {
+          resource: 'answer'
+        }
+      };
+
+      const wrapper = ({
+        children
+      }) => /*#__PURE__*/React.createElement(CustomDataProvider, {
+        data: data
+      }, children);
+
+      const {
+        result,
+        waitFor
+      } = renderHook(() => useDataQuery(query, {
+        lazy: true
+      }), {
+        wrapper
+      });
+      expect(spy).not.toHaveBeenCalled();
+      const initialRefetch = result.current.refetch;
+      act(() => {
+        initialRefetch();
+      });
+      await waitFor(() => {
+        expect(result.current).toMatchObject({
+          loading: false,
+          called: true,
+          data: {
+            x: 1
+          }
+        });
+      });
+      expect(spy).toHaveBeenCalledTimes(1);
+      act(() => {
+        initialRefetch();
+      });
+      await waitFor(() => {
+        expect(result.current).toMatchObject({
+          loading: false,
+          called: true,
+          data: {
+            x: 2
+          }
+        });
+      });
+      expect(spy).toHaveBeenCalledTimes(2);
+      expect(initialRefetch).toBe(result.current.refetch);
+    });
     it('Should only trigger a single request when refetch is called on a lazy query with new variables', async () => {
       const spy = jest.fn((type, query) => {
         if (query.id === '1') {

package/build/es/react/hooks/useStaticInput.js

@@ -1,10 +1,11 @@
-import { useState, useEffect, useRef } from 'react';
+import { useState, useEffect, useRef, useDebugValue } from 'react';
 export const useStaticInput = (staticValue, {
   warn = false,
   name = 'input'
 } = {}) => {
   const originalValue = useRef(staticValue);
   const [value, setValue] = useState(() => originalValue.current);
+  useDebugValue(value, debugValue => "".concat(name, ": ").concat(JSON.stringify(debugValue)));
   useEffect(() => {
     if (warn && originalValue.current !== staticValue) {
       console.warn("The ".concat(name, " should be static, don't create it within the render loop!"));

package/build/types/links/RestAPILink/queryToRequestOptions/multipartFormDataMatchers.d.ts

@@ -4,4 +4,3 @@ export declare const isFileResourceUpload: (type: FetchType, { resource }: Resol
 export declare const isMessageConversationAttachment: (type: FetchType, { resource }: ResolvedResourceQuery) => boolean;
 export declare const isStaticContentUpload: (type: FetchType, { resource }: ResolvedResourceQuery) => boolean;
 export declare const isAppInstall: (type: FetchType, { resource }: ResolvedResourceQuery) => boolean;
-export declare const isSvgConversion: (type: FetchType, { resource }: ResolvedResourceQuery) => boolean;

package/build/types/links/RestAPILink/queryToRequestOptions/requestContentType.d.ts

@@ -1,7 +1,7 @@
 import { ResolvedResourceQuery, FetchType } from '../../../engine';
-declare type RequestContentType = 'application/json' | 'application/json-patch+json' | 'text/plain' | 'multipart/form-data' | null;
-export declare const FORM_DATA_ERROR_MSG = "Could not convert data to FormData: object does not have own enumerable string-keyed properties";
+declare type RequestContentType = 'application/json' | 'application/json-patch+json' | 'text/plain' | 'multipart/form-data' | 'application/x-www-form-urlencoded' | null;
+export declare const getConversionErrorMessage: (outputType: string) => string;
 export declare const requestContentType: (type: FetchType, query: ResolvedResourceQuery) => null | RequestContentType;
 export declare const requestHeadersForContentType: (contentType: RequestContentType) => undefined | Record<'Content-Type', string>;
-export declare const requestBodyForContentType: (contentType: RequestContentType, { data }: ResolvedResourceQuery) => undefined | string | FormData;
+export declare const requestBodyForContentType: (contentType: RequestContentType, { data }: ResolvedResourceQuery) => undefined | string | FormData | URLSearchParams;
 export {};

package/build/types/links/RestAPILink/queryToRequestOptions/xWwwFormUrlencodedMatchers.d.ts

@@ -0,0 +1,2 @@
+import { ResolvedResourceQuery, FetchType } from '../../../engine';
+export declare const isSvgConversion: (type: FetchType, { resource }: ResolvedResourceQuery) => boolean;

package/build/types/react/hooks/mergeAndCompareVariables.d.ts

@@ -0,0 +1,6 @@
+import type { QueryVariables } from '../../engine';
+export declare const mergeAndCompareVariables: (previousVariables?: QueryVariables | undefined, newVariables?: QueryVariables | undefined, previousHash?: string | undefined) => {
+    identical: boolean;
+    mergedVariablesHash: string | undefined;
+    mergedVariables: QueryVariables | undefined;
+};

package/build/types/react/hooks/useDataQuery.d.ts

@@ -1,3 +1,3 @@
-import { Query, QueryOptions } from '../../engine';
-import { QueryRenderInput } from '../../types';
+import type { Query, QueryOptions } from '../../engine';
+import type { QueryRenderInput } from '../../types';
 export declare const useDataQuery: (query: Query, { onComplete: userOnSuccess, onError: userOnError, variables: initialVariables, lazy: initialLazy, }?: QueryOptions) => QueryRenderInput;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@dhis2/app-service-data",
-    "version": "3.4.1",
+    "version": "3.4.4",
     "main": "./build/cjs/index.js",
     "module": "./build/es/index.js",
     "types": "build/types/index.d.ts",
@@ -22,7 +22,7 @@
         "build/**"
     ],
     "peerDependencies": {
-        "@dhis2/app-service-config": "3.4.1",
+        "@dhis2/app-service-config": "3.4.4",
         "@dhis2/cli-app-scripts": "^7.1.1",
         "prop-types": "^15.7.2",
         "react": "^16.8",