perf: [10196] validate bad_int in the table

Author: arianber7

src/FileUtils/AgGridUtils.js

@@ -87,17 +87,13 @@ const _getColTypeFromTypeArray = (typeArray) => {
 };
 
 const _validateFormat = (rows, hasHeader, cols, options) => {
-  const colsData = cols.map((col) => ({
-    ...col,
-    type: _getColTypeFromTypeArray(col.type),
-  }));
   const errors = [];
-  const knownColsCount = colsData.length;
+  const knownColsCount = cols.length;
   const startIndex = hasHeader ? 1 : 0;
 
-  const colMeta = colsData.map((col) => ({
+  const colMeta = cols.map((col) => ({
     field: col.field,
-    type: col.type,
+    type: _getColTypeFromTypeArray(col.type),
     acceptsEmptyFields: col.acceptsEmptyFields ?? col.cellEditorParams?.acceptsEmptyFields ?? false,
     colOptions: {
       ...options,
@@ -119,7 +115,7 @@ const _validateFormat = (rows, hasHeader, cols, options) => {
     const row = rows[rowIndex];
     while (row[row.length - 1] === undefined && row.length > knownColsCount) row.pop();
 
-    if (row.length < knownColsCount && !row.includes(undefined)) {
+    if (row.length !== knownColsCount || row.includes(undefined)) {
       _forgeColumnsCountError(row, rowIndex + 1, cols, errors);
     }
 
@@ -128,8 +124,7 @@ const _validateFormat = (rows, hasHeader, cols, options) => {
       const value = row[colIndex];
       if (value === undefined) continue;
 
-      const typeStr = Array.isArray(type) ? type[0] : type;
-      const validationResult = ValidationUtils.isValid(value, typeStr, colOptions, acceptsEmptyFields);
+      const validationResult = ValidationUtils.isValid(value, type, colOptions, acceptsEmptyFields);
       if (validationResult !== true) {
         const { summary: errorSummary, context: errorContext } = validationResult;
         const errorLoc = `Line ${rowIndex + 1}, Column ${colIndex + 1} ("${field}")`;