WordImpress / Give

includes/admin/class-i18n-module.php

Spacing +46 added lines, -46 removed lines

@@ -65,18 +65,18 @@ 
 	 *
 	 * @param $args
 	 */
-	public function __construct( $args ) {
+	public function __construct($args) {
 
 		// Only for admins.
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
-		foreach ( $args as $key => $arg ) {
+		foreach ($args as $key => $arg) {
 			$this->$key = $arg;
 		}
 
-		add_action( 'admin_init', array( $this, 'init' ) );
+		add_action('admin_init', array($this, 'init'));
 
 
 	}
@@ -87,19 +87,19 @@ 
 	function init() {
 
 		// First get user's locale (4.7+).
-		$this->locale = function_exists( 'get_user_locale' ) ? get_user_locale() : get_locale();
+		$this->locale = function_exists('get_user_locale') ? get_user_locale() : get_locale();
 
 		// This plugin is en_US native.
-		if ( 'en_US' === $this->locale ) {
+		if ('en_US' === $this->locale) {
 			return;
 		}
 
 		if (
 			! $this->hide_promo()
-			&& ( ! empty( $_GET['post_type'] ) && 'give_forms' === $_GET['post_type'] )
-			&& ( ! empty( $_GET['page'] ) && 'give-settings' === $_GET['page'] )
+			&& ( ! empty($_GET['post_type']) && 'give_forms' === $_GET['post_type'])
+			&& ( ! empty($_GET['page']) && 'give-settings' === $_GET['page'])
 		) {
-			add_action( $this->hook, array( $this, 'promo' ) );
+			add_action($this->hook, array($this, 'promo'));
 		}
 	}
 
@@ -112,11 +112,11 @@ 
 	 * @return bool
 	 */
 	private function hide_promo() {
-		$hide_promo = Give_Cache::get( 'give_i18n_give_promo_hide', true );
-		if ( ! $hide_promo ) {
-			if ( filter_input( INPUT_GET, 'remove_i18n_promo', FILTER_VALIDATE_INT ) === 1 ) {
+		$hide_promo = Give_Cache::get('give_i18n_give_promo_hide', true);
+		if ( ! $hide_promo) {
+			if (filter_input(INPUT_GET, 'remove_i18n_promo', FILTER_VALIDATE_INT) === 1) {
 				// No expiration time, so this would normally not expire, but it wouldn't be copied to other sites etc.
-				Give_Cache::set( 'give_i18n_give_promo_hide', true, null, true );
+				Give_Cache::set('give_i18n_give_promo_hide', true, null, true);
 				$hide_promo = true;
 			}
 		}
@@ -135,20 +135,20 @@ 
 		$message = false;
 
 		// Using a translation less than 90% complete.
-		if ( $this->translation_exists && $this->translation_loaded && $this->percent_translated < 90 ) {
-			$message = __( 'As you can see, there is a translation of this plugin in %1$s. This translation is currently %3$d%% complete. We need your help to make it complete and to fix any errors. Please register at %4$s to help %5$s to %1$s!', 'give' );
-		} elseif ( ! $this->translation_loaded && $this->translation_exists ) {
-			$message = __( 'You\'re using WordPress in %1$s. While %2$s has been %3$d%% translated to %1$s, it has not been shipped with the plugin yet. You can help! Register at %4$s to help complete the translation to %1$s!', 'give' );
-		} elseif ( ! $this->translation_exists ) {
-			$message = __( 'You\'re using WordPress in a language we don\'t support yet. We\'d love for %2$s to be translated in that language too, but unfortunately, it isn\'t right now. You can change that! Register at %4$s to help translate it!', 'give' );
+		if ($this->translation_exists && $this->translation_loaded && $this->percent_translated < 90) {
+			$message = __('As you can see, there is a translation of this plugin in %1$s. This translation is currently %3$d%% complete. We need your help to make it complete and to fix any errors. Please register at %4$s to help %5$s to %1$s!', 'give');
+		} elseif ( ! $this->translation_loaded && $this->translation_exists) {
+			$message = __('You\'re using WordPress in %1$s. While %2$s has been %3$d%% translated to %1$s, it has not been shipped with the plugin yet. You can help! Register at %4$s to help complete the translation to %1$s!', 'give');
+		} elseif ( ! $this->translation_exists) {
+			$message = __('You\'re using WordPress in a language we don\'t support yet. We\'d love for %2$s to be translated in that language too, but unfortunately, it isn\'t right now. You can change that! Register at %4$s to help translate it!', 'give');
 		}
 
 		// Links.
-		$registration_link = sprintf( '<a href="%1$s" target="_blank">%2$s</a>', 'https://wordpress.org/support/register.php', esc_html__( 'WordPress.org', 'give' ) );
-		$translations_link = sprintf( '<a href="%1$s" target="_blank">%2$s</a>', 'https://translate.wordpress.org/projects/wp-plugins/give', esc_html__( 'complete the translation', 'give' ) );
+		$registration_link = sprintf('<a href="%1$s" target="_blank">%2$s</a>', 'https://wordpress.org/support/register.php', esc_html__('WordPress.org', 'give'));
+		$translations_link = sprintf('<a href="%1$s" target="_blank">%2$s</a>', 'https://translate.wordpress.org/projects/wp-plugins/give', esc_html__('complete the translation', 'give'));
 
 		// Message.
-		$message = sprintf( $message, $this->locale_name, 'Give', $this->percent_translated, $registration_link, $translations_link );
+		$message = sprintf($message, $this->locale_name, 'Give', $this->percent_translated, $registration_link, $translations_link);
 
 		return $message;
 
@@ -162,7 +162,7 @@ 
 		$this->translation_details();
 		$message = $this->promo_message();
 
-		if ( $message ) { ?>
+		if ($message) { ?>
 
 			<style>
 				/* Banner specific styles */
@@ -227,12 +227,12 @@ 
 				                                                                                                                             style="font-size: 110px; text-decoration: none;"></span></a>
 
 				<div class="give-i18n-notice-content">
-					<a href="<?php echo esc_url( add_query_arg( array( 'remove_i18n_promo' => '1' ) ) ); ?>" class="dismiss"><span class="dashicons dashicons-dismiss"></span></a>
+					<a href="<?php echo esc_url(add_query_arg(array('remove_i18n_promo' => '1'))); ?>" class="dismiss"><span class="dashicons dashicons-dismiss"></span></a>
 
-					<h2 style="margin: 10px 0;"><?php printf( esc_html__( 'Help Translate Give to %s', 'give' ), $this->locale_name ); ?></h2>
+					<h2 style="margin: 10px 0;"><?php printf(esc_html__('Help Translate Give to %s', 'give'), $this->locale_name); ?></h2>
 					<p><?php echo $message; ?></p>
 					<p>
-						<a href="https://wordpress.org/support/register.php" target="_blank"><?php _e( 'Register now &raquo;', 'give' ); ?></a>
+						<a href="https://wordpress.org/support/register.php" target="_blank"><?php _e('Register now &raquo;', 'give'); ?></a>
 					</p>
 				</div>
 			</div>
@@ -249,11 +249,11 @@ 
 	 */
 	private function find_or_initialize_translation_details() {
 
-		$set = Give_Cache::get( "give_i18n_give_{$this->locale}", true );
+		$set = Give_Cache::get("give_i18n_give_{$this->locale}", true);
 
-		if ( ! $set ) {
+		if ( ! $set) {
 			$set = $this->retrieve_translation_details();
-			Give_Cache::set( "give_i18n_give_{$this->locale}", $set, DAY_IN_SECONDS, true );
+			Give_Cache::set("give_i18n_give_{$this->locale}", $set, DAY_IN_SECONDS, true);
 		}
 
 		return $set;
@@ -267,10 +267,10 @@ 
 	private function translation_details() {
 		$set = $this->find_or_initialize_translation_details();
 
-		$this->translation_exists = ! is_null( $set );
-		$this->translation_loaded = is_textdomain_loaded( 'give' );
+		$this->translation_exists = ! is_null($set);
+		$this->translation_loaded = is_textdomain_loaded('give');
 
-		$this->parse_translation_set( $set );
+		$this->parse_translation_set($set);
 	}
 
 	/**
@@ -282,26 +282,26 @@ 
 	 */
 	private function retrieve_translation_details() {
 
-		$api_url = trailingslashit( $this->glotpress_url );
+		$api_url = trailingslashit($this->glotpress_url);
 
-		$resp = wp_remote_get( $api_url );
+		$resp = wp_remote_get($api_url);
 
-		if ( is_wp_error( $resp ) || wp_remote_retrieve_response_code( $resp ) === '404' ) {
+		if (is_wp_error($resp) || wp_remote_retrieve_response_code($resp) === '404') {
 			return null;
 		}
 
-		$body = wp_remote_retrieve_body( $resp );
-		unset( $resp );
+		$body = wp_remote_retrieve_body($resp);
+		unset($resp);
 
-		if ( $body ) {
-			$body = json_decode( $body );
+		if ($body) {
+			$body = json_decode($body);
 
-			foreach ( $body->translation_sets as $set ) {
-				if ( ! property_exists( $set, 'wp_locale' ) ) {
+			foreach ($body->translation_sets as $set) {
+				if ( ! property_exists($set, 'wp_locale')) {
 					continue;
 				}
 
-				if ( $this->locale == $set->wp_locale ) {
+				if ($this->locale == $set->wp_locale) {
 					return $set;
 				}
 			}
@@ -317,8 +317,8 @@ 
 	 *
 	 * @access private
 	 */
-	private function parse_translation_set( $set ) {
-		if ( $this->translation_exists && is_object( $set ) ) {
+	private function parse_translation_set($set) {
+		if ($this->translation_exists && is_object($set)) {
 			$this->locale_name        = $set->name;
 			$this->percent_translated = $set->percent_translated;
 		} else {
@@ -328,7 +328,7 @@ 
 	}
 }
 
-$give_i18n = new Give_i18n_Banner( array(
+$give_i18n = new Give_i18n_Banner(array(
 		'hook'          => 'admin_notices',
 		'glotpress_url' => 'https://translate.wordpress.org/api/projects/wp-plugins/give/stable/',
-	) );
+	));

includes/payments/actions.php

Spacing +57 added lines, -57 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,21 +28,21 @@ 
  *
  * @return void
  */
-function give_complete_purchase( $payment_id, $new_status, $old_status ) {
+function give_complete_purchase($payment_id, $new_status, $old_status) {
 
 	// Make sure that payments are only completed once.
-	if ( $old_status == 'publish' || $old_status == 'complete' ) {
+	if ($old_status == 'publish' || $old_status == 'complete') {
 		return;
 	}
 
 	// Make sure the payment completion is only processed when new status is complete.
-	if ( $new_status != 'publish' && $new_status != 'complete' ) {
+	if ($new_status != 'publish' && $new_status != 'complete') {
 		return;
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$creation_date  = get_post_field( 'post_date', $payment_id, 'raw' );
+	$creation_date  = get_post_field('post_date', $payment_id, 'raw');
 	$payment_meta   = $payment->payment_meta;
 	$completed_date = $payment->completed_date;
 	$user_info      = $payment->user_info;
@@ -58,12 +58,12 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_pre_complete_donation', $payment_id );
+	do_action('give_pre_complete_donation', $payment_id);
 
 	// Ensure these actions only run once, ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
-		give_record_donation_in_log( $form_id, $payment_id, $price_id, $creation_date );
+		give_record_donation_in_log($form_id, $payment_id, $price_id, $creation_date);
 
 		/**
 		 * Fires after logging donation record.
@@ -74,29 +74,29 @@ 
 		 * @param int $payment_id The ID number of the payment.
 		 * @param array $payment_meta The payment meta.
 		 */
-		do_action( 'give_complete_form_donation', $form_id, $payment_id, $payment_meta );
+		do_action('give_complete_form_donation', $form_id, $payment_id, $payment_meta);
 
 	}
 
 	// Increase the earnings for this form ID.
-	give_increase_earnings( $form_id, $amount );
-	give_increase_donation_count( $form_id );
+	give_increase_earnings($form_id, $amount);
+	give_increase_donation_count($form_id);
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 
 	// Increase the donor's donation stats.
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 	$donor->increase_purchase_count();
-	$donor->increase_value( $amount );
+	$donor->increase_value($amount);
 
-	give_increase_total_earnings( $amount );
+	give_increase_total_earnings($amount);
 
 	// Ensure this action only runs once ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
 		// Save the completed date.
-		$payment->completed_date = current_time( 'mysql' );
+		$payment->completed_date = current_time('mysql');
 		$payment->save();
 
 		/**
@@ -106,12 +106,12 @@ 
 		 *
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_complete_donation', $payment_id );
+		do_action('give_complete_donation', $payment_id);
 	}
 
 }
 
-add_action( 'give_update_payment_status', 'give_complete_purchase', 100, 3 );
+add_action('give_update_payment_status', 'give_complete_purchase', 100, 3);
 
 
 /**
@@ -125,20 +125,20 @@ 
  *
  * @return void
  */
-function give_record_status_change( $payment_id, $new_status, $old_status ) {
+function give_record_status_change($payment_id, $new_status, $old_status) {
 
 	// Get the list of statuses so that status in the payment note can be translated.
 	$stati      = give_get_payment_statuses();
-	$old_status = isset( $stati[ $old_status ] ) ? $stati[ $old_status ] : $old_status;
-	$new_status = isset( $stati[ $new_status ] ) ? $stati[ $new_status ] : $new_status;
+	$old_status = isset($stati[$old_status]) ? $stati[$old_status] : $old_status;
+	$new_status = isset($stati[$new_status]) ? $stati[$new_status] : $new_status;
 
 	// translators: 1: old status 2: new status.
-	$status_change = sprintf( esc_html__( 'Status changed from %1$s to %2$s.', 'give' ), $old_status, $new_status );
+	$status_change = sprintf(esc_html__('Status changed from %1$s to %2$s.', 'give'), $old_status, $new_status);
 
-	give_insert_payment_note( $payment_id, $status_change );
+	give_insert_payment_note($payment_id, $status_change);
 }
 
-add_action( 'give_update_payment_status', 'give_record_status_change', 100, 3 );
+add_action('give_update_payment_status', 'give_record_status_change', 100, 3);
 
 
 /**
@@ -154,25 +154,25 @@ 
  *
  * @return void
  */
-function give_update_old_payments_with_totals( $data ) {
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_upgrade_payments_nonce' ) ) {
+function give_update_old_payments_with_totals($data) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_upgrade_payments_nonce')) {
 		return;
 	}
 
-	if ( get_option( 'give_payment_totals_upgraded' ) ) {
+	if (get_option('give_payment_totals_upgraded')) {
 		return;
 	}
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'offset' => 0,
-		'number' => - 1,
+		'number' => -1,
 		'mode'   => 'all',
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 
-			$payment = new Give_Payment( $payment->ID );
+			$payment = new Give_Payment($payment->ID);
 			$meta    = $payment->get_meta();
 
 			$payment->total = $meta['amount'];
@@ -181,10 +181,10 @@ 
 		}
 	}
 
-	add_option( 'give_payment_totals_upgraded', 1 );
+	add_option('give_payment_totals_upgraded', 1);
 }
 
-add_action( 'give_upgrade_payments', 'give_update_old_payments_with_totals' );
+add_action('give_upgrade_payments', 'give_update_old_payments_with_totals');
 
 /**
  * Mark Abandoned Donations
@@ -198,17 +198,17 @@ 
 function give_mark_abandoned_donations() {
 	$args = array(
 		'status' => 'pending',
-		'number' => - 1,
+		'number' => -1,
 		'output' => 'give_payments',
 	);
 
-	add_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	add_filter('posts_where', 'give_filter_where_older_than_week');
 
-	$payments = give_get_payments( $args );
+	$payments = give_get_payments($args);
 
-	remove_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	remove_filter('posts_where', 'give_filter_where_older_than_week');
 
-	if ( $payments ) {
+	if ($payments) {
 		/**
 		 * Filter payment gateways:  Used to set payment gateways which can be skip while transferring pending payment to abandon.
 		 *
@@ -216,13 +216,13 @@ 
 		 *
 		 * @param array $skip_payment_gateways Array of payment gateways
 		 */
-		$skip_payment_gateways = apply_filters( 'give_mark_abandoned_donation_gateways', array( 'offline' ) );
+		$skip_payment_gateways = apply_filters('give_mark_abandoned_donation_gateways', array('offline'));
 
-		foreach ( $payments as $payment ) {
-			$gateway = give_get_payment_gateway( $payment );
+		foreach ($payments as $payment) {
+			$gateway = give_get_payment_gateway($payment);
 
 			// Skip payment gateways.
-			if ( in_array( $gateway, $skip_payment_gateways ) ) {
+			if (in_array($gateway, $skip_payment_gateways)) {
 				continue;
 			}
 
@@ -232,7 +232,7 @@ 
 	}
 }
 
-Give_Cron::add_weekly_event( 'give_mark_abandoned_donations' );
+Give_Cron::add_weekly_event('give_mark_abandoned_donations');
 
 
 /**
@@ -244,15 +244,15 @@ 
  *
  * @return void
  */
-function give_refresh_thismonth_stat_transients( $payment_ID ) {
+function give_refresh_thismonth_stat_transients($payment_ID) {
 	// Monthly stats.
-	Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+	Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 }
 
-add_action( 'save_post_give_payment', 'give_refresh_thismonth_stat_transients' );
+add_action('save_post_give_payment', 'give_refresh_thismonth_stat_transients');
 
 /*
  * Add meta in payment that store page id and page url.
@@ -264,18 +264,18 @@ 
  *
  * @param {integer} $payment_id Payment id for which the meta value should be updated.
  */
-function give_payment_save_page_data( $payment_id ) {
-	$page_url = ( ! empty( $_REQUEST['give-current-url'] ) ? esc_url( $_REQUEST['give-current-url'] ) : false );
+function give_payment_save_page_data($payment_id) {
+	$page_url = ( ! empty($_REQUEST['give-current-url']) ? esc_url($_REQUEST['give-current-url']) : false);
 	// Check $page_url is not empty.
-	if ( $page_url ) {
-		update_post_meta( $payment_id, '_give_current_url', $page_url );
-		$page_id = url_to_postid( $page_url );
+	if ($page_url) {
+		update_post_meta($payment_id, '_give_current_url', $page_url);
+		$page_id = url_to_postid($page_url);
 		// Check $page_id is not empty.
-		if ( $page_id ) {
-			update_post_meta( $payment_id, '_give_current_page_id', $page_id );
+		if ($page_id) {
+			update_post_meta($payment_id, '_give_current_page_id', $page_id);
 		}
 	}
 }
 
 // Fire when payment is save.
-add_action( 'give_insert_payment', 'give_payment_save_page_data' );
\ No newline at end of file
+add_action('give_insert_payment', 'give_payment_save_page_data');
\ No newline at end of file

includes/class-give-cron.php

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -51,7 +51,7 @@ 
 	 * @return static
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->setup();
 		}
@@ -66,12 +66,12 @@ 
 	 * @since 1.8.13
 	 */
 	private function setup() {
-		add_filter( 'cron_schedules', array( self::$instance, '__add_schedules' ) );
-		add_action( 'wp', array( self::$instance, '__schedule_events' ) );
+		add_filter('cron_schedules', array(self::$instance, '__add_schedules'));
+		add_action('wp', array(self::$instance, '__schedule_events'));
 
 		// Load async event only when cron is running.
-		if( defined( 'DOING_CRON' ) && DOING_CRON ) {
-			add_action( 'init', array( self::$instance, '__load_async_events' ) );
+		if (defined('DOING_CRON') && DOING_CRON) {
+			add_action('init', array(self::$instance, '__load_async_events'));
 		}
 	}
 
@@ -82,23 +82,23 @@ 
 	 * @since 1.8.13
 	 */
 	public function __load_async_events() {
-		$async_events = get_option( 'give_async_events', array() );
+		$async_events = get_option('give_async_events', array());
 
 		// Bailout.
-		if ( empty( $async_events ) ) {
+		if (empty($async_events)) {
 			return;
 		}
 
-		foreach ( $async_events as $index => $event ) {
+		foreach ($async_events as $index => $event) {
 			// Set cron name.
 			$cron_name = "give_async_scheduled_events_{$index}";
 
 			// Setup cron.
-			wp_schedule_single_event( current_time( 'timestamp', 1 ), $cron_name, $event['params'] );
+			wp_schedule_single_event(current_time('timestamp', 1), $cron_name, $event['params']);
 
 			// Add cron action.
-			add_action( $cron_name, $event['callback'], 10, count( $event['params'] ) );
-			add_action( $cron_name, array( $this, '__delete_async_events' ), 9999, 0 );
+			add_action($cron_name, $event['callback'], 10, count($event['params']));
+			add_action($cron_name, array($this, '__delete_async_events'), 9999, 0);
 		}
 	}
 
@@ -108,13 +108,13 @@ 
 	 * @since 1.8.13
 	 */
 	public function __delete_async_events() {
-		$async_events    = get_option( 'give_async_events', array() );
-		$cron_name_parts = explode( '_', current_action() );
-		$cron_id         = end( $cron_name_parts );
+		$async_events    = get_option('give_async_events', array());
+		$cron_name_parts = explode('_', current_action());
+		$cron_id         = end($cron_name_parts);
 
-		if ( ! empty( $async_events[ $cron_id ] ) ) {
-			unset( $async_events[ $cron_id ] );
-			update_option( 'give_async_events', $async_events );
+		if ( ! empty($async_events[$cron_id])) {
+			unset($async_events[$cron_id]);
+			update_option('give_async_events', $async_events);
 		}
 	}
 
@@ -128,11 +128,11 @@ 
 	 *
 	 * @return array            An array of non-default cron schedules.
 	 */
-	public function __add_schedules( $schedules = array() ) {
+	public function __add_schedules($schedules = array()) {
 		// Adds once weekly to the existing schedules.
 		$schedules['weekly'] = array(
 			'interval' => 604800,
-			'display'  => __( 'Once Weekly', 'give' ),
+			'display'  => __('Once Weekly', 'give'),
 		);
 
 		return $schedules;
@@ -160,8 +160,8 @@ 
 	 * @return void
 	 */
 	private function weekly_events() {
-		if ( ! wp_next_scheduled( 'give_weekly_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'weekly', 'give_weekly_scheduled_events' );
+		if ( ! wp_next_scheduled('give_weekly_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'weekly', 'give_weekly_scheduled_events');
 		}
 	}
 
@@ -174,8 +174,8 @@ 
 	 * @return void
 	 */
 	private function daily_events() {
-		if ( ! wp_next_scheduled( 'give_daily_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'daily', 'give_daily_scheduled_events' );
+		if ( ! wp_next_scheduled('give_daily_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'daily', 'give_daily_scheduled_events');
 		}
 	}
 
@@ -189,8 +189,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_cron_action( $type = 'weekly' ) {
-		switch ( $type ) {
+	public static function get_cron_action($type = 'weekly') {
+		switch ($type) {
 			case 'daily':
 				$cron_action = 'give_daily_scheduled_events';
 				break;
@@ -212,9 +212,9 @@ 
 	 * @param        $action
 	 * @param string $type
 	 */
-	private static function add_event( $action, $type = 'weekly' ) {
-		$cron_event = self::get_cron_action( $type );
-		add_action( $cron_event, $action );
+	private static function add_event($action, $type = 'weekly') {
+		$cron_event = self::get_cron_action($type);
+		add_action($cron_event, $action);
 	}
 
 	/**
@@ -225,8 +225,8 @@ 
 	 *
 	 * @param $action
 	 */
-	public static function add_weekly_event( $action ) {
-		self::add_event( $action, 'weekly' );
+	public static function add_weekly_event($action) {
+		self::add_event($action, 'weekly');
 	}
 
 	/**
@@ -237,8 +237,8 @@ 
 	 *
 	 * @param $action
 	 */
-	public static function add_daily_event( $action ) {
-		self::add_event( $action, 'daily' );
+	public static function add_daily_event($action) {
+		self::add_event($action, 'daily');
 	}
 
 	/**
@@ -251,17 +251,17 @@ 
 	 * @param string $action
 	 * @param array  $args
 	 */
-	public static function add_async_event( $action, $args = array() ) {
+	public static function add_async_event($action, $args = array()) {
 
 		// Cache async events.
-		$async_events             = get_option( 'give_async_events', array() );
-		$async_events[ uniqid() ] = array(
+		$async_events             = get_option('give_async_events', array());
+		$async_events[uniqid()] = array(
 			'callback' => $action,
 			'params'   => $args,
 		);
 
 
-		update_option( 'give_async_events', $async_events );
+		update_option('give_async_events', $async_events);
 	}
 }
 

includes/install.php

Spacing +61 added lines, -61 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,15 +26,15 @@ 
  * @global     $wpdb
  * @return void
  */
-function give_install( $network_wide = false ) {
+function give_install($network_wide = false) {
 
 	global $wpdb;
 
-	if ( is_multisite() && $network_wide ) {
+	if (is_multisite() && $network_wide) {
 
-		foreach ( $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs LIMIT 100" ) as $blog_id ) {
+		foreach ($wpdb->get_col("SELECT blog_id FROM $wpdb->blogs LIMIT 100") as $blog_id) {
 
-			switch_to_blog( $blog_id );
+			switch_to_blog($blog_id);
 			give_run_install();
 			restore_current_blog();
 
@@ -61,31 +61,31 @@ 
 	give_setup_post_types();
 
 	// Add Upgraded From Option.
-	$current_version = get_option( 'give_version' );
-	if ( $current_version ) {
-		update_option( 'give_version_upgraded_from', $current_version );
+	$current_version = get_option('give_version');
+	if ($current_version) {
+		update_option('give_version_upgraded_from', $current_version);
 	}
 
 	// Setup some default options.
 	$options = array();
 
 	//Fresh Install? Setup Test Mode, Base Country (US), Test Gateway, Currency.
-	if ( empty( $current_version ) ) {
-		$options = array_merge( $options, give_get_default_settings() );
+	if (empty($current_version)) {
+		$options = array_merge($options, give_get_default_settings());
 	}
 
 	// Populate the default values.
-	update_option( 'give_settings', array_merge( $give_options, $options ) );
+	update_option('give_settings', array_merge($give_options, $options));
 
 	/**
 	 * Run plugin upgrades.
 	 *
 	 * @since 1.8
 	 */
-	do_action( 'give_upgrades' );
+	do_action('give_upgrades');
 
-	if ( GIVE_VERSION !== get_option( 'give_version' ) ) {
-		update_option( 'give_version', GIVE_VERSION );
+	if (GIVE_VERSION !== get_option('give_version')) {
+		update_option('give_version', GIVE_VERSION);
 	}
 
 	// Create Give roles.
@@ -96,7 +96,7 @@ 
 	// Set api version, end point and refresh permalink.
 	$api = new Give_API();
 	$api->add_endpoint();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	flush_rewrite_rules();
 
@@ -105,11 +105,11 @@ 
 	$give_sessions->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created.
-	Give_Cache::set( '_give_installed', $options, 30, true );
+	Give_Cache::set('_give_installed', $options, 30, true);
 
-	if ( ! $current_version ) {
+	if ( ! $current_version) {
 
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		// When new upgrade routines are added, mark them as complete on fresh install.
 		$upgrade_routines = array(
@@ -125,18 +125,18 @@ 
 			'v1813_update_donor_user_roles'
 		);
 
-		foreach ( $upgrade_routines as $upgrade ) {
-			give_set_upgrade_complete( $upgrade );
+		foreach ($upgrade_routines as $upgrade) {
+			give_set_upgrade_complete($upgrade);
 		}
 	}
 
 	// Bail if activating from network, or bulk.
-	if ( is_network_admin() || isset( $_GET['activate-multi'] ) ) {
+	if (is_network_admin() || isset($_GET['activate-multi'])) {
 		return;
 	}
 
 	// Add the transient to redirect.
-	Give_Cache::set( '_give_activation_redirect', true, 30, true );
+	Give_Cache::set('_give_activation_redirect', true, 30, true);
 
 	// Set 'Donation Form' meta box enabled by default.
 	give_nav_donation_metabox_enabled();
@@ -156,11 +156,11 @@ 
  * @param  int $site_id The Site ID.
  * @param  array $meta Blog Meta.
  */
-function give_on_create_blog( $blog_id, $user_id, $domain, $path, $site_id, $meta ) {
+function give_on_create_blog($blog_id, $user_id, $domain, $path, $site_id, $meta) {
 
-	if ( is_plugin_active_for_network( GIVE_PLUGIN_BASENAME ) ) {
+	if (is_plugin_active_for_network(GIVE_PLUGIN_BASENAME)) {
 
-		switch_to_blog( $blog_id );
+		switch_to_blog($blog_id);
 		give_install();
 		restore_current_blog();
 
@@ -168,7 +168,7 @@ 
 
 }
 
-add_action( 'wpmu_new_blog', 'give_on_create_blog', 10, 6 );
+add_action('wpmu_new_blog', 'give_on_create_blog', 10, 6);
 
 
 /**
@@ -181,13 +181,13 @@ 
  *
  * @return array          The tables to drop.
  */
-function give_wpmu_drop_tables( $tables, $blog_id ) {
+function give_wpmu_drop_tables($tables, $blog_id) {
 
-	switch_to_blog( $blog_id );
+	switch_to_blog($blog_id);
 	$donors_db     = new Give_DB_Donors();
 	$donor_meta_db = new Give_DB_Donor_Meta();
 
-	if ( $donors_db->installed() ) {
+	if ($donors_db->installed()) {
 		$tables[] = $donors_db->table_name;
 		$tables[] = $donor_meta_db->table_name;
 	}
@@ -197,7 +197,7 @@ 
 
 }
 
-add_filter( 'wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2 );
+add_filter('wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2);
 
 /**
  * Post-installation
@@ -209,16 +209,16 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options     = Give_Cache::get( '_give_installed', true );
-	$give_table_check = get_option( '_give_table_check', false );
+	$give_options     = Give_Cache::get('_give_installed', true);
+	$give_table_check = get_option('_give_table_check', false);
 
-	if ( false === $give_table_check || current_time( 'timestamp' ) > $give_table_check ) {
+	if (false === $give_table_check || current_time('timestamp') > $give_table_check) {
 
-		if ( ! @Give()->donor_meta->installed() ) {
+		if ( ! @Give()->donor_meta->installed()) {
 
 			// Create the donor meta database.
 			// (this ensures it creates it on multisite instances where it is network activated).
@@ -226,7 +226,7 @@ 
 
 		}
 
-		if ( ! @Give()->donors->installed() ) {
+		if ( ! @Give()->donors->installed()) {
 			// Create the donor database.
 			// (this ensures it creates it on multisite instances where it is network activated).
 			@Give()->donors->create_table();
@@ -238,22 +238,22 @@ 
 			 *
 			 * @param array $give_options Give plugin options.
 			 */
-			do_action( 'give_after_install', $give_options );
+			do_action('give_after_install', $give_options);
 		}
 
-		update_option( '_give_table_check', ( current_time( 'timestamp' ) + WEEK_IN_SECONDS ) );
+		update_option('_give_table_check', (current_time('timestamp') + WEEK_IN_SECONDS));
 
 	}
 
 	// Delete the transient
-	if ( false !== $give_options ) {
-		Give_Cache::delete( Give_Cache::get_key( '_give_installed' ) );
+	if (false !== $give_options) {
+		Give_Cache::delete(Give_Cache::get_key('_give_installed'));
 	}
 
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -268,11 +268,11 @@ 
 
 	global $wp_roles;
 
-	if ( ! is_object( $wp_roles ) ) {
+	if ( ! is_object($wp_roles)) {
 		return;
 	}
 
-	if ( ! array_key_exists( 'give_manager', $wp_roles->roles ) ) {
+	if ( ! array_key_exists('give_manager', $wp_roles->roles)) {
 
 		// Create Give plugin roles
 		$roles = new Give_Roles();
@@ -283,7 +283,7 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
+add_action('admin_init', 'give_install_roles_on_network');
 
 /**
  * Default core setting values.
@@ -321,14 +321,14 @@ 
 		'uninstall_on_delete'                         => 'disabled',
 		'the_content_filter'                          => 'enabled',
 		'scripts_footer'                              => 'disabled',
-		'agree_to_terms_label'                        => __( 'Agree to Terms?', 'give' ),
+		'agree_to_terms_label'                        => __('Agree to Terms?', 'give'),
 		'agreement_text'                              => give_get_default_agreement_text(),
 
 		// Paypal IPN verification.
 		'paypal_verification'                         => 'enabled',
 
 		// Default is manual gateway.
-		'gateways'                                    => array( 'manual' => 1, 'offline' => 1 ),
+		'gateways'                                    => array('manual' => 1, 'offline' => 1),
 		'default_gateway'                             => 'manual',
 
 		// Offline gateway setup.
@@ -356,7 +356,7 @@ 
  */
 function give_get_default_agreement_text() {
 
-	$org_name = get_bloginfo( 'name' );
+	$org_name = get_bloginfo('name');
 
 	$agreement = sprintf(
 		'<p>Acceptance of any contribution, gift or grant is at the discretion of the %1$s. The  %1$s will not accept any gift unless it can be used or expended consistently with the purpose and mission of the  %1$s.</p>
@@ -370,7 +370,7 @@ 
 		$org_name
 	);
 
-	return apply_filters( 'give_get_default_agreement_text', $agreement, $org_name );
+	return apply_filters('give_get_default_agreement_text', $agreement, $org_name);
 }
 
 
@@ -379,22 +379,22 @@ 
  *
  * @since 1.8.11
  */
-function give_create_pages(){
+function give_create_pages() {
 
 	// Bailout if pages already created.
-	if( get_option( 'give_install_pages_created') ) {
+	if (get_option('give_install_pages_created')) {
 		return false;
 	}
 
 	$options = array();
 
 	// Checks if the Success Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'success_page' ) ) ) {
+	if ( ! get_post(give_get_option('success_page'))) {
 
 		// Donation Confirmation (Success) Page
 		$success = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Confirmation', 'give' ),
+				'post_title'     => esc_html__('Donation Confirmation', 'give'),
 				'post_content'   => '[give_receipt]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -408,13 +408,13 @@ 
 	}
 
 	// Checks if the Failure Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'failure_page' ) ) ) {
+	if ( ! get_post(give_get_option('failure_page'))) {
 
 		// Failed Donation Page
 		$failed = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Failed', 'give' ),
-				'post_content'   => esc_html__( 'We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give' ),
+				'post_title'     => esc_html__('Donation Failed', 'give'),
+				'post_content'   => esc_html__('We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -426,11 +426,11 @@ 
 	}
 
 	// Checks if the History Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'history_page' ) ) ) {
+	if ( ! get_post(give_get_option('history_page'))) {
 		// Donation History Page
 		$history = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation History', 'give' ),
+				'post_title'     => esc_html__('Donation History', 'give'),
 				'post_content'   => '[donation_history]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -442,10 +442,10 @@ 
 		$options['history_page'] = $history;
 	}
 
-	if( ! empty( $options ) ) {
-		update_option( 'give_settings', array_merge( give_get_settings(), $options ) );
+	if ( ! empty($options)) {
+		update_option('give_settings', array_merge(give_get_settings(), $options));
 	}
 
-	add_option( 'give_install_pages_created', 1, '', 'no' );
+	add_option('give_install_pages_created', 1, '', 'no');
 }
-add_action( 'admin_init', 'give_create_pages', -1 );
+add_action('admin_init', 'give_create_pages', -1);

includes/filters.php

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,34 +26,34 @@ 
  *
  * @return void
  */
-function give_set_settings_with_disable_prefix( $old_settings, $settings ) {
+function give_set_settings_with_disable_prefix($old_settings, $settings) {
 	// Bailout.
-	if( ! function_exists( 'give_v18_renamed_core_settings' ) ) {
+	if ( ! function_exists('give_v18_renamed_core_settings')) {
 		return;
 	}
 
 	// Get old setting names.
-	$old_settings   = array_flip( give_v18_renamed_core_settings() );
+	$old_settings   = array_flip(give_v18_renamed_core_settings());
 	$update_setting = false;
 
-	foreach ( $settings as $key => $value ) {
+	foreach ($settings as $key => $value) {
 
 		// Check 1. Check if new option is really updated or not.
 		// Check 2. Continue if key is not renamed.
-		if ( ! isset( $old_settings[ $key ] ) ) {
+		if ( ! isset($old_settings[$key])) {
 			continue;
 		}
 
 		// Set old setting.
-		$settings[ $old_settings[ $key ] ] = 'on';
+		$settings[$old_settings[$key]] = 'on';
 
 		// Do not need to set old setting if new setting is not set.
 		if (
-			( give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'disable_' ) ) )
-			|| ( ! give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'enable_' ) ) )
+			(give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'disable_')))
+			|| ( ! give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'enable_')))
 
 		) {
-			unset( $settings[ $old_settings[ $key ] ] );
+			unset($settings[$old_settings[$key]]);
 		}
 
 		// Tell bot to update setting.
@@ -61,8 +61,8 @@ 
 	}
 
 	// Update setting if any old setting set.
-	if ( $update_setting ) {
-		update_option( 'give_settings', $settings );
+	if ($update_setting) {
+		update_option('give_settings', $settings);
 	}
 }
-add_action( 'update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2 );
+add_action('update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2);

includes/class-give-gravatars.php

Spacing +94 added lines, -94 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -62,50 +62,50 @@ 
 	 *
 	 * @return bool If the gravatar exists or not
 	 */
-	public function validate_gravatar( $id_or_email ) {
+	public function validate_gravatar($id_or_email) {
 		//id or email code borrowed from wp-includes/pluggable.php
 		$email = '';
-		if ( is_numeric( $id_or_email ) ) {
+		if (is_numeric($id_or_email)) {
 			$id   = (int) $id_or_email;
-			$user = get_userdata( $id );
-			if ( $user ) {
+			$user = get_userdata($id);
+			if ($user) {
 				$email = $user->user_email;
 			}
-		} elseif ( is_object( $id_or_email ) ) {
+		} elseif (is_object($id_or_email)) {
 			// No avatar for pingbacks or trackbacks
-			$allowed_comment_types = apply_filters( 'get_avatar_comment_types', array( 'comment' ) );
-			if ( ! empty( $id_or_email->comment_type ) && ! in_array( $id_or_email->comment_type, (array) $allowed_comment_types ) ) {
+			$allowed_comment_types = apply_filters('get_avatar_comment_types', array('comment'));
+			if ( ! empty($id_or_email->comment_type) && ! in_array($id_or_email->comment_type, (array) $allowed_comment_types)) {
 				return false;
 			}
 
-			if ( ! empty( $id_or_email->user_id ) ) {
+			if ( ! empty($id_or_email->user_id)) {
 				$id   = (int) $id_or_email->user_id;
-				$user = get_userdata( $id );
-				if ( $user ) {
+				$user = get_userdata($id);
+				if ($user) {
 					$email = $user->user_email;
 				}
-			} elseif ( ! empty( $id_or_email->comment_author_email ) ) {
+			} elseif ( ! empty($id_or_email->comment_author_email)) {
 				$email = $id_or_email->comment_author_email;
 			}
 		} else {
 			$email = $id_or_email;
 		}
 
-		$hashkey = md5( strtolower( trim( $email ) ) );
-		$uri     = 'http://www.gravatar.com/avatar/' . $hashkey . '?d=404';
+		$hashkey = md5(strtolower(trim($email)));
+		$uri     = 'http://www.gravatar.com/avatar/'.$hashkey.'?d=404';
 
-		$data = wp_cache_get( $hashkey );
-		if ( false === $data ) {
-			$response = wp_remote_head( $uri );
-			if ( is_wp_error( $response ) ) {
+		$data = wp_cache_get($hashkey);
+		if (false === $data) {
+			$response = wp_remote_head($uri);
+			if (is_wp_error($response)) {
 				$data = 'not200';
 			} else {
 				$data = $response['response']['code'];
 			}
-			wp_cache_set( $hashkey, $data, $group = '', $expire = 60 * 5 );
+			wp_cache_set($hashkey, $data, $group = '', $expire = 60 * 5);
 
 		}
-		if ( $data == '200' ) {
+		if ($data == '200') {
 			return true;
 		} else {
 			return false;
@@ -122,19 +122,19 @@ 
 	 *
 	 * @return array        IDs if logs, false otherwise
 	 */
-	public function get_log_ids( $form_id = '' ) {
+	public function get_log_ids($form_id = '') {
 
 		// get Give_Logging class
 		global $give_logs;
 
 		// get log for this form
-		$logs = $give_logs->get_logs( $form_id );
+		$logs = $give_logs->get_logs($form_id);
 
-		if ( $logs ) {
+		if ($logs) {
 			$log_ids = array();
 
 			// make an array with all the donor IDs
-			foreach ( $logs as $log ) {
+			foreach ($logs as $log) {
 				$log_ids[] = $log->ID;
 			}
 
@@ -155,51 +155,51 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_payment_ids( $form_id = '' ) {
+	public function get_payment_ids($form_id = '') {
 
 		$give_options = give_get_settings();
 
-		$log_ids = $this->get_log_ids( $form_id );
+		$log_ids = $this->get_log_ids($form_id);
 
-		if ( $log_ids ) {
+		if ($log_ids) {
 
 			$payment_ids = array();
 
-			foreach ( $log_ids as $id ) {
+			foreach ($log_ids as $id) {
 				// get the payment ID for each corresponding log ID
-				$payment_ids[] = give_get_meta( $id, '_give_log_payment_id', true );
+				$payment_ids[] = give_get_meta($id, '_give_log_payment_id', true);
 			}
 
 			// remove donors who have donated more than once so we can have unique avatars
 			$unique_emails = array();
 
-			foreach ( $payment_ids as $key => $id ) {
+			foreach ($payment_ids as $key => $id) {
 
-				$email = give_get_meta( $id, '_give_payment_user_email', true );
+				$email = give_get_meta($id, '_give_payment_user_email', true);
 
-				if ( isset ( $give_options['give_donors_gravatars_has_gravatar_account'] ) ) {
-					if ( ! $this->validate_gravatar( $email ) ) {
+				if (isset ($give_options['give_donors_gravatars_has_gravatar_account'])) {
+					if ( ! $this->validate_gravatar($email)) {
 						continue;
 					}
 				}
 
-				$unique_emails[ $id ] = give_get_meta( $id, '_give_payment_user_email', true );
+				$unique_emails[$id] = give_get_meta($id, '_give_payment_user_email', true);
 
 			}
 
 			$unique_ids = array();
 
 			// strip duplicate emails
-			$unique_emails = array_unique( $unique_emails );
+			$unique_emails = array_unique($unique_emails);
 
 			// convert the unique IDs back into simple array
-			foreach ( $unique_emails as $id => $email ) {
+			foreach ($unique_emails as $id => $email) {
 				$unique_ids[] = $id;
 			}
 
 			// randomize the payment IDs if enabled
-			if ( isset( $give_options['give_donors_gravatars_random_gravatars'] ) ) {
-				shuffle( $unique_ids );
+			if (isset($give_options['give_donors_gravatars_random_gravatars'])) {
+				shuffle($unique_ids);
 			}
 
 			// return our unique IDs
@@ -220,22 +220,22 @@ 
 	 *
 	 * @return string
 	 */
-	public function gravatars( $form_id = false, $title = '' ) {
+	public function gravatars($form_id = false, $title = '') {
 
 		// unique $payment_ids 
-		$payment_ids = $this->get_payment_ids( $form_id );
+		$payment_ids = $this->get_payment_ids($form_id);
 
 		$give_options = give_get_settings();
 
 		// return if no ID
-		if ( ! $form_id ) {
+		if ( ! $form_id) {
 			return;
 		}
 
 		// minimum amount of donations before showing gravatars
 		// if the number of items in array is not greater or equal to the number specified, then exit
-		if ( isset( $give_options['give_donors_gravatars_min_purchases_required'] ) && '' != $give_options['give_donors_gravatars_min_purchases_required'] ) {
-			if ( ! ( count( $payment_ids ) >= $give_options['give_donors_gravatars_min_purchases_required'] ) ) {
+		if (isset($give_options['give_donors_gravatars_min_purchases_required']) && '' != $give_options['give_donors_gravatars_min_purchases_required']) {
+			if ( ! (count($payment_ids) >= $give_options['give_donors_gravatars_min_purchases_required'])) {
 				return;
 			}
 		}
@@ -246,28 +246,28 @@ 
 		echo '<div id="give-purchase-gravatars">';
 
 
-		if ( isset ( $title ) ) {
+		if (isset ($title)) {
 
-			if ( $title ) {
-				echo apply_filters( 'give_donors_gravatars_title', '<h3 class="give-gravatars-title">' . esc_attr( $title ) . '</h3>' );
-			} elseif ( isset( $give_options['give_donors_gravatars_heading'] ) ) {
-				echo apply_filters( 'give_donors_gravatars_title', '<h3 class="give-gravatars-title">' . esc_attr( $give_options['give_donors_gravatars_heading'] ) . '</h2>' );
+			if ($title) {
+				echo apply_filters('give_donors_gravatars_title', '<h3 class="give-gravatars-title">'.esc_attr($title).'</h3>');
+			} elseif (isset($give_options['give_donors_gravatars_heading'])) {
+				echo apply_filters('give_donors_gravatars_title', '<h3 class="give-gravatars-title">'.esc_attr($give_options['give_donors_gravatars_heading']).'</h2>');
 			}
 
 		}
 		echo '<ul class="give-purchase-gravatars-list">';
 		$i = 0;
 
-		if ( $payment_ids ) {
-			foreach ( $payment_ids as $id ) {
+		if ($payment_ids) {
+			foreach ($payment_ids as $id) {
 
 				// Give saves a blank option even when the control is turned off, hence the extra check
-				if ( isset( $give_options['give_donors_gravatars_maximum_number'] ) && '' != $give_options['give_donors_gravatars_maximum_number'] && $i == $give_options['give_donors_gravatars_maximum_number'] ) {
+				if (isset($give_options['give_donors_gravatars_maximum_number']) && '' != $give_options['give_donors_gravatars_maximum_number'] && $i == $give_options['give_donors_gravatars_maximum_number']) {
 					continue;
 				}
 
 				// get the payment meta
-				$payment_meta = give_get_meta( $id, '_give_payment_meta', true );
+				$payment_meta = give_get_meta($id, '_give_payment_meta', true);
 
 				$user_info = $payment_meta['user_info'];
 
@@ -275,21 +275,21 @@ 
 				$name = $user_info['first_name'];
 
 				// get donor's email
-				$email = give_get_meta( $id, '_give_payment_user_email', true );
+				$email = give_get_meta($id, '_give_payment_user_email', true);
 
 				// set gravatar size and provide filter
-				$size = isset( $give_options['give_donors_gravatars_gravatar_size'] ) ? apply_filters( 'give_donors_gravatars_gravatar_size', $give_options['give_donors_gravatars_gravatar_size'] ) : '';
+				$size = isset($give_options['give_donors_gravatars_gravatar_size']) ? apply_filters('give_donors_gravatars_gravatar_size', $give_options['give_donors_gravatars_gravatar_size']) : '';
 
 				// default image
-				$default_image = apply_filters( 'give_donors_gravatars_gravatar_default_image', false );
+				$default_image = apply_filters('give_donors_gravatars_gravatar_default_image', false);
 
 				// assemble output
 				$output .= '<li>';
 
-				$output .= get_avatar( $email, $size, $default_image, $name );
+				$output .= get_avatar($email, $size, $default_image, $name);
 				$output .= '</li>';
 
-				$i ++;
+				$i++;
 
 			} // end foreach
 		}
@@ -298,7 +298,7 @@ 
 		echo '</ul>';
 		echo '</div>';
 
-		return apply_filters( 'give_donors_gravatars', ob_get_clean() );
+		return apply_filters('give_donors_gravatars', ob_get_clean());
 	}
 
 	/**
@@ -310,7 +310,7 @@ 
 	 * @return void
 	 */
 	public function register_widget() {
-		register_widget( 'Give_Donors_Gravatars_Widget' );
+		register_widget('Give_Donors_Gravatars_Widget');
 	}
 
 	/**
@@ -324,19 +324,19 @@ 
 	 *
 	 * @return string
 	 */
-	public function shortcode( $atts, $content = null ) {
+	public function shortcode($atts, $content = null) {
 
-		$atts = shortcode_atts( array(
+		$atts = shortcode_atts(array(
 			'id'    => '',
 			'title' => ''
-		), $atts, 'give_donors_gravatars' );
+		), $atts, 'give_donors_gravatars');
 
 		// if no ID is passed on single give_forms pages, get the correct ID
-		if ( is_singular( 'give_forms' ) ) {
+		if (is_singular('give_forms')) {
 			$id = get_the_ID();
 		}
 
-		$content = $this->gravatars( $atts['id'], $atts['title'] );
+		$content = $this->gravatars($atts['id'], $atts['title']);
 
 		return $content;
 
@@ -352,56 +352,56 @@ 
 	 *
 	 * @return array           Gravatar settings.
 	 */
-	public function settings( $settings ) {
+	public function settings($settings) {
 
 		$give_gravatar_settings = array(
 			array(
-				'name' => esc_html__( 'Donator Gravatars', 'give' ),
+				'name' => esc_html__('Donator Gravatars', 'give'),
 				'desc' => '<hr>',
 				'id'   => 'give_title',
 				'type' => 'give_title'
 			),
 			array(
-				'name' => esc_html__( 'Heading', 'give' ),
-				'desc' => esc_html__( 'The heading to display above the Gravatars.', 'give' ),
+				'name' => esc_html__('Heading', 'give'),
+				'desc' => esc_html__('The heading to display above the Gravatars.', 'give'),
 				'type' => 'text',
 				'id'   => 'give_donors_gravatars_heading'
 			),
 			array(
-				'name'    => esc_html__( 'Gravatar Size', 'give' ),
-				'desc'    => esc_html__( 'The size of each Gravatar in pixels (512px maximum).', 'give' ),
+				'name'    => esc_html__('Gravatar Size', 'give'),
+				'desc'    => esc_html__('The size of each Gravatar in pixels (512px maximum).', 'give'),
 				'type'    => 'text_small',
 				'id'      => 'give_donors_gravatars_gravatar_size',
 				'default' => '64'
 			),
 			array(
-				'name' => esc_html__( 'Minimum Unique Donations Required', 'give' ),
-				'desc' => esc_html__( 'The minimum number of unique donations a form must have before the Gravatars are shown. Leave blank for no minimum.', 'give' ),
+				'name' => esc_html__('Minimum Unique Donations Required', 'give'),
+				'desc' => esc_html__('The minimum number of unique donations a form must have before the Gravatars are shown. Leave blank for no minimum.', 'give'),
 				'type' => 'text_small',
 				'id'   => 'give_donors_gravatars_min_purchases_required',
 			),
 			array(
-				'name'    => esc_html__( 'Maximum Gravatars To Show', 'give' ),
-				'desc'    => esc_html__( 'The maximum number of gravatars to show. Leave blank for no limit.', 'give' ),
+				'name'    => esc_html__('Maximum Gravatars To Show', 'give'),
+				'desc'    => esc_html__('The maximum number of gravatars to show. Leave blank for no limit.', 'give'),
 				'type'    => 'text',
 				'id'      => 'give_donors_gravatars_maximum_number',
 				'default' => '20',
 			),
 			array(
-				'name' => esc_html__( 'Gravatar Visibility', 'give' ),
-				'desc' => esc_html__( 'Show only donors with a Gravatar account.', 'give' ),
+				'name' => esc_html__('Gravatar Visibility', 'give'),
+				'desc' => esc_html__('Show only donors with a Gravatar account.', 'give'),
 				'id'   => 'give_donors_gravatars_has_gravatar_account',
 				'type' => 'checkbox',
 			),
 			array(
-				'name' => esc_html__( 'Randomize Gravatars', 'give' ),
-				'desc' => esc_html__( 'Randomize the Gravatars.', 'give' ),
+				'name' => esc_html__('Randomize Gravatars', 'give'),
+				'desc' => esc_html__('Randomize the Gravatars.', 'give'),
 				'id'   => 'give_donors_gravatars_random_gravatars',
 				'type' => 'checkbox',
 			),
 		);
 
-		return array_merge( $settings, $give_gravatar_settings );
+		return array_merge($settings, $give_gravatar_settings);
 	}
 
 }
@@ -427,7 +427,7 @@ 
 		// widget settings
 		$widget_ops = array(
 			'classname'   => 'give-donors-gravatars',
-			'description' => esc_html__( 'Displays gravatars of people who have donated using your your form. Will only show on the single form page.', 'give' ),
+			'description' => esc_html__('Displays gravatars of people who have donated using your your form. Will only show on the single form page.', 'give'),
 		);
 
 		// widget control settings
@@ -440,7 +440,7 @@ 
 		// create the widget
 		parent::__construct(
 			'give_donors_gravatars_widget',
-			esc_html__( 'Give Donors Gravatars', 'give' ),
+			esc_html__('Give Donors Gravatars', 'give'),
 			$widget_ops,
 			$control_ops
 		);
@@ -460,29 +460,29 @@ 
 	 *
 	 * @return void
 	 */
-	public function widget( $args, $instance ) {
+	public function widget($args, $instance) {
 
 		//@TODO: Don't extract it!!!
-		extract( $args );
+		extract($args);
 
-		if ( ! is_singular( 'give_forms' ) ) {
+		if ( ! is_singular('give_forms')) {
 			return;
 		}
 
 		// Variables from widget settings
-		$title = apply_filters( 'widget_title', $instance['title'] );
+		$title = apply_filters('widget_title', $instance['title']);
 
 		// Used by themes. Opens the widget
 		echo $before_widget;
 
 		// Display the widget title
-		if ( $title ) {
-			echo $before_title . $title . $after_title;
+		if ($title) {
+			echo $before_title.$title.$after_title;
 		}
 
 		$gravatars = new Give_Donors_Gravatars();
 
-		echo $gravatars->gravatars( get_the_ID(), null ); // remove title
+		echo $gravatars->gravatars(get_the_ID(), null); // remove title
 
 		// Used by themes. Closes the widget
 		echo $after_widget;
@@ -502,11 +502,11 @@ 
 	 *
 	 * @return array Updated settings to save.
 	 */
-	public function update( $new_instance, $old_instance ) {
+	public function update($new_instance, $old_instance) {
 
 		$instance = $old_instance;
 
-		$instance['title'] = strip_tags( $new_instance['title'] );
+		$instance['title'] = strip_tags($new_instance['title']);
 
 		return $instance;
 
@@ -524,19 +524,19 @@ 
 	 *
 	 * @return void
 	 */
-	public function form( $instance ) {
+	public function form($instance) {
 
 		// Set up some default widget settings.
 		$defaults = array(
 			'title' => '',
 		);
 
-		$instance = wp_parse_args( (array) $instance, $defaults ); ?>
+		$instance = wp_parse_args((array) $instance, $defaults); ?>
 
 		<!-- Title -->
 		<p>
-			<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'give' ) ?></label>
-			<input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo $instance['title']; ?>" />
+			<label for="<?php echo $this->get_field_id('title'); ?>"><?php esc_html_e('Title:', 'give') ?></label>
+			<input class="widefat" id="<?php echo $this->get_field_id('title'); ?>" name="<?php echo $this->get_field_name('title'); ?>" type="text" value="<?php echo $instance['title']; ?>" />
 		</p>
 
 		<?php

includes/class-give-roles.php

Spacing +60 added lines, -60 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 	 * @access public
 	 */
 	public function __construct() {
-		add_filter( 'give_map_meta_cap', array( $this, 'meta_caps' ), 10, 4 );
+		add_filter('give_map_meta_cap', array($this, 'meta_caps'), 10, 4);
 	}
 
 	/**
@@ -49,7 +49,7 @@ 
 	 * @return void
 	 */
 	public function add_roles() {
-		add_role( 'give_manager', __( 'Give Manager', 'give' ), array(
+		add_role('give_manager', __('Give Manager', 'give'), array(
 			'read'                   => true,
 			'edit_posts'             => true,
 			'delete_posts'           => true,
@@ -78,25 +78,25 @@ 
 			'publish_posts'          => true,
 			'read_private_pages'     => true,
 			'read_private_posts'     => true,
-		) );
+		));
 
-		add_role( 'give_accountant', __( 'Give Accountant', 'give' ), array(
+		add_role('give_accountant', __('Give Accountant', 'give'), array(
 			'read'         => true,
 			'edit_posts'   => false,
 			'delete_posts' => false
-		) );
+		));
 
-		add_role( 'give_worker', __( 'Give Worker', 'give' ), array(
+		add_role('give_worker', __('Give Worker', 'give'), array(
 			'read'         => true,
 			'edit_posts'   => true,
 			'edit_pages'   => true,
 			'upload_files' => true,
 			'delete_posts' => false,
-		) );
+		));
 
-		add_role( 'give_donor', __( 'Give Donor', 'give' ), array(
+		add_role('give_donor', __('Give Donor', 'give'), array(
 			'read'         => true,
-		) );
+		));
 
 	}
 
@@ -115,38 +115,38 @@ 
 	public function add_caps() {
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
-			$wp_roles->add_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'manage_give_settings' );
+		if (is_object($wp_roles)) {
+			$wp_roles->add_cap('give_manager', 'view_give_reports');
+			$wp_roles->add_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->add_cap('give_manager', 'export_give_reports');
+			$wp_roles->add_cap('give_manager', 'manage_give_settings');
 
-			$wp_roles->add_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'manage_give_settings' );
+			$wp_roles->add_cap('administrator', 'view_give_reports');
+			$wp_roles->add_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->add_cap('administrator', 'export_give_reports');
+			$wp_roles->add_cap('administrator', 'manage_give_settings');
 
 			// Add the main post type capabilities.
 			$capabilities = $this->get_core_caps();
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->add_cap( 'administrator', $cap );
-					$wp_roles->add_cap( 'give_manager', $cap );
-					$wp_roles->add_cap( 'give_worker', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->add_cap('administrator', $cap);
+					$wp_roles->add_cap('give_manager', $cap);
+					$wp_roles->add_cap('give_worker', $cap);
 				}
 			}
 
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'read_private_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_payments' );
+			$wp_roles->add_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->add_cap('give_accountant', 'read_private_give_forms');
+			$wp_roles->add_cap('give_accountant', 'view_give_reports');
+			$wp_roles->add_cap('give_accountant', 'export_give_reports');
+			$wp_roles->add_cap('give_accountant', 'edit_give_payments');
 
 		}
 	}
@@ -164,10 +164,10 @@ 
 	public function get_core_caps() {
 		$capabilities = array();
 
-		$capability_types = array( 'give_form', 'give_payment' );
+		$capability_types = array('give_form', 'give_payment');
 
-		foreach ( $capability_types as $capability_type ) {
-			$capabilities[ $capability_type ] = array(
+		foreach ($capability_types as $capability_type) {
+			$capabilities[$capability_type] = array(
 				// Post type.
 				"edit_{$capability_type}",
 				"read_{$capability_type}",
@@ -213,22 +213,22 @@ 
 	 *
 	 * @return array  $caps    Meta capabilities.
 	 */
-	public function meta_caps( $caps, $cap, $user_id, $args ) {
+	public function meta_caps($caps, $cap, $user_id, $args) {
 
-		switch ( $cap ) {
+		switch ($cap) {
 
 			case 'view_give_form_stats' :
 
-				if ( empty( $args[0] ) ) {
+				if (empty($args[0])) {
 					break;
 				}
 
-				$form = get_post( $args[0] );
-				if ( empty( $form ) ) {
+				$form = get_post($args[0]);
+				if (empty($form)) {
 					break;
 				}
 
-				if ( user_can( $user_id, 'view_give_reports' ) || $user_id == $form->post_author ) {
+				if (user_can($user_id, 'view_give_reports') || $user_id == $form->post_author) {
 					$caps = array();
 				}
 
@@ -255,41 +255,41 @@ 
 
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
+		if (is_object($wp_roles)) {
 			// Give Manager Capabilities.
-			$wp_roles->remove_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'manage_give_settings' );
+			$wp_roles->remove_cap('give_manager', 'view_give_reports');
+			$wp_roles->remove_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('give_manager', 'export_give_reports');
+			$wp_roles->remove_cap('give_manager', 'manage_give_settings');
 
 			// Site Administrator Capabilities.
-			$wp_roles->remove_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'manage_give_settings' );
+			$wp_roles->remove_cap('administrator', 'view_give_reports');
+			$wp_roles->remove_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('administrator', 'export_give_reports');
+			$wp_roles->remove_cap('administrator', 'manage_give_settings');
 
 			// Remove the Main Post Type Capabilities.
 			$capabilities = $this->get_core_caps();
 
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->remove_cap( 'give_manager', $cap );
-					$wp_roles->remove_cap( 'administrator', $cap );
-					$wp_roles->remove_cap( 'give_worker', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->remove_cap('give_manager', $cap);
+					$wp_roles->remove_cap('administrator', $cap);
+					$wp_roles->remove_cap('give_worker', $cap);
 				}
 			}
 
 			/** Give Accountant Capabilities */
-			$wp_roles->remove_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'read_private_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_accountant', 'export_give_reports' );
+			$wp_roles->remove_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'read_private_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'view_give_reports');
+			$wp_roles->remove_cap('give_accountant', 'export_give_reports');
 
 		}
 	}

includes/class-give-db-donor-meta.php

Spacing +26 added lines, -26 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,7 +33,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->customermeta = $this->table_name  = $wpdb->prefix . 'give_customermeta';
+		$wpdb->customermeta = $this->table_name = $wpdb->prefix.'give_customermeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -71,13 +71,13 @@ 
 	 *
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single is true.
 	 */
-	public function get_meta( $donor_id = 0, $meta_key = '', $single = false ) {
-		$donor_id = $this->sanitize_donor_id( $donor_id );
-		if ( false === $donor_id ) {
+	public function get_meta($donor_id = 0, $meta_key = '', $single = false) {
+		$donor_id = $this->sanitize_donor_id($donor_id);
+		if (false === $donor_id) {
 			return false;
 		}
 
-		return get_metadata( 'customer', $donor_id, $meta_key, $single );
+		return get_metadata('customer', $donor_id, $meta_key, $single);
 	}
 
 	/**
@@ -95,13 +95,13 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function add_meta( $donor_id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$donor_id = $this->sanitize_donor_id( $donor_id );
-		if ( false === $donor_id ) {
+	public function add_meta($donor_id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$donor_id = $this->sanitize_donor_id($donor_id);
+		if (false === $donor_id) {
 			return false;
 		}
 
-		return add_metadata( 'customer', $donor_id, $meta_key, $meta_value, $unique );
+		return add_metadata('customer', $donor_id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -124,13 +124,13 @@ 
 	 *
 	 * @return  bool                  False on failure, true if success.
 	 */
-	public function update_meta( $donor_id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$donor_id = $this->sanitize_donor_id( $donor_id );
-		if ( false === $donor_id ) {
+	public function update_meta($donor_id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$donor_id = $this->sanitize_donor_id($donor_id);
+		if (false === $donor_id) {
 			return false;
 		}
 
-		return update_metadata( 'customer', $donor_id, $meta_key, $meta_value, $prev_value );
+		return update_metadata('customer', $donor_id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -151,8 +151,8 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $donor_id = 0, $meta_key = '', $meta_value = '' ) {
-		return delete_metadata( 'customer', $donor_id, $meta_key, $meta_value );
+	public function delete_meta($donor_id = 0, $meta_key = '', $meta_value = '') {
+		return delete_metadata('customer', $donor_id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -165,9 +165,9 @@ 
 	 *
 	 * @return  bool  False for failure. True for success.
 	 */
-	public function delete_all_meta( $donor_id = 0 ) {
+	public function delete_all_meta($donor_id = 0) {
 		global $wpdb;
-		$wpdb->delete( $this->table_name, array( 'customer_id' => $donor_id ), array( '%d' ) );
+		$wpdb->delete($this->table_name, array('customer_id' => $donor_id), array('%d'));
 	}
 
 	/**
@@ -180,7 +180,7 @@ 
 	 */
 	public function create_table() {
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
 		$sql = "CREATE TABLE {$this->table_name} (
 			meta_id bigint(20) NOT NULL AUTO_INCREMENT,
@@ -192,9 +192,9 @@ 
 			KEY meta_key (meta_key)
 			) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 	/**
@@ -207,23 +207,23 @@ 
 	 *
 	 * @return int|bool                The normalized donor ID or false if it's found to not be valid.
 	 */
-	private function sanitize_donor_id( $donor_id ) {
-		if ( ! is_numeric( $donor_id ) ) {
+	private function sanitize_donor_id($donor_id) {
+		if ( ! is_numeric($donor_id)) {
 			return false;
 		}
 
 		$donor_id = (int) $donor_id;
 
 		// We were given a non positive number.
-		if ( absint( $donor_id ) !== $donor_id ) {
+		if (absint($donor_id) !== $donor_id) {
 			return false;
 		}
 
-		if ( empty( $donor_id ) ) {
+		if (empty($donor_id)) {
 			return false;
 		}
 
-		return absint( $donor_id );
+		return absint($donor_id);
 
 	}
 

includes/template-functions.php

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,7 +21,7 @@ 
  * @return string
  */
 function give_get_templates_dir() {
-	return GIVE_PLUGIN_DIR . 'templates';
+	return GIVE_PLUGIN_DIR.'templates';
 }
 
 /**
@@ -31,7 +31,7 @@ 
  * @return string
  */
 function give_get_templates_url() {
-	return GIVE_PLUGIN_URL . 'templates';
+	return GIVE_PLUGIN_URL.'templates';
 }
 
 /**
@@ -44,24 +44,24 @@ 
  * @param string $template_path Template file path. Default is empty.
  * @param string $default_path  Default path. Default is empty.
  */
-function give_get_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
-	if ( ! empty( $args ) && is_array( $args ) ) {
-		extract( $args );
+function give_get_template($template_name, $args = array(), $template_path = '', $default_path = '') {
+	if ( ! empty($args) && is_array($args)) {
+		extract($args);
 	}
 
-	$template_names = array( $template_name . '.php' );
+	$template_names = array($template_name.'.php');
 
-	$located = give_locate_template( $template_names, $template_path, $default_path );
+	$located = give_locate_template($template_names, $template_path, $default_path);
 
-	if ( ! file_exists( $located ) ) {
+	if ( ! file_exists($located)) {
 		/* translators: %s: the template */
-		Give()->notices->print_frontend_notice( sprintf( __( 'The %s template was not found.', 'give' ), $located ), true );
+		Give()->notices->print_frontend_notice(sprintf(__('The %s template was not found.', 'give'), $located), true);
 
 		return;
 	}
 
 	// Allow 3rd party plugin filter template file from their plugin.
-	$located = apply_filters( 'give_get_template', $located, $template_name, $args, $template_path, $default_path );
+	$located = apply_filters('give_get_template', $located, $template_name, $args, $template_path, $default_path);
 
 	/**
 	 * Fires in give template, before the file is included.
@@ -75,9 +75,9 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_before_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_before_template_part', $template_name, $template_path, $located, $args);
 
-	include( $located );
+	include($located);
 
 	/**
 	 * Fires in give template, after the file is included.
@@ -91,7 +91,7 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_after_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_after_template_part', $template_name, $template_path, $located, $args);
 }
 
 /**
@@ -107,7 +107,7 @@ 
  *
  * @return string
  */
-function give_get_template_part( $slug, $name = null, $load = true ) {
+function give_get_template_part($slug, $name = null, $load = true) {
 
 	/**
 	 * Fires in give template part, before the template part is retrieved.
@@ -119,20 +119,20 @@ 
 	 * @param string $slug Template part file slug {slug}.php.
 	 * @param string $name Template part file name {slug}-{name}.php.
 	 */
-	do_action( "get_template_part_{$slug}", $slug, $name );
+	do_action("get_template_part_{$slug}", $slug, $name);
 
 	// Setup possible parts
 	$templates = array();
-	if ( isset( $name ) ) {
-		$templates[] = $slug . '-' . $name . '.php';
+	if (isset($name)) {
+		$templates[] = $slug.'-'.$name.'.php';
 	}
-	$templates[] = $slug . '.php';
+	$templates[] = $slug.'.php';
 
 	// Allow template parts to be filtered
-	$templates = apply_filters( 'give_get_template_part', $templates, $slug, $name );
+	$templates = apply_filters('give_get_template_part', $templates, $slug, $name);
 
 	// Return the part that is found
-	return give_locate_template( $templates, $load, false );
+	return give_locate_template($templates, $load, false);
 }
 
 /**
@@ -153,37 +153,37 @@ 
  *
  * @return string The template filename if one is located.
  */
-function give_locate_template( $template_names, $load = false, $require_once = true ) {
+function give_locate_template($template_names, $load = false, $require_once = true) {
 	// No file found yet
 	$located = false;
 
 	// Try to find a template file
-	foreach ( (array) $template_names as $template_name ) {
+	foreach ((array) $template_names as $template_name) {
 
 		// Continue if template is empty
-		if ( empty( $template_name ) ) {
+		if (empty($template_name)) {
 			continue;
 		}
 
 		// Trim off any slashes from the template name
-		$template_name = ltrim( $template_name, '/' );
+		$template_name = ltrim($template_name, '/');
 
 		// try locating this template file by looping through the template paths
-		foreach ( give_get_theme_template_paths() as $template_path ) {
+		foreach (give_get_theme_template_paths() as $template_path) {
 
-			if ( file_exists( $template_path . $template_name ) ) {
-				$located = $template_path . $template_name;
+			if (file_exists($template_path.$template_name)) {
+				$located = $template_path.$template_name;
 				break;
 			}
 		}
 
-		if ( $located ) {
+		if ($located) {
 			break;
 		}
 	}
 
-	if ( ( true == $load ) && ! empty( $located ) ) {
-		load_template( $located, $require_once );
+	if ((true == $load) && ! empty($located)) {
+		load_template($located, $require_once);
 	}
 
 	return $located;
@@ -200,17 +200,17 @@ 
 	$template_dir = give_get_theme_template_dir_name();
 
 	$file_paths = array(
-		1   => trailingslashit( get_stylesheet_directory() ) . $template_dir,
-		10  => trailingslashit( get_template_directory() ) . $template_dir,
+		1   => trailingslashit(get_stylesheet_directory()).$template_dir,
+		10  => trailingslashit(get_template_directory()).$template_dir,
 		100 => give_get_templates_dir(),
 	);
 
-	$file_paths = apply_filters( 'give_template_paths', $file_paths );
+	$file_paths = apply_filters('give_template_paths', $file_paths);
 
 	// sort the file paths based on priority
-	ksort( $file_paths, SORT_NUMERIC );
+	ksort($file_paths, SORT_NUMERIC);
 
-	return array_map( 'trailingslashit', $file_paths );
+	return array_map('trailingslashit', $file_paths);
 }
 
 /**
@@ -222,7 +222,7 @@ 
  * @return string
  */
 function give_get_theme_template_dir_name() {
-	return trailingslashit( apply_filters( 'give_templates_dir', 'give' ) );
+	return trailingslashit(apply_filters('give_templates_dir', 'give'));
 }
 
 /**
@@ -232,10 +232,10 @@ 
  * @return void
  */
 function give_version_in_header() {
-	echo '<meta name="generator" content="Give v' . GIVE_VERSION . '" />' . "\n";
+	echo '<meta name="generator" content="Give v'.GIVE_VERSION.'" />'."\n";
 }
 
-add_action( 'wp_head', 'give_version_in_header' );
+add_action('wp_head', 'give_version_in_header');
 
 /**
  * Determines if we're currently on the Donations History page.
@@ -245,9 +245,9 @@ 
  */
 function give_is_donation_history_page() {
 
-	$ret = is_page( give_get_option( 'history_page' ) );
+	$ret = is_page(give_get_option('history_page'));
 
-	return apply_filters( 'give_is_donation_history_page', $ret );
+	return apply_filters('give_is_donation_history_page', $ret);
 }
 
 /**
@@ -259,25 +259,25 @@ 
  *
  * @return array Modified array of classes
  */
-function give_add_body_classes( $class ) {
+function give_add_body_classes($class) {
 	$classes = (array) $class;
 
-	if ( give_is_success_page() ) {
+	if (give_is_success_page()) {
 		$classes[] = 'give-success';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_failed_transaction_page() ) {
+	if (give_is_failed_transaction_page()) {
 		$classes[] = 'give-failed-transaction';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_donation_history_page() ) {
+	if (give_is_donation_history_page()) {
 		$classes[] = 'give-donation-history';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		$classes[] = 'give-test-mode';
 		$classes[] = 'give-page';
 	}
@@ -286,7 +286,7 @@ 
 	/* @var WP_Theme $current_theme */
 	$current_theme = wp_get_theme();
 
-	switch ( $current_theme->get_template() ) {
+	switch ($current_theme->get_template()) {
 
 		case 'Divi':
 			$classes[] = 'give-divi';
@@ -303,10 +303,10 @@ 
 
 	}
 
-	return array_unique( $classes );
+	return array_unique($classes);
 }
 
-add_filter( 'body_class', 'give_add_body_classes' );
+add_filter('body_class', 'give_add_body_classes');
 
 
 /**
@@ -322,22 +322,22 @@ 
  *
  * @return array
  */
-function give_add_post_class( $classes, $class = '', $post_id = '' ) {
-	if ( ! $post_id || 'give_forms' !== get_post_type( $post_id ) ) {
+function give_add_post_class($classes, $class = '', $post_id = '') {
+	if ( ! $post_id || 'give_forms' !== get_post_type($post_id)) {
 		return $classes;
 	}
 
 	//@TODO: Add classes for custom taxonomy and form configurations (multi vs single donations, etc).
 
-	if ( false !== ( $key = array_search( 'hentry', $classes ) ) ) {
-		unset( $classes[ $key ] );
+	if (false !== ($key = array_search('hentry', $classes))) {
+		unset($classes[$key]);
 	}
 
 	return $classes;
 }
 
 
-add_filter( 'post_class', 'give_add_post_class', 20, 3 );
+add_filter('post_class', 'give_add_post_class', 20, 3);
 
 /**
  * Get the placeholder image URL for forms etc
@@ -347,74 +347,74 @@ 
  */
 function give_get_placeholder_img_src() {
 
-	$placeholder_url = '//placehold.it/600x600&text=' . urlencode( esc_attr__( 'Give Placeholder Image', 'give' ) );
+	$placeholder_url = '//placehold.it/600x600&text='.urlencode(esc_attr__('Give Placeholder Image', 'give'));
 
-	return apply_filters( 'give_placeholder_img_src', $placeholder_url );
+	return apply_filters('give_placeholder_img_src', $placeholder_url);
 }
 
 
 /**
  * Global
  */
-if ( ! function_exists( 'give_output_content_wrapper' ) ) {
+if ( ! function_exists('give_output_content_wrapper')) {
 
 	/**
 	 * Output the start of the page wrapper.
 	 */
 	function give_output_content_wrapper() {
-		give_get_template_part( 'global/wrapper-start' );
+		give_get_template_part('global/wrapper-start');
 	}
 }
-if ( ! function_exists( 'give_output_content_wrapper_end' ) ) {
+if ( ! function_exists('give_output_content_wrapper_end')) {
 
 	/**
 	 * Output the end of the page wrapper.
 	 */
 	function give_output_content_wrapper_end() {
-		give_get_template_part( 'global/wrapper-end' );
+		give_get_template_part('global/wrapper-end');
 	}
 }
 
 /**
  * Single Give Form
  */
-if ( ! function_exists( 'give_left_sidebar_pre_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_pre_wrap')) {
 	function give_left_sidebar_pre_wrap() {
-		echo apply_filters( 'give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">' );
+		echo apply_filters('give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">');
 	}
 }
 
-if ( ! function_exists( 'give_left_sidebar_post_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_post_wrap')) {
 	function give_left_sidebar_post_wrap() {
-		echo apply_filters( 'give_left_sidebar_post_wrap', '</div>' );
+		echo apply_filters('give_left_sidebar_post_wrap', '</div>');
 	}
 }
 
-if ( ! function_exists( 'give_get_forms_sidebar' ) ) {
+if ( ! function_exists('give_get_forms_sidebar')) {
 	function give_get_forms_sidebar() {
-		give_get_template_part( 'single-give-form/sidebar' );
+		give_get_template_part('single-give-form/sidebar');
 	}
 }
 
-if ( ! function_exists( 'give_show_form_images' ) ) {
+if ( ! function_exists('give_show_form_images')) {
 
 	/**
 	 * Output the donation form featured image.
 	 */
 	function give_show_form_images() {
-		if ( give_is_setting_enabled( give_get_option( 'form_featured_img' ) ) ) {
-			give_get_template_part( 'single-give-form/featured-image' );
+		if (give_is_setting_enabled(give_get_option('form_featured_img'))) {
+			give_get_template_part('single-give-form/featured-image');
 		}
 	}
 }
 
-if ( ! function_exists( 'give_template_single_title' ) ) {
+if ( ! function_exists('give_template_single_title')) {
 
 	/**
 	 * Output the form title.
 	 */
 	function give_template_single_title() {
-		give_get_template_part( 'single-give-form/title' );
+		give_get_template_part('single-give-form/title');
 	}
 }
 
@@ -422,7 +422,7 @@ 
  * Conditional Functions
  */
 
-if ( ! function_exists( 'is_give_form' ) ) {
+if ( ! function_exists('is_give_form')) {
 
 	/**
 	 * is_give_form
@@ -434,11 +434,11 @@ 
 	 * @return bool
 	 */
 	function is_give_form() {
-		return is_singular( array( 'give_form' ) );
+		return is_singular(array('give_form'));
 	}
 }
 
-if ( ! function_exists( 'is_give_category' ) ) {
+if ( ! function_exists('is_give_category')) {
 
 	/**
 	 * is_give_category
@@ -453,12 +453,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_category( $term = '' ) {
-		return is_tax( 'give_forms_category', $term );
+	function is_give_category($term = '') {
+		return is_tax('give_forms_category', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_tag' ) ) {
+if ( ! function_exists('is_give_tag')) {
 
 	/**
 	 * is_give_tag
@@ -473,12 +473,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_tag( $term = '' ) {
-		return is_tax( 'give_forms_tag', $term );
+	function is_give_tag($term = '') {
+		return is_tax('give_forms_tag', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_taxonomy' ) ) {
+if ( ! function_exists('is_give_taxonomy')) {
 
 	/**
 	 * is_give_taxonomy
@@ -490,6 +490,6 @@ 
 	 * @return bool
 	 */
 	function is_give_taxonomy() {
-		return is_tax( get_object_taxonomies( 'give_form' ) );
+		return is_tax(get_object_taxonomies('give_form'));
 	}
 }