woothemes / sensei

includes/class-sensei-settings-api.php

Indentation +24 added lines, -24 removed lines

@@ -57,7 +57,7 @@ 
 	 */
 	public function register_hook_listener() {
 
-        add_action( 'admin_menu', array( $this, 'register_settings_screen' ), 60 );
+		add_action( 'admin_menu', array( $this, 'register_settings_screen' ), 60 );
 		add_action( 'admin_init', array( $this, 'settings_fields' ) );
 		add_action( 'init', array( $this, 'general_init' ) );
 
@@ -293,14 +293,14 @@ 
 	 * @return void
 	 */
 	public function settings_screen ()
-    {
+	{
 
-        ?>
+		?>
         <div id="woothemes-sensei" class="wrap <?php echo esc_attr($this->token); ?>">
         <?php screen_icon('woothemes-sensei'); ?>
         <h2><?php echo esc_html($this->name); ?><?php if ('' != $this->settings_version) {
-                echo ' <span class="version">' . $this->settings_version . '</span>';
-            } ?></h2>
+				echo ' <span class="version">' . $this->settings_version . '</span>';
+			} ?></h2>
         <p class="powered-by-woo"><?php _e('Powered by', 'woothemes-sensei'); ?><a href="http://www.woothemes.com/"
                                                                                    title="WooThemes"><img
                     src="<?php echo Sensei()->plugin_url; ?>assets/images/woothemes.png" alt="WooThemes"/></a></p>
@@ -308,26 +308,26 @@ 
         <form action="options.php" method="post">
 
         <?php
-        $this->settings_tabs();
-        settings_fields($this->token);
-        $page = 'woothemes-sensei-settings';
-        foreach ($this->sections as $section_id => $section) {
+		$this->settings_tabs();
+		settings_fields($this->token);
+		$page = 'woothemes-sensei-settings';
+		foreach ($this->sections as $section_id => $section) {
 
-            echo '<section id="' . $section_id . '">';
+			echo '<section id="' . $section_id . '">';
 
-            if ($section['name'])
-                echo "<h2>{$section['name']}</h2>\n";
+			if ($section['name'])
+				echo "<h2>{$section['name']}</h2>\n";
 
-            echo '<table class="form-table">';
-            do_settings_fields($page, $section_id );
-            echo '</table>';
+			echo '<table class="form-table">';
+			do_settings_fields($page, $section_id );
+			echo '</table>';
 
-            echo '</section>';
+			echo '</section>';
 
-        }
+		}
 
-        submit_button();
-        ?>
+		submit_button();
+		?>
 	</form>
 	<?php do_action( 'settings_after_form' ); ?>
 </div><!--/#woothemes-sensei-->
@@ -376,7 +376,7 @@ 
 	 * @return void
 	 */
 	public function settings_errors () {
-        settings_errors( $this->token . '-errors' );
+		settings_errors( $this->token . '-errors' );
 	} // End settings_errors()
 
 	/**
@@ -442,9 +442,9 @@ 
 		echo '<input id="' . $args['key'] . '" name="' . $this->token . '[' . esc_attr( $args['key'] ) . ']" type="checkbox" value="1"' . checked( esc_attr( $options[$args['key']] ), '1', false ) . ' />' . "\n";
 		if ( $has_description ) {
 			echo wp_kses( $args['data']['description'], array( 'a' => array(
-																	        'href' => array(),
-																	        'title' => array()
-																	    )
+																			'href' => array(),
+																			'title' => array()
+																		)
 															)
 						) . '</label>' . "\n";
 		}
@@ -833,7 +833,7 @@ 
 	 */
 	public function enqueue_scripts () {
 
-        $suffix = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
+		$suffix = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
 
 		wp_enqueue_script( 'farbtastic' );
 		wp_enqueue_script( 'woothemes-sensei-settings', esc_url( Sensei()->plugin_url . 'assets/js/settings' . $suffix . '.js' ), array( 'jquery', 'farbtastic' ), Sensei()->version );

Spacing +223 added lines, -223 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * A settings API (wrapping the WordPress Settings API).
@@ -31,7 +31,7 @@ 
 	 * @access public
 	 * @since  1.0.0
 	 */
-	public function __construct () {
+	public function __construct() {
 
 		$this->token = 'woothemes-sensei-settings';
 		$this->page_slug = 'woothemes-sensei-settings-api';
@@ -57,9 +57,9 @@ 
 	 */
 	public function register_hook_listener() {
 
-        add_action( 'admin_menu', array( $this, 'register_settings_screen' ), 60 );
-		add_action( 'admin_init', array( $this, 'settings_fields' ) );
-		add_action( 'init', array( $this, 'general_init' ) );
+        add_action('admin_menu', array($this, 'register_settings_screen'), 60);
+		add_action('admin_init', array($this, 'settings_fields'));
+		add_action('init', array($this, 'general_init'));
 
 	} // End setup_settings()
 
@@ -73,7 +73,7 @@ 
 		$this->init_sections();
 		$this->init_fields();
 		$this->get_settings();
-		if ( $this->has_tabs == true ) {
+		if ($this->has_tabs == true) {
 			$this->create_tabs();
 		} // End If Statement
 	} // End general_init()
@@ -84,9 +84,9 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function init_sections () {
+	public function init_sections() {
 		// Override this function in your class and assign the array of sections to $this->sections.
-		_e( 'Override init_sections() in your class.', 'woothemes-sensei' );
+		_e('Override init_sections() in your class.', 'woothemes-sensei');
 	} // End init_sections()
 
 	/**
@@ -95,9 +95,9 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function init_fields () {
+	public function init_fields() {
 		// Override this function in your class and assign the array of sections to $this->fields.
-		_e( 'Override init_fields() in your class.', 'woothemes-sensei' );
+		_e('Override init_fields() in your class.', 'woothemes-sensei');
 	} // End init_fields()
 
 	/**
@@ -106,34 +106,34 @@ 
 	 * @since  1.1.0
 	 * @return void
 	 */
-	public function settings_tabs () {
+	public function settings_tabs() {
 
-		if ( ! $this->has_tabs ) { return; }
+		if ( ! $this->has_tabs) { return; }
 
-		if ( count( $this->tabs ) > 0 ) {
+		if (count($this->tabs) > 0) {
 			$html = '';
 
-			$html .= '<ul id="settings-sections" class="subsubsub hide-if-no-js">' . "\n";
+			$html .= '<ul id="settings-sections" class="subsubsub hide-if-no-js">'."\n";
 
 			$sections = array(
-						'all' => array( 'href' => '#all', 'name' => __( 'All', 'woothemes-sensei' ), 'class' => 'current all tab' )
+						'all' => array('href' => '#all', 'name' => __('All', 'woothemes-sensei'), 'class' => 'current all tab')
 					);
 
-			foreach ( $this->tabs as $k => $v ) {
-				$sections[$k] = array( 'href' => '#' . esc_attr( $k ), 'name' => esc_attr( $v['name'] ), 'class' => 'tab' );
+			foreach ($this->tabs as $k => $v) {
+				$sections[$k] = array('href' => '#'.esc_attr($k), 'name' => esc_attr($v['name']), 'class' => 'tab');
 			}
 
 			$count = 1;
-			foreach ( $sections as $k => $v ) {
+			foreach ($sections as $k => $v) {
 				$count++;
-				$html .= '<li><a href="' . $v['href'] . '"';
-				if ( isset( $v['class'] ) && ( $v['class'] != '' ) ) { $html .= ' class="' . esc_attr( $v['class'] ) . '"'; }
-				$html .= '>' . esc_attr( $v['name'] ) . '</a>';
-				if ( $count <= count( $sections ) ) { $html .= ' | '; }
-				$html .= '</li>' . "\n";
+				$html .= '<li><a href="'.$v['href'].'"';
+				if (isset($v['class']) && ($v['class'] != '')) { $html .= ' class="'.esc_attr($v['class']).'"'; }
+				$html .= '>'.esc_attr($v['name']).'</a>';
+				if ($count <= count($sections)) { $html .= ' | '; }
+				$html .= '</li>'."\n";
 			}
 
-			$html .= '</ul><div class="clear"></div>' . "\n";
+			$html .= '</ul><div class="clear"></div>'."\n";
 
 			echo $html;
 		}
@@ -145,10 +145,10 @@ 
 	 * @since  1.1.0
 	 * @return void
 	 */
-	private function create_tabs () {
-		if ( count( $this->sections ) > 0 ) {
+	private function create_tabs() {
+		if (count($this->sections) > 0) {
 			$tabs = array();
-			foreach ( $this->sections as $k => $v ) {
+			foreach ($this->sections as $k => $v) {
 				$tabs[$k] = $v;
 			}
 
@@ -162,10 +162,10 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function create_sections () {
-		if ( count( $this->sections ) > 0 ) {
-			foreach ( $this->sections as $k => $v ) {
-				add_settings_section( $k, $v['name'], array( $this, 'section_description' ), $this->token );
+	public function create_sections() {
+		if (count($this->sections) > 0) {
+			foreach ($this->sections as $k => $v) {
+				add_settings_section($k, $v['name'], array($this, 'section_description'), $this->token);
 			}
 		}
 	} // End create_sections()
@@ -176,18 +176,18 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function create_fields () {
-		if ( count( $this->sections ) > 0 ) {
+	public function create_fields() {
+		if (count($this->sections) > 0) {
 			// $this->parse_fields( $this->fields );
 
-			foreach ( $this->fields as $k => $v ) {
-				$method = $this->determine_method( $v, 'form' );
+			foreach ($this->fields as $k => $v) {
+				$method = $this->determine_method($v, 'form');
 				$name = $v['name'];
-				if ( $v['type'] == 'info' ) { $name = ''; }
-				add_settings_field( $k, $name, $method, $this->token, $v['section'], array( 'key' => $k, 'data' => $v ) );
+				if ($v['type'] == 'info') { $name = ''; }
+				add_settings_field($k, $name, $method, $this->token, $v['section'], array('key' => $k, 'data' => $v));
 
 				// Let the API know that we have a colourpicker field.
-				if ( $v['type'] == 'range' && $this->has_range == false ) { $this->has_range = true; }
+				if ($v['type'] == 'range' && $this->has_range == false) { $this->has_range = true; }
 			}
 		}
 	} // End create_fields()
@@ -199,43 +199,43 @@ 
 	 * @param  array $data
 	 * @return callable,  array or string
 	 */
-	protected function determine_method ( $data, $type = 'form' ) {
+	protected function determine_method($data, $type = 'form') {
 		$method = '';
 
-		if ( ! in_array( $type, array( 'form', 'validate', 'check' ) ) ) { return; }
+		if ( ! in_array($type, array('form', 'validate', 'check'))) { return; }
 
 		// Check for custom functions.
-		if ( isset( $data[$type] ) ) {
-			if ( function_exists( $data[$type] ) ) {
+		if (isset($data[$type])) {
+			if (function_exists($data[$type])) {
 				$method = $data[$type];
 			}
 
-			if ( $method == '' && method_exists( $this, $data[$type] ) ) {
-				if ( $type == 'form' ) {
-					$method = array( $this, $data[$type] );
+			if ($method == '' && method_exists($this, $data[$type])) {
+				if ($type == 'form') {
+					$method = array($this, $data[$type]);
 				} else {
 					$method = $data[$type];
 				}
 			}
 		}
 
-		if ( $method == '' && method_exists ( $this, $type . '_field_' . $data['type'] ) ) {
-			if ( $type == 'form' ) {
-				$method = array( $this, $type . '_field_' . $data['type'] );
+		if ($method == '' && method_exists($this, $type.'_field_'.$data['type'])) {
+			if ($type == 'form') {
+				$method = array($this, $type.'_field_'.$data['type']);
 			} else {
-				$method = $type . '_field_' . $data['type'];
+				$method = $type.'_field_'.$data['type'];
 			}
 		}
 
-		if ( $method == '' && function_exists ( $this->token . '_' . $type . '_field_' . $data['type'] ) ) {
-			$method = $this->token . '_' . $type . '_field_' . $data['type'];
+		if ($method == '' && function_exists($this->token.'_'.$type.'_field_'.$data['type'])) {
+			$method = $this->token.'_'.$type.'_field_'.$data['type'];
 		}
 
-		if ( $method == '' ) {
-			if ( $type == 'form' ) {
-				$method = array( $this, $type . '_field_text' );
+		if ($method == '') {
+			if ($type == 'form') {
+				$method = array($this, $type.'_field_text');
 			} else {
-				$method = $type . '_field_text';
+				$method = $type.'_field_text';
 			}
 		}
 
@@ -249,10 +249,10 @@ 
 	 * @param  array $fields
 	 * @return void
 	 */
-	public function parse_fields ( $fields ) {
-		foreach ( $fields as $k => $v ) {
-			if ( isset( $v['section'] ) && ( $v['section'] != '' ) && ( isset( $this->sections[$v['section']] ) ) ) {
-				if ( ! isset( $this->sections[$v['section']]['fields'] ) ) {
+	public function parse_fields($fields) {
+		foreach ($fields as $k => $v) {
+			if (isset($v['section']) && ($v['section'] != '') && (isset($this->sections[$v['section']]))) {
+				if ( ! isset($this->sections[$v['section']]['fields'])) {
 					$this->sections[$v['section']]['fields'] = array();
 				}
 
@@ -269,19 +269,19 @@ 
 	 * @since 1.0.0
 	 * @return void
 	 */
-	public function register_settings_screen () {
+	public function register_settings_screen() {
 
-		if ( current_user_can( 'manage_sensei' ) ) {
-			$hook = add_submenu_page( 'sensei', $this->name, $this->menu_label, 'manage_sensei', $this->page_slug, array( $this, 'settings_screen' ) );
+		if (current_user_can('manage_sensei')) {
+			$hook = add_submenu_page('sensei', $this->name, $this->menu_label, 'manage_sensei', $this->page_slug, array($this, 'settings_screen'));
 
 			$this->hook = $hook;
 		}
 
-		if ( isset( $_GET['page'] ) && ( $_GET['page'] == $this->page_slug ) ) {
+		if (isset($_GET['page']) && ($_GET['page'] == $this->page_slug)) {
 
-			add_action( 'admin_notices', array( $this, 'settings_errors' ) );
-			add_action( 'admin_print_scripts', array( $this, 'enqueue_scripts' ) );
-			add_action( 'admin_print_styles', array( $this, 'enqueue_styles' ) );
+			add_action('admin_notices', array($this, 'settings_errors'));
+			add_action('admin_print_scripts', array($this, 'enqueue_scripts'));
+			add_action('admin_print_styles', array($this, 'enqueue_styles'));
 
 		}
 	} // End register_settings_screen()
@@ -292,14 +292,14 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function settings_screen ()
+	public function settings_screen()
     {
 
         ?>
         <div id="woothemes-sensei" class="wrap <?php echo esc_attr($this->token); ?>">
         <?php screen_icon('woothemes-sensei'); ?>
         <h2><?php echo esc_html($this->name); ?><?php if ('' != $this->settings_version) {
-                echo ' <span class="version">' . $this->settings_version . '</span>';
+                echo ' <span class="version">'.$this->settings_version.'</span>';
             } ?></h2>
         <p class="powered-by-woo"><?php _e('Powered by', 'woothemes-sensei'); ?><a href="http://www.woothemes.com/"
                                                                                    title="WooThemes"><img
@@ -313,13 +313,13 @@ 
         $page = 'woothemes-sensei-settings';
         foreach ($this->sections as $section_id => $section) {
 
-            echo '<section id="' . $section_id . '">';
+            echo '<section id="'.$section_id.'">';
 
             if ($section['name'])
                 echo "<h2>{$section['name']}</h2>\n";
 
             echo '<table class="form-table">';
-            do_settings_fields($page, $section_id );
+            do_settings_fields($page, $section_id);
             echo '</table>';
 
             echo '</section>';
@@ -329,7 +329,7 @@ 
         submit_button();
         ?>
 	</form>
-	<?php do_action( 'settings_after_form' ); ?>
+	<?php do_action('settings_after_form'); ?>
 </div><!--/#woothemes-sensei-->
 <?php
 	} // End settings_screen()
@@ -340,16 +340,16 @@ 
 	 * @since  1.0.0
 	 * @return array
 	 */
-	public function get_settings () {
-		if ( ! is_array( $this->settings ) ) {
-			$this->settings = get_option( $this->token, array() );
+	public function get_settings() {
+		if ( ! is_array($this->settings)) {
+			$this->settings = get_option($this->token, array());
 		}
 
-		foreach ( $this->fields as $k => $v ) {
-			if ( ! isset( $this->settings[$k] ) && isset( $v['default'] ) ) {
+		foreach ($this->fields as $k => $v) {
+			if ( ! isset($this->settings[$k]) && isset($v['default'])) {
 				$this->settings[$k] = $v['default'];
 			}
-			if ( $v['type'] == 'checkbox' && $this->settings[$k] != true ) {
+			if ($v['type'] == 'checkbox' && $this->settings[$k] != true) {
 				$this->settings[$k] = 0;
 			}
 		}
@@ -363,8 +363,8 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function settings_fields () {
-		register_setting( $this->token, $this->token, array( $this, 'validate_fields' ) );
+	public function settings_fields() {
+		register_setting($this->token, $this->token, array($this, 'validate_fields'));
 		$this->create_sections();
 		$this->create_fields();
 	} // End settings_fields()
@@ -375,8 +375,8 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function settings_errors () {
-        settings_errors( $this->token . '-errors' );
+	public function settings_errors() {
+        settings_errors($this->token.'-errors');
 	} // End settings_errors()
 
 	/**
@@ -385,9 +385,9 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function section_description ( $section ) {
-		if ( isset( $this->sections[$section['id']]['description'] ) ) {
-			echo wpautop( $this->sections[$section['id']]['description'] );
+	public function section_description($section) {
+		if (isset($this->sections[$section['id']]['description'])) {
+			echo wpautop($this->sections[$section['id']]['description']);
 		}
 	} // End section_description_main()
 
@@ -398,12 +398,12 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_text ( $args ) {
+	public function form_field_text($args) {
 		$options = $this->get_settings();
 
-		echo '<input id="' . esc_attr( $args['key'] ) . '" name="' . $this->token . '[' . esc_attr( $args['key'] ) . ']" size="40" type="text" value="' . esc_attr( $options[$args['key']] ) . '" />' . "\n";
-		if ( isset( $args['data']['description'] ) ) {
-			echo '<span class="description">' . $args['data']['description'] . '</span>' . "\n";
+		echo '<input id="'.esc_attr($args['key']).'" name="'.$this->token.'['.esc_attr($args['key']).']" size="40" type="text" value="'.esc_attr($options[$args['key']]).'" />'."\n";
+		if (isset($args['data']['description'])) {
+			echo '<span class="description">'.$args['data']['description'].'</span>'."\n";
 		}
 	} // End form_field_text()
 
@@ -414,13 +414,13 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_color ( $args ) {
+	public function form_field_color($args) {
 		$options = $this->get_settings();
 
-		echo '<input id="' . esc_attr( $args['key'] ) . '" name="' . $this->token . '[' . esc_attr( $args['key'] ) . ']" size="40" type="text" class="color" value="' . esc_attr( $options[$args['key']] ) . '" />' . "\n";
+		echo '<input id="'.esc_attr($args['key']).'" name="'.$this->token.'['.esc_attr($args['key']).']" size="40" type="text" class="color" value="'.esc_attr($options[$args['key']]).'" />'."\n";
 		echo '<div style="position:absolute;background:#FFF;z-index:99;border-radius:100%;" class="colorpicker"></div>';
-		if ( isset( $args['data']['description'] ) ) {
-			echo '<span class="description">' . $args['data']['description'] . '</span>' . "\n";
+		if (isset($args['data']['description'])) {
+			echo '<span class="description">'.$args['data']['description'].'</span>'."\n";
 		}
 	} // End form_field_text()
 
@@ -431,22 +431,22 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_checkbox ( $args ) {
+	public function form_field_checkbox($args) {
 		$options = $this->get_settings();
 
 		$has_description = false;
-		if ( isset( $args['data']['description'] ) ) {
+		if (isset($args['data']['description'])) {
 			$has_description = true;
-			echo '<label for="' . esc_attr( $args['key'] ) . '">' . "\n";
+			echo '<label for="'.esc_attr($args['key']).'">'."\n";
 		}
-		echo '<input id="' . $args['key'] . '" name="' . $this->token . '[' . esc_attr( $args['key'] ) . ']" type="checkbox" value="1"' . checked( esc_attr( $options[$args['key']] ), '1', false ) . ' />' . "\n";
-		if ( $has_description ) {
-			echo wp_kses( $args['data']['description'], array( 'a' => array(
+		echo '<input id="'.$args['key'].'" name="'.$this->token.'['.esc_attr($args['key']).']" type="checkbox" value="1"'.checked(esc_attr($options[$args['key']]), '1', false).' />'."\n";
+		if ($has_description) {
+			echo wp_kses($args['data']['description'], array('a' => array(
 																	        'href' => array(),
 																	        'title' => array()
 																	    )
 															)
-						) . '</label>' . "\n";
+						).'</label>'."\n";
 		}
 	} // End form_field_checkbox()
 
@@ -457,12 +457,12 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_textarea ( $args ) {
+	public function form_field_textarea($args) {
 		$options = $this->get_settings();
 
-		echo '<textarea id="' . esc_attr( $args['key'] ) . '" name="' . $this->token . '[' . esc_attr( $args['key'] ) . ']" cols="42" rows="5">' . esc_html( $options[$args['key']] ) . '</textarea>' . "\n";
-		if ( isset( $args['data']['description'] ) ) {
-			echo '<p><span class="description">' . esc_html( $args['data']['description'] ) . '</span></p>' . "\n";
+		echo '<textarea id="'.esc_attr($args['key']).'" name="'.$this->token.'['.esc_attr($args['key']).']" cols="42" rows="5">'.esc_html($options[$args['key']]).'</textarea>'."\n";
+		if (isset($args['data']['description'])) {
+			echo '<p><span class="description">'.esc_html($args['data']['description']).'</span></p>'."\n";
 		}
 	} // End form_field_textarea()
 
@@ -473,20 +473,20 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_select ( $args ) {
+	public function form_field_select($args) {
 		$options = $this->get_settings();
 
-		if ( isset( $args['data']['options'] ) && ( count( (array)$args['data']['options'] ) > 0 ) ) {
+		if (isset($args['data']['options']) && (count((array) $args['data']['options']) > 0)) {
 			$html = '';
-			$html .= '<select class="" id="' . esc_attr( $args['key'] ) . '" name="' . esc_attr( $this->token ) . '[' . esc_attr( $args['key'] ) . ']">' . "\n";
-				foreach ( $args['data']['options'] as $k => $v ) {
-					$html .= '<option value="' . esc_attr( $k ) . '"' . selected( esc_attr( $options[$args['key']] ), $k, false ) . '>' . $v . '</option>' . "\n";
+			$html .= '<select class="" id="'.esc_attr($args['key']).'" name="'.esc_attr($this->token).'['.esc_attr($args['key']).']">'."\n";
+				foreach ($args['data']['options'] as $k => $v) {
+					$html .= '<option value="'.esc_attr($k).'"'.selected(esc_attr($options[$args['key']]), $k, false).'>'.$v.'</option>'."\n";
 				}
-			$html .= '</select>' . "\n";
+			$html .= '</select>'."\n";
 			echo $html;
 
-			if ( isset( $args['data']['description'] ) ) {
-				echo '<p><span class="description">' . esc_html( $args['data']['description'] ) . '</span></p>' . "\n";
+			if (isset($args['data']['description'])) {
+				echo '<p><span class="description">'.esc_html($args['data']['description']).'</span></p>'."\n";
 			}
 		}
 	} // End form_field_select()
@@ -498,18 +498,18 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_radio ( $args ) {
+	public function form_field_radio($args) {
 		$options = $this->get_settings();
 
-		if ( isset( $args['data']['options'] ) && ( count( (array)$args['data']['options'] ) > 0 ) ) {
+		if (isset($args['data']['options']) && (count((array) $args['data']['options']) > 0)) {
 			$html = '';
-			foreach ( $args['data']['options'] as $k => $v ) {
-				$html .= '<input type="radio" name="' . $this->token . '[' . esc_attr( $args['key'] ) . ']" value="' . esc_attr( $k ) . '"' . checked( esc_attr( $options[$args['key']] ), $k, false ) . ' /> ' . $v . '<br />' . "\n";
+			foreach ($args['data']['options'] as $k => $v) {
+				$html .= '<input type="radio" name="'.$this->token.'['.esc_attr($args['key']).']" value="'.esc_attr($k).'"'.checked(esc_attr($options[$args['key']]), $k, false).' /> '.$v.'<br />'."\n";
 			}
 			echo $html;
 
-			if ( isset( $args['data']['description'] ) ) {
-				echo '<span class="description">' . esc_html( $args['data']['description'] ) . '</span>' . "\n";
+			if (isset($args['data']['description'])) {
+				echo '<span class="description">'.esc_html($args['data']['description']).'</span>'."\n";
 			}
 		}
 	} // End form_field_radio()
@@ -521,28 +521,28 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_multicheck ( $args ) {
+	public function form_field_multicheck($args) {
 		$options = $this->get_settings();
 
-		if ( isset( $args['data']['options'] ) && ( count( (array)$args['data']['options'] ) > 0 ) ) {
-			$html = '<div class="multicheck-container" style="margin-bottom:10px;">' . "\n";
-			foreach ( $args['data']['options'] as $k => $v ) {
+		if (isset($args['data']['options']) && (count((array) $args['data']['options']) > 0)) {
+			$html = '<div class="multicheck-container" style="margin-bottom:10px;">'."\n";
+			foreach ($args['data']['options'] as $k => $v) {
 				$checked = '';
 
-				if( isset( $options[ $args['key'] ] ) ) {
-					if ( in_array( $k, (array)$options[ $args['key'] ] ) ) { $checked = ' checked="checked"'; }
+				if (isset($options[$args['key']])) {
+					if (in_array($k, (array) $options[$args['key']])) { $checked = ' checked="checked"'; }
 				} else {
-					if ( in_array( $k, $args['data']['defaults'] ) ) { $checked = ' checked="checked"'; }
+					if (in_array($k, $args['data']['defaults'])) { $checked = ' checked="checked"'; }
 				}
-				$html .= '<label for="checkbox-' . esc_attr( $k ) . '">' . "\n";
-				$html .= '<input type="checkbox" name="' . esc_attr( $this->token ) . '[' . esc_attr( $args['key'] ) . '][]" class="multicheck multicheck-' . esc_attr( $args['key'] ) . '" value="' . esc_attr( $k ) . '" id="checkbox-' . esc_attr( $k ) . '" ' . $checked . ' /> ' . $v . "\n";
-				$html .= '</label><br />' . "\n";
+				$html .= '<label for="checkbox-'.esc_attr($k).'">'."\n";
+				$html .= '<input type="checkbox" name="'.esc_attr($this->token).'['.esc_attr($args['key']).'][]" class="multicheck multicheck-'.esc_attr($args['key']).'" value="'.esc_attr($k).'" id="checkbox-'.esc_attr($k).'" '.$checked.' /> '.$v."\n";
+				$html .= '</label><br />'."\n";
 			}
-			$html .= '</div>' . "\n";
+			$html .= '</div>'."\n";
 			echo $html;
 
-			if ( isset( $args['data']['description'] ) ) {
-				echo '<span class="description">' . esc_html( $args['data']['description'] ) . '</span>' . "\n";
+			if (isset($args['data']['description'])) {
+				echo '<span class="description">'.esc_html($args['data']['description']).'</span>'."\n";
 			}
 		}
 	} // End form_field_multicheck()
@@ -554,20 +554,20 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_range ( $args ) {
+	public function form_field_range($args) {
 		$options = $this->get_settings();
 
-		if ( isset( $args['data']['options'] ) && ( count( (array)$args['data']['options'] ) > 0 ) ) {
+		if (isset($args['data']['options']) && (count((array) $args['data']['options']) > 0)) {
 			$html = '';
-			$html .= '<select id="' . esc_attr( $args['key'] ) . '" name="' . esc_attr( $this->token ) . '[' . esc_attr( $args['key'] ) . ']" class="range-input">' . "\n";
-				foreach ( $args['data']['options'] as $k => $v ) {
-					$html .= '<option value="' . esc_attr( $k ) . '"' . selected( esc_attr( $options[$args['key']] ), $k, false ) . '>' . $v . '</option>' . "\n";
+			$html .= '<select id="'.esc_attr($args['key']).'" name="'.esc_attr($this->token).'['.esc_attr($args['key']).']" class="range-input">'."\n";
+				foreach ($args['data']['options'] as $k => $v) {
+					$html .= '<option value="'.esc_attr($k).'"'.selected(esc_attr($options[$args['key']]), $k, false).'>'.$v.'</option>'."\n";
 				}
-			$html .= '</select>' . "\n";
+			$html .= '</select>'."\n";
 			echo $html;
 
-			if ( isset( $args['data']['description'] ) ) {
-				echo '<p><span class="description">' . esc_html( $args['data']['description'] ) . '</span></p>' . "\n";
+			if (isset($args['data']['description'])) {
+				echo '<p><span class="description">'.esc_html($args['data']['description']).'</span></p>'."\n";
 			}
 		}
 	} // End form_field_range()
@@ -579,18 +579,18 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_images ( $args ) {
+	public function form_field_images($args) {
 		$options = $this->get_settings();
 
-		if ( isset( $args['data']['options'] ) && ( count( (array)$args['data']['options'] ) > 0 ) ) {
+		if (isset($args['data']['options']) && (count((array) $args['data']['options']) > 0)) {
 			$html = '';
-			foreach ( $args['data']['options'] as $k => $v ) {
-				$html .= '<input type="radio" name="' . esc_attr( $this->token ) . '[' . esc_attr( $args['key'] ) . ']" value="' . esc_attr( $k ) . '"' . checked( esc_attr( $options[$args['key']] ), $k, false ) . ' /> ' . $v . '<br />' . "\n";
+			foreach ($args['data']['options'] as $k => $v) {
+				$html .= '<input type="radio" name="'.esc_attr($this->token).'['.esc_attr($args['key']).']" value="'.esc_attr($k).'"'.checked(esc_attr($options[$args['key']]), $k, false).' /> '.$v.'<br />'."\n";
 			}
 			echo $html;
 
-			if ( isset( $args['data']['description'] ) ) {
-				echo '<span class="description">' . esc_html( $args['data']['description'] ) . '</span>' . "\n";
+			if (isset($args['data']['description'])) {
+				echo '<span class="description">'.esc_html($args['data']['description']).'</span>'."\n";
 			}
 		}
 	} // End form_field_images()
@@ -602,19 +602,19 @@ 
 	 * @param  array $args
 	 * @return void
 	 */
-	public function form_field_info ( $args ) {
+	public function form_field_info($args) {
 		$class = '';
-		if ( isset( $args['data']['class'] ) ) {
-			$class = ' ' . esc_attr( $args['data']['class'] );
+		if (isset($args['data']['class'])) {
+			$class = ' '.esc_attr($args['data']['class']);
 		}
-		$html = '<div id="' . $args['key'] . '" class="info-box' . $class . '">' . "\n";
-		if ( isset( $args['data']['name'] ) && ( $args['data']['name'] != '' ) ) {
-			$html .= '<h3 class="title">' . esc_html( $args['data']['name'] ) . '</h3>' . "\n";
+		$html = '<div id="'.$args['key'].'" class="info-box'.$class.'">'."\n";
+		if (isset($args['data']['name']) && ($args['data']['name'] != '')) {
+			$html .= '<h3 class="title">'.esc_html($args['data']['name']).'</h3>'."\n";
 		}
-		if ( isset( $args['data']['description'] ) && ( $args['data']['description'] != '' ) ) {
-			$html .= '<p>' . esc_html( $args['data']['description'] ) . '</p>' . "\n";
+		if (isset($args['data']['description']) && ($args['data']['description'] != '')) {
+			$html .= '<p>'.esc_html($args['data']['description']).'</p>'."\n";
 		}
-		$html .= '</div>' . "\n";
+		$html .= '</div>'."\n";
 
 		echo $html;
 	} // End form_field_info()
@@ -626,14 +626,14 @@ 
 	 * @since  1.9.0
 	 * @param  array $args
 	 */
-	public function form_field_button( $args ) {
+	public function form_field_button($args) {
 		$options = $this->get_settings();
 
-		if ( isset( $args['data']['target'] ) && isset( $args['data']['label'] ) ) {
-			printf( '<a href="%s" class="button button-secondary">%s</a> ', esc_url( $args['data']['target'] ), esc_html( $args['data']['label'] ) );
+		if (isset($args['data']['target']) && isset($args['data']['label'])) {
+			printf('<a href="%s" class="button button-secondary">%s</a> ', esc_url($args['data']['target']), esc_html($args['data']['label']));
 
-			if ( isset( $args['data']['description'] ) ) {
-				echo '<span class="description">' . esc_html( $args['data']['description'] ) . '</span>' . "\n";
+			if (isset($args['data']['description'])) {
+				echo '<span class="description">'.esc_html($args['data']['description']).'</span>'."\n";
 			}
 		}
 	} // End form_field_button()
@@ -647,23 +647,23 @@ 
 	 * @uses   $this->parse_errors()
 	 * @return array $options
 	 */
-	public function validate_fields ( $input ) {
+	public function validate_fields($input) {
 		$options = $this->get_settings();
 
-		foreach ( $this->fields as $k => $v ) {
+		foreach ($this->fields as $k => $v) {
 			// Make sure checkboxes are present even when false.
-			if ( $v['type'] == 'checkbox' && ! isset( $input[$k] ) ) { $input[$k] = false; }
-			if ( $v['type'] == 'multicheck' && ! isset( $input[$k] ) ) { $input[$k] = false; }
+			if ($v['type'] == 'checkbox' && ! isset($input[$k])) { $input[$k] = false; }
+			if ($v['type'] == 'multicheck' && ! isset($input[$k])) { $input[$k] = false; }
 
-			if ( isset( $input[$k] ) ) {
+			if (isset($input[$k])) {
 				// Perform checks on required fields.
-				if ( isset( $v['required'] ) && ( $v['required'] == true ) ) {
-					if ( in_array( $v['type'], $this->get_array_field_types() ) && ( count( (array) $input[$k] ) <= 0 ) ) {
-						$this->add_error( $k, $v );
+				if (isset($v['required']) && ($v['required'] == true)) {
+					if (in_array($v['type'], $this->get_array_field_types()) && (count((array) $input[$k]) <= 0)) {
+						$this->add_error($k, $v);
 						continue;
 					} else {
-						if ( $input[$k] == '' ) {
-							$this->add_error( $k, $v );
+						if ($input[$k] == '') {
+							$this->add_error($k, $v);
 							continue;
 						}
 					}
@@ -672,28 +672,28 @@ 
 				$value = $input[$k];
 
 				// Check if the field is valid.
-				$method = $this->determine_method( $v, 'check' );
+				$method = $this->determine_method($v, 'check');
 
-				if ( function_exists ( $method ) ) {
-					$is_valid = $method( $value );
+				if (function_exists($method)) {
+					$is_valid = $method($value);
 				} else {
-					if ( method_exists( $this, $method ) ) {
-						$is_valid = $this->$method( $value );
+					if (method_exists($this, $method)) {
+						$is_valid = $this->$method($value);
 					}
 				}
 
-				if ( ! $is_valid ) {
-					$this->add_error( $k, $v );
+				if ( ! $is_valid) {
+					$this->add_error($k, $v);
 					continue;
 				}
 
-				$method = $this->determine_method( $v, 'validate' );
+				$method = $this->determine_method($v, 'validate');
 
-				if ( function_exists ( $method ) ) {
-					$options[$k] = $method( $value );
+				if (function_exists($method)) {
+					$options[$k] = $method($value);
 				} else {
-					if ( method_exists( $this, $method ) ) {
-						$options[$k] = $this->$method( $value );
+					if (method_exists($this, $method)) {
+						$options[$k] = $this->$method($value);
 					}
 				}
 			}
@@ -711,8 +711,8 @@ 
 	 * @param  string $input
 	 * @return string
 	 */
-	public function validate_field_text ( $input ) {
-		return trim( esc_attr( $input ) );
+	public function validate_field_text($input) {
+		return trim(esc_attr($input));
 	} // End validate_field_text()
 
 	/**
@@ -722,11 +722,11 @@ 
 	 * @param  string $input
 	 * @return string
 	 */
-	public function validate_field_checkbox ( $input ) {
-		if ( ! isset( $input ) ) {
+	public function validate_field_checkbox($input) {
+		if ( ! isset($input)) {
 			return 0;
 		} else {
-			return (bool)$input;
+			return (bool) $input;
 		}
 	} // End validate_field_checkbox()
 
@@ -737,10 +737,10 @@ 
 	 * @param  string $input
 	 * @return string
 	 */
-	public function validate_field_multicheck ( $input ) {
+	public function validate_field_multicheck($input) {
 		$input = (array) $input;
 
-		$input = array_map( 'esc_attr', $input );
+		$input = array_map('esc_attr', $input);
 
 		return $input;
 	} // End validate_field_multicheck()
@@ -752,8 +752,8 @@ 
 	 * @param  string $input
 	 * @return string
 	 */
-	public function validate_field_range ( $input ) {
-		$input = number_format( floatval( $input ), 0 );
+	public function validate_field_range($input) {
+		$input = number_format(floatval($input), 0);
 
 		return $input;
 	} // End validate_field_range()
@@ -765,8 +765,8 @@ 
 	 * @param  string $input
 	 * @return string
 	 */
-	public function validate_field_url ( $input ) {
-		return trim( esc_url( $input ) );
+	public function validate_field_url($input) {
+		return trim(esc_url($input));
 	} // End validate_field_url()
 
 	/**
@@ -775,7 +775,7 @@ 
 	 * @since  1.1.0
 	 * @return boolean Is the value valid?
 	 */
-	public function check_field_text ( $input ) {
+	public function check_field_text($input) {
 		$is_valid = true;
 
 		return $is_valid;
@@ -789,11 +789,11 @@ 
 	 * @param  array $data
 	 * @return void
 	 */
-	protected function add_error ( $key, $data ) {
-		if ( isset( $data['error_message'] ) ) {
+	protected function add_error($key, $data) {
+		if (isset($data['error_message'])) {
 			$message = $data['error_message'];
 		} else {
-			$message = sprintf( __( '%s is a required field', 'woothemes-sensei' ), $data['name'] );
+			$message = sprintf(__('%s is a required field', 'woothemes-sensei'), $data['name']);
 		}
 		$this->errors[$key] = $message;
 	} // End add_error()
@@ -804,14 +804,14 @@ 
 	 * @since   1.0.0
 	 * @return  void
 	 */
-	protected function parse_errors () {
-		if ( count ( $this->errors ) > 0 ) {
-			foreach ( $this->errors as $k => $v ) {
-				add_settings_error( $this->token . '-errors', $k, $v, 'error' );
+	protected function parse_errors() {
+		if (count($this->errors) > 0) {
+			foreach ($this->errors as $k => $v) {
+				add_settings_error($this->token.'-errors', $k, $v, 'error');
 			}
 		} else {
-			$message = sprintf( __( '%s updated', 'woothemes-sensei' ), $this->name );
-			add_settings_error( $this->token . '-errors', $this->token, $message, 'updated' );
+			$message = sprintf(__('%s updated', 'woothemes-sensei'), $this->name);
+			add_settings_error($this->token.'-errors', $this->token, $message, 'updated');
 		}
 	} // End parse_errors()
 
@@ -821,8 +821,8 @@ 
 	 * @since  1.0.0
 	 * @return array
 	 */
-	protected function get_array_field_types () {
-		return array( 'multicheck' );
+	protected function get_array_field_types() {
+		return array('multicheck');
 	} // End get_array_field_types()
 
 	/**
@@ -831,21 +831,21 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function enqueue_scripts () {
+	public function enqueue_scripts() {
 
-        $suffix = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
+        $suffix = defined('SCRIPT_DEBUG') && SCRIPT_DEBUG ? '' : '.min';
 
-		wp_enqueue_script( 'farbtastic' );
-		wp_enqueue_script( 'woothemes-sensei-settings', esc_url( Sensei()->plugin_url . 'assets/js/settings' . $suffix . '.js' ), array( 'jquery', 'farbtastic' ), Sensei()->version );
+		wp_enqueue_script('farbtastic');
+		wp_enqueue_script('woothemes-sensei-settings', esc_url(Sensei()->plugin_url.'assets/js/settings'.$suffix.'.js'), array('jquery', 'farbtastic'), Sensei()->version);
 
-		if ( $this->has_range ) {
-			wp_enqueue_script( 'woothemes-sensei-settings-ranges', esc_url( Sensei()->plugin_url . 'assets/js/ranges' . $suffix . '.js' ), array( 'jquery-ui-slider' ), Sensei()->version );
+		if ($this->has_range) {
+			wp_enqueue_script('woothemes-sensei-settings-ranges', esc_url(Sensei()->plugin_url.'assets/js/ranges'.$suffix.'.js'), array('jquery-ui-slider'), Sensei()->version);
 		}
 
-		wp_register_script( 'woothemes-sensei-settings-imageselectors', esc_url( Sensei()->plugin_url . 'assets/js/image-selectors' . $suffix . '.js' ), array( 'jquery' ), Sensei()->version );
+		wp_register_script('woothemes-sensei-settings-imageselectors', esc_url(Sensei()->plugin_url.'assets/js/image-selectors'.$suffix.'.js'), array('jquery'), Sensei()->version);
 
-		if ( $this->has_imageselector ) {
-			wp_enqueue_script( 'woothemes-sensei-settings-imageselectors' );
+		if ($this->has_imageselector) {
+			wp_enqueue_script('woothemes-sensei-settings-imageselectors');
 		}
 
 	} // End enqueue_scripts()
@@ -856,12 +856,12 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function enqueue_styles () {
+	public function enqueue_styles() {
 
-		wp_enqueue_style( $this->token . '-admin' );
+		wp_enqueue_style($this->token.'-admin');
 
-		wp_enqueue_style( 'farbtastic' );
-		wp_enqueue_style( 'woothemes-sensei-settings-api', esc_url( Sensei()->plugin_url . 'assets/css/settings.css' ), array( 'farbtastic' ), Sensei()->version );
+		wp_enqueue_style('farbtastic');
+		wp_enqueue_style('woothemes-sensei-settings-api', esc_url(Sensei()->plugin_url.'assets/css/settings.css'), array('farbtastic'), Sensei()->version);
 
 		$this->enqueue_field_styles();
 	} // End enqueue_styles()
@@ -872,16 +872,16 @@ 
 	 * @since  1.0.0
 	 * @return void
 	 */
-	public function enqueue_field_styles () {
+	public function enqueue_field_styles() {
 
-		if ( $this->has_range ) {
-			wp_enqueue_style( 'woothemes-sensei-settings-ranges', esc_url( Sensei()->plugin_url . 'assets/css/ranges.css' ), '', Sensei()->version );
+		if ($this->has_range) {
+			wp_enqueue_style('woothemes-sensei-settings-ranges', esc_url(Sensei()->plugin_url.'assets/css/ranges.css'), '', Sensei()->version);
 		}
 
-		wp_register_style( 'woothemes-sensei-settings-imageselectors', esc_url( Sensei()->plugin_url . 'assets/css/image-selectors.css' ), '', Sensei()->version );
+		wp_register_style('woothemes-sensei-settings-imageselectors', esc_url(Sensei()->plugin_url.'assets/css/image-selectors.css'), '', Sensei()->version);
 
-		if ( $this->has_imageselector ) {
-			wp_enqueue_style( 'woothemes-sensei-settings-imageselectors' );
+		if ($this->has_imageselector) {
+			wp_enqueue_style('woothemes-sensei-settings-imageselectors');
 		}
 	} // End enqueue_field_styles()
 } // End Class
@@ -891,4 +891,4 @@ 
  * @ignore only for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Settings_API extends Sensei_Settings_API{}
+class WooThemes_Sensei_Settings_API extends Sensei_Settings_API {}

Braces +7 added lines, -3 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * A settings API (wrapping the WordPress Settings API).
@@ -315,8 +318,9 @@ 
 
             echo '<section id="' . $section_id . '">';
 
-            if ($section['name'])
-                echo "<h2>{$section['name']}</h2>\n";
+            if ($section['name']) {
+                            echo "<h2>{$section['name']}</h2>\n";
+            }
 
             echo '<table class="form-table">';
             do_settings_fields($page, $section_id );