From c20f5041f83dcdc0ebca8fb9f38c417d3e4febae Mon Sep 17 00:00:00 2001
From: marvzhang <tikazyq@163.com>
Date: Thu, 16 Apr 2020 13:54:37 +0800
Subject: [PATCH] =?UTF-8?q?=E4=BC=98=E5=8C=96=E6=97=A5=E5=BF=97?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/components/ScrollView/LogView.vue     | 67 +++++++++----------
 frontend/src/store/modules/task.js            | 11 ++-
 frontend/src/utils/log.js                     |  6 +-
 3 files changed, 46 insertions(+), 38 deletions(-)

diff --git a/frontend/src/components/ScrollView/LogView.vue b/frontend/src/components/ScrollView/LogView.vue
index c0754d13..97882b96 100644
--- a/frontend/src/components/ScrollView/LogView.vue
+++ b/frontend/src/components/ScrollView/LogView.vue
@@ -2,16 +2,12 @@
   <div class="log-view-container">
     <div class="filter-wrapper">
       <div class="left">
-        <el-button
-          size="small"
-          type="primary"
-          icon="el-icon-download"
+        <el-switch
+          v-model="isLogAutoScroll"
+          :inactive-text="$t('Auto-Scroll')"
           style="margin-right: 10px"
-          :disabled="isToBottom"
-          @click="onAutoScroll"
         >
-          {{$t('Auto-Scroll')}}
-        </el-button>
+        </el-switch>
         <el-input
           v-model="logKeyword"
           size="small"
@@ -85,9 +81,6 @@
             :class="currentLogIndex === item.index ? 'active' : ''"
             @click="onClickError(item)"
           >
-<!--            <span class="line-no">-->
-<!--              {{item.index}}-->
-<!--            </span>-->
             <span class="line-content">
               {{item.msg}}
             </span>
@@ -125,7 +118,6 @@ export default {
     return {
       item: LogItem,
       searchString: '',
-      isToBottom: false,
       isScrolling: false,
       isScrolling2nd: false,
       errorRegex: this.$utils.log.errorRegex,
@@ -137,7 +129,8 @@ export default {
   computed: {
     ...mapState('task', [
       'taskForm',
-      'taskLogTotal'
+      'taskLogTotal',
+      'logKeyword'
     ]),
     ...mapGetters('task', [
       'logData',
@@ -175,6 +168,14 @@ export default {
         this.$store.commit('task/SET_TASK_LOG_PAGE_SIZE', value)
       }
     },
+    isLogAutoScroll: {
+      get () {
+        return this.$store.state.task.isLogAutoScroll
+      },
+      set (value) {
+        this.$store.commit('task/SET_IS_LOG_AUTO_SCROLL', value)
+      }
+    },
     filteredLogData () {
       return this.logData.filter(d => {
         if (!this.searchString) return true
@@ -194,6 +195,19 @@ export default {
     taskLogPageSize () {
       this.$emit('search')
       this.$st.sendEv('任务详情', '日志', '改变日志每页条数')
+    },
+    isLogAutoScroll () {
+      if (this.isLogAutoScroll) {
+        this.$store.dispatch('task/getTaskLog', {
+          id: this.$route.params.id,
+          keyword: this.logKeyword
+        }).then(() => {
+          this.toBottom()
+        })
+        this.$st.sendEv('任务详情', '日志', '点击自动滚动')
+      } else {
+        this.$st.sendEv('任务详情', '日志', '取消自动滚动')
+      }
     }
   },
   methods: {
@@ -214,33 +228,14 @@ export default {
       }
     },
     onToBottom () {
-      if (this.isScrolling) return
-      this.isToBottom = true
     },
     onScroll () {
-      if (this.isScrolling2nd) {
-        this.isToBottom = false
-      }
-      this.isScrolling = true
-      setTimeout(() => {
-        this.isScrolling2nd = true
-        setTimeout(() => {
-          this.isScrolling2nd = false
-        }, 50)
-      }, 50)
-      setTimeout(() => {
-        this.isScrolling = false
-      }, 100)
     },
     toBottom () {
       this.$el.querySelector('.log-view').scrollTo({ top: 99999999 })
-      setTimeout(() => {
-        this.isToBottom = true
-      }, 50)
     },
     onAutoScroll () {
-      this.toBottom()
-      this.$st.sendEv('任务详情', '日志', '点击自动滚动')
+
     },
     toggleErrors () {
       this.isErrorsCollapsed = !this.isErrorsCollapsed
@@ -251,9 +246,9 @@ export default {
     },
     onClickError (item) {
       this.currentLogIndex = item.index
-      this.isToBottom = false
+      this.isLogAutoScroll = false
       const handle = setInterval(() => {
-        this.isToBottom = false
+        this.isLogAutoScroll = false
       }, 10)
       setTimeout(() => {
         clearInterval(handle)
@@ -267,7 +262,7 @@ export default {
   mounted () {
     this.currentLogIndex = 0
     this.handle = setInterval(() => {
-      if (this.isToBottom) {
+      if (this.isLogAutoScroll) {
         this.toBottom()
       }
     }, 200)
diff --git a/frontend/src/store/modules/task.js b/frontend/src/store/modules/task.js
index dfb8e350..3fe46dd7 100644
--- a/frontend/src/store/modules/task.js
+++ b/frontend/src/store/modules/task.js
@@ -11,6 +11,7 @@ const state = {
   taskLogPage: 1,
   taskLogPageSize: 5000,
   currentLogIndex: 0,
+  isLogAutoScroll: false,
   taskResultsData: [],
   taskResultsColumns: [],
   taskResultsTotalCount: 0,
@@ -129,6 +130,9 @@ const mutations = {
   },
   SET_TASK_LOG_PAGE_SIZE (state, value) {
     state.taskLogPageSize = value
+  },
+  SET_IS_LOG_AUTO_SCROLL (state, value) {
+    state.isLogAutoScroll = value
   }
 }
 
@@ -173,7 +177,7 @@ const actions = {
         dispatch('getTaskList')
       })
   },
-  getTaskLog ({ state, commit }, { id, keyword }) {
+  getTaskLog ({ state, commit }, { id, keyword, isAutoScrolling }) {
     return request.get(`/tasks/${id}/log`, {
       keyword,
       page_num: state.taskLogPage,
@@ -182,6 +186,11 @@ const actions = {
       .then(response => {
         commit('SET_TASK_LOG', response.data.data || [])
         commit('SET_TASK_LOG_TOTAL', response.data.total || 0)
+
+        // auto switch to next page if not reaching the end
+        if (state.isLogAutoScroll && state.taskLogTotal > (state.taskLogPage * state.taskLogPageSize)) {
+          commit('SET_TASK_LOG_PAGE', Math.ceil(state.taskLogTotal / state.taskLogPageSize))
+        }
       })
   },
   getTaskErrorLog ({ state, commit }, id) {
diff --git a/frontend/src/utils/log.js b/frontend/src/utils/log.js
index 8b1ea0ab..23f9ff64 100644
--- a/frontend/src/utils/log.js
+++ b/frontend/src/utils/log.js
@@ -1,5 +1,9 @@
 const regexToken = ' :,.'
 
 export default {
-  errorRegex: new RegExp(`(?:[${regexToken}]|^)((?:error|exception|traceback)s?)(?:[${regexToken}]|$)`, 'gi')
+  // errorRegex: new RegExp(`(?:[${regexToken}]|^)((?:error|exception|traceback)s?)(?:[${regexToken}]|$)`, 'gi')
+  errorRegex: new RegExp(`(?:${regexToken})(error)(?:${regexToken})`, 'gi'),
+  errorWhitelist: [
+    'log_count/ERROR'
+  ]
 }