From e517cec28ef9e7b268e5eba2ba2d6264182a4019 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E7=8E=8B=E8=89=AF?= <841369634@qq.com> Date: Mon, 27 Apr 2026 18:07:35 +0800 Subject: [PATCH 1/2] perf: Return the obtained `chat_record` --- apps/chat/serializers/chat.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/chat/serializers/chat.py b/apps/chat/serializers/chat.py index 4514dc457bc..60e65a93f09 100644 --- a/apps/chat/serializers/chat.py +++ b/apps/chat/serializers/chat.py @@ -384,8 +384,9 @@ def get_chat_record(chat_info, chat_record_id): return chat_record_list[-1] chat_record = QuerySet(ChatRecord).filter(id=chat_record_id, chat_id=chat_info.chat_id).first() if chat_record is None: - if not is_valid_uuid(chat_record_id): - raise ChatException(500, _("Conversation record does not exist")) + raise ChatException(500, _("Conversation record does not exist")) + else: + return chat_record chat_record = QuerySet(ChatRecord).filter(id=chat_record_id).first() return chat_record From 4fd4e69ff19bd9f6d7e1cbde7e606b4c1b517bf8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=88=98=E7=91=9E=E6=96=8C?= Date: Tue, 16 Jun 2026 17:53:32 +0800 Subject: [PATCH 2/2] chore: simplify chat record return logic Refactor chat record retrieval logic to simplify return statement. --- apps/chat/serializers/chat.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/chat/serializers/chat.py b/apps/chat/serializers/chat.py index 60e65a93f09..58f23136638 100644 --- a/apps/chat/serializers/chat.py +++ b/apps/chat/serializers/chat.py @@ -385,8 +385,8 @@ def get_chat_record(chat_info, chat_record_id): chat_record = QuerySet(ChatRecord).filter(id=chat_record_id, chat_id=chat_info.chat_id).first() if chat_record is None: raise ChatException(500, _("Conversation record does not exist")) - else: - return chat_record + + return chat_record chat_record = QuerySet(ChatRecord).filter(id=chat_record_id).first() return chat_record