gmakstutis/ragflow
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix

Browse source
This commit is contained in:
Billy Bao 2025-11-14 13:43:18 +08:00
parent 72c20022f6
commit 8d7bf4c2ab
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
api/db/services/knowledgebase_service.py
View file

@ -393,10 +393,11 @@ 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()
if dataset_name == "": dataset_size = len(dataset_name)
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 len(dataset_name.encode("utf-8")) > DATASET_NAME_LIMIT: if dataset_size > DATASET_NAME_LIMIT:
return get_data_error_result(message=f"Dataset name length is {len(dataset_name)} which is larger than {DATASET_NAME_LIMIT}") return get_data_error_result(message=f"Dataset name length is {dataset_size} which is larger than {DATASET_NAME_LIMIT}")
# Deduplicate name within tenant # Deduplicate name within tenant
dataset_name = duplicate_name( dataset_name = duplicate_name(
@ -409,9 +410,10 @@ class KnowledgebaseService(CommonService):
# Verify tenant exists # Verify tenant exists
ok, _t = TenantService.get_by_id(tenant_id) ok, _t = TenantService.get_by_id(tenant_id)
if not ok: if not ok:
return False, "Tenant not found." return get_data_error_result(message="Tenant does not exist.")
# Build payload # Build payload
parser_config = get_parser_config(parser_id, kwargs.get("parser_config"))
kb_id = get_uuid() kb_id = get_uuid()
payload = { payload = {
"id": kb_id, "id": kb_id,
@ -419,11 +421,11 @@ class KnowledgebaseService(CommonService):
"tenant_id": tenant_id, "tenant_id": tenant_id,
"created_by": tenant_id, "created_by": tenant_id,
"parser_id": (parser_id or "naive"), "parser_id": (parser_id or "naive"),
"parser_config": parser_config,
**kwargs **kwargs
} }
# Default parser_config (align with kb_app.create) — do not accept external overrides
payload["parser_config"] = get_parser_config(parser_id, kwargs.get("parser_config"))
return payload return payload