@teselagen/ui 0.5.21-beta.4 → 0.5.21-beta.5

package/index.cjs.js

@@ -48395,57 +48395,81 @@ const DataTable = /* @__PURE__ */ __name((_G) => {
     reduxFormSearchInput,
     urlConnected
   ]);
-  if (!isTableParamsConnected) {
-    const updateSearch = /* @__PURE__ */ __name((val) => {
-      change("reduxFormSearchInput", val || "");
-    }, "updateSearch");
-    const setNewParams2 = /* @__PURE__ */ __name((newParams) => {
-      urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
-      change("reduxFormQueryParams", newParams);
-    }, "setNewParams2");
-    const bindThese = makeDataTableHandlers({
-      setNewParams: setNewParams2,
-      updateSearch,
-      defaults: props.defaults,
-      onlyOneFilter: props.onlyOneFilter
-    });
-    const boundDispatchProps = {};
-    Object.keys(bindThese).forEach(function(key) {
-      const action = bindThese[key];
-      boundDispatchProps[key] = function(...args) {
-        action(...args, currentParams);
-      };
-    });
-    const changeFormValue = /* @__PURE__ */ __name((...args) => change(...args), "changeFormValue");
-    props.tableParams = __spreadProps(__spreadValues(__spreadValues(__spreadValues({
-      changeFormValue,
-      selectedEntities
-    }, _tableParams), props), boundDispatchProps), {
-      isTableParamsConnected: true
-      //let the table know not to do local sorting/filtering etc.
-    });
-  }
-  props = __spreadValues(__spreadValues({}, props), props.tableParams);
-  if (!isTableParamsConnected) {
-    const additionalFilterToUse = typeof props.additionalFilter === "function" ? props.additionalFilter.bind(void 0, ownProps) : () => props.additionalFilter;
-    const additionalOrFilterToUse = typeof props.additionalOrFilter === "function" ? props.additionalOrFilter.bind(void 0, ownProps) : () => props.additionalOrFilter;
-    props = __spreadValues(__spreadValues({}, props), getQueryParams({
-      doNotCoercePageSize,
-      currentParams,
-      entities: props.entities,
-      // for local table
-      urlConnected,
-      defaults: props.defaults,
-      schema: convertedSchema,
-      isInfinite,
-      isLocalCall,
-      additionalFilter: additionalFilterToUse,
-      additionalOrFilter: additionalOrFilterToUse,
-      noOrderError: props.noOrderError,
-      isCodeModel: props.isCodeModel,
-      ownProps: props
-    }));
-  }
+  const tableParams = React$1.useMemo(() => {
+    if (!isTableParamsConnected) {
+      const updateSearch = /* @__PURE__ */ __name((val) => {
+        change("reduxFormSearchInput", val || "");
+      }, "updateSearch");
+      const setNewParams2 = /* @__PURE__ */ __name((newParams) => {
+        urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
+        change("reduxFormQueryParams", newParams);
+      }, "setNewParams2");
+      const bindThese = makeDataTableHandlers({
+        setNewParams: setNewParams2,
+        updateSearch,
+        defaults: props.defaults,
+        onlyOneFilter: props.onlyOneFilter
+      });
+      const boundDispatchProps = {};
+      Object.keys(bindThese).forEach(function(key) {
+        const action = bindThese[key];
+        boundDispatchProps[key] = function(...args) {
+          action(...args, currentParams);
+        };
+      });
+      const changeFormValue = /* @__PURE__ */ __name((...args) => change(...args), "changeFormValue");
+      return __spreadProps(__spreadValues(__spreadValues(__spreadValues({
+        changeFormValue,
+        selectedEntities
+      }, _tableParams), props), boundDispatchProps), {
+        isTableParamsConnected: true
+        //let the table know not to do local sorting/filtering etc.
+      });
+    }
+    return _tableParams;
+  }, [
+    _tableParams,
+    change,
+    currentParams,
+    history.replace,
+    isTableParamsConnected,
+    props,
+    selectedEntities,
+    urlConnected
+  ]);
+  props = __spreadValues(__spreadValues({}, props), tableParams);
+  const queryParams = React$1.useMemo(() => {
+    if (!isTableParamsConnected) {
+      const additionalFilterToUse = typeof props.additionalFilter === "function" ? props.additionalFilter.bind(void 0, ownProps) : () => props.additionalFilter;
+      const additionalOrFilterToUse = typeof props.additionalOrFilter === "function" ? props.additionalOrFilter.bind(void 0, ownProps) : () => props.additionalOrFilter;
+      return getQueryParams({
+        doNotCoercePageSize,
+        currentParams,
+        entities: props.entities,
+        // for local table
+        urlConnected,
+        defaults: props.defaults,
+        schema: convertedSchema,
+        isInfinite,
+        isLocalCall,
+        additionalFilter: additionalFilterToUse,
+        additionalOrFilter: additionalOrFilterToUse,
+        noOrderError: props.noOrderError,
+        isCodeModel: props.isCodeModel,
+        ownProps: props
+      });
+    }
+    return {};
+  }, [
+    convertedSchema,
+    currentParams,
+    doNotCoercePageSize,
+    isInfinite,
+    isLocalCall,
+    isTableParamsConnected,
+    urlConnected
+  ]);
+  props = __spreadValues(__spreadValues({}, props), queryParams);
   const {
     addFilters = noop$3,
     additionalFilters,
@@ -64431,22 +64455,13 @@ function withTableParams(compOrOpts, pTopLevelOpts) {
     const mergedOpts = getMergedOpts(topLevelOptions, ownProps);
     const { formName, urlConnected, history, defaults: defaults2, onlyOneFilter } = mergedOpts;
     function updateSearch(val) {
-      setTimeout(function() {
-        dispatch(reduxForm.change(formName, "reduxFormSearchInput", val || ""));
-      });
+      dispatch(reduxForm.change(formName, "reduxFormSearchInput", val || ""));
     }
     __name(updateSearch, "updateSearch");
-    let setNewParams;
-    if (urlConnected) {
-      setNewParams = /* @__PURE__ */ __name(function(newParams) {
-        setCurrentParamsOnUrl(newParams, history.replace);
-        dispatch(reduxForm.change(formName, "reduxFormQueryParams", newParams));
-      }, "setNewParams");
-    } else {
-      setNewParams = /* @__PURE__ */ __name(function(newParams) {
-        dispatch(reduxForm.change(formName, "reduxFormQueryParams", newParams));
-      }, "setNewParams");
-    }
+    const setNewParams = /* @__PURE__ */ __name((newParams) => {
+      urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
+      dispatch(reduxForm.change(formName, "reduxFormQueryParams", newParams));
+    }, "setNewParams");
     return {
       bindThese: makeDataTableHandlers({
         setNewParams,

package/index.es.js

@@ -48377,57 +48377,81 @@ const DataTable = /* @__PURE__ */ __name((_G) => {
     reduxFormSearchInput,
     urlConnected
   ]);
-  if (!isTableParamsConnected) {
-    const updateSearch = /* @__PURE__ */ __name((val) => {
-      change2("reduxFormSearchInput", val || "");
-    }, "updateSearch");
-    const setNewParams2 = /* @__PURE__ */ __name((newParams) => {
-      urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
-      change2("reduxFormQueryParams", newParams);
-    }, "setNewParams2");
-    const bindThese = makeDataTableHandlers({
-      setNewParams: setNewParams2,
-      updateSearch,
-      defaults: props.defaults,
-      onlyOneFilter: props.onlyOneFilter
-    });
-    const boundDispatchProps = {};
-    Object.keys(bindThese).forEach(function(key) {
-      const action = bindThese[key];
-      boundDispatchProps[key] = function(...args) {
-        action(...args, currentParams);
-      };
-    });
-    const changeFormValue = /* @__PURE__ */ __name((...args) => change2(...args), "changeFormValue");
-    props.tableParams = __spreadProps(__spreadValues(__spreadValues(__spreadValues({
-      changeFormValue,
-      selectedEntities
-    }, _tableParams), props), boundDispatchProps), {
-      isTableParamsConnected: true
-      //let the table know not to do local sorting/filtering etc.
-    });
-  }
-  props = __spreadValues(__spreadValues({}, props), props.tableParams);
-  if (!isTableParamsConnected) {
-    const additionalFilterToUse = typeof props.additionalFilter === "function" ? props.additionalFilter.bind(void 0, ownProps) : () => props.additionalFilter;
-    const additionalOrFilterToUse = typeof props.additionalOrFilter === "function" ? props.additionalOrFilter.bind(void 0, ownProps) : () => props.additionalOrFilter;
-    props = __spreadValues(__spreadValues({}, props), getQueryParams({
-      doNotCoercePageSize,
-      currentParams,
-      entities: props.entities,
-      // for local table
-      urlConnected,
-      defaults: props.defaults,
-      schema: convertedSchema,
-      isInfinite,
-      isLocalCall,
-      additionalFilter: additionalFilterToUse,
-      additionalOrFilter: additionalOrFilterToUse,
-      noOrderError: props.noOrderError,
-      isCodeModel: props.isCodeModel,
-      ownProps: props
-    }));
-  }
+  const tableParams = useMemo(() => {
+    if (!isTableParamsConnected) {
+      const updateSearch = /* @__PURE__ */ __name((val) => {
+        change2("reduxFormSearchInput", val || "");
+      }, "updateSearch");
+      const setNewParams2 = /* @__PURE__ */ __name((newParams) => {
+        urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
+        change2("reduxFormQueryParams", newParams);
+      }, "setNewParams2");
+      const bindThese = makeDataTableHandlers({
+        setNewParams: setNewParams2,
+        updateSearch,
+        defaults: props.defaults,
+        onlyOneFilter: props.onlyOneFilter
+      });
+      const boundDispatchProps = {};
+      Object.keys(bindThese).forEach(function(key) {
+        const action = bindThese[key];
+        boundDispatchProps[key] = function(...args) {
+          action(...args, currentParams);
+        };
+      });
+      const changeFormValue = /* @__PURE__ */ __name((...args) => change2(...args), "changeFormValue");
+      return __spreadProps(__spreadValues(__spreadValues(__spreadValues({
+        changeFormValue,
+        selectedEntities
+      }, _tableParams), props), boundDispatchProps), {
+        isTableParamsConnected: true
+        //let the table know not to do local sorting/filtering etc.
+      });
+    }
+    return _tableParams;
+  }, [
+    _tableParams,
+    change2,
+    currentParams,
+    history.replace,
+    isTableParamsConnected,
+    props,
+    selectedEntities,
+    urlConnected
+  ]);
+  props = __spreadValues(__spreadValues({}, props), tableParams);
+  const queryParams = useMemo(() => {
+    if (!isTableParamsConnected) {
+      const additionalFilterToUse = typeof props.additionalFilter === "function" ? props.additionalFilter.bind(void 0, ownProps) : () => props.additionalFilter;
+      const additionalOrFilterToUse = typeof props.additionalOrFilter === "function" ? props.additionalOrFilter.bind(void 0, ownProps) : () => props.additionalOrFilter;
+      return getQueryParams({
+        doNotCoercePageSize,
+        currentParams,
+        entities: props.entities,
+        // for local table
+        urlConnected,
+        defaults: props.defaults,
+        schema: convertedSchema,
+        isInfinite,
+        isLocalCall,
+        additionalFilter: additionalFilterToUse,
+        additionalOrFilter: additionalOrFilterToUse,
+        noOrderError: props.noOrderError,
+        isCodeModel: props.isCodeModel,
+        ownProps: props
+      });
+    }
+    return {};
+  }, [
+    convertedSchema,
+    currentParams,
+    doNotCoercePageSize,
+    isInfinite,
+    isLocalCall,
+    isTableParamsConnected,
+    urlConnected
+  ]);
+  props = __spreadValues(__spreadValues({}, props), queryParams);
   const {
     addFilters = noop$3,
     additionalFilters,
@@ -64413,22 +64437,13 @@ function withTableParams(compOrOpts, pTopLevelOpts) {
     const mergedOpts = getMergedOpts(topLevelOptions, ownProps);
     const { formName, urlConnected, history, defaults: defaults2, onlyOneFilter } = mergedOpts;
     function updateSearch(val) {
-      setTimeout(function() {
-        dispatch(change(formName, "reduxFormSearchInput", val || ""));
-      });
+      dispatch(change(formName, "reduxFormSearchInput", val || ""));
     }
     __name(updateSearch, "updateSearch");
-    let setNewParams;
-    if (urlConnected) {
-      setNewParams = /* @__PURE__ */ __name(function(newParams) {
-        setCurrentParamsOnUrl(newParams, history.replace);
-        dispatch(change(formName, "reduxFormQueryParams", newParams));
-      }, "setNewParams");
-    } else {
-      setNewParams = /* @__PURE__ */ __name(function(newParams) {
-        dispatch(change(formName, "reduxFormQueryParams", newParams));
-      }, "setNewParams");
-    }
+    const setNewParams = /* @__PURE__ */ __name((newParams) => {
+      urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
+      dispatch(change(formName, "reduxFormQueryParams", newParams));
+    }, "setNewParams");
     return {
       bindThese: makeDataTableHandlers({
         setNewParams,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teselagen/ui",
-  "version": "0.5.21-beta.4",
+  "version": "0.5.21-beta.5",
   "main": "./src/index.js",
   "exports": {
     ".": {

package/src/DataTable/index.js

@@ -278,63 +278,74 @@ const DataTable = ({
     urlConnected
   ]);
 
-  if (!isTableParamsConnected) {
-    const updateSearch = val => {
-      change("reduxFormSearchInput", val || "");
-    };
+  const tableParams = useMemo(() => {
+    if (!isTableParamsConnected) {
+      const updateSearch = val => {
+        change("reduxFormSearchInput", val || "");
+      };
 
-    const setNewParams = newParams => {
-      urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
-      change("reduxFormQueryParams", newParams); //we always will update the redux params as a workaround for withRouter not always working if inside a redux-connected container https://github.com/ReactTraining/react-router/issues/5037
-    };
+      const setNewParams = newParams => {
+        urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
+        change("reduxFormQueryParams", newParams); //we always will update the redux params as a workaround for withRouter not always working if inside a redux-connected container https://github.com/ReactTraining/react-router/issues/5037
+      };
 
-    const bindThese = makeDataTableHandlers({
-      setNewParams,
-      updateSearch,
-      defaults: props.defaults,
-      onlyOneFilter: props.onlyOneFilter
-    });
+      const bindThese = makeDataTableHandlers({
+        setNewParams,
+        updateSearch,
+        defaults: props.defaults,
+        onlyOneFilter: props.onlyOneFilter
+      });
 
-    const boundDispatchProps = {};
-    //bind currentParams to actions
-    Object.keys(bindThese).forEach(function (key) {
-      const action = bindThese[key];
-      boundDispatchProps[key] = function (...args) {
-        action(...args, currentParams);
-      };
-    });
+      const boundDispatchProps = {};
+      //bind currentParams to actions
+      Object.keys(bindThese).forEach(function (key) {
+        const action = bindThese[key];
+        boundDispatchProps[key] = function (...args) {
+          action(...args, currentParams);
+        };
+      });
 
-    const changeFormValue = (...args) => change(...args);
+      const changeFormValue = (...args) => change(...args);
 
-    props.tableParams = {
-      changeFormValue,
-      selectedEntities,
-      ..._tableParams,
-      ...props,
-      ...boundDispatchProps,
-      isTableParamsConnected: true //let the table know not to do local sorting/filtering etc.
-    };
-  }
+      return {
+        changeFormValue,
+        selectedEntities,
+        ..._tableParams,
+        ...props,
+        ...boundDispatchProps,
+        isTableParamsConnected: true //let the table know not to do local sorting/filtering etc.
+      };
+    }
+    return _tableParams;
+  }, [
+    _tableParams,
+    change,
+    currentParams,
+    history.replace,
+    isTableParamsConnected,
+    props,
+    selectedEntities,
+    urlConnected
+  ]);
 
   props = {
     ...props,
-    ...props.tableParams
+    ...tableParams
   };
 
-  if (!isTableParamsConnected) {
-    const additionalFilterToUse =
-      typeof props.additionalFilter === "function"
-        ? props.additionalFilter.bind(this, ownProps)
-        : () => props.additionalFilter;
+  const queryParams = useMemo(() => {
+    if (!isTableParamsConnected) {
+      const additionalFilterToUse =
+        typeof props.additionalFilter === "function"
+          ? props.additionalFilter.bind(this, ownProps)
+          : () => props.additionalFilter;
 
-    const additionalOrFilterToUse =
-      typeof props.additionalOrFilter === "function"
-        ? props.additionalOrFilter.bind(this, ownProps)
-        : () => props.additionalOrFilter;
+      const additionalOrFilterToUse =
+        typeof props.additionalOrFilter === "function"
+          ? props.additionalOrFilter.bind(this, ownProps)
+          : () => props.additionalOrFilter;
 
-    props = {
-      ...props,
-      ...getQueryParams({
+      return getQueryParams({
         doNotCoercePageSize,
         currentParams,
         entities: props.entities, // for local table
@@ -348,9 +359,24 @@ const DataTable = ({
         noOrderError: props.noOrderError,
         isCodeModel: props.isCodeModel,
         ownProps: props
-      })
-    };
-  }
+      });
+    }
+    return {};
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [
+    convertedSchema,
+    currentParams,
+    doNotCoercePageSize,
+    isInfinite,
+    isLocalCall,
+    isTableParamsConnected,
+    urlConnected
+  ]);
+
+  props = {
+    ...props,
+    ...queryParams
+  };
 
   const {
     addFilters = noop,

package/src/DataTable/utils/withTableParams.js

@@ -182,22 +182,13 @@ export default function withTableParams(compOrOpts, pTopLevelOpts) {
       mergedOpts;
 
     function updateSearch(val) {
-      setTimeout(function () {
-        dispatch(change(formName, "reduxFormSearchInput", val || ""));
-      });
+      dispatch(change(formName, "reduxFormSearchInput", val || ""));
     }
 
-    let setNewParams;
-    if (urlConnected) {
-      setNewParams = function (newParams) {
-        setCurrentParamsOnUrl(newParams, history.replace);
-        dispatch(change(formName, "reduxFormQueryParams", newParams)); //we always will update the redux params as a workaround for withRouter not always working if inside a redux-connected container https://github.com/ReactTraining/react-router/issues/5037
-      };
-    } else {
-      setNewParams = function (newParams) {
-        dispatch(change(formName, "reduxFormQueryParams", newParams));
-      };
-    }
+    const setNewParams = newParams => {
+      urlConnected && setCurrentParamsOnUrl(newParams, history.replace);
+      dispatch(change(formName, "reduxFormQueryParams", newParams)); //we always will update the redux params as a workaround for withRouter not always working if inside a redux-connected container https://github.com/ReactTraining/react-router/issues/5037
+    };
     return {
       bindThese: makeDataTableHandlers({
         setNewParams,