lightspeeddevelopment / lsx-sharing

lsx-sharing.php

Spacing +5 added lines, -5 removed lines

@@ -13,14 +13,14 @@
  */
 
 // If this file is called directly, abort.
-if ( ! defined( 'WPINC' ) ) {
+if ( ! defined('WPINC')) {
 	die;
 }
 
-define( 'LSX_SHARING_PATH', plugin_dir_path( __FILE__ ) );
-define( 'LSX_SHARING_CORE', __FILE__ );
-define( 'LSX_SHARING_URL', plugin_dir_url( __FILE__ ) );
-define( 'LSX_SHARING_VER', '1.2.0' );
+define('LSX_SHARING_PATH', plugin_dir_path(__FILE__));
+define('LSX_SHARING_CORE', __FILE__);
+define('LSX_SHARING_URL', plugin_dir_url(__FILE__));
+define('LSX_SHARING_VER', '1.2.0');
 
 /* ======================= Below is the Plugin Class init ========================= */
 require_once LSX_SHARING_PATH . '/classes/class-sharing.php';

includes/functions.php

Spacing +14 added lines, -14 removed lines

@@ -12,12 +12,12 @@ 
  *
  * @return boolean
  */
-function is_button_disabled( $post_type = '' , $service = '' ) {
+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 ] ) ) {
+	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 ] ) ) {
+	} elseif (true === $sharing->is_new_options && ! empty($sharing->options[$post_type . '_disable_' . $service])) {
 		$option = true;
 	}
 	return $option;
@@ -28,12 +28,12 @@ 
  *
  * @return boolean
  */
-function is_pt_disabled( $post_type = '' ) {
+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 ] ) ) {
+	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' ] ) ) {
+	} elseif (true === $sharing->is_new_options && isset($sharing->options[$post_type . '_disable_pt'])) {
 		$option = true;
 	}
 	return $option;
@@ -47,9 +47,9 @@ 
 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'] ) ) {
+	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'] ) ) {
+	} elseif (true === $sharing->is_new_options && isset($sharing->options['global_disable_all'])) {
 		$option = true;
 	}
 	return $option;
@@ -60,15 +60,15 @@ 
  *
  * @return string
  */
-function get_sharing_text( $post_type = '' ) {
+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'] ) ) {
+	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'] ) ) {
+	} 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'];
 		}
 	}

classes/class-sharing.php

Spacing +11 added lines, -11 removed lines

@@ -49,7 +49,7 @@ 
 	 * Constructor.
 	 */
 	public function __construct() {
-		add_action( 'init', array( $this, 'set_options' ), 50 );
+		add_action('init', array($this, 'set_options'), 50);
 		$this->load_vendors();
 		$this->load_includes();
 		$this->load_classes();
@@ -64,7 +64,7 @@ 
 	 */
 	public static function get_instance() {
 		// If the single instance hasn't been set, set it now.
-		if ( null === self::$instance ) {
+		if (null === self::$instance) {
 			self::$instance = new self();
 		}
 		return self::$instance;
@@ -75,7 +75,7 @@ 
 	 */
 	private function load_vendors() {
 		// Configure custom fields.
-		if ( ! class_exists( 'CMB2' ) ) {
+		if ( ! class_exists('CMB2')) {
 			require_once LSX_SHARING_PATH . 'vendor/CMB2/init.php';
 		}
 	}
@@ -103,19 +103,19 @@ 
 	 * Set options.
 	 */
 	public function set_options() {
-		if ( function_exists( 'tour_operator' ) ) {
-			$this->options = get_option( '_lsx-to_settings', false );
+		if (function_exists('tour_operator')) {
+			$this->options = get_option('_lsx-to_settings', false);
 		} else {
-			$this->options = get_option( '_lsx_settings', false );
+			$this->options = get_option('_lsx_settings', false);
 
-			if ( false === $this->options ) {
-				$this->options = get_option( '_lsx_lsx-settings', false );
+			if (false === $this->options) {
+				$this->options = get_option('_lsx_lsx-settings', false);
 			}
 		}
 
-		$new_options = get_option( 'lsx-sharing-settings' );
-		if ( ! empty( $new_options ) ) {
-			if ( '' !== $new_options && false !== $new_options ) {
+		$new_options = get_option('lsx-sharing-settings');
+		if ( ! empty($new_options)) {
+			if ('' !== $new_options && false !== $new_options) {
 				$this->is_new_options = true;
 				$this->options        = $new_options;
 			}

classes/class-frontend.php

Spacing +4 added lines, -4 removed lines

@@ -47,7 +47,7 @@ 
 	 */
 	public static function get_instance() {
 		// If the single instance hasn't been set, set it now.
-		if ( null == self::$instance ) {
+		if (null == self::$instance) {
 			self::$instance = new self();
 		}
 		return self::$instance;
@@ -65,8 +65,8 @@ 
 	/**
 	 * Backwards compatabile function for the sharing buttons.
 	 */
-	public function sharing_buttons( $buttons = array( 'facebook', 'twitter', 'pinterest' ), $echo = false, $post_id = false ) {
-		wc_deprecated_function( 'LSX_Sharing_Frontend::sharing_buttons()', '1.2.0', 'lsx_sharing()->frontend->output->sharing_buttons()' );
-		$this->output->sharing_buttons( $buttons, $echo, $post_id );
+	public function sharing_buttons($buttons = array('facebook', 'twitter', 'pinterest'), $echo = false, $post_id = false) {
+		wc_deprecated_function('LSX_Sharing_Frontend::sharing_buttons()', '1.2.0', 'lsx_sharing()->frontend->output->sharing_buttons()');
+		$this->output->sharing_buttons($buttons, $echo, $post_id);
 	}
 }

classes/frontend/class-output.php

Spacing +48 added lines, -48 removed lines

@@ -21,13 +21,13 @@ 
 	 * Contructor
 	 */
 	public function __construct() {
-		add_action( 'wp_enqueue_scripts', array( $this, 'assets' ), 5 );
-		add_filter( 'wp_kses_allowed_html', array( $this, 'wp_kses_allowed_html' ), 10, 2 );
-		add_shortcode( 'lsx_sharing_buttons', array( $this, 'sharing_buttons_shortcode' ) );
+		add_action('wp_enqueue_scripts', array($this, 'assets'), 5);
+		add_filter('wp_kses_allowed_html', array($this, 'wp_kses_allowed_html'), 10, 2);
+		add_shortcode('lsx_sharing_buttons', array($this, 'sharing_buttons_shortcode'));
 		// Storefront (storefront_loop_post, storefront_single_post).
-		add_action( 'storefront_post_content_before', array( $this, 'sharing_buttons_template' ), 20 );
+		add_action('storefront_post_content_before', array($this, 'sharing_buttons_template'), 20);
 		// WooCommerce.
-		add_action( 'woocommerce_share', array( $this, 'sharing_buttons_template' ) );
+		add_action('woocommerce_share', array($this, 'sharing_buttons_template'));
 	}
 
 	/**
@@ -39,7 +39,7 @@ 
 	 */
 	public static function get_instance() {
 		// If the single instance hasn't been set, set it now.
-		if ( null == self::$instance ) {
+		if (null == self::$instance) {
 			self::$instance = new self();
 		}
 		return self::$instance;
@@ -49,14 +49,14 @@ 
 	 * Enques the assets.
 	 */
 	public function assets() {
-		if ( defined( 'WP_DEBUG' ) && true === WP_DEBUG ) {
+		if (defined('WP_DEBUG') && true === WP_DEBUG) {
 			$min = '';
 		} else {
 			$min = '.min';
 		}
 		/* Remove assets completely if all sharing options are off */
 
-		if ( \lsx\sharing\includes\functions\is_disabled() ) {
+		if (\lsx\sharing\includes\functions\is_disabled()) {
 			return '';
 		}
 
@@ -64,26 +64,26 @@ 
 		global $post;
 		$post_id    = false;
 		$share_post = $post;
-		if ( false !== $post_id ) {
-			$share_post = get_post( $post_id );
-			$post_type  = get_post_type( $post_id );
+		if (false !== $post_id) {
+			$share_post = get_post($post_id);
+			$post_type  = get_post_type($post_id);
 		} else {
 			$post_type = get_post_type();
 		}
 
 		/* Only show the assets if the post type sharing option is on */
-		if ( ! \lsx\sharing\includes\functions\is_pt_disabled( $post_type ) ) {
+		if ( ! \lsx\sharing\includes\functions\is_pt_disabled($post_type)) {
 
-			wp_enqueue_script( 'lsx-sharing', LSX_SHARING_URL . 'assets/js/lsx-sharing' . $min . '.js', array( 'jquery' ), LSX_SHARING_VER, true );
+			wp_enqueue_script('lsx-sharing', LSX_SHARING_URL . 'assets/js/lsx-sharing' . $min . '.js', array('jquery'), LSX_SHARING_VER, true);
 
-			$params = apply_filters( 'lsx_sharing_js_params', array(
-				'ajax_url' => admin_url( 'admin-ajax.php' ),
+			$params = apply_filters('lsx_sharing_js_params', array(
+				'ajax_url' => admin_url('admin-ajax.php'),
 			));
 
-			wp_localize_script( 'lsx-sharing', 'lsx_sharing_params', $params );
+			wp_localize_script('lsx-sharing', 'lsx_sharing_params', $params);
 
-			wp_enqueue_style( 'lsx-sharing', LSX_SHARING_URL . 'assets/css/lsx-sharing.css', array(), LSX_SHARING_VER );
-			wp_style_add_data( 'lsx-sharing', 'rtl', 'replace' );
+			wp_enqueue_style('lsx-sharing', LSX_SHARING_URL . 'assets/css/lsx-sharing.css', array(), LSX_SHARING_VER);
+			wp_style_add_data('lsx-sharing', 'rtl', 'replace');
 
 		}
 
@@ -92,49 +92,49 @@ 
 	/**
 	 * Display/return sharing buttons.
 	 */
-	public function sharing_buttons( $buttons = array( 'facebook', 'twitter', 'pinterest' ), $echo = false, $post_id = false ) {
+	public function sharing_buttons($buttons = array('facebook', 'twitter', 'pinterest'), $echo = false, $post_id = false) {
 		$sharing_content = '';
 
-		if ( ( is_preview() || is_admin() ) && ! ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+		if ((is_preview() || is_admin()) && ! (defined('DOING_AJAX') && DOING_AJAX)) {
 			return '';
 		}
 
 		//Set our variables
 		global $post;
 		$share_post = $post;
-		if ( false !== $post_id ) {
-			$share_post = get_post( $post_id );
-			$post_type = get_post_type( $post_id );
+		if (false !== $post_id) {
+			$share_post = get_post($post_id);
+			$post_type = get_post_type($post_id);
 		} else {
 			$post_type = get_post_type();
 		}
 
-		if ( \lsx\sharing\includes\functions\is_disabled() || \lsx\sharing\includes\functions\is_pt_disabled( $post_type ) ) {
+		if (\lsx\sharing\includes\functions\is_disabled() || \lsx\sharing\includes\functions\is_pt_disabled($post_type)) {
 			return '';
 		}
 
-		if ( ( is_array( $buttons ) && count( $buttons ) > 0 ) ) {
+		if ((is_array($buttons) && count($buttons) > 0)) {
 			$sharing_content .= '<div class="lsx-sharing-content"><p>';
 
-			$sharing_text = \lsx\sharing\includes\functions\get_sharing_text( $post_type );
-			if ( '' !== $sharing_text ) {
+			$sharing_text = \lsx\sharing\includes\functions\get_sharing_text($post_type);
+			if ('' !== $sharing_text) {
 				$sharing_content .= '<span class="lsx-sharing-label">' . $sharing_text . '</span>';
 			}
 
-			foreach ( $buttons as $id => $button ) {
-				$button_obj = new \lsx\sharing\classes\frontend\Button( $button, $this->options, $post_type );
+			foreach ($buttons as $id => $button) {
+				$button_obj = new \lsx\sharing\classes\frontend\Button($button, $this->options, $post_type);
 
-				if ( ! empty( $button_obj ) ) {
-					$url = $button_obj->get_link( $share_post );
+				if ( ! empty($button_obj)) {
+					$url = $button_obj->get_link($share_post);
 
-					if ( ! empty( $url ) ) {
-						if ( 'email' === $button ) {
-							if ( ! isset( $this->options['display'] ) || empty( $this->options['display']['sharing_email_form_id'] ) ) {
+					if ( ! empty($url)) {
+						if ('email' === $button) {
+							if ( ! isset($this->options['display']) || empty($this->options['display']['sharing_email_form_id'])) {
 								continue;
 							}
-							$sharing_content .= '<span class="lsx-sharing-button lsx-sharing-button-' . esc_attr( $button ) . '"><a href="#lsx-sharing-email" data-toggle="modal" data-link="' . esc_url( $url ) . '"><span class="fa" aria-hidden="true"></span></a></span>';
+							$sharing_content .= '<span class="lsx-sharing-button lsx-sharing-button-' . esc_attr($button) . '"><a href="#lsx-sharing-email" data-toggle="modal" data-link="' . esc_url($url) . '"><span class="fa" aria-hidden="true"></span></a></span>';
 						} else {
-							$sharing_content .= '<span class="lsx-sharing-button lsx-sharing-button-' . esc_attr( $button ) . '"><a href="' . esc_url( $url ) . '" target="_blank" rel="noopener noreferrer"><span class="fa" aria-hidden="true"></span></a></span>';
+							$sharing_content .= '<span class="lsx-sharing-button lsx-sharing-button-' . esc_attr($button) . '"><a href="' . esc_url($url) . '" target="_blank" rel="noopener noreferrer"><span class="fa" aria-hidden="true"></span></a></span>';
 						}
 					}
 				}
@@ -142,8 +142,8 @@ 
 			$sharing_content .= '</p></div>';
 		}
 
-		if ( $echo ) {
-			echo wp_kses_post( $sharing_content );
+		if ($echo) {
+			echo wp_kses_post($sharing_content);
 		} else {
 			return $sharing_content;
 		}
@@ -152,20 +152,20 @@ 
 	/**
 	 * Sharing buttons shortcode.
 	 */
-	public function sharing_buttons_shortcode( $atts ) {
-		$atts = shortcode_atts( array(
+	public function sharing_buttons_shortcode($atts) {
+		$atts = shortcode_atts(array(
 			'buttons' => '',
-		), $atts, 'lsx_sharing_buttons' );
+		), $atts, 'lsx_sharing_buttons');
 
-		if ( empty( $atts['buttons'] ) ) {
+		if (empty($atts['buttons'])) {
 			return '';
 		}
 
-		$no_whitespaces = preg_replace( '/\s*,\s*/', ',', filter_var( $atts['buttons'], FILTER_SANITIZE_STRING ) );
-		$buttons        = explode( ',', $no_whitespaces );
+		$no_whitespaces = preg_replace('/\s*,\s*/', ',', filter_var($atts['buttons'], FILTER_SANITIZE_STRING));
+		$buttons        = explode(',', $no_whitespaces);
 
-		if ( is_array( $buttons ) && count( $buttons ) > 0 ) {
-			return $this->sharing_buttons( $buttons );
+		if (is_array($buttons) && count($buttons) > 0) {
+			return $this->sharing_buttons($buttons);
 		}
 	}
 
@@ -173,13 +173,13 @@ 
 	 * Display buttons (template hook).
 	 */
 	public function sharing_buttons_template() {
-		echo wp_kses_post( $this->sharing_buttons() );
+		echo wp_kses_post($this->sharing_buttons());
 	}
 
 	/**
 	 * Allow data params for Bootstrap modal.
 	 */
-	public function wp_kses_allowed_html( $allowedtags, $context ) {
+	public function wp_kses_allowed_html($allowedtags, $context) {
 		$allowedtags['a']['data-toggle'] = true;
 		$allowedtags['a']['data-link']   = true;
 		return $allowedtags;

classes/frontend/class-button.php

Spacing +38 added lines, -38 removed lines

@@ -35,12 +35,12 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $service, $options, $prefix = 'sharing' ) {
+	public function __construct($service, $options, $prefix = 'sharing') {
 		$this->options = $options;
-		if ( ! in_array( $service, $this->services, true ) ) {
+		if ( ! in_array($service, $this->services, true)) {
 			return;
 		}
-		if ( \lsx\sharing\includes\functions\is_button_disabled( 'global', $service ) || \lsx\sharing\includes\functions\is_button_disabled( $prefix, $service ) ) {
+		if (\lsx\sharing\includes\functions\is_button_disabled('global', $service) || \lsx\sharing\includes\functions\is_button_disabled($prefix, $service)) {
 			return '';
 		}
 		$this->service = $service;
@@ -49,42 +49,42 @@ 
 	/**
 	 * Get service link to share.
 	 */
-	public function get_link( $post ) {
-		if ( empty( $post ) ) {
+	public function get_link($post) {
+		if (empty($post)) {
 			return '';
 		}
 
-		if ( 'email' === $this->service ) {
-			return $this->get_link_email( $post );
-		} elseif ( 'facebook' === $this->service ) {
-			return $this->get_link_facebook( $post );
-		} elseif ( 'twitter' === $this->service ) {
-			return $this->get_link_twitter( $post );
-		} elseif ( 'pinterest' === $this->service ) {
-			return $this->get_link_pinterest( $post );
+		if ('email' === $this->service) {
+			return $this->get_link_email($post);
+		} elseif ('facebook' === $this->service) {
+			return $this->get_link_facebook($post);
+		} elseif ('twitter' === $this->service) {
+			return $this->get_link_twitter($post);
+		} elseif ('pinterest' === $this->service) {
+			return $this->get_link_pinterest($post);
 		}
 	}
 
 	/**
 	 * Get Facebook link to share.
 	 */
-	public function get_link_facebook( $post ) {
-		$permalink = get_permalink( $post->ID );
-		$permalink = apply_filters( 'lsx_sharing_facebook_url', $permalink, $post );
-		$title     = apply_filters( 'the_title', $post->post_title );
+	public function get_link_facebook($post) {
+		$permalink = get_permalink($post->ID);
+		$permalink = apply_filters('lsx_sharing_facebook_url', $permalink, $post);
+		$title     = apply_filters('the_title', $post->post_title);
 
-		return 'https://www.facebook.com/sharer.php?display=page&u=' . rawurlencode( $permalink ) . '&t=' . rawurlencode( $title );
+		return 'https://www.facebook.com/sharer.php?display=page&u=' . rawurlencode($permalink) . '&t=' . rawurlencode($title);
 	}
 
 	/**
 	 * Get Twitter link to share.
 	 */
-	public function get_link_twitter( $post ) {
-		$permalink = get_permalink( $post->ID );
-		$permalink = apply_filters( 'lsx_sharing_twitter_url', $permalink, $post );
-		$title     = apply_filters( 'the_title', $post->post_title );
+	public function get_link_twitter($post) {
+		$permalink = get_permalink($post->ID);
+		$permalink = apply_filters('lsx_sharing_twitter_url', $permalink, $post);
+		$title     = apply_filters('the_title', $post->post_title);
 
-		if ( function_exists( 'mb_stripos' ) ) {
+		if (function_exists('mb_stripos')) {
 			$strlen = 'mb_strlen';
 			$substr = 'mb_substr';
 		} else {
@@ -94,34 +94,34 @@ 
 
 		$short_url_length = 24;
 
-		if ( ( $strlen( $title ) + $short_url_length ) > 140 ) {
-			$text = $substr( $title, 0, ( 140 - $short_url_length - 1 ) ) . "\xE2\x80\xA6";
+		if (($strlen($title) + $short_url_length) > 140) {
+			$text = $substr($title, 0, (140 - $short_url_length - 1)) . "\xE2\x80\xA6";
 		} else {
 			$text = $title;
 		}
 
-		return 'https://twitter.com/intent/tweet?text=' . rawurlencode( $text ) . '&url=' . rawurlencode( $permalink );
+		return 'https://twitter.com/intent/tweet?text=' . rawurlencode($text) . '&url=' . rawurlencode($permalink);
 	}
 
 	/**
 	 * Get Pinterest link to share.
 	 */
-	public function get_link_pinterest( $post ) {
-		$permalink = get_permalink( $post->ID );
-		$permalink = apply_filters( 'lsx_sharing_pinterest_url', $permalink, $post );
-		$title     = apply_filters( 'the_title', $post->post_title );
-
-		if ( ! has_post_thumbnail( $post ) ) {
-			if ( class_exists( 'lsx\legacy\Placeholders' ) ) {
-				$image = lsx\legacy\Placeholders::placeholder_url( null, $post->post_type );
-			} elseif ( class_exists( 'LSX_Placeholders' ) ) {
-				$image = LSX_Placeholders::placeholder_url( null, $post->post_type );
+	public function get_link_pinterest($post) {
+		$permalink = get_permalink($post->ID);
+		$permalink = apply_filters('lsx_sharing_pinterest_url', $permalink, $post);
+		$title     = apply_filters('the_title', $post->post_title);
+
+		if ( ! has_post_thumbnail($post)) {
+			if (class_exists('lsx\legacy\Placeholders')) {
+				$image = lsx\legacy\Placeholders::placeholder_url(null, $post->post_type);
+			} elseif (class_exists('LSX_Placeholders')) {
+				$image = LSX_Placeholders::placeholder_url(null, $post->post_type);
 			}
 		} else {
-			$image = get_the_post_thumbnail_url( $post->ID, 'large' );
+			$image = get_the_post_thumbnail_url($post->ID, 'large');
 		}
 
-		return 'https://www.pinterest.com/pin/create/button/?url=' . rawurlencode( $permalink ) . '&media=' . rawurlencode( $image ) . '&description=' . rawurlencode( $title );
+		return 'https://www.pinterest.com/pin/create/button/?url=' . rawurlencode($permalink) . '&media=' . rawurlencode($image) . '&description=' . rawurlencode($title);
 	}
 
 	/**

classes/admin/class-settings-theme.php

Spacing +42 added lines, -42 removed lines

@@ -37,14 +37,14 @@ 
 	 * Contructor
 	 */
 	public function __construct() {
-		add_filter( 'cmb2_enqueue_css', array( $this, 'disable_cmb2_styles' ), 1, 1 );
-		add_action( 'cmb2_before_form', array( $this, 'generate_navigation' ), 10, 4 );
-		add_action( 'cmb2_before_title_field_row', array( $this, 'output_tab_open_div' ), 10, 1 );
-		add_action( 'cmb2_after_tab_closing_field_row', array( $this, 'output_tab_closing_div' ), 10, 1 );
-		add_action( 'cmb2_render_tab_closing', array( $this, 'cmb2_render_callback_for_tab_closing' ), 10, 5 );
-		add_filter( 'cmb2_sanitize_tab_closing', array( $this, 'cmb2_sanitize_tab_closing_callback' ), 10, 2 );
-		add_action( 'cmb2_after_form', array( $this, 'navigation_js' ), 10, 4 );
-		add_filter( 'cmb2_options_page_redirect_url', array( $this, 'add_tab_argument' ), 10, 1 );
+		add_filter('cmb2_enqueue_css', array($this, 'disable_cmb2_styles'), 1, 1);
+		add_action('cmb2_before_form', array($this, 'generate_navigation'), 10, 4);
+		add_action('cmb2_before_title_field_row', array($this, 'output_tab_open_div'), 10, 1);
+		add_action('cmb2_after_tab_closing_field_row', array($this, 'output_tab_closing_div'), 10, 1);
+		add_action('cmb2_render_tab_closing', array($this, 'cmb2_render_callback_for_tab_closing'), 10, 5);
+		add_filter('cmb2_sanitize_tab_closing', array($this, 'cmb2_sanitize_tab_closing_callback'), 10, 2);
+		add_action('cmb2_after_form', array($this, 'navigation_js'), 10, 4);
+		add_filter('cmb2_options_page_redirect_url', array($this, 'add_tab_argument'), 10, 1);
 	}
 
 	/**
@@ -56,7 +56,7 @@ 
 	 */
 	public static function get_instance() {
 		// If the single instance hasn't been set, set it now.
-		if ( null == self::$instance ) {
+		if (null == self::$instance) {
 			self::$instance = new self();
 		}
 		return self::$instance;
@@ -67,10 +67,10 @@ 
 	 *
 	 * @return bool $enabled Whether to enable (enqueue) styles.
 	 */
-	public function disable_cmb2_styles( $enabled ) {
-		if ( is_admin() ) {
+	public function disable_cmb2_styles($enabled) {
+		if (is_admin()) {
 			$current_screen = get_current_screen();
-			if ( is_object( $current_screen ) && 'settings_page_lsx-sharing-settings' === $current_screen->id ) {
+			if (is_object($current_screen) && 'settings_page_lsx-sharing-settings' === $current_screen->id) {
 				$enabled = false;
 			}
 		}
@@ -86,14 +86,14 @@ 
 	 * @param object $cmb2_obj
 	 * @return void
 	 */
-	public function generate_navigation( $cmb_id, $object_id, $object_type, $cmb2_obj ) {
-		if ( 'lsx_sharing_settings' === $cmb_id && 'lsx-sharing-settings' === $object_id && 'options-page' === $object_type ) {
+	public function generate_navigation($cmb_id, $object_id, $object_type, $cmb2_obj) {
+		if ('lsx_sharing_settings' === $cmb_id && 'lsx-sharing-settings' === $object_id && 'options-page' === $object_type) {
 			$this->navigation      = array();
 			$this->is_options_page = true;
-			if ( isset( $cmb2_obj->meta_box['fields'] ) && ! empty( $cmb2_obj->meta_box['fields'] ) ) {
-				foreach ( $cmb2_obj->meta_box['fields'] as $field_index => $field ) {
-					if ( 'title' === $field['type'] ) {
-						$this->navigation[ $field_index ] = $field['name'];
+			if (isset($cmb2_obj->meta_box['fields']) && ! empty($cmb2_obj->meta_box['fields'])) {
+				foreach ($cmb2_obj->meta_box['fields'] as $field_index => $field) {
+					if ('title' === $field['type']) {
+						$this->navigation[$field_index] = $field['name'];
 					}
 				}
 			}
@@ -107,32 +107,32 @@ 
 	 * @return void
 	 */
 	public function output_navigation() {
-		if ( ! empty( $this->navigation ) ) {
+		if ( ! empty($this->navigation)) {
 			?>
 			<div class="wp-filter hide-if-no-js">
 				<ul class="filter-links">
 					<?php
 					$first_tab    = true;
-					$total        = count( $this->navigation );
+					$total        = count($this->navigation);
 					$count        = 0;
 					$separator    = ' |';
 					$selected_tab = '';
-					if ( isset( $_GET['cmb_tab'] ) && '' !== $_GET['cmb_tab'] ) {
-						$selected_tab  = sanitize_text_field( $_GET['cmb_tab'] );
+					if (isset($_GET['cmb_tab']) && '' !== $_GET['cmb_tab']) {
+						$selected_tab  = sanitize_text_field($_GET['cmb_tab']);
 						$selected_tab  = 'settings_' . $selected_tab;
 					}
-					foreach ( $this->navigation as $key => $label ) {
+					foreach ($this->navigation as $key => $label) {
 						$count++;
 						$current_css = '';
-						if ( ( true === $first_tab && '' === $selected_tab ) || $key === $selected_tab ) {
+						if ((true === $first_tab && '' === $selected_tab) || $key === $selected_tab) {
 							$first_tab   = false;
 							$current_css = 'current';
 						}
-						if ( $count === $total ) {
+						if ($count === $total) {
 							$separator = '';
 						}
 						?>
-							<li><a href="#" class="<?php echo esc_attr( $current_css ); ?>" data-sort="<?php echo esc_attr( $key ); ?>_tab"><?php echo esc_attr( $label ); ?></a><?php echo esc_attr( $separator ); ?></li>
+							<li><a href="#" class="<?php echo esc_attr($current_css); ?>" data-sort="<?php echo esc_attr($key); ?>_tab"><?php echo esc_attr($label); ?></a><?php echo esc_attr($separator); ?></li>
 						<?php
 					}
 					?>
@@ -148,10 +148,10 @@ 
 	 * @param object $field CMB2_Field();
 	 * @return void
 	 */
-	public function output_tab_open_div( $field ) {
-		if ( true === $this->is_options_page && isset( $field->args['type'] ) && 'title' === $field->args['type'] ) {
+	public function output_tab_open_div($field) {
+		if (true === $this->is_options_page && isset($field->args['type']) && 'title' === $field->args['type']) {
 			?>
-			<div id="<?php echo esc_attr( $field->args['id'] ); ?>_tab" class="tab tab-nav hidden">
+			<div id="<?php echo esc_attr($field->args['id']); ?>_tab" class="tab tab-nav hidden">
 			<?php
 		}
 	}
@@ -162,19 +162,19 @@ 
 	 * @param object $field CMB2_Field();
 	 * @return void
 	 */
-	public function output_tab_closing_div( $field ) {
-		if ( true === $this->is_options_page && isset( $field->args['type'] ) && 'tab_closing' === $field->args['type'] ) {
+	public function output_tab_closing_div($field) {
+		if (true === $this->is_options_page && isset($field->args['type']) && 'tab_closing' === $field->args['type']) {
 			?>
 			</div>
 			<?php
 		}
 	}
 
-	public function cmb2_render_callback_for_tab_closing( $field, $escaped_value, $object_id, $object_type, $field_type_object ) {
+	public function cmb2_render_callback_for_tab_closing($field, $escaped_value, $object_id, $object_type, $field_type_object) {
 		return;
 	}
 
-	public function cmb2_sanitize_tab_closing_callback( $override_value, $value ) {
+	public function cmb2_sanitize_tab_closing_callback($override_value, $value) {
 		return '';
 	}
 
@@ -187,8 +187,8 @@ 
 	 * @param object $cmb2_obj
 	 * @return void
 	 */
-	public function navigation_js( $cmb_id, $object_id, $object_type, $cmb2_obj ) {
-		if ( 'lsx_sharing_settings' === $cmb_id && 'lsx-sharing-settings' === $object_id && 'options-page' === $object_type ) {
+	public function navigation_js($cmb_id, $object_id, $object_type, $cmb2_obj) {
+		if ('lsx_sharing_settings' === $cmb_id && 'lsx-sharing-settings' === $object_id && 'options-page' === $object_type) {
 			?>
 			<script>
 				var LSX_SHARING_CMB2 = Object.create( null );
@@ -270,14 +270,14 @@ 
 	 * @param string $url
 	 * @return void
 	 */
-	public function add_tab_argument( $url ) {
-		if ( isset( $_POST['cmb_tab'] ) && '' !== $_POST['cmb_tab'] ) { // @codingStandardsIgnoreLine
-			$tab_selection = sanitize_text_field( $_POST['cmb_tab'] ); // @codingStandardsIgnoreLine
-			$tab_selection = str_replace( array( 'settings_', '_tab' ), '', $tab_selection ); // @codingStandardsIgnoreLine
-			if ( 'single' !== $tab_selection ) {
-				$url = add_query_arg( 'cmb_tab', $tab_selection, $url );
+	public function add_tab_argument($url) {
+		if (isset($_POST['cmb_tab']) && '' !== $_POST['cmb_tab']) { // @codingStandardsIgnoreLine
+			$tab_selection = sanitize_text_field($_POST['cmb_tab']); // @codingStandardsIgnoreLine
+			$tab_selection = str_replace(array('settings_', '_tab'), '', $tab_selection); // @codingStandardsIgnoreLine
+			if ('single' !== $tab_selection) {
+				$url = add_query_arg('cmb_tab', $tab_selection, $url);
 			} else {
-				$url = remove_query_arg( 'cmb_tab', $url );
+				$url = remove_query_arg('cmb_tab', $url);
 			}
 		}
 		return $url;

classes/admin/class-settings.php

Spacing +66 added lines, -66 removed lines

@@ -21,10 +21,10 @@ 
 	 * 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' ) );
+		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'));
 	}
 
 	/**
@@ -36,7 +36,7 @@ 
 	 */
 	public static function get_instance() {
 		// If the single instance hasn't been set, set it now.
-		if ( null == self::$instance ) {
+		if (null == self::$instance) {
 			self::$instance = new self();
 		}
 		return self::$instance;
@@ -50,23 +50,23 @@ 
 	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' ),
+			'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 );
+		$cmb = new_cmb2_box($args);
+		do_action('lsx_sharing_settings_page', $cmb);
 	}
 
 	/**
 	 * Enqueue JS and CSS.
 	 */
-	public function assets( $hook ) {
-		wp_enqueue_script( 'lsx-sharing-admin', LSX_SHARING_URL . 'assets/js/src/lsx-sharing-admin.js', array( 'jquery' ), LSX_SHARING_VER, true );
-		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_script('lsx-sharing-admin', LSX_SHARING_URL . 'assets/js/src/lsx-sharing-admin.js', array('jquery'), LSX_SHARING_VER, true);
+		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 ) {
+	public function configure_general_fields($cmb) {
 		$global_args = array(
-			'title' => __( 'Global', 'lsx-search' ),
-			'desc'  => esc_html__( 'Control the sharing WordPress post types.', 'lsx-search' ),
+			'title' => __('Global', 'lsx-search'),
+			'desc'  => esc_html__('Control the sharing WordPress post types.', 'lsx-search'),
 		);
-		$this->get_fields( $cmb, 'global', $global_args );
+		$this->get_fields($cmb, 'global', $global_args);
 	}
 
 	/**
@@ -89,54 +89,54 @@ 
 	 * @param string $position either top of bottom.
 	 * @return void
 	 */
-	public function configure_archive_fields( $cmb ) {
+	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_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(
 							/* translators: %s: The subscription info */
-							__( 'Control the filters which show on your <a target="_blank" href="%1$s">%2$s</a> posts.', 'lsx-search' ),
+							__('Control the filters which show 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_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(
 							/* translators: %s: The subscription info */
-							__( 'Control the filters which show on your <a target="_blank" href="%1$s">%2$s</a> product pages.', 'lsx-search' ),
+							__('Control the filters 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;
@@ -146,17 +146,17 @@ 
 						break;
 
 					default:
-						$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 );
+						$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 filters which show on your <a target="_blank" href="%1$s">%2$s</a> singles.', 'lsx-search' ),
+								__('Control the filters 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,
 							);
@@ -165,9 +165,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);
 			}
 		}
 	}
@@ -180,7 +180,7 @@ 
 	 * @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',
@@ -190,21 +190,21 @@ 
 				'description' => $args['desc'],
 			)
 		);
-		if ( 'global' === $section ) {
+		if ('global' === $section) {
 			$cmb->add_field(
 				array(
-					'name'        => esc_html__( 'Disable all', 'lsx-sharing' ),
+					'name'        => esc_html__('Disable all', 'lsx-sharing'),
 					'id'          => $section . '_disable_all',
-					'description' => esc_html__( 'Disable all share buttons on the site', 'lsx-sharing' ),
+					'description' => esc_html__('Disable all share buttons on the site', 'lsx-sharing'),
 					'type'        => 'checkbox',
 				)
 			);
 		} else {
 			$cmb->add_field(
 				array(
-					'name'        => esc_html__( 'Disable', 'lsx-sharing' ) . ' ' . $args['title'],
+					'name'        => esc_html__('Disable', 'lsx-sharing') . ' ' . $args['title'],
 					'id'          => $section . '_disable_pt',
-					'description' => esc_html__( 'Disable all share buttons on this post type', 'lsx-sharing' ),
+					'description' => esc_html__('Disable all share buttons on this post type', 'lsx-sharing'),
 					'type'        => 'checkbox',
 				)
 			);
@@ -212,44 +212,44 @@ 
 
 		$cmb->add_field(
 			array(
-				'name'        => esc_html__( 'Label text', 'lsx-sharing' ),
+				'name'        => esc_html__('Label text', 'lsx-sharing'),
 				'id'          => $section . '_label_text',
-				'description' => esc_html__( 'A default label for the sharing.', 'lsx-sharing' ),
+				'description' => esc_html__('A default label for the sharing.', 'lsx-sharing'),
 				'type'        => 'text',
 			)
 		);
-		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' ),
+					'name'        => esc_html__('Disable Facebook', 'lsx-sharing'),
 					'id'          => $section . '_disable_facebook',
-					'description' => esc_html__( 'Disable Facebook share button.', 'lsx-sharing' ),
+					'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' ),
+					'name'        => esc_html__('Disable Twitter', 'lsx-sharing'),
 					'id'          => $section . '_disable_twitter',
-					'description' => esc_html__( 'Disable Twitter share button.', 'lsx-sharing' ),
+					'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' ),
+					'name'        => esc_html__('Disable Pinterest', 'lsx-sharing'),
 					'id'          => $section . '_disable_pinterest',
-					'description' => esc_html__( 'Disable Pinterest button.', 'lsx-sharing' ),
+					'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',

classes/class-admin.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 	 */
 	public static function get_instance() {
 		// If the single instance hasn't been set, set it now.
-		if ( null === self::$instance ) {
+		if (null === self::$instance) {
 			self::$instance = new self();
 		}
 		return self::$instance;