Completed
Push — master ( 0af146...0e0d02 )
by Devin
13s
created
includes/actions.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
  */
11 11
 
12 12
 // Exit if accessed directly.
13
-if ( ! defined( 'ABSPATH' ) ) {
13
+if ( ! defined('ABSPATH')) {
14 14
 	exit;
15 15
 }
16 16
 
@@ -25,14 +25,14 @@  discard block
 block discarded – undo
25 25
  */
26 26
 function give_get_actions() {
27 27
 
28
-	$_get_action = ! empty( $_GET['give_action'] ) ? $_GET['give_action'] : null;
28
+	$_get_action = ! empty($_GET['give_action']) ? $_GET['give_action'] : null;
29 29
 
30 30
 	// Add backward compatibility to give-action param ( $_GET or $_POST )
31
-	if(  doing_action( 'admin_init' ) && empty( $_get_action ) ) {
32
-		$_get_action = ! empty( $_GET['give-action'] ) ? $_GET['give-action'] : null;
31
+	if (doing_action('admin_init') && empty($_get_action)) {
32
+		$_get_action = ! empty($_GET['give-action']) ? $_GET['give-action'] : null;
33 33
 	}
34 34
 
35
-	if ( isset( $_get_action ) ) {
35
+	if (isset($_get_action)) {
36 36
 		/**
37 37
 		 * Fires in WordPress init or admin init, when give_action is present in $_GET.
38 38
 		 *
@@ -40,13 +40,13 @@  discard block
 block discarded – undo
40 40
 		 *
41 41
 		 * @param array $_GET Array of HTTP GET variables.
42 42
 		 */
43
-		do_action( "give_{$_get_action}", $_GET );
43
+		do_action("give_{$_get_action}", $_GET);
44 44
 	}
45 45
 
46 46
 }
47 47
 
48
-add_action( 'init', 'give_get_actions' );
49
-add_action( 'admin_init', 'give_get_actions' );
48
+add_action('init', 'give_get_actions');
49
+add_action('admin_init', 'give_get_actions');
50 50
 
51 51
 /**
52 52
  * Hooks Give actions, when present in the $_POST superglobal. Every give_action
@@ -59,15 +59,15 @@  discard block
 block discarded – undo
59 59
  */
60 60
 function give_post_actions() {
61 61
 
62
-	$_post_action = ! empty( $_POST['give_action'] ) ? $_POST['give_action'] : null;
62
+	$_post_action = ! empty($_POST['give_action']) ? $_POST['give_action'] : null;
63 63
 
64 64
 
65 65
 	// Add backward compatibility to give-action param ( $_GET or $_POST )
66
-	if(  doing_action( 'admin_init' ) && empty( $_post_action ) ) {
67
-		$_post_action = ! empty( $_POST['give-action'] ) ? $_POST['give-action'] : null;
66
+	if (doing_action('admin_init') && empty($_post_action)) {
67
+		$_post_action = ! empty($_POST['give-action']) ? $_POST['give-action'] : null;
68 68
 	}
69 69
 
70
-	if ( isset( $_post_action ) ) {
70
+	if (isset($_post_action)) {
71 71
 		/**
72 72
 		 * Fires in WordPress init or admin init, when give_action is present in $_POST.
73 73
 		 *
@@ -75,13 +75,13 @@  discard block
 block discarded – undo
75 75
 		 *
76 76
 		 * @param array $_POST Array of HTTP POST variables.
77 77
 		 */
78
-		do_action( "give_{$_post_action}", $_POST );
78
+		do_action("give_{$_post_action}", $_POST);
79 79
 	}
80 80
 
81 81
 }
82 82
 
83
-add_action( 'init', 'give_post_actions' );
84
-add_action( 'admin_init', 'give_post_actions' );
83
+add_action('init', 'give_post_actions');
84
+add_action('admin_init', 'give_post_actions');
85 85
 
86 86
 /**
87 87
  * Connect WordPress user with Donor.
@@ -91,27 +91,27 @@  discard block
 block discarded – undo
91 91
  * @param  array $user_data User Data
92 92
  * @return void
93 93
  */
94
-function give_connect_donor_to_wpuser( $user_id, $user_data ){
95
-	$donor = new Give_Customer( $user_data['user_email'] );
94
+function give_connect_donor_to_wpuser($user_id, $user_data) {
95
+	$donor = new Give_Customer($user_data['user_email']);
96 96
 
97 97
 	// Validate donor id and check if do nor is already connect to wp user or not.
98
-	if( $donor->id && ! $donor->user_id ) {
98
+	if ($donor->id && ! $donor->user_id) {
99 99
 
100 100
 		// Update donor user_id.
101
-		if( $donor->update( array( 'user_id' => $user_id ) ) ) {
102
-			$donor_note = sprintf( esc_html__( 'WordPress user #%d is connected to #%d', 'give' ), $user_id, $donor->id );
103
-			$donor->add_note( $donor_note );
101
+		if ($donor->update(array('user_id' => $user_id))) {
102
+			$donor_note = sprintf(esc_html__('WordPress user #%d is connected to #%d', 'give'), $user_id, $donor->id);
103
+			$donor->add_note($donor_note);
104 104
 
105 105
 			// Update user_id meta in payments.
106
-			if( ! empty( $donor->payment_ids ) && ( $donations = explode( ',', $donor->payment_ids ) ) ) {
107
-				foreach ( $donations as $donation  ) {
108
-					update_post_meta( $donation, '_give_payment_user_id', $user_id );
106
+			if ( ! empty($donor->payment_ids) && ($donations = explode(',', $donor->payment_ids))) {
107
+				foreach ($donations as $donation) {
108
+					update_post_meta($donation, '_give_payment_user_id', $user_id);
109 109
 				}
110 110
 			}
111 111
 		}
112 112
 	}
113 113
 }
114
-add_action( 'give_insert_user', 'give_connect_donor_to_wpuser', 10, 2 );
114
+add_action('give_insert_user', 'give_connect_donor_to_wpuser', 10, 2);
115 115
 
116 116
 
117 117
 /**
@@ -125,24 +125,24 @@  discard block
 block discarded – undo
125 125
 function give_validate_license_when_site_migrated() {
126 126
 	// Store current site address if not already stored.
127 127
 	$homeurl = home_url();
128
-	if( ! get_option( 'give_site_address_before_migrate' ) ) {
128
+	if ( ! get_option('give_site_address_before_migrate')) {
129 129
 		// Update site address.
130
-		update_option( 'give_site_address_before_migrate', $homeurl );
130
+		update_option('give_site_address_before_migrate', $homeurl);
131 131
 
132 132
 		return;
133 133
 	}
134 134
 
135
-	if( $homeurl !== get_option( 'give_site_address_before_migrate' ) ) {
135
+	if ($homeurl !== get_option('give_site_address_before_migrate')) {
136 136
 		// Immediately run cron.
137
-		wp_schedule_single_event( time() , 'give_validate_license_when_site_migrated' );
137
+		wp_schedule_single_event(time(), 'give_validate_license_when_site_migrated');
138 138
 
139 139
 		// Update site address.
140
-		update_option( 'give_site_address_before_migrate', home_url() );
140
+		update_option('give_site_address_before_migrate', home_url());
141 141
 	}
142 142
 
143 143
 }
144
-add_action( 'init', 'give_validate_license_when_site_migrated' );
145
-add_action( 'admin_init', 'give_validate_license_when_site_migrated' );
144
+add_action('init', 'give_validate_license_when_site_migrated');
145
+add_action('admin_init', 'give_validate_license_when_site_migrated');
146 146
 
147 147
 
148 148
 /**
@@ -151,15 +151,15 @@  discard block
 block discarded – undo
151 151
  * @since 1.8
152 152
  * @param $data
153 153
  */
154
-function give_donor_batch_export_complete( $data ) {
154
+function give_donor_batch_export_complete($data) {
155 155
 	// Remove donor ids cache.
156
-	if(
157
-		isset( $data['class'] )
156
+	if (
157
+		isset($data['class'])
158 158
 		&& 'Give_Batch_Customers_Export' === $data['class']
159
-		&& ! empty( $data['forms'] )
160
-		&& isset( $data['give_export_option']['query_id'] )
159
+		&& ! empty($data['forms'])
160
+		&& isset($data['give_export_option']['query_id'])
161 161
 	) {
162
-		delete_transient( $data['give_export_option']['query_id'] );
162
+		delete_transient($data['give_export_option']['query_id']);
163 163
 	}
164 164
 }
165
-add_action('give_file_export_complete', 'give_donor_batch_export_complete' );
165
+add_action('give_file_export_complete', 'give_donor_batch_export_complete');
Please login to merge, or discard this patch.
includes/gateways/offline-donations.php 1 patch
Spacing   +152 added lines, -152 removed lines patch added patch discarded remove patch
@@ -18,17 +18,17 @@  discard block
 block discarded – undo
18 18
  *
19 19
  * @return array
20 20
  */
21
-function give_offline_register_gateway( $gateways ) {
21
+function give_offline_register_gateway($gateways) {
22 22
 	// Format: ID => Name
23 23
 	$gateways['offline'] = array(
24
-		'admin_label'    => esc_attr__( 'Offline Donation', 'give' ),
25
-		'checkout_label' => esc_attr__( 'Offline Donation', 'give' )
24
+		'admin_label'    => esc_attr__('Offline Donation', 'give'),
25
+		'checkout_label' => esc_attr__('Offline Donation', 'give')
26 26
 	);
27 27
 
28 28
 	return $gateways;
29 29
 }
30 30
 
31
-add_filter( 'give_payment_gateways', 'give_offline_register_gateway', 1 );
31
+add_filter('give_payment_gateways', 'give_offline_register_gateway', 1);
32 32
 
33 33
 
34 34
 /**
@@ -40,9 +40,9 @@  discard block
 block discarded – undo
40 40
  *
41 41
  * @return void
42 42
  */
43
-function give_offline_payment_cc_form( $form_id ) {
43
+function give_offline_payment_cc_form($form_id) {
44 44
 	// Get offline payment instruction.
45
-	$offline_instructions = give_get_offline_payment_instruction( $form_id, true );
45
+	$offline_instructions = give_get_offline_payment_instruction($form_id, true);
46 46
 
47 47
 	ob_start();
48 48
 
@@ -53,10 +53,10 @@  discard block
 block discarded – undo
53 53
 	 *
54 54
 	 * @param int $form_id Give form id.
55 55
 	 */
56
-	do_action( 'give_before_offline_info_fields', $form_id );
56
+	do_action('give_before_offline_info_fields', $form_id);
57 57
 	?>
58 58
 	<fieldset id="give_offline_payment_info">
59
-		<?php echo stripslashes( $offline_instructions ); ?>
59
+		<?php echo stripslashes($offline_instructions); ?>
60 60
 	</fieldset>
61 61
 	<?php
62 62
 	/**
@@ -66,35 +66,35 @@  discard block
 block discarded – undo
66 66
 	 *
67 67
 	 * @param int $form_id Give form id.
68 68
 	 */
69
-	do_action( 'give_after_offline_info_fields', $form_id );
69
+	do_action('give_after_offline_info_fields', $form_id);
70 70
 
71 71
 	echo ob_get_clean();
72 72
 }
73 73
 
74
-add_action( 'give_offline_cc_form', 'give_offline_payment_cc_form' );
74
+add_action('give_offline_cc_form', 'give_offline_payment_cc_form');
75 75
 
76 76
 /**
77 77
  * Give Offline Billing Field
78 78
  *
79 79
  * @param $form_id
80 80
  */
81
-function give_offline_billing_fields( $form_id ) {
81
+function give_offline_billing_fields($form_id) {
82 82
 	//Enable Default CC fields (billing info)
83
-	$post_offline_cc_fields        = get_post_meta( $form_id, '_give_offline_donation_enable_billing_fields_single', true );
84
-	$post_offline_customize_option = get_post_meta( $form_id, '_give_customize_offline_donations', true );
83
+	$post_offline_cc_fields        = get_post_meta($form_id, '_give_offline_donation_enable_billing_fields_single', true);
84
+	$post_offline_customize_option = get_post_meta($form_id, '_give_customize_offline_donations', true);
85 85
 
86
-	$global_offline_cc_fields      = give_get_option( 'give_offline_donation_enable_billing_fields' );
86
+	$global_offline_cc_fields      = give_get_option('give_offline_donation_enable_billing_fields');
87 87
 
88 88
 	//Output CC Address fields if global option is on and user hasn't elected to customize this form's offline donation options
89 89
 	if (
90
-		( give_is_setting_enabled( $post_offline_customize_option, 'global' ) && give_is_setting_enabled( $global_offline_cc_fields ) )
91
-		|| ( give_is_setting_enabled( $post_offline_customize_option, 'enabled' ) && give_is_setting_enabled( $post_offline_cc_fields ) )
90
+		(give_is_setting_enabled($post_offline_customize_option, 'global') && give_is_setting_enabled($global_offline_cc_fields))
91
+		|| (give_is_setting_enabled($post_offline_customize_option, 'enabled') && give_is_setting_enabled($post_offline_cc_fields))
92 92
 	) {
93
-		give_default_cc_address_fields( $form_id );
93
+		give_default_cc_address_fields($form_id);
94 94
 	}
95 95
 }
96 96
 
97
-add_action( 'give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1 );
97
+add_action('give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1);
98 98
 
99 99
 /**
100 100
  * Process the payment
@@ -105,16 +105,16 @@  discard block
 block discarded – undo
105 105
  *
106 106
  * @return void
107 107
  */
108
-function give_offline_process_payment( $purchase_data ) {
108
+function give_offline_process_payment($purchase_data) {
109 109
 
110
-	$purchase_summary = give_get_purchase_summary( $purchase_data );
110
+	$purchase_summary = give_get_purchase_summary($purchase_data);
111 111
 
112 112
 	// setup the payment details
113 113
 	$payment_data = array(
114 114
 		'price'           => $purchase_data['price'],
115 115
 		'give_form_title' => $purchase_data['post_data']['give-form-title'],
116
-		'give_form_id'    => intval( $purchase_data['post_data']['give-form-id'] ),
117
-		'give_price_id'   => isset( $purchase_data['post_data']['give-price-id'] ) ? $purchase_data['post_data']['give-price-id'] : '',
116
+		'give_form_id'    => intval($purchase_data['post_data']['give-form-id']),
117
+		'give_price_id'   => isset($purchase_data['post_data']['give-price-id']) ? $purchase_data['post_data']['give-price-id'] : '',
118 118
 		'date'            => $purchase_data['date'],
119 119
 		'user_email'      => $purchase_data['user_email'],
120 120
 		'purchase_key'    => $purchase_data['purchase_key'],
@@ -126,20 +126,20 @@  discard block
 block discarded – undo
126 126
 
127 127
 
128 128
 	// record the pending payment
129
-	$payment = give_insert_payment( $payment_data );
129
+	$payment = give_insert_payment($payment_data);
130 130
 
131
-	if ( $payment ) {
132
-		give_offline_send_admin_notice( $payment );
133
-		give_offline_send_donor_instructions( $payment );
131
+	if ($payment) {
132
+		give_offline_send_admin_notice($payment);
133
+		give_offline_send_donor_instructions($payment);
134 134
 		give_send_to_success_page();
135 135
 	} else {
136 136
 		// if errors are present, send the user back to the donation form so they can be corrected
137
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
137
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
138 138
 	}
139 139
 
140 140
 }
141 141
 
142
-add_action( 'give_gateway_offline', 'give_offline_process_payment' );
142
+add_action('give_gateway_offline', 'give_offline_process_payment');
143 143
 
144 144
 
145 145
 /**
@@ -152,59 +152,59 @@  discard block
 block discarded – undo
152 152
  * @since       1.0
153 153
  * @return void
154 154
  */
155
-function give_offline_send_donor_instructions( $payment_id = 0 ) {
155
+function give_offline_send_donor_instructions($payment_id = 0) {
156 156
 
157
-	$payment_data                      = give_get_payment_meta( $payment_id );
158
-	$post_offline_customization_option = get_post_meta( $payment_data['form_id'], '_give_customize_offline_donations', true );
157
+	$payment_data                      = give_get_payment_meta($payment_id);
158
+	$post_offline_customization_option = get_post_meta($payment_data['form_id'], '_give_customize_offline_donations', true);
159 159
 
160 160
 	//Customize email content depending on whether the single form has been customized
161
-	$email_content = give_get_option( 'global_offline_donation_email' );
161
+	$email_content = give_get_option('global_offline_donation_email');
162 162
 
163
-	if ( give_is_setting_enabled( $post_offline_customization_option, 'enabled' ) ) {
164
-		$email_content = get_post_meta( $payment_data['form_id'], '_give_offline_donation_email', true );
163
+	if (give_is_setting_enabled($post_offline_customization_option, 'enabled')) {
164
+		$email_content = get_post_meta($payment_data['form_id'], '_give_offline_donation_email', true);
165 165
 	}
166 166
 
167
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
167
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
168 168
 
169 169
 	/**
170 170
 	 * Filters the from name.
171 171
 	 *
172 172
 	 * @since 1.7
173 173
 	 */
174
-	$from_name = apply_filters( 'give_donation_from_name', $from_name, $payment_id, $payment_data );
174
+	$from_name = apply_filters('give_donation_from_name', $from_name, $payment_id, $payment_data);
175 175
 
176
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
176
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
177 177
 
178 178
 	/**
179 179
 	 * Filters the from email.
180 180
 	 *
181 181
 	 * @since 1.7
182 182
 	 */
183
-	$from_email = apply_filters( 'give_donation_from_address', $from_email, $payment_id, $payment_data );
183
+	$from_email = apply_filters('give_donation_from_address', $from_email, $payment_id, $payment_data);
184 184
 
185
-	$to_email = give_get_payment_user_email( $payment_id );
185
+	$to_email = give_get_payment_user_email($payment_id);
186 186
 
187
-	$subject = give_get_option( 'offline_donation_subject', __( 'Offline Donation Instructions', 'give' ) );
188
-	if ( give_is_setting_enabled( $post_offline_customization_option,  'enabled' ) ) {
189
-		$subject = get_post_meta( $payment_data['form_id'], '_give_offline_donation_subject', true );
187
+	$subject = give_get_option('offline_donation_subject', __('Offline Donation Instructions', 'give'));
188
+	if (give_is_setting_enabled($post_offline_customization_option, 'enabled')) {
189
+		$subject = get_post_meta($payment_data['form_id'], '_give_offline_donation_subject', true);
190 190
 	}
191 191
 
192
-	$subject = apply_filters( 'give_offline_donation_subject', wp_strip_all_tags( $subject ), $payment_id );
193
-	$subject = give_do_email_tags( $subject, $payment_id );
192
+	$subject = apply_filters('give_offline_donation_subject', wp_strip_all_tags($subject), $payment_id);
193
+	$subject = give_do_email_tags($subject, $payment_id);
194 194
 
195
-	$attachments = apply_filters( 'give_offline_donation_attachments', array(), $payment_id, $payment_data );
196
-	$message     = give_do_email_tags( $email_content, $payment_id );
195
+	$attachments = apply_filters('give_offline_donation_attachments', array(), $payment_id, $payment_data);
196
+	$message     = give_do_email_tags($email_content, $payment_id);
197 197
 
198 198
 	$emails = Give()->emails;
199 199
 
200
-	$emails->__set( 'from_name', $from_name );
201
-	$emails->__set( 'from_email', $from_email );
202
-	$emails->__set( 'heading', __( 'Offline Donation Instructions', 'give' ) );
200
+	$emails->__set('from_name', $from_name);
201
+	$emails->__set('from_email', $from_email);
202
+	$emails->__set('heading', __('Offline Donation Instructions', 'give'));
203 203
 
204
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data );
205
-	$emails->__set( 'headers', $headers );
204
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data);
205
+	$emails->__set('headers', $headers);
206 206
 
207
-	$emails->send( $to_email, $subject, $message, $attachments );
207
+	$emails->send($to_email, $subject, $message, $attachments);
208 208
 
209 209
 }
210 210
 
@@ -221,52 +221,52 @@  discard block
 block discarded – undo
221 221
  * @return void
222 222
  *
223 223
  */
224
-function give_offline_send_admin_notice( $payment_id = 0 ) {
224
+function give_offline_send_admin_notice($payment_id = 0) {
225 225
 
226 226
 	/* Send an email notification to the admin */
227 227
 	$admin_email = give_get_admin_notice_emails();
228
-	$user_info   = give_get_payment_meta_user_info( $payment_id );
228
+	$user_info   = give_get_payment_meta_user_info($payment_id);
229 229
 
230
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 ) {
231
-		$user_data = get_userdata( $user_info['id'] );
230
+	if (isset($user_info['id']) && $user_info['id'] > 0) {
231
+		$user_data = get_userdata($user_info['id']);
232 232
 		$name      = $user_data->display_name;
233
-	} elseif ( isset( $user_info['first_name'] ) && isset( $user_info['last_name'] ) ) {
234
-		$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
233
+	} elseif (isset($user_info['first_name']) && isset($user_info['last_name'])) {
234
+		$name = $user_info['first_name'].' '.$user_info['last_name'];
235 235
 	} else {
236 236
 		$name = $user_info['email'];
237 237
 	}
238 238
 
239
-	$amount = give_currency_filter( give_format_amount( give_get_payment_amount( $payment_id ) ) );
239
+	$amount = give_currency_filter(give_format_amount(give_get_payment_amount($payment_id)));
240 240
 
241
-	$admin_subject = apply_filters( 'give_offline_admin_donation_notification_subject', __( 'New Pending Donation', 'give' ), $payment_id );
241
+	$admin_subject = apply_filters('give_offline_admin_donation_notification_subject', __('New Pending Donation', 'give'), $payment_id);
242 242
 
243
-	$admin_message = __( 'Dear Admin,', 'give' ) . "\n\n";
244
-	$admin_message .= __( 'An offline donation has been made on your website:', 'give' ) . ' ' . get_bloginfo( 'name' ) . ' ';
245
-	$admin_message .= __( 'Hooray! The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give' ) . "\n\n";
243
+	$admin_message = __('Dear Admin,', 'give')."\n\n";
244
+	$admin_message .= __('An offline donation has been made on your website:', 'give').' '.get_bloginfo('name').' ';
245
+	$admin_message .= __('Hooray! The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give')."\n\n";
246 246
 
247 247
 
248
-	$admin_message .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
249
-	$admin_message .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n\n";
248
+	$admin_message .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
249
+	$admin_message .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n\n";
250 250
 
251 251
 	$admin_message .= sprintf(
252 252
 		'<a href="%1$s">%2$s</a>',
253
-		admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id=' . $payment_id ),
254
-		__( 'Click Here to View and/or Update Donation Details', 'give' )
255
-	) . "\n\n";
253
+		admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id='.$payment_id),
254
+		__('Click Here to View and/or Update Donation Details', 'give')
255
+	)."\n\n";
256 256
 
257
-	$admin_message = apply_filters( 'give_offline_admin_donation_notification', $admin_message, $payment_id );
258
-	$admin_message = give_do_email_tags( $admin_message, $payment_id );
257
+	$admin_message = apply_filters('give_offline_admin_donation_notification', $admin_message, $payment_id);
258
+	$admin_message = give_do_email_tags($admin_message, $payment_id);
259 259
 
260
-	$attachments   = apply_filters( 'give_offline_admin_donation_notification_attachments', array(), $payment_id );
261
-	$admin_headers = apply_filters( 'give_offline_admin_donation_notification_headers', array(), $payment_id );
260
+	$attachments   = apply_filters('give_offline_admin_donation_notification_attachments', array(), $payment_id);
261
+	$admin_headers = apply_filters('give_offline_admin_donation_notification_headers', array(), $payment_id);
262 262
 
263 263
 	//Send Email
264 264
 	$emails = Give()->emails;
265
-	if ( ! empty( $admin_headers ) ) {
266
-		$emails->__set( 'headers', $admin_headers );
265
+	if ( ! empty($admin_headers)) {
266
+		$emails->__set('headers', $admin_headers);
267 267
 	}
268 268
 
269
-	$emails->send( $admin_email, $admin_subject, $admin_message, $attachments );
269
+	$emails->send($admin_email, $admin_subject, $admin_message, $attachments);
270 270
 
271 271
 }
272 272
 
@@ -278,20 +278,20 @@  discard block
 block discarded – undo
278 278
  *
279 279
  * @return array
280 280
  */
281
-function give_offline_add_settings( $settings ) {
281
+function give_offline_add_settings($settings) {
282 282
 
283 283
 	// Bailout: Do not show offline gateways setting in to metabox if its disabled globally.
284
-	if ( in_array( 'offline', give_get_option( 'gateways' ) ) ) {
284
+	if (in_array('offline', give_get_option('gateways'))) {
285 285
 		return $settings;
286 286
 	}
287 287
 
288 288
 	//Vars
289 289
 	$prefix = '_give_';
290 290
 
291
-	$is_gateway_active = give_is_gateway_active( 'offline' );
291
+	$is_gateway_active = give_is_gateway_active('offline');
292 292
 
293 293
 	//this gateway isn't active
294
-	if ( ! $is_gateway_active ) {
294
+	if ( ! $is_gateway_active) {
295 295
 		//return settings and bounce
296 296
 		return $settings;
297 297
 	}
@@ -300,34 +300,34 @@  discard block
 block discarded – undo
300 300
 	$check_settings = array(
301 301
 
302 302
 		array(
303
-			'name'    => __( 'Offline Donations', 'give' ),
304
-			'desc'    => __( 'Do you want to customize the donation instructions for this form?', 'give' ),
305
-			'id'      => $prefix . 'customize_offline_donations',
303
+			'name'    => __('Offline Donations', 'give'),
304
+			'desc'    => __('Do you want to customize the donation instructions for this form?', 'give'),
305
+			'id'      => $prefix.'customize_offline_donations',
306 306
 			'type'    => 'radio_inline',
307 307
 			'default' => 'global',
308
-			'options' => apply_filters( 'give_forms_content_options_select', array(
309
-					'global'   => __( 'Global Option', 'give' ),
310
-					'enabled'  => __( 'Customize', 'give' ),
311
-					'disabled' => __( 'Disable', 'give' ),
308
+			'options' => apply_filters('give_forms_content_options_select', array(
309
+					'global'   => __('Global Option', 'give'),
310
+					'enabled'  => __('Customize', 'give'),
311
+					'disabled' => __('Disable', 'give'),
312 312
 				)
313 313
 			),
314 314
 		),
315 315
 		array(
316
-			'name'        => __( 'Billing Fields', 'give' ),
317
-			'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' ),
318
-			'id'          => $prefix . 'offline_donation_enable_billing_fields_single',
316
+			'name'        => __('Billing Fields', 'give'),
317
+			'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'),
318
+			'id'          => $prefix.'offline_donation_enable_billing_fields_single',
319 319
 			'row_classes' => 'give-subfield',
320 320
 			'type'        => 'radio_inline',
321 321
 			'default'     => 'disabled',
322 322
 			'options'     => array(
323
-				'enabled'  => __( 'Enabled', 'give' ),
324
-				'disabled' => __( 'Disabled', 'give' ),
323
+				'enabled'  => __('Enabled', 'give'),
324
+				'disabled' => __('Disabled', 'give'),
325 325
 			),
326 326
 		),
327 327
 		array(
328
-			'id'          => $prefix . 'offline_checkout_notes',
329
-			'name'        => __( 'Donation Instructions', 'give' ),
330
-			'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' ),
328
+			'id'          => $prefix.'offline_checkout_notes',
329
+			'name'        => __('Donation Instructions', 'give'),
330
+			'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'),
331 331
 			'default'     => give_get_default_offline_donation_content(),
332 332
 			'type'        => 'wysiwyg',
333 333
 			'row_classes' => 'give-subfield',
@@ -336,17 +336,17 @@  discard block
 block discarded – undo
336 336
 			)
337 337
 		),
338 338
 		array(
339
-			'id'          => $prefix . 'offline_donation_subject',
340
-			'name'        => __( 'Email Subject', 'give' ),
341
-			'desc'        => __( 'Enter the subject line for the donation receipt email.', 'give' ),
342
-			'default'     => __( '{form_title} - Offline Donation Instructions', 'give' ),
339
+			'id'          => $prefix.'offline_donation_subject',
340
+			'name'        => __('Email Subject', 'give'),
341
+			'desc'        => __('Enter the subject line for the donation receipt email.', 'give'),
342
+			'default'     => __('{form_title} - Offline Donation Instructions', 'give'),
343 343
 			'row_classes' => 'give-subfield',
344 344
 			'type'        => 'text'
345 345
 		),
346 346
 		array(
347
-			'id'          => $prefix . 'offline_donation_email',
348
-			'name'        => __( 'Email Instructions', 'give' ),
349
-			'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' ),
347
+			'id'          => $prefix.'offline_donation_email',
348
+			'name'        => __('Email Instructions', 'give'),
349
+			'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'),
350 350
 			'default'     => give_get_default_offline_donation_email_content(),
351 351
 			'type'        => 'wysiwyg',
352 352
 			'row_classes' => 'give-subfield',
@@ -362,10 +362,10 @@  discard block
 block discarded – undo
362 362
 		),
363 363
 	);
364 364
 
365
-	return array_merge( $settings, $check_settings );
365
+	return array_merge($settings, $check_settings);
366 366
 }
367 367
 
368
-add_filter( 'give_forms_offline_donations_metabox_fields', 'give_offline_add_settings' );
368
+add_filter('give_forms_offline_donations_metabox_fields', 'give_offline_add_settings');
369 369
 
370 370
 
371 371
 /**
@@ -377,32 +377,32 @@  discard block
 block discarded – undo
377 377
  */
378 378
 function give_get_default_offline_donation_content() {
379 379
 
380
-	$sitename = get_bloginfo( 'sitename' );
380
+	$sitename = get_bloginfo('sitename');
381 381
 
382
-	$default_text = '<p>' . __( 'In order to make an offline donation we ask that you please follow these instructions', 'give' ) . ': </p>';
382
+	$default_text = '<p>'.__('In order to make an offline donation we ask that you please follow these instructions', 'give').': </p>';
383 383
 	$default_text .= '<ol>';
384 384
 	$default_text .= '<li>';
385 385
 	$default_text .= sprintf(
386 386
 		/* translators: %s: site name */
387
-		__( 'Make a check payable to "%s"', 'give' ),
387
+		__('Make a check payable to "%s"', 'give'),
388 388
 		$sitename
389 389
 	);
390 390
 	$default_text .= '</li>';
391 391
 	$default_text .= '<li>';
392 392
 	$default_text .= sprintf(
393 393
 		/* translators: %s: site name */
394
-		__( 'On the memo line of the check, please indicate that the donation is for "%s"', 'give' ),
394
+		__('On the memo line of the check, please indicate that the donation is for "%s"', 'give'),
395 395
 		$sitename
396 396
 	);
397 397
 	$default_text .= '</li>';
398
-	$default_text .= '<li>' . __( 'Please mail your check to:', 'give' ) . '</li>';
398
+	$default_text .= '<li>'.__('Please mail your check to:', 'give').'</li>';
399 399
 	$default_text .= '</ol>';
400
-	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>' . $sitename . '</em><br>';
400
+	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>'.$sitename.'</em><br>';
401 401
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>123 G Street </em><br>';
402 402
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>San Diego, CA 92101 </em><br>';
403
-	$default_text .= '<p>' . __( 'All contributions will be gratefully acknowledged and are tax deductible.', 'give' ) . '</p>';
403
+	$default_text .= '<p>'.__('All contributions will be gratefully acknowledged and are tax deductible.', 'give').'</p>';
404 404
 
405
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
405
+	return apply_filters('give_default_offline_donation_content', $default_text);
406 406
 
407 407
 }
408 408
 
@@ -415,34 +415,34 @@  discard block
 block discarded – undo
415 415
  */
416 416
 function give_get_default_offline_donation_email_content() {
417 417
 
418
-	$sitename      = get_bloginfo( 'sitename' );
419
-	$default_text  = '<p>' . __( 'Dear {name},', 'give' ) . '</p>';
420
-	$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>';
418
+	$sitename      = get_bloginfo('sitename');
419
+	$default_text  = '<p>'.__('Dear {name},', 'give').'</p>';
420
+	$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>';
421 421
 	$default_text .= '<ol>';
422 422
 	$default_text .= '<li>';
423 423
 	$default_text .= sprintf(
424 424
 		/* translators: %s: site name */
425
-		__( 'Make a check payable to "%s"', 'give' ),
425
+		__('Make a check payable to "%s"', 'give'),
426 426
 		$sitename
427 427
 	);
428 428
 	$default_text .= '</li>';
429 429
 	$default_text .= '<li>';
430 430
 	$default_text .= sprintf(
431 431
 		/* translators: %s: site name */
432
-		__( 'On the memo line of the check, please indicate that the donation is for "%s"', 'give' ),
432
+		__('On the memo line of the check, please indicate that the donation is for "%s"', 'give'),
433 433
 		$sitename
434 434
 	);
435 435
 	$default_text .= '</li>';
436
-	$default_text .= '<li>' . __( 'Please mail your check to:', 'give' ) . '</li>';
436
+	$default_text .= '<li>'.__('Please mail your check to:', 'give').'</li>';
437 437
 	$default_text .= '</ol>';
438
-	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>' . $sitename . '</em><br>';
438
+	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>'.$sitename.'</em><br>';
439 439
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>123 G Street </em><br>';
440 440
 	$default_text .= '&nbsp;&nbsp;&nbsp;&nbsp;<em>San Diego, CA 92101 </em><br>';
441
-	$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>';
442
-	$default_text .= '<p>' . __( 'Sincerely,', 'give' ) . '</p>';
443
-	$default_text .= '<p>' . $sitename . '</p>';
441
+	$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>';
442
+	$default_text .= '<p>'.__('Sincerely,', 'give').'</p>';
443
+	$default_text .= '<p>'.$sitename.'</p>';
444 444
 
445
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
445
+	return apply_filters('give_default_offline_donation_content', $default_text);
446 446
 
447 447
 }
448 448
 
@@ -456,17 +456,17 @@  discard block
 block discarded – undo
456 456
  *
457 457
  * @return string
458 458
  */
459
-function give_offline_donation_receipt_status_notice( $notice, $id ) {
460
-	$payment = new Give_Payment( $id );
459
+function give_offline_donation_receipt_status_notice($notice, $id) {
460
+	$payment = new Give_Payment($id);
461 461
 
462
-	if ( 'offline' !== $payment->gateway || $payment->is_completed() ) {
462
+	if ('offline' !== $payment->gateway || $payment->is_completed()) {
463 463
 		return $notice;
464 464
 	}
465 465
 
466
-	return give_output_error( __('Payment Pending: Please follow the instructions below to complete your donation.', 'give'), false, 'warning' );
466
+	return give_output_error(__('Payment Pending: Please follow the instructions below to complete your donation.', 'give'), false, 'warning');
467 467
 }
468 468
 
469
-add_filter( 'give_receipt_status_notice', 'give_offline_donation_receipt_status_notice', 10, 2 );
469
+add_filter('give_receipt_status_notice', 'give_offline_donation_receipt_status_notice', 10, 2);
470 470
 
471 471
 /**
472 472
  * Add offline payment instruction on payment receipt.
@@ -477,26 +477,26 @@  discard block
 block discarded – undo
477 477
  *
478 478
  * @return mixed
479 479
  */
480
-function give_offline_payment_receipt_after( $payment ) {
480
+function give_offline_payment_receipt_after($payment) {
481 481
 	// Get payment object.
482
-	$payment = new Give_Payment( $payment->ID );
482
+	$payment = new Give_Payment($payment->ID);
483 483
 
484 484
 	// Bailout.
485
-	if ( 'offline' !== $payment->gateway ) {
485
+	if ('offline' !== $payment->gateway) {
486 486
 		return false;
487 487
 	}
488 488
 
489 489
 	?>
490 490
 	<tr>
491
-		<td scope="row"><strong><?php esc_html_e( 'Offline Payment Instruction:', 'give' ); ?></strong></td>
491
+		<td scope="row"><strong><?php esc_html_e('Offline Payment Instruction:', 'give'); ?></strong></td>
492 492
 		<td>
493
-			<?php echo give_get_offline_payment_instruction( $payment->form_id, true ); ?>
493
+			<?php echo give_get_offline_payment_instruction($payment->form_id, true); ?>
494 494
 		</td>
495 495
 	</tr>
496 496
 	<?php
497 497
 }
498 498
 
499
-add_filter( 'give_payment_receipt_after', 'give_offline_payment_receipt_after' );
499
+add_filter('give_payment_receipt_after', 'give_offline_payment_receipt_after');
500 500
 
501 501
 /**
502 502
  * Get offline payment instructions.
@@ -508,27 +508,27 @@  discard block
 block discarded – undo
508 508
  *
509 509
  * @return string
510 510
  */
511
-function give_get_offline_payment_instruction( $form_id, $wpautop = false ) {
511
+function give_get_offline_payment_instruction($form_id, $wpautop = false) {
512 512
 	// Bailout.
513
-	if ( ! $form_id ) {
513
+	if ( ! $form_id) {
514 514
 		return '';
515 515
 	}
516 516
 
517
-	$post_offline_customization_option = get_post_meta( $form_id, '_give_customize_offline_donations', true );
518
-	$post_offline_instructions         = get_post_meta( $form_id, '_give_offline_checkout_notes', true );
519
-	$global_offline_instruction        = give_get_option( 'global_offline_donation_content' );
517
+	$post_offline_customization_option = get_post_meta($form_id, '_give_customize_offline_donations', true);
518
+	$post_offline_instructions         = get_post_meta($form_id, '_give_offline_checkout_notes', true);
519
+	$global_offline_instruction        = give_get_option('global_offline_donation_content');
520 520
 	$offline_instructions              = $global_offline_instruction;
521 521
 
522
-	if ( $post_offline_customization_option == 'yes' ) {
522
+	if ($post_offline_customization_option == 'yes') {
523 523
 		$offline_instructions = $post_offline_instructions;
524 524
 	}
525 525
 
526
-	$settings_url = admin_url( 'post.php?post=' . $form_id . '&action=edit&message=1' );
526
+	$settings_url = admin_url('post.php?post='.$form_id.'&action=edit&message=1');
527 527
 
528 528
 	/* translators: %s: form settings url */
529
-	$offline_instructions = ! empty( $offline_instructions ) ? $offline_instructions : sprintf( __( 'Please enter offline donation instructions in <a href="%s">this form\'s settings</a>.', 'give' ), $settings_url );
529
+	$offline_instructions = ! empty($offline_instructions) ? $offline_instructions : sprintf(__('Please enter offline donation instructions in <a href="%s">this form\'s settings</a>.', 'give'), $settings_url);
530 530
 
531
-	return ( $wpautop ? wpautop( $offline_instructions ) : $offline_instructions );
531
+	return ($wpautop ? wpautop($offline_instructions) : $offline_instructions);
532 532
 }
533 533
 
534 534
 
@@ -542,24 +542,24 @@  discard block
 block discarded – undo
542 542
  *
543 543
  * @return array
544 544
  */
545
-function give_filter_offline_gateway( $gateway_list, $form_id ) {
545
+function give_filter_offline_gateway($gateway_list, $form_id) {
546 546
 	if (
547 547
 		// Show offline payment gateway if enable for new donation form.
548
-		( false === strpos( $_SERVER['REQUEST_URI'], '/wp-admin/post-new.php?post_type=give_forms' ) )
548
+		(false === strpos($_SERVER['REQUEST_URI'], '/wp-admin/post-new.php?post_type=give_forms'))
549 549
 		&& $form_id
550
-		&& ! give_is_setting_enabled( get_post_meta( $form_id, '_give_customize_offline_donations', true ), array(
550
+		&& ! give_is_setting_enabled(get_post_meta($form_id, '_give_customize_offline_donations', true), array(
551 551
 			'enabled',
552 552
 			'global',
553
-		) )
553
+		))
554 554
 	) {
555
-		unset( $gateway_list['offline'] );
555
+		unset($gateway_list['offline']);
556 556
 	}
557 557
 
558 558
 	// Output.
559 559
 	return $gateway_list;
560 560
 }
561 561
 
562
-add_filter( 'give_enabled_payment_gateways', 'give_filter_offline_gateway', 10, 2 );
562
+add_filter('give_enabled_payment_gateways', 'give_filter_offline_gateway', 10, 2);
563 563
 
564 564
 /**
565 565
  * Set default gateway to global default payment gateway
@@ -573,10 +573,10 @@  discard block
 block discarded – undo
573 573
  *
574 574
  * @return void
575 575
  */
576
-function _give_customize_offline_donations_on_save_callback( $meta_key, $meta_value, $postid ) {
577
-	if ( ( 'no' === $meta_value ) && ( 'offline' === get_post_meta( $postid, '_give_default_gateway', true ) ) ) {
578
-		update_post_meta( $postid, '_give_default_gateway', 'global' );
576
+function _give_customize_offline_donations_on_save_callback($meta_key, $meta_value, $postid) {
577
+	if (('no' === $meta_value) && ('offline' === get_post_meta($postid, '_give_default_gateway', true))) {
578
+		update_post_meta($postid, '_give_default_gateway', 'global');
579 579
 	}
580 580
 }
581 581
 
582
-add_filter( 'give_save__give_customize_offline_donations', '_give_customize_offline_donations_on_save_callback', 10, 3 );
582
+add_filter('give_save__give_customize_offline_donations', '_give_customize_offline_donations_on_save_callback', 10, 3);
Please login to merge, or discard this patch.
includes/admin/abstract-admin-settings-page.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@  discard block
 block discarded – undo
9 9
  * @since       1.8
10 10
  */
11 11
 
12
-if ( ! defined( 'ABSPATH' ) ) {
12
+if ( ! defined('ABSPATH')) {
13 13
 	exit; // Exit if accessed directly
14 14
 }
15 15
 
16
-if ( ! class_exists( 'Give_Settings_Page' ) ) :
16
+if ( ! class_exists('Give_Settings_Page')) :
17 17
 
18 18
 	/**
19 19
 	 * Give_Settings_Page.
@@ -62,11 +62,11 @@  discard block
 block discarded – undo
62 62
 			// Get current setting page.
63 63
 			$this->current_setting_page = give_get_current_setting_page();
64 64
 
65
-			add_filter( "give_default_setting_tab_section_{$this->id}", array( $this, 'set_default_setting_tab' ), 10 );
66
-			add_filter( "{$this->current_setting_page}_tabs_array", array( $this, 'add_settings_page' ), 20 );
67
-			add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array( $this, 'output_sections' ) );
68
-			add_action( "{$this->current_setting_page}_settings_{$this->id}_page", array( $this, 'output' ) );
69
-			add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
65
+			add_filter("give_default_setting_tab_section_{$this->id}", array($this, 'set_default_setting_tab'), 10);
66
+			add_filter("{$this->current_setting_page}_tabs_array", array($this, 'add_settings_page'), 20);
67
+			add_action("{$this->current_setting_page}_sections_{$this->id}_page", array($this, 'output_sections'));
68
+			add_action("{$this->current_setting_page}_settings_{$this->id}_page", array($this, 'output'));
69
+			add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
70 70
 		}
71 71
 
72 72
 		/**
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 		 *
79 79
 		 * @return string
80 80
 		 */
81
-		function set_default_setting_tab( $setting_tab ) {
81
+		function set_default_setting_tab($setting_tab) {
82 82
 			return $this->default_tab;
83 83
 		}
84 84
 
@@ -91,8 +91,8 @@  discard block
 block discarded – undo
91 91
 		 *
92 92
 		 * @return array
93 93
 		 */
94
-		public function add_settings_page( $pages ) {
95
-			$pages[ $this->id ] = $this->label;
94
+		public function add_settings_page($pages) {
95
+			$pages[$this->id] = $this->label;
96 96
 
97 97
 			return $pages;
98 98
 		}
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 			 *
112 112
 			 * @param  array $settings
113 113
 			 */
114
-			$settings = apply_filters( 'give_get_settings_' . $this->id, array() );
114
+			$settings = apply_filters('give_get_settings_'.$this->id, array());
115 115
 
116 116
 			// Output.
117 117
 			return $settings;
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 		 * @return array
125 125
 		 */
126 126
 		public function get_sections() {
127
-			return apply_filters( 'give_get_sections_' . $this->id, array() );
127
+			return apply_filters('give_get_sections_'.$this->id, array());
128 128
 		}
129 129
 
130 130
 		/**
@@ -141,29 +141,29 @@  discard block
 block discarded – undo
141 141
 			$sections = $this->get_sections();
142 142
 
143 143
 			// Show section settings only if setting section exist.
144
-			if ( $current_section && ! in_array( $current_section, array_keys( $sections ) ) ) {
145
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
144
+			if ($current_section && ! in_array($current_section, array_keys($sections))) {
145
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
146 146
 				$GLOBALS['give_hide_save_button'] = true;
147 147
 
148 148
 				return;
149 149
 			}
150 150
 
151 151
 			// Bailout.
152
-			if ( empty( $sections ) ) {
152
+			if (empty($sections)) {
153 153
 				return;
154 154
 			}
155 155
 
156
-			if ( is_null( $this->current_setting_page ) ) {
156
+			if (is_null($this->current_setting_page)) {
157 157
 				$this->current_setting_page = give_get_current_setting_page();
158 158
 			}
159 159
 
160 160
 			echo '<ul class="subsubsub">';
161 161
 
162 162
 			// Get section keys.
163
-			$array_keys = array_keys( $sections );
163
+			$array_keys = array_keys($sections);
164 164
 
165
-			foreach ( $sections as $id => $label ) {
166
-				echo '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=' . $this->current_setting_page . '&tab=' . $this->id . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $current_section == $id ? 'current' : '' ) . '">' . $label . '</a> ' . ( end( $array_keys ) == $id ? '' : '|' ) . ' </li>';
165
+			foreach ($sections as $id => $label) {
166
+				echo '<li><a href="'.admin_url('edit.php?post_type=give_forms&page='.$this->current_setting_page.'&tab='.$this->id.'&section='.sanitize_title($id)).'" class="'.($current_section == $id ? 'current' : '').'">'.$label.'</a> '.(end($array_keys) == $id ? '' : '|').' </li>';
167 167
 			}
168 168
 
169 169
 			echo '</ul><br class="clear" /><hr>';
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
 		public function output() {
179 179
 			$settings = $this->get_settings();
180 180
 
181
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
181
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
182 182
 		}
183 183
 
184 184
 		/**
@@ -191,14 +191,14 @@  discard block
 block discarded – undo
191 191
 			$settings        = $this->get_settings();
192 192
 			$current_section = give_get_current_setting_section();
193 193
 
194
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
194
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
195 195
 
196 196
 			/**
197 197
 			 * Trigger Action
198 198
 			 *
199 199
 			 * @since 1.8
200 200
 			 */
201
-			do_action( 'give_update_options_' . $this->id . '_' . $current_section );
201
+			do_action('give_update_options_'.$this->id.'_'.$current_section);
202 202
 		}
203 203
 	}
204 204
 
Please login to merge, or discard this patch.
includes/admin/reporting/class-settings-export.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@  discard block
 block discarded – undo
9 9
  * @since       1.8
10 10
  */
11 11
 
12
-if ( ! defined( 'ABSPATH' ) ) {
12
+if ( ! defined('ABSPATH')) {
13 13
 	exit; // Exit if accessed directly
14 14
 }
15 15
 
16
-if ( ! class_exists( 'Give_Settings_Export' ) ) :
16
+if ( ! class_exists('Give_Settings_Export')) :
17 17
 
18 18
 	/**
19 19
 	 * Give_Settings_Export.
@@ -43,16 +43,16 @@  discard block
 block discarded – undo
43 43
 		 */
44 44
 		public function __construct() {
45 45
 			$this->id    = 'export';
46
-			$this->label = esc_html__( 'Export', 'give' );
46
+			$this->label = esc_html__('Export', 'give');
47 47
 
48
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
49
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
50
-			add_action( 'give_admin_field_report_export', array( $this, 'render_report_export_field' ), 10, 2 );
48
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
49
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
50
+			add_action('give_admin_field_report_export', array($this, 'render_report_export_field'), 10, 2);
51 51
 
52 52
 			// Do not use main donor for this tab.
53
-			if( give_get_current_setting_tab() === $this->id ) {
54
-				add_action( 'give-reports_open_form', '__return_empty_string' );
55
-				add_action( 'give-reports_close_form', '__return_empty_string' );
53
+			if (give_get_current_setting_tab() === $this->id) {
54
+				add_action('give-reports_open_form', '__return_empty_string');
55
+				add_action('give-reports_close_form', '__return_empty_string');
56 56
 			}
57 57
 		}
58 58
 
@@ -63,8 +63,8 @@  discard block
 block discarded – undo
63 63
 		 * @param  array $pages Lst of pages.
64 64
 		 * @return array
65 65
 		 */
66
-		public function add_settings_page( $pages ) {
67
-			$pages[ $this->id ] = $this->label;
66
+		public function add_settings_page($pages) {
67
+			$pages[$this->id] = $this->label;
68 68
 
69 69
 			return $pages;
70 70
 		}
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 			 * @param  array $settings
87 87
 			 */
88 88
 			$settings = apply_filters(
89
-				'give_get_settings_' . $this->id,
89
+				'give_get_settings_'.$this->id,
90 90
 				array(
91 91
 					array(
92 92
 						'id'   => 'give_reports_export',
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 					),
96 96
 					array(
97 97
 						'id'   => 'export',
98
-						'name' => esc_html__( 'Export', 'give' ),
98
+						'name' => esc_html__('Export', 'give'),
99 99
 						'type' => 'report_export',
100 100
 					),
101 101
 					array(
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 		public function output() {
120 120
 			$settings = $this->get_settings();
121 121
 
122
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
122
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
123 123
 		}
124 124
 
125 125
 		/**
@@ -131,8 +131,8 @@  discard block
 block discarded – undo
131 131
 		 * @param $field
132 132
 		 * @param $option_value
133 133
 		 */
134
-		public function render_report_export_field( $field, $option_value ) {
135
-			do_action( 'give_reports_tab_export');
134
+		public function render_report_export_field($field, $option_value) {
135
+			do_action('give_reports_tab_export');
136 136
 		}
137 137
 	}
138 138
 
Please login to merge, or discard this patch.
includes/admin/reporting/reports.php 1 patch
Spacing   +108 added lines, -108 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
  */
17 17
 
18 18
 // Exit if accessed directly.
19
-if ( ! defined( 'ABSPATH' ) ) {
19
+if ( ! defined('ABSPATH')) {
20 20
 	exit;
21 21
 }
22 22
 
@@ -29,8 +29,8 @@  discard block
 block discarded – undo
29 29
  * @return void
30 30
  */
31 31
 function give_reports_page() {
32
-	$current_page = admin_url( 'edit.php?post_type=give_forms&page=give-reports' );
33
-	$active_tab   = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
32
+	$current_page = admin_url('edit.php?post_type=give_forms&page=give-reports');
33
+	$active_tab   = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
34 34
 	$views        = give_reports_default_views();
35 35
 	?>
36 36
 	<div class="wrap give-settings-page">
@@ -38,17 +38,17 @@  discard block
 block discarded – undo
38 38
 		<h1 class="screen-reader-text"><?php echo get_admin_page_title(); ?></h1>
39 39
 
40 40
 		<h2 class="nav-tab-wrapper">
41
-			<?php foreach ( $views as $tab => $label ) { ?>
42
-				<a href="<?php echo esc_url( add_query_arg( array(
41
+			<?php foreach ($views as $tab => $label) { ?>
42
+				<a href="<?php echo esc_url(add_query_arg(array(
43 43
 					'tab'              => $tab,
44 44
 					'settings-updated' => false,
45
-				), $current_page ) ); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php echo esc_html( $label ); ?></a>
45
+				), $current_page)); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php echo esc_html($label); ?></a>
46 46
 			<?php } ?>
47
-			<?php if ( current_user_can( 'export_give_reports' ) ) { ?>
48
-				<a href="<?php echo esc_url( add_query_arg( array(
47
+			<?php if (current_user_can('export_give_reports')) { ?>
48
+				<a href="<?php echo esc_url(add_query_arg(array(
49 49
 					'tab'              => 'export',
50 50
 					'settings-updated' => false,
51
-				), $current_page ) ); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php esc_html_e( 'Export', 'give' ); ?></a>
51
+				), $current_page)); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php esc_html_e('Export', 'give'); ?></a>
52 52
 			<?php }
53 53
 			/**
54 54
 			 * Fires in the report tabs.
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 			 *
58 58
 			 * @since 1.0
59 59
 			 */
60
-			do_action( 'give_reports_tabs' );
60
+			do_action('give_reports_tabs');
61 61
 			?>
62 62
 		</h2>
63 63
 
@@ -67,10 +67,10 @@  discard block
 block discarded – undo
67 67
 		 *
68 68
 		 * @since 1.0
69 69
 		 */
70
-		do_action( 'give_reports_page_top' );
70
+		do_action('give_reports_page_top');
71 71
 
72 72
 		// Set $active_tab prior to hook firing.
73
-		if ( in_array( $active_tab, array_keys( $views ) ) ) {
73
+		if (in_array($active_tab, array_keys($views))) {
74 74
 			$active_tab = 'reports';
75 75
 		}
76 76
 
@@ -79,14 +79,14 @@  discard block
 block discarded – undo
79 79
 		 *
80 80
 		 * @since 1.0
81 81
 		 */
82
-		do_action( "give_reports_tab_{$active_tab}" );
82
+		do_action("give_reports_tab_{$active_tab}");
83 83
 
84 84
 		/**
85 85
 		 * Fires after the report page.
86 86
 		 *
87 87
 		 * @since 1.0
88 88
 		 */
89
-		do_action( 'give_reports_page_bottom' );
89
+		do_action('give_reports_page_bottom');
90 90
 		?>
91 91
 	</div><!-- .wrap -->
92 92
 	<?php
@@ -100,13 +100,13 @@  discard block
 block discarded – undo
100 100
  */
101 101
 function give_reports_default_views() {
102 102
 	$views = array(
103
-		'earnings' => esc_html__( 'Income', 'give' ),
104
-		'forms'    => esc_html__( 'Forms', 'give' ),
105
-		'donors'   => esc_html__( 'Donors', 'give' ),
106
-		'gateways' => esc_html__( 'Donation Methods', 'give' ),
103
+		'earnings' => esc_html__('Income', 'give'),
104
+		'forms'    => esc_html__('Forms', 'give'),
105
+		'donors'   => esc_html__('Donors', 'give'),
106
+		'gateways' => esc_html__('Donation Methods', 'give'),
107 107
 	);
108 108
 
109
-	$views = apply_filters( 'give_report_views', $views );
109
+	$views = apply_filters('give_report_views', $views);
110 110
 
111 111
 	return $views;
112 112
 }
@@ -121,15 +121,15 @@  discard block
 block discarded – undo
121 121
  * @since 1.0
122 122
  * @return string $view Report View
123 123
  */
124
-function give_get_reporting_view( $default = 'earnings' ) {
124
+function give_get_reporting_view($default = 'earnings') {
125 125
 
126
-	if ( ! isset( $_GET['view'] ) || ! in_array( $_GET['view'], array_keys( give_reports_default_views() ) ) ) {
126
+	if ( ! isset($_GET['view']) || ! in_array($_GET['view'], array_keys(give_reports_default_views()))) {
127 127
 		$view = $default;
128 128
 	} else {
129 129
 		$view = $_GET['view'];
130 130
 	}
131 131
 
132
-	return apply_filters( 'give_get_reporting_view', $view );
132
+	return apply_filters('give_get_reporting_view', $view);
133 133
 }
134 134
 
135 135
 /**
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 	$current_view = 'earnings';
143 143
 	$views        = give_reports_default_views();
144 144
 
145
-	if ( isset( $_GET['tab'] ) && array_key_exists( $_GET['tab'], $views ) ) {
145
+	if (isset($_GET['tab']) && array_key_exists($_GET['tab'], $views)) {
146 146
 		$current_view = $_GET['tab'];
147 147
 	}
148 148
 
@@ -151,10 +151,10 @@  discard block
 block discarded – undo
151 151
 	 *
152 152
 	 * @since 1.0
153 153
 	 */
154
-	do_action( "give_reports_view_{$current_view}" );
154
+	do_action("give_reports_view_{$current_view}");
155 155
 }
156 156
 
157
-add_action( 'give_reports_tab_reports', 'give_reports_tab_reports' );
157
+add_action('give_reports_tab_reports', 'give_reports_tab_reports');
158 158
 
159 159
 /**
160 160
  * Renders the Reports Page Views Drop Downs
@@ -164,19 +164,19 @@  discard block
 block discarded – undo
164 164
  */
165 165
 function give_report_views() {
166 166
 	$views        = give_reports_default_views();
167
-	$current_view = isset( $_GET['view'] ) ? $_GET['view'] : 'earnings';
167
+	$current_view = isset($_GET['view']) ? $_GET['view'] : 'earnings';
168 168
 	/**
169 169
 	 * Fires before the report page actions form.
170 170
 	 *
171 171
 	 * @since 1.0
172 172
 	 */
173
-	do_action( 'give_report_view_actions_before' );
173
+	do_action('give_report_view_actions_before');
174 174
 	?>
175 175
 	<form id="give-reports-filter" method="get">
176 176
 		<select id="give-reports-view" name="view">
177
-			<option value="-1"><?php esc_html_e( 'Report Type', 'give' ); ?></option>
178
-			<?php foreach ( $views as $view_id => $label ) : ?>
179
-				<option value="<?php echo esc_attr( $view_id ); ?>" <?php selected( $view_id, $current_view ); ?>><?php echo $label; ?></option>
177
+			<option value="-1"><?php esc_html_e('Report Type', 'give'); ?></option>
178
+			<?php foreach ($views as $view_id => $label) : ?>
179
+				<option value="<?php echo esc_attr($view_id); ?>" <?php selected($view_id, $current_view); ?>><?php echo $label; ?></option>
180 180
 			<?php endforeach; ?>
181 181
 		</select>
182 182
 
@@ -188,12 +188,12 @@  discard block
 block discarded – undo
188 188
 		 *
189 189
 		 * @since 1.0
190 190
 		 */
191
-		do_action( 'give_report_view_actions' );
191
+		do_action('give_report_view_actions');
192 192
 		?>
193 193
 
194 194
 		<input type="hidden" name="post_type" value="give_forms"/>
195 195
 		<input type="hidden" name="page" value="give-reports"/>
196
-		<?php submit_button( esc_html__( 'Show', 'give' ), 'secondary', 'submit', false ); ?>
196
+		<?php submit_button(esc_html__('Show', 'give'), 'secondary', 'submit', false); ?>
197 197
 	</form>
198 198
 	<?php
199 199
 	/**
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 	 *
202 202
 	 * @since 1.0
203 203
 	 */
204
-	do_action( 'give_report_view_actions_after' );
204
+	do_action('give_report_view_actions_after');
205 205
 }
206 206
 
207 207
 /**
@@ -214,18 +214,18 @@  discard block
 block discarded – undo
214 214
  */
215 215
 function give_reports_forms_table() {
216 216
 
217
-	if ( isset( $_GET['form-id'] ) ) {
217
+	if (isset($_GET['form-id'])) {
218 218
 		return;
219 219
 	}
220 220
 
221
-	include( dirname( __FILE__ ) . '/class-form-reports-table.php' );
221
+	include(dirname(__FILE__).'/class-form-reports-table.php');
222 222
 
223 223
 	$give_table = new Give_Form_Reports_Table();
224 224
 	$give_table->prepare_items();
225 225
 	$give_table->display();
226 226
 }
227 227
 
228
-add_action( 'give_reports_view_forms', 'give_reports_forms_table' );
228
+add_action('give_reports_view_forms', 'give_reports_forms_table');
229 229
 
230 230
 /**
231 231
  * Renders the detailed report for a specific give form
@@ -234,20 +234,20 @@  discard block
 block discarded – undo
234 234
  * @return void
235 235
  */
236 236
 function give_reports_form_details() {
237
-	if ( ! isset( $_GET['form-id'] ) ) {
237
+	if ( ! isset($_GET['form-id'])) {
238 238
 		return;
239 239
 	}
240 240
 	?>
241 241
 	<div class="tablenav top reports-forms-details-wrap">
242 242
 		<div class="actions bulkactions">
243
-			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e( 'Go Back', 'give' ); ?></button>
243
+			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e('Go Back', 'give'); ?></button>
244 244
 		</div>
245 245
 	</div>
246 246
 	<?php
247
-	give_reports_graph_of_form( absint( $_GET['form-id'] ) );
247
+	give_reports_graph_of_form(absint($_GET['form-id']));
248 248
 }
249 249
 
250
-add_action( 'give_reports_view_forms', 'give_reports_form_details' );
250
+add_action('give_reports_view_forms', 'give_reports_form_details');
251 251
 
252 252
 /**
253 253
  * Renders the Reports Donors Table
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
  * @return void
259 259
  */
260 260
 function give_reports_donors_table() {
261
-	include( dirname( __FILE__ ) . '/class-donor-reports-table.php' );
261
+	include(dirname(__FILE__).'/class-donor-reports-table.php');
262 262
 
263 263
 	$give_table = new Give_Donor_Reports_Table();
264 264
 	$give_table->prepare_items();
@@ -270,11 +270,11 @@  discard block
 block discarded – undo
270 270
 		 *
271 271
 		 * @since 1.0
272 272
 		 */
273
-		do_action( 'give_logs_donors_table_top' );
273
+		do_action('give_logs_donors_table_top');
274 274
 		?>
275 275
 		<form id="give-donors-filter" method="get">
276 276
 			<?php
277
-			$give_table->search_box( esc_html__( 'Search', 'give' ), 'give-donors' );
277
+			$give_table->search_box(esc_html__('Search', 'give'), 'give-donors');
278 278
 			$give_table->display();
279 279
 			?>
280 280
 			<input type="hidden" name="post_type" value="give_forms"/>
@@ -287,13 +287,13 @@  discard block
 block discarded – undo
287 287
 		 *
288 288
 		 * @since 1.0
289 289
 		 */
290
-		do_action( 'give_logs_donors_table_bottom' );
290
+		do_action('give_logs_donors_table_bottom');
291 291
 		?>
292 292
 	</div>
293 293
 	<?php
294 294
 }
295 295
 
296
-add_action( 'give_reports_view_donors', 'give_reports_donors_table' );
296
+add_action('give_reports_view_donors', 'give_reports_donors_table');
297 297
 
298 298
 
299 299
 /**
@@ -305,14 +305,14 @@  discard block
 block discarded – undo
305 305
  * @return void
306 306
  */
307 307
 function give_reports_gateways_table() {
308
-	include( dirname( __FILE__ ) . '/class-gateways-reports-table.php' );
308
+	include(dirname(__FILE__).'/class-gateways-reports-table.php');
309 309
 
310 310
 	$give_table = new Give_Gateawy_Reports_Table();
311 311
 	$give_table->prepare_items();
312 312
 	$give_table->display();
313 313
 }
314 314
 
315
-add_action( 'give_reports_view_gateways', 'give_reports_gateways_table' );
315
+add_action('give_reports_view_gateways', 'give_reports_gateways_table');
316 316
 
317 317
 
318 318
 /**
@@ -324,13 +324,13 @@  discard block
 block discarded – undo
324 324
 function give_reports_earnings() {
325 325
 	?>
326 326
 	<div class="tablenav top reports-table-nav">
327
-		<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e( 'Income Report', 'give' ); ?></span></h3>
327
+		<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e('Income Report', 'give'); ?></span></h3>
328 328
 	</div>
329 329
 	<?php
330 330
 	give_reports_graph();
331 331
 }
332 332
 
333
-add_action( 'give_reports_view_earnings', 'give_reports_earnings' );
333
+add_action('give_reports_view_earnings', 'give_reports_earnings');
334 334
 
335 335
 
336 336
 /**
@@ -351,14 +351,14 @@  discard block
 block discarded – undo
351 351
 				 *
352 352
 				 * @since 1.0
353 353
 				 */
354
-				do_action( 'give_reports_tab_export_content_top' );
354
+				do_action('give_reports_tab_export_content_top');
355 355
 				?>
356 356
 
357 357
 				<table class="widefat export-options-table give-table">
358 358
 					<thead>
359 359
 						<tr>
360
-							<th scope="col"><?php esc_html_e( 'Export Type', 'give' ); ?></th>
361
-							<th scope="col"><?php esc_html_e( 'Export Options', 'give' ); ?></th>
360
+							<th scope="col"><?php esc_html_e('Export Type', 'give'); ?></th>
361
+							<th scope="col"><?php esc_html_e('Export Options', 'give'); ?></th>
362 362
 						</tr>
363 363
 					</thead>
364 364
 					<tbody>
@@ -370,42 +370,42 @@  discard block
 block discarded – undo
370 370
 						 *
371 371
 						 * @since 1.0
372 372
 						 */
373
-						do_action( 'give_reports_tab_export_table_top' );
373
+						do_action('give_reports_tab_export_table_top');
374 374
 						?>
375 375
 						<tr class="give-export-pdf-sales-earnings">
376 376
 							<td scope="row" class="row-title">
377
-								<h3><span><?php esc_html_e( 'Export PDF of Donations and Income', 'give' ); ?></span>
377
+								<h3><span><?php esc_html_e('Export PDF of Donations and Income', 'give'); ?></span>
378 378
 								</h3>
379
-								<p><?php esc_html_e( 'Download a PDF of Donations and Income reports for all forms for the current year.', 'give' ); ?></p>
379
+								<p><?php esc_html_e('Download a PDF of Donations and Income reports for all forms for the current year.', 'give'); ?></p>
380 380
 							</td>
381 381
 							<td>
382
-								<a class="button" href="<?php echo wp_nonce_url( add_query_arg( array( 'give-action' => 'generate_pdf' ) ), 'give_generate_pdf' ); ?>"><?php esc_html_e( 'Generate PDF', 'give' ); ?></a>
382
+								<a class="button" href="<?php echo wp_nonce_url(add_query_arg(array('give-action' => 'generate_pdf')), 'give_generate_pdf'); ?>"><?php esc_html_e('Generate PDF', 'give'); ?></a>
383 383
 							</td>
384 384
 						</tr>
385 385
 						<tr class="alternate give-export-sales-earnings">
386 386
 							<td scope="row" class="row-title">
387
-								<h3><span><?php esc_html_e( 'Export Income and Donation Stats', 'give' ); ?></span></h3>
388
-								<p><?php esc_html_e( 'Download a CSV of income and donations over time.', 'give' ); ?></p>
387
+								<h3><span><?php esc_html_e('Export Income and Donation Stats', 'give'); ?></span></h3>
388
+								<p><?php esc_html_e('Download a CSV of income and donations over time.', 'give'); ?></p>
389 389
 							</td>
390 390
 							<td>
391 391
 								<form method="post">
392 392
 									<?php
393 393
 									printf(
394 394
 									/* translators: 1: start date dropdown 2: end date dropdown */
395
-										esc_html__( '%1$s to %2$s', 'give' ),
396
-										Give()->html->year_dropdown( 'start_year' ) . ' ' . Give()->html->month_dropdown( 'start_month' ),
397
-										Give()->html->year_dropdown( 'end_year' ) . ' ' . Give()->html->month_dropdown( 'end_month' )
395
+										esc_html__('%1$s to %2$s', 'give'),
396
+										Give()->html->year_dropdown('start_year').' '.Give()->html->month_dropdown('start_month'),
397
+										Give()->html->year_dropdown('end_year').' '.Give()->html->month_dropdown('end_month')
398 398
 									);
399 399
 									?>
400 400
 									<input type="hidden" name="give-action" value="earnings_export"/>
401
-									<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>" class="button-secondary"/>
401
+									<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>" class="button-secondary"/>
402 402
 								</form>
403 403
 							</td>
404 404
 						</tr>
405 405
 						<tr class="give-export-payment-history">
406 406
 							<td scope="row" class="row-title">
407
-								<h3><span><?php esc_html_e( 'Export Donation History', 'give' ); ?></span></h3>
408
-								<p><?php esc_html_e( 'Download a CSV of all donations recorded.', 'give' ); ?></p>
407
+								<h3><span><?php esc_html_e('Export Donation History', 'give'); ?></span></h3>
408
+								<p><?php esc_html_e('Download a CSV of all donations recorded.', 'give'); ?></p>
409 409
 							</td>
410 410
 							<td>
411 411
 								<form id="give-export-payments" class="give-export-form" method="post">
@@ -413,27 +413,27 @@  discard block
 block discarded – undo
413 413
 									$args = array(
414 414
 										'id'          => 'give-payment-export-start',
415 415
 										'name'        => 'start',
416
-										'placeholder' => esc_attr__( 'Start date', 'give' ),
416
+										'placeholder' => esc_attr__('Start date', 'give'),
417 417
 									);
418
-									echo Give()->html->date_field( $args ); ?>
418
+									echo Give()->html->date_field($args); ?>
419 419
 									<?php
420 420
 									$args = array(
421 421
 										'id'          => 'give-payment-export-end',
422 422
 										'name'        => 'end',
423
-										'placeholder' => esc_attr__( 'End date', 'give' ),
423
+										'placeholder' => esc_attr__('End date', 'give'),
424 424
 									);
425
-									echo Give()->html->date_field( $args ); ?>
425
+									echo Give()->html->date_field($args); ?>
426 426
 									<select name="status">
427
-										<option value="any"><?php esc_html_e( 'All Statuses', 'give' ); ?></option>
427
+										<option value="any"><?php esc_html_e('All Statuses', 'give'); ?></option>
428 428
 										<?php
429 429
 										$statuses = give_get_payment_statuses();
430
-										foreach ( $statuses as $status => $label ) {
431
-											echo '<option value="' . $status . '">' . $label . '</option>';
430
+										foreach ($statuses as $status => $label) {
431
+											echo '<option value="'.$status.'">'.$label.'</option>';
432 432
 										}
433 433
 										?>
434 434
 									</select>
435 435
 									<?php
436
-									if ( give_is_setting_enabled( give_get_option( 'categories' ) ) ) {
436
+									if (give_is_setting_enabled(give_get_option('categories'))) {
437 437
 										echo Give()->html->category_dropdown(
438 438
 											'give_forms_categories[]',
439 439
 											0,
@@ -443,13 +443,13 @@  discard block
 block discarded – undo
443 443
 												'multiple'        => true,
444 444
 												'selected'        => array(),
445 445
 												'show_option_all' => false,
446
-												'placeholder'     => __( 'Choose one or more from categories', 'give' ),
446
+												'placeholder'     => __('Choose one or more from categories', 'give'),
447 447
 											)
448 448
 										);
449 449
 										$add_break = true;
450 450
 									}
451 451
 
452
-									if ( give_is_setting_enabled( give_get_option( 'tags' ) ) ) {
452
+									if (give_is_setting_enabled(give_get_option('tags'))) {
453 453
 										echo Give()->html->tags_dropdown(
454 454
 											'give_forms_tags[]',
455 455
 											0,
@@ -459,17 +459,17 @@  discard block
 block discarded – undo
459 459
 												'multiple'        => true,
460 460
 												'selected'        => array(),
461 461
 												'show_option_all' => false,
462
-												'placeholder'     => __( 'Choose one or more from tags', 'give' ),
462
+												'placeholder'     => __('Choose one or more from tags', 'give'),
463 463
 											)
464 464
 										);
465 465
 										$add_break = true;
466 466
 									}
467 467
 
468
-									wp_nonce_field( 'give_ajax_export', 'give_ajax_export' );
468
+									wp_nonce_field('give_ajax_export', 'give_ajax_export');
469 469
 									?>
470 470
 									<input type="hidden" name="give-export-class" value="Give_Batch_Payments_Export"/>
471 471
 									<span>
472
-									<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>" class="button-secondary"/>
472
+									<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>" class="button-secondary"/>
473 473
 									<span class="spinner"></span>
474 474
 								</span>
475 475
 								</form>
@@ -477,8 +477,8 @@  discard block
 block discarded – undo
477 477
 						</tr>
478 478
 						<tr class="alternate give-export-donors">
479 479
 							<td scope="row" class="row-title">
480
-								<h3><span><?php esc_html_e( 'Export Donors in CSV', 'give' ); ?></span></h3>
481
-								<p><?php esc_html_e( 'Download an export of donors for all donation forms or only those who have given to a particular form.', 'give' ); ?></p>
480
+								<h3><span><?php esc_html_e('Export Donors in CSV', 'give'); ?></span></h3>
481
+								<p><?php esc_html_e('Download an export of donors for all donation forms or only those who have given to a particular form.', 'give'); ?></p>
482 482
 							</td>
483 483
 							<td>
484 484
 								<form method="post" id="give_donor_export" class="give-export-form">
@@ -489,46 +489,46 @@  discard block
 block discarded – undo
489 489
 										'id'     => 'give_customer_export_form',
490 490
 										'chosen' => true,
491 491
 									);
492
-									echo Give()->html->forms_dropdown( $args ); ?>
492
+									echo Give()->html->forms_dropdown($args); ?>
493 493
 
494
-									<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>" class="button-secondary"/>
494
+									<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>" class="button-secondary"/>
495 495
 
496 496
 									<div id="export-donor-options-wrap" class="give-clearfix">
497
-										<p><?php esc_html_e( 'Export Columns:', 'give' ); ?></p>
497
+										<p><?php esc_html_e('Export Columns:', 'give'); ?></p>
498 498
 										<ul id="give-export-option-ul">
499 499
 											<li>
500
-												<label for="give-export-fullname"><input type="checkbox" checked name="give_export_option[full_name]" id="give-export-fullname"><?php esc_html_e( 'Name', 'give' ); ?>
500
+												<label for="give-export-fullname"><input type="checkbox" checked name="give_export_option[full_name]" id="give-export-fullname"><?php esc_html_e('Name', 'give'); ?>
501 501
 												</label>
502 502
 											</li>
503 503
 											<li>
504
-												<label for="give-export-email"><input type="checkbox" checked name="give_export_option[email]" id="give-export-email"><?php esc_html_e( 'Email', 'give' ); ?>
504
+												<label for="give-export-email"><input type="checkbox" checked name="give_export_option[email]" id="give-export-email"><?php esc_html_e('Email', 'give'); ?>
505 505
 												</label>
506 506
 											</li>
507 507
 											<li>
508
-												<label for="give-export-address"><input type="checkbox" checked name="give_export_option[address]" id="give-export-address"><?php esc_html_e( 'Address', 'give' ); ?>
508
+												<label for="give-export-address"><input type="checkbox" checked name="give_export_option[address]" id="give-export-address"><?php esc_html_e('Address', 'give'); ?>
509 509
 												</label>
510 510
 											</li>
511 511
 											<li>
512
-												<label for="give-export-userid"><input type="checkbox" checked name="give_export_option[userid]" id="give-export-userid"><?php esc_html_e( 'User ID', 'give' ); ?>
512
+												<label for="give-export-userid"><input type="checkbox" checked name="give_export_option[userid]" id="give-export-userid"><?php esc_html_e('User ID', 'give'); ?>
513 513
 												</label>
514 514
 											</li>
515 515
 											<li>
516
-												<label for="give-export-first-donation-date"><input type="checkbox" checked name="give_export_option[date_first_donated]" id="give-export-first-donation-date"><?php esc_html_e( 'First Donation Date', 'give' ); ?>
516
+												<label for="give-export-first-donation-date"><input type="checkbox" checked name="give_export_option[date_first_donated]" id="give-export-first-donation-date"><?php esc_html_e('First Donation Date', 'give'); ?>
517 517
 												</label>
518 518
 											</li>
519 519
 											<li>
520
-												<label for="give-export-donation-number"><input type="checkbox" checked name="give_export_option[donations]" id="give-export-donation-number"><?php esc_html_e( 'Number of Donations', 'give' ); ?>
520
+												<label for="give-export-donation-number"><input type="checkbox" checked name="give_export_option[donations]" id="give-export-donation-number"><?php esc_html_e('Number of Donations', 'give'); ?>
521 521
 												</label>
522 522
 											</li>
523 523
 											<li>
524
-												<label for="give-export-donation-sum"><input type="checkbox" checked name="give_export_option[donation_sum]" id="give-export-donation-sum"><?php esc_html_e( 'Total Donated', 'give' ); ?>
524
+												<label for="give-export-donation-sum"><input type="checkbox" checked name="give_export_option[donation_sum]" id="give-export-donation-sum"><?php esc_html_e('Total Donated', 'give'); ?>
525 525
 												</label>
526 526
 											</li>
527 527
 										</ul>
528 528
 									</div>
529
-									<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
529
+									<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
530 530
 									<input type="hidden" name="give-export-class" value="Give_Batch_Customers_Export"/>
531
-									<input type="hidden" name="give_export_option[query_id]" value="<?php echo uniqid( 'give_' ); ?>"/>
531
+									<input type="hidden" name="give_export_option[query_id]" value="<?php echo uniqid('give_'); ?>"/>
532 532
 									<input type="hidden" name="give_action" value="email_export"/>
533 533
 								</form>
534 534
 							</td>
@@ -541,7 +541,7 @@  discard block
 block discarded – undo
541 541
 						 *
542 542
 						 * @since 1.0
543 543
 						 */
544
-						do_action( 'give_reports_tab_export_table_bottom' );
544
+						do_action('give_reports_tab_export_table_bottom');
545 545
 						?>
546 546
 					</tbody>
547 547
 				</table>
@@ -552,7 +552,7 @@  discard block
 block discarded – undo
552 552
 				 *
553 553
 				 * @since 1.0
554 554
 				 */
555
-				do_action( 'give_reports_tab_export_content_bottom' );
555
+				do_action('give_reports_tab_export_content_bottom');
556 556
 				?>
557 557
 
558 558
 			</div>
@@ -563,7 +563,7 @@  discard block
 block discarded – undo
563 563
 	<?php
564 564
 }
565 565
 
566
-add_action( 'give_reports_tab_export', 'give_reports_tab_export' );
566
+add_action('give_reports_tab_export', 'give_reports_tab_export');
567 567
 
568 568
 /**
569 569
  * Renders the Reports page
@@ -573,7 +573,7 @@  discard block
 block discarded – undo
573 573
  */
574 574
 function give_reports_tab_logs() {
575 575
 
576
-	require( GIVE_PLUGIN_DIR . 'includes/admin/reporting/logs.php' );
576
+	require(GIVE_PLUGIN_DIR.'includes/admin/reporting/logs.php');
577 577
 
578 578
 	// Get current section.
579 579
 	$current_section = $_GET['section'] = give_get_current_setting_section();
@@ -583,7 +583,7 @@  discard block
 block discarded – undo
583 583
 	 *
584 584
 	 * @since 1.0
585 585
 	 */
586
-	do_action( "give_logs_view_{$current_section}" );
586
+	do_action("give_logs_view_{$current_section}");
587 587
 }
588 588
 
589 589
 /**
@@ -594,9 +594,9 @@  discard block
 block discarded – undo
594 594
  */
595 595
 function give_estimated_monthly_stats() {
596 596
 
597
-	$estimated = get_transient( 'give_estimated_monthly_stats' );
597
+	$estimated = get_transient('give_estimated_monthly_stats');
598 598
 
599
-	if ( false === $estimated ) {
599
+	if (false === $estimated) {
600 600
 
601 601
 		$estimated = array(
602 602
 			'earnings' => 0,
@@ -605,22 +605,22 @@  discard block
 block discarded – undo
605 605
 
606 606
 		$stats = new Give_Payment_Stats;
607 607
 
608
-		$to_date_earnings = $stats->get_earnings( 0, 'this_month' );
609
-		$to_date_sales    = $stats->get_sales( 0, 'this_month' );
608
+		$to_date_earnings = $stats->get_earnings(0, 'this_month');
609
+		$to_date_sales    = $stats->get_sales(0, 'this_month');
610 610
 
611
-		$current_day   = date( 'd', current_time( 'timestamp' ) );
612
-		$current_month = date( 'n', current_time( 'timestamp' ) );
613
-		$current_year  = date( 'Y', current_time( 'timestamp' ) );
614
-		$days_in_month = cal_days_in_month( CAL_GREGORIAN, $current_month, $current_year );
611
+		$current_day   = date('d', current_time('timestamp'));
612
+		$current_month = date('n', current_time('timestamp'));
613
+		$current_year  = date('Y', current_time('timestamp'));
614
+		$days_in_month = cal_days_in_month(CAL_GREGORIAN, $current_month, $current_year);
615 615
 
616
-		$estimated['earnings'] = ( $to_date_earnings / $current_day ) * $days_in_month;
617
-		$estimated['sales']    = ( $to_date_sales / $current_day ) * $days_in_month;
616
+		$estimated['earnings'] = ($to_date_earnings / $current_day) * $days_in_month;
617
+		$estimated['sales']    = ($to_date_sales / $current_day) * $days_in_month;
618 618
 
619 619
 		// Cache for one day
620
-		set_transient( 'give_estimated_monthly_stats', $estimated, 86400 );
620
+		set_transient('give_estimated_monthly_stats', $estimated, 86400);
621 621
 	}
622 622
 
623
-	return maybe_unserialize( $estimated );
623
+	return maybe_unserialize($estimated);
624 624
 }
625 625
 
626 626
 // @TODO: After release 1.8 Donations -> Reports generates with new setting api, so we can remove some old code from this file.
Please login to merge, or discard this patch.
includes/admin/reporting/class-settings-gateways.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@  discard block
 block discarded – undo
9 9
  * @since       1.8
10 10
  */
11 11
 
12
-if ( ! defined( 'ABSPATH' ) ) {
12
+if ( ! defined('ABSPATH')) {
13 13
 	exit; // Exit if accessed directly
14 14
 }
15 15
 
16
-if ( ! class_exists( 'Give_Settings_Gateways' ) ) :
16
+if ( ! class_exists('Give_Settings_Gateways')) :
17 17
 
18 18
 	/**
19 19
 	 * Give_Settings_Gateways.
@@ -43,16 +43,16 @@  discard block
 block discarded – undo
43 43
 		 */
44 44
 		public function __construct() {
45 45
 			$this->id    = 'gateways';
46
-			$this->label = esc_html__( 'Donation Methods', 'give' );
46
+			$this->label = esc_html__('Donation Methods', 'give');
47 47
 
48
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
49
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
50
-			add_action( 'give_admin_field_report_gateways', array( $this, 'render_report_gateways_field' ), 10, 2 );
48
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
49
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
50
+			add_action('give_admin_field_report_gateways', array($this, 'render_report_gateways_field'), 10, 2);
51 51
 
52 52
 			// Do not use main form for this tab.
53
-			if( give_get_current_setting_tab() === $this->id ) {
54
-				add_action( 'give-reports_open_form', '__return_empty_string' );
55
-				add_action( 'give-reports_close_form', '__return_empty_string' );
53
+			if (give_get_current_setting_tab() === $this->id) {
54
+				add_action('give-reports_open_form', '__return_empty_string');
55
+				add_action('give-reports_close_form', '__return_empty_string');
56 56
 			}
57 57
 		}
58 58
 
@@ -63,8 +63,8 @@  discard block
 block discarded – undo
63 63
 		 * @param  array $pages Lst of pages.
64 64
 		 * @return array
65 65
 		 */
66
-		public function add_settings_page( $pages ) {
67
-			$pages[ $this->id ] = $this->label;
66
+		public function add_settings_page($pages) {
67
+			$pages[$this->id] = $this->label;
68 68
 
69 69
 			return $pages;
70 70
 		}
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 			 * @param  array $settings
87 87
 			 */
88 88
 			$settings = apply_filters(
89
-				'give_get_settings_' . $this->id,
89
+				'give_get_settings_'.$this->id,
90 90
 				array(
91 91
 					array(
92 92
 						'id'   => 'give_reports_gateways',
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 					),
96 96
 					array(
97 97
 						'id'   => 'gateways',
98
-						'name' => esc_html__( 'Gateways', 'give' ),
98
+						'name' => esc_html__('Gateways', 'give'),
99 99
 						'type' => 'report_gateways',
100 100
 					),
101 101
 					array(
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 		public function output() {
120 120
 			$settings = $this->get_settings();
121 121
 
122
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
122
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
123 123
 		}
124 124
 
125 125
 		/**
@@ -131,8 +131,8 @@  discard block
 block discarded – undo
131 131
 		 * @param $field
132 132
 		 * @param $option_value
133 133
 		 */
134
-		public function render_report_gateways_field( $field, $option_value ) {
135
-			do_action( 'give_reports_view_gateways');
134
+		public function render_report_gateways_field($field, $option_value) {
135
+			do_action('give_reports_view_gateways');
136 136
 		}
137 137
 	}
138 138
 
Please login to merge, or discard this patch.
includes/admin/reporting/class-settings-donors.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@  discard block
 block discarded – undo
9 9
  * @since       1.8
10 10
  */
11 11
 
12
-if ( ! defined( 'ABSPATH' ) ) {
12
+if ( ! defined('ABSPATH')) {
13 13
 	exit; // Exit if accessed directly
14 14
 }
15 15
 
16
-if ( ! class_exists( 'Give_Settings_Donors' ) ) :
16
+if ( ! class_exists('Give_Settings_Donors')) :
17 17
 
18 18
 	/**
19 19
 	 * Give_Settings_Donors.
@@ -43,16 +43,16 @@  discard block
 block discarded – undo
43 43
 		 */
44 44
 		public function __construct() {
45 45
 			$this->id    = 'donors';
46
-			$this->label = esc_html__( 'Donors', 'give' );
46
+			$this->label = esc_html__('Donors', 'give');
47 47
 
48
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
49
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
50
-			add_action( 'give_admin_field_report_donors', array( $this, 'render_report_donors_field' ), 10, 2 );
48
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
49
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
50
+			add_action('give_admin_field_report_donors', array($this, 'render_report_donors_field'), 10, 2);
51 51
 
52 52
 			// Do not use main donor for this tab.
53
-			if( give_get_current_setting_tab() === $this->id ) {
54
-				add_action( 'give-reports_open_form', '__return_empty_string' );
55
-				add_action( 'give-reports_close_form', '__return_empty_string' );
53
+			if (give_get_current_setting_tab() === $this->id) {
54
+				add_action('give-reports_open_form', '__return_empty_string');
55
+				add_action('give-reports_close_form', '__return_empty_string');
56 56
 			}
57 57
 		}
58 58
 
@@ -63,8 +63,8 @@  discard block
 block discarded – undo
63 63
 		 * @param  array $pages Lst of pages.
64 64
 		 * @return array
65 65
 		 */
66
-		public function add_settings_page( $pages ) {
67
-			$pages[ $this->id ] = $this->label;
66
+		public function add_settings_page($pages) {
67
+			$pages[$this->id] = $this->label;
68 68
 
69 69
 			return $pages;
70 70
 		}
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 			 * @param  array $settings
87 87
 			 */
88 88
 			$settings = apply_filters(
89
-				'give_get_settings_' . $this->id,
89
+				'give_get_settings_'.$this->id,
90 90
 				array(
91 91
 					array(
92 92
 						'id'   => 'give_reports_donors',
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 					),
96 96
 					array(
97 97
 						'id'   => 'donors',
98
-						'name' => esc_html__( 'Donors', 'give' ),
98
+						'name' => esc_html__('Donors', 'give'),
99 99
 						'type' => 'report_donors',
100 100
 					),
101 101
 					array(
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 		public function output() {
120 120
 			$settings = $this->get_settings();
121 121
 
122
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
122
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
123 123
 		}
124 124
 
125 125
 		/**
@@ -131,8 +131,8 @@  discard block
 block discarded – undo
131 131
 		 * @param $field
132 132
 		 * @param $option_value
133 133
 		 */
134
-		public function render_report_donors_field( $field, $option_value ) {
135
-			do_action( 'give_reports_view_donors');
134
+		public function render_report_donors_field($field, $option_value) {
135
+			do_action('give_reports_view_donors');
136 136
 		}
137 137
 	}
138 138
 
Please login to merge, or discard this patch.
includes/admin/reporting/class-settings-forms.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@  discard block
 block discarded – undo
9 9
  * @since       1.8
10 10
  */
11 11
 
12
-if ( ! defined( 'ABSPATH' ) ) {
12
+if ( ! defined('ABSPATH')) {
13 13
 	exit; // Exit if accessed directly
14 14
 }
15 15
 
16
-if ( ! class_exists( 'Give_Settings_Forms' ) ) :
16
+if ( ! class_exists('Give_Settings_Forms')) :
17 17
 
18 18
 	/**
19 19
 	 * Give_Settings_Forms.
@@ -43,16 +43,16 @@  discard block
 block discarded – undo
43 43
 		 */
44 44
 		public function __construct() {
45 45
 			$this->id    = 'forms';
46
-			$this->label = esc_html__( 'Forms', 'give' );
46
+			$this->label = esc_html__('Forms', 'give');
47 47
 
48
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
49
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
50
-			add_action( 'give_admin_field_report_forms', array( $this, 'render_report_forms_field' ), 10, 2 );
48
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
49
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
50
+			add_action('give_admin_field_report_forms', array($this, 'render_report_forms_field'), 10, 2);
51 51
 
52 52
 			// Do not use main form for this tab.
53
-			if( give_get_current_setting_tab() === $this->id ) {
54
-				add_action( 'give-reports_open_form', '__return_empty_string' );
55
-				add_action( 'give-reports_close_form', '__return_empty_string' );
53
+			if (give_get_current_setting_tab() === $this->id) {
54
+				add_action('give-reports_open_form', '__return_empty_string');
55
+				add_action('give-reports_close_form', '__return_empty_string');
56 56
 			}
57 57
 		}
58 58
 
@@ -63,8 +63,8 @@  discard block
 block discarded – undo
63 63
 		 * @param  array $pages Lst of pages.
64 64
 		 * @return array
65 65
 		 */
66
-		public function add_settings_page( $pages ) {
67
-			$pages[ $this->id ] = $this->label;
66
+		public function add_settings_page($pages) {
67
+			$pages[$this->id] = $this->label;
68 68
 
69 69
 			return $pages;
70 70
 		}
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 			 * @param  array $settings
87 87
 			 */
88 88
 			$settings = apply_filters(
89
-				'give_get_settings_' . $this->id,
89
+				'give_get_settings_'.$this->id,
90 90
 				array(
91 91
 					array(
92 92
 						'id'   => 'give_reports_forms',
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 					),
96 96
 					array(
97 97
 						'id'   => 'forms',
98
-						'name' => esc_html__( 'Forms', 'give' ),
98
+						'name' => esc_html__('Forms', 'give'),
99 99
 						'type' => 'report_forms',
100 100
 					),
101 101
 					array(
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 		public function output() {
120 120
 			$settings = $this->get_settings();
121 121
 
122
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
122
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
123 123
 		}
124 124
 
125 125
 		/**
@@ -131,8 +131,8 @@  discard block
 block discarded – undo
131 131
 		 * @param $field
132 132
 		 * @param $option_value
133 133
 		 */
134
-		public function render_report_forms_field( $field, $option_value ) {
135
-			do_action( 'give_reports_view_forms');
134
+		public function render_report_forms_field($field, $option_value) {
135
+			do_action('give_reports_view_forms');
136 136
 		}
137 137
 	}
138 138
 
Please login to merge, or discard this patch.
includes/admin/reporting/class-donor-reports-table.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -185,8 +185,8 @@
 block discarded – undo
185 185
 				
186 186
 			case 'num_donations' :
187 187
 				$value = '<a href="' .
188
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
189
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
188
+						 admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
189
+						 ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
190 190
 				break;
191 191
 
192 192
 			case 'amount_spent' :
Please login to merge, or discard this patch.
Spacing   +64 added lines, -64 removed lines patch added patch discarded remove patch
@@ -10,13 +10,13 @@  discard block
 block discarded – undo
10 10
  */
11 11
 
12 12
 // Exit if accessed directly.
13
-if ( ! defined( 'ABSPATH' ) ) {
13
+if ( ! defined('ABSPATH')) {
14 14
 	exit;
15 15
 }
16 16
 
17 17
 // Load WP_List_Table if not loaded
18
-if ( ! class_exists( 'WP_List_Table' ) ) {
19
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
18
+if ( ! class_exists('WP_List_Table')) {
19
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
20 20
 }
21 21
 
22 22
 /**
@@ -62,11 +62,11 @@  discard block
 block discarded – undo
62 62
 		global $status, $page;
63 63
 
64 64
 		// Set parent defaults
65
-		parent::__construct( array(
66
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
67
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
65
+		parent::__construct(array(
66
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
67
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
68 68
 			'ajax'     => false                        // Does this table support ajax?
69
-		) );
69
+		));
70 70
 
71 71
 	}
72 72
 
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 	 *
84 84
 	 * @return false
85 85
 	 */
86
-	public function search_box( $text, $input_id ) {
86
+	public function search_box($text, $input_id) {
87 87
 		return false;
88 88
 	}
89 89
 
@@ -98,20 +98,20 @@  discard block
 block discarded – undo
98 98
 	 *
99 99
 	 * @return void
100 100
 	 */
101
-	public function give_search_box( $text, $input_id ) {
102
-		$input_id = $input_id . '-search-input';
101
+	public function give_search_box($text, $input_id) {
102
+		$input_id = $input_id.'-search-input';
103 103
 
104
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
105
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
104
+		if ( ! empty($_REQUEST['orderby'])) {
105
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
106 106
 		}
107
-		if ( ! empty( $_REQUEST['order'] ) ) {
108
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
107
+		if ( ! empty($_REQUEST['order'])) {
108
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
109 109
 		}
110 110
 		?>
111 111
 		<p class="search-box donor-search" role="search">
112 112
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
113 113
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
114
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
114
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
115 115
 		</p>
116 116
 	<?php
117 117
 	}
@@ -124,33 +124,33 @@  discard block
 block discarded – undo
124 124
 	 *
125 125
 	 * @param string $which
126 126
 	 */
127
-	protected function display_tablenav( $which ) {
127
+	protected function display_tablenav($which) {
128 128
 
129
-		if ( 'top' === $which ) {
130
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
129
+		if ('top' === $which) {
130
+			wp_nonce_field('bulk-'.$this->_args['plural']);
131 131
 		}
132 132
 		?>
133
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
133
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
134 134
 
135
-			<?php if ( 'top' === $which ) { ?>
135
+			<?php if ('top' === $which) { ?>
136 136
 				<h3 class="alignleft reports-earnings-title">
137
-					<span><?php esc_html_e( 'Donors Report', 'give' ); ?></span>
137
+					<span><?php esc_html_e('Donors Report', 'give'); ?></span>
138 138
 				</h3>
139 139
 			<?php } ?>
140 140
 
141 141
 			<div class="alignright tablenav-right">
142 142
 				<div class="actions bulkactions">
143 143
 					<?php
144
-					if ( 'top' === $which ) {
145
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
144
+					if ('top' === $which) {
145
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
146 146
 					}
147 147
 
148
-					$this->bulk_actions( $which ); ?>
148
+					$this->bulk_actions($which); ?>
149 149
 
150 150
 				</div>
151 151
 				<?php
152
-				$this->extra_tablenav( $which );
153
-				$this->pagination( $which );
152
+				$this->extra_tablenav($which);
153
+				$this->pagination($which);
154 154
 				?>
155 155
 			</div>
156 156
 
@@ -172,33 +172,33 @@  discard block
 block discarded – undo
172 172
 	 *
173 173
 	 * @return string Column Name
174 174
 	 */
175
-	public function column_default( $item, $column_name ) {
175
+	public function column_default($item, $column_name) {
176 176
 	    
177
-		switch ( $column_name ) {
177
+		switch ($column_name) {
178 178
 
179 179
 			case 'name' :
180
-				$name = '#' . $item['id'] . ' ';
181
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
182
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
183
-				$value = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
180
+				$name = '#'.$item['id'].' ';
181
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
182
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
183
+				$value = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
184 184
 				break;
185 185
 				
186 186
 			case 'num_donations' :
187
-				$value = '<a href="' .
188
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
189
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
187
+				$value = '<a href="'.
188
+				         admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
189
+				         ).'">'.esc_html($item['num_donations']).'</a>';
190 190
 				break;
191 191
 
192 192
 			case 'amount_spent' :
193
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
193
+				$value = give_currency_filter(give_format_amount($item[$column_name]));
194 194
 				break;
195 195
 
196 196
 			default:
197
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
197
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
198 198
 				break;
199 199
 		}
200 200
 
201
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
201
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
202 202
 	}
203 203
 
204 204
 	/**
@@ -210,13 +210,13 @@  discard block
 block discarded – undo
210 210
 	 */
211 211
 	public function get_columns() {
212 212
 		$columns = array(
213
-			'name'          => esc_html__( 'Name', 'give' ),
214
-			'email'         => esc_html__( 'Email', 'give' ),
215
-			'num_donations' => esc_html__( 'Donations', 'give' ),
216
-			'amount_spent'  => esc_html__( 'Total Donated', 'give' )
213
+			'name'          => esc_html__('Name', 'give'),
214
+			'email'         => esc_html__('Email', 'give'),
215
+			'num_donations' => esc_html__('Donations', 'give'),
216
+			'amount_spent'  => esc_html__('Total Donated', 'give')
217 217
 		);
218 218
 
219
-		return apply_filters( 'give_report_donor_columns', $columns );
219
+		return apply_filters('give_report_donor_columns', $columns);
220 220
 
221 221
 	}
222 222
 
@@ -229,10 +229,10 @@  discard block
 block discarded – undo
229 229
 	 */
230 230
 	public function get_sortable_columns() {
231 231
 		return array(
232
-			'id'            => array( 'id', true ),
233
-			'name'          => array( 'name', true ),
234
-			'num_donations' => array( 'purchase_count', false ),
235
-			'amount_spent'  => array( 'purchase_value', false ),
232
+			'id'            => array('id', true),
233
+			'name'          => array('name', true),
234
+			'num_donations' => array('purchase_count', false),
235
+			'amount_spent'  => array('purchase_value', false),
236 236
 		);
237 237
 	}
238 238
 
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
 	 * @since  1.0
244 244
 	 * @return void
245 245
 	 */
246
-	public function bulk_actions( $which = '' ) {
246
+	public function bulk_actions($which = '') {
247 247
 
248 248
 	}
249 249
 
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
 	 * @return int Current page number
256 256
 	 */
257 257
 	public function get_paged() {
258
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
258
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
259 259
 	}
260 260
 
261 261
 	/**
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
 	 * @return mixed string If search is present, false otherwise
267 267
 	 */
268 268
 	public function get_search() {
269
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
269
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
270 270
 	}
271 271
 
272 272
 	/**
@@ -283,10 +283,10 @@  discard block
 block discarded – undo
283 283
 
284 284
 		$data    = array();
285 285
 		$paged   = $this->get_paged();
286
-		$offset  = $this->per_page * ( $paged - 1 );
286
+		$offset  = $this->per_page * ($paged - 1);
287 287
 		$search  = $this->get_search();
288
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
289
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
288
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
289
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
290 290
 
291 291
 		$args = array(
292 292
 			'number'  => $this->per_page,
@@ -295,21 +295,21 @@  discard block
 block discarded – undo
295 295
 			'orderby' => $orderby
296 296
 		);
297 297
 
298
-		if ( is_email( $search ) ) {
298
+		if (is_email($search)) {
299 299
 			$args['email'] = $search;
300
-		} elseif ( is_numeric( $search ) ) {
300
+		} elseif (is_numeric($search)) {
301 301
 			$args['id'] = $search;
302 302
 		}
303 303
 
304
-		$donors = Give()->customers->get_customers( $args );
304
+		$donors = Give()->customers->get_customers($args);
305 305
 
306
-		if ( $donors ) {
306
+		if ($donors) {
307 307
 
308
-			$this->count = count( $donors );
308
+			$this->count = count($donors);
309 309
 
310
-			foreach ( $donors as $donor ) {
310
+			foreach ($donors as $donor) {
311 311
 
312
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
312
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
313 313
 
314 314
 				$data[] = array(
315 315
 					'id'            => $donor->id,
@@ -342,16 +342,16 @@  discard block
 block discarded – undo
342 342
 		$hidden   = array(); // No hidden columns
343 343
 		$sortable = $this->get_sortable_columns();
344 344
 
345
-		$this->_column_headers = array( $columns, $hidden, $sortable );
345
+		$this->_column_headers = array($columns, $hidden, $sortable);
346 346
 
347 347
 		$this->items = $this->reports_data();
348 348
 
349 349
 		$this->total = give_count_total_customers();
350 350
 
351
-		$this->set_pagination_args( array(
351
+		$this->set_pagination_args(array(
352 352
 			'total_items' => $this->total,
353 353
 			'per_page'    => $this->per_page,
354
-			'total_pages' => ceil( $this->total / $this->per_page )
355
-		) );
354
+			'total_pages' => ceil($this->total / $this->per_page)
355
+		));
356 356
 	}
357 357
 }
358 358
\ No newline at end of file
Please login to merge, or discard this patch.