postpromoterpro / post-promoter-pro

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'] ) ) {
+			if (isset($_REQUEST['oauth_verifier'])) {
 
 				//load twitter class
 				$twitter       = $this->load();
@@ -67,13 +67,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;
 
@@ -86,7 +86,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,
@@ -95,10 +95,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();
 
@@ -115,9 +115,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' );
 			}
 
@@ -126,7 +126,7 @@ 
 			$request_token = $this->twitter->getRequestToken( 'oob' );
 
 			// 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'];
@@ -143,14 +143,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 {
@@ -165,13 +165,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/twitter-functions.php

Spacing +143 added lines, -143 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,17 +83,17 @@ 
  */
 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 .= '<span id="tw-oob-auth-link-wrapper"><a id="tw-oob-auth-link" href="' . $tw_authurl . '" target="_blank"><img src="' . PPP_URL . '/includes/images/sign-in-with-twitter-gray.png" /></a></span>';
-		$string .= '<span style="display:none;" id="tw-oob-pin-notice">' . __( 'You are being directed to Twitter to authenticate. When complete, return here and enter the PIN you were provided.', 'ppp-txt' ) . '</span>';
-		$string .= '<span style="display:none;" id="tw-oob-pin-wrapper"><input type="text" size="10" placeholder="Enter your PIN" value="" id="tw-oob-pin" data-nonce="' . wp_create_nonce( 'ppp-tw-pin' ) . '" data-user="0" /> <a href="#" class="button-secondary tw-oob-pin-submit">' . __( 'Submit', 'ppp-txt' ) . '</a><span class="spinner"></span></span>';
+		$string .= '<span id="tw-oob-auth-link-wrapper"><a id="tw-oob-auth-link" href="'.$tw_authurl.'" target="_blank"><img src="'.PPP_URL.'/includes/images/sign-in-with-twitter-gray.png" /></a></span>';
+		$string .= '<span style="display:none;" id="tw-oob-pin-notice">'.__( 'You are being directed to Twitter to authenticate. When complete, return here and enter the PIN you were provided.', 'ppp-txt' ).'</span>';
+		$string .= '<span style="display:none;" id="tw-oob-pin-wrapper"><input type="text" size="10" placeholder="Enter your PIN" value="" id="tw-oob-pin" data-nonce="'.wp_create_nonce( 'ppp-tw-pin' ).'" data-user="0" /> <a href="#" class="button-secondary tw-oob-pin-submit">'.__( 'Submit', 'ppp-txt' ).'</a><span class="spinner"></span></span>';
 	} 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;
@@ -106,27 +106,27 @@ 
 
 	ppp_set_social_tokens();
 
-	$nonce = isset( $_POST['nonce'] ) ? sanitize_text_field( $_POST['nonce'] ) : '';
+	$nonce = isset($_POST['nonce']) ? sanitize_text_field( $_POST['nonce'] ) : '';
 	$nonce_verified = wp_verify_nonce( $nonce, 'ppp-tw-pin' );
 
-	if ( ! $nonce_verified ) {
+	if ( ! $nonce_verified) {
 		wp_die();
 	}
 
-	$pin = isset( $_POST['pin'] ) ? absint( $_POST['pin'] ) : false;
+	$pin = isset($_POST['pin']) ? absint( $_POST['pin'] ) : false;
 
-	if ( empty( $pin ) ) {
+	if (empty($pin)) {
 		wp_die();
 	}
 
 	$_REQUEST['oauth_verifier'] = $pin;
 
-	if ( empty( $_POST['user_auth'] ) ) {
+	if (empty($_POST['user_auth'])) {
 		$twitter = new PPP_Twitter;
 		$twitter->ppp_initialize_twitter();
 		$settings = get_option( 'ppp_social_settings', true );
 
-		if ( ! empty( $settings['twitter']['user']->id ) ) {
+		if ( ! empty($settings['twitter']['user']->id)) {
 			echo 1;
 		} else {
 			echo 0;
@@ -136,7 +136,7 @@ 
 		$twitter->init();
 
 		$user = get_user_meta( get_current_user_id(), '_ppp_twitter_data', true );
-		if ( ! empty( $user['user']->id ) ) {
+		if ( ! empty($user['user']->id)) {
 			echo 1;
 		} else {
 			echo 0;
@@ -152,13 +152,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();
@@ -176,17 +176,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( PostPromoterPro::get_manage_capability() ) ) {
+		if ($user_id !== get_current_user_id() || ! current_user_can( PostPromoterPro::get_manage_capability() )) {
 			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 );
 		}
 	}
@@ -217,12 +217,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 );
@@ -249,12 +249,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 );
 		}
@@ -275,8 +275,8 @@ 
 		);
 		$other_rt = get_users( $args );
 
-		if ( $other_rt ){
-			foreach ( $other_rt as $user ) {
+		if ($other_rt) {
+			foreach ($other_rt as $user) {
 				$twitter_user = new PPP_Twitter_User( $user->ID );
 				$twitter_user->retweet( $status->id_str );
 			}
@@ -296,9 +296,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, $scheduled );
 
-	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 );
@@ -312,22 +312,22 @@ 
  */
 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 ( ! empty( $ppp_tweets ) ) {
+	if ( ! empty($ppp_tweets)) {
 		$name_array    = explode( '_', $name );
 		$index         = $name_array[1];
-		if ( isset( $ppp_tweets[ $index ] ) ) {
-			$share_content = $ppp_tweets[ $index ]['text'];
+		if (isset($ppp_tweets[$index])) {
+			$share_content = $ppp_tweets[$index]['text'];
 		}
 	}
 
 	// 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 ) );
 }
@@ -339,12 +339,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;
 }
@@ -355,7 +355,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 );
 	}
@@ -393,7 +393,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;
 	}
 
@@ -410,7 +410,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;
 	}
 
@@ -469,16 +469,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 ); ?>
@@ -487,7 +487,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'          => '',
@@ -510,7 +510,7 @@ 
 								<tr class="ppp-add-repeatable-wrapper">
 									<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>
@@ -536,38 +536,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['share_on_publish'][ $post->post_type ]['twitter'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['share_on_publish'][$post->post_type]['twitter']) ? 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>
-			<textarea <?php if ( $disabled ): ?>readonly<?php endif; ?> class="ppp-tweet-text-repeatable" type="text" name="_ppp_share_on_publish_text"><?php echo esc_attr( $ppp_share_on_publish_text ); ?></textarea>
-			<?php $length = ! empty( $ppp_share_on_publish_text ) ? strlen( $ppp_share_on_publish_text ) : 0; ?>
+			<textarea <?php if ($disabled): ?>readonly<?php endif; ?> class="ppp-tweet-text-repeatable" type="text" name="_ppp_share_on_publish_text"><?php echo esc_attr( $ppp_share_on_publish_text ); ?></textarea>
+			<?php $length = ! empty($ppp_share_on_publish_text) ? strlen( $ppp_share_on_publish_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>
 
@@ -586,7 +586,7 @@ 
 function ppp_render_tweet_row( $key, $args = array(), $post_id ) {
 	global $post, $has_past_shares;
 
-	if ( ! empty( $args['date'] ) && ! empty( $args['time'] ) ) {
+	if ( ! empty($args['date']) && ! empty($args['time'])) {
 		$share_time = ppp_generate_timestamp( $args['date'], $args['time'] );
 		$readonly   = ppp_generate_timestamp() > $share_time ? 'readonly="readonly" ' : false;
 	} else {
@@ -594,11 +594,11 @@ 
 		$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++;
 	}
 	?>
@@ -613,7 +613,7 @@ 
 
 		<td>
 			<textarea class="ppp-tweet-text-repeatable" type="text" name="_ppp_tweets[<?php echo $key; ?>][text]" <?php echo $readonly; ?>><?php echo esc_attr( $args['text'] ); ?></textarea>
-			<?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>
 
@@ -648,24 +648,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 );
 
@@ -682,29 +682,29 @@ 
 function ppp_tw_share_on_publish( $new_status, $old_status, $post ) {
 	global $ppp_options;
 
-	$from_meta = ! empty( $_POST['ppp_post_edit'] ) ? false : get_post_meta( $post->ID, '_ppp_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_share_on_publish'] );
+	$from_meta = ! empty($_POST['ppp_post_edit']) ? false : get_post_meta( $post->ID, '_ppp_share_on_publish', true );
+	$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 );
 
@@ -722,11 +722,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 );
 		}
@@ -747,8 +747,8 @@ 
 		);
 		$other_rt = get_users( $args );
 
-		if ( $other_rt ){
-			foreach ( $other_rt as $user ) {
+		if ($other_rt) {
+			foreach ($other_rt as $user) {
 				$twitter_user = new PPP_Twitter_User( $user->ID );
 				$twitter_user->retweet( $status->id_str );
 			}
@@ -769,20 +769,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';
 		}
 
 	}
@@ -802,7 +802,7 @@ 
 
 	$ret = false;
 
-	if ( ! empty( $ppp_share_settings['twitter']['cards_enabled'] ) ) {
+	if ( ! empty($ppp_share_settings['twitter']['cards_enabled'])) {
 		$ret = true;
 	}
 
@@ -817,13 +817,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;
 	}
 
@@ -841,20 +841,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 );
@@ -874,15 +874,15 @@ 
 	$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;
 
 		$thumb_id = ppp_get_attachment_id_from_image_url( $image_url );
-		if ( ! empty( $thumb_id ) ) {
+		if ( ! empty($thumb_id)) {
 			$alt_text = ppp_get_attachment_alt_text( $thumb_id );
 			// When adding media via the WP Uploader, any 'alt text' supplied will be used as the accessible alt text.
-			if ( ! empty( $alt_text ) ) {
+			if ( ! empty($alt_text)) {
 				$elements['twitter:image:alt'] = esc_attr( $alt_text );
 			}
 		}
@@ -890,15 +890,15 @@ 
 		$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 ) );
@@ -917,13 +917,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->post_content );
 	}
 
@@ -941,10 +941,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;
@@ -959,7 +959,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
@@ -978,15 +978,15 @@ 
 function ppp_tw_profile_settings( $user ) {
 	global $ppp_social_settings;
 
-	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( PostPromoterPro::get_manage_capability() ) ) {
+	if ($user->ID !== get_current_user_id() && ! current_user_can( PostPromoterPro::get_manage_capability() )) {
 		return;
 	}
 
-	if ( ! isset( $ppp_social_settings['twitter'] ) || is_null( $ppp_social_settings['twitter'] ) ) {
+	if ( ! isset($ppp_social_settings['twitter']) || is_null( $ppp_social_settings['twitter'] )) {
 		return;
 	}
 
@@ -1001,12 +1001,12 @@ 
 			$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 ) );
 
-				$string  = '<span id="tw-oob-auth-link-wrapper"><a id="tw-oob-auth-link" href="' . $tw_authurl . '" target="_blank"><img src="' . PPP_URL . '/includes/images/sign-in-with-twitter-gray.png" /></a></span>';
-				$string .= '<span style="display:none;" id="tw-oob-pin-notice">' . __( 'You are being directed to Twitter to authenticate. When complete, return here and enter the PIN you were provided.', 'ppp-txt' ) . '</span>';
-				$string .= '<span style="display:none;" id="tw-oob-pin-wrapper"><input type="text" size="10" placeholder="Enter your PIN" value="" id="tw-oob-pin" data-nonce="' . wp_create_nonce( 'ppp-tw-pin' ) . '" data-user="1" /> <a href="#" class="button-secondary tw-oob-pin-submit">' . __( 'Submit', 'ppp-txt' ) . '</a><span class="spinner"></span></span>';
+				$string  = '<span id="tw-oob-auth-link-wrapper"><a id="tw-oob-auth-link" href="'.$tw_authurl.'" target="_blank"><img src="'.PPP_URL.'/includes/images/sign-in-with-twitter-gray.png" /></a></span>';
+				$string .= '<span style="display:none;" id="tw-oob-pin-notice">'.__( 'You are being directed to Twitter to authenticate. When complete, return here and enter the PIN you were provided.', 'ppp-txt' ).'</span>';
+				$string .= '<span style="display:none;" id="tw-oob-pin-wrapper"><input type="text" size="10" placeholder="Enter your PIN" value="" id="tw-oob-pin" data-nonce="'.wp_create_nonce( 'ppp-tw-pin' ).'" data-user="1" /> <a href="#" class="button-secondary tw-oob-pin-submit">'.__( 'Submit', 'ppp-txt' ).'</a><span class="spinner"></span></span>';
 				$string .= '<input type="hidden" name="ppp_user_auth" value="1" />';
 
 				echo $string;
@@ -1015,7 +1015,7 @@ 
 				?>
 				<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
@@ -1024,12 +1024,12 @@ 
 			</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 );
 			$share_others_on_publish 	= get_user_meta( $user->ID, '_ppp_share_others_on_publish', true );
-			$share_others_scheduled  	= get_user_meta( $user->ID, '_ppp_share_others_scheduled' , true );
+			$share_others_scheduled  	= get_user_meta( $user->ID, '_ppp_share_others_scheduled', true );
 		?>
 		<tr>
 			<th><?php _e( 'Sharing Options', 'ppp-txt' ); ?></th>
@@ -1077,19 +1077,19 @@ 
 function ppp_tw_save_profile( $user_id ) {
 	global $ppp_social_settings;
 
-	if ( ! isset( $ppp_social_settings['twitter'] ) || is_null( $ppp_social_settings['twitter'] ) ) {
+	if ( ! isset($ppp_social_settings['twitter']) || is_null( $ppp_social_settings['twitter'] )) {
 		return;
 	}
 
-	$share_on_publish 			= ! empty( $_POST['share_on_publish'] ) ? true : false;
-	$share_scheduled  			= ! empty( $_POST['share_scheduled'] )  ? true : false;
-	$share_others_on_publish 	= ! empty( $_POST['share_others_on_publish'] ) ? true : false;
-	$share_others_scheduled  	= ! empty( $_POST['share_others_scheduled'] )  ? true : false;
+	$share_on_publish 			= ! empty($_POST['share_on_publish']) ? true : false;
+	$share_scheduled  			= ! empty($_POST['share_scheduled']) ? true : false;
+	$share_others_on_publish 	= ! empty($_POST['share_others_on_publish']) ? true : false;
+	$share_others_scheduled  	= ! empty($_POST['share_others_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 );
 	update_user_meta( $user_id, '_ppp_share_others_on_publish', $share_others_on_publish );
-	update_user_meta( $user_id, '_ppp_share_others_scheduled', $share_others_scheduled  );
+	update_user_meta( $user_id, '_ppp_share_others_scheduled', $share_others_scheduled );
 
 }
 add_action( 'personal_options_update', 'ppp_tw_save_profile' );
@@ -1098,14 +1098,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,
 		);
 	}
@@ -1116,9 +1116,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/facebook-functions.php

Spacing +115 added lines, -115 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -12,7 +12,7 @@ 
 function ppp_facebook_enabled() {
 	global $ppp_social_settings;
 
-	if ( isset( $ppp_social_settings['facebook'] ) && !empty( $ppp_social_settings['facebook'] ) ) {
+	if (isset($ppp_social_settings['facebook']) && ! empty($ppp_social_settings['facebook'])) {
 		return true;
 	}
 
@@ -50,10 +50,10 @@ 
  */
 function ppp_fb_account_list_avatar( $string = '' ) {
 
-	if ( ppp_facebook_enabled() ) {
+	if (ppp_facebook_enabled()) {
 		global $ppp_social_settings;
 		$avatar_url = $ppp_social_settings['facebook']->avatar;
-		$string = '<img class="ppp-social-icon" src="' . $avatar_url . '" />';
+		$string = '<img class="ppp-social-icon" src="'.$avatar_url.'" />';
 	}
 
 	return $string;
@@ -67,9 +67,9 @@ 
  */
 function ppp_fb_account_list_name( $string = '' ) {
 
-	if ( ppp_facebook_enabled() ) {
+	if (ppp_facebook_enabled()) {
 		global $ppp_social_settings;
-		$string  = $ppp_social_settings['facebook']->name;
+		$string = $ppp_social_settings['facebook']->name;
 	}
 
 	return $string;
@@ -84,22 +84,22 @@ 
 function ppp_fb_account_list_actions( $string = '' ) {
 	global $ppp_facebook_oauth, $ppp_social_settings;
 
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 
 		$fb_authurl = $ppp_facebook_oauth->ppp_get_facebook_auth_url( admin_url( 'admin.php?page=ppp-social-settings' ) );
 
-		$string .= '<a class="button-primary" href="' . $fb_authurl . '">' . __( 'Connect to Facebook', 'ppp-txt' ) . '</a>';
+		$string .= '<a class="button-primary" href="'.$fb_authurl.'">'.__( 'Connect to Facebook', 'ppp-txt' ).'</a>';
 	} else {
-		$string  .= '<a class="button-primary" href="' . admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=facebook' ) . '" >' . __( 'Disconnect from Facebook', 'ppp-txt' ) . '</a>&nbsp;';
+		$string .= '<a class="button-primary" href="'.admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=facebook' ).'" >'.__( 'Disconnect from Facebook', 'ppp-txt' ).'</a>&nbsp;';
 
 		$refresh_date = (int) get_option( '_ppp_facebook_refresh', true );
 
-		if ( defined( 'PPP_FB_APP_ID' ) && current_time( 'timestamp' ) > $refresh_date ) {
+		if (defined( 'PPP_FB_APP_ID' ) && current_time( 'timestamp' ) > $refresh_date) {
 			$token       = $ppp_social_settings['facebook']->access_token;
 			$url         = $ppp_facebook_oauth->ppp_get_facebook_auth_url( admin_url( 'admin.php?page=ppp-social-settings' ) );
-			$refresh_url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token=' . $token, $url );
+			$refresh_url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token='.$token, $url );
 
-			$string  .= '<a class="button-secondary" href="' . $refresh_url . '" >' . __( 'Re-Authorize Facebook', 'ppp-txt' ) . '</a>&nbsp;';
+			$string .= '<a class="button-secondary" href="'.$refresh_url.'" >'.__( 'Re-Authorize Facebook', 'ppp-txt' ).'</a>&nbsp;';
 		}
 	}
 
@@ -114,27 +114,27 @@ 
  */
 function ppp_fb_account_list_extras( $string ) {
 
-	if ( ppp_facebook_enabled() ) {
+	if (ppp_facebook_enabled()) {
 		global $ppp_social_settings, $ppp_facebook_oauth, $ppp_options;
 		$pages = $ppp_facebook_oauth->ppp_get_fb_user_pages( $ppp_social_settings['facebook']->access_token );
-		$selected = isset( $ppp_social_settings['facebook']->page ) ? stripslashes( $ppp_social_settings['facebook']->page ) : 'me';
+		$selected = isset($ppp_social_settings['facebook']->page) ? stripslashes( $ppp_social_settings['facebook']->page ) : 'me';
 
-		if ( !empty( $pages ) ) {
-			$string = '<label>' . __( 'Publish as:', 'ppp-txt' ) . '</label><br />';
+		if ( ! empty($pages)) {
+			$string = '<label>'.__( 'Publish as:', 'ppp-txt' ).'</label><br />';
 			$string .= '<select id="fb-page">';
-			foreach ( $pages as $page ) {
-				$value = $page->name . '|' . $page->access_token . '|' . $page->id;
-				$string .= '<option ' . selected( $value, $selected, false ) . ' value="' . $value . '">' . $page->name . '</option>';
+			foreach ($pages as $page) {
+				$value = $page->name.'|'.$page->access_token.'|'.$page->id;
+				$string .= '<option '.selected( $value, $selected, false ).' value="'.$value.'">'.$page->name.'</option>';
 			}
 			$string .= '</select><span class="spinner"></span>';
 		}
 
-		if ( ! empty( $ppp_options['enable_debug'] ) ) {
-			$days_left  = absint( round( ( $ppp_social_settings['facebook']->expires_on - current_time( 'timestamp' ) ) / DAY_IN_SECONDS ) );
-			$refresh_in = absint( round( ( get_option( '_ppp_facebook_refresh' ) - current_time( 'timestamp' ) ) / DAY_IN_SECONDS ) );
+		if ( ! empty($ppp_options['enable_debug'])) {
+			$days_left  = absint( round( ($ppp_social_settings['facebook']->expires_on - current_time( 'timestamp' )) / DAY_IN_SECONDS ) );
+			$refresh_in = absint( round( (get_option( '_ppp_facebook_refresh' ) - current_time( 'timestamp' )) / DAY_IN_SECONDS ) );
 
-			$string .= '<br />' . sprintf( __( 'Token expires in %s days' , 'ppp-txt' ), $days_left );
-			$string .= '<br />' . sprintf( __( 'Refresh notice in %s days', 'ppp-txt' ), $refresh_in );
+			$string .= '<br />'.sprintf( __( 'Token expires in %s days', 'ppp-txt' ), $days_left );
+			$string .= '<br />'.sprintf( __( 'Refresh notice in %s days', 'ppp-txt' ), $refresh_in );
 		}
 	}
 
@@ -148,7 +148,7 @@ 
  * @return void
  */
 function ppp_set_fb_token_constants( $social_tokens ) {
-	if ( !empty( $social_tokens ) && property_exists( $social_tokens, 'facebook' ) ) {
+	if ( ! empty($social_tokens) && property_exists( $social_tokens, 'facebook' )) {
 		define( 'PPP_FB_APP_ID', $social_tokens->facebook->app_id );
 		define( 'PPP_FB_APP_SECRET', $social_tokens->facebook->app_secret );
 	}
@@ -162,17 +162,17 @@ 
 function ppp_capture_facebook_oauth() {
 	$should_capture = false;
 
-	if ( isset( $_GET['state'] ) && strpos( $_GET['state'], 'ppp-local-keys-fb' ) !== false ) {
+	if (isset($_GET['state']) && strpos( $_GET['state'], 'ppp-local-keys-fb' ) !== false) {
 		// Local config
 		$should_capture = true;
 	}
 
-	if ( isset( $_REQUEST['fb_access_token'] ) ) {
+	if (isset($_REQUEST['fb_access_token'])) {
 		// Returning from remote config
 		$should_capture = true;
 	}
 
-	if ( $should_capture && ( isset( $_REQUEST['page'] ) && $_REQUEST['page'] == 'ppp-social-settings' ) ) {
+	if ($should_capture && (isset($_REQUEST['page']) && $_REQUEST['page'] == 'ppp-social-settings')) {
 		global $ppp_facebook_oauth;
 		$ppp_facebook_oauth->ppp_initialize_facebook();
 		wp_redirect( admin_url( 'admin.php?page=ppp-social-settings' ) );
@@ -189,8 +189,8 @@ 
 function ppp_disconnect_facebook() {
 	global $ppp_social_settings;
 	$ppp_social_settings = get_option( 'ppp_social_settings' );
-	if ( isset( $ppp_social_settings['facebook'] ) ) {
-		unset( $ppp_social_settings['facebook'] );
+	if (isset($ppp_social_settings['facebook'])) {
+		unset($ppp_social_settings['facebook']);
 		update_option( 'ppp_social_settings', $ppp_social_settings );
 		delete_option( '_ppp_facebook_refresh' );
 	}
@@ -216,13 +216,13 @@ 
  */
 function ppp_fb_execute_refresh() {
 
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return;
 	}
 
 	$refresh_date = (int) get_option( '_ppp_facebook_refresh', true );
 
-	if ( ( empty( $_GET['page' ] ) || $_GET['page'] !== 'ppp-social-settings' ) && current_time( 'timestamp' ) > $refresh_date ) {
+	if ((empty($_GET['page']) || $_GET['page'] !== 'ppp-social-settings') && current_time( 'timestamp' ) > $refresh_date) {
 		add_action( 'admin_notices', 'ppp_facebook_refresh_notice' );
 	}
 }
@@ -234,12 +234,12 @@ 
  */
 function ppp_facebook_refresh_notice() {
 
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return;
 	}
 
-	$has_dismissed = get_transient( 'ppp-dismiss-refresh-fb' . get_current_user_id() );
-	if ( false !== $has_dismissed ) {
+	$has_dismissed = get_transient( 'ppp-dismiss-refresh-fb'.get_current_user_id() );
+	if (false !== $has_dismissed) {
 		return;
 	}
 
@@ -250,14 +250,14 @@ 
 
 	$token = $ppp_social_settings['facebook']->access_token;
 	$url = $ppp_facebook_oauth->ppp_get_facebook_auth_url( admin_url( 'admin.php?page=ppp-social-settings' ) );
-	$url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token=' . $token, $url );
+	$url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token='.$token, $url );
 
-	$days_left = (int) round( ( $ppp_social_settings['facebook']->expires_on - current_time( 'timestamp' ) ) / DAY_IN_SECONDS );
+	$days_left = (int) round( ($ppp_social_settings['facebook']->expires_on - current_time( 'timestamp' )) / DAY_IN_SECONDS );
 	?>
 	<div class="notice notice-warning is-dismissible" data-service="fb">
-		<?php if ( $days_left > 0 ): ?>
+		<?php if ($days_left > 0): ?>
 			<p><strong>Post Promoter Pro: </strong><?php printf( __( 'Your Facebook authentication expires in %d days. Please <a href="%s">refresh access</a>.', 'ppp-txt' ), $days_left, $url ); ?></p>
-		<?php elseif ( $days_left < 1 ): ?>
+		<?php elseif ($days_left < 1): ?>
 			<p><strong>Post Promoter Pro: </strong><?php printf( __( 'Your Facebook authentication has expired. Please <a href="%s">refresh access</a>.', 'ppp-txt' ), $url ); ?></p>
 		<?php endif; ?>
 	</div>
@@ -272,10 +272,10 @@ 
  */
 function ppp_fb_dismiss_notice() {
 
-	$nag = sanitize_key( $_POST[ 'nag' ] );
+	$nag = sanitize_key( $_POST['nag'] );
 
-	if ( $nag === $_POST[ 'nag' ] ) {
-		set_transient( $nag . get_current_user_id(), true, DAY_IN_SECONDS );
+	if ($nag === $_POST['nag']) {
+		set_transient( $nag.get_current_user_id(), true, DAY_IN_SECONDS );
 	}
 
 
@@ -303,18 +303,18 @@ 
  * @param  string  $name    The name of the Cron
  * @return void
  */
-function ppp_fb_scheduled_share(  $post_id = 0, $index = 1, $name = ''  ) {
+function ppp_fb_scheduled_share( $post_id = 0, $index = 1, $name = '' ) {
 	global $ppp_options;
 
 	$link = ppp_generate_link( $post_id, $name );
 
 	$post_meta     = get_post_meta( $post_id, '_ppp_fb_shares', 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_fb_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_fb_use_media( $post_id, $index );
@@ -372,7 +372,7 @@ 
  */
 function ppp_fb_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return $tabs;
 	}
 
@@ -389,7 +389,7 @@ 
  */
 function ppp_fb_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return $content;
 	}
 
@@ -405,7 +405,7 @@ 
  */
 function ppp_fb_add_metabox_content( $post ) {
 	global $ppp_options, $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_fb_share_on_publish               = get_post_meta( $post->ID, '_ppp_fb_share_on_publish', true );
 	$ppp_share_on_publish_title            = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_title', true );
@@ -414,9 +414,9 @@ 
 
 	$show_share_on_publish = false;
 
-	$share_by_default      = empty( $ppp_share_settings['share_on_publish'][ $post->post_type ]['facebook'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['share_on_publish'][$post->post_type]['facebook']) ? false : true;
 
-	if ( $ppp_fb_share_on_publish == '1' || ( $ppp_fb_share_on_publish == '' && $share_by_default ) ) {
+	if ($ppp_fb_share_on_publish == '1' || ($ppp_fb_share_on_publish == '' && $share_by_default)) {
 		$show_share_on_publish = true;
 	}
 
@@ -425,11 +425,11 @@ 
 		<div class="ppp-post-override-wrap">
 			<p><h3><?php _e( 'Share on Facebook', 'ppp-txt' ); ?></h3></p>
 			<p>
-				<?php $disabled = ( $post->post_status === 'publish' && time() > strtotime( $post->post_date ) ) ? true : false; ?>
+				<?php $disabled = ($post->post_status === 'publish' && time() > strtotime( $post->post_date )) ? true : false; ?>
 				<label for="ppp_fb_share_on_publish"><?php _e( 'Share this post on Facebook&hellip;', 'ppp-txt' ); ?></label>
 				<select name="_ppp_fb_share_on_publish" id="ppp_fb_share_on_publish" class="ppp-toggle-share-on-publish">
-					<option value="-1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ( $disabled ): ?>disabled<?php endif; ?>><?php _e( 'Do not share this post', 'ppp-txt' ); ?></option>
-					<option value="1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ( $disabled ): ?>disabled<?php endif; ?>><?php _e( 'When this post is published', 'ppp-txt' ); ?></option>
+					<option value="-1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ($disabled): ?>disabled<?php endif; ?>><?php _e( 'Do not share this post', 'ppp-txt' ); ?></option>
+					<option value="1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ($disabled): ?>disabled<?php endif; ?>><?php _e( 'When this post is published', 'ppp-txt' ); ?></option>
 					<option value="0" <?php selected( false, $show_share_on_publish, true ); ?>><?php _e( 'After this post is published', 'ppp-txt' ); ?></option>
 				</select>
 			</p>
@@ -445,7 +445,7 @@ 
 								<th style="width: 10px;"></th>
 							</tr>
 						</thead>
-						<tbody id="fb-share-on-publish" class="ppp-share-on-publish" <?php if ( false === $show_share_on_publish ) : echo 'style="display: none;"'; endif; ?>>
+						<tbody id="fb-share-on-publish" class="ppp-share-on-publish" <?php if (false === $show_share_on_publish) : echo 'style="display: none;"'; endif; ?>>
 							<?php
 								$args = array(
 									'text'          => $ppp_share_on_publish_title,
@@ -456,18 +456,18 @@ 
 								ppp_render_fb_share_on_publish_row( $args );
 							?>
 						</tbody>
-						<tbody id="fb-schedule-share" class="ppp-schedule-share" <?php if ( true === $show_share_on_publish ) : echo 'style="display: none;"'; endif; ?>>
+						<tbody id="fb-schedule-share" class="ppp-schedule-share" <?php if (true === $show_share_on_publish) : echo 'style="display: none;"'; endif; ?>>
 							<?php $shares = get_post_meta( $post->ID, '_ppp_fb_shares', true ); ?>
-							<?php if ( ! empty( $shares ) ) : ?>
+							<?php if ( ! empty($shares)) : ?>
 
-								<?php foreach ( $shares 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 ($shares 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_fb_row_args', compact( 'date','time','text','image','attachment_id' ), $value );
+									$args = apply_filters( 'ppp_fb_row_args', compact( 'date', 'time', 'text', 'image', 'attachment_id' ), $value );
 									?>
 
 									<?php ppp_render_fb_share_row( $key, $args, $post->ID ); ?>
@@ -547,8 +547,8 @@ 
 
 	$share_time = ppp_generate_timestamp( $args['date'], $args['time'] );
 	$readonly   = ppp_generate_timestamp() > $share_time ? 'readonly="readonly" ' : false;
-	$no_date    = ! empty( $readonly ) ? ' hasDatepicker' : '';
-	$hide       = ! empty( $readonly ) ? 'display: none;' : '';
+	$no_date    = ! empty($readonly) ? ' hasDatepicker' : '';
+	$hide       = ! empty($readonly) ? 'display: none;' : '';
 	?>
 	<tr class="ppp-fb-wrapper ppp-repeatable-row ppp-repeatable-facebook scheduled-row" data-key="<?php echo esc_attr( $key ); ?>">
 		<td>
@@ -593,23 +593,23 @@ 
  */
 function ppp_fb_save_post_meta_boxes( $post_id, $post ) {
 
-	if ( ! ppp_should_save( $post_id, $post ) ) {
+	if ( ! ppp_should_save( $post_id, $post )) {
 		return;
 	}
 
-	$ppp_fb_share_on_publish            = ( isset( $_REQUEST['_ppp_fb_share_on_publish'] ) )               ? $_REQUEST['_ppp_fb_share_on_publish']               : '-1';
-	$ppp_share_on_publish_title         = ( isset( $_REQUEST['_ppp_fb_share_on_publish_title'] ) )         ? $_REQUEST['_ppp_fb_share_on_publish_title']         : '';
-	$ppp_share_on_publish_image_url     = ( isset( $_REQUEST['_ppp_fb_share_on_publish_image_url'] ) )     ? $_REQUEST['_ppp_fb_share_on_publish_image_url']     : '';
-	$ppp_share_on_publish_attachment_id = ( isset( $_REQUEST['_ppp_fb_share_on_publish_attachment_id'] ) ) ? $_REQUEST['_ppp_fb_share_on_publish_attachment_id'] : '';
+	$ppp_fb_share_on_publish            = (isset($_REQUEST['_ppp_fb_share_on_publish'])) ? $_REQUEST['_ppp_fb_share_on_publish'] : '-1';
+	$ppp_share_on_publish_title         = (isset($_REQUEST['_ppp_fb_share_on_publish_title'])) ? $_REQUEST['_ppp_fb_share_on_publish_title'] : '';
+	$ppp_share_on_publish_image_url     = (isset($_REQUEST['_ppp_fb_share_on_publish_image_url'])) ? $_REQUEST['_ppp_fb_share_on_publish_image_url'] : '';
+	$ppp_share_on_publish_attachment_id = (isset($_REQUEST['_ppp_fb_share_on_publish_attachment_id'])) ? $_REQUEST['_ppp_fb_share_on_publish_attachment_id'] : '';
 
-	update_post_meta( $post_id, '_ppp_fb_share_on_publish',               $ppp_fb_share_on_publish );
-	update_post_meta( $post_id, '_ppp_fb_share_on_publish_title',         $ppp_share_on_publish_title );
-	update_post_meta( $post_id, '_ppp_fb_share_on_publish_image_url',     $ppp_share_on_publish_image_url );
+	update_post_meta( $post_id, '_ppp_fb_share_on_publish', $ppp_fb_share_on_publish );
+	update_post_meta( $post_id, '_ppp_fb_share_on_publish_title', $ppp_share_on_publish_title );
+	update_post_meta( $post_id, '_ppp_fb_share_on_publish_image_url', $ppp_share_on_publish_image_url );
 	update_post_meta( $post_id, '_ppp_fb_share_on_publish_attachment_id', $ppp_share_on_publish_attachment_id );
 
-	$fb_data = ( isset( $_REQUEST['_ppp_fb_shares'] ) && empty( $ppp_fb_share_on_publish ) ) ? $_REQUEST['_ppp_fb_shares'] : array();
-	foreach ( $fb_data as $index => $share ) {
-		$fb_data[ $index ]['text'] = sanitize_text_field( $share['text'] );
+	$fb_data = (isset($_REQUEST['_ppp_fb_shares']) && empty($ppp_fb_share_on_publish)) ? $_REQUEST['_ppp_fb_shares'] : array();
+	foreach ($fb_data as $index => $share) {
+		$fb_data[$index]['text'] = sanitize_text_field( $share['text'] );
 	}
 
 	update_post_meta( $post_id, '_ppp_fb_shares', $fb_data );
@@ -626,10 +626,10 @@ 
 function ppp_fb_share_on_publish( $new_status, $old_status, $post ) {
 	global $ppp_options;
 
-	$from_meta = ! empty( $_POST['ppp_post_edit'] ) ? false : get_post_meta( $post->ID, '_ppp_fb_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_fb_share_on_publish'] ) ? $_POST['_ppp_fb_share_on_publish'] : '0';
+	$from_meta = ! empty($_POST['ppp_post_edit']) ? false : get_post_meta( $post->ID, '_ppp_fb_share_on_publish', true );
+	$from_post = isset($_POST['_ppp_fb_share_on_publish']) ? $_POST['_ppp_fb_share_on_publish'] : '0';
 
-	if ( '1' != $from_meta && '1' != $from_post ) {
+	if ('1' != $from_meta && '1' != $from_post) {
 		return;
 	}
 
@@ -641,28 +641,28 @@ 
 	$image_url     = '';
 
 	// Determine if we're seeing the share on publish in meta or $_POST
-	if ( $from_meta && ! $from_post ) {
-		$title         = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_title',         true );
+	if ($from_meta && ! $from_post) {
+		$title         = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_title', true );
 		$attachment_id = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_attachment_id', true );
-		$image_url     = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_image_url',     true );
+		$image_url     = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_image_url', true );
 	} else {
-		$title         = isset( $_POST['_ppp_fb_share_on_publish_title'] )         ? $_POST['_ppp_fb_share_on_publish_title']         : '';
-		$attachment_id = isset( $_POST['_ppp_fb_share_on_publish_attachment_id'] ) ? $_POST['_ppp_fb_share_on_publish_attachment_id'] : 0;
-		$image_url     = isset( $_POST['_ppp_fb_share_on_publish_image_url'] )     ? $_POST['_ppp_fb_share_on_publish_image_url']     : '';
+		$title         = isset($_POST['_ppp_fb_share_on_publish_title']) ? $_POST['_ppp_fb_share_on_publish_title'] : '';
+		$attachment_id = isset($_POST['_ppp_fb_share_on_publish_attachment_id']) ? $_POST['_ppp_fb_share_on_publish_attachment_id'] : 0;
+		$image_url     = isset($_POST['_ppp_fb_share_on_publish_image_url']) ? $_POST['_ppp_fb_share_on_publish_image_url'] : '';
 	}
 
 	$thumbnail = '';
-	if ( empty( $attachment_id ) && ! empty( $image_url ) ) {
+	if (empty($attachment_id) && ! empty($image_url)) {
 		$thumbnail = $image_url;
 	} else {
 		$thumbnail = ppp_post_has_media( $post->ID, 'fb', true, $attachment_id );
 	}
 
-	$name = 'sharedate_0_' . $post->ID . '_fb';
+	$name = 'sharedate_0_'.$post->ID.'_fb';
 
-	$default_title = isset( $ppp_options['default_text'] ) ? $ppp_options['default_text'] : '';
+	$default_title = isset($ppp_options['default_text']) ? $ppp_options['default_text'] : '';
 	// If an override was found, use it, otherwise try the default text content
-	if ( empty( $title ) && empty( $default_title ) ) {
+	if (empty($title) && empty($default_title)) {
 		$title = get_the_title( $post->ID );
 	}
 
@@ -702,20 +702,20 @@ 
 function ppp_fb_generate_timestamps( $times, $post_id ) {
 	$fb_shares = get_post_meta( $post_id, '_ppp_fb_shares', true );
 
-	if ( empty( $fb_shares ) ) {
+	if (empty($fb_shares)) {
 		$fb_shares = array();
 	}
 
-	foreach ( $fb_shares as $key => $data ) {
-		if ( ! array_filter( $data ) ) {
+	foreach ($fb_shares 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 ) ) . '_fb';
-			$times[ $time_key ] = 'sharedate_' . $key . '_' . $post_id . '_fb';
+		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 ) ).'_fb';
+			$times[$time_key] = 'sharedate_'.$key.'_'.$post_id.'_fb';
 		}
 
 	}
@@ -765,22 +765,22 @@ 
  */
 function ppp_fb_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'] : '';
 	$fb_shares    = get_post_meta( $post_id, '_ppp_fb_shares', true );
 
-	if ( ! empty( $fb_shares ) ) {
+	if ( ! empty($fb_shares)) {
 		$name_array    = explode( '_', $name );
 		$index         = $name_array[1];
-		if ( isset( $fb_shares[ $index ] ) ) {
-			$share_content = $fb_shares[ $index ]['text'];
+		if (isset($fb_shares[$index])) {
+			$share_content = $fb_shares[$index]['text'];
 		}
 	}
 
 	// 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_fb', $share_content, array( 'post_id' => $post_id ) );
 }
@@ -792,7 +792,7 @@ 
  * @return bool         Whether or not this tweet should contain a media post
  */
 function ppp_fb_use_media( $post_id, $index ) {
-	if ( empty( $post_id ) || empty( $index ) ) {
+	if (empty($post_id) || empty($index)) {
 		return false;
 	}
 
@@ -808,9 +808,9 @@ 
 
 	ppp_set_social_tokens();
 
-	$account = isset( $_POST['account'] ) ? $_POST['account'] : false;
+	$account = isset($_POST['account']) ? $_POST['account'] : false;
 
-	if ( !empty( $account ) ) {
+	if ( ! empty($account)) {
 		$ppp_social_settings['facebook']->page = $account;
 
 		update_option( 'ppp_social_settings', $ppp_social_settings );
@@ -826,14 +826,14 @@ 
 function ppp_fb_calendar_on_publish_event( $events, $post_id ) {
 	$share_on_publish = get_post_meta( $post_id, '_ppp_fb_share_on_publish', true );
 
-	if ( ! empty( $share_on_publish ) ) {
+	if ( ! empty($share_on_publish)) {
 		$share_text = get_post_meta( $post_id, '_ppp_fb_share_on_publish_title', true );
 		$events[] = array(
-			'id' => $post_id . '-share-on-publish',
-			'title' => ( ! empty( $share_text ) ) ? $share_text : ppp_fb_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-fb cal-post-' . $post_id,
+			'id' => $post_id.'-share-on-publish',
+			'title' => ( ! empty($share_text)) ? $share_text : ppp_fb_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-fb cal-post-'.$post_id,
 			'belongsTo' => $post_id,
 		);
 	}
@@ -844,9 +844,9 @@ 
 
 function ppp_fb_get_post_shares( $items, $post_id ) {
 	$shares = get_post_meta( $post_id, '_ppp_fb_shares', true );
-	if ( empty( $shares ) ) { return $items; }
+	if (empty($shares)) { return $items; }
 
-	foreach ( $shares as $key => $share ) {
+	foreach ($shares as $key => $share) {
 		$items[] = array( 'id' => $key, 'service' => 'fb' );
 	}
 	return $items;
@@ -864,11 +864,11 @@ 
  */
 function ppp_fb_clear_open_graph_cache( $post_id, $post_after, $post_before ) {
 	$post_types = ppp_allowed_post_types();
-	if ( ! in_array( $post_after->post_type, $post_types ) ) {
+	if ( ! in_array( $post_after->post_type, $post_types )) {
 		return;
 	}
 
-	if ( 'publish' == $post_after->post_status ) {
+	if ('publish' == $post_after->post_status) {
 		global $ppp_facebook_oauth;
 
 		return $ppp_facebook_oauth->clear_og_cache( $post_id );

includes/admin/upgrades.php

Spacing +50 added lines, -50 removed lines

@@ -1,13 +1,13 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
 function ppp_upgrade_notices() {
 
-	if ( isset( $_GET['page'] ) && ( $_GET['page'] == 'ppp-about' || $_GET['page'] == 'ppp-upgrades' ) ) {
+	if (isset($_GET['page']) && ($_GET['page'] == 'ppp-about' || $_GET['page'] == 'ppp-upgrades')) {
 		return; // Don't show notices on the upgrades page
 	}
 
@@ -15,11 +15,11 @@ 
 
 	// Sequential Orders was the first stepped upgrade, so check if we have a stalled upgrade
 	$resume_upgrade = ppp_maybe_resume_upgrade();
-	if ( ! empty( $resume_upgrade ) ) {
+	if ( ! empty($resume_upgrade)) {
 
 		$resume_url = add_query_arg( $resume_upgrade, admin_url( 'index.php' ) );
 		printf(
-			'<div class="error"><p>' . __( 'Post Promoter Pro needs to complete a database upgrade that was previously started, click <a href="%s">here</a> to resume the upgrade.', 'ppp-txt' ) . '</p></div>',
+			'<div class="error"><p>'.__( 'Post Promoter Pro needs to complete a database upgrade that was previously started, click <a href="%s">here</a> to resume the upgrade.', 'ppp-txt' ).'</p></div>',
 			esc_url( $resume_url )
 		);
 
@@ -28,16 +28,16 @@ 
 		// Include all 'Stepped' upgrade process notices in this else statement,
 		// to avoid having a pending, and new upgrade suggested at the same time
 
-		if ( version_compare( $ppp_version, '2.2', '<' ) || ! ppp_has_upgrade_completed( 'upgrade_post_meta' ) ) {
+		if (version_compare( $ppp_version, '2.2', '<' ) || ! ppp_has_upgrade_completed( 'upgrade_post_meta' )) {
 			printf(
-				'<div class="notice notice-info"><p>' . __( 'Post Promoter Pro needs to upgrade share override data, click <a href="%s">here</a> to start the upgrade.', 'ppp-txt' ) . '</p></div>',
+				'<div class="notice notice-info"><p>'.__( 'Post Promoter Pro needs to upgrade share override data, click <a href="%s">here</a> to start the upgrade.', 'ppp-txt' ).'</p></div>',
 				esc_url( admin_url( 'index.php?page=ppp-upgrades&ppp-upgrade=upgrade_post_meta' ) )
 			);
 		}
 
-		if ( version_compare( $ppp_version, '2.3.19', '<' ) || ! ppp_has_upgrade_completed( 'fix_scheduled_shares_2319' ) ) {
+		if (version_compare( $ppp_version, '2.3.19', '<' ) || ! ppp_has_upgrade_completed( 'fix_scheduled_shares_2319' )) {
 			printf(
-				'<div class="notice notice-info"><p>' . __( 'Post Promoter Pro needs to fix an issue with scheduled shares, click <a href="%s">here</a> to start the upgrade.', 'ppp-txt' ) . '</p></div>',
+				'<div class="notice notice-info"><p>'.__( 'Post Promoter Pro needs to fix an issue with scheduled shares, click <a href="%s">here</a> to start the upgrade.', 'ppp-txt' ).'</p></div>',
 				esc_url( admin_url( 'index.php?page=ppp-upgrades&ppp-upgrade=fix_scheduled_shares_2319' ) )
 			);
 		}
@@ -59,17 +59,17 @@ 
 	$upgrades_executed = false;
 
 	// We don't have a version yet, so we need to run the upgrader
-	if ( !$ppp_version && PPP_VERSION == '1.3' ) {
+	if ( ! $ppp_version && PPP_VERSION == '1.3') {
 		ppp_v13_upgrades();
 		$upgrades_executed = true;
 	}
 
-	if ( version_compare( $ppp_version, 2.1, '<' ) ) {
+	if (version_compare( $ppp_version, 2.1, '<' )) {
 		ppp_v21_upgrades();
 		$upgrades_executed = true;
 	}
 
-	if ( $upgrades_executed || version_compare( $ppp_version, PPP_VERSION, '<' ) ) {
+	if ($upgrades_executed || version_compare( $ppp_version, PPP_VERSION, '<' )) {
 		set_transient( '_ppp_activation_redirect', '1', 60 );
 		update_option( 'ppp_version', PPP_VERSION );
 	}
@@ -87,7 +87,7 @@ 
 
 	$doing_upgrade = get_option( 'ppp_doing_upgrade', false );
 
-	if ( empty( $doing_upgrade ) ) {
+	if (empty($doing_upgrade)) {
 		return false;
 	}
 
@@ -103,22 +103,22 @@ 
  */
 function ppp_v13_upgrades() {
 	global $ppp_share_settings;
-	$uq_status = ( isset( $ppp_share_settings['ppp_unique_links'] ) && $ppp_share_settings['ppp_unique_links'] == '1' ) ? $ppp_share_settings['ppp_unique_links'] : 0;
-	$ga_status = ( isset( $ppp_share_settings['ppp_ga_tags'] ) && $ppp_share_settings['ppp_ga_tags'] == '1' ) ? $ppp_share_settings['ppp_ga_tags'] : 0;
+	$uq_status = (isset($ppp_share_settings['ppp_unique_links']) && $ppp_share_settings['ppp_unique_links'] == '1') ? $ppp_share_settings['ppp_unique_links'] : 0;
+	$ga_status = (isset($ppp_share_settings['ppp_ga_tags']) && $ppp_share_settings['ppp_ga_tags'] == '1') ? $ppp_share_settings['ppp_ga_tags'] : 0;
 
-	if ( $uq_status ) {
+	if ($uq_status) {
 		$ppp_share_settings['analytics'] = 'unique_links';
-		unset( $ppp_share_settings['ppp_unique_links'] );
-	} elseif ( $ga_status ) {
+		unset($ppp_share_settings['ppp_unique_links']);
+	} elseif ($ga_status) {
 		$ppp_share_settings['analytics'] = 'google_analytics';
-		unset( $ppp_share_settings['ppp_ga_tags'] );
+		unset($ppp_share_settings['ppp_ga_tags']);
 	}
 
 	update_option( 'ppp_share_settings', $ppp_share_settings );
 
 	global $ppp_options;
 	$ppp_options['default_text'] = '{post_title}';
-	$ppp_options['days'] = array( 'day1' => 'on', 'day2' => 'on', 'day3' => 'on', 'day4' => 'on', 'day5' => 'on', 'day6' => 'on');
+	$ppp_options['days'] = array( 'day1' => 'on', 'day2' => 'on', 'day3' => 'on', 'day4' => 'on', 'day5' => 'on', 'day6' => 'on' );
 
 	update_option( 'ppp_options', $ppp_options );
 }
@@ -140,7 +140,7 @@ 
  */
 function ppp_v22_postmeta_upgrade() {
 
-	if( ! current_user_can( PostPromoterPro::get_manage_capability() ) ) {
+	if ( ! current_user_can( PostPromoterPro::get_manage_capability() )) {
 		wp_die( __( 'You do not have permission to do upgrades', 'ppp-txt' ), __( 'Error', 'ppp-txt' ), array( 'response' => 403 ) );
 	}
 
@@ -149,16 +149,16 @@ 
 	global $wpdb;
 
 
-	$step   = isset( $_GET['step'] ) ? absint( $_GET['step'] ) : 1;
+	$step   = isset($_GET['step']) ? absint( $_GET['step'] ) : 1;
 	$number = 25;
-	$offset = $step == 1 ? 0 : ( $step - 1 ) * $number;
+	$offset = $step == 1 ? 0 : ($step - 1) * $number;
 
-	if ( $step < 2 ) {
+	if ($step < 2) {
 		// Check if we have any payments before moving on
 		$sql = "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_ppp_post_override_data' LIMIT 1";
 		$has_overrides = $wpdb->get_col( $sql );
 
-		if( empty( $has_overrides ) ) {
+		if (empty($has_overrides)) {
 			// We had no payments, just complete
 			update_option( 'ppp_version', preg_replace( '/[^0-9.].*/', '', PPP_VERSION ) );
 			ppp_set_upgrade_complete( 'upgrade_post_meta' );
@@ -167,9 +167,9 @@ 
 		}
 	}
 
-	$total = isset( $_GET['total'] ) ? absint( $_GET['total'] ) : false;
+	$total = isset($_GET['total']) ? absint( $_GET['total'] ) : false;
 
-	if ( empty( $total ) || $total <= 1 ) {
+	if (empty($total) || $total <= 1) {
 		$total_sql = "SELECT COUNT(post_id) as total FROM $wpdb->postmeta WHERE meta_key = '_ppp_post_override_data'";
 		$results   = $wpdb->get_row( $total_sql, 0 );
 
@@ -179,48 +179,48 @@ 
 	$results       = $wpdb->get_results( $wpdb->prepare( "SELECT post_id, meta_value FROM $wpdb->postmeta WHERE meta_key = '_ppp_post_override_data' ORDER BY meta_id DESC LIMIT %d,%d;", $offset, $number ) );
 	$new_post_meta = array();
 
-	if ( $results ) {
-		foreach ( $results as $result ) {
+	if ($results) {
+		foreach ($results as $result) {
 
 			$share_key     = 1;
 
 			$override_data = unserialize( $result->meta_value );
 
-			foreach ( $override_data as $day => $values ) {
+			foreach ($override_data as $day => $values) {
 
-				if ( ! isset( $values['enabled'] ) ) {
+				if ( ! isset($values['enabled'])) {
 					continue;
 				}
 
-				$text = ! empty( $values['text'] ) ? $values['text'] : '';
-				$time = ! empty( $values['time'] ) ? $values['time'] : '8:00am';
+				$text = ! empty($values['text']) ? $values['text'] : '';
+				$time = ! empty($values['time']) ? $values['time'] : '8:00am';
 
 				$post          = get_post( $result->post_id );
 				$days_ahead    = substr( $day, -1 );
-				$date          = date( 'm\/d\/Y', strtotime( $post->post_date . '+' . $days_ahead . ' days' ) );
+				$date          = date( 'm\/d\/Y', strtotime( $post->post_date.'+'.$days_ahead.' days' ) );
 				$image         = '';
 				$attachment_id = '';
 
-				if ( ! empty( $values['use_image'] ) ) {
+				if ( ! empty($values['use_image'])) {
 					$thumb_id  = get_post_thumbnail_id( $result->post_id );
 					$thumb_url = wp_get_attachment_image_src( $thumb_id, 'ppp-tw-share-image', true );
 
-					if ( isset( $thumb_url[0] ) && ! empty( $thumb_url[0] ) && !strpos( $thumb_url[0], 'wp-includes/images/media/default.png' ) ) {
+					if (isset($thumb_url[0]) && ! empty($thumb_url[0]) && ! strpos( $thumb_url[0], 'wp-includes/images/media/default.png' )) {
 						$thumb_url = $thumb_url[0];
 					}
 
-					if ( ! empty( $thumb_id ) && ! empty( $thumb_url ) ) {
+					if ( ! empty($thumb_id) && ! empty($thumb_url)) {
 						$attachment_id = $thumb_id;
 						$image         = $thumb_url;
 					}
 				}
 
-				$new_post_meta[$share_key] = array (
+				$new_post_meta[$share_key] = array(
 					'date'          => $date,
 					'time'          => $time,
 					'text'          => $text,
-					'image'         => ! empty( $image ) ? $image : '',
-					'attachment_id' => ! empty( $attachment_id ) ? $attachment_id : ''
+					'image'         => ! empty($image) ? $image : '',
+					'attachment_id' => ! empty($attachment_id) ? $attachment_id : ''
 				);
 
 				$share_key++;
@@ -255,7 +255,7 @@ 
 
 function ppp_fix_scheduled_shares_2319() {
 
-	if( ! current_user_can( PostPromoterPro::get_manage_capability() ) ) {
+	if ( ! current_user_can( PostPromoterPro::get_manage_capability() )) {
 		wp_die( __( 'You do not have permission to do upgrades', 'ppp-txt' ), __( 'Error', 'ppp-txt' ), array( 'response' => 403 ) );
 	}
 
@@ -264,16 +264,16 @@ 
 	global $wpdb;
 
 
-	$step   = isset( $_GET['step'] ) ? absint( $_GET['step'] ) : 1;
+	$step   = isset($_GET['step']) ? absint( $_GET['step'] ) : 1;
 	$number = 25;
-	$offset = $step == 1 ? 0 : ( $step - 1 ) * $number;
+	$offset = $step == 1 ? 0 : ($step - 1) * $number;
 
-	if ( $step < 2 ) {
+	if ($step < 2) {
 		// Check if we have any payments before moving on
 		$sql = "SELECT DISTINCT post_id FROM $wpdb->postmeta WHERE meta_key IN ( '_ppp_tweets', '_ppp_fb_shares', '_ppp_li_shares' ) LIMIT 1";
 		$has_shares = $wpdb->get_col( $sql );
 
-		if( empty( $has_shares ) ) {
+		if (empty($has_shares)) {
 			// We had no payments, just complete
 			update_option( 'ppp_version', preg_replace( '/[^0-9.].*/', '', PPP_VERSION ) );
 			ppp_set_upgrade_complete( 'fix_scheduled_shares_2319' );
@@ -282,9 +282,9 @@ 
 		}
 	}
 
-	$total = isset( $_GET['total'] ) ? absint( $_GET['total'] ) : false;
+	$total = isset($_GET['total']) ? absint( $_GET['total'] ) : false;
 
-	if ( empty( $total ) || $total <= 1 ) {
+	if (empty($total) || $total <= 1) {
 		$total_sql = "SELECT COUNT( DISTINCT post_id ) as total FROM $wpdb->postmeta WHERE meta_key IN ( '_ppp_tweets', '_ppp_fb_shares', '_ppp_li_shares' )";
 		$results   = $wpdb->get_row( $total_sql, 0 );
 
@@ -293,19 +293,19 @@ 
 
 	$results = $wpdb->get_results( $wpdb->prepare( "SELECT DISTINCT post_id FROM $wpdb->postmeta WHERE meta_key IN ( '_ppp_tweets', '_ppp_fb_shares', '_ppp_li_shares' ) ORDER BY post_id DESC LIMIT %d,%d;", $offset, $number ) );
 
-	if ( $results ) {
+	if ($results) {
 		$allowed_post_types = ppp_allowed_post_types();
 
-		foreach ( $results as $result ) {
+		foreach ($results as $result) {
 			$post = get_post( $result->post_id );
-			if ( ! in_array( $post->post_type, $allowed_post_types ) ) {
+			if ( ! in_array( $post->post_type, $allowed_post_types )) {
 				continue;
 			}
 
 			ppp_remove_scheduled_shares( $result->post_id );
 			$timestamps = ppp_get_timestamps( $result->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( $result->post_id, $name ) );
 			}

includes/libs/facebook.php

Spacing +58 added lines, -58 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,13 +11,13 @@ 
  * Handles all facebook functions
  *
  */
-if( !class_exists( 'PPP_Facebook' ) ) {
+if ( ! class_exists( 'PPP_Facebook' )) {
 
 	class PPP_Facebook {
 
 		var $facebook;
 
-		public function __construct(){
+		public function __construct() {
 			ppp_maybe_start_session();
 		}
 
@@ -30,8 +30,8 @@ 
 		public function ppp_load_facebook() {
 
 
-			if( !class_exists( 'Facebook' ) ) {
-				require_once ( PPP_PATH . '/includes/libs/facebook/facebook.php' );
+			if ( ! class_exists( 'Facebook' )) {
+				require_once (PPP_PATH.'/includes/libs/facebook/facebook.php');
 			}
 
 			ppp_set_social_tokens();
@@ -40,7 +40,7 @@ 
 					'appId' => PPP_FB_APP_ID,
 					'secret' => PPP_FB_APP_SECRET,
 					'cookie' => true
-			));
+			) );
 
 			return true;
 
@@ -55,70 +55,70 @@ 
 			$facebook = $this->ppp_load_facebook();
 
 			//when user is going to logged in and verified successfully session will create
-			if ( isset( $_REQUEST['fb_access_token'] ) && isset( $_REQUEST['expires_in'] ) ) {
+			if (isset($_REQUEST['fb_access_token']) && isset($_REQUEST['expires_in'])) {
 
 				$access_token = $_REQUEST['fb_access_token'];
 				$expires_in   = $_REQUEST['expires_in'];
 
-			} elseif ( isset( $_GET['state'] ) && strpos( $_GET['state'], 'ppp-local-keys-fb' ) !== false ) {
-				$access_code = isset( $_GET['code'] ) ? $_GET['code'] : false;
+			} elseif (isset($_GET['state']) && strpos( $_GET['state'], 'ppp-local-keys-fb' ) !== false) {
+				$access_code = isset($_GET['code']) ? $_GET['code'] : false;
 
-				if ( empty( $access_code ) ) {
+				if (empty($access_code)) {
 					return;
 				}
 
-				$params  = '?client_id=' . PPP_FB_APP_ID;
-				$params .= '&client_secret=' . PPP_FB_APP_SECRET;
-				$params .= '&code=' . $access_code;
-				$params .= '&redirect_uri=' . admin_url( 'admin.php?page=ppp-social-settings' );
-				$url = 'https://graph.facebook.com/oauth/access_token' . $params;
+				$params  = '?client_id='.PPP_FB_APP_ID;
+				$params .= '&client_secret='.PPP_FB_APP_SECRET;
+				$params .= '&code='.$access_code;
+				$params .= '&redirect_uri='.admin_url( 'admin.php?page=ppp-social-settings' );
+				$url = 'https://graph.facebook.com/oauth/access_token'.$params;
 
 				parse_str( wp_remote_retrieve_body( wp_remote_post( $url ) ), $result );
 
-				$access_token = ! empty( $result['access_token'] ) ? $result['access_token'] : false;
-				$expires_in   = ! empty( $result['expires'] ) ? $result['expires'] : false;
+				$access_token = ! empty($result['access_token']) ? $result['access_token'] : false;
+				$expires_in   = ! empty($result['expires']) ? $result['expires'] : false;
 
 			}
 
-			if ( ! empty( $access_token ) ) {
+			if ( ! empty($access_token)) {
 				global $ppp_social_settings;
 				$ppp_social_settings = get_option( 'ppp_social_settings' );
 
 				//check facebook class is loaded or not
-				if( !$facebook ) {
+				if ( ! $facebook) {
 					return false;
 				}
 
 				$data = new stdClass();
 				$data->access_token = $access_token;
 
-				$expires_in = empty( $expires_in ) ? 60 * 24 * 60 * 60 : $expires_in; // days * hours * minutes * seconds
+				$expires_in = empty($expires_in) ? 60 * 24 * 60 * 60 : $expires_in; // days * hours * minutes * seconds
 				$data->expires_on = current_time( 'timestamp' ) + $expires_in;
 
-				update_option( '_ppp_facebook_refresh', current_time( 'timestamp' ) + round( $expires_in/1.25 ) );
+				update_option( '_ppp_facebook_refresh', current_time( 'timestamp' ) + round( $expires_in / 1.25 ) );
 
 				// Now that we have a valid auth, get some user info
 				$user_info = $this->ppp_get_fb_user( $data->access_token );
 
-				if ( $user_info ) {
-					if ( !empty( $user_info->name ) ) {
+				if ($user_info) {
+					if ( ! empty($user_info->name)) {
 						$data->name = $user_info->name;
 					} else {
-						$parsed_name = $user_info->first_name . ' ' . $user_info->last_name;
+						$parsed_name = $user_info->first_name.' '.$user_info->last_name;
 						$data->name = $parsed_name;
 					}
 					$data->userid = $user_info->id;
 					$data->avatar = $this->ppp_fb_get_profile_picture( array( 'type' => 'square' ), $data->userid );
 
-					if ( ! empty( $ppp_social_settings['facebook']->page ) ) {
+					if ( ! empty($ppp_social_settings['facebook']->page)) {
 						$current_page = $ppp_social_settings['facebook']->page;
 						$page_parts   = explode( '|', $current_page );
 
 						$pages = $this->ppp_get_fb_user_pages( $data->access_token );
 
-						foreach ( $pages as $page ) {
-							if ( $page->id == $page_parts[2] ) {
-								$data->page = $page->name . '|' . $page->access_token . '|' . $page->id;
+						foreach ($pages as $page) {
+							if ($page->id == $page_parts[2]) {
+								$data->page = $page->name.'|'.$page->access_token.'|'.$page->id;
 								continue;
 							}
 						}
@@ -131,7 +131,7 @@ 
 				}
 
 
-				$url = remove_query_arg( array( 'fb_access_token' , 'expires_in' ) );
+				$url = remove_query_arg( array( 'fb_access_token', 'expires_in' ) );
 				wp_redirect( $url );
 				die();
 			}
@@ -149,12 +149,12 @@ 
 			$facebook = $this->ppp_load_facebook();
 
 			//check facebook class is exis or not
-			if( !$facebook ) {
+			if ( ! $facebook) {
 				return false;
 			}
 
 			global $ppp_social_settings;
-			$user = json_decode( wp_remote_retrieve_body( wp_remote_get( 'https://graph.facebook.com/me?access_token=' . $access_token ) ) );
+			$user = json_decode( wp_remote_retrieve_body( wp_remote_get( 'https://graph.facebook.com/me?access_token='.$access_token ) ) );
 
 			return $user;
 
@@ -166,7 +166,7 @@ 
 			$facebook = $this->ppp_load_facebook();
 
 			// check facebook cleast is exists or not
-			if( !$facebook ) {
+			if ( ! $facebook) {
 				return false;
 			}
 
@@ -174,20 +174,20 @@ 
 			$facebook_settings = $ppp_social_settings['facebook'];
 
 
-			if ( ! isset( $facebook_settings->available_pages ) ||
-				 ! isset( $facebook_settings->pages_last_updated ) ||
-				 $facebook_settings->pages_last_updated < current_time( 'timestamp' ) ) {
+			if ( ! isset($facebook_settings->available_pages) ||
+				 ! isset($facebook_settings->pages_last_updated) ||
+				 $facebook_settings->pages_last_updated < current_time( 'timestamp' )) {
 
-				$all_pages = json_decode( wp_remote_retrieve_body( wp_remote_get( 'https://graph.facebook.com/me/accounts?access_token=' . $access_token ) ) );
+				$all_pages = json_decode( wp_remote_retrieve_body( wp_remote_get( 'https://graph.facebook.com/me/accounts?access_token='.$access_token ) ) );
 				$pages = array();
 
-				if ( !empty( $all_pages ) ) {
-					foreach ( $all_pages->data as $page ) {
-						if ( in_array( 'CREATE_CONTENT', $page->tasks ) ) {
+				if ( ! empty($all_pages)) {
+					foreach ($all_pages->data as $page) {
+						if (in_array( 'CREATE_CONTENT', $page->tasks )) {
 							$pages[] = $page;
 
-							if ( ! empty( $ppp_social_settings['facebook']->page ) && strpos( $ppp_social_settings['facebook']->page, $page->id ) ) {
-								$ppp_social_settings['facebook']->page = $page->name . '|' . $page->access_token . '|' . $page->id;
+							if ( ! empty($ppp_social_settings['facebook']->page) && strpos( $ppp_social_settings['facebook']->page, $page->id )) {
+								$ppp_social_settings['facebook']->page = $page->name.'|'.$page->access_token.'|'.$page->id;
 							}
 						}
 					}
@@ -198,7 +198,7 @@ 
 
 				$pages = (object) $pages;
 				$ppp_social_settings['facebook']->available_pages = $pages;
-				$ppp_social_settings['facebook']->pages_last_updated = current_time( 'timestamp' ) + ( HOUR_IN_SECONDS / 4 );
+				$ppp_social_settings['facebook']->pages_last_updated = current_time( 'timestamp' ) + (HOUR_IN_SECONDS / 4);
 				update_option( 'ppp_social_settings', $ppp_social_settings );
 			} else {
 				$pages = $facebook_settings->available_pages;
@@ -220,7 +220,7 @@ 
 			$facebook = $this->ppp_load_facebook();
 
 			//check facebook class is exis or not
-			if( !$facebook ) {
+			if ( ! $facebook) {
 				return false;
 			}
 
@@ -231,27 +231,27 @@ 
 		 * Get auth url for facebook
 		 *
 		 */
-		public function ppp_get_facebook_auth_url ( $return_url ) {
+		public function ppp_get_facebook_auth_url( $return_url ) {
 
 			//load facebook class
 			$facebook = $this->ppp_load_facebook();
 
 			//check facebook class is exis or not
-			if( !$facebook ) {
+			if ( ! $facebook) {
 				return false;
 			}
 
-			if ( ! PPP_LOCAL_TOKENS ) {
+			if ( ! PPP_LOCAL_TOKENS) {
 				$base_url = 'https://postpromoterpro.com/?ppp-social-auth';
-				$url  = $base_url . '&ppp-service=fb&ppp-license-key=' . trim( get_option( '_ppp_license_key' ) );
+				$url  = $base_url.'&ppp-service=fb&ppp-license-key='.trim( get_option( '_ppp_license_key' ) );
 				$url .= '&nocache';
-				$url .= '&return_url=' . esc_url( $return_url );
+				$url .= '&return_url='.esc_url( $return_url );
 			} else {
 				$url  = 'https://graph.facebook.com/oauth/authorize?';
-				$url .= 'client_id=' . PPP_FB_APP_ID;
+				$url .= 'client_id='.PPP_FB_APP_ID;
 				$url .= '&scope=public_profile,publish_actions,manage_pages,publish_pages';
 				$url .= '&state=ppp-local-keys-fb';
-				$url .= '&redirect_uri=' . esc_url( $return_url ) . '&nocache';
+				$url .= '&redirect_uri='.esc_url( $return_url ).'&nocache';
 			}
 
 			return $url;
@@ -263,14 +263,14 @@ 
 		 * Getting the the profile image of the connected Facebook user.
 		 *
 		 */
-		public function ppp_fb_get_profile_picture( $args=array(), $user ) {
+		public function ppp_fb_get_profile_picture( $args = array(), $user ) {
 
-			if( isset( $args['type'] ) && !empty( $args['type'] ) ) {
+			if (isset($args['type']) && ! empty($args['type'])) {
 				$type = $args['type'];
 			} else {
 				$type = 'large';
 			}
-			$url = 'https://graph.facebook.com/' . $user . '/picture?type=' . $type;
+			$url = 'https://graph.facebook.com/'.$user.'/picture?type='.$type;
 			return $url;
 		}
 
@@ -278,7 +278,7 @@ 
 			global $ppp_social_settings;
 			$facebook_settings = $ppp_social_settings['facebook'];
 
-			if ( !isset( $facebook_settings->page ) || strtolower( $facebook_settings->page ) === 'me' ) {
+			if ( ! isset($facebook_settings->page) || strtolower( $facebook_settings->page ) === 'me') {
 				$account      = 'me';
 				$access_token = $facebook_settings->access_token;
 			} else {
@@ -287,9 +287,9 @@ 
 				$access_token = $page_info[1];
 			}
 
-			$url = 'https://graph.facebook.com/' . $account . '/feed?access_token=' . $access_token;
+			$url = 'https://graph.facebook.com/'.$account.'/feed?access_token='.$access_token;
 			$args = array( 'link' => $link, 'message' => $message );
-			if ( !empty( $image ) ) {
+			if ( ! empty($image)) {
 				$args['picture'] = $image;
 			}
 			$results = json_decode( wp_remote_retrieve_body( wp_remote_post( $url, array( 'body' => $args ) ) ) );
@@ -308,7 +308,7 @@ 
 		public function ppp_get_data_from_url( $url ) {
 
 			//Use wp_remote_post and wp_remote_get
-			$data	= wp_remote_retrieve_body( wp_remote_get( $url ) );
+			$data = wp_remote_retrieve_body( wp_remote_get( $url ) );
 
 			return $data;
 		}
@@ -320,7 +320,7 @@ 
 		 */
 		public function clear_og_cache( $post_id ) {
 			$post_url = get_permalink( $post_id );
-			if ( ! empty( $post_url ) ) {
+			if ( ! empty($post_url)) {
 				$args     = array( 'body' => array( 'id' => $post_url, 'scrape' => true ) );
 				$response = json_decode( wp_remote_retrieve_body( wp_remote_post( 'https://graph.facebook.com/', $args ) ) );
 			}

includes/admin/admin-ajax.php

Spacing +7 added lines, -7 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -10,7 +10,7 @@ 
 	$date = sanitize_text_field( $_POST['date'] );
 	$time = sanitize_text_field( $_POST['time'] );
 
-	$offset = (int) -( get_option( 'gmt_offset' ) ); // Make the timestamp in the users' timezone, b/c that makes more sense
+	$offset = (int) -(get_option( 'gmt_offset' )); // Make the timestamp in the users' timezone, b/c that makes more sense
 
 	$share_time = explode( ':', $time );
 
@@ -18,11 +18,11 @@ 
 	$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;
 	}
 
@@ -38,15 +38,15 @@ 
 }
 add_action( 'wp_ajax_ppp_has_schedule_conflict', 'ppp_check_for_schedule_conflict' );
 
-add_action( 'wp_ajax_ppp_local_url_notice_dismiss', function() {
+add_action( 'wp_ajax_ppp_local_url_notice_dismiss', function () {
 	$error_message = __( 'There was an error dismissing the local URL notice. Please try again.', 'ppp-txt' );
-	if( ! wp_verify_nonce( $_POST['nonce'], 'ppp_local_url_notice_nonce' ) || ! current_user_can( post_promoter_pro()->get_manage_capability() ) ) {
+	if ( ! wp_verify_nonce( $_POST['nonce'], 'ppp_local_url_notice_nonce' ) || ! current_user_can( post_promoter_pro()->get_manage_capability() )) {
 		wp_send_json_error( array(
 			'error' => $error_message,
 		) );
 	}
 	$updated = update_option( 'ppp_local_url_notice_dismissed', true );
-	if( $updated === false ) {
+	if ($updated === false) {
 		wp_send_json_error( array(
 			'error' => $error_message,
 		) );

includes/general-functions.php

Spacing +40 added lines, -40 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -10,12 +10,12 @@ 
  * @return void
  */
 function ppp_maybe_start_session() {
-	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');
 	}
 
 	$ret = false;
-	if ( ( is_admin() || ( defined( 'DOING_CRON' ) && DOING_CRON ) ) && session_status() == PHP_SESSION_NONE && ! headers_sent() ) {
+	if ((is_admin() || (defined( 'DOING_CRON' ) && DOING_CRON)) && session_status() == PHP_SESSION_NONE && ! headers_sent()) {
 		$ret = session_start();
 	}
 
@@ -30,8 +30,8 @@ 
 	global $ppp_share_settings;
 	$result = false;
 
-	if ( isset( $ppp_share_settings['analytics'] ) && $ppp_share_settings['analytics'] !== 'none' ) {
-		$result =  true;
+	if (isset($ppp_share_settings['analytics']) && $ppp_share_settings['analytics'] !== 'none') {
+		$result = true;
 	}
 
 	return apply_filters( 'ppp_is_link_tracking_enabled', $result );
@@ -72,7 +72,7 @@ 
 function ppp_is_shortener_enabled() {
 	global $ppp_share_settings;
 
-	return ( isset( $ppp_share_settings['shortener'] ) && !empty( $ppp_share_settings['shortener'] ) && $ppp_share_settings != '-1' );
+	return (isset($ppp_share_settings['shortener']) && ! empty($ppp_share_settings['shortener']) && $ppp_share_settings != '-1');
 }
 
 /**
@@ -104,9 +104,9 @@ 
 	$post_types = get_post_types( $post_type_args, NULL, 'and' );
 
 	$unsupported_post_types = array( 'wp_log', 'attachment' );
-	foreach ( $unsupported_post_types as $unsupported_post_type ) {
-		if ( array_key_exists( $unsupported_post_type, $post_types ) ) {
-			unset( $post_types[ $unsupported_post_type ] );
+	foreach ($unsupported_post_types as $unsupported_post_type) {
+		if (array_key_exists( $unsupported_post_type, $post_types )) {
+			unset($post_types[$unsupported_post_type]);
 		}
 	}
 
@@ -134,16 +134,16 @@ 
 function ppp_should_save( $post_id, $post ) {
 	$ret = true;
 
-	if ( empty( $_POST ) ) {
+	if (empty($_POST)) {
 		$ret = false;
 	}
 
-	if ( wp_is_post_revision( $post_id ) ) {
+	if (wp_is_post_revision( $post_id )) {
 		$ret = false;
 	}
 
 	global $ppp_options;
-	if ( !isset( $ppp_options['post_types'] ) || !is_array( $ppp_options['post_types'] ) || !array_key_exists( $post->post_type, $ppp_options['post_types'] ) ) {
+	if ( ! isset($ppp_options['post_types']) || ! is_array( $ppp_options['post_types'] ) || ! array_key_exists( $post->post_type, $ppp_options['post_types'] )) {
 		$ret = false;
 	}
 
@@ -159,7 +159,7 @@ 
 function ppp_set_uploads_dir() {
 	$upload_path = ppp_get_upload_path();
 
-	if ( false === get_transient( 'ppp_check_protection_files' ) ) {
+	if (false === get_transient( 'ppp_check_protection_files' )) {
 
 		// Make sure the /ppp folder is created
 		wp_mkdir_p( $upload_path );
@@ -168,22 +168,22 @@ 
 		$rules  = "Options -Indexes\n";
 		$rules .= "deny from all\n";
 
-		$htaccess_exists = file_exists( $upload_path . '/.htaccess' );
+		$htaccess_exists = file_exists( $upload_path.'/.htaccess' );
 
-		if ( $htaccess_exists ) {
-			$contents = @file_get_contents( $upload_path . '/.htaccess' );
-			if ( $contents !== $rules || ! $contents ) {
+		if ($htaccess_exists) {
+			$contents = @file_get_contents( $upload_path.'/.htaccess' );
+			if ($contents !== $rules || ! $contents) {
 				// Update the .htaccess rules if they don't match
-				@file_put_contents( $upload_path . '/.htaccess', $rules );
+				@file_put_contents( $upload_path.'/.htaccess', $rules );
 			}
-		} elseif( wp_is_writable( $upload_path ) ) {
+		} elseif (wp_is_writable( $upload_path )) {
 			// Create the file if it doesn't exist
-			@file_put_contents( $upload_path . '/.htaccess', $rules );
+			@file_put_contents( $upload_path.'/.htaccess', $rules );
 		}
 
 		// Top level blank index.php
-		if ( ! file_exists( $upload_path . '/index.php' ) && wp_is_writable( $upload_path ) ) {
-			@file_put_contents( $upload_path . '/index.php', '<?php' . PHP_EOL . '// Silence is golden.' );
+		if ( ! file_exists( $upload_path.'/index.php' ) && wp_is_writable( $upload_path )) {
+			@file_put_contents( $upload_path.'/index.php', '<?php'.PHP_EOL.'// Silence is golden.' );
 		}
 
 		// Check for the files once per day
@@ -200,7 +200,7 @@ 
  */
 function ppp_get_upload_path() {
 	$wp_upload_dir = wp_upload_dir();
-	return $wp_upload_dir['basedir'] . '/ppp';
+	return $wp_upload_dir['basedir'].'/ppp';
 }
 
 /**
@@ -213,7 +213,7 @@ 
 
 	$completed_upgrades = get_option( 'ppp_completed_upgrades' );
 
-	if ( false === $completed_upgrades ) {
+	if (false === $completed_upgrades) {
 		$completed_upgrades = array();
 	}
 
@@ -230,7 +230,7 @@ 
  */
 function ppp_has_upgrade_completed( $upgrade_action = '' ) {
 
-	if ( empty( $upgrade_action ) ) {
+	if (empty($upgrade_action)) {
 		return false;
 	}
 
@@ -249,7 +249,7 @@ 
  */
 function ppp_set_upgrade_complete( $upgrade_action = '' ) {
 
-	if ( empty( $upgrade_action ) ) {
+	if (empty($upgrade_action)) {
 		return false;
 	}
 
@@ -274,48 +274,48 @@ 
 	$url = strtolower( trim( get_home_url( '/' ) ) );
 
 	// Need to get the host...so let's add the scheme so we can use parse_url
-	if ( false === strpos( $url, 'http://' ) && false === strpos( $url, 'https://' ) ) {
-		$url = 'http://' . $url;
+	if (false === strpos( $url, 'http://' ) && false === strpos( $url, 'https://' )) {
+		$url = 'http://'.$url;
 	}
 
 	$url_parts = parse_url( $url );
-	$host      = ! empty( $url_parts['host'] ) ? $url_parts['host'] : false;
+	$host      = ! empty($url_parts['host']) ? $url_parts['host'] : false;
 
-	if ( ! empty( $url ) && ! empty( $host ) ) {
+	if ( ! empty($url) && ! empty($host)) {
 
-		if ( false !== ip2long( $host ) ) {
-			if ( ! filter_var( $host, FILTER_VALIDATE_IP, FILTER_FLAG_NO_PRIV_RANGE | FILTER_FLAG_NO_RES_RANGE ) ) {
+		if (false !== ip2long( $host )) {
+			if ( ! filter_var( $host, FILTER_VALIDATE_IP, FILTER_FLAG_NO_PRIV_RANGE | FILTER_FLAG_NO_RES_RANGE )) {
 				$is_local_url = true;
 			}
-		} else if ( 'localhost' === $host ) {
+		} else if ('localhost' === $host) {
 			$is_local_url = true;
 		}
 
 		$check_tlds = apply_filters( 'ppp_validate_tlds', true );
-		if ( $check_tlds ) {
+		if ($check_tlds) {
 			$tlds_to_check = apply_filters( 'ppp_url_tlds', array(
 				'.dev', '.local', '.test'
 			) );
 
-			foreach ( $tlds_to_check as $tld ) {
-				if ( false !== strpos( $host, $tld ) ) {
+			foreach ($tlds_to_check as $tld) {
+				if (false !== strpos( $host, $tld )) {
 					$is_local_url = true;
 					continue;
 				}
 			}
 		}
 
-		if ( substr_count( $host, '.' ) > 1 ) {
+		if (substr_count( $host, '.' ) > 1) {
 			$subdomains_to_check = apply_filters( 'ppp_url_subdomains', array(
 				'dev.', '*.staging.',
 			) );
 
-			foreach ( $subdomains_to_check as $subdomain ) {
+			foreach ($subdomains_to_check as $subdomain) {
 
 				$subdomain = str_replace( '.', '(.)', $subdomain );
 				$subdomain = str_replace( array( '*', '(.)' ), '(.*)', $subdomain );
 
-				if ( preg_match( '/^(' . $subdomain . ')/', $host ) ) {
+				if (preg_match( '/^('.$subdomain.')/', $host )) {
 					$is_local_url = true;
 					continue;
 				}

includes/linkedin-functions.php

Spacing +114 added lines, -114 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -12,7 +12,7 @@ 
 function ppp_linkedin_enabled() {
 	global $ppp_social_settings;
 
-	if ( isset( $ppp_social_settings['linkedin'] ) && !empty( $ppp_social_settings['linkedin'] ) ) {
+	if (isset($ppp_social_settings['linkedin']) && ! empty($ppp_social_settings['linkedin'])) {
 		return true;
 	}
 
@@ -60,9 +60,9 @@ 
  */
 function ppp_li_account_list_name( $string = '' ) {
 
-	if ( ppp_linkedin_enabled() ) {
+	if (ppp_linkedin_enabled()) {
 		global $ppp_social_settings;
-		$string .= $ppp_social_settings['linkedin']->firstName . ' ' . $ppp_social_settings['linkedin']->lastName;
+		$string .= $ppp_social_settings['linkedin']->firstName.' '.$ppp_social_settings['linkedin']->lastName;
 	}
 
 	return $string;
@@ -77,22 +77,22 @@ 
 function ppp_li_account_list_actions( $string = '' ) {
 	global $ppp_linkedin_oauth, $ppp_social_settings;
 
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		$li_authurl = $ppp_linkedin_oauth->ppp_get_linkedin_auth_url( admin_url( 'admin.php?page=ppp-social-settings' ) );
 
-		$string .= '<a class="button-primary" href="' . $li_authurl . '">' . __( 'Connect to Linkedin', 'ppp-txt' ) . '</a>';
+		$string .= '<a class="button-primary" href="'.$li_authurl.'">'.__( 'Connect to Linkedin', 'ppp-txt' ).'</a>';
 	} else {
-		$string  .= '<a class="button-primary" href="' . admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=linkedin' ) . '" >' . __( 'Disconnect from Linkedin', 'ppp-txt' ) . '</a>&nbsp;';
+		$string .= '<a class="button-primary" href="'.admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=linkedin' ).'" >'.__( 'Disconnect from Linkedin', 'ppp-txt' ).'</a>&nbsp;';
 
 		$refresh_date = (int) get_option( '_ppp_linkedin_refresh', true );
 
 
-		if ( defined( 'LINKEDIN_KEY' ) && current_time( 'timestamp' ) > $refresh_date ) {
+		if (defined( 'LINKEDIN_KEY' ) && current_time( 'timestamp' ) > $refresh_date) {
 			$token       = $ppp_social_settings['linkedin']->access_token;
 			$url         = $ppp_linkedin_oauth->ppp_get_linkedin_auth_url( admin_url( 'admin.php?page=ppp-social-settings' ) );
-			$refresh_url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token=' . $token, $url );
+			$refresh_url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token='.$token, $url );
 
-			$string  .= '<a class="button-secondary" href="' . $refresh_url . '" >' . __( 'Re-Authorize Linkedin', 'ppp-txt' ) . '</a>&nbsp;';
+			$string .= '<a class="button-secondary" href="'.$refresh_url.'" >'.__( 'Re-Authorize Linkedin', 'ppp-txt' ).'</a>&nbsp;';
 		}
 	}
 
@@ -106,14 +106,14 @@ 
  * @return string         The HTML for the LinkedIn Extras column
  */
 function ppp_li_account_list_extras( $string ) {
-	if ( ppp_linkedin_enabled() ) {
+	if (ppp_linkedin_enabled()) {
 		global $ppp_social_settings, $ppp_options;
-		if ( ! empty( $ppp_options['enable_debug'] ) ) {
-			$days_left  = absint( round( ( $ppp_social_settings['linkedin']->expires_on - current_time( 'timestamp' ) ) / DAY_IN_SECONDS ) );
-			$refresh_in = absint( round( ( get_option( '_ppp_linkedin_refresh' ) - current_time( 'timestamp' ) ) / DAY_IN_SECONDS ) );
+		if ( ! empty($ppp_options['enable_debug'])) {
+			$days_left  = absint( round( ($ppp_social_settings['linkedin']->expires_on - current_time( 'timestamp' )) / DAY_IN_SECONDS ) );
+			$refresh_in = absint( round( (get_option( '_ppp_linkedin_refresh' ) - current_time( 'timestamp' )) / DAY_IN_SECONDS ) );
 
-			$string .= '<br />' . sprintf( __( 'Token expires in %s days' , 'ppp-txt' ), $days_left );
-			$string .= '<br />' . sprintf( __( 'Refresh notice in %s days', 'ppp-txt' ), $refresh_in );
+			$string .= '<br />'.sprintf( __( 'Token expires in %s days', 'ppp-txt' ), $days_left );
+			$string .= '<br />'.sprintf( __( 'Refresh notice in %s days', 'ppp-txt' ), $refresh_in );
 		}
 	}
 
@@ -129,17 +129,17 @@ 
 function ppp_capture_linkedin_oauth() {
 	$should_capture = false;
 
-	if ( isset( $_GET['state'] ) && strpos( $_GET['state'], 'ppp-local-keys-li' ) !== false ) {
+	if (isset($_GET['state']) && strpos( $_GET['state'], 'ppp-local-keys-li' ) !== false) {
 		// Local config
 		$should_capture = true;
 	}
 
-	if ( isset( $_REQUEST['li_access_token'] ) ) {
+	if (isset($_REQUEST['li_access_token'])) {
 		// Returning from remote config
 		$should_capture = true;
 	}
 
-	if ( $should_capture && ( isset( $_REQUEST['page'] ) && $_REQUEST['page'] == 'ppp-social-settings' ) ) {
+	if ($should_capture && (isset($_REQUEST['page']) && $_REQUEST['page'] == 'ppp-social-settings')) {
 		global $ppp_linkedin_oauth;
 		$ppp_linkedin_oauth->ppp_initialize_linkedin();
 		wp_redirect( admin_url( 'admin.php?page=ppp-social-settings' ) );
@@ -155,8 +155,8 @@ 
 function ppp_disconnect_linkedin() {
 	global $ppp_social_settings;
 	$ppp_social_settings = get_option( 'ppp_social_settings' );
-	if ( isset( $ppp_social_settings['linkedin'] ) ) {
-		unset( $ppp_social_settings['linkedin'] );
+	if (isset($ppp_social_settings['linkedin'])) {
+		unset($ppp_social_settings['linkedin']);
 		update_option( 'ppp_social_settings', $ppp_social_settings );
 		delete_option( '_ppp_linkedin_refresh' );
 	}
@@ -182,13 +182,13 @@ 
  */
 function ppp_li_execute_refresh() {
 
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return;
 	}
 
 	$refresh_date = (int) get_option( '_ppp_linkedin_refresh', true );
 
-	if ( ( empty( $_GET['page' ] ) || $_GET['page'] !== 'ppp-social-settings' ) && current_time( 'timestamp' ) > $refresh_date ) {
+	if ((empty($_GET['page']) || $_GET['page'] !== 'ppp-social-settings') && current_time( 'timestamp' ) > $refresh_date) {
 		add_action( 'admin_notices', 'ppp_linkedin_refresh_notice' );
 	}
 }
@@ -200,12 +200,12 @@ 
  */
 function ppp_linkedin_refresh_notice() {
 
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return;
 	}
 
-	$has_dismissed = get_transient( 'ppp-dismiss-refresh-li' . get_current_user_id() );
-	if ( false !== $has_dismissed ) {
+	$has_dismissed = get_transient( 'ppp-dismiss-refresh-li'.get_current_user_id() );
+	if (false !== $has_dismissed) {
 		return;
 	}
 
@@ -216,14 +216,14 @@ 
 
 	$token = $ppp_social_settings['linkedin']->access_token;
 	$url = $ppp_linkedin_oauth->ppp_get_linkedin_auth_url( admin_url( 'admin.php?page=ppp-social-settings' ) );
-	$url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token=' . $token, $url );
+	$url = str_replace( '?ppp-social-auth', '?ppp-social-auth&ppp-refresh=true&access_token='.$token, $url );
 
-	$days_left = (int) round( ( $ppp_social_settings['linkedin']->expires_on - current_time( 'timestamp' ) ) / DAY_IN_SECONDS );
+	$days_left = (int) round( ($ppp_social_settings['linkedin']->expires_on - current_time( 'timestamp' )) / DAY_IN_SECONDS );
 	?>
 	<div class="notice notice-warning is-dismissible" data-service="li">
-		<?php if ( $days_left > 0 ): ?>
+		<?php if ($days_left > 0): ?>
 			<p><strong>Post Promoter Pro: </strong><?php printf( __( 'Your LinkedIn authentication expires in %d days. Please <a href="%s">refresh access</a>.', 'ppp-txt' ), $days_left, $url ); ?></p>
-		<?php elseif ( $days_left < 1 ): ?>
+		<?php elseif ($days_left < 1): ?>
 			<p><strong>Post Promoter Pro: </strong><?php printf( __( 'Your LinkedIn authentication has expired. Please <a href="%s">refresh access</a>.', 'ppp-txt' ), $url ); ?></p>
 		<?php endif; ?>
 	</div>
@@ -238,10 +238,10 @@ 
  */
 function ppp_li_dismiss_notice() {
 
-	$nag = sanitize_key( $_POST[ 'nag' ] );
+	$nag = sanitize_key( $_POST['nag'] );
 
-	if ( $nag === $_POST[ 'nag' ] ) {
-		set_transient( $nag . get_current_user_id(), true, DAY_IN_SECONDS );
+	if ($nag === $_POST['nag']) {
+		set_transient( $nag.get_current_user_id(), true, DAY_IN_SECONDS );
 	}
 
 
@@ -254,7 +254,7 @@ 
  * @return void
  */
 function ppp_set_li_token_constants( $social_tokens ) {
-	if ( !empty( $social_tokens ) && property_exists( $social_tokens, 'linkedin' ) ) {
+	if ( ! empty($social_tokens) && property_exists( $social_tokens, 'linkedin' )) {
 		define( 'LINKEDIN_KEY', $social_tokens->linkedin->api_key );
 		define( 'LINKEDIN_SECRET', $social_tokens->linkedin->secret_key );
 	}
@@ -271,7 +271,7 @@ 
  */
 function ppp_li_share( $title, $description, $link, $media ) {
 	global $ppp_linkedin_oauth;
-	$args = array (
+	$args = array(
 		'title' => ppp_entities_and_slashes( $title ),
 		'description' => ppp_entities_and_slashes( $description ),
 		'submitted-url' => $link,
@@ -312,7 +312,7 @@ 
  */
 function ppp_li_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return $tabs;
 	}
 
@@ -329,7 +329,7 @@ 
  */
 function ppp_li_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return $content;
 	}
 
@@ -367,7 +367,7 @@ 
  */
 function ppp_li_add_metabox_content( $post ) {
 	global $ppp_options, $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_li_share_on_publish               = get_post_meta( $post->ID, '_ppp_li_share_on_publish', true );
 	$ppp_share_on_publish_title            = get_post_meta( $post->ID, '_ppp_li_share_on_publish_title', true );
@@ -377,9 +377,9 @@ 
 
 	$show_share_on_publish = false;
 
-	$share_by_default      = empty( $ppp_share_settings['share_on_publish'][ $post->post_type ]['linkedin'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['share_on_publish'][$post->post_type]['linkedin']) ? false : true;
 
-	if ( $ppp_li_share_on_publish == '1' || ( $ppp_li_share_on_publish == '' && $share_by_default ) ) {
+	if ($ppp_li_share_on_publish == '1' || ($ppp_li_share_on_publish == '' && $share_by_default)) {
 		$show_share_on_publish = true;
 	}
 	?>
@@ -387,11 +387,11 @@ 
 		<div class="ppp-post-override-wrap">
 			<p><h3><?php _e( 'Share on LinkedIn', 'ppp-txt' ); ?></h3></p>
 			<p>
-				<?php $disabled = ( $post->post_status === 'publish' && time() > strtotime( $post->post_date ) ) ? true : false; ?>
+				<?php $disabled = ($post->post_status === 'publish' && time() > strtotime( $post->post_date )) ? true : false; ?>
 				<label for="ppp_li_share_on_publish"><?php _e( 'Share this post on LinkedIn&hellip;', 'ppp-txt' ); ?></label>
 				<select name="_ppp_li_share_on_publish" id="ppp_li_share_on_publish" class="ppp-toggle-share-on-publish">
-					<option value="-1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ( $disabled ): ?>disabled<?php endif; ?>><?php _e( 'Do not share this post', 'ppp-txt' ); ?></option>
-					<option value="1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ( $disabled ): ?>disabled<?php endif; ?>><?php _e( 'When this post is published', 'ppp-txt' ); ?></option>
+					<option value="-1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ($disabled): ?>disabled<?php endif; ?>><?php _e( 'Do not share this post', 'ppp-txt' ); ?></option>
+					<option value="1" <?php selected( true, $show_share_on_publish, true ); ?><?php if ($disabled): ?>disabled<?php endif; ?>><?php _e( 'When this post is published', 'ppp-txt' ); ?></option>
 					<option value="0" <?php selected( false, $show_share_on_publish, true ); ?>><?php _e( 'After this post is published', 'ppp-txt' ); ?></option>
 				</select>
 			</p>
@@ -407,7 +407,7 @@ 
 								<th style="width: 10px;"></th>
 							</tr>
 						</thead>
-						<tbody id="li-share-on-publish" class="ppp-share-on-publish" <?php if ( false === $show_share_on_publish ) : echo 'style="display: none;"'; endif; ?>>
+						<tbody id="li-share-on-publish" class="ppp-share-on-publish" <?php if (false === $show_share_on_publish) : echo 'style="display: none;"'; endif; ?>>
 							<?php
 								$args = array(
 									'text'          => $ppp_share_on_publish_title,
@@ -419,19 +419,19 @@ 
 								ppp_render_li_share_on_publish_row( $args );
 							?>
 						</tbody>
-						<tbody id="li-schedule-share" class="ppp-schedule-share" <?php if ( true === $show_share_on_publish ) : echo 'style="display: none;"'; endif; ?>>
+						<tbody id="li-schedule-share" class="ppp-schedule-share" <?php if (true === $show_share_on_publish) : echo 'style="display: none;"'; endif; ?>>
 							<?php $shares = get_post_meta( $post->ID, '_ppp_li_shares', true ); ?>
-							<?php if ( ! empty( $shares ) ) : ?>
+							<?php if ( ! empty($shares)) : ?>
 
-								<?php foreach ( $shares as $key => $value ) :
-									$date          = isset( $value['date'] )          ? $value['date']          : '';
-									$time          = isset( $value['time'] )          ? $value['time']          : '';
-									$text          = isset( $value['text'] )          ? $value['text']          : '';
-									$desc          = isset( $value['desc'] )          ? $value['desc']          : '';
-									$image         = isset( $value['image'] )         ? $value['image']         : '';
-									$attachment_id = isset( $value['attachment_id'] ) ? $value['attachment_id'] : '';
+								<?php foreach ($shares as $key => $value) :
+									$date          = isset($value['date']) ? $value['date'] : '';
+									$time          = isset($value['time']) ? $value['time'] : '';
+									$text          = isset($value['text']) ? $value['text'] : '';
+									$desc          = isset($value['desc']) ? $value['desc'] : '';
+									$image         = isset($value['image']) ? $value['image'] : '';
+									$attachment_id = isset($value['attachment_id']) ? $value['attachment_id'] : '';
 
-									$args = apply_filters( 'ppp_fb_row_args', compact( 'date','time','text', 'desc', 'image','attachment_id' ), $value );
+									$args = apply_filters( 'ppp_fb_row_args', compact( 'date', 'time', 'text', 'desc', 'image', 'attachment_id' ), $value );
 									?>
 
 									<?php ppp_render_li_share_row( $key, $args ); ?>
@@ -467,7 +467,7 @@ 
 function ppp_render_li_share_on_publish_row( $args = array() ) {
 	global $post;
 	$readonly = $post->post_status !== 'publish' ? '' : 'readonly="readonly" ';
-	$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-li-wrapper ppp-repeatable-row on-publish-row">
 		<td colspan="2" class="ppp-on-plublish-date-column">
@@ -495,7 +495,7 @@ 
 	<tr>
 		<td colspan="2"></td>
 		<td colspan="3">
-			<textarea <?php if ( $disabled ): ?>readonly<?php endif; ?> name="_ppp_li_share_on_publish_desc" placeholder="<?php _e( 'Link Description', 'ppp-txt' ); ?>"><?php echo esc_attr( $args['desc'] ); ?></textarea>
+			<textarea <?php if ($disabled): ?>readonly<?php endif; ?> name="_ppp_li_share_on_publish_desc" placeholder="<?php _e( 'Link Description', 'ppp-txt' ); ?>"><?php echo esc_attr( $args['desc'] ); ?></textarea>
 		</td>
 	</tr>
 <?php
@@ -515,8 +515,8 @@ 
 
 	$share_time = ppp_generate_timestamp( $args['date'], $args['time'] );
 	$readonly   = ppp_generate_timestamp() > $share_time ? 'readonly="readonly" ' : false;
-	$no_date    = ! empty( $readonly ) ? ' hasDatepicker' : '';
-	$hide       = ! empty( $readonly ) ? 'display: none;' : '';
+	$no_date    = ! empty($readonly) ? ' hasDatepicker' : '';
+	$hide       = ! empty($readonly) ? 'display: none;' : '';
 	?>
 	<tr class="ppp-li-wrapper ppp-repeatable-row ppp-repeatable-linkedin scheduled-row" data-key="<?php echo esc_attr( $key ); ?>">
 		<td>
@@ -567,26 +567,26 @@ 
  */
 function ppp_li_save_post_meta_boxes( $post_id, $post ) {
 
-	if ( ! ppp_should_save( $post_id, $post ) ) {
+	if ( ! ppp_should_save( $post_id, $post )) {
 		return;
 	}
 
-	$ppp_li_share_on_publish            = ( isset( $_REQUEST['_ppp_li_share_on_publish'] ) )               ? $_REQUEST['_ppp_li_share_on_publish']               : '-1';
-	$ppp_share_on_publish_title         = ( isset( $_REQUEST['_ppp_li_share_on_publish_title'] ) )         ? $_REQUEST['_ppp_li_share_on_publish_title']         : '';
-	$ppp_share_on_publish_desc          = ( isset( $_REQUEST['_ppp_li_share_on_publish_desc'] ) )          ? $_REQUEST['_ppp_li_share_on_publish_desc']          : '';
-	$ppp_share_on_publish_image_url     = ( isset( $_REQUEST['_ppp_li_share_on_publish_image_url'] ) )     ? $_REQUEST['_ppp_li_share_on_publish_image_url']     : '';
-	$ppp_share_on_publish_attachment_id = ( isset( $_REQUEST['_ppp_li_share_on_publish_attachment_id'] ) ) ? $_REQUEST['_ppp_li_share_on_publish_attachment_id'] : '';
+	$ppp_li_share_on_publish            = (isset($_REQUEST['_ppp_li_share_on_publish'])) ? $_REQUEST['_ppp_li_share_on_publish'] : '-1';
+	$ppp_share_on_publish_title         = (isset($_REQUEST['_ppp_li_share_on_publish_title'])) ? $_REQUEST['_ppp_li_share_on_publish_title'] : '';
+	$ppp_share_on_publish_desc          = (isset($_REQUEST['_ppp_li_share_on_publish_desc'])) ? $_REQUEST['_ppp_li_share_on_publish_desc'] : '';
+	$ppp_share_on_publish_image_url     = (isset($_REQUEST['_ppp_li_share_on_publish_image_url'])) ? $_REQUEST['_ppp_li_share_on_publish_image_url'] : '';
+	$ppp_share_on_publish_attachment_id = (isset($_REQUEST['_ppp_li_share_on_publish_attachment_id'])) ? $_REQUEST['_ppp_li_share_on_publish_attachment_id'] : '';
 
-	update_post_meta( $post_id, '_ppp_li_share_on_publish',               $ppp_li_share_on_publish );
-	update_post_meta( $post_id, '_ppp_li_share_on_publish_title',         $ppp_share_on_publish_title );
-	update_post_meta( $post_id, '_ppp_li_share_on_publish_desc',          $ppp_share_on_publish_desc );
-	update_post_meta( $post_id, '_ppp_li_share_on_publish_image_url',     $ppp_share_on_publish_image_url );
+	update_post_meta( $post_id, '_ppp_li_share_on_publish', $ppp_li_share_on_publish );
+	update_post_meta( $post_id, '_ppp_li_share_on_publish_title', $ppp_share_on_publish_title );
+	update_post_meta( $post_id, '_ppp_li_share_on_publish_desc', $ppp_share_on_publish_desc );
+	update_post_meta( $post_id, '_ppp_li_share_on_publish_image_url', $ppp_share_on_publish_image_url );
 	update_post_meta( $post_id, '_ppp_li_share_on_publish_attachment_id', $ppp_share_on_publish_attachment_id );
 
-	$li_data = ( isset( $_REQUEST['_ppp_li_shares'] ) && empty( $ppp_li_share_on_publish ) ) ? $_REQUEST['_ppp_li_shares'] : array();
-	foreach ( $li_data as $index => $share ) {
-		$li_data[ $index ]['text'] = sanitize_text_field( $share['text'] );
-		$li_data[ $index ]['desc'] = sanitize_text_field( $share['desc'] );
+	$li_data = (isset($_REQUEST['_ppp_li_shares']) && empty($ppp_li_share_on_publish)) ? $_REQUEST['_ppp_li_shares'] : array();
+	foreach ($li_data as $index => $share) {
+		$li_data[$index]['text'] = sanitize_text_field( $share['text'] );
+		$li_data[$index]['desc'] = sanitize_text_field( $share['desc'] );
 	}
 
 	update_post_meta( $post_id, '_ppp_li_shares', $li_data );
@@ -602,10 +602,10 @@ 
  */
 function ppp_li_share_on_publish( $new_status, $old_status, $post ) {
 	global $ppp_options;
-	$from_meta = ! empty( $_POST['ppp_post_edit'] ) ? false : get_post_meta( $post->ID, '_ppp_li_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_li_share_on_publish'] ) ? $_POST['_ppp_li_share_on_publish']: '0';
+	$from_meta = ! empty($_POST['ppp_post_edit']) ? false : get_post_meta( $post->ID, '_ppp_li_share_on_publish', true );
+	$from_post = isset($_POST['_ppp_li_share_on_publish']) ? $_POST['_ppp_li_share_on_publish'] : '0';
 
-	if ( '1' != $from_meta && '1' != $from_post ) {
+	if ('1' != $from_meta && '1' != $from_post) {
 		return;
 	}
 
@@ -618,30 +618,30 @@ 
 	$image_url     = '';
 
 	// Determine if we're seeing the share on publish in meta or $_POST
-	if ( $from_meta && ! $from_post ) {
-		$title         = get_post_meta( $post->ID, '_ppp_li_share_on_publish_title'        , true );
-		$desc          = get_post_meta( $post->ID, '_ppp_li_share_on_publish_desc'         , true );
+	if ($from_meta && ! $from_post) {
+		$title         = get_post_meta( $post->ID, '_ppp_li_share_on_publish_title', true );
+		$desc          = get_post_meta( $post->ID, '_ppp_li_share_on_publish_desc', true );
 		$attachment_id = get_post_meta( $post->ID, '_ppp_li_share_on_publish_attachment_id', true );
-		$image_url     = get_post_meta( $post->ID, '_ppp_li_share_on_publish_image_url'    , true );
+		$image_url     = get_post_meta( $post->ID, '_ppp_li_share_on_publish_image_url', true );
 	} else {
-		$title         = isset( $_POST['_ppp_li_share_on_publish_title'] )         ? $_POST['_ppp_li_share_on_publish_title']         : '';
-		$desc          = isset( $_POST['_ppp_li_share_on_publish_desc'] )          ? $_POST['_ppp_li_share_on_publish_desc']          : false;
-		$attachment_id = isset( $_POST['_ppp_li_share_on_publish_attachment_id'] ) ? $_POST['_ppp_li_share_on_publish_attachment_id'] : 0;
-		$image_url     = isset( $_POST['_ppp_li_share_on_publish_image_url'] )     ? $_POST['_ppp_li_share_on_publish_image_url']     : '';
+		$title         = isset($_POST['_ppp_li_share_on_publish_title']) ? $_POST['_ppp_li_share_on_publish_title'] : '';
+		$desc          = isset($_POST['_ppp_li_share_on_publish_desc']) ? $_POST['_ppp_li_share_on_publish_desc'] : false;
+		$attachment_id = isset($_POST['_ppp_li_share_on_publish_attachment_id']) ? $_POST['_ppp_li_share_on_publish_attachment_id'] : 0;
+		$image_url     = isset($_POST['_ppp_li_share_on_publish_image_url']) ? $_POST['_ppp_li_share_on_publish_image_url'] : '';
 	}
 
 	$thumbnail = '';
-	if ( empty( $attachment_id ) && ! empty( $image_url ) ) {
+	if (empty($attachment_id) && ! empty($image_url)) {
 		$thumbnail = $image_url;
 	} else {
 		$thumbnail = ppp_post_has_media( $post->ID, 'li', true, $attachment_id );
 	}
 
-	$name = 'sharedate_0_' . $post->ID . '_li';
+	$name = 'sharedate_0_'.$post->ID.'_li';
 
-	$default_title = isset( $ppp_options['default_text'] ) ? $ppp_options['default_text'] : '';
+	$default_title = isset($ppp_options['default_text']) ? $ppp_options['default_text'] : '';
 	// If an override was found, use it, otherwise try the default text content
-	if ( empty( $title ) && empty( $default_title ) ) {
+	if (empty($title) && empty($default_title)) {
 		$title = get_the_title( $post->ID );
 	}
 
@@ -676,18 +676,18 @@ 
  * @param  string  $name    The name of the Cron
  * @return void
  */
-function ppp_li_scheduled_share(  $post_id = 0, $index = 1, $name = ''  ) {
+function ppp_li_scheduled_share( $post_id = 0, $index = 1, $name = '' ) {
 	global $ppp_options;
 
 	$link = ppp_generate_link( $post_id, $name );
 
 	$post_meta     = get_post_meta( $post_id, '_ppp_li_shares', 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_li_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_li_use_media( $post_id, $index );
@@ -724,7 +724,7 @@ 
  * @return bool         Whether or not this tweet should contain a media post
  */
 function ppp_li_use_media( $post_id, $index ) {
-	if ( empty( $post_id ) || empty( $index ) ) {
+	if (empty($post_id) || empty($index)) {
 		return false;
 	}
 
@@ -757,22 +757,22 @@ 
  */
 function ppp_li_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'] : '';
 	$li_shares    = get_post_meta( $post_id, '_ppp_li_shares', true );
 
-	if ( ! empty( $li_shares ) ) {
+	if ( ! empty($li_shares)) {
 		$name_array    = explode( '_', $name );
 		$index         = $name_array[1];
-		if ( isset( $li_shares[ $index ] ) ) {
-			$share_content = $li_shares[ $index ]['text'];
+		if (isset($li_shares[$index])) {
+			$share_content = $li_shares[$index]['text'];
 		}
 	}
 
 	// 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_li', $share_content, array( 'post_id' => $post_id ) );
 }
@@ -789,8 +789,8 @@ 
 	$description = '';
 	$li_shares   = get_post_meta( $post_id, '_ppp_li_shares', true );
 
-	if ( ! empty( $li_shares[ $index ] ) ) {
-		$description = ! empty( $li_shares[ $index ]['desc'] ) ? $li_shares[ $index ]['desc'] : '';
+	if ( ! empty($li_shares[$index])) {
+		$description = ! empty($li_shares[$index]['desc']) ? $li_shares[$index]['desc'] : '';
 	}
 
 	return $description;
@@ -807,20 +807,20 @@ 
 function ppp_li_generate_timestamps( $times, $post_id ) {
 	$li_shares = get_post_meta( $post_id, '_ppp_li_shares', true );
 
-	if ( empty( $li_shares ) ) {
+	if (empty($li_shares)) {
 		$li_shares = array();
 	}
 
-	foreach ( $li_shares as $key => $data ) {
-		if ( ! array_filter( $data ) ) {
+	foreach ($li_shares 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 ) ) . '_li';
-			$times[ $time_key ] = 'sharedate_' . $key . '_' . $post_id . '_li';
+		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 ) ).'_li';
+			$times[$time_key] = 'sharedate_'.$key.'_'.$post_id.'_li';
 		}
 
 	}
@@ -840,14 +840,14 @@ 
 function ppp_li_calendar_on_publish_event( $events, $post_id ) {
 	$share_on_publish = get_post_meta( $post_id, '_ppp_li_share_on_publish', true );
 
-	if ( ! empty( $share_on_publish ) ) {
+	if ( ! empty($share_on_publish)) {
 		$share_text = get_post_meta( $post_id, '_ppp_li_share_on_publish_title', true );
 		$events[] = array(
-			'id'        => $post_id . '-share-on-publish',
-			'title'     => ( ! empty( $share_text ) ) ? $share_text : ppp_li_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-li cal-post-' . $post_id,
+			'id'        => $post_id.'-share-on-publish',
+			'title'     => ( ! empty($share_text)) ? $share_text : ppp_li_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-li cal-post-'.$post_id,
 			'belongsTo' => $post_id,
 		);
 	}
@@ -858,9 +858,9 @@ 
 
 function ppp_li_get_post_shares( $items, $post_id ) {
 	$shares = get_post_meta( $post_id, '_ppp_li_shares', true );
-	if ( empty( $shares ) ) { return $items; }
+	if (empty($shares)) { return $items; }
 
-	foreach ( $shares as $key => $share ) {
+	foreach ($shares as $key => $share) {
 		$items[] = array( 'id' => $key, 'service' => 'li' );
 	}
 	return $items;

includes/libs/linkedin.php

Spacing +34 added lines, -34 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,13 +11,13 @@ 
  * Handles all linkedin functions
  *
  */
-if( !class_exists( 'PPP_Linkedin' ) ) {
+if ( ! class_exists( 'PPP_Linkedin' )) {
 
 	class PPP_Linkedin {
 
 		var $linkedin;
 
-		public function __construct(){
+		public function __construct() {
 			ppp_maybe_start_session();
 		}
 
@@ -28,24 +28,24 @@ 
 		 */
 		public function ppp_load_linkedin() {
 
-				if( !class_exists( 'LinkedIn' ) ) {
-					require_once ( PPP_PATH . '/includes/libs/linkedin/linkedin_oAuth.php' );
+				if ( ! class_exists( 'LinkedIn' )) {
+					require_once (PPP_PATH.'/includes/libs/linkedin/linkedin_oAuth.php');
 				}
 
 
 				ppp_set_social_tokens();
 
-				if ( ! defined( 'LINKEDIN_KEY' ) || ! defined( 'LINKEDIN_SECRET' ) ) {
+				if ( ! defined( 'LINKEDIN_KEY' ) || ! defined( 'LINKEDIN_SECRET' )) {
 					return false;
 				}
 
 				global $ppp_social_settings;
 				$config = array( 'appKey' => LINKEDIN_KEY, 'appSecret' => LINKEDIN_SECRET );
-				if ( isset( $ppp_social_settings['linkedin']->access_token ) ) {
+				if (isset($ppp_social_settings['linkedin']->access_token)) {
 					$config['accessToken'] = $ppp_social_settings['linkedin']->access_token;
 				}
 
-				if ( !$this->linkedin ) {
+				if ( ! $this->linkedin) {
 					$this->linkedin = new LinkedIn( $config );
 				}
 
@@ -60,36 +60,36 @@ 
 			$linkedin = $this->ppp_load_linkedin();
 
 			//when user is going to logged in and verified successfully session will create
-			if ( isset( $_REQUEST['li_access_token'] ) && isset( $_REQUEST['expires_in'] ) ) {
+			if (isset($_REQUEST['li_access_token']) && isset($_REQUEST['expires_in'])) {
 
 				$access_token = $_REQUEST['li_access_token'];
 				$expires_in   = $_REQUEST['expires_in'];
 
-			} elseif ( isset( $_GET['state'] ) && strpos( $_GET['state'], 'ppp-local-keys-li' ) !== false ) {
-				$access_code = isset( $_GET['code'] ) ? $_GET['code'] : false;
+			} elseif (isset($_GET['state']) && strpos( $_GET['state'], 'ppp-local-keys-li' ) !== false) {
+				$access_code = isset($_GET['code']) ? $_GET['code'] : false;
 
-				if ( empty( $access_code ) ) {
+				if (empty($access_code)) {
 					return;
 				}
 
-				$params  = 'grant_type=authorization_code&client_id=' . LINKEDIN_KEY;
-				$params .= '&client_secret=' . LINKEDIN_SECRET;
-				$params .= '&code=' . $access_code;
-				$params .= '&redirect_uri=' . admin_url( 'admin.php?page=ppp-social-settings' );
-				$url = 'https://www.linkedin.com/uas/oauth2/accessToken?' . $params;
+				$params  = 'grant_type=authorization_code&client_id='.LINKEDIN_KEY;
+				$params .= '&client_secret='.LINKEDIN_SECRET;
+				$params .= '&code='.$access_code;
+				$params .= '&redirect_uri='.admin_url( 'admin.php?page=ppp-social-settings' );
+				$url = 'https://www.linkedin.com/uas/oauth2/accessToken?'.$params;
 				$response = json_decode( wp_remote_retrieve_body( wp_remote_post( $url ) ) );
 
-				$access_token = isset( $response->access_token ) ? $response->access_token : false;
-				$expires_in   = isset( $response->expires_in ) ? $response->expires_in : false;
+				$access_token = isset($response->access_token) ? $response->access_token : false;
+				$expires_in   = isset($response->expires_in) ? $response->expires_in : false;
 
 			}
 
-			if ( ! empty( $access_token ) && ! empty( $expires_in ) ) {
+			if ( ! empty($access_token) && ! empty($expires_in)) {
 				global $ppp_social_settings;
 				$ppp_social_settings = get_option( 'ppp_social_settings' );
 
 				//check linkedin class is loaded or not
-				if( !$linkedin ) {
+				if ( ! $linkedin) {
 					return false;
 				}
 
@@ -99,7 +99,7 @@ 
 				$expires_in = (int) $expires_in;
 				$data->expires_on = current_time( 'timestamp' ) + $expires_in;
 
-				update_option( '_ppp_linkedin_refresh', current_time( 'timestamp' ) + round( $expires_in/1.25 ) );
+				update_option( '_ppp_linkedin_refresh', current_time( 'timestamp' ) + round( $expires_in / 1.25 ) );
 
 				$ppp_social_settings['linkedin'] = $data;
 				update_option( 'ppp_social_settings', $ppp_social_settings );
@@ -112,7 +112,7 @@ 
 
 				update_option( 'ppp_social_settings', $ppp_social_settings );
 
-				$url = remove_query_arg( array( 'li_access_token' , 'expires_in' ) );
+				$url = remove_query_arg( array( 'li_access_token', 'expires_in' ) );
 				wp_redirect( $url );
 				die();
 			}
@@ -122,19 +122,19 @@ 
 		 * Get auth url for linkedin
 		 *
 		 */
-		public function ppp_get_linkedin_auth_url ( $return_url ) {
+		public function ppp_get_linkedin_auth_url( $return_url ) {
 
-			if ( ! PPP_LOCAL_TOKENS ) {
+			if ( ! PPP_LOCAL_TOKENS) {
 				$base_url = 'https://postpromoterpro.com/?ppp-social-auth';
-				$url  = $base_url . '&ppp-service=li&ppp-license-key=' . trim( get_option( '_ppp_license_key' ) );
+				$url  = $base_url.'&ppp-service=li&ppp-license-key='.trim( get_option( '_ppp_license_key' ) );
 				$url .= '&nocache';
-				$url .= '&return_url=' . esc_url( $return_url );
+				$url .= '&return_url='.esc_url( $return_url );
 			} else {
 				$url  = 'https://www.linkedin.com/uas/oauth2/authorization?response_type=code';
-				$url .= '&client_id=' . LINKEDIN_KEY;
+				$url .= '&client_id='.LINKEDIN_KEY;
 				$url .= '&scope=w_share%20r_basicprofile';
 				$url .= '&state=ppp-local-keys-li';
-				$url .= '&redirect_uri=' . $return_url;
+				$url .= '&redirect_uri='.$return_url;
 			}
 
 			return $url;
@@ -144,7 +144,7 @@ 
 		 * Share somethign on linkedin
 		 */
 		public function ppp_linkedin_share( $args ) {
-			if ( empty( $args ) ) {
+			if (empty($args)) {
 				return false;
 			}
 
@@ -163,11 +163,11 @@ 
 					'description' => $args['description'],
 					'title' => $args['title'],
 				),
-				'owner' => 'urn:li:person:' . $li_user_info->id,
+				'owner' => 'urn:li:person:'.$li_user_info->id,
 			);
 
 
-			if ( $args['submitted-image-url'] !== false ) {
+			if ($args['submitted-image-url'] !== false) {
 				$share['content']['contentEntities'][0]['thumbnails'] = array(
 					array(
 						'resolvedUrl' => $args['submitted-image-url'],
@@ -175,7 +175,7 @@ 
 				);
 			}
 
-			$headers = array( 'X-Restli-Protocol-Version' => '2.0.0', 'Authorization' => 'Bearer ' . $ppp_social_settings['linkedin']->access_token, 'Content-Type' => 'application/json' );
+			$headers = array( 'X-Restli-Protocol-Version' => '2.0.0', 'Authorization' => 'Bearer '.$ppp_social_settings['linkedin']->access_token, 'Content-Type' => 'application/json' );
 			$request = wp_remote_post( $url, array( 'httpversion' => '1.1', 'timeout' => '30', 'headers' => $headers, 'body' => json_encode( $share ) ) );
 
 			return json_decode( wp_remote_retrieve_body( $request ) );
@@ -184,7 +184,7 @@ 
 		public function ppp_linkedin_profile( $access_token ) {
 			$url = 'https://api.linkedin.com/v2/me';
 
-			$headers = array( 'Authorization' => 'Bearer ' . $access_token );
+			$headers = array( 'Authorization' => 'Bearer '.$access_token );
 
 			$request = wp_remote_get( $url, array( 'headers' => $headers ) );
 			$data = wp_remote_retrieve_body( $request );