@appcorp/fusion-storybook 0.1.77 → 0.1.79

package/base-modules/class/context.d.ts

@@ -115,7 +115,7 @@ export declare const useClassModule: () => {
     handleEdit: (row?: TableRow) => void;
     handleFilters: () => void;
     handleMoreActions: () => void;
-    handlePageChange: (page: number) => void;
+    handlePageChange: (pageOrEvent?: unknown) => void;
     handlePageLimitChange: (k: string, value: object) => void;
     handleSearch: (query: string) => void;
     handleSubmit: () => void;

package/base-modules/class/context.js

@@ -272,12 +272,19 @@ export const useClassModule = () => {
             payload: { drawer: CLASS_DRAWER.MORE_ACTIONS_DRAWER },
         });
     }, [dispatch]);
-    const handlePageChange = useCallback((page) => {
+    const handlePageChange = useCallback((pageOrEvent) => {
+        // The factory passes handlePageChange directly as the Next button's onClick
+        // handler (receiving a MouseEvent), but wraps Previous as () => handlePageChange(currentPage - 1).
+        // We detect which case we're in: a valid number means an explicit page target
+        // (Previous path); anything else means "advance to next page".
+        const page = typeof pageOrEvent === "number" && !isNaN(pageOrEvent)
+            ? pageOrEvent
+            : state.currentPage + 1;
         dispatch({
             type: CLASS_ACTION_TYPES.SET_CURRENT_PAGE,
             payload: { currentPage: page },
         });
-    }, [dispatch]);
+    }, [dispatch, state.currentPage]);
     const handlePageLimitChange = useCallback((k, value) => {
         const val = Object.assign({}, value);
         dispatch({
@@ -317,9 +324,8 @@ export const useClassModule = () => {
             type: CLASS_ACTION_TYPES.SET_CURRENT_PAGE,
             payload: { currentPage: 1 },
         });
-        listFetchNow();
         closeDrawer();
-    }, [dispatch, listFetchNow, closeDrawer]);
+    }, [dispatch, closeDrawer]);
     const clearFilters = useCallback(() => {
         dispatch({
             type: CLASS_ACTION_TYPES.SET_FILTERS,
@@ -417,8 +423,18 @@ export const useClassModule = () => {
     // ============================================================================
     // 1.4.9 EFFECTS
     // ============================================================================
-    // Initial load via cache; re-fetch directly from API on page/pageLimit/filter/search changes
+    // Always keep the ref current so the main effect can call the latest listFetchNow
+    // without including it in the dependency array (listFetchNow changes identity every
+    // render because useModuleEntityV2 passes an inline headers object to useFetch).
+    // Declared first so the ref is populated before the fetch effect runs.
     useEffect(() => {
+        listFetchNowRef.current = listFetchNow;
+    });
+    // Initial load via cache; re-fetch directly from API on page/pageLimit/filter/search changes.
+    // listFetchNow is intentionally excluded from deps — it changes every render due to the
+    // inline headers object in useModuleEntityV2. We call it via the stable ref instead.
+    useEffect(() => {
+        var _a;
         if (!schoolId)
             return;
         const currentPage = Number(listParams.currentPage) || 1;
@@ -444,14 +460,11 @@ export const useClassModule = () => {
             })();
         }
         else {
-            // Bypass cache for pagination, pageLimit, filter and search changes
-            listFetchNow();
+            // Bypass cache for pagination, pageLimit, filter and search changes.
+            // Use ref to avoid the infinite-loop caused by listFetchNow's unstable identity.
+            (_a = listFetchNowRef.current) === null || _a === void 0 ? void 0 : _a.call(listFetchNowRef);
         }
-    }, [dispatch, listFetchNow, listParams, schoolId, showToast, t]);
-    // Sync ref to always point at latest listFetchNow (avoids stale closure in callbacks)
-    useEffect(() => {
-        listFetchNowRef.current = listFetchNow;
-    });
+    }, [dispatch, listParams, schoolId, showToast, t]);
     // ============================================================================
     // 1.4.10 RETURN
     // ============================================================================

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@appcorp/fusion-storybook",
-  "version": "0.1.77",
+  "version": "0.1.79",
   "scripts": {
     "build-storybook": "storybook build",
     "build:next": "next build",