From e5a54c31e7d8314a2689d568713d639837cbb624 Mon Sep 17 00:00:00 2001
From: "Jingwei.Zhang" <zhang.jingwei@99cloud.net>
Date: Tue, 2 Aug 2022 12:52:19 +0800
Subject: [PATCH] fix: fix the limit value when fetch api in the list page

Fix the limit value by the page size options, when the limit in the store is not in the page size options, use the default limit value.

Change-Id: I3868ebf9b33304ace884b0fa77eae28d89e80c20
---
 src/containers/List/index.jsx | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/containers/List/index.jsx b/src/containers/List/index.jsx
index 91600a63..72cd9219 100644
--- a/src/containers/List/index.jsx
+++ b/src/containers/List/index.jsx
@@ -401,7 +401,8 @@ export default class BaseList extends React.Component {
   }
 
   get pageSizeOptions() {
-    return undefined;
+    // should be array of numbers
+    return [10, 20, 50, 100];
   }
 
   get hideTotal() {
@@ -483,7 +484,7 @@ export default class BaseList extends React.Component {
     const pagination = {
       total,
       current: Number(page),
-      pageSize: limit || 10,
+      pageSize: this.getTablePageSize(limit),
       // eslint-disable-next-line no-shadow
       showTotal: (total) => t('Total {total} items', { total }),
       showSizeChanger: true,
@@ -607,6 +608,10 @@ export default class BaseList extends React.Component {
       newParams[this.allProjectsKey] = true;
     }
     if (this.isFilterByBackend) {
+      const { limit } = newParams;
+      if (limit) {
+        newParams.limit = this.getTablePageSize(limit);
+      }
       this.fetchListWithTry(() =>
         this.fetchDataByPage(this.updateFetchParamsByPage(newParams))
       );
@@ -1052,6 +1057,12 @@ export default class BaseList extends React.Component {
 
   onCloseSuccessHint = () => {};
 
+  getTablePageSize = (limit) => {
+    const defaultOptions = [10, 20, 50, 100];
+    const options = this.pageSizeOptions || defaultOptions;
+    return options.includes(limit) ? limit : options[0] || defaultOptions[0];
+  };
+
   debounceSetTableHeight() {
     return debounce(this.setTableHeight, 1000);
   }