WordImpress / Give

includes/gateways/offline-donations.php

Spacing +115 added lines, -115 removed lines

@@ -18,17 +18,17 @@ 
  *
  * @return array
  */
-function give_offline_register_gateway( $gateways ) {
+function give_offline_register_gateway($gateways) {
 	// Format: ID => Name
 	$gateways['offline'] = array(
 		'admin_label'    => 'Offline Donation',
-		'checkout_label' => __( 'Offline Donation', 'give' )
+		'checkout_label' => __('Offline Donation', 'give')
 	);
 
 	return $gateways;
 }
 
-add_filter( 'give_payment_gateways', 'give_offline_register_gateway', 1 );
+add_filter('give_payment_gateways', 'give_offline_register_gateway', 1);
 
 
 /**
@@ -39,10 +39,10 @@ 
  * @return void
  */
 function give_offline_disable_abandoned_orders() {
-	remove_action( 'give_weekly_scheduled_events', 'give_mark_abandoned_orders' );
+	remove_action('give_weekly_scheduled_events', 'give_mark_abandoned_orders');
 }
 
-add_action( 'plugins_loaded', 'give_offline_disable_abandoned_orders' );
+add_action('plugins_loaded', 'give_offline_disable_abandoned_orders');
 
 
 /**
@@ -54,48 +54,48 @@ 
  *
  * @return void
  */
-function give_offline_payment_cc_form( $form_id ) {
+function give_offline_payment_cc_form($form_id) {
 
-	$post_offline_customization_option = get_post_meta( $form_id, '_give_customize_offline_donations', true );
-	$post_offline_instructions         = get_post_meta( $form_id, '_give_offline_checkout_notes', true );
-	$global_offline_instruction        = give_get_option( 'global_offline_donation_content' );
+	$post_offline_customization_option = get_post_meta($form_id, '_give_customize_offline_donations', true);
+	$post_offline_instructions         = get_post_meta($form_id, '_give_offline_checkout_notes', true);
+	$global_offline_instruction        = give_get_option('global_offline_donation_content');
 	$offline_instructions              = $global_offline_instruction;
 
-	if ( $post_offline_customization_option == 'yes' ) {
+	if ($post_offline_customization_option == 'yes') {
 		$offline_instructions = $post_offline_instructions;
 	}
 
 
 	ob_start(); ?>
-	<?php do_action( 'give_before_offline_info_fields', $form_id ); ?>
+	<?php do_action('give_before_offline_info_fields', $form_id); ?>
 	<fieldset id="give_offline_payment_info">
 		<?php
-		$settings_url         = admin_url( 'post.php?post=' . $form_id . '&action=edit&message=1' );
-		$offline_instructions = ! empty( $offline_instructions ) ? $offline_instructions : sprintf( __( 'Please enter offline donation instructions in the %s.', 'give' ), '<a href="' . $settings_url . '">' . __( 'this form\'s settings', 'give' ) . '</a>' );
-		echo wpautop( stripslashes( $offline_instructions ) );
+		$settings_url         = admin_url('post.php?post='.$form_id.'&action=edit&message=1');
+		$offline_instructions = ! empty($offline_instructions) ? $offline_instructions : sprintf(__('Please enter offline donation instructions in the %s.', 'give'), '<a href="'.$settings_url.'">'.__('this form\'s settings', 'give').'</a>');
+		echo wpautop(stripslashes($offline_instructions));
 		?>
 	</fieldset>
-	<?php do_action( 'give_after_offline_info_fields', $form_id ); ?>
+	<?php do_action('give_after_offline_info_fields', $form_id); ?>
 	<?php
 	echo ob_get_clean();
 }
 
-add_action( 'give_offline_cc_form', 'give_offline_payment_cc_form' );
+add_action('give_offline_cc_form', 'give_offline_payment_cc_form');
 
 
 
-function give_offline_billing_fields($form_id){
+function give_offline_billing_fields($form_id) {
 	//Enable Default CC fields (billing info)
-		$post_offline_cc_fields   = get_post_meta( $form_id, '_give_offline_donation_enable_billing_fields_single', true );
-		$global_offline_cc_fields = give_get_option( 'give_offline_donation_enable_billing_fields' );
+		$post_offline_cc_fields   = get_post_meta($form_id, '_give_offline_donation_enable_billing_fields_single', true);
+		$global_offline_cc_fields = give_get_option('give_offline_donation_enable_billing_fields');
 
 
-	if ( $global_offline_cc_fields == 'on' ||  $post_offline_cc_fields == 'on' ) {
-		give_default_cc_address_fields( $form_id );
+	if ($global_offline_cc_fields == 'on' || $post_offline_cc_fields == 'on') {
+		give_default_cc_address_fields($form_id);
 	}
 }
 
-add_action('give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1 );
+add_action('give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1);
 
 /**
  * Process the payment
@@ -103,15 +103,15 @@ 
  * @since  1.0
  * @return void
  */
-function give_offline_process_payment( $purchase_data ) {
+function give_offline_process_payment($purchase_data) {
 
-	$purchase_summary = give_get_purchase_summary( $purchase_data );
+	$purchase_summary = give_get_purchase_summary($purchase_data);
 
 	// setup the payment details
 	$payment_data = array(
 		'price'           => $purchase_data['price'],
 		'give_form_title' => $purchase_data['post_data']['give-form-title'],
-		'give_form_id'    => intval( $purchase_data['post_data']['give-form-id'] ),
+		'give_form_id'    => intval($purchase_data['post_data']['give-form-id']),
 		'date'            => $purchase_data['date'],
 		'user_email'      => $purchase_data['user_email'],
 		'purchase_key'    => $purchase_data['purchase_key'],
@@ -123,20 +123,20 @@ 
 
 
 	// record the pending payment
-	$payment = give_insert_payment( $payment_data );
+	$payment = give_insert_payment($payment_data);
 
-	if ( $payment ) {
-		give_offline_send_admin_notice( $payment );
-		give_offline_send_donor_instructions( $payment );
+	if ($payment) {
+		give_offline_send_admin_notice($payment);
+		give_offline_send_donor_instructions($payment);
 		give_send_to_success_page();
 	} else {
 		// if errors are present, send the user back to the donation form so they can be corrected
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
 	}
 
 }
 
-add_action( 'give_gateway_offline', 'give_offline_process_payment' );
+add_action('give_gateway_offline', 'give_offline_process_payment');
 
 
 /**
@@ -149,47 +149,47 @@ 
  * @since       1.0
  * @return void
  */
-function give_offline_send_donor_instructions( $payment_id = 0 ) {
+function give_offline_send_donor_instructions($payment_id = 0) {
 
-	$payment_data                      = give_get_payment_meta( $payment_id );
-	$post_offline_customization_option = get_post_meta( $payment_data['form_id'], '_give_customize_offline_donations', true );
+	$payment_data                      = give_get_payment_meta($payment_id);
+	$post_offline_customization_option = get_post_meta($payment_data['form_id'], '_give_customize_offline_donations', true);
 
 	//Customize email content depending on whether the single form has been customized
-	$email_content = give_get_option( 'global_offline_donation_email' );
+	$email_content = give_get_option('global_offline_donation_email');
 
-	if ( $post_offline_customization_option === 'yes' ) {
-		$email_content = get_post_meta( $payment_data['form_id'], '_give_offline_donation_email', true );
+	if ($post_offline_customization_option === 'yes') {
+		$email_content = get_post_meta($payment_data['form_id'], '_give_offline_donation_email', true);
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
-	$from_name = apply_filters( 'give_purchase_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
+	$from_name = apply_filters('give_purchase_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
-	$from_email = apply_filters( 'give_purchase_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
+	$from_email = apply_filters('give_purchase_from_address', $from_email, $payment_id, $payment_data);
 
-	$to_email = give_get_payment_user_email( $payment_id );
+	$to_email = give_get_payment_user_email($payment_id);
 
-	$subject = give_get_option( 'offline_donation_subject', __( 'Offline Donation Instructions', 'give' ) );
-	if ( $post_offline_customization_option === 'yes' ) {
-		$subject = get_post_meta( $payment_data['form_id'], '_give_offline_donation_subject', true );
+	$subject = give_get_option('offline_donation_subject', __('Offline Donation Instructions', 'give'));
+	if ($post_offline_customization_option === 'yes') {
+		$subject = get_post_meta($payment_data['form_id'], '_give_offline_donation_subject', true);
 	}
 
-	$subject = apply_filters( 'give_offline_donation_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = apply_filters('give_offline_donation_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
-	$attachments = apply_filters( 'give_offline_donation_attachments', array(), $payment_id, $payment_data );
-	$message     = give_do_email_tags( $email_content, $payment_id );
+	$attachments = apply_filters('give_offline_donation_attachments', array(), $payment_id, $payment_data);
+	$message     = give_do_email_tags($email_content, $payment_id);
 
 	$emails = Give()->emails;
 
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Offline Donation Instructions', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Offline Donation Instructions', 'give'));
 
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data);
+	$emails->__set('headers', $headers);
 
-	$emails->send( $to_email, $subject, $message, $attachments );
+	$emails->send($to_email, $subject, $message, $attachments);
 
 }
 
@@ -206,37 +206,37 @@ 
  * @return void
  *
  */
-function give_offline_send_admin_notice( $payment_id = 0 ) {
+function give_offline_send_admin_notice($payment_id = 0) {
 
 	/* Send an email notification to the admin */
 	$admin_email = give_get_admin_notice_emails();
-	$user_info   = give_get_payment_meta_user_info( $payment_id );
+	$user_info   = give_get_payment_meta_user_info($payment_id);
 
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 ) {
-		$user_data = get_userdata( $user_info['id'] );
+	if (isset($user_info['id']) && $user_info['id'] > 0) {
+		$user_data = get_userdata($user_info['id']);
 		$name      = $user_data->display_name;
-	} elseif ( isset( $user_info['first_name'] ) && isset( $user_info['last_name'] ) ) {
-		$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
+	} elseif (isset($user_info['first_name']) && isset($user_info['last_name'])) {
+		$name = $user_info['first_name'].' '.$user_info['last_name'];
 	} else {
 		$name = $user_info['email'];
 	}
 
-	$amount = give_currency_filter( give_format_amount( give_get_payment_amount( $payment_id ) ) );
+	$amount = give_currency_filter(give_format_amount(give_get_payment_amount($payment_id)));
 
-	$admin_subject = apply_filters( 'give_offline_admin_donation_notification_subject', __( 'New Pending Donation', 'give' ), $payment_id );
+	$admin_subject = apply_filters('give_offline_admin_donation_notification_subject', __('New Pending Donation', 'give'), $payment_id);
 
-	$admin_message = __( 'Dear Admin,', 'give' ) . "\n\n" . __( 'An offline donation has been made', 'give' ) . ".\n\n";
+	$admin_message = __('Dear Admin,', 'give')."\n\n".__('An offline donation has been made', 'give').".\n\n";
 
-	$order_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id=' . $payment_id );
-	$admin_message .= __( 'Donor: ', 'give' ) . " " . html_entity_decode( $name, ENT_COMPAT, 'UTF-8' ) . "\n";
-	$admin_message .= __( 'Amount: ', 'give' ) . " " . html_entity_decode( $amount, ENT_COMPAT, 'UTF-8' ) . "\n\n";
-	$admin_message .= __( 'This is a pending donation awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give' ) . "\n\n";
-	$admin_message .= sprintf( __( 'View Donation Details: %s.', 'give' ), $order_url ) . "\n\n";
-	$admin_message = apply_filters( 'give_offline_admin_donation_notification', $admin_message, $payment_id );
-	$admin_headers = apply_filters( 'give_offline_admin_donation_notification_headers', array(), $payment_id );
-	$attachments   = apply_filters( 'give_offline_admin_donation_notification_attachments', array(), $payment_id );
+	$order_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id='.$payment_id);
+	$admin_message .= __('Donor: ', 'give')." ".html_entity_decode($name, ENT_COMPAT, 'UTF-8')."\n";
+	$admin_message .= __('Amount: ', 'give')." ".html_entity_decode($amount, ENT_COMPAT, 'UTF-8')."\n\n";
+	$admin_message .= __('This is a pending donation awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give')."\n\n";
+	$admin_message .= sprintf(__('View Donation Details: %s.', 'give'), $order_url)."\n\n";
+	$admin_message = apply_filters('give_offline_admin_donation_notification', $admin_message, $payment_id);
+	$admin_headers = apply_filters('give_offline_admin_donation_notification_headers', array(), $payment_id);
+	$attachments   = apply_filters('give_offline_admin_donation_notification_attachments', array(), $payment_id);
 
-	wp_mail( $admin_email, $admin_subject, $admin_message, $admin_headers, $attachments );
+	wp_mail($admin_email, $admin_subject, $admin_message, $admin_headers, $attachments);
 
 }
 
@@ -247,15 +247,15 @@ 
  * @since  1.0
  * @return array
  */
-function give_offline_add_settings( $settings ) {
+function give_offline_add_settings($settings) {
 
 	//Vars
 	$prefix = '_give_';
 
-	$is_gateway_active = give_is_gateway_active( 'offline' );
+	$is_gateway_active = give_is_gateway_active('offline');
 
 	//this gateway isn't active
-	if ( ! $is_gateway_active ) {
+	if ( ! $is_gateway_active) {
 		//return settings and bounce
 		return $settings;
 	}
@@ -264,27 +264,27 @@ 
 	$check_settings = array(
 
 		array(
-			'name'    => __( 'Customize Offline Donations', 'give' ),
-			'desc'    => __( 'If you would like to customize the donation instructions for this specific forms check this option.', 'give' ),
-			'id'      => $prefix . 'customize_offline_donations',
+			'name'    => __('Customize Offline Donations', 'give'),
+			'desc'    => __('If you would like to customize the donation instructions for this specific forms check this option.', 'give'),
+			'id'      => $prefix.'customize_offline_donations',
 			'type'    => 'radio_inline',
 			'default' => 'no',
 			'options' => array(
-				'yes' => __( 'Yes', 'give' ),
-				'no'  => __( 'No', 'give' ),
+				'yes' => __('Yes', 'give'),
+				'no'  => __('No', 'give'),
 			),
 		),
 		array(
-			'name'        => __( 'Request Billing Information', 'give' ),
-			'desc'        => __( 'This option will enable the billing details section for this form\'s offline donation payment gateway. The fieldset will appear above the offline donation instructions.', 'give' ),
-			'id'          => $prefix . 'offline_donation_enable_billing_fields_single',
+			'name'        => __('Request Billing Information', 'give'),
+			'desc'        => __('This option will enable the billing details section for this form\'s offline donation payment gateway. The fieldset will appear above the offline donation instructions.', 'give'),
+			'id'          => $prefix.'offline_donation_enable_billing_fields_single',
 			'row_classes' => 'give-subfield',
 			'type'        => 'checkbox'
 		),
 		array(
-			'id'          => $prefix . 'offline_checkout_notes',
-			'name'        => __( 'Offline Donation Instructions', 'give' ),
-			'desc'        => __( 'Enter the instructions you want to display to the donor during the donation process. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ),
+			'id'          => $prefix.'offline_checkout_notes',
+			'name'        => __('Offline Donation Instructions', 'give'),
+			'desc'        => __('Enter the instructions you want to display to the donor during the donation process. Most likely this would include important information like mailing address and who to make the check out to.', 'give'),
 			'default'     => give_get_default_offline_donation_content(),
 			'type'        => 'wysiwyg',
 			'row_classes' => 'give-subfield',
@@ -293,17 +293,17 @@ 
 			)
 		),
 		array(
-			'id'          => $prefix . 'offline_donation_subject',
-			'name'        => __( 'Offline Donation Email Instructions Subject', 'give' ),
-			'desc'        => __( 'Enter the subject line for the donation receipt email.', 'give' ),
-			'default'     => __( '{donation} - Offline Donation Instructions', 'give' ),
+			'id'          => $prefix.'offline_donation_subject',
+			'name'        => __('Offline Donation Email Instructions Subject', 'give'),
+			'desc'        => __('Enter the subject line for the donation receipt email.', 'give'),
+			'default'     => __('{donation} - Offline Donation Instructions', 'give'),
 			'row_classes' => 'give-subfield',
 			'type'        => 'text'
 		),
 		array(
-			'id'          => $prefix . 'offline_donation_email',
-			'name'        => __( 'Offline Donation Email Instructions', 'give' ),
-			'desc'        => __( 'Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ),
+			'id'          => $prefix.'offline_donation_email',
+			'name'        => __('Offline Donation Email Instructions', 'give'),
+			'desc'        => __('Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give'),
 			'default'     => give_get_default_offline_donation_email_content(),
 			'type'        => 'wysiwyg',
 			'row_classes' => 'give-subfield',
@@ -313,10 +313,10 @@ 
 		)
 	);
 
-	return array_merge( $settings, $check_settings );
+	return array_merge($settings, $check_settings);
 }
 
-add_filter( 'give_forms_display_options_metabox_fields', 'give_offline_add_settings' );
+add_filter('give_forms_display_options_metabox_fields', 'give_offline_add_settings');
 
 
 /**
@@ -328,20 +328,20 @@ 
  */
 function give_get_default_offline_donation_content() {
 
-	$sitename = get_bloginfo( 'sitename' );
+	$sitename = get_bloginfo('sitename');
 
-	$default_text = '<p>' . __( 'In order to make an offline donation we ask that you please follow these instructions', 'give' ) . ': </p>';
+	$default_text = '<p>'.__('In order to make an offline donation we ask that you please follow these instructions', 'give').': </p>';
 	$default_text .= '<ol>';
-	$default_text .= '<li>' . _x( 'Make a check payable to ', 'Step 1 for donating offline by check', 'give' ) . '"' . $sitename . '"' . '</li>';
-	$default_text .= '<li>' . _x( 'On the memo line of the check, please indicate that the donation is for ', 'Step 2 for donating by check; this explains who the check should be written to', 'give' ) . '"' . $sitename . '"' . '</li>';
-	$default_text .= '<li>' . _x( 'Please mail your check to:', 'Step 3; where to mail the check', 'give' ) . '</li>';
+	$default_text .= '<li>'._x('Make a check payable to ', 'Step 1 for donating offline by check', 'give').'"'.$sitename.'"'.'</li>';
+	$default_text .= '<li>'._x('On the memo line of the check, please indicate that the donation is for ', 'Step 2 for donating by check; this explains who the check should be written to', 'give').'"'.$sitename.'"'.'</li>';
+	$default_text .= '<li>'._x('Please mail your check to:', 'Step 3; where to mail the check', 'give').'</li>';
 	$default_text .= '</ol>';
-	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>' . $sitename . '</em><br>';
+	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>'.$sitename.'</em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>123 G Street </em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>San Diego, CA 92101 </em><br>';
-	$default_text .= '<p>' . __( 'All contributions will be gratefully acknowledged and are tax deductible.', 'give' ) . '</p>';
+	$default_text .= '<p>'.__('All contributions will be gratefully acknowledged and are tax deductible.', 'give').'</p>';
 
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
+	return apply_filters('give_default_offline_donation_content', $default_text);
 
 }
 
@@ -354,21 +354,21 @@ 
  */
 function give_get_default_offline_donation_email_content() {
 
-	$sitename     = get_bloginfo( 'sitename' );
-	$default_text = '<p>' . __( 'Dear {name},', 'give' ) . '</p>';
-	$default_text .= '<p>' . __( 'Thank you for your offline donation request! Your generosity is greatly appreciated. In order to make an offline donation we ask that you please follow these instructions', 'give' ) . ': </p>';
+	$sitename     = get_bloginfo('sitename');
+	$default_text = '<p>'.__('Dear {name},', 'give').'</p>';
+	$default_text .= '<p>'.__('Thank you for your offline donation request! Your generosity is greatly appreciated. In order to make an offline donation we ask that you please follow these instructions', 'give').': </p>';
 	$default_text .= '<ol>';
-	$default_text .= '<li>' . _x( 'Make a check payable to ', 'Step 1 for donating offline by check', 'give' ) . '"' . $sitename . '"' . '</li>';
-	$default_text .= '<li>' . _x( 'On the memo line of the check, please indicate that the donation is for ', 'Step 2 for donating by check; this explains who the check should be written to', 'give' ) . '"' . $sitename . '"' . '</li>';
-	$default_text .= '<li>' . _x( 'Please mail your check to:', 'Step 3; where to mail the check', 'give' ) . '</li>';
+	$default_text .= '<li>'._x('Make a check payable to ', 'Step 1 for donating offline by check', 'give').'"'.$sitename.'"'.'</li>';
+	$default_text .= '<li>'._x('On the memo line of the check, please indicate that the donation is for ', 'Step 2 for donating by check; this explains who the check should be written to', 'give').'"'.$sitename.'"'.'</li>';
+	$default_text .= '<li>'._x('Please mail your check to:', 'Step 3; where to mail the check', 'give').'</li>';
 	$default_text .= '</ol>';
-	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>' . $sitename . '</em><br>';
+	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>'.$sitename.'</em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>123 G Street </em><br>';
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>San Diego, CA 92101 </em><br>';
-	$default_text .= '<p>' . __( 'Once your donation has been received we will mark it as complete and you will receive an email receipt for your records. Please contact us with any questions you may have!', 'give' ) . '</p>';
-	$default_text .= '<p>' . __( 'Sincerely,', 'give' ) . '</p>';
-	$default_text .= '<p>' . $sitename . '</p>';
+	$default_text .= '<p>'.__('Once your donation has been received we will mark it as complete and you will receive an email receipt for your records. Please contact us with any questions you may have!', 'give').'</p>';
+	$default_text .= '<p>'.__('Sincerely,', 'give').'</p>';
+	$default_text .= '<p>'.$sitename.'</p>';
 
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
+	return apply_filters('give_default_offline_donation_content', $default_text);
 
 }

includes/gateways/paypal-standard.php

Spacing +157 added lines, -157 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,7 +21,7 @@ 
  * @access private
  * @since  1.0
  */
-add_action( 'give_paypal_cc_form', '__return_false' );
+add_action('give_paypal_cc_form', '__return_false');
 
 /**
  * Process PayPal Purchase
@@ -32,13 +32,13 @@ 
  *
  * @return void
  */
-function give_process_paypal_purchase( $purchase_data ) {
+function give_process_paypal_purchase($purchase_data) {
 
-	if ( ! wp_verify_nonce( $purchase_data['gateway_nonce'], 'give-gateway' ) ) {
-		wp_die( __( 'Nonce verification has failed', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($purchase_data['gateway_nonce'], 'give-gateway')) {
+		wp_die(__('Nonce verification has failed', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	$form_id = intval( $purchase_data['post_data']['give-form-id'] );
+	$form_id = intval($purchase_data['post_data']['give-form-id']);
 
 	// Collect payment data
 	$payment_data = array(
@@ -55,59 +55,59 @@ 
 	);
 
 	// Record the pending payment
-	$payment = give_insert_payment( $payment_data );
+	$payment = give_insert_payment($payment_data);
 
 	// Check payment
-	if ( ! $payment ) {
+	if ( ! $payment) {
 		// Record the error
-		give_record_gateway_error( __( 'Payment Error', 'give' ), sprintf( __( 'Payment creation failed before sending buyer to PayPal. Payment data: %s', 'give' ), json_encode( $payment_data ) ), $payment );
+		give_record_gateway_error(__('Payment Error', 'give'), sprintf(__('Payment creation failed before sending buyer to PayPal. Payment data: %s', 'give'), json_encode($payment_data)), $payment);
 		// Problems? send back
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
 	} else {
 		// Only send to PayPal if the pending payment is created successfully
-		$listener_url = add_query_arg( 'give-listener', 'IPN', home_url( 'index.php' ) );
+		$listener_url = add_query_arg('give-listener', 'IPN', home_url('index.php'));
 
 		// Get the success url
-		$return_url = add_query_arg( array(
+		$return_url = add_query_arg(array(
 			'payment-confirmation' => 'paypal',
 			'payment-id'           => $payment
 
-		), get_permalink( give_get_option( 'success_page' ) ) );
+		), get_permalink(give_get_option('success_page')));
 
 		// Get the PayPal redirect uri
-		$paypal_redirect = trailingslashit( give_get_paypal_redirect() ) . '?';
+		$paypal_redirect = trailingslashit(give_get_paypal_redirect()).'?';
 
 		//Item name - pass level name if variable priced
 		$item_name = $purchase_data['post_data']['give-form-title'];
 
 		//Verify has variable prices
-		if ( give_has_variable_prices( $form_id ) && isset( $purchase_data['post_data']['give-price-id'] ) ) {
+		if (give_has_variable_prices($form_id) && isset($purchase_data['post_data']['give-price-id'])) {
 
-			$item_price_level_text = give_get_price_option_name( $form_id, $purchase_data['post_data']['give-price-id'] );
+			$item_price_level_text = give_get_price_option_name($form_id, $purchase_data['post_data']['give-price-id']);
 
-			$price_level_amount = give_get_price_option_amount( $form_id, $purchase_data['post_data']['give-price-id'] );
+			$price_level_amount = give_get_price_option_amount($form_id, $purchase_data['post_data']['give-price-id']);
 
 			//Donation given doesn't match selected level (must be a custom amount)
-			if ( $price_level_amount != give_sanitize_amount( $purchase_data['price'] ) ) {
-				$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
+			if ($price_level_amount != give_sanitize_amount($purchase_data['price'])) {
+				$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
 				//user custom amount text if any, fallback to default if not
-				$item_name .= ' - ' . (! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' ));
+				$item_name .= ' - '.( ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give'));
 
 			} //Is there any donation level text?
-			elseif ( ! empty( $item_price_level_text ) ) {
-				$item_name .= ' - ' . $item_price_level_text;
+			elseif ( ! empty($item_price_level_text)) {
+				$item_name .= ' - '.$item_price_level_text;
 			}
 
 		} //Single donation: Custom Amount
-		elseif ( give_get_form_price( $form_id ) !== give_sanitize_amount( $purchase_data['price'] ) ) {
-			$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
+		elseif (give_get_form_price($form_id) !== give_sanitize_amount($purchase_data['price'])) {
+			$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
 			//user custom amount text if any, fallback to default if not
-			$item_name .= ' - ' . (! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' ));
+			$item_name .= ' - '.( ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give'));
 		}
 
 		// Setup PayPal arguments
 		$paypal_args = array(
-			'business'      => give_get_option( 'paypal_email', false ),
+			'business'      => give_get_option('paypal_email', false),
 			'email'         => $purchase_data['user_email'],
 			'invoice'       => $purchase_data['purchase_key'],
 			'amount'        => $purchase_data['price'],
@@ -118,25 +118,25 @@ 
 			'shipping'      => '0',
 			'no_note'       => '1',
 			'currency_code' => give_get_currency(),
-			'charset'       => get_bloginfo( 'charset' ),
+			'charset'       => get_bloginfo('charset'),
 			'custom'        => $payment,
 			'rm'            => '2',
 			'return'        => $return_url,
-			'cancel_return' => give_get_failed_transaction_uri( '?payment-id=' . $payment ),
+			'cancel_return' => give_get_failed_transaction_uri('?payment-id='.$payment),
 			'notify_url'    => $listener_url,
 			'page_style'    => give_get_paypal_page_style(),
-			'cbt'           => get_bloginfo( 'name' ),
+			'cbt'           => get_bloginfo('name'),
 			'bn'            => 'givewp_SP'
 		);
 
-		if ( ! empty( $purchase_data['user_info']['address'] ) ) {
+		if ( ! empty($purchase_data['user_info']['address'])) {
 			$paypal_args['address1'] = $purchase_data['user_info']['address']['line1'];
 			$paypal_args['address2'] = $purchase_data['user_info']['address']['line2'];
 			$paypal_args['city']     = $purchase_data['user_info']['address']['city'];
 			$paypal_args['country']  = $purchase_data['user_info']['address']['country'];
 		}
 
-		if ( give_get_option( 'paypal_button_type' ) === 'standard' ) {
+		if (give_get_option('paypal_button_type') === 'standard') {
 			$paypal_extra_args = array(
 				'cmd' => '_xclick',
 			);
@@ -146,25 +146,25 @@ 
 			);
 		}
 
-		$paypal_args = array_merge( $paypal_extra_args, $paypal_args );
+		$paypal_args = array_merge($paypal_extra_args, $paypal_args);
 
 
-		$paypal_args = apply_filters( 'give_paypal_redirect_args', $paypal_args, $purchase_data );
+		$paypal_args = apply_filters('give_paypal_redirect_args', $paypal_args, $purchase_data);
 
 		// Build query
-		$paypal_redirect .= http_build_query( $paypal_args );
+		$paypal_redirect .= http_build_query($paypal_args);
 
 		// Fix for some sites that encode the entities
-		$paypal_redirect = str_replace( '&', '&', $paypal_redirect );
+		$paypal_redirect = str_replace('&', '&', $paypal_redirect);
 
 		// Redirect to PayPal
-		wp_redirect( $paypal_redirect );
+		wp_redirect($paypal_redirect);
 		exit;
 	}
 
 }
 
-add_action( 'give_gateway_paypal', 'give_process_paypal_purchase' );
+add_action('give_gateway_paypal', 'give_process_paypal_purchase');
 
 /**
  * Listens for a PayPal IPN requests and then sends to the processing function
@@ -174,12 +174,12 @@ 
  */
 function give_listen_for_paypal_ipn() {
 	// Regular PayPal IPN
-	if ( isset( $_GET['give-listener'] ) && $_GET['give-listener'] == 'IPN' ) {
-		do_action( 'give_verify_paypal_ipn' );
+	if (isset($_GET['give-listener']) && $_GET['give-listener'] == 'IPN') {
+		do_action('give_verify_paypal_ipn');
 	}
 }
 
-add_action( 'init', 'give_listen_for_paypal_ipn' );
+add_action('init', 'give_listen_for_paypal_ipn');
 
 /**
  * Process PayPal IPN
@@ -190,7 +190,7 @@ 
 function give_process_paypal_ipn() {
 
 	// Check the request method is POST
-	if ( isset( $_SERVER['REQUEST_METHOD'] ) && $_SERVER['REQUEST_METHOD'] != 'POST' ) {
+	if (isset($_SERVER['REQUEST_METHOD']) && $_SERVER['REQUEST_METHOD'] != 'POST') {
 		return;
 	}
 
@@ -198,11 +198,11 @@ 
 	$post_data = '';
 
 	// Fallback just in case post_max_size is lower than needed
-	if ( ini_get( 'allow_url_fopen' ) ) {
-		$post_data = file_get_contents( 'php://input' );
+	if (ini_get('allow_url_fopen')) {
+		$post_data = file_get_contents('php://input');
 	} else {
 		// If allow_url_fopen is not enabled, then make sure that post_max_size is large enough
-		ini_set( 'post_max_size', '12M' );
+		ini_set('post_max_size', '12M');
 	}
 	// Start the encoded data collection with notification command
 	$encoded_data = 'cmd=_notify-validate';
@@ -211,27 +211,27 @@ 
 	$arg_separator = give_get_php_arg_separator_output();
 
 	// Verify there is a post_data
-	if ( $post_data || strlen( $post_data ) > 0 ) {
+	if ($post_data || strlen($post_data) > 0) {
 		// Append the data
-		$encoded_data .= $arg_separator . $post_data;
+		$encoded_data .= $arg_separator.$post_data;
 	} else {
 		// Check if POST is empty
-		if ( empty( $_POST ) ) {
+		if (empty($_POST)) {
 			// Nothing to do
 			return;
 		} else {
 			// Loop through each POST
-			foreach ( $_POST as $key => $value ) {
+			foreach ($_POST as $key => $value) {
 				// Encode the value and append the data
-				$encoded_data .= $arg_separator . "$key=" . urlencode( $value );
+				$encoded_data .= $arg_separator."$key=".urlencode($value);
 			}
 		}
 	}
 
 	// Convert collected post data to an array
-	parse_str( $encoded_data, $encoded_data_array );
+	parse_str($encoded_data, $encoded_data_array);
 
-	if ( ! give_get_option( 'disable_paypal_verification' ) ) {
+	if ( ! give_get_option('disable_paypal_verification')) {
 
 		// Validate the IPN
 
@@ -253,16 +253,16 @@ 
 		);
 
 		// Get response
-		$api_response = wp_remote_post( give_get_paypal_redirect(), $remote_post_vars );
+		$api_response = wp_remote_post(give_get_paypal_redirect(), $remote_post_vars);
 
-		if ( is_wp_error( $api_response ) ) {
-			give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( __( 'Invalid IPN verification response. IPN data: %s', 'give' ), json_encode( $api_response ) ) );
+		if (is_wp_error($api_response)) {
+			give_record_gateway_error(__('IPN Error', 'give'), sprintf(__('Invalid IPN verification response. IPN data: %s', 'give'), json_encode($api_response)));
 
 			return; // Something went wrong
 		}
 
-		if ( $api_response['body'] !== 'VERIFIED' && give_get_option( 'disable_paypal_verification', false ) ) {
-			give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( __( 'Invalid IPN verification response. IPN data: %s', 'give' ), json_encode( $api_response ) ) );
+		if ($api_response['body'] !== 'VERIFIED' && give_get_option('disable_paypal_verification', false)) {
+			give_record_gateway_error(__('IPN Error', 'give'), sprintf(__('Invalid IPN verification response. IPN data: %s', 'give'), json_encode($api_response)));
 
 			return; // Response not okay
 		}
@@ -270,7 +270,7 @@ 
 	}
 
 	// Check if $post_data_array has been populated
-	if ( ! is_array( $encoded_data_array ) && ! empty( $encoded_data_array ) ) {
+	if ( ! is_array($encoded_data_array) && ! empty($encoded_data_array)) {
 		return;
 	}
 
@@ -279,21 +279,21 @@ 
 		'payment_status' => ''
 	);
 
-	$encoded_data_array = wp_parse_args( $encoded_data_array, $defaults );
+	$encoded_data_array = wp_parse_args($encoded_data_array, $defaults);
 
-	$payment_id = isset( $encoded_data_array['custom'] ) ? absint( $encoded_data_array['custom'] ) : 0;
+	$payment_id = isset($encoded_data_array['custom']) ? absint($encoded_data_array['custom']) : 0;
 
-	if ( has_action( 'give_paypal_' . $encoded_data_array['txn_type'] ) ) {
+	if (has_action('give_paypal_'.$encoded_data_array['txn_type'])) {
 		// Allow PayPal IPN types to be processed separately
-		do_action( 'give_paypal_' . $encoded_data_array['txn_type'], $encoded_data_array, $payment_id );
+		do_action('give_paypal_'.$encoded_data_array['txn_type'], $encoded_data_array, $payment_id);
 	} else {
 		// Fallback to web accept just in case the txn_type isn't present
-		do_action( 'give_paypal_web_accept', $encoded_data_array, $payment_id );
+		do_action('give_paypal_web_accept', $encoded_data_array, $payment_id);
 	}
 	exit;
 }
 
-add_action( 'give_verify_paypal_ipn', 'give_process_paypal_ipn' );
+add_action('give_verify_paypal_ipn', 'give_process_paypal_ipn');
 
 /**
  * Process web accept (one time) payment IPNs
@@ -304,185 +304,185 @@ 
  *
  * @return void
  */
-function give_process_paypal_web_accept_and_cart( $data, $payment_id ) {
+function give_process_paypal_web_accept_and_cart($data, $payment_id) {
 
-	if ( $data['txn_type'] != 'web_accept' && $data['txn_type'] != 'cart' && $data['payment_status'] != 'Refunded' ) {
+	if ($data['txn_type'] != 'web_accept' && $data['txn_type'] != 'cart' && $data['payment_status'] != 'Refunded') {
 		return;
 	}
 
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
 	// Collect payment details
-	$purchase_key   = isset( $data['invoice'] ) ? $data['invoice'] : $data['item_number'];
+	$purchase_key   = isset($data['invoice']) ? $data['invoice'] : $data['item_number'];
 	$paypal_amount  = $data['mc_gross'];
-	$payment_status = strtolower( $data['payment_status'] );
-	$currency_code  = strtolower( $data['mc_currency'] );
-	$business_email = isset( $data['business'] ) && is_email( $data['business'] ) ? trim( $data['business'] ) : trim( $data['receiver_email'] );
-	$payment_meta   = give_get_payment_meta( $payment_id );
+	$payment_status = strtolower($data['payment_status']);
+	$currency_code  = strtolower($data['mc_currency']);
+	$business_email = isset($data['business']) && is_email($data['business']) ? trim($data['business']) : trim($data['receiver_email']);
+	$payment_meta   = give_get_payment_meta($payment_id);
 
 
-	if ( give_get_payment_gateway( $payment_id ) != 'paypal' ) {
+	if (give_get_payment_gateway($payment_id) != 'paypal') {
 		return; // this isn't a PayPal standard IPN
 	}
 
 	// Verify payment recipient
-	if ( strcasecmp( $business_email, trim( give_get_option( 'paypal_email' ) ) ) != 0 ) {
+	if (strcasecmp($business_email, trim(give_get_option('paypal_email'))) != 0) {
 
-		give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( __( 'Invalid business email in IPN response. IPN data: %s', 'give' ), json_encode( $data ) ), $payment_id );
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, __( 'Payment failed due to invalid PayPal business email.', 'give' ) );
+		give_record_gateway_error(__('IPN Error', 'give'), sprintf(__('Invalid business email in IPN response. IPN data: %s', 'give'), json_encode($data)), $payment_id);
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, __('Payment failed due to invalid PayPal business email.', 'give'));
 
 		return;
 	}
 
 	// Verify payment currency
-	if ( $currency_code != strtolower( $payment_meta['currency'] ) ) {
+	if ($currency_code != strtolower($payment_meta['currency'])) {
 
-		give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( __( 'Invalid currency in IPN response. IPN data: %s', 'give' ), json_encode( $data ) ), $payment_id );
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, __( 'Payment failed due to invalid currency in PayPal IPN.', 'give' ) );
+		give_record_gateway_error(__('IPN Error', 'give'), sprintf(__('Invalid currency in IPN response. IPN data: %s', 'give'), json_encode($data)), $payment_id);
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, __('Payment failed due to invalid currency in PayPal IPN.', 'give'));
 
 		return;
 	}
 
-	if ( ! give_get_payment_user_email( $payment_id ) ) {
+	if ( ! give_get_payment_user_email($payment_id)) {
 
 		// No email associated with purchase, so store from PayPal
-		give_update_payment_meta( $payment_id, '_give_payment_user_email', $data['payer_email'] );
+		give_update_payment_meta($payment_id, '_give_payment_user_email', $data['payer_email']);
 
 		// Setup and store the donors's details
 		$address            = array();
-		$address['line1']   = ! empty( $data['address_street'] ) ? sanitize_text_field( $data['address_street'] ) : false;
-		$address['city']    = ! empty( $data['address_city'] ) ? sanitize_text_field( $data['address_city'] ) : false;
-		$address['state']   = ! empty( $data['address_state'] ) ? sanitize_text_field( $data['address_state'] ) : false;
-		$address['country'] = ! empty( $data['address_country_code'] ) ? sanitize_text_field( $data['address_country_code'] ) : false;
-		$address['zip']     = ! empty( $data['address_zip'] ) ? sanitize_text_field( $data['address_zip'] ) : false;
+		$address['line1']   = ! empty($data['address_street']) ? sanitize_text_field($data['address_street']) : false;
+		$address['city']    = ! empty($data['address_city']) ? sanitize_text_field($data['address_city']) : false;
+		$address['state']   = ! empty($data['address_state']) ? sanitize_text_field($data['address_state']) : false;
+		$address['country'] = ! empty($data['address_country_code']) ? sanitize_text_field($data['address_country_code']) : false;
+		$address['zip']     = ! empty($data['address_zip']) ? sanitize_text_field($data['address_zip']) : false;
 
 		$user_info = array(
 			'id'         => '-1',
-			'email'      => sanitize_text_field( $data['payer_email'] ),
-			'first_name' => sanitize_text_field( $data['first_name'] ),
-			'last_name'  => sanitize_text_field( $data['last_name'] ),
+			'email'      => sanitize_text_field($data['payer_email']),
+			'first_name' => sanitize_text_field($data['first_name']),
+			'last_name'  => sanitize_text_field($data['last_name']),
 			'discount'   => '',
 			'address'    => $address
 		);
 
 		$payment_meta['user_info'] = $user_info;
-		give_update_payment_meta( $payment_id, '_give_payment_meta', $payment_meta );
+		give_update_payment_meta($payment_id, '_give_payment_meta', $payment_meta);
 	}
 
-	if ( $payment_status == 'refunded' || $payment_status == 'reversed' ) {
+	if ($payment_status == 'refunded' || $payment_status == 'reversed') {
 
 		// Process a refund
-		give_process_paypal_refund( $data, $payment_id );
+		give_process_paypal_refund($data, $payment_id);
 
 	} else {
 
-		if ( get_post_status( $payment_id ) == 'publish' ) {
+		if (get_post_status($payment_id) == 'publish') {
 			return; // Only complete payments once
 		}
 
 		// Retrieve the total purchase amount (before PayPal)
-		$payment_amount = give_get_payment_amount( $payment_id );
+		$payment_amount = give_get_payment_amount($payment_id);
 
-		if ( number_format( (float) $paypal_amount, 2 ) < number_format( (float) $payment_amount, 2 ) ) {
+		if (number_format((float) $paypal_amount, 2) < number_format((float) $payment_amount, 2)) {
 			// The prices don't match
-			give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( __( 'Invalid payment amount in IPN response. IPN data: %s', 'give' ), json_encode( $data ) ), $payment_id );
-			give_update_payment_status( $payment_id, 'failed' );
-			give_insert_payment_note( $payment_id, __( 'Payment failed due to invalid amount in PayPal IPN.', 'give' ) );
+			give_record_gateway_error(__('IPN Error', 'give'), sprintf(__('Invalid payment amount in IPN response. IPN data: %s', 'give'), json_encode($data)), $payment_id);
+			give_update_payment_status($payment_id, 'failed');
+			give_insert_payment_note($payment_id, __('Payment failed due to invalid amount in PayPal IPN.', 'give'));
 
 			return;
 		}
-		if ( $purchase_key != give_get_payment_key( $payment_id ) ) {
+		if ($purchase_key != give_get_payment_key($payment_id)) {
 			// Purchase keys don't match
-			give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( __( 'Invalid purchase key in IPN response. IPN data: %s', 'give' ), json_encode( $data ) ), $payment_id );
-			give_update_payment_status( $payment_id, 'failed' );
-			give_insert_payment_note( $payment_id, __( 'Payment failed due to invalid purchase key in PayPal IPN.', 'give' ) );
+			give_record_gateway_error(__('IPN Error', 'give'), sprintf(__('Invalid purchase key in IPN response. IPN data: %s', 'give'), json_encode($data)), $payment_id);
+			give_update_payment_status($payment_id, 'failed');
+			give_insert_payment_note($payment_id, __('Payment failed due to invalid purchase key in PayPal IPN.', 'give'));
 
 			return;
 		}
 
-		if ( $payment_status == 'completed' || give_is_test_mode() ) {
-			give_insert_payment_note( $payment_id, sprintf( __( 'PayPal Transaction ID: %s', 'give' ), $data['txn_id'] ) );
-			give_set_payment_transaction_id( $payment_id, $data['txn_id'] );
-			give_update_payment_status( $payment_id, 'publish' );
-		} else if ( 'pending' == $payment_status && isset( $data['pending_reason'] ) ) {
+		if ($payment_status == 'completed' || give_is_test_mode()) {
+			give_insert_payment_note($payment_id, sprintf(__('PayPal Transaction ID: %s', 'give'), $data['txn_id']));
+			give_set_payment_transaction_id($payment_id, $data['txn_id']);
+			give_update_payment_status($payment_id, 'publish');
+		} else if ('pending' == $payment_status && isset($data['pending_reason'])) {
 
 			// Look for possible pending reasons, such as an echeck
 
 			$note = '';
 
-			switch ( strtolower( $data['pending_reason'] ) ) {
+			switch (strtolower($data['pending_reason'])) {
 
 				case 'echeck' :
 
-					$note = __( 'Payment made via eCheck and will clear automatically in 5-8 days', 'give' );
+					$note = __('Payment made via eCheck and will clear automatically in 5-8 days', 'give');
 
 					break;
 
 				case 'address' :
 
-					$note = __( 'Payment requires a confirmed donor address and must be accepted manually through PayPal', 'give' );
+					$note = __('Payment requires a confirmed donor address and must be accepted manually through PayPal', 'give');
 
 					break;
 
 				case 'intl' :
 
-					$note = __( 'Payment must be accepted manually through PayPal due to international account regulations', 'give' );
+					$note = __('Payment must be accepted manually through PayPal due to international account regulations', 'give');
 
 					break;
 
 				case 'multi-currency' :
 
-					$note = __( 'Payment received in non-shop currency and must be accepted manually through PayPal', 'give' );
+					$note = __('Payment received in non-shop currency and must be accepted manually through PayPal', 'give');
 
 					break;
 
 				case 'paymentreview' :
 				case 'regulatory_review' :
 
-					$note = __( 'Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations', 'give' );
+					$note = __('Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations', 'give');
 
 					break;
 
 				case 'unilateral' :
 
-					$note = __( 'Payment was sent to non-confirmed or non-registered email address.', 'give' );
+					$note = __('Payment was sent to non-confirmed or non-registered email address.', 'give');
 
 					break;
 
 				case 'upgrade' :
 
-					$note = __( 'PayPal account must be upgraded before this payment can be accepted', 'give' );
+					$note = __('PayPal account must be upgraded before this payment can be accepted', 'give');
 
 					break;
 
 				case 'verify' :
 
-					$note = __( 'PayPal account is not verified. Verify account in order to accept this payment', 'give' );
+					$note = __('PayPal account is not verified. Verify account in order to accept this payment', 'give');
 
 					break;
 
 				case 'other' :
 
-					$note = __( 'Payment is pending for unknown reasons. Contact PayPal support for assistance', 'give' );
+					$note = __('Payment is pending for unknown reasons. Contact PayPal support for assistance', 'give');
 
 					break;
 
 			}
 
-			if ( ! empty( $note ) ) {
+			if ( ! empty($note)) {
 
-				give_insert_payment_note( $payment_id, $note );
+				give_insert_payment_note($payment_id, $note);
 
 			}
 		}
 	}
 }
 
-add_action( 'give_paypal_web_accept', 'give_process_paypal_web_accept_and_cart', 10, 2 );
+add_action('give_paypal_web_accept', 'give_process_paypal_web_accept_and_cart', 10, 2);
 
 /**
  * Process PayPal IPN Refunds
@@ -493,32 +493,32 @@ 
  *
  * @return void
  */
-function give_process_paypal_refund( $data, $payment_id = 0 ) {
+function give_process_paypal_refund($data, $payment_id = 0) {
 
 	// Collect payment details
 
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
-	if ( get_post_status( $payment_id ) == 'refunded' ) {
+	if (get_post_status($payment_id) == 'refunded') {
 		return; // Only refund payments once
 	}
 
-	$payment_amount = give_get_payment_amount( $payment_id );
+	$payment_amount = give_get_payment_amount($payment_id);
 	$refund_amount  = $data['payment_gross'] * - 1;
 
-	if ( number_format( (float) $refund_amount, 2 ) < number_format( (float) $payment_amount, 2 ) ) {
+	if (number_format((float) $refund_amount, 2) < number_format((float) $payment_amount, 2)) {
 
-		give_insert_payment_note( $payment_id, sprintf( __( 'Partial PayPal refund processed: %s', 'give' ), $data['parent_txn_id'] ) );
+		give_insert_payment_note($payment_id, sprintf(__('Partial PayPal refund processed: %s', 'give'), $data['parent_txn_id']));
 
 		return; // This is a partial refund
 
 	}
 
-	give_insert_payment_note( $payment_id, sprintf( __( 'PayPal Payment #%s Refunded for reason: %s', 'give' ), $data['parent_txn_id'], $data['reason_code'] ) );
-	give_insert_payment_note( $payment_id, sprintf( __( 'PayPal Refund Transaction ID: %s', 'give' ), $data['txn_id'] ) );
-	give_update_payment_status( $payment_id, 'refunded' );
+	give_insert_payment_note($payment_id, sprintf(__('PayPal Payment #%s Refunded for reason: %s', 'give'), $data['parent_txn_id'], $data['reason_code']));
+	give_insert_payment_note($payment_id, sprintf(__('PayPal Refund Transaction ID: %s', 'give'), $data['txn_id']));
+	give_update_payment_status($payment_id, 'refunded');
 }
 
 /**
@@ -530,24 +530,24 @@ 
  *
  * @return string
  */
-function give_get_paypal_redirect( $ssl_check = false ) {
+function give_get_paypal_redirect($ssl_check = false) {
 
-	if ( is_ssl() || ! $ssl_check ) {
+	if (is_ssl() || ! $ssl_check) {
 		$protocal = 'https://';
 	} else {
 		$protocal = 'http://';
 	}
 
 	// Check the current payment mode
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		// Test mode
-		$paypal_uri = $protocal . 'www.sandbox.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocal.'www.sandbox.paypal.com/cgi-bin/webscr';
 	} else {
 		// Live mode
-		$paypal_uri = $protocal . 'www.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocal.'www.paypal.com/cgi-bin/webscr';
 	}
 
-	return apply_filters( 'give_paypal_uri', $paypal_uri );
+	return apply_filters('give_paypal_uri', $paypal_uri);
 }
 
 /**
@@ -558,9 +558,9 @@ 
  */
 function give_get_paypal_page_style() {
 
-	$page_style = trim( give_get_option( 'paypal_page_style', 'PayPal' ) );
+	$page_style = trim(give_get_option('paypal_page_style', 'PayPal'));
 
-	return apply_filters( 'give_paypal_page_style', $page_style );
+	return apply_filters('give_paypal_page_style', $page_style);
 }
 
 /**
@@ -571,27 +571,27 @@ 
  * @since 1.0
  * @return string
  */
-function give_paypal_success_page_content( $content ) {
+function give_paypal_success_page_content($content) {
 
-	if ( ! isset( $_GET['payment-id'] ) && ! give_get_purchase_session() ) {
+	if ( ! isset($_GET['payment-id']) && ! give_get_purchase_session()) {
 		return $content;
 	}
 
-	$payment_id = isset( $_GET['payment-id'] ) ? absint( $_GET['payment-id'] ) : false;
+	$payment_id = isset($_GET['payment-id']) ? absint($_GET['payment-id']) : false;
 
-	if ( ! $payment_id ) {
+	if ( ! $payment_id) {
 		$session    = give_get_purchase_session();
-		$payment_id = give_get_purchase_id_by_key( $session['purchase_key'] );
+		$payment_id = give_get_purchase_id_by_key($session['purchase_key']);
 	}
 
-	$payment = get_post( $payment_id );
+	$payment = get_post($payment_id);
 
-	if ( $payment && 'pending' == $payment->post_status ) {
+	if ($payment && 'pending' == $payment->post_status) {
 
 		// Payment is still pending so show processing indicator to fix the Race Condition, issue #
 		ob_start();
 
-		give_get_template_part( 'payment', 'processing' );
+		give_get_template_part('payment', 'processing');
 
 		$content = ob_get_clean();
 
@@ -601,7 +601,7 @@ 
 
 }
 
-add_filter( 'give_payment_confirm_paypal', 'give_paypal_success_page_content' );
+add_filter('give_payment_confirm_paypal', 'give_paypal_success_page_content');
 
 /**
  * Given a Payment ID, extract the transaction ID
@@ -612,22 +612,22 @@ 
  *
  * @return string                   Transaction ID
  */
-function give_paypal_get_payment_transaction_id( $payment_id ) {
+function give_paypal_get_payment_transaction_id($payment_id) {
 
 	$transaction_id = '';
-	$notes          = give_get_payment_notes( $payment_id );
+	$notes          = give_get_payment_notes($payment_id);
 
-	foreach ( $notes as $note ) {
-		if ( preg_match( '/^PayPal Transaction ID: ([^\s]+)/', $note->comment_content, $match ) ) {
+	foreach ($notes as $note) {
+		if (preg_match('/^PayPal Transaction ID: ([^\s]+)/', $note->comment_content, $match)) {
 			$transaction_id = $match[1];
 			continue;
 		}
 	}
 
-	return apply_filters( 'give_paypal_set_payment_transaction_id', $transaction_id, $payment_id );
+	return apply_filters('give_paypal_set_payment_transaction_id', $transaction_id, $payment_id);
 }
 
-add_filter( 'give_get_payment_transaction_id-paypal', 'give_paypal_get_payment_transaction_id', 10, 1 );
+add_filter('give_get_payment_transaction_id-paypal', 'give_paypal_get_payment_transaction_id', 10, 1);
 
 /**
  * Given a transaction ID, generate a link to the PayPal transaction ID details
@@ -639,13 +639,13 @@ 
  *
  * @return string                 A link to the PayPal transaction details
  */
-function give_paypal_link_transaction_id( $transaction_id, $payment_id ) {
+function give_paypal_link_transaction_id($transaction_id, $payment_id) {
 
 	$paypal_base_url = 'https://history.paypal.com/cgi-bin/webscr?cmd=_history-details-from-hub&id=';
-	$transaction_url = '<a href="' . esc_url( $paypal_base_url . $transaction_id ) . '" target="_blank">' . $transaction_id . '</a>';
+	$transaction_url = '<a href="'.esc_url($paypal_base_url.$transaction_id).'" target="_blank">'.$transaction_id.'</a>';
 
-	return apply_filters( 'give_paypal_link_payment_details_transaction_id', $transaction_url );
+	return apply_filters('give_paypal_link_payment_details_transaction_id', $transaction_url);
 
 }
 
-add_filter( 'give_payment_details_transaction_id-paypal', 'give_paypal_link_transaction_id', 10, 2 );
+add_filter('give_payment_details_transaction_id-paypal', 'give_paypal_link_transaction_id', 10, 2);

includes/install.php

Spacing +36 added lines, -36 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,25 +32,25 @@ 
 	give_setup_post_types();
 
 	// Clear the permalinks
-	flush_rewrite_rules( false );
+	flush_rewrite_rules(false);
 
 	// 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();
 
 	// 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'))) {
 
 		// Purchase Confirmation (Success) Page
 		$success = wp_insert_post(
 			array(
-				'post_title'     => __( 'Donation Confirmation', 'give' ),
-				'post_content'   => __( 'Thank you for your donation! [give_receipt]', 'give' ),
+				'post_title'     => __('Donation Confirmation', 'give'),
+				'post_content'   => __('Thank you for your donation! [give_receipt]', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -63,13 +63,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 Purchase Page
 		$failed = wp_insert_post(
 			array(
-				'post_title'     => __( 'Transaction Failed', 'give' ),
-				'post_content'   => __( 'Your transaction failed, please try again or contact site support.', 'give' ),
+				'post_title'     => __('Transaction Failed', 'give'),
+				'post_content'   => __('Your transaction failed, please try again or contact site support.', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -81,11 +81,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'))) {
 		// Purchase History (History) Page
 		$history = wp_insert_post(
 			array(
-				'post_title'     => __( 'Donation History', 'give' ),
+				'post_title'     => __('Donation History', 'give'),
 				'post_content'   => '[donation_history]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -98,7 +98,7 @@ 
 	}
 
 	//Fresh Install? Setup Test Mode, Base Country (US), Test Gateway, Currency
-	if ( empty( $current_version ) ) {
+	if (empty($current_version)) {
 		$options['base_country']       = 'US';
 		$options['test_mode']          = 1;
 		$options['currency']           = 'USD';
@@ -115,12 +115,12 @@ 
 	}
 
 	// Populate some default values
-	update_option( 'give_settings', array_merge( $give_options, $options ) );
-	update_option( 'give_version', GIVE_VERSION );
+	update_option('give_settings', array_merge($give_options, $options));
+	update_option('give_version', GIVE_VERSION);
 
 	//Update Version Number
-	if ( $current_version ) {
-		update_option( 'give_version_upgraded_from', $current_version );
+	if ($current_version) {
+		update_option('give_version_upgraded_from', $current_version);
 	}
 
 	// Create Give roles
@@ -129,7 +129,7 @@ 
 	$roles->add_caps();
 
 	$api = new Give_API();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	// Create the customers database
 	@Give()->customers->create_table();
@@ -138,32 +138,32 @@ 
 	Give()->session->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created
-	set_transient( '_give_installed', $options, 30 );
+	set_transient('_give_installed', $options, 30);
 
 	// Bail if activating from network, or bulk
-	if ( is_network_admin() || isset( $_GET['activate-multi'] ) ) {
+	if (is_network_admin() || isset($_GET['activate-multi'])) {
 		return;
 	}
 
-	if ( ! $current_version ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+	if ( ! $current_version) {
+		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(
 			'upgrade_donor_payments_association'
 		);
 
-		foreach ( $upgrade_routines as $upgrade ) {
-			give_set_upgrade_complete( $upgrade );
+		foreach ($upgrade_routines as $upgrade) {
+			give_set_upgrade_complete($upgrade);
 		}
 	}
 
 	// Add the transient to redirect
-	set_transient( '_give_activation_redirect', true, 30 );
+	set_transient('_give_activation_redirect', true, 30);
 
 }
 
-register_activation_hook( GIVE_PLUGIN_FILE, 'give_install' );
+register_activation_hook(GIVE_PLUGIN_FILE, 'give_install');
 
 /**
  * Post-installation
@@ -175,14 +175,14 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options = get_transient( '_give_installed' );
+	$give_options = get_transient('_give_installed');
 
 	// Exit if not in admin or the transient doesn't exist
-	if ( false === $give_options ) {
+	if (false === $give_options) {
 		return;
 	}
 
@@ -190,13 +190,13 @@ 
 	@Give()->customers->create_table();
 
 	// Delete the transient
-	delete_transient( '_give_installed' );
+	delete_transient('_give_installed');
 
-	do_action( 'give_after_install', $give_options );
+	do_action('give_after_install', $give_options);
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -211,11 +211,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 shop roles
 		$roles = new Give_Roles;
@@ -226,4 +226,4 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
+add_action('admin_init', 'give_install_roles_on_network');

includes/login-register.php

Spacing +59 added lines, -59 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,10 +25,10 @@ 
  *
  * @return string Login form
  */
-function give_login_form( $redirect = '' ) {
+function give_login_form($redirect = '') {
 	global $give_options, $give_login_redirect;
 
-	if ( empty( $redirect ) ) {
+	if (empty($redirect)) {
 		$redirect = give_get_current_page_url();
 	}
 
@@ -36,9 +36,9 @@ 
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'login' );
+	give_get_template_part('shortcode', 'login');
 
-	return apply_filters( 'give_login_form', ob_get_clean() );
+	return apply_filters('give_login_form', ob_get_clean());
 }
 
 /**
@@ -52,10 +52,10 @@ 
  *
  * @return string Register form
  */
-function give_register_form( $redirect = '' ) {
+function give_register_form($redirect = '') {
 	global $give_options, $give_register_redirect;
 
-	if ( empty( $redirect ) ) {
+	if (empty($redirect)) {
 		$redirect = give_get_current_page_url();
 	}
 
@@ -63,11 +63,11 @@ 
 
 	ob_start();
 
-	if ( ! is_user_logged_in() ) {
-		give_get_template_part( 'shortcode', 'register' );
+	if ( ! is_user_logged_in()) {
+		give_get_template_part('shortcode', 'register');
 	}
 
-	return apply_filters( 'give_register_form', ob_get_clean() );
+	return apply_filters('give_register_form', ob_get_clean());
 }
 
 /**
@@ -79,34 +79,34 @@ 
  *
  * @return void
  */
-function give_process_login_form( $data ) {
-	if ( wp_verify_nonce( $data['give_login_nonce'], 'give-login-nonce' ) ) {
-		$user_data = get_user_by( 'login', $data['give_user_login'] );
-		if ( ! $user_data ) {
-			$user_data = get_user_by( 'email', $data['give_user_login'] );
+function give_process_login_form($data) {
+	if (wp_verify_nonce($data['give_login_nonce'], 'give-login-nonce')) {
+		$user_data = get_user_by('login', $data['give_user_login']);
+		if ( ! $user_data) {
+			$user_data = get_user_by('email', $data['give_user_login']);
 		}
-		if ( $user_data ) {
+		if ($user_data) {
 			$user_ID    = $user_data->ID;
 			$user_email = $user_data->user_email;
-			if ( wp_check_password( $data['give_user_pass'], $user_data->user_pass, $user_data->ID ) ) {
-				give_log_user_in( $user_data->ID, $data['give_user_login'], $data['give_user_pass'] );
+			if (wp_check_password($data['give_user_pass'], $user_data->user_pass, $user_data->ID)) {
+				give_log_user_in($user_data->ID, $data['give_user_login'], $data['give_user_pass']);
 			} else {
-				give_set_error( 'password_incorrect', __( 'The password you entered is incorrect', 'give' ) );
+				give_set_error('password_incorrect', __('The password you entered is incorrect', 'give'));
 			}
 		} else {
-			give_set_error( 'username_incorrect', __( 'The username you entered does not exist', 'give' ) );
+			give_set_error('username_incorrect', __('The username you entered does not exist', 'give'));
 		}
 		// Check for errors and redirect if none present
 		$errors = give_get_errors();
-		if ( ! $errors ) {
-			$redirect = apply_filters( 'give_login_redirect', $data['give_redirect'], $user_ID );
-			wp_redirect( $redirect );
+		if ( ! $errors) {
+			$redirect = apply_filters('give_login_redirect', $data['give_redirect'], $user_ID);
+			wp_redirect($redirect);
 			give_die();
 		}
 	}
 }
 
-add_action( 'give_user_login', 'give_process_login_form' );
+add_action('give_user_login', 'give_process_login_form');
 
 /**
  * Log User In
@@ -119,15 +119,15 @@ 
  *
  * @return void
  */
-function give_log_user_in( $user_id, $user_login, $user_pass ) {
-	if ( $user_id < 1 ) {
+function give_log_user_in($user_id, $user_login, $user_pass) {
+	if ($user_id < 1) {
 		return;
 	}
 
-	wp_set_auth_cookie( $user_id );
-	wp_set_current_user( $user_id, $user_login );
-	do_action( 'wp_login', $user_login, get_userdata( $user_id ) );
-	do_action( 'give_log_user_in', $user_id, $user_login, $user_pass );
+	wp_set_auth_cookie($user_id);
+	wp_set_current_user($user_id, $user_login);
+	do_action('wp_login', $user_login, get_userdata($user_id));
+	do_action('give_log_user_in', $user_id, $user_login, $user_pass);
 }
 
 
@@ -140,70 +140,70 @@ 
  *
  * @return void
  */
-function give_process_register_form( $data ) {
+function give_process_register_form($data) {
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 		return;
 	}
 
-	if ( empty( $_POST['give_register_submit'] ) ) {
+	if (empty($_POST['give_register_submit'])) {
 		return;
 	}
 
-	do_action( 'give_pre_process_register_form' );
+	do_action('give_pre_process_register_form');
 
-	if ( empty( $data['give_user_login'] ) ) {
-		give_set_error( 'empty_username', __( 'Invalid username', 'give' ) );
+	if (empty($data['give_user_login'])) {
+		give_set_error('empty_username', __('Invalid username', 'give'));
 	}
 
-	if ( username_exists( $data['give_user_login'] ) ) {
-		give_set_error( 'username_unavailable', __( 'Username already taken', 'give' ) );
+	if (username_exists($data['give_user_login'])) {
+		give_set_error('username_unavailable', __('Username already taken', 'give'));
 	}
 
-	if ( ! validate_username( $data['give_user_login'] ) ) {
-		give_set_error( 'username_invalid', __( 'Invalid username', 'give' ) );
+	if ( ! validate_username($data['give_user_login'])) {
+		give_set_error('username_invalid', __('Invalid username', 'give'));
 	}
 
-	if ( email_exists( $data['give_user_email'] ) ) {
-		give_set_error( 'email_unavailable', __( 'Email address already taken', 'give' ) );
+	if (email_exists($data['give_user_email'])) {
+		give_set_error('email_unavailable', __('Email address already taken', 'give'));
 	}
 
-	if ( empty( $data['give_user_email'] ) || ! is_email( $data['give_user_email'] ) ) {
-		give_set_error( 'email_invalid', __( 'Invalid email', 'give' ) );
+	if (empty($data['give_user_email']) || ! is_email($data['give_user_email'])) {
+		give_set_error('email_invalid', __('Invalid email', 'give'));
 	}
 
-	if ( ! empty( $data['give_payment_email'] ) && $data['give_payment_email'] != $data['give_user_email'] && ! is_email( $data['give_payment_email'] ) ) {
-		give_set_error( 'payment_email_invalid', __( 'Invalid payment email', 'give' ) );
+	if ( ! empty($data['give_payment_email']) && $data['give_payment_email'] != $data['give_user_email'] && ! is_email($data['give_payment_email'])) {
+		give_set_error('payment_email_invalid', __('Invalid payment email', 'give'));
 	}
 
-	if ( empty( $_POST['give_user_pass'] ) ) {
-		give_set_error( 'empty_password', __( 'Please enter a password', 'give' ) );
+	if (empty($_POST['give_user_pass'])) {
+		give_set_error('empty_password', __('Please enter a password', 'give'));
 	}
 
-	if ( ( ! empty( $_POST['give_user_pass'] ) && empty( $_POST['give_user_pass2'] ) ) || ( $_POST['give_user_pass'] !== $_POST['give_user_pass2'] ) ) {
-		give_set_error( 'password_mismatch', __( 'Passwords do not match', 'give' ) );
+	if (( ! empty($_POST['give_user_pass']) && empty($_POST['give_user_pass2'])) || ($_POST['give_user_pass'] !== $_POST['give_user_pass2'])) {
+		give_set_error('password_mismatch', __('Passwords do not match', 'give'));
 	}
 
-	do_action( 'give_process_register_form' );
+	do_action('give_process_register_form');
 
 	// Check for errors and redirect if none present
 	$errors = give_get_errors();
 
-	if ( empty( $errors ) ) {
+	if (empty($errors)) {
 
-		$redirect = apply_filters( 'give_register_redirect', $data['give_redirect'] );
+		$redirect = apply_filters('give_register_redirect', $data['give_redirect']);
 
-		give_register_and_login_new_user( array(
+		give_register_and_login_new_user(array(
 			'user_login'      => $data['give_user_login'],
 			'user_pass'       => $data['give_user_pass'],
 			'user_email'      => $data['give_user_email'],
-			'user_registered' => date( 'Y-m-d H:i:s' ),
-			'role'            => get_option( 'default_role' )
-		) );
+			'user_registered' => date('Y-m-d H:i:s'),
+			'role'            => get_option('default_role')
+		));
 
-		wp_redirect( $redirect );
+		wp_redirect($redirect);
 		give_die();
 	}
 }
 
-add_action( 'give_user_register', 'give_process_register_form' );
\ No newline at end of file
+add_action('give_user_register', 'give_process_register_form');
\ No newline at end of file

includes/misc-functions.php

Braces +3 added lines, -1 removed lines

@@ -142,8 +142,10 @@
 
 	if ( is_front_page() ) :
 		$page_url = home_url();
-	else :
+	else {
+		:
 		$page_url = 'http';
+	}
 
 		if ( isset( $_SERVER["HTTPS"] ) && $_SERVER["HTTPS"] == "on" ) {
 			$page_url .= "s";

Doc Comments +5 added lines, -5 removed lines

@@ -335,7 +335,7 @@ 
  * Returns the webhost this site is using if possible
  *
  * @since 1.0
- * @return mixed string $host if detected, false otherwise
+ * @return string string $host if detected, false otherwise
  */
 function give_get_host() {
 	$host = false;
@@ -506,7 +506,7 @@ 
  *
  * @since 1.0
  *
- * @param unknown $n
+ * @param integer $n
  *
  * @return string Short month name
  */
@@ -636,9 +636,9 @@ 
 /**
  * Create SVG library function
  *
- * @param $icon
+ * @param string $icon
  *
- * @return mixed
+ * @return string
  */
 function give_svg_icons( $icon ) {
 
@@ -695,7 +695,7 @@ 
 	 *
 	 * @param array $input     A multi-dimensional array (record set) from which to pull
 	 *                         a column of values.
-	 * @param mixed $columnKey The column of values to return. This value may be the
+	 * @param string $columnKey The column of values to return. This value may be the
 	 *                         integer key of the column you wish to retrieve, or it
 	 *                         may be the string key name for an associative array.
 	 * @param mixed $indexKey  (Optional.) The column to use as the index/keys for

Indentation +8 added lines, -8 removed lines

@@ -727,10 +727,10 @@ 
 		}
 
 		if ( ! is_int( $params[1] )
-		     && ! is_float( $params[1] )
-		     && ! is_string( $params[1] )
-		     && $params[1] !== null
-		     && ! ( is_object( $params[1] ) && method_exists( $params[1], '__toString' ) )
+			 && ! is_float( $params[1] )
+			 && ! is_string( $params[1] )
+			 && $params[1] !== null
+			 && ! ( is_object( $params[1] ) && method_exists( $params[1], '__toString' ) )
 		) {
 			trigger_error( 'array_column(): The column key should be either a string or an integer', E_USER_WARNING );
 
@@ -738,10 +738,10 @@ 
 		}
 
 		if ( isset( $params[2] )
-		     && ! is_int( $params[2] )
-		     && ! is_float( $params[2] )
-		     && ! is_string( $params[2] )
-		     && ! ( is_object( $params[2] ) && method_exists( $params[2], '__toString' ) )
+			 && ! is_int( $params[2] )
+			 && ! is_float( $params[2] )
+			 && ! is_string( $params[2] )
+			 && ! ( is_object( $params[2] ) && method_exists( $params[2], '__toString' ) )
 		) {
 			trigger_error( 'array_column(): The index key should be either a string or an integer', E_USER_WARNING );
 

Spacing +154 added lines, -154 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
 function give_is_test_mode() {
 	global $give_options;
 
-	$ret = ! empty( $give_options['test_mode'] );
+	$ret = ! empty($give_options['test_mode']);
 
-	return (bool) apply_filters( 'give_is_test_mode', $ret );
+	return (bool) apply_filters('give_is_test_mode', $ret);
 }
 
 /**
@@ -37,9 +37,9 @@ 
  */
 function give_get_currency() {
 	global $give_options;
-	$currency = isset( $give_options['currency'] ) ? $give_options['currency'] : 'USD';
+	$currency = isset($give_options['currency']) ? $give_options['currency'] : 'USD';
 
-	return apply_filters( 'give_currency', $currency );
+	return apply_filters('give_currency', $currency);
 }
 
 
@@ -51,36 +51,36 @@ 
  */
 function give_get_currencies() {
 	$currencies = array(
-		'USD'  => __( 'US Dollars ($)', 'give' ),
-		'EUR'  => __( 'Euros (€)', 'give' ),
-		'GBP'  => __( 'Pounds Sterling (£)', 'give' ),
-		'AUD'  => __( 'Australian Dollars ($)', 'give' ),
-		'BRL'  => __( 'Brazilian Real (R$)', 'give' ),
-		'CAD'  => __( 'Canadian Dollars ($)', 'give' ),
-		'CZK'  => __( 'Czech Koruna', 'give' ),
-		'DKK'  => __( 'Danish Krone', 'give' ),
-		'HKD'  => __( 'Hong Kong Dollar ($)', 'give' ),
-		'HUF'  => __( 'Hungarian Forint', 'give' ),
-		'ILS'  => __( 'Israeli Shekel (₪)', 'give' ),
-		'JPY'  => __( 'Japanese Yen (¥)', 'give' ),
-		'MYR'  => __( 'Malaysian Ringgits', 'give' ),
-		'MXN'  => __( 'Mexican Peso ($)', 'give' ),
-		'NZD'  => __( 'New Zealand Dollar ($)', 'give' ),
-		'NOK'  => __( 'Norwegian Krone (Kr.)', 'give' ),
-		'PHP'  => __( 'Philippine Pesos', 'give' ),
-		'PLN'  => __( 'Polish Zloty', 'give' ),
-		'SGD'  => __( 'Singapore Dollar ($)', 'give' ),
-		'SEK'  => __( 'Swedish Krona', 'give' ),
-		'CHF'  => __( 'Swiss Franc', 'give' ),
-		'TWD'  => __( 'Taiwan New Dollars', 'give' ),
-		'THB'  => __( 'Thai Baht (฿)', 'give' ),
-		'INR'  => __( 'Indian Rupee (₹)', 'give' ),
-		'TRY'  => __( 'Turkish Lira (₺)', 'give' ),
-		'RIAL' => __( 'Iranian Rial (﷼)', 'give' ),
-		'RUB'  => __( 'Russian Rubles', 'give' )
+		'USD'  => __('US Dollars ($)', 'give'),
+		'EUR'  => __('Euros (€)', 'give'),
+		'GBP'  => __('Pounds Sterling (£)', 'give'),
+		'AUD'  => __('Australian Dollars ($)', 'give'),
+		'BRL'  => __('Brazilian Real (R$)', 'give'),
+		'CAD'  => __('Canadian Dollars ($)', 'give'),
+		'CZK'  => __('Czech Koruna', 'give'),
+		'DKK'  => __('Danish Krone', 'give'),
+		'HKD'  => __('Hong Kong Dollar ($)', 'give'),
+		'HUF'  => __('Hungarian Forint', 'give'),
+		'ILS'  => __('Israeli Shekel (₪)', 'give'),
+		'JPY'  => __('Japanese Yen (¥)', 'give'),
+		'MYR'  => __('Malaysian Ringgits', 'give'),
+		'MXN'  => __('Mexican Peso ($)', 'give'),
+		'NZD'  => __('New Zealand Dollar ($)', 'give'),
+		'NOK'  => __('Norwegian Krone (Kr.)', 'give'),
+		'PHP'  => __('Philippine Pesos', 'give'),
+		'PLN'  => __('Polish Zloty', 'give'),
+		'SGD'  => __('Singapore Dollar ($)', 'give'),
+		'SEK'  => __('Swedish Krona', 'give'),
+		'CHF'  => __('Swiss Franc', 'give'),
+		'TWD'  => __('Taiwan New Dollars', 'give'),
+		'THB'  => __('Thai Baht (฿)', 'give'),
+		'INR'  => __('Indian Rupee (₹)', 'give'),
+		'TRY'  => __('Turkish Lira (₺)', 'give'),
+		'RIAL' => __('Iranian Rial (﷼)', 'give'),
+		'RUB'  => __('Russian Rubles', 'give')
 	);
 
-	return apply_filters( 'give_currencies', $currencies );
+	return apply_filters('give_currencies', $currencies);
 }
 
 
@@ -94,12 +94,12 @@ 
  *
  * @return string           The symbol to use for the currency
  */
-function give_currency_symbol( $currency = '' ) {
+function give_currency_symbol($currency = '') {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
-	switch ( $currency ) :
+	switch ($currency) :
 		case "GBP" :
 			$symbol = '£';
 			break;
@@ -128,7 +128,7 @@ 
 			break;
 	endswitch;
 
-	return apply_filters( 'give_currency_symbol', $symbol, $currency );
+	return apply_filters('give_currency_symbol', $symbol, $currency);
 }
 
 
@@ -140,25 +140,25 @@ 
  */
 function give_get_current_page_url() {
 
-	if ( is_front_page() ) :
+	if (is_front_page()) :
 		$page_url = home_url();
 	else :
 		$page_url = 'http';
 
-		if ( isset( $_SERVER["HTTPS"] ) && $_SERVER["HTTPS"] == "on" ) {
+		if (isset($_SERVER["HTTPS"]) && $_SERVER["HTTPS"] == "on") {
 			$page_url .= "s";
 		}
 
 		$page_url .= "://";
 
-		if ( isset( $_SERVER["SERVER_PORT"] ) && $_SERVER["SERVER_PORT"] != "80" ) {
-			$page_url .= $_SERVER["SERVER_NAME"] . ":" . $_SERVER["SERVER_PORT"] . $_SERVER["REQUEST_URI"];
+		if (isset($_SERVER["SERVER_PORT"]) && $_SERVER["SERVER_PORT"] != "80") {
+			$page_url .= $_SERVER["SERVER_NAME"].":".$_SERVER["SERVER_PORT"].$_SERVER["REQUEST_URI"];
 		} else {
-			$page_url .= $_SERVER["SERVER_NAME"] . $_SERVER["REQUEST_URI"];
+			$page_url .= $_SERVER["SERVER_NAME"].$_SERVER["REQUEST_URI"];
 		}
 	endif;
 
-	return apply_filters( 'give_get_current_page_url', esc_url( $page_url ) );
+	return apply_filters('give_get_current_page_url', esc_url($page_url));
 }
 
 
@@ -179,15 +179,15 @@ 
 	 */
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( count( $gateways ) == 1 && ! isset( $gateways['paypal'] ) && ! isset( $gateways['manual'] ) ) {
+	if (count($gateways) == 1 && ! isset($gateways['paypal']) && ! isset($gateways['manual'])) {
 		$ret = true;
-	} else if ( count( $gateways ) == 1 ) {
+	} else if (count($gateways) == 1) {
 		$ret = false;
-	} else if ( count( $gateways ) == 2 && isset( $gateways['paypal'] ) && isset( $gateways['manual'] ) ) {
+	} else if (count($gateways) == 2 && isset($gateways['paypal']) && isset($gateways['manual'])) {
 		$ret = false;
 	}
 
-	return (bool) apply_filters( 'give_verify_credit_cards', $ret );
+	return (bool) apply_filters('give_verify_credit_cards', $ret);
 }
 
 
@@ -202,9 +202,9 @@ 
 
 	global $give_options;
 
-	$ret = ! empty( $give_options['logged_in_only'] );
+	$ret = ! empty($give_options['logged_in_only']);
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret );
+	return (bool) apply_filters('give_logged_in_only', $ret);
 
 }
 
@@ -218,26 +218,26 @@ 
 function give_get_timezone_id() {
 
 	// if site timezone string exists, return it
-	if ( $timezone = get_option( 'timezone_string' ) ) {
+	if ($timezone = get_option('timezone_string')) {
 		return $timezone;
 	}
 
 	// get UTC offset, if it isn't set return UTC
-	if ( ! ( $utc_offset = 3600 * get_option( 'gmt_offset', 0 ) ) ) {
+	if ( ! ($utc_offset = 3600 * get_option('gmt_offset', 0))) {
 		return 'UTC';
 	}
 
 	// attempt to guess the timezone string from the UTC offset
-	$timezone = timezone_name_from_abbr( '', $utc_offset );
+	$timezone = timezone_name_from_abbr('', $utc_offset);
 
 	// last try, guess timezone string manually
-	if ( $timezone === false ) {
+	if ($timezone === false) {
 
-		$is_dst = date( 'I' );
+		$is_dst = date('I');
 
-		foreach ( timezone_abbreviations_list() as $abbr ) {
-			foreach ( $abbr as $city ) {
-				if ( $city['dst'] == $is_dst && $city['offset'] == $utc_offset ) {
+		foreach (timezone_abbreviations_list() as $abbr) {
+			foreach ($abbr as $city) {
+				if ($city['dst'] == $is_dst && $city['offset'] == $utc_offset) {
 					return $city['timezone_id'];
 				}
 			}
@@ -261,17 +261,17 @@ 
 
 	$ip = '127.0.0.1';
 
-	if ( ! empty( $_SERVER['HTTP_CLIENT_IP'] ) ) {
+	if ( ! empty($_SERVER['HTTP_CLIENT_IP'])) {
 		//check ip from share internet
 		$ip = $_SERVER['HTTP_CLIENT_IP'];
-	} elseif ( ! empty( $_SERVER['HTTP_X_FORWARDED_FOR'] ) ) {
+	} elseif ( ! empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
 		//to check ip is pass from proxy
 		$ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
-	} elseif ( ! empty( $_SERVER['REMOTE_ADDR'] ) ) {
+	} elseif ( ! empty($_SERVER['REMOTE_ADDR'])) {
 		$ip = $_SERVER['REMOTE_ADDR'];
 	}
 
-	return apply_filters( 'give_get_ip', $ip );
+	return apply_filters('give_get_ip', $ip);
 }
 
 
@@ -286,8 +286,8 @@ 
  *
  * @uses  Give()->session->set()
  */
-function give_set_purchase_session( $purchase_data = array() ) {
-	Give()->session->set( 'give_purchase', $purchase_data );
+function give_set_purchase_session($purchase_data = array()) {
+	Give()->session->set('give_purchase', $purchase_data);
 }
 
 /**
@@ -301,7 +301,7 @@ 
  * @return mixed array | false
  */
 function give_get_purchase_session() {
-	return Give()->session->get( 'give_purchase' );
+	return Give()->session->get('give_purchase');
 }
 
 /**
@@ -316,14 +316,14 @@ 
  *
  * @return string
  */
-function give_get_purchase_summary( $purchase_data, $email = true ) {
+function give_get_purchase_summary($purchase_data, $email = true) {
 	$summary = '';
 
-	if ( $email ) {
-		$summary .= $purchase_data['user_email'] . ' - ';
+	if ($email) {
+		$summary .= $purchase_data['user_email'].' - ';
 	}
 
-	$summary .= get_the_title( $purchase_data['post_data']['give-form-id'] );
+	$summary .= get_the_title($purchase_data['post_data']['give-form-id']);
 
 	return $summary;
 }
@@ -340,31 +340,31 @@ 
 function give_get_host() {
 	$host = false;
 
-	if ( defined( 'WPE_APIKEY' ) ) {
+	if (defined('WPE_APIKEY')) {
 		$host = 'WP Engine';
-	} elseif ( defined( 'PAGELYBIN' ) ) {
+	} elseif (defined('PAGELYBIN')) {
 		$host = 'Pagely';
-	} elseif ( DB_HOST == 'localhost:/tmp/mysql5.sock' ) {
+	} elseif (DB_HOST == 'localhost:/tmp/mysql5.sock') {
 		$host = 'ICDSoft';
-	} elseif ( DB_HOST == 'mysqlv5' ) {
+	} elseif (DB_HOST == 'mysqlv5') {
 		$host = 'NetworkSolutions';
-	} elseif ( strpos( DB_HOST, 'ipagemysql.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, 'ipagemysql.com') !== false) {
 		$host = 'iPage';
-	} elseif ( strpos( DB_HOST, 'ipowermysql.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, 'ipowermysql.com') !== false) {
 		$host = 'IPower';
-	} elseif ( strpos( DB_HOST, '.gridserver.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.gridserver.com') !== false) {
 		$host = 'MediaTemple Grid';
-	} elseif ( strpos( DB_HOST, '.pair.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.pair.com') !== false) {
 		$host = 'pair Networks';
-	} elseif ( strpos( DB_HOST, '.stabletransit.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.stabletransit.com') !== false) {
 		$host = 'Rackspace Cloud';
-	} elseif ( strpos( DB_HOST, '.sysfix.eu' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.sysfix.eu') !== false) {
 		$host = 'SysFix.eu Power Hosting';
-	} elseif ( strpos( $_SERVER['SERVER_NAME'], 'Flywheel' ) !== false ) {
+	} elseif (strpos($_SERVER['SERVER_NAME'], 'Flywheel') !== false) {
 		$host = 'Flywheel';
 	} else {
 		// Adding a general fallback for data gathering
-		$host = 'DBH: ' . DB_HOST . ', SRV: ' . $_SERVER['SERVER_NAME'];
+		$host = 'DBH: '.DB_HOST.', SRV: '.$_SERVER['SERVER_NAME'];
 	}
 
 	return $host;
@@ -380,67 +380,67 @@ 
  *
  * @return bool true if host matches, false if not
  */
-function give_is_host( $host = false ) {
+function give_is_host($host = false) {
 
 	$return = false;
 
-	if ( $host ) {
-		$host = str_replace( ' ', '', strtolower( $host ) );
+	if ($host) {
+		$host = str_replace(' ', '', strtolower($host));
 
-		switch ( $host ) {
+		switch ($host) {
 			case 'wpengine':
-				if ( defined( 'WPE_APIKEY' ) ) {
+				if (defined('WPE_APIKEY')) {
 					$return = true;
 				}
 				break;
 			case 'pagely':
-				if ( defined( 'PAGELYBIN' ) ) {
+				if (defined('PAGELYBIN')) {
 					$return = true;
 				}
 				break;
 			case 'icdsoft':
-				if ( DB_HOST == 'localhost:/tmp/mysql5.sock' ) {
+				if (DB_HOST == 'localhost:/tmp/mysql5.sock') {
 					$return = true;
 				}
 				break;
 			case 'networksolutions':
-				if ( DB_HOST == 'mysqlv5' ) {
+				if (DB_HOST == 'mysqlv5') {
 					$return = true;
 				}
 				break;
 			case 'ipage':
-				if ( strpos( DB_HOST, 'ipagemysql.com' ) !== false ) {
+				if (strpos(DB_HOST, 'ipagemysql.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'ipower':
-				if ( strpos( DB_HOST, 'ipowermysql.com' ) !== false ) {
+				if (strpos(DB_HOST, 'ipowermysql.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'mediatemplegrid':
-				if ( strpos( DB_HOST, '.gridserver.com' ) !== false ) {
+				if (strpos(DB_HOST, '.gridserver.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'pairnetworks':
-				if ( strpos( DB_HOST, '.pair.com' ) !== false ) {
+				if (strpos(DB_HOST, '.pair.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'rackspacecloud':
-				if ( strpos( DB_HOST, '.stabletransit.com' ) !== false ) {
+				if (strpos(DB_HOST, '.stabletransit.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'sysfix.eu':
 			case 'sysfix.eupowerhosting':
-				if ( strpos( DB_HOST, '.sysfix.eu' ) !== false ) {
+				if (strpos(DB_HOST, '.sysfix.eu') !== false) {
 					$return = true;
 				}
 				break;
 			case 'flywheel':
-				if ( strpos( $_SERVER['SERVER_NAME'], 'Flywheel' ) !== false ) {
+				if (strpos($_SERVER['SERVER_NAME'], 'Flywheel') !== false) {
 					$return = true;
 				}
 				break;
@@ -461,8 +461,8 @@ 
  * @return string $post_id
  */
 function give_get_admin_post_id() {
-	$post_id = isset( $_GET['post'] ) ? $_GET['post'] : null;
-	if ( ! $post_id && isset( $_POST['post_id'] ) ) {
+	$post_id = isset($_GET['post']) ? $_GET['post'] : null;
+	if ( ! $post_id && isset($_POST['post_id'])) {
 		$post_id = $_POST['post_id'];
 	}
 
@@ -480,11 +480,11 @@ 
  *
  * @return bool $ret True if guest checkout is enabled, false otherwise
  */
-function give_no_guest_checkout( $form_id ) {
+function give_no_guest_checkout($form_id) {
 
-	$ret = get_post_meta( $form_id, '_give_logged_in_only', true );
+	$ret = get_post_meta($form_id, '_give_logged_in_only', true);
 
-	return (bool) apply_filters( 'give_no_guest_checkout', $ret );
+	return (bool) apply_filters('give_no_guest_checkout', $ret);
 }
 
 
@@ -495,7 +495,7 @@ 
  * @return string Arg separator output
  */
 function give_get_php_arg_separator_output() {
-	return ini_get( 'arg_separator.output' );
+	return ini_get('arg_separator.output');
 }
 
 
@@ -510,10 +510,10 @@ 
  *
  * @return string Short month name
  */
-function give_month_num_to_name( $n ) {
-	$timestamp = mktime( 0, 0, 0, $n, 1, 2005 );
+function give_month_num_to_name($n) {
+	$timestamp = mktime(0, 0, 0, $n, 1, 2005);
 
-	return date_i18n( "M", $timestamp );
+	return date_i18n("M", $timestamp);
 }
 
 
@@ -526,10 +526,10 @@ 
  *
  * @return bool Whether or not function is disabled.
  */
-function give_is_func_disabled( $function ) {
-	$disabled = explode( ',', ini_get( 'disable_functions' ) );
+function give_is_func_disabled($function) {
+	$disabled = explode(',', ini_get('disable_functions'));
 
-	return in_array( $function, $disabled );
+	return in_array($function, $disabled);
 }
 
 
@@ -544,7 +544,7 @@ 
 
 		<form action="//givewp.us3.list-manage.com/subscribe/post?u=3ccb75d68bda4381e2f45794c&amp;id=12a081aa13" method="post" id="mc-embedded-subscribe-form" name="mc-embedded-subscribe-form" class="validate" target="_blank" novalidate>
 			<div class="give-newsletter-confirmation">
-				<p><?php _e( 'Thanks for Subscribing!', 'give' ); ?> :)</p>
+				<p><?php _e('Thanks for Subscribing!', 'give'); ?> :)</p>
 			</div>
 
 			<table class="form-table give-newsletter-form">
@@ -640,7 +640,7 @@ 
  *
  * @return mixed
  */
-function give_svg_icons( $icon ) {
+function give_svg_icons($icon) {
 
 	// Store your SVGs in an associative array
 	$svgs = array(
@@ -652,7 +652,7 @@ 
 	);
 
 	// Return the chosen icon's SVG string
-	return $svgs[ $icon ];
+	return $svgs[$icon];
 }
 
 /**
@@ -664,15 +664,15 @@ 
  *
  * @return mixed
  */
-function modify_nav_menu_meta_box_object( $post_type ) {
-	if ( isset( $post_type->name ) && $post_type->name == 'give_forms' ) {
+function modify_nav_menu_meta_box_object($post_type) {
+	if (isset($post_type->name) && $post_type->name == 'give_forms') {
 		$post_type->labels->name = 'Donation Forms';
 	}
 
 	return $post_type;
 }
 
-add_filter( 'nav_menu_meta_box_object', 'modify_nav_menu_meta_box_object' );
+add_filter('nav_menu_meta_box_object', 'modify_nav_menu_meta_box_object');
 
 
 /**
@@ -685,7 +685,7 @@ 
  * @license    http://opensource.org/licenses/MIT MIT
  */
 
-if ( ! function_exists( 'array_column' ) ) {
+if ( ! function_exists('array_column')) {
 	/**
 	 * Returns the values from a single column of the input array, identified by
 	 * the $columnKey.
@@ -704,56 +704,56 @@ 
 	 *
 	 * @return array
 	 */
-	function array_column( $input = null, $columnKey = null, $indexKey = null ) {
+	function array_column($input = null, $columnKey = null, $indexKey = null) {
 		// Using func_get_args() in order to check for proper number of
 		// parameters and trigger errors exactly as the built-in array_column()
 		// does in PHP 5.5.
 		$argc   = func_num_args();
 		$params = func_get_args();
 
-		if ( $argc < 2 ) {
-			trigger_error( "array_column() expects at least 2 parameters, {$argc} given", E_USER_WARNING );
+		if ($argc < 2) {
+			trigger_error("array_column() expects at least 2 parameters, {$argc} given", E_USER_WARNING);
 
 			return null;
 		}
 
-		if ( ! is_array( $params[0] ) ) {
+		if ( ! is_array($params[0])) {
 			trigger_error(
-				'array_column() expects parameter 1 to be array, ' . gettype( $params[0] ) . ' given',
+				'array_column() expects parameter 1 to be array, '.gettype($params[0]).' given',
 				E_USER_WARNING
 			);
 
 			return null;
 		}
 
-		if ( ! is_int( $params[1] )
-		     && ! is_float( $params[1] )
-		     && ! is_string( $params[1] )
+		if ( ! is_int($params[1])
+		     && ! is_float($params[1])
+		     && ! is_string($params[1])
 		     && $params[1] !== null
-		     && ! ( is_object( $params[1] ) && method_exists( $params[1], '__toString' ) )
+		     && ! (is_object($params[1]) && method_exists($params[1], '__toString'))
 		) {
-			trigger_error( 'array_column(): The column key should be either a string or an integer', E_USER_WARNING );
+			trigger_error('array_column(): The column key should be either a string or an integer', E_USER_WARNING);
 
 			return false;
 		}
 
-		if ( isset( $params[2] )
-		     && ! is_int( $params[2] )
-		     && ! is_float( $params[2] )
-		     && ! is_string( $params[2] )
-		     && ! ( is_object( $params[2] ) && method_exists( $params[2], '__toString' ) )
+		if (isset($params[2])
+		     && ! is_int($params[2])
+		     && ! is_float($params[2])
+		     && ! is_string($params[2])
+		     && ! (is_object($params[2]) && method_exists($params[2], '__toString'))
 		) {
-			trigger_error( 'array_column(): The index key should be either a string or an integer', E_USER_WARNING );
+			trigger_error('array_column(): The index key should be either a string or an integer', E_USER_WARNING);
 
 			return false;
 		}
 
 		$paramsInput     = $params[0];
-		$paramsColumnKey = ( $params[1] !== null ) ? (string) $params[1] : null;
+		$paramsColumnKey = ($params[1] !== null) ? (string) $params[1] : null;
 
 		$paramsIndexKey = null;
-		if ( isset( $params[2] ) ) {
-			if ( is_float( $params[2] ) || is_int( $params[2] ) ) {
+		if (isset($params[2])) {
+			if (is_float($params[2]) || is_int($params[2])) {
 				$paramsIndexKey = (int) $params[2];
 			} else {
 				$paramsIndexKey = (string) $params[2];
@@ -762,26 +762,26 @@ 
 
 		$resultArray = array();
 
-		foreach ( $paramsInput as $row ) {
+		foreach ($paramsInput as $row) {
 			$key    = $value = null;
 			$keySet = $valueSet = false;
 
-			if ( $paramsIndexKey !== null && array_key_exists( $paramsIndexKey, $row ) ) {
+			if ($paramsIndexKey !== null && array_key_exists($paramsIndexKey, $row)) {
 				$keySet = true;
-				$key    = (string) $row[ $paramsIndexKey ];
+				$key    = (string) $row[$paramsIndexKey];
 			}
 
-			if ( $paramsColumnKey === null ) {
+			if ($paramsColumnKey === null) {
 				$valueSet = true;
 				$value    = $row;
-			} elseif ( is_array( $row ) && array_key_exists( $paramsColumnKey, $row ) ) {
+			} elseif (is_array($row) && array_key_exists($paramsColumnKey, $row)) {
 				$valueSet = true;
-				$value    = $row[ $paramsColumnKey ];
+				$value    = $row[$paramsColumnKey];
 			}
 
-			if ( $valueSet ) {
-				if ( $keySet ) {
-					$resultArray[ $key ] = $value;
+			if ($valueSet) {
+				if ($keySet) {
+					$resultArray[$key] = $value;
 				} else {
 					$resultArray[] = $value;
 				}
@@ -803,39 +803,39 @@ 
  *
  * @return bool Whether the receipt is visible or not.
  */
-function give_can_view_receipt( $payment_key = '' ) {
+function give_can_view_receipt($payment_key = '') {
 
 	$return = false;
 
-	if ( empty( $payment_key ) ) {
+	if (empty($payment_key)) {
 		return $return;
 	}
 
 	global $give_receipt_args;
 
-	$give_receipt_args['id'] = give_get_purchase_id_by_key( $payment_key );
+	$give_receipt_args['id'] = give_get_purchase_id_by_key($payment_key);
 
-	$user_id = (int) give_get_payment_user_id( $give_receipt_args['id'] );
+	$user_id = (int) give_get_payment_user_id($give_receipt_args['id']);
 
-	$payment_meta = give_get_payment_meta( $give_receipt_args['id'] );
+	$payment_meta = give_get_payment_meta($give_receipt_args['id']);
 
-	if ( is_user_logged_in() ) {
-		if ( $user_id === (int) get_current_user_id() ) {
+	if (is_user_logged_in()) {
+		if ($user_id === (int) get_current_user_id()) {
 			$return = true;
-		} elseif ( wp_get_current_user()->user_email === give_get_payment_user_email( $give_receipt_args['id'] ) ) {
+		} elseif (wp_get_current_user()->user_email === give_get_payment_user_email($give_receipt_args['id'])) {
 			$return = true;
-		} elseif ( current_user_can( 'view_give_sensitive_data' ) ) {
+		} elseif (current_user_can('view_give_sensitive_data')) {
 			$return = true;
 		}
 	}
 
 	$session = give_get_purchase_session();
-	if ( ! empty( $session ) && ! is_user_logged_in() ) {
-		if ( $session['purchase_key'] === $payment_meta['key'] ) {
+	if ( ! empty($session) && ! is_user_logged_in()) {
+		if ($session['purchase_key'] === $payment_meta['key']) {
 			$return = true;
 		}
 	}
 
-	return (bool) apply_filters( 'give_can_view_receipt', $return, $payment_key );
+	return (bool) apply_filters('give_can_view_receipt', $return, $payment_key);
 
 }
\ No newline at end of file

includes/payments/class-donators-gravatars.php

Spacing +96 added lines, -96 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,50 +50,50 @@ 
 	 */
 
 	// https://gist.github.com/justinph/5197810
-	function validate_gravatar( $id_or_email ) {
+	function validate_gravatar($id_or_email) {
 		//id or email code borrowed from wp-includes/pluggable.php
 		$email = '';
-		if ( is_numeric( $id_or_email ) ) {
+		if (is_numeric($id_or_email)) {
 			$id   = (int) $id_or_email;
-			$user = get_userdata( $id );
-			if ( $user ) {
+			$user = get_userdata($id);
+			if ($user) {
 				$email = $user->user_email;
 			}
-		} elseif ( is_object( $id_or_email ) ) {
+		} elseif (is_object($id_or_email)) {
 			// No avatar for pingbacks or trackbacks
-			$allowed_comment_types = apply_filters( 'get_avatar_comment_types', array( 'comment' ) );
-			if ( ! empty( $id_or_email->comment_type ) && ! in_array( $id_or_email->comment_type, (array) $allowed_comment_types ) ) {
+			$allowed_comment_types = apply_filters('get_avatar_comment_types', array('comment'));
+			if ( ! empty($id_or_email->comment_type) && ! in_array($id_or_email->comment_type, (array) $allowed_comment_types)) {
 				return false;
 			}
 
-			if ( ! empty( $id_or_email->user_id ) ) {
+			if ( ! empty($id_or_email->user_id)) {
 				$id   = (int) $id_or_email->user_id;
-				$user = get_userdata( $id );
-				if ( $user ) {
+				$user = get_userdata($id);
+				if ($user) {
 					$email = $user->user_email;
 				}
-			} elseif ( ! empty( $id_or_email->comment_author_email ) ) {
+			} elseif ( ! empty($id_or_email->comment_author_email)) {
 				$email = $id_or_email->comment_author_email;
 			}
 		} else {
 			$email = $id_or_email;
 		}
 
-		$hashkey = md5( strtolower( trim( $email ) ) );
-		$uri     = 'http://www.gravatar.com/avatar/' . $hashkey . '?d=404';
+		$hashkey = md5(strtolower(trim($email)));
+		$uri     = 'http://www.gravatar.com/avatar/'.$hashkey.'?d=404';
 
-		$data = wp_cache_get( $hashkey );
-		if ( false === $data ) {
-			$response = wp_remote_head( $uri );
-			if ( is_wp_error( $response ) ) {
+		$data = wp_cache_get($hashkey);
+		if (false === $data) {
+			$response = wp_remote_head($uri);
+			if (is_wp_error($response)) {
 				$data = 'not200';
 			} else {
 				$data = $response['response']['code'];
 			}
-			wp_cache_set( $hashkey, $data, $group = '', $expire = 60 * 5 );
+			wp_cache_set($hashkey, $data, $group = '', $expire = 60 * 5);
 
 		}
-		if ( $data == '200' ) {
+		if ($data == '200') {
 			return true;
 		} else {
 			return false;
@@ -108,17 +108,17 @@ 
 	 *
 	 * @param int $form_id
 	 */
-	function get_log_ids( $form_id = '' ) {
+	function get_log_ids($form_id = '') {
 
 		// get Give_Logging class
 		global $give_logs;
 
 		// get log for this form
-		$logs = $give_logs->get_logs( $form_id );
+		$logs = $give_logs->get_logs($form_id);
 
-		if ( $logs ) {
+		if ($logs) {
 			// make an array with all the donor IDs
-			foreach ( $logs as $log ) {
+			foreach ($logs as $log) {
 				$log_ids[] = $log->ID;
 			}
 
@@ -135,49 +135,49 @@ 
 	 *
 	 * @since 1.0
 	 */
-	function get_payment_ids( $form_id = '' ) {
+	function get_payment_ids($form_id = '') {
 
 		global $give_options;
 
-		$log_ids = $this->get_log_ids( $form_id );
+		$log_ids = $this->get_log_ids($form_id);
 
-		if ( $log_ids ) {
+		if ($log_ids) {
 
 			$payment_ids = array();
 
-			foreach ( $log_ids as $id ) {
+			foreach ($log_ids as $id) {
 				// get the payment ID for each corresponding log ID
-				$payment_ids[] = get_post_meta( $id, '_give_log_payment_id', true );
+				$payment_ids[] = get_post_meta($id, '_give_log_payment_id', true);
 			}
 
 			// remove donors who have purchased more than once so we can have unique avatars
 			$unique_emails = array();
 
-			foreach ( $payment_ids as $key => $id ) {
+			foreach ($payment_ids as $key => $id) {
 
-				$email = get_post_meta( $id, '_give_payment_user_email', true );
+				$email = get_post_meta($id, '_give_payment_user_email', true);
 
-				if ( isset ( $give_options['give_donators_gravatars_has_gravatar_account'] ) ) {
-					if ( ! $this->validate_gravatar( $email ) ) {
+				if (isset ($give_options['give_donators_gravatars_has_gravatar_account'])) {
+					if ( ! $this->validate_gravatar($email)) {
 						continue;
 					}
 				}
 
-				$unique_emails[ $id ] = get_post_meta( $id, '_give_payment_user_email', true );
+				$unique_emails[$id] = get_post_meta($id, '_give_payment_user_email', true);
 
 			}
 
 			// strip duplicate emails
-			$unique_emails = array_unique( $unique_emails );
+			$unique_emails = array_unique($unique_emails);
 
 			// convert the unique IDs back into simple array
-			foreach ( $unique_emails as $id => $email ) {
+			foreach ($unique_emails as $id => $email) {
 				$unique_ids[] = $id;
 			}
 
 			// randomize the payment IDs if enabled
-			if ( isset( $give_options['give_donators_gravatars_random_gravatars'] ) ) {
-				shuffle( $unique_ids );
+			if (isset($give_options['give_donators_gravatars_random_gravatars'])) {
+				shuffle($unique_ids);
 			}
 
 			// return our unique IDs
@@ -193,10 +193,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	function gravatars( $form_id = false, $title = '' ) {
+	function gravatars($form_id = false, $title = '') {
 
 		// unique $payment_ids 
-		$payment_ids = $this->get_payment_ids( $form_id );
+		$payment_ids = $this->get_payment_ids($form_id);
 
 		//			var_dump( $payment_ids );
 		//			 var_dump( $this->get_log_ids( get_the_ID() ) );
@@ -204,14 +204,14 @@ 
 		global $give_options;
 
 		// return if no ID
-		if ( ! $form_id ) {
+		if ( ! $form_id) {
 			return;
 		}
 
 		// minimum amount of purchases before showing gravatars
 		// if the number of items in array is not greater or equal to the number specified, then exit
-		if ( isset( $give_options['give_donators_gravatars_min_purchases_required'] ) && '' != $give_options['give_donators_gravatars_min_purchases_required'] ) {
-			if ( ! ( count( $payment_ids ) >= $give_options['give_donators_gravatars_min_purchases_required'] ) ) {
+		if (isset($give_options['give_donators_gravatars_min_purchases_required']) && '' != $give_options['give_donators_gravatars_min_purchases_required']) {
+			if ( ! (count($payment_ids) >= $give_options['give_donators_gravatars_min_purchases_required'])) {
 				return;
 			}
 		}
@@ -222,51 +222,51 @@ 
 		echo '<div id="give-purchase-gravatars">';
 
 
-		if ( isset ( $title ) ) {
+		if (isset ($title)) {
 
-			if ( $title ) {
-				echo apply_filters( 'give_donators_gravatars_title', '<h3 class="give-gravatars-title">' . esc_attr( $title ) . '</h3>' );
-			} elseif ( isset( $give_options['give_donators_gravatars_heading'] ) ) {
-				echo apply_filters( 'give_donators_gravatars_title', '<h3 class="give-gravatars-title">' . esc_attr( $give_options['give_donators_gravatars_heading'] ) . '</h2>' );
+			if ($title) {
+				echo apply_filters('give_donators_gravatars_title', '<h3 class="give-gravatars-title">'.esc_attr($title).'</h3>');
+			} elseif (isset($give_options['give_donators_gravatars_heading'])) {
+				echo apply_filters('give_donators_gravatars_title', '<h3 class="give-gravatars-title">'.esc_attr($give_options['give_donators_gravatars_heading']).'</h2>');
 			}
 
 		}
 		echo '<ul class="give-purchase-gravatars-list">';
 		$i = 0;
 
-		if ( $payment_ids ) {
-			foreach ( $payment_ids as $id ) {
+		if ($payment_ids) {
+			foreach ($payment_ids as $id) {
 
 				// Give saves a blank option even when the control is turned off, hence the extra check
-				if ( isset( $give_options['give_donators_gravatars_maximum_number'] ) && '' != $give_options['give_donators_gravatars_maximum_number'] && $i == $give_options['give_donators_gravatars_maximum_number'] ) {
+				if (isset($give_options['give_donators_gravatars_maximum_number']) && '' != $give_options['give_donators_gravatars_maximum_number'] && $i == $give_options['give_donators_gravatars_maximum_number']) {
 					continue;
 				}
 
 				// get the payment meta
-				$payment_meta = get_post_meta( $id, '_give_payment_meta', true );
+				$payment_meta = get_post_meta($id, '_give_payment_meta', true);
 
 				// unserialize the payment meta
-				$user_info = maybe_unserialize( $payment_meta['user_info'] );
+				$user_info = maybe_unserialize($payment_meta['user_info']);
 
 				// get donor's first name
 				$name = $user_info['first_name'];
 
 				// get donor's email
-				$email = get_post_meta( $id, '_give_payment_user_email', true );
+				$email = get_post_meta($id, '_give_payment_user_email', true);
 
 				// set gravatar size and provide filter
-				$size = isset( $give_options['give_donators_gravatars_gravatar_size'] ) ? apply_filters( 'give_donators_gravatars_gravatar_size', $give_options['give_donators_gravatars_gravatar_size'] ) : '';
+				$size = isset($give_options['give_donators_gravatars_gravatar_size']) ? apply_filters('give_donators_gravatars_gravatar_size', $give_options['give_donators_gravatars_gravatar_size']) : '';
 
 				// default image
-				$default_image = apply_filters( 'give_donators_gravatars_gravatar_default_image', false );
+				$default_image = apply_filters('give_donators_gravatars_gravatar_default_image', false);
 
 				// assemble output
 				$output .= '<li>';
 
-				$output .= get_avatar( $email, $size, $default_image, $name );
+				$output .= get_avatar($email, $size, $default_image, $name);
 				$output .= '</li>';
 
-				$i ++;
+				$i++;
 
 			} // end foreach
 		}
@@ -275,7 +275,7 @@ 
 		echo '</ul>';
 		echo '</div>';
 
-		return apply_filters( 'give_donators_gravatars', ob_get_clean() );
+		return apply_filters('give_donators_gravatars', ob_get_clean());
 	}
 
 	/**
@@ -284,7 +284,7 @@ 
 	 * @since 1.0
 	 */
 	function register_widget() {
-		register_widget( 'Give_Donators_Gravatars_Widget' );
+		register_widget('Give_Donators_Gravatars_Widget');
 	}
 
 	/**
@@ -293,19 +293,19 @@ 
 	 * @since 1.0
 	 * @todo  set the ID to get_the_ID() if ID parameter is not passed through. Otherwise it will incorrectly get other gravatars
 	 */
-	function shortcode( $atts, $content = null ) {
+	function shortcode($atts, $content = null) {
 
-		$atts = shortcode_atts( array(
+		$atts = shortcode_atts(array(
 			'id'    => '',
 			'title' => ''
-		), $atts, 'give_donators_gravatars' );
+		), $atts, 'give_donators_gravatars');
 
 		// if no ID is passed on single give_forms pages, get the correct ID
-		if ( is_singular( 'give_forms' ) ) {
+		if (is_singular('give_forms')) {
 			$id = get_the_ID();
 		}
 
-		$content = $this->gravatars( $atts['id'], $atts['title'] );
+		$content = $this->gravatars($atts['id'], $atts['title']);
 
 		return $content;
 
@@ -316,56 +316,56 @@ 
 	 *
 	 * @since 1.0
 	 */
-	function settings( $settings ) {
+	function settings($settings) {
 
 		$give_gravatar_settings = array(
 			array(
-				'name' => __( 'Donator Gravatars', 'give' ),
+				'name' => __('Donator Gravatars', 'give'),
 				'desc' => '<hr>',
 				'id'   => 'give_title',
 				'type' => 'give_title'
 			),
 			array(
-				'name' => __( 'Heading', 'give' ),
-				'desc' => __( 'The heading to display above the Gravatars', 'give' ),
+				'name' => __('Heading', 'give'),
+				'desc' => __('The heading to display above the Gravatars', 'give'),
 				'type' => 'text',
 				'id'   => 'give_donators_gravatars_heading'
 			),
 			array(
-				'name'    => __( 'Gravatar Size', 'give' ),
-				'desc'    => __( 'The size of each Gravatar in pixels (512px maximum)', 'give' ),
+				'name'    => __('Gravatar Size', 'give'),
+				'desc'    => __('The size of each Gravatar in pixels (512px maximum)', 'give'),
 				'type'    => 'text_small',
 				'id'      => 'give_donators_gravatars_gravatar_size',
 				'default' => '64'
 			),
 			array(
-				'name' => __( 'Minimum Unique Purchases Required', 'give' ),
-				'desc' => sprintf( __( 'The minimum number of unique purchases a %s must have before the Gravatars are shown. Leave blank for no minimum.', 'give' ), strtolower( give_get_forms_label_singular() ) ),
+				'name' => __('Minimum Unique Purchases Required', 'give'),
+				'desc' => sprintf(__('The minimum number of unique purchases a %s must have before the Gravatars are shown. Leave blank for no minimum.', 'give'), strtolower(give_get_forms_label_singular())),
 				'type' => 'text_small',
 				'id'   => 'give_donators_gravatars_min_purchases_required',
 			),
 			array(
-				'name'    => __( 'Maximum Gravatars To Show', 'give' ),
-				'desc'    => __( 'The maximum number of gravatars to show. Leave blank for no limit.', 'give' ),
+				'name'    => __('Maximum Gravatars To Show', 'give'),
+				'desc'    => __('The maximum number of gravatars to show. Leave blank for no limit.', 'give'),
 				'type'    => 'text',
 				'id'      => 'give_donators_gravatars_maximum_number',
 				'default' => '20',
 			),
 			array(
-				'name' => __( 'Gravatar Visibility', 'give' ),
-				'desc' => __( 'Only show donators with a Gravatar account', 'give' ),
+				'name' => __('Gravatar Visibility', 'give'),
+				'desc' => __('Only show donators with a Gravatar account', 'give'),
 				'id'   => 'give_donators_gravatars_has_gravatar_account',
 				'type' => 'checkbox',
 			),
 			array(
-				'name' => __( 'Randomize Gravatars', 'give' ),
-				'desc' => __( 'Randomize the Gravatars', 'give' ),
+				'name' => __('Randomize Gravatars', 'give'),
+				'desc' => __('Randomize the Gravatars', 'give'),
 				'id'   => 'give_donators_gravatars_random_gravatars',
 				'type' => 'checkbox',
 			),
 		);
 
-		return array_merge( $settings, $give_gravatar_settings );
+		return array_merge($settings, $give_gravatar_settings);
 	}
 
 }
@@ -383,12 +383,12 @@ 
 	 */
 	function give_donators_gravatars_widget() {
 
-		$give_label_singular = function_exists( 'give_get_forms_label_singular' ) ? strtolower( give_get_forms_label_singular() ) : null;
+		$give_label_singular = function_exists('give_get_forms_label_singular') ? strtolower(give_get_forms_label_singular()) : null;
 
 		// widget settings
 		$widget_ops = array(
 			'classname'   => 'give-donators-gravatars',
-			'description' => sprintf( __( 'Displays gravatars of people who have donated using your your %s. Will only show on the single %s page.', 'give' ), $give_label_singular, $give_label_singular )
+			'description' => sprintf(__('Displays gravatars of people who have donated using your your %s. Will only show on the single %s page.', 'give'), $give_label_singular, $give_label_singular)
 		);
 
 		// widget control settings
@@ -401,7 +401,7 @@ 
 		// create the widget
 		$this->WP_Widget(
 			'give_donators_gravatars_widget',
-			__( 'Give Donators Gravatars', 'give' ),
+			__('Give Donators Gravatars', 'give'),
 			$widget_ops,
 			$control_ops
 		);
@@ -412,29 +412,29 @@ 
 	/*
 	 * Outputs the content of the widget
 	 */
-	function widget( $args, $instance ) {
+	function widget($args, $instance) {
 		global $give_options;
 
-		extract( $args );
+		extract($args);
 
-		if ( ! is_singular( 'give_forms' ) ) {
+		if ( ! is_singular('give_forms')) {
 			return;
 		}
 
 		// Variables from widget settings
-		$title = apply_filters( 'widget_title', $instance['title'] );
+		$title = apply_filters('widget_title', $instance['title']);
 
 		// Used by themes. Opens the widget
 		echo $before_widget;
 
 		// Display the widget title
-		if ( $title ) {
-			echo $before_title . $title . $after_title;
+		if ($title) {
+			echo $before_title.$title.$after_title;
 		}
 
 		$gravatars = new Give_Donators_Gravatars();
 
-		echo $gravatars->gravatars( get_the_ID(), null ); // remove title
+		echo $gravatars->gravatars(get_the_ID(), null); // remove title
 
 		// Used by themes. Closes the widget
 		echo $after_widget;
@@ -444,11 +444,11 @@ 
 	/*
 	 * Update function. Processes widget options to be saved
 	 */
-	function update( $new_instance, $old_instance ) {
+	function update($new_instance, $old_instance) {
 
 		$instance = $old_instance;
 
-		$instance['title'] = strip_tags( $new_instance['title'] );
+		$instance['title'] = strip_tags($new_instance['title']);
 
 		return $instance;
 
@@ -457,19 +457,19 @@ 
 	/*
 	 * Form function. Displays the actual form on the widget page
 	 */
-	function form( $instance ) {
+	function form($instance) {
 
 		// Set up some default widget settings.
 		$defaults = array(
 			'title' => '',
 		);
 
-		$instance = wp_parse_args( (array) $instance, $defaults ); ?>
+		$instance = wp_parse_args((array) $instance, $defaults); ?>
 
 		<!-- Title -->
 		<p>
-			<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'give' ) ?></label>
-			<input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo $instance['title']; ?>" />
+			<label for="<?php echo $this->get_field_id('title'); ?>"><?php _e('Title:', 'give') ?></label>
+			<input class="widefat" id="<?php echo $this->get_field_id('title'); ?>" name="<?php echo $this->get_field_name('title'); ?>" type="text" value="<?php echo $instance['title']; ?>" />
 		</p>
 
 

includes/payments/class-payment-stats.php

Spacing +44 added lines, -44 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,46 +39,46 @@ 
 	 *
 	 * @return float|int
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global sale stats
-			add_filter( 'give_count_payments_where', array( $this, 'count_where' ) );
+			add_filter('give_count_payments_where', array($this, 'count_where'));
 
-			if ( is_array( $status ) ) {
+			if (is_array($status)) {
 				$count = 0;
-				foreach ( $status as $payment_status ) {
+				foreach ($status as $payment_status) {
 					$count += give_count_payments()->$payment_status;
 				}
 			} else {
 				$count = give_count_payments()->$status;
 			}
 
-			remove_filter( 'give_count_payments_where', array( $this, 'count_where' ) );
+			remove_filter('give_count_payments_where', array($this, 'count_where'));
 
 		} else {
 
 			// Product specific stats
 			global $give_logs;
 
-			add_filter( 'posts_where', array( $this, 'payments_where' ) );
+			add_filter('posts_where', array($this, 'payments_where'));
 
-			$count = $give_logs->get_log_count( $form_id, 'sale' );
+			$count = $give_logs->get_log_count($form_id, 'sale');
 
-			remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+			remove_filter('posts_where', array($this, 'payments_where'));
 
 		}
 
@@ -99,31 +99,31 @@ 
 	 *
 	 * @return float|int
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false ) {
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false) {
 
 		global $wpdb;
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global earning stats
 			$args = array(
 				'post_type'              => 'give_payment',
 				'nopaging'               => true,
-				'post_status'            => array( 'publish', 'revoked' ),
+				'post_status'            => array('publish', 'revoked'),
 				'fields'                 => 'ids',
 				'update_post_term_cache' => false,
 				'suppress_filters'       => false,
@@ -134,19 +134,19 @@ 
 				// This is not a valid query arg, but is used for cache keying
 			);
 
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-			$earnings = get_transient( $key );
-			if ( false === $earnings ) {
-				$sales    = get_posts( $args );
+			$earnings = get_transient($key);
+			if (false === $earnings) {
+				$sales    = get_posts($args);
 				$earnings = 0;
-				if ( $sales ) {
-					$sales = implode( ',', $sales );
-					$earnings += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})" );
+				if ($sales) {
+					$sales = implode(',', $sales);
+					$earnings += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})");
 				}
 				// Cache the results for one hour
-				set_transient( $key, $earnings, HOUR_IN_SECONDS );
+				set_transient($key, $earnings, HOUR_IN_SECONDS);
 			}
 
 		} else {
@@ -167,33 +167,33 @@ 
 				// This is not a valid query arg, but is used for cache keying
 			);
 
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 			//Set transient for faster stats
-			$earnings = get_transient( $key );
+			$earnings = get_transient($key);
 
-			if ( false === $earnings ) {
+			if (false === $earnings) {
 
-				$log_ids  = $give_logs->get_connected_logs( $args, 'sale' );
+				$log_ids  = $give_logs->get_connected_logs($args, 'sale');
 				$earnings = 0;
 
-				if ( $log_ids ) {
-					$log_ids     = implode( ',', $log_ids );
-					$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids);" );
-					foreach ( $payment_ids as $payment_id ) {
+				if ($log_ids) {
+					$log_ids     = implode(',', $log_ids);
+					$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids);");
+					foreach ($payment_ids as $payment_id) {
 						$earnings += give_get_payment_amount($payment_id);
 					}
 				}
 
 				// Cache the results for one hour
-				set_transient( $key, $earnings, 60 * 60 );
+				set_transient($key, $earnings, 60 * 60);
 			}
 		}
 
 		//remove our filter
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 		//return earnings
-		return round( $earnings, give_currency_decimal_filter() );
+		return round($earnings, give_currency_decimal_filter());
 
 	}
 
@@ -207,16 +207,16 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 
 		global $wpdb;
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT post_id as form_id, max(meta_value) as sales
 				FROM $wpdb->postmeta WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}

includes/payments/class-payments-query.php

Spacing +116 added lines, -116 removed lines

@@ -50,10 +50,10 @@ 
 	 *
 	 * @param $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments', // Use 'posts' to get standard post objects
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -73,7 +73,7 @@ 
 			'give_forms'      => null
 		);
 
-		$this->args = wp_parse_args( $args, $defaults );
+		$this->args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -84,11 +84,11 @@ 
 	 * @access public
 	 * @since  1.0
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -98,8 +98,8 @@ 
 	 * @access public
 	 * @since  1.0
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -111,19 +111,19 @@ 
 	 */
 	public function init() {
 
-		add_action( 'give_pre_get_payments', array( $this, 'date_filter_pre' ) );
-		add_action( 'give_post_get_payments', array( $this, 'date_filter_post' ) );
-
-		add_action( 'give_pre_get_payments', array( $this, 'orderby' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'status' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'month' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'per_page' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'page' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'user' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'search' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'mode' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'children' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'give_forms' ) );
+		add_action('give_pre_get_payments', array($this, 'date_filter_pre'));
+		add_action('give_post_get_payments', array($this, 'date_filter_post'));
+
+		add_action('give_pre_get_payments', array($this, 'orderby'));
+		add_action('give_pre_get_payments', array($this, 'status'));
+		add_action('give_pre_get_payments', array($this, 'month'));
+		add_action('give_pre_get_payments', array($this, 'per_page'));
+		add_action('give_pre_get_payments', array($this, 'page'));
+		add_action('give_pre_get_payments', array($this, 'user'));
+		add_action('give_pre_get_payments', array($this, 'search'));
+		add_action('give_pre_get_payments', array($this, 'mode'));
+		add_action('give_pre_get_payments', array($this, 'children'));
+		add_action('give_pre_get_payments', array($this, 'give_forms'));
 	}
 
 	/**
@@ -139,16 +139,16 @@ 
 	 */
 	public function get_payments() {
 
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 
-		$query = new WP_Query( $this->args );
+		$query = new WP_Query($this->args);
 
-		if ( 'payments' != $this->args['output'] ) {
+		if ('payments' != $this->args['output']) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
-			while ( $query->have_posts() ) {
+		if ($query->have_posts()) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$details = new stdClass;
@@ -158,23 +158,23 @@ 
 				$details->ID          = $payment_id;
 				$details->date        = get_post()->post_date;
 				$details->post_status = get_post()->post_status;
-				$details->total       = give_get_payment_amount( $payment_id );
-				$details->fees        = give_get_payment_fees( $payment_id );
-				$details->key         = give_get_payment_key( $payment_id );
-				$details->gateway     = give_get_payment_gateway( $payment_id );
-				$details->user_info   = give_get_payment_meta_user_info( $payment_id );
-
-				if ( give_get_option( 'enable_sequential' ) ) {
-					$details->payment_number = give_get_payment_number( $payment_id );
+				$details->total       = give_get_payment_amount($payment_id);
+				$details->fees        = give_get_payment_fees($payment_id);
+				$details->key         = give_get_payment_key($payment_id);
+				$details->gateway     = give_get_payment_gateway($payment_id);
+				$details->user_info   = give_get_payment_meta_user_info($payment_id);
+
+				if (give_get_option('enable_sequential')) {
+					$details->payment_number = give_get_payment_number($payment_id);
 				}
 
-				$this->payments[] = apply_filters( 'give_payment', $details, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $details, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 		}
 
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 
 		return $this->payments;
 	}
@@ -187,13 +187,13 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -205,11 +205,11 @@ 
 	 * @return void
 	 */
 	public function date_filter_post() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -220,12 +220,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset ( $this->args['status'] ) ) {
+		if ( ! isset ($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -236,12 +236,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset ( $this->args['page'] ) ) {
+		if ( ! isset ($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -253,17 +253,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -274,12 +274,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset ( $this->args['month'] ) ) {
+		if ( ! isset ($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -290,13 +290,13 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount' :
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 			default :
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -309,20 +309,20 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_user_id';
 		} else {
 			$user_key = '_give_payment_user_email';
 		}
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => $user_key,
 			'value' => $this->args['user']
-		) );
+		));
 	}
 
 	/**
@@ -334,33 +334,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_user_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -369,19 +369,19 @@ 
 				'compare' => 'LIKE'
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_user_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) )
+				'value' => trim(str_replace('user:', '', strtolower($search)))
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			if ( give_get_option( 'enable_sequential' ) ) {
+			if (give_get_option('enable_sequential')) {
 
 				$search_meta = array(
 					'key'     => '_give_payment_number',
@@ -389,19 +389,19 @@ 
 					'compare' => 'LIKE'
 				);
 
-				$this->__set( 'meta_query', $search_meta );
+				$this->__set('meta_query', $search_meta);
 
 				$this->args['meta_query']['relation'] = 'OR';
 
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
 		} elseif (
-			give_get_option( 'enable_sequential' ) &&
+			give_get_option('enable_sequential') &&
 			(
-				false !== strpos( $search, give_get_option( 'sequential_prefix' ) ) ||
-				false !== strpos( $search, give_get_option( 'sequential_postfix' ) )
+				false !== strpos($search, give_get_option('sequential_prefix')) ||
+				false !== strpos($search, give_get_option('sequential_postfix'))
 			)
 		) {
 
@@ -411,28 +411,28 @@ 
 				'compare' => 'LIKE'
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
 
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$this->__set( 'give_forms', str_replace( '#', '', $search ) );
-			$this->__unset( 's' );
+			$this->__set('give_forms', str_replace('#', '', $search));
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -446,16 +446,16 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => '_give_payment_mode',
 			'value' => $this->args['mode']
-		) );
+		));
 	}
 
 	/**
@@ -466,10 +466,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -481,7 +481,7 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
@@ -490,7 +490,7 @@ 
 		$args = array(
 			//			'post_id'                => $this->args['give_forms'],
 			'log_type'               => 'sale',
-			'post_status'            => array( 'publish' ),
+			'post_status'            => array('publish'),
 			'nopaging'               => true,
 			'no_found_rows'          => true,
 			'update_post_term_cache' => false,
@@ -499,31 +499,31 @@ 
 			'fields'                 => 'ids'
 		);
 
-		if ( is_array( $this->args['give_forms'] ) ) {
-			unset( $args['post_parent'] );
+		if (is_array($this->args['give_forms'])) {
+			unset($args['post_parent']);
 			$args['post_parent__in'] = $this->args['give_forms'];
 		}
 
-		$sales = $give_logs->get_connected_logs( $args );
+		$sales = $give_logs->get_connected_logs($args);
 
-		if ( ! empty( $sales ) ) {
+		if ( ! empty($sales)) {
 
 			$payments = array();
 
-			foreach ( $sales as $sale ) {
-				$payments[] = get_post_meta( $sale, '_give_log_payment_id', true );
+			foreach ($sales as $sale) {
+				$payments[] = get_post_meta($sale, '_give_log_payment_id', true);
 			}
 
-			$this->__set( 'post__in', $payments );
+			$this->__set('post__in', $payments);
 
 		} else {
 
 			// Set post_parent to something crazy so it doesn't find anything
-			$this->__set( 'post_parent', 999999999999999 );
+			$this->__set('post_parent', 999999999999999);
 
 		}
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 }

includes/post-types.php

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * Post Type Functions
- *
- * @package     Give
- * @subpackage  Functions
- * @copyright   Copyright (c) 2015, WordImpress
- * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
- * @since       1.0
- */
+	 * Post Type Functions
+	 *
+	 * @package     Give
+	 * @subpackage  Functions
+	 * @copyright   Copyright (c) 2015, WordImpress
+	 * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
+	 * @since       1.0
+	 */
 
 // Exit if accessed directly
 if ( ! defined( 'ABSPATH' ) ) {

Spacing +162 added lines, -162 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,40 +23,40 @@ 
 function give_setup_post_types() {
 
 	/** Give Forms Post Type */
-	$give_forms_singular = give_get_option( 'disable_forms_singular' ) !== 'on' ? true : false;
+	$give_forms_singular = give_get_option('disable_forms_singular') !== 'on' ? true : false;
 
-	$give_forms_archives = give_get_option( 'disable_forms_archives' ) !== 'on' ? true : false;
+	$give_forms_archives = give_get_option('disable_forms_archives') !== 'on' ? true : false;
 
-	$give_forms_slug = defined( 'GIVE_SLUG' ) ? GIVE_SLUG : 'donations';
+	$give_forms_slug = defined('GIVE_SLUG') ? GIVE_SLUG : 'donations';
 	//support for old 'GIVE_FORMS_SLUG' constant
-	if ( defined( 'GIVE_FORMS_SLUG' ) ) {
+	if (defined('GIVE_FORMS_SLUG')) {
 		$give_forms_slug = GIVE_FORMS_SLUG;
 	}
 
-	$give_forms_rewrite = defined( 'GIVE_DISABLE_FORMS_REWRITE' ) && GIVE_DISABLE_FORMS_REWRITE ? false : array(
+	$give_forms_rewrite = defined('GIVE_DISABLE_FORMS_REWRITE') && GIVE_DISABLE_FORMS_REWRITE ? false : array(
 		'slug'       => $give_forms_slug,
 		'with_front' => false
 	);
 
-	$give_forms_labels = apply_filters( 'give_forms_labels', array(
-		'name'               => __( 'Donation %2$s', 'give' ),
+	$give_forms_labels = apply_filters('give_forms_labels', array(
+		'name'               => __('Donation %2$s', 'give'),
 		'singular_name'      => '%1$s',
-		'add_new'            => __( 'Add %1$s', 'give' ),
-		'add_new_item'       => __( 'Add New Donation %1$s', 'give' ),
-		'edit_item'          => __( 'Edit Donation %1$s', 'give' ),
-		'new_item'           => __( 'New %1$s', 'give' ),
-		'all_items'          => __( 'All %2$s', 'give' ),
-		'view_item'          => __( 'View %1$s', 'give' ),
-		'search_items'       => __( 'Search %2$s', 'give' ),
-		'not_found'          => __( 'No %2$s found', 'give' ),
-		'not_found_in_trash' => __( 'No %2$s found in Trash', 'give' ),
+		'add_new'            => __('Add %1$s', 'give'),
+		'add_new_item'       => __('Add New Donation %1$s', 'give'),
+		'edit_item'          => __('Edit Donation %1$s', 'give'),
+		'new_item'           => __('New %1$s', 'give'),
+		'all_items'          => __('All %2$s', 'give'),
+		'view_item'          => __('View %1$s', 'give'),
+		'search_items'       => __('Search %2$s', 'give'),
+		'not_found'          => __('No %2$s found', 'give'),
+		'not_found_in_trash' => __('No %2$s found in Trash', 'give'),
 		'parent_item_colon'  => '',
-		'menu_name'          => apply_filters( 'give_menu_name', __( 'Donations', 'give' ) ),
-		'name_admin_bar'     => apply_filters( 'give_name_admin_bar_name', __( 'Donation Form', 'give' ) )
-	) );
+		'menu_name'          => apply_filters('give_menu_name', __('Donations', 'give')),
+		'name_admin_bar'     => apply_filters('give_name_admin_bar_name', __('Donation Form', 'give'))
+	));
 
-	foreach ( $give_forms_labels as $key => $value ) {
-		$give_forms_labels[ $key ] = sprintf( $value, give_get_forms_label_singular(), give_get_forms_label_plural() );
+	foreach ($give_forms_labels as $key => $value) {
+		$give_forms_labels[$key] = sprintf($value, give_get_forms_label_singular(), give_get_forms_label_plural());
 	}
 
 	//Default give_forms supports
@@ -69,14 +69,14 @@ 
 	);
 
 	//Has the user disabled the excerpt
-	if ( give_get_option( 'disable_forms_excerpt' ) === 'on' ) {
-		unset( $give_form_supports[2] );
+	if (give_get_option('disable_forms_excerpt') === 'on') {
+		unset($give_form_supports[2]);
 	}
 
 	//Has user disabled the featured image?
-	if ( give_get_option( 'disable_form_featured_img' ) === 'on' ) {
-		unset( $give_form_supports[1] );
-		remove_action( 'give_before_single_form_summary', 'give_show_form_images' );
+	if (give_get_option('disable_form_featured_img') === 'on') {
+		unset($give_form_supports[1]);
+		remove_action('give_before_single_form_summary', 'give_show_form_images');
 	}
 
 	$give_forms_args = array(
@@ -92,36 +92,36 @@ 
 		'has_archive'        => $give_forms_archives,
 		'menu_icon'          => 'dashicons-give',
 		'hierarchical'       => false,
-		'supports'           => apply_filters( 'give_forms_supports', $give_form_supports ),
+		'supports'           => apply_filters('give_forms_supports', $give_form_supports),
 	);
-	register_post_type( 'give_forms', apply_filters( 'give_forms_post_type_args', $give_forms_args ) );
+	register_post_type('give_forms', apply_filters('give_forms_post_type_args', $give_forms_args));
 
 
 	/** Give Campaigns Post Type */
-	$give_campaigns_archives = defined( 'GIVE_DISABLE_CAMPAIGNS_ARCHIVE' ) && GIVE_DISABLE_CAMPAIGNS_ARCHIVE ? false : true;
-	$give_campaigns_slug     = defined( 'GIVE_CAMPAIGNS_SLUG' ) ? GIVE_CAMPAIGNS_SLUG : 'campaigns';
-	$give_campaigns_rewrite  = defined( 'GIVE_DISABLE_CAMPAIGNS_REWRITE' ) && GIVE_DISABLE_CAMPAIGNS_REWRITE ? false : array(
+	$give_campaigns_archives = defined('GIVE_DISABLE_CAMPAIGNS_ARCHIVE') && GIVE_DISABLE_CAMPAIGNS_ARCHIVE ? false : true;
+	$give_campaigns_slug     = defined('GIVE_CAMPAIGNS_SLUG') ? GIVE_CAMPAIGNS_SLUG : 'campaigns';
+	$give_campaigns_rewrite  = defined('GIVE_DISABLE_CAMPAIGNS_REWRITE') && GIVE_DISABLE_CAMPAIGNS_REWRITE ? false : array(
 		'slug'       => $give_campaigns_slug,
 		'with_front' => false
 	);
 
-	$give_campaigns_labels = apply_filters( 'give_campaign_labels', array(
+	$give_campaigns_labels = apply_filters('give_campaign_labels', array(
 		'name'               => '%2$s',
 		'singular_name'      => '%1$s',
-		'add_new'            => __( 'Add %1$s', 'give' ),
-		'add_new_item'       => __( 'Add New %1$s', 'give' ),
-		'edit_item'          => __( 'Edit %1$s', 'give' ),
-		'new_item'           => __( 'New %1$s', 'give' ),
-		'all_items'          => __( 'All %2$s', 'give' ),
-		'view_item'          => __( 'View %1$s', 'give' ),
-		'search_items'       => __( 'Search %2$s', 'give' ),
-		'not_found'          => __( 'No %2$s found', 'give' ),
-		'not_found_in_trash' => __( 'No %2$s found in Trash', 'give' ),
+		'add_new'            => __('Add %1$s', 'give'),
+		'add_new_item'       => __('Add New %1$s', 'give'),
+		'edit_item'          => __('Edit %1$s', 'give'),
+		'new_item'           => __('New %1$s', 'give'),
+		'all_items'          => __('All %2$s', 'give'),
+		'view_item'          => __('View %1$s', 'give'),
+		'search_items'       => __('Search %2$s', 'give'),
+		'not_found'          => __('No %2$s found', 'give'),
+		'not_found_in_trash' => __('No %2$s found in Trash', 'give'),
 		'parent_item_colon'  => '',
-	) );
+	));
 
-	foreach ( $give_campaigns_labels as $key => $value ) {
-		$give_campaigns_labels[ $key ] = sprintf( $value, give_get_campaigns_label_singular(), give_get_campaigns_label_plural() );
+	foreach ($give_campaigns_labels as $key => $value) {
+		$give_campaigns_labels[$key] = sprintf($value, give_get_campaigns_label_singular(), give_get_campaigns_label_plural());
 	}
 
 	$give_campaigns_args = array(
@@ -136,48 +136,48 @@ 
 		'capability_type'    => 'give_campaigns',
 		'has_archive'        => $give_campaigns_archives,
 		'hierarchical'       => false,
-		'supports'           => apply_filters( 'give_campaigns_supports', array(
+		'supports'           => apply_filters('give_campaigns_supports', array(
 			'title',
 			'thumbnail',
 			'excerpt',
 			'revisions',
 			'author'
-		) ),
+		)),
 	);
 	//	register_post_type( 'give_campaigns', apply_filters( 'give_campaigns_post_type_args', $give_campaigns_args ) );
 
 	/** Payment Post Type */
 	$payment_labels = array(
-		'name'               => _x( 'Donations', 'post type general name', 'give' ),
-		'singular_name'      => _x( 'Donation', 'post type singular name', 'give' ),
-		'add_new'            => __( 'Add New', 'give' ),
-		'add_new_item'       => __( 'Add New Donation', 'give' ),
-		'edit_item'          => __( 'Edit Donation', 'give' ),
-		'new_item'           => __( 'New Donation', 'give' ),
-		'all_items'          => __( 'All Donations', 'give' ),
-		'view_item'          => __( 'View Donation', 'give' ),
-		'search_items'       => __( 'Search Donations', 'give' ),
-		'not_found'          => __( 'No Donations found', 'give' ),
-		'not_found_in_trash' => __( 'No Donations found in Trash', 'give' ),
+		'name'               => _x('Donations', 'post type general name', 'give'),
+		'singular_name'      => _x('Donation', 'post type singular name', 'give'),
+		'add_new'            => __('Add New', 'give'),
+		'add_new_item'       => __('Add New Donation', 'give'),
+		'edit_item'          => __('Edit Donation', 'give'),
+		'new_item'           => __('New Donation', 'give'),
+		'all_items'          => __('All Donations', 'give'),
+		'view_item'          => __('View Donation', 'give'),
+		'search_items'       => __('Search Donations', 'give'),
+		'not_found'          => __('No Donations found', 'give'),
+		'not_found_in_trash' => __('No Donations found in Trash', 'give'),
 		'parent_item_colon'  => '',
-		'menu_name'          => __( 'Transactions', 'give' )
+		'menu_name'          => __('Transactions', 'give')
 	);
 
 	$payment_args = array(
-		'labels'          => apply_filters( 'give_payment_labels', $payment_labels ),
+		'labels'          => apply_filters('give_payment_labels', $payment_labels),
 		'public'          => false,
 		'query_var'       => false,
 		'rewrite'         => false,
 		'map_meta_cap'    => true,
 		'capability_type' => 'give_payment',
-		'supports'        => array( 'title' ),
+		'supports'        => array('title'),
 		'can_export'      => true
 	);
-	register_post_type( 'give_payment', $payment_args );
+	register_post_type('give_payment', $payment_args);
 
 }
 
-add_action( 'init', 'give_setup_post_types', 1 );
+add_action('init', 'give_setup_post_types', 1);
 
 
 /**
@@ -190,30 +190,30 @@ 
  */
 function give_setup_taxonomies() {
 
-	$slug = defined( 'GIVE_FORMS_SLUG' ) ? GIVE_FORMS_SLUG : 'donations';
+	$slug = defined('GIVE_FORMS_SLUG') ? GIVE_FORMS_SLUG : 'donations';
 
 	/** Categories */
 	$category_labels = array(
-		'name'              => sprintf( _x( '%s Categories', 'taxonomy general name', 'give' ), give_get_forms_label_singular() ),
-		'singular_name'     => _x( 'Category', 'taxonomy singular name', 'give' ),
-		'search_items'      => __( 'Search Categories', 'give' ),
-		'all_items'         => __( 'All Categories', 'give' ),
-		'parent_item'       => __( 'Parent Category', 'give' ),
-		'parent_item_colon' => __( 'Parent Category:', 'give' ),
-		'edit_item'         => __( 'Edit Category', 'give' ),
-		'update_item'       => __( 'Update Category', 'give' ),
-		'add_new_item'      => sprintf( __( 'Add New %s Category', 'give' ), give_get_forms_label_singular() ),
-		'new_item_name'     => __( 'New Category Name', 'give' ),
-		'menu_name'         => __( 'Categories', 'give' ),
+		'name'              => sprintf(_x('%s Categories', 'taxonomy general name', 'give'), give_get_forms_label_singular()),
+		'singular_name'     => _x('Category', 'taxonomy singular name', 'give'),
+		'search_items'      => __('Search Categories', 'give'),
+		'all_items'         => __('All Categories', 'give'),
+		'parent_item'       => __('Parent Category', 'give'),
+		'parent_item_colon' => __('Parent Category:', 'give'),
+		'edit_item'         => __('Edit Category', 'give'),
+		'update_item'       => __('Update Category', 'give'),
+		'add_new_item'      => sprintf(__('Add New %s Category', 'give'), give_get_forms_label_singular()),
+		'new_item_name'     => __('New Category Name', 'give'),
+		'menu_name'         => __('Categories', 'give'),
 	);
 
-	$category_args = apply_filters( 'give_forms_category_args', array(
+	$category_args = apply_filters('give_forms_category_args', array(
 			'hierarchical' => true,
-			'labels'       => apply_filters( 'give_forms_category_labels', $category_labels ),
+			'labels'       => apply_filters('give_forms_category_labels', $category_labels),
 			'show_ui'      => true,
 			'query_var'    => 'give_forms_category',
 			'rewrite'      => array(
-				'slug'         => $slug . '/category',
+				'slug'         => $slug.'/category',
 				'with_front'   => false,
 				'hierarchical' => true
 			),
@@ -227,34 +227,34 @@ 
 	);
 
 	//Does the user want categories?
-	if ( give_get_option( 'enable_categories' ) == 'on' ) {
-		register_taxonomy( 'give_forms_category', array( 'give_forms' ), $category_args );
-		register_taxonomy_for_object_type( 'give_forms_category', 'give_forms' );
+	if (give_get_option('enable_categories') == 'on') {
+		register_taxonomy('give_forms_category', array('give_forms'), $category_args);
+		register_taxonomy_for_object_type('give_forms_category', 'give_forms');
 	}
 
 
 	/** Tags */
 	$tag_labels = array(
-		'name'                  => sprintf( _x( '%s Tags', 'taxonomy general name', 'give' ), give_get_forms_label_singular() ),
-		'singular_name'         => _x( 'Tag', 'taxonomy singular name', 'give' ),
-		'search_items'          => __( 'Search Tags', 'give' ),
-		'all_items'             => __( 'All Tags', 'give' ),
-		'parent_item'           => __( 'Parent Tag', 'give' ),
-		'parent_item_colon'     => __( 'Parent Tag:', 'give' ),
-		'edit_item'             => __( 'Edit Tag', 'give' ),
-		'update_item'           => __( 'Update Tag', 'give' ),
-		'add_new_item'          => __( 'Add New Tag', 'give' ),
-		'new_item_name'         => __( 'New Tag Name', 'give' ),
-		'menu_name'             => __( 'Tags', 'give' ),
-		'choose_from_most_used' => sprintf( __( 'Choose from most used %s tags', 'give' ), give_get_forms_label_singular() ),
+		'name'                  => sprintf(_x('%s Tags', 'taxonomy general name', 'give'), give_get_forms_label_singular()),
+		'singular_name'         => _x('Tag', 'taxonomy singular name', 'give'),
+		'search_items'          => __('Search Tags', 'give'),
+		'all_items'             => __('All Tags', 'give'),
+		'parent_item'           => __('Parent Tag', 'give'),
+		'parent_item_colon'     => __('Parent Tag:', 'give'),
+		'edit_item'             => __('Edit Tag', 'give'),
+		'update_item'           => __('Update Tag', 'give'),
+		'add_new_item'          => __('Add New Tag', 'give'),
+		'new_item_name'         => __('New Tag Name', 'give'),
+		'menu_name'             => __('Tags', 'give'),
+		'choose_from_most_used' => sprintf(__('Choose from most used %s tags', 'give'), give_get_forms_label_singular()),
 	);
 
-	$tag_args = apply_filters( 'give_forms_tag_args', array(
+	$tag_args = apply_filters('give_forms_tag_args', array(
 			'hierarchical' => false,
-			'labels'       => apply_filters( 'give_forms_tag_labels', $tag_labels ),
+			'labels'       => apply_filters('give_forms_tag_labels', $tag_labels),
 			'show_ui'      => true,
 			'query_var'    => 'give_forms_tag',
-			'rewrite'      => array( 'slug' => $slug . '/tag', 'with_front' => false, 'hierarchical' => true ),
+			'rewrite'      => array('slug' => $slug.'/tag', 'with_front' => false, 'hierarchical' => true),
 			'capabilities' => array(
 				'manage_terms' => 'manage_give_forms_terms',
 				'edit_terms'   => 'edit_give_forms_terms',
@@ -265,15 +265,15 @@ 
 		)
 	);
 
-	if ( give_get_option( 'enable_tags' ) == 'on' ) {
-		register_taxonomy( 'give_forms_tag', array( 'give_forms' ), $tag_args );
-		register_taxonomy_for_object_type( 'give_forms_tag', 'give_forms' );
+	if (give_get_option('enable_tags') == 'on') {
+		register_taxonomy('give_forms_tag', array('give_forms'), $tag_args);
+		register_taxonomy_for_object_type('give_forms_tag', 'give_forms');
 	}
 
 
 }
 
-add_action( 'init', 'give_setup_taxonomies', 0 );
+add_action('init', 'give_setup_taxonomies', 0);
 
 
 /**
@@ -284,11 +284,11 @@ 
  */
 function give_get_default_form_labels() {
 	$defaults = array(
-		'singular' => __( 'Form', 'give' ),
-		'plural'   => __( 'Forms', 'give' )
+		'singular' => __('Form', 'give'),
+		'plural'   => __('Forms', 'give')
 	);
 
-	return apply_filters( 'give_default_form_name', $defaults );
+	return apply_filters('give_default_form_name', $defaults);
 }
 
 /**
@@ -299,11 +299,11 @@ 
  */
 function give_get_default_campaign_labels() {
 	$defaults = array(
-		'singular' => __( 'Campaign', 'give' ),
-		'plural'   => __( 'Campaigns', 'give' )
+		'singular' => __('Campaign', 'give'),
+		'plural'   => __('Campaigns', 'give')
 	);
 
-	return apply_filters( 'give_default_campaign_name', $defaults );
+	return apply_filters('give_default_campaign_name', $defaults);
 }
 
 /**
@@ -315,10 +315,10 @@ 
  *
  * @return string $defaults['singular'] Singular label
  */
-function give_get_forms_label_singular( $lowercase = false ) {
+function give_get_forms_label_singular($lowercase = false) {
 	$defaults = give_get_default_form_labels();
 
-	return ( $lowercase ) ? strtolower( $defaults['singular'] ) : $defaults['singular'];
+	return ($lowercase) ? strtolower($defaults['singular']) : $defaults['singular'];
 }
 
 /**
@@ -327,10 +327,10 @@ 
  * @since 1.0
  * @return string $defaults['plural'] Plural label
  */
-function give_get_forms_label_plural( $lowercase = false ) {
+function give_get_forms_label_plural($lowercase = false) {
 	$defaults = give_get_default_form_labels();
 
-	return ( $lowercase ) ? strtolower( $defaults['plural'] ) : $defaults['plural'];
+	return ($lowercase) ? strtolower($defaults['plural']) : $defaults['plural'];
 }
 
 /**
@@ -342,10 +342,10 @@ 
  *
  * @return string $defaults['singular'] Singular label
  */
-function give_get_campaigns_label_singular( $lowercase = false ) {
+function give_get_campaigns_label_singular($lowercase = false) {
 	$defaults = give_get_default_campaign_labels();
 
-	return ( $lowercase ) ? strtolower( $defaults['singular'] ) : $defaults['singular'];
+	return ($lowercase) ? strtolower($defaults['singular']) : $defaults['singular'];
 }
 
 /**
@@ -354,10 +354,10 @@ 
  * @since 1.0
  * @return string $defaults['plural'] Plural label
  */
-function give_get_campaigns_label_plural( $lowercase = false ) {
+function give_get_campaigns_label_plural($lowercase = false) {
 	$defaults = give_get_default_campaign_labels();
 
-	return ( $lowercase ) ? strtolower( $defaults['plural'] ) : $defaults['plural'];
+	return ($lowercase) ? strtolower($defaults['plural']) : $defaults['plural'];
 }
 
 /**
@@ -369,26 +369,26 @@ 
  *
  * @return string $title New placeholder text
  */
-function give_change_default_title( $title ) {
+function give_change_default_title($title) {
 	// If a frontend plugin uses this filter (check extensions before changing this function)
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		$label = give_get_forms_label_singular();
-		$title = sprintf( __( 'Enter %s title here', 'give' ), $label );
+		$title = sprintf(__('Enter %s title here', 'give'), $label);
 
 		return $title;
 	}
 
 	$screen = get_current_screen();
 
-	if ( 'give_forms' == $screen->post_type ) {
+	if ('give_forms' == $screen->post_type) {
 		$label = give_get_forms_label_singular();
-		$title = sprintf( __( 'Enter %s title here', 'give' ), $label );
+		$title = sprintf(__('Enter %s title here', 'give'), $label);
 	}
 
 	return $title;
 }
 
-add_filter( 'enter_title_here', 'give_change_default_title' );
+add_filter('enter_title_here', 'give_change_default_title');
 
 /**
  * Registers Custom Post Statuses which are used by the Payments
@@ -398,42 +398,42 @@ 
  */
 function give_register_post_type_statuses() {
 	// Payment Statuses
-	register_post_status( 'refunded', array(
-		'label'                     => _x( 'Refunded', 'Refunded payment status', 'give' ),
+	register_post_status('refunded', array(
+		'label'                     => _x('Refunded', 'Refunded payment status', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Refunded <span class="count">(%s)</span>', 'Refunded <span class="count">(%s)</span>', 'give' )
-	) );
-	register_post_status( 'failed', array(
-		'label'                     => _x( 'Failed', 'Failed payment status', 'give' ),
+		'label_count'               => _n_noop('Refunded <span class="count">(%s)</span>', 'Refunded <span class="count">(%s)</span>', 'give')
+	));
+	register_post_status('failed', array(
+		'label'                     => _x('Failed', 'Failed payment status', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'give' )
-	) );
-	register_post_status( 'revoked', array(
-		'label'                     => _x( 'Revoked', 'Revoked payment status', 'give' ),
+		'label_count'               => _n_noop('Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'give')
+	));
+	register_post_status('revoked', array(
+		'label'                     => _x('Revoked', 'Revoked payment status', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Revoked <span class="count">(%s)</span>', 'Revoked <span class="count">(%s)</span>', 'give' )
-	) );
-	register_post_status( 'abandoned', array(
-		'label'                     => _x( 'Abandoned', 'Abandoned payment status', 'give' ),
+		'label_count'               => _n_noop('Revoked <span class="count">(%s)</span>', 'Revoked <span class="count">(%s)</span>', 'give')
+	));
+	register_post_status('abandoned', array(
+		'label'                     => _x('Abandoned', 'Abandoned payment status', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Abandoned <span class="count">(%s)</span>', 'Abandoned <span class="count">(%s)</span>', 'give' )
-	) );
+		'label_count'               => _n_noop('Abandoned <span class="count">(%s)</span>', 'Abandoned <span class="count">(%s)</span>', 'give')
+	));
 
 }
 
-add_action( 'init', 'give_register_post_type_statuses' );
+add_action('init', 'give_register_post_type_statuses');
 
 /**
  * Updated Messages
@@ -446,45 +446,45 @@ 
  *
  * @return array $messages New post updated messages
  */
-function give_updated_messages( $messages ) {
+function give_updated_messages($messages) {
 	global $post, $post_ID;
 
-	$url1 = '<a href="' . get_permalink( $post_ID ) . '">';
+	$url1 = '<a href="'.get_permalink($post_ID).'">';
 	$url2 = give_get_forms_label_singular();
 	$url3 = '</a>';
 
 	$messages['give_forms'] = array(
-		1 => sprintf( __( '%2$s updated. %1$sView %2$s%3$s.', 'give' ), $url1, $url2, $url3 ),
-		4 => sprintf( __( '%2$s updated. %1$sView %2$s%3$s.', 'give' ), $url1, $url2, $url3 ),
-		6 => sprintf( __( '%2$s published. %1$sView %2$s%3$s.', 'give' ), $url1, $url2, $url3 ),
-		7 => sprintf( __( '%2$s saved. %1$sView %2$s%3$s.', 'give' ), $url1, $url2, $url3 ),
-		8 => sprintf( __( '%2$s submitted. %1$sView %2$s%3$s.', 'give' ), $url1, $url2, $url3 )
+		1 => sprintf(__('%2$s updated. %1$sView %2$s%3$s.', 'give'), $url1, $url2, $url3),
+		4 => sprintf(__('%2$s updated. %1$sView %2$s%3$s.', 'give'), $url1, $url2, $url3),
+		6 => sprintf(__('%2$s published. %1$sView %2$s%3$s.', 'give'), $url1, $url2, $url3),
+		7 => sprintf(__('%2$s saved. %1$sView %2$s%3$s.', 'give'), $url1, $url2, $url3),
+		8 => sprintf(__('%2$s submitted. %1$sView %2$s%3$s.', 'give'), $url1, $url2, $url3)
 	);
 
 	return $messages;
 }
 
-add_filter( 'post_updated_messages', 'give_updated_messages' );
+add_filter('post_updated_messages', 'give_updated_messages');
 
 
 /**
  * Setup Post Type Images
  */
-add_action( 'after_setup_theme', 'give_add_thumbnail_support', 10 );
-add_action( 'after_setup_theme', 'give_add_image_sizes', 10 );
+add_action('after_setup_theme', 'give_add_thumbnail_support', 10);
+add_action('after_setup_theme', 'give_add_image_sizes', 10);
 
 /**
  * Ensure post thumbnail support is turned on
  */
 function give_add_thumbnail_support() {
-	if ( give_get_option( 'disable_form_featured_img' ) === 'on' ) {
+	if (give_get_option('disable_form_featured_img') === 'on') {
 		return;
 	}
-	if ( ! current_theme_supports( 'post-thumbnails' ) ) {
-		add_theme_support( 'post-thumbnails' );
+	if ( ! current_theme_supports('post-thumbnails')) {
+		add_theme_support('post-thumbnails');
 	}
-	add_post_type_support( 'give_forms', 'thumbnail' );
-	add_post_type_support( 'give_campaigns', 'thumbnail' );
+	add_post_type_support('give_forms', 'thumbnail');
+	add_post_type_support('give_campaigns', 'thumbnail');
 }
 
 /**
@@ -493,11 +493,11 @@ 
  * @since 1.0
  */
 function give_add_image_sizes() {
-	$give_form_thumbnail = give_get_image_size( 'give_form_thumbnail' );
-	$give_form_single    = give_get_image_size( 'give_form_single' );
+	$give_form_thumbnail = give_get_image_size('give_form_thumbnail');
+	$give_form_single    = give_get_image_size('give_form_single');
 
-	add_image_size( 'give_form_thumbnail', $give_form_thumbnail['width'], $give_form_thumbnail['height'], $give_form_thumbnail['crop'] );
-	add_image_size( 'give_form_single', $give_form_single['width'], $give_form_single['height'], $give_form_single['crop'] );
+	add_image_size('give_form_thumbnail', $give_form_thumbnail['width'], $give_form_thumbnail['height'], $give_form_thumbnail['crop']);
+	add_image_size('give_form_single', $give_form_single['width'], $give_form_single['height'], $give_form_single['crop']);
 }
 
 /**
@@ -509,19 +509,19 @@ 
 function give_widgets_init() {
 
 	//Single Give Forms (disabled if single turned off in settings)
-	if ( give_get_option( 'disable_forms_singular' ) !== 'on' && give_get_option( 'disable_form_sidebar' ) !== 'on' ) {
+	if (give_get_option('disable_forms_singular') !== 'on' && give_get_option('disable_form_sidebar') !== 'on') {
 
-		register_sidebar( apply_filters( 'give_forms_single_sidebar', array(
-			'name'          => __( 'Give Single Form Sidebar', 'give' ),
+		register_sidebar(apply_filters('give_forms_single_sidebar', array(
+			'name'          => __('Give Single Form Sidebar', 'give'),
 			'id'            => 'give-forms-sidebar',
-			'description'   => __( 'Widgets in this area will be shown on the single Give forms aside area. This sidebar will not display for embedded forms.', 'give' ),
+			'description'   => __('Widgets in this area will be shown on the single Give forms aside area. This sidebar will not display for embedded forms.', 'give'),
 			'before_widget' => '<div id="%1$s" class="widget %2$s">',
 			'after_widget'  => '</div>',
 			'before_title'  => '<h3 class="widgettitle widget-title">',
 			'after_title'   => '</h3>',
-		) ) );
+		)));
 
 	}
 }
 
-add_action( 'widgets_init', 'give_widgets_init', 999 );
\ No newline at end of file
+add_action('widgets_init', 'give_widgets_init', 999);
\ No newline at end of file