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

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 field
      *
-     * @param [array] $args
      * @return void
      */
     function amarkal_get_settings_value( $slug, $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 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();
         return $manager->add_settings_page($args);
     }
 }
 
-if(!function_exists('amarkal_get_settings_page'))
+if (!function_exists('amarkal_get_settings_page'))
 {
     /**
      * Get a settings page instance
@@ -44,14 +44,14 @@ 
      * @param string $slug
      * @return Amarkal\Settings\SettingsPage
      */
-    function amarkal_get_settings_page( $slug )
+    function amarkal_get_settings_page($slug)
     {
         $manager = Amarkal\Settings\Manager::get_instance();
         return $manager->get_settings_page($slug);
     }
 }
 
-if(!function_exists('amarkal_get_settings_value'))
+if (!function_exists('amarkal_get_settings_value'))
 {
     /**
      * Get the value of the given settings field
@@ -59,7 +59,7 @@ 
      * @param [array] $args
      * @return void
      */
-    function amarkal_get_settings_value( $slug, $field_name )
+    function amarkal_get_settings_value($slug, $field_name)
     {
         $manager = Amarkal\Settings\Manager::get_instance();
         $page = $manager->get_settings_page($slug);

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

Doc Comments -1 removed lines

@@ -50,7 +50,6 @@
      * Get a settings page from the list of registered settings pages.
      * 
      * @param string $slug
-     * @param string $parent_slug
      * @return SettingsPage
      * @throws \RuntimeException If no settings page was found for the given slug/parent_slug
      */

Spacing +7 added lines, -7 removed lines

@@ -21,7 +21,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance ) 
+        if (null === static::$instance) 
         {
             static::$instance = new static();
         }
@@ -34,10 +34,10 @@ 
      * @param array $args
      * @throws \RuntimeException
      */
-    public function add_settings_page( $args )
+    public function add_settings_page($args)
     {
         $slug = $args['slug'];
-        if(array_key_exists($slug,$this->settings_pages))
+        if (array_key_exists($slug, $this->settings_pages))
         {
             throw new \RuntimeException("A settings page with slug '$slug' has already been registered");
         }
@@ -54,9 +54,9 @@ 
      * @return SettingsPage
      * @throws \RuntimeException If no settings page was found for the given slug/parent_slug
      */
-    public function get_settings_page( $slug )
+    public function get_settings_page($slug)
     {
-        if(!array_key_exists($slug, $this->settings_pages))
+        if (!array_key_exists($slug, $this->settings_pages))
         {
             throw new \RuntimeException("The settings page '$slug' does not exist");
         }
@@ -86,7 +86,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'));
     }
     
     /**
@@ -102,7 +102,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();

RequestHandler.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance )
+        if (null === static::$instance)
         {
             static::$instance = new static();
         }
@@ -29,15 +29,15 @@ 
     
     public function init()
     {
-        \add_action('wp_ajax_amarkal_settings_save', array( $this, 'action_save'));
-        \add_action('wp_ajax_amarkal_settings_reset_all', array( $this, 'action_reset_all'));
-        \add_action('wp_ajax_amarkal_settings_reset_section', array( $this, 'action_reset_section'));
+        \add_action('wp_ajax_amarkal_settings_save', array($this, 'action_save'));
+        \add_action('wp_ajax_amarkal_settings_reset_all', array($this, 'action_reset_all'));
+        \add_action('wp_ajax_amarkal_settings_reset_section', array($this, 'action_reset_section'));
     }
     
     public function action_save()
     {
         $this->set_request_data();
-        $settings_page  = $this->get_request_settings_page();
+        $settings_page = $this->get_request_settings_page();
         
         \wp_send_json($settings_page->update($this->request_data));
     }
@@ -69,11 +69,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(),

SettingsPage.php

Spacing +18 added lines, -18 removed lines

@@ -47,15 +47,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->fields   = new \Amarkal\UI\ComponentList();
         $this->sections = array();
         $this->form     = new \Amarkal\UI\Form($this->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'));
     }
 
     /**
@@ -64,11 +64,11 @@ 
      * @param array $args
      * @return void
      */
-    public function add_section( array $args )
+    public function add_section(array $args)
     {
         $args = \array_merge($this->default_section_args(), $args);
         $slug = $args['slug'];
-        if(array_key_exists($slug, $this->sections))
+        if (array_key_exists($slug, $this->sections))
         {
             throw new \RuntimeException("A section with '$slug' has already been created for this page.");  
         }
@@ -81,7 +81,7 @@ 
      * @param array $args
      * @return void
      */
-    public function add_field( array $args )
+    public function add_field(array $args)
     {
         $this->fields->add_component($args);
     }
@@ -93,7 +93,7 @@ 
      */
     public function get_field_values()
     {
-        if(!isset($this->values))
+        if (!isset($this->values))
         {
             $this->values = array_merge($this->form->reset(), $this->get_old_instance());
         }
@@ -106,7 +106,7 @@ 
      * @param string $name
      * @return any
      */
-    public function get_field_value( $name )
+    public function get_field_value($name)
     {
         $values = $this->get_field_values();
         return $values[$name];
@@ -134,7 +134,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');
@@ -158,9 +158,9 @@ 
      * @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();
             $this->values = $this->form->update($new_instance, $old_instance);
@@ -185,7 +185,7 @@ 
      */
     public function reset()
     {
-        if($this->can_update())
+        if ($this->can_update())
         {
             \delete_option($this->config['slug']);
             $this->values = $this->form->reset();
@@ -207,9 +207,9 @@ 
      * @param string $slug
      * @return void
      */
-    public function reset_section( $slug ) 
+    public function reset_section($slug) 
     {
-        if($this->can_update())
+        if ($this->can_update())
         {
             $old_instance = $this->get_old_instance();
             $final_instance = $this->form->reset_components($this->get_section_fields($slug));
@@ -258,9 +258,9 @@ 
     private function get_section_fields($slug)
     {
         $fields = array();
-        foreach($this->fields->get_all() as $c)
+        foreach ($this->fields->get_all() as $c)
         {
-            if($c->section === $slug)
+            if ($c->section === $slug)
             {
                 $fields[] = $c;
             }
@@ -276,7 +276,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;
         }
@@ -290,7 +290,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,