WordImpress / Give

includes/admin/admin-footer.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,14 +24,14 @@ 
  *
  * @return      string
  */
-function give_admin_rate_us( $footer_text ) {
+function give_admin_rate_us($footer_text) {
 	global $typenow;
 
-	if ( $typenow == 'give_forms' ) {
+	if ($typenow == 'give_forms') {
 		$rate_text = sprintf(
 			/* translators: %s: Link to 5 star rating */
-			__( 'If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give' ),
-			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="' . esc_attr__( 'Thanks :)', 'give' ) . '">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
+			__('If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give'),
+			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="'.esc_attr__('Thanks :)', 'give').'">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
 		);
 
 		return $rate_text;
@@ -40,4 +40,4 @@ 
 	}
 }
 
-add_filter( 'admin_footer_text', 'give_admin_rate_us' );
+add_filter('admin_footer_text', 'give_admin_rate_us');

includes/admin/admin-actions.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
  *
  * @since 1.8
  *
- * @return bool
+ * @return false|null
  */
 function give_redirect_to_clean_url_admin_pages() {
 	// Give admin pages.

Spacing +117 added lines, -118 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,19 +20,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -51,13 +51,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -67,21 +67,21 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');
 
 
 /**
@@ -95,25 +95,25 @@ 
  */
 function give_hide_outdated_php_notice() {
 
-	if ( ! isset( $_POST['_give_hide_outdated_php_notices_shortly'] ) ) {
+	if ( ! isset($_POST['_give_hide_outdated_php_notices_shortly'])) {
 		give_die();
 	}
 
 	// Transient key name.
 	$transient_key = "_give_hide_outdated_php_notices_shortly";
 
-	if ( Give_Cache::get( $transient_key, true ) ) {
+	if (Give_Cache::get($transient_key, true)) {
 		return;
 	}
 
 	// Hide notice for 24 hours.
-	Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+	Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 	give_die();
 
 }
 
-add_action( 'wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice' );
+add_action('wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice');
 
 /**
  * Register admin notices.
@@ -122,43 +122,43 @@ 
  */
 function _give_register_admin_notices() {
 	// Bailout.
-	if( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
 	// Add PHP version update notice
-	if ( function_exists( 'phpversion' ) && version_compare( GIVE_REQUIRED_PHP_VERSION, phpversion(), '>' ) ) {
+	if (function_exists('phpversion') && version_compare(GIVE_REQUIRED_PHP_VERSION, phpversion(), '>')) {
 
-		$notice_desc = '<p><strong>' . __( 'Your site could be faster and more secure with a newer PHP version.', 'give' ) . '</strong></p>';
-		$notice_desc .= '<p>' . __( 'Hey, we\'ve noticed that you\'re running an outdated version of PHP. PHP is the programming language that WordPress and Give are built on. The version that is currently used for your site is no longer supported. Newer versions of PHP are both faster and more secure. In fact, your version of PHP no longer receives security updates, which is why we\'re sending you this notice.', 'give' ) . '</p>';
-		$notice_desc .= '<p>' . __( 'Hosts have the ability to update your PHP version, but sometimes they don\'t dare to do that because they\'re afraid they\'ll break your site.', 'give' ) . '</p>';
-		$notice_desc .= '<p><strong>' . __( 'To which version should I update?', 'give' ) . '</strong></p>';
-		$notice_desc .= '<p>' .  __( 'You should update your PHP version to either 5.6 or to 7.0 or 7.1. On a normal WordPress site, switching to PHP 5.6 should never cause issues. We would however actually recommend you switch to PHP7. There are some plugins that are not ready for PHP7 though, so do some testing first. PHP7 is much faster than PHP 5.6. It\'s also the only PHP version still in active development and therefore the better option for your site in the long run.', 'give' ) . '</p>';
-		$notice_desc .= '<p><strong>' . __( 'Can\'t update? Ask your host!', 'give' ) . '</strong></p>';
-		$notice_desc .= '<p>' . sprintf( __( 'If you cannot upgrade your PHP version yourself, you can send an email to your host. If they don\'t want to upgrade your PHP version, we would suggest you switch hosts. Have a look at one of the recommended %1$sWordPress hosting partners%2$s.', 'give' ), sprintf( '<a href="%1$s" target="_blank">', esc_url( 'https://wordpress.org/hosting/' ) ), '</a>' ) . '</p>';
+		$notice_desc = '<p><strong>'.__('Your site could be faster and more secure with a newer PHP version.', 'give').'</strong></p>';
+		$notice_desc .= '<p>'.__('Hey, we\'ve noticed that you\'re running an outdated version of PHP. PHP is the programming language that WordPress and Give are built on. The version that is currently used for your site is no longer supported. Newer versions of PHP are both faster and more secure. In fact, your version of PHP no longer receives security updates, which is why we\'re sending you this notice.', 'give').'</p>';
+		$notice_desc .= '<p>'.__('Hosts have the ability to update your PHP version, but sometimes they don\'t dare to do that because they\'re afraid they\'ll break your site.', 'give').'</p>';
+		$notice_desc .= '<p><strong>'.__('To which version should I update?', 'give').'</strong></p>';
+		$notice_desc .= '<p>'.__('You should update your PHP version to either 5.6 or to 7.0 or 7.1. On a normal WordPress site, switching to PHP 5.6 should never cause issues. We would however actually recommend you switch to PHP7. There are some plugins that are not ready for PHP7 though, so do some testing first. PHP7 is much faster than PHP 5.6. It\'s also the only PHP version still in active development and therefore the better option for your site in the long run.', 'give').'</p>';
+		$notice_desc .= '<p><strong>'.__('Can\'t update? Ask your host!', 'give').'</strong></p>';
+		$notice_desc .= '<p>'.sprintf(__('If you cannot upgrade your PHP version yourself, you can send an email to your host. If they don\'t want to upgrade your PHP version, we would suggest you switch hosts. Have a look at one of the recommended %1$sWordPress hosting partners%2$s.', 'give'), sprintf('<a href="%1$s" target="_blank">', esc_url('https://wordpress.org/hosting/')), '</a>').'</p>';
 
-		Give()->notices->register_notice( array(
+		Give()->notices->register_notice(array(
 			'id'          => 'give-invalid-php-version',
 			'type'        => 'error',
 			'description' => $notice_desc,
 			'dismissible_type' => 'user',
 			'dismiss_interval' => 'shortly',
-		) );
+		));
 	}
 
 	// Add payment bulk notice.
 	if (
-		current_user_can( 'edit_give_payments' )
-		&& isset( $_GET['action'] )
-		&& ! empty( $_GET['action'] )
-		&& isset( $_GET['payment'] )
-		&& ! empty( $_GET['payment'] )
+		current_user_can('edit_give_payments')
+		&& isset($_GET['action'])
+		&& ! empty($_GET['action'])
+		&& isset($_GET['payment'])
+		&& ! empty($_GET['payment'])
 	) {
-		$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+		$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-		switch ( $_GET['action'] ) {
+		switch ($_GET['action']) {
 			case 'delete':
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'          => 'bulk_action_delete',
 					'type'        => 'updated',
 					'description' => sprintf(
@@ -170,12 +170,12 @@ 
 						),
 						$payment_count ),
 					'show'        => true,
-				) );
+				));
 
 				break;
 
 			case 'resend-receipt':
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'          => 'bulk_action_resend_receipt',
 					'type'        => 'updated',
 					'description' => sprintf(
@@ -188,196 +188,196 @@ 
 						$payment_count
 					),
 					'show'        => true,
-				) );
+				));
 				break;
 		}
 	}
 
 	// Add give message notices.
-	if ( ! empty( $_GET['give-message'] ) ) {
+	if ( ! empty($_GET['give-message'])) {
 		// Donation reports errors.
-		if ( current_user_can( 'view_give_reports' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('view_give_reports')) {
+			switch ($_GET['give-message']) {
 				case 'donation_deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been deleted.', 'give' ),
+						'description' => __('The donation has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'email_sent' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-sent',
 						'type'        => 'updated',
-						'description' => __( 'The donation receipt has been resent.', 'give' ),
+						'description' => __('The donation receipt has been resent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'refreshed-reports' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-refreshed-reports',
 						'type'        => 'updated',
-						'description' => __( 'The reports cache has been cleared.', 'give' ),
+						'description' => __('The reports cache has been cleared.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'donation-note-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-note-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been deleted.', 'give' ),
+						'description' => __('The donation note has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Give settings notices and errors.
-		if ( current_user_can( 'manage_give_settings' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('manage_give_settings')) {
+			switch ($_GET['give-message']) {
 				case 'settings-imported' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-settings-imported',
 						'type'        => 'updated',
-						'description' => __( 'The settings have been imported.', 'give' ),
+						'description' => __('The settings have been imported.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-generated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-generated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been generated.', 'give' ),
+						'description' => __('API keys have been generated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-exists' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-exists',
 						'type'        => 'updated',
-						'description' => __( 'The specified user already has API keys.', 'give' ),
+						'description' => __('The specified user already has API keys.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-regenerated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-regenerated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been regenerated.', 'give' ),
+						'description' => __('API keys have been regenerated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-revoked' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-revoked',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been revoked.', 'give' ),
+						'description' => __('API keys have been revoked.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'sent-test-email' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-sent-test-email',
 						'type'        => 'updated',
-						'description' => __( 'The test email has been sent.', 'give' ),
+						'description' => __('The test email has been sent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'matched-success-failure-page':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-matched-success-failure-page',
 						'type'        => 'updated',
-						'description' => __( 'You cannot set the success and failed pages to the same page', 'give' ),
+						'description' => __('You cannot set the success and failed pages to the same page', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 		// Payments errors.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'note-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-note-added',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been added.', 'give' ),
+						'description' => __('The donation note has been added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'payment-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-updated',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been updated.', 'give' ),
+						'description' => __('The donation has been updated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Donor Notices.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'donor-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donor has been deleted.', 'give' ),
+						'description' => __('The donor has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-added',
 						'type'        => 'updated',
-						'description' => __( 'Donor email added.', 'give' ),
+						'description' => __('Donor email added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-removed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-removed',
 						'type'        => 'updated',
-						'description' => __( 'Donor email removed.', 'give' ),
+						'description' => __('Donor email removed.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-remove-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-remove-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to remove donor email.', 'give' ),
+						'description' => __('Failed to remove donor email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-updated',
 						'type'        => 'updated',
-						'description' => __( 'Primary email updated for donor.', 'give' ),
+						'description' => __('Primary email updated for donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to set primary email.', 'give' ),
+						'description' => __('Failed to set primary email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 }
 
-add_action( 'admin_notices', '_give_register_admin_notices', - 1 );
+add_action('admin_notices', '_give_register_admin_notices', - 1);
 
 
 /**
@@ -387,27 +387,26 @@ 
  *
  * @return bool
  */
-function _give_show_test_mode_notice_in_admin_bar( $wp_admin_bar ) {
-	$is_test_mode = ! empty( $_POST['test_mode'] ) ?
-		give_is_setting_enabled( $_POST['test_mode'] ) :
-		give_is_test_mode();
+function _give_show_test_mode_notice_in_admin_bar($wp_admin_bar) {
+	$is_test_mode = ! empty($_POST['test_mode']) ?
+		give_is_setting_enabled($_POST['test_mode']) : give_is_test_mode();
 
 	if (
-		! current_user_can( 'view_give_reports' ) ||
+		! current_user_can('view_give_reports') ||
 		! $is_test_mode
 	) {
 		return false;
 	}
 
 	// Add the main siteadmin menu item.
-	$wp_admin_bar->add_menu( array(
+	$wp_admin_bar->add_menu(array(
 		'id'     => 'give-test-notice',
-		'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+		'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 		'parent' => 'top-secondary',
-		'title'  => esc_html__( 'Give Test Mode Active', 'give' ),
-		'meta'   => array( 'class' => 'give-test-mode-active' ),
-	) );
+		'title'  => esc_html__('Give Test Mode Active', 'give'),
+		'meta'   => array('class' => 'give-test-mode-active'),
+	));
 
 	return true;
 }
-add_action( 'admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1 );
\ No newline at end of file
+add_action('admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1);
\ No newline at end of file

includes/class-give-logging.php

Doc Comments +1 added lines, -1 removed lines

@@ -463,7 +463,7 @@
 	 * @since  1.7
 	 * @access public
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function delete_cache() {
 		global $wpdb;

Spacing +80 added lines, -80 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
 	 */
 	public function __setup_hooks() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache' ) );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
 	}
 
 	/**
@@ -68,7 +68,7 @@ 
 		/* Logs post type */
 		$log_args = array(
 			'labels'              => array(
-				'name' => esc_html__( 'Logs', 'give' ),
+				'name' => esc_html__('Logs', 'give'),
 			),
 			'public'              => false,
 			'exclude_from_search' => true,
@@ -77,11 +77,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true,
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -95,9 +95,9 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array(
+		register_taxonomy('give_log_type', 'give_log', array(
 			'public' => false,
-		) );
+		));
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 			'api_request',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -132,8 +132,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -152,7 +152,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -160,7 +160,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -177,12 +177,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type,
-		) );
+		));
 	}
 
 	/**
@@ -196,7 +196,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -205,7 +205,7 @@ 
 			'log_type'     => false,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		/**
 		 * Fires before inserting log entry.
@@ -215,20 +215,20 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				give_update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -241,7 +241,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -257,7 +257,7 @@ 
 	 *
 	 * @return bool|null       True if successful, false otherwise.
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
 		/**
 		 * Fires before updating log entry.
@@ -267,7 +267,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -275,15 +275,15 @@ 
 			'post_parent' => 0,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					give_update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
@@ -297,7 +297,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -312,20 +312,20 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false,
 			'date_query'     => null
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -336,28 +336,28 @@ 
 		}
 
 		// Retrieve logs based on specific timeframe
-		if ( !empty ( $query_args['date_query'] ) && is_array( $query_args['date_query'] ) ) {
-			if ( ! empty( $query_args['date_query']['start_date'] ) ) {
-				$query_args['date_query']['after'] =  array(
-					'year'  => date( 'Y', strtotime( $query_args['date_query']['start_date'] ) ),
-					'month' => date( 'm', strtotime( $query_args['date_query']['start_date'] ) ),
-					'day'   => date( 'd', strtotime( $query_args['date_query']['start_date'] ) )
+		if ( ! empty ($query_args['date_query']) && is_array($query_args['date_query'])) {
+			if ( ! empty($query_args['date_query']['start_date'])) {
+				$query_args['date_query']['after'] = array(
+					'year'  => date('Y', strtotime($query_args['date_query']['start_date'])),
+					'month' => date('m', strtotime($query_args['date_query']['start_date'])),
+					'day'   => date('d', strtotime($query_args['date_query']['start_date']))
 				);
 			}
 
-			if ( ! empty( $query_args['date_query']['end_date'] ) ) {
-				$query_args['date_query']['before'] =  array(
-					'year'  => date( 'Y', strtotime( $query_args['date_query']['end_date'] ) ),
-					'month' => date( 'm', strtotime( $query_args['date_query']['end_date'] ) ),
-					'day'   => date( 'd', strtotime( $query_args['date_query']['end_date'] ) )
+			if ( ! empty($query_args['date_query']['end_date'])) {
+				$query_args['date_query']['before'] = array(
+					'year'  => date('Y', strtotime($query_args['date_query']['end_date'])),
+					'month' => date('m', strtotime($query_args['date_query']['end_date'])),
+					'day'   => date('d', strtotime($query_args['date_query']['end_date']))
 				);
 			}
 			$query_args['date_query']['inclusive'] = true;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -380,20 +380,20 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( $object_id ) {
+		if ($object_id) {
 			$query_args['post_parent'] = $object_id;
 		}
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -403,24 +403,24 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
 		// Get cache key for current query.
-		$cache_key = Give_Cache::get_key( 'get_log_count', $query_args );
+		$cache_key = Give_Cache::get_key('get_log_count', $query_args);
 
 		// check if cache already exist or not.
-		if ( ! ( $logs_count = Give_Cache::get( $cache_key ) ) ) {
-			$logs       = new WP_Query( $query_args );
+		if ( ! ($logs_count = Give_Cache::get($cache_key))) {
+			$logs       = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 
 			// Cache results.
-			Give_Cache::set( $cache_key, $logs_count );
+			Give_Cache::set($cache_key, $logs_count);
 		}
 
 		return $logs_count;
@@ -440,16 +440,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -459,15 +459,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -480,9 +480,9 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
+	public function background_process_delete_cache($post_id) {
 		// Delete log cache immediately
-		wp_schedule_single_event( time() - 5, 'give_delete_log_cache' );
+		wp_schedule_single_event(time() - 5, 'give_delete_log_cache');
 	}
 
 	/**
@@ -510,11 +510,11 @@ 
 		);
 
 		// Bailout.
-		if ( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
-		Give_Cache::delete( $cache_option_names );
+		Give_Cache::delete($cache_option_names);
 	}
 }
 
@@ -536,10 +536,10 @@ 
  *
  * @return int             ID of the new log entry.
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	/* @var Give_Logging $give_logs */
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }

includes/emails/class-give-email-tags.php

Doc Comments +1 added lines, -1 removed lines

@@ -530,7 +530,7 @@
  *
  * @param int $payment_id
  *
- * @return int payment_id
+ * @return string payment_id
  */
 function give_email_tag_payment_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );

Spacing +105 added lines, -105 removed lines

@@ -23,7 +23,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,9 +55,9 @@ 
 	 * @param string   $description Email tag description text
 	 * @param callable $func        Hook to run when email tag is found
 	 */
-	public function add( $tag, $description, $func ) {
-		if ( is_callable( $func ) ) {
-			$this->tags[ $tag ] = array(
+	public function add($tag, $description, $func) {
+		if (is_callable($func)) {
+			$this->tags[$tag] = array(
 				'tag'         => $tag,
 				'description' => $description,
 				'func'        => $func
@@ -72,8 +72,8 @@ 
 	 *
 	 * @param string $tag Email tag to remove hook from
 	 */
-	public function remove( $tag ) {
-		unset( $this->tags[ $tag ] );
+	public function remove($tag) {
+		unset($this->tags[$tag]);
 	}
 
 	/**
@@ -85,8 +85,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function email_tag_exists( $tag ) {
-		return array_key_exists( $tag, $this->tags );
+	public function email_tag_exists($tag) {
+		return array_key_exists($tag, $this->tags);
 	}
 
 	/**
@@ -110,16 +110,16 @@ 
 	 *
 	 * @return string Content with email tags filtered out.
 	 */
-	public function do_tags( $content, $payment_id ) {
+	public function do_tags($content, $payment_id) {
 
 		// Check if there is at least one tag added.
-		if ( empty( $this->tags ) || ! is_array( $this->tags ) ) {
+		if (empty($this->tags) || ! is_array($this->tags)) {
 			return $content;
 		}
 
 		$this->payment_id = $payment_id;
 
-		$new_content = preg_replace_callback( "/{([A-z0-9\-\_]+)}/s", array( $this, 'do_tag' ), $content );
+		$new_content = preg_replace_callback("/{([A-z0-9\-\_]+)}/s", array($this, 'do_tag'), $content);
 
 		$this->payment_id = null;
 
@@ -135,17 +135,17 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function do_tag( $m ) {
+	public function do_tag($m) {
 
 		// Get tag
 		$tag = $m[1];
 
 		// Return tag if tag not set
-		if ( ! $this->email_tag_exists( $tag ) ) {
+		if ( ! $this->email_tag_exists($tag)) {
 			return $m[0];
 		}
 
-		return call_user_func( $this->tags[ $tag ]['func'], $this->payment_id, $tag );
+		return call_user_func($this->tags[$tag]['func'], $this->payment_id, $tag);
 	}
 
 }
@@ -159,8 +159,8 @@ 
  * @param string   $description Description of the email tag added
  * @param callable $func        Hook to run when email tag is found
  */
-function give_add_email_tag( $tag, $description, $func ) {
-	Give()->email_tags->add( $tag, $description, $func );
+function give_add_email_tag($tag, $description, $func) {
+	Give()->email_tags->add($tag, $description, $func);
 }
 
 /**
@@ -170,8 +170,8 @@ 
  *
  * @param string $tag Email tag to remove hook from
  */
-function give_remove_email_tag( $tag ) {
-	Give()->email_tags->remove( $tag );
+function give_remove_email_tag($tag) {
+	Give()->email_tags->remove($tag);
 }
 
 /**
@@ -183,8 +183,8 @@ 
  *
  * @return bool
  */
-function give_email_tag_exists( $tag ) {
-	return Give()->email_tags->email_tag_exists( $tag );
+function give_email_tag_exists($tag) {
+	return Give()->email_tags->email_tag_exists($tag);
 }
 
 /**
@@ -211,9 +211,9 @@ 
 	$email_tags = give_get_email_tags();
 
 	ob_start();
-	if ( count( $email_tags ) > 0 ) : ?>
+	if (count($email_tags) > 0) : ?>
 		<div class="give-email-tags-wrap">
-			<?php foreach ( $email_tags as $email_tag ) : ?>
+			<?php foreach ($email_tags as $email_tag) : ?>
 				<span class="give_<?php echo $email_tag['tag']; ?>_tag">
 					<code>{<?php echo $email_tag['tag']; ?>}</code> - <?php echo $email_tag['description']; ?>
 				</span>
@@ -235,12 +235,12 @@ 
  *
  * @return string Content with email tags filtered out.
  */
-function give_do_email_tags( $content, $payment_id ) {
+function give_do_email_tags($content, $payment_id) {
 
 	// Replace all tags
-	$content = Give()->email_tags->do_tags( $content, $payment_id );
+	$content = Give()->email_tags->do_tags($content, $payment_id);
 
-	$content = apply_filters( 'give_email_template_tags', $content, give_get_payment_meta( $payment_id ), $payment_id );
+	$content = apply_filters('give_email_template_tags', $content, give_get_payment_meta($payment_id), $payment_id);
 
 	// Return content
 	return $content;
@@ -259,9 +259,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_add_email_tags' );
+	do_action('give_add_email_tags');
 }
-add_action( 'init', 'give_load_email_tags', - 999 );
+add_action('init', 'give_load_email_tags', - 999);
 
 
 /**
@@ -275,97 +275,97 @@ 
 	$email_tags = array(
 		array(
 			'tag'         => 'donation',
-			'description' => __( 'The donation form name, and the donation level (if applicable).', 'give' ),
+			'description' => __('The donation form name, and the donation level (if applicable).', 'give'),
 			'function'    => 'give_email_tag_donation'
 		),
 		array(
 			'tag'         => 'form_title',
-			'description' => __( 'The donation form name.', 'give' ),
+			'description' => __('The donation form name.', 'give'),
 			'function'    => 'give_email_tag_form_title'
 		),
 		array(
 			'tag'         => 'amount',
-			'description' => __( 'The total donation amount with currency sign.', 'give' ),
+			'description' => __('The total donation amount with currency sign.', 'give'),
 			'function'    => 'give_email_tag_amount'
 		),
 		array(
 			'tag'         => 'price',
-			'description' => __( 'The total donation amount with currency sign.', 'give' ),
+			'description' => __('The total donation amount with currency sign.', 'give'),
 			'function'    => 'give_email_tag_price'
 		),
 		array(
 			'tag'         => 'name',
-			'description' => __( 'The donor\'s first name.', 'give' ),
+			'description' => __('The donor\'s first name.', 'give'),
 			'function'    => 'give_email_tag_first_name'
 		),
 		array(
 			'tag'         => 'fullname',
-			'description' => __( 'The donor\'s full name, first and last.', 'give' ),
+			'description' => __('The donor\'s full name, first and last.', 'give'),
 			'function'    => 'give_email_tag_fullname'
 		),
 		array(
 			'tag'         => 'username',
-			'description' => __( 'The donor\'s user name on the site, if they registered an account.', 'give' ),
+			'description' => __('The donor\'s user name on the site, if they registered an account.', 'give'),
 			'function'    => 'give_email_tag_username'
 		),
 		array(
 			'tag'         => 'user_email',
-			'description' => __( 'The donor\'s email address.', 'give' ),
+			'description' => __('The donor\'s email address.', 'give'),
 			'function'    => 'give_email_tag_user_email'
 		),
 		array(
 			'tag'         => 'billing_address',
-			'description' => __( 'The donor\'s billing address.', 'give' ),
+			'description' => __('The donor\'s billing address.', 'give'),
 			'function'    => 'give_email_tag_billing_address'
 		),
 		array(
 			'tag'         => 'date',
-			'description' => __( 'The date of the donation.', 'give' ),
+			'description' => __('The date of the donation.', 'give'),
 			'function'    => 'give_email_tag_date'
 		),
 		array(
 			'tag'         => 'payment_id',
-			'description' => __( 'The unique ID number for this donation.', 'give' ),
+			'description' => __('The unique ID number for this donation.', 'give'),
 			'function'    => 'give_email_tag_payment_id'
 		),
 		array(
 			'tag'         => 'receipt_id',
-			'description' => __( 'The unique ID number for this donation receipt.', 'give' ),
+			'description' => __('The unique ID number for this donation receipt.', 'give'),
 			'function'    => 'give_email_tag_receipt_id'
 		),
 		array(
 			'tag'         => 'payment_method',
-			'description' => __( 'The method of payment used for this donation.', 'give' ),
+			'description' => __('The method of payment used for this donation.', 'give'),
 			'function'    => 'give_email_tag_payment_method'
 		),
 		array(
 			'tag'         => 'sitename',
-			'description' => __( 'The name of your site.', 'give' ),
+			'description' => __('The name of your site.', 'give'),
 			'function'    => 'give_email_tag_sitename'
 		),
 		array(
 			'tag'         => 'receipt_link',
-			'description' => __( 'The donation receipt direct link, to view the receipt on the website.', 'give' ),
+			'description' => __('The donation receipt direct link, to view the receipt on the website.', 'give'),
 			'function'    => 'give_email_tag_receipt_link'
 		),
 		array(
 			'tag'         => 'receipt_link_url',
-			'description' => __( 'The donation receipt direct URL, to view the receipt on the website.', 'give' ),
+			'description' => __('The donation receipt direct URL, to view the receipt on the website.', 'give'),
 			'function'    => 'give_email_tag_receipt_link_url'
 		),
 	);
 
 	// Apply give_email_tags filter
-	$email_tags = apply_filters( 'give_email_tags', $email_tags );
+	$email_tags = apply_filters('give_email_tags', $email_tags);
 
 	// Add email tags
-	foreach ( $email_tags as $email_tag ) {
-		give_add_email_tag( $email_tag['tag'], $email_tag['description'], $email_tag['function'] );
+	foreach ($email_tags as $email_tag) {
+		give_add_email_tag($email_tag['tag'], $email_tag['description'], $email_tag['function']);
 	}
 
 }
 
-add_action( 'give_add_email_tags', 'give_setup_email_tags' );
+add_action('give_add_email_tags', 'give_setup_email_tags');
 
 
 /**
@@ -377,15 +377,15 @@ 
  *
  * @return string name
  */
-function give_email_tag_first_name( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_first_name($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['name'];
 }
@@ -399,15 +399,15 @@ 
  *
  * @return string fullname
  */
-function give_email_tag_fullname( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_fullname($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['fullname'];
 }
@@ -421,15 +421,15 @@ 
  *
  * @return string username.
  */
-function give_email_tag_username( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_username($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['username'];
 }
@@ -443,8 +443,8 @@ 
  *
  * @return string user_email
  */
-function give_email_tag_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -458,9 +458,9 @@ 
  *
  * @return string billing_address
  */
-function give_email_tag_billing_address( $payment_id ) {
-	$user_info    = give_get_payment_meta_user_info( $payment_id );
-	$user_address = ! empty( $user_info['address'] ) ? $user_info['address'] : array(
+function give_email_tag_billing_address($payment_id) {
+	$user_info    = give_get_payment_meta_user_info($payment_id);
+	$user_address = ! empty($user_info['address']) ? $user_info['address'] : array(
 		'line1'   => '',
 		'line2'   => '',
 		'city'    => '',
@@ -469,11 +469,11 @@ 
 		'zip'     => ''
 	);
 
-	$return = $user_address['line1'] . "\n";
-	if ( ! empty( $user_address['line2'] ) ) {
-		$return .= $user_address['line2'] . "\n";
+	$return = $user_address['line1']."\n";
+	if ( ! empty($user_address['line2'])) {
+		$return .= $user_address['line2']."\n";
 	}
-	$return .= $user_address['city'] . ' ' . $user_address['zip'] . ' ' . $user_address['state'] . "\n";
+	$return .= $user_address['city'].' '.$user_address['zip'].' '.$user_address['state']."\n";
 	$return .= $user_address['country'];
 
 	return $return;
@@ -488,10 +488,10 @@ 
  *
  * @return string date
  */
-function give_email_tag_date( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_date($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return date_i18n( give_date_format(), strtotime( $payment->date ) );
+	return date_i18n(give_date_format(), strtotime($payment->date));
 }
 
 /**
@@ -503,11 +503,11 @@ 
  *
  * @return string amount
  */
-function give_email_tag_amount( $payment_id ) {
-	$payment     = new Give_Payment( $payment_id );
-	$give_amount = give_currency_filter( give_format_amount( $payment->total ), $payment->currency );
+function give_email_tag_amount($payment_id) {
+	$payment     = new Give_Payment($payment_id);
+	$give_amount = give_currency_filter(give_format_amount($payment->total), $payment->currency);
 
-	return html_entity_decode( $give_amount, ENT_COMPAT, 'UTF-8' );
+	return html_entity_decode($give_amount, ENT_COMPAT, 'UTF-8');
 }
 
 /**
@@ -519,8 +519,8 @@ 
  *
  * @return string price
  */
-function give_email_tag_price( $payment_id ) {
-	return give_email_tag_amount( $payment_id );
+function give_email_tag_price($payment_id) {
+	return give_email_tag_amount($payment_id);
 }
 
 /**
@@ -532,8 +532,8 @@ 
  *
  * @return int payment_id
  */
-function give_email_tag_payment_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -547,8 +547,8 @@ 
  *
  * @return string receipt_id
  */
-function give_email_tag_receipt_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_receipt_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -562,14 +562,14 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_donation( $payment_id ) {
-	$payment      = new Give_Payment( $payment_id );
+function give_email_tag_donation($payment_id) {
+	$payment      = new Give_Payment($payment_id);
 	$payment_meta = $payment->payment_meta;
-	$level_title  = give_has_variable_prices( $payment->form_id );
+	$level_title  = give_has_variable_prices($payment->form_id);
 	$separator    = $level_title ? '-' : '';
-	$form_title   = strip_tags( give_get_payment_form_title( $payment_meta, false, $separator ) );
+	$form_title   = strip_tags(give_get_payment_form_title($payment_meta, false, $separator));
 
-	return ! empty( $form_title ) ? $form_title : '';
+	return ! empty($form_title) ? $form_title : '';
 
 }
 
@@ -582,11 +582,11 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_form_title( $payment_id ) {
-	$payment      = new Give_Payment( $payment_id );
+function give_email_tag_form_title($payment_id) {
+	$payment      = new Give_Payment($payment_id);
 	$payment_meta = $payment->payment_meta;
 
-	return isset( $payment_meta['form_title'] ) ? strip_tags( $payment_meta['form_title'] ) : '';
+	return isset($payment_meta['form_title']) ? strip_tags($payment_meta['form_title']) : '';
 
 }
 
@@ -599,10 +599,10 @@ 
  *
  * @return string gateway
  */
-function give_email_tag_payment_method( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_method($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return give_get_gateway_checkout_label( $payment->gateway );
+	return give_get_gateway_checkout_label($payment->gateway);
 }
 
 /**
@@ -614,8 +614,8 @@ 
  *
  * @return string sitename
  */
-function give_email_tag_sitename( $payment_id ) {
-	return wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES );
+function give_email_tag_sitename($payment_id) {
+	return wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES);
 }
 
 /**
@@ -627,19 +627,19 @@ 
  *
  * @return string receipt_link
  */
-function give_email_tag_receipt_link( $payment_id ) {
+function give_email_tag_receipt_link($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt'
-	), home_url() ) );
-	$formatted   = sprintf(
+	), home_url()));
+	$formatted = sprintf(
 		'<a href="%1$s">%2$s</a>',
 		$receipt_url,
-		esc_html__( 'View it in your browser', 'give' )
+		esc_html__('View it in your browser', 'give')
 	);
 
-	if ( give_get_option( 'email_template' ) !== 'none' ) {
+	if (give_get_option('email_template') !== 'none') {
 		return $formatted;
 	} else {
 		return $receipt_url;
@@ -658,12 +658,12 @@ 
  *
  * @return string receipt_url
  */
-function give_email_tag_receipt_link_url( $payment_id ) {
+function give_email_tag_receipt_link_url($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt'
-	), home_url() ) );
+	), home_url()));
 
 	return $receipt_url;
 

includes/forms/functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -155,7 +155,7 @@ 
  *
  * Used to redirect a user back to the donation form if there are errors present.
  *
- * @param array $args
+ * @param string $args
  *
  * @access public
  * @since  1.0
@@ -437,7 +437,7 @@ 
  *
  * @param int $form_id Give Form ID
  *
- * @return int $earnings Earnings for a certain form
+ * @return double $earnings Earnings for a certain form
  */
 function give_get_form_earnings_stats( $form_id = 0 ) {
 	$give_form = new Give_Donate_Form( $form_id );

Spacing +204 added lines, -204 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,14 +23,14 @@ 
 
 	global $typenow;
 
-	if ( $typenow != 'give_forms' ) {
+	if ($typenow != 'give_forms') {
 		return true;
 	}
 
 	return false;
 }
 
-add_filter( 'give_shortcode_button_condition', 'give_shortcode_button_condition' );
+add_filter('give_shortcode_button_condition', 'give_shortcode_button_condition');
 
 
 /**
@@ -40,11 +40,11 @@ 
  *
  * @return int|false
  */
-function get_form_id_from_args( $args ) {
+function get_form_id_from_args($args) {
 
-	if ( isset( $args['form_id'] ) && $args['form_id'] != 0 ) {
+	if (isset($args['form_id']) && $args['form_id'] != 0) {
 
-		return intval( $args['form_id'] );
+		return intval($args['form_id']);
 	}
 
 	return false;
@@ -59,23 +59,23 @@ 
  *
  * @return bool
  */
-function give_is_float_labels_enabled( $args ) {
+function give_is_float_labels_enabled($args) {
 
 	$float_labels = '';
 
-	if ( ! empty( $args['float_labels'] ) ) {
+	if ( ! empty($args['float_labels'])) {
 		$float_labels = $args['float_labels'];
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = give_get_meta( $args['form_id'], '_give_form_floating_labels', true );
+	if (empty($float_labels)) {
+		$float_labels = give_get_meta($args['form_id'], '_give_form_floating_labels', true);
 	}
 
-	if ( empty( $float_labels ) || ( 'global' === $float_labels ) ) {
-		$float_labels = give_get_option( 'floatlabels', 'disabled' );
+	if (empty($float_labels) || ('global' === $float_labels)) {
+		$float_labels = give_get_option('floatlabels', 'disabled');
 	}
 
-	return give_is_setting_enabled( $float_labels );
+	return give_is_setting_enabled($float_labels);
 }
 
 /**
@@ -91,7 +91,7 @@ 
 
 	$can_checkout = true;
 
-	return (bool) apply_filters( 'give_can_checkout', $can_checkout );
+	return (bool) apply_filters('give_can_checkout', $can_checkout);
 }
 
 /**
@@ -105,9 +105,9 @@ 
 function give_get_success_page_uri() {
 	$give_options = give_get_settings();
 
-	$success_page = isset( $give_options['success_page'] ) ? get_permalink( absint( $give_options['success_page'] ) ) : get_bloginfo( 'url' );
+	$success_page = isset($give_options['success_page']) ? get_permalink(absint($give_options['success_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_success_page_uri', $success_page );
+	return apply_filters('give_get_success_page_uri', $success_page);
 }
 
 /**
@@ -119,9 +119,9 @@ 
  */
 function give_is_success_page() {
 	$give_options    = give_get_settings();
-	$is_success_page = isset( $give_options['success_page'] ) ? is_page( $give_options['success_page'] ) : false;
+	$is_success_page = isset($give_options['success_page']) ? is_page($give_options['success_page']) : false;
 
-	return apply_filters( 'give_is_success_page', $is_success_page );
+	return apply_filters('give_is_success_page', $is_success_page);
 }
 
 /**
@@ -135,17 +135,17 @@ 
  * @since       1.0
  * @return      void
  */
-function give_send_to_success_page( $query_string = null ) {
+function give_send_to_success_page($query_string = null) {
 
 	$redirect = give_get_success_page_uri();
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$redirect .= $query_string;
 	}
 
-	$gateway = isset( $_REQUEST['give-gateway'] ) ? $_REQUEST['give-gateway'] : '';
+	$gateway = isset($_REQUEST['give-gateway']) ? $_REQUEST['give-gateway'] : '';
 
-	wp_redirect( apply_filters( 'give_success_page_redirect', $redirect, $gateway, $query_string ) );
+	wp_redirect(apply_filters('give_success_page_redirect', $redirect, $gateway, $query_string));
 	give_die();
 }
 
@@ -161,19 +161,19 @@ 
  * @since  1.0
  * @return Void
  */
-function give_send_back_to_checkout( $args = array() ) {
+function give_send_back_to_checkout($args = array()) {
 
-	$url     = isset( $_POST['give-current-url'] ) ? sanitize_text_field( $_POST['give-current-url'] ) : '';
+	$url     = isset($_POST['give-current-url']) ? sanitize_text_field($_POST['give-current-url']) : '';
 	$form_id = 0;
 
 	// Set the form_id.
-	if ( isset( $_POST['give-form-id'] ) ) {
-		$form_id = sanitize_text_field( $_POST['give-form-id'] );
+	if (isset($_POST['give-form-id'])) {
+		$form_id = sanitize_text_field($_POST['give-form-id']);
 	}
 
 	// Need a URL to continue. If none, redirect back to single form.
-	if ( empty( $url ) ) {
-		wp_safe_redirect( get_permalink( $form_id ) );
+	if (empty($url)) {
+		wp_safe_redirect(get_permalink($form_id));
 		give_die();
 	}
 
@@ -182,41 +182,41 @@ 
 	);
 
 	// Set the $level_id.
-	if ( isset( $_POST['give-price-id'] ) ) {
-		$defaults['level-id'] = sanitize_text_field( $_POST['give-price-id'] );
+	if (isset($_POST['give-price-id'])) {
+		$defaults['level-id'] = sanitize_text_field($_POST['give-price-id']);
 	}
 
 	// Check for backward compatibility.
-	if ( is_string( $args ) ) {
-		$args = str_replace( '?', '', $args );
+	if (is_string($args)) {
+		$args = str_replace('?', '', $args);
 	}
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	// Merge URL query with $args to maintain third-party URL parameters after redirect.
-	$url_data = wp_parse_url( $url );
+	$url_data = wp_parse_url($url);
 
 	// Check if an array to prevent notices before parsing.
-	if ( isset( $url_data['query'] ) && ! empty( $url_data['query'] ) ) {
-		parse_str( $url_data['query'], $query );
+	if (isset($url_data['query']) && ! empty($url_data['query'])) {
+		parse_str($url_data['query'], $query);
 
 		// Precaution: don't allow any CC info.
-		unset( $query['card_number'] );
-		unset( $query['card_cvc'] );
+		unset($query['card_number']);
+		unset($query['card_cvc']);
 
 	} else {
 		// No $url_data so pass empty array.
 		$query = array();
 	}
 
-	$new_query        = array_merge( $args, $query );
-	$new_query_string = http_build_query( $new_query );
+	$new_query        = array_merge($args, $query);
+	$new_query_string = http_build_query($new_query);
 
 	// Assemble URL parts.
-	$redirect = home_url( '/' . $url_data['path'] . '?' . $new_query_string . '#give-form-' . $form_id . '-wrap' );
+	$redirect = home_url('/'.$url_data['path'].'?'.$new_query_string.'#give-form-'.$form_id.'-wrap');
 
 	// Redirect them.
-	wp_safe_redirect( apply_filters( 'give_send_back_to_checkout', $redirect, $args ) );
+	wp_safe_redirect(apply_filters('give_send_back_to_checkout', $redirect, $args));
 	give_die();
 
 }
@@ -232,16 +232,16 @@ 
  * @since       1.0
  * @return      string
  */
-function give_get_success_page_url( $query_string = null ) {
+function give_get_success_page_url($query_string = null) {
 
-	$success_page = give_get_option( 'success_page', 0 );
-	$success_page = get_permalink( $success_page );
+	$success_page = give_get_option('success_page', 0);
+	$success_page = get_permalink($success_page);
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$success_page .= $query_string;
 	}
 
-	return apply_filters( 'give_success_page_url', $success_page );
+	return apply_filters('give_success_page_url', $success_page);
 
 }
 
@@ -254,15 +254,15 @@ 
  *
  * @return mixed Full URL to the Failed Donation Page, if present, home page if it doesn't exist.
  */
-function give_get_failed_transaction_uri( $extras = false ) {
+function give_get_failed_transaction_uri($extras = false) {
 	$give_options = give_get_settings();
 
-	$uri = ! empty( $give_options['failure_page'] ) ? trailingslashit( get_permalink( $give_options['failure_page'] ) ) : home_url();
-	if ( $extras ) {
+	$uri = ! empty($give_options['failure_page']) ? trailingslashit(get_permalink($give_options['failure_page'])) : home_url();
+	if ($extras) {
 		$uri .= $extras;
 	}
 
-	return apply_filters( 'give_get_failed_transaction_uri', $uri );
+	return apply_filters('give_get_failed_transaction_uri', $uri);
 }
 
 /**
@@ -273,9 +273,9 @@ 
  */
 function give_is_failed_transaction_page() {
 	$give_options = give_get_settings();
-	$ret          = isset( $give_options['failure_page'] ) ? is_page( $give_options['failure_page'] ) : false;
+	$ret          = isset($give_options['failure_page']) ? is_page($give_options['failure_page']) : false;
 
-	return apply_filters( 'give_is_failure_page', $ret );
+	return apply_filters('give_is_failure_page', $ret);
 }
 
 /**
@@ -287,18 +287,18 @@ 
  */
 function give_listen_for_failed_payments() {
 
-	$failed_page = give_get_option( 'failure_page', 0 );
+	$failed_page = give_get_option('failure_page', 0);
 
-	if ( ! empty( $failed_page ) && is_page( $failed_page ) && ! empty( $_GET['payment-id'] ) ) {
+	if ( ! empty($failed_page) && is_page($failed_page) && ! empty($_GET['payment-id'])) {
 
-		$payment_id = absint( $_GET['payment-id'] );
-		give_update_payment_status( $payment_id, 'failed' );
+		$payment_id = absint($_GET['payment-id']);
+		give_update_payment_status($payment_id, 'failed');
 
 	}
 
 }
 
-add_action( 'template_redirect', 'give_listen_for_failed_payments' );
+add_action('template_redirect', 'give_listen_for_failed_payments');
 
 /**
  * Retrieve the Donation History page URI
@@ -311,9 +311,9 @@ 
 function give_get_history_page_uri() {
 	$give_options = give_get_settings();
 
-	$history_page = isset( $give_options['history_page'] ) ? get_permalink( absint( $give_options['history_page'] ) ) : get_bloginfo( 'url' );
+	$history_page = isset($give_options['history_page']) ? get_permalink(absint($give_options['history_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_history_page_uri', $history_page );
+	return apply_filters('give_get_history_page_uri', $history_page);
 }
 
 /**
@@ -326,11 +326,11 @@ 
  * @since       1.0
  * @return      bool
  */
-function give_field_is_required( $field = '', $form_id ) {
+function give_field_is_required($field = '', $form_id) {
 
-	$required_fields = give_get_required_fields( $form_id );
+	$required_fields = give_get_required_fields($form_id);
 
-	return array_key_exists( $field, $required_fields );
+	return array_key_exists($field, $required_fields);
 }
 
 /**
@@ -348,14 +348,14 @@ 
  *
  * @return void
  */
-function give_record_donation_in_log( $give_form_id = 0, $payment_id, $price_id = false, $donation_date = null ) {
+function give_record_donation_in_log($give_form_id = 0, $payment_id, $price_id = false, $donation_date = null) {
 	global $give_logs;
 
 	$log_data = array(
 		'post_parent'   => $give_form_id,
 		'log_type'      => 'sale',
-		'post_date'     => isset( $donation_date ) ? $donation_date : null,
-		'post_date_gmt' => isset( $donation_date ) ? $donation_date : null,
+		'post_date'     => isset($donation_date) ? $donation_date : null,
+		'post_date_gmt' => isset($donation_date) ? $donation_date : null,
 	);
 
 	$log_meta = array(
@@ -363,7 +363,7 @@ 
 		'price_id'   => (int) $price_id,
 	);
 
-	$give_logs->insert_log( $log_data, $log_meta );
+	$give_logs->insert_log($log_data, $log_meta);
 }
 
 
@@ -377,11 +377,11 @@ 
  *
  * @return bool|int
  */
-function give_increase_donation_count( $form_id = 0, $quantity = 1 ) {
+function give_increase_donation_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->increase_sales( $quantity );
+	return $form->increase_sales($quantity);
 }
 
 /**
@@ -394,11 +394,11 @@ 
  *
  * @return bool|int
  */
-function give_decrease_donation_count( $form_id = 0, $quantity = 1 ) {
+function give_decrease_donation_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->decrease_sales( $quantity );
+	return $form->decrease_sales($quantity);
 }
 
 /**
@@ -411,10 +411,10 @@ 
  *
  * @return bool|int
  */
-function give_increase_earnings( $give_form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $give_form_id );
+function give_increase_earnings($give_form_id = 0, $amount) {
+	$form = new Give_Donate_Form($give_form_id);
 
-	return $form->increase_earnings( $amount );
+	return $form->increase_earnings($amount);
 }
 
 /**
@@ -427,10 +427,10 @@ 
  *
  * @return bool|int
  */
-function give_decrease_earnings( $form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $form_id );
+function give_decrease_earnings($form_id = 0, $amount) {
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->decrease_earnings( $amount );
+	return $form->decrease_earnings($amount);
 }
 
 
@@ -443,8 +443,8 @@ 
  *
  * @return int $earnings Earnings for a certain form
  */
-function give_get_form_earnings_stats( $form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $form_id );
+function give_get_form_earnings_stats($form_id = 0) {
+	$give_form = new Give_Donate_Form($form_id);
 
 	return $give_form->earnings;
 }
@@ -459,8 +459,8 @@ 
  *
  * @return int $sales Amount of sales for a certain form
  */
-function give_get_form_sales_stats( $give_form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $give_form_id );
+function give_get_form_sales_stats($give_form_id = 0) {
+	$give_form = new Give_Donate_Form($give_form_id);
 
 	return $give_form->sales;
 }
@@ -475,16 +475,16 @@ 
  *
  * @return float $sales Average monthly sales
  */
-function give_get_average_monthly_form_sales( $form_id = 0 ) {
-	$sales        = give_get_form_sales_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_sales($form_id = 0) {
+	$sales        = give_get_form_sales_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$sales = ( $sales / $months );
+	if ($months > 0) {
+		$sales = ($sales / $months);
 	}
 
 	return $sales;
@@ -500,16 +500,16 @@ 
  *
  * @return float $earnings Average monthly earnings
  */
-function give_get_average_monthly_form_earnings( $form_id = 0 ) {
-	$earnings     = give_get_form_earnings_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_earnings($form_id = 0) {
+	$earnings     = give_get_form_earnings_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$earnings = ( $earnings / $months );
+	if ($months > 0) {
+		$earnings = ($earnings / $months);
 	}
 
 	return $earnings < 0 ? 0 : $earnings;
@@ -529,23 +529,23 @@ 
  *
  * @return string $price_name Name of the price option
  */
-function give_get_price_option_name( $form_id = 0, $price_id = 0, $payment_id = 0 ) {
+function give_get_price_option_name($form_id = 0, $price_id = 0, $payment_id = 0) {
 
-	$prices     = give_get_variable_prices( $form_id );
+	$prices     = give_get_variable_prices($form_id);
 	$price_name = '';
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( intval( $price['_give_id']['level_id'] ) == intval( $price_id ) ) {
+		if (intval($price['_give_id']['level_id']) == intval($price_id)) {
 
-			$price_text     = isset( $price['_give_text'] ) ? $price['_give_text'] : '';
-			$price_fallback = give_currency_filter( give_format_amount( $price['_give_amount'] ), '', true );
-			$price_name     = ! empty( $price_text ) ? $price_text : $price_fallback;
+			$price_text     = isset($price['_give_text']) ? $price['_give_text'] : '';
+			$price_fallback = give_currency_filter(give_format_amount($price['_give_amount']), '', true);
+			$price_name     = ! empty($price_text) ? $price_text : $price_fallback;
 
 		}
 	}
 
-	return apply_filters( 'give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id );
+	return apply_filters('give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id);
 }
 
 
@@ -558,23 +558,23 @@ 
  *
  * @return string $range A fully formatted price range
  */
-function give_price_range( $form_id = 0 ) {
-	$low        = give_get_lowest_price_option( $form_id );
-	$high       = give_get_highest_price_option( $form_id );
-	$order_type = ! empty( $_REQUEST['order'] ) ? $_REQUEST['order'] : 'asc';
+function give_price_range($form_id = 0) {
+	$low        = give_get_lowest_price_option($form_id);
+	$high       = give_get_highest_price_option($form_id);
+	$order_type = ! empty($_REQUEST['order']) ? $_REQUEST['order'] : 'asc';
 
 	$range = sprintf(
 		'<span class="give_price_range_%1$s">%2$s</span>
 				<span class="give_price_range_sep">&nbsp;&ndash;&nbsp;</span>
 				<span class="give_price_range_%3$s">%4$s</span>',
 		'asc' === $order_type ? 'low' : 'high',
-		'asc' === $order_type ? give_currency_filter( give_format_amount( $low ) ) : give_currency_filter( give_format_amount( $high ) ),
+		'asc' === $order_type ? give_currency_filter(give_format_amount($low)) : give_currency_filter(give_format_amount($high)),
 		'asc' === $order_type ? 'high' : 'low',
-		'asc' === $order_type ? give_currency_filter( give_format_amount( $high ) ) : give_currency_filter( give_format_amount( $low ) )
+		'asc' === $order_type ? give_currency_filter(give_format_amount($high)) : give_currency_filter(give_format_amount($low))
 
 	);
 
-	return apply_filters( 'give_price_range', $range, $form_id, $low, $high );
+	return apply_filters('give_price_range', $range, $form_id, $low, $high);
 }
 
 
@@ -589,35 +589,35 @@ 
  *
  * @return int ID of the lowest price
  */
-function give_get_lowest_price_id( $form_id = 0 ) {
+function give_get_lowest_price_id($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$min = $min_id = 0;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, $price['_give_amount'] );
+				$min = min($min, $price['_give_amount']);
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $price['_give_id']['level_id'];
 			}
 		}
@@ -635,22 +635,22 @@ 
  *
  * @return float Amount of the lowest price
  */
-function give_get_lowest_price_option( $form_id = 0 ) {
-	if ( empty( $form_id ) ) {
+function give_get_lowest_price_option($form_id = 0) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	if ( ! ( $low = get_post_meta( $form_id, '_give_levels_minimum_amount', true ) ) ) {
+	if ( ! ($low = get_post_meta($form_id, '_give_levels_minimum_amount', true))) {
 		// Backward compatibility.
-		$prices = wp_list_pluck( give_get_variable_prices( $form_id ), '_give_amount' );
-		$low    = ! empty( $prices ) ? min( $prices ) : 0;
+		$prices = wp_list_pluck(give_get_variable_prices($form_id), '_give_amount');
+		$low    = ! empty($prices) ? min($prices) : 0;
 	}
 
-	return give_sanitize_amount( $low );
+	return give_sanitize_amount($low);
 }
 
 /**
@@ -662,23 +662,23 @@ 
  *
  * @return float Amount of the highest price
  */
-function give_get_highest_price_option( $form_id = 0 ) {
+function give_get_highest_price_option($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	if ( ! ( $high = get_post_meta( $form_id, '_give_levels_maximum_amount', true ) ) ) {
+	if ( ! ($high = get_post_meta($form_id, '_give_levels_maximum_amount', true))) {
 		// Backward compatibility.
-		$prices = wp_list_pluck( give_get_variable_prices( $form_id ), '_give_amount' );
-		$high   = ! empty( $prices ) ? max( $prices ) : 0;
+		$prices = wp_list_pluck(give_get_variable_prices($form_id), '_give_amount');
+		$high   = ! empty($prices) ? max($prices) : 0;
 	}
 
-	return give_sanitize_amount( $high );
+	return give_sanitize_amount($high);
 }
 
 /**
@@ -690,15 +690,15 @@ 
  *
  * @return mixed string|int Price of the form
  */
-function give_get_form_price( $form_id = 0 ) {
+function give_get_form_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'price' );
+	return $form->__get('price');
 }
 
 /**
@@ -710,15 +710,15 @@ 
  *
  * @return mixed string|int Minimum price of the form
  */
-function give_get_form_minimum_price( $form_id = 0 ) {
+function give_get_form_minimum_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'minimum_price' );
+	return $form->__get('minimum_price');
 
 }
 
@@ -733,51 +733,51 @@ 
  *
  * @return int $formatted_price
  */
-function give_price( $form_id = 0, $echo = true, $price_id = false ) {
+function give_price($form_id = 0, $echo = true, $price_id = false) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$prices = give_get_variable_prices( $form_id );
+		$prices = give_get_variable_prices($form_id);
 
-		if ( false !== $price_id ) {
+		if (false !== $price_id) {
 
 			// loop through multi-prices to see which is default
-			foreach ( $prices as $price ) {
+			foreach ($prices as $price) {
 				// this is the default price
-				if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+				if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 					$price = (float) $price['_give_amount'];
 				};
 			}
 		} else {
 
-			$price = give_get_lowest_price_option( $form_id );
+			$price = give_get_lowest_price_option($form_id);
 		}
 
-		$price = give_sanitize_amount( $price );
+		$price = give_sanitize_amount($price);
 
 	} else {
 
-		$price = give_get_form_price( $form_id );
+		$price = give_get_form_price($form_id);
 
 	}
 
-	$price           = apply_filters( 'give_form_price', give_sanitize_amount( $price ), $form_id );
-	$formatted_price = '<span class="give_price" id="give_price_' . $form_id . '">' . $price . '</span>';
-	$formatted_price = apply_filters( 'give_form_price_after_html', $formatted_price, $form_id, $price );
+	$price           = apply_filters('give_form_price', give_sanitize_amount($price), $form_id);
+	$formatted_price = '<span class="give_price" id="give_price_'.$form_id.'">'.$price.'</span>';
+	$formatted_price = apply_filters('give_form_price_after_html', $formatted_price, $form_id, $price);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_price;
 	} else {
 		return $formatted_price;
 	}
 }
 
-add_filter( 'give_form_price', 'give_format_amount', 10 );
-add_filter( 'give_form_price', 'give_currency_filter', 20 );
+add_filter('give_form_price', 'give_format_amount', 10);
+add_filter('give_form_price', 'give_currency_filter', 20);
 
 
 /**
@@ -790,19 +790,19 @@ 
  *
  * @return float $amount Amount of the price option
  */
-function give_get_price_option_amount( $form_id = 0, $price_id = 0 ) {
-	$prices = give_get_variable_prices( $form_id );
+function give_get_price_option_amount($form_id = 0, $price_id = 0) {
+	$prices = give_get_variable_prices($form_id);
 
 	$amount = 0.00;
 
-	foreach ( $prices as $price ) {
-		if ( isset( $price['_give_id']['level_id'] ) && $price['_give_id']['level_id'] == $price_id ) {
-			$amount = isset( $price['_give_amount'] ) ? $price['_give_amount'] : 0.00;
+	foreach ($prices as $price) {
+		if (isset($price['_give_id']['level_id']) && $price['_give_id']['level_id'] == $price_id) {
+			$amount = isset($price['_give_amount']) ? $price['_give_amount'] : 0.00;
 			break;
 		};
 	}
 
-	return apply_filters( 'give_get_price_option_amount', give_sanitize_amount( $amount ), $form_id, $price_id );
+	return apply_filters('give_get_price_option_amount', give_sanitize_amount($amount), $form_id, $price_id);
 }
 
 /**
@@ -814,13 +814,13 @@ 
  *
  * @return mixed string|int Goal of the form
  */
-function give_get_form_goal( $form_id = 0 ) {
+function give_get_form_goal($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->goal;
 
@@ -836,27 +836,27 @@ 
  *
  * @return string $formatted_goal
  */
-function give_goal( $form_id = 0, $echo = true ) {
+function give_goal($form_id = 0, $echo = true) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	$goal = give_get_form_goal( $form_id );
+	$goal = give_get_form_goal($form_id);
 
-	$goal           = apply_filters( 'give_form_goal', give_sanitize_amount( $goal ), $form_id );
-	$formatted_goal = '<span class="give_price" id="give_price_' . $form_id . '">' . $goal . '</span>';
-	$formatted_goal = apply_filters( 'give_form_price_after_html', $formatted_goal, $form_id, $goal );
+	$goal           = apply_filters('give_form_goal', give_sanitize_amount($goal), $form_id);
+	$formatted_goal = '<span class="give_price" id="give_price_'.$form_id.'">'.$goal.'</span>';
+	$formatted_goal = apply_filters('give_form_price_after_html', $formatted_goal, $form_id, $goal);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_goal;
 	} else {
 		return $formatted_goal;
 	}
 }
 
-add_filter( 'give_form_goal', 'give_format_amount', 10 );
-add_filter( 'give_form_goal', 'give_currency_filter', 20 );
+add_filter('give_form_goal', 'give_format_amount', 10);
+add_filter('give_form_goal', 'give_currency_filter', 20);
 
 
 /**
@@ -868,15 +868,15 @@ 
  *
  * @return bool  $ret Whether or not the logged_in_only setting is set
  */
-function give_logged_in_only( $form_id ) {
+function give_logged_in_only($form_id) {
 	// If _give_logged_in_only is set to enable then guest can donate from that specific form.
 	// Otherwise it is member only donation form.
-	$val = give_get_meta( $form_id, '_give_logged_in_only', true );
-	$val = ! empty( $val ) ? $val : 'enabled';
+	$val = give_get_meta($form_id, '_give_logged_in_only', true);
+	$val = ! empty($val) ? $val : 'enabled';
 
-	$ret = ! give_is_setting_enabled( $val );
+	$ret = ! give_is_setting_enabled($val);
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret, $form_id );
+	return (bool) apply_filters('give_logged_in_only', $ret, $form_id);
 }
 
 
@@ -889,11 +889,11 @@ 
  *
  * @return string
  */
-function give_show_login_register_option( $form_id ) {
+function give_show_login_register_option($form_id) {
 
-	$show_register_form = give_get_meta( $form_id, '_give_show_register_form', true );
+	$show_register_form = give_get_meta($form_id, '_give_show_register_form', true);
 
-	return apply_filters( 'give_show_register_form', $show_register_form, $form_id );
+	return apply_filters('give_show_register_form', $show_register_form, $form_id);
 
 }
 
@@ -909,12 +909,12 @@ 
  *
  * @return array
  */
-function _give_get_prefill_form_field_values( $form_id ) {
+function _give_get_prefill_form_field_values($form_id) {
 	$logged_in_donor_info = array();
 
-	if ( is_user_logged_in() ) :
-		$donor_data    = get_userdata( get_current_user_id() );
-		$donor_address = get_user_meta( get_current_user_id(), '_give_user_address', true );
+	if (is_user_logged_in()) :
+		$donor_data    = get_userdata(get_current_user_id());
+		$donor_address = get_user_meta(get_current_user_id(), '_give_user_address', true);
 
 		$logged_in_donor_info = array(
 			// First name.
@@ -927,42 +927,42 @@ 
 			'give_email'      => $donor_data->user_email,
 
 			// Street address 1.
-			'card_address'    => ( ! empty( $donor_address['line1'] ) ? $donor_address['line1'] : '' ),
+			'card_address'    => ( ! empty($donor_address['line1']) ? $donor_address['line1'] : ''),
 
 			// Street address 2.
-			'card_address_2'  => ( ! empty( $donor_address['line2'] ) ? $donor_address['line2'] : '' ),
+			'card_address_2'  => ( ! empty($donor_address['line2']) ? $donor_address['line2'] : ''),
 
 			// Country.
-			'billing_country' => ( ! empty( $donor_address['country'] ) ? $donor_address['country'] : '' ),
+			'billing_country' => ( ! empty($donor_address['country']) ? $donor_address['country'] : ''),
 
 			// State.
-			'card_state'      => ( ! empty( $donor_address['state'] ) ? $donor_address['state'] : '' ),
+			'card_state'      => ( ! empty($donor_address['state']) ? $donor_address['state'] : ''),
 
 			// City.
-			'card_city'       => ( ! empty( $donor_address['city'] ) ? $donor_address['city'] : '' ),
+			'card_city'       => ( ! empty($donor_address['city']) ? $donor_address['city'] : ''),
 
 			// Zipcode
-			'card_zip'        => ( ! empty( $donor_address['zip'] ) ? $donor_address['zip'] : '' ),
+			'card_zip'        => ( ! empty($donor_address['zip']) ? $donor_address['zip'] : ''),
 		);
 	endif;
 
 	// Bailout: Auto fill form field values only form form which donor is donating.
 	if (
-		empty( $_GET['form-id'] )
+		empty($_GET['form-id'])
 		|| ! $form_id
-		|| ( $form_id !== absint( $_GET['form-id'] ) )
+		|| ($form_id !== absint($_GET['form-id']))
 	) {
 		return $logged_in_donor_info;
 	}
 
 	// Get purchase data.
-	$give_purchase_data = Give()->session->get( 'give_purchase' );
+	$give_purchase_data = Give()->session->get('give_purchase');
 
 	// Get donor info from form data.
-	$give_donor_info_in_session = empty( $give_purchase_data['post_data'] )
+	$give_donor_info_in_session = empty($give_purchase_data['post_data'])
 		? array()
 		: $give_purchase_data['post_data'];
 
 	// Output.
-	return wp_parse_args( $give_donor_info_in_session, $logged_in_donor_info );
+	return wp_parse_args($give_donor_info_in_session, $logged_in_donor_info);
 }

includes/payments/class-give-payment.php

Doc Comments +3 added lines, -3 removed lines

@@ -392,7 +392,7 @@ 
 	 *
 	 * @param  int|bool $payment_id A given payment
 	 *
-	 * @return mixed void|false
+	 * @return false|null void|false
 	 */
 	public function __construct( $payment_id = false ) {
 
@@ -463,7 +463,7 @@ 
 	 *
 	 * @param  string $name The attribute to get
 	 *
-	 * @return boolean       If the item is set or not
+	 * @return boolean|null       If the item is set or not
 	 */
 	public function __isset( $name ) {
 		if ( property_exists( $this, $name ) ) {
@@ -1302,7 +1302,7 @@ 
 	 *
 	 * @param  string $note The note to add
 	 *
-	 * @return void
+	 * @return false|null
 	 */
 	public function add_note( $note = false ) {
 		// Bail if no note specified.

Indentation +1 added lines, -1 removed lines

@@ -1050,7 +1050,7 @@
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
 					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
-					     && $args['price_id'] == $price['_give_id']['level_id']
+						 && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}

Spacing +281 added lines, -281 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -371,13 +371,13 @@ 
 	 *
 	 * @return mixed void|false
 	 */
-	public function __construct( $payment_id = false ) {
+	public function __construct($payment_id = false) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -390,11 +390,11 @@ 
 	 *
 	 * @return mixed        The value.
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			$value = call_user_func( array( $this, 'get_' . $key ) );
+			$value = call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
@@ -416,18 +416,18 @@ 
 	 * @param  string $key The property name
 	 * @param  mixed  $value The value of the property
 	 */
-	public function __set( $key, $value ) {
-		$ignore = array( '_ID' );
+	public function __set($key, $value) {
+		$ignore = array('_ID');
 
-		if ( $key === 'status' ) {
+		if ($key === 'status') {
 			$this->old_status = $this->status;
 		}
 
-		if ( ! in_array( $key, $ignore ) ) {
-			$this->pending[ $key ] = $value;
+		if ( ! in_array($key, $ignore)) {
+			$this->pending[$key] = $value;
 		}
 
-		if ( '_ID' !== $key ) {
+		if ('_ID' !== $key) {
 			$this->$key = $value;
 		}
 	}
@@ -442,9 +442,9 @@ 
 	 *
 	 * @return boolean       If the item is set or not
 	 */
-	public function __isset( $name ) {
-		if ( property_exists( $this, $name ) ) {
-			return false === empty( $this->$name );
+	public function __isset($name) {
+		if (property_exists($this, $name)) {
+			return false === empty($this->$name);
 		} else {
 			return null;
 		}
@@ -460,20 +460,20 @@ 
 	 *
 	 * @return bool            If the setup was successful or not
 	 */
-	private function setup_payment( $payment_id ) {
+	private function setup_payment($payment_id) {
 		$this->pending = array();
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = get_post( $payment_id );
+		$payment = get_post($payment_id);
 
-		if ( ! $payment || is_wp_error( $payment ) ) {
+		if ( ! $payment || is_wp_error($payment)) {
 			return false;
 		}
 
-		if ( 'give_payment' !== $payment->post_type ) {
+		if ('give_payment' !== $payment->post_type) {
 			return false;
 		}
 
@@ -487,13 +487,13 @@ 
 		 * @param Give_Payment $this Payment object.
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_pre_setup_payment', $this, $payment_id );
+		do_action('give_pre_setup_payment', $this, $payment_id);
 
 		// Primary Identifier.
-		$this->ID = absint( $payment_id );
+		$this->ID = absint($payment_id);
 
 		// Protected ID that can never be changed.
-		$this->_ID = absint( $payment_id );
+		$this->_ID = absint($payment_id);
 
 		// We have a payment, get the generic payment_meta item to reduce calls to it.
 		$this->payment_meta = $this->get_meta();
@@ -508,7 +508,7 @@ 
 		$this->parent_payment = $payment->post_parent;
 
 		$all_payment_statuses  = give_get_payment_statuses();
-		$this->status_nicename = array_key_exists( $this->status, $all_payment_statuses ) ? $all_payment_statuses[ $this->status ] : ucfirst( $this->status );
+		$this->status_nicename = array_key_exists($this->status, $all_payment_statuses) ? $all_payment_statuses[$this->status] : ucfirst($this->status);
 
 		// Currency Based.
 		$this->total      = $this->setup_total();
@@ -546,7 +546,7 @@ 
 		 * @param Give_Payment $this Payment object.
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -564,8 +564,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_payment_setup( $payment_id ) {
-		$this->setup_payment( $payment_id );
+	public function update_payment_setup($payment_id) {
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -580,24 +580,24 @@ 
 
 		// Construct the payment title.
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} elseif ( ! empty( $this->first_name ) && empty( $this->last_name ) ) {
+		if ( ! empty($this->first_name) && ! empty($this->last_name)) {
+			$payment_title = $this->first_name.' '.$this->last_name;
+		} elseif ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} elseif ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} elseif ( ! empty($this->email) && is_email($this->email)) {
 			$payment_title = $this->email;
 		}
 
 		// Set Key.
-		if ( empty( $this->key ) ) {
+		if (empty($this->key)) {
 
-			$auth_key             = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-			$this->key            = strtolower( md5( $this->email . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );  // Unique key
+			$auth_key             = defined('AUTH_KEY') ? AUTH_KEY : '';
+			$this->key            = strtolower(md5($this->email.date('Y-m-d H:i:s').$auth_key.uniqid('give', true))); // Unique key
 			$this->pending['key'] = $this->key;
 		}
 
 		// Set IP.
-		if ( empty( $this->ip ) ) {
+		if (empty($this->ip)) {
 
 			$this->ip            = give_get_ip();
 			$this->pending['ip'] = $this->ip;
@@ -623,57 +623,57 @@ 
 			'status'       => $this->status,
 		);
 
-		$args = apply_filters( 'give_insert_payment_args', array(
+		$args = apply_filters('give_insert_payment_args', array(
 			'post_title'    => $payment_title,
 			'post_status'   => $this->status,
 			'post_type'     => 'give_payment',
-			'post_date'     => ! empty( $this->date ) ? $this->date : null,
-			'post_date_gmt' => ! empty( $this->date ) ? get_gmt_from_date( $this->date ) : null,
+			'post_date'     => ! empty($this->date) ? $this->date : null,
+			'post_date_gmt' => ! empty($this->date) ? get_gmt_from_date($this->date) : null,
 			'post_parent'   => $this->parent_payment,
-		), $payment_data );
+		), $payment_data);
 
 		// Create a blank payment
-		$payment_id = wp_insert_post( $args );
+		$payment_id = wp_insert_post($args);
 
-		if ( ! empty( $payment_id ) ) {
+		if ( ! empty($payment_id)) {
 
 			$this->ID  = $payment_id;
 			$this->_ID = $payment_id;
 
 			$donor = new stdClass;
 
-			if ( did_action( 'give_pre_process_donation' ) && is_user_logged_in() ) {
-				$donor = new Give_Donor( get_current_user_id(), true );
+			if (did_action('give_pre_process_donation') && is_user_logged_in()) {
+				$donor = new Give_Donor(get_current_user_id(), true);
 
 				// Donor is logged in but used a different email to purchase with so assign to their donor record.
-				if ( ! empty( $donor->id ) && $this->email != $donor->email ) {
-					$donor->add_email( $this->email );
+				if ( ! empty($donor->id) && $this->email != $donor->email) {
+					$donor->add_email($this->email);
 				}
 			}
 
-			if ( empty( $donor->id ) ) {
-				$donor = new Give_Donor( $this->email );
+			if (empty($donor->id)) {
+				$donor = new Give_Donor($this->email);
 			}
 
-			if ( empty( $donor->id ) ) {
+			if (empty($donor->id)) {
 
 				$donor_data = array(
-					'name'    => ! is_email( $payment_title ) ? $this->first_name . ' ' . $this->last_name : '',
+					'name'    => ! is_email($payment_title) ? $this->first_name.' '.$this->last_name : '',
 					'email'   => $this->email,
 					'user_id' => $this->user_id,
 				);
 
-				$donor->create( $donor_data );
+				$donor->create($donor_data);
 
 			}
 
 			$this->customer_id            = $donor->id;
 			$this->pending['customer_id'] = $this->customer_id;
-			$donor->attach_payment( $this->ID, false );
+			$donor->attach_payment($this->ID, false);
 
-			$this->payment_meta = apply_filters( 'give_payment_meta', $this->payment_meta, $payment_data );
+			$this->payment_meta = apply_filters('give_payment_meta', $this->payment_meta, $payment_data);
 
-			$this->update_meta( '_give_payment_meta', $this->payment_meta );
+			$this->update_meta('_give_payment_meta', $this->payment_meta);
 			$this->new = true;
 		}// End if().
 
@@ -695,11 +695,11 @@ 
 		$saved = false;
 
 		// Must have an ID.
-		if ( empty( $this->ID ) ) {
+		if (empty($this->ID)) {
 
 			$payment_id = $this->insert_payment();
 
-			if ( false === $payment_id ) {
+			if (false === $payment_id) {
 				$saved = false;
 			} else {
 				$this->ID = $payment_id;
@@ -707,48 +707,48 @@ 
 		}
 
 		// Set ID if not matching.
-		if ( $this->ID !== $this->_ID ) {
+		if ($this->ID !== $this->_ID) {
 			$this->ID = $this->_ID;
 		}
 
 		// If we have something pending, let's save it.
-		if ( ! empty( $this->pending ) ) {
+		if ( ! empty($this->pending)) {
 
 			$total_increase = 0;
 			$total_decrease = 0;
 
-			foreach ( $this->pending as $key => $value ) {
+			foreach ($this->pending as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'donations':
 						// Update totals for pending donations.
-						foreach ( $this->pending[ $key ] as $item ) {
+						foreach ($this->pending[$key] as $item) {
 
-							$quantity = isset( $item['quantity'] ) ? $item['quantity'] : 1;
-							$price_id = isset( $item['price_id'] ) ? $item['price_id'] : 0;
+							$quantity = isset($item['quantity']) ? $item['quantity'] : 1;
+							$price_id = isset($item['price_id']) ? $item['price_id'] : 0;
 
-							switch ( $item['action'] ) {
+							switch ($item['action']) {
 
 								case 'add':
 
 									$price = $item['price'];
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status) {
 
 										// Add sales logs.
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
 
 										$y = 0;
-										while ( $y < $quantity ) {
+										while ($y < $quantity) {
 
-											give_record_donation_in_log( $item['id'], $this->ID, $price_id, $log_date );
-											$y ++;
+											give_record_donation_in_log($item['id'], $this->ID, $price_id, $log_date);
+											$y++;
 										}
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price);
 
 										$total_increase += $price;
 									}
@@ -773,15 +773,15 @@ 
 										),
 									);
 
-									$found_logs = get_posts( $log_args );
-									foreach ( $found_logs as $log ) {
-										wp_delete_post( $log->ID, true );
+									$found_logs = get_posts($log_args);
+									foreach ($found_logs as $log) {
+										wp_delete_post($log->ID, true);
 									}
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'] );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -792,43 +792,43 @@ 
 						break;
 
 					case 'status':
-						$this->update_status( $this->status );
+						$this->update_status($this->status);
 						break;
 
 					case 'gateway':
-						$this->update_meta( '_give_payment_gateway', $this->gateway );
+						$this->update_meta('_give_payment_gateway', $this->gateway);
 						break;
 
 					case 'mode':
-						$this->update_meta( '_give_payment_mode', $this->mode );
+						$this->update_meta('_give_payment_mode', $this->mode);
 						break;
 
 					case 'transaction_id':
-						$this->update_meta( '_give_payment_transaction_id', $this->transaction_id );
+						$this->update_meta('_give_payment_transaction_id', $this->transaction_id);
 						break;
 
 					case 'ip':
-						$this->update_meta( '_give_payment_user_ip', $this->ip );
+						$this->update_meta('_give_payment_user_ip', $this->ip);
 						break;
 
 					case 'customer_id':
-						$this->update_meta( '_give_payment_customer_id', $this->customer_id );
+						$this->update_meta('_give_payment_customer_id', $this->customer_id);
 						break;
 
 					case 'user_id':
-						$this->update_meta( '_give_payment_user_id', $this->user_id );
+						$this->update_meta('_give_payment_user_id', $this->user_id);
 						break;
 
 					case 'form_title':
-						$this->update_meta( '_give_payment_form_title', $this->form_title );
+						$this->update_meta('_give_payment_form_title', $this->form_title);
 						break;
 
 					case 'form_id':
-						$this->update_meta( '_give_payment_form_id', $this->form_id );
+						$this->update_meta('_give_payment_form_id', $this->form_id);
 						break;
 
 					case 'price_id':
-						$this->update_meta( '_give_payment_price_id', $this->price_id );
+						$this->update_meta('_give_payment_price_id', $this->price_id);
 						break;
 
 					case 'first_name':
@@ -844,15 +844,15 @@ 
 						break;
 
 					case 'email':
-						$this->update_meta( '_give_payment_user_email', $this->email );
+						$this->update_meta('_give_payment_user_email', $this->email);
 						break;
 
 					case 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
@@ -862,11 +862,11 @@ 
 							'edit_date' => true,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'completed_date':
-						$this->update_meta( '_give_completed_date', $this->completed_date );
+						$this->update_meta('_give_completed_date', $this->completed_date);
 						break;
 
 					case 'parent_payment':
@@ -875,7 +875,7 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
@@ -886,33 +886,33 @@ 
 						 *
 						 * @param Give_Payment $this Payment object.
 						 */
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				}// End switch().
 			}// End foreach().
 
-			if ( 'pending' !== $this->status ) {
+			if ('pending' !== $this->status) {
 
-				$donor = new Give_Donor( $this->customer_id );
+				$donor = new Give_Donor($this->customer_id);
 
 				$total_change = $total_increase - $total_decrease;
-				if ( $total_change < 0 ) {
+				if ($total_change < 0) {
 
-					$total_change = - ( $total_change );
+					$total_change = - ($total_change);
 					// Decrease the donor's donation stats.
-					$donor->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$donor->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
-				} elseif ( $total_change > 0 ) {
+				} elseif ($total_change > 0) {
 
 					// Increase the donor's donation stats.
-					$donor->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$donor->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 				}
 			}
 
-			$this->update_meta( '_give_payment_total', $this->total );
+			$this->update_meta('_give_payment_total', $this->total);
 
 			$new_meta = array(
 				'form_title' => $this->form_title,
@@ -923,12 +923,12 @@ 
 			);
 
 			$meta        = $this->get_meta();
-			$merged_meta = array_merge( $meta, $new_meta );
+			$merged_meta = array_merge($meta, $new_meta);
 
 			// Only save the payment meta if it's changed.
-			if ( md5( serialize( $meta ) ) !== md5( serialize( $merged_meta ) ) ) {
-				$updated = $this->update_meta( '_give_payment_meta', $merged_meta );
-				if ( false !== $updated ) {
+			if (md5(serialize($meta)) !== md5(serialize($merged_meta))) {
+				$updated = $this->update_meta('_give_payment_meta', $merged_meta);
+				if (false !== $updated) {
 					$saved = true;
 				}
 			}
@@ -937,8 +937,8 @@ 
 			$saved         = true;
 		}// End if().
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -956,12 +956,12 @@ 
 	 *
 	 * @return bool           True when successful, false otherwise
 	 */
-	public function add_donation( $form_id = 0, $args = array(), $options = array() ) {
+	public function add_donation($form_id = 0, $args = array(), $options = array()) {
 
-		$donation = new Give_Donate_Form( $form_id );
+		$donation = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a give donation form.
-		if ( ! $donation || $donation->post_type !== 'give_forms' ) {
+		if ( ! $donation || $donation->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -971,59 +971,59 @@ 
 			'price_id' => false,
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_payment_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_payment_add_donation_args', $args, $donation->ID), $defaults);
 
 		// Allow overriding the price.
-		if ( false !== $args['price'] ) {
+		if (false !== $args['price']) {
 			$item_price = $args['price'];
 		} else {
 
 			// Deal with variable pricing.
-			if ( give_has_variable_prices( $donation->ID ) ) {
-				$prices     = maybe_unserialize( give_get_meta( $form_id, '_give_donation_levels', true ) );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices     = maybe_unserialize(give_get_meta($form_id, '_give_donation_levels', true));
 				$item_price = '';
 				// Loop through prices.
-				foreach ( $prices as $price ) {
+				foreach ($prices as $price) {
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
-					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
+					if ((isset($args['price_id']) && isset($price['_give_id']['level_id']))
 					     && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}
 				}
 				// Fallback to the lowest price point.
-				if ( $item_price == '' ) {
-					$item_price       = give_get_lowest_price_option( $donation->ID );
-					$args['price_id'] = give_get_lowest_price_id( $donation->ID );
+				if ($item_price == '') {
+					$item_price       = give_get_lowest_price_option($donation->ID);
+					$args['price_id'] = give_get_lowest_price_id($donation->ID);
 				}
 			} else {
 				// Simple form price.
-				$item_price = give_get_form_price( $donation->ID );
+				$item_price = give_get_form_price($donation->ID);
 			}
 		}
 
 		// Sanitizing the price here so we don't have a dozen calls later.
-		$item_price = give_sanitize_amount( $item_price );
-		$total      = round( $item_price, give_currency_decimal_filter() );
+		$item_price = give_sanitize_amount($item_price);
+		$total      = round($item_price, give_currency_decimal_filter());
 
 		// Add Options.
 		$default_options = array();
-		if ( false !== $args['price_id'] ) {
+		if (false !== $args['price_id']) {
 			$default_options['price_id'] = (int) $args['price_id'];
 		}
-		$options = wp_parse_args( $options, $default_options );
+		$options = wp_parse_args($options, $default_options);
 
 		// Do not allow totals to go negative.
-		if ( $total < 0 ) {
+		if ($total < 0) {
 			$total = 0;
 		}
 
 		$donation = array(
 			'name'     => $donation->post_title,
 			'id'       => $donation->ID,
-			'price'    => round( $total, give_currency_decimal_filter() ),
-			'subtotal' => round( $total, give_currency_decimal_filter() ),
+			'price'    => round($total, give_currency_decimal_filter()),
+			'subtotal' => round($total, give_currency_decimal_filter()),
 			'price_id' => $args['price_id'],
 			'action'   => 'add',
 			'options'  => $options,
@@ -1031,7 +1031,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1048,7 +1048,7 @@ 
 	 *
 	 * @return bool           If the item was removed or not
 	 */
-	public function remove_donation( $form_id, $args = array() ) {
+	public function remove_donation($form_id, $args = array()) {
 
 		// Set some defaults.
 		$defaults = array(
@@ -1056,12 +1056,12 @@ 
 			'price'    => false,
 			'price_id' => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$form = new Give_Donate_Form( $form_id );
+		$form = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a valid give donation form.
-		if ( ! $form || $form->post_type !== 'give_forms' ) {
+		if ( ! $form || $form->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -1074,7 +1074,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1090,13 +1090,13 @@ 
 	 *
 	 * @return void
 	 */
-	public function add_note( $note = false ) {
+	public function add_note($note = false) {
 		// Bail if no note specified.
-		if ( ! $note ) {
+		if ( ! $note) {
 			return false;
 		}
 
-		give_insert_payment_note( $this->ID, $note );
+		give_insert_payment_note($this->ID, $note);
 	}
 
 	/**
@@ -1109,8 +1109,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function increase_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
 		$this->recalculate_total();
@@ -1126,11 +1126,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function decrease_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1159,24 +1159,24 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated.
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		// standardize the 'complete(d)' status.
-		if ( $status == 'completed' || $status == 'complete' ) {
+		if ($status == 'completed' || $status == 'complete') {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes.
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
-		if ( $do_change ) {
+		if ($do_change) {
 
 			/**
 			 * Fires before changing payment status.
@@ -1187,21 +1187,21 @@ 
 			 * @param string $status The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' ),
+				'edit_date'   => current_time('mysql'),
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions.
-			switch ( $status ) {
+			switch ($status) {
 				case 'refunded':
 					$this->process_refund();
 					break;
@@ -1228,7 +1228,7 @@ 
 			 * @param string $status The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		}// End if().
 
@@ -1263,33 +1263,33 @@ 
 	 *
 	 * @return mixed             The value from the post meta
 	 */
-	public function get_meta( $meta_key = '_give_payment_meta', $single = true ) {
+	public function get_meta($meta_key = '_give_payment_meta', $single = true) {
 
-		$meta = give_get_meta( $this->ID, $meta_key, $single );
+		$meta = give_get_meta($this->ID, $meta_key, $single);
 
-		if ( $meta_key === '_give_payment_meta' ) {
+		if ($meta_key === '_give_payment_meta') {
 			$meta = (array) $meta;
 
-			if ( empty( $meta['key'] ) ) {
+			if (empty($meta['key'])) {
 				$meta['key'] = $this->setup_payment_key();
 			}
 
-			if ( empty( $meta['form_title'] ) ) {
+			if (empty($meta['form_title'])) {
 				$meta['form_title'] = $this->setup_form_title();
 			}
 
-			if ( empty( $meta['email'] ) ) {
+			if (empty($meta['email'])) {
 				$meta['email'] = $this->setup_email();
 			}
 
-			if ( empty( $meta['date'] ) ) {
-				$meta['date'] = get_post_field( 'post_date', $this->ID );
+			if (empty($meta['date'])) {
+				$meta['date'] = get_post_field('post_date', $this->ID);
 			}
 		}
 
-		$meta = apply_filters( "give_get_payment_meta_{$meta_key}", $meta, $this->ID );
+		$meta = apply_filters("give_get_payment_meta_{$meta_key}", $meta, $this->ID);
 
-		return apply_filters( 'give_get_payment_meta', $meta, $this->ID, $meta_key );
+		return apply_filters('give_get_payment_meta', $meta, $this->ID, $meta_key);
 	}
 
 	/**
@@ -1304,23 +1304,23 @@ 
 	 *
 	 * @return int|bool           Meta ID if the key didn't exist, true on successful update, false on failure
 	 */
-	public function update_meta( $meta_key = '', $meta_value = '', $prev_value = '' ) {
-		if ( empty( $meta_key ) ) {
+	public function update_meta($meta_key = '', $meta_value = '', $prev_value = '') {
+		if (empty($meta_key)) {
 			return false;
 		}
 
-		if ( $meta_key == 'key' || $meta_key == 'date' ) {
+		if ($meta_key == 'key' || $meta_key == 'date') {
 
 			$current_meta              = $this->get_meta();
-			$current_meta[ $meta_key ] = $meta_value;
+			$current_meta[$meta_key] = $meta_value;
 
 			$meta_key   = '_give_payment_meta';
 			$meta_value = $current_meta;
 
-		} elseif ( $meta_key == 'email' || $meta_key == '_give_payment_user_email' ) {
+		} elseif ($meta_key == 'email' || $meta_key == '_give_payment_user_email') {
 
-			$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
-			give_update_meta( $this->ID, '_give_payment_user_email', $meta_value );
+			$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
+			give_update_meta($this->ID, '_give_payment_user_email', $meta_value);
 
 			$current_meta                       = $this->get_meta();
 			$current_meta['user_info']['email'] = $meta_value;
@@ -1330,9 +1330,9 @@ 
 
 		}
 
-		$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
+		$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
 
-		return give_update_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return give_update_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1347,14 +1347,14 @@ 
 		$process_refund = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'refunded' != $this->status ) {
+		if ('publish' != $this->old_status || 'refunded' != $this->status) {
 			$process_refund = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_refund = apply_filters( 'give_should_process_refund', $process_refund, $this );
+		$process_refund = apply_filters('give_should_process_refund', $process_refund, $this);
 
-		if ( false === $process_refund ) {
+		if (false === $process_refund) {
 			return;
 		}
 
@@ -1365,13 +1365,13 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_pre_refund_payment', $this );
+		do_action('give_pre_refund_payment', $this);
 
-		$decrease_earnings       = apply_filters( 'give_decrease_store_earnings_on_refund', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_refund', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_refund', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_store_earnings_on_refund', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_refund', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_refund', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1384,7 +1384,7 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1411,26 +1411,26 @@ 
 		$process_pending = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'pending' != $this->status ) {
+		if ('publish' != $this->old_status || 'pending' != $this->status) {
 			$process_pending = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_pending = apply_filters( 'give_should_process_pending', $process_pending, $this );
+		$process_pending = apply_filters('give_should_process_pending', $process_pending, $this);
 
-		if ( false === $process_pending ) {
+		if (false === $process_pending) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_pending', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_pending', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_pending', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_pending', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_pending', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_pending', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1448,26 +1448,26 @@ 
 		$process_cancelled = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
+		if ('publish' != $this->old_status || 'cancelled' != $this->status) {
 			$process_cancelled = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
+		$process_cancelled = apply_filters('give_should_process_cancelled', $process_cancelled, $this);
 
-		if ( false === $process_cancelled ) {
+		if (false === $process_cancelled) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_cancelled', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_cancelled', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_cancelled', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_cancelled', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_cancelled', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_cancelled', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1483,26 +1483,26 @@ 
 		$process_revoked = true;
 
 		// If the payment was not in publish, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
+		if ('publish' != $this->old_status || 'revoked' != $this->status) {
 			$process_revoked = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
+		$process_revoked = apply_filters('give_should_process_revoked', $process_revoked, $this);
 
-		if ( false === $process_revoked ) {
+		if (false === $process_revoked) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_revoked', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_revoked', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_revoked', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_revoked', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_revoked', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_revoked', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1520,25 +1520,25 @@ 
 	 *
 	 * @return void
 	 */
-	private function maybe_alter_stats( $alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count ) {
+	private function maybe_alter_stats($alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count) {
 
-		give_undo_donation( $this->ID );
+		give_undo_donation($this->ID);
 
 		// Decrease store earnings.
-		if ( true === $alter_store_earnings ) {
-			give_decrease_total_earnings( $this->total );
+		if (true === $alter_store_earnings) {
+			give_decrease_total_earnings($this->total);
 		}
 
 		// Decrement the stats for the donor.
-		if ( ! empty( $this->customer_id ) ) {
+		if ( ! empty($this->customer_id)) {
 
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( true === $alter_customer_value ) {
-				$donor->decrease_value( $this->total );
+			if (true === $alter_customer_value) {
+				$donor->decrease_value($this->total);
 			}
 
-			if ( true === $alter_customer_purchase_count ) {
+			if (true === $alter_customer_purchase_count) {
 				$donor->decrease_donation_count();
 			}
 		}
@@ -1587,13 +1587,13 @@ 
 	 * @return string The date the payment was completed
 	 */
 	private function setup_completed_date() {
-		$payment = get_post( $this->ID );
+		$payment = get_post($this->ID);
 
-		if ( 'pending' == $payment->post_status || 'preapproved' == $payment->post_status ) {
+		if ('pending' == $payment->post_status || 'preapproved' == $payment->post_status) {
 			return false; // This payment was never completed.
 		}
 
-		$date = ( $date = $this->get_meta( '_give_completed_date', true ) ) ? $date : $payment->modified_date;
+		$date = ($date = $this->get_meta('_give_completed_date', true)) ? $date : $payment->modified_date;
 
 		return $date;
 	}
@@ -1607,7 +1607,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1619,18 +1619,18 @@ 
 	 * @return float The payment total
 	 */
 	private function setup_total() {
-		$amount = $this->get_meta( '_give_payment_total', true );
+		$amount = $this->get_meta('_give_payment_total', true);
 
-		if ( empty( $amount ) && '0.00' != $amount ) {
-			$meta = $this->get_meta( '_give_payment_meta', true );
-			$meta = maybe_unserialize( $meta );
+		if (empty($amount) && '0.00' != $amount) {
+			$meta = $this->get_meta('_give_payment_meta', true);
+			$meta = maybe_unserialize($meta);
 
-			if ( isset( $meta['amount'] ) ) {
+			if (isset($meta['amount'])) {
 				$amount = $meta['amount'];
 			}
 		}
 
-		return round( floatval( $amount ), give_currency_decimal_filter() );
+		return round(floatval($amount), give_currency_decimal_filter());
 	}
 
 	/**
@@ -1656,7 +1656,7 @@ 
 	 * @return string The currency for the payment
 	 */
 	private function setup_currency() {
-		$currency = isset( $this->payment_meta['currency'] ) ? $this->payment_meta['currency'] : apply_filters( 'give_payment_currency_default', give_get_currency(), $this );
+		$currency = isset($this->payment_meta['currency']) ? $this->payment_meta['currency'] : apply_filters('give_payment_currency_default', give_get_currency(), $this);
 
 		return $currency;
 	}
@@ -1670,7 +1670,7 @@ 
 	 * @return string The gateway
 	 */
 	private function setup_gateway() {
-		$gateway = $this->get_meta( '_give_payment_gateway', true );
+		$gateway = $this->get_meta('_give_payment_gateway', true);
 
 		return $gateway;
 	}
@@ -1684,11 +1684,11 @@ 
 	 * @return string The donation ID
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) ) {
+		if (empty($transaction_id)) {
 			$gateway        = $this->gateway;
-			$transaction_id = apply_filters( "give_get_payment_transaction_id-{$gateway}", $this->ID );
+			$transaction_id = apply_filters("give_get_payment_transaction_id-{$gateway}", $this->ID);
 		}
 
 		return $transaction_id;
@@ -1703,7 +1703,7 @@ 
 	 * @return string The IP address for the payment
 	 */
 	private function setup_ip() {
-		$ip = $this->get_meta( '_give_payment_user_ip', true );
+		$ip = $this->get_meta('_give_payment_user_ip', true);
 
 		return $ip;
 	}
@@ -1717,7 +1717,7 @@ 
 	 * @return int The Donor ID.
 	 */
 	private function setup_donor_id() {
-		$customer_id = $this->get_meta( '_give_payment_customer_id', true );
+		$customer_id = $this->get_meta('_give_payment_customer_id', true);
 
 		return $customer_id;
 	}
@@ -1731,7 +1731,7 @@ 
 	 * @return int The User ID
 	 */
 	private function setup_user_id() {
-		$user_id = $this->get_meta( '_give_payment_user_id', true );
+		$user_id = $this->get_meta('_give_payment_user_id', true);
 
 		return $user_id;
 	}
@@ -1745,10 +1745,10 @@ 
 	 * @return string The email address for the payment.
 	 */
 	private function setup_email() {
-		$email = $this->get_meta( '_give_payment_user_email', true );
+		$email = $this->get_meta('_give_payment_user_email', true);
 
-		if ( empty( $email ) ) {
-			$email = Give()->donors->get_column( 'email', $this->customer_id );
+		if (empty($email)) {
+			$email = Give()->donors->get_column('email', $this->customer_id);
 		}
 
 		return $email;
@@ -1768,15 +1768,15 @@ 
 			'last_name'  => $this->last_name,
 		);
 
-		$user_info = isset( $this->payment_meta['user_info'] ) ? maybe_unserialize( $this->payment_meta['user_info'] ) : array();
-		$user_info = wp_parse_args( $user_info, $defaults );
+		$user_info = isset($this->payment_meta['user_info']) ? maybe_unserialize($this->payment_meta['user_info']) : array();
+		$user_info = wp_parse_args($user_info, $defaults);
 
-		if ( empty( $user_info ) ) {
+		if (empty($user_info)) {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( $donor->id > 0 ) {
-				$name      = explode( ' ', $donor->name, 2 );
+			if ($donor->id > 0) {
+				$name      = explode(' ', $donor->name, 2);
 				$user_info = array(
 					'first_name' => $name[0],
 					'last_name'  => $name[1],
@@ -1786,29 +1786,29 @@ 
 			}
 		} else {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
-			if ( $donor->id > 0 ) {
-				foreach ( $user_info as $key => $value ) {
-					if ( ! empty( $value ) ) {
+			$donor = new Give_Donor($this->customer_id);
+			if ($donor->id > 0) {
+				foreach ($user_info as $key => $value) {
+					if ( ! empty($value)) {
 						continue;
 					}
 
-					switch ( $key ) {
+					switch ($key) {
 						case 'first_name':
-							$name = explode( ' ', $donor->name, 2 );
+							$name = explode(' ', $donor->name, 2);
 
-							$user_info[ $key ] = $name[0];
+							$user_info[$key] = $name[0];
 							break;
 
 						case 'last_name':
-							$name      = explode( ' ', $donor->name, 2 );
-							$last_name = ! empty( $name[1] ) ? $name[1] : '';
+							$name      = explode(' ', $donor->name, 2);
+							$last_name = ! empty($name[1]) ? $name[1] : '';
 
-							$user_info[ $key ] = $last_name;
+							$user_info[$key] = $last_name;
 							break;
 
 						case 'email':
-							$user_info[ $key ] = $donor->email;
+							$user_info[$key] = $donor->email;
 							break;
 					}
 				}
@@ -1829,7 +1829,7 @@ 
 	 */
 	private function setup_address() {
 
-		$address = ! empty( $this->payment_meta['user_info']['address'] ) ? $this->payment_meta['user_info']['address'] : array(
+		$address = ! empty($this->payment_meta['user_info']['address']) ? $this->payment_meta['user_info']['address'] : array(
 			'line1'   => '',
 			'line2'   => '',
 			'city'    => '',
@@ -1851,7 +1851,7 @@ 
 	 */
 	private function setup_form_title() {
 
-		$form_id = $this->get_meta( '_give_payment_form_title', true );
+		$form_id = $this->get_meta('_give_payment_form_title', true);
 
 		return $form_id;
 	}
@@ -1866,7 +1866,7 @@ 
 	 */
 	private function setup_form_id() {
 
-		$form_id = $this->get_meta( '_give_payment_form_id', true );
+		$form_id = $this->get_meta('_give_payment_form_id', true);
 
 		return $form_id;
 	}
@@ -1880,7 +1880,7 @@ 
 	 * @return int The Form Price ID.
 	 */
 	private function setup_price_id() {
-		$price_id = $this->get_meta( '_give_payment_price_id', true );
+		$price_id = $this->get_meta('_give_payment_price_id', true);
 
 		return $price_id;
 	}
@@ -1894,7 +1894,7 @@ 
 	 * @return string The Payment Key.
 	 */
 	private function setup_payment_key() {
-		$key = $this->get_meta( '_give_payment_purchase_key', true );
+		$key = $this->get_meta('_give_payment_purchase_key', true);
 
 		return $key;
 	}
@@ -1910,11 +1910,11 @@ 
 	private function setup_payment_number() {
 		$number = $this->ID;
 
-		if ( give_get_option( 'enable_sequential' ) ) {
+		if (give_get_option('enable_sequential')) {
 
-			$number = $this->get_meta( '_give_payment_number', true );
+			$number = $this->get_meta('_give_payment_number', true);
 
-			if ( ! $number ) {
+			if ( ! $number) {
 
 				$number = $this->ID;
 
@@ -1932,7 +1932,7 @@ 
 	 * @return array The payment object as an array.
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -1945,7 +1945,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -1957,7 +1957,7 @@ 
 	 * @return string Date payment was completed.
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_payment_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_payment_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -1969,7 +1969,7 @@ 
 	 * @return float Payment subtotal.
 	 */
 	private function get_subtotal() {
-		return apply_filters( 'give_get_payment_subtotal', $this->subtotal, $this->ID, $this );
+		return apply_filters('give_get_payment_subtotal', $this->subtotal, $this->ID, $this);
 	}
 
 	/**
@@ -1981,7 +1981,7 @@ 
 	 * @return string Payment currency code.
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_payment_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_payment_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -1993,7 +1993,7 @@ 
 	 * @return string Gateway used.
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_payment_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_payment_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -2005,7 +2005,7 @@ 
 	 * @return string Donation ID from merchant processor.
 	 */
 	private function get_transaction_id() {
-		return apply_filters( 'give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this );
+		return apply_filters('give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this);
 	}
 
 	/**
@@ -2017,7 +2017,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_payment_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_payment_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -2029,7 +2029,7 @@ 
 	 * @return int Payment donor ID.
 	 */
 	private function get_donor_id() {
-		return apply_filters( 'give_payment_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_payment_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -2041,7 +2041,7 @@ 
 	 * @return int Payment user ID.
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_payment_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_payment_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -2053,7 +2053,7 @@ 
 	 * @return string Payment donor email.
 	 */
 	private function get_email() {
-		return apply_filters( 'give_payment_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_payment_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2065,7 +2065,7 @@ 
 	 * @return array Payment user info.
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_payment_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_payment_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2077,7 +2077,7 @@ 
 	 * @return array Payment billing address.
 	 */
 	private function get_address() {
-		return apply_filters( 'give_payment_address', $this->address, $this->ID, $this );
+		return apply_filters('give_payment_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2089,7 +2089,7 @@ 
 	 * @return string Payment key.
 	 */
 	private function get_key() {
-		return apply_filters( 'give_payment_key', $this->key, $this->ID, $this );
+		return apply_filters('give_payment_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2101,7 +2101,7 @@ 
 	 * @return string Payment form id
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_payment_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_payment_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2113,7 +2113,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 
 }

includes/class-give-cron.php

Spacing +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -34,8 +34,8 @@ 
 	 * @see    Give_Cron::weekly_events()
 	 */
 	public function __construct() {
-		add_filter( 'cron_schedules', array( $this, 'add_schedules' ) );
-		add_action( 'wp', array( $this, 'schedule_Events' ) );
+		add_filter('cron_schedules', array($this, 'add_schedules'));
+		add_action('wp', array($this, 'schedule_Events'));
 	}
 
 	/**
@@ -48,11 +48,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'  => esc_html__( 'Once Weekly', 'give' )
+			'display'  => esc_html__('Once Weekly', 'give')
 		);
 
 		return $schedules;
@@ -80,8 +80,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');
 		}
 	}
 
@@ -94,8 +94,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');
 		}
 	}
 

includes/admin/settings/class-settings-email.php

Indentation +21 added lines, -21 removed lines

@@ -79,13 +79,13 @@ 
 							'default' => get_bloginfo( 'admin_email' ),
 							'type'    => 'text'
 						),
-                        array(
-                            'name'  => esc_html__( 'Email Settings Docs Link', 'give' ),
-                            'id'    => 'email_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-emails' ),
-                            'title' => __( 'Email Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Email Settings Docs Link', 'give' ),
+							'id'    => 'email_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-emails' ),
+							'title' => __( 'Email Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_email_settings_1',
 							'type' => 'sectionend'
@@ -118,13 +118,13 @@ 
 							'type'    => 'wysiwyg',
 							'default' => give_get_default_donation_receipt_email()
 						),
-                        array(
-                            'name'  => esc_html__( 'Donation Receipt Settings Docs Link', 'give' ),
-                            'id'    => 'donation_receipt_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-receipt' ),
-                            'title' => __( 'Donation Receipt Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Donation Receipt Settings Docs Link', 'give' ),
+							'id'    => 'donation_receipt_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-donation-receipt' ),
+							'title' => __( 'Donation Receipt Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_email_settings_2',
 							'type' => 'sectionend'
@@ -175,13 +175,13 @@ 
 							'type'    => 'textarea',
 							'default' => get_bloginfo( 'admin_email' )
 						),
-                        array(
-                            'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
-                            'id'    => 'donation_notification_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
-                            'title' => __( 'Donation Notification Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
+							'id'    => 'donation_notification_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
+							'title' => __( 'Donation Notification Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_email_settings_3',
 							'type' => 'sectionend'

Spacing +46 added lines, -46 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Email' ) ) :
+if ( ! class_exists('Give_Settings_Email')) :
 
 	/**
 	 * Give_Settings_Email.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'emails';
-			$this->label = esc_html__( 'Emails', 'give' );
+			$this->label = esc_html__('Emails', 'give');
 
 			$this->default_tab = 'email-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'email-settings' :
 					$settings = array(
 						// Section 1: Email
@@ -54,36 +54,36 @@ 
 						),
 						array(
 							'id'      => 'email_template',
-							'name'    => esc_html__( 'Email Template', 'give' ),
-							'desc'    => esc_html__( 'Choose your template from the available registered template types.', 'give' ),
+							'name'    => esc_html__('Email Template', 'give'),
+							'desc'    => esc_html__('Choose your template from the available registered template types.', 'give'),
 							'type'    => 'select',
 							'options' => give_get_email_templates()
 						),
 						array(
 							'id'   => 'email_logo',
-							'name' => esc_html__( 'Logo', 'give' ),
-							'desc' => esc_html__( 'Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give' ),
+							'name' => esc_html__('Logo', 'give'),
+							'desc' => esc_html__('Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give'),
 							'type' => 'file'
 						),
 						array(
 							'id'      => 'from_name',
-							'name'    => esc_html__( 'From Name', 'give' ),
-							'desc'    => esc_html__( 'The name which appears in the "From" field in all Give donation emails.', 'give' ),
-							'default' => get_bloginfo( 'name' ),
+							'name'    => esc_html__('From Name', 'give'),
+							'desc'    => esc_html__('The name which appears in the "From" field in all Give donation emails.', 'give'),
+							'default' => get_bloginfo('name'),
 							'type'    => 'text'
 						),
 						array(
 							'id'      => 'from_email',
-							'name'    => esc_html__( 'From Email', 'give' ),
-							'desc'    => esc_html__( 'Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give' ),
-							'default' => get_bloginfo( 'admin_email' ),
+							'name'    => esc_html__('From Email', 'give'),
+							'desc'    => esc_html__('Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give'),
+							'default' => get_bloginfo('admin_email'),
 							'type'    => 'text'
 						),
                         array(
-                            'name'  => esc_html__( 'Email Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Email Settings Docs Link', 'give'),
                             'id'    => 'email_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-emails' ),
-                            'title' => __( 'Email Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-emails'),
+                            'title' => __('Email Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -102,27 +102,27 @@ 
 						),
 						array(
 							'id'      => 'donation_subject',
-							'name'    => esc_html__( 'Donation Email Subject', 'give' ),
-							'desc'    => esc_html__( 'Enter the subject line for the donation receipt email.', 'give' ),
-							'default' => esc_attr__( 'Donation Receipt', 'give' ),
+							'name'    => esc_html__('Donation Email Subject', 'give'),
+							'desc'    => esc_html__('Enter the subject line for the donation receipt email.', 'give'),
+							'default' => esc_attr__('Donation Receipt', 'give'),
 							'type'    => 'text'
 						),
 						array(
 							'id'      => 'donation_receipt',
-							'name'    => esc_html__( 'Donation Receipt', 'give' ),
+							'name'    => esc_html__('Donation Receipt', 'give'),
 							'desc'    => sprintf(
 							/* translators: %s: emails tags list */
-								__( 'Enter the email that is sent to users after completing a successful donation. HTML is accepted.<br /><strong>Available template tags:</strong> %s', 'give' ),
+								__('Enter the email that is sent to users after completing a successful donation. HTML is accepted.<br /><strong>Available template tags:</strong> %s', 'give'),
 								'<br/>'.give_get_emails_tags_list()
 							),
 							'type'    => 'wysiwyg',
 							'default' => give_get_default_donation_receipt_email()
 						),
                         array(
-                            'name'  => esc_html__( 'Donation Receipt Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Donation Receipt Settings Docs Link', 'give'),
                             'id'    => 'donation_receipt_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-receipt' ),
-                            'title' => __( 'Donation Receipt Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-donation-receipt'),
+                            'title' => __('Donation Receipt Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -141,28 +141,28 @@ 
 						),
 						array(
 							'id'      => 'admin_notices',
-							'name'    => esc_html__( 'Admin Notifications', 'give' ),
-							'desc'    => esc_html__( 'Enable/Disable all admin notifications from Give completely.', 'give' ),
+							'name'    => esc_html__('Admin Notifications', 'give'),
+							'desc'    => esc_html__('Enable/Disable all admin notifications from Give completely.', 'give'),
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
 							'id'      => 'donation_notification_subject',
-							'name'    => esc_html__( 'Donation Notification Subject', 'give' ),
-							'desc'    => esc_html__( 'Enter the subject line for the admin donation notification email.', 'give' ),
+							'name'    => esc_html__('Donation Notification Subject', 'give'),
+							'desc'    => esc_html__('Enter the subject line for the admin donation notification email.', 'give'),
 							'type'    => 'text',
-							'default' => esc_attr__( 'New Donation - #{payment_id}', 'give' )
+							'default' => esc_attr__('New Donation - #{payment_id}', 'give')
 						),
 						array(
 							'id'      => 'donation_notification',
-							'name'    => esc_html__( 'Donation Notification', 'give' ),
+							'name'    => esc_html__('Donation Notification', 'give'),
 							'desc'    => sprintf(
 							/* translators: %s: emails tags list */
-								__( 'Enter the content of the email that is sent to notify an admin of a new donation. HTML is accepted. <br /><strong>Available template tags:</strong> %s', 'give' ),
+								__('Enter the content of the email that is sent to notify an admin of a new donation. HTML is accepted. <br /><strong>Available template tags:</strong> %s', 'give'),
 								'<br/>'.give_get_emails_tags_list()
 							),
 							'type'    => 'wysiwyg',
@@ -170,16 +170,16 @@ 
 						),
 						array(
 							'id'      => 'admin_notice_emails',
-							'name'    => esc_html__( 'Donation Notification Emails', 'give' ),
-							'desc'    => __( 'Enter the email address(es) that should receive a notification anytime a donation is made, please only enter <span class="give-underline">one email address per line</span> and <strong>not separated by commas</strong>.', 'give' ),
+							'name'    => esc_html__('Donation Notification Emails', 'give'),
+							'desc'    => __('Enter the email address(es) that should receive a notification anytime a donation is made, please only enter <span class="give-underline">one email address per line</span> and <strong>not separated by commas</strong>.', 'give'),
 							'type'    => 'textarea',
-							'default' => get_bloginfo( 'admin_email' )
+							'default' => get_bloginfo('admin_email')
 						),
                         array(
-                            'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Donation Notification Settings Docs Link', 'give'),
                             'id'    => 'donation_notification_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
-                            'title' => __( 'Donation Notification Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-donation-notification'),
+                            'title' => __('Donation Notification Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -194,7 +194,7 @@ 
 			 * Filter the emails settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_emails', $settings );
+			$settings = apply_filters('give_settings_emails', $settings);
 
 			/**
 			 * Filter the settings.
@@ -202,7 +202,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -216,12 +216,12 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'email-settings'            => esc_html__( 'Email Settings', 'give' ),
-				'donation-receipt'          => esc_html__( 'Donation Receipt', 'give' ),
-				'new-donation-notification' => esc_html__( 'New Donation Notification', 'give' )
+				'email-settings'            => esc_html__('Email Settings', 'give'),
+				'donation-receipt'          => esc_html__('Donation Receipt', 'give'),
+				'new-donation-notification' => esc_html__('New Donation Notification', 'give')
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/admin/shortcodes/shortcode-give-profile-editor.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,9 +21,9 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['label'] = esc_html__( 'Profile Editor', 'give' );
+		$this->shortcode['label'] = esc_html__('Profile Editor', 'give');
 
-		parent::__construct( 'give_profile_editor' );
+		parent::__construct('give_profile_editor');
 	}
 }