WordImpress / Give

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

Indentation +2 added lines, -2 removed lines

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

Spacing +64 added lines, -64 removed lines

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

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

Spacing +51 added lines, -51 removed lines

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

includes/class-give-license-handler.php

Doc Comments +3 added lines, -3 removed lines

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

Spacing +151 added lines, -151 removed lines

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

includes/payments/class-payment-stats.php

Doc Comments +3 added lines, -3 removed lines

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

Spacing +42 added lines, -42 removed lines

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

includes/gateways/offline-donations.php

Indentation +4 added lines, -4 removed lines

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

Spacing +139 added lines, -139 removed lines

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

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

Spacing +27 added lines, -28 removed lines

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

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

Indentation +1 added lines, -1 removed lines

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

Spacing +103 added lines, -105 removed lines

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

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

Spacing +38 added lines, -38 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -118,14 +118,14 @@ 
 	 *
 	 * @param int $_step
 	 */
-	public function __construct( $_step = 1 ) {
+	public function __construct($_step = 1) {
 
 		$upload_dir     = wp_upload_dir();
 		$this->filetype = '.csv';
-		$this->filename = 'give-' . $this->export_type . $this->filetype;
-		$this->file     = trailingslashit( $upload_dir['basedir'] ) . $this->filename;
+		$this->filename = 'give-'.$this->export_type.$this->filetype;
+		$this->file     = trailingslashit($upload_dir['basedir']).$this->filename;
 
-		if ( ! is_writeable( $upload_dir['basedir'] ) ) {
+		if ( ! is_writeable($upload_dir['basedir'])) {
 			$this->is_writable = false;
 		}
 
@@ -141,22 +141,22 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to export data.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to export data.', 'give'), esc_html__('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
-		if ( $this->step < 2 ) {
+		if ($this->step < 2) {
 
 			// Make sure we start with a fresh file on step 1.
-			@unlink( $this->file );
+			@unlink($this->file);
 			$this->print_csv_cols();
 		}
 
 		$rows = $this->print_csv_rows();
 
-		if ( $rows ) {
+		if ($rows) {
 			return true;
 		} else {
 			return false;
@@ -176,14 +176,14 @@ 
 		$col_data = '';
 		$cols     = $this->get_csv_cols();
 		$i        = 1;
-		foreach ( $cols as $col_id => $column ) {
-			$col_data .= '"' . addslashes( $column ) . '"';
-			$col_data .= $i == count( $cols ) ? '' : ',';
-			$i ++;
+		foreach ($cols as $col_id => $column) {
+			$col_data .= '"'.addslashes($column).'"';
+			$col_data .= $i == count($cols) ? '' : ',';
+			$i++;
 		}
 		$col_data .= "\r\n";
 
-		$this->stash_step_data( $col_data );
+		$this->stash_step_data($col_data);
 
 		return $col_data;
 
@@ -202,23 +202,23 @@ 
 		$data     = $this->get_data();
 		$cols     = $this->get_csv_cols();
 
-		if ( $data ) {
+		if ($data) {
 
 			// 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 ) ) {
-						$row_data .= '"' . addslashes( preg_replace( '/"/', "'", $column ) ) . '"';
-						$row_data .= $i == count( $cols ) ? '' : ',';
-						$i ++;
+					if (array_key_exists($col_id, $cols)) {
+						$row_data .= '"'.addslashes(preg_replace('/"/', "'", $column)).'"';
+						$row_data .= $i == count($cols) ? '' : ',';
+						$i++;
 					}
 				}
 				$row_data .= "\r\n";
 			}
 
-			$this->stash_step_data( $row_data );
+			$this->stash_step_data($row_data);
 
 			return $row_data;
 		}
@@ -246,18 +246,18 @@ 
 
 		$file = '';
 
-		if ( @file_exists( $this->file ) ) {
+		if (@file_exists($this->file)) {
 
-			if ( ! is_writeable( $this->file ) ) {
+			if ( ! is_writeable($this->file)) {
 				$this->is_writable = false;
 			}
 
-			$file = @file_get_contents( $this->file );
+			$file = @file_get_contents($this->file);
 
 		} else {
 
-			@file_put_contents( $this->file, '' );
-			@chmod( $this->file, 0664 );
+			@file_put_contents($this->file, '');
+			@chmod($this->file, 0664);
 
 		}
 
@@ -273,18 +273,18 @@ 
 	 *
 	 * @return void
 	 */
-	protected function stash_step_data( $data = '' ) {
+	protected function stash_step_data($data = '') {
 
 		$file = $this->get_file();
 		$file .= $data;
-		@file_put_contents( $this->file, $file );
+		@file_put_contents($this->file, $file);
 
 		// If we have no rows after this step, mark it as an empty export.
-		$file_rows    = file( $this->file, FILE_SKIP_EMPTY_LINES );
+		$file_rows    = file($this->file, FILE_SKIP_EMPTY_LINES);
 		$default_cols = $this->get_csv_cols();
-		$default_cols = empty( $default_cols ) ? 0 : 1;
+		$default_cols = empty($default_cols) ? 0 : 1;
 
-		$this->is_empty = count( $file_rows ) == $default_cols ? true : false;
+		$this->is_empty = count($file_rows) == $default_cols ? true : false;
 
 	}
 
@@ -302,7 +302,7 @@ 
 
 		$file = $this->get_file();
 
-		@unlink( $this->file );
+		@unlink($this->file);
 
 		echo $file;
 
@@ -311,7 +311,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( 'give_file_export_complete', $_REQUEST );
+		do_action('give_file_export_complete', $_REQUEST);
 
 		give_die();
 	}
@@ -323,7 +323,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -334,7 +334,7 @@ 
 	 * @param array             $request The Form Data passed into the batch processing.
 	 * @param Give_Batch_Export $export
 	 */
-	public function unset_properties( $request, $export ) {
+	public function unset_properties($request, $export) {
 	}
 
 	/**

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

Doc Comments +1 added lines, -1 removed lines

@@ -143,7 +143,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 +67 added lines, -67 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';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_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 donor-search" 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( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h2 class="alignleft reports-earnings-title">
-					<?php esc_html_e( 'Donors Report', 'give' ); ?>
+					<?php esc_html_e('Donors Report', 'give'); ?>
 				</h2>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,35 +172,35 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['id'] ) ),
-						esc_html( $item['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['id'])),
+						esc_html($item['num_donations'])
 					);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -212,13 +212,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -231,10 +231,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -245,7 +245,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -257,7 +257,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;
 	}
 
 	/**
@@ -268,7 +268,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;
 	}
 
 	/**
@@ -287,15 +287,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -323,9 +323,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);
 	}
 
 	/**
@@ -337,10 +337,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,
@@ -349,10 +349,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;
@@ -379,16 +379,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->reports_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),
+		));
 	}
 }