amarkal / amarkal-admin-notification

bootstrap.php

Braces +6 added lines, -2 removed lines

@@ -15,7 +15,9 @@ 
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'WP_ADMIN_NOTIFICATION' ) ) return;
+if( defined( 'WP_ADMIN_NOTIFICATION' ) ) {
+    return;
+}
 define( 'WP_ADMIN_NOTIFICATION', true );
 
 /**
@@ -59,7 +61,9 @@ 
         $offset = 0;
         foreach($dismissed as $id)
         {
-            if( $id === $handle ) break;
+            if( $id === $handle ) {
+                break;
+            }
             $offset++;
         }
         array_splice( $dismissed, $offset, 1);

handler.php

Braces +11 added lines, -4 removed lines

@@ -37,8 +37,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()
@@ -53,7 +54,9 @@ 
         {
             foreach( $this->notifications as $handle => $notification )
             {
-                if( $notification['network'] ) $this->render_notification( $handle, $notification );
+                if( $notification['network'] ) {
+                    $this->render_notification( $handle, $notification );
+                }
             }
         }
         
@@ -91,13 +94,17 @@ 
             add_action( 'wp_ajax_dismiss_admin_notification', array( $this, 'dismiss_notification' ) );
             add_action( 'admin_head', array( $this, 'render_script' ) );
             $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();
+            }
             var_dump($this->dismissed_notices);
         }
         
         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>',