postpromoterpro / post-promoter-pro

includes/EDD_SL_Plugin_Updater.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,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.
 	 */
 	function check_update( $_transient_data ) {
 

Spacing +41 added lines, -41 removed lines

@@ -50,7 +50,7 @@ 
 		add_filter( 'pre_set_site_transient_update_plugins', array( $this, 'check_update' ) );
 		add_filter( 'plugins_api', array( $this, 'plugins_api_filter' ), 10, 3 );
 
-		add_action( 'after_plugin_row_' . $this->name, array( $this, 'show_update_notification' ), 10, 2 );
+		add_action( 'after_plugin_row_'.$this->name, array( $this, 'show_update_notification' ), 10, 2 );
 	}
 
 	/**
@@ -70,30 +70,30 @@ 
 
 		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 ] ) ) {
+		if (empty($_transient_data->response) || empty($_transient_data->response[$this->name])) {
 
 			$version_info = $this->api_request( 'plugin_latest_version', array( 'slug' => $this->slug ) );
 
-			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)) {
 
 				$this->did_check = true;
 
-				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 = time();
-				$_transient_data->checked[ $this->name ] = $this->version;
+				$_transient_data->checked[$this->name] = $this->version;
 
 			}
 
@@ -110,15 +110,15 @@ 
 	 */
 	public function show_update_notification( $file, $plugin ) {
 
-		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;
 		}
 
@@ -127,12 +127,12 @@ 
 
 		$update_cache = get_site_transient( 'update_plugins' );
 
-		if ( ! is_object( $update_cache ) || empty( $update_cache->response ) || empty( $update_cache->response[ $this->name ] ) ) {
+		if ( ! is_object( $update_cache ) || empty($update_cache->response) || empty($update_cache->response[$this->name])) {
 
-			$cache_key    = md5( 'edd_plugin_' .sanitize_key( $this->name ) . '_version_info' );
+			$cache_key    = md5( 'edd_plugin_'.sanitize_key( $this->name ).'_version_info' );
 			$version_info = get_transient( $cache_key );
 
-			if( false === $version_info ) {
+			if (false === $version_info) {
 
 				$version_info = $this->api_request( 'plugin_latest_version', array( 'slug' => $this->slug ) );
 
@@ -140,39 +140,39 @@ 
 			}
 
 
-			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 = time();
-			$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' );
-			echo '<tr class="plugin-update-tr"><td colspan="' . $wp_list_table->get_column_count() . '" class="plugin-update colspanchange"><div class="update-message">';
+			echo '<tr class="plugin-update-tr"><td colspan="'.$wp_list_table->get_column_count().'" class="plugin-update colspanchange"><div class="update-message">';
 
-			$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. <a target="_blank" class="thickbox" href="%2$s">View version %3$s details</a>.', 'edd' ),
 					esc_html( $version_info->name ),
@@ -185,7 +185,7 @@ 
 					esc_html( $version_info->name ),
 					esc_url( $changelog_link ),
 					esc_html( $version_info->new_version ),
-					esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) )
+					esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ).$this->name, 'upgrade-plugin_'.$this->name ) )
 				);
 			}
 
@@ -207,13 +207,13 @@ 
 	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;
 
@@ -230,7 +230,7 @@ 
 
 		$api_response = $this->api_request( 'plugin_information', $to_send );
 
-		if ( false !== $api_response ) {
+		if (false !== $api_response) {
 			$_data = $api_response;
 		}
 
@@ -247,7 +247,7 @@ 
 	 */
 	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;
@@ -270,21 +270,21 @@ 
 
 		$data = array_merge( $this->api_data, $_data );
 
-		if ( $data['slug'] != $this->slug )
+		if ($data['slug'] != $this->slug)
 			return;
 
-		if ( empty( $data['license'] ) )
+		if (empty($data['license']))
 			return;
 
-		if( $this->api_url == home_url() ) {
+		if ($this->api_url == home_url()) {
 			return false; // Don't allow a plugin to ping itself
 		}
 
 		$api_params = array(
 			'edd_action' => 'get_version',
 			'license'    => $data['license'],
-			'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'       => $data['slug'],
 			'author'     => $data['author'],
 			'url'        => home_url()
@@ -292,11 +292,11 @@ 
 
 		$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;
@@ -308,26 +308,26 @@ 
 	public function show_changelog() {
 
 
-		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', 'edd' ), __( 'Error', 'edd' ), array( 'response' => 403 ) );
 		}
 
 		$response = $this->api_request( 'plugin_latest_version', array( 'slug' => $_REQUEST['slug'] ) );
 
-		if( $response && isset( $response->sections['changelog'] ) ) {
-			echo '<div style="background:#fff;padding:10px;">' . $response->sections['changelog'] . '</div>';
+		if ($response && isset($response->sections['changelog'])) {
+			echo '<div style="background:#fff;padding:10px;">'.$response->sections['changelog'].'</div>';
 		}
 
 

Braces +6 added lines, -4 removed lines

@@ -270,11 +270,13 @@
 
 		$data = array_merge( $this->api_data, $_data );
 
-		if ( $data['slug'] != $this->slug )
-			return;
+		if ( $data['slug'] != $this->slug ) {
+					return;
+		}
 
-		if ( empty( $data['license'] ) )
-			return;
+		if ( empty( $data['license'] ) ) {
+					return;
+		}
 
 		if( $this->api_url == home_url() ) {
 			return false; // Don't allow a plugin to ping itself

includes/facebook-functions.php

Doc Comments +2 added lines, -2 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';

Spacing +60 added lines, -60 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;
@@ -83,13 +83,13 @@ 
  */
 function ppp_fb_account_list_actions( $string = '' ) {
 
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		global $ppp_facebook_oauth, $ppp_social_settings;
 		$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;';
 	}
 
 	return $string;
@@ -103,28 +103,28 @@ 
  */
 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">';
-			$string .= '<option value="me">' . __( 'Me', 'ppp-txt' ) . '</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 .= '<option value="me">'.__( 'Me', 'ppp-txt' ).'</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 );
 		}
 	}
 
@@ -138,7 +138,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 );
 	}
@@ -152,17 +152,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' ) );
@@ -179,8 +179,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' );
 	}
@@ -206,13 +206,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 ( current_time( 'timestamp' ) > $refresh_date ) {
+	if (current_time( 'timestamp' ) > $refresh_date) {
 		add_action( 'admin_notices', 'ppp_facebook_refresh_notice' );
 	}
 }
@@ -224,7 +224,7 @@ 
  */
 function ppp_facebook_refresh_notice() {
 
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return;
 	}
 
@@ -235,14 +235,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 = absint( round( ( $ppp_social_settings['facebook']->expires_on - current_time( 'timestamp' ) ) / DAY_IN_SECONDS ) );
+	$days_left = absint( round( ($ppp_social_settings['facebook']->expires_on - current_time( 'timestamp' )) / DAY_IN_SECONDS ) );
 	?>
 	<div class="update-nag">
-		<?php if ( $days_left > 0 ): ?>
+		<?php if ($days_left > 0): ?>
 			<p><strong>Post Promoter Pro: </strong><?php printf( __( 'Your Facebook authentication expires in within %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>
@@ -277,7 +277,7 @@ 
  */
 function ppp_fb_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return $tabs;
 	}
 
@@ -294,7 +294,7 @@ 
  */
 function ppp_fb_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( ! ppp_facebook_enabled() ) {
+	if ( ! ppp_facebook_enabled()) {
 		return $content;
 	}
 
@@ -310,33 +310,33 @@ 
  */
 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 );
 
 	$show_share_on_publish = false;
 
-	$share_by_default      = empty( $ppp_share_settings['facebook']['share_on_publish'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['facebook']['share_on_publish']) ? 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;
 	}
 
 	?>
 	<p>
-	<?php $disabled = ( $post->post_status === 'publish' && time() > strtotime( $post->post_date ) ) ? true : false; ?>
-	<input <?php if ( $disabled ): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_fb_share_on_publish" id="ppp_fb_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />&nbsp;
+	<?php $disabled = ($post->post_status === 'publish' && time() > strtotime( $post->post_date )) ? true : false; ?>
+	<input <?php if ($disabled): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_fb_share_on_publish" id="ppp_fb_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />&nbsp;
 		<label for="ppp_fb_share_on_publish"><?php _e( 'Share this post on Facebook at the time of publishing?', 'ppp-txt' ); ?></label>
-		<p class="ppp_share_on_publish_text"<?php if ( false === $show_share_on_publish ) : ?> style="display: none;"<?php endif; ?>>
+		<p class="ppp_share_on_publish_text"<?php if (false === $show_share_on_publish) : ?> style="display: none;"<?php endif; ?>>
 			<?php _e( 'Link Message', 'ppp-txt' ); ?>:<br />
 				<input
-				<?php if ( $disabled ): ?>readonly<?php endif; ?>
+				<?php if ($disabled): ?>readonly<?php endif; ?>
 				class="ppp-share-text"
 				type="text"
 				placeholder="<?php echo esc_attr( $default_text ); ?>"
 				name="_ppp_fb_share_on_publish_title"
-				<?php if ( isset( $ppp_share_on_publish_title ) ) {?>value="<?php echo esc_attr( $ppp_share_on_publish_title ); ?>"<?php ;}?>
+				<?php if (isset($ppp_share_on_publish_title)) {?>value="<?php echo esc_attr( $ppp_share_on_publish_title ); ?>"<?php; }?>
 			/>
 				<span id="ppp-fb-image">
 					<br /><br />
@@ -357,12 +357,12 @@ 
  */
 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'] : '0';
-	$ppp_share_on_publish_title = ( isset( $_REQUEST['_ppp_fb_share_on_publish_title'] ) ) ? $_REQUEST['_ppp_fb_share_on_publish_title'] : '';
+	$ppp_fb_share_on_publish = (isset($_REQUEST['_ppp_fb_share_on_publish'])) ? $_REQUEST['_ppp_fb_share_on_publish'] : '0';
+	$ppp_share_on_publish_title = (isset($_REQUEST['_ppp_fb_share_on_publish_title'])) ? $_REQUEST['_ppp_fb_share_on_publish_title'] : '';
 
 	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 );
@@ -379,37 +379,37 @@ 
 function ppp_fb_share_on_publish( $new_status, $old_status, $post ) {
 	global $ppp_options;
 	$from_meta = get_post_meta( $post->ID, '_ppp_fb_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_fb_share_on_publish'] );
+	$from_post = isset($_POST['_ppp_fb_share_on_publish']);
 
-	if ( empty( $from_meta ) && empty( $from_post ) ) {
+	if (empty($from_meta) && empty($from_post)) {
 		return;
 	}
 
 	// Determine if we're seeing the share on publish in meta or $_POST
-	if ( $from_meta && !$from_post ) {
+	if ($from_meta && ! $from_post) {
 		$ppp_share_on_publish_title = get_post_meta( $post->ID, '_ppp_fb_share_on_publish_title', true );
 	} else {
-		$ppp_share_on_publish_title = isset( $_POST['_ppp_fb_share_on_publish_title'] ) ? $_POST['_ppp_fb_share_on_publish_title'] : '';
+		$ppp_share_on_publish_title = isset($_POST['_ppp_fb_share_on_publish_title']) ? $_POST['_ppp_fb_share_on_publish_title'] : '';
 	}
 
 	$thumbnail = ppp_post_has_media( $post->ID, 'fb', true );
 
-	$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
-	$share_title = ( isset( $ppp_share_on_publish_title ) && !empty( $ppp_share_on_publish_title ) ) ? $ppp_share_on_publish_title : $default_title;
+	$share_title = (isset($ppp_share_on_publish_title) && ! empty($ppp_share_on_publish_title)) ? $ppp_share_on_publish_title : $default_title;
 
 	// If the content is still empty, just use the post title
-	$share_title = ( isset( $share_title ) && !empty( $share_title ) ) ? $share_title : get_the_title( $post->ID );
+	$share_title = (isset($share_title) && ! empty($share_title)) ? $share_title : get_the_title( $post->ID );
 
 	$share_title = apply_filters( 'ppp_share_content', $share_title, array( 'post_id' => $post->ID ) );
 	$share_link = ppp_generate_link( $post->ID, $name, true );
 
 	$status['facebook'] = ppp_fb_share( $share_link, $share_title, $thumbnail );
 
-	if ( isset( $ppp_options['enable_debug'] ) && $ppp_options['enable_debug'] == '1' ) {
-		update_post_meta( $post->ID, '_ppp-' . $name . '-status', $status );
+	if (isset($ppp_options['enable_debug']) && $ppp_options['enable_debug'] == '1') {
+		update_post_meta( $post->ID, '_ppp-'.$name.'-status', $status );
 	}
 }
 add_action( 'ppp_share_on_publish', 'ppp_fb_share_on_publish', 10, 3 );
@@ -424,9 +424,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 );

includes/general-functions.php

Doc Comments +2 added lines, -1 removed lines

@@ -7,7 +7,7 @@ 
 
 /**
  * Checks to see if a session is set and it's appropriate to start one, and starts it if necessary
- * @return void
+ * @return boolean
  */
 function ppp_maybe_start_session() {
 	$ret = false;
@@ -73,6 +73,7 @@ 
 
 /**
  * Strips slashes and html_entities_decode for sending to the networks.
+ * @param string $string
  */
 function ppp_entities_and_slashes( $string ) {
 	return stripslashes( html_entity_decode( $string, ENT_COMPAT, 'UTF-8' ) );

Spacing +19 added lines, -19 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -11,7 +11,7 @@ 
  */
 function ppp_maybe_start_session() {
 	$ret = false;
-	if ( ( is_admin() || ( defined( 'DOING_CRON' ) && DOING_CRON ) ) && !isset( $_SESSION ) && !defined( 'DOING_AJAX' ) ) {
+	if ((is_admin() || (defined( 'DOING_CRON' ) && DOING_CRON)) && ! isset($_SESSION) && ! defined( 'DOING_AJAX' )) {
 		$ret = session_start();
 	}
 
@@ -26,8 +26,8 @@ 
 	global $ppp_share_settings;
 	$result = false;
 
-	if ( isset( $ppp_share_settings['analytics'] ) && $ppp_share_settings['analytics'] !== 'none' ) {
-		$result =  true;
+	if (isset($ppp_share_settings['analytics']) && $ppp_share_settings['analytics'] !== 'none') {
+		$result = true;
 	}
 
 	return apply_filters( 'ppp_is_link_tracking_enabled', $result );
@@ -68,7 +68,7 @@ 
 function ppp_is_shortener_enabled() {
 	global $ppp_share_settings;
 
-	return ( isset( $ppp_share_settings['shortener'] ) && !empty( $ppp_share_settings['shortener'] ) && $ppp_share_settings != '-1' );
+	return (isset($ppp_share_settings['shortener']) && ! empty($ppp_share_settings['shortener']) && $ppp_share_settings != '-1');
 }
 
 /**
@@ -107,16 +107,16 @@ 
 function ppp_should_save( $post_id, $post ) {
 	$ret = true;
 
-	if ( empty( $_POST ) ) {
+	if (empty($_POST)) {
 		$ret = false;
 	}
 
-	if ( wp_is_post_revision( $post_id ) ) {
+	if (wp_is_post_revision( $post_id )) {
 		$ret = false;
 	}
 
 	global $ppp_options;
-	if ( !isset( $ppp_options['post_types'] ) || !is_array( $ppp_options['post_types'] ) || !array_key_exists( $post->post_type, $ppp_options['post_types'] ) ) {
+	if ( ! isset($ppp_options['post_types']) || ! is_array( $ppp_options['post_types'] ) || ! array_key_exists( $post->post_type, $ppp_options['post_types'] )) {
 		$ret = false;
 	}
 
@@ -132,7 +132,7 @@ 
 function ppp_set_uploads_dir() {
 	$upload_path = ppp_get_upload_path();
 
-	if ( false === get_transient( 'ppp_check_protection_files' ) ) {
+	if (false === get_transient( 'ppp_check_protection_files' )) {
 
 		// Make sure the /ppp folder is created
 		wp_mkdir_p( $upload_path );
@@ -141,22 +141,22 @@ 
 		$rules  = "Options -Indexes\n";
 		$rules .= "deny from all\n";
 
-		$htaccess_exists = file_exists( $upload_path . '/.htaccess' );
+		$htaccess_exists = file_exists( $upload_path.'/.htaccess' );
 
-		if ( $htaccess_exists ) {
-			$contents = @file_get_contents( $upload_path . '/.htaccess' );
-			if ( $contents !== $rules || ! $contents ) {
+		if ($htaccess_exists) {
+			$contents = @file_get_contents( $upload_path.'/.htaccess' );
+			if ($contents !== $rules || ! $contents) {
 				// Update the .htaccess rules if they don't match
-				@file_put_contents( $upload_path . '/.htaccess', $rules );
+				@file_put_contents( $upload_path.'/.htaccess', $rules );
 			}
-		} elseif( wp_is_writable( $upload_path ) ) {
+		} elseif (wp_is_writable( $upload_path )) {
 			// Create the file if it doesn't exist
-			@file_put_contents( $upload_path . '/.htaccess', $rules );
+			@file_put_contents( $upload_path.'/.htaccess', $rules );
 		}
 
 		// Top level blank index.php
-		if ( ! file_exists( $upload_path . '/index.php' ) && wp_is_writable( $upload_path ) ) {
-			@file_put_contents( $upload_path . '/index.php', '<?php' . PHP_EOL . '// Silence is golden.' );
+		if ( ! file_exists( $upload_path.'/index.php' ) && wp_is_writable( $upload_path )) {
+			@file_put_contents( $upload_path.'/index.php', '<?php'.PHP_EOL.'// Silence is golden.' );
 		}
 
 		// Check for the files once per day
@@ -173,5 +173,5 @@ 
  */
 function ppp_get_upload_path() {
 	$wp_upload_dir = wp_upload_dir();
-	return $wp_upload_dir['basedir'] . '/ppp';
+	return $wp_upload_dir['basedir'].'/ppp';
 }

includes/linkedin-functions.php

Doc Comments +3 added lines, -3 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';
@@ -263,7 +263,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';

Spacing +54 added lines, -54 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;
@@ -77,13 +77,13 @@ 
  */
 function ppp_li_account_list_actions( $string = '' ) {
 
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		global $ppp_linkedin_oauth, $ppp_social_settings;
 		$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;';
 	}
 
 	return $string;
@@ -96,14 +96,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 );
 		}
 	}
 
@@ -119,17 +119,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' ) );
@@ -145,8 +145,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' );
 	}
@@ -172,13 +172,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 ( current_time( 'timestamp' ) > $refresh_date ) {
+	if (current_time( 'timestamp' ) > $refresh_date) {
 		add_action( 'admin_notices', 'ppp_linkedin_refresh_notice' );
 	}
 }
@@ -190,7 +190,7 @@ 
  */
 function ppp_linkedin_refresh_notice() {
 
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return;
 	}
 
@@ -201,14 +201,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 = absint( round( ( $ppp_social_settings['linkedin']->expires_on - current_time( 'timestamp' ) ) / DAY_IN_SECONDS ) );
+	$days_left = absint( round( ($ppp_social_settings['linkedin']->expires_on - current_time( 'timestamp' )) / DAY_IN_SECONDS ) );
 	?>
 	<div class="update-nag">
-		<?php if ( $days_left > 0 ): ?>
+		<?php if ($days_left > 0): ?>
 			<p><strong>Post Promoter Pro: </strong><?php printf( __( 'Your LinkedIn authentication expires in within %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>
@@ -221,7 +221,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 );
 	}
@@ -238,7 +238,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,
@@ -279,7 +279,7 @@ 
  */
 function ppp_li_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return $tabs;
 	}
 
@@ -296,7 +296,7 @@ 
  */
 function ppp_li_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( ! ppp_linkedin_enabled() ) {
+	if ( ! ppp_linkedin_enabled()) {
 		return $content;
 	}
 
@@ -321,7 +321,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 );
@@ -329,32 +329,32 @@ 
 
 	$show_share_on_publish = false;
 
-	$share_by_default      = empty( $ppp_share_settings['linkedin']['share_on_publish'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['linkedin']['share_on_publish']) ? 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;
 	}
 	?>
 	<p>
-	<?php $disabled = ( $post->post_status === 'publish' && time() > strtotime( $post->post_date ) ) ? true : false; ?>
-	<input <?php if ( $disabled ): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_li_share_on_publish" id="ppp_li_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />&nbsp;
+	<?php $disabled = ($post->post_status === 'publish' && time() > strtotime( $post->post_date )) ? true : false; ?>
+	<input <?php if ($disabled): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_li_share_on_publish" id="ppp_li_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />&nbsp;
 		<label for="ppp_li_share_on_publish"><?php _e( 'Share this post on LinkedIn at the time of publishing?', 'ppp-txt' ); ?></label>
-		<p class="ppp_share_on_publish_text"<?php if ( false === $show_share_on_publish ) : ?> style="display: none;"<?php endif; ?>
+		<p class="ppp_share_on_publish_text"<?php if (false === $show_share_on_publish) : ?> style="display: none;"<?php endif; ?>
 				<span class="left" id="ppp-li-image">
 					<?php echo get_the_post_thumbnail( $post->ID, 'ppp-li-share-image', array( 'class' => 'left' ) ); ?>
 				</span>
 				<?php _e( 'Link Title', 'ppp-txt' ); ?>:<br />
 				<input
-				<?php if ( $disabled ): ?>readonly<?php endif; ?>
+				<?php if ($disabled): ?>readonly<?php endif; ?>
 				class="ppp-share-text"
 				type="text"
 				placeholder="<?php echo esc_attr( $default_text ); ?>"
 				name="_ppp_li_share_on_publish_title"
-				<?php if ( isset( $ppp_share_on_publish_title ) ) {?>value="<?php echo esc_attr( $ppp_share_on_publish_title ); ?>"<?php ;}?>
+				<?php if (isset($ppp_share_on_publish_title)) {?>value="<?php echo esc_attr( $ppp_share_on_publish_title ); ?>"<?php; }?>
 			/>
 			<br />
 			<?php _e( 'Link Description', 'ppp-txt' ); ?>:<br />
-			<textarea <?php if ( $disabled ): ?>readonly<?php endif; ?> name="_ppp_li_share_on_publish_desc"><?php echo esc_attr( $ppp_share_on_publish_desc ); ?></textarea>
+			<textarea <?php if ($disabled): ?>readonly<?php endif; ?> name="_ppp_li_share_on_publish_desc"><?php echo esc_attr( $ppp_share_on_publish_desc ); ?></textarea>
 			<br /><?php _e( 'Note: If set, the Featured image will be attached to this share', 'ppp-txt' ); ?>
 		</p>
 	</p>
@@ -370,13 +370,13 @@ 
  */
 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'] : '0';
-	$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_li_share_on_publish = (isset($_REQUEST['_ppp_li_share_on_publish'])) ? $_REQUEST['_ppp_li_share_on_publish'] : '0';
+	$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'] : '';
 
 	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 );
@@ -396,39 +396,39 @@ 
 function ppp_li_share_on_publish( $new_status, $old_status, $post ) {
 	global $ppp_options;
 	$from_meta = get_post_meta( $post->ID, '_ppp_li_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_li_share_on_publish'] );
+	$from_post = isset($_POST['_ppp_li_share_on_publish']);
 
-	if ( empty( $from_meta ) && empty( $from_post ) ) {
+	if (empty($from_meta) && empty($from_post)) {
 		return;
 	}
 
 	// Determine if we're seeing the share on publish in meta or $_POST
-	if ( $from_meta && !$from_post ) {
+	if ($from_meta && ! $from_post) {
 		$ppp_share_on_publish_title = get_post_meta( $post->ID, '_ppp_li_share_on_publish_title', true );
 		$ppp_share_on_publish_desc = get_post_meta( $post->ID, '_ppp_li_share_on_publish_desc', true );
 	} else {
-		$ppp_share_on_publish_title = isset( $_POST['_ppp_li_share_on_publish_title'] ) ? $_POST['_ppp_li_share_on_publish_title'] : '';
-		$ppp_share_on_publish_desc = isset( $_POST['_ppp_li_share_on_publish_desc'] ) ? $_POST['_ppp_li_share_on_publish_desc'] : false;
+		$ppp_share_on_publish_title = isset($_POST['_ppp_li_share_on_publish_title']) ? $_POST['_ppp_li_share_on_publish_title'] : '';
+		$ppp_share_on_publish_desc = isset($_POST['_ppp_li_share_on_publish_desc']) ? $_POST['_ppp_li_share_on_publish_desc'] : false;
 	}
 
 	$thumbnail = ppp_post_has_media( $post->ID, 'li', true );
 
-	$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
-	$share_title = ( isset( $ppp_share_on_publish_title ) && !empty( $ppp_share_on_publish_title ) ) ? $ppp_share_on_publish_title : $default_title;
+	$share_title = (isset($ppp_share_on_publish_title) && ! empty($ppp_share_on_publish_title)) ? $ppp_share_on_publish_title : $default_title;
 
 	// If the content is still empty, just use the post title
-	$share_title = ( isset( $share_title ) && !empty( $share_title ) ) ? $share_title : get_the_title( $post->ID );
+	$share_title = (isset($share_title) && ! empty($share_title)) ? $share_title : get_the_title( $post->ID );
 
 	$share_title = apply_filters( 'ppp_share_content', $share_title, array( 'post_id' => $post->ID ) );
 	$share_link = ppp_generate_link( $post->ID, $name, true );
 
 	$status['linkedin'] = ppp_li_share( $share_title, $ppp_share_on_publish_desc, $share_link, $thumbnail );
 
-	if ( isset( $ppp_options['enable_debug'] ) && $ppp_options['enable_debug'] == '1' ) {
-		update_post_meta( $post->ID, '_ppp-' . $name . '-status', $status );
+	if (isset($ppp_options['enable_debug']) && $ppp_options['enable_debug'] == '1') {
+		update_post_meta( $post->ID, '_ppp-'.$name.'-status', $status );
 	}
 }
 add_action( 'ppp_share_on_publish', 'ppp_li_share_on_publish', 10, 3 );

includes/share-functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
  * @param string $old_status The old status of the poast
  * @param  object $post      The Post Object
  *
- * @return void
+ * @return null|boolean
  */
 function ppp_share_on_publish( $new_status, $old_status, $post ) {
 	// don't publish password protected posts
@@ -137,7 +137,7 @@ 
 
 /**
  * Get the Social Share Tokens from the API
- * @return void
+ * @return null|false
  */
 function ppp_set_social_tokens() {
 	if ( ( defined( 'PPP_TW_CONSUMER_KEY' ) && defined( 'PPP_TW_CONSUMER_SECRET' ) ) ||

Indentation +3 added lines, -3 removed lines

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

Spacing +47 added lines, -47 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -16,17 +16,17 @@ 
  */
 function ppp_share_on_publish( $new_status, $old_status, $post ) {
 	// don't publish password protected posts
-	if ( '' !== $post->post_password ) {
+	if ('' !== $post->post_password) {
 		return;
 	}
 
-	if ( $new_status == 'publish' && $old_status != 'publish' ) {
+	if ($new_status == 'publish' && $old_status != 'publish') {
 		global $ppp_options;
 
-		$allowed_post_types = isset( $ppp_options['post_types'] ) ? $ppp_options['post_types'] : array();
+		$allowed_post_types = isset($ppp_options['post_types']) ? $ppp_options['post_types'] : array();
 		$allowed_post_types = apply_filters( 'ppp_schedule_share_post_types', $allowed_post_types );
 
-		if ( !isset( $post->post_status ) || !array_key_exists( $post->post_type, $allowed_post_types ) ) {
+		if ( ! isset($post->post_status) || ! array_key_exists( $post->post_type, $allowed_post_types )) {
 			return false;
 		}
 
@@ -43,17 +43,17 @@ 
  */
 function ppp_get_timestamps( $post_id ) {
 	// Make the timestamp in the users' timezone, b/c that makes more sense
-	$offset = (int) -( get_option( 'gmt_offset' ) );
+	$offset = (int) -(get_option( 'gmt_offset' ));
 
 	$ppp_tweets = get_post_meta( $post_id, '_ppp_tweets', true );
 
-	if ( empty( $ppp_tweets ) ) {
+	if (empty($ppp_tweets)) {
 		$ppp_tweets = array();
 	}
 
 	$times = array();
-	foreach ( $ppp_tweets as $key => $data ) {
-		if ( ! array_filter( $data ) ) {
+	foreach ($ppp_tweets as $key => $data) {
+		if ( ! array_filter( $data )) {
 			continue;
 		}
 
@@ -62,11 +62,11 @@ 
 		$minutes = (int) substr( $share_time[1], 0, 2 );
 		$ampm = strtolower( substr( $share_time[1], -2 ) );
 
-		if ( $ampm == 'pm' && $hours != 12 ) {
+		if ($ampm == 'pm' && $hours != 12) {
 			$hours = $hours + 12;
 		}
 
-		if ( $ampm == 'am' && $hours == 12 ) {
+		if ($ampm == 'am' && $hours == 12) {
 			$hours = 00;
 		}
 
@@ -74,8 +74,8 @@ 
 		$date      = explode( '/', $data['date'] );
 		$timestamp = mktime( $hours, $minutes, 0, $date[0], $date[1], $date[2] );
 
-		if ( $timestamp > current_time( 'timestamp', 1 ) ) { // Make sure the timestamp we're getting is in the future
-			$times[strtotime( date_i18n( 'd-m-Y H:i:s', $timestamp , true ) )] = 'sharedate_' . $key . '_' . $post_id;
+		if ($timestamp > current_time( 'timestamp', 1 )) { // Make sure the timestamp we're getting is in the future
+			$times[strtotime( date_i18n( 'd-m-Y H:i:s', $timestamp, true ) )] = 'sharedate_'.$key.'_'.$post_id;
 		}
 
 	}
@@ -94,21 +94,21 @@ 
 
 	// If we've already started to share this, don't share it again.
 	// Compensates for wp-cron's race conditions
-	if ( get_transient( 'ppp_sharing' . $name ) === 'true' ) {
+	if (get_transient( 'ppp_sharing'.$name ) === 'true') {
 		return;
 	}
 
 	// For 60 seconds, don't allow another share to go for this post
-	set_transient( 'ppp_sharing' . $name, 'true', 60 );
+	set_transient( 'ppp_sharing'.$name, 'true', 60 );
 
 	$share_message = ppp_tw_build_share_message( $post_id, $name );
 
 	$name_parts    = explode( '_', $name );
 	$post_meta     = get_post_meta( $post_id, '_ppp_tweets', true );
 	$this_share    = $post_meta[$name_parts[1]];
-	$attachment_id = isset( $this_share['attachment_id'] ) ? $this_share['attachment_id'] : false;
+	$attachment_id = isset($this_share['attachment_id']) ? $this_share['attachment_id'] : false;
 
-	if ( empty( $attachment_id ) && ! empty( $this_share['image'] ) ) {
+	if (empty($attachment_id) && ! empty($this_share['image'])) {
 		$media = $this_share['image'];
 	} else {
 		$media = ppp_post_has_media( $post_id, 'tw', ppp_tw_use_media( $post_id, $name_parts[1] ), $attachment_id );
@@ -118,20 +118,20 @@ 
 
 	$status['twitter'] = ppp_send_tweet( $share_message, $post_id, $media );
 
-	if ( ! empty( $status['twitter']->id_str ) ) {
+	if ( ! empty($status['twitter']->id_str)) {
 		$post      = get_post( $post_id );
 		$author_id = $post->post_author;
 		$author_rt = get_user_meta( $author_id, '_ppp_share_scheduled', true );
 
-		if ( $author_rt ) {
+		if ($author_rt) {
 			$twitter_user = new PPP_Twitter_User( $author_id );
 			$twitter_user->retweet( $status['twitter']->id_str );
 		}
 
 	}
 
-	if ( isset( $ppp_options['enable_debug'] ) && $ppp_options['enable_debug'] == '1' ) {
-		update_post_meta( $post_id, '_ppp-' . $name . '-status', $status );
+	if (isset($ppp_options['enable_debug']) && $ppp_options['enable_debug'] == '1') {
+		update_post_meta( $post_id, '_ppp-'.$name.'-status', $status );
 	}
 }
 
@@ -140,26 +140,26 @@ 
  * @return void
  */
 function ppp_set_social_tokens() {
-	if ( ( defined( 'PPP_TW_CONSUMER_KEY' ) && defined( 'PPP_TW_CONSUMER_SECRET' ) ) ||
-	     ( defined( 'LINKEDIN_KEY' ) && defined( 'LINKEDIN_SECRET' ) ) ||
-	     ( defined( 'bitly_clientid' ) && defined( 'bitly_secret' ) ) ||
-	     ( defined( 'PPP_FB_APP_ID' ) && defined( 'PPP_FB_APP_SECRET' ) )
+	if ((defined( 'PPP_TW_CONSUMER_KEY' ) && defined( 'PPP_TW_CONSUMER_SECRET' )) ||
+	     (defined( 'LINKEDIN_KEY' ) && defined( 'LINKEDIN_SECRET' )) ||
+	     (defined( 'bitly_clientid' ) && defined( 'bitly_secret' )) ||
+	     (defined( 'PPP_FB_APP_ID' ) && defined( 'PPP_FB_APP_SECRET' ))
 	   ) {
 		return;
 	}
 
 	$social_tokens = ppp_has_local_tokens();
 
-	if ( false === $social_tokens ) {
+	if (false === $social_tokens) {
 		define( 'PPP_LOCAL_TOKENS', false );
 		$social_tokens = get_transient( 'ppp_social_tokens' );
 
-		if ( ! $social_tokens ) {
+		if ( ! $social_tokens) {
 			$license = trim( get_option( '_ppp_license_key' ) );
-			$url = PPP_STORE_URL . '/?ppp-get-tokens&ppp-license-key=' . $license . '&ver=' . md5( time() . $license );
+			$url = PPP_STORE_URL.'/?ppp-get-tokens&ppp-license-key='.$license.'&ver='.md5( time().$license );
 			$response = wp_remote_get( $url, array( 'timeout' => 15, 'sslverify' => false ) );
 
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error( $response )) {
 				return false;
 			}
 
@@ -171,16 +171,16 @@ 
 		define( 'PPP_LOCAL_TOKENS', true );
 		delete_transient( 'ppp_social_tokens' );
 
-		if ( isset( $social_tokens->options ) ) {
-			foreach ( $social_tokens->options as $constant => $value ) {
+		if (isset($social_tokens->options)) {
+			foreach ($social_tokens->options as $constant => $value) {
 
 				$constant = strtoupper( $constant );
 
-				if ( defined( $constant ) ) {
+				if (defined( $constant )) {
 					continue;
 				}
 
-				switch( $constant ) {
+				switch ($constant) {
 
 					case 'NO_AUTO_UPDATE':
 						// Avoid the call to the API to check for software updates
@@ -193,7 +193,7 @@ 
 		}
 	}
 
-	if ( false === PPP_LOCAL_TOKENS && ! isset( $social_tokens->error ) && ( isset( $social_tokens->twitter ) || isset( $social_tokens->facebook ) || isset( $social_tokens->linkedin ) ) ) {
+	if (false === PPP_LOCAL_TOKENS && ! isset($social_tokens->error) && (isset($social_tokens->twitter) || isset($social_tokens->facebook) || isset($social_tokens->linkedin))) {
 		set_transient( 'ppp_social_tokens', $social_tokens, WEEK_IN_SECONDS );
 	}
 
@@ -206,22 +206,22 @@ 
  */
 function ppp_has_local_tokens() {
 
-	$token_file   = apply_filters( 'ppp_local_social_token_path', ppp_get_upload_path() . '/ppp-social-tokens.json' );
+	$token_file   = apply_filters( 'ppp_local_social_token_path', ppp_get_upload_path().'/ppp-social-tokens.json' );
 	$local_tokens = false;
 
-	if ( ! file_exists( $token_file ) ) {
+	if ( ! file_exists( $token_file )) {
 		return $local_tokens;
 	}
 
 	$local_tokens = json_decode( file_get_contents( $token_file ) );
 
 	// Failed to parse as JSON
-	if ( false === $local_tokens ) {
+	if (false === $local_tokens) {
 		return $local_tokens;
 	}
 
 	// No social tokens found in the format we accept or it was empty
-	if ( empty( $local_tokens ) || ( ! isset( $local_tokens->twitter ) || ! isset( $local_tokens->facebook ) || ! isset( $local_tokens->linkedin ) ) ) {
+	if (empty($local_tokens) || ( ! isset($local_tokens->twitter) || ! isset($local_tokens->facebook) || ! isset($local_tokens->linkedin))) {
 		return false;
 	}
 
@@ -239,13 +239,13 @@ 
 	global $ppp_share_settings;
 	$share_link = get_permalink( $post_id );
 
-	if ( ppp_link_tracking_enabled() ) {
+	if (ppp_link_tracking_enabled()) {
 		$share_link = ppp_generate_link_tracking( $share_link, $post_id, $name );
 	}
 
-	if ( ppp_is_shortener_enabled() && $scheduled ) {
+	if (ppp_is_shortener_enabled() && $scheduled) {
 		$shortener_name = $ppp_share_settings['shortener'];
-		$share_link = apply_filters( 'ppp_apply_shortener-' . $shortener_name, $share_link );
+		$share_link = apply_filters( 'ppp_apply_shortener-'.$shortener_name, $share_link );
 	}
 
 
@@ -260,13 +260,13 @@ 
  * @return string             The URL to post, with proper analytics applied if necessary
  */
 function ppp_generate_link_tracking( $share_link, $post_id, $name ) {
-	if ( ppp_link_tracking_enabled() ) {
+	if (ppp_link_tracking_enabled()) {
 		global $ppp_share_settings;
 		$link_tracking_type = $ppp_share_settings['analytics'];
 
 		// Given the setting name, devs can extend this and apply a filter of ppp_analytics-[setting value]
 		// to apply their own rules for link tracking
-		$share_link = apply_filters( 'ppp_analytics-' . $link_tracking_type, $share_link, $post_id, $name );
+		$share_link = apply_filters( 'ppp_analytics-'.$link_tracking_type, $share_link, $post_id, $name );
 	}
 
 	$share_link = apply_filters( 'ppp_generate_link_tracking', $share_link, $post_id, $name );
@@ -282,14 +282,14 @@ 
  * @return mixed             If a thumbnail is found returns the URL, otherwise returns false
  */
 function ppp_post_has_media( $post_id, $network, $use_media, $attachment_id = false ) {
-	if ( !$use_media || empty( $post_id ) || empty( $network ) ) {
+	if ( ! $use_media || empty($post_id) || empty($network)) {
 		return false;
 	}
 
-	$thumb_id = empty( $attachment_id ) ? get_post_thumbnail_id( $post_id ) : $attachment_id;
-	$thumb_url = wp_get_attachment_image_src( $thumb_id, 'ppp-' . $network . '-share-image', true );
+	$thumb_id = empty($attachment_id) ? get_post_thumbnail_id( $post_id ) : $attachment_id;
+	$thumb_url = wp_get_attachment_image_src( $thumb_id, 'ppp-'.$network.'-share-image', true );
 
-	if ( isset( $thumb_url[0] ) && ! empty( $thumb_url[0] ) && !strpos( $thumb_url[0], 'wp-includes/images/media/default.png' ) ) {
+	if (isset($thumb_url[0]) && ! empty($thumb_url[0]) && ! strpos( $thumb_url[0], 'wp-includes/images/media/default.png' )) {
 		return $thumb_url[0];
 	}
 

includes/twitter-functions.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +98 added lines, -98 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -12,7 +12,7 @@ 
 function ppp_twitter_enabled() {
 	global $ppp_social_settings;
 
-	if ( isset( $ppp_social_settings['twitter'] ) && !empty( $ppp_social_settings['twitter'] ) ) {
+	if (isset($ppp_social_settings['twitter']) && ! empty($ppp_social_settings['twitter'])) {
 		return true;
 	}
 
@@ -50,10 +50,10 @@ 
  */
 function ppp_tw_account_list_avatar( $string = '' ) {
 
-	if ( ppp_twitter_enabled() ) {
+	if (ppp_twitter_enabled()) {
 		global $ppp_social_settings;
 		$avatar_url = $ppp_social_settings['twitter']['user']->profile_image_url_https;
-		$string .= '<img class="ppp-social-icon" src="' . $avatar_url . '" />';
+		$string .= '<img class="ppp-social-icon" src="'.$avatar_url.'" />';
 	}
 
 	return $string;
@@ -67,7 +67,7 @@ 
  */
 function ppp_tw_account_list_name( $string = '' ) {
 
-	if ( ppp_twitter_enabled() ) {
+	if (ppp_twitter_enabled()) {
 		global $ppp_social_settings;
 		$string .= $ppp_social_settings['twitter']['user']->name;
 	}
@@ -83,15 +83,15 @@ 
  */
 function ppp_tw_account_list_actions( $string = '' ) {
 
-	if ( ! ppp_twitter_enabled() ) {
+	if ( ! ppp_twitter_enabled()) {
 		global $ppp_twitter_oauth, $ppp_social_settings;
 		$tw_auth = $ppp_twitter_oauth->ppp_verify_twitter_credentials();
 		$tw_authurl = $ppp_twitter_oauth->ppp_get_twitter_auth_url();
 
-		$string .= '<a href="' . $tw_authurl . '"><img src="' . PPP_URL . '/includes/images/sign-in-with-twitter-gray.png" /></a>';
+		$string .= '<a href="'.$tw_authurl.'"><img src="'.PPP_URL.'/includes/images/sign-in-with-twitter-gray.png" /></a>';
 	} else {
-		$string .= '<a class="button-primary" href="' . admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=twitter' ) . '" >' . __( 'Disconnect from Twitter', 'ppp-txt' ) . '</a>&nbsp;';
-		$string .= '<a class="button-secondary" href="https://twitter.com/settings/applications" target="blank">' . __( 'Revoke Access via Twitter', 'ppp-txt' ) . '</a>';
+		$string .= '<a class="button-primary" href="'.admin_url( 'admin.php?page=ppp-social-settings&ppp_social_disconnect=true&ppp_network=twitter' ).'" >'.__( 'Disconnect from Twitter', 'ppp-txt' ).'</a>&nbsp;';
+		$string .= '<a class="button-secondary" href="https://twitter.com/settings/applications" target="blank">'.__( 'Revoke Access via Twitter', 'ppp-txt' ).'</a>';
 	}
 
 	return $string;
@@ -103,13 +103,13 @@ 
  * @return void
  */
 function ppp_capture_twitter_oauth() {
-	if ( isset( $_REQUEST['oauth_verifier'] ) && isset( $_REQUEST['oauth_token'] ) ) {
+	if (isset($_REQUEST['oauth_verifier']) && isset($_REQUEST['oauth_token'])) {
 		$current_screen = get_current_screen();
-		if ( 'user-edit' === $current_screen->base ) {
-			$user_id = ! empty( $_GET['user_id'] ) && is_numeric( $_GET['user_id'] ) ? $_GET['user_id'] : false;
+		if ('user-edit' === $current_screen->base) {
+			$user_id = ! empty($_GET['user_id']) && is_numeric( $_GET['user_id'] ) ? $_GET['user_id'] : false;
 			$twitter = new PPP_Twitter_User( $user_id );
 			$twitter->init();
-			$redirect = admin_url( 'user-edit.php?updated=1&user_id=' . $user_id );
+			$redirect = admin_url( 'user-edit.php?updated=1&user_id='.$user_id );
 		} else {
 			global $ppp_twitter_oauth;
 			$ppp_twitter_oauth->ppp_initialize_twitter();
@@ -127,17 +127,17 @@ 
  * @return void
  */
 function ppp_disconnect_twitter() {
-	if ( ! empty( $_GET['user_id'] ) ) {
+	if ( ! empty($_GET['user_id'])) {
 		$user_id = (int) sanitize_text_field( $_GET['user_id'] );
-		if ( $user_id !== get_current_user_id() || ! current_user_can( 'manage_options' ) ) {
+		if ($user_id !== get_current_user_id() || ! current_user_can( 'manage_options' )) {
 			wp_die( __( 'Unable to disconnect Twitter account', 'ppp-txt' ) );
 		}
 		delete_user_meta( $user_id, '_ppp_twitter_data' );
 	} else {
 		global $ppp_social_settings;
 		$ppp_social_settings = get_option( 'ppp_social_settings' );
-		if ( isset( $ppp_social_settings['twitter'] ) ) {
-			unset( $ppp_social_settings['twitter'] );
+		if (isset($ppp_social_settings['twitter'])) {
+			unset($ppp_social_settings['twitter']);
 			update_option( 'ppp_social_settings', $ppp_social_settings );
 		}
 	}
@@ -165,7 +165,7 @@ 
 	$share_content = ppp_tw_generate_share_content( $post_id, $name );
 	$share_link    = ppp_generate_link( $post_id, $name, $scheduled );
 
-	return apply_filters( 'ppp_tw_build_share_message', $share_content . ' ' . $share_link );
+	return apply_filters( 'ppp_tw_build_share_message', $share_content.' '.$share_link );
 }
 
 /**
@@ -176,10 +176,10 @@ 
  */
 function ppp_tw_generate_share_content( $post_id, $name, $is_scheduled = true ) {
 	global $ppp_options;
-	$default_text = isset( $ppp_options['default_text'] ) ? $ppp_options['default_text'] : '';
+	$default_text = isset($ppp_options['default_text']) ? $ppp_options['default_text'] : '';
 	$ppp_tweets   = get_post_meta( $post_id, '_ppp_tweets', true );
 
-	if ( $is_scheduled && ! empty( $ppp_tweets ) ) {
+	if ($is_scheduled && ! empty($ppp_tweets)) {
 		$ppp_post_override_data = get_post_meta( $post_id, '_ppp_post_override_data', true );
 		$name_array    = explode( '_', $name );
 		$index         = $name_array[1];
@@ -187,10 +187,10 @@ 
 	}
 
 	// If an override was found, use it, otherwise try the default text content
-	$share_content = ( isset( $share_content ) && !empty( $share_content ) ) ? $share_content : $default_text;
+	$share_content = (isset($share_content) && ! empty($share_content)) ? $share_content : $default_text;
 
 	// If the content is still empty, just use the post title
-	$share_content = ( isset( $share_content ) && !empty( $share_content ) ) ? $share_content : get_the_title( $post_id );
+	$share_content = (isset($share_content) && ! empty($share_content)) ? $share_content : get_the_title( $post_id );
 
 	return apply_filters( 'ppp_share_content', $share_content, array( 'post_id' => $post_id ) );
 }
@@ -202,12 +202,12 @@ 
  * @return bool         Whether or not this tweet should contain a media post
  */
 function ppp_tw_use_media( $post_id, $index ) {
-	if ( empty( $post_id ) || empty( $index ) ) {
+	if (empty($post_id) || empty($index)) {
 		return false;
 	}
 
 	$share_data = get_post_meta( $post_id, '_ppp_tweets', true );
-	$use_media  = ! empty( $share_data[$index]['attachment_id'] ) || ! empty( $share_data[$index]['image'] ) ? true : false;
+	$use_media  = ! empty($share_data[$index]['attachment_id']) || ! empty($share_data[$index]['image']) ? true : false;
 
 	return $use_media;
 }
@@ -218,7 +218,7 @@ 
  * @return void
  */
 function ppp_set_tw_token_constants( $social_tokens ) {
-	if ( !empty( $social_tokens ) && property_exists( $social_tokens, 'twitter' ) ) {
+	if ( ! empty($social_tokens) && property_exists( $social_tokens, 'twitter' )) {
 		define( 'PPP_TW_CONSUMER_KEY', $social_tokens->twitter->consumer_token );
 		define( 'PPP_TW_CONSUMER_SECRET', $social_tokens->twitter->consumer_secret );
 	}
@@ -256,7 +256,7 @@ 
  */
 function ppp_tw_add_meta_tab( $tabs ) {
 	global $ppp_social_settings;
-	if ( !isset( $ppp_social_settings['twitter'] ) ) {
+	if ( ! isset($ppp_social_settings['twitter'])) {
 		return $tabs;
 	}
 
@@ -273,7 +273,7 @@ 
  */
 function ppp_tw_register_metabox_content( $content ) {
 	global $ppp_social_settings;
-	if ( !isset( $ppp_social_settings['twitter'] ) ) {
+	if ( ! isset($ppp_social_settings['twitter'])) {
 		return $content;
 	}
 
@@ -299,16 +299,16 @@ 
  */
 function ppp_tw_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_post_exclude = get_post_meta( $post->ID, '_ppp_post_exclude', true );
 
 	$ppp_share_on_publish  = get_post_meta( $post->ID, '_ppp_share_on_publish', true );
 	$show_share_on_publish = false;
 
-	$share_by_default      = empty( $ppp_share_settings['twitter']['share_on_publish'] ) ? false : true;
+	$share_by_default      = empty($ppp_share_settings['twitter']['share_on_publish']) ? false : true;
 
-	if ( $ppp_share_on_publish == '1' || ( $ppp_share_on_publish == '' && $share_by_default ) ) {
+	if ($ppp_share_on_publish == '1' || ($ppp_share_on_publish == '' && $share_by_default)) {
 		$show_share_on_publish = true;
 	}
 	$ppp_share_on_publish_text = get_post_meta( $post->ID, '_ppp_share_on_publish_text', true );
@@ -316,22 +316,22 @@ 
 	?>
 		<p>
 			<p>
-			<?php $disabled = ( $post->post_status === 'publish' && time() > strtotime( $post->post_date ) ) ? true : false; ?>
-			<input <?php if ( $disabled ): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_share_on_publish" id="ppp_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />&nbsp;
+			<?php $disabled = ($post->post_status === 'publish' && time() > strtotime( $post->post_date )) ? true : false; ?>
+			<input <?php if ($disabled): ?>readonly<?php endif; ?> type="checkbox" name="_ppp_share_on_publish" id="ppp_share_on_publish" value="1" <?php checked( true, $show_share_on_publish, true ); ?> />&nbsp;
 				<label for="ppp_share_on_publish"><?php _e( 'Tweet this post at the time of publishing?', 'ppp-txt' ); ?></label>
-				<p class="ppp_share_on_publish_text"<?php if ( false === $show_share_on_publish ) : ?> style="display: none;"<?php endif; ?>>
+				<p class="ppp_share_on_publish_text"<?php if (false === $show_share_on_publish) : ?> style="display: none;"<?php endif; ?>>
 						<input
-						<?php if ( $disabled ): ?>readonly<?php endif; ?>
+						<?php if ($disabled): ?>readonly<?php endif; ?>
 						class="ppp-share-text"
 						type="text"
 						placeholder="<?php echo esc_attr( $default_text ); ?>"
 						name="_ppp_share_on_publish_text"
-						<?php if ( isset( $ppp_share_on_publish_text ) ) {?>value="<?php echo esc_attr( $ppp_share_on_publish_text ); ?>"<?php ;}?>
-						<?php $length = ! empty($ppp_share_on_publish_text ) ? strlen( $ppp_share_on_publish_text ) : 0; ?>
+						<?php if (isset($ppp_share_on_publish_text)) {?>value="<?php echo esc_attr( $ppp_share_on_publish_text ); ?>"<?php; }?>
+						<?php $length = ! empty($ppp_share_on_publish_text) ? strlen( $ppp_share_on_publish_text ) : 0; ?>
 					/>&nbsp;<span class="ppp-text-length"><?php echo $length; ?></span>
 					<br />
 					<span>
-						<input class="ppp-tw-featured-image-input" <?php if ( $disabled ): ?>readonly<?php endif; ?> id="ppp-share-on-publish-image" type="checkbox" name="_ppp_share_on_publish_include_image" value="1" <?php checked( '1', $ppp_share_on_publish_include_image, true ); ?>/>
+						<input class="ppp-tw-featured-image-input" <?php if ($disabled): ?>readonly<?php endif; ?> id="ppp-share-on-publish-image" type="checkbox" name="_ppp_share_on_publish_include_image" value="1" <?php checked( '1', $ppp_share_on_publish_include_image, true ); ?>/>
 						&nbsp;<label for="ppp-share-on-publish-image"><?php _e( 'Attach Featured Image In Tweet', 'ppp-txt' ); ?></label>
 					</span>
 				</p>
@@ -353,16 +353,16 @@ 
 							</thead>
 							<tbody>
 								<?php $tweets = get_post_meta( $post->ID, '_ppp_tweets', true ); ?>
-								<?php if ( ! empty( $tweets ) ) : ?>
+								<?php if ( ! empty($tweets)) : ?>
 
-									<?php foreach ( $tweets as $key => $value ) :
-										$date          = isset( $value['date'] )          ? $value['date']          : '';
-										$time          = isset( $value['time'] )          ? $value['time']          : '';
-										$text          = isset( $value['text'] )          ? $value['text']          : '';
-										$image         = isset( $value['image'] )         ? $value['image']         : '';
-										$attachment_id = isset( $value['attachment_id'] ) ? $value['attachment_id'] : '';
+									<?php foreach ($tweets as $key => $value) :
+										$date          = isset($value['date']) ? $value['date'] : '';
+										$time          = isset($value['time']) ? $value['time'] : '';
+										$text          = isset($value['text']) ? $value['text'] : '';
+										$image         = isset($value['image']) ? $value['image'] : '';
+										$attachment_id = isset($value['attachment_id']) ? $value['attachment_id'] : '';
 
-										$args = apply_filters( 'ppp_tweet_row_args', compact( 'date','time','text','image','attachment_id' ), $value );
+										$args = apply_filters( 'ppp_tweet_row_args', compact( 'date', 'time', 'text', 'image', 'attachment_id' ), $value );
 										?>
 
 										<?php ppp_render_tweet_row( $key, $args, $post->ID ); ?>
@@ -397,10 +397,10 @@ 
 function ppp_render_tweet_row( $key, $args = array(), $post_id ) {
 	global $post;
 
-	$share_time     = strtotime( $args['date'] . ' ' . $args['time'] );
+	$share_time     = strtotime( $args['date'].' '.$args['time'] );
 	$readonly       = current_time( 'timestamp' ) > $share_time ? 'readonly="readonly" ' : false;
-	$no_date        = ! empty( $readonly ) ? ' hasDatepicker' : '';
-	$hide           = ! empty( $readonly ) ? 'display: none;' : '';
+	$no_date        = ! empty($readonly) ? ' hasDatepicker' : '';
+	$hide           = ! empty($readonly) ? 'display: none;' : '';
 	?>
 	<tr class="ppp-tweet-wrapper ppp-repeatable-row" data-key="<?php echo esc_attr( $key ); ?>">
 		<td>
@@ -413,7 +413,7 @@ 
 
 		<td>
 			<input <?php echo $readonly; ?>class="ppp-tweet-text-repeatable" type="text" name="_ppp_tweets[<?php echo $key; ?>][text]" value="<?php echo esc_attr( $args['text'] ); ?>" />
-			<?php $length = ! empty( $args['text'] ) ? strlen( $args['text'] ) : 0; ?>
+			<?php $length = ! empty($args['text']) ? strlen( $args['text'] ) : 0; ?>
 			&nbsp;<span class="ppp-text-length"><?php echo $length; ?></span>
 		</td>
 
@@ -432,7 +432,7 @@ 
 		</td>
 
 		<td>
-			<a href="#" class="ppp-repeatable-row ppp-remove-repeatable" data-type="tweet" style="background: url(<?php echo admin_url('/images/xit.gif'); ?>) no-repeat;<?php echo $hide; ?>">&times;</a>
+			<a href="#" class="ppp-repeatable-row ppp-remove-repeatable" data-type="tweet" style="background: url(<?php echo admin_url( '/images/xit.gif' ); ?>) no-repeat;<?php echo $hide; ?>">&times;</a>
 		</td>
 
 	</tr>
@@ -447,24 +447,24 @@ 
  */
 function ppp_tw_save_post_meta_boxes( $post_id, $post ) {
 
-	if ( ! ppp_should_save( $post_id, $post ) ) {
+	if ( ! ppp_should_save( $post_id, $post )) {
 		return;
 	}
 
-	$ppp_post_exclude = ( isset( $_REQUEST['_ppp_post_exclude'] ) ) ? $_REQUEST['_ppp_post_exclude'] : '0';
+	$ppp_post_exclude = (isset($_REQUEST['_ppp_post_exclude'])) ? $_REQUEST['_ppp_post_exclude'] : '0';
 
-	$ppp_share_on_publish = ( isset( $_REQUEST['_ppp_share_on_publish'] ) ) ? $_REQUEST['_ppp_share_on_publish'] : '0';
-	$ppp_share_on_publish_text = ( isset( $_REQUEST['_ppp_share_on_publish_text'] ) ) ? $_REQUEST['_ppp_share_on_publish_text'] : '';
-	$ppp_share_on_publish_include_image = ( isset( $_REQUEST['_ppp_share_on_publish_include_image'] ) ) ? $_REQUEST['_ppp_share_on_publish_include_image'] : '';
+	$ppp_share_on_publish = (isset($_REQUEST['_ppp_share_on_publish'])) ? $_REQUEST['_ppp_share_on_publish'] : '0';
+	$ppp_share_on_publish_text = (isset($_REQUEST['_ppp_share_on_publish_text'])) ? $_REQUEST['_ppp_share_on_publish_text'] : '';
+	$ppp_share_on_publish_include_image = (isset($_REQUEST['_ppp_share_on_publish_include_image'])) ? $_REQUEST['_ppp_share_on_publish_include_image'] : '';
 
 
 	update_post_meta( $post_id, '_ppp_share_on_publish', $ppp_share_on_publish );
 	update_post_meta( $post_id, '_ppp_share_on_publish_text', $ppp_share_on_publish_text );
 	update_post_meta( $post_id, '_ppp_share_on_publish_include_image', $ppp_share_on_publish_include_image );
 
-	$tweet_data = isset( $_REQUEST['_ppp_tweets'] ) ? $_REQUEST['_ppp_tweets'] : array();
-	foreach ( $tweet_data as $index => $tweet ) {
-		$tweet_data[ $index ]['text'] = sanitize_text_field( $tweet['text'] );
+	$tweet_data = isset($_REQUEST['_ppp_tweets']) ? $_REQUEST['_ppp_tweets'] : array();
+	foreach ($tweet_data as $index => $tweet) {
+		$tweet_data[$index]['text'] = sanitize_text_field( $tweet['text'] );
 	}
 	update_post_meta( $post_id, '_ppp_tweets', $tweet_data );
 
@@ -482,34 +482,34 @@ 
 	global $ppp_options;
 
 	$from_meta = get_post_meta( $post->ID, '_ppp_share_on_publish', true );
-	$from_post = isset( $_POST['_ppp_share_on_publish'] );
+	$from_post = isset($_POST['_ppp_share_on_publish']);
 
-	if ( empty( $from_meta ) && empty( $from_post ) ) {
+	if (empty($from_meta) && empty($from_post)) {
 		return;
 	}
 
 	// Determine if we're seeing the share on publish in meta or $_POST
-	if ( $from_meta && !$from_post ) {
+	if ($from_meta && ! $from_post) {
 		$ppp_share_on_publish_text = get_post_meta( $post->ID, '_ppp_share_on_publish_text', true );
 		$use_media = get_post_meta( $post->ID, '_ppp_share_on_publish_include_image', true );
 	} else {
-		$ppp_share_on_publish_text = isset( $_POST['_ppp_share_on_publish_text'] ) ? $_POST['_ppp_share_on_publish_text'] : '';
-		$use_media = isset( $_POST['_ppp_share_on_publish_include_image'] ) ? $_POST['_ppp_share_on_publish_include_image'] : false;
+		$ppp_share_on_publish_text = isset($_POST['_ppp_share_on_publish_text']) ? $_POST['_ppp_share_on_publish_text'] : '';
+		$use_media = isset($_POST['_ppp_share_on_publish_include_image']) ? $_POST['_ppp_share_on_publish_include_image'] : false;
 	}
 
-	$share_content = ( !empty( $ppp_share_on_publish_text ) ) ? $ppp_share_on_publish_text : ppp_tw_generate_share_content( $post->ID, null, false );
+	$share_content = ( ! empty($ppp_share_on_publish_text)) ? $ppp_share_on_publish_text : ppp_tw_generate_share_content( $post->ID, null, false );
 	$share_content = apply_filters( 'ppp_share_content', $share_content, array( 'post_id' => $post->ID ) );
-	$name = 'sharedate_0_' . $post->ID;
+	$name = 'sharedate_0_'.$post->ID;
 	$media = ppp_post_has_media( $post->ID, 'tw', $use_media );
 	$share_link = ppp_generate_link( $post->ID, $name, true );
 
-	$status['twitter'] = ppp_send_tweet( $share_content . ' ' . $share_link, $post->ID, $media );
+	$status['twitter'] = ppp_send_tweet( $share_content.' '.$share_link, $post->ID, $media );
 
-	if ( ! empty( $status['twitter']->id_str ) ) {
+	if ( ! empty($status['twitter']->id_str)) {
 		$author_id = $post->post_author;
 		$author_rt = get_user_meta( $author_id, '_ppp_share_on_publish', true );
 
-		if ( $author_rt ) {
+		if ($author_rt) {
 			$twitter_user = new PPP_Twitter_User( $author_id );
 			$twitter_user->retweet( $status['twitter']->id_str );
 		}
@@ -517,8 +517,8 @@ 
 	}
 
 
-	if ( isset( $ppp_options['enable_debug'] ) && $ppp_options['enable_debug'] == '1' ) {
-		update_post_meta( $post->ID, '_ppp-' . $name . '-status', $status );
+	if (isset($ppp_options['enable_debug']) && $ppp_options['enable_debug'] == '1') {
+		update_post_meta( $post->ID, '_ppp-'.$name.'-status', $status );
 	}
 }
 add_action( 'ppp_share_on_publish', 'ppp_tw_share_on_publish', 10, 3 );
@@ -534,7 +534,7 @@ 
  */
 function ppp_tw_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 );
 	}
 
@@ -552,7 +552,7 @@ 
 
 	$ret = false;
 
-	if ( ! empty( $ppp_share_settings['twitter']['cards_enabled'] ) ) {
+	if ( ! empty($ppp_share_settings['twitter']['cards_enabled'])) {
 		$ret = true;
 	}
 
@@ -567,13 +567,13 @@ 
  */
 function ppp_tw_card_meta() {
 
-	if ( ! is_single() || ! ppp_twitter_enabled() || ! ppp_tw_cards_enabled() ) {
+	if ( ! is_single() || ! ppp_twitter_enabled() || ! ppp_tw_cards_enabled()) {
 		return;
 	}
 
 	global $post, $ppp_options;
 
-	if ( ! array_key_exists( $post->post_type, $ppp_options['post_types'] ) ) {
+	if ( ! array_key_exists( $post->post_type, $ppp_options['post_types'] )) {
 		return;
 	}
 
@@ -591,20 +591,20 @@ 
 
 	$return = '';
 
-	if ( ! is_single() || ! ppp_tw_cards_enabled() ) {
+	if ( ! is_single() || ! ppp_tw_cards_enabled()) {
 		return $return;
 	}
 
 	global $post, $ppp_social_settings;
 
 
-	if ( empty( $post ) ) {
+	if (empty($post)) {
 		return;
 	}
 
 	$elements = ppp_tw_default_meta_elements();
-	foreach ( $elements as $name => $content ) {
-		$return .= '<meta name="' . $name . '" content="' . $content . '" />' . "\n";
+	foreach ($elements as $name => $content) {
+		$return .= '<meta name="'.$name.'" content="'.$content.'" />'."\n";
 	}
 
 	return apply_filters( 'ppp_tw_card_meta', $return );
@@ -624,22 +624,22 @@ 
 	$elements = array();
 
 	$image_url = ppp_post_has_media( $post->ID, 'tw', true );
-	if ( $image_url ) {
+	if ($image_url) {
 		$elements['twitter:card']      = 'summary_large_image';
 		$elements['twitter:image:src'] = $image_url;
 	} else {
 		$elements['twitter:card'] = 'summary';
 	}
 
-	$elements['twitter:site']        = '@' . $ppp_social_settings['twitter']['user']->screen_name;
+	$elements['twitter:site']        = '@'.$ppp_social_settings['twitter']['user']->screen_name;
 	$elements['twitter:title']       = esc_attr( strip_tags( $post->post_title ) );
 	$elements['twitter:description'] = esc_attr( strip_tags( ppp_tw_get_card_description() ) );
 
 	$author_twitter_handle = get_user_meta( $post->post_author, 'twitter', true );
-	if ( ! empty( $author_twitter_handle ) ) {
+	if ( ! empty($author_twitter_handle)) {
 
-		if ( strpos( $author_twitter_handle, '@' ) === false ) {
-			$author_twitter_handle = '@' . $author_twitter_handle;
+		if (strpos( $author_twitter_handle, '@' ) === false) {
+			$author_twitter_handle = '@'.$author_twitter_handle;
 		}
 
 		$elements['twitter:creator'] = esc_attr( strip_tags( $author_twitter_handle ) );
@@ -658,17 +658,17 @@ 
 function ppp_tw_get_card_description() {
 	global $post;
 
-	if ( ! is_single() || empty( $post ) ) {
+	if ( ! is_single() || empty($post)) {
 		return false;
 	}
 
 	$excerpt = $post->post_excerpt;
 	$max_len = apply_filters( 'ppp_tw_cart_desc_length', 200 );
 
-	if ( empty( $excerpt ) ) {
+	if (empty($excerpt)) {
 		$excerpt_pre = substr( $post->post_content, 0, $max_len );
 		$last_space  = strrpos( $excerpt_pre, ' ' );
-		$excerpt     = substr( $excerpt_pre, 0, $last_space ) . '...';
+		$excerpt     = substr( $excerpt_pre, 0, $last_space ).'...';
 	}
 
 	return apply_filters( 'ppp_tw_card_desc', $excerpt );
@@ -683,7 +683,7 @@ 
  */
 function ppp_tw_add_contact_method( $user_contactmethods ) {
 
-	if ( ! isset( $user_contactmethods['twitter'] ) ) {
+	if ( ! isset($user_contactmethods['twitter'])) {
 		$user_contactmethods['twitter'] = __( 'Twitter', 'ppp-txt' );
 	}
 	// Returns the contact methods
@@ -699,11 +699,11 @@ 
  */
 function ppp_tw_profile_settings( $user ) {
 
-	if ( $user->ID == get_current_user_id() && ! current_user_can( 'edit_posts' ) ) {
+	if ($user->ID == get_current_user_id() && ! current_user_can( 'edit_posts' )) {
 		return;
 	}
 
-	if ( $user->ID !== get_current_user_id() && ! current_user_can( 'manage_options' ) ) {
+	if ($user->ID !== get_current_user_id() && ! current_user_can( 'manage_options' )) {
 		return;
 	}
 
@@ -717,16 +717,16 @@ 
 			<?php
 			$twitter = new PPP_Twitter_User( get_current_user_id() );
 			$tw_user = get_user_meta( $user->ID, '_ppp_twitter_data', true );
-			if ( empty( $tw_user ) ) {
-				$tw_authurl = $twitter->get_auth_url( admin_url( 'user-edit.php?user_id=' . $user->ID ) );
+			if (empty($tw_user)) {
+				$tw_authurl = $twitter->get_auth_url( admin_url( 'user-edit.php?user_id='.$user->ID ) );
 
-				echo '<a href="' . $tw_authurl . '"><img src="' . PPP_URL . '/includes/images/sign-in-with-twitter-gray.png" /></a>';
+				echo '<a href="'.$tw_authurl.'"><img src="'.PPP_URL.'/includes/images/sign-in-with-twitter-gray.png" /></a>';
 			} else {
 				$connected = true;
 				?>
 				<p><strong><?php _e( 'Signed in as', 'ppp-txt' ); ?>: </strong><?php echo $tw_user['user']->screen_name; ?></p>
 				<p>
-					<a class="button-primary" href="<?php echo admin_url( 'user-edit.php?user_id=' . $user->ID . '&ppp_social_disconnect=true&ppp_network=twitter&user_id=' . $user->ID ); ?>" ><?php _e( 'Disconnect from Twitter', 'ppp-txt' ); ?></a>&nbsp;
+					<a class="button-primary" href="<?php echo admin_url( 'user-edit.php?user_id='.$user->ID.'&ppp_social_disconnect=true&ppp_network=twitter&user_id='.$user->ID ); ?>" ><?php _e( 'Disconnect from Twitter', 'ppp-txt' ); ?></a>&nbsp;
 					<a class="button-secondary" href="https://twitter.com/settings/applications" target="blank"><?php _e( 'Revoke Access via Twitter', 'ppp-txt' ); ?></a>
 				</p>
 				<?php
@@ -735,10 +735,10 @@ 
 			</td>
 		</tr>
 
-		<?php if ( $connected ) : ?>
+		<?php if ($connected) : ?>
 		<?php
 			$share_on_publish = get_user_meta( $user->ID, '_ppp_share_on_publish', true );
-			$share_scheduled  = get_user_meta( $user->ID, '_ppp_share_scheduled' , true );
+			$share_scheduled  = get_user_meta( $user->ID, '_ppp_share_scheduled', true );
 		?>
 		<tr>
 			<th><?php _e( 'Sharing Options', 'ppp-txt' ); ?></th>
@@ -769,11 +769,11 @@ 
  */
 function ppp_tw_save_profile( $user_id ) {
 
-	$share_on_publish = ! empty( $_POST['share_on_publish'] ) ? true : false;
-	$share_scheduled  = ! empty( $_POST['share_scheduled'] )  ? true : false;
+	$share_on_publish = ! empty($_POST['share_on_publish']) ? true : false;
+	$share_scheduled  = ! empty($_POST['share_scheduled']) ? true : false;
 
 	update_user_meta( $user_id, '_ppp_share_on_publish', $share_on_publish );
-	update_user_meta( $user_id, '_ppp_share_scheduled' , $share_scheduled  );
+	update_user_meta( $user_id, '_ppp_share_scheduled', $share_scheduled );
 
 }
 add_action( 'personal_options_update', 'ppp_tw_save_profile' );

Braces +5 added lines, -2 removed lines

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

post-promoter-pro.php

Doc Comments +2 added lines, -2 removed lines

@@ -291,7 +291,7 @@ 
 
 	/**
 	 * Deactivates the license key
-	 * @return void
+	 * @return null|false
 	 */
 	public function deactivate_license() {
 		// listen for our activate button to be clicked
@@ -335,7 +335,7 @@ 
 
 	/**
 	 * Activates the license key provided
-	 * @return void
+	 * @return null|false
 	 */
 	public function activate_license() {
 		// listen for our activate button to be clicked

Indentation +33 added lines, -33 removed lines

@@ -181,43 +181,43 @@ 
 	 */
 	public function ppp_setup_admin_menu() {
 		add_menu_page( __( 'Post Promoter', 'ppp-txt' ),
-		               __( 'Post Promoter', 'ppp-txt' ),
-		               apply_filters( 'ppp_manage_role', 'administrator' ),
-		               'ppp-options',
-		               'ppp_admin_page'
-		             );
+					   __( 'Post Promoter', 'ppp-txt' ),
+					   apply_filters( 'ppp_manage_role', 'administrator' ),
+					   'ppp-options',
+					   'ppp_admin_page'
+					 );
 
 		add_submenu_page( 'ppp-options',
-			              __( 'Social Settings', 'ppp-txt' ),
-			              __( 'Social Settings', 'ppp-txt' ),
-			              'manage_options',
-			              'ppp-social-settings',
-			              'ppp_display_social'
-			            );
+						  __( 'Social Settings', 'ppp-txt' ),
+						  __( 'Social Settings', 'ppp-txt' ),
+						  'manage_options',
+						  'ppp-social-settings',
+						  'ppp_display_social'
+						);
 
 		add_submenu_page( 'ppp-options',
-			              __( 'Schedule', 'ppp-txt' ),
-			              __( 'Schedule', 'ppp-txt' ),
-			              'manage_options',
-			              'ppp-schedule-info',
-			              'ppp_display_schedule'
-			            );
+						  __( 'Schedule', 'ppp-txt' ),
+						  __( 'Schedule', 'ppp-txt' ),
+						  'manage_options',
+						  'ppp-schedule-info',
+						  'ppp_display_schedule'
+						);
 
 		add_submenu_page( 'ppp-options',
-			              __( 'System Info', 'ppp-txt' ),
-			              __( 'System Info', 'ppp-txt' ),
-			              'manage_options',
-			              'ppp-system-info',
-			              'ppp_display_sysinfo'
-			            );
+						  __( 'System Info', 'ppp-txt' ),
+						  __( 'System Info', 'ppp-txt' ),
+						  'manage_options',
+						  'ppp-system-info',
+						  'ppp_display_sysinfo'
+						);
 
 		add_submenu_page( null,
-			              __( 'PPP Upgrades', 'ppp-txt' ),
-			              __( 'PPP Upgrades', 'ppp-txt' ),
-			              'manage_options',
-			              'ppp-upgrades',
-			              'ppp_upgrades_screen'
-			            );
+						  __( 'PPP Upgrades', 'ppp-txt' ),
+						  __( 'PPP Upgrades', 'ppp-txt' ),
+						  'manage_options',
+						  'ppp-upgrades',
+						  'ppp_upgrades_screen'
+						);
 
 	}
 
@@ -280,10 +280,10 @@ 
 		<div class="updated">
 			<p>
 				<?php printf(
-			         __( 'Post Promoter Pro requires your license key to work, please <a href="%s">enter it now</a>.', 'ppp-txt' ),
-			              admin_url( 'admin.php?page=ppp-options' )
-			         );
-			    ?>
+					 __( 'Post Promoter Pro requires your license key to work, please <a href="%s">enter it now</a>.', 'ppp-txt' ),
+						  admin_url( 'admin.php?page=ppp-options' )
+					 );
+				?>
 			</p>
 		</div>
 		<?php

Spacing +44 added lines, -44 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,7 +27,7 @@ 
 	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 {
 			register_activation_hook( PPP_FILE, array( $this, 'activation_setup' ) );
@@ -37,21 +37,21 @@ 
 			$ppp_social_settings = get_option( 'ppp_social_settings' );
 			$ppp_share_settings  = get_option( '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';
-
-			if ( is_admin() ) {
-				include PPP_PATH . '/includes/admin/upgrades.php';
-				include PPP_PATH . '/includes/admin/do-upgrades.php';
-				include PPP_PATH . '/includes/admin/actions.php';
-				include PPP_PATH . '/includes/admin/admin-pages.php';
-				include PPP_PATH . '/includes/admin/admin-ajax.php';
-				include PPP_PATH . '/includes/admin/meta-boxes.php';
-				include PPP_PATH . '/includes/admin/welcome.php';
-				include PPP_PATH . '/includes/admin/dashboard.php';
+			include PPP_PATH.'/includes/general-functions.php';
+			include PPP_PATH.'/includes/share-functions.php';
+			include PPP_PATH.'/includes/cron-functions.php';
+			include PPP_PATH.'/includes/filters.php';
+			include PPP_PATH.'/includes/libs/social-loader.php';
+
+			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';
 
 				add_action( 'admin_init', array( $this, 'ppp_register_settings' ) );
 				add_action( 'admin_init', 'ppp_upgrade_plugin', 1 );
@@ -69,8 +69,8 @@ 
 			}
 
 			add_action( 'init', array( $this, 'get_actions' ) );
-			add_action( 'save_post', 'ppp_schedule_share', 11, 2);
-			add_action( 'transition_post_status', 'ppp_share_on_publish', 99, 3);
+			add_action( 'save_post', 'ppp_schedule_share', 11, 2 );
+			add_action( 'transition_post_status', 'ppp_share_on_publish', 99, 3 );
 			add_action( 'init', 'ppp_add_image_sizes' );
 		}
 
@@ -82,7 +82,7 @@ 
 	 * @access public
 	 */
 	public static function getInstance() {
-		if ( !self::$ppp_instance ) {
+		if ( ! self::$ppp_instance) {
 			self::$ppp_instance = new PostPromoterPro();
 		}
 
@@ -107,7 +107,7 @@ 
 	 */
 	public function activation_setup() {
 		// If the settings already exist, don't do this
-		if ( get_option( 'ppp_options' ) ) {
+		if (get_option( 'ppp_options' )) {
 			return;
 		}
 
@@ -129,28 +129,28 @@ 
 	 * @access public
 	 */
 	public function load_custom_scripts( $hook ) {
-		if ( 'toplevel_page_ppp-options' != $hook
+		if ('toplevel_page_ppp-options' != $hook
 			  && 'post-promoter_page_ppp-social-settings' != $hook
 			  && 'post-new.php' != $hook
-			  && 'post.php' != $hook ) {
+			  && 'post.php' != $hook) {
 					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 );
 	}
 
 	public function load_styles( $hook ) {
-		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' );
 
 		// 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' );
 		}
 	}
@@ -163,7 +163,7 @@ 
 	 * @access public
 	 */
 	public function plugin_settings_links( $links, $file ) {
-		if ( $file != PPP_FILE ) {
+		if ($file != PPP_FILE) {
 			return $links;
 		}
 
@@ -250,22 +250,22 @@ 
 	 */
 	public function plugin_updater() {
 
-		if ( defined( 'NO_AUTO_UPDATE' ) && true === NO_AUTO_UPDATE ) {
+		if (defined( 'NO_AUTO_UPDATE' ) && true === NO_AUTO_UPDATE) {
 			return;
 		}
 
 		$license_key = trim( get_option( '_ppp_license_key' ) );
 
-		if ( empty( $license_key ) ) {
+		if (empty($license_key)) {
 			add_action( 'admin_notices', array( $this, 'no_license_nag' ) );
 			return;
 		}
 
 		// setup the updater
 		$edd_updater = new EDD_SL_Plugin_Updater( PPP_STORE_URL, __FILE__, array(
-				'version'   => PPP_VERSION,         // current version number
-				'license'   => $license_key,        // license key (used get_option above to retrieve from DB)
-				'item_name' => PPP_PLUGIN_NAME,     // name of this plugin
+				'version'   => PPP_VERSION, // current version number
+				'license'   => $license_key, // license key (used get_option above to retrieve from DB)
+				'item_name' => PPP_PLUGIN_NAME, // name of this plugin
 				'author'    => 'Post Promoter Pro'  // author of this plugin
 			)
 		);
@@ -295,10 +295,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
@@ -318,7 +318,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;
 			}
 
@@ -326,7 +326,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' );
 			}
 
@@ -339,10 +339,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
@@ -362,7 +362,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;
 			}
 
@@ -383,15 +383,15 @@ 
 	 */
 	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;
 	}
 
 	public function get_actions() {
-		if ( isset( $_GET['ppp_action'] ) ) {
-			do_action( 'ppp_' . $_GET['ppp_action'], $_GET );
+		if (isset($_GET['ppp_action'])) {
+			do_action( 'ppp_'.$_GET['ppp_action'], $_GET );
 		}
 	}
 }

includes/admin/actions.php

Spacing +34 added lines, -34 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -14,14 +14,14 @@ 
  */
 function ppp_disconnect_social() {
 
-	if ( isset( $_GET['ppp_social_disconnect'] ) && isset( $_GET['ppp_network'] ) ) {
+	if (isset($_GET['ppp_social_disconnect']) && isset($_GET['ppp_network'])) {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
+		if ( ! current_user_can( 'manage_options' )) {
 			wp_die( __( 'You do not have permission to view this page', 'ppp-txt' ) );
 		}
 
 		$network = $_GET['ppp_network'];
-		do_action( 'ppp_disconnect-' . $network );
+		do_action( 'ppp_disconnect-'.$network );
 	}
 }
 add_action( 'admin_init', 'ppp_disconnect_social', 10 );
@@ -35,8 +35,8 @@ 
 
 	$tabs = apply_filters( 'ppp_metabox_tabs', array() );
 	$i = 0;
-	foreach ( $tabs as $key => $values ) {
-		if ( $i === 0 ) {
+	foreach ($tabs as $key => $values) {
+		if ($i === 0) {
 			$visibleKey = $key;
 			$class = 'tabs';
 		} else {
@@ -44,7 +44,7 @@ 
 		}
 
 		?><li class="<?php echo $class; ?>"><a href="#<?php echo $key; ?>"><?php
-		if ( $values['class'] !== false ) {
+		if ($values['class'] !== false) {
 			?>
 			<span class="dashicons <?php echo $values['class']; ?>"></span>&nbsp;
 			<?php
@@ -65,15 +65,15 @@ 
 	$tabs = apply_filters( 'ppp_admin_tabs', array() );
 	$i = 0;
 	?><h2 id="ppp-social-connect-tabs" class="nav-tab-wrapper"><?php
-	foreach ( $tabs as $key => $values ) {
-		if ( $i === 0 ) {
+	foreach ($tabs as $key => $values) {
+		if ($i === 0) {
 			$visibleSettingTab = $key;
 			$class = ' nav-tab-active';
 		} else {
 			$class = '';
 		}
 		?><a class="nav-tab<?php echo $class; ?>" href='#<?php echo $key; ?>'><?php
-		if ( $values['class'] !== false ) {
+		if ($values['class'] !== false) {
 			?>
 			<span class="dashicons <?php echo $values['class']; ?>"></span>&nbsp;
 			<?php
@@ -94,14 +94,14 @@ 
 function ppp_generate_metabox_content( $post ) {
 	global $visibleKey;
 	$tab_content = apply_filters( 'ppp_metabox_content', array() );
-	if ( empty( $tab_content ) ) {
+	if (empty($tab_content)) {
 		printf( __( 'No social media accounts active. <a href="%s">Connect with your accounts now</a>.', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-social-settings' ) );
 	} else {
-		foreach ( $tab_content as $service ) {
-			$hidden = ( $visibleKey == $service ) ? '' : ' hidden';
+		foreach ($tab_content as $service) {
+			$hidden = ($visibleKey == $service) ? '' : ' hidden';
 			?>
 			<div class="wp-tab-panel tabs-panel<?php echo $hidden; ?>" id="<?php echo $service; ?>">
-				<?php do_action( 'ppp_generate_metabox_content-' . $service, $post ); ?>
+				<?php do_action( 'ppp_generate_metabox_content-'.$service, $post ); ?>
 			</div>
 			<?php
 		}
@@ -116,14 +116,14 @@ 
 function ppp_generate_social_account_content() {
 	global $visibleSettingTab;
 	$tab_content = apply_filters( 'ppp_admin_social_content', array() );
-	if ( empty( $tab_content ) ) {
+	if (empty($tab_content)) {
 		printf( __( 'No social media accounts active. <a href="%s">Connect with your accounts now</a>.', 'ppp-txt' ), admin_url( 'admin.php?page=ppp-social-settings' ) );
 	} else {
-		foreach ( $tab_content as $service ) {
-			$hidden = ( $visibleSettingTab == $service ) ? '' : ' hidden';
+		foreach ($tab_content as $service) {
+			$hidden = ($visibleSettingTab == $service) ? '' : ' hidden';
 			?>
 			<div class="ppp-social-connect<?php echo $hidden; ?>" id="<?php echo $service; ?>">
-				<?php do_action( 'ppp_connect_display-' . $service ); ?>
+				<?php do_action( 'ppp_connect_display-'.$service ); ?>
 			</div>
 			<?php
 		}
@@ -136,30 +136,30 @@ 
  * @return void
  */
 function ppp_list_view_maybe_take_action() {
-	if ( ! isset( $_GET['page'] ) || $_GET['page'] !== 'ppp-schedule-info' ) {
+	if ( ! isset($_GET['page']) || $_GET['page'] !== 'ppp-schedule-info') {
 		return;
 	}
 
-	if ( ! isset( $_GET['action'] ) ) {
+	if ( ! isset($_GET['action'])) {
 		return;
 	}
 
 	// Get the necessary info for the actions
-	$post_id = isset( $_GET['post_id'] ) ? $_GET['post_id'] : 0;
-	$name    = isset( $_GET['name'] ) ? $_GET['name'] : '';
-	$index   = isset( $_GET['index'] ) ? $_GET['index'] : 0;
-	$delete  = isset( $_GET['delete_too'] ) ? true : false;
+	$post_id = isset($_GET['post_id']) ? $_GET['post_id'] : 0;
+	$name    = isset($_GET['name']) ? $_GET['name'] : '';
+	$index   = isset($_GET['index']) ? $_GET['index'] : 0;
+	$delete  = isset($_GET['delete_too']) ? true : false;
 
-	switch( $_GET['action'] ) {
+	switch ($_GET['action']) {
 		case 'delete_item':
-			if ( ! empty( $post_id ) && ! empty( $name ) || empty( $index ) ) {
-				ppp_remove_scheduled_share( array( (int)$post_id, $name ) ); // Remove the item in cron
+			if ( ! empty($post_id) && ! empty($name) || empty($index)) {
+				ppp_remove_scheduled_share( array( (int) $post_id, $name ) ); // Remove the item in cron
 
 				// Remove the item from postmeta if it exists.
 				$current_post_meta = get_post_meta( $post_id, '_ppp_tweets', true );
 
-				if ( isset( $current_post_meta[$index] ) ) {
-					unset( $current_post_meta[$index] );
+				if (isset($current_post_meta[$index])) {
+					unset($current_post_meta[$index]);
 					update_post_meta( $post_id, '_ppp_tweets', $current_post_meta );
 				}
 
@@ -168,17 +168,17 @@ 
 			}
 			break;
 		case 'share_now':
-			if ( ! empty( $post_id ) && ! empty( $name ) ) {
+			if ( ! empty($post_id) && ! empty($name)) {
 				ppp_share_post( $post_id, $name );
 
-				if ( $delete && ! empty( $index ) ) {
-					ppp_remove_scheduled_share( array( (int)$post_id, $name ) ); // Remove the item in cron
+				if ($delete && ! empty($index)) {
+					ppp_remove_scheduled_share( array( (int) $post_id, $name ) ); // Remove the item in cron
 
 					// Remove the item from postmeta if it exists.
 					$current_post_meta = get_post_meta( $post_id, '_ppp_tweets', true );
 
-					if ( isset( $current_post_meta[$index] ) ) {
-						unset( $current_post_meta[$index] );
+					if (isset($current_post_meta[$index])) {
+						unset($current_post_meta[$index]);
 						update_post_meta( $post_id, '_ppp_tweets', $current_post_meta );
 					}
 

includes/admin/admin-ajax.php

Spacing +4 added lines, -4 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined( 'ABSPATH' )) {
 	exit;
 }
 
@@ -10,7 +10,7 @@ 
 	$date = sanitize_text_field( $_POST['date'] );
 	$time = sanitize_text_field( $_POST['time'] );
 
-	$offset = (int) -( get_option( 'gmt_offset' ) ); // Make the timestamp in the users' timezone, b/c that makes more sense
+	$offset = (int) -(get_option( 'gmt_offset' )); // Make the timestamp in the users' timezone, b/c that makes more sense
 
 	$share_time = explode( ':', $time );
 
@@ -18,11 +18,11 @@ 
 	$minutes = (int) substr( $share_time[1], 0, 2 );
 	$ampm = strtolower( substr( $share_time[1], -2 ) );
 
-	if ( $ampm == 'pm' && $hours != 12 ) {
+	if ($ampm == 'pm' && $hours != 12) {
 		$hours = $hours + 12;
 	}
 
-	if ( $ampm == 'am' && $hours == 12 ) {
+	if ($ampm == 'am' && $hours == 12) {
 		$hours = 00;
 	}