fix dataset name verification
This commit is contained in:
parent
fd19a42d06
commit
68930d6fc7
1 changed files with 3 additions and 4 deletions
|
|
@ -393,11 +393,10 @@ class KnowledgebaseService(CommonService):
|
||||||
if not isinstance(name, str):
|
if not isinstance(name, str):
|
||||||
return get_data_error_result(message="Dataset name must be string.")
|
return get_data_error_result(message="Dataset name must be string.")
|
||||||
dataset_name = name.strip()
|
dataset_name = name.strip()
|
||||||
dataset_size = len(dataset_name)
|
if len(dataset_name) == 0:
|
||||||
if dataset_size == 0:
|
|
||||||
return get_data_error_result(message="Dataset name can't be empty.")
|
return get_data_error_result(message="Dataset name can't be empty.")
|
||||||
if dataset_size > DATASET_NAME_LIMIT:
|
if len(dataset_name.encode("utf-8")) > DATASET_NAME_LIMIT:
|
||||||
return get_data_error_result(message=f"Dataset name length is {dataset_size} which is larger than {DATASET_NAME_LIMIT}")
|
return get_data_error_result(message=f"Dataset name length is {len(dataset_name)} which is larger than {DATASET_NAME_LIMIT}")
|
||||||
|
|
||||||
# Deduplicate name within tenant
|
# Deduplicate name within tenant
|
||||||
dataset_name = duplicate_name(
|
dataset_name = duplicate_name(
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue