WordImpress / Give

includes/admin/emails/class-donor-register-email.php

Doc Comments +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
 		 * @param string                    $email_preview_header
 		 * @param Give_Donor_Register_Email $email
 		 *
-		 * @return bool
+		 * @return string|null
 		 */
 		public function email_preview_header( $email_preview_header, $email ) {
 			// Bailout.

Spacing +37 added lines, -37 removed lines

@@ -10,11 +10,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Donor_Register_Email' ) ) :
+if ( ! class_exists('Give_Donor_Register_Email')) :
 
 	/**
 	 * Give_Donor_Register_Email
@@ -31,34 +31,34 @@ 
 		 * @since   2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                    => 'donor-register',
-				'label'                 => __( 'User Registration Information', 'give' ),
-				'description'           => __( 'Sent to the donor when they register for an account on the site.', 'give' ),
+				'label'                 => __('User Registration Information', 'give'),
+				'description'           => __('Sent to the donor when they register for an account on the site.', 'give'),
 				'notification_status'   => 'enabled',
 				'email_tag_contex'      => 'donor',
 				'form_metabox_setting'  => false,
-				'recipient_group_name'  => __( 'Donor', 'give' ),
-				'email_tag_context'     => array( 'donor', 'general' ),
+				'recipient_group_name'  => __('Donor', 'give'),
+				'email_tag_context'     => array('donor', 'general'),
 				'default_email_subject' => sprintf(
 				/* translators: %s: site name */
-					esc_attr__( '[%s] Your username and password', 'give' ),
-					get_bloginfo( 'name' )
+					esc_attr__('[%s] Your username and password', 'give'),
+					get_bloginfo('name')
 				),
 				'default_email_message' => $this->get_default_email_message(),
-			) );
+			));
 
 			// Setup action hook.
 			add_action(
 				"give_{$this->config['id']}_email_notification",
-				array( $this, 'setup_email_notification' ),
+				array($this, 'setup_email_notification'),
 				10,
 				2
 			);
 
 			add_filter(
 				'give_email_preview_header',
-				array( $this, 'email_preview_header' ),
+				array($this, 'email_preview_header'),
 				10,
 				2
 			);
@@ -73,13 +73,13 @@ 
 		 * @return string
 		 */
 		function get_default_email_message() {
-			$message = esc_attr__( 'Username: {username}', 'give' ) . "\r\n\r\n";
+			$message = esc_attr__('Username: {username}', 'give')."\r\n\r\n";
 
-			$message .= __( 'To reset your password, simply click the link below which will take you to a web page where you can create a new password.', 'give' ) . "\r\n";
-			$message .= '{reset_password_link}' . "\r\n\r\n";
+			$message .= __('To reset your password, simply click the link below which will take you to a web page where you can create a new password.', 'give')."\r\n";
+			$message .= '{reset_password_link}'."\r\n\r\n";
 
-			$message .= __( 'After resetting password, Please login to your account with link below.', 'give' ) . "\r\n";
-			$message .= '<a href="' . wp_login_url() . '"> ' . esc_attr__( 'Click Here to Login &raquo;', 'give' ) . '</a>' . "\r\n";
+			$message .= __('After resetting password, Please login to your account with link below.', 'give')."\r\n";
+			$message .= '<a href="'.wp_login_url().'"> '.esc_attr__('Click Here to Login &raquo;', 'give').'</a>'."\r\n";
 
 			/**
 			 * Filter the default email message
@@ -98,7 +98,7 @@ 
 		 * @since 2.0
 		 */
 		public function setup_email_data() {
-			Give()->emails->__set( 'heading', esc_html__( 'New User Registration', 'give' ) );
+			Give()->emails->__set('heading', esc_html__('New User Registration', 'give'));
 		}
 
 		/**
@@ -112,13 +112,13 @@ 
 		 *
 		 * @return string
 		 */
-		public function setup_email_notification( $user_id, $user_data ) {
+		public function setup_email_notification($user_id, $user_data) {
 			$this->setup_email_data();
 
 			$this->recipient_email = $user_data['user_email'];
-			$this->send_email_notification( array(
+			$this->send_email_notification(array(
 				'user_id' => $user_id,
-			) );
+			));
 		}
 
 		/**
@@ -132,42 +132,42 @@ 
 		 *
 		 * @return bool
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
+		public function email_preview_header($email_preview_header, $email) {
 			// Bailout.
-			if ( $this->config['id'] !== $email->config['id'] ) {
+			if ($this->config['id'] !== $email->config['id']) {
 				return $email_preview_header;
 			}
 
 			// Payment receipt switcher
-			$user_id = give_check_variable( give_clean( $_GET ), 'isset', 0, 'user_id' );
+			$user_id = give_check_variable(give_clean($_GET), 'isset', 0, 'user_id');
 
 			// Get payments.
 			$donors  = new Give_API();
-			$donors  = give_check_variable( $donors->get_donors(), 'empty', array(), 'donors' );
+			$donors  = give_check_variable($donors->get_donors(), 'empty', array(), 'donors');
 			$options = array();
 
 			// Default option.
-			$options[0] = esc_html__( 'No donor(s) found.', 'give' );
+			$options[0] = esc_html__('No donor(s) found.', 'give');
 
 			// Provide nice human readable options.
-			if ( $donors ) {
-				$options[0] = esc_html__( '- Select a donor -', 'give' );
-				foreach ( $donors as $donor ) {
+			if ($donors) {
+				$options[0] = esc_html__('- Select a donor -', 'give');
+				foreach ($donors as $donor) {
 					// Exclude customers for which wp user not exist.
-					if ( ! $donor['info']['user_id'] ) {
+					if ( ! $donor['info']['user_id']) {
 						continue;
 					}
-					$options[ $donor['info']['user_id'] ] = esc_html( '#' . $donor['info']['donor_id'] . ' - ' . $donor['info']['email'] );
+					$options[$donor['info']['user_id']] = esc_html('#'.$donor['info']['donor_id'].' - '.$donor['info']['email']);
 				}
 			}
 
-			$request_url_data = wp_parse_url( $_SERVER['REQUEST_URI'] );
+			$request_url_data = wp_parse_url($_SERVER['REQUEST_URI']);
 			$query            = $request_url_data['query'];
 
 			// Remove user id query param if set from request url.
-			$query = remove_query_arg( array( 'user_id' ), $query );
+			$query = remove_query_arg(array('user_id'), $query);
 
-			$request_url = home_url( '/?' . str_replace( '', '', $query ) );
+			$request_url = home_url('/?'.str_replace('', '', $query));
 			?>
 
 			<!-- Start constructing HTML output.-->
@@ -185,12 +185,12 @@ 
 				</script>
 
 				<label for="give_preview_email_user_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">
-					<?php echo esc_html__( 'Preview email with a donor:', 'give' ); ?>
+					<?php echo esc_html__('Preview email with a donor:', 'give'); ?>
 				</label>
 
 				<?php
 				// The select field with 100 latest transactions
-				echo Give()->html->select( array(
+				echo Give()->html->select(array(
 					'name'             => 'preview_email_user_id',
 					'selected'         => $user_id,
 					'id'               => 'give_preview_email_user_id',
@@ -200,7 +200,7 @@ 
 					'select_atts'      => 'onchange="change_preview()"',
 					'show_option_all'  => false,
 					'show_option_none' => false,
-				) );
+				));
 				?>
 			</div>
 			<?php

includes/admin/emails/class-email-notification-util.php

Doc Comments +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
 	 * @param Give_Email_Notification $email
 	 * @param int $form_id
 	 *
-	 * @return string
+	 * @return boolean
 	 */
 	public static function is_email_notification_active( Give_Email_Notification $email, $form_id = null ) {
 		$notification_status = $email->get_notification_status( $form_id );

Spacing +36 added lines, -36 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,7 +46,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 		}
 
@@ -64,7 +64,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function has_preview( Give_Email_Notification $email ) {
+	public static function has_preview(Give_Email_Notification $email) {
 		return $email->config['has_preview'];
 	}
 
@@ -78,7 +78,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function has_recipient_field( Give_Email_Notification $email ) {
+	public static function has_recipient_field(Give_Email_Notification $email) {
 		return $email->config['has_recipient_field'];
 	}
 
@@ -92,7 +92,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_notification_status_editable( Give_Email_Notification $email ) {
+	public static function is_notification_status_editable(Give_Email_Notification $email) {
 		$user_can_edit = $email->config['notification_status_editable'];
 
 		return (bool) $user_can_edit;
@@ -108,7 +108,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_content_type_editable( Give_Email_Notification $email ) {
+	public static function is_content_type_editable(Give_Email_Notification $email) {
 		return $email->config['content_type_editable'];
 	}
 
@@ -122,7 +122,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_email_preview_has_header( Give_Email_Notification $email ) {
+	public static function is_email_preview_has_header(Give_Email_Notification $email) {
 		return $email->config['has_preview_header'];
 	}
 
@@ -136,7 +136,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_email_preview( Give_Email_Notification $email ) {
+	public static function is_email_preview(Give_Email_Notification $email) {
 		return $email->config['has_preview'];
 	}
 
@@ -150,7 +150,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_show_on_emails_setting_page( Give_Email_Notification $email ){
+	public static function is_show_on_emails_setting_page(Give_Email_Notification $email) {
 		return $email->config['show_on_emails_setting_page'];
 	}
 
@@ -165,8 +165,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function can_use_form_email_options( Give_Email_Notification $email, $form_id = null ){
-		return give_is_setting_enabled( give_get_meta( $form_id, '_give_email_options', true ) );
+	public static function can_use_form_email_options(Give_Email_Notification $email, $form_id = null) {
+		return give_is_setting_enabled(give_get_meta($form_id, '_give_email_options', true));
 	}
 
 	/**
@@ -180,13 +180,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function is_email_notification_active( Give_Email_Notification $email, $form_id = null ) {
-		$notification_status = $email->get_notification_status( $form_id );
+	public static function is_email_notification_active(Give_Email_Notification $email, $form_id = null) {
+		$notification_status = $email->get_notification_status($form_id);
 
 
-		$notification_status = empty( $form_id )
-			? give_is_setting_enabled( $notification_status )
-			: give_is_setting_enabled( give_get_option( "{$email->config['id']}_notification", $email->config['notification_status'] ) ) && give_is_setting_enabled( $notification_status, array( 'enabled', 'global' ) );
+		$notification_status = empty($form_id)
+			? give_is_setting_enabled($notification_status)
+			: give_is_setting_enabled(give_get_option("{$email->config['id']}_notification", $email->config['notification_status'])) && give_is_setting_enabled($notification_status, array('enabled', 'global'));
 			// To check if email notification is active or not on a per-form basis, email notification must be globally active—otherwise it will be considered disabled.
 
 		return $notification_status;
@@ -203,8 +203,8 @@ 
 		$is_preview = false;
 
 		if (
-			current_user_can( 'manage_give_settings' )
-			&& ! empty( $_GET['give_action'] )
+			current_user_can('manage_give_settings')
+			&& ! empty($_GET['give_action'])
 			&& 'preview_email' === $_GET['give_action']
 		) {
 			$is_preview = true;
@@ -224,8 +224,8 @@ 
 		$is_preview = false;
 
 		if (
-			current_user_can( 'manage_give_settings' )
-			&& ! empty( $_GET['give_action'] )
+			current_user_can('manage_give_settings')
+			&& ! empty($_GET['give_action'])
 			&& 'send_preview_email' === $_GET['give_action']
 		) {
 			$is_preview = true;
@@ -245,13 +245,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_formatted_email_type( $content_type ) {
+	public static function get_formatted_email_type($content_type) {
 		$email_contents = array(
-			'text/html'  => __( 'HTML', 'give' ),
-			'text/plain' => __( 'Plain', 'give' ),
+			'text/html'  => __('HTML', 'give'),
+			'text/plain' => __('Plain', 'give'),
 		);
 
-		return $email_contents[ $content_type ];
+		return $email_contents[$content_type];
 	}
 
 
@@ -268,40 +268,40 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_value( Give_Email_Notification $email, $option_name, $form_id = null, $default = false ) {
+	public static function get_value(Give_Email_Notification $email, $option_name, $form_id = null, $default = false) {
 		// If form id set then option name can be contain _give_ prefix which is only used for meta key,
 		// So make sure you are using correct option name.
-		$global_option_name = ( 0 === strpos( $option_name, '_give_' )
-			? str_replace( '_give_', '', $option_name )
-			: $option_name );
-		$option_value = give_get_option( $global_option_name, $default );
+		$global_option_name = (0 === strpos($option_name, '_give_')
+			? str_replace('_give_', '', $option_name)
+			: $option_name);
+		$option_value = give_get_option($global_option_name, $default);
 
 		if (
-			! empty( $form_id )
+			! empty($form_id)
 			&& give_is_setting_enabled(
 				give_get_meta(
 					$form_id,
-					Give_Email_Setting_Field::get_prefix( $email, $form_id ) . 'notification',
+					Give_Email_Setting_Field::get_prefix($email, $form_id).'notification',
 					true,
 					'global'
 				)
 			)
 		) {
-			$option_value = get_post_meta( $form_id, $option_name, true );
+			$option_value = get_post_meta($form_id, $option_name, true);
 
 			// Get only email field value from recipients setting.
-			if( Give_Email_Setting_Field::get_prefix( $email, $form_id ) . 'recipient' === $option_name ) {
-				$option_value = wp_list_pluck( $option_value, 'email' );
+			if (Give_Email_Setting_Field::get_prefix($email, $form_id).'recipient' === $option_name) {
+				$option_value = wp_list_pluck($option_value, 'email');
 			}
 		}
 
-		$option_value = empty( $option_value ) ? $default : $option_value;
+		$option_value = empty($option_value) ? $default : $option_value;
 
 		/**
 		 * Filter the setting value
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_email_setting_value', $option_value, $option_name, $email, $form_id, $default );
+		return apply_filters('give_email_setting_value', $option_value, $option_name, $email, $form_id, $default);
 	}
 }

includes/admin/emails/class-email-notifications.php

Doc Comments +2 added lines, -2 removed lines

@@ -231,7 +231,7 @@ 
 	 *
 	 * @since  2.0
 	 * @access public
-	 * @return bool|null
+	 * @return false|null
 	 */
 	public function preview_email() {
 		// Bailout.
@@ -369,7 +369,7 @@ 
 	 *
 	 * @since  2.0
 	 * @access public
-	 * @return bool|null
+	 * @return false|null
 	 */
 	public function send_preview_email() {
 		// Bailout.

Spacing +82 added lines, -82 removed lines

@@ -52,7 +52,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 		}
 
@@ -66,31 +66,31 @@ 
 	 */
 	public function init() {
 		// Load files.
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/ajax-handler.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-setting-field.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/filters.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/emails/ajax-handler.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-setting-field.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/emails/filters.php';
 
 		// Load email notifications.
 		$this->add_emails_notifications();
 
-		add_filter( 'give_metabox_form_data_settings', array( $this, 'add_metabox_setting_fields' ), 10, 2 );
-		add_action( 'init', array( $this, 'preview_email' ) );
-		add_action( 'init', array( $this, 'send_preview_email' ) );
-		add_action( 'init', array( $this, 'validate_settings' ) );
+		add_filter('give_metabox_form_data_settings', array($this, 'add_metabox_setting_fields'), 10, 2);
+		add_action('init', array($this, 'preview_email'));
+		add_action('init', array($this, 'send_preview_email'));
+		add_action('init', array($this, 'validate_settings'));
 
 		/* @var Give_Email_Notification $email */
-		foreach ( $this->get_email_notifications() as $email ) {
+		foreach ($this->get_email_notifications() as $email) {
 			// Setup email section.
-			if( Give_Email_Notification_Util::is_show_on_emails_setting_page( $email ) ) {
-				add_filter( 'give_get_sections_emails', array( $email, 'add_section' ) );
-				add_filter( "give_hide_section_{$email->config['id']}_on_emails_page", array( $email, 'hide_section' ) );
+			if (Give_Email_Notification_Util::is_show_on_emails_setting_page($email)) {
+				add_filter('give_get_sections_emails', array($email, 'add_section'));
+				add_filter("give_hide_section_{$email->config['id']}_on_emails_page", array($email, 'hide_section'));
 			}
 
 			// Setup email preview.
-			if ( Give_Email_Notification_Util::is_email_preview_has_header( $email ) ) {
-				add_action( "give_{$email->config['id']}_email_preview", array( $this, 'email_preview_header' ) );
-				add_filter( "give_{$email->config['id']}_email_preview_data", array( $this, 'email_preview_data' ) );
-				add_filter( "give_{$email->config['id']}_email_preview_message", array( $this, 'email_preview_message' ), 1, 2 );
+			if (Give_Email_Notification_Util::is_email_preview_has_header($email)) {
+				add_action("give_{$email->config['id']}_email_preview", array($this, 'email_preview_header'));
+				add_filter("give_{$email->config['id']}_email_preview_data", array($this, 'email_preview_data'));
+				add_filter("give_{$email->config['id']}_email_preview_message", array($this, 'email_preview_message'), 1, 2);
 			}
 		}
 	}
@@ -107,64 +107,64 @@ 
 	 *
 	 * @return array
 	 */
-	public function add_metabox_setting_fields( $settings, $post_id ) {
+	public function add_metabox_setting_fields($settings, $post_id) {
 		$emails = $this->get_email_notifications();
 
 		// Bailout.
-		if ( empty( $emails ) ) {
+		if (empty($emails)) {
 			return $settings;
 		}
 
 		// Email notification setting.
 		$settings['email_notification_options'] = array(
 			'id'         => 'email_notification_options',
-			'title'      => __( 'Email Notifications', 'give' ),
+			'title'      => __('Email Notifications', 'give'),
 			'icon-html' => '<span class="dashicons dashicons-email-alt"></span>',
 			'fields'     => array(
 				array(
-					'name'        => __( 'Email Options', 'give' ),
-					'description' => __( '', 'give' ),
+					'name'        => __('Email Options', 'give'),
+					'description' => __('', 'give'),
 					'id'          => '_give_email_options',
 					'type'        => 'radio_inline',
 					'default'     => 'global',
 					'options'     => array(
-						'global'   => __( 'Global Options' ),
-						'enabled'  => __( 'Customize', 'give' ),
+						'global'   => __('Global Options'),
+						'enabled'  => __('Customize', 'give'),
 					),
 				),
 				array(
 					'id'      => '_give_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',
 					'default' => 'default',
 					'options' => give_get_email_templates(),
 				),
 				array(
 					'id'   => '_give_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'      => '_give_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'      => '_give_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'  => 'email_notification_docs',
 					'type'  => 'docs_link',
 					'url'   => 'http://docs.givewp.com/email-notification',
-					'title' => __( 'Email Notification', 'give' ),
+					'title' => __('Email Notification', 'give'),
 				),
 			),
 
@@ -173,7 +173,7 @@ 
 			 *
 			 * @since 2.0
 			 */
-			'sub-fields' => apply_filters( 'give_email_notification_options_metabox_fields', array(), $post_id ),
+			'sub-fields' => apply_filters('give_email_notification_options_metabox_fields', array(), $post_id),
 		);
 
 		return $settings;
@@ -187,13 +187,13 @@ 
 	 */
 	private function add_emails_notifications() {
 		$this->emails = array(
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-new-donation-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-donation-receipt-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-new-offline-donation-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-offline-donation-instruction-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-new-donor-register-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-donor-register-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-access-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-new-donation-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-donation-receipt-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-new-offline-donation-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-offline-donation-instruction-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-new-donor-register-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-donor-register-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-access-email.php',
 		);
 
 		/**
@@ -201,15 +201,15 @@ 
 		 *
 		 * @since 2.0
 		 */
-		$this->emails = apply_filters( 'give_email_notifications', $this->emails, $this );
+		$this->emails = apply_filters('give_email_notifications', $this->emails, $this);
 
 		// Bailout.
-		if ( empty( $this->emails ) ) {
+		if (empty($this->emails)) {
 			return;
 		}
 
 		// Initiate email notifications.
-		foreach ( $this->emails as $email ) {
+		foreach ($this->emails as $email) {
 			$email->init();
 		}
 	}
@@ -236,41 +236,41 @@ 
 	 */
 	public function preview_email() {
 		// Bailout.
-		if ( ! Give_Email_Notification_Util::can_preview_email() ) {
+		if ( ! Give_Email_Notification_Util::can_preview_email()) {
 			return false;
 		}
 
 		// Security check.
-		give_validate_nonce( $_GET['_wpnonce'], 'give-preview-email' );
+		give_validate_nonce($_GET['_wpnonce'], 'give-preview-email');
 
 		// Get email type.
-		$email_type = isset( $_GET['email_type'] ) ? esc_attr( $_GET['email_type'] ) : '';
+		$email_type = isset($_GET['email_type']) ? esc_attr($_GET['email_type']) : '';
 
 		/* @var Give_Email_Notification $email */
-		foreach ( $this->get_email_notifications() as $email ) {
-			if ( $email_type !== $email->config['id'] ) {
+		foreach ($this->get_email_notifications() as $email) {
+			if ($email_type !== $email->config['id']) {
 				continue;
 			}
 
 			// Set form id.
-			$form_id = empty( $_GET['form_id']  ) ? null : absint( $_GET['form_id'] );
+			$form_id = empty($_GET['form_id']) ? null : absint($_GET['form_id']);
 
 			// Call setup email data to apply filter and other thing to email.
 			$email->setup_email_data();
 
 			// Decode message.
-			$email_message = $email->preview_email_template_tags( $email->get_email_message( $form_id ) );
+			$email_message = $email->preview_email_template_tags($email->get_email_message($form_id));
 
 			// Set email template.
-			Give()->emails->html    = true;
-			Give()->emails->__set( 'template', $email->get_email_template( $form_id ) );
+			Give()->emails->html = true;
+			Give()->emails->__set('template', $email->get_email_template($form_id));
 
-			if ( 'text/plain' === $email->config['content_type'] ) {
+			if ('text/plain' === $email->config['content_type']) {
 				// Give()->emails->__set( 'html', false );
-				Give()->emails->__set( 'template', 'none' );
+				Give()->emails->__set('template', 'none');
 			}
 
-			if ( $email_message = Give()->emails->build_email( $email_message ) ) {
+			if ($email_message = Give()->emails->build_email($email_message)) {
 
 				/**
 				 * Filter the email preview data
@@ -279,14 +279,14 @@ 
 				 *
 				 * @param array
 				 */
-				$email_preview_data = apply_filters( "give_{$email_type}_email_preview_data", array() );
+				$email_preview_data = apply_filters("give_{$email_type}_email_preview_data", array());
 
 				/**
 				 * Fire the give_{$email_type}_email_preview action
 				 *
 				 * @since 2.0
 				 */
-				do_action( "give_{$email_type}_email_preview", $email );
+				do_action("give_{$email_type}_email_preview", $email);
 
 				/**
 				 * Filter the email message
@@ -297,7 +297,7 @@ 
 				 * @param array                   $email_preview_data
 				 * @param Give_Email_Notification $email
 				 */
-				echo apply_filters( "give_{$email_type}_email_preview_message", $email_message, $email_preview_data, $email );
+				echo apply_filters("give_{$email_type}_email_preview_message", $email_message, $email_preview_data, $email);
 
 				exit();
 			}
@@ -313,7 +313,7 @@ 
 	 *
 	 * @param Give_Email_Notification $email
 	 */
-	public function email_preview_header( $email ) {
+	public function email_preview_header($email) {
 		/**
 		 * Filter the all email preview headers.
 		 *
@@ -321,7 +321,7 @@ 
 		 *
 		 * @param Give_Email_Notification $email
 		 */
-		$email_preview_header = apply_filters( 'give_email_preview_header', give_get_preview_email_header(), $email );
+		$email_preview_header = apply_filters('give_email_preview_header', give_get_preview_email_header(), $email);
 
 		echo $email_preview_header;
 	}
@@ -336,9 +336,9 @@ 
 	 *
 	 * @return array
 	 */
-	public function email_preview_data( $email_preview_data ) {
-		$email_preview_data['payment_id'] = absint( give_check_variable( give_clean( $_GET ), 'isset', 0, 'preview_id' ) );
-		$email_preview_data['user_id']    = absint( give_check_variable( give_clean( $_GET ), 'isset', 0, 'user_id' ) );
+	public function email_preview_data($email_preview_data) {
+		$email_preview_data['payment_id'] = absint(give_check_variable(give_clean($_GET), 'isset', 0, 'preview_id'));
+		$email_preview_data['user_id']    = absint(give_check_variable(give_clean($_GET), 'isset', 0, 'user_id'));
 
 		return $email_preview_data;
 	}
@@ -354,12 +354,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function email_preview_message( $email_message, $email_preview_data ) {
+	public function email_preview_message($email_message, $email_preview_data) {
 		if (
-			! empty( $email_preview_data['payment_id'] )
-			|| ! empty( $email_preview_data['user_id'] )
+			! empty($email_preview_data['payment_id'])
+			|| ! empty($email_preview_data['user_id'])
 		) {
-			$email_message = give_do_email_tags( $email_message, $email_preview_data );
+			$email_message = give_do_email_tags($email_message, $email_preview_data);
 		}
 
 		return $email_message;
@@ -374,26 +374,26 @@ 
 	 */
 	public function send_preview_email() {
 		// Bailout.
-		if ( ! Give_Email_Notification_Util::can_send_preview_email() ) {
+		if ( ! Give_Email_Notification_Util::can_send_preview_email()) {
 			return false;
 		}
 
 		// Security check.
-		give_validate_nonce( $_GET['_wpnonce'], 'give-send-preview-email' );
+		give_validate_nonce($_GET['_wpnonce'], 'give-send-preview-email');
 
 		// Get email type.
-		$email_type = give_check_variable( give_clean( $_GET ), 'isset', '', 'email_type' );
+		$email_type = give_check_variable(give_clean($_GET), 'isset', '', 'email_type');
 
 		/* @var Give_Email_Notification $email */
-		foreach ( $this->get_email_notifications() as $email ) {
-			if ( $email_type === $email->config['id'] && Give_Email_Notification_Util::is_email_preview( $email ) ) {
+		foreach ($this->get_email_notifications() as $email) {
+			if ($email_type === $email->config['id'] && Give_Email_Notification_Util::is_email_preview($email)) {
 				$email->send_preview_email();
 				break;
 			}
 		}
 
 		// Remove the test email query arg.
-		wp_redirect( remove_query_arg( 'give_action' ) );
+		wp_redirect(remove_query_arg('give_action'));
 		exit;
 	}
 
@@ -405,7 +405,7 @@ 
 	 * @access public
 	 */
 	public function load() {
-		add_action( 'init', array( $this, 'init' ), -1 );
+		add_action('init', array($this, 'init'), -1);
 	}
 
 
@@ -420,7 +420,7 @@ 
 		if (
 			! Give_Admin_Settings::is_saving_settings() ||
 			'emails' !== give_get_current_setting_tab() ||
-			! isset( $_GET['section'] )
+			! isset($_GET['section'])
 		) {
 			return;
 		}
@@ -428,18 +428,18 @@ 
 		// Get email type.
 		$email_type = give_get_current_setting_section();
 
-		if ( ! empty( $_POST["{$email_type}_recipient"] ) ) {
-			$_POST["{$email_type}_recipient"] = array_unique( array_filter( $_POST["{$email_type}_recipient"] ) );
+		if ( ! empty($_POST["{$email_type}_recipient"])) {
+			$_POST["{$email_type}_recipient"] = array_unique(array_filter($_POST["{$email_type}_recipient"]));
 		}
 	}
 }
 
 // Helper class.
-require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/abstract-email-notification.php';
-require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-notification-util.php';
+require_once GIVE_PLUGIN_DIR.'includes/admin/emails/abstract-email-notification.php';
+require_once GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-notification-util.php';
 
 // Add backward compatibility.
-require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/backward-compatibility.php';
+require_once GIVE_PLUGIN_DIR.'includes/admin/emails/backward-compatibility.php';
 
 /**
  * Initialize functionality.

includes/admin/payments/class-payments-table.php

Doc Comments +3 added lines, -3 removed lines

@@ -528,7 +528,7 @@ 
 	/**
 	 * Get donor email html.
 	 *
-	 * @param object $payment Contains all the data of the payment.
+	 * @param Give_Payment $payment Contains all the data of the payment.
 	 *
 	 * @access public
 	 * @since  1.0
@@ -556,7 +556,7 @@ 
 	/**
 	 * Get Row Actions
 	 *
-	 * @param object $payment Payment Data.
+	 * @param Give_Payment $payment Payment Data.
 	 *
 	 * @since 1.6
 	 *
@@ -677,7 +677,7 @@ 
 	/**
 	 * Get donor html.
 	 *
-	 * @param object $payment Contains all the data of the payment.
+	 * @param Give_Payment $payment Contains all the data of the payment.
 	 *
 	 * @access public
 	 * @since  1.0

Spacing +217 added lines, -218 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -136,15 +136,15 @@ 
 		// Set parent defaults.
 		parent::__construct(
 			array(
-				'singular' => give_get_forms_label_singular(),    // Singular name of the listed records.
-				'plural'   => give_get_forms_label_plural(),      // Plural name of the listed records.
-				'ajax'     => false,                              // Does this table support ajax?
+				'singular' => give_get_forms_label_singular(), // Singular name of the listed records.
+				'plural'   => give_get_forms_label_plural(), // Plural name of the listed records.
+				'ajax'     => false, // Does this table support ajax?
 			)
 		);
 
 		$this->process_bulk_action();
 		$this->get_payment_counts();
-		$this->base_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+		$this->base_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 	}
 
 	/**
@@ -153,31 +153,31 @@ 
 	 * @return void
 	 */
 	public function advanced_filters() {
-		$start_date = isset( $_GET['start-date'] ) ? give_clean( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? give_clean( $_GET['end-date'] ) : null;
-		$status     = isset( $_GET['status'] ) ? give_clean( $_GET['status'] ) : '';
-		$donor      = isset( $_GET['donor'] ) ? absint( $_GET['donor'] ) : '';
-		$search     = isset( $_GET['s'] ) ? give_clean( $_GET['s'] ) : '';
-		$form_id    = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : 0;
+		$start_date = isset($_GET['start-date']) ? give_clean($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? give_clean($_GET['end-date']) : null;
+		$status     = isset($_GET['status']) ? give_clean($_GET['status']) : '';
+		$donor      = isset($_GET['donor']) ? absint($_GET['donor']) : '';
+		$search     = isset($_GET['s']) ? give_clean($_GET['s']) : '';
+		$form_id    = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : 0;
 		?>
 		<div id="give-payment-filters" class="give-filters">
-			<?php $this->search_box( __( 'Search', 'give' ), 'give-payments' ); ?>
+			<?php $this->search_box(__('Search', 'give'), 'give-payments'); ?>
 			<div id="give-payment-date-filters">
 				<div class="give-filter give-filter-half">
 					<label for="start-date"
-					       class="give-start-date-label"><?php _e( 'Start Date', 'give' ); ?></label>
+					       class="give-start-date-label"><?php _e('Start Date', 'give'); ?></label>
 					<input type="text" id="start-date" name="start-date" class="give_datepicker"
 					       value="<?php echo $start_date; ?>" placeholder="mm/dd/yyyy"/>
 				</div>
 				<div class="give-filter give-filter-half">
-					<label for="end-date" class="give-end-date-label"><?php _e( 'End Date', 'give' ); ?></label>
+					<label for="end-date" class="give-end-date-label"><?php _e('End Date', 'give'); ?></label>
 					<input type="text" id="end-date" name="end-date" class="give_datepicker"
 					       value="<?php echo $end_date; ?>" placeholder="mm/dd/yyyy"/>
 				</div>
 			</div>
 			<div id="give-payment-form-filter" class="give-filter">
 				<label for="give-donation-forms-filter"
-				       class="give-donation-forms-filter-label"><?php _e( 'Form', 'give' ); ?></label>
+				       class="give-donation-forms-filter-label"><?php _e('Form', 'give'); ?></label>
 				<?php
 				// Filter Donations by Donation Forms.
 				echo Give()->html->forms_dropdown(
@@ -187,7 +187,7 @@ 
 						'class'    => 'give-donation-forms-filter',
 						'selected' => $form_id, // Make sure to have $form_id set to 0, if there is no selection.
 						'chosen'   => true,
-						'number'   => - 1,
+						'number'   => -1,
 					)
 				);
 				?>
@@ -199,26 +199,26 @@ 
 			 *
 			 * @since 1.8.18
 			 */
-			do_action( 'give_payment_table_advanced_filters' );
+			do_action('give_payment_table_advanced_filters');
 
 
-			if ( ! empty( $status ) ) {
-				echo sprintf( '<input type="hidden" name="status" value="%s"/>', esc_attr( $status ) );
+			if ( ! empty($status)) {
+				echo sprintf('<input type="hidden" name="status" value="%s"/>', esc_attr($status));
 			}
 
-			if ( ! empty( $donor ) ) {
-				echo sprintf( '<input type="hidden" name="donor" value="%s"/>', absint( $donor ) );
+			if ( ! empty($donor)) {
+				echo sprintf('<input type="hidden" name="donor" value="%s"/>', absint($donor));
 			}
 			?>
 
 			<div class="give-filter">
-				<?php submit_button( __( 'Apply', 'give' ), 'secondary', '', false ); ?>
+				<?php submit_button(__('Apply', 'give'), 'secondary', '', false); ?>
 				<?php
 				// Clear active filters button.
-				if ( ! empty( $start_date ) || ! empty( $end_date ) || ! empty( $donor ) || ! empty( $search ) || ! empty( $status ) || ! empty( $form_id ) ) :
+				if ( ! empty($start_date) || ! empty($end_date) || ! empty($donor) || ! empty($search) || ! empty($status) || ! empty($form_id)) :
 					?>
-					<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>"
-					   class="button give-clear-filters-button"><?php _e( 'Clear Filters', 'give' ); ?></a>
+					<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>"
+					   class="button give-clear-filters-button"><?php _e('Clear Filters', 'give'); ?></a>
 				<?php endif; ?>
 			</div>
 		</div>
@@ -237,18 +237,18 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		if ( empty( $_REQUEST['s'] ) && ! $this->has_items() ) {
+	public function search_box($text, $input_id) {
+		if (empty($_REQUEST['s']) && ! $this->has_items()) {
 			return;
 		}
 
-		$input_id = $input_id . '-search-input';
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<div class="give-filter give-filter-search" role="search">
@@ -260,13 +260,13 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payment_history_search' );
+			do_action('give_payment_history_search');
 			?>
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?><br/>
+			)); ?><br/>
 		</div>
 		<?php
 	}
@@ -281,48 +281,48 @@ 
 	 */
 	public function get_views() {
 
-		$current = isset( $_GET['status'] ) ? $_GET['status'] : '';
+		$current = isset($_GET['status']) ? $_GET['status'] : '';
 		$views   = array();
 		$tabs    = array(
 			'all'         => array(
 				'total_count',
-				__( 'All', 'give' ),
+				__('All', 'give'),
 			),
 			'publish'     => array(
 				'complete_count',
-				__( 'Completed', 'give' ),
+				__('Completed', 'give'),
 			),
 			'pending'     => array(
 				'pending_count',
-				__( 'Pending', 'give' ),
+				__('Pending', 'give'),
 			),
 			'processing'  => array(
 				'processing_count',
-				__( 'Processing', 'give' ),
+				__('Processing', 'give'),
 			),
 			'refunded'    => array(
 				'refunded_count',
-				__( 'Refunded', 'give' ),
+				__('Refunded', 'give'),
 			),
 			'revoked'     => array(
 				'revoked_count',
-				__( 'Revoked', 'give' ),
+				__('Revoked', 'give'),
 			),
 			'failed'      => array(
 				'failed_count',
-				__( 'Failed', 'give' ),
+				__('Failed', 'give'),
 			),
 			'cancelled'   => array(
 				'cancelled_count',
-				__( 'Cancelled', 'give' ),
+				__('Cancelled', 'give'),
 			),
 			'abandoned'   => array(
 				'abandoned_count',
-				__( 'Abandoned', 'give' ),
+				__('Abandoned', 'give'),
 			),
 			'preapproval' => array(
 				'preapproval_count',
-				__( 'Preapproval Pending', 'give' ),
+				__('Preapproval Pending', 'give'),
 			),
 		);
 
@@ -331,12 +331,12 @@ 
 		 *
 		 * @since 1.8.18
 		 */
-		$args = (array) apply_filters( 'give_payments_table_status_remove_query_arg', array( 'paged', '_wpnonce', '_wp_http_referer' ) );
+		$args = (array) apply_filters('give_payments_table_status_remove_query_arg', array('paged', '_wpnonce', '_wp_http_referer'));
 
 		// Build URL.
-		$staus_url = remove_query_arg( $args );
+		$staus_url = remove_query_arg($args);
 
-		foreach ( $tabs as $key => $tab ) {
+		foreach ($tabs as $key => $tab) {
 			$count_key = $tab[0];
 			$name      = $tab[1];
 			$count     = $this->$count_key;
@@ -351,16 +351,15 @@ 
 			 *
 			 * @since 1.8.12
 			 */
-			if ( 'all' === $key || $key === $current || apply_filters( 'give_payments_table_show_all_status', 0 < $count, $key, $count ) ) {
+			if ('all' === $key || $key === $current || apply_filters('give_payments_table_show_all_status', 0 < $count, $key, $count)) {
 
 				$staus_url = 'all' === $key ?
-					add_query_arg( array( 'status' => false ), $staus_url ) :
-					add_query_arg( array( 'status' => $key ), $staus_url );
+					add_query_arg(array('status' => false), $staus_url) : add_query_arg(array('status' => $key), $staus_url);
 
-				$views[ $key ] = sprintf(
+				$views[$key] = sprintf(
 					'<a href="%s"%s>%s&nbsp;<span class="count">(%s)</span></a>',
-					esc_url( $staus_url ),
-					( ( 'all' === $key && empty( $current ) ) ) ? ' class="current"' : ( $current == $key ? 'class="current"' : '' ),
+					esc_url($staus_url),
+					(('all' === $key && empty($current))) ? ' class="current"' : ($current == $key ? 'class="current"' : ''),
 					$name,
 					$count
 				);
@@ -375,7 +374,7 @@ 
 		 * @param array $views
 		 * @param Give_Payment_History_Table
 		 */
-		return apply_filters( 'give_payments_table_views', $views, $this );
+		return apply_filters('give_payments_table_views', $views, $this);
 	}
 
 	/**
@@ -389,18 +388,18 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'donation'      => __( 'Donation', 'give' ),
-			'donation_form' => __( 'Donation Form', 'give' ),
-			'status'        => __( 'Status', 'give' ),
-			'date'          => __( 'Date', 'give' ),
-			'amount'        => __( 'Amount', 'give' ),
+			'donation'      => __('Donation', 'give'),
+			'donation_form' => __('Donation Form', 'give'),
+			'status'        => __('Status', 'give'),
+			'date'          => __('Date', 'give'),
+			'amount'        => __('Amount', 'give'),
 		);
 
-		if ( current_user_can( 'view_give_payments' ) ) {
-			$columns['details'] = __( 'Details', 'give' );
+		if (current_user_can('view_give_payments')) {
+			$columns['details'] = __('Details', 'give');
 		}
 
-		return apply_filters( 'give_payments_table_columns', $columns );
+		return apply_filters('give_payments_table_columns', $columns);
 	}
 
 	/**
@@ -413,14 +412,14 @@ 
 	 */
 	public function get_sortable_columns() {
 		$columns = array(
-			'donation'      => array( 'ID', true ),
-			'donation_form' => array( 'donation_form', false ),
-			'status'        => array( 'status', false ),
-			'amount'        => array( 'amount', false ),
-			'date'          => array( 'date', false ),
+			'donation'      => array('ID', true),
+			'donation_form' => array('donation_form', false),
+			'status'        => array('status', false),
+			'amount'        => array('amount', false),
+			'date'          => array('date', false),
 		);
 
-		return apply_filters( 'give_payments_table_sortable_columns', $columns );
+		return apply_filters('give_payments_table_sortable_columns', $columns);
 	}
 
 	/**
@@ -446,43 +445,43 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $payment, $column_name ) {
+	public function column_default($payment, $column_name) {
 
-		$single_donation_url = esc_url( add_query_arg( 'id', $payment->ID, admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details' ) ) );
-		$row_actions         = $this->get_row_actions( $payment );
+		$single_donation_url = esc_url(add_query_arg('id', $payment->ID, admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details')));
+		$row_actions         = $this->get_row_actions($payment);
 		$value               = '';
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'donation' :
-				if ( current_user_can( 'view_give_payments' ) ) {
-					$value = Give()->tooltips->render_link( array(
-						'label'       => sprintf( __( 'View Donation #%s', 'give' ), $payment->ID ),
+				if (current_user_can('view_give_payments')) {
+					$value = Give()->tooltips->render_link(array(
+						'label'       => sprintf(__('View Donation #%s', 'give'), $payment->ID),
 						'tag_content' => "#$payment->ID",
 						'link'        => $single_donation_url,
-					) );
+					));
 				} else {
 					$value = "#{$payment->ID}";
 				}
 
 				$value .= sprintf(
 					'&nbsp;%1$s&nbsp;%2$s<br>',
-					__( 'by', 'give' ),
-					$this->get_donor( $payment )
+					__('by', 'give'),
+					$this->get_donor($payment)
 				);
 
-				$value .= $this->get_donor_email( $payment );
-				$value .= $this->row_actions( $row_actions );
+				$value .= $this->get_donor_email($payment);
+				$value .= $this->row_actions($row_actions);
 				break;
 
 			case 'amount':
-				$amount = ! empty( $payment->total ) ? $payment->total : 0;
-				$value  = give_donation_amount( $payment, true );
-				$value .= sprintf( '<br><small>%1$s %2$s</small>', __( 'via', 'give' ), give_get_gateway_admin_label( $payment->gateway ) );
+				$amount = ! empty($payment->total) ? $payment->total : 0;
+				$value  = give_donation_amount($payment, true);
+				$value .= sprintf('<br><small>%1$s %2$s</small>', __('via', 'give'), give_get_gateway_admin_label($payment->gateway));
 				break;
 
 			case 'donation_form':
-				$form_title = empty( $payment->form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $payment->form_id ) : $payment->form_title;
-				$value      = '<a href="' . admin_url( 'post.php?post=' . $payment->form_id . '&action=edit' ) . '">' . $form_title . '</a>';
+				$form_title = empty($payment->form_title) ? sprintf(__('Untitled (#%s)', 'give'), $payment->form_id) : $payment->form_title;
+				$value      = '<a href="'.admin_url('post.php?post='.$payment->form_id.'&action=edit').'">'.$form_title.'</a>';
 				$level      = give_get_donation_form_title(
 					$payment,
 					array(
@@ -490,44 +489,44 @@ 
 					)
 				);
 
-				if ( ! empty( $level ) ) {
+				if ( ! empty($level)) {
 					$value .= $level;
 				}
 
 				break;
 
 			case 'date':
-				$date  = strtotime( $payment->date );
-				$value = date_i18n( give_date_format(), $date );
+				$date  = strtotime($payment->date);
+				$value = date_i18n(give_date_format(), $date);
 				break;
 
 			case 'status':
-				$value = $this->get_payment_status( $payment );
+				$value = $this->get_payment_status($payment);
 				break;
 
 
 			case 'details' :
-				if ( current_user_can( 'view_give_payments' ) ) {
-					$value = Give()->tooltips->render_link( array(
-						'label'       => sprintf( __( 'View Donation #%s', 'give' ), $payment->ID ),
+				if (current_user_can('view_give_payments')) {
+					$value = Give()->tooltips->render_link(array(
+						'label'       => sprintf(__('View Donation #%s', 'give'), $payment->ID),
 						'tag_content' => '<span class="dashicons dashicons-visibility"></span>',
 						'link'        => $single_donation_url,
 						'attributes'  => array(
 							'class' => 'give-payment-details-link button button-small',
 						),
-					) );
+					));
 
 					$value = "<div class=\"give-payment-details-link-wrap\">{$value}</div>";
 				}
 				break;
 
 			default:
-				$value = isset( $payment->$column_name ) ? $payment->$column_name : '';
+				$value = isset($payment->$column_name) ? $payment->$column_name : '';
 				break;
 
 		}// End switch().
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, $column_name );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, $column_name);
 	}
 
 	/**
@@ -540,22 +539,22 @@ 
 	 *
 	 * @return string Data shown in the Email column
 	 */
-	public function get_donor_email( $payment ) {
+	public function get_donor_email($payment) {
 
-		$email = give_get_payment_user_email( $payment->ID );
+		$email = give_get_payment_user_email($payment->ID);
 
-		if ( empty( $email ) ) {
-			$email = __( '(unknown)', 'give' );
+		if (empty($email)) {
+			$email = __('(unknown)', 'give');
 		}
 
 
-		$value = Give()->tooltips->render_link( array(
+		$value = Give()->tooltips->render_link(array(
 			'link'        => "mailto:{$email}",
-			'label'       => __( 'Email donor', 'give' ),
+			'label'       => __('Email donor', 'give'),
 			'tag_content' => $email,
-		) );
+		));
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'email' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'email');
 	}
 
 	/**
@@ -567,18 +566,18 @@ 
 	 *
 	 * @return array $actions
 	 */
-	function get_row_actions( $payment ) {
+	function get_row_actions($payment) {
 
 		$actions = array();
-		$email   = give_get_payment_user_email( $payment->ID );
+		$email   = give_get_payment_user_email($payment->ID);
 
 		// Add search term string back to base URL.
-		$search_terms = ( isset( $_GET['s'] ) ? trim( $_GET['s'] ) : '' );
-		if ( ! empty( $search_terms ) ) {
-			$this->base_url = add_query_arg( 's', $search_terms, $this->base_url );
+		$search_terms = (isset($_GET['s']) ? trim($_GET['s']) : '');
+		if ( ! empty($search_terms)) {
+			$this->base_url = add_query_arg('s', $search_terms, $this->base_url);
 		}
 
-		if ( give_is_payment_complete( $payment->ID ) && ! empty( $email ) ) {
+		if (give_is_payment_complete($payment->ID) && ! empty($email)) {
 
 			$actions['email_links'] = sprintf(
 				'<a class="resend-single-donation-receipt" href="%1$s" aria-label="%2$s">%3$s</a>', wp_nonce_url(
@@ -588,12 +587,12 @@ 
 						'purchase_id' => $payment->ID,
 					), $this->base_url
 				), 'give_payment_nonce'
-			), sprintf( __( 'Resend Donation %s Receipt', 'give' ), $payment->ID ), __( 'Resend Receipt', 'give' )
+			), sprintf(__('Resend Donation %s Receipt', 'give'), $payment->ID), __('Resend Receipt', 'give')
 			);
 
 		}
 
-		if ( current_user_can( 'view_give_payments' ) ) {
+		if (current_user_can('view_give_payments')) {
 			$actions['delete'] = sprintf(
 				'<a class="delete-single-donation" href="%1$s" aria-label="%2$s">%3$s</a>',
 				wp_nonce_url(
@@ -603,11 +602,11 @@ 
 							'purchase_id' => $payment->ID,
 						), $this->base_url
 					), 'give_donation_nonce'
-				), sprintf( __( 'Delete Donation %s', 'give' ), $payment->ID ), __( 'Delete', 'give' )
+				), sprintf(__('Delete Donation %s', 'give'), $payment->ID), __('Delete', 'give')
 			);
 		}
 
-		return apply_filters( 'give_payment_row_actions', $actions, $payment );
+		return apply_filters('give_payment_row_actions', $actions, $payment);
 	}
 
 
@@ -621,30 +620,30 @@ 
 	 *
 	 * @return string Data shown in the Email column
 	 */
-	function get_payment_status( $payment ) {
+	function get_payment_status($payment) {
 		$value = sprintf(
 			'<div class="give-donation-status status-%1$s"><span class="give-donation-status-icon"></span>&nbsp;%2$s</div>',
 			$payment->status,
-			give_get_payment_status( $payment, true )
+			give_get_payment_status($payment, true)
 		);
 
-		if ( $payment->mode == 'test' ) {
-			$value .= Give()->tooltips->render_span( array(
-				'label'       => __( 'This donation was made in test mode.', 'give' ),
-				'tag_content' => __( 'Test', 'give' ),
+		if ($payment->mode == 'test') {
+			$value .= Give()->tooltips->render_span(array(
+				'label'       => __('This donation was made in test mode.', 'give'),
+				'tag_content' => __('Test', 'give'),
 				'attributes'  => array(
 					'class' => 'give-item-label give-item-label-orange give-test-mode-transactions-label',
 				),
 
 
-			) );
+			));
 		}
 
-		if ( true === $payment->import && true === (bool) apply_filters( 'give_payment_show_importer_label', false ) ) {
+		if (true === $payment->import && true === (bool) apply_filters('give_payment_show_importer_label', false)) {
 			$value .= sprintf(
 				'&nbsp;<span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="%1$s">%2$s</span>',
-				__( 'This donation was imported.', 'give' ),
-				__( 'Import', 'give' )
+				__('This donation was imported.', 'give'),
+				__('Import', 'give')
 			);
 		}
 
@@ -661,8 +660,8 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function column_cb( $payment ) {
-		return sprintf( '<input type="checkbox" name="%1$s[]" value="%2$s" />', 'payment', $payment->ID );
+	public function column_cb($payment) {
+		return sprintf('<input type="checkbox" name="%1$s[]" value="%2$s" />', 'payment', $payment->ID);
 	}
 
 	/**
@@ -675,8 +674,8 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function get_payment_id( $payment ) {
-		return '<span class="give-payment-id">' . give_get_payment_number( $payment->ID ) . '</span>';
+	public function get_payment_id($payment) {
+		return '<span class="give-payment-id">'.give_get_payment_number($payment->ID).'</span>';
 	}
 
 	/**
@@ -689,32 +688,32 @@ 
 	 *
 	 * @return string Data shown in the User column
 	 */
-	public function get_donor( $payment ) {
+	public function get_donor($payment) {
 
-		$donor_id           = give_get_payment_donor_id( $payment->ID );
-		$donor_billing_name = give_get_donor_name_by( $payment->ID, 'donation' );
-		$donor_name         = give_get_donor_name_by( $donor_id, 'donor' );
+		$donor_id           = give_get_payment_donor_id($payment->ID);
+		$donor_billing_name = give_get_donor_name_by($payment->ID, 'donation');
+		$donor_name         = give_get_donor_name_by($donor_id, 'donor');
 
 		$value = '';
-		if ( ! empty( $donor_id ) ) {
+		if ( ! empty($donor_id)) {
 
 			// Check whether the donor name and WP_User name is same or not.
-			if ( sanitize_title( $donor_billing_name ) !== sanitize_title( $donor_name ) ) {
-				$value .= $donor_billing_name . ' (';
+			if (sanitize_title($donor_billing_name) !== sanitize_title($donor_name)) {
+				$value .= $donor_billing_name.' (';
 			}
 
-			$value .= '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id" ) ) . '">' . $donor_name . '</a>';
+			$value .= '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id")).'">'.$donor_name.'</a>';
 
 			// Check whether the donor name and WP_User name is same or not.
-			if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
+			if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
 				$value .= ')';
 			}
 		} else {
-			$email  = give_get_payment_user_email( $payment->ID );
-			$value .= '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-payment-history&s=$email" ) ) . '">' . __( '(donor missing)', 'give' ) . '</a>';
+			$email  = give_get_payment_user_email($payment->ID);
+			$value .= '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-payment-history&s=$email")).'">'.__('(donor missing)', 'give').'</a>';
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'donor' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'donor');
 	}
 
 	/**
@@ -727,20 +726,20 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete'                 => __( 'Delete', 'give' ),
-			'set-status-publish'     => __( 'Set To Completed', 'give' ),
-			'set-status-pending'     => __( 'Set To Pending', 'give' ),
-			'set-status-processing'  => __( 'Set To Processing', 'give' ),
-			'set-status-refunded'    => __( 'Set To Refunded', 'give' ),
-			'set-status-revoked'     => __( 'Set To Revoked', 'give' ),
-			'set-status-failed'      => __( 'Set To Failed', 'give' ),
-			'set-status-cancelled'   => __( 'Set To Cancelled', 'give' ),
-			'set-status-abandoned'   => __( 'Set To Abandoned', 'give' ),
-			'set-status-preapproval' => __( 'Set To Preapproval', 'give' ),
-			'resend-receipt'         => __( 'Resend Email Receipts', 'give' ),
+			'delete'                 => __('Delete', 'give'),
+			'set-status-publish'     => __('Set To Completed', 'give'),
+			'set-status-pending'     => __('Set To Pending', 'give'),
+			'set-status-processing'  => __('Set To Processing', 'give'),
+			'set-status-refunded'    => __('Set To Refunded', 'give'),
+			'set-status-revoked'     => __('Set To Revoked', 'give'),
+			'set-status-failed'      => __('Set To Failed', 'give'),
+			'set-status-cancelled'   => __('Set To Cancelled', 'give'),
+			'set-status-abandoned'   => __('Set To Abandoned', 'give'),
+			'set-status-preapproval' => __('Set To Preapproval', 'give'),
+			'resend-receipt'         => __('Resend Email Receipts', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_bulk_actions', $actions );
+		return apply_filters('give_payments_table_bulk_actions', $actions);
 	}
 
 	/**
@@ -752,59 +751,59 @@ 
 	 * @return void
 	 */
 	public function process_bulk_action() {
-		$ids    = isset( $_GET['payment'] ) ? $_GET['payment'] : false;
+		$ids    = isset($_GET['payment']) ? $_GET['payment'] : false;
 		$action = $this->current_action();
 
-		if ( ! is_array( $ids ) ) {
-			$ids = array( $ids );
+		if ( ! is_array($ids)) {
+			$ids = array($ids);
 		}
 
-		if ( empty( $action ) ) {
+		if (empty($action)) {
 			return;
 		}
 
-		foreach ( $ids as $id ) {
+		foreach ($ids as $id) {
 
 			// Detect when a bulk action is being triggered.
-			switch ( $this->current_action() ) {
+			switch ($this->current_action()) {
 
 				case 'delete':
-					give_delete_donation( $id );
+					give_delete_donation($id);
 					break;
 
 				case 'set-status-publish':
-					give_update_payment_status( $id, 'publish' );
+					give_update_payment_status($id, 'publish');
 					break;
 
 				case 'set-status-pending':
-					give_update_payment_status( $id, 'pending' );
+					give_update_payment_status($id, 'pending');
 					break;
 
 				case 'set-status-processing':
-					give_update_payment_status( $id, 'processing' );
+					give_update_payment_status($id, 'processing');
 					break;
 
 				case 'set-status-refunded':
-					give_update_payment_status( $id, 'refunded' );
+					give_update_payment_status($id, 'refunded');
 					break;
 				case 'set-status-revoked':
-					give_update_payment_status( $id, 'revoked' );
+					give_update_payment_status($id, 'revoked');
 					break;
 
 				case 'set-status-failed':
-					give_update_payment_status( $id, 'failed' );
+					give_update_payment_status($id, 'failed');
 					break;
 
 				case 'set-status-cancelled':
-					give_update_payment_status( $id, 'cancelled' );
+					give_update_payment_status($id, 'cancelled');
 					break;
 
 				case 'set-status-abandoned':
-					give_update_payment_status( $id, 'abandoned' );
+					give_update_payment_status($id, 'abandoned');
 					break;
 
 				case 'set-status-preapproval':
-					give_update_payment_status( $id, 'preapproval' );
+					give_update_payment_status($id, 'preapproval');
 					break;
 
 				case 'resend-receipt':
@@ -813,7 +812,7 @@ 
 					 *
 					 * @since 2.0
 					 */
-					do_action( 'give_donation-receipt_email_notification', $id );
+					do_action('give_donation-receipt_email_notification', $id);
 					break;
 			}// End switch().
 
@@ -825,7 +824,7 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payments_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_payments_table_do_bulk_action', $id, $this->current_action());
 		}// End foreach().
 
 	}
@@ -842,32 +841,32 @@ 
 
 		$args = array();
 
-		if ( isset( $_GET['user'] ) ) {
-			$args['user'] = urldecode( $_GET['user'] );
-		} elseif ( isset( $_GET['donor'] ) ) {
-			$args['donor'] = absint( $_GET['donor'] );
-		} elseif ( isset( $_GET['s'] ) ) {
-			$is_user = strpos( $_GET['s'], strtolower( 'user:' ) ) !== false;
-			if ( $is_user ) {
-				$args['user'] = absint( trim( str_replace( 'user:', '', strtolower( $_GET['s'] ) ) ) );
-				unset( $args['s'] );
+		if (isset($_GET['user'])) {
+			$args['user'] = urldecode($_GET['user']);
+		} elseif (isset($_GET['donor'])) {
+			$args['donor'] = absint($_GET['donor']);
+		} elseif (isset($_GET['s'])) {
+			$is_user = strpos($_GET['s'], strtolower('user:')) !== false;
+			if ($is_user) {
+				$args['user'] = absint(trim(str_replace('user:', '', strtolower($_GET['s']))));
+				unset($args['s']);
 			} else {
-				$args['s'] = sanitize_text_field( $_GET['s'] );
+				$args['s'] = sanitize_text_field($_GET['s']);
 			}
 		}
 
-		if ( ! empty( $_GET['start-date'] ) ) {
-			$args['start-date'] = urldecode( $_GET['start-date'] );
+		if ( ! empty($_GET['start-date'])) {
+			$args['start-date'] = urldecode($_GET['start-date']);
 		}
 
-		if ( ! empty( $_GET['end-date'] ) ) {
-			$args['end-date'] = urldecode( $_GET['end-date'] );
+		if ( ! empty($_GET['end-date'])) {
+			$args['end-date'] = urldecode($_GET['end-date']);
 		}
 
-		$args['form_id'] = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
-		$args['gateway'] = ! empty( $_GET['gateway'] ) ? give_clean( $_GET['gateway'] ) : null;
+		$args['form_id'] = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
+		$args['gateway'] = ! empty($_GET['gateway']) ? give_clean($_GET['gateway']) : null;
 
-		$payment_count           = give_count_payments( $args );
+		$payment_count           = give_count_payments($args);
 		$this->complete_count    = $payment_count->publish;
 		$this->pending_count     = $payment_count->pending;
 		$this->processing_count  = $payment_count->processing;
@@ -878,7 +877,7 @@ 
 		$this->abandoned_count   = $payment_count->abandoned;
 		$this->preapproval_count = $payment_count->preapproval;
 
-		foreach ( $payment_count as $count ) {
+		foreach ($payment_count as $count) {
 			$this->total_count += $count;
 		}
 
@@ -896,25 +895,25 @@ 
 	public function payments_data() {
 
 		$per_page   = $this->per_page;
-		$orderby    = isset( $_GET['orderby'] ) ? urldecode( $_GET['orderby'] ) : 'ID';
-		$order      = isset( $_GET['order'] ) ? $_GET['order'] : 'DESC';
-		$user       = isset( $_GET['user'] ) ? $_GET['user'] : null;
-		$donor      = isset( $_GET['donor'] ) ? $_GET['donor'] : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : give_get_payment_status_keys();
-		$meta_key   = isset( $_GET['meta_key'] ) ? $_GET['meta_key'] : null;
-		$year       = isset( $_GET['year'] ) ? $_GET['year'] : null;
-		$month      = isset( $_GET['m'] ) ? $_GET['m'] : null;
-		$day        = isset( $_GET['day'] ) ? $_GET['day'] : null;
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : null;
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : $start_date;
-		$form_id    = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
-		$gateway    = ! empty( $_GET['gateway'] ) ? give_clean( $_GET['gateway'] ) : null;
+		$orderby    = isset($_GET['orderby']) ? urldecode($_GET['orderby']) : 'ID';
+		$order      = isset($_GET['order']) ? $_GET['order'] : 'DESC';
+		$user       = isset($_GET['user']) ? $_GET['user'] : null;
+		$donor      = isset($_GET['donor']) ? $_GET['donor'] : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : give_get_payment_status_keys();
+		$meta_key   = isset($_GET['meta_key']) ? $_GET['meta_key'] : null;
+		$year       = isset($_GET['year']) ? $_GET['year'] : null;
+		$month      = isset($_GET['m']) ? $_GET['m'] : null;
+		$day        = isset($_GET['day']) ? $_GET['day'] : null;
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : null;
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : $start_date;
+		$form_id    = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
+		$gateway    = ! empty($_GET['gateway']) ? give_clean($_GET['gateway']) : null;
 
 		$args = array(
 			'output'     => 'payments',
 			'number'     => $per_page,
-			'page'       => isset( $_GET['paged'] ) ? $_GET['paged'] : null,
+			'page'       => isset($_GET['paged']) ? $_GET['paged'] : null,
 			'orderby'    => $orderby,
 			'order'      => $order,
 			'user'       => $user,
@@ -931,9 +930,9 @@ 
 			'give_forms' => $form_id,
 		);
 
-		if ( is_string( $search ) && false !== strpos( $search, 'txn:' ) ) {
+		if (is_string($search) && false !== strpos($search, 'txn:')) {
 			$args['search_in_notes'] = true;
-			$args['s']               = trim( str_replace( 'txn:', '', $args['s'] ) );
+			$args['s']               = trim(str_replace('txn:', '', $args['s']));
 		}
 
 		/**
@@ -941,9 +940,9 @@ 
 		 *
 		 * @since 1.8.18
 		 */
-		$args = (array) apply_filters( 'give_payment_table_payments_query', $args );
+		$args = (array) apply_filters('give_payment_table_payments_query', $args);
 
-		$p_query = new Give_Payments_Query( $args );
+		$p_query = new Give_Payments_Query($args);
 
 		return $p_query->get_payments();
 
@@ -964,17 +963,17 @@ 
 	 */
 	public function prepare_items() {
 
-		wp_reset_vars( array( 'action', 'payment', 'orderby', 'order', 's' ) );
+		wp_reset_vars(array('action', 'payment', 'orderby', 'order', 's'));
 
 		$columns  = $this->get_columns();
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 		$data     = $this->payments_data();
-		$status   = isset( $_GET['status'] ) ? $_GET['status'] : 'any';
+		$status   = isset($_GET['status']) ? $_GET['status'] : 'any';
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'publish':
 				$total_items = $this->complete_count;
 				break;
@@ -1007,8 +1006,8 @@ 
 				break;
 			default:
 				// Retrieve the count of the non-default-Give status.
-				$count       = wp_count_posts( 'give_payment' );
-				$total_items = isset( $count->{$status} ) ? $count->{$status} : 0;
+				$count       = wp_count_posts('give_payment');
+				$total_items = isset($count->{$status} ) ? $count->{$status} : 0;
 				break;
 		}
 
@@ -1019,7 +1018,7 @@ 
 		 *
 		 * @since 1.8.19
 		 */
-		$total_items = (int) apply_filters( 'give_payment_table_pagination_total_count', $total_items, $this );
+		$total_items = (int) apply_filters('give_payment_table_pagination_total_count', $total_items, $this);
 
 		$this->set_pagination_args(
 			array(
@@ -1027,7 +1026,7 @@ 
 				// We have to calculate the total number of items.
 				'per_page'    => $this->per_page,
 				// We have to determine how many items to show on a page.
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 				// We have to calculate the total number of pages.
 			)
 		);

includes/admin/tools/logs/class-api-requests-logs-list-table.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +64 added lines, -64 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 	}
 
 	/**
@@ -63,20 +63,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 		<?php
 	}
@@ -91,10 +91,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'      => __( 'Log ID', 'give' ),
-			'ip'      => __( 'Request IP', 'give' ),
-			'date'    => __( 'Date', 'give' ),
-			'details' => __( 'Request Details', 'give' ),
+			'ID'      => __('Log ID', 'give'),
+			'ip'      => __('Request IP', 'give'),
+			'date'    => __('Date', 'give'),
+			'details' => __('Request Details', 'give'),
 		);
 
 		return $columns;
@@ -111,10 +111,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return esc_attr( $item[ $column_name ] );
+				return esc_attr($item[$column_name]);
 		}
 	}
 
@@ -128,52 +128,52 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_details( $item ) {
-		echo Give()->tooltips->render_link( array(
-			'label'       => __( 'View Request', 'give' ),
+	public function column_details($item) {
+		echo Give()->tooltips->render_link(array(
+			'label'       => __('View Request', 'give'),
 			'tag_content' => '<span class="dashicons dashicons-visibility"></span>',
 			'link'        => "#TB_inline?width=640&amp;inlineId=log-details-{$item['ID']}",
 			'attributes'  => array(
 				'class' => 'thickbox give-error-log-details-link button button-small',
 			),
-		) );
+		));
 		?>
 		<div id="log-details-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 			// Print API Request.
 			echo sprintf(
 				'<p><strong>%1$s</strong></p><div>%2$s</div>',
-				__( 'API Request:', 'give' ),
-				Give()->logs->logmeta_db->get_meta( $item['ID'], '_give_log_api_query', true )
+				__('API Request:', 'give'),
+				Give()->logs->logmeta_db->get_meta($item['ID'], '_give_log_api_query', true)
 			);
 
 			// Print Log Content, if not empty.
-			if ( ! empty( $item['log_content'] ) ) {
+			if ( ! empty($item['log_content'])) {
 				echo sprintf(
 					'<p><strong>%1$s</strong></p><div>%2$s</div>',
-					__( 'Error', 'give' ),
-					esc_html( $item['log_content'] )
+					__('Error', 'give'),
+					esc_html($item['log_content'])
 				);
 			}
 
 			// Print User who requested data using API.
 			echo sprintf(
 				'<p><strong>%1$s</strong></p><div>%2$s</div>',
-				__( 'API User:', 'give' ),
-				Give()->logs->logmeta_db->get_meta( $item['ID'], '_give_log_user', true )
+				__('API User:', 'give'),
+				Give()->logs->logmeta_db->get_meta($item['ID'], '_give_log_user', true)
 			);
 
 			// Print the logged key used by API.
 			echo sprintf(
 				'<p><strong>%1$s</strong></p><div>%2$s</div>',
-				__( 'API Key:', 'give' ),
-				Give()->logs->logmeta_db->get_meta( $item['ID'], '_give_log_key', true )
+				__('API Key:', 'give'),
+				Give()->logs->logmeta_db->get_meta($item['ID'], '_give_log_key', true)
 			);
 
 			// Print the API Request Date.
 			echo sprintf(
 				'<p><strong>%1$s</strong></p><div>%2$s</div>',
-				__( 'Request Date:', 'give' ),
+				__('Request Date:', 'give'),
 				$item['log_date']
 			);
 			?>
@@ -190,7 +190,7 @@ 
 	 * @return string|bool String if search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -206,19 +206,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -241,40 +241,40 @@ 
 		$meta_query = array();
 		$search     = $this->get_search();
 
-		if ( $search ) {
-			if ( filter_var( $search, FILTER_VALIDATE_IP ) ) {
+		if ($search) {
+			if (filter_var($search, FILTER_VALIDATE_IP)) {
 
 				// This is an IP address search.
 				$key = '_give_log_request_ip';
 
-			} elseif ( is_email( $search ) ) {
+			} elseif (is_email($search)) {
 
 				// This is an email search.
-				$userdata = get_user_by( 'email', $search );
+				$userdata = get_user_by('email', $search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
 				$key = '_give_log_user';
 
-			} elseif ( 32 === strlen( $search ) ) {
+			} elseif (32 === strlen($search)) {
 
 				// Look for an API key.
 				$key = '_give_log_key';
 
-			} elseif ( stristr( $search, 'token:' ) ) {
+			} elseif (stristr($search, 'token:')) {
 
 				// Look for an API token.
-				$search = str_ireplace( 'token:', '', $search );
+				$search = str_ireplace('token:', '', $search);
 				$key    = '_give_log_token';
 
 			} else {
 
 				// This is (probably) a user ID search.
-				$userdata = get_userdata( $search );
+				$userdata = get_userdata($search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
@@ -302,7 +302,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -315,7 +315,7 @@ 
 	 *
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -337,14 +337,14 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$logs = Give()->logs->get_connected_logs( $log_query );
+		$logs = Give()->logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'          => $log->ID,
-					'ip'          => Give()->logs->logmeta_db->get_meta( $log->ID, '_give_log_request_ip', true ),
+					'ip'          => Give()->logs->logmeta_db->get_meta($log->ID, '_give_log_request_ip', true),
 					'date'        => $log->log_date,
 					'log_content' => $log->log_content,
 					'log_date'    => $log->log_date,
@@ -372,14 +372,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = Give()->logs->get_log_count( 0, 'api_request' );
+		$total_items           = Give()->logs->get_log_count(0, 'api_request');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/class-give-cache.php

Doc Comments +3 added lines, -3 removed lines

@@ -121,7 +121,7 @@ 
 	 *
 	 * @param  string $cache_key
 	 * @param  bool   $custom_key
-	 * @param  mixed  $query_args
+	 * @param  string  $query_args
 	 *
 	 * @return mixed
 	 */
@@ -237,7 +237,7 @@ 
 	 *
 	 * @param bool  $force If set to true then all cached values will be delete instead of only expired
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public static function delete_all_expired( $force = false ) {
 		global $wpdb;
@@ -669,7 +669,7 @@ 
 	 * @since  2.0
 	 * @access private
 	 *
-	 * @param $group
+	 * @param string $group
 	 *
 	 * @return mixed
 	 */

Spacing +124 added lines, -124 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,7 +53,7 @@ 
 	 * @return static
 	 */
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) && ! ( self::$instance instanceof Give_Cache ) ) {
+		if ( ! isset(self::$instance) && ! (self::$instance instanceof Give_Cache)) {
 			self::$instance = new Give_Cache();
 		}
 
@@ -68,17 +68,17 @@ 
 	 */
 	public function setup() {
 		// Currently enable cache only for backend.
-		self::$instance->is_cache = ( defined( 'GIVE_CACHE' ) ? GIVE_CACHE : give_is_setting_enabled( give_get_option( 'cache', 'enabled' ) ) ) && is_admin();
+		self::$instance->is_cache = (defined('GIVE_CACHE') ? GIVE_CACHE : give_is_setting_enabled(give_get_option('cache', 'enabled'))) && is_admin();
 
 		// weekly delete all expired cache.
-		Give_Cron::add_weekly_event( array( $this, 'delete_all_expired' ) );
+		Give_Cron::add_weekly_event(array($this, 'delete_all_expired'));
 
-		add_action( 'save_post_give_forms', array( $this, 'delete_form_related_cache' ) );
-		add_action( 'save_post_give_payment', array( $this, 'delete_payment_related_cache' ) );
-		add_action( 'give_deleted_give-donors_cache', array( $this, 'delete_donor_related_cache' ), 10, 3 );
-		add_action( 'give_deleted_give-donations_cache', array( $this, 'delete_donations_related_cache' ), 10, 3 );
+		add_action('save_post_give_forms', array($this, 'delete_form_related_cache'));
+		add_action('save_post_give_payment', array($this, 'delete_payment_related_cache'));
+		add_action('give_deleted_give-donors_cache', array($this, 'delete_donor_related_cache'), 10, 3);
+		add_action('give_deleted_give-donations_cache', array($this, 'delete_donations_related_cache'), 10, 3);
 
-		add_action( 'give_save_settings_give_settings', array( $this, 'flush_cache' ) );
+		add_action('give_save_settings_give_settings', array($this, 'flush_cache'));
 	}
 
 	/**
@@ -92,18 +92,18 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_key( $action, $query_args = null, $is_prefix = true ) {
+	public static function get_key($action, $query_args = null, $is_prefix = true) {
 		// Bailout.
-		if ( empty( $action ) ) {
-			return new WP_Error( 'give_invalid_cache_key_action', __( 'Do not pass empty action to generate cache key.', 'give' ) );
+		if (empty($action)) {
+			return new WP_Error('give_invalid_cache_key_action', __('Do not pass empty action to generate cache key.', 'give'));
 		}
 
 		// Set cache key.
 		$cache_key = $is_prefix ? "give_cache_{$action}" : $action;
 
 		// Bailout.
-		if ( ! empty( $query_args ) ) {
-			$cache_key = "{$cache_key}_" . substr( md5( serialize( $query_args ) ), 0, 15 );
+		if ( ! empty($query_args)) {
+			$cache_key = "{$cache_key}_".substr(md5(serialize($query_args)), 0, 15);
 		}
 
 		/**
@@ -111,7 +111,7 @@ 
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_get_cache_key', $cache_key, $action, $query_args );
+		return apply_filters('give_get_cache_key', $cache_key, $action, $query_args);
 	}
 
 	/**
@@ -125,26 +125,26 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get( $cache_key, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if ( ! $custom_key ) {
-				return new WP_Error( 'give_invalid_cache_key', __( 'Cache key format should be give_cache_*', 'give' ) );
+	public static function get($cache_key, $custom_key = false, $query_args = array()) {
+		if ( ! self::is_valid_cache_key($cache_key)) {
+			if ( ! $custom_key) {
+				return new WP_Error('give_invalid_cache_key', __('Cache key format should be give_cache_*', 'give'));
 			}
 
-			$cache_key = self::get_key( $cache_key, $query_args );
+			$cache_key = self::get_key($cache_key, $query_args);
 		}
 
-		$option = get_option( $cache_key );
+		$option = get_option($cache_key);
 
 		// Backward compatibility (<1.8.7).
-		if ( ! is_array( $option ) || empty( $option ) || ! array_key_exists( 'expiration', $option ) ) {
+		if ( ! is_array($option) || empty($option) || ! array_key_exists('expiration', $option)) {
 			return $option;
 		}
 
 		// Get current time.
-		$current_time = current_time( 'timestamp', 1 );
+		$current_time = current_time('timestamp', 1);
 
-		if ( empty( $option['expiration'] ) || ( $current_time < $option['expiration'] ) ) {
+		if (empty($option['expiration']) || ($current_time < $option['expiration'])) {
 			$option = $option['data'];
 		} else {
 			$option = false;
@@ -166,23 +166,23 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function set( $cache_key, $data, $expiration = null, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if ( ! $custom_key ) {
-				return new WP_Error( 'give_invalid_cache_key', __( 'Cache key format should be give_cache_*', 'give' ) );
+	public static function set($cache_key, $data, $expiration = null, $custom_key = false, $query_args = array()) {
+		if ( ! self::is_valid_cache_key($cache_key)) {
+			if ( ! $custom_key) {
+				return new WP_Error('give_invalid_cache_key', __('Cache key format should be give_cache_*', 'give'));
 			}
 
-			$cache_key = self::get_key( $cache_key, $query_args );
+			$cache_key = self::get_key($cache_key, $query_args);
 		}
 
 		$option_value = array(
 			'data'       => $data,
-			'expiration' => ! is_null( $expiration )
-				? ( $expiration + current_time( 'timestamp', 1 ) )
+			'expiration' => ! is_null($expiration)
+				? ($expiration + current_time('timestamp', 1))
 				: null,
 		);
 
-		$result = update_option( $cache_key, $option_value, 'no' );
+		$result = update_option($cache_key, $option_value, 'no');
 
 		return $result;
 	}
@@ -198,27 +198,27 @@ 
 	 *
 	 * @return bool|WP_Error
 	 */
-	public static function delete( $cache_keys ) {
+	public static function delete($cache_keys) {
 		$result       = true;
 		$invalid_keys = array();
 
-		if ( ! empty( $cache_keys ) ) {
-			$cache_keys = is_array( $cache_keys ) ? $cache_keys : array( $cache_keys );
+		if ( ! empty($cache_keys)) {
+			$cache_keys = is_array($cache_keys) ? $cache_keys : array($cache_keys);
 
-			foreach ( $cache_keys as $cache_key ) {
-				if ( ! self::is_valid_cache_key( $cache_key ) ) {
+			foreach ($cache_keys as $cache_key) {
+				if ( ! self::is_valid_cache_key($cache_key)) {
 					$invalid_keys[] = $cache_key;
 					$result         = false;
 				}
 
-				delete_option( $cache_key );
+				delete_option($cache_key);
 			}
 		}
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			$result = new WP_Error(
 				'give_invalid_cache_key',
-				__( 'Cache key format should be give_cache_*', 'give' ),
+				__('Cache key format should be give_cache_*', 'give'),
 				$invalid_keys
 			);
 		}
@@ -239,7 +239,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function delete_all_expired( $force = false ) {
+	public static function delete_all_expired($force = false) {
 		global $wpdb;
 		$options = $wpdb->get_results(
 			$wpdb->prepare(
@@ -253,30 +253,30 @@ 
 		);
 
 		// Bailout.
-		if ( empty( $options ) ) {
+		if (empty($options)) {
 			return false;
 		}
 
-		$current_time = current_time( 'timestamp', 1 );
+		$current_time = current_time('timestamp', 1);
 
 		// Delete log cache.
-		foreach ( $options as $option ) {
-			$option['option_value'] = maybe_unserialize( $option['option_value'] );
+		foreach ($options as $option) {
+			$option['option_value'] = maybe_unserialize($option['option_value']);
 
 			if (
 				(
-					! self::is_valid_cache_key( $option['option_name'] )
-					|| ! is_array( $option['option_value'] ) // Backward compatibility (<1.8.7).
-					|| ! array_key_exists( 'expiration', $option['option_value'] ) // Backward compatibility (<1.8.7).
-					|| empty( $option['option_value']['expiration'] )
-					|| ( $current_time < $option['option_value']['expiration'] )
+					! self::is_valid_cache_key($option['option_name'])
+					|| ! is_array($option['option_value']) // Backward compatibility (<1.8.7).
+					|| ! array_key_exists('expiration', $option['option_value']) // Backward compatibility (<1.8.7).
+					|| empty($option['option_value']['expiration'])
+					|| ($current_time < $option['option_value']['expiration'])
 				)
 				&& ! $force
 			) {
 				continue;
 			}
 
-			self::delete( $option['option_name'] );
+			self::delete($option['option_name']);
 		}
 	}
 
@@ -294,12 +294,12 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_options_like( $option_name, $fields = false ) {
+	public static function get_options_like($option_name, $fields = false) {
 		global $wpdb;
 
 		$field_names = $fields ? 'option_name, option_value' : 'option_name';
 
-		if ( $fields ) {
+		if ($fields) {
 			$options = $wpdb->get_results(
 				$wpdb->prepare(
 					"SELECT {$field_names }
@@ -323,10 +323,10 @@ 
 			);
 		}
 
-		if ( ! empty( $options ) && $fields ) {
-			foreach ( $options as $index => $option ) {
-				$option['option_value'] = maybe_unserialize( $option['option_value'] );
-				$options[ $index ]      = $option;
+		if ( ! empty($options) && $fields) {
+			foreach ($options as $index => $option) {
+				$option['option_value'] = maybe_unserialize($option['option_value']);
+				$options[$index]      = $option;
 			}
 		}
 
@@ -343,8 +343,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_valid_cache_key( $cache_key ) {
-		$is_valid = ( false !== strpos( $cache_key, 'give_cache_' ) );
+	public static function is_valid_cache_key($cache_key) {
+		$is_valid = (false !== strpos($cache_key, 'give_cache_'));
 
 
 		/**
@@ -352,7 +352,7 @@ 
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_is_valid_cache_key', $is_valid, $cache_key );
+		return apply_filters('give_is_valid_cache_key', $is_valid, $cache_key);
 	}
 
 
@@ -367,14 +367,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_group( $id, $group = '' ) {
+	public static function get_group($id, $group = '') {
 		$cached_data = null;
 
 		// Bailout.
-		if ( self::$instance->is_cache && ! empty( $id ) ) {
-			$group = self::$instance->filter_group_name( $group );
+		if (self::$instance->is_cache && ! empty($id)) {
+			$group = self::$instance->filter_group_name($group);
 
-			$cached_data = wp_cache_get( $id, $group );
+			$cached_data = wp_cache_get($id, $group);
 			$cached_data = false !== $cached_data ? $cached_data : null;
 		}
 
@@ -394,17 +394,17 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function set_group( $id, $data, $group = '', $expire = 0 ) {
+	public static function set_group($id, $data, $group = '', $expire = 0) {
 		$status = false;
 
 		// Bailout.
-		if ( ! self::$instance->is_cache || empty( $id ) ) {
+		if ( ! self::$instance->is_cache || empty($id)) {
 			return $status;
 		}
 
-		$group = self::$instance->filter_group_name( $group );
+		$group = self::$instance->filter_group_name($group);
 
-		$status = wp_cache_set( $id, $data, $group, $expire );
+		$status = wp_cache_set($id, $data, $group, $expire);
 
 		return $status;
 	}
@@ -420,15 +420,15 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function set_db_query( $id, $data ) {
+	public static function set_db_query($id, $data) {
 		$status = false;
 
 		// Bailout.
-		if ( ! self::$instance->is_cache || empty( $id ) ) {
+		if ( ! self::$instance->is_cache || empty($id)) {
 			return $status;
 		}
 
-		return self::set_group( $id, $data, 'give-db-queries', 0 );
+		return self::set_group($id, $data, 'give-db-queries', 0);
 	}
 
 	/**
@@ -441,8 +441,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_db_query( $id ) {
-		return self::get_group( $id, 'give-db-queries' );
+	public static function get_db_query($id) {
+		return self::get_group($id, 'give-db-queries');
 	}
 
 	/**
@@ -457,20 +457,20 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function delete_group( $ids, $group = '', $expire = 0 ) {
+	public static function delete_group($ids, $group = '', $expire = 0) {
 		$status = false;
 
 		// Bailout.
-		if ( ! self::$instance->is_cache || empty( $ids ) ) {
+		if ( ! self::$instance->is_cache || empty($ids)) {
 			return $status;
 		}
 
-		$group = self::$instance->filter_group_name( $group );
+		$group = self::$instance->filter_group_name($group);
 
 		// Delete single or multiple cache items from cache.
-		if ( ! is_array( $ids ) ) {
-			$status = wp_cache_delete( $ids, $group, $expire );
-			self::$instance->get_incrementer( true );
+		if ( ! is_array($ids)) {
+			$status = wp_cache_delete($ids, $group, $expire);
+			self::$instance->get_incrementer(true);
 
 			/**
 			 * Fire action when cache deleted for specific id.
@@ -481,12 +481,12 @@ 
 			 * @param string $group
 			 * @param int    $expire
 			 */
-			do_action( "give_deleted_{$group}_cache", $ids, $group, $expire, $status );
+			do_action("give_deleted_{$group}_cache", $ids, $group, $expire, $status);
 
 		} else {
-			foreach ( $ids as $id ) {
-				$status = wp_cache_delete( $id, $group, $expire );
-				self::$instance->get_incrementer( true );
+			foreach ($ids as $id) {
+				$status = wp_cache_delete($id, $group, $expire);
+				self::$instance->get_incrementer(true);
 
 				/**
 				 * Fire action when cache deleted for specific id .
@@ -497,7 +497,7 @@ 
 				 * @param string $group
 				 * @param int    $expire
 				 */
-				do_action( "give_deleted_{$group}_cache", $id, $group, $expire, $status );
+				do_action("give_deleted_{$group}_cache", $id, $group, $expire, $status);
 			}
 		}
 
@@ -514,30 +514,30 @@ 
 	 *
 	 * @param int $form_id
 	 */
-	public function delete_form_related_cache( $form_id ) {
+	public function delete_form_related_cache($form_id) {
 		// If this is just a revision, don't send the email.
-		if ( wp_is_post_revision( $form_id ) ) {
+		if (wp_is_post_revision($form_id)) {
 			return;
 		}
 
 		$donation_query = new Give_Payments_Query(
 			array(
-				'number'     => - 1,
+				'number'     => -1,
 				'give_forms' => $form_id,
 			)
 		);
 
 		$donations = $donation_query->get_payments();
 
-		if ( ! empty( $donations ) ) {
+		if ( ! empty($donations)) {
 			/* @var Give_Payment $donation */
-			foreach ( $donations as $donation ) {
-				wp_cache_delete( $donation->ID, $this->filter_group_name( 'give-donations' ) );
-				wp_cache_delete( $donation->donor_id, $this->filter_group_name( 'give-donors' ) );
+			foreach ($donations as $donation) {
+				wp_cache_delete($donation->ID, $this->filter_group_name('give-donations'));
+				wp_cache_delete($donation->donor_id, $this->filter_group_name('give-donors'));
 			}
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -549,22 +549,22 @@ 
 	 *
 	 * @param int $donation_id
 	 */
-	public function delete_payment_related_cache( $donation_id ) {
+	public function delete_payment_related_cache($donation_id) {
 		// If this is just a revision, don't send the email.
-		if ( wp_is_post_revision( $donation_id ) ) {
+		if (wp_is_post_revision($donation_id)) {
 			return;
 		}
 
 		/* @var Give_Payment $donation */
-		$donation = new Give_Payment( $donation_id );
+		$donation = new Give_Payment($donation_id);
 
-		if ( $donation && $donation->donor_id ) {
-			wp_cache_delete( $donation->donor_id, $this->filter_group_name( 'give-donors' ) );
+		if ($donation && $donation->donor_id) {
+			wp_cache_delete($donation->donor_id, $this->filter_group_name('give-donors'));
 		}
 
-		wp_cache_delete( $donation->ID, $this->filter_group_name( 'give-donations' ) );
+		wp_cache_delete($donation->ID, $this->filter_group_name('give-donations'));
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -578,17 +578,17 @@ 
 	 * @param string $group
 	 * @param int    $expire
 	 */
-	public function delete_donor_related_cache( $id, $group, $expire ) {
-		$donor        = new Give_Donor( $id );
-		$donation_ids = array_map( 'trim', (array) explode( ',', trim( $donor->payment_ids ) ) );
+	public function delete_donor_related_cache($id, $group, $expire) {
+		$donor        = new Give_Donor($id);
+		$donation_ids = array_map('trim', (array) explode(',', trim($donor->payment_ids)));
 
-		if ( ! empty( $donation_ids ) ) {
-			foreach ( $donation_ids as $donation ) {
-				wp_cache_delete( $donation, $this->filter_group_name( 'give-donations' ) );
+		if ( ! empty($donation_ids)) {
+			foreach ($donation_ids as $donation) {
+				wp_cache_delete($donation, $this->filter_group_name('give-donations'));
 			}
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -602,15 +602,15 @@ 
 	 * @param string $group
 	 * @param int    $expire
 	 */
-	public function delete_donations_related_cache( $id, $group, $expire ) {
+	public function delete_donations_related_cache($id, $group, $expire) {
 		/* @var Give_Payment $donation */
-		$donation = new Give_Payment( $id );
+		$donation = new Give_Payment($id);
 
-		if ( $donation && $donation->donor_id ) {
-			wp_cache_delete( $donation->donor_id, $this->filter_group_name( 'give-donors' ) );
+		if ($donation && $donation->donor_id) {
+			wp_cache_delete($donation->donor_id, $this->filter_group_name('give-donors'));
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 
@@ -628,12 +628,12 @@ 
 	 *
 	 * @return string
 	 */
-	private function get_incrementer( $refresh = false, $incrementer_key = 'give-cache-incrementer-db-queries' ) {
-		$incrementer_value = wp_cache_get( $incrementer_key );
+	private function get_incrementer($refresh = false, $incrementer_key = 'give-cache-incrementer-db-queries') {
+		$incrementer_value = wp_cache_get($incrementer_key);
 
-		if ( false === $incrementer_value || true === $refresh ) {
-			$incrementer_value = microtime( true );
-			wp_cache_set( $incrementer_key, $incrementer_value );
+		if (false === $incrementer_value || true === $refresh) {
+			$incrementer_value = microtime(true);
+			wp_cache_set($incrementer_key, $incrementer_value);
 		}
 
 		return $incrementer_value;
@@ -650,11 +650,11 @@ 
 	public function flush_cache() {
 		if (
 			Give_Admin_Settings::is_saving_settings() &&
-			isset( $_POST['cache'] ) &&
-			give_is_setting_enabled( give_clean( $_POST['cache'] ) )
+			isset($_POST['cache']) &&
+			give_is_setting_enabled(give_clean($_POST['cache']))
 		) {
-			$this->get_incrementer( true );
-			$this->get_incrementer( true, 'give-cache-incrementer' );
+			$this->get_incrementer(true);
+			$this->get_incrementer(true, 'give-cache-incrementer');
 		}
 	}
 
@@ -669,11 +669,11 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function filter_group_name( $group ) {
-		if ( ! empty( $group ) ) {
-			$incrementer = self::$instance->get_incrementer( false, 'give-cache-incrementer' );
+	private function filter_group_name($group) {
+		if ( ! empty($group)) {
+			$incrementer = self::$instance->get_incrementer(false, 'give-cache-incrementer');
 
-			if ( 'give-db-queries' === $group ) {
+			if ('give-db-queries' === $group) {
 				$incrementer = self::$instance->get_incrementer();
 			}
 

includes/class-give-db-donors.php

Doc Comments +2 added lines, -2 removed lines

@@ -207,7 +207,7 @@ 
 	 * NOTE: This should not be called directly as it does not make necessary changes to
 	 * the payment meta and logs. Use give_donor_delete() instead.
 	 *
-	 * @param  bool|string|int $_id_or_email ID or Email of Donor.
+	 * @param  integer $_id_or_email ID or Email of Donor.
 	 *
 	 * @since  1.0
 	 * @access public
@@ -407,7 +407,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $field ID or email. Default is 'id'.
-	 * @param  mixed  $value The Customer ID or email to search. Default is 0.
+	 * @param  integer  $value The Customer ID or email to search. Default is 0.
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */

Spacing +107 added lines, -107 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -42,7 +42,7 @@ 
 		$this->bc_200_params();
 
 		// Set hooks and register table only if instance loading first time.
-		if ( ! ( Give()->donors instanceof Give_DB_Donors ) ) {
+		if ( ! (Give()->donors instanceof Give_DB_Donors)) {
 			// Install table.
 			$this->register_table();
 		}
@@ -91,7 +91,7 @@ 
 			'purchase_value'  => 0.00,
 			'purchase_count'  => 0,
 			'notes'           => '',
-			'date_created'    => date( 'Y-m-d H:i:s' ),
+			'date_created'    => date('Y-m-d H:i:s'),
 			'token'           => '',
 			'verify_key'      => '',
 			'verify_throttle' => '',
@@ -108,40 +108,40 @@ 
 	 *
 	 * @return int|bool
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 
 		$defaults = array(
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
-		if ( empty( $args['email'] ) ) {
+		if (empty($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
-		$donor = $this->get_donor_by( 'email', $args['email'] );
+		$donor = $this->get_donor_by('email', $args['email']);
 
 		// update an existing donor.
-		if ( $donor ) {
+		if ($donor) {
 
 			// Update the payment IDs attached to the donor
-			if ( ! empty( $args['payment_ids'] ) ) {
+			if ( ! empty($args['payment_ids'])) {
 
-				if ( empty( $donor->payment_ids ) ) {
+				if (empty($donor->payment_ids)) {
 
 					$donor->payment_ids = $args['payment_ids'];
 
 				} else {
 
-					$existing_ids       = array_map( 'absint', explode( ',', $donor->payment_ids ) );
-					$payment_ids        = array_map( 'absint', explode( ',', $args['payment_ids'] ) );
-					$payment_ids        = array_merge( $payment_ids, $existing_ids );
-					$donor->payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+					$existing_ids       = array_map('absint', explode(',', $donor->payment_ids));
+					$payment_ids        = array_map('absint', explode(',', $args['payment_ids']));
+					$payment_ids        = array_merge($payment_ids, $existing_ids);
+					$donor->payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 				}
 
@@ -149,13 +149,13 @@ 
 
 			}
 
-			$this->update( $donor->id, $args );
+			$this->update($donor->id, $args);
 
 			return $donor->id;
 
 		} else {
 
-			return $this->insert( $args, 'donor' );
+			return $this->insert($args, 'donor');
 
 		}
 
@@ -172,12 +172,12 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 
-		$status = parent::update( $row_id, $data, $where );
+		$status = parent::update($row_id, $data, $where);
 
-		if ( $status ) {
-			Give_Cache::delete_group( $row_id, 'give-donors' );
+		if ($status) {
+			Give_Cache::delete_group($row_id, 'give-donors');
 		}
 
 		return $status;
@@ -191,11 +191,11 @@ 
 	 *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
-		$donor_id = parent::insert( $data, $type );
+	public function insert($data, $type = '') {
+		$donor_id = parent::insert($data, $type);
 
-		if ( $donor_id ) {
-			Give_Cache::delete_group( $donor_id, 'give-donors' );
+		if ($donor_id) {
+			Give_Cache::delete_group($donor_id, 'give-donors');
 		}
 
 		return $donor_id;
@@ -214,16 +214,16 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete( $_id_or_email = false ) {
+	public function delete($_id_or_email = false) {
 
-		if ( empty( $_id_or_email ) ) {
+		if (empty($_id_or_email)) {
 			return false;
 		}
 
-		$column = is_email( $_id_or_email ) ? 'email' : 'id';
-		$donor  = $this->get_donor_by( $column, $_id_or_email );
+		$column = is_email($_id_or_email) ? 'email' : 'id';
+		$donor  = $this->get_donor_by($column, $_id_or_email);
 
-		if ( $donor->id > 0 ) {
+		if ($donor->id > 0) {
 
 			global $wpdb;
 
@@ -232,11 +232,11 @@ 
 			 *
 			 * @since 1.8.14
 			 */
-			Give()->donor_meta->delete_all_meta( $donor->id );
+			Give()->donor_meta->delete_all_meta($donor->id);
 
 			// Cache already deleted in delete_all_meta fn.
 
-			return $wpdb->delete( $this->table_name, array( 'id' => $donor->id ), array( '%d' ) );
+			return $wpdb->delete($this->table_name, array('id' => $donor->id), array('%d'));
 
 		} else {
 			return false;
@@ -257,10 +257,10 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete_by_user_id( $user_id = false ) {
+	public function delete_by_user_id($user_id = false) {
 		global $wpdb;
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return false;
 		}
 
@@ -269,14 +269,14 @@ 
 		 *
 		 * @since 1.8.14
 		 */
-		$donor = new Give_Donor( $user_id, true );
-		if ( ! empty( $donor->id ) ) {
-			Give()->donor_meta->delete_all_meta( $donor->id );
+		$donor = new Give_Donor($user_id, true);
+		if ( ! empty($donor->id)) {
+			Give()->donor_meta->delete_all_meta($donor->id);
 		}
 
 		// Cache is already deleted in delete_all_meta fn.
 
-		return $wpdb->delete( $this->table_name, array( 'user_id' => $user_id ), array( '%d' ) );
+		return $wpdb->delete($this->table_name, array('user_id' => $user_id), array('%d'));
 	}
 
 	/**
@@ -290,14 +290,14 @@ 
 	 *
 	 * @return bool          True is exists, false otherwise.
 	 */
-	public function exists( $value = '', $field = 'email' ) {
+	public function exists($value = '', $field = 'email') {
 
 		$columns = $this->get_columns();
-		if ( ! array_key_exists( $field, $columns ) ) {
+		if ( ! array_key_exists($field, $columns)) {
 			return false;
 		}
 
-		return (bool) $this->get_column_by( 'id', $field, $value );
+		return (bool) $this->get_column_by('id', $field, $value);
 
 	}
 
@@ -312,16 +312,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function attach_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function attach_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		// Attach the payment, but don't increment stats, as this function previously did not
-		return $donor->attach_payment( $payment_id, false );
+		return $donor->attach_payment($payment_id, false);
 
 	}
 
@@ -336,16 +336,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function remove_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		// Remove the payment, but don't decrease stats, as this function previously did not
-		return $donor->remove_payment( $payment_id, false );
+		return $donor->remove_payment($payment_id, false);
 
 	}
 
@@ -359,18 +359,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function increment_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function increment_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		$increased_count = $donor->increase_purchase_count();
-		$increased_value = $donor->increase_value( $amount );
+		$increased_value = $donor->increase_value($amount);
 
-		return ( $increased_count && $increased_value ) ? true : false;
+		return ($increased_count && $increased_value) ? true : false;
 
 	}
 
@@ -385,18 +385,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function decrement_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function decrement_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		$decreased_count = $donor->decrease_donation_count();
-		$decreased_value = $donor->decrease_value( $amount );
+		$decreased_value = $donor->decrease_value($amount);
 
-		return ( $decreased_count && $decreased_value ) ? true : false;
+		return ($decreased_count && $decreased_value) ? true : false;
 
 	}
 
@@ -411,44 +411,44 @@ 
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */
-	public function get_donor_by( $field = 'id', $value = 0 ) {
-		$value = sanitize_text_field( $value );
+	public function get_donor_by($field = 'id', $value = 0) {
+		$value = sanitize_text_field($value);
 
 		// Bailout.
-		if ( empty( $field ) || empty( $value ) ) {
+		if (empty($field) || empty($value)) {
 			return null;
 		}
 
 		// Verify values.
-		if ( 'id' === $field || 'user_id' === $field ) {
+		if ('id' === $field || 'user_id' === $field) {
 			// Make sure the value is numeric to avoid casting objects, for example,
 			// to int 1.
-			if ( ! is_numeric( $value ) ) {
+			if ( ! is_numeric($value)) {
 				return false;
 			}
 
-			$value = absint( $value );
+			$value = absint($value);
 
-			if ( $value < 1 ) {
+			if ($value < 1) {
 				return false;
 			}
 
-		} elseif ( 'email' === $field ) {
+		} elseif ('email' === $field) {
 
-			if ( ! is_email( $value ) ) {
+			if ( ! is_email($value)) {
 				return false;
 			}
 
-			$value = trim( $value );
+			$value = trim($value);
 		}
 
 		// Bailout
-		if ( ! $value ) {
+		if ( ! $value) {
 			return false;
 		}
 
 		// Set query params.
-		switch ( $field ) {
+		switch ($field) {
 			case 'id':
 				$args['donor'] = $value;
 				break;
@@ -463,9 +463,9 @@ 
 		}
 
 		// Get donors.
-		$donor = new Give_Donors_Query( $args );
+		$donor = new Give_Donors_Query($args);
 
-		if ( ! $donor = $donor->get_donors() ) {
+		if ( ! $donor = $donor->get_donors()) {
 			// Look for donor from an additional email.
 			$args = array(
 				'meta_query' => array(
@@ -476,15 +476,15 @@ 
 				),
 			);
 
-			$donor = new Give_Donors_Query( $args );
+			$donor = new Give_Donors_Query($args);
 			$donor = $donor->get_donors();
 
-			if ( empty( $donor ) ) {
+			if (empty($donor)) {
 				return false;
 			}
 		}
 
-		return current( $donor );
+		return current($donor);
 	}
 
 	/**
@@ -497,10 +497,10 @@ 
 	 *
 	 * @return array|object|null Donors array or object. Null if not found.
 	 */
-	public function get_donors( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function get_donors($args = array()) {
+		$this->bc_1814_params($args);
 
-		$donors = new Give_Donors_Query( $args );
+		$donors = new Give_Donors_Query($args);
 
 		return $donors->get_donors();
 
@@ -517,21 +517,21 @@ 
 	 *
 	 * @return int         Total number of donors.
 	 */
-	public function count( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function count($args = array()) {
+		$this->bc_1814_params($args);
 		$args['count'] = true;
 
-		$cache_key = md5( 'give_donors_count' . serialize( $args ) );
-		$count     = Give_Cache::get_group( $cache_key, 'donors' );
+		$cache_key = md5('give_donors_count'.serialize($args));
+		$count     = Give_Cache::get_group($cache_key, 'donors');
 
-		if ( is_null( $count ) ) {
-			$donors = new Give_Donors_Query( $args );
+		if (is_null($count)) {
+			$donors = new Give_Donors_Query($args);
 			$count  = $donors->get_donors();
 
-			Give_Cache::set_group( $cache_key, $count, 'donors', 3600 );
+			Give_Cache::set_group($cache_key, $count, 'donors', 3600);
 		}
 
-		return absint( $count );
+		return absint($count);
 
 	}
 
@@ -545,9 +545,9 @@ 
 	 */
 	public function create_table() {
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
-		$sql = "CREATE TABLE " . $this->table_name . " (
+		$sql = "CREATE TABLE ".$this->table_name." (
 		id bigint(20) NOT NULL AUTO_INCREMENT,
 		user_id bigint(20) NOT NULL,
 		email varchar(50) NOT NULL,
@@ -565,9 +565,9 @@ 
 		KEY user (user_id)
 		) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 	/**
@@ -582,8 +582,8 @@ 
 		global $wpdb;
 
 		if (
-			! give_has_upgrade_completed( 'v20_rename_donor_tables' ) &&
-			$wpdb->query( $wpdb->prepare( "SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customers" ) )
+			! give_has_upgrade_completed('v20_rename_donor_tables') &&
+			$wpdb->query($wpdb->prepare("SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customers"))
 		) {
 			$wpdb->donors = $this->table_name = "{$wpdb->prefix}give_customers";
 		}
@@ -597,41 +597,41 @@ 
 	 *
 	 * @param $args
 	 */
-	private function bc_1814_params( &$args ) {
+	private function bc_1814_params(&$args) {
 		// Backward compatibility: user_id
-		if ( ! empty( $args['user_id'] ) ) {
+		if ( ! empty($args['user_id'])) {
 			$args['user'] = $args['user_id'];
 		}
 
 		// Backward compatibility: id
-		if ( ! empty( $args['id'] ) ) {
+		if ( ! empty($args['id'])) {
 			$args['donor'] = $args['id'];
 		}
 
 		// Backward compatibility: name
-		if ( ! empty( $args['name'] ) ) {
+		if ( ! empty($args['name'])) {
 			$args['s'] = "name:{$args['name']}";
 		}
 
 		// Backward compatibility: date
 		// Donors created for a specific date or in a date range.
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				if ( ! empty( $args['date']['start'] ) ) {
-					$args['date_query']['after'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
+				if ( ! empty($args['date']['start'])) {
+					$args['date_query']['after'] = date('Y-m-d H:i:s', strtotime($args['date']['start']));
 				}
 
-				if ( ! empty( $args['date']['end'] ) ) {
-					$args['date_query']['before'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+				if ( ! empty($args['date']['end'])) {
+					$args['date_query']['before'] = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 				}
 
 			} else {
 
-				$args['date_query']['year']  = date( 'Y', strtotime( $args['date'] ) );
-				$args['date_query']['month'] = date( 'm', strtotime( $args['date'] ) );
-				$args['date_query']['day']   = date( 'd', strtotime( $args['date'] ) );
+				$args['date_query']['year']  = date('Y', strtotime($args['date']));
+				$args['date_query']['month'] = date('m', strtotime($args['date']));
+				$args['date_query']['day']   = date('d', strtotime($args['date']));
 			}
 		}
 	}

includes/class-give-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);