WordImpress / Give

includes/class-give-logging.php

Doc Comments +1 added lines, -1 removed lines

@@ -512,7 +512,7 @@
 	 * @since  1.7
 	 * @access public
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function delete_cache() {
 		// Add log related keys to delete.

Spacing +137 added lines, -137 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 		 * Setup properties
 		 */
 
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-logs.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-logs-meta.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-db-logs.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-db-logs-meta.php';
 		$this->log_db     = new Give_DB_Logs();
 		$this->logmeta_db = new Give_DB_Log_Meta();
 
@@ -62,22 +62,22 @@ 
 		 * Setup hooks.
 		 */
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache' ) );
-		add_action( 'update_log_metadata', array( $this, 'bc_200_set_payment_as_log_parent' ), 10, 4 );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
+		add_action('update_log_metadata', array($this, 'bc_200_set_payment_as_log_parent'), 10, 4);
 
 		// Backward compatibility.
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Create the log post type
-			add_action( 'init', array( $this, 'register_post_type' ), -2 );
+			add_action('init', array($this, 'register_post_type'), -2);
 		}
 
 		// Create types taxonomy and default types
 		// @todo: remove this taxonomy, some addon use this taxonomy with there custom log post type for example: recurring
 		// Do not use this taxonomy with your log type because we will remove it in future releases.
-		add_action( 'init', array( $this, 'register_taxonomy' ), -2 );
+		add_action('init', array($this, 'register_taxonomy'), -2);
 	}
 
 
@@ -95,7 +95,7 @@ 
 		/* Logs post type */
 		$log_args = array(
 			'labels'              => array(
-				'name' => esc_html__( 'Logs', 'give' ),
+				'name' => esc_html__('Logs', 'give'),
 			),
 			'public'              => false,
 			'exclude_from_search' => true,
@@ -104,11 +104,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true,
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -122,9 +122,9 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array(
+		register_taxonomy('give_log_type', 'give_log', array(
 			'public' => false,
-		) );
+		));
 	}
 
 	/**
@@ -145,7 +145,7 @@ 
 			'update',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -160,8 +160,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -180,7 +180,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = '' ) {
+	public function add($title = '', $message = '', $parent = 0, $type = '') {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -188,7 +188,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -205,12 +205,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = '', $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = '', $paged = null) {
+		return $this->get_connected_logs(array(
 			'log_parent' => $object_id,
 			'paged'      => $paged,
 			'log_type'   => $type,
-		) );
+		));
 	}
 
 	/**
@@ -224,7 +224,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$log_id = 0;
 
 		$defaults = array(
@@ -237,28 +237,28 @@ 
 			'post_status' => 'publish',
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
-		$this->bc_200_validate_params( $args, $log_meta );
+		$args = wp_parse_args($log_data, $defaults);
+		$this->bc_200_validate_params($args, $log_meta);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			global $wpdb;
 
 			// Backward Compatibility.
-			if ( ! $wpdb->get_var( "SELECT ID from {$this->log_db->table_name} ORDER BY id DESC LIMIT 1" ) ) {
-				$latest_log_id = $wpdb->get_var( "SELECT ID from $wpdb->posts ORDER BY id DESC LIMIT 1" );
-				$latest_log_id = empty( $latest_log_id ) ? 1 : ++ $latest_log_id;
+			if ( ! $wpdb->get_var("SELECT ID from {$this->log_db->table_name} ORDER BY id DESC LIMIT 1")) {
+				$latest_log_id = $wpdb->get_var("SELECT ID from $wpdb->posts ORDER BY id DESC LIMIT 1");
+				$latest_log_id = empty($latest_log_id) ? 1 : ++ $latest_log_id;
 
 				$args['ID'] = $latest_log_id;
-				$this->log_db->insert( $args );
+				$this->log_db->insert($args);
 			}
 		}
 
-		$log_id = $this->log_db->add( $args );
+		$log_id = $this->log_db->add($args);
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				$this->logmeta_db->update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				$this->logmeta_db->update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -280,7 +280,7 @@ 
 	 *
 	 * @return bool|null       True if successful, false otherwise.
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 		$log_id = 0;
 
 		/**
@@ -291,7 +291,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'log_parent'  => 0,
@@ -301,28 +301,28 @@ 
 			'post_status' => 'publish',
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
-		$this->bc_200_validate_params( $args, $log_meta );
+		$args = wp_parse_args($log_data, $defaults);
+		$this->bc_200_validate_params($args, $log_meta);
 
 		// Store the log entry
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$log_id = wp_update_post( $args );
+			$log_id = wp_update_post($args);
 
-			if ( $log_id && ! empty( $log_meta ) ) {
-				foreach ( (array) $log_meta as $key => $meta ) {
-					if ( ! empty( $meta ) ) {
-						give_update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+			if ($log_id && ! empty($log_meta)) {
+				foreach ((array) $log_meta as $key => $meta) {
+					if ( ! empty($meta)) {
+						give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 					}
 				}
 			}
 		} else {
-			$log_id = $this->log_db->add( $args );
+			$log_id = $this->log_db->add($args);
 
-			if ( $log_id && ! empty( $log_meta ) ) {
-				foreach ( (array) $log_meta as $key => $meta ) {
-					if ( ! empty( $meta ) ) {
-						$this->logmeta_db->update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+			if ($log_id && ! empty($log_meta)) {
+				foreach ((array) $log_meta as $key => $meta) {
+					if ( ! empty($meta)) {
+						$this->logmeta_db->update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 					}
 				}
 			}
@@ -337,7 +337,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -354,30 +354,30 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 		$logs = array();
 
-		$defaults   = array(
+		$defaults = array(
 			'number'      => 20,
-			'paged'       => get_query_var( 'paged' ),
+			'paged'       => get_query_var('paged'),
 			'log_type'    => false,
 
 			// Backward compatibility.
 			'post_type'   => 'give_log',
 			'post_status' => 'publish',
 		);
-		$query_args = wp_parse_args( $args, $defaults );
-		$this->bc_200_validate_params( $query_args );
+		$query_args = wp_parse_args($args, $defaults);
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs = get_posts( $query_args );
-			$this->bc_200_add_new_properties( $logs );
+			$logs = get_posts($query_args);
+			$this->bc_200_add_new_properties($logs);
 		} else {
-			$logs = $this->log_db->get_logs( $query_args );
+			$logs = $this->log_db->get_logs($query_args);
 		}
 
-		return ( ! empty( $logs ) ? $logs : false );
+		return ( ! empty($logs) ? $logs : false);
 	}
 
 	/**
@@ -395,41 +395,41 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = '', $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = '', $meta_query = null, $date_query = null) {
 		$logs_count = 0;
 
 		$query_args = array(
-			'number'      => - 1,
+			'number'      => -1,
 
 			// Backward comatibility.
 			'post_type'   => 'give_log',
 			'post_status' => 'publish',
 		);
 
-		if ( $object_id ) {
+		if ($object_id) {
 			$query_args['log_parent'] = $object_id;
 		}
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['log_type'] = $type;
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
-		$this->bc_200_validate_params( $query_args );
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs       = new WP_Query( $query_args );
+			$logs       = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 		} else {
-			$logs_count = $this->log_db->count( $query_args );
+			$logs_count = $this->log_db->count($query_args);
 		}
 
 		return $logs_count;
@@ -449,10 +449,10 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = '', $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = '', $meta_query = null) {
 		$query_args = array(
 			'log_parent'  => $object_id,
-			'number'      => - 1,
+			'number'      => -1,
 			'fields'      => 'ID',
 
 			// Backward compatibility.
@@ -460,32 +460,32 @@ 
 			'post_status' => 'publish',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['log_type'] = $type;
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$this->bc_200_validate_params( $query_args );
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs = get_posts( $query_args );
+			$logs = get_posts($query_args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					wp_delete_post( $log, true );
+			if ($logs) {
+				foreach ($logs as $log) {
+					wp_delete_post($log, true);
 				}
 			}
 		} else {
-			$logs = $this->log_db->get_logs( $query_args );
+			$logs = $this->log_db->get_logs($query_args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					if ( $this->log_db->delete( $log->ID ) ) {
-						$this->logmeta_db->delete_row( $log->ID );
+			if ($logs) {
+				foreach ($logs as $log) {
+					if ($this->log_db->delete($log->ID)) {
+						$this->logmeta_db->delete_row($log->ID);
 					}
 				}
 			}
@@ -502,9 +502,9 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
+	public function background_process_delete_cache($post_id) {
 		// Delete log cache immediately
-		wp_schedule_single_event( time() - 5, 'give_delete_log_cache' );
+		wp_schedule_single_event(time() - 5, 'give_delete_log_cache');
 	}
 
 	/**
@@ -517,17 +517,17 @@ 
 	 */
 	public function delete_cache() {
 		// Add log related keys to delete.
-		$cache_give_logs      = Give_Cache::get_options_like( 'give_logs' );
-		$cache_give_log_count = Give_Cache::get_options_like( 'log_count' );
+		$cache_give_logs      = Give_Cache::get_options_like('give_logs');
+		$cache_give_log_count = Give_Cache::get_options_like('log_count');
 
-		$cache_option_names = array_merge( $cache_give_logs, $cache_give_log_count );
+		$cache_option_names = array_merge($cache_give_logs, $cache_give_log_count);
 
 		// Bailout.
-		if ( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
-		Give_Cache::delete( $cache_option_names );
+		Give_Cache::delete($cache_option_names);
 	}
 
 	/**
@@ -539,7 +539,7 @@ 
 	 * @param array $log_query
 	 * @param array $log_meta
 	 */
-	private function bc_200_validate_params( &$log_query, &$log_meta = array() ) {
+	private function bc_200_validate_params(&$log_query, &$log_meta = array()) {
 		$query_params = array(
 			'log_title'    => 'post_title',
 			'log_parent'   => 'post_parent',
@@ -551,41 +551,41 @@ 
 			'meta_query'   => 'meta_query',
 		);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Set old params.
-			foreach ( $query_params as $new_query_param => $old_query_param ) {
+			foreach ($query_params as $new_query_param => $old_query_param) {
 
-				if ( isset( $log_query[ $old_query_param ] ) && empty( $log_query[ $new_query_param ] ) ) {
-					$log_query[ $new_query_param ] = $log_query[ $old_query_param ];
+				if (isset($log_query[$old_query_param]) && empty($log_query[$new_query_param])) {
+					$log_query[$new_query_param] = $log_query[$old_query_param];
 					continue;
-				} elseif ( ! isset( $log_query[ $new_query_param ] ) ) {
+				} elseif ( ! isset($log_query[$new_query_param])) {
 					continue;
-				} elseif( empty( $log_query[ $new_query_param ] ) ) {
+				} elseif (empty($log_query[$new_query_param])) {
 					continue;
 				}
 
-				switch ( $new_query_param ) {
+				switch ($new_query_param) {
 					case 'log_type':
 						$log_query['tax_query'] = array(
 							array(
 								'taxonomy' => 'give_log_type',
 								'field'    => 'slug',
-								'terms'    => $log_query[ $new_query_param ],
+								'terms'    => $log_query[$new_query_param],
 							),
 						);
 						break;
 
 					case 'meta_query':
-						if( ! empty( $log_query['meta_query'] ) && empty( $log_query['post_parent'] ) ) {
-							foreach ( $log_query['meta_query'] as $index => $meta_query ){
-								if( ! is_array( $meta_query ) || empty( $meta_query['key'] ) ) {
+						if ( ! empty($log_query['meta_query']) && empty($log_query['post_parent'])) {
+							foreach ($log_query['meta_query'] as $index => $meta_query) {
+								if ( ! is_array($meta_query) || empty($meta_query['key'])) {
 									continue;
 								}
 
-								switch ( $meta_query['key'] ) {
+								switch ($meta_query['key']) {
 									case '_give_log_form_id':
 										$log_query['post_parent'] = $meta_query['value'];
-										unset( $log_query['meta_query'][$index] );
+										unset($log_query['meta_query'][$index]);
 										break;
 								}
 							}
@@ -593,40 +593,40 @@ 
 						break;
 
 					default:
-						switch( $new_query_param ){
+						switch ($new_query_param) {
 							case 'log_parent':
 								$log_query['meta_query'][] = array(
 									'key' => '_give_log_payment_id',
-									'value' => $log_query[ $new_query_param ]
+									'value' => $log_query[$new_query_param]
 								);
 
 								break;
 
 							default:
-								$log_query[ $old_query_param ] = $log_query[ $new_query_param ];
+								$log_query[$old_query_param] = $log_query[$new_query_param];
 						}
 				}
 			}
 		} else {
 			// Set only old params.
-			$query_params = array_flip( $query_params );
-			foreach ( $query_params as $old_query_param => $new_query_param ) {
-				if ( isset( $log_query[ $new_query_param ] ) && empty( $log_query[ $old_query_param ] ) ) {
-					$log_query[ $old_query_param ] = $log_query[ $new_query_param ];
+			$query_params = array_flip($query_params);
+			foreach ($query_params as $old_query_param => $new_query_param) {
+				if (isset($log_query[$new_query_param]) && empty($log_query[$old_query_param])) {
+					$log_query[$old_query_param] = $log_query[$new_query_param];
 					continue;
-				} elseif ( ! isset( $log_query[ $old_query_param ] ) ) {
+				} elseif ( ! isset($log_query[$old_query_param])) {
 					continue;
 				}
 
-				switch ( $old_query_param ) {
+				switch ($old_query_param) {
 					case 'tax_query':
-						if ( isset( $log_query[ $old_query_param ][0]['terms'] ) ) {
-							$log_query[ $new_query_param ] = $log_query[ $old_query_param ][0]['terms'];
+						if (isset($log_query[$old_query_param][0]['terms'])) {
+							$log_query[$new_query_param] = $log_query[$old_query_param][0]['terms'];
 						}
 						break;
 
 					default:
-						$log_query[ $new_query_param ] = $log_query[ $old_query_param ];
+						$log_query[$new_query_param] = $log_query[$old_query_param];
 				}
 			}
 		}
@@ -640,8 +640,8 @@ 
 	 *
 	 * @param  array $logs
 	 */
-	private function bc_200_add_new_properties( &$logs ) {
-		if ( empty( $logs ) ) {
+	private function bc_200_add_new_properties(&$logs) {
+		if (empty($logs)) {
 			return;
 		}
 
@@ -654,30 +654,30 @@ 
 			'log_type'     => 'give_log_type',
 		);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
-			foreach ( $logs as $index => $log ) {
-				foreach ( $query_params as $new_query_param => $old_query_param ) {
-					if ( ! property_exists( $log, $old_query_param ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
+			foreach ($logs as $index => $log) {
+				foreach ($query_params as $new_query_param => $old_query_param) {
+					if ( ! property_exists($log, $old_query_param)) {
 						/**
 						 *  Set unmatched properties.
 						 */
 
 						// 1. log_type
-						$term = get_the_terms( $log->ID, 'give_log_type' );
-						$term = ! is_wp_error( $term ) && ! empty( $term ) ? $term[0] : array();
+						$term = get_the_terms($log->ID, 'give_log_type');
+						$term = ! is_wp_error($term) && ! empty($term) ? $term[0] : array();
 
-						$logs[ $index ]->{$new_query_param} = ! empty( $term ) ? $term->slug : '';
+						$logs[$index]->{$new_query_param} = ! empty($term) ? $term->slug : '';
 
 						continue;
 					}
 
-					switch ( $old_query_param ) {
+					switch ($old_query_param) {
 						case 'post_parent':
-							$logs[ $index ]->{$new_query_param} = give_get_meta( $log->ID, '_give_log_payment_id', true );
+							$logs[$index]->{$new_query_param} = give_get_meta($log->ID, '_give_log_payment_id', true);
 							break;
 
 						default:
-							$logs[ $index ]->{$new_query_param} = $log->{$old_query_param};
+							$logs[$index]->{$new_query_param} = $log->{$old_query_param};
 					}
 				}
 			}
@@ -697,10 +697,10 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function bc_200_set_payment_as_log_parent( $check, $log_id, $meta_key, $meta_value ) {
+	public function bc_200_set_payment_as_log_parent($check, $log_id, $meta_key, $meta_value) {
 		global $wpdb;
 		$update_status = false;
-		$post_type     = get_post_type( $log_id );
+		$post_type     = get_post_type($log_id);
 
 		// Bailout.
 		if (
@@ -720,9 +720,9 @@ 
 			)
 		);
 
-		if ( $form_id ) {
-			$this->logmeta_db->delete_meta( $log_id, '_give_log_payment_id' );
-			$this->logmeta_db->update_meta( $log_id, '_give_log_form_id', $form_id );
+		if ($form_id) {
+			$this->logmeta_db->delete_meta($log_id, '_give_log_payment_id');
+			$this->logmeta_db->update_meta($log_id, '_give_log_form_id', $form_id);
 
 			$update_status = $wpdb->update(
 				$this->log_db->table_name,

includes/install.php

Doc Comments +1 added lines, -1 removed lines

@@ -397,7 +397,7 @@
  *
  * @since 1.8.11
  *
- * @return void
+ * @return false|null
  */
 function give_create_pages() {
 

Spacing +60 added lines, -60 removed lines

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

includes/payments/backward-compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
  *                              metadata entries with the specified value.
  *                              Otherwise, update all entries.
  *
- * @return mixed
+ * @return null|boolean
  */
 function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
 	// Bailout.

Spacing +166 added lines, -171 removed lines

@@ -9,74 +9,74 @@ 
  *
  * @return void
  */
-function _give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value ) {
+function _give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value) {
 	// Bailout
-	if ( empty( $meta_value ) ) {
+	if (empty($meta_value)) {
 		return;
-	} elseif ( ! is_array( $meta_value ) ) {
+	} elseif ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10 );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10);
 
 	// Date payment meta.
-	if ( ! empty( $meta_value['date'] ) ) {
-		give_update_meta( $object_id, '_give_payment_date', $meta_value['date'] );
+	if ( ! empty($meta_value['date'])) {
+		give_update_meta($object_id, '_give_payment_date', $meta_value['date']);
 	}
 
 	// Currency payment meta.
-	if ( ! empty( $meta_value['currency'] ) ) {
-		give_update_meta( $object_id, '_give_payment_currency', $meta_value['currency'] );
+	if ( ! empty($meta_value['currency'])) {
+		give_update_meta($object_id, '_give_payment_currency', $meta_value['currency']);
 	}
 
 	// User information.
-	if ( ! empty( $meta_value['user_info'] ) ) {
+	if ( ! empty($meta_value['user_info'])) {
 		// Donor first name.
-		if ( ! empty( $meta_value['user_info']['first_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name'] );
+		if ( ! empty($meta_value['user_info']['first_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name']);
 		}
 
 		// Donor last name.
-		if ( ! empty( $meta_value['user_info']['last_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name'] );
+		if ( ! empty($meta_value['user_info']['last_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name']);
 		}
 
 		// Donor address payment meta.
-		if ( ! empty( $meta_value['user_info']['address'] ) ) {
+		if ( ! empty($meta_value['user_info']['address'])) {
 
 			// Address1.
-			if ( ! empty( $meta_value['user_info']['address']['line1'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1'] );
+			if ( ! empty($meta_value['user_info']['address']['line1'])) {
+				give_update_meta($object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1']);
 			}
 
 			// Address2.
-			if ( ! empty( $meta_value['user_info']['address']['line2'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2'] );
+			if ( ! empty($meta_value['user_info']['address']['line2'])) {
+				give_update_meta($object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2']);
 			}
 
 			// City.
-			if ( ! empty( $meta_value['user_info']['address']['city'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city'] );
+			if ( ! empty($meta_value['user_info']['address']['city'])) {
+				give_update_meta($object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city']);
 			}
 
 			// Zip.
-			if ( ! empty( $meta_value['user_info']['address']['zip'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip'] );
+			if ( ! empty($meta_value['user_info']['address']['zip'])) {
+				give_update_meta($object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip']);
 			}
 
 			// State.
-			if ( ! empty( $meta_value['user_info']['address']['state'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state'] );
+			if ( ! empty($meta_value['user_info']['address']['state'])) {
+				give_update_meta($object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state']);
 			}
 
 			// Country.
-			if ( ! empty( $meta_value['user_info']['address']['country'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country'] );
+			if ( ! empty($meta_value['user_info']['address']['country'])) {
+				give_update_meta($object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country']);
 			}
 		}
 	}// End if().
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 }
 
 /**
@@ -89,103 +89,99 @@ 
  *
  * @return array
  */
-function _give_20_bc_give_payment_meta_value( $object_id, $meta_value ) {
+function _give_20_bc_give_payment_meta_value($object_id, $meta_value) {
 	$cache_key = "_give_payment_meta_{$object_id}";
-	$cache     = Give_Cache::get_db_query( $cache_key );
+	$cache     = Give_Cache::get_db_query($cache_key);
 
-	if ( ! is_null( $cache ) ) {
+	if ( ! is_null($cache)) {
 		return $cache;
 	}
 
 	// Set default value to array.
-	if ( ! is_array( $meta_value ) ) {
+	if ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
 	// Donation key.
-	$meta_value['key'] = give_get_meta( $object_id, '_give_payment_purchase_key', true );
+	$meta_value['key'] = give_get_meta($object_id, '_give_payment_purchase_key', true);
 
 	// Donation form.
-	$meta_value['form_title'] = give_get_meta( $object_id, '_give_payment_form_title', true );
+	$meta_value['form_title'] = give_get_meta($object_id, '_give_payment_form_title', true);
 
 	// Donor email.
-	$meta_value['email'] = give_get_meta( $object_id, '_give_payment_donor_email', true );
-	$meta_value['email'] = ! empty( $meta_value['email'] ) ?
-		$meta_value['email'] :
-		Give()->donors->get_column( 'email', give_get_payment_donor_id( $object_id ) );
+	$meta_value['email'] = give_get_meta($object_id, '_give_payment_donor_email', true);
+	$meta_value['email'] = ! empty($meta_value['email']) ?
+		$meta_value['email'] : Give()->donors->get_column('email', give_get_payment_donor_id($object_id));
 
 	// Form id.
-	$meta_value['form_id'] = give_get_meta( $object_id, '_give_payment_form_id', true );
+	$meta_value['form_id'] = give_get_meta($object_id, '_give_payment_form_id', true);
 
 	// Price id.
-	$meta_value['price_id'] = give_get_meta( $object_id, '_give_payment_price_id', true );
+	$meta_value['price_id'] = give_get_meta($object_id, '_give_payment_price_id', true);
 
 	// Date.
-	$meta_value['date'] = give_get_meta( $object_id, '_give_payment_date', true );
-	$meta_value['date'] = ! empty( $meta_value['date'] ) ?
-		$meta_value['date'] :
-		get_post_field( 'post_date', $object_id );
+	$meta_value['date'] = give_get_meta($object_id, '_give_payment_date', true);
+	$meta_value['date'] = ! empty($meta_value['date']) ?
+		$meta_value['date'] : get_post_field('post_date', $object_id);
 
 	// Currency.
-	$meta_value['currency'] = give_get_meta( $object_id, '_give_payment_currency', true );
+	$meta_value['currency'] = give_get_meta($object_id, '_give_payment_currency', true);
 
 	// Decode donor data.
-	$donor_names = give_get_donor_name_by( give_get_meta( $object_id, '_give_payment_donor_id', true ), 'donor' );
-	$donor_names = explode( ' ', $donor_names, 2 );
+	$donor_names = give_get_donor_name_by(give_get_meta($object_id, '_give_payment_donor_id', true), 'donor');
+	$donor_names = explode(' ', $donor_names, 2);
 
 	// Donor first name.
-	$donor_data['first_name'] = give_get_meta( $object_id, '_give_donor_billing_first_name', true );
-	$donor_data['first_name'] = ! empty( $donor_data['first_name'] ) ?
-		$donor_data['first_name'] :
-		$donor_names[0];
+	$donor_data['first_name'] = give_get_meta($object_id, '_give_donor_billing_first_name', true);
+	$donor_data['first_name'] = ! empty($donor_data['first_name']) ?
+		$donor_data['first_name'] : $donor_names[0];
 
 	// Donor last name.
-	$donor_data['last_name'] = give_get_meta( $object_id, '_give_donor_billing_last_name', true );
-	$donor_data['last_name'] = ! empty( $donor_data['last_name'] ) ?
-		$donor_data['last_name'] :
-		( isset( $donor_names[1] ) ? $donor_names[1] : '' );
+	$donor_data['last_name'] = give_get_meta($object_id, '_give_donor_billing_last_name', true);
+	$donor_data['last_name'] = ! empty($donor_data['last_name']) ?
+		$donor_data['last_name'] : (isset($donor_names[1]) ? $donor_names[1] : '');
 
 	// Donor email.
 	$donor_data['email'] = $meta_value['email'];
 
 	// User ID.
-	$donor_data['id'] = give_get_payment_user_id( $object_id );
+	$donor_data['id'] = give_get_payment_user_id($object_id);
 
 	$donor_data['address'] = false;
 
 	// Address1.
-	if ( $address1 = give_get_meta( $object_id, '_give_donor_billing_address1', true ) ) {
+	if ($address1 = give_get_meta($object_id, '_give_donor_billing_address1', true)) {
 		$donor_data['address']['line1'] = $address1;
 	}
 
 	// Address2.
-	if ( $address2 = give_get_meta( $object_id, '_give_donor_billing_address2', true ) ) {
+	if ($address2 = give_get_meta($object_id, '_give_donor_billing_address2', true)) {
 		$donor_data['address']['line2'] = $address2;
 	}
 
 	// City.
-	if ( $city = give_get_meta( $object_id, '_give_donor_billing_city', true ) ) {
+	if ($city = give_get_meta($object_id, '_give_donor_billing_city', true)) {
 		$donor_data['address']['city'] = $city;
 	}
 
 	// Zip.
-	if ( $zip = give_get_meta( $object_id, '_give_donor_billing_zip', true ) ) {
+	if ($zip = give_get_meta($object_id, '_give_donor_billing_zip', true)) {
 		$donor_data['address']['zip'] = $zip;
 	}
 
 	// State.
-	if ( $state = give_get_meta( $object_id, '_give_donor_billing_state', true ) ) {
+	if ($state = give_get_meta($object_id, '_give_donor_billing_state', true)) {
 		$donor_data['address']['state'] = $state;
 	}
 
 	// Country.
-	if ( $country = give_get_meta( $object_id, '_give_donor_billing_country', true ) ) {
+	if ($country = give_get_meta($object_id, '_give_donor_billing_country', true)) {
 		$donor_data['address']['country'] = $country;
 	}
 
-	$meta_value['user_info'] = maybe_unserialize( $donor_data );
+	$meta_value['user_info'] = maybe_unserialize($donor_data);
 
-	Give_Cache::set_db_query( $cache_key, $meta_value );
+	Give_Cache::set_db_query($cache_key, $meta_value);
 
 	return $meta_value;
 }
@@ -209,37 +205,37 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
+function _give_20_bc_saving_old_payment_meta($check, $object_id, $meta_key, $meta_value, $prev_value) {
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, array(
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, array(
 			'_give_payment_meta',
 			'_give_payment_user_email',
 			'_give_payment_customer_id',
 			'give_payment_user_ip',
-		) )
+		))
 	) {
 		return $check;
 	}
 
-	if ( '_give_payment_meta' === $meta_key ) {
-		_give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value );
-	} elseif ( '_give_payment_user_email' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_email', $meta_value );
+	if ('_give_payment_meta' === $meta_key) {
+		_give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value);
+	} elseif ('_give_payment_user_email' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_email', $meta_value);
 		$check = true;
-	} elseif ( '_give_payment_customer_id' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_id', $meta_value );
+	} elseif ('_give_payment_customer_id' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_id', $meta_value);
 		$check = true;
-	} elseif ( 'give_payment_user_ip' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_ip', $meta_value );
+	} elseif ('give_payment_user_ip' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_ip', $meta_value);
 		$check = true;
 	}
 
 	return $check;
 }
 
-add_filter( 'update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5 );
+add_filter('update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5);
 
 
 /**
@@ -254,7 +250,7 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_old_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_old_payment_meta($check, $object_id, $meta_key, $single) {
 	global $wpdb;
 
 	// Deprecated meta keys.
@@ -265,32 +261,31 @@ 
 	);
 
 	// Add _give_payment_meta to backward compatibility
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		$old_meta_keys[] = '_give_payment_meta';
 	}
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $old_meta_keys )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $old_meta_keys)
 	) {
 		return $check;
 	}
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check     = Give_Cache::get_db_query( $cache_key );
+	$check     = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle old meta keys.
 			case '_give_payment_meta':
-				remove_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta' );
+				remove_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta');
 
 				// if ( $meta_value = give_get_meta( $object_id, '_give_payment_meta' ) ) {
-				$meta_value = ! empty( $meta_value ) ?
-					current( $meta_value ) :
-					(array) maybe_unserialize(
+				$meta_value = ! empty($meta_value) ?
+					current($meta_value) : (array) maybe_unserialize(
 						$wpdb->get_var(
 							$wpdb->prepare(
 								"
@@ -304,44 +299,44 @@ 
 							)
 						)
 					);
-				$check      = _give_20_bc_give_payment_meta_value( $object_id, $meta_value );
+				$check = _give_20_bc_give_payment_meta_value($object_id, $meta_value);
 				// }
 
-				add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+				add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 				break;
 
 			case '_give_payment_customer_id':
-				if ( $donor_id = give_get_meta( $object_id, '_give_payment_donor_id', $single ) ) {
+				if ($donor_id = give_get_meta($object_id, '_give_payment_donor_id', $single)) {
 					$check = $donor_id;
 				}
 				break;
 
 			case '_give_payment_user_email':
-				if ( $donor_email = give_get_meta( $object_id, '_give_payment_donor_email', $single ) ) {
+				if ($donor_email = give_get_meta($object_id, '_give_payment_donor_email', $single)) {
 					$check = $donor_email;
 				}
 				break;
 
 			case '_give_payment_user_ip':
-				if ( $donor_ip = give_get_meta( $object_id, '_give_payment_donor_ip', $single ) ) {
+				if ($donor_ip = give_get_meta($object_id, '_give_payment_donor_ip', $single)) {
 					$check = $donor_ip;
 				}
 				break;
 		}// End switch().
 
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! is_null( $check ) ) {
-		$check = array( $check );
+	if ( ! is_null($check)) {
+		$check = array($check);
 	}
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 
 /**
@@ -356,9 +351,9 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_new_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_new_payment_meta($check, $object_id, $meta_key, $single) {
 	// Bailout: do not apply backward compatibility if upgrade done.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return $check;
 	}
 
@@ -380,26 +375,26 @@ 
 	);
 
 	// metadata_exists fx will cause of firing get_post_metadata filter again so remove it to prevent infinite loop.
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta' );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta');
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $new_meta_keys ) ||
-		metadata_exists( 'post', $object_id, $meta_key )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $new_meta_keys) ||
+		metadata_exists('post', $object_id, $meta_key)
 	) {
-		add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+		add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 		return $check;
 	}
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check    = Give_Cache::get_db_query( $cache_key );
+	$check = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle new meta keys.
 			case '_give_payment_donor_id':
@@ -442,9 +437,9 @@ 
 			case '_give_donor_billing_country':
 			case '_give_payment_date':
 			case '_give_payment_currency':
-				$donation_meta = Give_Cache::get_db_query( "_give_payment_meta_{$object_id}" );
+				$donation_meta = Give_Cache::get_db_query("_give_payment_meta_{$object_id}");
 
-				if ( is_null( $donation_meta ) ) {
+				if (is_null($donation_meta)) {
 					$donation_meta = $wpdb->get_var(
 						$wpdb->prepare(
 							"SELECT meta_value FROM {$wpdb->postmeta} WHERE post_id=%d AND meta_key=%s",
@@ -452,51 +447,51 @@ 
 							'_give_payment_meta'
 						)
 					);
-					$donation_meta = maybe_unserialize( $donation_meta );
-					$donation_meta = ! is_array( $donation_meta ) ? array() : $donation_meta;
-					Give_Cache::set_db_query( "_give_payment_meta_{$object_id}", $donation_meta );
+					$donation_meta = maybe_unserialize($donation_meta);
+					$donation_meta = ! is_array($donation_meta) ? array() : $donation_meta;
+					Give_Cache::set_db_query("_give_payment_meta_{$object_id}", $donation_meta);
 				}
 
 				// Get results.
-				if ( empty( $donation_meta ) ) {
+				if (empty($donation_meta)) {
 					$check = '';
-				} elseif ( in_array( $meta_key, array( '_give_payment_date', '_give_payment_currency' ) ) ) {
-					$payment_meta_key = str_replace( '_give_payment_', '', $meta_key );
+				} elseif (in_array($meta_key, array('_give_payment_date', '_give_payment_currency'))) {
+					$payment_meta_key = str_replace('_give_payment_', '', $meta_key);
 
-					if ( isset( $donation_meta[ $payment_meta_key ] ) ) {
-						$check = $donation_meta[ $payment_meta_key ];
+					if (isset($donation_meta[$payment_meta_key])) {
+						$check = $donation_meta[$payment_meta_key];
 					}
 				} else {
-					$payment_meta_key = str_replace( '_give_donor_billing_', '', $meta_key );
+					$payment_meta_key = str_replace('_give_donor_billing_', '', $meta_key);
 
-					switch ( $payment_meta_key ) {
+					switch ($payment_meta_key) {
 						case 'address1':
-							if ( isset( $donation_meta['user_info']['address']['line1'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line1'])) {
 								$check = $donation_meta['user_info']['address']['line1'];
 							}
 							break;
 
 						case 'address2':
-							if ( isset( $donation_meta['user_info']['address']['line2'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line2'])) {
 								$check = $donation_meta['user_info']['address']['line2'];
 							}
 							break;
 
 						case 'first_name':
-							if ( isset( $donation_meta['user_info']['first_name'] ) ) {
+							if (isset($donation_meta['user_info']['first_name'])) {
 								$check = $donation_meta['user_info']['first_name'];
 							}
 							break;
 
 						case 'last_name':
-							if ( isset( $donation_meta['user_info']['last_name'] ) ) {
+							if (isset($donation_meta['user_info']['last_name'])) {
 								$check = $donation_meta['user_info']['last_name'];
 							}
 							break;
 
 						default:
-							if ( isset( $donation_meta['user_info']['address'][ $payment_meta_key ] ) ) {
-								$check = $donation_meta['user_info']['address'][ $payment_meta_key ];
+							if (isset($donation_meta['user_info']['address'][$payment_meta_key])) {
+								$check = $donation_meta['user_info']['address'][$payment_meta_key];
 							}
 					}
 				}
@@ -505,19 +500,19 @@ 
 		}// End switch().
 
 		// Set cache.
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! $single ) {
-		$check = array( $check );
+	if ( ! $single) {
+		$check = array($check);
 	}
 
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 
 /**
@@ -529,9 +524,9 @@ 
  *
  * @return void
  */
-function _give_20_bc_support_deprecated_meta_key_query( $query ) {
+function _give_20_bc_support_deprecated_meta_key_query($query) {
 	// Bailout.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
@@ -541,21 +536,21 @@ 
 		// '_give_payment_user_ip'     => '_give_payment_donor_ip',
 	);
 
-	$deprecated_meta_keys = array_flip( $new_meta_keys );
+	$deprecated_meta_keys = array_flip($new_meta_keys);
 
 	// Set meta keys.
 	$meta_keys = array();
 
 
 	// Bailout.
-	if ( ! empty( $query->query_vars['meta_key'] ) ) {
-		if ( in_array( $query->query_vars['meta_key'], $new_meta_keys ) ) {
+	if ( ! empty($query->query_vars['meta_key'])) {
+		if (in_array($query->query_vars['meta_key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $query->query_vars['meta_key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($query->query_vars['meta_key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -566,35 +561,35 @@ 
 						'value' => $query->query_vars['meta_value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_key']],
 						'value' => $query->query_vars['meta_value'],
 					),
 				)
 			);
 
 			// Unset single meta query.
-			unset( $query->query_vars['meta_key'] );
-			unset( $query->query_vars['meta_value'] );
+			unset($query->query_vars['meta_key']);
+			unset($query->query_vars['meta_value']);
 		}
 	} elseif (
-		! empty( $query->query_vars['meta_query'] ) &&
-		( 1 === count( $query->query_vars['meta_query'] ) )
+		! empty($query->query_vars['meta_query']) &&
+		(1 === count($query->query_vars['meta_query']))
 	) {
-		$meta_query = current( $query->query_vars['meta_query'] );
+		$meta_query = current($query->query_vars['meta_query']);
 
-		if ( empty( $meta_query[0]['key'] ) ) {
+		if (empty($meta_query[0]['key'])) {
 			return;
 		}
 
-		if ( in_array( $meta_query[0]['key'], $new_meta_keys ) ) {
+		if (in_array($meta_query[0]['key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $meta_query[0]['key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($meta_query[0]['key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		} else {
 			return;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -605,7 +600,7 @@ 
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_query'][0]['key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_query'][0]['key']],
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 				)
@@ -614,7 +609,7 @@ 
 	}
 }
 
-add_action( 'pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query' );
+add_action('pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query');
 
 
 /**
@@ -627,49 +622,49 @@ 
  * @param Give_Payment $payment
  * @param string       $key
  */
-function _give_20_bc_payment_save( $payment, $key ) {
+function _give_20_bc_payment_save($payment, $key) {
 	// Bailout.
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
-	switch ( $key ) {
+	switch ($key) {
 		case 'user_info':
-			if ( empty( $payment->user_info ) ) {
+			if (empty($payment->user_info)) {
 				// Bailout.
 				break;
-			} elseif ( is_string( $payment->user_info ) ) {
+			} elseif (is_string($payment->user_info)) {
 				// Check if value serialize.
-				$payment->user_info = maybe_unserialize( $payment->user_info );
+				$payment->user_info = maybe_unserialize($payment->user_info);
 			}
 
 
 			// Save first name.
-			if ( isset( $payment->user_info['first_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_first_name', $payment->user_info['first_name'] );
+			if (isset($payment->user_info['first_name'])) {
+				$payment->update_meta('_give_donor_billing_first_name', $payment->user_info['first_name']);
 			}
 
 
 			// Save last name.
-			if ( isset( $payment->user_info['last_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_last_name', $payment->user_info['last_name'] );
+			if (isset($payment->user_info['last_name'])) {
+				$payment->update_meta('_give_donor_billing_last_name', $payment->user_info['last_name']);
 			}
 
 
 			// Save address.
-			if ( ! empty( $payment->user_info['address'] ) ) {
-				foreach ( $payment->user_info['address'] as $address_name => $address ) {
-					switch ( $address_name ) {
+			if ( ! empty($payment->user_info['address'])) {
+				foreach ($payment->user_info['address'] as $address_name => $address) {
+					switch ($address_name) {
 						case 'line1':
-							$payment->update_meta( '_give_donor_billing_address1', $address );
+							$payment->update_meta('_give_donor_billing_address1', $address);
 							break;
 
 						case 'line2':
-							$payment->update_meta( '_give_donor_billing_address2', $address );
+							$payment->update_meta('_give_donor_billing_address2', $address);
 							break;
 
 						default:
-							$payment->update_meta( "_give_donor_billing_{$address_name}", $address );
+							$payment->update_meta("_give_donor_billing_{$address_name}", $address);
 					}
 				}
 			}
@@ -678,7 +673,7 @@ 
 	}
 }
 
-add_action( 'give_payment_save', '_give_20_bc_payment_save', 10, 2 );
+add_action('give_payment_save', '_give_20_bc_payment_save', 10, 2);
 
 
 /**
@@ -691,16 +686,16 @@ 
  *
  * @return mixed
  */
-function __give_20_bc_flush_cache( $check, $object_id ) {
+function __give_20_bc_flush_cache($check, $object_id) {
 	if (
-		! give_has_upgrade_completed( 'v20_move_metadata_into_new_table' ) &&
-		'give_payment' === get_post_type( $object_id )
+		! give_has_upgrade_completed('v20_move_metadata_into_new_table') &&
+		'give_payment' === get_post_type($object_id)
 	) {
-		Give_Cache::delete_group( $object_id, 'give-donations' );
+		Give_Cache::delete_group($object_id, 'give-donations');
 	}
 
 	return $check;
 }
 
-add_action( 'update_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
-add_action( 'add_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
+add_action('update_post_metadata', '__give_20_bc_flush_cache', 9999, 2);
+add_action('add_post_metadata', '__give_20_bc_flush_cache', 9999, 2);

includes/payments/class-payments-query.php

Doc Comments +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 * @param $value
 	 */
 	public function __set( $query_var, $value ) {
@@ -124,7 +124,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 */
 	public function __unset( $query_var ) {
 		unset( $this->args[ $query_var ] );
@@ -468,7 +468,7 @@ 
 	 * @param string   $order
 	 * @param WP_Query $query
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function custom_orderby( $order, $query ) {
 

Spacing +149 added lines, -149 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -66,10 +66,10 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments',
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -96,7 +96,7 @@ 
 			'count'           => false,
 		);
 
-		$this->args = $this->_args = wp_parse_args( $args, $defaults );
+		$this->args = $this->_args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -110,11 +110,11 @@ 
 	 * @param $query_var
 	 * @param $value
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -126,8 +126,8 @@ 
 	 *
 	 * @param $query_var
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 		$this->give_forms();
 		$this->gateway_filter();
 
-		add_filter( 'posts_orderby', array( $this, 'custom_orderby' ), 10, 2 );
+		add_filter('posts_orderby', array($this, 'custom_orderby'), 10, 2);
 
 		/**
 		 * Fires after setup filters.
@@ -176,7 +176,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @access private
 	 */
 	private function unset_filters() {
-		remove_filter( 'posts_orderby', array( $this, 'custom_orderby' ) );
+		remove_filter('posts_orderby', array($this, 'custom_orderby'));
 
 		/**
 		 * Fires after retrieving payments.
@@ -195,7 +195,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 	}
 
 
@@ -212,11 +212,11 @@ 
 	 * @return array
 	 */
 	public function get_payments() {
-		$cache_key      = Give_Cache::get_key( 'give_payment_query', $this->args, false );
-		$this->payments = Give_Cache::get_db_query( $cache_key );
+		$cache_key      = Give_Cache::get_key('give_payment_query', $this->args, false);
+		$this->payments = Give_Cache::get_db_query($cache_key);
 
 		// Return cached result.
-		if ( ! is_null( $this->payments ) ) {
+		if ( ! is_null($this->payments)) {
 			return $this->payments;
 		}
 
@@ -224,7 +224,7 @@ 
 		// Modify the query/query arguments before we retrieve payments.
 		$this->set_filters();
 
-		$query          = new WP_Query( $this->args );
+		$query          = new WP_Query($this->args);
 		$this->payments = array();
 
 		$custom_output = array(
@@ -232,24 +232,24 @@ 
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
-			while ( $query->have_posts() ) {
+		if ($query->have_posts()) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 		}
 
-		Give_Cache::set_db_query( $cache_key, $this->payments );
+		Give_Cache::set_db_query($cache_key, $this->payments);
 
 		// Remove query filters after we retrieve payments.
 		$this->unset_filters();
@@ -268,31 +268,31 @@ 
 	public function get_payment_by_group() {
 		global $wpdb;
 
-		$allowed_groups = array( 'post_status' );
+		$allowed_groups = array('post_status');
 		$result         = array();
 
 
-		if ( in_array( $this->args['group_by'], $allowed_groups ) ) {
+		if (in_array($this->args['group_by'], $allowed_groups)) {
 			// Set only count in result.
-			if ( $this->args['count'] ) {
+			if ($this->args['count']) {
 
 				$this->set_filters();
 
-				$new_results = $wpdb->get_results( $this->get_sql(), ARRAY_N );
+				$new_results = $wpdb->get_results($this->get_sql(), ARRAY_N);
 
 				$this->unset_filters();
 
-				foreach ( $new_results as $results ) {
-					$result[ $results[0] ] = $results[1];
+				foreach ($new_results as $results) {
+					$result[$results[0]] = $results[1];
 				}
 
-				switch ( $this->args['group_by'] ) {
+				switch ($this->args['group_by']) {
 					case 'post_status':
 
 						/* @var Give_Payment $donation */
-						foreach ( give_get_payment_status_keys() as $status ) {
-							if ( ! isset( $result[ $status ] ) ) {
-								$result[ $status ] = 0;
+						foreach (give_get_payment_status_keys() as $status) {
+							if ( ! isset($result[$status])) {
+								$result[$status] = 0;
 							}
 						}
 
@@ -302,8 +302,8 @@ 
 				$donations = $this->get_payments();
 
 				/* @var $donation Give_Payment */
-				foreach ( $donations as $donation ) {
-					$result[ $donation->{$this->args['group_by']} ][] = $donation;
+				foreach ($donations as $donation) {
+					$result[$donation->{$this->args['group_by']}][] = $donation;
 				}
 			}
 		}
@@ -314,7 +314,7 @@ 
 		 *
 		 * @since 1.8.17
 		 */
-		return apply_filters( 'give_get_payment_by_group', $result, $this );
+		return apply_filters('give_get_payment_by_group', $result, $this);
 	}
 
 	/**
@@ -326,29 +326,29 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
-		$is_start_date = property_exists( __CLASS__, 'start_date' );
-		$is_end_date   = property_exists( __CLASS__, 'end_date' );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
+		$is_start_date = property_exists(__CLASS__, 'start_date');
+		$is_end_date   = property_exists(__CLASS__, 'end_date');
 
-		if ( $is_start_date || $is_end_date ) {
+		if ($is_start_date || $is_end_date) {
 			$date_query = array();
 
-			if ( $is_start_date && ! is_wp_error( $this->start_date ) ) {
-				$date_query['after'] = date( 'Y-m-d H:i:s', $this->start_date );
+			if ($is_start_date && ! is_wp_error($this->start_date)) {
+				$date_query['after'] = date('Y-m-d H:i:s', $this->start_date);
 			}
 
-			if ( $is_end_date && ! is_wp_error( $this->end_date ) ) {
-				$date_query['before'] = date( 'Y-m-d H:i:s', $this->end_date );
+			if ($is_end_date && ! is_wp_error($this->end_date)) {
+				$date_query['before'] = date('Y-m-d H:i:s', $this->end_date);
 			}
 
 			// Include Start Date and End Date while querying.
 			$date_query['inclusive'] = true;
 
-			$this->__set( 'date_query', $date_query );
+			$this->__set('date_query', $date_query);
 
 		}
 
@@ -363,12 +363,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset( $this->args['status'] ) ) {
+		if ( ! isset($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -380,12 +380,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset( $this->args['page'] ) ) {
+		if ( ! isset($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -398,17 +398,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -420,12 +420,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset( $this->args['month'] ) ) {
+		if ( ! isset($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -437,23 +437,23 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount':
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 
 			case 'status':
-				$this->__set( 'orderby', 'post_status' );
+				$this->__set('orderby', 'post_status');
 				break;
 
 			case 'donation_form':
-				$this->__set( 'orderby', 'meta_value' );
-				$this->__set( 'meta_key', '_give_payment_form_title' );
+				$this->__set('orderby', 'meta_value');
+				$this->__set('meta_key', '_give_payment_form_title');
 				break;
 
 			default:
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -470,19 +470,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function custom_orderby( $order, $query ) {
+	public function custom_orderby($order, $query) {
 
-		if ( ! empty( $query->query['post_type'] ) ) {
-			$post_types = is_array( $query->query['post_type'] ) ? $query->query['post_type'] : array( $query->query['post_type'] );
+		if ( ! empty($query->query['post_type'])) {
+			$post_types = is_array($query->query['post_type']) ? $query->query['post_type'] : array($query->query['post_type']);
 
-			if ( ! in_array( 'give_payment', $post_types ) || is_array( $query->query['orderby'] ) ) {
+			if ( ! in_array('give_payment', $post_types) || is_array($query->query['orderby'])) {
 				return $order;
 			}
 
 			global $wpdb;
-			switch ( $query->query['orderby'] ) {
+			switch ($query->query['orderby']) {
 				case 'post_status':
-					$order = $wpdb->posts . '.post_status ' . strtoupper( $query->query['order'] );
+					$order = $wpdb->posts.'.post_status '.strtoupper($query->query['order']);
 					break;
 			}
 		}
@@ -499,11 +499,11 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_donor_id';
 		} else {
 			$user_key = '_give_payment_donor_email';
@@ -525,16 +525,16 @@ 
 	 * @return  void
 	 */
 	public function donor() {
-		if ( is_null( $this->args['donor'] ) || ! is_numeric( $this->args['donor'] ) ) {
+		if (is_null($this->args['donor']) || ! is_numeric($this->args['donor'])) {
 			return;
 		}
 
 		$donor_meta_type = Give()->donor_meta->meta_type;
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => "_give_payment_{$donor_meta_type}_id",
 			'value' => (int) $this->args['donor'],
-		) );
+		));
 	}
 
 	/**
@@ -547,33 +547,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_donor_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -582,19 +582,19 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_donor_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) ),
+				'value' => trim(str_replace('user:', '', strtolower($search))),
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			if ( give_get_option( 'enable_sequential' ) ) {
+			if (give_get_option('enable_sequential')) {
 
 				$search_meta = array(
 					'key'     => '_give_payment_number',
@@ -602,19 +602,19 @@ 
 					'compare' => 'LIKE',
 				);
 
-				$this->__set( 'meta_query', $search_meta );
+				$this->__set('meta_query', $search_meta);
 
 				$this->args['meta_query']['relation'] = 'OR';
 
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
 		} elseif (
-			give_get_option( 'enable_sequential' ) &&
+			give_get_option('enable_sequential') &&
 			(
-				false !== strpos( $search, give_get_option( 'sequential_prefix' ) ) ||
-				false !== strpos( $search, give_get_option( 'sequential_postfix' ) )
+				false !== strpos($search, give_get_option('sequential_prefix')) ||
+				false !== strpos($search, give_get_option('sequential_postfix'))
 			)
 		) {
 
@@ -624,29 +624,29 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -661,8 +661,8 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
@@ -684,10 +684,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -700,13 +700,13 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['give_forms'] ) ) {
+		if (is_array($this->args['give_forms'])) {
 			$compare = 'IN';
 		}
 
@@ -719,7 +719,7 @@ 
 			)
 		);
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 
@@ -733,13 +733,13 @@ 
 	 */
 	public function gateway_filter() {
 
-		if ( empty( $this->args['gateway'] ) ) {
+		if (empty($this->args['gateway'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['gateway'] ) ) {
+		if (is_array($this->args['gateway'])) {
 			$compare = 'IN';
 		}
 
@@ -753,7 +753,7 @@ 
 			)
 		);
 
-		$this->__unset( 'gateway' );
+		$this->__unset('gateway');
 
 	}
 
@@ -773,9 +773,9 @@ 
 		global $wpdb;
 
 		$where = "WHERE {$wpdb->posts}.post_type = 'give_payment'";
-		$where .= " AND {$wpdb->posts}.post_status IN ('" . implode( "','", $this->args['post_status'] ) . "')";
+		$where .= " AND {$wpdb->posts}.post_status IN ('".implode("','", $this->args['post_status'])."')";
 
-		if( is_numeric( $this->args['post_parent'] ) ) {
+		if (is_numeric($this->args['post_parent'])) {
 			$where .= " AND {$wpdb->posts}.post_parent={$this->args['post_parent']}";
 		}
 
@@ -784,42 +784,42 @@ 
 		$group_by = '';
 
 		// Set group by.
-		if ( ! empty( $this->args['group_by'] ) ) {
+		if ( ! empty($this->args['group_by'])) {
 			$group_by = "GROUP BY {$wpdb->posts}.{$this->args['group_by']}";
 		}
 
 		// Set offset.
 		if (
-			empty( $this->args['nopaging'] ) &&
-			empty( $this->args['offset'] ) &&
-			( ! empty( $this->args['page'] ) && 0 < $this->args['page'] )
+			empty($this->args['nopaging']) &&
+			empty($this->args['offset']) &&
+			( ! empty($this->args['page']) && 0 < $this->args['page'])
 		) {
-			$this->args['offset'] = $this->args['posts_per_page'] * ( $this->args['page'] - 1 );
+			$this->args['offset'] = $this->args['posts_per_page'] * ($this->args['page'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$wpdb->posts}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$wpdb->posts}.{$this->args['fields']}";
-			} elseif ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$wpdb->posts}." . implode( " , {$wpdb->posts}.", $this->args['fields'] );
+			} elseif (is_array($this->args['fields'])) {
+				$fields = "{$wpdb->posts}.".implode(" , {$wpdb->posts}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$wpdb->posts}.ID)";
 
-			if ( ! empty( $this->args['group_by'] ) ) {
+			if ( ! empty($this->args['group_by'])) {
 				$fields = "{$wpdb->posts}.{$this->args['group_by']}, {$fields}";
 			}
 		}
 
 		// Date query.
-		if ( ! empty( $this->args['date_query'] ) ) {
-			$date_query_obj = new WP_Date_Query( $this->args['date_query'] );
-			$where          .= str_replace(
+		if ( ! empty($this->args['date_query'])) {
+			$date_query_obj = new WP_Date_Query($this->args['date_query']);
+			$where .= str_replace(
 				array(
 					"\n",
 					'(   (',
@@ -835,22 +835,22 @@ 
 		}
 
 		// Meta query.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_obj = new WP_Meta_Query( $this->args['meta_query'] );
-			$where          = implode( ' ', $meta_query_obj->get_sql( 'post', $wpdb->posts, 'ID' ) ) . " {$where}";
-			$where          = Give()->payment_meta->__rename_meta_table_name( $where, 'posts_where' );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_obj = new WP_Meta_Query($this->args['meta_query']);
+			$where          = implode(' ', $meta_query_obj->get_sql('post', $wpdb->posts, 'ID'))." {$where}";
+			$where          = Give()->payment_meta->__rename_meta_table_name($where, 'posts_where');
 		}
 
 		// Set sql query.
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$wpdb->posts} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			( empty( $this->args['nopaging'] ) ? absint( $this->args['posts_per_page'] ) : 99999999999 )
+			absint($this->args['offset']),
+			(empty($this->args['nopaging']) ? absint($this->args['posts_per_page']) : 99999999999)
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}

includes/admin/abstract-admin-settings-page.php

Spacing +31 added lines, -31 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Page' ) ) :
+if ( ! class_exists('Give_Settings_Page')) :
 
 	/**
 	 * Give_Settings_Page.
@@ -70,25 +70,25 @@ 
 			// Get current setting page.
 			$this->current_setting_page = give_get_current_setting_page();
 
-			add_filter( "give_default_setting_tab_section_{$this->id}", array( $this, 'set_default_setting_tab' ), 10 );
-			add_filter( "{$this->current_setting_page}_tabs_array", array( $this, 'add_settings_page' ), 20 );
-			add_action( "{$this->current_setting_page}_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter("give_default_setting_tab_section_{$this->id}", array($this, 'set_default_setting_tab'), 10);
+			add_filter("{$this->current_setting_page}_tabs_array", array($this, 'add_settings_page'), 20);
+			add_action("{$this->current_setting_page}_settings_{$this->id}_page", array($this, 'output'));
 
 			// Output section only if exist.
 			$sections = $this->get_sections();
-			if ( ! empty( $sections ) ) {
-				add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array(
+			if ( ! empty($sections)) {
+				add_action("{$this->current_setting_page}_sections_{$this->id}_page", array(
 					$this,
 					'output_sections',
-				) );
+				));
 			}
 
 			// Save hide button by default.
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Enable saving feature.
-			if ( $this->enable_save ) {
-				add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
+			if ($this->enable_save) {
+				add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
 			}
 		}
 
@@ -113,7 +113,7 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			return $this->default_tab;
 		}
 
@@ -126,8 +126,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -146,7 +146,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, array() );
+			$settings = apply_filters('give_get_settings_'.$this->id, array());
 
 			// Output.
 			return $settings;
@@ -159,7 +159,7 @@ 
 		 * @return array
 		 */
 		public function get_sections() {
-			return apply_filters( 'give_get_sections_' . $this->id, array() );
+			return apply_filters('give_get_sections_'.$this->id, array());
 		}
 
 		/**
@@ -176,39 +176,39 @@ 
 			$sections = $this->get_sections();
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
 			// Show section settings only if setting section exist.
-			if ( $current_section && ! in_array( $current_section, array_keys( $sections ) ) ) {
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+			if ($current_section && ! in_array($current_section, array_keys($sections))) {
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
-			if ( is_null( $this->current_setting_page ) ) {
+			if (is_null($this->current_setting_page)) {
 				$this->current_setting_page = give_get_current_setting_page();
 			}
 
 			$section_list = array();
-			foreach ( $sections as $id => $label ) {
+			foreach ($sections as $id => $label) {
 				/**
 				 * Fire the filter to hide particular section on tab.
 				 *
 				 * @since 2.0
 				 */
-				if ( apply_filters( "give_hide_section_{$id}_on_{$this->id}_page", false, $sections, $this->id ) ) {
+				if (apply_filters("give_hide_section_{$id}_on_{$this->id}_page", false, $sections, $this->id)) {
 					continue;
 				}
 
-				$section_list[] = '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=' . $this->current_setting_page . '&tab=' . $this->id . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $current_section == $id ? 'current' : '' ) . '">' . $label . '</a>';
+				$section_list[] = '<li><a href="'.admin_url('edit.php?post_type=give_forms&page='.$this->current_setting_page.'&tab='.$this->id.'&section='.sanitize_title($id)).'" class="'.($current_section == $id ? 'current' : '').'">'.$label.'</a>';
 			}
 
 			echo sprintf(
 				'<ul class="subsubsub">%s</ul><br class="clear" /><hr>',
-				implode( ' | </li>', $section_list )
+				implode(' | </li>', $section_list)
 			);
 		}
 
@@ -221,13 +221,13 @@ 
 		 * @return void
 		 */
 		public function output() {
-			if ( $this->enable_save ) {
+			if ($this->enable_save) {
 				$GLOBALS['give_hide_save_button'] = false;
 			}
 
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -240,14 +240,14 @@ 
 			$settings        = $this->get_settings();
 			$current_section = give_get_current_setting_section();
 
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
 
 			/**
 			 * Trigger Action
 			 *
 			 * @since 1.8
 			 */
-			do_action( 'give_update_options_' . $this->id . '_' . $current_section );
+			do_action('give_update_options_'.$this->id.'_'.$current_section);
 		}
 
 		/**
@@ -264,11 +264,11 @@ 
 			$section         = $this->get_sections();
 			$current_section = give_get_current_setting_section();
 
-			if ( array_key_exists( $current_section, $section ) ) {
-				$heading[] = $section[ $current_section ];
+			if (array_key_exists($current_section, $section)) {
+				$heading[] = $section[$current_section];
 			}
 
-			return array_unique( $heading );
+			return array_unique($heading);
 		}
 
 		/**
@@ -282,7 +282,7 @@ 
 		public function get_heading_html() {
 			return sprintf(
 				'<h1 class="wp-heading-inline">%s</h1><hr class="wp-header-end">',
-				implode( ' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading() )
+				implode(' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading())
 			);
 		}
 	}

includes/admin/donors/donors.php

Braces +5 added lines, -2 removed lines

@@ -402,8 +402,11 @@
 								<span class="donor-user-id info-item editable">
 										<?php if ( ! empty( $userdata ) ) : ?>
 											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
-										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'None', 'give' ); ?></span>
+										<?php else {
+	: ?>
+											<span data-key="user_id"><?php _e( 'None', 'give' );
+}
+?></span>
 										<?php endif; ?>
 									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
 

Spacing +213 added lines, -213 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  *
  * @return string
  */
-function __give_get_format_address( $address, $address_args = array() ) {
+function __give_get_format_address($address, $address_args = array()) {
 	$address_html = '';
 	$address_args = wp_parse_args(
 		$address_args,
@@ -40,7 +40,7 @@ 
 	$address_id = $address_args['type'];
 
 	// Bailout.
-	if ( empty( $address ) || ! is_array( $address ) ) {
+	if (empty($address) || ! is_array($address)) {
 		return $address_html;
 	}
 
@@ -49,19 +49,19 @@ 
 	$address_html .= sprintf(
 		'<span data-address-type="line1">%1$s</span>%2$s',
 		$address['line1'],
-		( ! empty( $address['line2'] ) ? '<br>' : '' )
+		( ! empty($address['line2']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="line2">%1$s</span>%2$s',
 		$address['line2'],
-		( ! empty( $address['city'] ) ? '<br>' : '' )
+		( ! empty($address['city']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="city">%1$s</span><span data-address-type="state">%2$s</span><span data-address-type="zip">%3$s</span>%4$s',
 		$address['city'],
-		( ! empty( $address['state'] ) ? ", {$address['state']}" : '' ),
-		( ! empty( $address['zip'] ) ? " {$address['zip']}" : '' ),
-		( ! empty( $address['country'] ) ? '<br>' : '' )
+		( ! empty($address['state']) ? ", {$address['state']}" : ''),
+		( ! empty($address['zip']) ? " {$address['zip']}" : ''),
+		( ! empty($address['country']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="country">%s</span><br>',
@@ -71,8 +71,8 @@ 
 	// Address action.
 	$address_html .= sprintf(
 		'<br><a href="#" class="js-edit">%1$s</a> | <a href="#" class="js-remove">%2$s</a>',
-		__( 'Edit', 'give' ),
-		__( 'Remove', 'give' )
+		__('Edit', 'give'),
+		__('Remove', 'give')
 	);
 
 	/**
@@ -80,14 +80,14 @@ 
 	 *
 	 * @since 2.0
 	 */
-	$address_label = apply_filters( "give_donor_{$address_args['type']}_address_label", ucfirst( $address_args['type'] ), $address_args );
+	$address_label = apply_filters("give_donor_{$address_args['type']}_address_label", ucfirst($address_args['type']), $address_args);
 
 	// Set unique id and index for multi type address.
-	if ( isset( $address_args['index'] ) ) {
+	if (isset($address_args['index'])) {
 		$address_label = "{$address_label} #{$address_args['index']}";
 	}
 
-	if ( isset( $address_args['id'] ) ) {
+	if (isset($address_args['id'])) {
 		$address_id = "{$address_id}_{$address_args['id']}";
 	}
 
@@ -112,9 +112,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -130,7 +130,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -144,7 +144,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -156,7 +156,7 @@ 
  */
 function give_donors_list() {
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/donors/class-donor-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/donors/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -169,12 +169,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
 
 		<hr class="wp-header-end">
-		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
-			<?php $donors_table->search_box( __( 'Search Donors', 'give' ), 'give-donors' ); ?>
+		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
+			<?php $donors_table->search_box(__('Search Donors', 'give'), 'give-donors'); ?>
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-donors" />
 			<input type="hidden" name="view" value="donors" />
@@ -191,7 +191,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -207,33 +207,33 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id          = (int) $_GET['id'];
-	$reconnect_user_id = ! empty( $_GET['user_id'] ) ? (int) $_GET['user_id'] : '';
-	$donor             = new Give_Donor( $donor_id );
+	$reconnect_user_id = ! empty($_GET['user_id']) ? (int) $_GET['user_id'] : '';
+	$donor             = new Give_Donor($donor_id);
 
 	// Reconnect User with Donor profile.
-	if ( $reconnect_user_id ) {
-		give_connect_user_donor_profile( $donor, array( 'user_id' => $reconnect_user_id ), array() );
+	if ($reconnect_user_id) {
+		give_connect_user_donor_profile($donor, array('user_id' => $reconnect_user_id), array());
 	}
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -242,9 +242,9 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 
@@ -252,7 +252,7 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor first name */
-				__( 'Edit Donor #%s: %s %s', 'give' ),
+				__('Edit Donor #%s: %s %s', 'give'),
 				$donor->id,
 				$donor->get_first_name(),
 				$donor->get_last_name()
@@ -262,26 +262,26 @@ 
 
 		<hr class="wp-header-end">
 
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 				<?php
-				foreach ( $donor_tabs as $key => $tab ) :
+				foreach ($donor_tabs as $key => $tab) :
 					$active = $key === $view ? true : false;
 					$class  = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 					printf(
-						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-						esc_attr( $class ),
-						sanitize_html_class( $tab['dashicon'] ),
-						esc_html( $tab['title'] )
+						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+						esc_attr($class),
+						sanitize_html_class($tab['dashicon']),
+						esc_html($tab['title'])
 					);
 				endforeach;
 				?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -301,9 +301,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -312,11 +312,11 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 
 	// Set Read only to the fields which needs to be locked.
 	$read_only = '';
-	if ( $donor->user_id ) {
+	if ($donor->user_id) {
 		$read_only = 'readonly="readonly"';
 	}
 	?>
@@ -324,14 +324,14 @@ 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
 		<form id="edit-donor-info" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
@@ -339,18 +339,18 @@ 
 						<span class="donor-name info-item edit-item">
 							<input <?php echo $read_only; ?> size="15" data-key="first_name"
 							                                 name="customerinfo[first_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_first_name() ); ?>"
-							                                 placeholder="<?php _e( 'First Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_first_name()); ?>"
+							                                 placeholder="<?php _e('First Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
 							<?php endif; ?>
 							<input <?php echo $read_only; ?> size="15" data-key="last_name"
 							                                 name="customerinfo[last_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_last_name() ); ?>"
-							                                 placeholder="<?php _e( 'Last Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_last_name()); ?>"
+							                                 placeholder="<?php _e('Last Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
@@ -360,12 +360,12 @@ 
 									data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php _e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php _e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
+					<?php if (current_user_can($donor_edit_role)) : ?>
 						<a href="#" id="edit-donor"
-						   class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+						   class="button info-item editable donor-edit-link"><?php _e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -375,7 +375,7 @@ 
 					<table class="widefat">
 						<tbody>
 						<tr class="alternate">
-							<th scope="col"><label for="tablecell"><?php _e( 'User:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('User:', 'give'); ?></label></th>
 							<td>
 									<span class="donor-user-id info-item edit-item">
 										<?php
@@ -392,22 +392,22 @@ 
 											'data'  => $data_atts,
 										);
 
-										if ( ! empty( $user_id ) ) {
-											$userdata              = get_userdata( $user_id );
+										if ( ! empty($user_id)) {
+											$userdata              = get_userdata($user_id);
 											$user_args['selected'] = $user_id;
 										}
 
-										echo Give()->html->ajax_user_search( $user_args );
+										echo Give()->html->ajax_user_search($user_args);
 										?>
 									</span>
 
 								<span class="donor-user-id info-item editable">
-										<?php if ( ! empty( $userdata ) ) : ?>
-											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+										<?php if ( ! empty($userdata)) : ?>
+											<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'None', 'give' ); ?></span>
+											<span data-key="user_id"><?php _e('None', 'give'); ?></span>
 										<?php endif; ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0):
 
 										echo sprintf(
 											'- <span class="disconnect-user">
@@ -416,11 +416,11 @@ 
 				                                       | <span class="view-user-profile">
  											                <a id="view-user-profile" href="%3$s" aria-label="%4$s">%5$s</a>
  										               </span>',
-											__( 'Disconnects the current user ID from this donor record.', 'give' ),
-											__( 'Disconnect User', 'give' ),
-											'user-edit.php?user_id=' . $donor->user_id,
-											__( 'View User Profile of current user ID.', 'give' ),
-											__( 'View User Profile', 'give' )
+											__('Disconnects the current user ID from this donor record.', 'give'),
+											__('Disconnect User', 'give'),
+											'user-edit.php?user_id='.$donor->user_id,
+											__('View User Profile of current user ID.', 'give'),
+											__('View User Profile', 'give')
 										);
 
 										endif; ?>
@@ -436,11 +436,11 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>"/>
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor"/>
 				<input type="submit" id="give-edit-donor-save" class="button-secondary"
-				       value="<?php _e( 'Update Donor', 'give' ); ?>"/>
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+				       value="<?php _e('Update Donor', 'give'); ?>"/>
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -455,24 +455,24 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations.
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->get_total_donation_amount(), array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->get_total_donation_amount(), array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -484,7 +484,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -497,11 +497,11 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_address', $donor );
+	do_action('give_donor_before_address', $donor);
 	?>
 
 	<div id="donor-address-wrapper" class="donor-section clear">
-		<h3><?php _e( 'Addresses', 'give' ); ?></h3>
+		<h3><?php _e('Addresses', 'give'); ?></h3>
 
 		<div class="postbox">
 			<div class="give-spinner-wrapper">
@@ -511,16 +511,16 @@ 
 				<div class="all-address">
 					<div class="give-grid-row">
 						<?php
-						if ( ! empty( $donor->address ) ) :
+						if ( ! empty($donor->address)) :
 							// Default address always will be at zero array index.
 							$is_set_as_default = null;
 
-							foreach ( $donor->address as $address_type => $addresses ) {
+							foreach ($donor->address as $address_type => $addresses) {
 
-								switch ( true ) {
-									case is_array( end( $addresses ) ):
+								switch (true) {
+									case is_array(end($addresses)):
 										$index = 1;
-										foreach ( $addresses as $id => $address ) {
+										foreach ($addresses as $id => $address) {
 											echo __give_get_format_address(
 												$address,
 												array(
@@ -530,11 +530,11 @@ 
 												)
 											);
 
-											$index ++;
+											$index++;
 										}
 										break;
 
-									case is_string( end( $addresses ) ):
+									case is_string(end($addresses)):
 										echo __give_get_format_address(
 											$addresses,
 											array(
@@ -547,13 +547,13 @@ 
 						endif;
 						?>
 					</div>
-					<span class="give-no-address-message<?php if ( ! empty( $donor->address ) ) {
+					<span class="give-no-address-message<?php if ( ! empty($donor->address)) {
 						echo ' give-hidden';
 					} ?>">
-						<?php _e( 'This donor does not have any addresses saved.', 'give' ); ?>
+						<?php _e('This donor does not have any addresses saved.', 'give'); ?>
 					</span>
 					<button class="button add-new-address">
-						<?php _e( 'Add Address', 'give' ); ?>
+						<?php _e('Add Address', 'give'); ?>
 					</button>
 				</div>
 
@@ -563,26 +563,26 @@ 
 							<tbody>
 							<tr>
 								<th class="col">
-									<label class="country"><?php esc_html_e( 'Country:', 'give' ); ?></label>
+									<label class="country"><?php esc_html_e('Country:', 'give'); ?></label>
 								</th>
 								<td>
 									<?php
-									echo Give()->html->select( array(
+									echo Give()->html->select(array(
 										'options'          => give_get_country_list(),
 										'name'             => 'country',
-										'selected'         => give_get_option( 'base_country' ),
+										'selected'         => give_get_option('base_country'),
 										'show_option_all'  => false,
 										'show_option_none' => false,
 										'chosen'           => true,
-										'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-										'data'             => array( 'search-type' => 'no_ajax' ),
-									) );
+										'placeholder'      => esc_attr__('Select a country', 'give'),
+										'data'             => array('search-type' => 'no_ajax'),
+									));
 									?>
 								</td>
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line1"><?php esc_html_e( 'Address 1:', 'give' ); ?></label>
+									<label for="line1"><?php esc_html_e('Address 1:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line1" name="line1" type="text" class="medium-text"/>
@@ -590,7 +590,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line2"><?php esc_html_e( 'Address 2:', 'give' ); ?></label>
+									<label for="line2"><?php esc_html_e('Address 2:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line2" type="text" name="line2" value="" class="medium-text"/>
@@ -599,7 +599,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="city"><?php esc_html_e( 'City:', 'give' ); ?></label>
+									<label for="city"><?php esc_html_e('City:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="city" type="text" name="city" value="" class="medium-text"/>
@@ -607,41 +607,41 @@ 
 							</tr>
 							<?php
 							$no_states_country = give_no_states_country_list();
-							$base_country      = give_get_option( 'base_country' );
-							if ( ! array_key_exists( $base_country, $no_states_country ) ) {
+							$base_country      = give_get_option('base_country');
+							if ( ! array_key_exists($base_country, $no_states_country)) {
 								?>
 								<tr class="give-field-wrap">
 									<th class="col">
-										<label for="state"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+										<label for="state"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 									</th>
 									<td>
 										<?php
-										$states     = give_get_states( $base_country );
+										$states     = give_get_states($base_country);
 										$state_args = array(
 											'name'  => 'state',
 											'class' => 'regular-text',
 										);
 
-										if ( empty( $states ) ) {
+										if (empty($states)) {
 
 											// Show Text field, if empty states.
-											$state_args = wp_parse_args( $state_args, array(
-												'value' => give_get_option( 'base_state' ),
-											) );
-											echo Give()->html->text( $state_args );
+											$state_args = wp_parse_args($state_args, array(
+												'value' => give_get_option('base_state'),
+											));
+											echo Give()->html->text($state_args);
 										} else {
 
 											// Show Chosen DropDown, if states are not empty.
-											$state_args = wp_parse_args( $state_args, array(
+											$state_args = wp_parse_args($state_args, array(
 												'options'          => $states,
-												'selected'         => give_get_option( 'base_state' ),
+												'selected'         => give_get_option('base_state'),
 												'show_option_all'  => false,
 												'show_option_none' => false,
 												'chosen'           => true,
-												'placeholder'      => __( 'Select a state', 'give' ),
-												'data'             => array( 'search-type' => 'no_ajax' ),
-											) );
-											echo Give()->html->select( $state_args );
+												'placeholder'      => __('Select a state', 'give'),
+												'data'             => array('search-type' => 'no_ajax'),
+											));
+											echo Give()->html->select($state_args);
 										}
 										?>
 									</td>
@@ -651,7 +651,7 @@ 
 							?>
 							<tr>
 								<th class="col">
-									<label for="zip"><?php esc_html_e( 'Zip / Postal Code:', 'give' ); ?></label>
+									<label for="zip"><?php esc_html_e('Zip / Postal Code:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="zip" type="text" name="zip" value="" class="medium-text"/>
@@ -659,12 +659,12 @@ 
 							</tr>
 							<tr>
 								<td colspan="2">
-									<?php wp_nonce_field( 'give-manage-donor-addresses', '_wpnonce', false ); ?>
+									<?php wp_nonce_field('give-manage-donor-addresses', '_wpnonce', false); ?>
 									<input type="hidden" name="address-action" value="add">
 									<input type="hidden" name="address-id" value="">
 									<input type="submit" class="button button-primary js-save"
-									       value="<?php _e( 'Save', 'give' ); ?>">&nbsp;&nbsp;<button
-											class="button js-cancel"><?php _e( 'Cancel', 'give' ); ?></button>
+									       value="<?php _e('Save', 'give'); ?>">&nbsp;&nbsp;<button
+											class="button js-cancel"><?php _e('Cancel', 'give'); ?></button>
 								</td>
 							</tr>
 							</tbody>
@@ -683,7 +683,7 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -696,46 +696,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 			<tr>
-				<th><?php _e( 'Email', 'give' ); ?></th>
-				<th><?php _e( 'Actions', 'give' ); ?></th>
+				<th><?php _e('Email', 'give'); ?></th>
+				<th><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 
 			<tbody>
-			<?php if ( ! empty( $donor->emails ) ) { ?>
+			<?php if ( ! empty($donor->emails)) { ?>
 
-				<?php foreach ( $donor->emails as $key => $email ) : ?>
+				<?php foreach ($donor->emails as $key => $email) : ?>
 					<tr data-key="<?php echo $key; ?>">
 						<td>
 							<?php echo $email; ?>
-							<?php if ( 'primary' === $key ) : ?>
+							<?php if ('primary' === $key) : ?>
 								<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 							<?php endif; ?>
 						</td>
 						<td>
-							<?php if ( 'primary' !== $key ) : ?>
+							<?php if ('primary' !== $key) : ?>
 								<?php
-								$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-								$promote_url = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+								$promote_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'set_donor_primary_email',
-								), $base_url ), 'give-set-donor-primary-email' );
-								$remove_url  = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								), $base_url), 'give-set-donor-primary-email');
+								$remove_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'remove_donor_email',
-								), $base_url ), 'give-remove-donor-email' );
+								), $base_url), 'give-remove-donor-email');
 								?>
-								<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+								<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 								&nbsp;|&nbsp;
-								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 							<?php endif; ?>
 						</td>
 					</tr>
@@ -745,14 +745,14 @@ 
 					<td colspan="2" class="add-donor-email-td">
 						<div class="add-donor-email-wrapper">
 							<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>"/>
-							<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
+							<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
 							<input type="email" name="additional-email" value=""
-							       placeholder="<?php _e( 'Email Address', 'give' ); ?>"/>&nbsp;
+							       placeholder="<?php _e('Email Address', 'give'); ?>"/>&nbsp;
 							<input type="checkbox" name="make-additional-primary" value="1"
 							       id="make-additional-primary"/>&nbsp;<label
-									for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
+									for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
 							<button class="button-secondary give-add-donor-email"
-							        id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+							        id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 							<span class="spinner"></span>
 						</div>
 						<div class="notice-wrap"></div>
@@ -760,50 +760,50 @@ 
 				</tr>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No Emails Found', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php _e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('ID', 'give'); ?></th>
+				<th scope="col"><?php _e('Amount', 'give'); ?></th>
+				<th scope="col"><?php _e('Date', 'give'); ?></th>
+				<th scope="col"><?php _e('Status', 'give'); ?></th>
+				<th scope="col"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
 						<td><?php echo $payment->ID; ?></td>
-						<td><?php echo give_donation_amount( $payment->ID, array( 'currency' => true, 'amount' => true, 'type' => 'donor' ) ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo give_donation_amount($payment->ID, array('currency' => true, 'amount' => true, 'type' => 'donor')); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 								/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								__( 'View Donation', 'give' )
+								__('View Donation', 'give')
 							);
 							?>
 
@@ -818,47 +818,47 @@ 
 							 * @param object $donor   The donor object being displayed.
 							 * @param object $payment The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php _e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php _e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php _e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 								/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								__( 'View Form', 'give' )
+								__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -866,7 +866,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -880,7 +880,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -893,7 +893,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -906,31 +906,31 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged       = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged       = absint( $paged );
+	$paged       = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged       = absint($paged);
 	$note_count  = $donor->get_notes_count();
-	$per_page    = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page    = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php _e( 'Notes', 'give' ); ?></h3>
+		<h3><?php _e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
 				<form id="give-add-donor-note" method="post"
-				      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br/>
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>"/>
 					<input type="hidden" name="give_action" value="add-donor-note"/>
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note"/>
 				</form>
 			</div>
@@ -945,26 +945,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php _e( 'No donor notes found.', 'give' ); ?>
+					<?php _e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -980,9 +980,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -991,16 +991,16 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
 		<form id="delete-donor" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -1008,20 +1008,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php _e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php _e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php _e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php _e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -1034,19 +1034,19 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>"/>
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor"/>
 					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary"
-					       value="<?php _e( 'Delete Donor', 'give' ); ?>"/>
+					       value="<?php _e('Delete Donor', 'give'); ?>"/>
 					<a id="give-delete-donor-cancel"
-					   href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>"
-					   class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+					   href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>"
+					   class="delete"><?php _e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -1062,5 +1062,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

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

Indentation +14 added lines, -14 removed lines

@@ -97,13 +97,13 @@ 
 								'disabled' => __( 'Disabled', 'give' ),
 							)
 						),
-                        array(
-                            'name'  => __( 'PayPal Standard Gateway Settings Docs Link', 'give' ),
-                            'id'    => 'paypal_standard_gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-paypal-standard' ),
-                            'title' => __( 'PayPal Standard Gateway Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => __( 'PayPal Standard Gateway Settings Docs Link', 'give' ),
+							'id'    => 'paypal_standard_gateway_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-paypal-standard' ),
+							'title' => __( 'PayPal Standard Gateway Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'type' => 'sectionend',
 							'id'   => 'give_title_gateway_settings_2',
@@ -183,13 +183,13 @@ 
 							'id'   => 'default_gateway',
 							'type' => 'default_gateway'
 						),
-                        array(
-                            'name'  => __( 'Gateways Docs Link', 'give' ),
-                            'id'    => 'gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateways' ),
-                            'title' => __( 'Gateway Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => __( 'Gateways Docs Link', 'give' ),
+							'id'    => 'gateway_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-gateways' ),
+							'title' => __( 'Gateway Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_gateway_settings_1',
 							'type' => 'sectionend'

Spacing +49 added lines, -49 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Gateways' ) ) :
+if ( ! class_exists('Give_Settings_Gateways')) :
 
 	/**
 	 * Give_Settings_Gateways.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'gateways';
-			$this->label = esc_html__( 'Payment Gateways', 'give' );
+			$this->label = esc_html__('Payment Gateways', 'give');
 
 			$this->default_tab = 'gateways-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings        = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'paypal-standard':
 					$settings = array(
 						// Section 2: PayPal Standard.
@@ -53,55 +53,55 @@ 
 							'id'   => 'give_title_gateway_settings_2',
 						),
 						array(
-							'name' => __( 'PayPal Email', 'give' ),
-							'desc' => __( 'Enter your PayPal account\'s email.', 'give' ),
+							'name' => __('PayPal Email', 'give'),
+							'desc' => __('Enter your PayPal account\'s email.', 'give'),
 							'id'   => 'paypal_email',
 							'type' => 'email',
 						),
 						array(
-							'name' => __( 'PayPal Page Style', 'give' ),
-							'desc' => __( 'Enter the name of the PayPal page style to use, or leave blank to use the default.', 'give' ),
+							'name' => __('PayPal Page Style', 'give'),
+							'desc' => __('Enter the name of the PayPal page style to use, or leave blank to use the default.', 'give'),
 							'id'   => 'paypal_page_style',
 							'type' => 'text',
 						),
 						array(
-							'name'    => __( 'PayPal Transaction Type', 'give' ),
-							'desc'    => __( 'Nonprofits must verify their status to withdraw donations they receive via PayPal. PayPal users that are not verified nonprofits must demonstrate how their donations will be used, once they raise more than $10,000. By default, Give transactions are sent to PayPal as donations. You may change the transaction type using this option if you feel you may not meet PayPal\'s donation requirements.', 'give' ),
+							'name'    => __('PayPal Transaction Type', 'give'),
+							'desc'    => __('Nonprofits must verify their status to withdraw donations they receive via PayPal. PayPal users that are not verified nonprofits must demonstrate how their donations will be used, once they raise more than $10,000. By default, Give transactions are sent to PayPal as donations. You may change the transaction type using this option if you feel you may not meet PayPal\'s donation requirements.', 'give'),
 							'id'      => 'paypal_button_type',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'donation' => __( 'Donation', 'give' ),
-								'standard' => __( 'Standard Transaction', 'give' )
+								'donation' => __('Donation', 'give'),
+								'standard' => __('Standard Transaction', 'give')
 							),
 							'default' => 'donation',
 						),
 						array(
-							'name'    => __( 'Billing Details', 'give' ),
-							'desc'    => __( 'This option will enable the billing details section for PayPal Standard which requires the donor\'s address to complete the donation. These fields are not required by PayPal to process the transaction, but you may have a need to collect the data.', 'give' ),
+							'name'    => __('Billing Details', 'give'),
+							'desc'    => __('This option will enable the billing details section for PayPal Standard which requires the donor\'s address to complete the donation. These fields are not required by PayPal to process the transaction, but you may have a need to collect the data.', 'give'),
 							'id'      => 'paypal_standard_billing_details',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
-							'name'    => __( 'PayPal IPN Verification', 'give' ),
-							'desc'    => __( 'If donations are not getting marked as complete, use a slightly less secure method of verifying donations.', 'give' ),
+							'name'    => __('PayPal IPN Verification', 'give'),
+							'desc'    => __('If donations are not getting marked as complete, use a slightly less secure method of verifying donations.', 'give'),
 							'id'      => 'paypal_verification',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
                         array(
-                            'name'  => __( 'PayPal Standard Gateway Settings Docs Link', 'give' ),
+                            'name'  => __('PayPal Standard Gateway Settings Docs Link', 'give'),
                             'id'    => 'paypal_standard_gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-paypal-standard' ),
-                            'title' => __( 'PayPal Standard Gateway Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-gateway-paypal-standard'),
+                            'title' => __('PayPal Standard Gateway Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -119,19 +119,19 @@ 
 							'id'   => 'give_title_gateway_settings_3',
 						),
 						array(
-							'name'    => __( 'Collect Billing Details', 'give' ),
-							'desc'    => __( 'Enable to request billing details for offline donations. Will appear above offline donation instructions. Can be enabled/disabled per form.', 'give' ),
+							'name'    => __('Collect Billing Details', 'give'),
+							'desc'    => __('Enable to request billing details for offline donations. Will appear above offline donation instructions. Can be enabled/disabled per form.', 'give'),
 							'id'      => 'give_offline_donation_enable_billing_fields',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' )
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give')
 							)
 						),
 						array(
-							'name'    => __( 'Offline Donation Instructions', 'give' ),
-							'desc'    => __( 'The following content will appear for all forms when the user selects the offline donation payment option. Note: You may customize the content per form as needed.', 'give' ),
+							'name'    => __('Offline Donation Instructions', 'give'),
+							'desc'    => __('The following content will appear for all forms when the user selects the offline donation payment option. Note: You may customize the content per form as needed.', 'give'),
 							'id'      => 'global_offline_donation_content',
 							'default' => give_get_default_offline_donation_content(),
 							'type'    => 'wysiwyg',
@@ -140,10 +140,10 @@ 
 							)
 						),
 						array(
-							'name'  => esc_html__( 'Offline Donations Settings Docs Link', 'give' ),
+							'name'  => esc_html__('Offline Donations Settings Docs Link', 'give'),
 							'id'    => 'offline_gateway_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/offlinegateway' ),
-							'title' => __( 'Offline Gateway Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/offlinegateway'),
+							'title' => __('Offline Gateway Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -161,33 +161,33 @@ 
 							'type' => 'title'
 						),
 						array(
-							'name'    => __( 'Test Mode', 'give' ),
-							'desc'    => __( 'While in test mode no live donations are processed. To fully use test mode, you must have a sandbox (test) account for the payment gateway you are testing.', 'give' ),
+							'name'    => __('Test Mode', 'give'),
+							'desc'    => __('While in test mode no live donations are processed. To fully use test mode, you must have a sandbox (test) account for the payment gateway you are testing.', 'give'),
 							'id'      => 'test_mode',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
-							'name' => __( 'Enabled Gateways', 'give' ),
-							'desc' => __( 'Enable your payment gateway. Can be ordered by dragging.', 'give' ),
+							'name' => __('Enabled Gateways', 'give'),
+							'desc' => __('Enable your payment gateway. Can be ordered by dragging.', 'give'),
 							'id'   => 'gateways',
 							'type' => 'enabled_gateways'
 						),
 						array(
-							'name' => __( 'Default Gateway', 'give' ),
-							'desc' => __( 'The gateway that will be selected by default.', 'give' ),
+							'name' => __('Default Gateway', 'give'),
+							'desc' => __('The gateway that will be selected by default.', 'give'),
 							'id'   => 'default_gateway',
 							'type' => 'default_gateway'
 						),
                         array(
-                            'name'  => __( 'Gateways Docs Link', 'give' ),
+                            'name'  => __('Gateways Docs Link', 'give'),
                             'id'    => 'gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateways' ),
-                            'title' => __( 'Gateway Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-gateways'),
+                            'title' => __('Gateway Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -202,7 +202,7 @@ 
 			 * Filter the payment gateways settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_gateways', $settings );
+			$settings = apply_filters('give_settings_gateways', $settings);
 
 			/**
 			 * Filter the settings.
@@ -211,7 +211,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -225,12 +225,12 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'gateways-settings' => __( 'Gateways', 'give' ),
-				'paypal-standard'   => __( 'PayPal Standard', 'give' ),
-				'offline-donations' => __( 'Offline Donations', 'give' )
+				'gateways-settings' => __('Gateways', 'give'),
+				'paypal-standard'   => __('PayPal Standard', 'give'),
+				'offline-donations' => __('Offline Donations', 'give')
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

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

Spacing +25 added lines, -25 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Email' ) ) :
+if ( ! class_exists('Give_Settings_Email')) :
 
 	/**
 	 * Give_Settings_Email.
@@ -27,13 +27,13 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'emails';
-			$this->label = esc_html__( 'Emails', 'give' );
+			$this->label = esc_html__('Emails', 'give');
 
 			$this->default_tab = 'email-settings';
 
 			parent::__construct();
 
-			add_action( 'give_admin_field_email_notification', array( $this, 'email_notification_setting' ) );
+			add_action('give_admin_field_email_notification', array($this, 'email_notification_setting'));
 		}
 
 		/**
@@ -46,13 +46,13 @@ 
 			$settings        = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'email-settings' :
 					$settings = array(
 
 						// Section 1: Email Notification Listing.
 						array(
-							'desc'       => __( 'Email notifications sent from Give are listed below. Click on an email to configure it.', 'give' ),
+							'desc'       => __('Email notifications sent from Give are listed below. Click on an email to configure it.', 'give'),
 							'type'       => 'title',
 							'id'         => 'give_email_notification_settings',
 							'table_html' => false,
@@ -67,42 +67,42 @@ 
 
 						// Section 2: Email Sender Setting
 						array(
-							'title' => __( 'Email Sender Options', 'give' ),
+							'title' => __('Email Sender Options', 'give'),
 							'id'    => 'give_title_email_settings_1',
 							'type'  => 'title',
 						),
 						array(
 							'id'      => 'email_template',
-							'name'    => esc_html__( 'Email Template', 'give' ),
-							'desc'    => esc_html__( 'Choose your template from the available registered template types.', 'give' ),
+							'name'    => esc_html__('Email Template', 'give'),
+							'desc'    => esc_html__('Choose your template from the available registered template types.', 'give'),
 							'type'    => 'select',
 							'options' => give_get_email_templates(),
 						),
 						array(
 							'id'   => 'email_logo',
-							'name' => esc_html__( 'Logo', 'give' ),
-							'desc' => esc_html__( 'Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give' ),
+							'name' => esc_html__('Logo', 'give'),
+							'desc' => esc_html__('Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give'),
 							'type' => 'file',
 						),
 						array(
 							'id'      => 'from_name',
-							'name'    => esc_html__( 'From Name', 'give' ),
-							'desc'    => esc_html__( 'The name which appears in the "From" field in all Give donation emails.', 'give' ),
-							'default' => get_bloginfo( 'name' ),
+							'name'    => esc_html__('From Name', 'give'),
+							'desc'    => esc_html__('The name which appears in the "From" field in all Give donation emails.', 'give'),
+							'default' => get_bloginfo('name'),
 							'type'    => 'text',
 						),
 						array(
 							'id'      => 'from_email',
-							'name'    => esc_html__( 'From Email', 'give' ),
-							'desc'    => esc_html__( 'Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give' ),
-							'default' => get_bloginfo( 'admin_email' ),
+							'name'    => esc_html__('From Email', 'give'),
+							'desc'    => esc_html__('Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give'),
+							'default' => get_bloginfo('admin_email'),
 							'type'    => 'text',
 						),
 						array(
-							'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
+							'name'  => esc_html__('Donation Notification Settings Docs Link', 'give'),
 							'id'    => 'donation_notification_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
-							'title' => __( 'Donation Notification Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-donation-notification'),
+							'title' => __('Donation Notification Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -117,7 +117,7 @@ 
 			 * Filter the emails settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_emails', $settings );
+			$settings = apply_filters('give_settings_emails', $settings);
 
 			/**
 			 * Filter the settings.
@@ -126,7 +126,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -140,10 +140,10 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'email-settings' => esc_html__( 'Email Settings', 'give' ),
+				'email-settings' => esc_html__('Email Settings', 'give'),
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 
 		/**
@@ -154,7 +154,7 @@ 
 		 */
 		public function email_notification_setting() {
 			// Load email notification table.
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-notification-table.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-notification-table.php';
 
 			// Init table.
 			$email_notifications_table = new Give_Email_Notification_Table();

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

Spacing +37 added lines, -37 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Advanced' ) ) :
+if ( ! class_exists('Give_Settings_Advanced')) :
 
 	/**
 	 * Give_Settings_Advanced.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'advanced';
-			$this->label = __( 'Advanced', 'give' );
+			$this->label = __('Advanced', 'give');
 
 			$this->default_tab = 'advanced-options';
 
@@ -45,7 +45,7 @@ 
 
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'advanced-options':
 					$settings = array(
 						array(
@@ -53,19 +53,19 @@ 
 							'type' => 'title',
 						),
 						array(
-							'name'    => __( 'Remove Data on Uninstall', 'give' ),
-							'desc'    => __( 'When the plugin is deleted, completely remove all Give data. This includes all Give settings, forms, form meta, donor, donor data, donations. Everything.', 'give' ),
+							'name'    => __('Remove Data on Uninstall', 'give'),
+							'desc'    => __('When the plugin is deleted, completely remove all Give data. This includes all Give settings, forms, form meta, donor, donor data, donations. Everything.', 'give'),
 							'id'      => 'uninstall_on_delete',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Yes, Remove all data', 'give' ),
-								'disabled' => __( 'No, keep my Give settings and donation data', 'give' ),
+								'enabled'  => __('Yes, Remove all data', 'give'),
+								'disabled' => __('No, keep my Give settings and donation data', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Default User Role', 'give' ),
-							'desc'    => __( 'Assign default user roles for donors when donors opt to register as a WP User.', 'give' ),
+							'name'    => __('Default User Role', 'give'),
+							'desc'    => __('Assign default user roles for donors when donors opt to register as a WP User.', 'give'),
 							'id'      => 'donor_default_user_role',
 							'type'    => 'select',
 							'default' => 'give_donor',
@@ -73,44 +73,44 @@ 
 						),
 						array(
 							/* translators: %s: the_content */
-							'name'    => sprintf( __( '%s filter', 'give' ), '<code>the_content</code>' ),
+							'name'    => sprintf(__('%s filter', 'give'), '<code>the_content</code>'),
 							/* translators: 1: https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content 2: the_content */
-							'desc'    => sprintf( __( 'If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give' ), esc_url( 'https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content' ), '<code>the_content</code>' ),
+							'desc'    => sprintf(__('If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give'), esc_url('https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content'), '<code>the_content</code>'),
 							'id'      => 'the_content_filter',
 							'default' => 'enabled',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Script Loading Location', 'give' ),
-							'desc'    => __( 'This allows you to load your Give scripts either in the <code>&lt;head&gt;</code> or footer of your website.', 'give' ),
+							'name'    => __('Script Loading Location', 'give'),
+							'desc'    => __('This allows you to load your Give scripts either in the <code>&lt;head&gt;</code> or footer of your website.', 'give'),
 							'id'      => 'scripts_footer',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'disabled' => __( 'Head', 'give' ),
-								'enabled'  => __( 'Footer', 'give' ),
+								'disabled' => __('Head', 'give'),
+								'enabled'  => __('Footer', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Akismet SPAM Protection', 'give' ),
-							'desc'    => __( 'Add a layer of SPAM protection to your donation submissions with Akismet. When enabled, donation submissions will be first sent to Akismet\'s API if you have the plugin activated and configured.', 'give' ),
+							'name'    => __('Akismet SPAM Protection', 'give'),
+							'desc'    => __('Add a layer of SPAM protection to your donation submissions with Akismet. When enabled, donation submissions will be first sent to Akismet\'s API if you have the plugin activated and configured.', 'give'),
 							'id'      => 'akismet_spam_protection',
 							'type'    => 'radio_inline',
-							'default' => ( give_check_akismet_key() ) ? 'enabled' : 'disabled',
+							'default' => (give_check_akismet_key()) ? 'enabled' : 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'  => __( 'Advanced Settings Docs Link', 'give' ),
+							'name'  => __('Advanced Settings Docs Link', 'give'),
 							'id'    => 'advanced_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-advanced' ),
-							'title' => __( 'Advanced Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-advanced'),
+							'title' => __('Advanced Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -126,20 +126,20 @@ 
 			 *
 			 * @since 2.0
 			 */
-			if( apply_filters( 'give_settings_advanced_show_cache_setting', false ) ) {
-				array_splice( $settings, 1, 0, array(
+			if (apply_filters('give_settings_advanced_show_cache_setting', false)) {
+				array_splice($settings, 1, 0, array(
 					array(
-						'name'    => __( 'Cache', 'give' ),
-						'desc'    => __( 'If caching is enabled the plugin will start caching custom post type related queries and reduce the overall load time.', 'give' ),
+						'name'    => __('Cache', 'give'),
+						'desc'    => __('If caching is enabled the plugin will start caching custom post type related queries and reduce the overall load time.', 'give'),
 						'id'      => 'cache',
 						'type'    => 'radio_inline',
 						'default' => 'enabled',
 						'options' => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					)
-				) );
+				));
 			}
 
 
@@ -147,7 +147,7 @@ 
 			 * Filter the advanced settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_advanced', $settings );
+			$settings = apply_filters('give_settings_advanced', $settings);
 
 			/**
 			 * Filter the settings.
@@ -156,7 +156,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -170,10 +170,10 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'advanced-options' => __( 'Advanced Options', 'give' ),
+				'advanced-options' => __('Advanced Options', 'give'),
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}