amarkal / amarkal-settings

functions.php

Doc Comments -1 removed lines

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

Spacing +7 added lines, -7 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();
         return $manager->add_settings_page($args);
     }
 }
 
-if(!function_exists('amarkal_get_settings_value'))
+if (!function_exists('amarkal_get_settings_value'))
 {
     /**
      * Get the value of the given settings
@@ -44,7 +44,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();
         return $manager->get_field_value($slug, $field_name);

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 +17 added lines, -17 removed lines

@@ -40,15 +40,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'));
     }
 
     /**
@@ -57,11 +57,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.");  
         }
@@ -74,7 +74,7 @@ 
      * @param array $args
      * @return void
      */
-    public function add_field( array $args )
+    public function add_field(array $args)
     {
         $this->fields->add_component($args);
     }
@@ -101,7 +101,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');
@@ -125,14 +125,14 @@ 
      * @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);
             
-            \update_option($this->config['slug'],$final_instance);
+            \update_option($this->config['slug'], $final_instance);
 
             return $this->results_array(
                 $this->get_errors(),
@@ -152,7 +152,7 @@ 
      */
     public function reset()
     {
-        if($this->can_update())
+        if ($this->can_update())
         {
             \delete_option($this->config['slug']);
 
@@ -173,9 +173,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));
@@ -225,9 +225,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;
             }
@@ -243,7 +243,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;
         }
@@ -257,7 +257,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,