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);

Spacing +14 added lines, -14 removed lines

@@ -10,29 +10,29 @@ 
  * @copyright 2016 Askupa Software
  */
 
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'WP_ADMIN_NOTIFICATION' ) ) return;
-define( 'WP_ADMIN_NOTIFICATION', true );
+if (defined('WP_ADMIN_NOTIFICATION')) return;
+define('WP_ADMIN_NOTIFICATION', true);
 
 /**
  * Require the handler class and initiate it:
  */
 require_once 'handler.php';
-if(!function_exists('_wp_admin_notification_init'))
+if (!function_exists('_wp_admin_notification_init'))
 {
     function _wp_admin_notification_init() 
     {
         $notifier = WPAdminNotifications::get_instance();
         $notifier->init();
     }
-    add_action( 'init', '_wp_admin_notification_init' );
+    add_action('init', '_wp_admin_notification_init');
 }
 
-if(!function_exists('wp_admin_notification'))
+if (!function_exists('wp_admin_notification'))
 {
     /**
      * Register an admin notification.
@@ -44,7 +44,7 @@ 
      * @param type $class
      * @param type $network
      */
-    function wp_admin_notification( $handle, $html, $type = 'success', $dismissible = false, $class = '', $network = false )
+    function wp_admin_notification($handle, $html, $type = 'success', $dismissible = false, $class = '', $network = false)
     {
         $notifier = WPAdminNotifications::get_instance();
         $notifier->register_notification($handle, array(
@@ -57,22 +57,22 @@ 
     }
 }
 
-if(!function_exists('wp_reset_admin_notification'))
+if (!function_exists('wp_reset_admin_notification'))
 {
     /**
      * 
      * @param string $handle
      */
-    function wp_reset_admin_notification( $handle )
+    function wp_reset_admin_notification($handle)
     {
-        $dismissed = get_option( 'wp_dismissed_notices' );
+        $dismissed = get_option('wp_dismissed_notices');
         $offset = 0;
-        foreach($dismissed as $id)
+        foreach ($dismissed as $id)
         {
-            if( $id === $handle ) break;
+            if ($id === $handle) break;
             $offset++;
         }
-        array_splice( $dismissed, $offset, 1);
-        update_option( 'wp_dismissed_notices', $dismissed );
+        array_splice($dismissed, $offset, 1);
+        update_option('wp_dismissed_notices', $dismissed);
     }
 }
\ No newline at end of file

handler.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>',

Spacing +21 added lines, -21 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if(!class_exists('WPAdminNotifications'))
+if (!class_exists('WPAdminNotifications'))
 {
     class WPAdminNotifications
     {
@@ -20,7 +20,7 @@ 
          */
         public static function get_instance()
         {
-            if( null === static::$instance ) 
+            if (null === static::$instance) 
             {
                 static::$instance = new static();
             }
@@ -29,53 +29,53 @@ 
         
         public function init()
         {
-            add_action( 'admin_notices', array( $this, 'render_notifications' ) );
-            add_action( 'network_admin_notices', array( $this, 'render_network_notifications' ) );
-            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
+            add_action('admin_notices', array($this, 'render_notifications'));
+            add_action('network_admin_notices', array($this, 'render_network_notifications'));
+            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 )
+        public function register_notification($handle, $options)
         {
-            if( !key_exists( $handle, $this->notifications ) )
+            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()
         {
-            foreach($this->notifications as $handle => $notification)
+            foreach ($this->notifications as $handle => $notification)
             {
-                $this->render_notification( $handle, $notification );
+                $this->render_notification($handle, $notification);
             }
         }
         
         public function render_network_notifications()
         {
-            foreach( $this->notifications as $handle => $notification )
+            foreach ($this->notifications as $handle => $notification)
             {
-                if( $notification['network'] ) $this->render_notification( $handle, $notification );
+                if ($notification['network']) $this->render_notification($handle, $notification);
             }
         }
         
         public function dismiss_notification()
         {
-            $id = filter_input( INPUT_POST, 'id' );
-            if( !in_array( $id, $this->dismissed_notices ) )
+            $id = filter_input(INPUT_POST, 'id');
+            if (!in_array($id, $this->dismissed_notices))
             {
                 $this->dismissed_notices[] = $id;
-                update_option( 'wp_dismissed_notices', $this->dismissed_notices);
+                update_option('wp_dismissed_notices', $this->dismissed_notices);
             }
             wp_die();
         }
         
         public function render_script()
         {
-            if( 0 === count( $this->notifications ) ) return;
+            if (0 === count($this->notifications)) return;
             ?>
             <script>
             jQuery(document).ready(function($){
@@ -90,15 +90,15 @@ 
             <?php
         }
         
-        private function render_notification( $id, $n )
+        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>',
                 $id,
                 $n['type'], 
-                $n['dismissible']?'is-dismissible ':'', 
+                $n['dismissible'] ? 'is-dismissible ' : '', 
                 $n['class'], 
                 $n['html'] 
             );