From c8133edddf2617aa38042aead93e96467085b657 Mon Sep 17 00:00:00 2001
From: Chloe7733 <yangyh0573@gmail.com>
Date: Mon, 8 Dec 2025 17:32:44 +0800
Subject: [PATCH] fix: add articleId validation to avoid potential NPE in
 detail API

---
 .../forum/web/front/article/rest/ArticleRestController.java   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/paicoding-web/src/main/java/com/github/paicoding/forum/web/front/article/rest/ArticleRestController.java b/paicoding-web/src/main/java/com/github/paicoding/forum/web/front/article/rest/ArticleRestController.java
index e1a462a8..e62ff1c2 100644
--- a/paicoding-web/src/main/java/com/github/paicoding/forum/web/front/article/rest/ArticleRestController.java
+++ b/paicoding-web/src/main/java/com/github/paicoding/forum/web/front/article/rest/ArticleRestController.java
@@ -86,6 +86,10 @@ public class ArticleRestController {
      */
     @GetMapping("/data/detail/{articleId}")
     public ResVo<ArticleDetailVo> detail(@PathVariable(name = "articleId") Long articleId) throws IOException {
+        // === Fix: validate articleId to avoid potential NPE ===
+        if (articleId == null || articleId <= 0) {
+            return ResVo.fail("Invalid articleId: " + articleId);
+        } 
         ArticleDetailVo vo = new ArticleDetailVo();
         // 文章相关信息
         ArticleDTO articleDTO = articleService.queryFullArticleInfo(articleId, ReqInfoContext.getReqInfo().getUserId());