Clean up code.

Removing test of notifications.

diff --git a/models/admin/settings/notificationTypes.php b/models/admin/settings/notificationTypes.php
index 8c25dd8..b596bd4 100644 (file)
--- a/models/admin/settings/notificationTypes.php
+++ b/models/admin/settings/notificationTypes.php
@@ -143,26 +143,6 @@ class GlmMembersAdmin_settings_notificationTypes extends GlmDataNotificationType
         }
 
         switch( $option ) {
-        case 'test': // TODO: remove this case (only for testing notifications)
-            require_once GLM_MEMBERS_BILLING_PLUGIN_CLASS_PATH .  '/notifications.php';
-            $Notifications = new GlmNotifications( $this->wpdb, $this->config );
-            /**
-             * test:
-             *
-             * Test the notification email.
-             * Currently this is just using a hard coded account id  of 1.
-             * After the test it will call wp_die
-             */
-
-            $notification_id = filter_var( $_REQUEST['id'], FILTER_VALIDATE_INT );
-            if ( $notification_id ) {
-                echo '<pre>$notification_id: ' . print_r( $notification_id, true ) . '</pre>';
-                $account = 1; // TODO: remove this. It's for testing only
-                $noticeReturned = $Notifications->sendEmailNotification( $notification_id, $account );
-                // var_dump( $noticeReturned );
-            }
-            wp_die('Testing here');
-            break;
 
         case 'add':
             /**

diff --git a/views/admin/settings/editNotificationType.html b/views/admin/settings/editNotificationType.html
index 59154e0..2eae49d 100644 (file)
--- a/views/admin/settings/editNotificationType.html
+++ b/views/admin/settings/editNotificationType.html
@@ -165,6 +165,7 @@
 
 <script>
 jQuery(document).ready(function($){
+
     // Flash certain elements for a short time after display
     $(".glm-flash-updated").fadeOut(500).fadeIn(500).fadeOut(500).fadeIn(500).fadeOut(500).fadeIn(500).fadeOut(500).fadeIn(500).fadeOut(500);