WordImpress / Give

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

Spacing +27 added lines, -27 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -37,7 +37,7 @@ 
 	 * @return bool Whether we can export or not
 	 */
 	public function can_export() {
-		return (bool) apply_filters( 'give_export_capability', current_user_can( 'export_give_reports' ) );
+		return (bool) apply_filters('give_export_capability', current_user_can('export_give_reports'));
 	}
 
 	/**
@@ -48,16 +48,16 @@ 
 	 * @return void
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 
 		nocache_headers();
-		header( 'Content-Type: text/csv; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=give-export-' . $this->export_type . '-' . date( 'm-d-Y' ) . '.csv' );
-		header( "Expires: 0" );
+		header('Content-Type: text/csv; charset=utf-8');
+		header('Content-Disposition: attachment; filename=give-export-'.$this->export_type.'-'.date('m-d-Y').'.csv');
+		header("Expires: 0");
 	}
 
 	/**
@@ -69,8 +69,8 @@ 
 	 */
 	public function csv_cols() {
 		$cols = array(
-			'id'   => __( 'ID', 'give' ),
-			'date' => __( 'Date', 'give' )
+			'id'   => __('ID', 'give'),
+			'date' => __('Date', 'give')
 		);
 
 		return $cols;
@@ -86,7 +86,7 @@ 
 	public function get_csv_cols() {
 		$cols = $this->csv_cols();
 
-		return apply_filters( "give_export_csv_cols_{$this->export_type}", $cols );
+		return apply_filters("give_export_csv_cols_{$this->export_type}", $cols);
 	}
 
 	/**
@@ -100,10 +100,10 @@ 
 	public function csv_cols_out() {
 		$cols = $this->get_csv_cols();
 		$i    = 1;
-		foreach ( $cols as $col_id => $column ) {
-			echo '"' . addslashes( $column ) . '"';
-			echo $i == count( $cols ) ? '' : ',';
-			$i ++;
+		foreach ($cols as $col_id => $column) {
+			echo '"'.addslashes($column).'"';
+			echo $i == count($cols) ? '' : ',';
+			$i++;
 		}
 		echo "\r\n";
 	}
@@ -120,16 +120,16 @@ 
 		$data = array(
 			0 => array(
 				'id'   => '',
-				'data' => date( 'F j, Y' )
+				'data' => date('F j, Y')
 			),
 			1 => array(
 				'id'   => '',
-				'data' => date( 'F j, Y' )
+				'data' => date('F j, Y')
 			)
 		);
 
-		$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;
 	}
@@ -147,14 +147,14 @@ 
 		$cols = $this->get_csv_cols();
 
 		// Output each row
-		foreach ( $data as $row ) {
+		foreach ($data as $row) {
 			$i = 1;
-			foreach ( $row as $col_id => $column ) {
+			foreach ($row as $col_id => $column) {
 				// Make sure the column is valid
-				if ( array_key_exists( $col_id, $cols ) ) {
-					echo '"' . addslashes( $column ) . '"';
-					echo $i == count( $cols ) ? '' : ',';
-					$i ++;
+				if (array_key_exists($col_id, $cols)) {
+					echo '"'.addslashes($column).'"';
+					echo $i == count($cols) ? '' : ',';
+					$i++;
 				}
 			}
 			echo "\r\n";
@@ -173,8 +173,8 @@ 
 	 * @return void
 	 */
 	public function export() {
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		// Set headers

includes/admin/tools/class-settings-export.php

Spacing +16 added lines, -16 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Export' ) ) :
+if ( ! class_exists('Give_Settings_Export')) :
 
 	/**
 	 * Give_Settings_Export.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'export';
-			$this->label = __( 'Export', 'give' );
+			$this->label = __('Export', 'give');
 
-			add_filter( 'give-tools_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-tools_settings_{$this->id}_page", array( $this, 'output' ) );
-			add_action( 'give_admin_field_tools_export', array( $this, 'render_export_field' ), 10, 2 );
+			add_filter('give-tools_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-tools_settings_{$this->id}_page", array($this, 'output'));
+			add_action('give_admin_field_tools_export', array($this, 'render_export_field'), 10, 2);
 
 			// Do not use main donor for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( 'give-tools_open_form', '__return_empty_string' );
-				add_action( 'give-tools_close_form', '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action('give-tools_open_form', '__return_empty_string');
+				add_action('give-tools_close_form', '__return_empty_string');
 			}
 		}
 
@@ -63,8 +63,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -86,7 +86,7 @@ 
 			 * @param  array $settings
 			 */
 			$settings = apply_filters(
-				'give_get_settings_' . $this->id,
+				'give_get_settings_'.$this->id,
 				array(
 					array(
 						'id'   => 'give_tools_export',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'export',
-						'name' => __( 'Export', 'give' ),
+						'name' => __('Export', 'give'),
 						'type' => 'tools_export',
 					),
 					array(
@@ -119,7 +119,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -131,8 +131,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_export_field( $field, $option_value ) {
-			include_once( 'views/html-admin-page-exports.php' );
+		public function render_export_field($field, $option_value) {
+			include_once('views/html-admin-page-exports.php');
 		}
 	}
 

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/actions.php

Doc Comments -1 removed lines

@@ -216,7 +216,6 @@
 /**
  * Set Donation Amount for Multi Level Donation Forms
  *
- * @param int    $form_id
  * @param object $form
  *
  * @since 1.8.9

Spacing +60 added lines, -60 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_get_actions() {
 
-	$_get_action = ! empty( $_GET['give_action'] ) ? $_GET['give_action'] : null;
+	$_get_action = ! empty($_GET['give_action']) ? $_GET['give_action'] : null;
 
 	// Add backward compatibility to give-action param ( $_GET )
-	if ( empty( $_get_action ) ) {
-		$_get_action = ! empty( $_GET['give-action'] ) ? $_GET['give-action'] : null;
+	if (empty($_get_action)) {
+		$_get_action = ! empty($_GET['give-action']) ? $_GET['give-action'] : null;
 	}
 
-	if ( isset( $_get_action ) ) {
+	if (isset($_get_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_GET.
 		 *
@@ -40,12 +40,12 @@ 
 		 *
 		 * @param array $_GET Array of HTTP GET variables.
 		 */
-		do_action( "give_{$_get_action}", $_GET );
+		do_action("give_{$_get_action}", $_GET);
 	}
 
 }
 
-add_action( 'init', 'give_get_actions' );
+add_action('init', 'give_get_actions');
 
 /**
  * Hooks Give actions, when present in the $_POST super global. Every give_action
@@ -58,15 +58,15 @@ 
  */
 function give_post_actions() {
 
-	$_post_action = ! empty( $_POST['give_action'] ) ? $_POST['give_action'] : null;
+	$_post_action = ! empty($_POST['give_action']) ? $_POST['give_action'] : null;
 
 
 	// Add backward compatibility to give-action param ( $_POST ).
-	if ( empty( $_post_action ) ) {
-		$_post_action = ! empty( $_POST['give-action'] ) ? $_POST['give-action'] : null;
+	if (empty($_post_action)) {
+		$_post_action = ! empty($_POST['give-action']) ? $_POST['give-action'] : null;
 	}
 
-	if ( isset( $_post_action ) ) {
+	if (isset($_post_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_POST.
 		 *
@@ -74,12 +74,12 @@ 
 		 *
 		 * @param array $_POST Array of HTTP POST variables.
 		 */
-		do_action( "give_{$_post_action}", $_POST );
+		do_action("give_{$_post_action}", $_POST);
 	}
 
 }
 
-add_action( 'init', 'give_post_actions' );
+add_action('init', 'give_post_actions');
 
 /**
  * Connect WordPress user with Donor.
@@ -91,29 +91,29 @@ 
  *
  * @return void
  */
-function give_connect_donor_to_wpuser( $user_id, $user_data ) {
+function give_connect_donor_to_wpuser($user_id, $user_data) {
 	/* @var Give_Donor $donor */
-	$donor = new Give_Donor( $user_data['user_email'] );
+	$donor = new Give_Donor($user_data['user_email']);
 
 	// Validate donor id and check if do nor is already connect to wp user or not.
-	if ( $donor->id && ! $donor->user_id ) {
+	if ($donor->id && ! $donor->user_id) {
 
 		// Update donor user_id.
-		if ( $donor->update( array( 'user_id' => $user_id ) ) ) {
-			$donor_note = sprintf( esc_html__( 'WordPress user #%d is connected to #%d', 'give' ), $user_id, $donor->id );
-			$donor->add_note( $donor_note );
+		if ($donor->update(array('user_id' => $user_id))) {
+			$donor_note = sprintf(esc_html__('WordPress user #%d is connected to #%d', 'give'), $user_id, $donor->id);
+			$donor->add_note($donor_note);
 
 			// Update user_id meta in payments.
-			if ( ! empty( $donor->payment_ids ) && ( $donations = explode( ',', $donor->payment_ids ) ) ) {
-				foreach ( $donations as $donation ) {
-					give_update_meta( $donation, '_give_payment_user_id', $user_id );
+			if ( ! empty($donor->payment_ids) && ($donations = explode(',', $donor->payment_ids))) {
+				foreach ($donations as $donation) {
+					give_update_meta($donation, '_give_payment_user_id', $user_id);
 				}
 			}
 		}
 	}
 }
 
-add_action( 'give_insert_user', 'give_connect_donor_to_wpuser', 10, 2 );
+add_action('give_insert_user', 'give_connect_donor_to_wpuser', 10, 2);
 
 
 /**
@@ -127,44 +127,44 @@ 
  */
 function give_validate_license_when_site_migrated() {
 	// Store current site address if not already stored.
-	$home_url_parts              = parse_url( home_url() );
-	$home_url                    = isset( $home_url_parts['host'] ) ? $home_url_parts['host'] : false;
-	$home_url                    .= isset( $home_url_parts['path'] ) ? $home_url_parts['path'] : '';
-	$site_address_before_migrate = get_option( 'give_site_address_before_migrate' );
+	$home_url_parts              = parse_url(home_url());
+	$home_url                    = isset($home_url_parts['host']) ? $home_url_parts['host'] : false;
+	$home_url .= isset($home_url_parts['path']) ? $home_url_parts['path'] : '';
+	$site_address_before_migrate = get_option('give_site_address_before_migrate');
 
 	// Need $home_url to proceed
-	if ( ! $home_url ) {
+	if ( ! $home_url) {
 		return;
 	}
 
 	// Save site address
-	if ( ! $site_address_before_migrate ) {
+	if ( ! $site_address_before_migrate) {
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', $home_url );
+		update_option('give_site_address_before_migrate', $home_url);
 
 		return;
 	}
 
 	// Backwards compat. for before when we were storing URL scheme.
-	if ( strpos( $site_address_before_migrate, 'http' ) ) {
-		$site_address_before_migrate = parse_url( $site_address_before_migrate );
-		$site_address_before_migrate = isset( $site_address_before_migrate['host'] ) ? $site_address_before_migrate['host'] : false;
+	if (strpos($site_address_before_migrate, 'http')) {
+		$site_address_before_migrate = parse_url($site_address_before_migrate);
+		$site_address_before_migrate = isset($site_address_before_migrate['host']) ? $site_address_before_migrate['host'] : false;
 		// Add path for multisite installs.
-		$site_address_before_migrate .= isset( $site_address_before_migrate['path'] ) ? $site_address_before_migrate['path'] : '';
+		$site_address_before_migrate .= isset($site_address_before_migrate['path']) ? $site_address_before_migrate['path'] : '';
 	}
 
 	// If the two URLs don't match run CRON.
-	if ( $home_url !== $site_address_before_migrate ) {
+	if ($home_url !== $site_address_before_migrate) {
 		// Immediately run cron.
-		wp_schedule_single_event( time(), 'give_validate_license_when_site_migrated' );
+		wp_schedule_single_event(time(), 'give_validate_license_when_site_migrated');
 
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', $home_url );
+		update_option('give_site_address_before_migrate', $home_url);
 	}
 
 }
 
-add_action( 'admin_init', 'give_validate_license_when_site_migrated' );
+add_action('admin_init', 'give_validate_license_when_site_migrated');
 
 
 /**
@@ -174,19 +174,19 @@ 
  *
  * @param $data
  */
-function give_donor_batch_export_complete( $data ) {
+function give_donor_batch_export_complete($data) {
 	// Remove donor ids cache.
 	if (
-		isset( $data['class'] )
+		isset($data['class'])
 		&& 'Give_Batch_Donors_Export' === $data['class']
-		&& ! empty( $data['forms'] )
-		&& isset( $data['give_export_option']['query_id'] )
+		&& ! empty($data['forms'])
+		&& isset($data['give_export_option']['query_id'])
 	) {
-		Give_Cache::delete( Give_Cache::get_key( $data['give_export_option']['query_id'] ) );
+		Give_Cache::delete(Give_Cache::get_key($data['give_export_option']['query_id']));
 	}
 }
 
-add_action( 'give_file_export_complete', 'give_donor_batch_export_complete' );
+add_action('give_file_export_complete', 'give_donor_batch_export_complete');
 
 /**
  * Print css for wordpress setting pages.
@@ -197,12 +197,12 @@ 
 	/* @var WP_Screen $screen */
 	$screen = get_current_screen();
 
-	if ( ! ( $screen instanceof WP_Screen ) ) {
+	if ( ! ($screen instanceof WP_Screen)) {
 		return false;
 	}
 
-	switch ( true ) {
-		case ( 'plugins' === $screen->base || 'plugins-network' === $screen->base ):
+	switch (true) {
+		case ('plugins' === $screen->base || 'plugins-network' === $screen->base):
 			?>
 			<style>
 				tr.active.update + tr.give-addon-notice-tr td {
@@ -239,7 +239,7 @@ 
 	}
 }
 
-add_action( 'admin_head', 'give_admin_quick_css' );
+add_action('admin_head', 'give_admin_quick_css');
 
 
 /**
@@ -252,28 +252,28 @@ 
  *
  * @return void
  */
-function give_set_donation_levels_max_min_amount( $form_id ) {
+function give_set_donation_levels_max_min_amount($form_id) {
 	if (
-		( 'set' === $_POST['_give_price_option'] ) ||
-		( in_array( '_give_donation_levels', $_POST ) && count( $_POST['_give_donation_levels'] ) <= 0 ) ||
-		! ( $donation_levels_amounts = wp_list_pluck( $_POST['_give_donation_levels'], '_give_amount' ) )
+		('set' === $_POST['_give_price_option']) ||
+		(in_array('_give_donation_levels', $_POST) && count($_POST['_give_donation_levels']) <= 0) ||
+		! ($donation_levels_amounts = wp_list_pluck($_POST['_give_donation_levels'], '_give_amount'))
 	) {
 		// Delete old meta.
-		give_delete_meta( $form_id, '_give_levels_minimum_amount' );
-		give_delete_meta( $form_id, '_give_levels_maximum_amount' );
+		give_delete_meta($form_id, '_give_levels_minimum_amount');
+		give_delete_meta($form_id, '_give_levels_maximum_amount');
 
 		return;
 	}
 
 	// Sanitize donation level amounts.
-	$donation_levels_amounts = array_map( 'give_maybe_sanitize_amount', $donation_levels_amounts );
+	$donation_levels_amounts = array_map('give_maybe_sanitize_amount', $donation_levels_amounts);
 
-	$min_amount = min( $donation_levels_amounts );
-	$max_amount = max( $donation_levels_amounts );
+	$min_amount = min($donation_levels_amounts);
+	$max_amount = max($donation_levels_amounts);
 
 	// Set Minimum and Maximum amount for Multi Level Donation Forms
-	give_update_meta( $form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount_for_db( $min_amount ) : 0 );
-	give_update_meta( $form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount_for_db( $max_amount ) : 0 );
+	give_update_meta($form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount_for_db($min_amount) : 0);
+	give_update_meta($form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount_for_db($max_amount) : 0);
 }
 
-add_action( 'give_pre_process_give_forms_meta', 'give_set_donation_levels_max_min_amount', 30 );
\ No newline at end of file
+add_action('give_pre_process_give_forms_meta', 'give_set_donation_levels_max_min_amount', 30);
\ No newline at end of file

includes/admin/donors/class-donor-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -253,7 +253,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return mixed string If search is present, false otherwise.
+	 * @return string|false string If search is present, false otherwise.
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Spacing +66 added lines, -66 removed lines

@@ -12,13 +12,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';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => __( 'Donor', 'give' ),     // Singular name of the listed records.
-			'plural'   => __( 'Donors', 'give' ),    // Plural name of the listed records.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		parent::__construct(array(
+			'singular' => __('Donor', 'give'), // Singular name of the listed records.
+			'plural'   => __('Donors', 'give'), // Plural name of the listed records.
+			'ajax'     => false, // Does this table support ajax?.
+		));
 
 	}
 
@@ -80,22 +80,22 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?>
+			)); ?>
 		</p>
 		<?php
 	}
@@ -111,31 +111,31 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($donor, $column_name) {
+		switch ($column_name) {
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor['id'])),
+					esc_html($donor['num_donations'])
 				);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $donor[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($donor[$column_name], array('sanitize' => false)));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $donor['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($donor['date_created']));
 				break;
 
 			default:
-				$value = isset( $donor[ $column_name ] ) ? $donor[ $column_name ] : null;
+				$value = isset($donor[$column_name]) ? $donor[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_donors_column_{$column_name}", $value, $donor['id'] );
+		return apply_filters("give_donors_column_{$column_name}", $value, $donor['id']);
 
 	}
 
@@ -146,13 +146,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = '#' . $donor['id'] . ' ';
-		$name     .= ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] );
-		$actions  = $this->get_row_actions( $donor );
+	public function column_name($donor) {
+		$name     = '#'.$donor['id'].' ';
+		$name .= ! empty($donor['name']) ? $donor['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+		$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']);
+		$actions  = $this->get_row_actions($donor);
 
-		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
+		return '<a href="'.esc_url($view_url).'">'.$name.'</a>'.$this->row_actions($actions);
 	}
 
 	/**
@@ -164,14 +164,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
-			'date_created'  => __( 'Date Created', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
+			'date_created'  => __('Date Created', 'give'),
 		);
 
-		return apply_filters( 'give_list_donors_columns', $columns );
+		return apply_filters('give_list_donors_columns', $columns);
 
 	}
 
@@ -185,13 +185,13 @@ 
 	public function get_sortable_columns() {
 
 		$columns = array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 
-		return apply_filters( 'give_list_donors_sortable_columns', $columns );
+		return apply_filters('give_list_donors_sortable_columns', $columns);
 	}
 
 	/**
@@ -204,19 +204,19 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $donor ) {
+	public function get_row_actions($donor) {
 
 		$actions = array(
 
-			'view' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] ), sprintf( esc_attr__( 'View "%s"', 'give' ), $donor['name'] ), __( 'View Donor', 'give' ) ),
+			'view' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']), sprintf(esc_attr__('View "%s"', 'give'), $donor['name']), __('View Donor', 'give')),
 
-			'notes' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor['id'] ), sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $donor['name'] ), __( 'Notes', 'give' ) ),
+			'notes' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor['id']), sprintf(esc_attr__('Notes for "%s"', 'give'), $donor['name']), __('Notes', 'give')),
 
-			'delete' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor['id'] ), sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'delete' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor['id']), sprintf(esc_attr__('Delete "%s"', 'give'), $donor['name']), __('Delete', 'give')),
 
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $donor );
+		return apply_filters('give_donor_row_actions', $actions, $donor);
 
 	}
 
@@ -230,7 +230,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place.
 	}
 
@@ -242,7 +242,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;
 	}
 
 	/**
@@ -253,7 +253,7 @@ 
 	 * @return mixed string If search is present, false otherwise.
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -270,13 +270,13 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? intval( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? intval($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -290,7 +290,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -305,9 +305,9 @@ 
 
 		$_donor_query['number'] = - 1;
 		$_donor_query['offset'] = 0;
-		$donors                 = Give()->donors->get_donors( $_donor_query );
+		$donors                 = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -319,10 +319,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -331,10 +331,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -357,16 +357,16 @@ 
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->donor_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/donors/donor-actions.php

Doc Comments +3 added lines, -3 removed lines

@@ -272,7 +272,7 @@ 
  *
  * @param  array $args The $_POST array being passed.
  *
- * @return int Whether it was a successful deletion.
+ * @return false|null Whether it was a successful deletion.
  */
 function give_donor_delete( $args ) {
 
@@ -546,7 +546,7 @@ 
  * Remove an email address to the donor from within the admin and log a donor note and redirect back to the donor interface for feedback.
  *
  * @since  1.7
- * @return bool|null
+ * @return false|null
  */
 function give_remove_donor_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
@@ -589,7 +589,7 @@ 
  * and redirect back to the donor interface for feedback
  *
  * @since  1.7
- * @return bool|null
+ * @return false|null
  */
 function give_set_donor_primary_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {

Spacing +190 added lines, -190 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,17 +23,17 @@ 
  *
  * @return array|bool $output Response messages
  */
-function give_edit_donor( $args ) {
+function give_edit_donor($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -41,15 +41,15 @@ 
 	$donor_id   = (int) $args['customerinfo']['id'];
 	$nonce      = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin’ uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin’ uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
-	if ( empty( $donor->id ) ) {
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -64,67 +64,67 @@ 
 		'country' => '',
 	);
 
-	$donor_info = wp_parse_args( $donor_info, $defaults );
+	$donor_info = wp_parse_args($donor_info, $defaults);
 
-	if ( (int) $donor_info['user_id'] !== (int) $donor->user_id ) {
+	if ((int) $donor_info['user_id'] !== (int) $donor->user_id) {
 
 		// Make sure we don't already have this user attached to a donor.
-		if ( ! empty( $donor_info['user_id'] ) && false !== Give()->donors->get_donor_by( 'user_id', $donor_info['user_id'] ) ) {
-			give_set_error( 'give-invalid-donor-user_id', sprintf( __( 'The User ID #%d is already associated with a different donor.', 'give' ), $donor_info['user_id'] ) );
+		if ( ! empty($donor_info['user_id']) && false !== Give()->donors->get_donor_by('user_id', $donor_info['user_id'])) {
+			give_set_error('give-invalid-donor-user_id', sprintf(__('The User ID #%d is already associated with a different donor.', 'give'), $donor_info['user_id']));
 		}
 
 		// Make sure it's actually a user.
-		$user = get_user_by( 'id', $donor_info['user_id'] );
-		if ( ! empty( $donor_info['user_id'] ) && false === $user ) {
-			give_set_error( 'give-invalid-user_id', sprintf( __( 'The User ID #%d does not exist. Please assign an existing user.', 'give' ), $donor_info['user_id'] ) );
+		$user = get_user_by('id', $donor_info['user_id']);
+		if ( ! empty($donor_info['user_id']) && false === $user) {
+			give_set_error('give-invalid-user_id', sprintf(__('The User ID #%d does not exist. Please assign an existing user.', 'give'), $donor_info['user_id']));
 		}
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
 	// Setup the donor address, if present.
 	$address = array();
-	if ( intval( $donor_info['user_id'] ) > 0 ) {
-
-		$current_address = (array) get_user_meta( $donor_info['user_id'], '_give_user_address', true );
-
-		if ( is_array( $current_address ) && 0 < count( $current_address ) ) {
-			$current_address    = wp_parse_args( $current_address, $defaults );
-			$address['line1']   = ! empty( $donor_info['line1'] ) ? $donor_info['line1'] : $current_address['line1'];
-			$address['line2']   = ! empty( $donor_info['line2'] ) ? $donor_info['line2'] : $current_address['line2'];
-			$address['city']    = ! empty( $donor_info['city'] ) ? $donor_info['city'] : $current_address['city'];
-			$address['country'] = ! empty( $donor_info['country'] ) ? $donor_info['country'] : $current_address['country'];
-			$address['zip']     = ! empty( $donor_info['zip'] ) ? $donor_info['zip'] : $current_address['zip'];
-			$address['state']   = ! empty( $donor_info['state'] ) ? $donor_info['state'] : $current_address['state'];
+	if (intval($donor_info['user_id']) > 0) {
+
+		$current_address = (array) get_user_meta($donor_info['user_id'], '_give_user_address', true);
+
+		if (is_array($current_address) && 0 < count($current_address)) {
+			$current_address    = wp_parse_args($current_address, $defaults);
+			$address['line1']   = ! empty($donor_info['line1']) ? $donor_info['line1'] : $current_address['line1'];
+			$address['line2']   = ! empty($donor_info['line2']) ? $donor_info['line2'] : $current_address['line2'];
+			$address['city']    = ! empty($donor_info['city']) ? $donor_info['city'] : $current_address['city'];
+			$address['country'] = ! empty($donor_info['country']) ? $donor_info['country'] : $current_address['country'];
+			$address['zip']     = ! empty($donor_info['zip']) ? $donor_info['zip'] : $current_address['zip'];
+			$address['state']   = ! empty($donor_info['state']) ? $donor_info['state'] : $current_address['state'];
 		} else {
-			$address['line1']   = ! empty( $donor_info['line1'] ) ? $donor_info['line1'] : '';
-			$address['line2']   = ! empty( $donor_info['line2'] ) ? $donor_info['line2'] : '';
-			$address['city']    = ! empty( $donor_info['city'] ) ? $donor_info['city'] : '';
-			$address['country'] = ! empty( $donor_info['country'] ) ? $donor_info['country'] : '';
-			$address['zip']     = ! empty( $donor_info['zip'] ) ? $donor_info['zip'] : '';
-			$address['state']   = ! empty( $donor_info['state'] ) ? $donor_info['state'] : '';
+			$address['line1']   = ! empty($donor_info['line1']) ? $donor_info['line1'] : '';
+			$address['line2']   = ! empty($donor_info['line2']) ? $donor_info['line2'] : '';
+			$address['city']    = ! empty($donor_info['city']) ? $donor_info['city'] : '';
+			$address['country'] = ! empty($donor_info['country']) ? $donor_info['country'] : '';
+			$address['zip']     = ! empty($donor_info['zip']) ? $donor_info['zip'] : '';
+			$address['state']   = ! empty($donor_info['state']) ? $donor_info['state'] : '';
 		}
 
 	}
 
 	// Sanitize the inputs.
 	$donor_data            = array();
-	$donor_data['name']    = strip_tags( stripslashes( $donor_info['name'] ) );
+	$donor_data['name']    = strip_tags(stripslashes($donor_info['name']));
 	$donor_data['user_id'] = $donor_info['user_id'];
 
-	$donor_data = apply_filters( 'give_edit_donor_info', $donor_data, $donor_id );
-	$address    = apply_filters( 'give_edit_donor_address', $address, $donor_id );
+	$donor_data = apply_filters('give_edit_donor_info', $donor_data, $donor_id);
+	$address    = apply_filters('give_edit_donor_address', $address, $donor_id);
 
-	$donor_data = array_map( 'sanitize_text_field', $donor_data );
-	$address    = array_map( 'sanitize_text_field', $address );
+	$donor_data = array_map('sanitize_text_field', $donor_data);
+	$address    = array_map('sanitize_text_field', $address);
 
-	$output = give_connect_user_donor_profile( $donor, $donor_data, $address );
+	$output = give_connect_user_donor_profile($donor, $donor_data, $address);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
@@ -132,7 +132,7 @@ 
 
 }
 
-add_action( 'give_edit-donor', 'give_edit_donor', 10, 1 );
+add_action('give_edit-donor', 'give_edit_donor', 10, 1);
 
 /**
  * Save a donor note.
@@ -143,40 +143,40 @@ 
  *
  * @return int         The Note ID that was saved, or 0 if nothing was saved.
  */
-function give_donor_save_note( $args ) {
+function give_donor_save_note($args) {
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! is_admin() || ! current_user_can( $donor_view_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_view_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
-	$donor_note = trim( sanitize_text_field( $args['donor_note'] ) );
+	$donor_note = trim(sanitize_text_field($args['donor_note']));
 	$donor_id   = (int) $args['customer_id'];
 	$nonce      = $args['add_donor_note_nonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'add-donor-note' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'add-donor-note')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( empty( $donor_note ) ) {
-		give_set_error( 'empty-donor-note', __( 'A note is required.', 'give' ) );
+	if (empty($donor_note)) {
+		give_set_error('empty-donor-note', __('A note is required.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
-	$donor    = new Give_Donor( $donor_id );
-	$new_note = $donor->add_note( $donor_note );
+	$donor    = new Give_Donor($donor_id);
+	$new_note = $donor->add_note($donor_note);
 
 	/**
 	 * Fires before inserting donor note.
@@ -186,22 +186,22 @@ 
 	 * @param int    $donor_id The ID of the donor.
 	 * @param string $new_note Note content.
 	 */
-	do_action( 'give_pre_insert_donor_note', $donor_id, $new_note );
+	do_action('give_pre_insert_donor_note', $donor_id, $new_note);
 
-	if ( ! empty( $new_note ) && ! empty( $donor->id ) ) {
+	if ( ! empty($new_note) && ! empty($donor->id)) {
 
 		ob_start();
 		?>
 		<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 			<span class="note-content-wrap">
-				<?php echo stripslashes( $new_note ); ?>
+				<?php echo stripslashes($new_note); ?>
 			</span>
 		</div>
 		<?php
 		$output = ob_get_contents();
 		ob_end_clean();
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+		if (defined('DOING_AJAX') && DOING_AJAX) {
 			echo $output;
 			exit;
 		}
@@ -214,7 +214,7 @@ 
 
 }
 
-add_action( 'give_add-donor-note', 'give_donor_save_note', 10, 1 );
+add_action('give_add-donor-note', 'give_donor_save_note', 10, 1);
 
 /**
  * Delete a donor.
@@ -225,41 +225,41 @@ 
  *
  * @return int Whether it was a successful deletion.
  */
-function give_donor_delete( $args ) {
+function give_donor_delete($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to delete donors.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to delete donors.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
 	$donor_id    = (int) $args['customer_id'];
-	$confirm     = ! empty( $args['give-donor-delete-confirm'] ) ? true : false;
-	$remove_data = ! empty( $args['give-donor-delete-records'] ) ? true : false;
+	$confirm     = ! empty($args['give-donor-delete-confirm']) ? true : false;
+	$remove_data = ! empty($args['give-donor-delete-records']) ? true : false;
 	$nonce       = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'delete-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'delete-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( ! $confirm ) {
-		give_set_error( 'donor-delete-no-confirm', __( 'Please confirm you want to delete this donor.', 'give' ) );
+	if ( ! $confirm) {
+		give_set_error('donor-delete-no-confirm', __('Please confirm you want to delete this donor.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
-		wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id ) );
+	if (give_get_errors()) {
+		wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id));
 		exit;
 	}
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
 	/**
 	 * Fires before deleting donor.
@@ -270,50 +270,50 @@ 
 	 * @param bool $confirm     Delete confirmation.
 	 * @param bool $remove_data Records delete confirmation.
 	 */
-	do_action( 'give_pre_delete_donor', $donor_id, $confirm, $remove_data );
+	do_action('give_pre_delete_donor', $donor_id, $confirm, $remove_data);
 
-	if ( $donor->id > 0 ) {
+	if ($donor->id > 0) {
 
-		$payments_array = explode( ',', $donor->payment_ids );
-		$success        = Give()->donors->delete( $donor->id );
+		$payments_array = explode(',', $donor->payment_ids);
+		$success        = Give()->donors->delete($donor->id);
 
-		if ( $success ) {
+		if ($success) {
 
-			if ( $remove_data ) {
+			if ($remove_data) {
 
 				// Remove all donations, logs, etc.
-				foreach ( $payments_array as $payment_id ) {
-					give_delete_donation( $payment_id );
+				foreach ($payments_array as $payment_id) {
+					give_delete_donation($payment_id);
 				}
 			} else {
 
 				// Just set the donations to customer_id of 0.
-				foreach ( $payments_array as $payment_id ) {
-					give_update_payment_meta( $payment_id, '_give_payment_customer_id', 0 );
+				foreach ($payments_array as $payment_id) {
+					give_update_payment_meta($payment_id, '_give_payment_customer_id', 0);
 				}
 			}
 
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&give-message=donor-deleted' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&give-message=donor-deleted');
 
 		} else {
 
-			give_set_error( 'give-donor-delete-failed', __( 'Error deleting donor.', 'give' ) );
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor_id );
+			give_set_error('give-donor-delete-failed', __('Error deleting donor.', 'give'));
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor_id);
 
 		}
 	} else {
 
-		give_set_error( 'give-donor-delete-invalid-id', __( 'Invalid Donor ID.', 'give' ) );
-		$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors' );
+		give_set_error('give-donor-delete-invalid-id', __('Invalid Donor ID.', 'give'));
+		$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors');
 
 	}
 
-	wp_redirect( $redirect );
+	wp_redirect($redirect);
 	exit;
 
 }
 
-add_action( 'give_delete-donor', 'give_donor_delete', 10, 1 );
+add_action('give_delete-donor', 'give_donor_delete', 10, 1);
 
 /**
  * Disconnect a user ID from a donor
@@ -324,17 +324,17 @@ 
  *
  * @return bool|array        If the disconnect was successful.
  */
-function give_disconnect_donor_user_id( $args ) {
+function give_disconnect_donor_user_id($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -342,14 +342,14 @@ 
 
 	$nonce = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
-	if ( empty( $donor->id ) ) {
+	$donor = new Give_Donor($donor_id);
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -363,34 +363,34 @@ 
 	 * @param int $donor_id The ID of the donor.
 	 * @param int $user_id  The ID of the user.
 	 */
-	do_action( 'give_pre_donor_disconnect_user_id', $donor_id, $user_id );
+	do_action('give_pre_donor_disconnect_user_id', $donor_id, $user_id);
 
 	$output     = array();
 	$donor_args = array(
 		'user_id' => 0,
 	);
 
-	if ( $donor->update( $donor_args ) ) {
+	if ($donor->update($donor_args)) {
 		global $wpdb;
 
-		if ( ! empty( $donor->payment_ids ) ) {
-			$wpdb->query( "UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $donor->payment_ids )" );
+		if ( ! empty($donor->payment_ids)) {
+			$wpdb->query("UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $donor->payment_ids )");
 		}
 
 		// Set Donor Disconnection status true, if user and donor are disconnected with each other.
-		update_user_meta( $user_id, '_give_is_donor_disconnected', true );
-		update_user_meta( $user_id, '_give_disconnected_donor_id', $donor->id );
-		$donor->update_meta( '_give_disconnected_user_id', $user_id );
+		update_user_meta($user_id, '_give_is_donor_disconnected', true);
+		update_user_meta($user_id, '_give_disconnected_donor_id', $donor->id);
+		$donor->update_meta('_give_disconnected_user_id', $user_id);
 
 		$output['success'] = true;
 
 	} else {
 
 		$output['success'] = false;
-		give_set_error( 'give-disconnect-user-fail', __( 'Failed to disconnect user from donor.', 'give' ) );
+		give_set_error('give-disconnect-user-fail', __('Failed to disconnect user from donor.', 'give'));
 	}
 
-	$output['redirect'] = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' ) . $donor_id;
+	$output['redirect'] = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id=').$donor_id;
 
 	/**
 	 * Fires after disconnecting user ID from a donor.
@@ -399,11 +399,11 @@ 
 	 *
 	 * @param int $donor_id The ID of the donor.
 	 */
-	do_action( 'give_post_donor_disconnect_user_id', $donor_id );
+	do_action('give_post_donor_disconnect_user_id', $donor_id);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
@@ -411,7 +411,7 @@ 
 
 }
 
-add_action( 'give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1 );
+add_action('give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1);
 
 /**
  * Add an email address to the donor from within the admin and log a donor note.
@@ -422,90 +422,90 @@ 
  *
  * @return mixed        If DOING_AJAX echos out JSON, otherwise returns array of success (bool) and message (string).
  */
-function give_add_donor_email( $args ) {
+function give_add_donor_email($args) {
 	/**
 	 * Define variable
 	 *
 	 * @since 1.8.14
 	 */
 	$donor_id = '';
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
 	$output = array();
-	if ( empty( $args ) || empty( $args['email'] ) || empty( $args['customer_id'] ) ) {
+	if (empty($args) || empty($args['email']) || empty($args['customer_id'])) {
 		$output['success'] = false;
-		if ( empty( $args['email'] ) ) {
-			$output['message'] = __( 'Email address is required.', 'give' );
-		} elseif ( empty( $args['customer_id'] ) ) {
-			$output['message'] = __( 'Donor ID is required.', 'give' );
+		if (empty($args['email'])) {
+			$output['message'] = __('Email address is required.', 'give');
+		} elseif (empty($args['customer_id'])) {
+			$output['message'] = __('Donor ID is required.', 'give');
 		} else {
-			$output['message'] = __( 'An error has occurred. Please try again.', 'give' );
+			$output['message'] = __('An error has occurred. Please try again.', 'give');
 		}
-	} elseif ( ! wp_verify_nonce( $args['_wpnonce'], 'give_add_donor_email' ) ) {
+	} elseif ( ! wp_verify_nonce($args['_wpnonce'], 'give_add_donor_email')) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Nonce verification failed.', 'give' ),
+			'message' => __('Nonce verification failed.', 'give'),
 		);
-	} elseif ( ! is_email( $args['email'] ) ) {
+	} elseif ( ! is_email($args['email'])) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Invalid email.', 'give' ),
+			'message' => __('Invalid email.', 'give'),
 		);
 	} else {
-		$email    = sanitize_email( $args['email'] );
+		$email    = sanitize_email($args['email']);
 		$donor_id = (int) $args['customer_id'];
 		$primary  = 'true' === $args['primary'] ? true : false;
-		$donor    = new Give_Donor( $donor_id );
-		if ( false === $donor->add_email( $email, $primary ) ) {
-			if ( in_array( $email, $donor->emails ) ) {
+		$donor    = new Give_Donor($donor_id);
+		if (false === $donor->add_email($email, $primary)) {
+			if (in_array($email, $donor->emails)) {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email already associated with this donor.', 'give' ),
+					'message' => __('Email already associated with this donor.', 'give'),
 				);
 			} else {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email address is already associated with another donor.', 'give' ),
+					'message' => __('Email address is already associated with another donor.', 'give'),
 				);
 			}
 		} else {
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&give-message=email-added' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&give-message=email-added');
 			$output   = array(
 				'success'  => true,
-				'message'  => __( 'Email successfully added to donor.', 'give' ),
+				'message'  => __('Email successfully added to donor.', 'give'),
 				'redirect' => $redirect,
 			);
 
 			$user       = wp_get_current_user();
-			$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-			$donor_note = sprintf( __( 'Email address %1$s added by %2$s', 'give' ), $email, $user_login );
-			$donor->add_note( $donor_note );
+			$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+			$donor_note = sprintf(__('Email address %1$s added by %2$s', 'give'), $email, $user_login);
+			$donor->add_note($donor_note);
 
-			if ( $primary ) {
-				$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $email, $user_login );
-				$donor->add_note( $donor_note );
+			if ($primary) {
+				$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $email, $user_login);
+				$donor->add_note($donor_note);
 			}
 		}
 	}// End if().
 
-	do_action( 'give_post_add_donor_email', $donor_id, $args );
+	do_action('give_post_add_donor_email', $donor_id, $args);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
 	return $output;
 }
 
-add_action( 'give_add_donor_email', 'give_add_donor_email', 10, 1 );
+add_action('give_add_donor_email', 'give_add_donor_email', 10, 1);
 
 
 /**
@@ -515,39 +515,39 @@ 
  * @return bool|null
  */
 function give_remove_donor_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
-	if ( ! wp_verify_nonce( $nonce, 'give-remove-donor-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-remove-donor-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
-	if ( $donor->remove_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-message', 'email-removed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	$donor = new Give_Donor($_GET['id']);
+	if ($donor->remove_email($_GET['email'])) {
+		$url        = add_query_arg('give-message', 'email-removed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s removed by %2$s', 'give' ), $_GET['email'], $user_login );
-		$donor->add_note( $donor_note );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s removed by %2$s', 'give'), $_GET['email'], $user_login);
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-message', 'email-remove-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-message', 'email-remove-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_remove_donor_email', 'give_remove_donor_email', 10 );
+add_action('give_remove_donor_email', 'give_remove_donor_email', 10);
 
 
 /**
@@ -558,41 +558,41 @@ 
  * @return bool|null
  */
 function give_set_donor_primary_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'give-set-donor-primary-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-set-donor-primary-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
+	$donor = new Give_Donor($_GET['id']);
 
-	if ( $donor->set_primary_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-message', 'primary-email-updated', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	if ($donor->set_primary_email($_GET['email'])) {
+		$url        = add_query_arg('give-message', 'primary-email-updated', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $_GET['email'], $user_login );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $_GET['email'], $user_login);
 
-		$donor->add_note( $donor_note );
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-message', 'primary-email-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-message', 'primary-email-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_set_donor_primary_email', 'give_set_donor_primary_email', 10 );
+add_action('give_set_donor_primary_email', 'give_set_donor_primary_email', 10);

includes/admin/tools/data/class-give-tools-recount-donor-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 * @access public
 	 * @since 1.5
 	 *
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Spacing +38 added lines, -38 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -57,32 +57,32 @@ 
 
 		$args = array(
 			'number'  => $this->per_step,
-			'offset'  => $this->per_step * ( $this->step - 1 ),
+			'offset'  => $this->per_step * ($this->step - 1),
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$allowed_payment_status = apply_filters( 'give_recount_donors_donation_statuses', give_get_payment_status_keys() );
+			$allowed_payment_status = apply_filters('give_recount_donors_donation_statuses', give_get_payment_status_keys());
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$attached_payment_ids = explode( ',', $donor->payment_ids );
+				$attached_payment_ids = explode(',', $donor->payment_ids);
 
 				$attached_args = array(
 					'post__in' => $attached_payment_ids,
-					'number'   => - 1,
+					'number'   => -1,
 					'status'   => $allowed_payment_status,
 				);
 
-				$attached_payments = (array) give_get_payments( $attached_args );
+				$attached_payments = (array) give_get_payments($attached_args);
 
 				$unattached_args = array(
 					'post__not_in' => $attached_payment_ids,
-					'number'       => - 1,
+					'number'       => -1,
 					'status'       => $allowed_payment_status,
 					'meta_query'   => array(
 						array(
@@ -93,29 +93,29 @@ 
 					),
 				);
 
-				$unattached_payments = give_get_payments( $unattached_args );
+				$unattached_payments = give_get_payments($unattached_args);
 
-				$payments = array_merge( $attached_payments, $unattached_payments );
+				$payments = array_merge($attached_payments, $unattached_payments);
 
 				$purchase_value = 0.00;
 				$purchase_count = 0;
 				$payment_ids    = array();
 
-				if ( $payments ) {
+				if ($payments) {
 
-					foreach ( $payments as $payment ) {
+					foreach ($payments as $payment) {
 
 						$should_process_payment = 'publish' == $payment->post_status ? true : false;
-						$should_process_payment = apply_filters( 'give_donor_recount_should_process_donation', $should_process_payment, $payment );
+						$should_process_payment = apply_filters('give_donor_recount_should_process_donation', $should_process_payment, $payment);
 
-						if ( true === $should_process_payment ) {
+						if (true === $should_process_payment) {
 
-							if ( apply_filters( 'give_donor_recount_should_increase_value', true, $payment ) ) {
-								$purchase_value += give_get_payment_amount( $payment->ID );
+							if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+								$purchase_value += give_get_payment_amount($payment->ID);
 							}
 
-							if ( apply_filters( 'give_donor_recount_should_increase_count', true, $payment ) ) {
-								$purchase_count ++;
+							if (apply_filters('give_donor_recount_should_increase_count', true, $payment)) {
+								$purchase_count++;
 							}
 						}
 
@@ -123,7 +123,7 @@ 
 					}
 				}
 
-				$payment_ids = implode( ',', $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
 
 				$donor_update_data = array(
 					'purchase_count' => $purchase_count,
@@ -131,8 +131,8 @@ 
 					'payment_ids'    => $payment_ids,
 				);
 
-				$donor_instance = new Give_Donor( $donor->id );
-				$donor_instance->update( $donor_update_data );
+				$donor_instance = new Give_Donor($donor->id);
+				$donor_instance->update($donor_update_data);
 
 			}// End foreach().
 
@@ -152,21 +152,21 @@ 
 	public function get_percentage_complete() {
 
 		$args = array(
-			'number'  => - 1,
+			'number'  => -1,
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$donors = Give()->donors->get_donors( $args );
-		$total     = count( $donors );
+		$donors = Give()->donors->get_donors($args);
+		$total = count($donors);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -180,7 +180,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -191,21 +191,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
 			$this->done    = true;
-			$this->message = esc_html__( 'Donor stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Donor stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -215,10 +215,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 

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;