From 8a11c23d428f20d7799b4c0140f9eff7a9989e06 Mon Sep 17 00:00:00 2001
From: Yan Yifeng <yanyifeng@splayer.org>
Date: Tue, 16 Jul 2019 16:26:04 +0800
Subject: [PATCH] refactor(subtitle): turn calculatedNoSub to new getter

1. Remove calculatedNoSub mutation and action from legacy vuex.
2. Add calculatedNoSub as new vuex getter.
---
 src/renderer/components/PlayingView/SubtitleControl.vue | 4 ----
 src/renderer/store/actionTypes.js                       | 1 -
 src/renderer/store/modules/Subtitle/index.js            | 8 --------
 src/renderer/store/modules/SubtitleManager.ts           | 1 +
 src/renderer/store/mutationTypes.js                     | 1 -
 5 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/src/renderer/components/PlayingView/SubtitleControl.vue b/src/renderer/components/PlayingView/SubtitleControl.vue
index e75c482f1b..8e7138bd56 100644
--- a/src/renderer/components/PlayingView/SubtitleControl.vue
+++ b/src/renderer/components/PlayingView/SubtitleControl.vue
@@ -210,9 +210,6 @@ export default {
     enabledSecondarySub(val: boolean) {
       if (!val) this.updateSubtitleType(true);
     },
-    computedAvailableItems(val: SubtitleControlListItem[]) {
-      this.updateNoSubtitle(!val.length);
-    },
     list(val: SubtitleControlListItem[]) {
       this.computedAvailableItems = val.map((sub: SubtitleControlListItem) => ({
         ...sub,
@@ -348,7 +345,6 @@ export default {
       changeSecondarySubtitle: smActions.changeSecondarySubtitle,
       refreshSubtitles: smActions.refreshSubtitles,
       deleteCurrentSubtitle: smActions.deleteSubtitlesByUuid,
-      updateNoSubtitle: subtitleActions.UPDATE_NO_SUBTITLE,
       updateSubtitleType: subtitleActions.UPDATE_SUBTITLE_TYPE,
     }),
     offCurrentSubtitle() {
diff --git a/src/renderer/store/actionTypes.js b/src/renderer/store/actionTypes.js
index 7b50063bc7..2638707cdc 100644
--- a/src/renderer/store/actionTypes.js
+++ b/src/renderer/store/actionTypes.js
@@ -41,7 +41,6 @@ export const Subtitle = {
   UPDATE_SUBTITLE_STYLE: 'UPDATE_SUBTITLE_STYLE',
   UPDATE_SUBTITLE_SIZE: 'UPDATE_SUBTITLE_SIZE',
   UPDATE_LAST_SUBTITLE_SIZE: 'UPDATE_LAST_SUBTITLE_SIZE',
-  UPDATE_NO_SUBTITLE: 'UPDATE_NO_SUBTITLE',
   UPDATE_SUBTITLE_TOP: 'UPDATE_SUBTITLE_TOP',
   REMOVE_LOCAL_SUBTITLE: 'REMOVE_LOCAL_SUBTITLE',
   UPDATE_SUBTITLE_TYPE: 'UPDATE_SUBTITLE_TYPE',
diff --git a/src/renderer/store/modules/Subtitle/index.js b/src/renderer/store/modules/Subtitle/index.js
index b6b451b183..54f20d4c6e 100644
--- a/src/renderer/store/modules/Subtitle/index.js
+++ b/src/renderer/store/modules/Subtitle/index.js
@@ -25,7 +25,6 @@ const state = {
   lastChosenSize: 1,
   subtitleDelay: 0,
   scaleNum: 1,
-  calculatedNoSub: true,
   subToTop: false,
   isFirstSubtitle: true,
   isPrimarySubSettings: true,
@@ -76,7 +75,6 @@ const getters = {
   chosenSize: state => state.chosenSize,
   lastChosenSize: state => state.lastChosenSize,
   scaleNum: state => state.scaleNum,
-  calculatedNoSub: state => state.calculatedNoSub,
   subToTop: state => state.subToTop,
   isFirstSubtitle: state => state.isFirstSubtitle,
   enabledSecondarySub: state => state.enabledSecondarySub,
@@ -150,9 +148,6 @@ const mutations = {
   [subtitleMutations.SUBTITLE_SIZE_UPDATE](state, payload) {
     state.chosenSize = payload;
   },
-  [subtitleMutations.NO_SUBTITLE_UPDATE](state, payload) {
-    state.calculatedNoSub = payload;
-  },
   [subtitleMutations.SUBTITLE_TOP_UPDATE](state, payload) {
     state.subToTop = payload;
   },
@@ -290,9 +285,6 @@ const actions = {
   [subtitleActions.UPDATE_SUBTITLE_SIZE]({ commit }, delta) {
     commit(subtitleMutations.SUBTITLE_SIZE_UPDATE, delta);
   },
-  [subtitleActions.UPDATE_NO_SUBTITLE]({ commit }, delta) {
-    commit(subtitleMutations.NO_SUBTITLE_UPDATE, delta);
-  },
   [subtitleActions.UPDATE_SUBTITLE_TOP]({ commit }, delta) {
     commit(subtitleMutations.SUBTITLE_TOP_UPDATE, delta);
   },
diff --git a/src/renderer/store/modules/SubtitleManager.ts b/src/renderer/store/modules/SubtitleManager.ts
index ea51b6e634..91d5d94965 100644
--- a/src/renderer/store/modules/SubtitleManager.ts
+++ b/src/renderer/store/modules/SubtitleManager.ts
@@ -70,6 +70,7 @@ const getters = {
   },
   primaryDelay({ primaryDelay }: SubtitleManagerState) { return primaryDelay; },
   secondaryDelay({ secondaryDelay }: SubtitleManagerState) { return secondaryDelay; },
+  calculatedNoSub(state: any, { list }: any) { return !list.length; },
 };
 const mutations = {
   [m.setPlaylistId](state: SubtitleManagerState, id: number) {
diff --git a/src/renderer/store/mutationTypes.js b/src/renderer/store/mutationTypes.js
index 0a5dd8a1f7..f3fe6b8e86 100644
--- a/src/renderer/store/mutationTypes.js
+++ b/src/renderer/store/mutationTypes.js
@@ -64,7 +64,6 @@ export const Subtitle = {
   SUBTITLE_STYLE_UPDATE: 'SUBTITLE_STYLE_UPDATE',
   SUBTITLE_SIZE_UPDATE: 'SUBTITLE_SIZE_UPDATE',
   LAST_SUBTITLE_SIZE_UPDATE: 'LAST_SUBTITLE_SIZE_UPDATE',
-  NO_SUBTITLE_UPDATE: 'NO_SUBTITLE_UPDATE',
   SUBTITLE_TOP_UPDATE: 'SUBTITLE_TOP_UPDATE',
   CURRENT_SUBTITLE_REMOVE: 'CURRENT_SUBTITLE_REMOVE',
   SUBTITLE_TYPE_UPDATE: 'SUBTITLE_TYPE_UPDATE',