postpromoterpro / post-promoter-pro

includes/twitter-functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 /**
  * Register Twitter as a servcie
  * @param  array $services The registered services
- * @return array           With Twitter added
+ * @return string[]           With Twitter added
  */
 function ppp_tw_register_service( $services = array() ) {
 	$services[] = 'tw';
@@ -240,7 +240,7 @@ 
 /**
  * Register the Twitter connection area for the Social Media Accounts section
  * @param  array $content The existing content tokens
- * @return array          The content tokens with Twitter added
+ * @return string[]          The content tokens with Twitter added
  */
 function ppp_tw_register_admin_social_content( $content ) {
 	$content[] = 'tw';

Braces +5 added lines, -2 removed lines

@@ -429,9 +429,12 @@
 									}
 									?>
 
-								<?php else: ?>
+								<?php else {
+	: ?>
 
-									<?php ppp_render_tweet_row( 1, array( 'date' => '', 'time' => '', 'text' => '', 'image' => '', 'attachment_id' => '' ), $post->ID, 1 ); ?>
+									<?php ppp_render_tweet_row( 1, array( 'date' => '', 'time' => '', 'text' => '', 'image' => '', 'attachment_id' => '' ), $post->ID, 1 );
+}
+?>
 
 								<?php endif; ?>
 

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' ) . '" /> <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' ).'" /> <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,26 +106,26 @@ 
 
 	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['ppp_user_auth'] ) ) {
+	if ( ! empty($_POST['ppp_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;
@@ -135,7 +135,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;
@@ -151,13 +151,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();
@@ -175,17 +175,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 );
 		}
 	}
@@ -216,12 +216,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 );
@@ -248,12 +248,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 );
 		}
@@ -274,8 +274,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 );
 			}
@@ -295,9 +295,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 );
@@ -311,22 +311,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 ) );
 }
@@ -338,12 +338,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;
 }
@@ -354,7 +354,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 );
 	}
@@ -392,7 +392,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;
 	}
 
@@ -409,7 +409,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;
 	}
 
@@ -468,16 +468,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 ); ?>
@@ -486,7 +486,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'          => '',
@@ -509,7 +509,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>
@@ -535,38 +535,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>
 
@@ -585,7 +585,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 {
@@ -593,11 +593,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++;
 	}
 	?>
@@ -612,7 +612,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>
 
@@ -647,24 +647,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 );
 
@@ -681,29 +681,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 );
 
@@ -721,11 +721,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 );
 		}
@@ -746,8 +746,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 );
 			}
@@ -768,20 +768,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';
 		}
 
 	}
@@ -801,7 +801,7 @@ 
 
 	$ret = false;
 
-	if ( ! empty( $ppp_share_settings['twitter']['cards_enabled'] ) ) {
+	if ( ! empty($ppp_share_settings['twitter']['cards_enabled'])) {
 		$ret = true;
 	}
 
@@ -816,13 +816,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;
 	}
 
@@ -840,20 +840,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 );
@@ -873,15 +873,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 );
 			}
 		}
@@ -889,15 +889,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 ) );
@@ -916,13 +916,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 );
 	}
 
@@ -940,10 +940,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;
@@ -958,7 +958,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
@@ -977,15 +977,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;
 	}
 
@@ -999,12 +999,12 @@ 
 			<?php
 			$twitter = new PPP_Twitter_User( get_current_user_id() );
 			$tw_user = get_user_meta( $user->ID, '_ppp_twitter_data', true );
-			if ( empty( $tw_user ) ) {
-				$tw_authurl = $twitter->get_auth_url( admin_url( 'user-edit.php?user_id=' . $user->ID ) );
+			if (empty($tw_user)) {
+				$tw_authurl = $twitter->get_auth_url( admin_url( 'user-edit.php?user_id='.$user->ID ) );
 
-				$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' ) . '" /> <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' ).'" /> <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;
@@ -1013,7 +1013,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
@@ -1022,12 +1022,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>
@@ -1075,19 +1075,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' );
@@ -1096,14 +1096,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,
 		);
 	}
@@ -1114,9 +1114,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/admin/admin-ajax.php

Spacing +4 added lines, -4 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;
 	}
 

includes/admin/class-accounts-table.php

Spacing +10 added lines, -10 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists( 'WP_List_Table' )) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -42,7 +42,7 @@ 
 	 * @return string              The Content to display
 	 */
 	public function column_default( $item, $column_name ) {
-		return $item[ $column_name ];
+		return $item[$column_name];
 	}
 
 	/**
@@ -74,13 +74,13 @@ 
 
 		$accounts = apply_filters( 'ppp_register_social_service', array() );
 
-		foreach ( $accounts as $account ) {
+		foreach ($accounts as $account) {
 			$data[$account] = array(
-				'icon'    => apply_filters( 'ppp_account_list_icon-' . $account, '' ),
-				'avatar'  => apply_filters( 'ppp_account_list_avatar-' . $account, '' ),
-				'name'    => apply_filters( 'ppp_account_list_name-' . $account, '' ),
-				'actions' => apply_filters( 'ppp_account_list_actions-' . $account, '' ),
-				'extras'  => apply_filters( 'ppp_account_list_extras-' . $account, '' ),
+				'icon'    => apply_filters( 'ppp_account_list_icon-'.$account, '' ),
+				'avatar'  => apply_filters( 'ppp_account_list_avatar-'.$account, '' ),
+				'name'    => apply_filters( 'ppp_account_list_name-'.$account, '' ),
+				'actions' => apply_filters( 'ppp_account_list_actions-'.$account, '' ),
+				'extras'  => apply_filters( 'ppp_account_list_extras-'.$account, '' ),
 			);
 		}
 

includes/admin/class-schedule-table.php

Switch Indentation +15 added lines, -15 removed lines

@@ -43,21 +43,21 @@
 	 */
 	public function column_default( $item, $column_name ) {
 		switch ( $column_name ) {
-		case 'date':
-		case 'post_title':
-			return $item[ $column_name ];
-		case 'content':
-			$content = $item[ $column_name ];
-			return $content;
-		case 'image':
-			if ( ! empty( $item['image_url'] ) ) {
-				$content = '<img src="' . $item['image_url'] . '" />';
-			} else {
-				$content = __( 'None', 'ppp-txt' );
-			}
-			return $content;
-		default:
-			return;
+			case 'date':
+			case 'post_title':
+				return $item[ $column_name ];
+			case 'content':
+				$content = $item[ $column_name ];
+				return $content;
+			case 'image':
+				if ( ! empty( $item['image_url'] ) ) {
+					$content = '<img src="' . $item['image_url'] . '" />';
+				} else {
+					$content = __( 'None', 'ppp-txt' );
+				}
+				return $content;
+			default:
+				return;
 		}
 	}
 

Spacing +38 added lines, -38 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists( 'WP_List_Table' )) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -42,16 +42,16 @@ 
 	 * @return string              The Content to display
 	 */
 	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+		switch ($column_name) {
 		case 'date':
 		case 'post_title':
-			return $item[ $column_name ];
+			return $item[$column_name];
 		case 'content':
-			$content = $item[ $column_name ];
+			$content = $item[$column_name];
 			return $content;
 		case 'image':
-			if ( ! empty( $item['image_url'] ) ) {
-				$content = '<img src="' . $item['image_url'] . '" />';
+			if ( ! empty($item['image_url'])) {
+				$content = '<img src="'.$item['image_url'].'" />';
 			} else {
 				$content = __( 'None', 'ppp-txt' );
 			}
@@ -82,15 +82,15 @@ 
 	 * @return string       The HTML to display for this column.
 	 */
 	public function column_post_title( $item ) {
-		$item_name = $item['name'] . '_' . $item['service'];
+		$item_name = $item['name'].'_'.$item['service'];
 		$actions   = array(
-			'edit'          => sprintf( __( '<a href="%s">Edit</a>', 'ppp-txt' ), admin_url( 'post.php?post=' . $item['post_id'] . '&action=edit#ppp_schedule_metabox' ) ),
-			'delete'        => sprintf( __( '<a href="%s">Delete</a>', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-schedule-info&action=delete_item&post_id=' . $item['post_id'] . '&name=' . $item_name . '&index=' . $item['index'] ) ),
-			'share'         => sprintf( __( '<a href="%s">Share Now</a>', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-schedule-info&action=share_now&post_id=' . $item['post_id'] . '&name=' . $item_name ) ),
-			'share_delete'  => sprintf( __( '<a href="%s">Share Now & Delete</a>', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-schedule-info&action=share_now&post_id=' . $item['post_id'] . '&name=' . $item_name . '&index=' . $item['index'] . '&delete_too=true' ) ),
+			'edit'          => sprintf( __( '<a href="%s">Edit</a>', 'ppp-txt' ), admin_url( 'post.php?post='.$item['post_id'].'&action=edit#ppp_schedule_metabox' ) ),
+			'delete'        => sprintf( __( '<a href="%s">Delete</a>', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-schedule-info&action=delete_item&post_id='.$item['post_id'].'&name='.$item_name.'&index='.$item['index'] ) ),
+			'share'         => sprintf( __( '<a href="%s">Share Now</a>', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-schedule-info&action=share_now&post_id='.$item['post_id'].'&name='.$item_name ) ),
+			'share_delete'  => sprintf( __( '<a href="%s">Share Now & Delete</a>', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-schedule-info&action=share_now&post_id='.$item['post_id'].'&name='.$item_name.'&index='.$item['index'].'&delete_too=true' ) ),
 		);
 
-		return sprintf( '<span class="dashicons icon-ppp-' . $item['service'] . '"></span>&nbsp;%1$s %2$s', $item['post_title'], $this->row_actions( $actions ) );
+		return sprintf( '<span class="dashicons icon-ppp-'.$item['service'].'"></span>&nbsp;%1$s %2$s', $item['post_title'], $this->row_actions( $actions ) );
 	}
 
 	/**
@@ -99,9 +99,9 @@ 
 	 * @return string       The HTML to display the date
 	 */
 	public function column_date( $item ) {
-		$date = date_i18n( get_option('date_format') . ' @ ' . get_option('time_format'), $item['date'] );
-		if ( $item['conflict'] ) {
-			$date .= '<br /><small style="color: red">' . __( 'Warning: Multiple items scheduled at this time.', 'ppp-txt' ) . '</small>';
+		$date = date_i18n( get_option( 'date_format' ).' @ '.get_option( 'time_format' ), $item['date'] );
+		if ($item['conflict']) {
+			$date .= '<br /><small style="color: red">'.__( 'Warning: Multiple items scheduled at this time.', 'ppp-txt' ).'</small>';
 		}
 
 		return $date;
@@ -123,48 +123,48 @@ 
 		$crons = ppp_get_shceduled_crons();
 
 		$cron_tally = array();
-		foreach ( $crons as $key => $cron ) {
+		foreach ($crons as $key => $cron) {
 			$ppp_data = $cron;
 			$timestamp = $ppp_data['timestamp'];
 
-			$cron_tally[$timestamp] = isset( $cron_tally[$timestamp] ) ? $cron_tally[$timestamp] + 1 : 1;
+			$cron_tally[$timestamp] = isset($cron_tally[$timestamp]) ? $cron_tally[$timestamp] + 1 : 1;
 
 			$name_parts = explode( '_', $ppp_data['args'][1] );
 			$post_id    = $ppp_data['args'][0];
 			$index      = $name_parts[1];
-			$service    = isset( $name_parts[3] ) ? $name_parts[3] : 'tw';
-			$builder    = 'ppp_' . $service . '_build_share_message';
-			$post_meta  = apply_filters( 'ppp_get_scheduled_items_' . $service, array(), $post_id );
+			$service    = isset($name_parts[3]) ? $name_parts[3] : 'tw';
+			$builder    = 'ppp_'.$service.'_build_share_message';
+			$post_meta  = apply_filters( 'ppp_get_scheduled_items_'.$service, array(), $post_id );
 			$image_url  = '';
 
-			if ( ! empty( $post_meta[$index]['attachment_id'] ) ) {
-				$image_url = ppp_post_has_media( $post_id, $service, true, $post_meta[ $index ]['attachment_id'] );
-			} elseif ( ! empty( $post_meta[ $index ]['image'] ) ) {
-				$image_url = $post_meta[ $index ]['image'];
+			if ( ! empty($post_meta[$index]['attachment_id'])) {
+				$image_url = ppp_post_has_media( $post_id, $service, true, $post_meta[$index]['attachment_id'] );
+			} elseif ( ! empty($post_meta[$index]['image'])) {
+				$image_url = $post_meta[$index]['image'];
 			}
 
-			$conflict   = $cron_tally[ $timestamp ] > 1 ? true : false;
+			$conflict   = $cron_tally[$timestamp] > 1 ? true : false;
 
-			$data[ $key ] = array(
+			$data[$key] = array(
 				'post_id'      => $post_id,
 				'post_title'   => get_the_title( $post_id ),
 				'service'      => $service,
 				'index'        => $index,
-				'date'         => $timestamp + ( get_option( 'gmt_offset' ) * 3600 ),
+				'date'         => $timestamp + (get_option( 'gmt_offset' ) * 3600),
 				'content'      => function_exists( $builder ) ? $builder( $post_id, $ppp_data['args'][1], false ) : '',
-				'name'         => 'sharedate_' . $index . '_' . $post_id,
+				'name'         => 'sharedate_'.$index.'_'.$post_id,
 				'conflict'     => $conflict,
 			);
 
-			if ( ! empty( $image_url ) ) {
-				$data[ $key ]['image_url'] = $image_url;
+			if ( ! empty($image_url)) {
+				$data[$key]['image_url'] = $image_url;
 			}
 
-			$description_function = 'ppp_' . $service . '_get_share_description';
-			if ( function_exists( $description_function ) ) {
+			$description_function = 'ppp_'.$service.'_get_share_description';
+			if (function_exists( $description_function )) {
 				$description = $description_function( $post_id, $index );
-				if ( ! empty( $description ) ) {
-					$data[ $key ]['content'] .= '<br />' . $description;
+				if ( ! empty($description)) {
+					$data[$key]['content'] .= '<br />'.$description;
 				}
 			}
 
@@ -172,9 +172,9 @@ 
 
 		$total_items = count( $data );
 
-		$offset = isset( $_GET['paged'] ) ? $_GET['paged'] : 1;
+		$offset = isset($_GET['paged']) ? $_GET['paged'] : 1;
 
-		$data = array_slice( $data, ( $offset - 1 ) * $per_page, $per_page, true );
+		$data = array_slice( $data, ($offset - 1) * $per_page, $per_page, true );
 		$this->items = $data;
 
 

includes/admin/do-upgrades.php

Spacing +10 added lines, -10 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' )) exit;
 
 /**
  * Render Upgrades Screen
@@ -19,12 +19,12 @@ 
  * @return void
 */
 function ppp_upgrades_screen() {
-	$action = isset( $_GET['ppp-upgrade'] ) ? sanitize_text_field( $_GET['ppp-upgrade'] ) : '';
-	$step   = isset( $_GET['step'] )        ? absint( $_GET['step'] )                     : 1;
-	$total  = isset( $_GET['total'] )       ? absint( $_GET['total'] )                    : false;
-	$custom = isset( $_GET['custom'] )      ? absint( $_GET['custom'] )                   : 0;
-	$number = isset( $_GET['number'] )      ? absint( $_GET['number'] )                   : 100;
-	$steps  = round( ( $total / $number ), 0 );
+	$action = isset($_GET['ppp-upgrade']) ? sanitize_text_field( $_GET['ppp-upgrade'] ) : '';
+	$step   = isset($_GET['step']) ? absint( $_GET['step'] ) : 1;
+	$total  = isset($_GET['total']) ? absint( $_GET['total'] ) : false;
+	$custom = isset($_GET['custom']) ? absint( $_GET['custom'] ) : 0;
+	$number = isset($_GET['number']) ? absint( $_GET['number'] ) : 100;
+	$steps  = round( ($total / $number), 0 );
 
 	$doing_upgrade_args = array(
 		'page'        => 'ppp-upgrades',
@@ -36,7 +36,7 @@ 
 	);
 
 	update_option( 'ppp_doing_upgrade', $doing_upgrade_args );
-	if ( $step > $steps ) {
+	if ($step > $steps) {
 		// Prevent a weird case where the estimate was off. Usually only a couple.
 		$steps = $step;
 	}
@@ -44,12 +44,12 @@ 
 	<div class="wrap">
 		<h1><?php _e( 'Post Promoter Pro - Upgrades', 'ppp-txt' ); ?></h1>
 
-		<?php if( ! empty( $action ) ) : ?>
+		<?php if ( ! empty($action)) : ?>
 
 			<div id="ppp-upgrade-status">
 				<p><?php _e( 'The upgrade process has started, please be patient. This could take several minutes. You will be automatically redirected when the upgrade is finished.', 'ppp-txt' ); ?></p>
 
-				<?php if( ! empty( $total ) ) : ?>
+				<?php if ( ! empty($total)) : ?>
 					<p><strong><?php printf( __( 'Step %d of approximately %d running', 'ppp-txt' ), $step, $steps ); ?></strong></p>
 				<?php endif; ?>
 			</div>

Braces +8 added lines, -3 removed lines

@@ -10,7 +10,9 @@ 
  */
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) exit;
+if ( !defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Render Upgrades Screen
@@ -57,11 +59,14 @@ 
 				setTimeout(function() { document.location.href = "index.php?ppp_action=<?php echo $action; ?>&step=<?php echo $step; ?>&total=<?php echo $total; ?>&custom=<?php echo $custom; ?>"; }, 250);
 			</script>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
 			<div id="ppp-upgrade-status">
 				<p>
-					<?php _e( 'The upgrade process has started, please be patient. This could take several minutes. You will be automatically redirected when the upgrade is finished.', 'ppp-txt' ); ?>
+					<?php _e( 'The upgrade process has started, please be patient. This could take several minutes. You will be automatically redirected when the upgrade is finished.', 'ppp-txt' );
+}
+?>
 				</p>
 			</div>
 			<script type="text/javascript">

includes/admin/meta-boxes.php

Spacing +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -12,11 +12,11 @@ 
 function ppp_register_meta_boxes() {
 	global $post, $ppp_options;
 
-	if ( !isset( $ppp_options['post_types'] ) || !is_array( $ppp_options['post_types'] ) ) {
+	if ( ! isset($ppp_options['post_types']) || ! is_array( $ppp_options['post_types'] )) {
 		return;
 	}
 
-	foreach ( $ppp_options['post_types'] as $post_type => $value ) {
+	foreach ($ppp_options['post_types'] as $post_type => $value) {
 		add_meta_box( 'ppp_schedule_metabox', 'Post Promoter Pro', 'ppp_schedule_callback', $post_type, 'normal', 'high' );
 	}
 }

includes/libs/bitly.php

Spacing +14 added lines, -14 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( !defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,13 +11,13 @@ 
  * Handles all bitly functions
  *
  */
-if( !class_exists( 'PPP_Bitly' ) ) {
+if ( ! class_exists( 'PPP_Bitly' )) {
 
 	class PPP_Bitly {
 
 		var $bitly;
 
-		public function __construct(){
+		public function __construct() {
 			ppp_maybe_start_session();
 		}
 
@@ -27,20 +27,20 @@ 
 		 * Handles to load twitter class
 		 */
 		public function ppp_load_bitly() {
-				if( !class_exists( 'Bitly' ) ) {
-					require_once ( PPP_PATH . '/includes/libs/bitly/bitly.php' );
+				if ( ! class_exists( 'Bitly' )) {
+					require_once (PPP_PATH.'/includes/libs/bitly/bitly.php');
 				}
 
 				ppp_set_social_tokens();
 
-				if ( ! defined( 'bitly_clientid' ) || ! defined( 'bitly_secret' ) ) {
+				if ( ! defined( 'bitly_clientid' ) || ! defined( 'bitly_secret' )) {
 					return false;
 				}
 
 				global $ppp_social_settings;
 
-				if ( isset( $ppp_social_settings['bitly'] ) ) {
-					if ( !defined( 'bitly_accesstoken' ) ) {
+				if (isset($ppp_social_settings['bitly'])) {
+					if ( ! defined( 'bitly_accesstoken' )) {
 						define( 'bitly_accesstoken', $ppp_social_settings['bitly']['access_token'] );
 					}
 					$this->bitly = new Bitly( bitly_clientid, bitly_secret, bitly_accesstoken );
@@ -54,7 +54,7 @@ 
 		public function revoke_access() {
 			global $ppp_social_settings;
 
-			unset( $ppp_social_settings['bitly'] );
+			unset($ppp_social_settings['bitly']);
 
 			update_option( 'ppp_social_settings', $ppp_social_settings );
 		}
@@ -63,12 +63,12 @@ 
 		 * Get auth codes for Bitly
 		 *
 		 */
-		public function ppp_get_bitly_auth_url () {
+		public function ppp_get_bitly_auth_url() {
 			//load bitly class
 			$bitly = $this->ppp_load_bitly();
 
 			//check bitly class is loaded or not
-			if( !$bitly ) {
+			if ( ! $bitly) {
 				return false;
 			}
 
@@ -77,13 +77,13 @@ 
 		}
 
 		public function ppp_make_bitly_link( $link = null ) {
-			if ( empty( $link ) ) {
+			if (empty($link)) {
 				return false;
 			}
 
 			$bitly = $this->ppp_load_bitly();
 
-			if ( !$bitly ) {
+			if ( ! $bitly) {
 				return false;
 			}
 
@@ -93,7 +93,7 @@ 
 		public function ppp_bitly_user_info() {
 			$bitly = $this->ppp_load_bitly();
 
-			if ( !$bitly ) {
+			if ( ! $bitly) {
 				return false;
 			}
 

includes/libs/twitter.php

Switch Indentation +10 added lines, -10 removed lines

@@ -146,16 +146,16 @@
 			// If last connection failed don't display authorization link.
 			switch( $this->twitter->http_code ) {
 
-			case 200:
-				$_SESSION['ppp_twt_oauth_token']        = $request_token['oauth_token'];
-				$_SESSION['ppp_twt_oauth_token_secret'] = $request_token['oauth_token_secret'];
-
-				$token = $request_token['oauth_token'];
-				$url = $this->twitter->getAuthorizeURL( $token, NULL );
-				break;
-			default:
-				$url = '';
-				break;
+				case 200:
+					$_SESSION['ppp_twt_oauth_token']        = $request_token['oauth_token'];
+					$_SESSION['ppp_twt_oauth_token_secret'] = $request_token['oauth_token_secret'];
+
+					$token = $request_token['oauth_token'];
+					$url = $this->twitter->getAuthorizeURL( $token, NULL );
+					break;
+				default:
+					$url = '';
+					break;
 			}
 			return $url;
 		}

Spacing +24 added lines, -24 removed lines

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

includes/share-functions.php

Indentation +3 added lines, -3 removed lines

@@ -141,9 +141,9 @@
  */
 function ppp_set_social_tokens() {
 	if ( ( defined( 'PPP_TW_CONSUMER_KEY' ) && defined( 'PPP_TW_CONSUMER_SECRET' ) ) ||
-	     ( defined( 'LINKEDIN_KEY' ) && defined( 'LINKEDIN_SECRET' ) ) ||
-	     ( defined( 'bitly_clientid' ) && defined( 'bitly_secret' ) ) ||
-	     ( defined( 'PPP_FB_APP_ID' ) && defined( 'PPP_FB_APP_SECRET' ) )
+		 ( defined( 'LINKEDIN_KEY' ) && defined( 'LINKEDIN_SECRET' ) ) ||
+		 ( defined( 'bitly_clientid' ) && defined( 'bitly_secret' ) ) ||
+		 ( defined( 'PPP_FB_APP_ID' ) && defined( 'PPP_FB_APP_SECRET' ) )
 	   ) {
 		return;
 	}

Spacing +47 added lines, -47 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -16,17 +16,17 @@ 
  */
 function ppp_share_on_publish( $new_status, $old_status, $post ) {
 	// don't publish password protected posts
-	if ( '' !== $post->post_password ) {
+	if ('' !== $post->post_password) {
 		return;
 	}
 
-	if ( $new_status == 'publish' && $old_status != 'publish' ) {
+	if ($new_status == 'publish' && $old_status != 'publish') {
 		global $ppp_options;
 
-		$allowed_post_types = isset( $ppp_options['post_types'] ) ? $ppp_options['post_types'] : array();
+		$allowed_post_types = isset($ppp_options['post_types']) ? $ppp_options['post_types'] : array();
 		$allowed_post_types = apply_filters( 'ppp_schedule_share_post_types', $allowed_post_types );
 
-		if ( !isset( $post->post_status ) || !array_key_exists( $post->post_type, $allowed_post_types ) ) {
+		if ( ! isset($post->post_status) || ! array_key_exists( $post->post_type, $allowed_post_types )) {
 			return false;
 		}
 
@@ -55,26 +55,26 @@ 
 function ppp_share_post( $post_id, $name ) {
 	global $ppp_options, $ppp_social_settings, $ppp_share_settings, $ppp_twitter_oauth;
 
-	if ( ppp_is_dev_or_staging() ) {
+	if (ppp_is_dev_or_staging()) {
 		return;
 	}
 
 	// If we've already started to share this, don't share it again.
 	// Compensates for wp-cron's race conditions
-	if ( get_transient( 'ppp_sharing' . $name ) === 'true' ) {
+	if (get_transient( 'ppp_sharing'.$name ) === 'true') {
 		return;
 	}
 
 	// For 10 seconds, don't allow another share to go for this post
-	set_transient( 'ppp_sharing' . $name, 'true', 10 );
+	set_transient( 'ppp_sharing'.$name, 'true', 10 );
 
 	$name_parts = explode( '_', $name );
 	$index      = $name_parts[1];
-	$service    = isset( $name_parts[3] ) ? $name_parts[3] : 'tw';
+	$service    = isset($name_parts[3]) ? $name_parts[3] : 'tw';
 
 	// If we're fired on a cron, check for stale cron runs
-	if ( defined( 'DOING_CRON' ) && DOING_CRON ) {
-		switch( $service ) {
+	if (defined( 'DOING_CRON' ) && DOING_CRON) {
+		switch ($service) {
 			case 'tw':
 				$post_meta = get_post_meta( $post_id, '_ppp_tweets', true );
 				break;
@@ -89,20 +89,20 @@ 
 		}
 
 		$share_data = array();
-		if ( isset( $post_meta[ $index ] ) ) {
-			$share_data = $post_meta[ $index ];
+		if (isset($post_meta[$index])) {
+			$share_data = $post_meta[$index];
 		}
 
 		$timestamp = ppp_generate_timestamp( $share_data['date'], $share_data['time'] );
 
 		// If the current time is more than 60 minutes (filterable) overdue, don't fire the share
 		$share_buffer = apply_filters( 'ppp_share_buffer', HOUR_IN_SECONDS );
-		if ( ( current_time( 'timestamp', 1 ) - $timestamp ) > $share_buffer ) {
+		if ((current_time( 'timestamp', 1 ) - $timestamp) > $share_buffer) {
 			return;
 		}
 	}
 
-	do_action( 'ppp_share_scheduled_' . $service, $post_id, $index, $name );
+	do_action( 'ppp_share_scheduled_'.$service, $post_id, $index, $name );
 
 	// Just in case we get caught in a weird state, kill this share immediately after we execute it.
 	wp_clear_scheduled_hook( 'ppp_share_post_event', array( $post_id, $name ) );
@@ -114,28 +114,28 @@ 
  * @return void
  */
 function ppp_set_social_tokens() {
-	if ( ( defined( 'PPP_TW_CONSUMER_KEY' ) && defined( 'PPP_TW_CONSUMER_SECRET' ) ) ||
-	     ( defined( 'LINKEDIN_KEY' ) && defined( 'LINKEDIN_SECRET' ) ) ||
-	     ( defined( 'bitly_clientid' ) && defined( 'bitly_secret' ) ) ||
-	     ( defined( 'PPP_FB_APP_ID' ) && defined( 'PPP_FB_APP_SECRET' ) )
+	if ((defined( 'PPP_TW_CONSUMER_KEY' ) && defined( 'PPP_TW_CONSUMER_SECRET' )) ||
+	     (defined( 'LINKEDIN_KEY' ) && defined( 'LINKEDIN_SECRET' )) ||
+	     (defined( 'bitly_clientid' ) && defined( 'bitly_secret' )) ||
+	     (defined( 'PPP_FB_APP_ID' ) && defined( 'PPP_FB_APP_SECRET' ))
 	   ) {
 		return;
 	}
 
 	$social_tokens = ppp_has_local_tokens();
 
-	if ( empty( $social_tokens ) ) {
-		if ( ! defined( 'PPP_LOCAL_TOKENS' ) ) {
+	if (empty($social_tokens)) {
+		if ( ! defined( 'PPP_LOCAL_TOKENS' )) {
 			define( 'PPP_LOCAL_TOKENS', false );
 		}
 		$social_tokens = get_transient( 'ppp_social_tokens' );
 
-		if ( ! $social_tokens ) {
+		if ( ! $social_tokens) {
 			$license = trim( get_option( '_ppp_license_key' ) );
-			$url = PPP_STORE_URL . '/?ppp-get-tokens&ppp-license-key=' . $license . '&ver=' . md5( time() . $license );
+			$url = PPP_STORE_URL.'/?ppp-get-tokens&ppp-license-key='.$license.'&ver='.md5( time().$license );
 			$response = wp_remote_get( $url, array( 'timeout' => 15, 'sslverify' => false ) );
 
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error( $response )) {
 				return false;
 			}
 
@@ -144,27 +144,27 @@ 
 		}
 
 	} else {
-		if ( ! defined ( 'PPP_LOCAL_TOKENS' ) ) {
+		if ( ! defined( 'PPP_LOCAL_TOKENS' )) {
 			define( 'PPP_LOCAL_TOKENS', true );
 		}
 
 		delete_transient( 'ppp_social_tokens' );
 
-		if ( isset( $social_tokens->options ) ) {
-			foreach ( $social_tokens->options as $constant => $value ) {
+		if (isset($social_tokens->options)) {
+			foreach ($social_tokens->options as $constant => $value) {
 
 				$constant = strtoupper( $constant );
 
-				if ( defined( $constant ) ) {
+				if (defined( $constant )) {
 					continue;
 				}
 
-				switch( $constant ) {
+				switch ($constant) {
 
 					case 'NO_AUTO_UPDATE':
 						// Avoid the call to the API to check for software updates
 						$value = is_bool( $value ) ? $value : false;
-						if ( ! defined( 'NO_AUTO_UPDATE' ) ) {
+						if ( ! defined( 'NO_AUTO_UPDATE' )) {
 							define( 'NO_AUTO_UPDATE', $value );
 						}
 						break;
@@ -174,7 +174,7 @@ 
 		}
 	}
 
-	if ( false === PPP_LOCAL_TOKENS && ! isset( $social_tokens->error ) && ( isset( $social_tokens->twitter ) || isset( $social_tokens->facebook ) || isset( $social_tokens->linkedin ) ) ) {
+	if (false === PPP_LOCAL_TOKENS && ! isset($social_tokens->error) && (isset($social_tokens->twitter) || isset($social_tokens->facebook) || isset($social_tokens->linkedin))) {
 		set_transient( 'ppp_social_tokens', $social_tokens, WEEK_IN_SECONDS );
 	}
 
@@ -187,22 +187,22 @@ 
  */
 function ppp_has_local_tokens() {
 
-	$token_file   = apply_filters( 'ppp_local_social_token_path', ppp_get_upload_path() . '/ppp-social-tokens.json' );
+	$token_file   = apply_filters( 'ppp_local_social_token_path', ppp_get_upload_path().'/ppp-social-tokens.json' );
 	$local_tokens = false;
 
-	if ( ! file_exists( $token_file ) ) {
+	if ( ! file_exists( $token_file )) {
 		return $local_tokens;
 	}
 
 	$local_tokens = json_decode( file_get_contents( $token_file ) );
 
 	// Failed to parse as JSON
-	if ( false === $local_tokens ) {
+	if (false === $local_tokens) {
 		return $local_tokens;
 	}
 
 	// No social tokens found in the format we accept or it was empty
-	if ( empty( $local_tokens ) || ( ! isset( $local_tokens->twitter ) || ! isset( $local_tokens->facebook ) || ! isset( $local_tokens->linkedin ) ) ) {
+	if (empty($local_tokens) || ( ! isset($local_tokens->twitter) || ! isset($local_tokens->facebook) || ! isset($local_tokens->linkedin))) {
 		return false;
 	}
 
@@ -220,13 +220,13 @@ 
 	global $ppp_share_settings;
 	$share_link = get_permalink( $post_id );
 
-	if ( ppp_link_tracking_enabled() ) {
+	if (ppp_link_tracking_enabled()) {
 		$share_link = ppp_generate_link_tracking( $share_link, $post_id, $name );
 	}
 
-	if ( ppp_is_shortener_enabled() && $scheduled ) {
+	if (ppp_is_shortener_enabled() && $scheduled) {
 		$shortener_name = $ppp_share_settings['shortener'];
-		$share_link = apply_filters( 'ppp_apply_shortener-' . $shortener_name, $share_link );
+		$share_link = apply_filters( 'ppp_apply_shortener-'.$shortener_name, $share_link );
 	}
 
 
@@ -241,13 +241,13 @@ 
  * @return string             The URL to post, with proper analytics applied if necessary
  */
 function ppp_generate_link_tracking( $share_link, $post_id, $name ) {
-	if ( ppp_link_tracking_enabled() ) {
+	if (ppp_link_tracking_enabled()) {
 		global $ppp_share_settings;
 		$link_tracking_type = $ppp_share_settings['analytics'];
 
 		// Given the setting name, devs can extend this and apply a filter of ppp_analytics-[setting value]
 		// to apply their own rules for link tracking
-		$share_link = apply_filters( 'ppp_analytics-' . $link_tracking_type, $share_link, $post_id, $name );
+		$share_link = apply_filters( 'ppp_analytics-'.$link_tracking_type, $share_link, $post_id, $name );
 	}
 
 	$share_link = apply_filters( 'ppp_generate_link_tracking', $share_link, $post_id, $name );
@@ -263,14 +263,14 @@ 
  * @return mixed             If a thumbnail is found returns the URL, otherwise returns false
  */
 function ppp_post_has_media( $post_id, $network, $use_media, $attachment_id = false ) {
-	if ( !$use_media || empty( $post_id ) || empty( $network ) ) {
+	if ( ! $use_media || empty($post_id) || empty($network)) {
 		return false;
 	}
 
-	$thumb_id = empty( $attachment_id ) ? get_post_thumbnail_id( $post_id ) : $attachment_id;
-	$thumb_url = wp_get_attachment_image_src( $thumb_id, 'ppp-' . $network . '-share-image', true );
+	$thumb_id = empty($attachment_id) ? get_post_thumbnail_id( $post_id ) : $attachment_id;
+	$thumb_url = wp_get_attachment_image_src( $thumb_id, 'ppp-'.$network.'-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' )) {
 		return $thumb_url[0];
 	}
 
@@ -292,14 +292,14 @@ 
 	$filename = end( $path );
 
 	// First check a social media share (thumbnail sizes)
-	$thumb_id = $wpdb->get_col( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_value LIKE '%s';", '%' . $wpdb->esc_like( $filename ) . '%' ) );
-	if ( ! empty( $thumb_id[0] ) ) {
+	$thumb_id = $wpdb->get_col( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_value LIKE '%s';", '%'.$wpdb->esc_like( $filename ).'%' ) );
+	if ( ! empty($thumb_id[0])) {
 		$attachment_id = $thumb_id[0];
 	} else {
 
 		// We didn't find one via the thumbnail sizes, try the full image URL.
 		$thumb_id = $wpdb->get_col( $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE guid='%s';", $image_url ) );
-		if ( ! empty( $thumb_id[0] ) ) {
+		if ( ! empty($thumb_id[0])) {
 			$attachment_id = $thumb_id[0];
 		}
 	}