postpromoterpro / post-promoter-pro

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/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/wpme-functions.php

Spacing +4 added lines, -4 removed lines

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

uninstall.php

Spacing +3 added lines, -3 removed lines

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

includes/linkedin-functions.php

Braces +5 added lines, -2 removed lines

@@ -429,9 +429,12 @@
 
 								<?php endforeach; ?>
 
-							<?php else: ?>
+							<?php else {
+	: ?>
 
-								<?php ppp_render_li_share_row( 1, array( 'date' => '', 'time' => '', 'text' => '', 'desc' => '', 'image' => '', 'attachment_id' => '' ) ); ?>
+								<?php ppp_render_li_share_row( 1, array( 'date' => '', 'time' => '', 'text' => '', 'desc' => '', 'image' => '', 'attachment_id' => '' ) );
+}
+?>
 
 							<?php endif; ?>
 						</tbody>

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_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,10 +60,10 @@ 
  */
 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 .= '<br />' . $ppp_social_settings['linkedin']->headline;
+		$string .= $ppp_social_settings['linkedin']->firstName.' '.$ppp_social_settings['linkedin']->lastName;
+		$string .= '<br />'.$ppp_social_settings['linkedin']->headline;
 	}
 
 	return $string;
@@ -78,22 +78,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;';
 		}
 	}
 
@@ -107,14 +107,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 );
 		}
 	}
 
@@ -130,17 +130,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' ) );
@@ -156,8 +156,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' );
 	}
@@ -183,13 +183,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' );
 	}
 }
@@ -201,12 +201,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;
 	}
 
@@ -217,14 +217,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>
@@ -239,10 +239,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 );
 	}
 
 
@@ -255,7 +255,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 );
 	}
@@ -272,7 +272,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,
@@ -313,7 +313,7 @@ 
  */
 function ppp_li_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return $tabs;
 	}
 
@@ -330,7 +330,7 @@ 
  */
 function ppp_li_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return $content;
 	}
 
@@ -368,7 +368,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 );
@@ -378,9 +378,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;
 	}
 	?>
@@ -388,11 +388,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>
@@ -408,7 +408,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,
@@ -420,19 +420,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 ); ?>
@@ -468,7 +468,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">
@@ -496,7 +496,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
@@ -516,8 +516,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>
@@ -568,26 +568,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 );
@@ -603,10 +603,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;
 	}
 
@@ -619,30 +619,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 );
 	}
 
@@ -677,18 +677,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 );
@@ -725,7 +725,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;
 	}
 
@@ -758,22 +758,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 ) );
 }
@@ -790,8 +790,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;
@@ -808,20 +808,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';
 		}
 
 	}
@@ -841,14 +841,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,
 		);
 	}
@@ -859,9 +859,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/facebook-functions.php

Braces +5 added lines, -2 removed lines

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

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 );