postpromoterpro / post-promoter-pro

includes/libs/social-loader.php

Spacing +11 added lines, -11 removed lines

@@ -1,31 +1,31 @@
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
-include_once( PPP_PATH . '/includes/wpme-functions.php' );
+include_once(PPP_PATH.'/includes/wpme-functions.php');
 
 global $ppp_twitter_oauth;
-include_once( PPP_PATH . '/includes/twitter-functions.php' );
-require_once( PPP_PATH . '/includes/libs/twitter.php');
+include_once(PPP_PATH.'/includes/twitter-functions.php');
+require_once(PPP_PATH.'/includes/libs/twitter.php');
 $ppp_twitter_oauth = new PPP_Twitter();
 
 // Also include the user specific Twitter Class ( separate for now )
-require_once( PPP_PATH . '/includes/libs/twitter-user.php' );
+require_once(PPP_PATH.'/includes/libs/twitter-user.php');
 
 global $ppp_facebook_oauth;
-include_once( PPP_PATH . '/includes/facebook-functions.php' );
-require_once( PPP_PATH . '/includes/libs/facebook.php');
+include_once(PPP_PATH.'/includes/facebook-functions.php');
+require_once(PPP_PATH.'/includes/libs/facebook.php');
 $ppp_facebook_oauth = new PPP_Facebook();
 
 global $ppp_bitly_oauth;
-include_once( PPP_PATH . '/includes/bitly-functions.php' );
-require_once( PPP_PATH . '/includes/libs/bitly.php' );
+include_once(PPP_PATH.'/includes/bitly-functions.php');
+require_once(PPP_PATH.'/includes/libs/bitly.php');
 $ppp_bitly_oauth = new PPP_Bitly();
 
 global $ppp_linkedin_oauth;
-include_once( PPP_PATH . '/includes/linkedin-functions.php' );
-require_once( PPP_PATH . '/includes/libs/linkedin.php' );
+include_once(PPP_PATH.'/includes/linkedin-functions.php');
+require_once(PPP_PATH.'/includes/libs/linkedin.php');
 $ppp_linkedin_oauth = new PPP_Linkedin();

includes/libs/twitter-user.php

Spacing +23 added lines, -23 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,7 +11,7 @@ 
  * Handles all twitter functions
  *
  */
-if( !class_exists( 'PPP_Twitter_User' ) ) {
+if ( ! class_exists( 'PPP_Twitter_User' )) {
 
 	class PPP_Twitter_User {
 
@@ -19,7 +19,7 @@ 
 			ppp_maybe_start_session();
 			$this->user_id = $_user_id;
 
-			if ( ! empty( $this->user_id ) ) {
+			if ( ! empty($this->user_id)) {
 				$this->verify_credentials();
 			}
 		}
@@ -30,13 +30,13 @@ 
 		 * Handles to load twitter class
 		 */
 		public function load() {
-				if( !class_exists( 'TwitterOAuth' ) ) {
-					require_once ( PPP_PATH . '/includes/libs/twitter/twitteroauth.php' );
+				if ( ! class_exists( 'TwitterOAuth' )) {
+					require_once (PPP_PATH.'/includes/libs/twitter/twitteroauth.php');
 				}
 
 				ppp_set_social_tokens();
 
-				if ( ! defined( 'PPP_TW_CONSUMER_KEY' ) || ! defined( 'PPP_TW_CONSUMER_SECRET' ) ) {
+				if ( ! defined( 'PPP_TW_CONSUMER_KEY' ) || ! defined( 'PPP_TW_CONSUMER_SECRET' )) {
 					return false;
 				}
 
@@ -56,7 +56,7 @@ 
 		public function init() {
 
 			//when user is going to logged in in twitter and verified successfully session will create
-			if ( isset( $_REQUEST['oauth_verifier'] ) && isset( $_REQUEST['oauth_token'] ) ) {
+			if (isset($_REQUEST['oauth_verifier']) && isset($_REQUEST['oauth_token'])) {
 				$ppp_social_settings = get_option( 'ppp_social_settings' );
 
 				//load twitter class
@@ -68,13 +68,13 @@ 
 
 				//session for verifier
 				$verifier['oauth_verifier']       = $_REQUEST['oauth_verifier'];
-				$_SESSION[ 'ppp_twt_user_cache' ] = $verifier;
+				$_SESSION['ppp_twt_user_cache'] = $verifier;
 
 				//getting user data from twitter
 				$response = $this->twitter->get( 'account/verify_credentials' );
 
 				//if user data get successfully
-				if ( $response->id_str ) {
+				if ($response->id_str) {
 
 					$data['user'] = $response;
 					$data['user']->accessToken = $ppp_tw_access_token;
@@ -88,7 +88,7 @@ 
 			$this->load();
 
 			$user_settings = get_user_meta( $this->user_id, '_ppp_twitter_data', true );
-			if ( ! empty( $user_settings ) ) {
+			if ( ! empty($user_settings)) {
 
 				$this->twitter = new TwitterOAuth(
 					PPP_TW_CONSUMER_KEY,
@@ -97,10 +97,10 @@ 
 					$user_settings['user']->accessToken['oauth_token_secret']
 				);
 
-				$response = $this->twitter->get('account/verify_credentials');
-				if ( is_object( $response ) && property_exists( $response, 'errors' ) && count( $response->errors ) > 0 ) {
-					foreach ( $response->errors as $error ) {
-						if ( $error->code == 89 ) { // Expired or revoked tokens
+				$response = $this->twitter->get( 'account/verify_credentials' );
+				if (is_object( $response ) && property_exists( $response, 'errors' ) && count( $response->errors ) > 0) {
+					foreach ($response->errors as $error) {
+						if ($error->code == 89) { // Expired or revoked tokens
 
 							$this->revoke_access();
 
@@ -117,9 +117,9 @@ 
 		 * Get auth url for twitter
 		 *
 		 */
-		public function get_auth_url ( $return_url = '' ) {
+		public function get_auth_url( $return_url = '' ) {
 
-			if ( empty( $return_url ) ) {
+			if (empty($return_url)) {
 				$return_url = admin_url( 'admin.php?page=ppp-social-settings' );
 			}
 
@@ -128,7 +128,7 @@ 
 			$request_token = $this->twitter->getRequestToken( $return_url );
 
 			// If last connection failed don't display authorization link.
-			switch( $this->twitter->http_code ) {
+			switch ($this->twitter->http_code) {
 				case 200:
 					$_SESSION['ppp_user_twt_oauth_token']        = $request_token['oauth_token'];
 					$_SESSION['ppp_user_twt_oauth_token_secret'] = $request_token['oauth_token_secret'];
@@ -145,14 +145,14 @@ 
 		}
 
 		public function send_tweet( $message = '', $media = null ) {
-			if ( empty( $message ) ) {
+			if (empty($message)) {
 				return false;
 			}
 
 			$verify = $this->verify_credentials();
-			if ( $verify === true ) {
+			if ($verify === true) {
 				$args = array();
-				if ( ! empty( $media ) ) {
+				if ( ! empty($media)) {
 					$endpoint = 'statuses/update_with_media';
 					$args['media[]'] = wp_remote_retrieve_body( wp_remote_get( $media ) );
 				} else {
@@ -167,13 +167,13 @@ 
 		}
 
 		public function retweet( $tweet_id ) {
-			if ( empty( $tweet_id ) ) {
+			if (empty($tweet_id)) {
 				return false;
 			}
 
 			$verify = $this->verify_credentials();
-			if ( $verify === true ) {
-				$endpoint = 'statuses/retweet/' . $tweet_id;
+			if ($verify === true) {
+				$endpoint = 'statuses/retweet/'.$tweet_id;
 
 				return $this->twitter->post( $endpoint, array(), true );
 			} else {

includes/libs/twitter.php

Spacing +25 added lines, -25 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,7 +11,7 @@ 
  * Handles all twitter functions
  *
  */
-if( !class_exists( 'PPP_Twitter' ) ) {
+if ( ! class_exists( 'PPP_Twitter' )) {
 
 	class PPP_Twitter {
 
@@ -29,13 +29,13 @@ 
 		 * Handles to load twitter class
 		 */
 		public function ppp_load_twitter() {
-				if( !class_exists( 'TwitterOAuth' ) ) {
-					require_once ( PPP_PATH . '/includes/libs/twitter/twitteroauth.php' );
+				if ( ! class_exists( 'TwitterOAuth' )) {
+					require_once (PPP_PATH.'/includes/libs/twitter/twitteroauth.php');
 				}
 
 				ppp_set_social_tokens();
 
-				if ( ! defined( 'PPP_TW_CONSUMER_KEY' ) || ! defined( 'PPP_TW_CONSUMER_SECRET' ) ) {
+				if ( ! defined( 'PPP_TW_CONSUMER_KEY' ) || ! defined( 'PPP_TW_CONSUMER_SECRET' )) {
 					return false;
 				}
 
@@ -47,7 +47,7 @@ 
 		public function revoke_access() {
 			global $ppp_social_settings;
 
-			unset( $ppp_social_settings['twitter'] );
+			unset($ppp_social_settings['twitter']);
 
 			update_option( 'ppp_social_settings', $ppp_social_settings );
 		}
@@ -59,32 +59,32 @@ 
 		public function ppp_initialize_twitter() {
 
 			//when user is going to logged in in twitter and verified successfully session will create
-			if ( isset( $_REQUEST['oauth_verifier'] ) && isset( $_REQUEST['oauth_token'] ) ) {
+			if (isset($_REQUEST['oauth_verifier']) && isset($_REQUEST['oauth_token'])) {
 				$ppp_social_settings = get_option( 'ppp_social_settings' );
 
 				//load twitter class
 				$twitter = $this->ppp_load_twitter();
 
 				//check twitter class is loaded or not
-				if( !$twitter ) {
+				if ( ! $twitter) {
 					return false;
 				}
 
 				$this->twitter = new TwitterOAuth( PPP_TW_CONSUMER_KEY, PPP_TW_CONSUMER_SECRET, $_SESSION['ppp_twt_oauth_token'], $_SESSION['ppp_twt_oauth_token_secret'] );
 
 				// Request access tokens from twitter
-				$ppp_tw_access_token = $this->twitter->getAccessToken($_REQUEST['oauth_verifier']);
+				$ppp_tw_access_token = $this->twitter->getAccessToken( $_REQUEST['oauth_verifier'] );
 
 				//session for verifier
 				$verifier['oauth_verifier'] = $_REQUEST['oauth_verifier'];
 
-				$_SESSION[ 'ppp_twt_user_cache' ] = $verifier;
+				$_SESSION['ppp_twt_user_cache'] = $verifier;
 
 				//getting user data from twitter
-				$response = $this->twitter->get('account/verify_credentials');
+				$response = $this->twitter->get( 'account/verify_credentials' );
 
 				//if user data get successfully
-				if ( $response->id_str ) {
+				if ($response->id_str) {
 
 					$data['user'] = $response;
 					$data['user']->accessToken = $ppp_tw_access_token;
@@ -99,7 +99,7 @@ 
 			$this->ppp_load_twitter();
 
 			global $ppp_social_settings;
-			if ( isset( $ppp_social_settings['twitter'] ) ) {
+			if (isset($ppp_social_settings['twitter'])) {
 
 				$this->twitter = new TwitterOAuth(
 					PPP_TW_CONSUMER_KEY,
@@ -108,11 +108,11 @@ 
 					$ppp_social_settings['twitter']['user']->accessToken['oauth_token_secret']
 				);
 
-				$response = $this->twitter->get('account/verify_credentials');
-				if ( is_object( $response ) && property_exists( $response, 'errors' ) && count( $response->errors ) > 0 ) {
-					foreach ( $response->errors as $error ) {
-						if ( $error->code == 89 ) { // Expired or revoked tokens
-							unset( $ppp_social_settings['twitter'] );
+				$response = $this->twitter->get( 'account/verify_credentials' );
+				if (is_object( $response ) && property_exists( $response, 'errors' ) && count( $response->errors ) > 0) {
+					foreach ($response->errors as $error) {
+						if ($error->code == 89) { // Expired or revoked tokens
+							unset($ppp_social_settings['twitter']);
 							update_option( 'ppp_social_settings', $ppp_social_settings );
 
 							return array( 'error' => __( 'Post Promoter Pro has been removed from your Twitter account. Please reauthorize to continue promoting your content.', 'ppp-txt' ) );
@@ -128,23 +128,23 @@ 
 		 * Get auth url for twitter
 		 *
 		 */
-		public function ppp_get_twitter_auth_url ( $return_url = '' ) {
+		public function ppp_get_twitter_auth_url( $return_url = '' ) {
 
-			if ( empty( $return_url ) ) {
+			if (empty($return_url)) {
 				$return_url = admin_url( 'admin.php?page=ppp-social-settings' );
 			}
 			//load twitter class
 			$twitter = $this->ppp_load_twitter();
 
 			//check twitter class is loaded or not
-			if( !$twitter ) {
+			if ( ! $twitter) {
 				return false;
 			}
 
 			$request_token = $this->twitter->getRequestToken( $return_url );
 
 			// If last connection failed don't display authorization link.
-			switch( $this->twitter->http_code ) {
+			switch ($this->twitter->http_code) {
 
 			case 200:
 				$_SESSION['ppp_twt_oauth_token']        = $request_token['oauth_token'];
@@ -161,14 +161,14 @@ 
 		}
 
 		public function ppp_tweet( $message = '', $media = null ) {
-			if ( empty( $message ) ) {
+			if (empty($message)) {
 				return false;
 			}
 
 			$verify = $this->ppp_verify_twitter_credentials();
-			if ( $verify === true ) {
+			if ($verify === true) {
 				$args = array();
-				if ( ! empty( $media ) ) {
+				if ( ! empty($media)) {
 					$endpoint = 'statuses/update_with_media';
 					$args['media[]'] = wp_remote_retrieve_body( wp_remote_get( $media ) );
 				} else {

includes/wpme-functions.php

Spacing +4 added lines, -4 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,7 +11,7 @@ 
  * @return void
  */
 function ppp_add_wpme_shortener( $selected_shortener ) {
-	if( class_exists( 'Jetpack' ) && Jetpack::is_module_active( 'shortlinks' ) ) { ?>
+	if (class_exists( 'Jetpack' ) && Jetpack::is_module_active( 'shortlinks' )) { ?>
 		<option value="wpme" <?php selected( $selected_shortener, 'wpme', true ); ?>>WP.me</option>
 	<?php }
 }
@@ -40,11 +40,11 @@ 
  * @return string      The link, after shortening through wp.me
  */
 function ppp_apply_wpme( $link ) {
-	if ( class_exists( 'Jetpack' ) && Jetpack::is_module_active( 'shortlinks' ) ) {
+	if (class_exists( 'Jetpack' ) && Jetpack::is_module_active( 'shortlinks' )) {
 		$id = url_to_postid( $link );
 		$result = wpme_get_shortlink( $id );
 
-		if ( ! empty( $result ) ) {
+		if ( ! empty($result)) {
 			return $result;
 		} else {
 			return $link;

uninstall.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,11 +11,11 @@ 
  */
 $options = get_option( 'ppp_options' );
 
-if ( isset( $options['delete_on_uninstall'] ) ) {
+if (isset($options['delete_on_uninstall'])) {
 	require_once 'includes/cron-functions.php';
 
 	$crons = ppp_get_shceduled_crons();
-	foreach( $crons as $cron ) {
+	foreach ($crons as $cron) {
 		$ppp_data   = $cron['ppp_share_post_event'];
 		$array_keys = array_keys( $ppp_data );
 		$hash_key   = $array_keys[0];

post-promoter-pro.php

Spacing +45 added lines, -45 removed lines

@@ -16,9 +16,9 @@ 
 
 define( 'PPP_STORE_URL', 'https://postpromoterpro.com' );
 define( 'PPP_PLUGIN_NAME', 'Post Promoter Pro' );
-if( !class_exists( 'EDD_SL_Plugin_Updater' ) ) {
+if ( ! class_exists( 'EDD_SL_Plugin_Updater' )) {
 	// load our custom updater
-	include( PPP_PATH . '/includes/EDD_SL_Plugin_Updater.php' );
+	include(PPP_PATH.'/includes/EDD_SL_Plugin_Updater.php');
 }
 
 class PostPromoterPro {
@@ -27,27 +27,27 @@ 
 	private function __construct() {
 		add_action( 'init', array( $this, 'ppp_loaddomain' ), 1 );
 
-		if ( ! is_callable( 'curl_init' ) ) {
+		if ( ! is_callable( 'curl_init' )) {
 			add_action( 'admin_notices', array( $this, 'no_curl' ) );
 		} else {
 			global $ppp_options, $ppp_social_settings, $ppp_share_settings;
 
-			include PPP_PATH . '/includes/general-functions.php';
-			include PPP_PATH . '/includes/share-functions.php';
-			include PPP_PATH . '/includes/cron-functions.php';
-			include PPP_PATH . '/includes/filters.php';
-			include PPP_PATH . '/includes/libs/social-loader.php';
-			include PPP_PATH . '/includes/libs/class-wp-logging.php';
-
-			if ( is_admin() ) {
-				include PPP_PATH . '/includes/admin/upgrades.php';
-				include PPP_PATH . '/includes/admin/do-upgrades.php';
-				include PPP_PATH . '/includes/admin/actions.php';
-				include PPP_PATH . '/includes/admin/admin-pages.php';
-				include PPP_PATH . '/includes/admin/admin-ajax.php';
-				include PPP_PATH . '/includes/admin/meta-boxes.php';
-				include PPP_PATH . '/includes/admin/welcome.php';
-				include PPP_PATH . '/includes/admin/dashboard.php';
+			include PPP_PATH.'/includes/general-functions.php';
+			include PPP_PATH.'/includes/share-functions.php';
+			include PPP_PATH.'/includes/cron-functions.php';
+			include PPP_PATH.'/includes/filters.php';
+			include PPP_PATH.'/includes/libs/social-loader.php';
+			include PPP_PATH.'/includes/libs/class-wp-logging.php';
+
+			if (is_admin()) {
+				include PPP_PATH.'/includes/admin/upgrades.php';
+				include PPP_PATH.'/includes/admin/do-upgrades.php';
+				include PPP_PATH.'/includes/admin/actions.php';
+				include PPP_PATH.'/includes/admin/admin-pages.php';
+				include PPP_PATH.'/includes/admin/admin-ajax.php';
+				include PPP_PATH.'/includes/admin/meta-boxes.php';
+				include PPP_PATH.'/includes/admin/welcome.php';
+				include PPP_PATH.'/includes/admin/dashboard.php';
 			}
 
 			register_activation_hook( PPP_FILE, array( $this, 'activation_setup' ) );
@@ -67,7 +67,7 @@ 
 	 * @access public
 	 */
 	public static function getInstance() {
-		if ( !self::$ppp_instance ) {
+		if ( ! self::$ppp_instance) {
 			self::$ppp_instance = new PostPromoterPro();
 		}
 
@@ -92,7 +92,7 @@ 
 	 */
 	public function activation_setup() {
 		// If the settings already exist, don't do this
-		if ( get_option( 'ppp_options' ) ) {
+		if (get_option( 'ppp_options' )) {
 			return;
 		}
 
@@ -110,7 +110,7 @@ 
 	}
 
 	private function hooks() {
-		if ( is_admin() ) {
+		if (is_admin()) {
 			add_action( 'admin_init', array( $this, 'ppp_register_settings' ) );
 			add_action( 'admin_init', 'ppp_upgrade_plugin', 1 );
 
@@ -127,8 +127,8 @@ 
 		}
 
 		add_action( 'init', array( $this, 'get_actions' ) );
-		add_action( 'save_post', 'ppp_schedule_share', 99, 2);
-		add_action( 'transition_post_status', 'ppp_share_on_publish', 99, 3);
+		add_action( 'save_post', 'ppp_schedule_share', 99, 2 );
+		add_action( 'transition_post_status', 'ppp_share_on_publish', 99, 3 );
 		add_action( 'init', 'ppp_add_image_sizes' );
 		add_filter( 'wp_log_types', array( $this, 'register_log_type' ), 10, 1 );
 	}
@@ -151,15 +151,15 @@ 
 
 		$allowed_pages = apply_filters( 'ppp_admin_scripts_pages', $allowed_pages, $hook );
 
-		if ( ! in_array( $hook, $allowed_pages ) ) {
+		if ( ! in_array( $hook, $allowed_pages )) {
 			return;
 		}
 
 		wp_enqueue_script( 'jquery-ui-core' );
 		wp_enqueue_script( 'jquery-ui-datepicker' );
 
-		$jquery_ui_timepicker_path = PPP_URL . 'includes/scripts/libs/jquery-ui-timepicker-addon.js';
-		wp_enqueue_script( 'ppp_timepicker_js', $jquery_ui_timepicker_path , array( 'jquery', 'jquery-ui-core' ), PPP_VERSION, true );
+		$jquery_ui_timepicker_path = PPP_URL.'includes/scripts/libs/jquery-ui-timepicker-addon.js';
+		wp_enqueue_script( 'ppp_timepicker_js', $jquery_ui_timepicker_path, array( 'jquery', 'jquery-ui-core' ), PPP_VERSION, true );
 		wp_enqueue_script( 'ppp_core_custom_js', PPP_URL.'includes/scripts/js/ppp_custom.js', 'jquery', PPP_VERSION, true );
 
 	}
@@ -168,11 +168,11 @@ 
 
 		// List of people who make it impossible to override their jQuery UI as it's in their core CSS...so only
 		// load ours if they don't exist
-		if ( ! wp_style_is( 'ot-admin-css' ) && ! wp_style_is( 'jquery-ui-css' ) ) {
+		if ( ! wp_style_is( 'ot-admin-css' ) && ! wp_style_is( 'jquery-ui-css' )) {
 			wp_enqueue_style( 'jquery-ui-css', '//ajax.googleapis.com/ajax/libs/jqueryui/1.8.2/themes/flick/jquery-ui.css' );
 		}
 
-		wp_register_style( 'ppp_admin_css', PPP_URL . 'includes/scripts/css/admin-style.css', false, PPP_VERSION );
+		wp_register_style( 'ppp_admin_css', PPP_URL.'includes/scripts/css/admin-style.css', false, PPP_VERSION );
 		wp_enqueue_style( 'ppp_admin_css' );
 
 	}
@@ -185,7 +185,7 @@ 
 	 * @access public
 	 */
 	public function plugin_settings_links( $links, $file ) {
-		if ( $file != PPP_FILE ) {
+		if ($file != PPP_FILE) {
 			return $links;
 		}
 
@@ -279,22 +279,22 @@ 
 	 */
 	public function plugin_updater() {
 
-		if ( defined( 'NO_AUTO_UPDATE' ) && true === NO_AUTO_UPDATE ) {
+		if (defined( 'NO_AUTO_UPDATE' ) && true === NO_AUTO_UPDATE) {
 			return;
 		}
 
 		$license_key = trim( get_option( '_ppp_license_key' ) );
 
-		if ( empty( $license_key ) ) {
+		if (empty($license_key)) {
 			add_action( 'admin_notices', array( $this, 'no_license_nag' ) );
 			return;
 		}
 
 		// setup the updater
 		$edd_updater = new EDD_SL_Plugin_Updater( PPP_STORE_URL, __FILE__, array(
-				'version'   => PPP_VERSION,         // current version number
-				'license'   => $license_key,        // license key (used get_option above to retrieve from DB)
-				'item_name' => PPP_PLUGIN_NAME,     // name of this plugin
+				'version'   => PPP_VERSION, // current version number
+				'license'   => $license_key, // license key (used get_option above to retrieve from DB)
+				'item_name' => PPP_PLUGIN_NAME, // name of this plugin
 				'author'    => 'Post Promoter Pro'  // author of this plugin
 			)
 		);
@@ -324,10 +324,10 @@ 
 	 */
 	public function deactivate_license() {
 		// listen for our activate button to be clicked
-		if( isset( $_POST['ppp_license_deactivate'] ) ) {
+		if (isset($_POST['ppp_license_deactivate'])) {
 
 			// run a quick security check
-			if( ! check_admin_referer( 'ppp_deactivate_nonce', 'ppp_deactivate_nonce' ) ) {
+			if ( ! check_admin_referer( 'ppp_deactivate_nonce', 'ppp_deactivate_nonce' )) {
 				return;
 			}
 			// get out if we didn't click the Activate button
@@ -347,7 +347,7 @@ 
 			$response = wp_remote_get( add_query_arg( $api_params, PPP_STORE_URL ), array( 'timeout' => 15, 'sslverify' => false ) );
 
 			// make sure the response came back okay
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error( $response )) {
 				return false;
 			}
 
@@ -355,7 +355,7 @@ 
 			$license_data = json_decode( wp_remote_retrieve_body( $response ) );
 
 			// $license_data->license will be either "deactivated" or "failed"
-			if( $license_data->license == 'deactivated' ) {
+			if ($license_data->license == 'deactivated') {
 				delete_option( '_ppp_license_key_status' );
 			}
 
@@ -368,10 +368,10 @@ 
 	 */
 	public function activate_license() {
 		// listen for our activate button to be clicked
-		if( isset( $_POST['ppp_license_activate'] ) ) {
+		if (isset($_POST['ppp_license_activate'])) {
 
 			// run a quick security check
-			if( ! check_admin_referer( 'ppp_activate_nonce', 'ppp_activate_nonce' ) ) {
+			if ( ! check_admin_referer( 'ppp_activate_nonce', 'ppp_activate_nonce' )) {
 				return;
 			}
 			// get out if we didn't click the Activate button
@@ -391,7 +391,7 @@ 
 			$response = wp_remote_get( add_query_arg( $api_params, PPP_STORE_URL ), array( 'timeout' => 15, 'sslverify' => false ) );
 
 			// make sure the response came back okay
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error( $response )) {
 				return false;
 			}
 
@@ -412,7 +412,7 @@ 
 	 */
 	public function ppp_sanitize_license( $new ) {
 		$old = get_option( '_ppp_license_key' );
-		if( $old && $old != $new ) {
+		if ($old && $old != $new) {
 			delete_option( '_ppp_license_key_status' ); // new license has been entered, so must reactivate
 		}
 		return $new;
@@ -424,8 +424,8 @@ 
 	 * @return void
 	 */
 	public function get_actions() {
-		if ( isset( $_GET['ppp_action'] ) ) {
-			do_action( 'ppp_' . $_GET['ppp_action'], $_GET );
+		if (isset($_GET['ppp_action'])) {
+			do_action( 'ppp_'.$_GET['ppp_action'], $_GET );
 		}
 	}
 

includes/filters.php

Spacing +13 added lines, -13 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -28,8 +28,8 @@ 
  */
 function ppp_replace_text_tokens( $string, $args = array() ) {
 	$tokens = wp_list_pluck( ppp_get_text_tokens(), 'token' );
-	foreach ( $tokens as $key => $token ) {
-		$string = apply_filters( 'ppp_replace_token-' . $token, $string, $args );
+	foreach ($tokens as $key => $token) {
+		$string = apply_filters( 'ppp_replace_token-'.$token, $string, $args );
 	}
 
 	return $string;
@@ -43,7 +43,7 @@ 
  * @return string         The string with the token {post_title} replaced
  */
 function ppp_post_title_token( $string, $args ) {
-	if ( !isset( $args['post_id'] ) ) {
+	if ( ! isset($args['post_id'])) {
 		return $string;
 	}
 
@@ -71,11 +71,11 @@ 
  */
 function ppp_generate_unique_link( $share_link, $post_id, $name ) {
 	$name_parts = explode( '_', $name );
-	$share_link .= strpos( $share_link, '?' ) ? '&' : '?' ;
+	$share_link .= strpos( $share_link, '?' ) ? '&' : '?';
 
 	$query_string_var = apply_filters( 'ppp_query_string_var', 'ppp' );
 
-	$share_link .= $query_string_var . '=' . $post_id . '-' . $name_parts[1];
+	$share_link .= $query_string_var.'='.$post_id.'-'.$name_parts[1];
 
 	return $share_link;
 }
@@ -90,9 +90,9 @@ 
  */
 function ppp_generate_google_utm_link( $share_link, $post_id, $name ) {
 	$name_parts = explode( '_', $name );
-	$service = isset( $name_parts[3] ) ? $name_parts[3] : 'tw';
+	$service = isset($name_parts[3]) ? $name_parts[3] : 'tw';
 
-	switch( $service ) {
+	switch ($service) {
 		case 'li':
 			$network = 'LinkedIn';
 			break;
@@ -107,17 +107,17 @@ 
 
 	$utm['source']   = $network;
 	$utm['medium']   = 'social';
-	$utm['term']     =  ppp_get_post_slug_by_id( $post_id );
+	$utm['term']     = ppp_get_post_slug_by_id( $post_id );
 	$utm['content']  = $name_parts[1]; // The day after publishing
 	$utm['campaign'] = 'PostPromoterPro';
 
-	$utm_string  = strpos( $share_link, '?' ) ? '&' : '?' ;
+	$utm_string = strpos( $share_link, '?' ) ? '&' : '?';
 	$first = true;
-	foreach ( $utm as $key => $value ) {
-		if ( !$first ) {
+	foreach ($utm as $key => $value) {
+		if ( ! $first) {
 			$utm_string .= '&';
 		}
-		$utm_string .= 'utm_' . $key . '=' . $value;
+		$utm_string .= 'utm_'.$key.'='.$value;
 		$first = false;
 	}
 

includes/twitter-functions.php

Spacing +118 added lines, -118 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -12,7 +12,7 @@ 
 function ppp_twitter_enabled() {
 	global $ppp_social_settings;
 
-	if ( isset( $ppp_social_settings['twitter'] ) && !empty( $ppp_social_settings['twitter'] ) ) {
+	if (isset($ppp_social_settings['twitter']) && ! empty($ppp_social_settings['twitter'])) {
 		return true;
 	}
 
@@ -50,10 +50,10 @@ 
  */
 function ppp_tw_account_list_avatar( $string = '' ) {
 
-	if ( ppp_twitter_enabled() ) {
+	if (ppp_twitter_enabled()) {
 		global $ppp_social_settings;
 		$avatar_url = $ppp_social_settings['twitter']['user']->profile_image_url_https;
-		$string .= '<img class="ppp-social-icon" src="' . $avatar_url . '" />';
+		$string .= '<img class="ppp-social-icon" src="'.$avatar_url.'" />';
 	}
 
 	return $string;
@@ -67,7 +67,7 @@ 
  */
 function ppp_tw_account_list_name( $string = '' ) {
 
-	if ( ppp_twitter_enabled() ) {
+	if (ppp_twitter_enabled()) {
 		global $ppp_social_settings;
 		$string .= $ppp_social_settings['twitter']['user']->name;
 	}
@@ -83,15 +83,15 @@ 
  */
 function ppp_tw_account_list_actions( $string = '' ) {
 
-	if ( ! ppp_twitter_enabled() ) {
+	if ( ! ppp_twitter_enabled()) {
 		global $ppp_twitter_oauth, $ppp_social_settings;
 		$tw_auth = $ppp_twitter_oauth->ppp_verify_twitter_credentials();
 		$tw_authurl = $ppp_twitter_oauth->ppp_get_twitter_auth_url();
 
-		$string .= '<a href="' . $tw_authurl . '"><img src="' . PPP_URL . '/includes/images/sign-in-with-twitter-gray.png" /></a>';
+		$string .= '<a href="'.$tw_authurl.'"><img src="'.PPP_URL.'/includes/images/sign-in-with-twitter-gray.png" /></a>';
 	} else {
-		$string .= '<a class="button-primary" href="' . admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=twitter' ) . '" >' . __( 'Disconnect from Twitter', 'ppp-txt' ) . '</a>&nbsp;';
-		$string .= '<a class="button-secondary" href="https://twitter.com/settings/applications" target="blank">' . __( 'Revoke Access via Twitter', 'ppp-txt' ) . '</a>';
+		$string .= '<a class="button-primary" href="'.admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=twitter' ).'" >'.__( 'Disconnect from Twitter', 'ppp-txt' ).'</a>&nbsp;';
+		$string .= '<a class="button-secondary" href="https://twitter.com/settings/applications" target="blank">'.__( 'Revoke Access via Twitter', 'ppp-txt' ).'</a>';
 	}
 
 	return $string;
@@ -103,13 +103,13 @@ 
  * @return void
  */
 function ppp_capture_twitter_oauth() {
-	if ( isset( $_REQUEST['oauth_verifier'] ) && isset( $_REQUEST['oauth_token'] ) ) {
+	if (isset($_REQUEST['oauth_verifier']) && isset($_REQUEST['oauth_token'])) {
 		$current_screen = get_current_screen();
-		if ( 'user-edit' === $current_screen->base ) {
-			$user_id = ! empty( $_GET['user_id'] ) && is_numeric( $_GET['user_id'] ) ? $_GET['user_id'] : false;
+		if ('user-edit' === $current_screen->base) {
+			$user_id = ! empty($_GET['user_id']) && is_numeric( $_GET['user_id'] ) ? $_GET['user_id'] : false;
 			$twitter = new PPP_Twitter_User( $user_id );
 			$twitter->init();
-			$redirect = admin_url( 'user-edit.php?updated=1&user_id=' . $user_id );
+			$redirect = admin_url( 'user-edit.php?updated=1&user_id='.$user_id );
 		} else {
 			global $ppp_twitter_oauth;
 			$ppp_twitter_oauth->ppp_initialize_twitter();
@@ -127,17 +127,17 @@ 
  * @return void
  */
 function ppp_disconnect_twitter() {
-	if ( ! empty( $_GET['user_id'] ) ) {
+	if ( ! empty($_GET['user_id'])) {
 		$user_id = (int) sanitize_text_field( $_GET['user_id'] );
-		if ( $user_id !== get_current_user_id() || ! current_user_can( 'manage_options' ) ) {
+		if ($user_id !== get_current_user_id() || ! current_user_can( 'manage_options' )) {
 			wp_die( __( 'Unable to disconnect Twitter account', 'ppp-txt' ) );
 		}
 		delete_user_meta( $user_id, '_ppp_twitter_data' );
 	} else {
 		global $ppp_social_settings;
 		$ppp_social_settings = get_option( 'ppp_social_settings' );
-		if ( isset( $ppp_social_settings['twitter'] ) ) {
-			unset( $ppp_social_settings['twitter'] );
+		if (isset($ppp_social_settings['twitter'])) {
+			unset($ppp_social_settings['twitter']);
 			update_option( 'ppp_social_settings', $ppp_social_settings );
 		}
 	}
@@ -168,12 +168,12 @@ 
 	global $ppp_options, $wp_logs, $wp_filter;
 
 	$post_meta     = get_post_meta( $post_id, '_ppp_tweets', true );
-	$this_share    = $post_meta[ $index ];
-	$attachment_id = isset( $this_share['attachment_id'] ) ? $this_share['attachment_id'] : false;
+	$this_share    = $post_meta[$index];
+	$attachment_id = isset($this_share['attachment_id']) ? $this_share['attachment_id'] : false;
 
 	$share_message = ppp_tw_build_share_message( $post_id, $name );
 
-	if ( empty( $attachment_id ) && ! empty( $this_share['image'] ) ) {
+	if (empty($attachment_id) && ! empty($this_share['image'])) {
 		$media = $this_share['image'];
 	} else {
 		$use_media = ppp_tw_use_media( $post_id, $index );
@@ -199,12 +199,12 @@ 
 
 	update_post_meta( $log_entry, '_ppp_share_status', $status );
 
-	if ( ! empty( $status->id_str ) ) {
+	if ( ! empty($status->id_str)) {
 		$post      = get_post( $post_id );
 		$author_id = $post->post_author;
 		$author_rt = get_user_meta( $author_id, '_ppp_share_scheduled', true );
 
-		if ( $author_rt ) {
+		if ($author_rt) {
 			$twitter_user = new PPP_Twitter_User( $author_id );
 			$twitter_user->retweet( $status->id_str );
 		}
@@ -224,9 +224,9 @@ 
 function ppp_tw_build_share_message( $post_id, $name, $scheduled = true, $include_link = true ) {
 	$share_content = ppp_tw_generate_share_content( $post_id, $name );
 
-	if ( $include_link ) {
+	if ($include_link) {
 		$share_link    = ppp_generate_link( $post_id, $name, $scheduled );
-		$share_content = $share_content . ' ' . $share_link;
+		$share_content = $share_content.' '.$share_link;
 	}
 
 	return apply_filters( 'ppp_tw_build_share_message', $share_content );
@@ -240,10 +240,10 @@ 
  */
 function ppp_tw_generate_share_content( $post_id, $name, $is_scheduled = true ) {
 	global $ppp_options;
-	$default_text = isset( $ppp_options['default_text'] ) ? $ppp_options['default_text'] : '';
+	$default_text = isset($ppp_options['default_text']) ? $ppp_options['default_text'] : '';
 	$ppp_tweets   = get_post_meta( $post_id, '_ppp_tweets', true );
 
-	if ( $is_scheduled && ! empty( $ppp_tweets ) ) {
+	if ($is_scheduled && ! empty($ppp_tweets)) {
 		$ppp_post_override_data = get_post_meta( $post_id, '_ppp_post_override_data', true );
 		$name_array    = explode( '_', $name );
 		$index         = $name_array[1];
@@ -251,10 +251,10 @@ 
 	}
 
 	// If an override was found, use it, otherwise try the default text content
-	$share_content = ( isset( $share_content ) && !empty( $share_content ) ) ? $share_content : $default_text;
+	$share_content = (isset($share_content) && ! empty($share_content)) ? $share_content : $default_text;
 
 	// If the content is still empty, just use the post title
-	$share_content = ( isset( $share_content ) && !empty( $share_content ) ) ? $share_content : get_the_title( $post_id );
+	$share_content = (isset($share_content) && ! empty($share_content)) ? $share_content : get_the_title( $post_id );
 
 	return apply_filters( 'ppp_share_content', $share_content, array( 'post_id' => $post_id ) );
 }
@@ -266,12 +266,12 @@ 
  * @return bool         Whether or not this tweet should contain a media post
  */
 function ppp_tw_use_media( $post_id, $index ) {
-	if ( empty( $post_id ) || empty( $index ) ) {
+	if (empty($post_id) || empty($index)) {
 		return false;
 	}
 
 	$share_data = get_post_meta( $post_id, '_ppp_tweets', true );
-	$use_media  = ! empty( $share_data[$index]['attachment_id'] ) || ! empty( $share_data[$index]['image'] ) ? true : false;
+	$use_media  = ! empty($share_data[$index]['attachment_id']) || ! empty($share_data[$index]['image']) ? true : false;
 
 	return $use_media;
 }
@@ -282,7 +282,7 @@ 
  * @return void
  */
 function ppp_set_tw_token_constants( $social_tokens ) {
-	if ( !empty( $social_tokens ) && property_exists( $social_tokens, 'twitter' ) ) {
+	if ( ! empty($social_tokens) && property_exists( $social_tokens, 'twitter' )) {
 		define( 'PPP_TW_CONSUMER_KEY', $social_tokens->twitter->consumer_token );
 		define( 'PPP_TW_CONSUMER_SECRET', $social_tokens->twitter->consumer_secret );
 	}
@@ -320,7 +320,7 @@ 
  */
 function ppp_tw_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( !isset( $ppp_social_settings['twitter'] ) ) {
+	if ( ! isset($ppp_social_settings['twitter'])) {
 		return $tabs;
 	}
 
@@ -337,7 +337,7 @@ 
  */
 function ppp_tw_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( !isset( $ppp_social_settings['twitter'] ) ) {
+	if ( ! isset($ppp_social_settings['twitter'])) {
 		return $content;
 	}
 
@@ -396,16 +396,16 @@ 
 							<tbody>
 								<?php ppp_render_tweet_share_on_publish_row(); ?>
 								<?php $tweets = get_post_meta( $post->ID, '_ppp_tweets', true ); ?>
-								<?php if ( ! empty( $tweets ) ) : ?>
+								<?php if ( ! empty($tweets)) : ?>
 
-									<?php foreach ( $tweets as $key => $value ) :
-										$date          = isset( $value['date'] )          ? $value['date']          : '';
-										$time          = isset( $value['time'] )          ? $value['time']          : '';
-										$text          = isset( $value['text'] )          ? $value['text']          : '';
-										$image         = isset( $value['image'] )         ? $value['image']         : '';
-										$attachment_id = isset( $value['attachment_id'] ) ? $value['attachment_id'] : '';
+									<?php foreach ($tweets as $key => $value) :
+										$date          = isset($value['date']) ? $value['date'] : '';
+										$time          = isset($value['time']) ? $value['time'] : '';
+										$text          = isset($value['text']) ? $value['text'] : '';
+										$image         = isset($value['image']) ? $value['image'] : '';
+										$attachment_id = isset($value['attachment_id']) ? $value['attachment_id'] : '';
 
-										$args = apply_filters( 'ppp_tweet_row_args', compact( 'date','time','text','image','attachment_id' ), $value );
+										$args = apply_filters( 'ppp_tweet_row_args', compact( 'date', 'time', 'text', 'image', 'attachment_id' ), $value );
 										?>
 
 										<?php ppp_render_tweet_row( $key, $args, $post->ID ); ?>
@@ -414,7 +414,7 @@ 
 									<?php endforeach; ?>
 
 									<?php
-									if ( ! empty( $has_past_shares ) && count ( $tweets ) == $has_past_shares ) {
+									if ( ! empty($has_past_shares) && count( $tweets ) == $has_past_shares) {
 										$args = array(
 											'date'          => '',
 											'time'          => '',
@@ -437,7 +437,7 @@ 
 								<tr>
 									<td class="submit" colspan="4" style="float: none; clear:both; background:#fff;">
 										<a class="button-secondary ppp-add-repeatable" style="margin: 6px 0;"><?php _e( 'Add New Tweet', 'ppp-txt' ); ?></a>
-										<?php if ( ! empty( $has_past_shares ) ) : ?>
+										<?php if ( ! empty($has_past_shares)) : ?>
 											<a class="button-secondary ppp-view-all" style="margin: 6px 0;"><?php _e( 'Toggle Past Tweets', 'ppp-txt' ); ?></a>
 										<?php endif; ?>
 									</td>
@@ -463,38 +463,38 @@ 
  */
 function ppp_render_tweet_share_on_publish_row() {
 	global $post, $ppp_share_settings;
-	$default_text = !empty( $ppp_options['default_text'] ) ? $ppp_options['default_text'] : __( 'Social Text', 'ppp-txt' );
+	$default_text = ! empty($ppp_options['default_text']) ? $ppp_options['default_text'] : __( 'Social Text', 'ppp-txt' );
 
 	$ppp_post_exclude = get_post_meta( $post->ID, '_ppp_post_exclude', true );
 
 	$ppp_share_on_publish  = get_post_meta( $post->ID, '_ppp_share_on_publish', true );
 	$show_share_on_publish = false;
 
-	$share_by_default      = empty( $ppp_share_settings['twitter']['share_on_publish'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['twitter']['share_on_publish']) ? false : true;
 
-	if ( $ppp_share_on_publish == '1' || ( $ppp_share_on_publish == '' && $share_by_default ) ) {
+	if ($ppp_share_on_publish == '1' || ($ppp_share_on_publish == '' && $share_by_default)) {
 		$show_share_on_publish = true;
 	}
 
 	$ppp_share_on_publish_text          = get_post_meta( $post->ID, '_ppp_share_on_publish_text', true );
 	$ppp_share_on_publish_include_image = get_post_meta( $post->ID, '_ppp_share_on_publish_include_image', true );
 
-	$disabled = ( $post->post_status === 'publish' && time() > strtotime( $post->post_date ) ) ? true : false;
+	$disabled = ($post->post_status === 'publish' && time() > strtotime( $post->post_date )) ? true : false;
 	?>
 	<tr class="ppp-tweet-wrapper ppp-repeatable-row on-publish-row">
 		<td colspan="2" class="ppp-on-plublish-date-column">
-			<input <?php if ( $disabled ): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_share_on_publish" id="ppp_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />
+			<input <?php if ($disabled): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_share_on_publish" id="ppp_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />
 			&nbsp;<label for="ppp_share_on_publish"><?php _e( 'Tweet On Publish', 'ppp-txt' ); ?></label>
 		</td>
 
 		<td>
-			<input <?php if ( $disabled ): ?>readonly<?php endif; ?> class="ppp-tweet-text-repeatable" type="text" name="_ppp_share_on_publish_text" value="<?php echo esc_attr( $ppp_share_on_publish_text ); ?>" />
-			<?php $length = ! empty( $args['text'] ) ? strlen( $args['text'] ) : 0; ?>
+			<input <?php if ($disabled): ?>readonly<?php endif; ?> class="ppp-tweet-text-repeatable" type="text" name="_ppp_share_on_publish_text" value="<?php echo esc_attr( $ppp_share_on_publish_text ); ?>" />
+			<?php $length = ! empty($args['text']) ? strlen( $args['text'] ) : 0; ?>
 			&nbsp;<span class="ppp-text-length"><?php echo $length; ?></span>
 		</td>
 
 		<td style="width: 200px" colspan="2">
-			<input class="ppp-tw-featured-image-input" <?php if ( $disabled ): ?>readonly<?php endif; ?> id="ppp-share-on-publish-image" type="checkbox" name="_ppp_share_on_publish_include_image" value="1" <?php checked( '1', $ppp_share_on_publish_include_image, true ); ?>/>
+			<input class="ppp-tw-featured-image-input" <?php if ($disabled): ?>readonly<?php endif; ?> id="ppp-share-on-publish-image" type="checkbox" name="_ppp_share_on_publish_include_image" value="1" <?php checked( '1', $ppp_share_on_publish_include_image, true ); ?>/>
 			&nbsp;<label for="ppp-share-on-publish-image"><?php _e( 'Featured Image', 'ppp-txt' ); ?></label>
 		</td>
 
@@ -513,13 +513,13 @@ 
 function ppp_render_tweet_row( $key, $args = array(), $post_id ) {
 	global $post, $has_past_shares;
 
-	$share_time     = strtotime( $args['date'] . ' ' . $args['time'] );
+	$share_time     = strtotime( $args['date'].' '.$args['time'] );
 	$readonly       = current_time( 'timestamp' ) > $share_time ? 'readonly="readonly" ' : false;
-	$no_date        = ! empty( $readonly ) ? ' hasDatepicker' : '';
-	$hide           = ! empty( $readonly ) ? 'display: none;' : '';
-	$shared         = ! empty( $readonly ) ? 'past-share' : '';
+	$no_date        = ! empty($readonly) ? ' hasDatepicker' : '';
+	$hide           = ! empty($readonly) ? 'display: none;' : '';
+	$shared         = ! empty($readonly) ? 'past-share' : '';
 
-	if ( ! empty( $readonly ) ) {
+	if ( ! empty($readonly)) {
 		$has_past_shares++;
 	}
 	?>
@@ -534,7 +534,7 @@ 
 
 		<td>
 			<input <?php echo $readonly; ?>class="ppp-tweet-text-repeatable" type="text" name="_ppp_tweets[<?php echo $key; ?>][text]" value="<?php echo esc_attr( $args['text'] ); ?>" />
-			<?php $length = ! empty( $args['text'] ) ? strlen( $args['text'] ) : 0; ?>
+			<?php $length = ! empty($args['text']) ? strlen( $args['text'] ) : 0; ?>
 			&nbsp;<span class="ppp-text-length"><?php echo $length; ?></span>
 		</td>
 
@@ -553,7 +553,7 @@ 
 		</td>
 
 		<td>
-			<a href="#" class="ppp-repeatable-row ppp-remove-repeatable" data-type="twitter" style="background: url(<?php echo admin_url('/images/xit.gif'); ?>) no-repeat;<?php echo $hide; ?>">&times;</a>
+			<a href="#" class="ppp-repeatable-row ppp-remove-repeatable" data-type="twitter" style="background: url(<?php echo admin_url( '/images/xit.gif' ); ?>) no-repeat;<?php echo $hide; ?>">&times;</a>
 		</td>
 
 	</tr>
@@ -568,24 +568,24 @@ 
  */
 function ppp_tw_save_post_meta_boxes( $post_id, $post ) {
 
-	if ( ! ppp_should_save( $post_id, $post ) ) {
+	if ( ! ppp_should_save( $post_id, $post )) {
 		return;
 	}
 
-	$ppp_post_exclude = ( isset( $_REQUEST['_ppp_post_exclude'] ) ) ? $_REQUEST['_ppp_post_exclude'] : '0';
+	$ppp_post_exclude = (isset($_REQUEST['_ppp_post_exclude'])) ? $_REQUEST['_ppp_post_exclude'] : '0';
 
-	$ppp_share_on_publish = ( isset( $_REQUEST['_ppp_share_on_publish'] ) ) ? $_REQUEST['_ppp_share_on_publish'] : '0';
-	$ppp_share_on_publish_text = ( isset( $_REQUEST['_ppp_share_on_publish_text'] ) ) ? $_REQUEST['_ppp_share_on_publish_text'] : '';
-	$ppp_share_on_publish_include_image = ( isset( $_REQUEST['_ppp_share_on_publish_include_image'] ) ) ? $_REQUEST['_ppp_share_on_publish_include_image'] : '';
+	$ppp_share_on_publish = (isset($_REQUEST['_ppp_share_on_publish'])) ? $_REQUEST['_ppp_share_on_publish'] : '0';
+	$ppp_share_on_publish_text = (isset($_REQUEST['_ppp_share_on_publish_text'])) ? $_REQUEST['_ppp_share_on_publish_text'] : '';
+	$ppp_share_on_publish_include_image = (isset($_REQUEST['_ppp_share_on_publish_include_image'])) ? $_REQUEST['_ppp_share_on_publish_include_image'] : '';
 
 
 	update_post_meta( $post_id, '_ppp_share_on_publish', $ppp_share_on_publish );
 	update_post_meta( $post_id, '_ppp_share_on_publish_text', $ppp_share_on_publish_text );
 	update_post_meta( $post_id, '_ppp_share_on_publish_include_image', $ppp_share_on_publish_include_image );
 
-	$tweet_data = isset( $_REQUEST['_ppp_tweets'] ) ? $_REQUEST['_ppp_tweets'] : array();
-	foreach ( $tweet_data as $index => $tweet ) {
-		$tweet_data[ $index ]['text'] = sanitize_text_field( $tweet['text'] );
+	$tweet_data = isset($_REQUEST['_ppp_tweets']) ? $_REQUEST['_ppp_tweets'] : array();
+	foreach ($tweet_data as $index => $tweet) {
+		$tweet_data[$index]['text'] = sanitize_text_field( $tweet['text'] );
 	}
 	update_post_meta( $post_id, '_ppp_tweets', $tweet_data );
 
@@ -603,28 +603,28 @@ 
 	global $ppp_options;
 
 	$from_meta = get_post_meta( $post->ID, '_ppp_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_share_on_publish'] );
+	$from_post = isset($_POST['_ppp_share_on_publish']);
 
-	if ( empty( $from_meta ) && empty( $from_post ) ) {
+	if (empty($from_meta) && empty($from_post)) {
 		return;
 	}
 
 	// Determine if we're seeing the share on publish in meta or $_POST
-	if ( $from_meta && !$from_post ) {
+	if ($from_meta && ! $from_post) {
 		$ppp_share_on_publish_text = get_post_meta( $post->ID, '_ppp_share_on_publish_text', true );
 		$use_media = get_post_meta( $post->ID, '_ppp_share_on_publish_include_image', true );
 	} else {
-		$ppp_share_on_publish_text = isset( $_POST['_ppp_share_on_publish_text'] ) ? $_POST['_ppp_share_on_publish_text'] : '';
-		$use_media = isset( $_POST['_ppp_share_on_publish_include_image'] ) ? $_POST['_ppp_share_on_publish_include_image'] : false;
+		$ppp_share_on_publish_text = isset($_POST['_ppp_share_on_publish_text']) ? $_POST['_ppp_share_on_publish_text'] : '';
+		$use_media = isset($_POST['_ppp_share_on_publish_include_image']) ? $_POST['_ppp_share_on_publish_include_image'] : false;
 	}
 
-	$share_content = ( !empty( $ppp_share_on_publish_text ) ) ? $ppp_share_on_publish_text : ppp_tw_generate_share_content( $post->ID, null, false );
+	$share_content = ( ! empty($ppp_share_on_publish_text)) ? $ppp_share_on_publish_text : ppp_tw_generate_share_content( $post->ID, null, false );
 	$share_content = apply_filters( 'ppp_share_content', $share_content, array( 'post_id' => $post->ID ) );
-	$name = 'sharedate_0_' . $post->ID;
+	$name = 'sharedate_0_'.$post->ID;
 	$media = ppp_post_has_media( $post->ID, 'tw', $use_media );
 	$share_link = ppp_generate_link( $post->ID, $name, true );
 
-	$status = ppp_send_tweet( $share_content . ' ' . $share_link, $post->ID, $media );
+	$status = ppp_send_tweet( $share_content.' '.$share_link, $post->ID, $media );
 
 	$log_title = ppp_tw_build_share_message( $post->ID, $name, false, false );
 
@@ -641,11 +641,11 @@ 
 
 	$log_entry = WP_Logging::insert_log( $log_data, $log_meta );
 
-	if ( ! empty( $status->id_str ) ) {
+	if ( ! empty($status->id_str)) {
 		$author_id = $post->post_author;
 		$author_rt = get_user_meta( $author_id, '_ppp_share_on_publish', true );
 
-		if ( $author_rt ) {
+		if ($author_rt) {
 			$twitter_user = new PPP_Twitter_User( $author_id );
 			$twitter_user->retweet( $status->id_str );
 		}
@@ -665,20 +665,20 @@ 
 function ppp_tw_generate_timestamps( $times, $post_id ) {
 	$ppp_tweets = get_post_meta( $post_id, '_ppp_tweets', true );
 
-	if ( empty( $ppp_tweets ) ) {
+	if (empty($ppp_tweets)) {
 		$ppp_tweets = array();
 	}
 
-	foreach ( $ppp_tweets as $key => $data ) {
-		if ( ! array_filter( $data ) ) {
+	foreach ($ppp_tweets as $key => $data) {
+		if ( ! array_filter( $data )) {
 			continue;
 		}
 
 		$timestamp = ppp_generate_timestamp( $data['date'], $data['time'] );
 
-		if ( $timestamp > current_time( 'timestamp', 1 ) ) { // Make sure the timestamp we're getting is in the future
-			$time_key           = strtotime( date_i18n( 'd-m-Y H:i:s', $timestamp , true ) ) . '_tw';
-			$times[ $time_key ] = 'sharedate_' . $key . '_' . $post_id . '_tw';
+		if ($timestamp > current_time( 'timestamp', 1 )) { // Make sure the timestamp we're getting is in the future
+			$time_key           = strtotime( date_i18n( 'd-m-Y H:i:s', $timestamp, true ) ).'_tw';
+			$times[$time_key] = 'sharedate_'.$key.'_'.$post_id.'_tw';
 		}
 
 	}
@@ -698,7 +698,7 @@ 
 
 	$ret = false;
 
-	if ( ! empty( $ppp_share_settings['twitter']['cards_enabled'] ) ) {
+	if ( ! empty($ppp_share_settings['twitter']['cards_enabled'])) {
 		$ret = true;
 	}
 
@@ -713,13 +713,13 @@ 
  */
 function ppp_tw_card_meta() {
 
-	if ( ! is_single() || ! ppp_twitter_enabled() || ! ppp_tw_cards_enabled() ) {
+	if ( ! is_single() || ! ppp_twitter_enabled() || ! ppp_tw_cards_enabled()) {
 		return;
 	}
 
 	global $post, $ppp_options;
 
-	if ( ! array_key_exists( $post->post_type, $ppp_options['post_types'] ) ) {
+	if ( ! array_key_exists( $post->post_type, $ppp_options['post_types'] )) {
 		return;
 	}
 
@@ -737,20 +737,20 @@ 
 
 	$return = '';
 
-	if ( ! is_single() || ! ppp_tw_cards_enabled() ) {
+	if ( ! is_single() || ! ppp_tw_cards_enabled()) {
 		return $return;
 	}
 
 	global $post, $ppp_social_settings;
 
 
-	if ( empty( $post ) ) {
+	if (empty($post)) {
 		return;
 	}
 
 	$elements = ppp_tw_default_meta_elements();
-	foreach ( $elements as $name => $content ) {
-		$return .= '<meta name="' . $name . '" content="' . $content . '" />' . "\n";
+	foreach ($elements as $name => $content) {
+		$return .= '<meta name="'.$name.'" content="'.$content.'" />'."\n";
 	}
 
 	return apply_filters( 'ppp_tw_card_meta', $return );
@@ -770,22 +770,22 @@ 
 	$elements = array();
 
 	$image_url = ppp_post_has_media( $post->ID, 'tw', true );
-	if ( $image_url ) {
+	if ($image_url) {
 		$elements['twitter:card']      = 'summary_large_image';
 		$elements['twitter:image:src'] = $image_url;
 	} else {
 		$elements['twitter:card'] = 'summary';
 	}
 
-	$elements['twitter:site']        = '@' . $ppp_social_settings['twitter']['user']->screen_name;
+	$elements['twitter:site']        = '@'.$ppp_social_settings['twitter']['user']->screen_name;
 	$elements['twitter:title']       = esc_attr( strip_tags( $post->post_title ) );
 	$elements['twitter:description'] = esc_attr( ppp_tw_get_card_description() );
 
 	$author_twitter_handle = get_user_meta( $post->post_author, 'twitter', true );
-	if ( ! empty( $author_twitter_handle ) ) {
+	if ( ! empty($author_twitter_handle)) {
 
-		if ( strpos( $author_twitter_handle, '@' ) === false ) {
-			$author_twitter_handle = '@' . $author_twitter_handle;
+		if (strpos( $author_twitter_handle, '@' ) === false) {
+			$author_twitter_handle = '@'.$author_twitter_handle;
 		}
 
 		$elements['twitter:creator'] = esc_attr( strip_tags( $author_twitter_handle ) );
@@ -804,13 +804,13 @@ 
 function ppp_tw_get_card_description() {
 	global $post;
 
-	if ( ! is_single() || empty( $post ) ) {
+	if ( ! is_single() || empty($post)) {
 		return false;
 	}
 
 	$excerpt = $post->post_excerpt;
 
-	if ( empty( $excerpt ) ) {
+	if (empty($excerpt)) {
 		$excerpt = ppp_tw_format_card_description( $post->content );
 	}
 
@@ -828,10 +828,10 @@ 
 	$max_len = apply_filters( 'ppp_tw_cart_desc_length', 200 );
 	$excerpt = strip_tags( $excerpt );
 
-	if ( strlen( $excerpt ) > $max_len ) {
+	if (strlen( $excerpt ) > $max_len) {
 		$excerpt_pre = substr( $excerpt, 0, $max_len );
 		$last_space  = strrpos( $excerpt_pre, ' ' );
-		$excerpt     = substr( $excerpt_pre, 0, $last_space ) . '...';
+		$excerpt     = substr( $excerpt_pre, 0, $last_space ).'...';
 	}
 
 	return $excerpt;
@@ -846,7 +846,7 @@ 
  */
 function ppp_tw_add_contact_method( $user_contactmethods ) {
 
-	if ( ! isset( $user_contactmethods['twitter'] ) ) {
+	if ( ! isset($user_contactmethods['twitter'])) {
 		$user_contactmethods['twitter'] = __( 'Twitter', 'ppp-txt' );
 	}
 	// Returns the contact methods
@@ -864,11 +864,11 @@ 
  */
 function ppp_tw_profile_settings( $user ) {
 
-	if ( $user->ID == get_current_user_id() && ! current_user_can( 'edit_posts' ) ) {
+	if ($user->ID == get_current_user_id() && ! current_user_can( 'edit_posts' )) {
 		return;
 	}
 
-	if ( $user->ID !== get_current_user_id() && ! current_user_can( 'manage_options' ) ) {
+	if ($user->ID !== get_current_user_id() && ! current_user_can( 'manage_options' )) {
 		return;
 	}
 
@@ -882,16 +882,16 @@ 
 			<?php
 			$twitter = new PPP_Twitter_User( get_current_user_id() );
 			$tw_user = get_user_meta( $user->ID, '_ppp_twitter_data', true );
-			if ( empty( $tw_user ) ) {
-				$tw_authurl = $twitter->get_auth_url( admin_url( 'user-edit.php?user_id=' . $user->ID ) );
+			if (empty($tw_user)) {
+				$tw_authurl = $twitter->get_auth_url( admin_url( 'user-edit.php?user_id='.$user->ID ) );
 
-				echo '<a href="' . $tw_authurl . '"><img src="' . PPP_URL . '/includes/images/sign-in-with-twitter-gray.png" /></a>';
+				echo '<a href="'.$tw_authurl.'"><img src="'.PPP_URL.'/includes/images/sign-in-with-twitter-gray.png" /></a>';
 			} else {
 				$connected = true;
 				?>
 				<p><strong><?php _e( 'Signed in as', 'ppp-txt' ); ?>: </strong><?php echo $tw_user['user']->screen_name; ?></p>
 				<p>
-					<a class="button-primary" href="<?php echo admin_url( 'user-edit.php?user_id=' . $user->ID . '&ppp_social_disconnect=true&ppp_network=twitter&user_id=' . $user->ID ); ?>" ><?php _e( 'Disconnect from Twitter', 'ppp-txt' ); ?></a>&nbsp;
+					<a class="button-primary" href="<?php echo admin_url( 'user-edit.php?user_id='.$user->ID.'&ppp_social_disconnect=true&ppp_network=twitter&user_id='.$user->ID ); ?>" ><?php _e( 'Disconnect from Twitter', 'ppp-txt' ); ?></a>&nbsp;
 					<a class="button-secondary" href="https://twitter.com/settings/applications" target="blank"><?php _e( 'Revoke Access via Twitter', 'ppp-txt' ); ?></a>
 				</p>
 				<?php
@@ -900,10 +900,10 @@ 
 			</td>
 		</tr>
 
-		<?php if ( $connected ) : ?>
+		<?php if ($connected) : ?>
 		<?php
 			$share_on_publish = get_user_meta( $user->ID, '_ppp_share_on_publish', true );
-			$share_scheduled  = get_user_meta( $user->ID, '_ppp_share_scheduled' , true );
+			$share_scheduled  = get_user_meta( $user->ID, '_ppp_share_scheduled', true );
 		?>
 		<tr>
 			<th><?php _e( 'Sharing Options', 'ppp-txt' ); ?></th>
@@ -936,11 +936,11 @@ 
  */
 function ppp_tw_save_profile( $user_id ) {
 
-	$share_on_publish = ! empty( $_POST['share_on_publish'] ) ? true : false;
-	$share_scheduled  = ! empty( $_POST['share_scheduled'] )  ? true : false;
+	$share_on_publish = ! empty($_POST['share_on_publish']) ? true : false;
+	$share_scheduled  = ! empty($_POST['share_scheduled']) ? true : false;
 
 	update_user_meta( $user_id, '_ppp_share_on_publish', $share_on_publish );
-	update_user_meta( $user_id, '_ppp_share_scheduled' , $share_scheduled  );
+	update_user_meta( $user_id, '_ppp_share_scheduled', $share_scheduled );
 
 }
 add_action( 'personal_options_update', 'ppp_tw_save_profile' );
@@ -949,14 +949,14 @@ 
 function ppp_tw_calendar_on_publish_event( $events, $post_id ) {
 	$share_on_publish = get_post_meta( $post_id, '_ppp_share_on_publish', true );
 
-	if ( ! empty( $share_on_publish ) ) {
+	if ( ! empty($share_on_publish)) {
 		$share_text = get_post_meta( $post_id, '_ppp_share_on_publish_text', true );
 		$events[] = array(
-			'id' => $post_id . '-share-on-publish',
-			'title' => ( ! empty( $share_text ) ) ? $share_text : ppp_tw_generate_share_content( $post_id, null, false ),
-			'start'     => date_i18n( 'Y-m-d/TH:i:s', strtotime( get_the_date( null, $post_id ) . ' ' . get_the_time( null, $post_id ) ) + 1 ),
-			'end'       => date_i18n( 'Y-m-d/TH:i:s', strtotime( get_the_date( null, $post_id ) . ' ' . get_the_time( null, $post_id ) ) + 1 ),
-			'className' => 'ppp-calendar-item-tw cal-post-' . $post_id,
+			'id' => $post_id.'-share-on-publish',
+			'title' => ( ! empty($share_text)) ? $share_text : ppp_tw_generate_share_content( $post_id, null, false ),
+			'start'     => date_i18n( 'Y-m-d/TH:i:s', strtotime( get_the_date( null, $post_id ).' '.get_the_time( null, $post_id ) ) + 1 ),
+			'end'       => date_i18n( 'Y-m-d/TH:i:s', strtotime( get_the_date( null, $post_id ).' '.get_the_time( null, $post_id ) ) + 1 ),
+			'className' => 'ppp-calendar-item-tw cal-post-'.$post_id,
 			'belongsTo' => $post_id,
 		);
 	}
@@ -967,9 +967,9 @@ 
 
 function ppp_tw_get_post_shares( $items, $post_id ) {
 	$tweets = get_post_meta( $post_id, '_ppp_tweets', true );
-	if ( empty( $tweets ) ) { return $items; }
+	if (empty($tweets)) { return $items; }
 
-	foreach ( $tweets as $key => $tweet ) {
+	foreach ($tweets as $key => $tweet) {
 		$items[] = array( 'id' => $key, 'service' => 'tw' );
 	}
 	return $items;

includes/cron-functions.php

Spacing +23 added lines, -23 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -13,23 +13,23 @@ 
  */
 function ppp_schedule_share( $post_id, $post ) {
 	global $ppp_options;
-	$allowed_post_types = isset( $ppp_options['post_types'] ) ? $ppp_options['post_types'] : array();
+	$allowed_post_types = isset($ppp_options['post_types']) ? $ppp_options['post_types'] : array();
 	$allowed_post_types = apply_filters( 'ppp_schedule_share_post_types', $allowed_post_types );
 
-	if ( ! isset( $_POST['post_status'] ) || ! array_key_exists( $post->post_type, $allowed_post_types ) ) {
+	if ( ! isset($_POST['post_status']) || ! array_key_exists( $post->post_type, $allowed_post_types )) {
 		return;
 	}
 
 	ppp_remove_scheduled_shares( $post_id );
 
-	if( ( $_POST['post_status'] == 'publish' && $_POST['original_post_status'] != 'publish' ) || // From anything to published
-		( $_POST['post_status'] == 'future' && $_POST['original_post_status'] == 'future' ) || // Updating a future post
-		( $_POST['post_status'] == 'publish' && $_POST['original_post_status'] == 'publish' ) ) { // Updating an already published post
+	if (($_POST['post_status'] == 'publish' && $_POST['original_post_status'] != 'publish') || // From anything to published
+		($_POST['post_status'] == 'future' && $_POST['original_post_status'] == 'future') || // Updating a future post
+		($_POST['post_status'] == 'publish' && $_POST['original_post_status'] == 'publish')) { // Updating an already published post
 		global $ppp_options, $ppp_social_settings;
 
 		$timestamps = ppp_get_timestamps( $post_id );
 
-		foreach ( $timestamps as $timestamp => $name ) {
+		foreach ($timestamps as $timestamp => $name) {
 			$timestamp = substr( $timestamp, 0, strlen( $timestamp ) - 3 );
 			wp_schedule_single_event( $timestamp, 'ppp_share_post_event', array( $post_id, $name ) );
 		}
@@ -48,7 +48,7 @@ 
 
 	$current_item_shares = ppp_get_shceduled_crons( $post_id );
 
-	foreach ( $current_item_shares as $share ) {
+	foreach ($current_item_shares as $share) {
 		wp_clear_scheduled_hook( 'ppp_share_post_event', array( $post_id, $share['args'][1] ) );
 	}
 
@@ -73,19 +73,19 @@ 
 	$all_crons = get_option( 'cron' );
 	$ppp_crons = array();
 
-	foreach ( $all_crons as $timestamp => $cron ) {
-		if ( ! isset( $cron['ppp_share_post_event'] ) ) {
+	foreach ($all_crons as $timestamp => $cron) {
+		if ( ! isset($cron['ppp_share_post_event'])) {
 			continue;
 		}
 
-		foreach ( $cron['ppp_share_post_event'] as $key => $single_event ) {
+		foreach ($cron['ppp_share_post_event'] as $key => $single_event) {
 			$name_parts = explode( '_', $single_event['args'][1] );
-			if ( false !== $post_id && $post_id != $name_parts[2] ) {
+			if (false !== $post_id && $post_id != $name_parts[2]) {
 				continue;
 			}
 
 			$single_event['timestamp'] = $timestamp;
-			$ppp_crons[ $key ]         = $single_event;
+			$ppp_crons[$key]         = $single_event;
 		}
 
 	}
@@ -103,26 +103,26 @@ 
  *
  */
 function ppp_has_cron_within( $time = 0, $within = 0 ) {
-	if ( empty( $time ) ) {
+	if (empty($time)) {
 		$time = current_time( 'timestamp' );
 	}
 
-	if ( empty( $within ) ) {
+	if (empty($within)) {
 		$within = ppp_get_default_conflict_window();
 	}
 
 	$crons = ppp_get_shceduled_crons();
 
-	if ( empty( $crons ) ) {
+	if (empty($crons)) {
 		return false;
 	}
 
 	$scheduled_times = wp_list_pluck( $crons, 'timestamp' );
 
 	$found_time = false;
-	foreach ( $scheduled_times as $key => $scheduled_time ) {
+	foreach ($scheduled_times as $key => $scheduled_time) {
 		$found_time = ppp_is_time_within( $scheduled_time, $time, $within );
-		if ( $found_time ) {
+		if ($found_time) {
 			break;
 		}
 	}
@@ -144,7 +144,7 @@ 
 	$min = $target_time - $within;
 	$max = $target_time + $within;
 
-	return ( ( $time >= $min ) && ( $time <= $max ) );
+	return (($time >= $min) && ($time <= $max));
 }
 
 /**
@@ -212,7 +212,7 @@ 
  */
 function ppp_unschedule_shares( $new_status, $old_status, $post ) {
 
-	if ( ( $old_status == 'publish' || $old_status == 'future' ) && ( $new_status != 'publish' && $new_status != 'future' ) ) {
+	if (($old_status == 'publish' || $old_status == 'future') && ($new_status != 'publish' && $new_status != 'future')) {
 		ppp_remove_scheduled_shares( $post->ID );
 	}
 
@@ -233,15 +233,15 @@ 
 	$minutes    = (int) substr( $share_time[1], 0, 2 );
 	$ampm       = strtolower( substr( $share_time[1], -2 ) );
 
-	if ( $ampm == 'pm' && $hours != 12 ) {
+	if ($ampm == 'pm' && $hours != 12) {
 		$hours = $hours + 12;
 	}
 
-	if ( $ampm == 'am' && $hours == 12 ) {
+	if ($ampm == 'am' && $hours == 12) {
 		$hours = 00;
 	}
 
-	$offset    = (int) -( get_option( 'gmt_offset' ) );
+	$offset    = (int) -(get_option( 'gmt_offset' ));
 	$hours     = $hours + $offset;
 	$date      = explode( '/', $date );
 	$timestamp = mktime( $hours, $minutes, 0, $date[0], $date[1], $date[2] );