From 3c59765b2ad56ab909e783c7841a39212e48dc5c Mon Sep 17 00:00:00 2001
From: chenjianxing <jianxing.chen@fit2cloud.com>
Date: Thu, 9 Apr 2020 13:51:04 +0800
Subject: [PATCH 1/2] =?UTF-8?q?=E6=B5=8B=E8=AF=95=E8=AF=A6=E6=83=85?=
 =?UTF-8?q?=E6=B7=BB=E5=8A=A0=E6=B5=8B=E8=AF=95=E8=AE=A1=E5=88=92=E8=8F=9C?=
 =?UTF-8?q?=E5=8D=95=E6=A0=8F?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../controller/TestPlanController.java        |  6 +++
 .../metersphere/service/TestPlanService.java  |  6 +++
 .../components/common/head/HeaderUser.vue     |  2 +
 .../components/track/case/TestCase.vue        | 37 +++++--------
 .../components/track/common/SelectMenu.vue    | 53 +++++++++++++++++++
 .../components/track/plan/TestPlanView.vue    | 52 ++++++++++++++----
 .../track/plan/components/TestPlanList.vue    |  6 +--
 7 files changed, 125 insertions(+), 37 deletions(-)
 create mode 100644 frontend/src/business/components/track/common/SelectMenu.vue

diff --git a/backend/src/main/java/io/metersphere/controller/TestPlanController.java b/backend/src/main/java/io/metersphere/controller/TestPlanController.java
index f8c1a7fcec..dbfa8375a4 100644
--- a/backend/src/main/java/io/metersphere/controller/TestPlanController.java
+++ b/backend/src/main/java/io/metersphere/controller/TestPlanController.java
@@ -34,6 +34,12 @@ public class TestPlanController {
         return PageUtils.setPageInfo(page, testPlanService.listTestPlan(request));
     }
 
+    @PostMapping("/list/all")
+    public List<TestPlan> listAll() {
+        String currentWorkspaceId = SessionUtils.getCurrentWorkspaceId();
+        return testPlanService.listTestAllPlan(currentWorkspaceId);
+    }
+
 
     @GetMapping("recent/{count}")
     public List<TestPlan> recentTestPlans(@PathVariable int count) {
diff --git a/backend/src/main/java/io/metersphere/service/TestPlanService.java b/backend/src/main/java/io/metersphere/service/TestPlanService.java
index d577adfc93..4d51e9be84 100644
--- a/backend/src/main/java/io/metersphere/service/TestPlanService.java
+++ b/backend/src/main/java/io/metersphere/service/TestPlanService.java
@@ -116,4 +116,10 @@ public class TestPlanService {
         testPlanTestCaseExample.setOrderByClause("update_time desc");
         return testPlanMapper.selectByExample(testPlanTestCaseExample);
     }
+
+    public List<TestPlan> listTestAllPlan(String currentWorkspaceId) {
+        TestPlanExample testPlanExample = new TestPlanExample();
+        testPlanExample.createCriteria().andWorkspaceIdEqualTo(currentWorkspaceId);
+        return testPlanMapper.selectByExample(testPlanExample);
+    }
 }
diff --git a/frontend/src/business/components/common/head/HeaderUser.vue b/frontend/src/business/components/common/head/HeaderUser.vue
index b106d7bd7d..c4f616fb38 100644
--- a/frontend/src/business/components/common/head/HeaderUser.vue
+++ b/frontend/src/business/components/common/head/HeaderUser.vue
@@ -130,6 +130,7 @@
         let orgId = data.id;
         this.$post("/user/switch/source/org/" + orgId, {}, response => {
           localStorage.setItem(TokenKey, JSON.stringify(response.data));
+          this.$router.push('/');
           window.location.reload();
         })
       },
@@ -141,6 +142,7 @@
         this.$post("/user/switch/source/ws/" + workspaceId, {}, response => {
           localStorage.setItem(TokenKey, JSON.stringify(response.data));
           localStorage.setItem("workspace_id", workspaceId);
+          this.$router.push('/');
           window.location.reload();
         })
       }
diff --git a/frontend/src/business/components/track/case/TestCase.vue b/frontend/src/business/components/track/case/TestCase.vue
index ef5d8dc8d4..cb36a94b39 100644
--- a/frontend/src/business/components/track/case/TestCase.vue
+++ b/frontend/src/business/components/track/case/TestCase.vue
@@ -3,29 +3,19 @@
     <el-container>
       <el-aside width="250px">
 
-        <el-menu :unique-opened="true" mode="horizontal" active-text-color="write"
-          class="project_menu">
-          <el-submenu index="1" popper-class="submenu">
-            <template v-slot:title>
-              {{currentProject == null ? '' : currentProject.name}}
-            </template>
-            <div style="height:400px;">
-              <el-scrollbar style="height:100%">
-                <label v-for="(item,index) in projects" :key="index">
-                  <el-menu-item @click="changeProject(item)">
-                    {{item.name}}
-                    <i class="el-icon-check" v-if="currentProject && item.id === currentProject.id"></i>
-                  </el-menu-item>
-                </label>
-              </el-scrollbar>
-          </div>
-          </el-submenu>
-        </el-menu>
+        <select-menu
+          :data="projects"
+          :current-data="currentProject"
+          :title="'项目'"
+          @dataChange="changeProject">
+        </select-menu>
+
         <node-tree class="node_tree"
                    :current-project="currentProject"
                    @nodeSelectEvent="refreshTable"
                    @refresh="refreshTable"
-                   ref="nodeTree"></node-tree>
+                   ref="nodeTree">
+        </node-tree>
       </el-aside>
 
       <el-main class="main-content">
@@ -34,10 +24,10 @@
           :current-project="currentProject"
           @openTestCaseEditDialog="openTestCaseEditDialog"
           @testCaseEdit="openTestCaseEditDialog"
-          ref="testCaseList"></test-case-list>
+          ref="testCaseList">
+        </test-case-list>
       </el-main>
 
-
     </el-container>
 
     <test-case-edit
@@ -53,10 +43,11 @@
   import TestCaseEdit from './components/TestCaseEdit';
   import {WORKSPACE_ID,CURRENT_PROJECT} from '../../../../common/constants';
   import TestCaseList from "./components/TestCaseList";
+  import SelectMenu from "../common/SelectMenu";
 
   export default {
     name: "TestCase",
-    components: {TestCaseList, NodeTree, TestCaseEdit},
+    components: {TestCaseList, NodeTree, TestCaseEdit, SelectMenu},
     comments: {},
     data() {
       return {
@@ -208,7 +199,7 @@
   }
 
   .node_tree {
-    margin: 10%;
+    margin: 5%;
   }
 
   .project_menu {
diff --git a/frontend/src/business/components/track/common/SelectMenu.vue b/frontend/src/business/components/track/common/SelectMenu.vue
new file mode 100644
index 0000000000..e7ba2b10d5
--- /dev/null
+++ b/frontend/src/business/components/track/common/SelectMenu.vue
@@ -0,0 +1,53 @@
+<template>
+  <el-menu :unique-opened="true" mode="horizontal" active-text-color="write"
+           class="project_menu">
+    <el-submenu index="1" popper-class="submenu">
+      <template v-slot:title>
+        <span class="menu-title">{{'[' + title + ']'}}</span>
+        <span> {{currentData == null ? '' : currentData.name}} </span>
+      </template>
+      <template v-slot:default>
+        <div style="height:400px;">
+          <el-scrollbar style="height:100%">
+            <label v-for="(item,index) in data" :key="index">
+              <el-menu-item @click="changeData(item)">
+                {{item.name}}
+                <i class="el-icon-check" v-if="currentData && item.id === currentData.id"></i>
+              </el-menu-item>
+            </label>
+          </el-scrollbar>
+        </div>
+      </template>
+    </el-submenu>
+  </el-menu>
+</template>
+
+<script>
+    export default {
+      name: "SelectMenu",
+      props: {
+        data: {
+          type: Array
+        },
+        currentData: {
+          type: Object
+        },
+        title: {
+          type: String
+        }
+      },
+      methods: {
+        changeData(data) {
+          this.$emit("dataChange", data);
+        }
+      }
+    }
+</script>
+
+<style scoped>
+
+  .menu-title {
+    color: darkgrey;
+  }
+
+</style>
diff --git a/frontend/src/business/components/track/plan/TestPlanView.vue b/frontend/src/business/components/track/plan/TestPlanView.vue
index f42cdcc4c2..dbc0e74e95 100644
--- a/frontend/src/business/components/track/plan/TestPlanView.vue
+++ b/frontend/src/business/components/track/plan/TestPlanView.vue
@@ -2,12 +2,21 @@
 
   <div class="plan_container">
     <el-container>
-      <el-aside class="node-tree" width="250px">
+      <el-aside class="aside-container" width="250px">
+        <select-menu
+          :data="testPlans"
+          :current-data="currentPlan"
+          :title="'计划'"
+          @dataChange="changePlan">
+        </select-menu>
+
         <plan-node-tree
-          :tree-nodes="treeNodes"
+          class="node_tree"
           :plan-id="planId"
           @nodeSelectEvent="getPlanCases"
-          ref="tree"></plan-node-tree>
+          ref="tree">
+        </plan-node-tree>
+
       </el-aside>
 
       <el-main>
@@ -30,7 +39,6 @@
       @refresh="refresh">
     </test-plan-test-case-edit>
 
-
   </div>
 
 </template>
@@ -41,13 +49,15 @@
     import TestCasePlanList from "./components/TestCasePlanList";
     import TestCaseRelevance from "./components/TestCaseRelevance";
     import TestPlanTestCaseEdit from "./components/TestPlanTestCaseEdit";
+    import SelectMenu from "../common/SelectMenu";
 
     export default {
       name: "TestPlanView",
-      components: {PlanNodeTree, TestCasePlanList, TestCaseRelevance, TestPlanTestCaseEdit},
+      components: {PlanNodeTree, TestCasePlanList, TestCaseRelevance, TestPlanTestCaseEdit, SelectMenu},
       data() {
         return {
-          treeNodes: []
+          testPlans: [],
+          currentPlan: {}
         }
       },
       computed: {
@@ -55,6 +65,14 @@
           return this.$route.params.planId;
         }
       },
+      created() {
+        this.getTestPlans();
+      },
+      watch: {
+        planId() {
+          this.getTestPlans();
+        }
+      },
       methods: {
         refresh() {
           this.getPlanCases();
@@ -71,7 +89,6 @@
           Object.assign(item, testCase);
           item.results = JSON.parse(item.results);
           item.steps = JSON.parse(item.steps);
-
           item.steptResults = [];
           for (let i = 0; i < item.steps.length; i++){
             if(item.results[i]){
@@ -82,6 +99,20 @@
           }
           this.$refs.testPlanTestCaseEdit.testCase = item;
           this.$refs.testPlanTestCaseEdit.dialog = true;
+        },
+        getTestPlans() {
+          this.result = this.$post('/test/plan/list/all', {}, response => {
+            this.testPlans = response.data;
+            this.testPlans.forEach(plan => {
+              if (this.planId && plan.id === this.planId) {
+                this.currentPlan = plan;
+              }
+            });
+          });
+        },
+        changePlan(plan) {
+          this.currentPlan = plan;
+          this.$router.push('/track/plan/view/' + plan.id);
         }
       }
     }
@@ -94,9 +125,12 @@
     height: 600px;
   }
 
-  .node-tree {
-    margin-top: 2%;
+  .aside-container {
     margin-left: 15px;
   }
 
+  .node_tree {
+    margin: 3%;
+  }
+
 </style>
diff --git a/frontend/src/business/components/track/plan/components/TestPlanList.vue b/frontend/src/business/components/track/plan/components/TestPlanList.vue
index 7f0f600b6c..52590812c3 100644
--- a/frontend/src/business/components/track/plan/components/TestPlanList.vue
+++ b/frontend/src/business/components/track/plan/components/TestPlanList.vue
@@ -139,7 +139,6 @@
           currentPage: 1,
           pageSize: 5,
           total: 0,
-          loadingRequire: {project: true, testCase: true},
           testId: null,
         }
       },
@@ -148,14 +147,11 @@
         this.initTableData();
       },
       methods: {
-        initTableData(nodeIds) {
+        initTableData() {
           let param = {
             name: this.condition,
           };
-          param.nodeIds = nodeIds;
-
           this.result = this.$post(this.buildPagePath(this.queryPath), param, response => {
-            this.loadingRequire.testCase = false;
             let data = response.data;
             this.total = data.itemCount;
             this.tableData = data.listObject;

From 9b34bb2819ab554634deb9adb38e07a7c5de6142 Mon Sep 17 00:00:00 2001
From: chenjianxing <jianxing.chen@fit2cloud.com>
Date: Thu, 9 Apr 2020 14:47:53 +0800
Subject: [PATCH 2/2] =?UTF-8?q?=E5=9B=BD=E9=99=85=E5=8C=96?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../components/track/case/TestCase.vue        |  3 +-
 .../components/track/plan/TestPlanView.vue    | 10 +++---
 ...ePlanList.vue => TestPlanTestCaseList.vue} | 32 +++++++++++++------
 frontend/src/i18n/en-US.js                    | 21 ++++++------
 frontend/src/i18n/zh-CN.js                    |  5 ++-
 5 files changed, 45 insertions(+), 26 deletions(-)
 rename frontend/src/business/components/track/plan/components/{TestCasePlanList.vue => TestPlanTestCaseList.vue} (84%)

diff --git a/frontend/src/business/components/track/case/TestCase.vue b/frontend/src/business/components/track/case/TestCase.vue
index cb36a94b39..9792f0a7c6 100644
--- a/frontend/src/business/components/track/case/TestCase.vue
+++ b/frontend/src/business/components/track/case/TestCase.vue
@@ -6,7 +6,7 @@
         <select-menu
           :data="projects"
           :current-data="currentProject"
-          :title="'项目'"
+          :title="$t('test_track.project')"
           @dataChange="changeProject">
         </select-menu>
 
@@ -184,7 +184,6 @@
 
 <style scoped>
   .case_container {
-    padding: 15px;
     width: 100%;
     height: 100%;
     background: white;
diff --git a/frontend/src/business/components/track/plan/TestPlanView.vue b/frontend/src/business/components/track/plan/TestPlanView.vue
index dbc0e74e95..9e4f8caa86 100644
--- a/frontend/src/business/components/track/plan/TestPlanView.vue
+++ b/frontend/src/business/components/track/plan/TestPlanView.vue
@@ -6,7 +6,7 @@
         <select-menu
           :data="testPlans"
           :current-data="currentPlan"
-          :title="'计划'"
+          :title="$t('test_track.plan')"
           @dataChange="changePlan">
         </select-menu>
 
@@ -20,12 +20,12 @@
       </el-aside>
 
       <el-main>
-        <test-case-plan-list
+        <test-plan-test-case-list
           @openTestCaseRelevanceDialog="openTestCaseRelevanceDialog"
           @editTestPlanTestCase="editTestPlanTestCase"
           @refresh="refresh"
           :plan-id="planId"
-          ref="testCasePlanList"></test-case-plan-list>
+          ref="testCasePlanList"></test-plan-test-case-list>
       </el-main>
     </el-container>
 
@@ -46,14 +46,14 @@
 <script>
 
     import PlanNodeTree from "./components/PlanNodeTree";
-    import TestCasePlanList from "./components/TestCasePlanList";
+    import TestPlanTestCaseList from "./components/TestPlanTestCaseList";
     import TestCaseRelevance from "./components/TestCaseRelevance";
     import TestPlanTestCaseEdit from "./components/TestPlanTestCaseEdit";
     import SelectMenu from "../common/SelectMenu";
 
     export default {
       name: "TestPlanView",
-      components: {PlanNodeTree, TestCasePlanList, TestCaseRelevance, TestPlanTestCaseEdit, SelectMenu},
+      components: {PlanNodeTree, TestPlanTestCaseList, TestCaseRelevance, TestPlanTestCaseEdit, SelectMenu},
       data() {
         return {
           testPlans: [],
diff --git a/frontend/src/business/components/track/plan/components/TestCasePlanList.vue b/frontend/src/business/components/track/plan/components/TestPlanTestCaseList.vue
similarity index 84%
rename from frontend/src/business/components/track/plan/components/TestCasePlanList.vue
rename to frontend/src/business/components/track/plan/components/TestPlanTestCaseList.vue
index 5783f1b235..37315b2bee 100644
--- a/frontend/src/business/components/track/plan/components/TestCasePlanList.vue
+++ b/frontend/src/business/components/track/plan/components/TestPlanTestCaseList.vue
@@ -10,7 +10,7 @@
             </el-col>
 
             <el-col :span="2" :offset="8">
-              <el-button icon="el-icon-circle-plus-outline" size="small" round
+              <el-button icon="el-icon-connection" size="small" round
                          @click="$emit('openTestCaseRelevanceDialog')" >{{$t('test_track.relevance_test_case')}}</el-button>
             </el-col>
 
@@ -75,11 +75,25 @@
           width="160"
           show-overflow-tooltip>
           <template v-slot:default="scope">
-            <span v-if="scope.row.status == 'Prepare'">{{$t('test_track.plan_status_prepare')}}</span>
-            <span v-if="scope.row.status == 'Pass'">{{$t('test_track.pass')}}</span>
-            <span v-if="scope.row.status == 'Failure'">{{$t('test_track.failure')}}</span>
-            <span v-if="scope.row.status == 'Blocking'">{{$t('test_track.blocking')}}</span>
-            <span v-if="scope.row.status == 'Skip'">{{$t('test_track.skip')}}</span>
+            <el-tag v-if="scope.row.status == 'Prepare'"
+                    e ffect="info"
+                    size="mini">{{$t('test_track.plan_status_prepare')}}</el-tag>
+            <el-tag v-if="scope.row.status == 'Pass'"
+                    type="success"
+                    effect="dark"
+                    size="mini">{{$t('test_track.pass')}}</el-tag>
+            <el-tag v-if="scope.row.status == 'Failure'"
+                    type="danger"
+                    effect="dark"
+                    size="mini">{{$t('test_track.failure')}}</el-tag>
+            <el-tag v-if="scope.row.status == 'Blocking'"
+                    type="warning"
+                    effect="dark"
+                    size="mini">{{$t('test_track.blocking')}}</el-tag>
+            <el-tag v-if="scope.row.status == 'Skip'"
+                    type="info"
+                    effect="dark"
+                    size="mini">{{$t('test_track.skip')}}</el-tag>
           </template>
         </el-table-column>
 
@@ -95,7 +109,7 @@
           :label="$t('commons.operating')">
           <template v-slot:default="scope">
             <el-button @click="handleEdit(scope.row)" type="primary" icon="el-icon-edit" size="mini" circle/>
-            <el-button @click="handleDelete(scope.row)" type="danger" icon="el-icon-delete" size="mini" circle/>
+            <el-button @click="handleDelete(scope.row)" type="danger" icon="el-icon-unlock" size="mini" circle/>
           </template>
         </el-table-column>
       </el-table>
@@ -126,7 +140,7 @@
   import PlanNodeTree from './PlanNodeTree';
 
   export default {
-      name: "TestCaseList",
+      name: "TestPlanTestCaseList",
       components: {PlanNodeTree},
       data() {
         return {
@@ -192,7 +206,7 @@
           this.$emit('editTestPlanTestCase', testCase);
         },
         handleDelete(testCase) {
-          this.$alert(this.$t('load_test.delete_confirm') + testCase.name + "？", '', {
+          this.$alert(this.$t('test_track.confirm_cancel_relevance') + ' ' + testCase.name + " ？", '', {
             confirmButtonText: this.$t('commons.confirm'),
             callback: (action) => {
               if (action === 'confirm') {
diff --git a/frontend/src/i18n/en-US.js b/frontend/src/i18n/en-US.js
index 5b1885e916..b5449bc32c 100644
--- a/frontend/src/i18n/en-US.js
+++ b/frontend/src/i18n/en-US.js
@@ -41,10 +41,10 @@ export default {
     'api': 'Api',
     'performance': 'Performance',
     'input_content': 'Please enter content',
-    'create': 'create',
-    'refresh': 'refresh',
-    'remark': 'remark',
-    'delete': 'delete',
+    'create': 'Create',
+    'refresh': 'Refresh',
+    'remark': 'Remark',
+    'delete': 'Delete',
     'not_filled': 'Not filled'
   },
   workspace: {
@@ -201,7 +201,7 @@ export default {
     'input_method': 'Please select method',
     'input_prerequisite': 'Please select prerequisite',
     'search_module': 'Search module',
-    'rename': 'rename',
+    'rename': 'Rename',
     'add_submodule': 'Add submodule',
     'add_module': 'Add module',
     'module_name': 'Module name',
@@ -209,17 +209,17 @@ export default {
     'plan_name': 'Test plan name',
     'plan_project': 'Related project',
     'plan_stage': 'Testing phase',
-    'plan_status': 'The current state',
+    'plan_status': 'Status',
     'smoke_test': 'Smoke test',
     'functional_test': 'Functional test',
     'integration_testing': 'Integration testing',
     'system_test': 'System test',
     'version_validation': 'Version validation',
-    'plan_principal': 'principal',
+    'plan_principal': 'Principal',
     'input_plan_principal': 'Please select principal',
     'input_plan_project': 'Please select project',
     'input_plan_stage': 'Please select stage',
-    'plan_status_prepare': ' Not started',
+    'plan_status_prepare': 'Not started',
     'plan_status_running': 'Starting',
     'plan_status_completed': 'Completed',
     'delete_module_confirm': 'Confirm delete module: ',
@@ -235,7 +235,10 @@ export default {
     'actual_result': 'Actual result',
     'step_result': 'Step result',
     'select_execute_result': 'Select execute result',
-    'save': 'Save'
+    'save': 'Save',
+    'confirm_cancel_relevance': 'Confirm cancel relevance',
+    'plan': 'Plan',
+    'project': 'Project',
   },
   test_resource_pool: {
     'type': 'type',
diff --git a/frontend/src/i18n/zh-CN.js b/frontend/src/i18n/zh-CN.js
index 5193913a10..eeda592b4b 100644
--- a/frontend/src/i18n/zh-CN.js
+++ b/frontend/src/i18n/zh-CN.js
@@ -235,7 +235,10 @@ export default {
     'actual_result': '实际结果',
     'step_result': '步骤执行结果',
     'select_execute_result': '选择执行结果',
-    'save': '保 存'
+    'save': '保 存',
+    'confirm_cancel_relevance': '确认取消关联',
+    'plan': '计划',
+    'project': '项目',
   },
   test_resource_pool: {
     'type': '类型',