amarkal / amarkal-admin-notification

Notifications.php

Braces +14 added lines, -5 removed lines

@@ -34,7 +34,9 @@ 
         add_action( 'wp_ajax_dismiss_admin_notification', array( $this, 'dismiss_notification' ) );
         add_action( 'admin_footer', array( $this, 'render_script' ) ); // Must be hooked to a late action hook
         $this->dismissed_notices = get_option('wp_dismissed_notices');
-        if( false === $this->dismissed_notices ) $this->dismissed_notices = array();
+        if( false === $this->dismissed_notices ) {
+            $this->dismissed_notices = array();
+        }
     }
 
     public function register_notification( $handle, $options )
@@ -42,8 +44,9 @@ 
         if( !key_exists( $handle, $this->notifications ) )
         {
             $this->notifications[$handle] = $options;
+        } else {
+            trigger_error( "The handle <strong>$handle</strong> has already been registered. Please choose a different handle for your notification." );
         }
-        else trigger_error( "The handle <strong>$handle</strong> has already been registered. Please choose a different handle for your notification." );
     }
 
     public function render_notifications()
@@ -58,7 +61,9 @@ 
     {
         foreach( $this->notifications as $handle => $notification )
         {
-            if( $notification['network'] ) $this->render_notification( $handle, $notification );
+            if( $notification['network'] ) {
+                $this->render_notification( $handle, $notification );
+            }
         }
     }
 
@@ -75,7 +80,9 @@ 
 
     public function render_script()
     {
-        if( 0 === count( $this->notifications ) ) return;
+        if( 0 === count( $this->notifications ) ) {
+            return;
+        }
         ?>
         <script>
         jQuery(document).ready(function($){
@@ -92,7 +99,9 @@ 
 
     private function render_notification( $id, $n )
     {
-        if( in_array( $id, $this->dismissed_notices ) ) return;
+        if( in_array( $id, $this->dismissed_notices ) ) {
+            return;
+        }
 
         printf( 
             '<div id="%s" class="notice notice-%s %s%s"><p>%s</p></div>',