amarkal / amarkal-admin-notification

Notifications.php

Braces +11 added lines, -4 removed lines

@@ -41,8 +41,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()
@@ -57,7 +58,9 @@ 
     {
         foreach( $this->notifications as $handle => $notification )
         {
-            if( $notification['network'] ) $this->render_notification( $handle, $notification );
+            if( $notification['network'] ) {
+                $this->render_notification( $handle, $notification );
+            }
         }
     }
 
@@ -74,7 +77,9 @@ 
 
     public function render_script()
     {
-        if( 0 === count( $this->notifications ) ) return;
+        if( 0 === count( $this->notifications ) ) {
+            return;
+        }
         ?>
         <script>
         jQuery(document).ready(function($){
@@ -91,7 +96,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 +20 added lines, -20 removed lines

@@ -20,7 +20,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance ) 
+        if (null === static::$instance) 
         {
             static::$instance = new static();
         }
@@ -29,52 +29,52 @@ 
 
     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
-        $this->dismissed_notices = get_site_option( 'amarkal_dismissed_notices', array() );
+        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_site_option('amarkal_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_site_option( 'amarkal_dismissed_notices', $this->dismissed_notices);
+            update_site_option('amarkal_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($){
@@ -89,15 +89,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'] 
         );

composer.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  */
 
 // Prevent direct file access
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 // Load module functions
 require_once 'functions.php';

functions.php

Spacing +20 added lines, -20 removed lines

@@ -11,15 +11,15 @@ 
  */
 
 // Prevent direct file access
-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( 'AMARKAL_ADMIN_NOTIFICATION' ) ) return false;
-define( 'AMARKAL_ADMIN_NOTIFICATION', true );
+if (defined('AMARKAL_ADMIN_NOTIFICATION')) return false;
+define('AMARKAL_ADMIN_NOTIFICATION', true);
 
-if(!function_exists('_amarkal_admin_notification_init'))
+if (!function_exists('_amarkal_admin_notification_init'))
 {
     /**
      * Initiate admin notifications
@@ -29,10 +29,10 @@ 
         $notifier = Amarkal\Admin\Notifications::get_instance();
         $notifier->init();
     }
-    add_action( 'init', '_amarkal_admin_notification_init' );
+    add_action('init', '_amarkal_admin_notification_init');
 }
 
-if(!function_exists('amarkal_admin_notification'))
+if (!function_exists('amarkal_admin_notification'))
 {
     /**
      * Register an admin notification.
@@ -44,7 +44,7 @@ 
      * @param type $class
      * @param type $network
      */
-    function amarkal_admin_notification( $handle, $html, $type = 'success', $dismissible = false, $class = '', $network = false )
+    function amarkal_admin_notification($handle, $html, $type = 'success', $dismissible = false, $class = '', $network = false)
     {
         $notifier = Amarkal\Admin\Notifications::get_instance();
         $notifier->register_notification($handle, array(
@@ -57,47 +57,47 @@ 
     }
 }
 
-if(!function_exists('amarkal_reset_admin_notification'))
+if (!function_exists('amarkal_reset_admin_notification'))
 {
     /**
      * Reset a dismissed admin notification.
      * 
      * @param string $handle
      */
-    function amarkal_reset_admin_notification( $handle )
+    function amarkal_reset_admin_notification($handle)
     {
-        $dismissed = get_site_option( 'amarkal_dismissed_notices', array() );
+        $dismissed = get_site_option('amarkal_dismissed_notices', array());
         $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_site_option( 'amarkal_dismissed_notices', $dismissed );
+        array_splice($dismissed, $offset, 1);
+        update_site_option('amarkal_dismissed_notices', $dismissed);
     }
 }
 
-if(!function_exists('wp_admin_notification'))
+if (!function_exists('wp_admin_notification'))
 {
     /**
      * @deprecated use amarkal_admin_notification instead
      */
-    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)
     {
         trigger_error('<b>wp_admin_notification()</b> has been deprecated. Use <b>amarkal_admin_notification</b> instead.');
-        amarkal_admin_notification( $handle, $html, $type, $dismissible, $class, $network );
+        amarkal_admin_notification($handle, $html, $type, $dismissible, $class, $network);
     }
 }
 
-if(!function_exists('wp_reset_admin_notification'))
+if (!function_exists('wp_reset_admin_notification'))
 {
     /**
      * @deprecated use amarkal_reset_admin_notification instead
      */
-    function wp_reset_admin_notification( $handle )
+    function wp_reset_admin_notification($handle)
     {
         trigger_error('<b>wp_reset_admin_notification()</b> has been deprecated. Use <b>amarkal_reset_admin_notification</b> instead.');
-        amarkal_reset_admin_notification( $handle );
+        amarkal_reset_admin_notification($handle);
     }
 }
\ No newline at end of file

Braces +6 added lines, -2 removed lines

@@ -16,7 +16,9 @@ 
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'AMARKAL_ADMIN_NOTIFICATION' ) ) return false;
+if( defined( 'AMARKAL_ADMIN_NOTIFICATION' ) ) {
+    return false;
+}
 define( 'AMARKAL_ADMIN_NOTIFICATION', true );
 
 if(!function_exists('_amarkal_admin_notification_init'))
@@ -70,7 +72,9 @@ 
         $offset = 0;
         foreach($dismissed as $id)
         {
-            if( $id === $handle ) break;
+            if( $id === $handle ) {
+                break;
+            }
             $offset++;
         }
         array_splice( $dismissed, $offset, 1);

bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -6,13 +6,13 @@
  */
 
 // Prevent direct file access
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 /**
  * Load module functions. If this amarkal module has not been loaded, 
  * functions.php will not return false.
  */
-if(false !== (require_once 'functions.php'))
+if (false !== (require_once 'functions.php'))
 {
     // Load required classes if not using composer
     require_once 'Notifications.php';