postpromoterpro / post-promoter-pro

includes/facebook-functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 /**
  * Register Facebook as a service
  * @param  array $services The Currently registered services
- * @return array           The services with Facebook added
+ * @return string[]           The services with Facebook added
  */
 function ppp_fb_register_service( $services = array() ) {
 	$services[] = 'fb';
@@ -190,7 +190,7 @@ 
 /**
  * Add query vars for Facebook
  * @param  array $vars Currenty Query Vars
- * @return array       Query vars array with facebook added
+ * @return string[]       Query vars array with facebook added
  */
 function ppp_fb_query_vars( $vars ) {
 	$vars[] = 'fb_access_token';
@@ -748,7 +748,7 @@ 
  * @since  2.3
  * @param  int     $post_id      Post ID
  * @param  string  $name         The cron name
- * @param  boolean $scheduled    If the item is being fired by a schedule (default, true), or retrieved for display (false)
+ * @param  boolean $is_scheduled    If the item is being fired by a schedule (default, true), or retrieved for display (false)
  * @return string                The formatted link to the post
  */
 function ppp_fb_generate_share_content( $post_id, $name, $is_scheduled = true ) {

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

includes/libs/class-wp-logging.php

Doc Comments +2 added lines, -2 removed lines

@@ -201,7 +201,7 @@ 
 	 * @since       1.0
 	 *
 	 *
-	 * @return     array
+	 * @return     boolean
 	*/
 
 	private static function valid_type( $type ) {
@@ -299,7 +299,7 @@ 
 	 * @uses 		wp_update_post()
 	 * @uses 		update_post_meta()
 	 *
-	 * @return      bool True if successful, false otherwise
+	 * @return      boolean|null True if successful, false otherwise
 	*/
 	public static function update_log( $log_data = array(), $log_meta = array() ) {
 

Indentation +10 added lines, -10 removed lines

@@ -123,7 +123,7 @@ 
 	 * @since       1.0
 	 *
 	 * @return     array
-	*/
+	 */
 
 	private static function log_types() {
 		$terms = array(
@@ -143,7 +143,7 @@ 
 	 * @uses 		register_post_type()
 	 *
 	 * @return     void
-	*/
+	 */
 
 	public function register_post_type() {
 
@@ -176,7 +176,7 @@ 
 	 * @uses 		wp_insert_term()
 	 *
 	 * @return     void
-	*/
+	 */
 
 	public function register_taxonomy() {
 
@@ -202,7 +202,7 @@ 
 	 *
 	 *
 	 * @return     array
-	*/
+	 */
 
 	private static function valid_type( $type ) {
 		return in_array( $type, self::log_types() );
@@ -221,7 +221,7 @@ 
 	 * @uses 		self::insert_log()
 	 *
 	 * @return      int The ID of the new log entry
-	*/
+	 */
 
 	public static function add( $title = '', $message = '', $parent = 0, $type = null ) {
 
@@ -250,7 +250,7 @@ 
 	 * @uses 		sanitize_key()
 	 *
 	 * @return      int The ID of the newly created log item
-	*/
+	 */
 
 	public static function insert_log( $log_data = array(), $log_meta = array() ) {
 
@@ -300,7 +300,7 @@ 
 	 * @uses 		update_post_meta()
 	 *
 	 * @return      bool True if successful, false otherwise
-	*/
+	 */
 	public static function update_log( $log_data = array(), $log_meta = array() ) {
 
 		do_action( 'wp_pre_update_log', $log_id );
@@ -337,7 +337,7 @@ 
 	 * @uses 		self::get_connected_logs()
 	 *
 	 * @return      array
-	*/
+	 */
 
 	public static function get_logs( $object_id = 0, $type = null, $paged = null ) {
 		return self::get_connected_logs( array( 'post_parent' => $object_id, 'paged' => $paged, 'log_type' => $type ) );
@@ -359,7 +359,7 @@ 
 	 * @uses 	self::valid_type()
 	 *
 	 * @return  array / false
-	*/
+	 */
 
 	public static function get_connected_logs( $args = array() ) {
 
@@ -407,7 +407,7 @@ 
 	 * @uses 	self::valid_type()
 	 *
 	 * @return  int
-	*/
+	 */
 
 	public static function get_log_count( $object_id = 0, $type = null, $meta_query = null ) {
 

Spacing +20 added lines, -20 removed lines

@@ -41,17 +41,17 @@ 
 	 * @uses $this->get_logs_to_prune()     Returns array of posts via get_posts of logs to prune
 	 * @uses $this->prune_old_logs()        Deletes the logs that we don't want anymore
 	 */
-	public function prune_logs(){
+	public function prune_logs() {
 
 		$should_we_prune = apply_filters( 'wp_logging_should_we_prune', false );
 
-		if ( $should_we_prune === false ){
+		if ($should_we_prune === false) {
 			return;
 		}
 
 		$logs_to_prune = $this->get_logs_to_prune();
 
-		if ( isset( $logs_to_prune ) && ! empty( $logs_to_prune ) ){
+		if (isset($logs_to_prune) && ! empty($logs_to_prune)) {
 			$this->prune_old_logs( $logs_to_prune );
 		}
 
@@ -69,11 +69,11 @@ 
 	 *
 	 * @filter wp_logging_force_delete_log         Allows user to override the force delete setting which bypasses the trash
 	 */
-	private function prune_old_logs( $logs ){
+	private function prune_old_logs( $logs ) {
 
 		$force = apply_filters( 'wp_logging_force_delete_log', true );
 
-		foreach( $logs as $l ){
+		foreach ($logs as $l) {
 			wp_delete_post( $l->ID, $force );
 		}
 
@@ -93,7 +93,7 @@ 
 	 * @filter wp_logging_prune_when           Users can change how long ago we are looking for logs to prune
 	 * @filter wp_logging_prune_query_args     Gives users access to change any query args for pruning
 	 */
-	private function get_logs_to_prune(){
+	private function get_logs_to_prune() {
 
 		$how_old = apply_filters( 'wp_logging_prune_when', '2 weeks ago' );
 
@@ -184,8 +184,8 @@ 
 
 		$types = self::log_types();
 
-		foreach ( $types as $type ) {
-			if( ! term_exists( $type, 'wp_log_type' ) ) {
+		foreach ($types as $type) {
+			if ( ! term_exists( $type, 'wp_log_type' )) {
 				wp_insert_term( $type, 'wp_log_type' );
 			}
 		}
@@ -270,15 +270,15 @@ 
 		$log_id = wp_insert_post( $args );
 
 		// set the log type, if any
-		if( $log_data['log_type'] && self::valid_type( $log_data['log_type'] ) ) {
+		if ($log_data['log_type'] && self::valid_type( $log_data['log_type'] )) {
 			wp_set_object_terms( $log_id, $log_data['log_type'], 'wp_log_type', false );
 		}
 
 
 		// set log meta, if any
-		if( $log_id && ! empty( $log_meta ) ) {
-			foreach( (array) $log_meta as $key => $meta ) {
-				update_post_meta( $log_id, '_wp_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				update_post_meta( $log_id, '_wp_log_'.sanitize_key( $key ), $meta );
 			}
 		}
 
@@ -316,10 +316,10 @@ 
 		// store the log entry
 		$log_id = wp_update_post( $args );
 
-		if( $log_id && ! empty( $log_meta ) ) {
-			foreach( (array) $log_meta as $key => $meta ) {
-				if( ! empty( $meta ) )
-					update_post_meta( $log_id, '_wp_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta))
+					update_post_meta( $log_id, '_wp_log_'.sanitize_key( $key ), $meta );
 			}
 		}
 
@@ -374,7 +374,7 @@ 
 
 		$query_args = wp_parse_args( $args, $defaults );
 
-		if( $query_args['log_type'] && self::valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && self::valid_type( $query_args['log_type'] )) {
 
 			$query_args['tax_query'] = array(
 				array(
@@ -388,7 +388,7 @@ 
 
 		$logs = get_posts( $query_args );
 
-		if( $logs )
+		if ($logs)
 			return $logs;
 
 		// no logs found
@@ -418,7 +418,7 @@ 
 			'post_status'    => 'publish'
 		);
 
-		if( ! empty( $type ) && self::valid_type( $type ) ) {
+		if ( ! empty($type) && self::valid_type( $type )) {
 
 			$query_args['tax_query'] = array(
 				array(
@@ -430,7 +430,7 @@ 
 
 		}
 
-		if( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 

Braces +6 added lines, -4 removed lines

@@ -318,8 +318,9 @@ 
 
 		if( $log_id && ! empty( $log_meta ) ) {
 			foreach( (array) $log_meta as $key => $meta ) {
-				if( ! empty( $meta ) )
-					update_post_meta( $log_id, '_wp_log_' . sanitize_key( $key ), $meta );
+				if( ! empty( $meta ) ) {
+									update_post_meta( $log_id, '_wp_log_' . sanitize_key( $key ), $meta );
+				}
 			}
 		}
 
@@ -388,8 +389,9 @@ 
 
 		$logs = get_posts( $query_args );
 
-		if( $logs )
-			return $logs;
+		if( $logs ) {
+					return $logs;
+		}
 
 		// no logs found
 		return false;

includes/linkedin-functions.php

Doc Comments +4 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
 /**
  * Registers LinkedIn as a service
  * @param  array $services The registered servcies
- * @return array           With LinkedIn added
+ * @return string[]           With LinkedIn added
  */
 function ppp_li_register_service( $services = array() ) {
 	$services[] = 'li';
@@ -156,7 +156,7 @@ 
 /**
  * Add query vars for Linkedin
  * @param  array $vars Currenty Query Vars
- * @return array       Query vars array with linkedin added
+ * @return string[]       Query vars array with linkedin added
  */
 function ppp_li_query_vars( $vars ) {
 	$vars[] = 'li_access_token';
@@ -286,7 +286,7 @@ 
 /**
  * Add the content box for LinkedIn in the social media settings
  * @param  array $content The existing content blocks
- * @return array          With LinkedIn
+ * @return string[]          With LinkedIn
  */
 function ppp_li_register_admin_social_content( $content ) {
 	$content[] = 'li';
@@ -497,7 +497,6 @@ 
  * @since  2.3
  * @param  int $key        The key in the array
  * @param  array  $args    Arguements for the current post's share data
- * @param  int    $post_id The post ID being edited
  * @return void
  */
 function ppp_render_li_share_row( $key, $args = array() ) {
@@ -740,7 +739,7 @@ 
  * @since  2.3
  * @param  int     $post_id      Post ID
  * @param  string  $name         The cron name
- * @param  boolean $scheduled    If the item is being fired by a schedule (default, true), or retrieved for display (false)
+ * @param  boolean $is_scheduled    If the item is being fired by a schedule (default, true), or retrieved for display (false)
  * @return string                The formatted link to the post
  */
 function ppp_li_generate_share_content( $post_id, $name, $is_scheduled = true ) {

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;

post-promoter-pro.php

Doc Comments +3 added lines, -3 removed lines

@@ -320,7 +320,7 @@ 
 
 	/**
 	 * Deactivates the license key
-	 * @return void
+	 * @return null|false
 	 */
 	public function deactivate_license() {
 		// listen for our activate button to be clicked
@@ -364,7 +364,7 @@ 
 
 	/**
 	 * Activates the license key provided
-	 * @return void
+	 * @return null|false
 	 */
 	public function activate_license() {
 		// listen for our activate button to be clicked
@@ -434,7 +434,7 @@ 
 	 *
 	 * @since  2.3
 	 * @param  array $log_types Array of log types
-	 * @return array
+	 * @return string[]
 	 */
 	public function register_log_type( $log_types ) {
 		$types[] = 'ppp_share';

Spacing +59 added lines, -59 removed lines

@@ -16,9 +16,9 @@ 
 
 define( 'PPP_STORE_URL', 'https://postpromoterpro.com' );
 define( 'PPP_PLUGIN_NAME', 'Post Promoter Pro' );
-if( !class_exists( 'EDD_SL_Plugin_Updater' ) ) {
+if ( ! class_exists( 'EDD_SL_Plugin_Updater' )) {
 	// load our custom updater
-	include( PPP_PATH . '/includes/EDD_SL_Plugin_Updater.php' );
+	include(PPP_PATH.'/includes/EDD_SL_Plugin_Updater.php');
 }
 
 class PostPromoterPro {
@@ -27,30 +27,30 @@ 
 	private function __construct() {
 		add_action( 'init', array( $this, 'ppp_loaddomain' ), 1 );
 
-		if ( ! is_callable( 'curl_init' ) ) {
+		if ( ! is_callable( 'curl_init' )) {
 			add_action( 'admin_notices', array( $this, 'no_curl' ) );
 		} else {
 			global $ppp_options, $ppp_social_settings, $ppp_share_settings;
 
-			include PPP_PATH . '/includes/general-functions.php';
-			include PPP_PATH . '/includes/share-functions.php';
-			include PPP_PATH . '/includes/cron-functions.php';
-			include PPP_PATH . '/includes/filters.php';
-			include PPP_PATH . '/includes/libs/social-loader.php';
+			include PPP_PATH.'/includes/general-functions.php';
+			include PPP_PATH.'/includes/share-functions.php';
+			include PPP_PATH.'/includes/cron-functions.php';
+			include PPP_PATH.'/includes/filters.php';
+			include PPP_PATH.'/includes/libs/social-loader.php';
 
-			if( ! class_exists( 'WP_Logging' ) ) {
-				include PPP_PATH . '/includes/libs/class-wp-logging.php';
+			if ( ! class_exists( 'WP_Logging' )) {
+				include PPP_PATH.'/includes/libs/class-wp-logging.php';
 			}
 
-			if ( is_admin() ) {
-				include PPP_PATH . '/includes/admin/upgrades.php';
-				include PPP_PATH . '/includes/admin/do-upgrades.php';
-				include PPP_PATH . '/includes/admin/actions.php';
-				include PPP_PATH . '/includes/admin/admin-pages.php';
-				include PPP_PATH . '/includes/admin/admin-ajax.php';
-				include PPP_PATH . '/includes/admin/meta-boxes.php';
-				include PPP_PATH . '/includes/admin/welcome.php';
-				include PPP_PATH . '/includes/admin/dashboard.php';
+			if (is_admin()) {
+				include PPP_PATH.'/includes/admin/upgrades.php';
+				include PPP_PATH.'/includes/admin/do-upgrades.php';
+				include PPP_PATH.'/includes/admin/actions.php';
+				include PPP_PATH.'/includes/admin/admin-pages.php';
+				include PPP_PATH.'/includes/admin/admin-ajax.php';
+				include PPP_PATH.'/includes/admin/meta-boxes.php';
+				include PPP_PATH.'/includes/admin/welcome.php';
+				include PPP_PATH.'/includes/admin/dashboard.php';
 			}
 
 			$ppp_options         = get_option( 'ppp_options' );
@@ -58,10 +58,10 @@ 
 			$ppp_share_settings  = get_option( 'ppp_share_settings' );
 
 			// Do some leg work on the social settings for Issue #257
-			if ( is_array( $ppp_share_settings ) && ! array_key_exists( 'share_on_publish', $ppp_share_settings ) ) {
-				$tw_share_on_publish = ! empty( $ppp_share_settings['twitter']['share_on_publish'] ) ? true : false;
-				$fb_share_on_publish = ! empty( $ppp_share_settings['facebook']['share_on_publish'] ) ? true : false;
-				$li_share_on_publish = ! empty( $ppp_share_settings['linkedin']['share_on_publish'] ) ? true : false;
+			if (is_array( $ppp_share_settings ) && ! array_key_exists( 'share_on_publish', $ppp_share_settings )) {
+				$tw_share_on_publish = ! empty($ppp_share_settings['twitter']['share_on_publish']) ? true : false;
+				$fb_share_on_publish = ! empty($ppp_share_settings['facebook']['share_on_publish']) ? true : false;
+				$li_share_on_publish = ! empty($ppp_share_settings['linkedin']['share_on_publish']) ? true : false;
 
 				unset(
 					$ppp_share_settings['twitter']['share_on_publish'],
@@ -70,10 +70,10 @@ 
 				);
 
 				$post_types = ppp_supported_post_types();
-				foreach ( $post_types as $key => $post_type ) {
-					$ppp_share_settings['share_on_publish'][ $key ]['twitter'] = $tw_share_on_publish;
-					$ppp_share_settings['share_on_publish'][ $key ]['facebook'] = $fb_share_on_publish;
-					$ppp_share_settings['share_on_publish'][ $key ]['linkedin'] = $li_share_on_publish;
+				foreach ($post_types as $key => $post_type) {
+					$ppp_share_settings['share_on_publish'][$key]['twitter'] = $tw_share_on_publish;
+					$ppp_share_settings['share_on_publish'][$key]['facebook'] = $fb_share_on_publish;
+					$ppp_share_settings['share_on_publish'][$key]['linkedin'] = $li_share_on_publish;
 				}
 
 				update_option( 'ppp_share_settings', $ppp_share_settings );
@@ -90,7 +90,7 @@ 
 	 * @access public
 	 */
 	public static function getInstance() {
-		if ( !self::$ppp_instance ) {
+		if ( ! self::$ppp_instance) {
 			self::$ppp_instance = new PostPromoterPro();
 		}
 
@@ -110,7 +110,7 @@ 
 	}
 
 	private function hooks() {
-		if ( is_admin() ) {
+		if (is_admin()) {
 			add_action( 'admin_init', array( $this, 'ppp_register_settings' ) );
 			add_action( 'admin_init', 'ppp_upgrade_plugin', 1 );
 
@@ -125,14 +125,14 @@ 
 			add_action( 'admin_enqueue_scripts', array( $this, 'load_styles' ), 99999 );
 			add_action( 'wp_trash_post', 'ppp_remove_scheduled_shares', 10, 1 );
 
-			if ( ppp_is_dev_or_staging() ) {
+			if (ppp_is_dev_or_staging()) {
 				add_action( 'admin_notices', array( $this, 'local_site_nag' ) );
 			}
 		}
 
 		add_action( 'init', array( $this, 'get_actions' ) );
-		add_action( 'wp_insert_post', 'ppp_schedule_share', 99, 2);
-		add_action( 'transition_post_status', 'ppp_share_on_publish', 99, 3);
+		add_action( 'wp_insert_post', 'ppp_schedule_share', 99, 2 );
+		add_action( 'transition_post_status', 'ppp_share_on_publish', 99, 3 );
 		add_action( 'init', 'ppp_add_image_sizes' );
 		add_filter( 'wp_log_types', array( $this, 'register_log_type' ), 10, 1 );
 	}
@@ -157,15 +157,15 @@ 
 
 		$allowed_pages = apply_filters( 'ppp_admin_scripts_pages', $allowed_pages, $hook );
 
-		if ( ! in_array( $hook, $allowed_pages ) ) {
+		if ( ! in_array( $hook, $allowed_pages )) {
 			return;
 		}
 
 		wp_enqueue_script( 'jquery-ui-core' );
 		wp_enqueue_script( 'jquery-ui-datepicker' );
 
-		$jquery_ui_timepicker_path = PPP_URL . 'includes/scripts/libs/jquery-ui-timepicker-addon.js';
-		wp_enqueue_script( 'ppp_timepicker_js', $jquery_ui_timepicker_path , array( 'jquery', 'jquery-ui-core' ), PPP_VERSION, true );
+		$jquery_ui_timepicker_path = PPP_URL.'includes/scripts/libs/jquery-ui-timepicker-addon.js';
+		wp_enqueue_script( 'ppp_timepicker_js', $jquery_ui_timepicker_path, array( 'jquery', 'jquery-ui-core' ), PPP_VERSION, true );
 		wp_enqueue_script( 'ppp_core_custom_js', PPP_URL.'includes/scripts/js/ppp_custom.js', 'jquery', PPP_VERSION, true );
 
 	}
@@ -176,15 +176,15 @@ 
 
 		// List of people who make it impossible to override their jQuery UI as it's in their core CSS...so only
 		// load ours if they don't exist
-		if ( ! wp_style_is( 'ot-admin-css' ) && ! wp_style_is( 'jquery-ui-css' ) ) {
+		if ( ! wp_style_is( 'ot-admin-css' ) && ! wp_style_is( 'jquery-ui-css' )) {
 			wp_enqueue_style( 'jquery-ui-css', '//ajax.googleapis.com/ajax/libs/jqueryui/1.8.2/themes/flick/jquery-ui.css' );
 		}
 
-		wp_register_style( 'ppp_admin_css', PPP_URL . 'includes/scripts/css/admin-style.css', false, PPP_VERSION );
+		wp_register_style( 'ppp_admin_css', PPP_URL.'includes/scripts/css/admin-style.css', false, PPP_VERSION );
 		wp_enqueue_style( 'ppp_admin_css' );
 
 		$sources = array_map( 'basename', (array) wp_list_pluck( $wp_styles->registered, 'src' ) );
-		if ( ! in_array( 'font-awesome.css', $sources ) || in_array( 'font-awesome.min.css', $sources )  ) {
+		if ( ! in_array( 'font-awesome.css', $sources ) || in_array( 'font-awesome.min.css', $sources )) {
 			wp_register_style( 'font-awesome', '//maxcdn.bootstrapcdn.com/font-awesome/4.6.3/css/font-awesome.min.css', false, null );
 			wp_enqueue_style( 'font-awesome' );
 		}
@@ -199,7 +199,7 @@ 
 	 * @access public
 	 */
 	public function plugin_settings_links( $links, $file ) {
-		if ( $file != PPP_FILE ) {
+		if ($file != PPP_FILE) {
 			return $links;
 		}
 
@@ -293,7 +293,7 @@ 
 	 * @access public
 	 */
 	public function ppp_loaddomain() {
-		load_plugin_textdomain( 'ppp-txt', false, dirname( plugin_basename( __FILE__ ) ) . '/languages/' );
+		load_plugin_textdomain( 'ppp-txt', false, dirname( plugin_basename( __FILE__ ) ).'/languages/' );
 	}
 
 	/**
@@ -302,24 +302,24 @@ 
 	 */
 	public function plugin_updater() {
 		global $ppp_options;
-		if ( defined( 'NO_AUTO_UPDATE' ) && true === NO_AUTO_UPDATE ) {
+		if (defined( 'NO_AUTO_UPDATE' ) && true === NO_AUTO_UPDATE) {
 			return;
 		}
 
 		$license_key = trim( get_option( '_ppp_license_key' ) );
 
-		if ( empty( $license_key ) ) {
+		if (empty($license_key)) {
 			add_action( 'admin_notices', array( $this, 'no_license_nag' ) );
 			return;
 		}
 
 		// setup the updater
 		$edd_updater = new EDD_SL_Plugin_Updater( PPP_STORE_URL, __FILE__, array(
-				'version'   => PPP_VERSION,         // current version number
-				'license'   => $license_key,        // license key (used get_option above to retrieve from DB)
-				'item_name' => PPP_PLUGIN_NAME,     // name of this plugin
-				'author'    => 'Post Promoter Pro',  // author of this plugin
-				'beta'      => ! empty( $ppp_options['enable_betas'] ) ? true : false, // If we should install beta versions
+				'version'   => PPP_VERSION, // current version number
+				'license'   => $license_key, // license key (used get_option above to retrieve from DB)
+				'item_name' => PPP_PLUGIN_NAME, // name of this plugin
+				'author'    => 'Post Promoter Pro', // author of this plugin
+				'beta'      => ! empty($ppp_options['enable_betas']) ? true : false, // If we should install beta versions
 			)
 		);
 	}
@@ -348,7 +348,7 @@ 
 	 */
 	public function local_site_nag() {
 		$dismissed = get_option( 'ppp_local_url_notice_dismissed' );
-		if ( ! empty( $dismissed ) ) {
+		if ( ! empty($dismissed)) {
 			return;
 		}
 		?>
@@ -385,10 +385,10 @@ 
 	 */
 	public function deactivate_license() {
 		// listen for our activate button to be clicked
-		if( isset( $_POST['ppp_license_deactivate'] ) ) {
+		if (isset($_POST['ppp_license_deactivate'])) {
 
 			// run a quick security check
-			if( ! check_admin_referer( 'ppp_deactivate_nonce', 'ppp_deactivate_nonce' ) ) {
+			if ( ! check_admin_referer( 'ppp_deactivate_nonce', 'ppp_deactivate_nonce' )) {
 				return;
 			}
 			// get out if we didn't click the Activate button
@@ -408,7 +408,7 @@ 
 			$response = wp_remote_get( add_query_arg( $api_params, PPP_STORE_URL ), array( 'timeout' => 15, 'sslverify' => false ) );
 
 			// make sure the response came back okay
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error( $response )) {
 				return false;
 			}
 
@@ -416,7 +416,7 @@ 
 			$license_data = json_decode( wp_remote_retrieve_body( $response ) );
 
 			// $license_data->license will be either "deactivated" or "failed"
-			if( $license_data->license == 'deactivated' ) {
+			if ($license_data->license == 'deactivated') {
 				delete_option( '_ppp_license_key_status' );
 			}
 
@@ -429,10 +429,10 @@ 
 	 */
 	public function activate_license() {
 		// listen for our activate button to be clicked
-		if( isset( $_POST['ppp_license_activate'] ) ) {
+		if (isset($_POST['ppp_license_activate'])) {
 
 			// run a quick security check
-			if( ! check_admin_referer( 'ppp_activate_nonce', 'ppp_activate_nonce' ) ) {
+			if ( ! check_admin_referer( 'ppp_activate_nonce', 'ppp_activate_nonce' )) {
 				return;
 			}
 			// get out if we didn't click the Activate button
@@ -452,7 +452,7 @@ 
 			$response = wp_remote_get( add_query_arg( $api_params, PPP_STORE_URL ), array( 'timeout' => 15, 'sslverify' => false ) );
 
 			// make sure the response came back okay
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error( $response )) {
 				return false;
 			}
 
@@ -473,7 +473,7 @@ 
 	 */
 	public function ppp_sanitize_license( $new ) {
 		$old = get_option( '_ppp_license_key' );
-		if( $old && $old != $new ) {
+		if ($old && $old != $new) {
 			delete_option( '_ppp_license_key_status' ); // new license has been entered, so must reactivate
 		}
 		return $new;
@@ -485,8 +485,8 @@ 
 	 * @return void
 	 */
 	public function get_actions() {
-		if ( isset( $_GET['ppp_action'] ) ) {
-			do_action( 'ppp_' . $_GET['ppp_action'], $_GET );
+		if (isset($_GET['ppp_action'])) {
+			do_action( 'ppp_'.$_GET['ppp_action'], $_GET );
 		}
 	}
 
@@ -520,7 +520,7 @@ 
  */
 function post_promoter_pro_activation_setup() {
 	// If the settings already exist, don't do this
-	if ( get_option( 'ppp_options' ) ) {
+	if (get_option( 'ppp_options' )) {
 		return;
 	}
 

includes/admin/admin-pages.php

Braces +5 added lines, -2 removed lines

@@ -177,8 +177,11 @@
 							<?php $post = wp_get_recent_posts( array( 'numberposts' => 1 ) ); ?>
 							<?php if ( count( $post ) > 0 ) :  ?>
 								<code><?php echo ppp_generate_link( $post[0]['ID'], 'sharedate_1_' . $post[0]['ID'], false ); ?></code></small>
-							<?php else : ?>
-								<em><?php _e( 'No posts available to generate link from.', 'ppp-txt' ); ?></em>
+							<?php else {
+	: ?>
+								<em><?php _e( 'No posts available to generate link from.', 'ppp-txt' );
+}
+?></em>
 							<?php endif; ?>
 						</p>
 					</td>

Spacing +66 added lines, -66 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -27,18 +27,18 @@ 
 						<?php _e( 'License Key', 'ppp-txt' ); ?><br /><span style="font-size: x-small;"><?php _e( 'Enter your license key', 'ppp-txt' ); ?></span>
 					</th>
 					<td>
-						<input id="ppp_license_key" name="_ppp_license_key" type="text" class="regular-text" value="<?php esc_attr_e( $license ); ?>" /><?php if ( $status !== false && $status == 'valid' ) { ?>
+						<input id="ppp_license_key" name="_ppp_license_key" type="text" class="regular-text" value="<?php esc_attr_e( $license ); ?>" /><?php if ($status !== false && $status == 'valid') { ?>
 						<span style="color:green;">&nbsp;<?php _e( 'active', 'ppp-txt' ); ?></span><?php } ?>
 					</td>
 				</tr>
 
-				<?php if ( false !== $license ) { ?>
+				<?php if (false !== $license) { ?>
 					<tr valign="top">
 						<th scope="row" valign="top">
 							<?php _e( 'Activate License', 'ppp-txt' ); ?>
 						</th>
 						<td>
-							<?php if ( $status !== false && $status == 'valid' ) { ?>
+							<?php if ($status !== false && $status == 'valid') { ?>
 								<?php wp_nonce_field( 'ppp_deactivate_nonce', 'ppp_deactivate_nonce' ); ?>
 								<input type="submit" class="button-secondary" name="ppp_license_deactivate" value="<?php _e( 'Deactivate License', 'ppp-txt' ); ?>"/>
 							<?php } else {
@@ -54,7 +54,7 @@ 
 						<span style="font-size: x-small;"><a href="#" onclick="jQuery('#ppp-text-helper').toggle(); return false;"><?php _e( 'Default Text Tips', 'ppp-txt' ); ?></a></span>
 					</th>
 					<td>
-						<?php $default_text = isset( $ppp_options['default_text'] ) ? $ppp_options['default_text'] : ''; ?>
+						<?php $default_text = isset($ppp_options['default_text']) ? $ppp_options['default_text'] : ''; ?>
 						<input type="text" class="regular-text" name="ppp_options[default_text]" value="<?php echo $default_text; ?>" placeholder="Post Title will be used if empty" size="50" />
 						<p id="ppp-text-helper" style="display: none">
 							<small>
@@ -64,7 +64,7 @@ 
 							<br />
 							<?php _e( 'Possible Replacements:', 'ppp-txt' ); ?>
 							<br />
-							<?php foreach ( ppp_get_text_tokens() as $token ) :  ?>
+							<?php foreach (ppp_get_text_tokens() as $token) :  ?>
 								<code>{<?php echo $token['token']; ?>}</code> - <?php echo $token['description']; ?><br />
 							<?php endforeach; ?>
 							</small>
@@ -76,8 +76,8 @@ 
 					<th scope="row"><?php _e( 'Post Types', 'ppp-txt' ); ?><br /><span style="font-size: x-small;"><?php _e( 'What post types do you want to schedule for?', 'ppp-txt' ); ?></span></th>
 					<td>
 						<?php $post_types = ppp_supported_post_types(); ?>
-						<?php foreach ( $post_types as $post_type => $type_data ) :  ?>
-							<?php $value = ( isset( $ppp_options['post_types'] ) && isset( $ppp_options['post_types'][ $post_type ] ) ) ? true : false; ?>
+						<?php foreach ($post_types as $post_type => $type_data) :  ?>
+							<?php $value = (isset($ppp_options['post_types']) && isset($ppp_options['post_types'][$post_type])) ? true : false; ?>
 							<input type="checkbox" name="ppp_options[post_types][<?php echo $post_type; ?>]" value="1" id="<?php echo $post_type; ?>" <?php checked( true, $value, true ); ?> />&nbsp;
 							<label for="<?php echo $post_type; ?>"><?php echo $type_data->labels->name; ?></label></br />
 						<?php endforeach; ?>
@@ -88,15 +88,15 @@ 
 					<th scope="row"><?php _e( 'Advanced', 'ppp-txt' ); ?><br /><span style="font-size: x-small;"><?php _e( 'Tools for troubleshooting and advanced usage', 'ppp-txt' ); ?></span></th>
 					<td>
 						<p>
-						<?php $debug_enabled = isset( $ppp_options['enable_debug'] ) ? true : false; ?>
+						<?php $debug_enabled = isset($ppp_options['enable_debug']) ? true : false; ?>
 						<input type="checkbox" name="ppp_options[enable_debug]" <?php checked( true, $debug_enabled, true ); ?> value="1" /> <?php _e( 'Enable Debug', 'ppp-txt' ); ?>
 						</p>
 						<p>
-						<?php $betas_enabled = isset( $ppp_options['enable_betas'] ) ? true : false; ?>
+						<?php $betas_enabled = isset($ppp_options['enable_betas']) ? true : false; ?>
 						<input type="checkbox" name="ppp_options[enable_betas]" <?php checked( true, $betas_enabled, true ); ?> value="1" /> <?php _e( 'Enable Beta Versions', 'ppp-txt' ); ?>
 						</p>
 						<p>
-						<?php $delete_on_uninstall = isset( $ppp_options['delete_on_uninstall'] ) ? true : false; ?>
+						<?php $delete_on_uninstall = isset($ppp_options['delete_on_uninstall']) ? true : false; ?>
 						<input type="checkbox" name="ppp_options[delete_on_uninstall]" <?php checked( true, $delete_on_uninstall, true ); ?> value="1" /> <?php _e( 'Delete All Data On Uninstall', 'ppp-txt' ); ?>
 						</p>
 					</td>
@@ -132,7 +132,7 @@ 
 			<?php wp_nonce_field( 'ppp-share-settings' ); ?>
 			<h3><?php _e( 'Social Media Accounts', 'ppp-txt' ); ?></h3>
 			<?php
-			require_once PPP_PATH . 'includes/admin/class-accounts-table.php';
+			require_once PPP_PATH.'includes/admin/class-accounts-table.php';
 
 			$accounts_table = new PPP_Accounts_Table();
 			$accounts_table->prepare_items();
@@ -140,7 +140,7 @@ 
 			$accounts_table->display();
 			?>
 			<table class="form-table">
-				<?php $analytics_option = isset( $ppp_share_settings['analytics'] ) ? $ppp_share_settings['analytics'] : 0; ?>
+				<?php $analytics_option = isset($ppp_share_settings['analytics']) ? $ppp_share_settings['analytics'] : 0; ?>
 				<tr valign="top">
 					<th scope="row" valign="top">
 						<?php _e( 'Analytics', 'ppp-txt' ); ?></span>
@@ -167,8 +167,8 @@ 
 						<hr />
 						<small><?php _e( 'Here is an example of what your link will look like', 'ppp-txt' ); ?>: <br />
 							<?php $post = wp_get_recent_posts( array( 'numberposts' => 1 ) ); ?>
-							<?php if ( count( $post ) > 0 ) :  ?>
-								<code><?php echo ppp_generate_link( $post[0]['ID'], 'sharedate_1_' . $post[0]['ID'], false ); ?></code></small>
+							<?php if (count( $post ) > 0) :  ?>
+								<code><?php echo ppp_generate_link( $post[0]['ID'], 'sharedate_1_'.$post[0]['ID'], false ); ?></code></small>
 							<?php else : ?>
 								<em><?php _e( 'No posts available to generate link from.', 'ppp-txt' ); ?></em>
 							<?php endif; ?>
@@ -183,34 +183,34 @@ 
 					</th>
 					<td id="ppp-share-on-publish-wrapper">
 						<?php $supported_post_types = ppp_supported_post_types(); ?>
-						<?php foreach ( $supported_post_types as $post_type => $post_type_details ) : ?>
-							<?php if ( ppp_twitter_enabled() || ppp_facebook_enabled() || ppp_linkedin_enabled() ) : ?>
+						<?php foreach ($supported_post_types as $post_type => $post_type_details) : ?>
+							<?php if (ppp_twitter_enabled() || ppp_facebook_enabled() || ppp_linkedin_enabled()) : ?>
 								<?php
-									$enabled_post_types   = ppp_allowed_post_types();
-									if ( ! in_array( $post_type, $enabled_post_types ) ) {
+									$enabled_post_types = ppp_allowed_post_types();
+									if ( ! in_array( $post_type, $enabled_post_types )) {
 										continue;
 									}
 								?>
-								<strong><?php echo $supported_post_types[ $post_type ]->label; ?></strong>
+								<strong><?php echo $supported_post_types[$post_type]->label; ?></strong>
 							<?php endif; ?>
-							<?php $post_type_settings = ! empty( $ppp_share_settings['share_on_publish'][ $post_type ] ) ? $ppp_share_settings['share_on_publish'][ $post_type ] : array(); ?>
-							<?php if ( ppp_twitter_enabled() ) : ?>
+							<?php $post_type_settings = ! empty($ppp_share_settings['share_on_publish'][$post_type]) ? $ppp_share_settings['share_on_publish'][$post_type] : array(); ?>
+							<?php if (ppp_twitter_enabled()) : ?>
 							<p>
-								<input type="checkbox" id="<?php echo $post_type; ?>-twitter-share-on-publish" value="1" <?php checked( true, ! empty( $post_type_settings['twitter'] ), true ); ?> name="ppp_share_settings[share_on_publish][<?php echo $post_type; ?>][twitter]" />
+								<input type="checkbox" id="<?php echo $post_type; ?>-twitter-share-on-publish" value="1" <?php checked( true, ! empty($post_type_settings['twitter']), true ); ?> name="ppp_share_settings[share_on_publish][<?php echo $post_type; ?>][twitter]" />
 								<label for="<?php echo $post_type; ?>-twitter-share-on-publish"><?php _e( 'Twitter', 'ppp-txt' ); ?></label>
 							</p>
 							<?php endif; ?>
 
-							<?php if ( ppp_facebook_enabled() ) : ?>
+							<?php if (ppp_facebook_enabled()) : ?>
 							<p>
-								<input type="checkbox" id="<?php echo $post_type; ?>-facebook-share-on-publish" value="1" <?php checked( true, ! empty( $post_type_settings['facebook'] ), true ); ?> name="ppp_share_settings[share_on_publish][<?php echo $post_type; ?>][facebook]" />
+								<input type="checkbox" id="<?php echo $post_type; ?>-facebook-share-on-publish" value="1" <?php checked( true, ! empty($post_type_settings['facebook']), true ); ?> name="ppp_share_settings[share_on_publish][<?php echo $post_type; ?>][facebook]" />
 								<label for="<?php echo $post_type; ?>-facebook-share-on-publish"><?php _e( 'Facebook', 'ppp-txt' ); ?></label>
 							</p>
 							<?php endif; ?>
 
-							<?php if ( ppp_linkedin_enabled() ) : ?>
+							<?php if (ppp_linkedin_enabled()) : ?>
 							<p>
-								<input type="checkbox" id="<?php echo $post_type; ?>-linkedin-share-on-publish" value="1" <?php checked( true, ! empty( $post_type_settings['linkedin'] ), true ); ?> name="ppp_share_settings[share_on_publish][<?php echo $post_type; ?>][linkedin]" />
+								<input type="checkbox" id="<?php echo $post_type; ?>-linkedin-share-on-publish" value="1" <?php checked( true, ! empty($post_type_settings['linkedin']), true ); ?> name="ppp_share_settings[share_on_publish][<?php echo $post_type; ?>][linkedin]" />
 								<label for="<?php echo $post_type; ?>-linkedin-share-on-publish"><?php _e( 'LinkedIn', 'ppp-txt' ); ?></label>
 							</p>
 							<?php endif; ?>
@@ -233,7 +233,7 @@ 
 				</tr>
 
 				<?php
-				$shortener = isset( $ppp_share_settings['shortener'] ) ? $ppp_share_settings['shortener'] : false;
+				$shortener = isset($ppp_share_settings['shortener']) ? $ppp_share_settings['shortener'] : false;
 				?>
 				<tr valign="top">
 					<th scope="row" valign="top">
@@ -246,8 +246,8 @@ 
 								<?php do_action( 'ppp_url_shorteners', $shortener ); ?>
 							</select>
 						</p>
-						<?php if ( $shortener ) : ?>
-							<?php do_action( 'ppp_shortener_settings-' . $shortener ); ?>
+						<?php if ($shortener) : ?>
+							<?php do_action( 'ppp_shortener_settings-'.$shortener ); ?>
 						<?php endif; ?>
 					</td>
 				</tr>
@@ -281,7 +281,7 @@ 
 		.wp-list-table .column-content { width: 50%; }
 	</style>
 	<?php
-	require_once PPP_PATH . 'includes/admin/class-schedule-table.php';
+	require_once PPP_PATH.'includes/admin/class-schedule-table.php';
 	$schedule_table = new PPP_Schedule_Table();
 	$schedule_table->prepare_items();
 	?>
@@ -289,7 +289,7 @@ 
 		<div id="icon-options-general" class="icon32"></div><h1><?php _e( 'Post Promoter Pro - Scheduled Shares', 'ppp-txt' ); ?></h1>
 		<?php $schedule_table->display() ?>
 	</div>
-	<?php if ( ppp_is_shortener_enabled() ) :  ?>
+	<?php if (ppp_is_shortener_enabled()) :  ?>
 	<p>
 		<small><?php _e( 'NOTICE: Schedule view does not show shortened links, they will be shortened at the time of sharing', 'ppp-txt' ); ?></small>
 	</p>
@@ -308,24 +308,24 @@ 
 	<div class="wrap">
 		<div id="icon-options-general" class="icon32"></div><h1><?php _e( 'Post Promoter Pro - System Info', 'ppp-txt' ); ?></h1>
 		<textarea style="font-family: Menlo, Monaco, monospace; white-space: pre" onclick="this.focus();this.select()" readonly cols="150" rows="35">
-	SITE_URL:                 <?php echo site_url() . "\n"; ?>
-	HOME_URL:                 <?php echo home_url() . "\n"; ?>
+	SITE_URL:                 <?php echo site_url()."\n"; ?>
+	HOME_URL:                 <?php echo home_url()."\n"; ?>
 
-	PPP Version:              <?php echo PPP_VERSION . "\n"; ?>
-	WordPress Version:        <?php echo get_bloginfo( 'version' ) . "\n"; ?>
+	PPP Version:              <?php echo PPP_VERSION."\n"; ?>
+	WordPress Version:        <?php echo get_bloginfo( 'version' )."\n"; ?>
 
 	PPP SETTINGS:
 	<?php
-	foreach ( $ppp_options as $name => $value ) {
-		if ( $value == false ) {
+	foreach ($ppp_options as $name => $value) {
+		if ($value == false) {
 			$value = 'false';
 		}
 
-		if ( $value == '1' ) {
+		if ($value == '1') {
 			$value = 'true';
 		}
 
-		echo $name . ': ' . maybe_serialize( $value ) . "\n\t";
+		echo $name.': '.maybe_serialize( $value )."\n\t";
 	}
 	?>
 
@@ -334,65 +334,65 @@ 
 	$plugins = get_plugins();
 	$active_plugins = get_option( 'active_plugins', array() );
 
-	foreach ( $plugins as $plugin_path => $plugin ) {
+	foreach ($plugins as $plugin_path => $plugin) {
 		// If the plugin isn't active, don't show it.
-		if ( ! in_array( $plugin_path, $active_plugins ) ) {
+		if ( ! in_array( $plugin_path, $active_plugins )) {
 			continue;
 		}
 
-		echo $plugin['Name']; ?>: <?php echo $plugin['Version'] ."\n\t";
+		echo $plugin['Name']; ?>: <?php echo $plugin['Version']."\n\t";
 
 	}
 	?>
 
 	CURRENT THEME:
 	<?php
-	if ( get_bloginfo( 'version' ) < '3.4' ) {
-		$theme_data = get_theme_data( get_stylesheet_directory() . '/style.css' );
-		echo $theme_data['Name'] . ': ' . $theme_data['Version'];
+	if (get_bloginfo( 'version' ) < '3.4') {
+		$theme_data = get_theme_data( get_stylesheet_directory().'/style.css' );
+		echo $theme_data['Name'].': '.$theme_data['Version'];
 	} else {
 		$theme_data = wp_get_theme();
-		echo $theme_data->Name . ': ' . $theme_data->Version;
+		echo $theme_data->Name.': '.$theme_data->Version;
 	}
 	?>
 
 
-	Multi-site:               <?php echo is_multisite() ? 'Yes' . "\n" : 'No' . "\n" ?>
+	Multi-site:               <?php echo is_multisite() ? 'Yes'."\n" : 'No'."\n" ?>
 
 	ADVANCED INFO:
-	PHP Version:              <?php echo PHP_VERSION . "\n"; ?>
-	MySQL Version:            <?php echo $wpdb->db_version() . "\n"; ?>
-	Web Server Info:          <?php echo $_SERVER['SERVER_SOFTWARE'] . "\n"; ?>
+	PHP Version:              <?php echo PHP_VERSION."\n"; ?>
+	MySQL Version:            <?php echo $wpdb->db_version()."\n"; ?>
+	Web Server Info:          <?php echo $_SERVER['SERVER_SOFTWARE']."\n"; ?>
 
-	PHP Memory Limit:         <?php echo ini_get( 'memory_limit' ) . "\n"; ?>
-	PHP Post Max Size:        <?php echo ini_get( 'post_max_size' ) . "\n"; ?>
-	PHP Time Limit:           <?php echo ini_get( 'max_execution_time' ) . "\n"; ?>
+	PHP Memory Limit:         <?php echo ini_get( 'memory_limit' )."\n"; ?>
+	PHP Post Max Size:        <?php echo ini_get( 'post_max_size' )."\n"; ?>
+	PHP Time Limit:           <?php echo ini_get( 'max_execution_time' )."\n"; ?>
 
-	WP_DEBUG:                 <?php echo defined( 'WP_DEBUG' ) ? WP_DEBUG ? 'Enabled' . "\n" : 'Disabled' . "\n" : 'Not set' . "\n" ?>
-	SCRIPT_DEBUG:             <?php echo defined( 'SCRIPT_DEBUG' ) ? SCRIPT_DEBUG ? 'Enabled' . "\n" : 'Disabled' . "\n" : 'Not set' . "\n" ?>
+	WP_DEBUG:                 <?php echo defined( 'WP_DEBUG' ) ? WP_DEBUG ? 'Enabled'."\n" : 'Disabled'."\n" : 'Not set'."\n" ?>
+	SCRIPT_DEBUG:             <?php echo defined( 'SCRIPT_DEBUG' ) ? SCRIPT_DEBUG ? 'Enabled'."\n" : 'Disabled'."\n" : 'Not set'."\n" ?>
 
-	WP Table Prefix:          <?php echo 'Length: '. strlen( $wpdb->prefix );
+	WP Table Prefix:          <?php echo 'Length: '.strlen( $wpdb->prefix );
 	echo ' Status:';
-	if ( strlen( $wpdb->prefix ) > 16 ) {echo ' ERROR: Too Long';
+	if (strlen( $wpdb->prefix ) > 16) {echo ' ERROR: Too Long';
 	} else { echo ' Acceptable';
 	} echo "\n"; ?>
 
-	Show On Front:            <?php echo get_option( 'show_on_front' ) . "\n" ?>
-	Page On Front:            <?php $id = get_option( 'page_on_front' ); echo get_the_title( $id ) . ' #' . $id . "\n" ?>
-	Page For Posts:           <?php $id = get_option( 'page_on_front' ); echo get_the_title( $id ) . ' #' . $id . "\n" ?>
+	Show On Front:            <?php echo get_option( 'show_on_front' )."\n" ?>
+	Page On Front:            <?php $id = get_option( 'page_on_front' ); echo get_the_title( $id ).' #'.$id."\n" ?>
+	Page For Posts:           <?php $id = get_option( 'page_on_front' ); echo get_the_title( $id ).' #'.$id."\n" ?>
 
-	Session:                  <?php echo isset( $_SESSION ) ? 'Enabled' : 'Disabled'; ?><?php echo "\n"; ?>
+	Session:                  <?php echo isset($_SESSION) ? 'Enabled' : 'Disabled'; ?><?php echo "\n"; ?>
 	Session Name:             <?php echo esc_html( ini_get( 'session.name' ) ); ?><?php echo "\n"; ?>
 	Cookie Path:              <?php echo esc_html( ini_get( 'session.cookie_path' ) ); ?><?php echo "\n"; ?>
 	Save Path:                <?php echo esc_html( ini_get( 'session.save_path' ) ); ?><?php echo "\n"; ?>
 	Use Cookies:              <?php echo ini_get( 'session.use_cookies' ) ? 'On' : 'Off'; ?><?php echo "\n"; ?>
 	Use Only Cookies:         <?php echo ini_get( 'session.use_only_cookies' ) ? 'On' : 'Off'; ?><?php echo "\n"; ?>
 
-	UPLOAD_MAX_FILESIZE:      <?php if ( function_exists( 'phpversion' ) ) { echo ini_get( 'upload_max_filesize' ); } ?><?php echo "\n"; ?>
-	POST_MAX_SIZE:            <?php if ( function_exists( 'phpversion' ) ) { echo ini_get( 'post_max_size' ); } ?><?php echo "\n"; ?>
+	UPLOAD_MAX_FILESIZE:      <?php if (function_exists( 'phpversion' )) { echo ini_get( 'upload_max_filesize' ); } ?><?php echo "\n"; ?>
+	POST_MAX_SIZE:            <?php if (function_exists( 'phpversion' )) { echo ini_get( 'post_max_size' ); } ?><?php echo "\n"; ?>
 	WordPress Memory Limit:   <?php echo WP_MEMORY_LIMIT; ?><?php echo "\n"; ?>
-	DISPLAY ERRORS:           <?php echo ( ini_get( 'display_errors' ) ) ? 'On (' . ini_get( 'display_errors' ) . ')' : 'N/A'; ?><?php echo "\n"; ?>
-	FSOCKOPEN:                <?php echo ( function_exists( 'fsockopen' ) ) ? __( 'Your server supports fsockopen.', 'ppp-txt' ) : __( 'Your server does not support fsockopen.', 'ppp-txt' ); ?><?php echo "\n"; ?>
+	DISPLAY ERRORS:           <?php echo (ini_get( 'display_errors' )) ? 'On ('.ini_get( 'display_errors' ).')' : 'N/A'; ?><?php echo "\n"; ?>
+	FSOCKOPEN:                <?php echo (function_exists( 'fsockopen' )) ? __( 'Your server supports fsockopen.', 'ppp-txt' ) : __( 'Your server does not support fsockopen.', 'ppp-txt' ); ?><?php echo "\n"; ?>
 		</textarea>
 	</div>
 	<?php

includes/bitly-functions.php

Spacing +12 added lines, -12 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -12,7 +12,7 @@ 
 function ppp_bitly_enabled() {
 	global $ppp_social_settings;
 
-	if ( isset( $ppp_social_settings['bitly'] ) && !empty( $ppp_social_settings['bitly'] ) ) {
+	if (isset($ppp_social_settings['bitly']) && ! empty($ppp_social_settings['bitly'])) {
 		return true;
 	}
 
@@ -37,7 +37,7 @@ 
 	global $ppp_bitly_oauth, $ppp_social_settings;
 	?>
 	<p>
-		<?php if ( !ppp_bitly_enabled() ) : ?>
+		<?php if ( ! ppp_bitly_enabled()) : ?>
 		<form autocomplete="off">
 			<input id="bitly-username" name="ppp-bitly-username" autocomplete="off" type="text" value="" placeholder="Bit.ly Username" size="25" /><br />
 			<input id="bitly-password" name="ppp-bitly-password" autocomplete="off" type="password" value="" placeholder="Bit.ly Password" size="25" /><br />
@@ -48,7 +48,7 @@ 
 			<input type="hidden" id="bitly-redirect-url" value="<?php echo admin_url( 'admin.php?page=ppp-social-settings' ); ?>" />
 		</form>
 		<?php endif; ?>
-		<?php if ( ppp_bitly_enabled() ) : ?>
+		<?php if (ppp_bitly_enabled()) : ?>
 			<div class="ppp-social-profile ppp-bitly-profile">
 				<div class="ppp-bitly-info">
 					<?php _e( 'Signed in as', 'ppp-txt' ); ?>:<br /><?php echo $ppp_social_settings['bitly']['login']; ?><br />
@@ -71,15 +71,15 @@ 
 function ppp_disconnect_bitly() {
 	global $ppp_social_settings;
 	$ppp_social_settings = get_option( 'ppp_social_settings' );
-	if ( isset( $ppp_social_settings['bitly'] ) ) {
-		unset( $ppp_social_settings['bitly'] );
+	if (isset($ppp_social_settings['bitly'])) {
+		unset($ppp_social_settings['bitly']);
 		update_option( 'ppp_social_settings', $ppp_social_settings );
 	}
 }
 add_action( 'ppp_disconnect-bitly', 'ppp_disconnect_bitly', 10 );
 
 function ppp_set_bitly_token_constants( $social_tokens ) {
-	if ( !empty( $social_tokens ) && property_exists( $social_tokens, 'bitly' ) ) {
+	if ( ! empty($social_tokens) && property_exists( $social_tokens, 'bitly' )) {
 		define( 'bitly_clientid', $social_tokens->bitly->client_id );
 		define( 'bitly_secret', $social_tokens->bitly->client_secret );
 	}
@@ -97,7 +97,7 @@ 
 	$result = $ppp_bitly_oauth->ppp_make_bitly_link( $link );
 	$result = json_decode( $result, true );
 
-	if ( isset ( $result['status_code'] ) && $result['status_code'] == 200 ) {
+	if (isset ($result['status_code']) && $result['status_code'] == 200) {
 		return $result['data']['url'];
 	} else {
 		return $link;
@@ -115,13 +115,13 @@ 
 	$url = 'https://api-ssl.bitly.com/oauth/access_token';
 	$username = $_POST['username'];
 	$password = $_POST['password'];
-	$headers = array( 'Authorization' => 'Basic ' . base64_encode( bitly_clientid . ':' . bitly_secret ) );
-	$body = 'grant_type=password&username=' . $username . '&password=' . $password;
+	$headers = array( 'Authorization' => 'Basic '.base64_encode( bitly_clientid.':'.bitly_secret ) );
+	$body = 'grant_type=password&username='.$username.'&password='.$password;
 	$result = wp_remote_post( $url, array( 'headers' => $headers, 'body' => $body ) );
 	$body = wp_remote_retrieve_body( $result );
 	$data = json_decode( $body );
 
-	if ( isset( $data->access_token ) ) {
+	if (isset($data->access_token)) {
 		global $ppp_social_settings;
 		$ppp_social_settings['bitly']['access_token'] = $data->access_token;
 
@@ -131,7 +131,7 @@ 
 
 		update_option( 'ppp_social_settings', $ppp_social_settings );
 		echo 1;
-	} elseif ( $data->status_txt == 'INVALID_LOGIN' ) {
+	} elseif ($data->status_txt == 'INVALID_LOGIN') {
 		echo 'INVALID_LOGIN';
 	} else {
 		echo 0;

includes/cron-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -225,7 +225,7 @@
  * @since  2.3
  * @param  string $date The Date
  * @param  string $time The time
- * @return long         A timestamp
+ * @return integer         A timestamp
  */
 function ppp_generate_timestamp( $date = false, $time = false ) {
 	// Just in case we need this, let's set it once

Spacing +24 added lines, -24 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -14,19 +14,19 @@ 
 function ppp_schedule_share( $post_id, $post ) {
 	$allowed_post_types = ppp_allowed_post_types();
 
-	if ( ! isset( $_POST['post_status'] ) || ! in_array( $post->post_type, $allowed_post_types ) ) {
+	if ( ! isset($_POST['post_status']) || ! in_array( $post->post_type, $allowed_post_types )) {
 		return;
 	}
 
 	ppp_remove_scheduled_shares( $post_id );
 
-	if( ( $_POST['post_status'] == 'publish' && $_POST['original_post_status'] != 'publish' ) || // From anything to published
-		( $_POST['post_status'] == 'future' && $_POST['original_post_status'] == 'future' ) || // Updating a future post
-		( $_POST['post_status'] == 'publish' && $_POST['original_post_status'] == 'publish' ) ) { // Updating an already published post
+	if (($_POST['post_status'] == 'publish' && $_POST['original_post_status'] != 'publish') || // From anything to published
+		($_POST['post_status'] == 'future' && $_POST['original_post_status'] == 'future') || // Updating a future post
+		($_POST['post_status'] == 'publish' && $_POST['original_post_status'] == 'publish')) { // Updating an already published post
 
 		$timestamps = ppp_get_timestamps( $post_id );
 
-		foreach ( $timestamps as $timestamp => $name ) {
+		foreach ($timestamps as $timestamp => $name) {
 			$timestamp = substr( $timestamp, 0, strlen( $timestamp ) - 3 );
 			wp_schedule_single_event( $timestamp, 'ppp_share_post_event', array( $post_id, $name ) );
 		}
@@ -45,7 +45,7 @@ 
 
 	$current_item_shares = ppp_get_shceduled_crons( $post_id );
 
-	foreach ( $current_item_shares as $share ) {
+	foreach ($current_item_shares as $share) {
 		wp_clear_scheduled_hook( 'ppp_share_post_event', array( $post_id, $share['args'][1] ) );
 	}
 
@@ -70,19 +70,19 @@ 
 	$all_crons = get_option( 'cron' );
 	$ppp_crons = array();
 
-	foreach ( $all_crons as $timestamp => $cron ) {
-		if ( ! isset( $cron['ppp_share_post_event'] ) ) {
+	foreach ($all_crons as $timestamp => $cron) {
+		if ( ! isset($cron['ppp_share_post_event'])) {
 			continue;
 		}
 
-		foreach ( $cron['ppp_share_post_event'] as $key => $single_event ) {
+		foreach ($cron['ppp_share_post_event'] as $key => $single_event) {
 			$name_parts = explode( '_', $single_event['args'][1] );
-			if ( false !== $post_id && $post_id != $name_parts[2] ) {
+			if (false !== $post_id && $post_id != $name_parts[2]) {
 				continue;
 			}
 
 			$single_event['timestamp'] = $timestamp;
-			$ppp_crons[ $key ]         = $single_event;
+			$ppp_crons[$key]         = $single_event;
 		}
 
 	}
@@ -111,26 +111,26 @@ 
  *
  */
 function ppp_has_cron_within( $time = 0, $within = 0 ) {
-	if ( empty( $time ) ) {
+	if (empty($time)) {
 		$time = current_time( 'timestamp' );
 	}
 
-	if ( empty( $within ) ) {
+	if (empty($within)) {
 		$within = ppp_get_default_conflict_window();
 	}
 
 	$crons = ppp_get_shceduled_crons();
 
-	if ( empty( $crons ) ) {
+	if (empty($crons)) {
 		return false;
 	}
 
 	$scheduled_times = wp_list_pluck( $crons, 'timestamp' );
 
 	$found_time = false;
-	foreach ( $scheduled_times as $key => $scheduled_time ) {
+	foreach ($scheduled_times as $key => $scheduled_time) {
 		$found_time = ppp_is_time_within( $scheduled_time, $time, $within );
-		if ( $found_time ) {
+		if ($found_time) {
 			break;
 		}
 	}
@@ -152,7 +152,7 @@ 
 	$min = $target_time - $within;
 	$max = $target_time + $within;
 
-	return ( ( $time >= $min ) && ( $time <= $max ) );
+	return (($time >= $min) && ($time <= $max));
 }
 
 /**
@@ -220,7 +220,7 @@ 
  */
 function ppp_unschedule_shares( $new_status, $old_status, $post ) {
 
-	if ( ( $old_status == 'publish' || $old_status == 'future' ) && ( $new_status != 'publish' && $new_status != 'future' ) ) {
+	if (($old_status == 'publish' || $old_status == 'future') && ($new_status != 'publish' && $new_status != 'future')) {
 		ppp_remove_scheduled_shares( $post->ID );
 	}
 
@@ -239,11 +239,11 @@ 
 	// Just in case we need this, let's set it once
 	$current_time = current_time( 'timestamp' );
 
-	if ( empty( $date ) ) {
+	if (empty($date)) {
 		$date = date( 'm/d/Y', $current_time );
 	}
 
-	if ( empty( $time ) ) {
+	if (empty($time)) {
 		$time = date( 'h:ia', $current_time );
 	}
 
@@ -252,15 +252,15 @@ 
 	$minutes    = (int) substr( $share_time[1], 0, 2 );
 	$ampm       = strtolower( substr( $share_time[1], -2 ) );
 
-	if ( $ampm == 'pm' && $hours != 12 ) {
+	if ($ampm == 'pm' && $hours != 12) {
 		$hours = $hours + 12;
 	}
 
-	if ( $ampm == 'am' && $hours == 12 ) {
+	if ($ampm == 'am' && $hours == 12) {
 		$hours = 00;
 	}
 
-	$offset    = (int) -( get_option( 'gmt_offset' ) );
+	$offset    = (int) -(get_option( 'gmt_offset' ));
 	$hours     = $hours + $offset;
 	$date      = explode( '/', $date );
 	$timestamp = mktime( (int) $hours, (int) $minutes, 0, (int) $date[0], (int) $date[1], (int) $date[2] );

includes/EDD_SL_Plugin_Updater.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	 * @uses api_request()
 	 *
 	 * @param array   $_transient_data Update array build by WordPress.
-	 * @return array Modified update array with custom plugin data.
+	 * @return stdClass Modified update array with custom plugin data.
 	 */
 	public function check_update( $_transient_data ) {
 

Braces +3 added lines, -1 removed lines

@@ -1,7 +1,9 @@
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Allows plugins to use their own update API.

Spacing +77 added lines, -77 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' )) exit;
 
 /**
  * Allows plugins to use their own update API.
@@ -38,11 +38,11 @@ 
 		$this->name        = plugin_basename( $_plugin_file );
 		$this->slug        = basename( $_plugin_file, '.php' );
 		$this->version     = $_api_data['version'];
-		$this->wp_override = isset( $_api_data['wp_override'] ) ? (bool) $_api_data['wp_override'] : false;
-		$this->beta        = ! empty( $this->api_data['beta'] ) ? true : false;
-		$this->cache_key   = md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) );
+		$this->wp_override = isset($_api_data['wp_override']) ? (bool) $_api_data['wp_override'] : false;
+		$this->beta        = ! empty($this->api_data['beta']) ? true : false;
+		$this->cache_key   = md5( serialize( $this->slug.$this->api_data['license'].$this->beta ) );
 
-		$edd_plugin_data[ $this->slug ] = $this->api_data;
+		$edd_plugin_data[$this->slug] = $this->api_data;
 
 		// Set up hooks.
 		$this->init();
@@ -60,8 +60,8 @@ 
 
 		add_filter( 'pre_set_site_transient_update_plugins', array( $this, 'check_update' ) );
 		add_filter( 'plugins_api', array( $this, 'plugins_api_filter' ), 10, 3 );
-		remove_action( 'after_plugin_row_' . $this->name, 'wp_plugin_update_row', 10 );
-		add_action( 'after_plugin_row_' . $this->name, array( $this, 'show_update_notification' ), 10, 2 );
+		remove_action( 'after_plugin_row_'.$this->name, 'wp_plugin_update_row', 10 );
+		add_action( 'after_plugin_row_'.$this->name, array( $this, 'show_update_notification' ), 10, 2 );
 		add_action( 'admin_init', array( $this, 'show_changelog' ) );
 
 	}
@@ -83,37 +83,37 @@ 
 
 		global $pagenow;
 
-		if ( ! is_object( $_transient_data ) ) {
+		if ( ! is_object( $_transient_data )) {
 			$_transient_data = new stdClass;
 		}
 
-		if ( 'plugins.php' == $pagenow && is_multisite() ) {
+		if ('plugins.php' == $pagenow && is_multisite()) {
 			return $_transient_data;
 		}
 
-		if ( ! empty( $_transient_data->response ) && ! empty( $_transient_data->response[ $this->name ] ) && false === $this->wp_override ) {
+		if ( ! empty($_transient_data->response) && ! empty($_transient_data->response[$this->name]) && false === $this->wp_override) {
 			return $_transient_data;
 		}
 
 		$version_info = $this->get_cached_version_info();
 
-		if ( false === $version_info ) {
+		if (false === $version_info) {
 			$version_info = $this->api_request( 'plugin_latest_version', array( 'slug' => $this->slug, 'beta' => $this->beta ) );
 
 			$this->set_version_info_cache( $version_info );
 
 		}
 
-		if ( false !== $version_info && is_object( $version_info ) && isset( $version_info->new_version ) ) {
+		if (false !== $version_info && is_object( $version_info ) && isset($version_info->new_version)) {
 
-			if ( version_compare( $this->version, $version_info->new_version, '<' ) ) {
+			if (version_compare( $this->version, $version_info->new_version, '<' )) {
 
-				$_transient_data->response[ $this->name ] = $version_info;
+				$_transient_data->response[$this->name] = $version_info;
 
 			}
 
 			$_transient_data->last_checked           = current_time( 'timestamp' );
-			$_transient_data->checked[ $this->name ] = $this->version;
+			$_transient_data->checked[$this->name] = $this->version;
 
 		}
 
@@ -128,19 +128,19 @@ 
 	 */
 	public function show_update_notification( $file, $plugin ) {
 
-		if ( is_network_admin() ) {
+		if (is_network_admin()) {
 			return;
 		}
 
-		if( ! current_user_can( 'update_plugins' ) ) {
+		if ( ! current_user_can( 'update_plugins' )) {
 			return;
 		}
 
-		if( ! is_multisite() ) {
+		if ( ! is_multisite()) {
 			return;
 		}
 
-		if ( $this->name != $file ) {
+		if ($this->name != $file) {
 			return;
 		}
 
@@ -151,56 +151,56 @@ 
 
 		$update_cache = is_object( $update_cache ) ? $update_cache : new stdClass();
 
-		if ( empty( $update_cache->response ) || empty( $update_cache->response[ $this->name ] ) ) {
+		if (empty($update_cache->response) || empty($update_cache->response[$this->name])) {
 
 			$version_info = $this->get_cached_version_info();
 
-			if ( false === $version_info ) {
+			if (false === $version_info) {
 				$version_info = $this->api_request( 'plugin_latest_version', array( 'slug' => $this->slug, 'beta' => $this->beta ) );
 
 				$this->set_version_info_cache( $version_info );
 			}
 
-			if ( ! is_object( $version_info ) ) {
+			if ( ! is_object( $version_info )) {
 				return;
 			}
 
-			if ( version_compare( $this->version, $version_info->new_version, '<' ) ) {
+			if (version_compare( $this->version, $version_info->new_version, '<' )) {
 
-				$update_cache->response[ $this->name ] = $version_info;
+				$update_cache->response[$this->name] = $version_info;
 
 			}
 
 			$update_cache->last_checked = current_time( 'timestamp' );
-			$update_cache->checked[ $this->name ] = $this->version;
+			$update_cache->checked[$this->name] = $this->version;
 
 			set_site_transient( 'update_plugins', $update_cache );
 
 		} else {
 
-			$version_info = $update_cache->response[ $this->name ];
+			$version_info = $update_cache->response[$this->name];
 
 		}
 
 		// Restore our filter
 		add_filter( 'pre_set_site_transient_update_plugins', array( $this, 'check_update' ) );
 
-		if ( ! empty( $update_cache->response[ $this->name ] ) && version_compare( $this->version, $version_info->new_version, '<' ) ) {
+		if ( ! empty($update_cache->response[$this->name]) && version_compare( $this->version, $version_info->new_version, '<' )) {
 
 			// build a plugin list row, with update notification
 			$wp_list_table = _get_list_table( 'WP_Plugins_List_Table' );
 			# <tr class="plugin-update-tr"><td colspan="' . $wp_list_table->get_column_count() . '" class="plugin-update colspanchange">
-			echo '<tr class="plugin-update-tr" id="' . $this->slug . '-update" data-slug="' . $this->slug . '" data-plugin="' . $this->slug . '/' . $file . '">';
+			echo '<tr class="plugin-update-tr" id="'.$this->slug.'-update" data-slug="'.$this->slug.'" data-plugin="'.$this->slug.'/'.$file.'">';
 			echo '<td colspan="3" class="plugin-update colspanchange">';
 			echo '<div class="update-message notice inline notice-warning notice-alt">';
 
-			$changelog_link = self_admin_url( 'index.php?edd_sl_action=view_plugin_changelog&plugin=' . $this->name . '&slug=' . $this->slug . '&TB_iframe=true&width=772&height=911' );
+			$changelog_link = self_admin_url( 'index.php?edd_sl_action=view_plugin_changelog&plugin='.$this->name.'&slug='.$this->slug.'&TB_iframe=true&width=772&height=911' );
 
-			if ( empty( $version_info->download_link ) ) {
+			if (empty($version_info->download_link)) {
 				printf(
 					__( 'There is a new version of %1$s available. %2$sView version %3$s details%4$s.', 'easy-digital-downloads' ),
 					esc_html( $version_info->name ),
-					'<a target="_blank" class="thickbox" href="' . esc_url( $changelog_link ) . '">',
+					'<a target="_blank" class="thickbox" href="'.esc_url( $changelog_link ).'">',
 					esc_html( $version_info->new_version ),
 					'</a>'
 				);
@@ -208,10 +208,10 @@ 
 				printf(
 					__( 'There is a new version of %1$s available. %2$sView version %3$s details%4$s or %5$supdate now%6$s.', 'easy-digital-downloads' ),
 					esc_html( $version_info->name ),
-					'<a target="_blank" class="thickbox" href="' . esc_url( $changelog_link ) . '">',
+					'<a target="_blank" class="thickbox" href="'.esc_url( $changelog_link ).'">',
 					esc_html( $version_info->new_version ),
 					'</a>',
-					'<a href="' . esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) ) .'">',
+					'<a href="'.esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ).$this->name, 'upgrade-plugin_'.$this->name ) ).'">',
 					'</a>'
 				);
 			}
@@ -234,13 +234,13 @@ 
 	 */
 	public function plugins_api_filter( $_data, $_action = '', $_args = null ) {
 
-		if ( $_action != 'plugin_information' ) {
+		if ($_action != 'plugin_information') {
 
 			return $_data;
 
 		}
 
-		if ( ! isset( $_args->slug ) || ( $_args->slug != $this->slug ) ) {
+		if ( ! isset($_args->slug) || ($_args->slug != $this->slug)) {
 
 			return $_data;
 
@@ -255,20 +255,20 @@ 
 			)
 		);
 
-		$cache_key = 'edd_api_request_' . md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) );
+		$cache_key = 'edd_api_request_'.md5( serialize( $this->slug.$this->api_data['license'].$this->beta ) );
 
 		// Get the transient where we store the api request for this plugin for 24 hours
 		$edd_api_request_transient = $this->get_cached_version_info( $cache_key );
 
 		//If we have no transient-saved value, run the API, set a fresh transient with the API value, and return that value too right now.
-		if ( empty( $edd_api_request_transient ) ) {
+		if (empty($edd_api_request_transient)) {
 
 			$api_response = $this->api_request( 'plugin_information', $to_send );
 
 			// Expires in 3 hours
 			$this->set_version_info_cache( $api_response, $cache_key );
 
-			if ( false !== $api_response ) {
+			if (false !== $api_response) {
 				$_data = $api_response;
 			}
 
@@ -277,20 +277,20 @@ 
 		}
 
 		// Convert sections into an associative array, since we're getting an object, but Core expects an array.
-		if ( isset( $_data->sections ) && ! is_array( $_data->sections ) ) {
+		if (isset($_data->sections) && ! is_array( $_data->sections )) {
 			$new_sections = array();
-			foreach ( $_data->sections as $key => $key ) {
-				$new_sections[ $key ] = $key;
+			foreach ($_data->sections as $key => $key) {
+				$new_sections[$key] = $key;
 			}
 
 			$_data->sections = $new_sections;
 		}
 
 		// Convert banners into an associative array, since we're getting an object, but Core expects an array.
-		if ( isset( $_data->banners ) && ! is_array( $_data->banners ) ) {
+		if (isset($_data->banners) && ! is_array( $_data->banners )) {
 			$new_banners = array();
-			foreach ( $_data->banners as $key => $key ) {
-				$new_banners[ $key ] = $key;
+			foreach ($_data->banners as $key => $key) {
+				$new_banners[$key] = $key;
 			}
 
 			$_data->banners = $new_banners;
@@ -308,7 +308,7 @@ 
 	 */
 	public function http_request_args( $args, $url ) {
 		// If it is an https request and we are performing a package download, disable ssl verification
-		if ( strpos( $url, 'https://' ) !== false && strpos( $url, 'edd_action=package_download' ) ) {
+		if (strpos( $url, 'https://' ) !== false && strpos( $url, 'edd_action=package_download' )) {
 			$args['sslverify'] = false;
 		}
 		return $args;
@@ -331,44 +331,44 @@ 
 
 		$data = array_merge( $this->api_data, $_data );
 
-		if ( $data['slug'] != $this->slug ) {
+		if ($data['slug'] != $this->slug) {
 			return;
 		}
 
-		if( $this->api_url == trailingslashit (home_url() ) ) {
+		if ($this->api_url == trailingslashit( home_url() )) {
 			return false; // Don't allow a plugin to ping itself
 		}
 
 		$api_params = array(
 			'edd_action' => 'get_version',
-			'license'    => ! empty( $data['license'] ) ? $data['license'] : '',
-			'item_name'  => isset( $data['item_name'] ) ? $data['item_name'] : false,
-			'item_id'    => isset( $data['item_id'] ) ? $data['item_id'] : false,
-			'version'    => isset( $data['version'] ) ? $data['version'] : false,
+			'license'    => ! empty($data['license']) ? $data['license'] : '',
+			'item_name'  => isset($data['item_name']) ? $data['item_name'] : false,
+			'item_id'    => isset($data['item_id']) ? $data['item_id'] : false,
+			'version'    => isset($data['version']) ? $data['version'] : false,
 			'slug'       => $data['slug'],
 			'author'     => $data['author'],
 			'url'        => home_url(),
-			'beta'       => ! empty( $data['beta'] ),
+			'beta'       => ! empty($data['beta']),
 		);
 
 		$request = wp_remote_post( $this->api_url, array( 'timeout' => 15, 'sslverify' => false, 'body' => $api_params ) );
 
-		if ( ! is_wp_error( $request ) ) {
+		if ( ! is_wp_error( $request )) {
 			$request = json_decode( wp_remote_retrieve_body( $request ) );
 		}
 
-		if ( $request && isset( $request->sections ) ) {
+		if ($request && isset($request->sections)) {
 			$request->sections = maybe_unserialize( $request->sections );
 		} else {
 			$request = false;
 		}
 
-		if ( $request && isset( $request->banners ) ) {
+		if ($request && isset($request->banners)) {
 			$request->banners = maybe_unserialize( $request->banners );
 		}
 
-		if( ! empty( $request->sections ) ) {
-			foreach( $request->sections as $key => $section ) {
+		if ( ! empty($request->sections)) {
+			foreach ($request->sections as $key => $section) {
 				$request->$key = (array) $section;
 			}
 		}
@@ -380,54 +380,54 @@ 
 
 		global $edd_plugin_data;
 
-		if( empty( $_REQUEST['edd_sl_action'] ) || 'view_plugin_changelog' != $_REQUEST['edd_sl_action'] ) {
+		if (empty($_REQUEST['edd_sl_action']) || 'view_plugin_changelog' != $_REQUEST['edd_sl_action']) {
 			return;
 		}
 
-		if( empty( $_REQUEST['plugin'] ) ) {
+		if (empty($_REQUEST['plugin'])) {
 			return;
 		}
 
-		if( empty( $_REQUEST['slug'] ) ) {
+		if (empty($_REQUEST['slug'])) {
 			return;
 		}
 
-		if( ! current_user_can( 'update_plugins' ) ) {
+		if ( ! current_user_can( 'update_plugins' )) {
 			wp_die( __( 'You do not have permission to install plugin updates', 'easy-digital-downloads' ), __( 'Error', 'easy-digital-downloads' ), array( 'response' => 403 ) );
 		}
 
-		$data         = $edd_plugin_data[ $_REQUEST['slug'] ];
-		$beta         = ! empty( $data['beta'] ) ? true : false;
-		$cache_key    = md5( 'edd_plugin_' . sanitize_key( $_REQUEST['plugin'] ) . '_' . $beta . '_version_info' );
+		$data         = $edd_plugin_data[$_REQUEST['slug']];
+		$beta         = ! empty($data['beta']) ? true : false;
+		$cache_key    = md5( 'edd_plugin_'.sanitize_key( $_REQUEST['plugin'] ).'_'.$beta.'_version_info' );
 		$version_info = $this->get_cached_version_info( $cache_key );
 
-		if( false === $version_info ) {
+		if (false === $version_info) {
 
 			$api_params = array(
 				'edd_action' => 'get_version',
-				'item_name'  => isset( $data['item_name'] ) ? $data['item_name'] : false,
-				'item_id'    => isset( $data['item_id'] ) ? $data['item_id'] : false,
+				'item_name'  => isset($data['item_name']) ? $data['item_name'] : false,
+				'item_id'    => isset($data['item_id']) ? $data['item_id'] : false,
 				'slug'       => $_REQUEST['slug'],
 				'author'     => $data['author'],
 				'url'        => home_url(),
-				'beta'       => ! empty( $data['beta'] )
+				'beta'       => ! empty($data['beta'])
 			);
 
 			$request = wp_remote_post( $this->api_url, array( 'timeout' => 15, 'sslverify' => false, 'body' => $api_params ) );
 
-			if ( ! is_wp_error( $request ) ) {
+			if ( ! is_wp_error( $request )) {
 				$version_info = json_decode( wp_remote_retrieve_body( $request ) );
 			}
 
 
-			if ( ! empty( $version_info ) && isset( $version_info->sections ) ) {
+			if ( ! empty($version_info) && isset($version_info->sections)) {
 				$version_info->sections = maybe_unserialize( $version_info->sections );
 			} else {
 				$version_info = false;
 			}
 
-			if( ! empty( $version_info ) ) {
-				foreach( $version_info->sections as $key => $section ) {
+			if ( ! empty($version_info)) {
+				foreach ($version_info->sections as $key => $section) {
 					$version_info->$key = (array) $section;
 				}
 			}
@@ -436,8 +436,8 @@ 
 
 		}
 
-		if( ! empty( $version_info ) && isset( $version_info->sections['changelog'] ) ) {
-			echo '<div style="background:#fff;padding:10px;">' . $version_info->sections['changelog'] . '</div>';
+		if ( ! empty($version_info) && isset($version_info->sections['changelog'])) {
+			echo '<div style="background:#fff;padding:10px;">'.$version_info->sections['changelog'].'</div>';
 		}
 
 		exit;
@@ -445,13 +445,13 @@ 
 
 	public function get_cached_version_info( $cache_key = '' ) {
 
-		if( empty( $cache_key ) ) {
+		if (empty($cache_key)) {
 			$cache_key = $this->cache_key;
 		}
 
 		$cache = get_option( $cache_key );
 
-		if( empty( $cache['timeout'] ) || current_time( 'timestamp' ) > $cache['timeout'] ) {
+		if (empty($cache['timeout']) || current_time( 'timestamp' ) > $cache['timeout']) {
 			return false; // Cache is expired
 		}
 
@@ -461,7 +461,7 @@ 
 
 	public function set_version_info_cache( $value = '', $cache_key = '' ) {
 
-		if( empty( $cache_key ) ) {
+		if (empty($cache_key)) {
 			$cache_key = $this->cache_key;
 		}
 

includes/filters.php

Spacing +13 added lines, -13 removed lines

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