From 23068a7c05a4f56688c8ddbe0d418efd3607df5d Mon Sep 17 00:00:00 2001 From: taojinlong Date: Fri, 29 Nov 2024 12:13:47 +0800 Subject: [PATCH 1/4] =?UTF-8?q?=20fix:=20=E4=BF=AE=E5=A4=8D=E4=B8=8A?= =?UTF-8?q?=E4=BC=A0excel=E7=9A=84=E9=97=AE=E9=A2=98?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../java/io/dataease/datasource/provider/ExcelUtils.java | 1 + .../io/dataease/datasource/server/DatasourceServer.java | 8 -------- .../data/datasource/form/ApiHttpRequestDraw.vue | 6 +++++- .../views/visualized/data/datasource/form/ExcelDetail.vue | 8 +++++--- 4 files changed, 11 insertions(+), 12 deletions(-) diff --git a/core/core-backend/src/main/java/io/dataease/datasource/provider/ExcelUtils.java b/core/core-backend/src/main/java/io/dataease/datasource/provider/ExcelUtils.java index 3ae742deb7..31c18fc1b4 100644 --- a/core/core-backend/src/main/java/io/dataease/datasource/provider/ExcelUtils.java +++ b/core/core-backend/src/main/java/io/dataease/datasource/provider/ExcelUtils.java @@ -486,6 +486,7 @@ public class ExcelUtils { tableFiled.setName(filedName); tableFiled.setOriginName(filedName); tableFiled.setFieldType(null); + tableFiled.setChecked(true); fields.add(tableFiled); } diff --git a/core/core-backend/src/main/java/io/dataease/datasource/server/DatasourceServer.java b/core/core-backend/src/main/java/io/dataease/datasource/server/DatasourceServer.java index 5370af6d50..848a67b145 100644 --- a/core/core-backend/src/main/java/io/dataease/datasource/server/DatasourceServer.java +++ b/core/core-backend/src/main/java/io/dataease/datasource/server/DatasourceServer.java @@ -810,19 +810,11 @@ public class DatasourceServer implements DatasourceApi { datasourceRequest.setDatasource(transDTO(coreDatasource)); List datasetTableDTOS = ExcelUtils.getTables(datasourceRequest); for (ExcelSheetData sheet : excelFileData.getSheets()) { - boolean find = false; for (DatasetTableDTO datasetTableDTO : datasetTableDTOS) { if (excelDataTableName(datasetTableDTO.getTableName()).equals(sheet.getTableName()) || isCsv(file.getOriginalFilename())) { - find = true; sheet.setDeTableName(datasetTableDTO.getTableName()); - datasourceRequest.setTable(datasetTableDTO.getTableName()); - List oldTableFields = ExcelUtils.getTableFields(datasourceRequest); - mergeFields(oldTableFields, sheet.getFields()); } } - if (!find) { - sheet.setNewSheet(true); - } } } } diff --git a/core/core-frontend/src/views/visualized/data/datasource/form/ApiHttpRequestDraw.vue b/core/core-frontend/src/views/visualized/data/datasource/form/ApiHttpRequestDraw.vue index 7dc1fba8c2..c6f2c62281 100644 --- a/core/core-frontend/src/views/visualized/data/datasource/form/ApiHttpRequestDraw.vue +++ b/core/core-frontend/src/views/visualized/data/datasource/form/ApiHttpRequestDraw.vue @@ -289,7 +289,11 @@ const saveItem = () => { } } else { for (let i = 0; i < apiItem.fields.length; i++) { - if (apiItem.fields[i].primaryKey && !apiItem.fields[i].length) { + if ( + apiItem.fields[i].primaryKey && + !apiItem.fields[i].length && + apiItem.fields[i].deExtractType === 0 + ) { ElMessage.error(t('datasource.primary_key_length') + apiItem.fields[i].name) return } diff --git a/core/core-frontend/src/views/visualized/data/datasource/form/ExcelDetail.vue b/core/core-frontend/src/views/visualized/data/datasource/form/ExcelDetail.vue index 5c79d1feed..8363eebf92 100644 --- a/core/core-frontend/src/views/visualized/data/datasource/form/ExcelDetail.vue +++ b/core/core-frontend/src/views/visualized/data/datasource/form/ExcelDetail.vue @@ -207,6 +207,7 @@ const saveExcelDs = (params, successCb, finallyCb) => { let effectExtField = false let changeFiled = false let selectNode = state.excelData[0]?.sheets + console.log(1) for (let i = 0; i < selectNode.length; i++) { if (selectNode[i].sheet) { if (selectNode[i].effectExtField) { @@ -243,6 +244,7 @@ const saveExcelDs = (params, successCb, finallyCb) => { sheetFileMd5.push(selectNode[i].fieldsMd5) } } + console.log(2) if (!selectedSheet.length) { ElMessage({ message: t('dataset.ple_select_excel'), @@ -644,7 +646,7 @@ defineExpose({ >
-
+