WordImpress / Give

includes/admin/class-api-keys-table.php

Indentation +2 added lines, -2 removed lines

@@ -254,8 +254,8 @@
 		$args = array(
 		  'id' => 'give-api-user-search',
 		  'name' => 'user_id',
-        );
-        echo Give()->html->ajax_user_search($args); ?>
+		);
+		echo Give()->html->ajax_user_search($args); ?>
 		<?php submit_button( esc_html__( 'Generate New API Keys', 'give' ), 'secondary', 'submit', false ); ?>
 		<?php
 		$give_api_is_bottom = true;

Spacing +64 added lines, -64 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -53,11 +53,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'API Key', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'API Keys', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('API Key', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('API Keys', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 		$this->query();
 	}
@@ -73,8 +73,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		return $item[ $column_name ];
+	public function column_default($item, $column_name) {
+		return $item[$column_name];
 	}
 
 	/**
@@ -87,8 +87,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_key( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['key'] ) . '"/>';
+	public function column_key($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['key']).'"/>';
 	}
 
 	/**
@@ -101,8 +101,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_token( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['token'] ) . '"/>';
+	public function column_token($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['token']).'"/>';
 	}
 
 	/**
@@ -115,8 +115,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_secret( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['secret'] ) . '"/>';
+	public function column_secret($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['secret']).'"/>';
 	}
 
 	/**
@@ -126,46 +126,46 @@ 
 	 * @since  1.1
 	 * @return string
 	 */
-	public function column_user( $item ) {
+	public function column_user($item) {
 
 		$actions = array();
 
-		if ( apply_filters( 'give_api_log_requests', true ) ) {
+		if (apply_filters('give_api_log_requests', true)) {
 			$actions['view'] = sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( add_query_arg( array(
+				esc_url(add_query_arg(array(
 					'section'   => 'api_requests',
 					'post_type' => 'give_forms',
 					'page'      => 'give-tools',
 					'tab'       => 'logs',
 					's'         => $item['email'],
-				), 'edit.php' ) ),
-				esc_html__( 'View API Log', 'give' )
+				), 'edit.php')),
+				esc_html__('View API Log', 'give')
 			);
 		}
 
 		$actions['reissue'] = sprintf(
 			'<a href="%s" class="give-regenerate-api-key">%s</a>',
-			esc_url( wp_nonce_url( add_query_arg( array(
+			esc_url(wp_nonce_url(add_query_arg(array(
 				'user_id'          => $item['id'],
 				'give_action'      => 'process_api_key',
 				'give_api_process' => 'regenerate',
-			) ), 'give-api-nonce' ) ),
-			esc_html__( 'Reissue', 'give' )
+			)), 'give-api-nonce')),
+			esc_html__('Reissue', 'give')
 		);
-		$actions['revoke']  = sprintf(
+		$actions['revoke'] = sprintf(
 			'<a href="%s" class="give-revoke-api-key give-delete">%s</a>',
-			esc_url( wp_nonce_url( add_query_arg( array(
+			esc_url(wp_nonce_url(add_query_arg(array(
 				'user_id'          => $item['id'],
 				'give_action'      => 'process_api_key',
 				'give_api_process' => 'revoke',
-			) ), 'give-api-nonce' ) ),
-			esc_html__( 'Revoke', 'give' )
+			)), 'give-api-nonce')),
+			esc_html__('Revoke', 'give')
 		);
 
-		$actions = apply_filters( 'give_api_row_actions', array_filter( $actions ) );
+		$actions = apply_filters('give_api_row_actions', array_filter($actions));
 
-		return sprintf( '%1$s %2$s', $item['user'], $this->row_actions( $actions ) );
+		return sprintf('%1$s %2$s', $item['user'], $this->row_actions($actions));
 	}
 
 	/**
@@ -189,10 +189,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'user'   => esc_html__( 'Username', 'give' ),
-			'key'    => esc_html__( 'Public Key', 'give' ),
-			'token'  => esc_html__( 'Token', 'give' ),
-			'secret' => esc_html__( 'Secret Key', 'give' ),
+			'user'   => esc_html__('Username', 'give'),
+			'key'    => esc_html__('Public Key', 'give'),
+			'token'  => esc_html__('Token', 'give'),
+			'secret' => esc_html__('Secret Key', 'give'),
 		);
 
 		return $columns;
@@ -206,20 +206,20 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -237,17 +237,17 @@ 
 	 *
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place.
 		static $give_api_is_bottom;
 
-		if ( $give_api_is_bottom ) {
+		if ($give_api_is_bottom) {
 			return;
 		}
 		?>
 		<input type="hidden" name="give_action" value="process_api_key"/>
 		<input type="hidden" name="give_api_process" value="generate"/>
-		<?php wp_nonce_field( 'give-api-nonce' );
+		<?php wp_nonce_field('give-api-nonce');
 		/**
 		 * API Key user search.
 		 */
@@ -256,7 +256,7 @@ 
 		  'name' => 'user_id',
         );
         echo Give()->html->ajax_user_search($args); ?>
-		<?php submit_button( esc_html__( 'Generate New API Keys', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Generate New API Keys', 'give'), 'secondary', 'submit', false); ?>
 		<?php
 		$give_api_is_bottom = true;
 	}
@@ -269,7 +269,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -280,21 +280,21 @@ 
 	 * @return array
 	 */
 	public function query() {
-		$users = get_users( array(
+		$users = get_users(array(
 			'meta_value' => 'give_user_secret_key',
 			'number'     => $this->per_page,
-			'offset'     => $this->per_page * ( $this->get_paged() - 1 ),
-		) );
-		$keys  = array();
-
-		foreach ( $users as $user ) {
-			$keys[ $user->ID ]['id']    = $user->ID;
-			$keys[ $user->ID ]['email'] = $user->user_email;
-			$keys[ $user->ID ]['user']  = '<a href="' . add_query_arg( 'user_id', $user->ID, 'user-edit.php' ) . '"><strong>' . $user->user_login . '</strong></a>';
-
-			$keys[ $user->ID ]['key']    = Give()->api->get_user_public_key( $user->ID );
-			$keys[ $user->ID ]['secret'] = Give()->api->get_user_secret_key( $user->ID );
-			$keys[ $user->ID ]['token']  = Give()->api->get_token( $user->ID );
+			'offset'     => $this->per_page * ($this->get_paged() - 1),
+		));
+		$keys = array();
+
+		foreach ($users as $user) {
+			$keys[$user->ID]['id']    = $user->ID;
+			$keys[$user->ID]['email'] = $user->user_email;
+			$keys[$user->ID]['user']  = '<a href="'.add_query_arg('user_id', $user->ID, 'user-edit.php').'"><strong>'.$user->user_login.'</strong></a>';
+
+			$keys[$user->ID]['key']    = Give()->api->get_user_public_key($user->ID);
+			$keys[$user->ID]['secret'] = Give()->api->get_user_secret_key($user->ID);
+			$keys[$user->ID]['token']  = Give()->api->get_token($user->ID);
 		}
 
 		return $keys;
@@ -311,7 +311,7 @@ 
 	public function total_items() {
 		global $wpdb;
 
-		if ( ! ( $total_items = Give_Cache::get( 'give_total_api_keys', true ) ) ) {
+		if ( ! ($total_items = Give_Cache::get('give_total_api_keys', true))) {
 			$total_items = $wpdb->get_var(
 				$wpdb->prepare(
 					"SELECT count(user_id)
@@ -320,7 +320,7 @@ 
 				)
 			);
 
-			Give_Cache::set( 'give_total_api_keys', $total_items, HOUR_IN_SECONDS, true );
+			Give_Cache::set('give_total_api_keys', $total_items, HOUR_IN_SECONDS, true);
 		}
 
 		return $total_items;
@@ -339,7 +339,7 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = array(); // Not sortable... for now
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$data = $this->query();
 
@@ -347,10 +347,10 @@ 
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/tools/export/class-batch-export-forms.php

Spacing +51 added lines, -51 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -41,20 +41,20 @@ 
 	public function csv_cols() {
 
 		$cols = array(
-			'ID'                      => __( 'ID', 'give' ),
-			'post_name'               => __( 'Slug', 'give' ),
-			'post_title'              => __( 'Name', 'give' ),
-			'post_date'               => __( 'Date Created', 'give' ),
-			'post_author'             => __( 'Author', 'give' ),
-			'post_content'            => __( 'Description', 'give' ),
-			'post_excerpt'            => __( 'Excerpt', 'give' ),
-			'post_status'             => __( 'Status', 'give' ),
-			'categories'              => __( 'Categories', 'give' ),
-			'tags'                    => __( 'Tags', 'give' ),
-			'give_price'              => __( 'Price', 'give' ),
-			'_thumbnail_id'           => __( 'Featured Image', 'give' ),
-			'_give_form_sales'        => __( 'Donations', 'give' ),
-			'_give_download_earnings' => __( 'Income', 'give' ),
+			'ID'                      => __('ID', 'give'),
+			'post_name'               => __('Slug', 'give'),
+			'post_title'              => __('Name', 'give'),
+			'post_date'               => __('Date Created', 'give'),
+			'post_author'             => __('Author', 'give'),
+			'post_content'            => __('Description', 'give'),
+			'post_excerpt'            => __('Excerpt', 'give'),
+			'post_status'             => __('Status', 'give'),
+			'categories'              => __('Categories', 'give'),
+			'tags'                    => __('Tags', 'give'),
+			'give_price'              => __('Price', 'give'),
+			'_thumbnail_id'           => __('Featured Image', 'give'),
+			'_give_form_sales'        => __('Donations', 'give'),
+			'_give_download_earnings' => __('Income', 'give'),
 		);
 
 		return $cols;
@@ -84,43 +84,43 @@ 
 			'paged'          => $this->step
 		);
 
-		$forms = new WP_Query( $args );
+		$forms = new WP_Query($args);
 
-		if ( $forms->posts ) {
-			foreach ( $forms->posts as $form ) {
+		if ($forms->posts) {
+			foreach ($forms->posts as $form) {
 
 				$row = [];
 
-				foreach ( $this->csv_cols() as $key => $value ) {
+				foreach ($this->csv_cols() as $key => $value) {
 
 					// Setup default value/
-					$row[ $key ] = '';
+					$row[$key] = '';
 
-					if ( in_array( $key, $meta ) ) {
+					if (in_array($key, $meta)) {
 
-						switch ( $key ) {
+						switch ($key) {
 
 							case '_thumbnail_id' :
 
-								$image_id    = get_post_thumbnail_id( $form->ID );
-								$row[ $key ] = wp_get_attachment_url( $image_id );
+								$image_id    = get_post_thumbnail_id($form->ID);
+								$row[$key] = wp_get_attachment_url($image_id);
 
 								break;
 
 							case 'give_price' :
 
-								if ( give_has_variable_prices( $form->ID ) ) {
+								if (give_has_variable_prices($form->ID)) {
 
 									$prices = [];
-									foreach ( give_get_variable_prices( $form->ID ) as $price ) {
-										$prices[] = $price['name'] . ': ' . $price['amount'];
+									foreach (give_get_variable_prices($form->ID) as $price) {
+										$prices[] = $price['name'].': '.$price['amount'];
 									}
 
-									$row[ $key ] = implode( ' | ', $prices );
+									$row[$key] = implode(' | ', $prices);
 
 								} else {
 
-									$row[ $key ] = give_get_form_price( $form->ID );
+									$row[$key] = give_get_form_price($form->ID);
 
 								}
 
@@ -128,44 +128,44 @@ 
 
 							default :
 
-								$row[ $key ] = give_get_meta( $form->ID, $key, TRUE );
+								$row[$key] = give_get_meta($form->ID, $key, TRUE);
 
 								break;
 
 						}
 
-					} elseif ( isset( $form->$key ) ) {
+					} elseif (isset($form->$key)) {
 
-						switch ( $key ) {
+						switch ($key) {
 
 							case 'post_author' :
 
-								$row[ $key ] = get_the_author_meta( 'user_login', $form->post_author );
+								$row[$key] = get_the_author_meta('user_login', $form->post_author);
 
 								break;
 
 							default :
 
-								$row[ $key ] = $form->$key;
+								$row[$key] = $form->$key;
 
 								break;
 						}
 
-					} elseif ( 'tags' == $key ) {
+					} elseif ('tags' == $key) {
 
-						$terms = get_the_terms( $form->ID, 'form_tag' );
-						if ( $terms ) {
-							$terms       = wp_list_pluck( $terms, 'name' );
-							$row[ $key ] = implode( ' | ', $terms );
+						$terms = get_the_terms($form->ID, 'form_tag');
+						if ($terms) {
+							$terms       = wp_list_pluck($terms, 'name');
+							$row[$key] = implode(' | ', $terms);
 						}
 
 
-					} elseif ( 'categories' == $key ) {
+					} elseif ('categories' == $key) {
 
-						$terms = get_the_terms( $form->ID, 'form_category' );
-						if ( $terms ) {
-							$terms       = wp_list_pluck( $terms, 'name' );
-							$row[ $key ] = implode( ' | ', $terms );
+						$terms = get_the_terms($form->ID, 'form_category');
+						if ($terms) {
+							$terms       = wp_list_pluck($terms, 'name');
+							$row[$key] = implode(' | ', $terms);
 						}
 
 					}
@@ -176,8 +176,8 @@ 
 
 			}
 
-			$data = apply_filters( 'give_export_get_data', $data );
-			$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+			$data = apply_filters('give_export_get_data', $data);
+			$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 			return $data;
 
@@ -197,20 +197,20 @@ 
 
 		$args = array(
 			'post_type'      => 'give_forms',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'any',
 			'fields'         => 'ids',
 		);
 
-		$forms  = new WP_Query( $args );
+		$forms = new WP_Query($args);
 		$total      = (int) $forms->post_count;
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 

includes/api/class-give-api.php

Doc Comments +3 added lines, -3 removed lines

@@ -216,7 +216,7 @@ 
 	 *
 	 * @access public
 	 * @since  1.1
-	 * @return array
+	 * @return string
 	 */
 	public function get_versions() {
 		return $this->versions;
@@ -302,7 +302,7 @@ 
 	 * @uses   Give_API::invalid_key()
 	 * @uses   Give_API::invalid_auth()
 	 * @since  1.1
-	 * @return bool
+	 * @return false|null
 	 */
 	private function validate_request() {
 		global $wp_query;
@@ -694,7 +694,7 @@ 
 	 *
 	 * @param array $args Arguments to override defaults
 	 *
-	 * @return array $dates
+	 * @return integer|null $dates
 	 */
 	public function get_dates( $args = array() ) {
 		$dates = array();

Spacing +511 added lines, -511 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -141,27 +141,27 @@ 
 			'v1' => 'GIVE_API_V1',
 		);
 
-		foreach ( $this->get_versions() as $version => $class ) {
-			require_once GIVE_PLUGIN_DIR . 'includes/api/class-give-api-' . $version . '.php';
+		foreach ($this->get_versions() as $version => $class) {
+			require_once GIVE_PLUGIN_DIR.'includes/api/class-give-api-'.$version.'.php';
 		}
 
-		add_action( 'init', array( $this, 'add_endpoint' ) );
-		add_action( 'wp', array( $this, 'process_query' ), - 1 );
-		add_filter( 'query_vars', array( $this, 'query_vars' ) );
-		add_action( 'show_user_profile', array( $this, 'user_key_field' ) );
-		add_action( 'edit_user_profile', array( $this, 'user_key_field' ) );
-		add_action( 'personal_options_update', array( $this, 'generate_api_key' ) );
-		add_action( 'edit_user_profile_update', array( $this, 'generate_api_key' ) );
-		add_action( 'give_process_api_key', array( $this, 'process_api_key' ) );
+		add_action('init', array($this, 'add_endpoint'));
+		add_action('wp', array($this, 'process_query'), - 1);
+		add_filter('query_vars', array($this, 'query_vars'));
+		add_action('show_user_profile', array($this, 'user_key_field'));
+		add_action('edit_user_profile', array($this, 'user_key_field'));
+		add_action('personal_options_update', array($this, 'generate_api_key'));
+		add_action('edit_user_profile_update', array($this, 'generate_api_key'));
+		add_action('give_process_api_key', array($this, 'process_api_key'));
 
 		// Setup a backwards compatibility check for user API Keys
-		add_filter( 'get_user_metadata', array( $this, 'api_key_backwards_compat' ), 10, 4 );
+		add_filter('get_user_metadata', array($this, 'api_key_backwards_compat'), 10, 4);
 
 		// Determine if JSON_PRETTY_PRINT is available
-		$this->pretty_print = defined( 'JSON_PRETTY_PRINT' ) ? JSON_PRETTY_PRINT : null;
+		$this->pretty_print = defined('JSON_PRETTY_PRINT') ? JSON_PRETTY_PRINT : null;
 
 		// Allow API request logging to be turned off
-		$this->log_requests = apply_filters( 'give_api_log_requests', $this->log_requests );
+		$this->log_requests = apply_filters('give_api_log_requests', $this->log_requests);
 
 		// Setup Give_Payment_Stats instance
 		$this->stats = new Give_Payment_Stats();
@@ -176,7 +176,7 @@ 
 	 * @since  1.1
 	 */
 	public function add_endpoint() {
-		add_rewrite_endpoint( 'give-api', EP_ALL );
+		add_rewrite_endpoint('give-api', EP_ALL);
 	}
 
 	/**
@@ -189,7 +189,7 @@ 
 	 *
 	 * @return string[] $vars New query vars
 	 */
-	public function query_vars( $vars ) {
+	public function query_vars($vars) {
 
 		$vars[] = 'token';
 		$vars[] = 'key';
@@ -240,11 +240,11 @@ 
 	 */
 	public function get_default_version() {
 
-		$version = get_option( 'give_default_api_version' );
+		$version = get_option('give_default_api_version');
 
-		if ( defined( 'GIVE_API_VERSION' ) ) {
+		if (defined('GIVE_API_VERSION')) {
 			$version = GIVE_API_VERSION;
-		} elseif ( ! $version ) {
+		} elseif ( ! $version) {
 			$version = 'v1';
 		}
 
@@ -265,14 +265,14 @@ 
 
 		$version = $wp_query->query_vars['give-api'];
 
-		if ( strpos( $version, '/' ) ) {
+		if (strpos($version, '/')) {
 
-			$version = explode( '/', $version );
-			$version = strtolower( $version[0] );
+			$version = explode('/', $version);
+			$version = strtolower($version[0]);
 
-			$wp_query->query_vars['give-api'] = str_replace( $version . '/', '', $wp_query->query_vars['give-api'] );
+			$wp_query->query_vars['give-api'] = str_replace($version.'/', '', $wp_query->query_vars['give-api']);
 
-			if ( array_key_exists( $version, $this->versions ) ) {
+			if (array_key_exists($version, $this->versions)) {
 
 				$this->queried_version = $version;
 
@@ -308,16 +308,16 @@ 
 		$this->override = false;
 
 		// Make sure we have both user and api key
-		if ( ! empty( $wp_query->query_vars['give-api'] ) && ( $wp_query->query_vars['give-api'] !== 'forms' || ! empty( $wp_query->query_vars['token'] ) ) ) {
+		if ( ! empty($wp_query->query_vars['give-api']) && ($wp_query->query_vars['give-api'] !== 'forms' || ! empty($wp_query->query_vars['token']))) {
 
-			if ( empty( $wp_query->query_vars['token'] ) || empty( $wp_query->query_vars['key'] ) ) {
+			if (empty($wp_query->query_vars['token']) || empty($wp_query->query_vars['key'])) {
 				$this->missing_auth();
 
 				return false;
 			}
 
 			// Retrieve the user by public API key and ensure they exist
-			if ( ! ( $user = $this->get_user( $wp_query->query_vars['key'] ) ) ) {
+			if ( ! ($user = $this->get_user($wp_query->query_vars['key']))) {
 
 				$this->invalid_key();
 
@@ -325,11 +325,11 @@ 
 
 			} else {
 
-				$token  = urldecode( $wp_query->query_vars['token'] );
-				$secret = $this->get_user_secret_key( $user );
-				$public = urldecode( $wp_query->query_vars['key'] );
+				$token  = urldecode($wp_query->query_vars['token']);
+				$secret = $this->get_user_secret_key($user);
+				$public = urldecode($wp_query->query_vars['key']);
 
-				if ( hash_equals( md5( $secret . $public ), $token ) ) {
+				if (hash_equals(md5($secret.$public), $token)) {
 					$this->is_valid_request = true;
 				} else {
 					$this->invalid_auth();
@@ -338,9 +338,9 @@ 
 				}
 
 			}
-		} elseif ( ! empty( $wp_query->query_vars['give-api'] ) && $wp_query->query_vars['give-api'] === 'forms' ) {
+		} elseif ( ! empty($wp_query->query_vars['give-api']) && $wp_query->query_vars['give-api'] === 'forms') {
 			$this->is_valid_request = true;
-			$wp_query->set( 'key', 'public' );
+			$wp_query->set('key', 'public');
 		}
 	}
 
@@ -356,25 +356,25 @@ 
 	 *
 	 * @return bool if user ID is found, false otherwise
 	 */
-	public function get_user( $key = '' ) {
+	public function get_user($key = '') {
 		global $wpdb, $wp_query;
 
-		if ( empty( $key ) ) {
-			$key = urldecode( $wp_query->query_vars['key'] );
+		if (empty($key)) {
+			$key = urldecode($wp_query->query_vars['key']);
 		}
 
-		if ( empty( $key ) ) {
+		if (empty($key)) {
 			return false;
 		}
 
-		$user = Give_Cache::get( md5( 'give_api_user_' . $key ), true );
+		$user = Give_Cache::get(md5('give_api_user_'.$key), true);
 
-		if ( false === $user ) {
-			$user = $wpdb->get_var( $wpdb->prepare( "SELECT user_id FROM $wpdb->usermeta WHERE meta_key = %s LIMIT 1", $key ) );
-			Give_Cache::set( md5( 'give_api_user_' . $key ), $user, DAY_IN_SECONDS, true );
+		if (false === $user) {
+			$user = $wpdb->get_var($wpdb->prepare("SELECT user_id FROM $wpdb->usermeta WHERE meta_key = %s LIMIT 1", $key));
+			Give_Cache::set(md5('give_api_user_'.$key), $user, DAY_IN_SECONDS, true);
 		}
 
-		if ( $user != null ) {
+		if ($user != null) {
 			$this->user_id = $user;
 
 			return $user;
@@ -390,19 +390,19 @@ 
 	 *
 	 * @return mixed|null|string
 	 */
-	public function get_user_public_key( $user_id = 0 ) {
+	public function get_user_public_key($user_id = 0) {
 		global $wpdb;
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return '';
 		}
 
-		$cache_key       = md5( 'give_api_user_public_key' . $user_id );
-		$user_public_key = Give_Cache::get( $cache_key, true );
+		$cache_key       = md5('give_api_user_public_key'.$user_id);
+		$user_public_key = Give_Cache::get($cache_key, true);
 
-		if ( empty( $user_public_key ) ) {
-			$user_public_key = $wpdb->get_var( $wpdb->prepare( "SELECT meta_key FROM $wpdb->usermeta WHERE meta_value = 'give_user_public_key' AND user_id = %d", $user_id ) );
-			Give_Cache::set( $cache_key, $user_public_key, HOUR_IN_SECONDS, true );
+		if (empty($user_public_key)) {
+			$user_public_key = $wpdb->get_var($wpdb->prepare("SELECT meta_key FROM $wpdb->usermeta WHERE meta_value = 'give_user_public_key' AND user_id = %d", $user_id));
+			Give_Cache::set($cache_key, $user_public_key, HOUR_IN_SECONDS, true);
 		}
 
 		return $user_public_key;
@@ -415,19 +415,19 @@ 
 	 *
 	 * @return mixed|null|string
 	 */
-	public function get_user_secret_key( $user_id = 0 ) {
+	public function get_user_secret_key($user_id = 0) {
 		global $wpdb;
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return '';
 		}
 
-		$cache_key       = md5( 'give_api_user_secret_key' . $user_id );
-		$user_secret_key = Give_Cache::get( $cache_key, true );
+		$cache_key       = md5('give_api_user_secret_key'.$user_id);
+		$user_secret_key = Give_Cache::get($cache_key, true);
 
-		if ( empty( $user_secret_key ) ) {
-			$user_secret_key = $wpdb->get_var( $wpdb->prepare( "SELECT meta_key FROM $wpdb->usermeta WHERE meta_value = 'give_user_secret_key' AND user_id = %d", $user_id ) );
-			Give_Cache::set( $cache_key, $user_secret_key, HOUR_IN_SECONDS, true );
+		if (empty($user_secret_key)) {
+			$user_secret_key = $wpdb->get_var($wpdb->prepare("SELECT meta_key FROM $wpdb->usermeta WHERE meta_value = 'give_user_secret_key' AND user_id = %d", $user_id));
+			Give_Cache::set($cache_key, $user_secret_key, HOUR_IN_SECONDS, true);
 		}
 
 		return $user_secret_key;
@@ -443,10 +443,10 @@ 
 	 */
 	private function missing_auth() {
 		$error          = array();
-		$error['error'] = __( 'You must specify both a token and API key.', 'give' );
+		$error['error'] = __('You must specify both a token and API key.', 'give');
 
 		$this->data = $error;
-		$this->output( 401 );
+		$this->output(401);
 	}
 
 	/**
@@ -460,10 +460,10 @@ 
 	 */
 	private function invalid_auth() {
 		$error          = array();
-		$error['error'] = __( 'Your request could not be authenticated.', 'give' );
+		$error['error'] = __('Your request could not be authenticated.', 'give');
 
 		$this->data = $error;
-		$this->output( 403 );
+		$this->output(403);
 	}
 
 	/**
@@ -477,10 +477,10 @@ 
 	 */
 	private function invalid_key() {
 		$error          = array();
-		$error['error'] = __( 'Invalid API key.', 'give' );
+		$error['error'] = __('Invalid API key.', 'give');
 
 		$this->data = $error;
-		$this->output( 403 );
+		$this->output(403);
 	}
 
 	/**
@@ -493,10 +493,10 @@ 
 	 */
 	private function invalid_version() {
 		$error          = array();
-		$error['error'] = __( 'Invalid API version.', 'give' );
+		$error['error'] = __('Invalid API version.', 'give');
 
 		$this->data = $error;
-		$this->output( 404 );
+		$this->output(404);
 	}
 
 	/**
@@ -512,10 +512,10 @@ 
 		global $wp_query;
 
 		// Start logging how long the request takes for logging
-		$before = microtime( true );
+		$before = microtime(true);
 
 		// Check for give-api var. Get out if not present
-		if ( empty( $wp_query->query_vars['give-api'] ) ) {
+		if (empty($wp_query->query_vars['give-api'])) {
 			return;
 		}
 
@@ -529,45 +529,45 @@ 
 		$this->validate_request();
 
 		// Only proceed if no errors have been noted
-		if ( ! $this->is_valid_request ) {
+		if ( ! $this->is_valid_request) {
 			return;
 		}
 
-		if ( ! defined( 'GIVE_DOING_API' ) ) {
-			define( 'GIVE_DOING_API', true );
+		if ( ! defined('GIVE_DOING_API')) {
+			define('GIVE_DOING_API', true);
 		}
 
 		$data         = array();
 		$this->routes = new $this->versions[$this->get_queried_version()];
 		$this->routes->validate_request();
 
-		switch ( $this->endpoint ) :
+		switch ($this->endpoint) :
 
 			case 'stats' :
 
-				$data = $this->routes->get_stats( array(
-					'type'      => isset( $wp_query->query_vars['type'] ) ? $wp_query->query_vars['type'] : null,
-					'form'      => isset( $wp_query->query_vars['form'] ) ? $wp_query->query_vars['form'] : null,
-					'date'      => isset( $wp_query->query_vars['date'] ) ? $wp_query->query_vars['date'] : null,
-					'startdate' => isset( $wp_query->query_vars['startdate'] ) ? $wp_query->query_vars['startdate'] : null,
-					'enddate'   => isset( $wp_query->query_vars['enddate'] ) ? $wp_query->query_vars['enddate'] : null,
-				) );
+				$data = $this->routes->get_stats(array(
+					'type'      => isset($wp_query->query_vars['type']) ? $wp_query->query_vars['type'] : null,
+					'form'      => isset($wp_query->query_vars['form']) ? $wp_query->query_vars['form'] : null,
+					'date'      => isset($wp_query->query_vars['date']) ? $wp_query->query_vars['date'] : null,
+					'startdate' => isset($wp_query->query_vars['startdate']) ? $wp_query->query_vars['startdate'] : null,
+					'enddate'   => isset($wp_query->query_vars['enddate']) ? $wp_query->query_vars['enddate'] : null,
+				));
 
 				break;
 
 			case 'forms' :
 
-				$form = isset( $wp_query->query_vars['form'] ) ? $wp_query->query_vars['form'] : null;
+				$form = isset($wp_query->query_vars['form']) ? $wp_query->query_vars['form'] : null;
 
-				$data = $this->routes->get_forms( $form );
+				$data = $this->routes->get_forms($form);
 
 				break;
 
 			case 'donors' :
 
-				$donor = isset( $wp_query->query_vars['donor'] ) ? $wp_query->query_vars['donor'] : null;
+				$donor = isset($wp_query->query_vars['donor']) ? $wp_query->query_vars['donor'] : null;
 
-				$data = $this->routes->get_donors( $donor );
+				$data = $this->routes->get_donors($donor);
 
 				break;
 
@@ -580,26 +580,26 @@ 
 				 * @params date startdate | required when date = range and format to be YYYYMMDD (i.e. 20170524)
 				 * @params date enddate | required when date = range and format to be YYYYMMDD (i.e. 20170524)
 				 */
-				$data = $this->routes->get_recent_donations( array(
-					'id'        => isset( $wp_query->query_vars['id'] ) ? $wp_query->query_vars['id'] : null,
-					'date'      => isset( $wp_query->query_vars['date'] ) ? $wp_query->query_vars['date'] : null,
-					'startdate' => isset( $wp_query->query_vars['startdate'] ) ? $wp_query->query_vars['startdate'] : null,
-					'enddate'   => isset( $wp_query->query_vars['enddate'] ) ? $wp_query->query_vars['enddate'] : null,
-				) );
+				$data = $this->routes->get_recent_donations(array(
+					'id'        => isset($wp_query->query_vars['id']) ? $wp_query->query_vars['id'] : null,
+					'date'      => isset($wp_query->query_vars['date']) ? $wp_query->query_vars['date'] : null,
+					'startdate' => isset($wp_query->query_vars['startdate']) ? $wp_query->query_vars['startdate'] : null,
+					'enddate'   => isset($wp_query->query_vars['enddate']) ? $wp_query->query_vars['enddate'] : null,
+				));
 
 				break;
 
 		endswitch;
 
 		// Allow extensions to setup their own return data
-		$this->data = apply_filters( 'give_api_output_data', $data, $this->endpoint, $this );
+		$this->data = apply_filters('give_api_output_data', $data, $this->endpoint, $this);
 
-		$after                       = microtime( true );
-		$request_time                = ( $after - $before );
+		$after                       = microtime(true);
+		$request_time                = ($after - $before);
 		$this->data['request_speed'] = $request_time;
 
 		// Log this API request, if enabled. We log it here because we have access to errors.
-		$this->log_request( $this->data );
+		$this->log_request($this->data);
 
 		// Send out data to the output function
 		$this->output();
@@ -629,25 +629,25 @@ 
 		global $wp_query;
 
 		// Whitelist our query options
-		$accepted = apply_filters( 'give_api_valid_query_modes', array(
+		$accepted = apply_filters('give_api_valid_query_modes', array(
 			'stats',
 			'forms',
 			'donors',
 			'donations',
-		) );
+		));
 
-		$query = isset( $wp_query->query_vars['give-api'] ) ? $wp_query->query_vars['give-api'] : null;
-		$query = str_replace( $this->queried_version . '/', '', $query );
+		$query = isset($wp_query->query_vars['give-api']) ? $wp_query->query_vars['give-api'] : null;
+		$query = str_replace($this->queried_version.'/', '', $query);
 
 		$error = array();
 
 		// Make sure our query is valid
-		if ( ! in_array( $query, $accepted ) ) {
-			$error['error'] = __( 'Invalid query.', 'give' );
+		if ( ! in_array($query, $accepted)) {
+			$error['error'] = __('Invalid query.', 'give');
 
 			$this->data = $error;
 			// 400 is Bad Request
-			$this->output( 400 );
+			$this->output(400);
 		}
 
 		$this->endpoint = $query;
@@ -664,7 +664,7 @@ 
 	public function get_paged() {
 		global $wp_query;
 
-		return isset( $wp_query->query_vars['page'] ) ? $wp_query->query_vars['page'] : 1;
+		return isset($wp_query->query_vars['page']) ? $wp_query->query_vars['page'] : 1;
 	}
 
 
@@ -679,13 +679,13 @@ 
 	public function per_page() {
 		global $wp_query;
 
-		$per_page = isset( $wp_query->query_vars['number'] ) ? $wp_query->query_vars['number'] : 10;
+		$per_page = isset($wp_query->query_vars['number']) ? $wp_query->query_vars['number'] : 10;
 
-		if ( $per_page < 0 && $this->get_query_mode() == 'donors' ) {
+		if ($per_page < 0 && $this->get_query_mode() == 'donors') {
 			$per_page = 99999999;
 		} // End if().
 
-		return apply_filters( 'give_api_results_per_page', $per_page );
+		return apply_filters('give_api_results_per_page', $per_page);
 	}
 
 	/**
@@ -698,7 +698,7 @@ 
 	 *
 	 * @return array $dates
 	 */
-	public function get_dates( $args = array() ) {
+	public function get_dates($args = array()) {
 		$dates = array();
 
 		$defaults = array(
@@ -709,60 +709,60 @@ 
 			'enddate'   => null,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$current_time = current_time( 'timestamp' );
+		$current_time = current_time('timestamp');
 
-		if ( 'range' === $args['date'] ) {
-			$startdate          = strtotime( $args['startdate'] );
-			$enddate            = strtotime( $args['enddate'] );
-			$dates['day_start'] = date( 'd', $startdate );
-			$dates['day_end']   = date( 'd', $enddate );
-			$dates['m_start']   = date( 'n', $startdate );
-			$dates['m_end']     = date( 'n', $enddate );
-			$dates['year']      = date( 'Y', $startdate );
-			$dates['year_end']  = date( 'Y', $enddate );
+		if ('range' === $args['date']) {
+			$startdate          = strtotime($args['startdate']);
+			$enddate            = strtotime($args['enddate']);
+			$dates['day_start'] = date('d', $startdate);
+			$dates['day_end']   = date('d', $enddate);
+			$dates['m_start']   = date('n', $startdate);
+			$dates['m_end']     = date('n', $enddate);
+			$dates['year']      = date('Y', $startdate);
+			$dates['year_end']  = date('Y', $enddate);
 		} else {
 			// Modify dates based on predefined ranges
-			switch ( $args['date'] ) :
+			switch ($args['date']) :
 
 				case 'this_month' :
 					$dates['day']     = null;
-					$dates['m_start'] = date( 'n', $current_time );
-					$dates['m_end']   = date( 'n', $current_time );
-					$dates['year']    = date( 'Y', $current_time );
+					$dates['m_start'] = date('n', $current_time);
+					$dates['m_end']   = date('n', $current_time);
+					$dates['year']    = date('Y', $current_time);
 					break;
 
 				case 'last_month' :
 					$dates['day']     = null;
-					$dates['m_start'] = date( 'n', $current_time ) == 1 ? 12 : date( 'n', $current_time ) - 1;
+					$dates['m_start'] = date('n', $current_time) == 1 ? 12 : date('n', $current_time) - 1;
 					$dates['m_end']   = $dates['m_start'];
-					$dates['year']    = date( 'n', $current_time ) == 1 ? date( 'Y', $current_time ) - 1 : date( 'Y', $current_time );
+					$dates['year']    = date('n', $current_time) == 1 ? date('Y', $current_time) - 1 : date('Y', $current_time);
 					break;
 
 				case 'today' :
-					$dates['day']     = date( 'd', $current_time );
-					$dates['m_start'] = date( 'n', $current_time );
-					$dates['m_end']   = date( 'n', $current_time );
-					$dates['year']    = date( 'Y', $current_time );
+					$dates['day']     = date('d', $current_time);
+					$dates['m_start'] = date('n', $current_time);
+					$dates['m_end']   = date('n', $current_time);
+					$dates['year']    = date('Y', $current_time);
 					break;
 
 				case 'yesterday' :
 
-					$year  = date( 'Y', $current_time );
-					$month = date( 'n', $current_time );
-					$day   = date( 'd', $current_time );
+					$year  = date('Y', $current_time);
+					$month = date('n', $current_time);
+					$day   = date('d', $current_time);
 
-					if ( $month == 1 && $day == 1 ) {
+					if ($month == 1 && $day == 1) {
 
-						$year  -= 1;
+						$year -= 1;
 						$month = 12;
-						$day   = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day   = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
-					} elseif ( $month > 1 && $day == 1 ) {
+					} elseif ($month > 1 && $day == 1) {
 
 						$month -= 1;
-						$day   = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
 					} else {
 
@@ -778,65 +778,65 @@ 
 					break;
 
 				case 'this_quarter' :
-					$month_now = date( 'n', $current_time );
+					$month_now = date('n', $current_time);
 
 					$dates['day'] = null;
 
-					if ( $month_now <= 3 ) {
+					if ($month_now <= 3) {
 
 						$dates['m_start'] = 1;
 						$dates['m_end']   = 3;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
-					} elseif ( $month_now <= 6 ) {
+					} elseif ($month_now <= 6) {
 
 						$dates['m_start'] = 4;
 						$dates['m_end']   = 6;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
-					} elseif ( $month_now <= 9 ) {
+					} elseif ($month_now <= 9) {
 
 						$dates['m_start'] = 7;
 						$dates['m_end']   = 9;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
 					} else {
 
 						$dates['m_start'] = 10;
 						$dates['m_end']   = 12;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
 					}
 					break;
 
 				case 'last_quarter' :
-					$month_now = date( 'n', $current_time );
+					$month_now = date('n', $current_time);
 
 					$dates['day'] = null;
 
-					if ( $month_now <= 3 ) {
+					if ($month_now <= 3) {
 
 						$dates['m_start'] = 10;
 						$dates['m_end']   = 12;
-						$dates['year']    = date( 'Y', $current_time ) - 1; // Previous year
+						$dates['year']    = date('Y', $current_time) - 1; // Previous year
 
-					} elseif ( $month_now <= 6 ) {
+					} elseif ($month_now <= 6) {
 
 						$dates['m_start'] = 1;
 						$dates['m_end']   = 3;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
-					} elseif ( $month_now <= 9 ) {
+					} elseif ($month_now <= 9) {
 
 						$dates['m_start'] = 4;
 						$dates['m_end']   = 6;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
 					} else {
 
 						$dates['m_start'] = 7;
 						$dates['m_end']   = 9;
-						$dates['year']    = date( 'Y', $current_time );
+						$dates['year']    = date('Y', $current_time);
 
 					}
 					break;
@@ -845,14 +845,14 @@ 
 					$dates['day']     = null;
 					$dates['m_start'] = null;
 					$dates['m_end']   = null;
-					$dates['year']    = date( 'Y', $current_time );
+					$dates['year']    = date('Y', $current_time);
 					break;
 
 				case 'last_year' :
 					$dates['day']     = null;
 					$dates['m_start'] = null;
 					$dates['m_end']   = null;
-					$dates['year']    = date( 'Y', $current_time ) - 1;
+					$dates['year']    = date('Y', $current_time) - 1;
 					break;
 
 			endswitch;
@@ -865,7 +865,7 @@ 
 		 *
 		 * @param array $dates The dates used for retrieving earnings.
 		 */
-		return apply_filters( 'give_api_stat_dates', $dates );
+		return apply_filters('give_api_stat_dates', $dates);
 	}
 
 	/**
@@ -879,78 +879,78 @@ 
 	 *
 	 * @return array $donors Multidimensional array of the donors.
 	 */
-	public function get_donors( $donor = null ) {
+	public function get_donors($donor = null) {
 
 		$donors = array();
 		$error  = array();
-		if ( ! user_can( $this->user_id, 'view_give_sensitive_data' ) && ! $this->override ) {
+		if ( ! user_can($this->user_id, 'view_give_sensitive_data') && ! $this->override) {
 			return $donors;
 		}
 
 		$paged    = $this->get_paged();
 		$per_page = $this->per_page();
-		$offset   = $per_page * ( $paged - 1 );
+		$offset   = $per_page * ($paged - 1);
 
-		if ( is_numeric( $donor ) ) {
+		if (is_numeric($donor)) {
 			$field = 'id';
 		} else {
 			$field = 'email';
 		}
 
-		$donor_query = Give()->donors->get_donors( array(
+		$donor_query = Give()->donors->get_donors(array(
 			'number' => $per_page,
 			'offset' => $offset,
 			$field   => $donor,
-		) );
+		));
 		$donor_count = 0;
 
-		if ( $donor_query ) {
+		if ($donor_query) {
 
-			foreach ( $donor_query as $donor_obj ) {
+			foreach ($donor_query as $donor_obj) {
 
-				$names      = explode( ' ', $donor_obj->name );
-				$first_name = ! empty( $names[0] ) ? $names[0] : '';
+				$names      = explode(' ', $donor_obj->name);
+				$first_name = ! empty($names[0]) ? $names[0] : '';
 				$last_name  = '';
-				if ( ! empty( $names[1] ) ) {
-					unset( $names[0] );
-					$last_name = implode( ' ', $names );
+				if ( ! empty($names[1])) {
+					unset($names[0]);
+					$last_name = implode(' ', $names);
 				}
 
-				$donors['donors'][ $donor_count ]['info']['user_id']      = '';
-				$donors['donors'][ $donor_count ]['info']['username']     = '';
-				$donors['donors'][ $donor_count ]['info']['display_name'] = '';
-				$donors['donors'][ $donor_count ]['info']['donor_id']     = $donor_obj->id;
-				$donors['donors'][ $donor_count ]['info']['first_name']   = $first_name;
-				$donors['donors'][ $donor_count ]['info']['last_name']    = $last_name;
-				$donors['donors'][ $donor_count ]['info']['email']        = $donor_obj->email;
+				$donors['donors'][$donor_count]['info']['user_id']      = '';
+				$donors['donors'][$donor_count]['info']['username']     = '';
+				$donors['donors'][$donor_count]['info']['display_name'] = '';
+				$donors['donors'][$donor_count]['info']['donor_id']     = $donor_obj->id;
+				$donors['donors'][$donor_count]['info']['first_name']   = $first_name;
+				$donors['donors'][$donor_count]['info']['last_name']    = $last_name;
+				$donors['donors'][$donor_count]['info']['email']        = $donor_obj->email;
 
-				if ( ! empty( $donor_obj->user_id ) ) {
+				if ( ! empty($donor_obj->user_id)) {
 
-					$user_data = get_userdata( $donor_obj->user_id );
+					$user_data = get_userdata($donor_obj->user_id);
 
 					// Donor with registered account.
-					$donors['donors'][ $donor_count ]['info']['user_id']      = $donor_obj->user_id;
-					$donors['donors'][ $donor_count ]['info']['username']     = $user_data->user_login;
-					$donors['donors'][ $donor_count ]['info']['display_name'] = $user_data->display_name;
+					$donors['donors'][$donor_count]['info']['user_id']      = $donor_obj->user_id;
+					$donors['donors'][$donor_count]['info']['username']     = $user_data->user_login;
+					$donors['donors'][$donor_count]['info']['display_name'] = $user_data->display_name;
 
 				}
 
-				$donors['donors'][ $donor_count ]['stats']['total_donations'] = $donor_obj->purchase_count;
-				$donors['donors'][ $donor_count ]['stats']['total_spent']     = $donor_obj->purchase_value;
+				$donors['donors'][$donor_count]['stats']['total_donations'] = $donor_obj->purchase_count;
+				$donors['donors'][$donor_count]['stats']['total_spent']     = $donor_obj->purchase_value;
 
-				$donor_count ++;
+				$donor_count++;
 
 			}
-		} elseif ( $donor ) {
+		} elseif ($donor) {
 
 			$error['error'] = sprintf( /* translators: %s: donor */
-				__( 'Donor %s not found.', 'give' ), $donor );
+				__('Donor %s not found.', 'give'), $donor );
 
 			return $error;
 
 		} else {
 
-			$error['error'] = __( 'No donors found.', 'give' );
+			$error['error'] = __('No donors found.', 'give');
 
 			return $error;
 
@@ -969,37 +969,37 @@ 
 	 *
 	 * @return array $donors Multidimensional array of the forms.
 	 */
-	public function get_forms( $form = null ) {
+	public function get_forms($form = null) {
 
 		$forms = array();
 		$error = array();
 
-		if ( $form == null ) {
+		if ($form == null) {
 			$forms['forms'] = array();
 
-			$form_list = get_posts( array(
+			$form_list = get_posts(array(
 				'post_type'        => 'give_forms',
 				'posts_per_page'   => $this->per_page(),
 				'suppress_filters' => true,
 				'paged'            => $this->get_paged(),
-			) );
+			));
 
-			if ( $form_list ) {
+			if ($form_list) {
 				$i = 0;
-				foreach ( $form_list as $form_info ) {
-					$forms['forms'][ $i ] = $this->get_form_data( $form_info );
-					$i ++;
+				foreach ($form_list as $form_info) {
+					$forms['forms'][$i] = $this->get_form_data($form_info);
+					$i++;
 				}
 			}
 		} else {
-			if ( get_post_type( $form ) == 'give_forms' ) {
-				$form_info = get_post( $form );
+			if (get_post_type($form) == 'give_forms') {
+				$form_info = get_post($form);
 
-				$forms['forms'][0] = $this->get_form_data( $form_info );
+				$forms['forms'][0] = $this->get_form_data($form_info);
 
 			} else {
 				$error['error'] = sprintf( /* translators: %s: form */
-					__( 'Form %s not found.', 'give' ), $form );
+					__('Form %s not found.', 'give'), $form );
 
 				return $error;
 			}
@@ -1017,7 +1017,7 @@ 
 	 *
 	 * @return array                Array of post data to return back in the API.
 	 */
-	private function get_form_data( $form_info ) {
+	private function get_form_data($form_info) {
 
 		$form = array();
 
@@ -1027,60 +1027,60 @@ 
 		$form['info']['create_date']   = $form_info->post_date;
 		$form['info']['modified_date'] = $form_info->post_modified;
 		$form['info']['status']        = $form_info->post_status;
-		$form['info']['link']          = html_entity_decode( $form_info->guid );
-		$form['info']['content']       = give_get_meta( $form_info->ID, '_give_form_content', true );
-		$form['info']['thumbnail']     = wp_get_attachment_url( get_post_thumbnail_id( $form_info->ID ) );
+		$form['info']['link']          = html_entity_decode($form_info->guid);
+		$form['info']['content']       = give_get_meta($form_info->ID, '_give_form_content', true);
+		$form['info']['thumbnail']     = wp_get_attachment_url(get_post_thumbnail_id($form_info->ID));
 
-		if ( give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) ) ) {
-			$form['info']['category'] = get_the_terms( $form_info, 'give_forms_category' );
-			$form['info']['tags']     = get_the_terms( $form_info, 'give_forms_tag' );
+		if (give_is_setting_enabled(give_get_option('categories', 'disabled'))) {
+			$form['info']['category'] = get_the_terms($form_info, 'give_forms_category');
+			$form['info']['tags']     = get_the_terms($form_info, 'give_forms_tag');
 		}
-		if ( give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) ) ) {
-			$form['info']['tags'] = get_the_terms( $form_info, 'give_forms_tag' );
+		if (give_is_setting_enabled(give_get_option('tags', 'disabled'))) {
+			$form['info']['tags'] = get_the_terms($form_info, 'give_forms_tag');
 		}
 
 		// Check whether any goal is to be achieved for the donation form.
-		$goal_option = give_get_meta( $form_info->ID, '_give_goal_option', true );
-		$goal_amount = give_get_meta( $form_info->ID, '_give_set_goal', true );
-		if ( give_is_setting_enabled( $goal_option ) && $goal_amount ) {
-			$total_income                         = give_get_form_earnings_stats( $form_info->ID );
-			$goal_percentage_completed            = ( $total_income < $goal_amount ) ? round( ( $total_income / $goal_amount ) * 100, 2 ) : 100;
-			$form['goal']['amount']               = isset( $goal_amount ) ? $goal_amount : '';
-			$form['goal']['percentage_completed'] = isset( $goal_percentage_completed ) ? $goal_percentage_completed : '';
+		$goal_option = give_get_meta($form_info->ID, '_give_goal_option', true);
+		$goal_amount = give_get_meta($form_info->ID, '_give_set_goal', true);
+		if (give_is_setting_enabled($goal_option) && $goal_amount) {
+			$total_income                         = give_get_form_earnings_stats($form_info->ID);
+			$goal_percentage_completed            = ($total_income < $goal_amount) ? round(($total_income / $goal_amount) * 100, 2) : 100;
+			$form['goal']['amount']               = isset($goal_amount) ? $goal_amount : '';
+			$form['goal']['percentage_completed'] = isset($goal_percentage_completed) ? $goal_percentage_completed : '';
 		}
 
-		if ( user_can( $this->user_id, 'view_give_reports' ) || $this->override ) {
-			$form['stats']['total']['donations']           = give_get_form_sales_stats( $form_info->ID );
-			$form['stats']['total']['earnings']            = give_get_form_earnings_stats( $form_info->ID );
-			$form['stats']['monthly_average']['donations'] = give_get_average_monthly_form_sales( $form_info->ID );
-			$form['stats']['monthly_average']['earnings']  = give_get_average_monthly_form_earnings( $form_info->ID );
+		if (user_can($this->user_id, 'view_give_reports') || $this->override) {
+			$form['stats']['total']['donations']           = give_get_form_sales_stats($form_info->ID);
+			$form['stats']['total']['earnings']            = give_get_form_earnings_stats($form_info->ID);
+			$form['stats']['monthly_average']['donations'] = give_get_average_monthly_form_sales($form_info->ID);
+			$form['stats']['monthly_average']['earnings']  = give_get_average_monthly_form_earnings($form_info->ID);
 		}
 
 		$counter = 0;
-		if ( give_has_variable_prices( $form_info->ID ) ) {
-			foreach ( give_get_variable_prices( $form_info->ID ) as $price ) {
-				$counter ++;
+		if (give_has_variable_prices($form_info->ID)) {
+			foreach (give_get_variable_prices($form_info->ID) as $price) {
+				$counter++;
 				// multi-level item
-				$level                                     = isset( $price['_give_text'] ) ? $price['_give_text'] : 'level-' . $counter;
-				$form['pricing'][ sanitize_key( $level ) ] = $price['_give_amount'];
+				$level                                     = isset($price['_give_text']) ? $price['_give_text'] : 'level-'.$counter;
+				$form['pricing'][sanitize_key($level)] = $price['_give_amount'];
 
 			}
 		} else {
-			$form['pricing']['amount'] = give_get_form_price( $form_info->ID );
+			$form['pricing']['amount'] = give_get_form_price($form_info->ID);
 		}
 
-		if ( user_can( $this->user_id, 'view_give_sensitive_data' ) || $this->override ) {
+		if (user_can($this->user_id, 'view_give_sensitive_data') || $this->override) {
 
 			/**
 			 * Fires when generating API sensitive data.
 			 *
 			 * @since 1.1
 			 */
-			do_action( 'give_api_sensitive_data' );
+			do_action('give_api_sensitive_data');
 
 		}
 
-		return apply_filters( 'give_api_forms_form', $form );
+		return apply_filters('give_api_forms_form', $form);
 
 	}
 
@@ -1095,7 +1095,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_stats( $args = array() ) {
+	public function get_stats($args = array()) {
 		$defaults = array(
 			'type'      => null,
 			'form'      => null,
@@ -1104,9 +1104,9 @@ 
 			'enddate'   => null,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$dates = $this->get_dates( $args );
+		$dates = $this->get_dates($args);
 
 		$stats     = array();
 		$earnings  = array(
@@ -1117,40 +1117,40 @@ 
 		);
 		$error     = array();
 
-		if ( ! user_can( $this->user_id, 'view_give_reports' ) && ! $this->override ) {
+		if ( ! user_can($this->user_id, 'view_give_reports') && ! $this->override) {
 			return $stats;
 		}
 
-		if ( $args['type'] == 'donations' ) {
+		if ($args['type'] == 'donations') {
 
-			if ( $args['form'] == null ) {
-				if ( $args['date'] == null ) {
+			if ($args['form'] == null) {
+				if ($args['date'] == null) {
 					$donations = $this->get_default_sales_stats();
-				} elseif ( $args['date'] === 'range' ) {
+				} elseif ($args['date'] === 'range') {
 					// Return donations for a date range.
 					// Ensure the end date is later than the start date.
-					if ( $args['enddate'] < $args['startdate'] ) {
-						$error['error'] = __( 'The end date must be later than the start date.', 'give' );
+					if ($args['enddate'] < $args['startdate']) {
+						$error['error'] = __('The end date must be later than the start date.', 'give');
 					}
 
 					// Ensure both the start and end date are specified
-					if ( empty( $args['startdate'] ) || empty( $args['enddate'] ) ) {
-						$error['error'] = __( 'Invalid or no date range specified.', 'give' );
+					if (empty($args['startdate']) || empty($args['enddate'])) {
+						$error['error'] = __('Invalid or no date range specified.', 'give');
 					}
 
 					$total = 0;
 
 					// Loop through the years
 					$y = $dates['year'];
-					while ( $y <= $dates['year_end'] ) :
+					while ($y <= $dates['year_end']) :
 
-						if ( $dates['year'] == $dates['year_end'] ) {
+						if ($dates['year'] == $dates['year_end']) {
 							$month_start = $dates['m_start'];
 							$month_end   = $dates['m_end'];
-						} elseif ( $y == $dates['year'] && $dates['year_end'] > $dates['year'] ) {
+						} elseif ($y == $dates['year'] && $dates['year_end'] > $dates['year']) {
 							$month_start = $dates['m_start'];
 							$month_end   = 12;
-						} elseif ( $y == $dates['year_end'] ) {
+						} elseif ($y == $dates['year_end']) {
 							$month_start = 1;
 							$month_end   = $dates['m_end'];
 						} else {
@@ -1159,116 +1159,116 @@ 
 						}
 
 						$i = $month_start;
-						while ( $i <= $month_end ) :
+						while ($i <= $month_end) :
 
-							if ( $i == $dates['m_start'] ) {
+							if ($i == $dates['m_start']) {
 								$d = $dates['day_start'];
 							} else {
 								$d = 1;
 							}
 
-							if ( $i == $dates['m_end'] ) {
+							if ($i == $dates['m_end']) {
 								$num_of_days = $dates['day_end'];
 							} else {
-								$num_of_days = cal_days_in_month( CAL_GREGORIAN, $i, $y );
+								$num_of_days = cal_days_in_month(CAL_GREGORIAN, $i, $y);
 							}
 
-							while ( $d <= $num_of_days ) :
-								$sale_count = give_get_sales_by_date( $d, $i, $y );
-								$date_key   = date( 'Ymd', strtotime( $y . '/' . $i . '/' . $d ) );
-								if ( ! isset( $donations['sales'][ $date_key ] ) ) {
-									$donations['sales'][ $date_key ] = 0;
+							while ($d <= $num_of_days) :
+								$sale_count = give_get_sales_by_date($d, $i, $y);
+								$date_key   = date('Ymd', strtotime($y.'/'.$i.'/'.$d));
+								if ( ! isset($donations['sales'][$date_key])) {
+									$donations['sales'][$date_key] = 0;
 								}
-								$donations['sales'][ $date_key ] += $sale_count;
+								$donations['sales'][$date_key] += $sale_count;
 								$total                           += $sale_count;
-								$d ++;
+								$d++;
 							endwhile;
-							$i ++;
+							$i++;
 						endwhile;
 
-						$y ++;
+						$y++;
 					endwhile;
 
 					$donations['totals'] = $total;
 				} else {
-					if ( $args['date'] == 'this_quarter' || $args['date'] == 'last_quarter' ) {
+					if ($args['date'] == 'this_quarter' || $args['date'] == 'last_quarter') {
 						$donations_count = 0;
 
 						// Loop through the months
 						$month = $dates['m_start'];
 
-						while ( $month <= $dates['m_end'] ) :
-							$donations_count += give_get_sales_by_date( null, $month, $dates['year'] );
-							$month ++;
+						while ($month <= $dates['m_end']) :
+							$donations_count += give_get_sales_by_date(null, $month, $dates['year']);
+							$month++;
 						endwhile;
 
-						$donations['donations'][ $args['date'] ] = $donations_count;
+						$donations['donations'][$args['date']] = $donations_count;
 					} else {
-						$donations['donations'][ $args['date'] ] = give_get_sales_by_date( $dates['day'], $dates['m_start'], $dates['year'] );
+						$donations['donations'][$args['date']] = give_get_sales_by_date($dates['day'], $dates['m_start'], $dates['year']);
 					}
 				}// End if().
-			} elseif ( $args['form'] == 'all' ) {
-				$forms = get_posts( array(
+			} elseif ($args['form'] == 'all') {
+				$forms = get_posts(array(
 					'post_type' => 'give_forms',
 					'nopaging'  => true,
-				) );
-				$i     = 0;
-				foreach ( $forms as $form_info ) {
-					$donations['donations'][ $i ] = array(
-						$form_info->post_name => give_get_form_sales_stats( $form_info->ID ),
+				));
+				$i = 0;
+				foreach ($forms as $form_info) {
+					$donations['donations'][$i] = array(
+						$form_info->post_name => give_get_form_sales_stats($form_info->ID),
 					);
-					$i ++;
+					$i++;
 				}
 			} else {
-				if ( get_post_type( $args['form'] ) == 'give_forms' ) {
-					$form_info                 = get_post( $args['form'] );
+				if (get_post_type($args['form']) == 'give_forms') {
+					$form_info                 = get_post($args['form']);
 					$donations['donations'][0] = array(
-						$form_info->post_name => give_get_form_sales_stats( $args['form'] ),
+						$form_info->post_name => give_get_form_sales_stats($args['form']),
 					);
 				} else {
 					$error['error'] = sprintf( /* translators: %s: form */
-						__( 'Form %s not found.', 'give' ), $args['form'] );
+						__('Form %s not found.', 'give'), $args['form'] );
 				}
 			}// End if().
 
-			if ( ! empty( $error ) ) {
+			if ( ! empty($error)) {
 				return $error;
 			}
 
 			return $donations;
 
-		} elseif ( $args['type'] == 'earnings' ) {
-			if ( $args['form'] == null ) {
-				if ( $args['date'] == null ) {
+		} elseif ($args['type'] == 'earnings') {
+			if ($args['form'] == null) {
+				if ($args['date'] == null) {
 					$earnings = $this->get_default_earnings_stats();
-				} elseif ( $args['date'] === 'range' ) {
+				} elseif ($args['date'] === 'range') {
 					// Return sales for a date range
 					// Ensure the end date is later than the start date
-					if ( $args['enddate'] < $args['startdate'] ) {
-						$error['error'] = __( 'The end date must be later than the start date.', 'give' );
+					if ($args['enddate'] < $args['startdate']) {
+						$error['error'] = __('The end date must be later than the start date.', 'give');
 					}
 
 					// Ensure both the start and end date are specified
-					if ( empty( $args['startdate'] ) || empty( $args['enddate'] ) ) {
-						$error['error'] = __( 'Invalid or no date range specified.', 'give' );
+					if (empty($args['startdate']) || empty($args['enddate'])) {
+						$error['error'] = __('Invalid or no date range specified.', 'give');
 					}
 
 					$total = (float) 0.00;
 
 					// Loop through the years
 					$y = $dates['year'];
-					if ( ! isset( $earnings['earnings'] ) ) {
+					if ( ! isset($earnings['earnings'])) {
 						$earnings['earnings'] = array();
 					}
-					while ( $y <= $dates['year_end'] ) :
+					while ($y <= $dates['year_end']) :
 
-						if ( $dates['year'] == $dates['year_end'] ) {
+						if ($dates['year'] == $dates['year_end']) {
 							$month_start = $dates['m_start'];
 							$month_end   = $dates['m_end'];
-						} elseif ( $y == $dates['year'] && $dates['year_end'] > $dates['year'] ) {
+						} elseif ($y == $dates['year'] && $dates['year_end'] > $dates['year']) {
 							$month_start = $dates['m_start'];
 							$month_end   = 12;
-						} elseif ( $y == $dates['year_end'] ) {
+						} elseif ($y == $dates['year_end']) {
 							$month_start = 1;
 							$month_end   = $dates['m_end'];
 						} else {
@@ -1277,94 +1277,94 @@ 
 						}
 
 						$i = $month_start;
-						while ( $i <= $month_end ) :
+						while ($i <= $month_end) :
 
-							if ( $i == $dates['m_start'] ) {
+							if ($i == $dates['m_start']) {
 								$d = $dates['day_start'];
 							} else {
 								$d = 1;
 							}
 
-							if ( $i == $dates['m_end'] ) {
+							if ($i == $dates['m_end']) {
 								$num_of_days = $dates['day_end'];
 							} else {
-								$num_of_days = cal_days_in_month( CAL_GREGORIAN, $i, $y );
+								$num_of_days = cal_days_in_month(CAL_GREGORIAN, $i, $y);
 							}
 
-							while ( $d <= $num_of_days ) :
-								$earnings_stat = give_get_earnings_by_date( $d, $i, $y );
-								$date_key      = date( 'Ymd', strtotime( $y . '/' . $i . '/' . $d ) );
-								if ( ! isset( $earnings['earnings'][ $date_key ] ) ) {
-									$earnings['earnings'][ $date_key ] = 0;
+							while ($d <= $num_of_days) :
+								$earnings_stat = give_get_earnings_by_date($d, $i, $y);
+								$date_key      = date('Ymd', strtotime($y.'/'.$i.'/'.$d));
+								if ( ! isset($earnings['earnings'][$date_key])) {
+									$earnings['earnings'][$date_key] = 0;
 								}
-								$earnings['earnings'][ $date_key ] += $earnings_stat;
+								$earnings['earnings'][$date_key] += $earnings_stat;
 								$total                             += $earnings_stat;
-								$d ++;
+								$d++;
 							endwhile;
 
-							$i ++;
+							$i++;
 						endwhile;
 
-						$y ++;
+						$y++;
 					endwhile;
 
 					$earnings['totals'] = $total;
 				} else {
-					if ( $args['date'] == 'this_quarter' || $args['date'] == 'last_quarter' ) {
+					if ($args['date'] == 'this_quarter' || $args['date'] == 'last_quarter') {
 						$earnings_count = (float) 0.00;
 
 						// Loop through the months
 						$month = $dates['m_start'];
 
-						while ( $month <= $dates['m_end'] ) :
-							$earnings_count += give_get_earnings_by_date( null, $month, $dates['year'] );
-							$month ++;
+						while ($month <= $dates['m_end']) :
+							$earnings_count += give_get_earnings_by_date(null, $month, $dates['year']);
+							$month++;
 						endwhile;
 
-						$earnings['earnings'][ $args['date'] ] = $earnings_count;
+						$earnings['earnings'][$args['date']] = $earnings_count;
 					} else {
-						$earnings['earnings'][ $args['date'] ] = give_get_earnings_by_date( $dates['day'], $dates['m_start'], $dates['year'] );
+						$earnings['earnings'][$args['date']] = give_get_earnings_by_date($dates['day'], $dates['m_start'], $dates['year']);
 					}
 				}// End if().
-			} elseif ( $args['form'] == 'all' ) {
-				$forms = get_posts( array(
+			} elseif ($args['form'] == 'all') {
+				$forms = get_posts(array(
 					'post_type' => 'give_forms',
 					'nopaging'  => true,
-				) );
+				));
 
 				$i = 0;
-				foreach ( $forms as $form_info ) {
-					$earnings['earnings'][ $i ] = array(
-						$form_info->post_name => give_get_form_earnings_stats( $form_info->ID ),
+				foreach ($forms as $form_info) {
+					$earnings['earnings'][$i] = array(
+						$form_info->post_name => give_get_form_earnings_stats($form_info->ID),
 					);
-					$i ++;
+					$i++;
 				}
 			} else {
-				if ( get_post_type( $args['form'] ) == 'give_forms' ) {
-					$form_info               = get_post( $args['form'] );
+				if (get_post_type($args['form']) == 'give_forms') {
+					$form_info               = get_post($args['form']);
 					$earnings['earnings'][0] = array(
-						$form_info->post_name => give_get_form_earnings_stats( $args['form'] ),
+						$form_info->post_name => give_get_form_earnings_stats($args['form']),
 					);
 				} else {
 					$error['error'] = sprintf( /* translators: %s: form */
-						__( 'Form %s not found.', 'give' ), $args['form'] );
+						__('Form %s not found.', 'give'), $args['form'] );
 				}
 			}// End if().
 
-			if ( ! empty( $error ) ) {
+			if ( ! empty($error)) {
 				return $error;
 			}
 
 			return $earnings;
-		} elseif ( $args['type'] == 'donors' ) {
+		} elseif ($args['type'] == 'donors') {
 			$donors                             = new Give_DB_Donors();
 			$stats['donations']['total_donors'] = $donors->count();
 
 			return $stats;
 
-		} elseif ( empty( $args['type'] ) ) {
-			$stats = array_merge( $stats, $this->get_default_sales_stats() );
-			$stats = array_merge( $stats, $this->get_default_earnings_stats() );
+		} elseif (empty($args['type'])) {
+			$stats = array_merge($stats, $this->get_default_sales_stats());
+			$stats = array_merge($stats, $this->get_default_earnings_stats());
 
 			return array(
 				'stats' => $stats,
@@ -1382,7 +1382,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_recent_donations( $args = array() ) {
+	public function get_recent_donations($args = array()) {
 		global $wp_query;
 
 		$defaults = array(
@@ -1392,33 +1392,33 @@ 
 			'enddate'   => null,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$donations = array();
 
-		if ( ! user_can( $this->user_id, 'view_give_reports' ) && ! $this->override ) {
+		if ( ! user_can($this->user_id, 'view_give_reports') && ! $this->override) {
 			return $donations;
 		}
 
-		if ( isset( $wp_query->query_vars['id'] ) ) {
+		if (isset($wp_query->query_vars['id'])) {
 			$query   = array();
-			$query[] = new Give_Payment( $wp_query->query_vars['id'] );
-		} elseif ( isset( $wp_query->query_vars['purchasekey'] ) ) {
+			$query[] = new Give_Payment($wp_query->query_vars['id']);
+		} elseif (isset($wp_query->query_vars['purchasekey'])) {
 			$query   = array();
-			$query[] = give_get_payment_by( 'key', $wp_query->query_vars['purchasekey'] );
-		} elseif ( isset( $wp_query->query_vars['email'] ) ) {
-			$args  = array(
+			$query[] = give_get_payment_by('key', $wp_query->query_vars['purchasekey']);
+		} elseif (isset($wp_query->query_vars['email'])) {
+			$args = array(
 				'fields'     => 'ids',
 				'meta_key'   => '_give_payment_user_email',
 				'meta_value' => $wp_query->query_vars['email'],
 				'number'     => $this->per_page(),
 				'page'       => $this->get_paged(),
 			);
-			$query = give_get_payments( $args );
-		} elseif ( isset( $wp_query->query_vars['date'] ) ) {
+			$query = give_get_payments($args);
+		} elseif (isset($wp_query->query_vars['date'])) {
 
-			$current_time = current_time( 'timestamp' );
-			$dates        = $this->get_dates( $args );
+			$current_time = current_time('timestamp');
+			$dates        = $this->get_dates($args);
 			$start_date   = '';
 			$end_date     = '';
 
@@ -1431,27 +1431,27 @@ 
 			 * @params date startdate | required when date = range and format to be YYYYMMDD (i.e. 20170524)
 			 * @params date enddate | required when date = range and format to be YYYYMMDD (i.e. 20170524)
 			 */
-			switch ( $wp_query->query_vars['date'] ) {
+			switch ($wp_query->query_vars['date']) {
 
 				case 'today':
 
 					// Set and Format Start and End Date to be date of today.
-					$start_date = $end_date = date( 'Y/m/d', $current_time );
+					$start_date = $end_date = date('Y/m/d', $current_time);
 
 					break;
 
 				case 'yesterday':
 
 					// Set and Format Start and End Date to be date of yesterday.
-					$start_date = $end_date = date( 'Y/m', $current_time ) . '/' . ( date( 'd', $current_time ) - 1 );
+					$start_date = $end_date = date('Y/m', $current_time).'/'.(date('d', $current_time) - 1);
 
 					break;
 
 				case 'range':
 
 					// Format Start Date and End Date for filtering payment based on date range.
-					$start_date = $dates['year'] . '/' . $dates['m_start'] . '/' . $dates['day_start'];
-					$end_date   = $dates['year_end'] . '/' . $dates['m_end'] . '/' . $dates['day_end'];
+					$start_date = $dates['year'].'/'.$dates['m_start'].'/'.$dates['day_start'];
+					$end_date   = $dates['year_end'].'/'.$dates['m_end'].'/'.$dates['day_end'];
 
 					break;
 
@@ -1465,22 +1465,22 @@ 
 				'page'       => $this->get_paged(),
 			);
 
-			$query = give_get_payments( $args );
+			$query = give_get_payments($args);
 		} else {
 			$args  = array(
 				'fields' => 'ids',
 				'number' => $this->per_page(),
 				'page'   => $this->get_paged(),
 			);
-			$query = give_get_payments( $args );
+			$query = give_get_payments($args);
 		}// End if().
 
-		if ( $query ) {
+		if ($query) {
 			$i = 0;
-			foreach ( $query as $payment ) {
+			foreach ($query as $payment) {
 
-				if ( is_numeric( $payment ) ) {
-					$payment      = new Give_Payment( $payment );
+				if (is_numeric($payment)) {
+					$payment      = new Give_Payment($payment);
 					$payment_meta = $payment->get_meta();
 					$user_info    = $payment->user_info;
 				}
@@ -1488,41 +1488,41 @@ 
 				$payment_meta = $payment->get_meta();
 				$user_info    = $payment->user_info;
 
-				$first_name = isset( $user_info['first_name'] ) ? $user_info['first_name'] : '';
-				$last_name  = isset( $user_info['last_name'] ) ? $user_info['last_name'] : '';
-
-				$donations['donations'][ $i ]['ID']             = $payment->number;
-				$donations['donations'][ $i ]['transaction_id'] = $payment->transaction_id;
-				$donations['donations'][ $i ]['key']            = $payment->key;
-				$donations['donations'][ $i ]['total']          = $payment->total;
-				$donations['donations'][ $i ]['status']         = give_get_payment_status( $payment, true );
-				$donations['donations'][ $i ]['gateway']        = $payment->gateway;
-				$donations['donations'][ $i ]['name']           = $first_name . ' ' . $last_name;
-				$donations['donations'][ $i ]['fname']          = $first_name;
-				$donations['donations'][ $i ]['lname']          = $last_name;
-				$donations['donations'][ $i ]['email']          = $payment->email;
-				$donations['donations'][ $i ]['date']           = $payment->date;
-
-				$form_id  = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : $payment_meta;
-				$price    = isset( $payment_meta['form_id'] ) ? give_get_form_price( $payment_meta['form_id'] ) : false;
-				$price_id = isset( $payment_meta['price_id'] ) ? $payment_meta['price_id'] : null;
-
-				$donations['donations'][ $i ]['form']['id']    = $form_id;
-				$donations['donations'][ $i ]['form']['name']  = get_the_title( $payment_meta['form_id'] );
-				$donations['donations'][ $i ]['form']['price'] = $price;
-
-				if ( give_has_variable_prices( $form_id ) ) {
-					if ( isset( $payment_meta['price_id'] ) ) {
-						$price_name                                         = give_get_price_option_name( $form_id, $payment_meta['price_id'], $payment->ID );
-						$donations['donations'][ $i ]['form']['price_name'] = $price_name;
-						$donations['donations'][ $i ]['form']['price_id']   = $price_id;
-						$donations['donations'][ $i ]['form']['price']      = give_get_price_option_amount( $form_id, $price_id );
+				$first_name = isset($user_info['first_name']) ? $user_info['first_name'] : '';
+				$last_name  = isset($user_info['last_name']) ? $user_info['last_name'] : '';
+
+				$donations['donations'][$i]['ID']             = $payment->number;
+				$donations['donations'][$i]['transaction_id'] = $payment->transaction_id;
+				$donations['donations'][$i]['key']            = $payment->key;
+				$donations['donations'][$i]['total']          = $payment->total;
+				$donations['donations'][$i]['status']         = give_get_payment_status($payment, true);
+				$donations['donations'][$i]['gateway']        = $payment->gateway;
+				$donations['donations'][$i]['name']           = $first_name.' '.$last_name;
+				$donations['donations'][$i]['fname']          = $first_name;
+				$donations['donations'][$i]['lname']          = $last_name;
+				$donations['donations'][$i]['email']          = $payment->email;
+				$donations['donations'][$i]['date']           = $payment->date;
+
+				$form_id  = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : $payment_meta;
+				$price    = isset($payment_meta['form_id']) ? give_get_form_price($payment_meta['form_id']) : false;
+				$price_id = isset($payment_meta['price_id']) ? $payment_meta['price_id'] : null;
+
+				$donations['donations'][$i]['form']['id']    = $form_id;
+				$donations['donations'][$i]['form']['name']  = get_the_title($payment_meta['form_id']);
+				$donations['donations'][$i]['form']['price'] = $price;
+
+				if (give_has_variable_prices($form_id)) {
+					if (isset($payment_meta['price_id'])) {
+						$price_name                                         = give_get_price_option_name($form_id, $payment_meta['price_id'], $payment->ID);
+						$donations['donations'][$i]['form']['price_name'] = $price_name;
+						$donations['donations'][$i]['form']['price_id']   = $price_id;
+						$donations['donations'][$i]['form']['price']      = give_get_price_option_amount($form_id, $price_id);
 
 					}
 				}
 
 				// Add custom meta to API
-				foreach ( $payment_meta as $meta_key => $meta_value ) {
+				foreach ($payment_meta as $meta_key => $meta_value) {
 
 					$exceptions = array(
 						'form_title',
@@ -1535,19 +1535,19 @@ 
 					);
 
 					// Don't clutter up results with dupes
-					if ( in_array( $meta_key, $exceptions ) ) {
+					if (in_array($meta_key, $exceptions)) {
 						continue;
 					}
 
-					$donations['donations'][ $i ]['payment_meta'][ $meta_key ] = $meta_value;
+					$donations['donations'][$i]['payment_meta'][$meta_key] = $meta_value;
 
 				}
 
-				$i ++;
+				$i++;
 			}// End foreach().
 		}// End if().
 
-		return apply_filters( 'give_api_donations_endpoint', $donations );
+		return apply_filters('give_api_donations_endpoint', $donations);
 	}
 
 	/**
@@ -1563,9 +1563,9 @@ 
 	public function get_output_format() {
 		global $wp_query;
 
-		$format = isset( $wp_query->query_vars['format'] ) ? $wp_query->query_vars['format'] : 'json';
+		$format = isset($wp_query->query_vars['format']) ? $wp_query->query_vars['format'] : 'json';
 
-		return apply_filters( 'give_api_output_format', $format );
+		return apply_filters('give_api_output_format', $format);
 	}
 
 
@@ -1582,8 +1582,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function log_request( $data = array() ) {
-		if ( ! $this->log_requests ) {
+	private function log_request($data = array()) {
+		if ( ! $this->log_requests) {
 			return;
 		}
 
@@ -1599,36 +1599,36 @@ 
 
 		$query = array(
 			'give-api'    => $wp_query->query_vars['give-api'],
-			'key'         => isset( $wp_query->query_vars['key'] ) ? $wp_query->query_vars['key'] : null,
-			'token'       => isset( $wp_query->query_vars['token'] ) ? $wp_query->query_vars['token'] : null,
-			'query'       => isset( $wp_query->query_vars['query'] ) ? $wp_query->query_vars['query'] : null,
-			'type'        => isset( $wp_query->query_vars['type'] ) ? $wp_query->query_vars['type'] : null,
-			'form'        => isset( $wp_query->query_vars['form'] ) ? $wp_query->query_vars['form'] : null,
-			'donor'       => isset( $wp_query->query_vars['donor'] ) ? $wp_query->query_vars['donor'] : null,
-			'date'        => isset( $wp_query->query_vars['date'] ) ? $wp_query->query_vars['date'] : null,
-			'startdate'   => isset( $wp_query->query_vars['startdate'] ) ? $wp_query->query_vars['startdate'] : null,
-			'enddate'     => isset( $wp_query->query_vars['enddate'] ) ? $wp_query->query_vars['enddate'] : null,
-			'id'          => isset( $wp_query->query_vars['id'] ) ? $wp_query->query_vars['id'] : null,
-			'purchasekey' => isset( $wp_query->query_vars['purchasekey'] ) ? $wp_query->query_vars['purchasekey'] : null,
-			'email'       => isset( $wp_query->query_vars['email'] ) ? $wp_query->query_vars['email'] : null,
+			'key'         => isset($wp_query->query_vars['key']) ? $wp_query->query_vars['key'] : null,
+			'token'       => isset($wp_query->query_vars['token']) ? $wp_query->query_vars['token'] : null,
+			'query'       => isset($wp_query->query_vars['query']) ? $wp_query->query_vars['query'] : null,
+			'type'        => isset($wp_query->query_vars['type']) ? $wp_query->query_vars['type'] : null,
+			'form'        => isset($wp_query->query_vars['form']) ? $wp_query->query_vars['form'] : null,
+			'donor'       => isset($wp_query->query_vars['donor']) ? $wp_query->query_vars['donor'] : null,
+			'date'        => isset($wp_query->query_vars['date']) ? $wp_query->query_vars['date'] : null,
+			'startdate'   => isset($wp_query->query_vars['startdate']) ? $wp_query->query_vars['startdate'] : null,
+			'enddate'     => isset($wp_query->query_vars['enddate']) ? $wp_query->query_vars['enddate'] : null,
+			'id'          => isset($wp_query->query_vars['id']) ? $wp_query->query_vars['id'] : null,
+			'purchasekey' => isset($wp_query->query_vars['purchasekey']) ? $wp_query->query_vars['purchasekey'] : null,
+			'email'       => isset($wp_query->query_vars['email']) ? $wp_query->query_vars['email'] : null,
 		);
 
 		$log_data = array(
 			'log_type'     => 'api_request',
-			'post_excerpt' => http_build_query( $query ),
-			'post_content' => ! empty( $data['error'] ) ? $data['error'] : '',
+			'post_excerpt' => http_build_query($query),
+			'post_content' => ! empty($data['error']) ? $data['error'] : '',
 		);
 
 		$log_meta = array(
 			'request_ip' => give_get_ip(),
 			'user'       => $this->user_id,
-			'key'        => isset( $wp_query->query_vars['key'] ) ? $wp_query->query_vars['key'] : null,
-			'token'      => isset( $wp_query->query_vars['token'] ) ? $wp_query->query_vars['token'] : null,
+			'key'        => isset($wp_query->query_vars['key']) ? $wp_query->query_vars['key'] : null,
+			'token'      => isset($wp_query->query_vars['token']) ? $wp_query->query_vars['token'] : null,
 			'time'       => $data['request_speed'],
 			'version'    => $this->get_queried_version(),
 		);
 
-		$give_logs->insert_log( $log_data, $log_meta );
+		$give_logs->insert_log($log_data, $log_meta);
 	}
 
 
@@ -1652,11 +1652,11 @@ 
 	 *
 	 * @param int       $status_code
 	 */
-	public function output( $status_code = 200 ) {
+	public function output($status_code = 200) {
 
 		$format = $this->get_output_format();
 
-		status_header( $status_code );
+		status_header($status_code);
 
 		/**
 		 * Fires before outputting the API.
@@ -1667,25 +1667,25 @@ 
 		 * @param Give_API $this   The Give_API object.
 		 * @param string   $format Output format, XML or JSON. Default is JSON.
 		 */
-		do_action( 'give_api_output_before', $this->data, $this, $format );
+		do_action('give_api_output_before', $this->data, $this, $format);
 
-		switch ( $format ) :
+		switch ($format) :
 
 			case 'xml' :
 
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/array2xml.php';
-				$xml = Array2XML::createXML( 'give', $this->data );
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/array2xml.php';
+				$xml = Array2XML::createXML('give', $this->data);
 				echo $xml->saveXML();
 
 				break;
 
 			case 'json' :
 
-				header( 'Content-Type: application/json' );
-				if ( ! empty( $this->pretty_print ) ) {
-					echo json_encode( $this->data, $this->pretty_print );
+				header('Content-Type: application/json');
+				if ( ! empty($this->pretty_print)) {
+					echo json_encode($this->data, $this->pretty_print);
 				} else {
-					echo json_encode( $this->data );
+					echo json_encode($this->data);
 				}
 
 				break;
@@ -1700,7 +1700,7 @@ 
 				 * @param array    $data Response data to return.
 				 * @param Give_API $this The Give_API object.
 				 */
-				do_action( "give_api_output_{$format}", $this->data, $this );
+				do_action("give_api_output_{$format}", $this->data, $this);
 
 				break;
 
@@ -1715,7 +1715,7 @@ 
 		 * @param Give_API $this   The Give_API object.
 		 * @param string   $format Output format, XML or JSON. Default is JSON.
 		 */
-		do_action( 'give_api_output_after', $this->data, $this, $format );
+		do_action('give_api_output_after', $this->data, $this, $format);
 
 		give_die();
 	}
@@ -1732,41 +1732,41 @@ 
 	 *
 	 * @return void
 	 */
-	function user_key_field( $user ) {
+	function user_key_field($user) {
 
-		if ( ( give_get_option( 'api_allow_user_keys', false ) || current_user_can( 'manage_give_settings' ) ) && current_user_can( 'edit_user', $user->ID ) ) {
+		if ((give_get_option('api_allow_user_keys', false) || current_user_can('manage_give_settings')) && current_user_can('edit_user', $user->ID)) {
 
-			$user = get_userdata( $user->ID );
+			$user = get_userdata($user->ID);
 			?>
 			<table class="form-table">
 				<tbody>
 				<tr>
 					<th>
-						<?php _e( 'Give API Keys', 'give' ); ?>
+						<?php _e('Give API Keys', 'give'); ?>
 					</th>
 					<td>
 						<?php
-						$public_key = $this->get_user_public_key( $user->ID );
-						$secret_key = $this->get_user_secret_key( $user->ID );
+						$public_key = $this->get_user_public_key($user->ID);
+						$secret_key = $this->get_user_secret_key($user->ID);
 						?>
-						<?php if ( empty( $user->give_user_public_key ) ) { ?>
+						<?php if (empty($user->give_user_public_key)) { ?>
 							<input name="give_set_api_key" type="checkbox" id="give_set_api_key" />
-							<span class="description"><?php _e( 'Generate API Key', 'give' ); ?></span>
+							<span class="description"><?php _e('Generate API Key', 'give'); ?></span>
 						<?php } else { ?>
-							<strong style="display:inline-block; width: 125px;"><?php _e( 'Public key:', 'give' ); ?>
+							<strong style="display:inline-block; width: 125px;"><?php _e('Public key:', 'give'); ?>
 								&nbsp;</strong>
-							<input type="text" disabled="disabled" class="regular-text" id="publickey" value="<?php echo esc_attr( $public_key ); ?>" />
+							<input type="text" disabled="disabled" class="regular-text" id="publickey" value="<?php echo esc_attr($public_key); ?>" />
 							<br />
-							<strong style="display:inline-block; width: 125px;"><?php _e( 'Secret key:', 'give' ); ?>
+							<strong style="display:inline-block; width: 125px;"><?php _e('Secret key:', 'give'); ?>
 								&nbsp;</strong>
-							<input type="text" disabled="disabled" class="regular-text" id="privatekey" value="<?php echo esc_attr( $secret_key ); ?>" />
+							<input type="text" disabled="disabled" class="regular-text" id="privatekey" value="<?php echo esc_attr($secret_key); ?>" />
 							<br />
-							<strong style="display:inline-block; width: 125px;"><?php _e( 'Token:', 'give' ); ?>
+							<strong style="display:inline-block; width: 125px;"><?php _e('Token:', 'give'); ?>
 								&nbsp;</strong>
-							<input type="text" disabled="disabled" class="regular-text" id="token" value="<?php echo esc_attr( $this->get_token( $user->ID ) ); ?>" />
+							<input type="text" disabled="disabled" class="regular-text" id="token" value="<?php echo esc_attr($this->get_token($user->ID)); ?>" />
 							<br />
 							<input name="give_revoke_api_key" type="checkbox" id="give_revoke_api_key" />
-							<span class="description"><label for="give_revoke_api_key"><?php _e( 'Revoke API Keys', 'give' ); ?></label></span>
+							<span class="description"><label for="give_revoke_api_key"><?php _e('Revoke API Keys', 'give'); ?></label></span>
 						<?php } ?>
 					</td>
 				</tr>
@@ -1785,61 +1785,61 @@ 
 	 *
 	 * @return void
 	 */
-	public function process_api_key( $args ) {
+	public function process_api_key($args) {
 
-		if ( ! wp_verify_nonce( $_REQUEST['_wpnonce'], 'give-api-nonce' ) ) {
-			wp_die( __( 'Nonce verification failed.', 'give' ), __( 'Error', 'give' ), array(
+		if ( ! wp_verify_nonce($_REQUEST['_wpnonce'], 'give-api-nonce')) {
+			wp_die(__('Nonce verification failed.', 'give'), __('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
-		if ( empty( $args['user_id'] ) ) {
-			wp_die( __( 'User ID Required.', 'give' ), __( 'Error', 'give' ), array(
+		if (empty($args['user_id'])) {
+			wp_die(__('User ID Required.', 'give'), __('Error', 'give'), array(
 				'response' => 401,
-			) );
+			));
 		}
 
-		if ( is_numeric( $args['user_id'] ) ) {
-			$user_id = isset( $args['user_id'] ) ? absint( $args['user_id'] ) : get_current_user_id();
+		if (is_numeric($args['user_id'])) {
+			$user_id = isset($args['user_id']) ? absint($args['user_id']) : get_current_user_id();
 		} else {
-			$userdata = get_user_by( 'login', $args['user_id'] );
+			$userdata = get_user_by('login', $args['user_id']);
 			$user_id  = $userdata->ID;
 		}
-		$process = isset( $args['give_api_process'] ) ? strtolower( $args['give_api_process'] ) : false;
+		$process = isset($args['give_api_process']) ? strtolower($args['give_api_process']) : false;
 
-		if ( $user_id == get_current_user_id() && ! give_get_option( 'allow_user_api_keys' ) && ! current_user_can( 'manage_give_settings' ) ) {
-			wp_die( sprintf( /* translators: %s: process */
-				__( 'You do not have permission to %s API keys for this user.', 'give' ), $process ), __( 'Error', 'give' ), array(
+		if ($user_id == get_current_user_id() && ! give_get_option('allow_user_api_keys') && ! current_user_can('manage_give_settings')) {
+			wp_die(sprintf( /* translators: %s: process */
+				__('You do not have permission to %s API keys for this user.', 'give'), $process ), __('Error', 'give'), array(
 				'response' => 403,
-			) );
-		} elseif ( ! current_user_can( 'manage_give_settings' ) ) {
-			wp_die( sprintf( /* translators: %s: process */
-				__( 'You do not have permission to %s API keys for this user.', 'give' ), $process ), __( 'Error', 'give' ), array(
+			));
+		} elseif ( ! current_user_can('manage_give_settings')) {
+			wp_die(sprintf( /* translators: %s: process */
+				__('You do not have permission to %s API keys for this user.', 'give'), $process ), __('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
-		switch ( $process ) {
+		switch ($process) {
 			case 'generate':
-				if ( $this->generate_api_key( $user_id ) ) {
-					Give_Cache::delete( Give_Cache::get_key( 'give_total_api_keys' ) );
-					wp_redirect( add_query_arg( 'give-message', 'api-key-generated', 'edit.php?post_type=give_forms&page=give-tools&tab=api' ) );
+				if ($this->generate_api_key($user_id)) {
+					Give_Cache::delete(Give_Cache::get_key('give_total_api_keys'));
+					wp_redirect(add_query_arg('give-message', 'api-key-generated', 'edit.php?post_type=give_forms&page=give-tools&tab=api'));
 					exit();
 				} else {
-					wp_redirect( add_query_arg( 'give-message', 'api-key-failed', 'edit.php?post_type=give_forms&page=give-tools&tab=api' ) );
+					wp_redirect(add_query_arg('give-message', 'api-key-failed', 'edit.php?post_type=give_forms&page=give-tools&tab=api'));
 					exit();
 				}
 				break;
 			case 'regenerate':
-				$this->generate_api_key( $user_id, true );
-				Give_Cache::delete( Give_Cache::get_key( 'give_total_api_keys' ) );
-				wp_redirect( add_query_arg( 'give-message', 'api-key-regenerated', 'edit.php?post_type=give_forms&page=give-tools&tab=api' ) );
+				$this->generate_api_key($user_id, true);
+				Give_Cache::delete(Give_Cache::get_key('give_total_api_keys'));
+				wp_redirect(add_query_arg('give-message', 'api-key-regenerated', 'edit.php?post_type=give_forms&page=give-tools&tab=api'));
 				exit();
 				break;
 			case 'revoke':
-				$this->revoke_api_key( $user_id );
-				Give_Cache::delete( Give_Cache::get_key( 'give_total_api_keys' ) );
-				wp_redirect( add_query_arg( 'give-message', 'api-key-revoked', 'edit.php?post_type=give_forms&page=give-tools&tab=api' ) );
+				$this->revoke_api_key($user_id);
+				Give_Cache::delete(Give_Cache::get_key('give_total_api_keys'));
+				wp_redirect(add_query_arg('give-message', 'api-key-revoked', 'edit.php?post_type=give_forms&page=give-tools&tab=api'));
 				exit();
 				break;
 			default;
@@ -1858,59 +1858,59 @@ 
 	 *
 	 * @return boolean True if (re)generated successfully, false otherwise.
 	 */
-	public function generate_api_key( $user_id = 0, $regenerate = false ) {
+	public function generate_api_key($user_id = 0, $regenerate = false) {
 
 		// Bail out, if user doesn't exists.
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return false;
 		}
 
-		$user = get_userdata( $user_id );
+		$user = get_userdata($user_id);
 
 		// Bail Out, if user object doesn't exists.
-		if ( ! $user ) {
+		if ( ! $user) {
 			return false;
 		}
 
 		$new_public_key = '';
 		$new_secret_key = '';
 
-		if( ! empty( $_POST['from'] ) && 'profile' === $_POST['from'] ) {
+		if ( ! empty($_POST['from']) && 'profile' === $_POST['from']) {
 			// For User Profile Page.
-			if( ! empty( $_POST['give_set_api_key'] ) ) {
+			if ( ! empty($_POST['give_set_api_key'])) {
 				// Generate API Key from User Profile page.
-				$new_public_key = $this->generate_public_key( $user->user_email );
-				$new_secret_key = $this->generate_private_key( $user->ID );
-			} elseif ( ! empty( $_POST['give_revoke_api_key'] ) ) {
+				$new_public_key = $this->generate_public_key($user->user_email);
+				$new_secret_key = $this->generate_private_key($user->ID);
+			} elseif ( ! empty($_POST['give_revoke_api_key'])) {
 				// Revoke API Key from User Profile page.
-				$this->revoke_api_key( $user->ID );
+				$this->revoke_api_key($user->ID);
 			} else {
 				return false;
 			}
 		} else {
 			// For Tools > API page.
-			$public_key = $this->get_user_public_key( $user_id );
+			$public_key = $this->get_user_public_key($user_id);
 
-			if ( empty( $public_key ) && ! $regenerate ) {
+			if (empty($public_key) && ! $regenerate) {
 				// Generating API for first time.
-				$new_public_key = $this->generate_public_key( $user->user_email );
-				$new_secret_key = $this->generate_private_key( $user->ID );
-			} elseif ( $public_key && $regenerate ) {
+				$new_public_key = $this->generate_public_key($user->user_email);
+				$new_secret_key = $this->generate_private_key($user->ID);
+			} elseif ($public_key && $regenerate) {
 				// API Key already exists and Regenerating API Key.
-				$this->revoke_api_key( $user->ID );
-				$new_public_key = $this->generate_public_key( $user->user_email );
-				$new_secret_key = $this->generate_private_key( $user->ID );
-			} elseif ( ! empty( $public_key ) && ! $regenerate ) {
+				$this->revoke_api_key($user->ID);
+				$new_public_key = $this->generate_public_key($user->user_email);
+				$new_secret_key = $this->generate_private_key($user->ID);
+			} elseif ( ! empty($public_key) && ! $regenerate) {
 				// Doing nothing, when API Key exists but still try to generate again instead of regenerating.
 				return false;
 			} else {
 				// Revoke API Key.
-				$this->revoke_api_key( $user->ID );
+				$this->revoke_api_key($user->ID);
 			}
 		}
 
-		update_user_meta( $user_id, $new_public_key, 'give_user_public_key' );
-		update_user_meta( $user_id, $new_secret_key, 'give_user_secret_key' );
+		update_user_meta($user_id, $new_public_key, 'give_user_public_key');
+		update_user_meta($user_id, $new_secret_key, 'give_user_secret_key');
 
 		return true;
 	}
@@ -1925,26 +1925,26 @@ 
 	 *
 	 * @return bool
 	 */
-	public function revoke_api_key( $user_id = 0 ) {
+	public function revoke_api_key($user_id = 0) {
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return false;
 		}
 
-		$user = get_userdata( $user_id );
+		$user = get_userdata($user_id);
 
-		if ( ! $user ) {
+		if ( ! $user) {
 			return false;
 		}
 
-		$public_key = $this->get_user_public_key( $user_id );
-		$secret_key = $this->get_user_secret_key( $user_id );
-		if ( ! empty( $public_key ) ) {
-			Give_Cache::delete( Give_Cache::get_key( md5( 'give_api_user_' . $public_key ) ) );
-			Give_Cache::delete( Give_Cache::get_key( md5( 'give_api_user_public_key' . $user_id ) ) );
-			Give_Cache::delete( Give_Cache::get_key( md5( 'give_api_user_secret_key' . $user_id ) ) );
-			delete_user_meta( $user_id, $public_key );
-			delete_user_meta( $user_id, $secret_key );
+		$public_key = $this->get_user_public_key($user_id);
+		$secret_key = $this->get_user_secret_key($user_id);
+		if ( ! empty($public_key)) {
+			Give_Cache::delete(Give_Cache::get_key(md5('give_api_user_'.$public_key)));
+			Give_Cache::delete(Give_Cache::get_key(md5('give_api_user_public_key'.$user_id)));
+			Give_Cache::delete(Give_Cache::get_key(md5('give_api_user_secret_key'.$user_id)));
+			delete_user_meta($user_id, $public_key);
+			delete_user_meta($user_id, $secret_key);
 		} else {
 			return false;
 		}
@@ -1966,9 +1966,9 @@ 
 	 *
 	 * @return string
 	 */
-	private function generate_public_key( $user_email = '' ) {
-		$auth_key = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-		$public   = hash( 'md5', $user_email . $auth_key . date( 'U' ) );
+	private function generate_public_key($user_email = '') {
+		$auth_key = defined('AUTH_KEY') ? AUTH_KEY : '';
+		$public   = hash('md5', $user_email.$auth_key.date('U'));
 
 		return $public;
 	}
@@ -1983,9 +1983,9 @@ 
 	 *
 	 * @return string
 	 */
-	private function generate_private_key( $user_id = 0 ) {
-		$auth_key = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-		$secret   = hash( 'md5', $user_id . $auth_key . date( 'U' ) );
+	private function generate_private_key($user_id = 0) {
+		$auth_key = defined('AUTH_KEY') ? AUTH_KEY : '';
+		$secret   = hash('md5', $user_id.$auth_key.date('U'));
 
 		return $secret;
 	}
@@ -2000,8 +2000,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_token( $user_id = 0 ) {
-		return hash( 'md5', $this->get_user_secret_key( $user_id ) . $this->get_user_public_key( $user_id ) );
+	public function get_token($user_id = 0) {
+		return hash('md5', $this->get_user_secret_key($user_id).$this->get_user_public_key($user_id));
 	}
 
 	/**
@@ -2015,9 +2015,9 @@ 
 
 		// Default sales return
 		$donations                               = array();
-		$donations['donations']['today']         = $this->stats->get_sales( 0, 'today' );
-		$donations['donations']['current_month'] = $this->stats->get_sales( 0, 'this_month' );
-		$donations['donations']['last_month']    = $this->stats->get_sales( 0, 'last_month' );
+		$donations['donations']['today']         = $this->stats->get_sales(0, 'today');
+		$donations['donations']['current_month'] = $this->stats->get_sales(0, 'this_month');
+		$donations['donations']['last_month']    = $this->stats->get_sales(0, 'last_month');
 		$donations['donations']['totals']        = give_get_total_donations();
 
 		return $donations;
@@ -2034,9 +2034,9 @@ 
 
 		// Default earnings return
 		$earnings                              = array();
-		$earnings['earnings']['today']         = $this->stats->get_earnings( 0, 'today' );
-		$earnings['earnings']['current_month'] = $this->stats->get_earnings( 0, 'this_month' );
-		$earnings['earnings']['last_month']    = $this->stats->get_earnings( 0, 'last_month' );
+		$earnings['earnings']['today']         = $this->stats->get_earnings(0, 'today');
+		$earnings['earnings']['current_month'] = $this->stats->get_earnings(0, 'this_month');
+		$earnings['earnings']['last_month']    = $this->stats->get_earnings(0, 'last_month');
 		$earnings['earnings']['totals']        = give_get_total_earnings();
 
 		return $earnings;
@@ -2056,25 +2056,25 @@ 
 	 *
 	 * @return string            The API key/secret for the user supplied
 	 */
-	public function api_key_backwards_compat( $check, $object_id, $meta_key, $single ) {
+	public function api_key_backwards_compat($check, $object_id, $meta_key, $single) {
 
-		if ( $meta_key !== 'give_user_public_key' && $meta_key !== 'give_user_secret_key' ) {
+		if ($meta_key !== 'give_user_public_key' && $meta_key !== 'give_user_secret_key') {
 			return $check;
 		}
 
 		$return = $check;
 
-		switch ( $meta_key ) {
+		switch ($meta_key) {
 			case 'give_user_public_key':
-				$return = Give()->api->get_user_public_key( $object_id );
+				$return = Give()->api->get_user_public_key($object_id);
 				break;
 			case 'give_user_secret_key':
-				$return = Give()->api->get_user_secret_key( $object_id );
+				$return = Give()->api->get_user_secret_key($object_id);
 				break;
 		}
 
-		if ( ! $single ) {
-			$return = array( $return );
+		if ( ! $single) {
+			$return = array($return);
 		}
 
 		return $return;

includes/class-give-db-donors.php

Doc Comments +3 added lines, -3 removed lines

@@ -161,7 +161,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param  bool|string|int $_id_or_email
+	 * @param  integer $_id_or_email
 	 *
 	 * @return bool|int
 	 */
@@ -316,7 +316,7 @@ 
 	 * @param  int          $user_id       User ID.
 	 * @param  WP_User|bool $old_user_data User data.
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function update_donor_email_on_user_update( $user_id = 0, $old_user_data = false ) {
 
@@ -373,7 +373,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $field ID or email. Default is 'id'.
-	 * @param  mixed  $value The Customer ID or email to search. Default is 0.
+	 * @param  integer  $value The Customer ID or email to search. Default is 0.
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */

Spacing +117 added lines, -117 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,14 +35,14 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_customers';
+		$this->table_name  = $wpdb->prefix.'give_customers';
 		$this->primary_key = 'id';
 		$this->version     = '1.0';
 
 		// Set hooks and register table only if instance loading first time.
-		if ( ! ( Give()->donors instanceof Give_DB_Donors ) ) {
+		if ( ! (Give()->donors instanceof Give_DB_Donors)) {
 			// Setup hook.
-			add_action( 'profile_update', array( $this, 'update_donor_email_on_user_update' ), 10, 2 );
+			add_action('profile_update', array($this, 'update_donor_email_on_user_update'), 10, 2);
 
 			// Install table.
 			$this->register_table();
@@ -89,7 +89,7 @@ 
 			'purchase_value' => 0.00,
 			'purchase_count' => 0,
 			'notes'          => '',
-			'date_created'   => date( 'Y-m-d H:i:s' ),
+			'date_created'   => date('Y-m-d H:i:s'),
 		);
 	}
 
@@ -103,40 +103,40 @@ 
 	 *
 	 * @return int|bool
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 
 		$defaults = array(
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
-		if ( empty( $args['email'] ) ) {
+		if (empty($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
-		$donor = $this->get_donor_by( 'email', $args['email'] );
+		$donor = $this->get_donor_by('email', $args['email']);
 
 		// update an existing donor.
-		if ( $donor ) {
+		if ($donor) {
 
 			// Update the payment IDs attached to the donor
-			if ( ! empty( $args['payment_ids'] ) ) {
+			if ( ! empty($args['payment_ids'])) {
 
-				if ( empty( $donor->payment_ids ) ) {
+				if (empty($donor->payment_ids)) {
 
 					$donor->payment_ids = $args['payment_ids'];
 
 				} else {
 
-					$existing_ids       = array_map( 'absint', explode( ',', $donor->payment_ids ) );
-					$payment_ids        = array_map( 'absint', explode( ',', $args['payment_ids'] ) );
-					$payment_ids        = array_merge( $payment_ids, $existing_ids );
-					$donor->payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+					$existing_ids       = array_map('absint', explode(',', $donor->payment_ids));
+					$payment_ids        = array_map('absint', explode(',', $args['payment_ids']));
+					$payment_ids        = array_merge($payment_ids, $existing_ids);
+					$donor->payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 				}
 
@@ -144,13 +144,13 @@ 
 
 			}
 
-			$this->update( $donor->id, $args );
+			$this->update($donor->id, $args);
 
 			return $donor->id;
 
 		} else {
 
-			return $this->insert( $args, 'donor' );
+			return $this->insert($args, 'donor');
 
 		}
 
@@ -169,16 +169,16 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete( $_id_or_email = false ) {
+	public function delete($_id_or_email = false) {
 
-		if ( empty( $_id_or_email ) ) {
+		if (empty($_id_or_email)) {
 			return false;
 		}
 
-		$column = is_email( $_id_or_email ) ? 'email' : 'id';
-		$donor  = $this->get_donor_by( $column, $_id_or_email );
+		$column = is_email($_id_or_email) ? 'email' : 'id';
+		$donor  = $this->get_donor_by($column, $_id_or_email);
 
-		if ( $donor->id > 0 ) {
+		if ($donor->id > 0) {
 
 			global $wpdb;
 
@@ -187,9 +187,9 @@ 
 			 *
 			 * @since 1.8.14
 			 */
-			Give()->donor_meta->delete_all_meta( $donor->id );
+			Give()->donor_meta->delete_all_meta($donor->id);
 
-			return $wpdb->delete( $this->table_name, array( 'id' => $donor->id ), array( '%d' ) );
+			return $wpdb->delete($this->table_name, array('id' => $donor->id), array('%d'));
 
 		} else {
 			return false;
@@ -210,9 +210,9 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete_by_user_id( $user_id = false ) {
+	public function delete_by_user_id($user_id = false) {
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return false;
 		}
 
@@ -221,14 +221,14 @@ 
 		 *
 		 * @since 1.8.14
 		 */
-		$donor = new Give_Donor( $user_id, true );
-		if ( ! empty( $donor->id ) ) {
-			Give()->donor_meta->delete_all_meta( $donor->id );
+		$donor = new Give_Donor($user_id, true);
+		if ( ! empty($donor->id)) {
+			Give()->donor_meta->delete_all_meta($donor->id);
 		}
 
 		global $wpdb;
 
-		return $wpdb->delete( $this->table_name, array( 'user_id' => $user_id ), array( '%d' ) );
+		return $wpdb->delete($this->table_name, array('user_id' => $user_id), array('%d'));
 	}
 
 	/**
@@ -242,14 +242,14 @@ 
 	 *
 	 * @return bool          True is exists, false otherwise.
 	 */
-	public function exists( $value = '', $field = 'email' ) {
+	public function exists($value = '', $field = 'email') {
 
 		$columns = $this->get_columns();
-		if ( ! array_key_exists( $field, $columns ) ) {
+		if ( ! array_key_exists($field, $columns)) {
 			return false;
 		}
 
-		return (bool) $this->get_column_by( 'id', $field, $value );
+		return (bool) $this->get_column_by('id', $field, $value);
 
 	}
 
@@ -264,16 +264,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function attach_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function attach_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		// Attach the payment, but don't increment stats, as this function previously did not
-		return $donor->attach_payment( $payment_id, false );
+		return $donor->attach_payment($payment_id, false);
 
 	}
 
@@ -288,16 +288,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function remove_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		// Remove the payment, but don't decrease stats, as this function previously did not
-		return $donor->remove_payment( $payment_id, false );
+		return $donor->remove_payment($payment_id, false);
 
 	}
 
@@ -311,18 +311,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function increment_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function increment_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		$increased_count = $donor->increase_purchase_count();
-		$increased_value = $donor->increase_value( $amount );
+		$increased_value = $donor->increase_value($amount);
 
-		return ( $increased_count && $increased_value ) ? true : false;
+		return ($increased_count && $increased_value) ? true : false;
 
 	}
 
@@ -337,18 +337,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function decrement_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function decrement_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		$decreased_count = $donor->decrease_donation_count();
-		$decreased_value = $donor->decrease_value( $amount );
+		$decreased_value = $donor->decrease_value($amount);
 
-		return ( $decreased_count && $decreased_value ) ? true : false;
+		return ($decreased_count && $decreased_value) ? true : false;
 
 	}
 
@@ -363,31 +363,31 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update_donor_email_on_user_update( $user_id = 0, $old_user_data = false ) {
+	public function update_donor_email_on_user_update($user_id = 0, $old_user_data = false) {
 
-		$donor = new Give_Donor( $user_id, true );
+		$donor = new Give_Donor($user_id, true);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
-		$user = get_userdata( $user_id );
+		$user = get_userdata($user_id);
 
-		if ( ! empty( $user ) && $user->user_email !== $donor->email ) {
+		if ( ! empty($user) && $user->user_email !== $donor->email) {
 
-			if ( ! $this->get_donor_by( 'email', $user->user_email ) ) {
+			if ( ! $this->get_donor_by('email', $user->user_email)) {
 
-				$success = $this->update( $donor->id, array( 'email' => $user->user_email ) );
+				$success = $this->update($donor->id, array('email' => $user->user_email));
 
-				if ( $success ) {
+				if ($success) {
 					// Update some payment meta if we need to
-					$payments_array = explode( ',', $donor->payment_ids );
+					$payments_array = explode(',', $donor->payment_ids);
 
-					if ( ! empty( $payments_array ) ) {
+					if ( ! empty($payments_array)) {
 
-						foreach ( $payments_array as $payment_id ) {
+						foreach ($payments_array as $payment_id) {
 
-							give_update_payment_meta( $payment_id, 'email', $user->user_email );
+							give_update_payment_meta($payment_id, 'email', $user->user_email);
 
 						}
 
@@ -401,7 +401,7 @@ 
 					 * @param  WP_User    $user  WordPress User object.
 					 * @param  Give_Donor $donor Give donor object.
 					 */
-					do_action( 'give_update_donor_email_on_user_update', $user, $donor );
+					do_action('give_update_donor_email_on_user_update', $user, $donor);
 
 				}
 
@@ -422,44 +422,44 @@ 
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */
-	public function get_donor_by( $field = 'id', $value = 0 ) {
-		$value = sanitize_text_field( $value );
+	public function get_donor_by($field = 'id', $value = 0) {
+		$value = sanitize_text_field($value);
 
 		// Bailout.
-		if ( empty( $field ) || empty( $value ) ) {
+		if (empty($field) || empty($value)) {
 			return null;
 		}
 
 		// Verify values.
-		if ( 'id' === $field || 'user_id' === $field ) {
+		if ('id' === $field || 'user_id' === $field) {
 			// Make sure the value is numeric to avoid casting objects, for example,
 			// to int 1.
-			if ( ! is_numeric( $value ) ) {
+			if ( ! is_numeric($value)) {
 				return false;
 			}
 
-			$value = absint( $value );
+			$value = absint($value);
 
-			if ( $value < 1 ) {
+			if ($value < 1) {
 				return false;
 			}
 
-		} elseif ( 'email' === $field ) {
+		} elseif ('email' === $field) {
 
-			if ( ! is_email( $value ) ) {
+			if ( ! is_email($value)) {
 				return false;
 			}
 
-			$value = trim( $value );
+			$value = trim($value);
 		}
 
 		// Bailout
-		if ( ! $value ) {
+		if ( ! $value) {
 			return false;
 		}
 
 		// Set query params.
-		switch ( $field ) {
+		switch ($field) {
 			case 'id':
 				$args['donor'] = $value;
 				break;
@@ -474,9 +474,9 @@ 
 		}
 
 		// Get donors.
-		$donor = new Give_Donors_Query( $args );
+		$donor = new Give_Donors_Query($args);
 
-		if ( ! $donor = $donor->get_donors() ) {
+		if ( ! $donor = $donor->get_donors()) {
 			// Look for donor from an additional email.
 			$args = array(
 				'meta_query' => array(
@@ -487,15 +487,15 @@ 
 				),
 			);
 
-			$donor = new Give_Donors_Query( $args );
+			$donor = new Give_Donors_Query($args);
 			$donor = $donor->get_donors();
 
-			if ( empty( $donor ) ) {
+			if (empty($donor)) {
 				return false;
 			}
 		}
 
-		return current( $donor );
+		return current($donor);
 	}
 
 	/**
@@ -508,18 +508,18 @@ 
 	 *
 	 * @return array|object|null Donors array or object. Null if not found.
 	 */
-	public function get_donors( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function get_donors($args = array()) {
+		$this->bc_1814_params($args);
 
-		$cache_key = md5( 'give_donors_' . serialize( $args ) );
+		$cache_key = md5('give_donors_'.serialize($args));
 
-		$donors = wp_cache_get( $cache_key, 'donors' );
+		$donors = wp_cache_get($cache_key, 'donors');
 
-		if ( $donors === false ) {
-			$donors = new Give_Donors_Query( $args );
+		if ($donors === false) {
+			$donors = new Give_Donors_Query($args);
 			$donors = $donors->get_donors();
 
-			wp_cache_set( $cache_key, $donors, 'donors', 3600 );
+			wp_cache_set($cache_key, $donors, 'donors', 3600);
 		}
 
 		return $donors;
@@ -537,21 +537,21 @@ 
 	 *
 	 * @return int         Total number of donors.
 	 */
-	public function count( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function count($args = array()) {
+		$this->bc_1814_params($args);
 		$args['count'] = true;
 
-		$cache_key = md5( 'give_donors_count' . serialize( $args ) );
-		$count     = wp_cache_get( $cache_key, 'donors' );
+		$cache_key = md5('give_donors_count'.serialize($args));
+		$count     = wp_cache_get($cache_key, 'donors');
 
-		if ( $count === false ) {
-			$donors = new Give_Donors_Query( $args );
+		if ($count === false) {
+			$donors = new Give_Donors_Query($args);
 			$count  = $donors->get_donors();
 
-			wp_cache_set( $cache_key, $count, 'donors', 3600 );
+			wp_cache_set($cache_key, $count, 'donors', 3600);
 		}
 
-		return absint( $count );
+		return absint($count);
 
 	}
 
@@ -565,9 +565,9 @@ 
 	 */
 	public function create_table() {
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
-		$sql = "CREATE TABLE " . $this->table_name . " (
+		$sql = "CREATE TABLE ".$this->table_name." (
 		id bigint(20) NOT NULL AUTO_INCREMENT,
 		user_id bigint(20) NOT NULL,
 		email varchar(50) NOT NULL,
@@ -582,9 +582,9 @@ 
 		KEY user (user_id)
 		) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 	/**
@@ -596,7 +596,7 @@ 
 	 * @return bool Returns if the donors table was installed and upgrade routine run.
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 	/**
@@ -607,41 +607,41 @@ 
 	 *
 	 * @param $args
 	 */
-	private function bc_1814_params( &$args ) {
+	private function bc_1814_params(&$args) {
 		// Backward compatibility: user_id
-		if ( ! empty( $args['user_id'] ) ) {
+		if ( ! empty($args['user_id'])) {
 			$args['user'] = $args['user_id'];
 		}
 
 		// Backward compatibility: id
-		if ( ! empty( $args['id'] ) ) {
+		if ( ! empty($args['id'])) {
 			$args['donor'] = $args['id'];
 		}
 
 		// Backward compatibility: name
-		if ( ! empty( $args['name'] ) ) {
+		if ( ! empty($args['name'])) {
 			$args['s'] = "name:{$args['name']}";
 		}
 
 		// Backward compatibility: date
 		// Donors created for a specific date or in a date range.
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				if ( ! empty( $args['date']['start'] ) ) {
-					$args['date_query']['after'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
+				if ( ! empty($args['date']['start'])) {
+					$args['date_query']['after'] = date('Y-m-d H:i:s', strtotime($args['date']['start']));
 				}
 
-				if ( ! empty( $args['date']['end'] ) ) {
-					$args['date_query']['before'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+				if ( ! empty($args['date']['end'])) {
+					$args['date_query']['before'] = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 				}
 
 			} else {
 
-				$args['date_query']['year']  = date( 'Y', strtotime( $args['date'] ) );
-				$args['date_query']['month'] = date( 'm', strtotime( $args['date'] ) );
-				$args['date_query']['day']   = date( 'd', strtotime( $args['date'] ) );
+				$args['date_query']['year']  = date('Y', strtotime($args['date']));
+				$args['date_query']['month'] = date('m', strtotime($args['date']));
+				$args['date_query']['day']   = date('d', strtotime($args['date']));
 			}
 
 		}

includes/class-give-license-handler.php

Doc Comments +3 added lines, -3 removed lines

@@ -703,7 +703,7 @@ 
 		 * @access private
 		 * @since  1.7
 		 *
-		 * @return bool
+		 * @return false|null
 		 */
 		private function __remove_license_key_from_subscriptions() {
 			$subscriptions = get_option( 'give_subscriptions', array() );
@@ -736,7 +736,7 @@ 
 		 * @param $plugin_data
 		 * @param $status
 		 *
-		 * @return bool
+		 * @return false|null
 		 */
 		public function plugin_page_notices( $plugin_file, $plugin_data, $status ) {
 			// Bailout.
@@ -758,7 +758,7 @@ 
 		 *
 		 * @since  1.8.7
 		 * @access public
-		 * @return array
+		 * @return string
 		 */
 		public function license_state_message() {
 			$message_data = array();

Spacing +151 added lines, -151 removed lines

@@ -10,11 +10,11 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_License' ) ) :
+if ( ! class_exists('Give_License')) :
 
 	/**
 	 * Give_License Class
@@ -143,27 +143,27 @@ 
 		 * @param string $_checkout_url
 		 * @param string $_account_url
 		 */
-		public function __construct( $_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null, $_checkout_url = null, $_account_url = null ) {
+		public function __construct($_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null, $_checkout_url = null, $_account_url = null) {
 
 			$give_options = give_get_settings();
 
 			$this->file             = $_file;
 			$this->item_name        = $_item_name;
-			$this->item_shortname   = 'give_' . preg_replace( '/[^a-zA-Z0-9_\s]/', '', str_replace( ' ', '_', strtolower( $this->item_name ) ) );
+			$this->item_shortname   = 'give_'.preg_replace('/[^a-zA-Z0-9_\s]/', '', str_replace(' ', '_', strtolower($this->item_name)));
 			$this->version          = $_version;
-			$this->license          = isset( $give_options[ $this->item_shortname . '_license_key' ] ) ? trim( $give_options[ $this->item_shortname . '_license_key' ] ) : '';
-			$this->license_data     = get_option( $this->item_shortname . '_license_active' );
+			$this->license          = isset($give_options[$this->item_shortname.'_license_key']) ? trim($give_options[$this->item_shortname.'_license_key']) : '';
+			$this->license_data     = get_option($this->item_shortname.'_license_active');
 			$this->author           = $_author;
-			$this->api_url          = is_null( $_api_url ) ? $this->api_url : $_api_url;
-			$this->checkout_url     = is_null( $_checkout_url ) ? $this->checkout_url : $_checkout_url;
-			$this->account_url      = is_null( $_account_url ) ? $this->account_url : $_account_url;
+			$this->api_url          = is_null($_api_url) ? $this->api_url : $_api_url;
+			$this->checkout_url     = is_null($_checkout_url) ? $this->checkout_url : $_checkout_url;
+			$this->account_url      = is_null($_account_url) ? $this->account_url : $_account_url;
 			$this->auto_updater_obj = null;
 
 			// Add Setting for Give Add-on activation status.
-			$is_addon_activated = get_option( 'give_is_addon_activated' );
-			if ( ! $is_addon_activated && is_object( $this ) && sizeof( $this ) > 0 ) {
-				update_option( 'give_is_addon_activated', true );
-				Give_Cache::set( 'give_cache_hide_license_notice_after_activation', true, DAY_IN_SECONDS );
+			$is_addon_activated = get_option('give_is_addon_activated');
+			if ( ! $is_addon_activated && is_object($this) && sizeof($this) > 0) {
+				update_option('give_is_addon_activated', true);
+				Give_Cache::set('give_cache_hide_license_notice_after_activation', true, DAY_IN_SECONDS);
 			}
 
 			// Setup hooks
@@ -184,7 +184,7 @@ 
 		 */
 		private function includes() {
 
-			if ( ! class_exists( 'EDD_SL_Plugin_Updater' ) ) {
+			if ( ! class_exists('EDD_SL_Plugin_Updater')) {
 				require_once 'admin/EDD_SL_Plugin_Updater.php';
 			}
 		}
@@ -202,30 +202,30 @@ 
 		private function hooks() {
 
 			// Register settings.
-			add_filter( 'give_settings_licenses', array( $this, 'settings' ), 1 );
+			add_filter('give_settings_licenses', array($this, 'settings'), 1);
 
 			// Activate license key on settings save.
-			add_action( 'admin_init', array( $this, 'activate_license' ), 10 );
+			add_action('admin_init', array($this, 'activate_license'), 10);
 
 			// Deactivate license key.
-			add_action( 'admin_init', array( $this, 'deactivate_license' ), 11 );
+			add_action('admin_init', array($this, 'deactivate_license'), 11);
 
 			// Updater.
-			add_action( 'admin_init', array( $this, 'auto_updater' ), 0 );
-			add_action( 'admin_notices', array( $this, 'notices' ) );
+			add_action('admin_init', array($this, 'auto_updater'), 0);
+			add_action('admin_notices', array($this, 'notices'));
 
 			// Check license weekly.
-			Give_Cron::add_weekly_event( array( $this, 'weekly_license_check' ) );
-			add_action( 'give_validate_license_when_site_migrated', array( $this, 'weekly_license_check' ) );
+			Give_Cron::add_weekly_event(array($this, 'weekly_license_check'));
+			add_action('give_validate_license_when_site_migrated', array($this, 'weekly_license_check'));
 
 			// Check subscription weekly.
-			Give_Cron::add_weekly_event( array( $this, 'weekly_subscription_check' ) );
-			add_action( 'give_validate_license_when_site_migrated', array( $this, 'weekly_subscription_check' ) );
+			Give_Cron::add_weekly_event(array($this, 'weekly_subscription_check'));
+			add_action('give_validate_license_when_site_migrated', array($this, 'weekly_subscription_check'));
 
 			// Show addon notice on plugin page.
-			$plugin_name = explode( 'plugins/', $this->file );
-			$plugin_name = end( $plugin_name );
-			add_action( "after_plugin_row_{$plugin_name}", array( $this, 'plugin_page_notices' ), 10, 3 );
+			$plugin_name = explode('plugins/', $this->file);
+			$plugin_name = end($plugin_name);
+			add_action("after_plugin_row_{$plugin_name}", array($this, 'plugin_page_notices'), 10, 3);
 
 		}
 
@@ -265,16 +265,16 @@ 
 		 *
 		 * @return array           License settings.
 		 */
-		public function settings( $settings ) {
+		public function settings($settings) {
 
 			$give_license_settings = array(
 				array(
 					'name'    => $this->item_name,
-					'id'      => $this->item_shortname . '_license_key',
+					'id'      => $this->item_shortname.'_license_key',
 					'desc'    => '',
 					'type'    => 'license_key',
 					'options' => array(
-						'license'      => get_option( $this->item_shortname . '_license_active' ),
+						'license'      => get_option($this->item_shortname.'_license_active'),
 						'shortname'    => $this->item_shortname,
 						'item_name'    => $this->item_name,
 						'api_url'      => $this->api_url,
@@ -285,7 +285,7 @@ 
 				),
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -300,18 +300,18 @@ 
 		 *
 		 * @return array           License settings content.
 		 */
-		public function license_settings_content( $settings ) {
+		public function license_settings_content($settings) {
 
 			$give_license_settings = array(
 				array(
-					'name' => __( 'Add-on Licenses', 'give' ),
+					'name' => __('Add-on Licenses', 'give'),
 					'desc' => '<hr>',
 					'type' => 'give_title',
 					'id'   => 'give_title',
 				),
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -326,62 +326,62 @@ 
 		 */
 		public function activate_license() {
 			// Bailout.
-			if ( ! $this->__is_user_can_edit_license() ) {
+			if ( ! $this->__is_user_can_edit_license()) {
 				return;
 			}
 
 			// Allow third party addon developers to handle license activation.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_activate_license', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_activate_license', $this);
 
 				return;
 			}
 
 			// Delete previous license setting if a empty license key submitted.
-			if ( empty( $_POST["{$this->item_shortname}_license_key"] ) ) {
+			if (empty($_POST["{$this->item_shortname}_license_key"])) {
 				$this->unset_license();
 
 				return;
 			}
 
 			// Do not simultaneously activate add-ons if the user want to deactivate a specific add-on.
-			if( $this->is_deactivating_license() ) {
+			if ($this->is_deactivating_license()) {
 				return;
 			}
 
 			// Check if plugin previously installed.
-			if ( $this->is_valid_license() ) {
+			if ($this->is_valid_license()) {
 				return;
 			}
 
 			// Get license key.
-			$this->license = sanitize_text_field( $_POST[ $this->item_shortname . '_license_key' ] );
+			$this->license = sanitize_text_field($_POST[$this->item_shortname.'_license_key']);
 
 			// Delete previous license key from subscription if previously added.
 			$this->__remove_license_key_from_subscriptions();
 
 			// Make sure there are no api errors.
-			if ( ! ( $license_data = $this->get_license_info( 'activate_license' ) ) ) {
+			if ( ! ($license_data = $this->get_license_info('activate_license'))) {
 				return;
 			}
 
 			// Make sure license is valid.
 			// return because admin will want to activate license again.
-			if ( ! $this->is_license( $license_data ) ) {
+			if ( ! $this->is_license($license_data)) {
 				// Add license key.
-				give_update_option( "{$this->item_shortname}_license_key", $this->license );
+				give_update_option("{$this->item_shortname}_license_key", $this->license);
 
 				return;
 			}
 
 			// Tell WordPress to look for updates.
-			set_site_transient( 'update_plugins', null );
+			set_site_transient('update_plugins', null);
 
 			// Add license data.
-			update_option( "{$this->item_shortname}_license_active", $license_data );
+			update_option("{$this->item_shortname}_license_active", $license_data);
 
 			// Add license key.
-			give_update_option( "{$this->item_shortname}_license_key", $this->license );
+			give_update_option("{$this->item_shortname}_license_key", $this->license);
 
 			// Check subscription for license key and store this to db (if any).
 			$this->__single_subscription_check();
@@ -399,19 +399,19 @@ 
 		 */
 		public function deactivate_license() {
 			// Bailout.
-			if ( ! $this->__is_user_can_edit_license() ) {
+			if ( ! $this->__is_user_can_edit_license()) {
 				return;
 			}
 
 			// Allow third party add-on developers to handle license deactivation.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_deactivate_license', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_deactivate_license', $this);
 
 				return;
 			}
 
 			// Run on deactivate button press.
-			if ( isset( $_POST[ $this->item_shortname . '_license_key_deactivate' ] ) ) {
+			if (isset($_POST[$this->item_shortname.'_license_key_deactivate'])) {
 				$this->unset_license();
 			}
 		}
@@ -427,30 +427,30 @@ 
 		public function weekly_license_check() {
 
 			if (
-				! empty( $_POST['give_settings'] ) ||
-				empty( $this->license )
+				! empty($_POST['give_settings']) ||
+				empty($this->license)
 			) {
 				return;
 			}
 
 			// Allow third party add-on developers to handle their license check.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_weekly_license_check', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_weekly_license_check', $this);
 
 				return;
 			}
 
 			// Make sure there are no api errors.
-			if ( ! ( $license_data = $this->get_license_info( 'check_license' ) ) ) {
+			if ( ! ($license_data = $this->get_license_info('check_license'))) {
 				return;
 			}
 
 			// Bailout.
-			if ( ! $this->is_license( $license_data ) ) {
+			if ( ! $this->is_license($license_data)) {
 				return;
 			}
 
-			update_option( $this->item_shortname . '_license_active', $license_data );
+			update_option($this->item_shortname.'_license_active', $license_data);
 
 			return;
 		}
@@ -466,21 +466,21 @@ 
 		public function weekly_subscription_check() {
 			// Bailout.
 			if (
-				! empty( $_POST['give_settings'] ) ||
-				empty( $this->license )
+				! empty($_POST['give_settings']) ||
+				empty($this->license)
 			) {
 				return;
 			}
 
 			// Remove old subscription data.
-			if ( absint( get_option( '_give_subscriptions_edit_last', true ) ) < current_time( 'timestamp', 1 ) ) {
-				delete_option( 'give_subscriptions' );
-				update_option( '_give_subscriptions_edit_last', strtotime( '+ 1 day', current_time( 'timestamp', 1 ) ) );
+			if (absint(get_option('_give_subscriptions_edit_last', true)) < current_time('timestamp', 1)) {
+				delete_option('give_subscriptions');
+				update_option('_give_subscriptions_edit_last', strtotime('+ 1 day', current_time('timestamp', 1)));
 			}
 
 			// Allow third party add-on developers to handle their subscription check.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_weekly_subscription_check', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_weekly_subscription_check', $this);
 
 				return;
 			}
@@ -497,7 +497,7 @@ 
 		 * @return void
 		 */
 		private function __single_subscription_check() {
-			if ( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return;
 			}
 
@@ -508,26 +508,26 @@ 
 			 * By default edd software licensing api does not have api to check subscription.
 			 * This is a custom feature to check subscriptions.
 			 */
-			$subscription_data = $this->get_license_info( 'check_subscription', true );
+			$subscription_data = $this->get_license_info('check_subscription', true);
 
-			if ( ! empty( $subscription_data['success'] ) && absint( $subscription_data['success'] ) ) {
+			if ( ! empty($subscription_data['success']) && absint($subscription_data['success'])) {
 
-				$subscriptions = get_option( 'give_subscriptions', array() );
+				$subscriptions = get_option('give_subscriptions', array());
 
 				// Update subscription data only if subscription does not exist already.
-				$subscriptions[ $subscription_data['id'] ] = $subscription_data;
+				$subscriptions[$subscription_data['id']] = $subscription_data;
 
 				// Initiate default set of license for subscription.
-				if ( ! isset( $subscriptions[ $subscription_data['id'] ]['licenses'] ) ) {
-					$subscriptions[ $subscription_data['id'] ]['licenses'] = array();
+				if ( ! isset($subscriptions[$subscription_data['id']]['licenses'])) {
+					$subscriptions[$subscription_data['id']]['licenses'] = array();
 				}
 
 				// Store licenses for subscription.
-				if ( ! in_array( $this->license, $subscriptions[ $subscription_data['id'] ]['licenses'] ) ) {
-					$subscriptions[ $subscription_data['id'] ]['licenses'][] = $this->license;
+				if ( ! in_array($this->license, $subscriptions[$subscription_data['id']]['licenses'])) {
+					$subscriptions[$subscription_data['id']]['licenses'][] = $this->license;
 				}
 
-				update_option( 'give_subscriptions', $subscriptions );
+				update_option('give_subscriptions', $subscriptions);
 			}
 		}
 
@@ -541,12 +541,12 @@ 
 		 */
 		public function notices() {
 
-			if ( ! current_user_can( 'manage_give_settings' ) ) {
+			if ( ! current_user_can('manage_give_settings')) {
 				return;
 			}
 
 			// Do not show licenses notices on license tab.
-			if ( 'licenses' === give_get_current_setting_tab() ) {
+			if ('licenses' === give_get_current_setting_tab()) {
 				return;
 			}
 
@@ -555,118 +555,118 @@ 
 			static $addon_license_key_in_subscriptions;
 
 			// Set default value.
-			$addon_license_key_in_subscriptions = ! empty( $addon_license_key_in_subscriptions ) ? $addon_license_key_in_subscriptions : array();
+			$addon_license_key_in_subscriptions = ! empty($addon_license_key_in_subscriptions) ? $addon_license_key_in_subscriptions : array();
 			$messages                           = array();
 
 			// Check whether admin has Give Add-on activated since 24 hours?
-			$is_license_notice_hidden = Give_Cache::get( 'give_cache_hide_license_notice_after_activation' );
+			$is_license_notice_hidden = Give_Cache::get('give_cache_hide_license_notice_after_activation');
 
 			// Display Invalid License notice, if its more than 24 hours since first Give Add-on activation.
 			if (
-				empty( $this->license )
-				&& empty( $showed_invalid_message )
-				&& ( false === $is_license_notice_hidden )
+				empty($this->license)
+				&& empty($showed_invalid_message)
+				&& (false === $is_license_notice_hidden)
 			) {
 
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'               => 'give-invalid-license',
 					'type'             => 'error',
 					'description'      => sprintf(
-						__( 'You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give' ),
-						admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' )
+						__('You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give'),
+						admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')
 					),
 					'dismissible_type' => 'user',
 					'dismiss_interval' => 'shortly',
-				) );
+				));
 
 				$showed_invalid_message = true;
 
 			}
 
 			// Get subscriptions.
-			$subscriptions = get_option( 'give_subscriptions' );
+			$subscriptions = get_option('give_subscriptions');
 
 			// Show subscription messages.
-			if ( ! empty( $subscriptions ) && ! $showed_subscriptions_message ) {
+			if ( ! empty($subscriptions) && ! $showed_subscriptions_message) {
 
-				foreach ( $subscriptions as $subscription ) {
+				foreach ($subscriptions as $subscription) {
 					// Subscription expires timestamp.
-					$subscription_expires = strtotime( $subscription['expires'] );
+					$subscription_expires = strtotime($subscription['expires']);
 
 					// Start showing subscriptions message before one week of renewal date.
-					if ( strtotime( '- 7 days', $subscription_expires ) > current_time( 'timestamp', 1 ) ) {
+					if (strtotime('- 7 days', $subscription_expires) > current_time('timestamp', 1)) {
 						continue;
 					}
 
 					// Check if subscription message already exist in messages.
-					if ( array_key_exists( $subscription['id'], $messages ) ) {
+					if (array_key_exists($subscription['id'], $messages)) {
 						continue;
 					}
 
 					// Check if license already expired.
-					if ( strtotime( $subscription['expires'] ) < current_time( 'timestamp', 1 ) ) {
-						Give()->notices->register_notice( array(
+					if (strtotime($subscription['expires']) < current_time('timestamp', 1)) {
+						Give()->notices->register_notice(array(
 							'id'               => "give-expired-subscription-{$subscription['id']}",
 							'type'             => 'error',
 							'description'      => sprintf(
-								__( 'Your Give add-on license expired for payment <a href="%1$s" target="_blank">#%2$d</a>. <a href="%3$s" target="_blank">Click to renew an existing license</a> or %4$s.', 'give' ),
-								urldecode( $subscription['invoice_url'] ),
+								__('Your Give add-on license expired for payment <a href="%1$s" target="_blank">#%2$d</a>. <a href="%3$s" target="_blank">Click to renew an existing license</a> or %4$s.', 'give'),
+								urldecode($subscription['invoice_url']),
 								$subscription['payment_id'],
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
-								Give()->notices->get_dismiss_link( array(
-									'title'            => __( 'Click here if already renewed', 'give' ),
+								Give()->notices->get_dismiss_link(array(
+									'title'            => __('Click here if already renewed', 'give'),
 									'dismissible_type' => 'user',
 									'dismiss_interval' => 'permanent',
-								) )
+								))
 							),
 							'dismissible_type' => 'user',
 							'dismiss_interval' => 'shortly',
-						) );
+						));
 					} else {
-						Give()->notices->register_notice( array(
+						Give()->notices->register_notice(array(
 							'id'               => "give-expires-subscription-{$subscription['id']}",
 							'type'             => 'error',
 							'description'      => sprintf(
-								__( 'Your Give add-on license will expire in %1$s for payment <a href="%2$s" target="_blank">#%3$d</a>. <a href="%4$s" target="_blank">Click to renew an existing license</a> or %5$s.', 'give' ),
-								human_time_diff( current_time( 'timestamp', 1 ), strtotime( $subscription['expires'] ) ),
-								urldecode( $subscription['invoice_url'] ),
+								__('Your Give add-on license will expire in %1$s for payment <a href="%2$s" target="_blank">#%3$d</a>. <a href="%4$s" target="_blank">Click to renew an existing license</a> or %5$s.', 'give'),
+								human_time_diff(current_time('timestamp', 1), strtotime($subscription['expires'])),
+								urldecode($subscription['invoice_url']),
 								$subscription['payment_id'],
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
-								Give()->notices->get_dismiss_link( array(
-									'title'            => __( 'Click here if already renewed', 'give' ),
+								Give()->notices->get_dismiss_link(array(
+									'title'            => __('Click here if already renewed', 'give'),
 									'dismissible_type' => 'user',
 									'dismiss_interval' => 'permanent',
-								) )
+								))
 							),
 							'dismissible_type' => 'user',
 							'dismiss_interval' => 'shortly',
-						) );
+						));
 					}
 
 					// Stop validation for these license keys.
-					$addon_license_key_in_subscriptions = array_merge( $addon_license_key_in_subscriptions, $subscription['licenses'] );
+					$addon_license_key_in_subscriptions = array_merge($addon_license_key_in_subscriptions, $subscription['licenses']);
 				}// End foreach().
 				$showed_subscriptions_message = true;
 			}// End if().
 
 			// Show Non Subscription Give Add-on messages.
 			if (
-				! in_array( $this->license, $addon_license_key_in_subscriptions )
-				&& ! empty( $this->license )
-				&& empty( $showed_invalid_message )
+				! in_array($this->license, $addon_license_key_in_subscriptions)
+				&& ! empty($this->license)
+				&& empty($showed_invalid_message)
 				&& ! $this->is_valid_license()
 			) {
 
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'               => 'give-invalid-license',
 					'type'             => 'error',
 					'description'      => sprintf(
-						__( 'You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give' ),
-						admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' )
+						__('You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give'),
+						admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')
 					),
 					'dismissible_type' => 'user',
 					'dismiss_interval' => 'shortly',
-				) );
+				));
 
 				$showed_invalid_message = true;
 
@@ -683,10 +683,10 @@ 
 		 *
 		 * @return bool
 		 */
-		public function is_valid_license( $licence_data = null ) {
-			$license_data = empty( $licence_data ) ? $this->license_data : $licence_data;
+		public function is_valid_license($licence_data = null) {
+			$license_data = empty($licence_data) ? $this->license_data : $licence_data;
 
-			if ( apply_filters( 'give_is_valid_license', ( $this->is_license( $license_data ) && 'valid' === $license_data->license ) ) ) {
+			if (apply_filters('give_is_valid_license', ($this->is_license($license_data) && 'valid' === $license_data->license))) {
 				return true;
 			}
 
@@ -704,10 +704,10 @@ 
 		 *
 		 * @return bool
 		 */
-		public function is_license( $licence_data = null ) {
-			$license_data = empty( $licence_data ) ? $this->license_data : $licence_data;
+		public function is_license($licence_data = null) {
+			$license_data = empty($licence_data) ? $this->license_data : $licence_data;
 
-			if ( apply_filters( 'give_is_license', ( is_object( $license_data ) && ! empty( $license_data ) && property_exists( $license_data, 'license' ) ) ) ) {
+			if (apply_filters('give_is_license', (is_object($license_data) && ! empty($license_data) && property_exists($license_data, 'license')))) {
 				return true;
 			}
 
@@ -723,7 +723,7 @@ 
 		 * @return bool
 		 */
 		private function __is_third_party_addon() {
-			return ( false === strpos( $this->api_url, 'givewp.com/' ) );
+			return (false === strpos($this->api_url, 'givewp.com/'));
 		}
 
 		/**
@@ -738,25 +738,25 @@ 
 		 * @return bool
 		 */
 		private function __remove_license_key_from_subscriptions() {
-			$subscriptions = get_option( 'give_subscriptions', array() );
+			$subscriptions = get_option('give_subscriptions', array());
 
 			// Bailout.
-			if ( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return false;
 			}
 
-			if ( ! empty( $subscriptions ) ) {
-				foreach ( $subscriptions as $subscription_id => $subscription ) {
-					$license_index = array_search( $this->license, $subscription['licenses'] );
-					if ( false !== $license_index ) {
+			if ( ! empty($subscriptions)) {
+				foreach ($subscriptions as $subscription_id => $subscription) {
+					$license_index = array_search($this->license, $subscription['licenses']);
+					if (false !== $license_index) {
 						// Remove license key.
-						unset( $subscriptions[ $subscription_id ]['licenses'][ $license_index ] );
+						unset($subscriptions[$subscription_id]['licenses'][$license_index]);
 
 						// Rearrange license keys.
-						$subscriptions[ $subscription_id ]['licenses'] = array_values( $subscriptions[ $subscription_id ]['licenses'] );
+						$subscriptions[$subscription_id]['licenses'] = array_values($subscriptions[$subscription_id]['licenses']);
 
 						// Update subscription information.
-						update_option( 'give_subscriptions', $subscriptions );
+						update_option('give_subscriptions', $subscriptions);
 						break;
 					}
 				}
@@ -770,17 +770,17 @@ 
 		 *
 		 * @return bool
 		 */
-		public function plugin_page_notices( $plugin_file, $plugin_data, $status ) {
+		public function plugin_page_notices($plugin_file, $plugin_data, $status) {
 			// Bailout.
-			if ( $this->is_valid_license() ) {
+			if ($this->is_valid_license()) {
 				return false;
 			}
 
 			$update_notice_wrap = '<tr class="give-addon-notice-tr active"><td colspan="3" class="colspanchange"><div class="notice inline notice-warning notice-alt give-invalid-license"><p><span class="dashicons dashicons-info"></span> %s</p></div></td></tr>';
 			$message            = $this->license_state_message();
 
-			if ( ! empty( $message['message'] ) ) {
-				echo sprintf( $update_notice_wrap, $message['message'] );
+			if ( ! empty($message['message'])) {
+				echo sprintf($update_notice_wrap, $message['message']);
 			}
 		}
 
@@ -795,11 +795,11 @@ 
 		public function license_state_message() {
 			$message_data = array();
 
-			if ( ! $this->is_valid_license() ) {
+			if ( ! $this->is_valid_license()) {
 
 				$message_data['message'] = sprintf(
 					'Please <a href="%1$s">activate your license</a> to receive updates and support for the %2$s add-on.',
-					esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' ) ),
+					esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')),
 					$this->item_name
 				);
 			}
@@ -818,7 +818,7 @@ 
 			// Bailout.
 			if (
 				! Give_Admin_Settings::verify_nonce() ||
-				! current_user_can( 'manage_give_settings' ) ||
+				! current_user_can('manage_give_settings') ||
 				'licenses' !== give_get_current_setting_tab()
 			) {
 				return false;
@@ -826,10 +826,10 @@ 
 
 			// Security check.
 			if (
-				isset( $_POST[ $this->item_shortname . '_license_key-nonce' ] ) &&
-				! wp_verify_nonce( $_REQUEST[ $this->item_shortname . '_license_key-nonce' ], $this->item_shortname . '_license_key-nonce' )
+				isset($_POST[$this->item_shortname.'_license_key-nonce']) &&
+				! wp_verify_nonce($_REQUEST[$this->item_shortname.'_license_key-nonce'], $this->item_shortname.'_license_key-nonce')
 			) {
-				wp_die( __( 'Nonce verification failed.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(__('Nonce verification failed.', 'give'), __('Error', 'give'), array('response' => 403));
 			}
 
 			return true;
@@ -847,9 +847,9 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function get_license_info( $edd_action = '', $response_in_array = false ) {
+		public function get_license_info($edd_action = '', $response_in_array = false) {
 
-			if ( empty( $edd_action ) ) {
+			if (empty($edd_action)) {
 				return false;
 			}
 
@@ -857,7 +857,7 @@ 
 			$api_params = array(
 				'edd_action' => $edd_action, // never change from "edd_" to "give_"!
 				'license'    => $this->license,
-				'item_name'  => urlencode( $this->item_name ),
+				'item_name'  => urlencode($this->item_name),
 				'url'        => home_url(),
 			);
 
@@ -872,11 +872,11 @@ 
 			);
 
 			// Make sure there are no errors.
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return false;
 			}
 
-			return json_decode( wp_remote_retrieve_body( $response ), $response_in_array );
+			return json_decode(wp_remote_retrieve_body($response), $response_in_array);
 		}
 
 
@@ -892,9 +892,9 @@ 
 			$this->__remove_license_key_from_subscriptions();
 
 			// Remove license from database.
-			delete_option( "{$this->item_shortname}_license_active" );
-			give_delete_option( "{$this->item_shortname}_license_key" );
-			unset( $_POST["{$this->item_shortname}_license_key"] );
+			delete_option("{$this->item_shortname}_license_active");
+			give_delete_option("{$this->item_shortname}_license_key");
+			unset($_POST["{$this->item_shortname}_license_key"]);
 
 			// Unset license param.
 			$this->license = '';
@@ -912,8 +912,8 @@ 
 		private function is_deactivating_license() {
 			$status = false;
 
-			foreach ( $_POST as $key => $value ) {
-				if ( false !== strpos( $key, 'license_key_deactivate' ) ) {
+			foreach ($_POST as $key => $value) {
+				if (false !== strpos($key, 'license_key_deactivate')) {
 					$status = true;
 					break;
 				}

includes/payments/class-payment-stats.php

Doc Comments +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 	 * @param  $end_date   string|bool  The end date for which we'd like to filter our sale stats. If false, we'll use the default end date of `this_month`
 	 * @param  $status     string|array The sale status(es) to count. Only valid when retrieving global stats
 	 *
-	 * @return float|int                Total amount of donations based on the passed arguments.
+	 * @return string                Total amount of donations based on the passed arguments.
 	 */
 	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
 
@@ -83,7 +83,7 @@ 
 	 * @param  $end_date    string|bool The end date for which we'd like to filter the donations stats. If false, method will use the default end date of `this_month`.
 	 * @param  $gateway_id  string|bool The gateway to get earnings for such as 'paypal' or 'stripe'.
 	 *
-	 * @return float|int                Total amount of donations based on the passed arguments.
+	 * @return string                Total amount of donations based on the passed arguments.
 	 */
 	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
 		$this->setup_dates( $start_date, $end_date );
@@ -168,7 +168,7 @@ 
 	 * @param  $end_date    string|bool The end date for which we'd like to filter our sale stats. If false, we'll use the default end date of `this_month`
 	 * @param  $gateway_id  string|bool The gateway to get earnings for such as 'paypal' or 'stripe'
 	 *
-	 * @return float|int                Total amount of donations based on the passed arguments.
+	 * @return string                Total amount of donations based on the passed arguments.
 	 */
 	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
 

Spacing +41 added lines, -41 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,17 +38,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -57,18 +57,18 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
-		if ( ! empty( $form_id ) ) {
+		if ( ! empty($form_id)) {
 			$args['give_forms'] = $form_id;
 		}
 
 		/* @var Give_Payments_Query $payments */
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		return count( $payments );
+		return count($payments);
 	}
 
 
@@ -85,16 +85,16 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
-		$this->setup_dates( $start_date, $end_date );
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -104,12 +104,12 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -117,39 +117,39 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		// Set transient for faster stats.
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 
-		if ( false === $earnings ) {
+		if (false === $earnings) {
 
 			$this->timestamp = false;
-			$payments        = new Give_Payments_Query( $args );
+			$payments        = new Give_Payments_Query($args);
 			$payments        = $payments->get_payments();
 			$earnings        = 0;
 
-			if ( ! empty( $payments ) ) {
-				foreach ( $payments as $payment ) {
-					$earnings += (float) give_donation_amount( $payment->ID, array( 'type' => 'stats' ) );
+			if ( ! empty($payments)) {
+				foreach ($payments as $payment) {
+					$earnings += (float) give_donation_amount($payment->ID, array('type' => 'stats'));
 				}
 
 			}
 
 			// Cache the results for one hour.
-			Give_Cache::set( $key, give_sanitize_amount_for_db( $earnings ), 60 * 60 );
+			Give_Cache::set($key, give_sanitize_amount_for_db($earnings), 60 * 60);
 		}
 
 		/**
@@ -163,10 +163,10 @@ 
 		 * @param  string|bool $end_date   Earning end date.
 		 * @param  string|bool $gateway_id Payment gateway id.
 		 */
-		$earnings = apply_filters( 'give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id );
+		$earnings = apply_filters('give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id);
 
 		//return earnings
-		return round( $earnings, give_get_price_decimals( $form_id ) );
+		return round($earnings, give_get_price_decimals($form_id));
 
 	}
 
@@ -183,17 +183,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings_cache_key($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -203,12 +203,12 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -216,19 +216,19 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		//return earnings
 		return $key;
@@ -245,16 +245,16 @@ 
 	 *
 	 * @return array       Best selling forms
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 
 		global $wpdb;
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT post_id as form_id, max(meta_value) as sales
 				FROM $wpdb->postmeta WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}

includes/gateways/offline-donations.php

Indentation +4 added lines, -4 removed lines

@@ -249,10 +249,10 @@
 	$admin_message .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n\n";
 
 	$admin_message .= sprintf(
-		                  '<a href="%1$s">%2$s</a>',
-		                  admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id ),
-		                  __( 'View Donation Details &raquo;', 'give' )
-	                  ) . "\n\n";
+						  '<a href="%1$s">%2$s</a>',
+						  admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id ),
+						  __( 'View Donation Details &raquo;', 'give' )
+					  ) . "\n\n";
 
 	$admin_message = apply_filters( 'give_offline_admin_donation_notification', $admin_message, $payment_id );
 	$admin_message = give_do_email_tags( $admin_message, $payment_id );

Spacing +148 added lines, -148 removed lines

@@ -18,17 +18,17 @@ 
  *
  * @return array
  */
-function give_offline_register_gateway( $gateways ) {
+function give_offline_register_gateway($gateways) {
 	// Format: ID => Name
 	$gateways['offline'] = array(
-		'admin_label'    => esc_attr__( 'Offline Donation', 'give' ),
-		'checkout_label' => esc_attr__( 'Offline Donation', 'give' )
+		'admin_label'    => esc_attr__('Offline Donation', 'give'),
+		'checkout_label' => esc_attr__('Offline Donation', 'give')
 	);
 
 	return $gateways;
 }
 
-add_filter( 'give_payment_gateways', 'give_offline_register_gateway', 1 );
+add_filter('give_payment_gateways', 'give_offline_register_gateway', 1);
 
 
 /**
@@ -40,9 +40,9 @@ 
  *
  * @return void
  */
-function give_offline_payment_cc_form( $form_id ) {
+function give_offline_payment_cc_form($form_id) {
 	// Get offline payment instruction.
-	$offline_instructions = give_get_offline_payment_instruction( $form_id, true );
+	$offline_instructions = give_get_offline_payment_instruction($form_id, true);
 
 	ob_start();
 
@@ -53,10 +53,10 @@ 
 	 *
 	 * @param int $form_id Give form id.
 	 */
-	do_action( 'give_before_offline_info_fields', $form_id );
+	do_action('give_before_offline_info_fields', $form_id);
 	?>
     <fieldset id="give_offline_payment_info">
-		<?php echo stripslashes( $offline_instructions ); ?>
+		<?php echo stripslashes($offline_instructions); ?>
     </fieldset>
 	<?php
 	/**
@@ -66,35 +66,35 @@ 
 	 *
 	 * @param int $form_id Give form id.
 	 */
-	do_action( 'give_after_offline_info_fields', $form_id );
+	do_action('give_after_offline_info_fields', $form_id);
 
 	echo ob_get_clean();
 }
 
-add_action( 'give_offline_cc_form', 'give_offline_payment_cc_form' );
+add_action('give_offline_cc_form', 'give_offline_payment_cc_form');
 
 /**
  * Give Offline Billing Field
  *
  * @param $form_id
  */
-function give_offline_billing_fields( $form_id ) {
+function give_offline_billing_fields($form_id) {
 	//Enable Default CC fields (billing info)
-	$post_offline_cc_fields        = give_get_meta( $form_id, '_give_offline_donation_enable_billing_fields_single', true );
-	$post_offline_customize_option = give_get_meta( $form_id, '_give_customize_offline_donations', true );
+	$post_offline_cc_fields        = give_get_meta($form_id, '_give_offline_donation_enable_billing_fields_single', true);
+	$post_offline_customize_option = give_get_meta($form_id, '_give_customize_offline_donations', true);
 
-	$global_offline_cc_fields = give_get_option( 'give_offline_donation_enable_billing_fields' );
+	$global_offline_cc_fields = give_get_option('give_offline_donation_enable_billing_fields');
 
 	//Output CC Address fields if global option is on and user hasn't elected to customize this form's offline donation options
 	if (
-		( give_is_setting_enabled( $post_offline_customize_option, 'global' ) && give_is_setting_enabled( $global_offline_cc_fields ) )
-		|| ( give_is_setting_enabled( $post_offline_customize_option, 'enabled' ) && give_is_setting_enabled( $post_offline_cc_fields ) )
+		(give_is_setting_enabled($post_offline_customize_option, 'global') && give_is_setting_enabled($global_offline_cc_fields))
+		|| (give_is_setting_enabled($post_offline_customize_option, 'enabled') && give_is_setting_enabled($post_offline_cc_fields))
 	) {
-		give_default_cc_address_fields( $form_id );
+		give_default_cc_address_fields($form_id);
 	}
 }
 
-add_action( 'give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1 );
+add_action('give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1);
 
 /**
  * Process the payment
@@ -105,18 +105,18 @@ 
  *
  * @return void
  */
-function give_offline_process_payment( $purchase_data ) {
+function give_offline_process_payment($purchase_data) {
 
 	// Setup the payment details.
 	$payment_data = array(
 		'price'           => $purchase_data['price'],
 		'give_form_title' => $purchase_data['post_data']['give-form-title'],
-		'give_form_id'    => intval( $purchase_data['post_data']['give-form-id'] ),
-		'give_price_id'   => isset( $purchase_data['post_data']['give-price-id'] ) ? $purchase_data['post_data']['give-price-id'] : '',
+		'give_form_id'    => intval($purchase_data['post_data']['give-form-id']),
+		'give_price_id'   => isset($purchase_data['post_data']['give-price-id']) ? $purchase_data['post_data']['give-price-id'] : '',
 		'date'            => $purchase_data['date'],
 		'user_email'      => $purchase_data['user_email'],
 		'purchase_key'    => $purchase_data['purchase_key'],
-		'currency'        => give_get_currency( $purchase_data['post_data']['give-form-id'], $purchase_data ),
+		'currency'        => give_get_currency($purchase_data['post_data']['give-form-id'], $purchase_data),
 		'user_info'       => $purchase_data['user_info'],
 		'status'          => 'pending',
 		'gateway'         => 'offline'
@@ -124,20 +124,20 @@ 
 
 
 	// record the pending payment
-	$payment = give_insert_payment( $payment_data );
+	$payment = give_insert_payment($payment_data);
 
-	if ( $payment ) {
-		give_offline_send_admin_notice( $payment );
-		give_offline_send_donor_instructions( $payment );
+	if ($payment) {
+		give_offline_send_admin_notice($payment);
+		give_offline_send_donor_instructions($payment);
 		give_send_to_success_page();
 	} else {
 		// if errors are present, send the user back to the donation form so they can be corrected
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
 	}
 
 }
 
-add_action( 'give_gateway_offline', 'give_offline_process_payment' );
+add_action('give_gateway_offline', 'give_offline_process_payment');
 
 
 /**
@@ -150,59 +150,59 @@ 
  * @since       1.0
  * @return void
  */
-function give_offline_send_donor_instructions( $payment_id = 0 ) {
+function give_offline_send_donor_instructions($payment_id = 0) {
 
-	$payment_data                      = give_get_payment_meta( $payment_id );
-	$post_offline_customization_option = give_get_meta( $payment_data['form_id'], '_give_customize_offline_donations', true );
+	$payment_data                      = give_get_payment_meta($payment_id);
+	$post_offline_customization_option = give_get_meta($payment_data['form_id'], '_give_customize_offline_donations', true);
 
 	//Customize email content depending on whether the single form has been customized
-	$email_content = give_get_option( 'global_offline_donation_email' );
+	$email_content = give_get_option('global_offline_donation_email');
 
-	if ( give_is_setting_enabled( $post_offline_customization_option, 'enabled' ) ) {
-		$email_content = give_get_meta( $payment_data['form_id'], '_give_offline_donation_email', true );
+	if (give_is_setting_enabled($post_offline_customization_option, 'enabled')) {
+		$email_content = give_get_meta($payment_data['form_id'], '_give_offline_donation_email', true);
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
 	/**
 	 * Filters the from name.
 	 *
 	 * @since 1.7
 	 */
-	$from_name = apply_filters( 'give_donation_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = apply_filters('give_donation_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/**
 	 * Filters the from email.
 	 *
 	 * @since 1.7
 	 */
-	$from_email = apply_filters( 'give_donation_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = apply_filters('give_donation_from_address', $from_email, $payment_id, $payment_data);
 
-	$to_email = give_get_payment_user_email( $payment_id );
+	$to_email = give_get_payment_user_email($payment_id);
 
-	$subject = give_get_option( 'offline_donation_subject', __( 'Offline Donation Instructions', 'give' ) );
-	if ( give_is_setting_enabled( $post_offline_customization_option, 'enabled' ) ) {
-		$subject = give_get_meta( $payment_data['form_id'], '_give_offline_donation_subject', true );
+	$subject = give_get_option('offline_donation_subject', __('Offline Donation Instructions', 'give'));
+	if (give_is_setting_enabled($post_offline_customization_option, 'enabled')) {
+		$subject = give_get_meta($payment_data['form_id'], '_give_offline_donation_subject', true);
 	}
 
-	$subject = apply_filters( 'give_offline_donation_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = apply_filters('give_offline_donation_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
-	$attachments = apply_filters( 'give_offline_donation_attachments', array(), $payment_id, $payment_data );
-	$message     = give_do_email_tags( $email_content, $payment_id );
+	$attachments = apply_filters('give_offline_donation_attachments', array(), $payment_id, $payment_data);
+	$message     = give_do_email_tags($email_content, $payment_id);
 
 	$emails = Give()->emails;
 
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Offline Donation Instructions', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Offline Donation Instructions', 'give'));
 
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data);
+	$emails->__set('headers', $headers);
 
-	$emails->send( $to_email, $subject, $message, $attachments );
+	$emails->send($to_email, $subject, $message, $attachments);
 
 }
 
@@ -219,54 +219,54 @@ 
  * @return void
  *
  */
-function give_offline_send_admin_notice( $payment_id = 0 ) {
+function give_offline_send_admin_notice($payment_id = 0) {
 
 	/* Send an email notification to the admin */
 	$admin_email = give_get_admin_notice_emails();
-	$user_info   = give_get_payment_meta_user_info( $payment_id );
+	$user_info   = give_get_payment_meta_user_info($payment_id);
 
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 ) {
-		$user_data = get_userdata( $user_info['id'] );
+	if (isset($user_info['id']) && $user_info['id'] > 0) {
+		$user_data = get_userdata($user_info['id']);
 		$name      = $user_data->display_name;
-	} elseif ( isset( $user_info['first_name'] ) && isset( $user_info['last_name'] ) ) {
-		$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
+	} elseif (isset($user_info['first_name']) && isset($user_info['last_name'])) {
+		$name = $user_info['first_name'].' '.$user_info['last_name'];
 	} else {
 		$name = $user_info['email'];
 	}
 
-	$amount = give_donation_amount( $payment_id );
+	$amount = give_donation_amount($payment_id);
 
-	$admin_subject = apply_filters( 'give_offline_admin_donation_notification_subject', __( 'New Pending Donation', 'give' ), $payment_id );
+	$admin_subject = apply_filters('give_offline_admin_donation_notification_subject', __('New Pending Donation', 'give'), $payment_id);
 
-	$admin_message = __( 'Dear Admin,', 'give' ) . "\n\n";
-	$admin_message .= sprintf(__( 'A new offline donation has been made on your website for %s.', 'give' ), $amount) . "\n\n";
-	$admin_message .= __( 'The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give' ) . "\n\n";
+	$admin_message = __('Dear Admin,', 'give')."\n\n";
+	$admin_message .= sprintf(__('A new offline donation has been made on your website for %s.', 'give'), $amount)."\n\n";
+	$admin_message .= __('The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give')."\n\n";
 
 
-	$admin_message .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
-	$admin_message .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n\n";
+	$admin_message .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
+	$admin_message .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n\n";
 
 	$admin_message .= sprintf(
 		                  '<a href="%1$s">%2$s</a>',
-		                  admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id ),
-		                  __( 'View Donation Details &raquo;', 'give' )
-	                  ) . "\n\n";
+		                  admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment_id),
+		                  __('View Donation Details &raquo;', 'give')
+	                  )."\n\n";
 
-	$admin_message = apply_filters( 'give_offline_admin_donation_notification', $admin_message, $payment_id );
-	$admin_message = give_do_email_tags( $admin_message, $payment_id );
+	$admin_message = apply_filters('give_offline_admin_donation_notification', $admin_message, $payment_id);
+	$admin_message = give_do_email_tags($admin_message, $payment_id);
 
-	$attachments   = apply_filters( 'give_offline_admin_donation_notification_attachments', array(), $payment_id );
-	$admin_headers = apply_filters( 'give_offline_admin_donation_notification_headers', array(), $payment_id );
+	$attachments   = apply_filters('give_offline_admin_donation_notification_attachments', array(), $payment_id);
+	$admin_headers = apply_filters('give_offline_admin_donation_notification_headers', array(), $payment_id);
 
 	//Send Email
 	$emails = Give()->emails;
-	$emails->__set( 'heading', __( 'New Offline Donation', 'give' ) );
+	$emails->__set('heading', __('New Offline Donation', 'give'));
 
-	if ( ! empty( $admin_headers ) ) {
-		$emails->__set( 'headers', $admin_headers );
+	if ( ! empty($admin_headers)) {
+		$emails->__set('headers', $admin_headers);
 	}
 
-	$emails->send( $admin_email, $admin_subject, $admin_message, $attachments );
+	$emails->send($admin_email, $admin_subject, $admin_message, $attachments);
 
 }
 
@@ -278,20 +278,20 @@ 
  *
  * @return array
  */
-function give_offline_add_settings( $settings ) {
+function give_offline_add_settings($settings) {
 
 	// Bailout: Do not show offline gateways setting in to metabox if its disabled globally.
-	if ( in_array( 'offline', (array) give_get_option( 'gateways' ) ) ) {
+	if (in_array('offline', (array) give_get_option('gateways'))) {
 		return $settings;
 	}
 
 	//Vars
 	$prefix = '_give_';
 
-	$is_gateway_active = give_is_gateway_active( 'offline' );
+	$is_gateway_active = give_is_gateway_active('offline');
 
 	//this gateway isn't active
-	if ( ! $is_gateway_active ) {
+	if ( ! $is_gateway_active) {
 		//return settings and bounce
 		return $settings;
 	}
@@ -300,34 +300,34 @@ 
 	$check_settings = array(
 
 		array(
-			'name'    => __( 'Offline Donations', 'give' ),
-			'desc'    => __( 'Do you want to customize the donation instructions for this form?', 'give' ),
-			'id'      => $prefix . 'customize_offline_donations',
+			'name'    => __('Offline Donations', 'give'),
+			'desc'    => __('Do you want to customize the donation instructions for this form?', 'give'),
+			'id'      => $prefix.'customize_offline_donations',
 			'type'    => 'radio_inline',
 			'default' => 'global',
-			'options' => apply_filters( 'give_forms_content_options_select', array(
-					'global'   => __( 'Global Option', 'give' ),
-					'enabled'  => __( 'Customize', 'give' ),
-					'disabled' => __( 'Disable', 'give' ),
+			'options' => apply_filters('give_forms_content_options_select', array(
+					'global'   => __('Global Option', 'give'),
+					'enabled'  => __('Customize', 'give'),
+					'disabled' => __('Disable', 'give'),
 				)
 			),
 		),
 		array(
-			'name'        => __( 'Billing Fields', 'give' ),
-			'desc'        => __( 'This option will enable the billing details section for this form\'s offline donation payment gateway. The fieldset will appear above the offline donation instructions.', 'give' ),
-			'id'          => $prefix . 'offline_donation_enable_billing_fields_single',
+			'name'        => __('Billing Fields', 'give'),
+			'desc'        => __('This option will enable the billing details section for this form\'s offline donation payment gateway. The fieldset will appear above the offline donation instructions.', 'give'),
+			'id'          => $prefix.'offline_donation_enable_billing_fields_single',
 			'row_classes' => 'give-subfield give-hidden',
 			'type'        => 'radio_inline',
 			'default'     => 'disabled',
 			'options'     => array(
-				'enabled'  => __( 'Enabled', 'give' ),
-				'disabled' => __( 'Disabled', 'give' ),
+				'enabled'  => __('Enabled', 'give'),
+				'disabled' => __('Disabled', 'give'),
 			),
 		),
 		array(
-			'id'          => $prefix . 'offline_checkout_notes',
-			'name'        => __( 'Donation Instructions', 'give' ),
-			'desc'        => __( 'Enter the instructions you want to display to the donor during the donation process. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ),
+			'id'          => $prefix.'offline_checkout_notes',
+			'name'        => __('Donation Instructions', 'give'),
+			'desc'        => __('Enter the instructions you want to display to the donor during the donation process. Most likely this would include important information like mailing address and who to make the check out to.', 'give'),
 			'default'     => give_get_default_offline_donation_content(),
 			'type'        => 'wysiwyg',
 			'row_classes' => 'give-subfield give-hidden',
@@ -336,17 +336,17 @@ 
 			),
 		),
 		array(
-			'id'          => $prefix . 'offline_donation_subject',
-			'name'        => __( 'Email Subject', 'give' ),
-			'desc'        => __( 'Enter the subject line for the donation receipt email.', 'give' ),
-			'default'     => __( '{form_title} - Offline Donation Instructions', 'give' ),
+			'id'          => $prefix.'offline_donation_subject',
+			'name'        => __('Email Subject', 'give'),
+			'desc'        => __('Enter the subject line for the donation receipt email.', 'give'),
+			'default'     => __('{form_title} - Offline Donation Instructions', 'give'),
 			'row_classes' => 'give-subfield give-hidden',
 			'type'        => 'text',
 		),
 		array(
-			'id'          => $prefix . 'offline_donation_email',
-			'name'        => __( 'Email Instructions', 'give' ),
-			'desc'        => __( 'Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ) . ' ' . __( 'Available template tags:', 'give' ) . give_get_emails_tags_list(),
+			'id'          => $prefix.'offline_donation_email',
+			'name'        => __('Email Instructions', 'give'),
+			'desc'        => __('Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give').' '.__('Available template tags:', 'give').give_get_emails_tags_list(),
 			'default'     => give_get_default_offline_donation_email_content(),
 			'type'        => 'wysiwyg',
 			'row_classes' => 'give-subfield give-hidden',
@@ -358,14 +358,14 @@ 
 			'name'  => 'offline_docs',
 			'type'  => 'docs_link',
 			'url'   => 'http://docs.givewp.com/settings-gateway-offline-donations',
-			'title' => __( 'Offline Donations', 'give' ),
+			'title' => __('Offline Donations', 'give'),
 		),
 	);
 
-	return array_merge( $settings, $check_settings );
+	return array_merge($settings, $check_settings);
 }
 
-add_filter( 'give_forms_offline_donations_metabox_fields', 'give_offline_add_settings' );
+add_filter('give_forms_offline_donations_metabox_fields', 'give_offline_add_settings');
 
 
 /**
@@ -377,32 +377,32 @@ 
  */
 function give_get_default_offline_donation_content() {
 
-	$sitename = get_bloginfo( 'sitename' );
+	$sitename = get_bloginfo('sitename');
 
-	$default_text = '<p>' . __( 'In order to make an offline donation we ask that you please follow these instructions', 'give' ) . ': </p>';
+	$default_text = '<p>'.__('In order to make an offline donation we ask that you please follow these instructions', 'give').': </p>';
 	$default_text .= '<ol>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'Make a check payable to "%s"', 'give' ),
+		__('Make a check payable to "%s"', 'give'),
 		$sitename
 	);
 	$default_text .= '</li>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'On the memo line of the check, please indicate that the donation is for "%s"', 'give' ),
+		__('On the memo line of the check, please indicate that the donation is for "%s"', 'give'),
 		$sitename
 	);
 	$default_text .= '</li>';
-	$default_text .= '<li>' . __( 'Please mail your check to:', 'give' ) . '</li>';
+	$default_text .= '<li>'.__('Please mail your check to:', 'give').'</li>';
 	$default_text .= '</ol>';
-	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>' . $sitename . '</em><br>';
+	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>'.$sitename.'</em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>111 Not A Real St.</em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>Anytown, CA 12345 </em><br>';
-	$default_text .= '<p>' . __( 'All contributions will be gratefully acknowledged and are tax deductible.', 'give' ) . '</p>';
+	$default_text .= '<p>'.__('All contributions will be gratefully acknowledged and are tax deductible.', 'give').'</p>';
 
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
+	return apply_filters('give_default_offline_donation_content', $default_text);
 
 }
 
@@ -415,34 +415,34 @@ 
  */
 function give_get_default_offline_donation_email_content() {
 
-	$sitename     = get_bloginfo( 'sitename' );
-	$default_text = '<p>' . __( 'Dear {name},', 'give' ) . '</p>';
-	$default_text .= '<p>' . __( 'Thank you for your offline donation request! Your generosity is greatly appreciated. In order to make an offline donation we ask that you please follow these instructions:', 'give' ) . '</p>';
+	$sitename     = get_bloginfo('sitename');
+	$default_text = '<p>'.__('Dear {name},', 'give').'</p>';
+	$default_text .= '<p>'.__('Thank you for your offline donation request! Your generosity is greatly appreciated. In order to make an offline donation we ask that you please follow these instructions:', 'give').'</p>';
 	$default_text .= '<ol>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'Make a check payable to "%s"', 'give' ),
+		__('Make a check payable to "%s"', 'give'),
 		$sitename
 	);
 	$default_text .= '</li>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'On the memo line of the check, please indicate that the donation is for "%s"', 'give' ),
+		__('On the memo line of the check, please indicate that the donation is for "%s"', 'give'),
 		$sitename
 	);
 	$default_text .= '</li>';
-	$default_text .= '<li>' . __( 'Please mail your check to:', 'give' ) . '</li>';
+	$default_text .= '<li>'.__('Please mail your check to:', 'give').'</li>';
 	$default_text .= '</ol>';
-	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>' . $sitename . '</em><br>';
+	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>'.$sitename.'</em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>111 Not A Real St.</em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>Anytown, CA 12345 </em><br>';
-	$default_text .= '<p>' . __( 'Once your donation has been received we will mark it as complete and you will receive an email receipt for your records. Please contact us with any questions you may have!', 'give' ) . '</p>';
-	$default_text .= '<p>' . __( 'Sincerely,', 'give' ) . '</p>';
-	$default_text .= '<p>' . $sitename . '</p>';
+	$default_text .= '<p>'.__('Once your donation has been received we will mark it as complete and you will receive an email receipt for your records. Please contact us with any questions you may have!', 'give').'</p>';
+	$default_text .= '<p>'.__('Sincerely,', 'give').'</p>';
+	$default_text .= '<p>'.$sitename.'</p>';
 
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
+	return apply_filters('give_default_offline_donation_content', $default_text);
 
 }
 
@@ -456,17 +456,17 @@ 
  *
  * @return string
  */
-function give_offline_donation_receipt_status_notice( $notice, $id ) {
-	$payment = new Give_Payment( $id );
+function give_offline_donation_receipt_status_notice($notice, $id) {
+	$payment = new Give_Payment($id);
 
-	if ( 'offline' !== $payment->gateway || $payment->is_completed() ) {
+	if ('offline' !== $payment->gateway || $payment->is_completed()) {
 		return $notice;
 	}
 
-	return Give()->notices->print_frontend_notice( __( 'Payment Pending: Please follow the instructions below to complete your donation.', 'give' ), false, 'warning' );
+	return Give()->notices->print_frontend_notice(__('Payment Pending: Please follow the instructions below to complete your donation.', 'give'), false, 'warning');
 }
 
-add_filter( 'give_receipt_status_notice', 'give_offline_donation_receipt_status_notice', 10, 2 );
+add_filter('give_receipt_status_notice', 'give_offline_donation_receipt_status_notice', 10, 2);
 
 /**
  * Get offline payment instructions.
@@ -478,27 +478,27 @@ 
  *
  * @return string
  */
-function give_get_offline_payment_instruction( $form_id, $wpautop = false ) {
+function give_get_offline_payment_instruction($form_id, $wpautop = false) {
 	// Bailout.
-	if ( ! $form_id ) {
+	if ( ! $form_id) {
 		return '';
 	}
 
-	$post_offline_customization_option = give_get_meta( $form_id, '_give_customize_offline_donations', true );
-	$post_offline_instructions         = give_get_meta( $form_id, '_give_offline_checkout_notes', true );
-	$global_offline_instruction        = give_get_option( 'global_offline_donation_content' );
+	$post_offline_customization_option = give_get_meta($form_id, '_give_customize_offline_donations', true);
+	$post_offline_instructions         = give_get_meta($form_id, '_give_offline_checkout_notes', true);
+	$global_offline_instruction        = give_get_option('global_offline_donation_content');
 	$offline_instructions              = $global_offline_instruction;
 
-	if ( give_is_setting_enabled( $post_offline_customization_option ) ) {
+	if (give_is_setting_enabled($post_offline_customization_option)) {
 		$offline_instructions = $post_offline_instructions;
 	}
 
-	$settings_url = admin_url( 'post.php?post=' . $form_id . '&action=edit&message=1' );
+	$settings_url = admin_url('post.php?post='.$form_id.'&action=edit&message=1');
 
 	/* translators: %s: form settings url */
-	$offline_instructions = ! empty( $offline_instructions ) ? $offline_instructions : sprintf( __( 'Please enter offline donation instructions in <a href="%s">this form\'s settings</a>.', 'give' ), $settings_url );
+	$offline_instructions = ! empty($offline_instructions) ? $offline_instructions : sprintf(__('Please enter offline donation instructions in <a href="%s">this form\'s settings</a>.', 'give'), $settings_url);
 
-	return ( $wpautop ? wpautop( $offline_instructions ) : $offline_instructions );
+	return ($wpautop ? wpautop($offline_instructions) : $offline_instructions);
 }
 
 
@@ -512,24 +512,24 @@ 
  *
  * @return array
  */
-function give_filter_offline_gateway( $gateway_list, $form_id ) {
+function give_filter_offline_gateway($gateway_list, $form_id) {
 	if (
 		// Show offline payment gateway if enable for new donation form.
-		( false === strpos( $_SERVER['REQUEST_URI'], '/wp-admin/post-new.php?post_type=give_forms' ) )
+		(false === strpos($_SERVER['REQUEST_URI'], '/wp-admin/post-new.php?post_type=give_forms'))
 		&& $form_id
-		&& ! give_is_setting_enabled( give_get_meta( $form_id, '_give_customize_offline_donations', true ), array(
+		&& ! give_is_setting_enabled(give_get_meta($form_id, '_give_customize_offline_donations', true), array(
 			'enabled',
 			'global',
-		) )
+		))
 	) {
-		unset( $gateway_list['offline'] );
+		unset($gateway_list['offline']);
 	}
 
 	// Output.
 	return $gateway_list;
 }
 
-add_filter( 'give_enabled_payment_gateways', 'give_filter_offline_gateway', 10, 2 );
+add_filter('give_enabled_payment_gateways', 'give_filter_offline_gateway', 10, 2);
 
 /**
  * Set default gateway to global default payment gateway
@@ -543,10 +543,10 @@ 
  *
  * @return void
  */
-function _give_customize_offline_donations_on_save_callback( $meta_key, $meta_value, $postid ) {
-	if ( ! give_is_setting_enabled( $meta_value, array( 'global', 'enabled' ) ) && ( 'offline' === give_get_meta( $postid, '_give_default_gateway', true ) ) ) {
-		give_update_meta( $postid, '_give_default_gateway', 'global' );
+function _give_customize_offline_donations_on_save_callback($meta_key, $meta_value, $postid) {
+	if ( ! give_is_setting_enabled($meta_value, array('global', 'enabled')) && ('offline' === give_get_meta($postid, '_give_default_gateway', true))) {
+		give_update_meta($postid, '_give_default_gateway', 'global');
 	}
 }
 
-add_filter( 'give_save__give_customize_offline_donations', '_give_customize_offline_donations_on_save_callback', 10, 3 );
+add_filter('give_save__give_customize_offline_donations', '_give_customize_offline_donations_on_save_callback', 10, 3);

includes/admin/tools/export/export-functions.php

Spacing +27 added lines, -28 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_do_ajax_export() {
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export.php';
 
-	parse_str( $_POST['form'], $form );
+	parse_str($_POST['form'], $form);
 
 	$_REQUEST = $form = (array) $form;
 
-	if ( ! wp_verify_nonce( $_REQUEST['give_ajax_export'], 'give_ajax_export' ) ) {
-		die( '-2' );
+	if ( ! wp_verify_nonce($_REQUEST['give_ajax_export'], 'give_ajax_export')) {
+		die('-2');
 	}
 
 	/**
@@ -42,28 +42,28 @@ 
 	 *
 	 * @param string $class Export class.
 	 */
-	do_action( 'give_batch_export_class_include', $form['give-export-class'] );
+	do_action('give_batch_export_class_include', $form['give-export-class']);
 
-	$step   = absint( $_POST['step'] );
-	$class  = sanitize_text_field( $form['give-export-class'] );
+	$step   = absint($_POST['step']);
+	$class  = sanitize_text_field($form['give-export-class']);
 
 	/* @var Give_Batch_Export $export */
-	$export = new $class( $step );
+	$export = new $class($step);
 
-	if ( ! $export->can_export() ) {
-		die( '-1' );
+	if ( ! $export->can_export()) {
+		die('-1');
 	}
 
-	if ( ! $export->is_writable ) {
+	if ( ! $export->is_writable) {
 		$json_args = array(
 			'error'   => true,
-			'message' => esc_html__( 'Export location or file not writable.', 'give' )
+			'message' => esc_html__('Export location or file not writable.', 'give')
 		);
 		echo json_encode($json_args);
 		exit;
 	}
 
-	$export->set_properties( give_clean( $_REQUEST ) );
+	$export->set_properties(give_clean($_REQUEST));
 
 	$export->pre_fetch();
 
@@ -71,7 +71,7 @@ 
 
 	$percentage = $export->get_percentage_complete();
 
-	if ( $ret ) {
+	if ($ret) {
 
 		$step += 1;
 		$json_data = array(
@@ -79,18 +79,17 @@ 
 			'percentage' => $percentage
 		);
 
-	} elseif ( true === $export->is_empty ) {
+	} elseif (true === $export->is_empty) {
 
 		$json_data = array(
 			'error'   => true,
-			'message' => esc_html__( 'No data found for export parameters.', 'give' )
+			'message' => esc_html__('No data found for export parameters.', 'give')
 		);
 
-	} elseif ( true === $export->done && true === $export->is_void ) {
+	} elseif (true === $export->done && true === $export->is_void) {
 
-		$message = ! empty( $export->message ) ?
-			$export->message :
-			esc_html__( 'Batch Processing Complete', 'give' );
+		$message = ! empty($export->message) ?
+			$export->message : esc_html__('Batch Processing Complete', 'give');
 
 		$json_data = array(
 			'success' => true,
@@ -99,23 +98,23 @@ 
 
 	} else {
 		
-		$args = array_merge( $_REQUEST, array(
+		$args = array_merge($_REQUEST, array(
 			'step'        => $step,
 			'class'       => $class,
-			'nonce'       => wp_create_nonce( 'give-batch-export' ),
+			'nonce'       => wp_create_nonce('give-batch-export'),
 			'give_action' => 'form_batch_export',
-		) );
+		));
 
 		$json_data = array(
 			'step' => 'done',
-			'url' => add_query_arg( $args, admin_url() )
+			'url' => add_query_arg($args, admin_url())
 		);
 
 	}
 
-	$export->unset_properties( give_clean( $_REQUEST ), $export );
-	echo json_encode( $json_data );
+	$export->unset_properties(give_clean($_REQUEST), $export);
+	echo json_encode($json_data);
 	exit;
 }
 
-add_action( 'wp_ajax_give_do_ajax_export', 'give_do_ajax_export' );
+add_action('wp_ajax_give_do_ajax_export', 'give_do_ajax_export');

includes/admin/tools/export/class-batch-export-donors.php

Indentation +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 
 					// Continue if donor already included.
 					if ( empty( $payment->customer_id ) ||
-					     in_array( $payment->customer_id, $cached_donor_ids )
+						 in_array( $payment->customer_id, $cached_donor_ids )
 					) {
 						continue;
 					}

Spacing +103 added lines, -105 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -70,23 +70,23 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 
 		// Set data from form submission
-		if ( isset( $_POST['form'] ) ) {
-			parse_str( $_POST['form'], $this->data );
+		if (isset($_POST['form'])) {
+			parse_str($_POST['form'], $this->data);
 		}
 
 		$this->form = $this->data['forms'];
 
 		// Setup donor ids cache.
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 			// Cache donor ids to output unique list of donor.
-			$this->query_id = give_clean( $_REQUEST['give_export_option']['query_id'] );
+			$this->query_id = give_clean($_REQUEST['give_export_option']['query_id']);
 			$this->cache_donor_ids();
 		}
 
-		$this->price_id = ! empty( $request['give_price_option'] ) && 'all' !== $request['give_price_option'] ? absint( $request['give_price_option'] ) : null;
+		$this->price_id = ! empty($request['give_price_option']) && 'all' !== $request['give_price_option'] ? absint($request['give_price_option']) : null;
 
 	}
 
@@ -101,12 +101,12 @@ 
 		// Fetch already cached donor ids.
 		$donor_ids = $this->donor_ids;
 
-		if ( $cached_donor_ids = Give_Cache::get( $this->query_id, true ) ) {
-			$donor_ids = array_unique( array_merge( $cached_donor_ids, $this->donor_ids ) );
+		if ($cached_donor_ids = Give_Cache::get($this->query_id, true)) {
+			$donor_ids = array_unique(array_merge($cached_donor_ids, $this->donor_ids));
 		}
 
-		$donor_ids = array_values( $donor_ids );
-		Give_Cache::set( $this->query_id, $donor_ids, HOUR_IN_SECONDS, true );
+		$donor_ids = array_values($donor_ids);
+		Give_Cache::set($this->query_id, $donor_ids, HOUR_IN_SECONDS, true);
 	}
 
 	/**
@@ -118,14 +118,14 @@ 
 	 */
 	public function csv_cols() {
 
-		$columns = isset( $this->data['give_export_option'] ) ? $this->data['give_export_option'] : array();
+		$columns = isset($this->data['give_export_option']) ? $this->data['give_export_option'] : array();
 
 		// We need columns.
-		if ( empty( $columns ) ) {
+		if (empty($columns)) {
 			return false;
 		}
 
-		$cols = $this->get_cols( $columns );
+		$cols = $this->get_cols($columns);
 
 		return $cols;
 	}
@@ -137,41 +137,41 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_cols( $columns ) {
+	private function get_cols($columns) {
 
 		$cols = array();
 
-		foreach ( $columns as $key => $value ) {
+		foreach ($columns as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 				case 'full_name' :
-					$cols['full_name'] = esc_html__( 'Full Name', 'give' );
+					$cols['full_name'] = esc_html__('Full Name', 'give');
 					break;
 				case 'email' :
-					$cols['email'] = esc_html__( 'Email Address', 'give' );
+					$cols['email'] = esc_html__('Email Address', 'give');
 					break;
 				case 'address' :
-					$cols['address_line1']   = esc_html__( 'Address', 'give' );
-					$cols['address_line2']   = esc_html__( 'Address 2', 'give' );
-					$cols['address_city']    = esc_html__( 'City', 'give' );
-					$cols['address_state']   = esc_html__( 'State', 'give' );
-					$cols['address_zip']     = esc_html__( 'Zip', 'give' );
-					$cols['address_country'] = esc_html__( 'Country', 'give' );
+					$cols['address_line1']   = esc_html__('Address', 'give');
+					$cols['address_line2']   = esc_html__('Address 2', 'give');
+					$cols['address_city']    = esc_html__('City', 'give');
+					$cols['address_state']   = esc_html__('State', 'give');
+					$cols['address_zip']     = esc_html__('Zip', 'give');
+					$cols['address_country'] = esc_html__('Country', 'give');
 					break;
 				case 'userid' :
-					$cols['userid'] = esc_html__( 'User ID', 'give' );
+					$cols['userid'] = esc_html__('User ID', 'give');
 					break;
 				case 'donation_form' :
-					$cols['donation_form'] = esc_html__( 'Donation Form', 'give' );
+					$cols['donation_form'] = esc_html__('Donation Form', 'give');
 					break;
 				case 'date_first_donated' :
-					$cols['date_first_donated'] = esc_html__( 'First Donation Date', 'give' );
+					$cols['date_first_donated'] = esc_html__('First Donation Date', 'give');
 					break;
 				case 'donations' :
-					$cols['donations'] = esc_html__( 'Number of Donations', 'give' );
+					$cols['donations'] = esc_html__('Number of Donations', 'give');
 					break;
 				case 'donation_sum' :
-					$cols['donation_sum'] = esc_html__( 'Sum of Donations', 'give' );
+					$cols['donation_sum'] = esc_html__('Sum of Donations', 'give');
 					break;
 			}
 		}
@@ -192,29 +192,29 @@ 
 		$i = 0;
 
 		$data             = array();
-		$cached_donor_ids = Give_Cache::get( $this->query_id, true );
+		$cached_donor_ids = Give_Cache::get($this->query_id, true);
 
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 
 			// Export donors for a specific donation form and also within specified timeframe
 			$args = array(
 				'output'     => 'payments', // Use 'posts' to get standard post objects
-				'post_type'  => array( 'give_payment' ),
+				'post_type'  => array('give_payment'),
 				'number'     => 30,
 				'paged'      => $this->step,
 				'status'     => 'publish',
 				'meta_key'   => '_give_payment_form_id',
-				'meta_value' => absint( $this->form ),
+				'meta_value' => absint($this->form),
 			);
 
 			// Check for date option filter
-			if ( ! empty( $this->data['donor_export_start_date'] ) || ! empty( $this->data['donor_export_end_date'] ) ) {
-				$args['start_date'] = ! empty( $this->data['donor_export_start_date'] ) ? date( 'Y-n-d 00:00:00', strtotime( $this->data['donor_export_start_date'] ) ) : date( 'Y-n-d 23:59:59', '1970-1-01 00:00:00' );
-				$args['end_date']   = ! empty( $this->data['donor_export_end_date'] ) ? date( 'Y-n-d 23:59:59', strtotime( $this->data['donor_export_end_date'] ) ) : date( 'Y-n-d 23:59:59', current_time( 'timestamp' ) );
+			if ( ! empty($this->data['donor_export_start_date']) || ! empty($this->data['donor_export_end_date'])) {
+				$args['start_date'] = ! empty($this->data['donor_export_start_date']) ? date('Y-n-d 00:00:00', strtotime($this->data['donor_export_start_date'])) : date('Y-n-d 23:59:59', '1970-1-01 00:00:00');
+				$args['end_date']   = ! empty($this->data['donor_export_end_date']) ? date('Y-n-d 23:59:59', strtotime($this->data['donor_export_end_date'])) : date('Y-n-d 23:59:59', current_time('timestamp'));
 			}
 
 			// Check for price option.
-			if ( null !== $this->price_id ) {
+			if (null !== $this->price_id) {
 				$args['meta_query'] = array(
 					array(
 						'key'   => '_give_payment_price_id',
@@ -223,45 +223,43 @@ 
 				);
 			}
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 
-			if ( $payments ) {
+			if ($payments) {
 				/* @var Give_Payment $payment */
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 					// Set donation sum.
-					$this->payment_stats[ $payment->customer_id ]['donation_sum'] = isset( $this->payment_stats[ $payment->customer_id ]['donation_sum'] ) ?
-						$this->payment_stats[ $payment->customer_id ]['donation_sum'] :
-						0;
-					$this->payment_stats[ $payment->customer_id ]['donation_sum'] += $payment->total;
+					$this->payment_stats[$payment->customer_id]['donation_sum'] = isset($this->payment_stats[$payment->customer_id]['donation_sum']) ?
+						$this->payment_stats[$payment->customer_id]['donation_sum'] : 0;
+					$this->payment_stats[$payment->customer_id]['donation_sum'] += $payment->total;
 
 					// Set donation count.
-					$this->payment_stats[ $payment->customer_id ]['donations'] = isset( $this->payment_stats[ $payment->customer_id ]['donations'] ) ?
-						++ $this->payment_stats[ $payment->customer_id ]['donations'] :
-						1;
+					$this->payment_stats[$payment->customer_id]['donations'] = isset($this->payment_stats[$payment->customer_id]['donations']) ?
+						++$this->payment_stats[$payment->customer_id]['donations'] : 1;
 
 					// Set donation form name.
-					$this->payment_stats[ $payment->customer_id ]['form_title'] = $payment->form_title;
+					$this->payment_stats[$payment->customer_id]['form_title'] = $payment->form_title;
 
 					// Continue if donor already included.
-					if ( empty( $payment->customer_id ) ||
-					     in_array( $payment->customer_id, $cached_donor_ids )
+					if (empty($payment->customer_id) ||
+					     in_array($payment->customer_id, $cached_donor_ids)
 					) {
 						continue;
 					}
 
 					$this->donor_ids[] = $cached_donor_ids[] = $payment->customer_id;
 
-					$i ++;
+					$i++;
 				}
 
-				if ( ! empty( $this->donor_ids ) ) {
-					foreach ( $this->donor_ids as $donor_id ) {
-						$donor                      = Give()->donors->get_donor_by( 'id', $donor_id );
-						$donor->donation_form_title = $this->payment_stats[ $donor_id ]['form_title'];
-						$donor->purchase_count      = $this->payment_stats[ $donor_id ]['donations'];
-						$donor->purchase_value      = $this->payment_stats[ $donor_id ]['donation_sum'];
-						$data[]                     = $this->set_donor_data( $i, $data, $donor );
+				if ( ! empty($this->donor_ids)) {
+					foreach ($this->donor_ids as $donor_id) {
+						$donor                      = Give()->donors->get_donor_by('id', $donor_id);
+						$donor->donation_form_title = $this->payment_stats[$donor_id]['form_title'];
+						$donor->purchase_count      = $this->payment_stats[$donor_id]['donations'];
+						$donor->purchase_value      = $this->payment_stats[$donor_id]['donation_sum'];
+						$data[]                     = $this->set_donor_data($i, $data, $donor);
 					}
 
 					// Cache donor ids only if admin export donor for specific form.
@@ -271,7 +269,7 @@ 
 		} else {
 
 			// Export all donors.
-			$offset = 30 * ( $this->step - 1 );
+			$offset = 30 * ($this->step - 1);
 
 			$args = array(
 				'number' => 30,
@@ -279,31 +277,31 @@ 
 			);
 
 			// Check for date option filter
-			if ( ! empty( $this->data['donor_export_start_date'] ) || ! empty( $this->data['donor_export_end_date'] ) ) {
+			if ( ! empty($this->data['donor_export_start_date']) || ! empty($this->data['donor_export_end_date'])) {
 				$args['date'] = array(
-					'start' => ! empty( $this->data['donor_export_start_date'] ) ? date( 'Y-n-d 00:00:00', strtotime( $this->data['donor_export_start_date'] ) ) : date( 'Y-n-d 23:59:59', '1970-1-01 00:00:00' ),
-					'end'   => ! empty( $this->data['donor_export_end_date'] ) ? date( 'Y-n-d 23:59:59', strtotime( $this->data['donor_export_end_date'] ) ) : date( 'Y-n-d 23:59:59', current_time( 'timestamp' ) ),
+					'start' => ! empty($this->data['donor_export_start_date']) ? date('Y-n-d 00:00:00', strtotime($this->data['donor_export_start_date'])) : date('Y-n-d 23:59:59', '1970-1-01 00:00:00'),
+					'end'   => ! empty($this->data['donor_export_end_date']) ? date('Y-n-d 23:59:59', strtotime($this->data['donor_export_end_date'])) : date('Y-n-d 23:59:59', current_time('timestamp')),
 				);
 			}
 
-			$donors = Give()->donors->get_donors( $args );
+			$donors = Give()->donors->get_donors($args);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
 				// Continue if donor already included.
-				if ( empty( $donor->id ) || empty( $donor->payment_ids ) ) {
+				if (empty($donor->id) || empty($donor->payment_ids)) {
 					continue;
 				}
 
-				$payment                    = new Give_Payment( $donor->payment_ids );
+				$payment                    = new Give_Payment($donor->payment_ids);
 				$donor->donation_form_title = $payment->form_title;
-				$data[]                     = $this->set_donor_data( $i, $data, $donor );
-				$i ++;
+				$data[]                     = $this->set_donor_data($i, $data, $donor);
+				$i++;
 			}
 		}// End if().
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 		return $data;
 	}
@@ -319,18 +317,18 @@ 
 		$percentage = 0;
 
 		// We can't count the number when getting them for a specific form.
-		if ( empty( $this->form ) ) {
+		if (empty($this->form)) {
 
 			$total = Give()->donors->count();
 
-			if ( $total > 0 ) {
+			if ($total > 0) {
 
-				$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+				$percentage = ((30 * $this->step) / $total) * 100;
 
 			}
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -346,51 +344,51 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function set_donor_data( $i, $data, $donor ) {
+	private function set_donor_data($i, $data, $donor) {
 
 		$columns = $this->csv_cols();
 
 		// Set address variable
 		$address = '';
-		if ( isset( $donor->user_id ) && $donor->user_id > 0 ) {
-			$address = give_get_donor_address( $donor->user_id );
+		if (isset($donor->user_id) && $donor->user_id > 0) {
+			$address = give_get_donor_address($donor->user_id);
 		}
 
 		// Set columns
-		if ( ! empty( $columns['full_name'] ) ) {
-			$data[ $i ]['full_name'] = $donor->name;
+		if ( ! empty($columns['full_name'])) {
+			$data[$i]['full_name'] = $donor->name;
 		}
-		if ( ! empty( $columns['email'] ) ) {
-			$data[ $i ]['email'] = $donor->email;
+		if ( ! empty($columns['email'])) {
+			$data[$i]['email'] = $donor->email;
 		}
-		if ( ! empty( $columns['address_line1'] ) ) {
-
-			$data[ $i ]['address_line1']   = isset( $address['line1'] ) ? $address['line1'] : '';
-			$data[ $i ]['address_line2']   = isset( $address['line2'] ) ? $address['line2'] : '';
-			$data[ $i ]['address_city']    = isset( $address['city'] ) ? $address['city'] : '';
-			$data[ $i ]['address_state']   = isset( $address['state'] ) ? $address['state'] : '';
-			$data[ $i ]['address_zip']     = isset( $address['zip'] ) ? $address['zip'] : '';
-			$data[ $i ]['address_country'] = isset( $address['country'] ) ? $address['country'] : '';
+		if ( ! empty($columns['address_line1'])) {
+
+			$data[$i]['address_line1']   = isset($address['line1']) ? $address['line1'] : '';
+			$data[$i]['address_line2']   = isset($address['line2']) ? $address['line2'] : '';
+			$data[$i]['address_city']    = isset($address['city']) ? $address['city'] : '';
+			$data[$i]['address_state']   = isset($address['state']) ? $address['state'] : '';
+			$data[$i]['address_zip']     = isset($address['zip']) ? $address['zip'] : '';
+			$data[$i]['address_country'] = isset($address['country']) ? $address['country'] : '';
 		}
-		if ( ! empty( $columns['userid'] ) ) {
-			$data[ $i ]['userid'] = ! empty( $donor->user_id ) ? $donor->user_id : '';
+		if ( ! empty($columns['userid'])) {
+			$data[$i]['userid'] = ! empty($donor->user_id) ? $donor->user_id : '';
 		}
-		if ( ! empty( $columns['donation_form'] ) ) {
-			$data[ $i ]['donation_form'] = ! empty( $donor->donation_form_title ) ? $donor->donation_form_title : '';
+		if ( ! empty($columns['donation_form'])) {
+			$data[$i]['donation_form'] = ! empty($donor->donation_form_title) ? $donor->donation_form_title : '';
 		}
-		if ( ! empty( $columns['date_first_donated'] ) ) {
-			$data[ $i ]['date_first_donated'] = date_i18n( give_date_format(), strtotime( $donor->date_created ) );
+		if ( ! empty($columns['date_first_donated'])) {
+			$data[$i]['date_first_donated'] = date_i18n(give_date_format(), strtotime($donor->date_created));
 		}
-		if ( ! empty( $columns['donations'] ) ) {
-			$data[ $i ]['donations'] = $donor->purchase_count;
+		if ( ! empty($columns['donations'])) {
+			$data[$i]['donations'] = $donor->purchase_count;
 		}
-		if ( ! empty( $columns['donation_sum'] ) ) {
-			$data[ $i ]['donation_sum'] = give_format_amount( $donor->purchase_value, array( 'sanitize' => false ) );
+		if ( ! empty($columns['donation_sum'])) {
+			$data[$i]['donation_sum'] = give_format_amount($donor->purchase_value, array('sanitize' => false));
 		}
 
-		$data[ $i ] = apply_filters( 'give_export_set_donor_data', $data[ $i ], $donor );
+		$data[$i] = apply_filters('give_export_set_donor_data', $data[$i], $donor);
 
-		return $data[ $i ];
+		return $data[$i];
 
 	}
 
@@ -400,9 +398,9 @@ 
 	 * @param array             $request
 	 * @param Give_Batch_Export $export
 	 */
-	public function unset_properties( $request, $export ) {
-		if ( $export->done ) {
-			Give_Cache::delete( "give_cache_{$this->query_id}" );
+	public function unset_properties($request, $export) {
+		if ($export->done) {
+			Give_Cache::delete("give_cache_{$this->query_id}");
 		}
 	}
 }
\ No newline at end of file