fix(cluster): correct the parameter range render (#121)

Author: Yuiham

apps/main/[3]cluster/[1]instances/[-2]_clusterId/[4]params/components/ParamsTable/index.tsx

@@ -223,7 +223,7 @@ function getColumns(t: TFunction<''>, form: FormInstance) {
       key: 'range',
       renderText(_, record) {
         return isArray(record.range) && record.range.length > 0
-          ? renderRange(record.type!, record.range, record.rangeType!)
+          ? renderRange(record.range, record.rangeType!)
           : null
       },
       editable: false,

apps/main/[3]cluster/[3]param-group/[-2]_paramGroupId/components/ParamCard/index.tsx

@@ -42,7 +42,7 @@ function getColumns(t: TFunction<''>) {
       key: 'range',
       renderText(_, record) {
         return isArray(record.range) && record.range.length > 0
-          ? renderRange(record.type!, record.range, record.rangeType!)
+          ? renderRange(record.range, record.rangeType!)
           : null
       },
     },

apps/main/[3]cluster/[3]param-group/components/EditableParamCard/helper.ts

@@ -1,52 +1,32 @@
-import { ParamValueDataType, ParamRangeType } from '@/api/model'
+import { ParamRangeType } from '@/api/model'
 
 const rangeRenderTable = [
   {
-    valueType: (type: number) =>
-      [ParamValueDataType.int, ParamValueDataType.float].includes(type),
     rangeType: (type: number) => type === ParamRangeType.continuous,
-    render: (range: string[]) => `${range[0]} ~ ${range[1]}`,
+    render: (range: string[]) => {
+      const [start, end] = range
+
+      return [start, end].filter((el) => el !== undefined).join(' ~ ')
+    },
   },
   {
-    valueType: (type: number) =>
-      [ParamValueDataType.int, ParamValueDataType.float].includes(type),
     rangeType: (type: number) => type === ParamRangeType.discrete,
     render: (range: string[]) =>
       range.filter((el) => el !== undefined).join(', '),
   },
   {
-    valueType: (type: number) =>
-      [ParamValueDataType.string, ParamValueDataType.array].includes(type),
-    rangeType: (type: number) =>
-      type === ParamRangeType.discrete || type === ParamRangeType.none,
-    render: (range: string[]) =>
-      range.filter((el) => el !== undefined).join(', '),
-  },
-  {
-    valueType: (type: number) => type === ParamValueDataType.boolean,
-    rangeType: (type: number) =>
-      type === ParamRangeType.discrete || type === ParamRangeType.none,
-    render: (range: string[]) => {
-      const [first, second] = range
-
-      return [first, second].filter((el) => el !== undefined).join(', ')
-    },
+    rangeType: (type: number) => type === ParamRangeType.none,
+    render: () => undefined,
   },
 ]
 
 /**
- * Render parameter range
- * @param type parameter type
- * @param range parameter range
+ * Render the parameter range
+ * @param range parameter range value
+ * @param rangeType parameter range type
  */
-export const renderRange = (
-  valType: number,
-  range: string[],
-  rangeType: number
-) => {
-  const target = rangeRenderTable.find(
-    (config) => config.valueType(valType) && config.rangeType(rangeType)
-  )
+export const renderRange = (range: string[], rangeType: number) => {
+  const target = rangeRenderTable.find((config) => config.rangeType(rangeType))
 
   return target?.render(range)
 }

apps/main/[3]cluster/[3]param-group/components/EditableParamCard/index.tsx

@@ -53,7 +53,7 @@ function getColumns(
       key: 'range',
       renderText(_, record) {
         return isArray(record.range) && record.range.length > 0
-          ? renderRange(record.type!, record.range, record.rangeType!)
+          ? renderRange(record.range, record.rangeType!)
           : null
       },
       editable: false,