amarkal / amarkal-settings

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

Page.php

Spacing +2 added lines, -2 removed lines

@@ -6,11 +6,11 @@
 {   
     private $config;
     
-    public function __construct( array $args = array() ) 
+    public function __construct(array $args = array()) 
     {
         $this->config = array_merge($this->default_args(), $args);
         
-        \add_action('admin_menu', array($this,'add_menu_page'));
+        \add_action('admin_menu', array($this, 'add_menu_page'));
     }
     
     public function add_menu_page()

functions.php

Braces +3 added lines, -1 removed lines

@@ -18,7 +18,9 @@
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'AMARKAL_SETTINGS' ) ) return false;
+if( defined( 'AMARKAL_SETTINGS' ) ) {
+    return false;
+}
 define( 'AMARKAL_SETTINGS', true );
 
 if(!function_exists('amarkal_add_settings_page'))

Doc Comments -1 removed lines

@@ -56,7 +56,6 @@
     /**
      * Get the value of the given settings
      *
-     * @param [array] $args
      * @return void
      */
     function amarkal_get_settings_value( $field_name )

Spacing +9 added lines, -9 removed lines

@@ -13,15 +13,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_SETTINGS' ) ) return false;
-define( 'AMARKAL_SETTINGS', true );
+if (defined('AMARKAL_SETTINGS')) return false;
+define('AMARKAL_SETTINGS', true);
 
-if(!function_exists('amarkal_add_settings_page'))
+if (!function_exists('amarkal_add_settings_page'))
 {
     /**
      * Add a new top-level settings page
@@ -29,14 +29,14 @@ 
      * @param [array] $args
      * @return void
      */
-    function amarkal_add_settings_page( $args )
+    function amarkal_add_settings_page($args)
     {
         $manager = Amarkal\Settings\Manager::get_instance();
         $manager->add_page($args);
     }
 }
 
-if(!function_exists('amarkal_add_settings_child_page'))
+if (!function_exists('amarkal_add_settings_child_page'))
 {
     /**
      * Add a child page to an existing top-level settings page
@@ -44,14 +44,14 @@ 
      * @param [array] $args
      * @return void
      */
-    function amarkal_add_settings_child_page( $args )
+    function amarkal_add_settings_child_page($args)
     {
         $manager = Amarkal\Settings\Manager::get_instance();
         $manager->add_child_page($args);
     }
 }
 
-if(!function_exists('amarkal_get_settings_value'))
+if (!function_exists('amarkal_get_settings_value'))
 {
     /**
      * Get the value of the given settings
@@ -59,7 +59,7 @@ 
      * @param [array] $args
      * @return void
      */
-    function amarkal_get_settings_value( $field_name )
+    function amarkal_get_settings_value($field_name)
     {
         $manager = Amarkal\Settings\Manager::get_instance();
         return $manager->get_field_value($field_name);

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 'RequestHandler.php';

Manager.php

Spacing +11 added lines, -11 removed lines

@@ -20,7 +20,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance ) 
+        if (null === static::$instance) 
         {
             static::$instance = new static();
         }
@@ -36,7 +36,7 @@ 
     public function add_page($args)
     {
         $slug = $args['slug'];
-        if(array_key_exists($slug,$this->pages))
+        if (array_key_exists($slug, $this->pages))
         {
             throw new \RuntimeException("A page with slug '$slug' has already been registered");
         }
@@ -52,7 +52,7 @@ 
      */
     public function get_page($slug)
     {
-        if(!array_key_exists($slug,$this->pages))
+        if (!array_key_exists($slug, $this->pages))
         {
             throw new \RuntimeException("The page '$slug' does not exist");
         }
@@ -67,7 +67,7 @@ 
     public function add_child_page($args)
     {
         $parent_slug = $args['parent_slug'];
-        if(!array_key_exists($parent_slug, $this->child_pages))
+        if (!array_key_exists($parent_slug, $this->child_pages))
         {
             $this->child_pages[$parent_slug] = array();
         }
@@ -84,7 +84,7 @@ 
      */
     public function get_child_page($slug, $parent_slug)
     {
-        if(!array_key_exists($parent_slug, $this->child_pages) ||
+        if (!array_key_exists($parent_slug, $this->child_pages) ||
            !array_key_exists($slug, $this->child_pages[$parent_slug]))
         {
             throw new \RuntimeException("The child page '$slug' does not exist for the parent '$parent_slug'");
@@ -100,13 +100,13 @@ 
      */
     public function get_field_value($field_name)
     {
-        foreach($this->child_pages as $parent_slug => $child_pages)
+        foreach ($this->child_pages as $parent_slug => $child_pages)
         {
             try
             {
                 return $this->get_field_value_for_parent($parent_slug, $field_name);
             }
-            catch(\RuntimeException $e)
+            catch (\RuntimeException $e)
             {
                 continue;
             }
@@ -125,7 +125,7 @@ 
      */
     public function get_field_value_for_parent($parent_slug, $field_name)
     {
-        foreach($this->child_pages[$parent_slug] as $cp)
+        foreach ($this->child_pages[$parent_slug] as $cp)
         {
             try 
             {
@@ -133,7 +133,7 @@ 
                 $value = \get_option($field_name, $component->default);
                 return $value;
             }
-            catch(\RuntimeException $e)
+            catch (\RuntimeException $e)
             {
                 continue;
             }
@@ -147,7 +147,7 @@ 
     public function register_scripts()
     {
         \wp_register_style('amarkal-settings', \Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/css/dist/amarkal-settings.min.css'));
-        \wp_register_script('amarkal-settings',\Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/js/dist/amarkal-settings.min.js'),array('amarkal-ui'));
+        \wp_register_script('amarkal-settings', \Amarkal\Core\Utility::path_to_url(__DIR__.'/assets/js/dist/amarkal-settings.min.js'), array('amarkal-ui'));
     }
     
     /**
@@ -163,7 +163,7 @@ 
      */
     private function init()
     {
-        \add_action('admin_init',array($this,'register_scripts'));
+        \add_action('admin_init', array($this, 'register_scripts'));
         
         $rh = RequestHandler::get_instance();
         $rh->init();

Braces +2 added lines, -4 removed lines

@@ -105,8 +105,7 @@ 
             try
             {
                 return $this->get_field_value_for_parent($parent_slug, $field_name);
-            }
-            catch(\RuntimeException $e)
+            } catch(\RuntimeException $e)
             {
                 continue;
             }
@@ -132,8 +131,7 @@ 
                 $component = $cp->get_component($field_name);
                 $value = \get_option($field_name, $component->default);
                 return $value;
-            }
-            catch(\RuntimeException $e)
+            } catch(\RuntimeException $e)
             {
                 continue;
             }

RequestHandler.php

Spacing +7 added lines, -7 removed lines

@@ -20,7 +20,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance )
+        if (null === static::$instance)
         {
             static::$instance = new static();
         }
@@ -29,14 +29,14 @@ 
     
     public function init()
     {
-        \add_action('wp_ajax_amarkal_settings_save', array( $this, 'save_settings'));
-        \add_action('wp_ajax_amarkal_settings_reset', array( $this, 'reset_settings'));
+        \add_action('wp_ajax_amarkal_settings_save', array($this, 'save_settings'));
+        \add_action('wp_ajax_amarkal_settings_reset', array($this, 'reset_settings'));
     }
     
     public function save_settings()
     {
         $this->set_request_data();
-        $child_page  = $this->get_request_child_page();
+        $child_page = $this->get_request_child_page();
         
         \wp_send_json($child_page->update($this->request_data));
     }
@@ -60,11 +60,11 @@ 
     
     private function set_request_data()
     {
-        $this->request_data = filter_input(INPUT_POST,'data',FILTER_DEFAULT,FILTER_REQUIRE_ARRAY);
+        $this->request_data = filter_input(INPUT_POST, 'data', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY);
         $nonce = $this->request_data['_amarkal_settings_nonce'];
 
-        if( !isset( $nonce ) ||
-            !\wp_verify_nonce($nonce, self::NONCE_ACTION) ) 
+        if (!isset($nonce) ||
+            !\wp_verify_nonce($nonce, self::NONCE_ACTION)) 
         {
             \wp_send_json(array(
                 'values' => array(),

ChildPage.php

Spacing +13 added lines, -13 removed lines

@@ -28,15 +28,15 @@ 
      * 
      * @param array $config
      */
-    public function __construct( array $config = array() ) 
+    public function __construct(array $config = array()) 
     {
         $this->config = array_merge($this->default_args(), $config);
         $this->form   = new \Amarkal\UI\Form(
             new \Amarkal\UI\ComponentList($this->config['fields'])
         );
         
-        \add_action('admin_menu', array($this,'add_submenu_page'));
-        \add_action('admin_enqueue_scripts', array($this,'enqueue_scripts'));
+        \add_action('admin_menu', array($this, 'add_submenu_page'));
+        \add_action('admin_enqueue_scripts', array($this, 'enqueue_scripts'));
     }
     
     /**
@@ -61,7 +61,7 @@ 
     public function enqueue_scripts()
     {
         // Only enqueue styles & scripts if this is a settings page
-        if($this->config['slug'] === filter_input(INPUT_GET, 'page'))
+        if ($this->config['slug'] === filter_input(INPUT_GET, 'page'))
         {
             \wp_enqueue_style('amarkal-settings');
             \wp_enqueue_script('amarkal-settings');
@@ -85,15 +85,15 @@ 
      * @param array $new_instance
      * @return array
      */
-    public function update( $new_instance )
+    public function update($new_instance)
     {
-        if($this->can_update())
+        if ($this->can_update())
         {
             $old_instance = $this->get_old_instance();
             $final_instance = $this->form->update($new_instance, $old_instance);
-            foreach($final_instance as $name => $value)
+            foreach ($final_instance as $name => $value)
             {
-                \update_option($name,$value);
+                \update_option($name, $value);
             }
             return $this->results_array(
                 $this->get_errors(),
@@ -113,9 +113,9 @@ 
      */
     public function reset()
     {
-        if($this->can_update())
+        if ($this->can_update())
         {
-            foreach($this->config['fields'] as $field)
+            foreach ($this->config['fields'] as $field)
             {
                 \delete_option($field['name']);
             }
@@ -157,7 +157,7 @@ 
     private function get_errors()
     {
         $errors = array();
-        foreach($this->form->get_errors() as $name => $error)
+        foreach ($this->form->get_errors() as $name => $error)
         {
             $errors[$name] = $error;
         }
@@ -171,7 +171,7 @@ 
      * @param array $values The list of values
      * @return array
      */
-    private function results_array( $errors = array(), $values = '' )
+    private function results_array($errors = array(), $values = '')
     {
         return array(
             'values' => $values,
@@ -198,7 +198,7 @@ 
     private function get_old_instance()
     {
         $old_instance = array();
-        foreach($this->form->get_component_list()->get_value_components() as $component)
+        foreach ($this->form->get_component_list()->get_value_components() as $component)
         {
             $old_instance[$component->name] = \get_option($component->name, $component->default);
         }