From 42b02e422f953a3de739d20b781996224abd5213 Mon Sep 17 00:00:00 2001
From: chengwang15 <66863386+chengwang15@users.noreply.github.com>
Date: Mon, 28 Feb 2022 20:30:05 +0800
Subject: [PATCH] #98 clear logMessageList & resultList after toggling menu
 (#101)

---
 src/components/Header/HeaderMenu.vue             |  7 ++++++-
 src/page/Workspace/Notebook/LogMessage/index.vue | 10 +++++++---
 src/store/module/notebook.js                     |  4 ++++
 src/store/type.js                                |  1 +
 4 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/src/components/Header/HeaderMenu.vue b/src/components/Header/HeaderMenu.vue
index d06fe05..dcdaa9c 100644
--- a/src/components/Header/HeaderMenu.vue
+++ b/src/components/Header/HeaderMenu.vue
@@ -34,7 +34,8 @@
   </el-menu>
 </template>
 <script>
-import { mapState } from 'vuex'
+import { mapMutations, mapState } from 'vuex'
+import { actionsTypes } from '../../store'
 export default {
   data () {
     return {}
@@ -48,7 +49,11 @@ export default {
     }
   },
   methods: {
+    ...mapMutations ({
+      resetList: actionsTypes.RESET_LIST
+    }),
     handleSelect (key) {
+      this.resetList()
       this.$router.push({name: key})
     }
   }
diff --git a/src/page/Workspace/Notebook/LogMessage/index.vue b/src/page/Workspace/Notebook/LogMessage/index.vue
index 51e1425..49bd98b 100644
--- a/src/page/Workspace/Notebook/LogMessage/index.vue
+++ b/src/page/Workspace/Notebook/LogMessage/index.vue
@@ -31,7 +31,7 @@ import { mapState, mapMutations, mapActions } from 'vuex'
   }
 })
 
-export default class ExcuteDetail extends Vue {
+export default class LogMessage extends Vue {
   logList = []
   timer = null
 
@@ -51,8 +51,7 @@ export default class ExcuteDetail extends Vue {
   @Watch('currentNotebook', { immediate: true, deep: true })
   onCurrentNotebookChange (newVal) {
     if (newVal && newVal.active === 'true' && !this.logMessageList.includes(this.cellId)) {
-      this.addLogMessage(this.cellId)
-      this.getLogs()
+      this.initLog()
     }
   }
 
@@ -60,6 +59,11 @@ export default class ExcuteDetail extends Vue {
     window.clearTimeout(this.timer)
   }
 
+  initLog () {
+    this.addLogMessage(this.cellId)
+    this.getLogs()
+  }
+
   pollingData () {
     window.clearTimeout(this.timer)
     if (this._isDestroyed) {
diff --git a/src/store/module/notebook.js b/src/store/module/notebook.js
index f238893..4f71b06 100644
--- a/src/store/module/notebook.js
+++ b/src/store/module/notebook.js
@@ -29,6 +29,10 @@ export default {
     [types.SET_LOG_MESSAGE_LIST]: (state, cellId) => {
       state.logMessageList.push(cellId)
     },
+    [types.RESET_LIST]: state => {
+      state.logMessageList = []
+      state.resultList = []
+    },
     [types.SET_RESULT_LIST]: (state, cellId) => {
       state.resultList.push(cellId)
     },
diff --git a/src/store/type.js b/src/store/type.js
index 2a529eb..04a4e34 100644
--- a/src/store/type.js
+++ b/src/store/type.js
@@ -31,6 +31,7 @@ export const SET_OPENED_NOTEBOOK = 'SET_OPENED_NOTEBOOK'
 export const SET_ACTIVE_NOTEBOOK = 'SET_ACTIVE_NOTEBOOK'
 export const SET_DEMO_LIST = 'SET_DEMO_LIST'
 export const SET_LOG_MESSAGE_LIST = 'SET_LOG_MESSAGE_LIST'
+export const RESET_LIST = 'RESET_LIST'
 export const SET_RESULT_LIST = 'SET_RESULT_LIST'
 export const CREATE_FOLDER = 'CREATE_FOLDER'
 export const MOVE_NOTEBOOK = 'MOVE_NOTEBOOK'