lightspeeddevelopment / lsx-sharing

includes/functions.php

Indentation +72 added lines, -72 removed lines

@@ -13,24 +13,24 @@ 
  * @return array
  */
 function get_restricted_post_types() {
-	$post_types = array(
-		'page',
-		'attachment',
-		'forum',
-		'topic',
-		'lesson',
-		'quiz',
-		'question',
-		'reply',
-		'popup',
-		'sensei_message',
-		'envira',
-		'soliloquy',
-		'certificate_template',
-		'certificate',
-		'project',
-	);
-	return apply_filters( 'lsx_sharing_get_restricted_post_types', $post_types );
+     $post_types = array(
+          'page',
+          'attachment',
+          'forum',
+          'topic',
+          'lesson',
+          'quiz',
+          'question',
+          'reply',
+          'popup',
+          'sensei_message',
+          'envira',
+          'soliloquy',
+          'certificate_template',
+          'certificate',
+          'project',
+     );
+     return apply_filters( 'lsx_sharing_get_restricted_post_types', $post_types );
 }
 
 /**
@@ -39,16 +39,16 @@ 
  * @return array
  */
 function get_to_post_types() {
-	$post_types = array(
-		'accommodation',
-		'tour',
-		'destination',
-		'review',
-		'special',
-		'vehicle',
-		'activity',
-	);
-	return apply_filters( 'lsx_sharing_get_to_post_types', $post_types );
+     $post_types = array(
+          'accommodation',
+          'tour',
+          'destination',
+          'review',
+          'special',
+          'vehicle',
+          'activity',
+     );
+     return apply_filters( 'lsx_sharing_get_to_post_types', $post_types );
 }
 
 /**
@@ -57,14 +57,14 @@ 
  * @return array
  */
 function get_hp_post_types() {
-	$post_types = array(
-		'workout',
-		'exercise',
-		'recipe',
-		'meal',
-		'tip',
-	);
-	return apply_filters( 'lsx_sharing_get_hp_post_types', $post_types );
+     $post_types = array(
+          'workout',
+          'exercise',
+          'recipe',
+          'meal',
+          'tip',
+     );
+     return apply_filters( 'lsx_sharing_get_hp_post_types', $post_types );
 }
 
 /**
@@ -73,14 +73,14 @@ 
  * @return boolean
  */
 function is_button_disabled( $post_type = '', $service = '' ) {
-	$sharing = lsx_sharing();
-	$option  = false;
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display'][ 'sharing_disable_' . $service ] ) ) {
-		$option = true;
-	} elseif ( true === $sharing->is_new_options && ! empty( $sharing->options[ $post_type . '_disable_' . $service ] ) ) {
-		$option = true;
-	}
-	return apply_filters( 'lsx_sharing_is_button_disabled', $option );
+     $sharing = lsx_sharing();
+     $option  = false;
+     if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display'][ 'sharing_disable_' . $service ] ) ) {
+          $option = true;
+     } elseif ( true === $sharing->is_new_options && ! empty( $sharing->options[ $post_type . '_disable_' . $service ] ) ) {
+          $option = true;
+     }
+     return apply_filters( 'lsx_sharing_is_button_disabled', $option );
 }
 
 /**
@@ -89,14 +89,14 @@ 
  * @return boolean
  */
 function is_pt_disabled( $post_type = '' ) {
-	$sharing = lsx_sharing();
-	$option  = false;
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display'][ 'sharing_disable_pt_' . $post_type ] ) ) {
-		$option = true;
-	} elseif ( true === $sharing->is_new_options && isset( $sharing->options[ $post_type . '_disable_pt' ] ) ) {
-		$option = true;
-	}
-	return apply_filters( 'lsx_sharing_is_pt_disabled', $option );
+     $sharing = lsx_sharing();
+     $option  = false;
+     if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display'][ 'sharing_disable_pt_' . $post_type ] ) ) {
+          $option = true;
+     } elseif ( true === $sharing->is_new_options && isset( $sharing->options[ $post_type . '_disable_pt' ] ) ) {
+          $option = true;
+     }
+     return apply_filters( 'lsx_sharing_is_pt_disabled', $option );
 }
 
 /**
@@ -105,14 +105,14 @@ 
  * @return boolean
  */
 function is_disabled() {
-	$sharing = lsx_sharing();
-	$option  = false;
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display']['sharing_disable_all'] ) ) {
-		$option = true;
-	} elseif ( true === $sharing->is_new_options && isset( $sharing->options['global_disable_all'] ) ) {
-		$option = true;
-	}
-	return apply_filters( 'lsx_sharing_is_disabled', $option );
+     $sharing = lsx_sharing();
+     $option  = false;
+     if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display']['sharing_disable_all'] ) ) {
+          $option = true;
+     } elseif ( true === $sharing->is_new_options && isset( $sharing->options['global_disable_all'] ) ) {
+          $option = true;
+     }
+     return apply_filters( 'lsx_sharing_is_disabled', $option );
 }
 
 /**
@@ -121,16 +121,16 @@ 
  * @return string
  */
 function get_sharing_text( $post_type = '' ) {
-	$sharing = lsx_sharing();
-	$text    = '';
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display']['sharing_label_text'] ) ) {
-		$text = $sharing->options['display']['sharing_label_text'];
-	} elseif ( true === $sharing->is_new_options ) {
-		if ( isset( $sharing->options[ $post_type . '_label_text' ] ) ) {
-			$text = $sharing->options[ $post_type . '_label_text' ];
-		} elseif ( isset( $sharing->options['global_label_text'] ) ) {
-			$text = $sharing->options['global_label_text'];
-		}
-	}
-	return $text;
+     $sharing = lsx_sharing();
+     $text    = '';
+     if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display']['sharing_label_text'] ) ) {
+          $text = $sharing->options['display']['sharing_label_text'];
+     } elseif ( true === $sharing->is_new_options ) {
+          if ( isset( $sharing->options[ $post_type . '_label_text' ] ) ) {
+               $text = $sharing->options[ $post_type . '_label_text' ];
+          } elseif ( isset( $sharing->options['global_label_text'] ) ) {
+               $text = $sharing->options['global_label_text'];
+          }
+     }
+     return $text;
 }

Spacing +43 added lines, -43 removed lines

@@ -12,8 +12,8 @@ 
  *
  * @return array
  */
-function get_restricted_post_types() {
-	$post_types = array(
+function get_restricted_post_types(){
+	$post_types=array(
 		'page',
 		'attachment',
 		'forum',
@@ -30,7 +30,7 @@ 
 		'certificate',
 		'project',
 	);
-	return apply_filters( 'lsx_sharing_get_restricted_post_types', $post_types );
+	return apply_filters('lsx_sharing_get_restricted_post_types',$post_types);
 }
 
 /**
@@ -38,8 +38,8 @@ 
  *
  * @return array
  */
-function get_to_post_types() {
-	$post_types = array(
+function get_to_post_types(){
+	$post_types=array(
 		'accommodation',
 		'tour',
 		'destination',
@@ -48,7 +48,7 @@ 
 		'vehicle',
 		'activity',
 	);
-	return apply_filters( 'lsx_sharing_get_to_post_types', $post_types );
+	return apply_filters('lsx_sharing_get_to_post_types',$post_types);
 }
 
 /**
@@ -56,15 +56,15 @@ 
  *
  * @return array
  */
-function get_hp_post_types() {
-	$post_types = array(
+function get_hp_post_types(){
+	$post_types=array(
 		'workout',
 		'exercise',
 		'recipe',
 		'meal',
 		'tip',
 	);
-	return apply_filters( 'lsx_sharing_get_hp_post_types', $post_types );
+	return apply_filters('lsx_sharing_get_hp_post_types',$post_types);
 }
 
 /**
@@ -72,15 +72,15 @@ 
  *
  * @return boolean
  */
-function is_button_disabled( $post_type = '', $service = '' ) {
-	$sharing = lsx_sharing();
-	$option  = false;
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display'][ 'sharing_disable_' . $service ] ) ) {
-		$option = true;
-	} elseif ( true === $sharing->is_new_options && ! empty( $sharing->options[ $post_type . '_disable_' . $service ] ) ) {
-		$option = true;
+function is_button_disabled($post_type='',$service=''){
+	$sharing=lsx_sharing();
+	$option=false;
+	if(false===$sharing->is_new_options&&isset($sharing->options['display'])&&!empty($sharing->options['display']['sharing_disable_'.$service])){
+		$option=true;
+	} elseif(true===$sharing->is_new_options&&!empty($sharing->options[$post_type.'_disable_'.$service])){
+		$option=true;
 	}
-	return apply_filters( 'lsx_sharing_is_button_disabled', $option );
+	return apply_filters('lsx_sharing_is_button_disabled',$option);
 }
 
 /**
@@ -88,15 +88,15 @@ 
  *
  * @return boolean
  */
-function is_pt_disabled( $post_type = '' ) {
-	$sharing = lsx_sharing();
-	$option  = false;
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display'][ 'sharing_disable_pt_' . $post_type ] ) ) {
-		$option = true;
-	} elseif ( true === $sharing->is_new_options && isset( $sharing->options[ $post_type . '_disable_pt' ] ) ) {
-		$option = true;
+function is_pt_disabled($post_type=''){
+	$sharing=lsx_sharing();
+	$option=false;
+	if(false===$sharing->is_new_options&&isset($sharing->options['display'])&&!empty($sharing->options['display']['sharing_disable_pt_'.$post_type])){
+		$option=true;
+	} elseif(true===$sharing->is_new_options&&isset($sharing->options[$post_type.'_disable_pt'])){
+		$option=true;
 	}
-	return apply_filters( 'lsx_sharing_is_pt_disabled', $option );
+	return apply_filters('lsx_sharing_is_pt_disabled',$option);
 }
 
 /**
@@ -104,15 +104,15 @@ 
  *
  * @return boolean
  */
-function is_disabled() {
-	$sharing = lsx_sharing();
-	$option  = false;
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display']['sharing_disable_all'] ) ) {
-		$option = true;
-	} elseif ( true === $sharing->is_new_options && isset( $sharing->options['global_disable_all'] ) ) {
-		$option = true;
+function is_disabled(){
+	$sharing=lsx_sharing();
+	$option=false;
+	if(false===$sharing->is_new_options&&isset($sharing->options['display'])&&!empty($sharing->options['display']['sharing_disable_all'])){
+		$option=true;
+	} elseif(true===$sharing->is_new_options&&isset($sharing->options['global_disable_all'])){
+		$option=true;
 	}
-	return apply_filters( 'lsx_sharing_is_disabled', $option );
+	return apply_filters('lsx_sharing_is_disabled',$option);
 }
 
 /**
@@ -120,16 +120,16 @@ 
  *
  * @return string
  */
-function get_sharing_text( $post_type = '' ) {
-	$sharing = lsx_sharing();
-	$text    = '';
-	if ( false === $sharing->is_new_options && isset( $sharing->options['display'] ) && ! empty( $sharing->options['display']['sharing_label_text'] ) ) {
-		$text = $sharing->options['display']['sharing_label_text'];
-	} elseif ( true === $sharing->is_new_options ) {
-		if ( isset( $sharing->options[ $post_type . '_label_text' ] ) ) {
-			$text = $sharing->options[ $post_type . '_label_text' ];
-		} elseif ( isset( $sharing->options['global_label_text'] ) ) {
-			$text = $sharing->options['global_label_text'];
+function get_sharing_text($post_type=''){
+	$sharing=lsx_sharing();
+	$text='';
+	if(false===$sharing->is_new_options&&isset($sharing->options['display'])&&!empty($sharing->options['display']['sharing_label_text'])){
+		$text=$sharing->options['display']['sharing_label_text'];
+	} elseif(true===$sharing->is_new_options){
+		if(isset($sharing->options[$post_type.'_label_text'])){
+			$text=$sharing->options[$post_type.'_label_text'];
+		} elseif(isset($sharing->options['global_label_text'])){
+			$text=$sharing->options['global_label_text'];
 		}
 	}
 	return $text;

classes/admin/class-settings.php

Indentation +216 added lines, -216 removed lines

@@ -9,257 +9,257 @@
 class Settings {
 
 
-    /**
-     * Holds class instance
-     *
-     * @since 1.0.0
-     *
-     * @var object \lsx\sharing\classes\admin\Settings()
-     */
-    protected static $instance = null;
+     /**
+      * Holds class instance
+      *
+      * @since 1.0.0
+      *
+      * @var object \lsx\sharing\classes\admin\Settings()
+      */
+     protected static $instance = null;
 
-    /**
-     * Contructor
-     */
-    public function __construct() {
-         add_action('cmb2_admin_init', array( $this, 'register_settings_page' ));
-        add_action('lsx_sharing_settings_page', array( $this, 'configure_general_fields' ), 15, 1);
-        add_action('lsx_sharing_settings_page', array( $this, 'configure_archive_fields' ), 15, 1);
-        add_action('admin_enqueue_scripts', array( $this, 'assets' ));
-    }
+     /**
+      * Contructor
+      */
+     public function __construct() {
+          add_action('cmb2_admin_init', array( $this, 'register_settings_page' ));
+          add_action('lsx_sharing_settings_page', array( $this, 'configure_general_fields' ), 15, 1);
+          add_action('lsx_sharing_settings_page', array( $this, 'configure_archive_fields' ), 15, 1);
+          add_action('admin_enqueue_scripts', array( $this, 'assets' ));
+     }
 
-    /**
-     * Return an instance of this class.
-     *
-     * @since 1.0.0
-     *
-     * @return object \lsx\sharing\classes\admin\Settings()    A single instance of this class.
-     */
-    public static function get_instance() {
-         // If the single instance hasn't been set, set it now.
-        if ( null == self::$instance ) {
-            self::$instance = new self();
-        }
-        return self::$instance;
-    }
+     /**
+      * Return an instance of this class.
+      *
+      * @since 1.0.0
+      *
+      * @return object \lsx\sharing\classes\admin\Settings()    A single instance of this class.
+      */
+     public static function get_instance() {
+          // If the single instance hasn't been set, set it now.
+          if ( null == self::$instance ) {
+               self::$instance = new self();
+          }
+          return self::$instance;
+     }
 
-    /**
-     * Configure fields for the Settings page.
-     *
-     * @return void
-     */
-    public function register_settings_page() {
-         $args = array(
-			 'id'           => 'lsx_sharing_settings',
-			 'title'        => '<h1>' . esc_html__('LSX Sharing Settings', 'lsx-search') . ' <span class="version">' . LSX_SHARING_VER . '</span></h1>',
-			 'menu_title'   => esc_html__('LSX Sharing', 'search'), // Falls back to 'title' (above).
-			 'object_types' => array( 'options-page' ),
-			 'option_key'   => 'lsx-sharing-settings', // The option key and admin menu page slug.
-			 'parent_slug'  => 'options-general.php',
-			 'capability'   => 'manage_options', // Cap required to view options-page.
-		 );
-        $cmb  = new_cmb2_box($args);
-        do_action('lsx_sharing_settings_page', $cmb);
-    }
+     /**
+      * Configure fields for the Settings page.
+      *
+      * @return void
+      */
+     public function register_settings_page() {
+          $args = array(
+                    'id'           => 'lsx_sharing_settings',
+                    'title'        => '<h1>' . esc_html__('LSX Sharing Settings', 'lsx-search') . ' <span class="version">' . LSX_SHARING_VER . '</span></h1>',
+                    'menu_title'   => esc_html__('LSX Sharing', 'search'), // Falls back to 'title' (above).
+                    'object_types' => array( 'options-page' ),
+                    'option_key'   => 'lsx-sharing-settings', // The option key and admin menu page slug.
+                    'parent_slug'  => 'options-general.php',
+                    'capability'   => 'manage_options', // Cap required to view options-page.
+               );
+          $cmb  = new_cmb2_box($args);
+          do_action('lsx_sharing_settings_page', $cmb);
+     }
 
-    /**
-     * Enqueue JS and CSS.
-     */
-    public function assets( $hook ) {
-         wp_enqueue_style('lsx-sharing-admin', LSX_SHARING_URL . 'assets/css/lsx-sharing-admin.css', array(), LSX_SHARING_VER);
-    }
+     /**
+      * Enqueue JS and CSS.
+      */
+     public function assets( $hook ) {
+          wp_enqueue_style('lsx-sharing-admin', LSX_SHARING_URL . 'assets/css/lsx-sharing-admin.css', array(), LSX_SHARING_VER);
+     }
 
-    /**
-     * Enable Business Directory Search settings only if LSX Search plugin is enabled.
-     *
-     * @return void
-     */
-    public function configure_general_fields( $cmb ) {
-         $global_args = array(
-			 'title' => __('Global', 'lsx-search'),
-			 'desc'  => esc_html__('These settings will control sharing icons everywhere.', 'lsx-search'),
-		 );
-        $this->get_fields($cmb, 'global', $global_args);
-    }
+     /**
+      * Enable Business Directory Search settings only if LSX Search plugin is enabled.
+      *
+      * @return void
+      */
+     public function configure_general_fields( $cmb ) {
+          $global_args = array(
+                    'title' => __('Global', 'lsx-search'),
+                    'desc'  => esc_html__('These settings will control sharing icons everywhere.', 'lsx-search'),
+               );
+          $this->get_fields($cmb, 'global', $global_args);
+     }
 
-    /**
-     * Enable Sharing settings only if LSX Search plugin is enabled.
-     *
-     * @param  object $cmb      The CMB2() class.
-     * @param  string $position either top of bottom.
-     * @return void
-     */
-    public function configure_archive_fields( $cmb ) {
-         $archives       = array();
-        $post_type_args = array(
-			'public' => true,
-        );
-        $post_types     = get_post_types($post_type_args);
-        if ( ! empty($post_types) ) {
-            foreach ( $post_types as $post_type_key => $post_type_value ) {
-                switch ( $post_type_key ) {
-                case 'post':
+     /**
+      * Enable Sharing settings only if LSX Search plugin is enabled.
+      *
+      * @param  object $cmb      The CMB2() class.
+      * @param  string $position either top of bottom.
+      * @return void
+      */
+     public function configure_archive_fields( $cmb ) {
+          $archives       = array();
+          $post_type_args = array(
+               'public' => true,
+          );
+          $post_types     = get_post_types($post_type_args);
+          if ( ! empty($post_types) ) {
+               foreach ( $post_types as $post_type_key => $post_type_value ) {
+                    switch ( $post_type_key ) {
+                    case 'post':
                     $page_url      = home_url();
                     $page_title    = __('Home', 'lsx-search');
                     $show_on_front = get_option('show_on_front');
                     if ( 'page' === $show_on_front ) {
-                            $page_for_posts = get_option('page_for_posts');
-                        if ( '' !== $page_for_posts ) {
-                            $page_title   = get_the_title($page_for_posts);
-                            $page_url     = get_permalink($page_for_posts);
-                        }
+                              $page_for_posts = get_option('page_for_posts');
+                         if ( '' !== $page_for_posts ) {
+                              $page_title   = get_the_title($page_for_posts);
+                              $page_url     = get_permalink($page_for_posts);
+                         }
                     }
                     $description = sprintf(
                     /* translators: %s: The subscription info */
-                        __('Control the sharing buttons on your <a target="_blank" href="%1$s">%2$s</a> posts.', 'lsx-search'),
-                        $page_url,
-                        $page_title
+                         __('Control the sharing buttons on your <a target="_blank" href="%1$s">%2$s</a> posts.', 'lsx-search'),
+                         $page_url,
+                         $page_title
                     );
                     $archives[ $post_type_key ] = array(
-						'title' => __('Blog', 'lsx-search'),
-						'desc'  => $description,
+                              'title' => __('Blog', 'lsx-search'),
+                              'desc'  => $description,
                     );
-                        break;
+                         break;
 
-                case 'product':
+                    case 'product':
                     $page_url = home_url();
                     $page_title    = __('Shop', 'lsx-search');
                     if ( function_exists('wc_get_page_id') ) {
-                        $shop_page  = wc_get_page_id('shop');
-                        $page_url   = get_permalink($shop_page);
-                        $page_title = get_the_title($shop_page);
+                         $shop_page  = wc_get_page_id('shop');
+                         $page_url   = get_permalink($shop_page);
+                         $page_title = get_the_title($shop_page);
                     }
                     $description = sprintf(
                     /* translators: %s: The subscription info */
-                        __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> product pages.', 'lsx-search'),
-                        $page_url,
-                        $page_title
+                         __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> product pages.', 'lsx-search'),
+                         $page_url,
+                         $page_title
                     );
                     $archives[ $post_type_key ] = array(
-						'title' => __('Shop', 'lsx-search'),
-						'desc'  => $description,
+                              'title' => __('Shop', 'lsx-search'),
+                              'desc'  => $description,
                     );
-                        break;
+                         break;
 
-                default:
+                    default:
                     if ( in_array($post_type_key, \lsx\sharing\includes\functions\get_restricted_post_types()) ) {
-                            break;
+                              break;
                     }
                     $temp_post_type = get_post_type_object($post_type_key);
                     if ( ! is_wp_error($temp_post_type) ) {
-                        $page_url    = get_post_type_archive_link($temp_post_type->name);
-                        $description = sprintf(
-                        /* translators: %s: The subscription info */
-                            __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> singles.', 'lsx-search'),
-                            $page_url,
-                            $temp_post_type->label
-                        );
+                         $page_url    = get_post_type_archive_link($temp_post_type->name);
+                         $description = sprintf(
+                         /* translators: %s: The subscription info */
+                              __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> singles.', 'lsx-search'),
+                              $page_url,
+                              $temp_post_type->label
+                         );
 
-                        $archives[ $post_type_key ] = array(
-							'title' => $temp_post_type->label,
-							'desc'  => $description,
-                        );
+                         $archives[ $post_type_key ] = array(
+                                   'title' => $temp_post_type->label,
+                                   'desc'  => $description,
+                         );
                     }
-                        break;
-                }
-            }
-        }
-        if ( ! empty($archives) ) {
-            foreach ( $archives as $archive_key => $archive_args ) {
-                $this->get_fields($cmb, $archive_key, $archive_args);
-            }
-        }
-    }
-    /**
-     * Gets the sharing fields and loops through them.
-     *
-     * @param  object $cmb
-     * @param  string $section
-     * @param  array  $args
-     * @return void
-     */
-    public function get_fields( $cmb, $section, $args ) {
-         $cmb->add_field(
-            array(
-				'id'          => 'settings_' . $section . '_sharing',
-				'type'        => 'title',
-				'name'        => $args['title'],
-				'default'     => $args['title'],
-				'description' => $args['desc'],
-            )
-        );
-        if ( 'global' === $section ) {
-            $cmb->add_field(
-                array(
-					'name'        => esc_html__('Disable all', 'lsx-sharing'),
-					'id'          => $section . '_disable_all',
-					'description' => esc_html__('Disable all sharing buttons on the site', 'lsx-sharing'),
-					'type'        => 'checkbox',
-                )
-            );
-        } else {
-            $cmb->add_field(
-                array(
-					'name'        => esc_html__('Disable', 'lsx-sharing'),
-					'id'          => $section . '_disable_pt',
-					'description' => esc_html__('Disable the share buttons on this post type', 'lsx-sharing'),
-					'type'        => 'checkbox',
-                )
-            );
-        }
+                         break;
+                    }
+               }
+          }
+          if ( ! empty($archives) ) {
+               foreach ( $archives as $archive_key => $archive_args ) {
+                    $this->get_fields($cmb, $archive_key, $archive_args);
+               }
+          }
+     }
+     /**
+      * Gets the sharing fields and loops through them.
+      *
+      * @param  object $cmb
+      * @param  string $section
+      * @param  array  $args
+      * @return void
+      */
+     public function get_fields( $cmb, $section, $args ) {
+          $cmb->add_field(
+               array(
+                    'id'          => 'settings_' . $section . '_sharing',
+                    'type'        => 'title',
+                    'name'        => $args['title'],
+                    'default'     => $args['title'],
+                    'description' => $args['desc'],
+               )
+          );
+          if ( 'global' === $section ) {
+               $cmb->add_field(
+                    array(
+                         'name'        => esc_html__('Disable all', 'lsx-sharing'),
+                         'id'          => $section . '_disable_all',
+                         'description' => esc_html__('Disable all sharing buttons on the site', 'lsx-sharing'),
+                         'type'        => 'checkbox',
+                    )
+               );
+          } else {
+               $cmb->add_field(
+                    array(
+                         'name'        => esc_html__('Disable', 'lsx-sharing'),
+                         'id'          => $section . '_disable_pt',
+                         'description' => esc_html__('Disable the share buttons on this post type', 'lsx-sharing'),
+                         'type'        => 'checkbox',
+                    )
+               );
+          }
 
-		if ( 'global' === $section ) {
-			$label_text_description = esc_html__( 'If no text is specified per post type this text will display.' , 'lsx-sharing' );
-		} else {
-			$label_text_description = esc_html__( 'This text will display alongside the sharing buttons.', 'lsx-sharing' );
-		}
+          if ( 'global' === $section ) {
+               $label_text_description = esc_html__( 'If no text is specified per post type this text will display.' , 'lsx-sharing' );
+          } else {
+               $label_text_description = esc_html__( 'This text will display alongside the sharing buttons.', 'lsx-sharing' );
+          }
 		
-        $cmb->add_field(
-			array(
-				'name'        => esc_html__( 'Label text', 'lsx-sharing' ),
-				'id'          => $section . '_label_text',
-				'type'        => 'text',
-				'description' => $label_text_description,
-			)
-		);
+          $cmb->add_field(
+               array(
+                    'name'        => esc_html__( 'Label text', 'lsx-sharing' ),
+                    'id'          => $section . '_label_text',
+                    'type'        => 'text',
+                    'description' => $label_text_description,
+               )
+          );
 		
-        if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'facebook') ) ) {
-            $cmb->add_field(
-                array(
-					'name'        => esc_html__('Disable Facebook', 'lsx-sharing'),
-					'id'          => $section . '_disable_facebook',
-					'description' => esc_html__('Disable Facebook share button.', 'lsx-sharing'),
-					'type'        => 'checkbox',
-                )
-            );
-        }
-        if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'twitter') ) ) {
-            $cmb->add_field(
-                array(
-					'name'        => esc_html__('Disable Twitter', 'lsx-sharing'),
-					'id'          => $section . '_disable_twitter',
-					'description' => esc_html__('Disable Twitter share button.', 'lsx-sharing'),
-					'type'        => 'checkbox',
-                )
-            );
-        }
-        if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'pinterest') ) ) {
-            $cmb->add_field(
-                array(
-					'name'        => esc_html__('Disable Pinterest', 'lsx-sharing'),
-					'id'          => $section . '_disable_pinterest',
-					'description' => esc_html__('Disable Pinterest button.', 'lsx-sharing'),
-					'type'        => 'checkbox',
-                )
-            );
-        }
+          if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'facebook') ) ) {
+               $cmb->add_field(
+                    array(
+                         'name'        => esc_html__('Disable Facebook', 'lsx-sharing'),
+                         'id'          => $section . '_disable_facebook',
+                         'description' => esc_html__('Disable Facebook share button.', 'lsx-sharing'),
+                         'type'        => 'checkbox',
+                    )
+               );
+          }
+          if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'twitter') ) ) {
+               $cmb->add_field(
+                    array(
+                         'name'        => esc_html__('Disable Twitter', 'lsx-sharing'),
+                         'id'          => $section . '_disable_twitter',
+                         'description' => esc_html__('Disable Twitter share button.', 'lsx-sharing'),
+                         'type'        => 'checkbox',
+                    )
+               );
+          }
+          if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'pinterest') ) ) {
+               $cmb->add_field(
+                    array(
+                         'name'        => esc_html__('Disable Pinterest', 'lsx-sharing'),
+                         'id'          => $section . '_disable_pinterest',
+                         'description' => esc_html__('Disable Pinterest button.', 'lsx-sharing'),
+                         'type'        => 'checkbox',
+                    )
+               );
+          }
 
-        do_action('lsx_sharing_settings_section', $cmb, $section);
-        $cmb->add_field(
-            array(
-				'id'   => $section . '_title_closing',
-				'type' => 'tab_closing',
-            )
-        );
-    }
+          do_action('lsx_sharing_settings_section', $cmb, $section);
+          $cmb->add_field(
+               array(
+                    'id'   => $section . '_title_closing',
+                    'type' => 'tab_closing',
+               )
+          );
+     }
 }

Spacing +95 added lines, -95 removed lines

@@ -6,7 +6,7 @@ 
  *
  * @package lsx-sharing
  */
-class Settings {
+class Settings{
 
 
     /**
@@ -16,16 +16,16 @@ 
      *
      * @var object \lsx\sharing\classes\admin\Settings()
      */
-    protected static $instance = null;
+    protected static $instance=null;
 
     /**
      * Contructor
      */
-    public function __construct() {
-         add_action('cmb2_admin_init', array( $this, 'register_settings_page' ));
-        add_action('lsx_sharing_settings_page', array( $this, 'configure_general_fields' ), 15, 1);
-        add_action('lsx_sharing_settings_page', array( $this, 'configure_archive_fields' ), 15, 1);
-        add_action('admin_enqueue_scripts', array( $this, 'assets' ));
+    public function __construct(){
+         add_action('cmb2_admin_init',array($this,'register_settings_page'));
+        add_action('lsx_sharing_settings_page',array($this,'configure_general_fields'),15,1);
+        add_action('lsx_sharing_settings_page',array($this,'configure_archive_fields'),15,1);
+        add_action('admin_enqueue_scripts',array($this,'assets'));
     }
 
     /**
@@ -35,10 +35,10 @@ 
      *
      * @return object \lsx\sharing\classes\admin\Settings()    A single instance of this class.
      */
-    public static function get_instance() {
+    public static function get_instance(){
          // If the single instance hasn't been set, set it now.
-        if ( null == self::$instance ) {
-            self::$instance = new self();
+        if(null==self::$instance){
+            self::$instance=new self();
         }
         return self::$instance;
     }
@@ -48,25 +48,25 @@ 
      *
      * @return void
      */
-    public function register_settings_page() {
-         $args = array(
+    public function register_settings_page(){
+         $args=array(
 			 'id'           => 'lsx_sharing_settings',
-			 'title'        => '<h1>' . esc_html__('LSX Sharing Settings', 'lsx-search') . ' <span class="version">' . LSX_SHARING_VER . '</span></h1>',
-			 'menu_title'   => esc_html__('LSX Sharing', 'search'), // Falls back to 'title' (above).
-			 'object_types' => array( 'options-page' ),
-			 'option_key'   => 'lsx-sharing-settings', // The option key and admin menu page slug.
+			 'title'        => '<h1>'.esc_html__('LSX Sharing Settings','lsx-search').' <span class="version">'.LSX_SHARING_VER.'</span></h1>',
+			 'menu_title'   => esc_html__('LSX Sharing','search'),// Falls back to 'title' (above).
+			 'object_types' => array('options-page'),
+			 'option_key'   => 'lsx-sharing-settings',// The option key and admin menu page slug.
 			 'parent_slug'  => 'options-general.php',
-			 'capability'   => 'manage_options', // Cap required to view options-page.
+			 'capability'   => 'manage_options',// Cap required to view options-page.
 		 );
-        $cmb  = new_cmb2_box($args);
-        do_action('lsx_sharing_settings_page', $cmb);
+        $cmb=new_cmb2_box($args);
+        do_action('lsx_sharing_settings_page',$cmb);
     }
 
     /**
      * Enqueue JS and CSS.
      */
-    public function assets( $hook ) {
-         wp_enqueue_style('lsx-sharing-admin', LSX_SHARING_URL . 'assets/css/lsx-sharing-admin.css', array(), LSX_SHARING_VER);
+    public function assets($hook){
+         wp_enqueue_style('lsx-sharing-admin',LSX_SHARING_URL.'assets/css/lsx-sharing-admin.css',array(),LSX_SHARING_VER);
     }
 
     /**
@@ -74,12 +74,12 @@ 
      *
      * @return void
      */
-    public function configure_general_fields( $cmb ) {
-         $global_args = array(
-			 'title' => __('Global', 'lsx-search'),
-			 'desc'  => esc_html__('These settings will control sharing icons everywhere.', 'lsx-search'),
+    public function configure_general_fields($cmb){
+         $global_args=array(
+			 'title' => __('Global','lsx-search'),
+			 'desc'  => esc_html__('These settings will control sharing icons everywhere.','lsx-search'),
 		 );
-        $this->get_fields($cmb, 'global', $global_args);
+        $this->get_fields($cmb,'global',$global_args);
     }
 
     /**
@@ -89,73 +89,73 @@ 
      * @param  string $position either top of bottom.
      * @return void
      */
-    public function configure_archive_fields( $cmb ) {
-         $archives       = array();
-        $post_type_args = array(
+    public function configure_archive_fields($cmb){
+         $archives=array();
+        $post_type_args=array(
 			'public' => true,
         );
-        $post_types     = get_post_types($post_type_args);
-        if ( ! empty($post_types) ) {
-            foreach ( $post_types as $post_type_key => $post_type_value ) {
-                switch ( $post_type_key ) {
+        $post_types=get_post_types($post_type_args);
+        if(!empty($post_types)){
+            foreach($post_types as $post_type_key => $post_type_value){
+                switch($post_type_key){
                 case 'post':
-                    $page_url      = home_url();
-                    $page_title    = __('Home', 'lsx-search');
-                    $show_on_front = get_option('show_on_front');
-                    if ( 'page' === $show_on_front ) {
-                            $page_for_posts = get_option('page_for_posts');
-                        if ( '' !== $page_for_posts ) {
-                            $page_title   = get_the_title($page_for_posts);
-                            $page_url     = get_permalink($page_for_posts);
+                    $page_url=home_url();
+                    $page_title=__('Home','lsx-search');
+                    $show_on_front=get_option('show_on_front');
+                    if('page'===$show_on_front){
+                            $page_for_posts=get_option('page_for_posts');
+                        if(''!==$page_for_posts){
+                            $page_title=get_the_title($page_for_posts);
+                            $page_url=get_permalink($page_for_posts);
                         }
                     }
-                    $description = sprintf(
+                    $description=sprintf(
                     /* translators: %s: The subscription info */
-                        __('Control the sharing buttons on your <a target="_blank" href="%1$s">%2$s</a> posts.', 'lsx-search'),
+                        __('Control the sharing buttons on your <a target="_blank" href="%1$s">%2$s</a> posts.','lsx-search'),
                         $page_url,
                         $page_title
                     );
-                    $archives[ $post_type_key ] = array(
-						'title' => __('Blog', 'lsx-search'),
+                    $archives[$post_type_key]=array(
+						'title' => __('Blog','lsx-search'),
 						'desc'  => $description,
                     );
                         break;
 
                 case 'product':
-                    $page_url = home_url();
-                    $page_title    = __('Shop', 'lsx-search');
-                    if ( function_exists('wc_get_page_id') ) {
-                        $shop_page  = wc_get_page_id('shop');
-                        $page_url   = get_permalink($shop_page);
-                        $page_title = get_the_title($shop_page);
+                    $page_url=home_url();
+                    $page_title=__('Shop','lsx-search');
+                    if(function_exists('wc_get_page_id')){
+                        $shop_page=wc_get_page_id('shop');
+                        $page_url=get_permalink($shop_page);
+                        $page_title=get_the_title($shop_page);
                     }
-                    $description = sprintf(
+                    $description=sprintf(
                     /* translators: %s: The subscription info */
-                        __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> product pages.', 'lsx-search'),
+                        __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> product pages.','lsx-search'),
                         $page_url,
                         $page_title
                     );
-                    $archives[ $post_type_key ] = array(
-						'title' => __('Shop', 'lsx-search'),
+                    $archives[$post_type_key]=array(
+						'title' => __('Shop','lsx-search'),
 						'desc'  => $description,
                     );
                         break;
 
                 default:
-                    if ( in_array($post_type_key, \lsx\sharing\includes\functions\get_restricted_post_types()) ) {
+                    if(in_array($post_type_key,\lsx\sharing\includes\functions\get_restricted_post_types())){
                             break;
                     }
-                    $temp_post_type = get_post_type_object($post_type_key);
-                    if ( ! is_wp_error($temp_post_type) ) {
-                        $page_url    = get_post_type_archive_link($temp_post_type->name);
-                        $description = sprintf(
+                    $temp_post_type=get_post_type_object($post_type_key);
+                    if(!is_wp_error($temp_post_type)){
+                        $page_url=get_post_type_archive_link($temp_post_type->name);
+                        $description=sprintf(
                         /* translators: %s: The subscription info */
-                            __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> singles.', 'lsx-search'),
+                            __('Control the sharing buttons which show on your <a target="_blank" href="%1$s">%2$s</a> singles.','lsx-search'),
                             $page_url,
                             $temp_post_type->label
                         );
 
-                        $archives[ $post_type_key ] = array(
+                        $archives[$post_type_key]=array(
 							'title' => $temp_post_type->label,
 							'desc'  => $description,
                         );
@@ -164,9 +164,9 @@ 
                 }
             }
         }
-        if ( ! empty($archives) ) {
-            foreach ( $archives as $archive_key => $archive_args ) {
-                $this->get_fields($cmb, $archive_key, $archive_args);
+        if(!empty($archives)){
+            foreach($archives as $archive_key => $archive_args){
+                $this->get_fields($cmb,$archive_key,$archive_args);
             }
         }
     }
@@ -178,86 +178,86 @@ 
      * @param  array  $args
      * @return void
      */
-    public function get_fields( $cmb, $section, $args ) {
+    public function get_fields($cmb,$section,$args){
          $cmb->add_field(
             array(
-				'id'          => 'settings_' . $section . '_sharing',
+				'id'          => 'settings_'.$section.'_sharing',
 				'type'        => 'title',
 				'name'        => $args['title'],
 				'default'     => $args['title'],
 				'description' => $args['desc'],
             )
         );
-        if ( 'global' === $section ) {
+        if('global'===$section){
             $cmb->add_field(
                 array(
-					'name'        => esc_html__('Disable all', 'lsx-sharing'),
-					'id'          => $section . '_disable_all',
-					'description' => esc_html__('Disable all sharing buttons on the site', 'lsx-sharing'),
+					'name'        => esc_html__('Disable all','lsx-sharing'),
+					'id'          => $section.'_disable_all',
+					'description' => esc_html__('Disable all sharing buttons on the site','lsx-sharing'),
 					'type'        => 'checkbox',
                 )
             );
-        } else {
+        }else{
             $cmb->add_field(
                 array(
-					'name'        => esc_html__('Disable', 'lsx-sharing'),
-					'id'          => $section . '_disable_pt',
-					'description' => esc_html__('Disable the share buttons on this post type', 'lsx-sharing'),
+					'name'        => esc_html__('Disable','lsx-sharing'),
+					'id'          => $section.'_disable_pt',
+					'description' => esc_html__('Disable the share buttons on this post type','lsx-sharing'),
 					'type'        => 'checkbox',
                 )
             );
         }
 
-		if ( 'global' === $section ) {
-			$label_text_description = esc_html__( 'If no text is specified per post type this text will display.' , 'lsx-sharing' );
-		} else {
-			$label_text_description = esc_html__( 'This text will display alongside the sharing buttons.', 'lsx-sharing' );
+		if('global'===$section){
+			$label_text_description=esc_html__('If no text is specified per post type this text will display.','lsx-sharing');
+		}else{
+			$label_text_description=esc_html__('This text will display alongside the sharing buttons.','lsx-sharing');
 		}
 		
         $cmb->add_field(
 			array(
-				'name'        => esc_html__( 'Label text', 'lsx-sharing' ),
-				'id'          => $section . '_label_text',
+				'name'        => esc_html__('Label text','lsx-sharing'),
+				'id'          => $section.'_label_text',
 				'type'        => 'text',
 				'description' => $label_text_description,
 			)
 		);
 		
-        if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'facebook') ) ) {
+        if('global'===$section||('global'!==$section&&!\lsx\sharing\includes\functions\is_button_disabled('global','facebook'))){
             $cmb->add_field(
                 array(
-					'name'        => esc_html__('Disable Facebook', 'lsx-sharing'),
-					'id'          => $section . '_disable_facebook',
-					'description' => esc_html__('Disable Facebook share button.', 'lsx-sharing'),
+					'name'        => esc_html__('Disable Facebook','lsx-sharing'),
+					'id'          => $section.'_disable_facebook',
+					'description' => esc_html__('Disable Facebook share button.','lsx-sharing'),
 					'type'        => 'checkbox',
                 )
             );
         }
-        if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'twitter') ) ) {
+        if('global'===$section||('global'!==$section&&!\lsx\sharing\includes\functions\is_button_disabled('global','twitter'))){
             $cmb->add_field(
                 array(
-					'name'        => esc_html__('Disable Twitter', 'lsx-sharing'),
-					'id'          => $section . '_disable_twitter',
-					'description' => esc_html__('Disable Twitter share button.', 'lsx-sharing'),
+					'name'        => esc_html__('Disable Twitter','lsx-sharing'),
+					'id'          => $section.'_disable_twitter',
+					'description' => esc_html__('Disable Twitter share button.','lsx-sharing'),
 					'type'        => 'checkbox',
                 )
             );
         }
-        if ( 'global' === $section || ( 'global' !== $section && ! \lsx\sharing\includes\functions\is_button_disabled('global', 'pinterest') ) ) {
+        if('global'===$section||('global'!==$section&&!\lsx\sharing\includes\functions\is_button_disabled('global','pinterest'))){
             $cmb->add_field(
                 array(
-					'name'        => esc_html__('Disable Pinterest', 'lsx-sharing'),
-					'id'          => $section . '_disable_pinterest',
-					'description' => esc_html__('Disable Pinterest button.', 'lsx-sharing'),
+					'name'        => esc_html__('Disable Pinterest','lsx-sharing'),
+					'id'          => $section.'_disable_pinterest',
+					'description' => esc_html__('Disable Pinterest button.','lsx-sharing'),
 					'type'        => 'checkbox',
                 )
             );
         }
 
-        do_action('lsx_sharing_settings_section', $cmb, $section);
+        do_action('lsx_sharing_settings_section',$cmb,$section);
         $cmb->add_field(
             array(
-				'id'   => $section . '_title_closing',
+				'id'   => $section.'_title_closing',
 				'type' => 'tab_closing',
             )
         );