refactor alert reducer to be functional

Author: ShMcK

lib/reducers/alert/index.js

@@ -10,13 +10,16 @@ var open = {
     action: 'pass',
 };
 var current = _alert;
-function setAlert(options) {
+function setAlert(options, color) {
+    if (color) {
+        var statusBarAlert = document.getElementsByClassName('cr-alert-replay')[0];
+        statusBarAlert.style.color = color;
+    }
     current = Object.assign({}, open, options);
     return current;
 }
 function alertReducer(alert, action) {
     if (alert === void 0) { alert = _alert; }
-    var statusBarAlert = document.getElementsByClassName('cr-alert-replay')[0];
     switch (action.type) {
         case _types_1.ALERT_REPLAY:
             return setAlert(current);
@@ -32,26 +35,24 @@ function alertReducer(alert, action) {
             var result = action.payload.result;
             switch (true) {
                 case result.pass && result.change > 0:
-                    statusBarAlert.style.color = '#73C990';
                     return setAlert({
                         message: result.msg,
                         duration: result.duration || 1500,
-                    });
+                    }, '#73C990');
                 case result.pass === false && result.change < 1:
-                    statusBarAlert.style.color = '#FF4081';
                     return setAlert({
                         message: result.msg,
                         action: 'fail',
                         duration: result.duration || 2500,
-                    });
+                    }, '#FF4081');
                 default:
-                    statusBarAlert.style.color = '#9DA5B4';
+                    break;
             }
             return setAlert({
                 message: result.msg,
                 action: 'note',
                 duration: result.duration || 2500,
-            });
+            }, '#9DA5B4');
         case _types_1.COMPLETE_PAGE:
             return setAlert({
                 message: "Page " + (action.payload.pagePosition + 1) + " Complete",

src/reducers/alert/index.ts

@@ -16,15 +16,18 @@ const open = {
 
 let current: CR.Alert = _alert;
 
-function setAlert(options) {
+function setAlert(options: Object, color?: string) {
+  if (color) {
+    let statusBarAlert = <HTMLElement>document.getElementsByClassName('cr-alert-replay')[0];
+    statusBarAlert.style.color = color;
+  }
   current = Object.assign({}, open, options);
   return current;
 }
 
 export default function alertReducer(
   alert = _alert, action: Action
 ): CR.Alert {
-  let statusBarAlert = <HTMLElement>document.getElementsByClassName('cr-alert-replay')[0];
   switch (action.type) {
 
     case ALERT_REPLAY:
@@ -46,28 +49,26 @@ export default function alertReducer(
       switch (true) {
         // pass
         case result.pass && result.change > 0:
-          statusBarAlert.style.color = '#73C990';
           return setAlert({
             message: result.msg,
             duration: result.duration || 1500,
-          });
+          }, '#73C990');
         // Fail
         case result.pass === false && result.change < 1:
-          statusBarAlert.style.color = '#FF4081';
           return setAlert({
             message: result.msg,
             action: 'fail',
             duration: result.duration || 2500,
-          });
+          }, '#FF4081');
         // Alert
         default:
-          statusBarAlert.style.color = '#9DA5B4';
+          break;
       }
       return setAlert({
         message: result.msg,
         action: 'note',
         duration: result.duration || 2500,
-      });
+      }, '#9DA5B4');
 
     case COMPLETE_PAGE:
       return setAlert({