Test Failed
Pull Request — master (#2204)
by Ravinder
04:20
created
includes/class-notices.php 1 patch
Spacing   +95 added lines, -95 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
 
@@ -53,21 +53,21 @@  discard block
 block discarded – undo
53 53
 	 * @since 1.0
54 54
 	 */
55 55
 	public function __construct() {
56
-		add_action( 'admin_notices', array( $this, 'render_admin_notices' ), 999 );
57
-		add_action( 'give_dismiss_notices', array( $this, 'dismiss_notices' ) );
56
+		add_action('admin_notices', array($this, 'render_admin_notices'), 999);
57
+		add_action('give_dismiss_notices', array($this, 'dismiss_notices'));
58 58
 
59
-		add_action( 'give_frontend_notices', array( $this, 'render_frontend_notices' ), 999 );
60
-		add_action( 'give_donation_form_before_personal_info', array( $this, 'render_frontend_notices' ) );
61
-		add_action( 'give_ajax_donation_errors', array( $this, 'render_frontend_notices' ) );
59
+		add_action('give_frontend_notices', array($this, 'render_frontend_notices'), 999);
60
+		add_action('give_donation_form_before_personal_info', array($this, 'render_frontend_notices'));
61
+		add_action('give_ajax_donation_errors', array($this, 'render_frontend_notices'));
62 62
 
63 63
 		/**
64 64
 		 * Backward compatibility for deprecated params.
65 65
 		 *
66 66
 		 * @since 1.8.14
67 67
 		 */
68
-		add_filter( 'give_register_notice_args', array( $this, 'bc_deprecated_params' ) );
69
-		add_filter( 'give_frontend_errors_args', array( $this, 'bc_deprecated_params' ) );
70
-		add_filter( 'give_frontend_notice_args', array( $this, 'bc_deprecated_params' ) );
68
+		add_filter('give_register_notice_args', array($this, 'bc_deprecated_params'));
69
+		add_filter('give_frontend_errors_args', array($this, 'bc_deprecated_params'));
70
+		add_filter('give_frontend_notice_args', array($this, 'bc_deprecated_params'));
71 71
 	}
72 72
 
73 73
 	/**
@@ -80,19 +80,19 @@  discard block
 block discarded – undo
80 80
 	 *
81 81
 	 * @return array
82 82
 	 */
83
-	public function bc_deprecated_params( $args ) {
83
+	public function bc_deprecated_params($args) {
84 84
 		/**
85 85
 		 *  Param: auto_dismissible
86 86
 		 *  deprecated in 1.8.14
87 87
 		 *
88 88
 		 *  Check if auto_dismissible is set and it true then unset and change dismissible parameter value to auto
89 89
 		 */
90
-		if ( isset( $args['auto_dismissible'] ) ) {
91
-			if ( ! empty( $args['auto_dismissible'] ) ) {
90
+		if (isset($args['auto_dismissible'])) {
91
+			if ( ! empty($args['auto_dismissible'])) {
92 92
 				$args['dismissible'] = 'auto';
93 93
 			}
94 94
 			// unset auto_dismissible as it has been deprecated.
95
-			unset( $args['auto_dismissible'] );
95
+			unset($args['auto_dismissible']);
96 96
 		}
97 97
 
98 98
 		return $args;
@@ -108,9 +108,9 @@  discard block
 block discarded – undo
108 108
 	 *
109 109
 	 * @return bool
110 110
 	 */
111
-	public function register_notice( $notice_args ) {
111
+	public function register_notice($notice_args) {
112 112
 		// Bailout.
113
-		if ( empty( $notice_args['id'] ) || array_key_exists( $notice_args['id'], self::$notices ) ) {
113
+		if (empty($notice_args['id']) || array_key_exists($notice_args['id'], self::$notices)) {
114 114
 			return false;
115 115
 		}
116 116
 
@@ -148,39 +148,39 @@  discard block
 block discarded – undo
148 148
 		 *
149 149
 		 * @since 1.8.14
150 150
 		 */
151
-		$notice_args = apply_filters( 'give_register_notice_args', $notice_args );
151
+		$notice_args = apply_filters('give_register_notice_args', $notice_args);
152 152
 
153 153
 		// Set extra dismiss links if any.
154
-		if ( false !== strpos( $notice_args['description'], 'data-dismiss-interval' ) ) {
154
+		if (false !== strpos($notice_args['description'], 'data-dismiss-interval')) {
155 155
 
156
-			preg_match_all( "/data-([^\"]*)=\"([^\"]*)\"/", $notice_args['description'], $extra_notice_dismiss_link );
156
+			preg_match_all("/data-([^\"]*)=\"([^\"]*)\"/", $notice_args['description'], $extra_notice_dismiss_link);
157 157
 
158
-			if ( ! empty( $extra_notice_dismiss_link ) ) {
159
-				$extra_notice_dismiss_links = array_chunk( current( $extra_notice_dismiss_link ), 3 );
160
-				foreach ( $extra_notice_dismiss_links as $extra_notice_dismiss_link ) {
158
+			if ( ! empty($extra_notice_dismiss_link)) {
159
+				$extra_notice_dismiss_links = array_chunk(current($extra_notice_dismiss_link), 3);
160
+				foreach ($extra_notice_dismiss_links as $extra_notice_dismiss_link) {
161 161
 					// Create array og key ==> value by parsing query string created after renaming data attributes.
162
-					$data_attribute_query_str = str_replace( array( 'data-', '-', '"' ), array(
162
+					$data_attribute_query_str = str_replace(array('data-', '-', '"'), array(
163 163
 						'',
164 164
 						'_',
165 165
 						'',
166
-					), implode( '&', $extra_notice_dismiss_link ) );
166
+					), implode('&', $extra_notice_dismiss_link));
167 167
 
168
-					$notice_args['extra_links'][] = wp_parse_args( $data_attribute_query_str );
168
+					$notice_args['extra_links'][] = wp_parse_args($data_attribute_query_str);
169 169
 				}
170 170
 			}
171 171
 		}
172 172
 
173 173
 
174
-		self::$notices[ $notice_args['id'] ] = $notice_args;
174
+		self::$notices[$notice_args['id']] = $notice_args;
175 175
 
176 176
 		// Auto set show param if not already set.
177
-		if ( ! isset( self::$notices[ $notice_args['id'] ]['show'] ) ) {
178
-			self::$notices[ $notice_args['id'] ]['show'] = $this->is_notice_dismissed( $notice_args ) ? false : true;
177
+		if ( ! isset(self::$notices[$notice_args['id']]['show'])) {
178
+			self::$notices[$notice_args['id']]['show'] = $this->is_notice_dismissed($notice_args) ? false : true;
179 179
 		}
180 180
 
181 181
 		// Auto set time interval for shortly.
182
-		if ( 'shortly' === self::$notices[ $notice_args['id'] ]['dismiss_interval'] ) {
183
-			self::$notices[ $notice_args['id'] ]['dismiss_interval_time'] = DAY_IN_SECONDS;
182
+		if ('shortly' === self::$notices[$notice_args['id']]['dismiss_interval']) {
183
+			self::$notices[$notice_args['id']]['dismiss_interval_time'] = DAY_IN_SECONDS;
184 184
 		}
185 185
 
186 186
 		return true;
@@ -194,44 +194,44 @@  discard block
 block discarded – undo
194 194
 	 */
195 195
 	public function render_admin_notices() {
196 196
 		// Bailout.
197
-		if ( empty( self::$notices ) ) {
197
+		if (empty(self::$notices)) {
198 198
 			return;
199 199
 		}
200 200
 
201 201
 		$output = '';
202 202
 
203
-		foreach ( self::$notices as $notice_id => $notice ) {
203
+		foreach (self::$notices as $notice_id => $notice) {
204 204
 			// Check flag set to true to show notice.
205
-			if ( ! $notice['show'] ) {
205
+			if ( ! $notice['show']) {
206 206
 				continue;
207 207
 			}
208 208
 
209 209
 			// Check if notice dismissible or not.
210
-			if ( ! self::$has_auto_dismissible_notice ) {
211
-				self::$has_auto_dismissible_notice = ( 'auto' === $notice['dismissible'] );
210
+			if ( ! self::$has_auto_dismissible_notice) {
211
+				self::$has_auto_dismissible_notice = ('auto' === $notice['dismissible']);
212 212
 			}
213 213
 
214 214
 			// Check if notice dismissible or not.
215
-			if ( ! self::$has_dismiss_interval_notice ) {
215
+			if ( ! self::$has_dismiss_interval_notice) {
216 216
 				self::$has_dismiss_interval_notice = $notice['dismiss_interval'];
217 217
 			}
218 218
 
219
-			$css_id = ( false === strpos( $notice['id'], 'give' ) ? "give-{$notice['id']}" : $notice['id'] );
219
+			$css_id = (false === strpos($notice['id'], 'give') ? "give-{$notice['id']}" : $notice['id']);
220 220
 
221
-			$css_class = 'give-notice notice ' . ( empty( $notice['dismissible'] ) ? 'non' : 'is' ) . "-dismissible {$notice['type']} notice-{$notice['type']}";
222
-			$output    .= sprintf(
223
-				'<div id="%1$s" class="%2$s" data-dismissible="%3$s" data-dismissible-type="%4$s" data-dismiss-interval="%5$s" data-notice-id="%6$s" data-security="%7$s" data-dismiss-interval-time="%8$s" style="display: none">' . " \n",
221
+			$css_class = 'give-notice notice '.(empty($notice['dismissible']) ? 'non' : 'is')."-dismissible {$notice['type']} notice-{$notice['type']}";
222
+			$output .= sprintf(
223
+				'<div id="%1$s" class="%2$s" data-dismissible="%3$s" data-dismissible-type="%4$s" data-dismiss-interval="%5$s" data-notice-id="%6$s" data-security="%7$s" data-dismiss-interval-time="%8$s" style="display: none">'." \n",
224 224
 				$css_id,
225 225
 				$css_class,
226
-				give_clean( $notice['dismissible'] ),
226
+				give_clean($notice['dismissible']),
227 227
 				$notice['dismissible_type'],
228 228
 				$notice['dismiss_interval'],
229 229
 				$notice['id'],
230
-				empty( $notice['dismissible_type'] ) ? '' : wp_create_nonce( "give_edit_{$notice_id}_notice" ),
230
+				empty($notice['dismissible_type']) ? '' : wp_create_nonce("give_edit_{$notice_id}_notice"),
231 231
 				$notice['dismiss_interval_time']
232 232
 			);
233 233
 
234
-			$output .= ( 0 === strpos( $notice['description'], '<div' ) || 0 === strpos( $notice['description'], '<p' ) ? $notice['description'] : "<p>{$notice['description']}</p>" );
234
+			$output .= (0 === strpos($notice['description'], '<div') || 0 === strpos($notice['description'], '<p') ? $notice['description'] : "<p>{$notice['description']}</p>");
235 235
 			$output .= "</div> \n";
236 236
 		}
237 237
 
@@ -249,18 +249,18 @@  discard block
 block discarded – undo
249 249
 	 *
250 250
 	 * @param int $form_id
251 251
 	 */
252
-	public function render_frontend_notices( $form_id = 0 ) {
252
+	public function render_frontend_notices($form_id = 0) {
253 253
 		$errors = give_get_errors();
254 254
 
255
-		$request_form_id = isset( $_REQUEST['form-id'] ) ? absint( $_REQUEST['form-id'] ) : 0;
255
+		$request_form_id = isset($_REQUEST['form-id']) ? absint($_REQUEST['form-id']) : 0;
256 256
 
257 257
 		// Sanity checks first: Ensure that gateway returned errors display on the appropriate form.
258
-		if ( ! isset( $_POST['give_ajax'] ) && $request_form_id !== $form_id ) {
258
+		if ( ! isset($_POST['give_ajax']) && $request_form_id !== $form_id) {
259 259
 			return;
260 260
 		}
261 261
 
262
-		if ( $errors ) {
263
-			self::print_frontend_errors( $errors );
262
+		if ($errors) {
263
+			self::print_frontend_errors($errors);
264 264
 
265 265
 			give_clear_errors();
266 266
 		}
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
 	 * @access private
274 274
 	 */
275 275
 	private function print_js() {
276
-		if ( self::$has_auto_dismissible_notice ) :
276
+		if (self::$has_auto_dismissible_notice) :
277 277
 			?>
278 278
 			<script>
279 279
 				jQuery(document).ready(function () {
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
 			<?php
290 290
 		endif;
291 291
 
292
-		if ( self::$has_dismiss_interval_notice ) :
292
+		if (self::$has_dismiss_interval_notice) :
293 293
 			?>
294 294
 			<script>
295 295
 				jQuery(document).ready(function () {
@@ -368,29 +368,29 @@  discard block
 block discarded – undo
368 368
 	 * @access public
369 369
 	 */
370 370
 	public function dismiss_notices() {
371
-		$_post     = give_clean( $_POST );
372
-		$notice_id = esc_attr( $_post['notice_id'] );
371
+		$_post     = give_clean($_POST);
372
+		$notice_id = esc_attr($_post['notice_id']);
373 373
 
374 374
 		// Bailout.
375 375
 		if (
376
-			empty( $notice_id ) ||
377
-			empty( $_post['dismissible_type'] ) ||
378
-			empty( $_post['dismiss_interval'] ) ||
379
-			! check_ajax_referer( "give_edit_{$notice_id}_notice", '_wpnonce' )
376
+			empty($notice_id) ||
377
+			empty($_post['dismissible_type']) ||
378
+			empty($_post['dismiss_interval']) ||
379
+			! check_ajax_referer("give_edit_{$notice_id}_notice", '_wpnonce')
380 380
 		) {
381 381
 			wp_send_json_error();
382 382
 		}
383 383
 
384
-		$notice_key = Give()->notices->get_notice_key( $notice_id, $_post['dismiss_interval'] );
385
-		if ( 'user' === $_post['dismissible_type'] ) {
384
+		$notice_key = Give()->notices->get_notice_key($notice_id, $_post['dismiss_interval']);
385
+		if ('user' === $_post['dismissible_type']) {
386 386
 			$current_user = wp_get_current_user();
387
-			$notice_key   = Give()->notices->get_notice_key( $notice_id, $_post['dismiss_interval'], $current_user->ID );
387
+			$notice_key   = Give()->notices->get_notice_key($notice_id, $_post['dismiss_interval'], $current_user->ID);
388 388
 		}
389 389
 
390
-		$notice_dismiss_time = ! empty( $_post['dismiss_interval_time'] ) ? $_post['dismiss_interval_time'] : null;
390
+		$notice_dismiss_time = ! empty($_post['dismiss_interval_time']) ? $_post['dismiss_interval_time'] : null;
391 391
 
392 392
 		// Save option to hide notice.
393
-		Give_Cache::set( $notice_key, true, $notice_dismiss_time, true );
393
+		Give_Cache::set($notice_key, true, $notice_dismiss_time, true);
394 394
 
395 395
 		wp_send_json_success();
396 396
 	}
@@ -408,18 +408,18 @@  discard block
 block discarded – undo
408 408
 	 *
409 409
 	 * @return string
410 410
 	 */
411
-	public function get_notice_key( $notice_id, $dismiss_interval = null, $user_id = 0 ) {
411
+	public function get_notice_key($notice_id, $dismiss_interval = null, $user_id = 0) {
412 412
 		$notice_key = "_give_notice_{$notice_id}";
413 413
 
414
-		if ( ! empty( $dismiss_interval ) ) {
414
+		if ( ! empty($dismiss_interval)) {
415 415
 			$notice_key .= "_{$dismiss_interval}";
416 416
 		}
417 417
 
418
-		if ( $user_id ) {
418
+		if ($user_id) {
419 419
 			$notice_key .= "_{$user_id}";
420 420
 		}
421 421
 
422
-		$notice_key = sanitize_key( $notice_key );
422
+		$notice_key = sanitize_key($notice_key);
423 423
 
424 424
 		return $notice_key;
425 425
 	}
@@ -432,11 +432,11 @@  discard block
 block discarded – undo
432 432
 	 *
433 433
 	 * @return string
434 434
 	 */
435
-	public function get_dismiss_link( $notice_args ) {
435
+	public function get_dismiss_link($notice_args) {
436 436
 		$notice_args = wp_parse_args(
437 437
 			$notice_args,
438 438
 			array(
439
-				'title'                 => __( 'Click here', 'give' ),
439
+				'title'                 => __('Click here', 'give'),
440 440
 				'dismissible_type'      => '',
441 441
 				'dismiss_interval'      => '',
442 442
 				'dismiss_interval_time' => null,
@@ -463,31 +463,31 @@  discard block
 block discarded – undo
463 463
 	 *
464 464
 	 * @return bool|null
465 465
 	 */
466
-	public function is_notice_dismissed( $notice ) {
467
-		$notice_key          = $this->get_notice_key( $notice['id'], $notice['dismiss_interval'] );
466
+	public function is_notice_dismissed($notice) {
467
+		$notice_key          = $this->get_notice_key($notice['id'], $notice['dismiss_interval']);
468 468
 		$is_notice_dismissed = false;
469 469
 
470
-		if ( 'user' === $notice['dismissible_type'] ) {
470
+		if ('user' === $notice['dismissible_type']) {
471 471
 			$current_user = wp_get_current_user();
472
-			$notice_key   = Give()->notices->get_notice_key( $notice['id'], $notice['dismiss_interval'], $current_user->ID );
472
+			$notice_key   = Give()->notices->get_notice_key($notice['id'], $notice['dismiss_interval'], $current_user->ID);
473 473
 		}
474 474
 
475
-		$notice_data = Give_Cache::get( $notice_key, true );
475
+		$notice_data = Give_Cache::get($notice_key, true);
476 476
 
477 477
 		// Find notice dismiss link status if notice has extra dismissible links.
478
-		if ( ( empty( $notice_data ) || is_wp_error( $notice_data ) ) && ! empty( $notice['extra_links'] ) ) {
478
+		if ((empty($notice_data) || is_wp_error($notice_data)) && ! empty($notice['extra_links'])) {
479 479
 
480
-			foreach ( $notice['extra_links'] as $extra_link ) {
481
-				$new_notice_data = wp_parse_args( $extra_link, $notice );
482
-				unset( $new_notice_data['extra_links'] );
480
+			foreach ($notice['extra_links'] as $extra_link) {
481
+				$new_notice_data = wp_parse_args($extra_link, $notice);
482
+				unset($new_notice_data['extra_links']);
483 483
 
484
-				if ( $is_notice_dismissed = $this->is_notice_dismissed( $new_notice_data ) ) {
484
+				if ($is_notice_dismissed = $this->is_notice_dismissed($new_notice_data)) {
485 485
 					return $is_notice_dismissed;
486 486
 				}
487 487
 			}
488 488
 		}
489 489
 
490
-		$is_notice_dismissed = ! empty( $notice_data ) && ! is_wp_error( $notice_data );
490
+		$is_notice_dismissed = ! empty($notice_data) && ! is_wp_error($notice_data);
491 491
 
492 492
 		return $is_notice_dismissed;
493 493
 	}
@@ -501,9 +501,9 @@  discard block
 block discarded – undo
501 501
 	 *
502 502
 	 * @param array $errors
503 503
 	 */
504
-	public static function print_frontend_errors( $errors ) {
504
+	public static function print_frontend_errors($errors) {
505 505
 		// Bailout.
506
-		if ( ! $errors ) {
506
+		if ( ! $errors) {
507 507
 			return;
508 508
 		}
509 509
 
@@ -518,37 +518,37 @@  discard block
 block discarded – undo
518 518
 		);
519 519
 
520 520
 		// Note: we will remove give_errors class in future.
521
-		$classes = apply_filters( 'give_error_class', array( 'give_notices', 'give_errors' ) );
521
+		$classes = apply_filters('give_error_class', array('give_notices', 'give_errors'));
522 522
 
523
-		echo sprintf( '<div class="%s">', implode( ' ', $classes ) );
523
+		echo sprintf('<div class="%s">', implode(' ', $classes));
524 524
 
525 525
 		// Loop error codes and display errors.
526
-		foreach ( $errors as $error_id => $error ) {
526
+		foreach ($errors as $error_id => $error) {
527 527
 			// Backward compatibility v<1.8.11
528
-			if ( is_string( $error ) ) {
528
+			if (is_string($error)) {
529 529
 				$error = array(
530 530
 					'message'     => $error,
531 531
 					'notice_args' => array(),
532 532
 				);
533 533
 			}
534 534
 
535
-			$notice_args = wp_parse_args( $error['notice_args'], $default_notice_args );
535
+			$notice_args = wp_parse_args($error['notice_args'], $default_notice_args);
536 536
 
537 537
 			/**
538 538
 			 * Filter to modify Frontend Errors args before errors is display.
539 539
 			 *
540 540
 			 * @since 1.8.14
541 541
 			 */
542
-			$notice_args = apply_filters( 'give_frontend_errors_args', $notice_args );
542
+			$notice_args = apply_filters('give_frontend_errors_args', $notice_args);
543 543
 
544 544
 			echo sprintf(
545 545
 				'<div class="give_error give_notice" id="give_error_%1$s" data-dismissible="%2$s" data-dismiss-interval="%3$d">
546 546
 						<p><strong>%4$s</strong>: %5$s</p>
547 547
 					</div>',
548 548
 				$error_id,
549
-				give_clean( $notice_args['dismissible'] ),
550
-				absint( $notice_args['dismiss_interval'] ),
551
-				esc_html__( 'Error', 'give' ),
549
+				give_clean($notice_args['dismissible']),
550
+				absint($notice_args['dismiss_interval']),
551
+				esc_html__('Error', 'give'),
552 552
 				$error['message']
553 553
 			);
554 554
 		}
@@ -570,8 +570,8 @@  discard block
 block discarded – undo
570 570
 	 *
571 571
 	 * @return  string
572 572
 	 */
573
-	public static function print_frontend_notice( $message, $echo = true, $notice_type = 'warning', $notice_args = array() ) {
574
-		if ( empty( $message ) ) {
573
+	public static function print_frontend_notice($message, $echo = true, $notice_type = 'warning', $notice_args = array()) {
574
+		if (empty($message)) {
575 575
 			return '';
576 576
 		}
577 577
 
@@ -585,14 +585,14 @@  discard block
 block discarded – undo
585 585
 			'dismiss_interval' => 5000,
586 586
 		);
587 587
 
588
-		$notice_args = wp_parse_args( $notice_args, $default_notice_args );
588
+		$notice_args = wp_parse_args($notice_args, $default_notice_args);
589 589
 
590 590
 		/**
591 591
 		 * Filter to modify Frontend notice args before notices is display.
592 592
 		 *
593 593
 		 * @since 1.8.14
594 594
 		 */
595
-		$notice_args = apply_filters( 'give_frontend_notice_args', $notice_args );
595
+		$notice_args = apply_filters('give_frontend_notice_args', $notice_args);
596 596
 
597 597
 		// Note: we will remove give_errors class in future.
598 598
 		$error = sprintf(
@@ -602,12 +602,12 @@  discard block
 block discarded – undo
602 602
 				</p>
603 603
 			</div>',
604 604
 			$notice_type,
605
-			give_clean( $notice_args['dismissible'] ),
606
-			absint( $notice_args['dismiss_interval'] ),
605
+			give_clean($notice_args['dismissible']),
606
+			absint($notice_args['dismiss_interval']),
607 607
 			$message
608 608
 		);
609 609
 
610
-		if ( ! $echo ) {
610
+		if ( ! $echo) {
611 611
 			return $error;
612 612
 		}
613 613
 
Please login to merge, or discard this patch.