From 0e9575a36a69c7b333c7d8307a75e0ad9507f52c Mon Sep 17 00:00:00 2001
From: Martin Weise <martin.weise@tuwien.ac.at>
Date: Thu, 13 Jun 2024 08:29:00 +0200
Subject: [PATCH] Hotfix column mapping and more error messages in the frontend
---
.../src/main/java/at/tuwien/mapper/MetadataMapper.java | 5 +++--
.../java/at/tuwien/service/impl/TableServiceImpl.java | 1 -
dbrepo-ui/pages/database/[database_id]/table/import.vue | 9 ++++-----
3 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/dbrepo-metadata-service/repositories/src/main/java/at/tuwien/mapper/MetadataMapper.java b/dbrepo-metadata-service/repositories/src/main/java/at/tuwien/mapper/MetadataMapper.java
index bb355f02c4..8e628f710a 100644
--- a/dbrepo-metadata-service/repositories/src/main/java/at/tuwien/mapper/MetadataMapper.java
+++ b/dbrepo-metadata-service/repositories/src/main/java/at/tuwien/mapper/MetadataMapper.java
@@ -829,8 +829,9 @@ public interface MetadataMapper {
final Pattern WHITESPACE = Pattern.compile("[\\s]");
String nowhitespace = WHITESPACE.matcher(data).replaceAll("_");
String normalized = Normalizer.normalize(nowhitespace, Normalizer.Form.NFD);
- String slug = NONLATIN.matcher(normalized).replaceAll("");
- final String name = slug.toLowerCase(Locale.ENGLISH);
+ String slug = NONLATIN.matcher(normalized).replaceAll("_");
+ final String name = slug.toLowerCase(Locale.ENGLISH)
+ .replaceAll("-", "_");
log.debug("mapping name {} to internal name {}", data, name);
return name;
}
diff --git a/dbrepo-metadata-service/services/src/main/java/at/tuwien/service/impl/TableServiceImpl.java b/dbrepo-metadata-service/services/src/main/java/at/tuwien/service/impl/TableServiceImpl.java
index e8fecdf300..593b612a9c 100644
--- a/dbrepo-metadata-service/services/src/main/java/at/tuwien/service/impl/TableServiceImpl.java
+++ b/dbrepo-metadata-service/services/src/main/java/at/tuwien/service/impl/TableServiceImpl.java
@@ -19,7 +19,6 @@ import at.tuwien.exception.*;
import at.tuwien.gateway.DataServiceGateway;
import at.tuwien.gateway.SearchServiceGateway;
import at.tuwien.mapper.MetadataMapper;
-import at.tuwien.mapper.SparqlMapper;
import at.tuwien.repository.DatabaseRepository;
import at.tuwien.service.*;
import lombok.extern.log4j.Log4j2;
diff --git a/dbrepo-ui/pages/database/[database_id]/table/import.vue b/dbrepo-ui/pages/database/[database_id]/table/import.vue
index 724205c7ba..07e56afea3 100644
--- a/dbrepo-ui/pages/database/[database_id]/table/import.vue
+++ b/dbrepo-ui/pages/database/[database_id]/table/import.vue
@@ -325,10 +325,9 @@ export default {
this.table = table
resolve(table)
})
- .catch((error) => {
- console.error('Failed to create table', error)
+ .catch(({code, message}) => {
const toast = useToastInstance()
- toast.error(this.$t(error.code))
+ toast.error(`${this.$t(code)}: ${message}`)
this.loading = false
reject(error)
})
@@ -347,10 +346,10 @@ export default {
toast.success(this.$t('success.import.dataset'))
this.cacheStore.reloadDatabase()
})
- .catch(({code}) => {
+ .catch(({code, message}) => {
console.error('Failed to import csv')
const toast = useToastInstance()
- toast.error(this.$t(code))
+ toast.error(`${this.$t(code)}: ${message}`)
this.loading = false
})
.finally(() => {
--
GitLab