From afaa7144a5e2eab465fa1ae97c51b2c1e3246a1d Mon Sep 17 00:00:00 2001 From: Kevin Hu Date: Thu, 27 Feb 2025 10:39:34 +0800 Subject: [PATCH] Fix: issue of no id for /datasets//documents (#5420) ### What problem does this PR solve? #5401 ### Type of change - [x] Bug Fix (non-breaking change which fixes an issue) --- api/apps/sdk/doc.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/api/apps/sdk/doc.py b/api/apps/sdk/doc.py index 05cb204c6..3d0413544 100644 --- a/api/apps/sdk/doc.py +++ b/api/apps/sdk/doc.py @@ -477,10 +477,12 @@ def list_docs(dataset_id, tenant_id): return get_error_data_result(message=f"You don't own the dataset {dataset_id}. ") id = request.args.get("id") name = request.args.get("name") - if not DocumentService.query(id=id, kb_id=dataset_id): + + if id and not DocumentService.query(id=id, kb_id=dataset_id): return get_error_data_result(message=f"You don't own the document {id}.") - if not DocumentService.query(name=name, kb_id=dataset_id): + if name and not DocumentService.query(name=name, kb_id=dataset_id): return get_error_data_result(message=f"You don't own the document {name}.") + page = int(request.args.get("page", 1)) keywords = request.args.get("keywords", "") page_size = int(request.args.get("page_size", 30))