fix(pagination): fix wrong page (#1867)

Co-authored-by: maxin <maxin@growingio.com>
growingio · Feb 28, 2022 · 37497f8 · 37497f8 · vercel · Feb 28, 2022
1 parent eb5edfa
commit 37497f8
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/pagination/hooks/usePagination.ts b/src/pagination/hooks/usePagination.ts
@@ -33,18 +33,18 @@ const usePagination = (
 
   const goToPage: ReturnType<typeof usePagination>['goToPage'] = useCallback(
     (page, event) => {
-      if (page) {
+      if (page > 0) {
         setCurrentPage(Math.min(page, maxPages));
-      }
-      if (isFunction(onChange)) {
-        onChange(page, pageSize, event);
+        if (isFunction(onChange)) {
+          onChange(page, pageSize, event);
+        }
       }
     },
     [maxPages, onChange, pageSize, setCurrentPage]
   );
 
   useEffect(() => {
-    if (currentPage > maxPages) {
+    if (currentPage > maxPages && maxPages > 0) {
       goToPage(maxPages, null);
     }
   }, [maxPages, currentPage, setCurrentPage, goToPage]);