From 79b8cee2513e30f68cf694474cecd2b42d31bb65 Mon Sep 17 00:00:00 2001
From: Oliver Hamlet <oliver.hamlet@gmail.com>
Date: Tue, 29 Dec 2015 19:13:34 +0000
Subject: [PATCH] Refactor Plugin change event handlers

---
 src/gui/html/js/events.js | 14 --------------
 src/gui/html/js/init.js   |  4 ++--
 src/gui/html/js/plugin.js | 14 ++++++++++++++
 3 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/src/gui/html/js/events.js b/src/gui/html/js/events.js
index cf05721c..f96f163c 100644
--- a/src/gui/html/js/events.js
+++ b/src/gui/html/js/events.js
@@ -1,18 +1,4 @@
 'use strict';
-function onPluginMessageChange(evt) {
-  document.getElementById('filterTotalMessageNo').textContent = parseInt(document.getElementById('filterTotalMessageNo').textContent, 10) + evt.detail.totalDiff;
-  document.getElementById('totalMessageNo').textContent = parseInt(document.getElementById('totalMessageNo').textContent, 10) + evt.detail.totalDiff;
-  document.getElementById('totalWarningNo').textContent = parseInt(document.getElementById('totalWarningNo').textContent, 10) + evt.detail.warningDiff;
-  document.getElementById('totalErrorNo').textContent = parseInt(document.getElementById('totalErrorNo').textContent, 10) + evt.detail.errorDiff;
-}
-function onPluginIsDirtyChange(evt) {
-  if (evt.detail.isDirty) {
-    document.getElementById('dirtyPluginNo').textContent = parseInt(document.getElementById('dirtyPluginNo').textContent, 10) + 1;
-  } else {
-    document.getElementById('dirtyPluginNo').textContent = parseInt(document.getElementById('dirtyPluginNo').textContent, 10) - 1;
-  }
-}
-
 function saveFilterState(evt) {
   loot.query('saveFilterState', evt.target.id, evt.target.checked).catch(handlePromiseError);
 }
diff --git a/src/gui/html/js/init.js b/src/gui/html/js/init.js
index 321805e2..68a5179a 100644
--- a/src/gui/html/js/init.js
+++ b/src/gui/html/js/init.js
@@ -93,8 +93,8 @@
     document.getElementById('cardsNav').addEventListener('dblclick', onSidebarClick);
 
     /* Set up handler for plugin message and dirty info changes. */
-    document.addEventListener('loot-plugin-message-change', onPluginMessageChange);
-    document.addEventListener('loot-plugin-isdirty-change', onPluginIsDirtyChange);
+    document.addEventListener('loot-plugin-message-change', Plugin.onMessageChange);
+    document.addEventListener('loot-plugin-isdirty-change', Plugin.onIsDirtyChange);
 
     /* Set up event handlers for game member variable changes. */
     document.addEventListener('loot-game-folder-change', Game.onFolderChange);
diff --git a/src/gui/html/js/plugin.js b/src/gui/html/js/plugin.js
index 875a85c0..6e740999 100644
--- a/src/gui/html/js/plugin.js
+++ b/src/gui/html/js/plugin.js
@@ -306,5 +306,19 @@
     getCardContent(filters) {
       return new PluginCardContent(this, filters);
     }
+
+    static onMessageChange(evt) {
+      document.getElementById('filterTotalMessageNo').textContent = parseInt(document.getElementById('filterTotalMessageNo').textContent, 10) + evt.detail.totalDiff;
+      document.getElementById('totalMessageNo').textContent = parseInt(document.getElementById('totalMessageNo').textContent, 10) + evt.detail.totalDiff;
+      document.getElementById('totalWarningNo').textContent = parseInt(document.getElementById('totalWarningNo').textContent, 10) + evt.detail.warningDiff;
+      document.getElementById('totalErrorNo').textContent = parseInt(document.getElementById('totalErrorNo').textContent, 10) + evt.detail.errorDiff;
+    }
+    static onIsDirtyChange(evt) {
+      if (evt.detail.isDirty) {
+        document.getElementById('dirtyPluginNo').textContent = parseInt(document.getElementById('dirtyPluginNo').textContent, 10) + 1;
+      } else {
+        document.getElementById('dirtyPluginNo').textContent = parseInt(document.getElementById('dirtyPluginNo').textContent, 10) - 1;
+      }
+    }
   };
 }));