studiokit-scaffolding-js 7.0.2 → 7.0.3-next.1.2

package/lib/css/components/_alert.css

@@ -8,6 +8,7 @@
 
 	&.MuiSnackbar-root {
 		position: fixed;
+		z-index: 3000;
 	}
 
 	& .MuiSvgIcon-root {

package/lib/hooks/useCollectionConfiguration.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.useCollectionConfiguration = void 0;
+var lodash_1 = require("lodash");
 var react_1 = require("react");
 var react_redux_1 = require("react-redux");
 var react_router_1 = require("react-router");
@@ -24,11 +25,12 @@ function useCollectionConfiguration(props, selectorFunction) {
     var _b = react_1.useState(modelStatus_1.MODEL_STATUS.UNINITIALIZED), modelStatus = _b[0], setModelStatus = _b[1];
     var previousModelStatus = usePrevious_1.usePrevious(modelStatus);
     var _c = react_1.useState(), fetchingId = _c[0], setFetchingId = _c[1];
-    var pathParams = react_1.useMemo(function () { return propPathParams || route_1.getPathParamsFromRouteMatchParams(routeMatchParams, modelName); }, [
-        modelName,
-        propPathParams,
-        routeMatchParams
-    ]);
+    // keep track of new and previous pathParams values
+    // this will almost always be a new array, which causes issues for hook deps
+    var pathParamsValue = react_1.useMemo(function () { return propPathParams || route_1.getPathParamsFromRouteMatchParams(routeMatchParams, modelName); }, [modelName, propPathParams, routeMatchParams]);
+    var previousPathParamsValue = usePrevious_1.usePrevious(pathParamsValue);
+    // only update pathParams if the value array actually contains different values, since it will always be a different ref
+    var pathParams = react_1.useMemo(function () { return (lodash_1.isEqual(pathParamsValue, previousPathParamsValue) ? previousPathParamsValue : pathParamsValue); }, [pathParamsValue, previousPathParamsValue]);
     var _d = react_redux_1.useSelector(function (state) {
         return selectorFunction({ guid: guid, modelName: modelName, pathParams: pathParams, routeMatchParams: routeMatchParams, state: state });
     }), model = _d.model, isCollectionItem = _d.isCollectionItem;
@@ -38,11 +40,11 @@ function useCollectionConfiguration(props, selectorFunction) {
     var previousQueryParams = usePrevious_1.usePrevious(queryParams);
     var modelArray = react_1.useMemo(function () { return (!isCollectionItem ? model_1.getModelArray(model, guid) : undefined); }, [guid, isCollectionItem, model]);
     var modelMinusRelations = react_1.useMemo(function () { return (isCollectionItem ? model_1.getModelMinusRelations(model) : undefined); }, [isCollectionItem, model]);
-    var changeModelStatus = function (newModelStatus, newFetchingId) {
+    var changeModelStatus = react_1.useCallback(function (newModelStatus, newFetchingId) {
         setFetchingId(newFetchingId);
         setModelStatus(newModelStatus);
-    };
-    var methodConfig = {
+    }, []);
+    var methodConfig = react_1.useMemo(function () { return ({
         modelName: modelName,
         pathParams: pathParams,
         queryParams: queryParams,
@@ -50,11 +52,11 @@ function useCollectionConfiguration(props, selectorFunction) {
         modelStatus: modelStatus,
         changeModelStatus: changeModelStatus,
         disableAutoLoad: disableAutoLoad
-    };
+    }); }, [modelName, pathParams, queryParams, guid, modelStatus, changeModelStatus, disableAutoLoad]);
     // update `modelStatus` when `model` fetch finishes
     react_1.useEffect(function () {
         model_1.handleModelFetchFinish(model, previousModel, fetchingId, changeModelStatus);
-    }, [fetchingId, model, previousModel]);
+    }, [fetchingId, model, previousModel, changeModelStatus]);
     return {
         previousModelName: previousModelName,
         guid: guid,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "studiokit-scaffolding-js",
-	"version": "7.0.2",
+	"version": "7.0.3-next.1.2",
 	"description": "Common scaffolding for Studio apps at Purdue",
 	"repository": "https://gitlab.com/purdue-informatics/studiokit/studiokit-scaffolding-js",
 	"license": "MIT",