diff --git a/test-track/backend/src/main/java/io/metersphere/service/TestReviewTestCaseService.java b/test-track/backend/src/main/java/io/metersphere/service/TestReviewTestCaseService.java
index 47c0e2cf43..7daf743b94 100644
--- a/test-track/backend/src/main/java/io/metersphere/service/TestReviewTestCaseService.java
+++ b/test-track/backend/src/main/java/io/metersphere/service/TestReviewTestCaseService.java
@@ -521,6 +521,9 @@ public class TestReviewTestCaseService {
 
     public void editTestCaseForMinder(String reviewId, List<TestCaseReviewTestCase> testCaseReviewTestCases) {
         List<String> caseIds = testCaseReviewTestCases.stream().map(TestCaseReviewTestCase::getCaseId).collect(Collectors.toList());
+        if (CollectionUtils.isEmpty(caseIds)) {
+            return;
+        }
         checkReviewCase(reviewId, caseIds);
         if (!CollectionUtils.isEmpty(testCaseReviewTestCases)) {
             testCaseReviewTestCases.forEach((item) -> {
diff --git a/test-track/frontend/src/business/head/TrackHeaderMenus.vue b/test-track/frontend/src/business/head/TrackHeaderMenus.vue
index a07365a2c6..e390035662 100644
--- a/test-track/frontend/src/business/head/TrackHeaderMenus.vue
+++ b/test-track/frontend/src/business/head/TrackHeaderMenus.vue
@@ -68,7 +68,7 @@ export default {
           this.pathName = '/track/review/all';
         } else if (to.path.indexOf("/track/plan") >= 0) {
           this.pathName = '/track/plan/all';
-        } if (to.path.indexOf("/track/case") >= 0) {
+        } else if (to.path.indexOf("/track/case") >= 0) {
           this.pathName = this.caseListPath;
         } else {
           this.pathName = to.path;