Completed
Push — master ( dce43d...f66823 )
by J.D.
04:19
created
src/admin/screens/modules-load.php 2 patches
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -9,22 +9,22 @@  discard block
 block discarded – undo
9 9
 
10 10
 global $status, $wp_version;
11 11
 
12
-if ( isset( $_POST['clear-recent-list'] ) ) {
12
+if ( isset( $_POST[ 'clear-recent-list' ] ) ) {
13 13
 	$action = 'clear-recent-list';
14
-} elseif ( isset( $_REQUEST['action'] ) && -1 !== (int) $_REQUEST['action'] ) { // WPCS: CSRF OK.
15
-	$action = sanitize_key( $_REQUEST['action'] );
16
-} elseif ( isset( $_REQUEST['action2'] ) && -1 !== (int) $_REQUEST['action2'] ) { // WPCS: CSRF OK.
17
-	$action = sanitize_key( $_REQUEST['action2'] );
14
+} elseif ( isset( $_REQUEST[ 'action' ] ) && -1 !== (int) $_REQUEST[ 'action' ] ) { // WPCS: CSRF OK.
15
+	$action = sanitize_key( $_REQUEST[ 'action' ] );
16
+} elseif ( isset( $_REQUEST[ 'action2' ] ) && -1 !== (int) $_REQUEST[ 'action2' ] ) { // WPCS: CSRF OK.
17
+	$action = sanitize_key( $_REQUEST[ 'action2' ] );
18 18
 } else {
19 19
 	$action = '';
20 20
 }
21 21
 
22
-$page   = ( isset( $_REQUEST['paged'] ) ) ? max( 1, absint( $_REQUEST['paged'] ) ) : 1; // WPCS: CSRF OK.
23
-$module = ( isset( $_REQUEST['module'] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST['module'] ) ) : ''; // WPCS: CSRF OK.
24
-$s      = ( isset( $_REQUEST['s'] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ) : ''; // WPCS: CSRF OK.
22
+$page   = ( isset( $_REQUEST[ 'paged' ] ) ) ? max( 1, absint( $_REQUEST[ 'paged' ] ) ) : 1; // WPCS: CSRF OK.
23
+$module = ( isset( $_REQUEST[ 'module' ] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 'module' ] ) ) : ''; // WPCS: CSRF OK.
24
+$s      = ( isset( $_REQUEST[ 's' ] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 's' ] ) ) : ''; // WPCS: CSRF OK.
25 25
 
26 26
 // Clean up request URI from temporary args for screen options/paging URI's to work as expected.
27
-$_SERVER['REQUEST_URI'] = remove_query_arg( array( 'error', 'deleted', 'activate', 'activate-multi', 'deactivate', 'deactivate-multi', '_error_nonce' ) );
27
+$_SERVER[ 'REQUEST_URI' ] = remove_query_arg( array( 'error', 'deleted', 'activate', 'activate-multi', 'deactivate', 'deactivate-multi', '_error_nonce' ) );
28 28
 
29 29
 $redirect_url = self_admin_url( "admin.php?page=wordpoints_modules&module_status={$status}&paged={$page}&s={$s}" );
30 30
 
@@ -90,8 +90,8 @@  discard block
 block discarded – undo
90 90
 
91 91
 		check_admin_referer( 'bulk-modules' );
92 92
 
93
-		$modules = isset( $_POST['checked'] )
94
-			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST['checked'] ) )
93
+		$modules = isset( $_POST[ 'checked' ] )
94
+			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST[ 'checked' ] ) )
95 95
 			: array();
96 96
 
97 97
 		// Only activate modules which are not already active.
@@ -218,8 +218,8 @@  discard block
 block discarded – undo
218 218
 
219 219
 		check_admin_referer( 'bulk-modules' );
220 220
 
221
-		$modules = isset( $_POST['checked'] )
222
-			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST['checked'] ) )
221
+		$modules = isset( $_POST[ 'checked' ] )
222
+			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST[ 'checked' ] ) )
223 223
 			: array();
224 224
 
225 225
 		$network_modules = array_filter( $modules, 'is_wordpoints_module_active_for_network' );
@@ -261,8 +261,8 @@  discard block
 block discarded – undo
261 261
 		check_admin_referer( 'bulk-modules' );
262 262
 
263 263
 		// $_POST = from the module form; $_GET = from the FTP details screen.
264
-		$modules = isset( $_REQUEST['checked'] )
265
-			? array_map( 'sanitize_text_field', (array) wp_unslash( $_REQUEST['checked'] ) )
264
+		$modules = isset( $_REQUEST[ 'checked' ] )
265
+			? array_map( 'sanitize_text_field', (array) wp_unslash( $_REQUEST[ 'checked' ] ) )
266 266
 			: array();
267 267
 
268 268
 		if ( empty( $modules ) ) {
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
 
288 288
 		$parent_file = 'admin.php';
289 289
 
290
-		if ( ! isset( $_REQUEST['verify-delete'] ) ) {
290
+		if ( ! isset( $_REQUEST[ 'verify-delete' ] ) ) {
291 291
 
292 292
 			wp_enqueue_script( 'jquery' );
293 293
 			require_once ABSPATH . 'wp-admin/admin-header.php';
@@ -307,15 +307,15 @@  discard block
 block discarded – undo
307 307
 
308 308
 					if ( '.' === dirname( $module ) ) {
309 309
 
310
-						$files_to_delete[] = $module_dir . '/' . $module;
310
+						$files_to_delete[ ] = $module_dir . '/' . $module;
311 311
 						$data = wordpoints_get_module_data( $module_dir . '/' . $module );
312 312
 
313 313
 						if ( ! empty( $data ) ) {
314 314
 
315 315
 							$module_info[ $module ] = $data;
316
-							$module_info[ $module ]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
316
+							$module_info[ $module ][ 'is_uninstallable' ] = is_uninstallable_wordpoints_module( $module );
317 317
 
318
-							if ( ! $module_info[ $module ]['network'] ) {
318
+							if ( ! $module_info[ $module ][ 'network' ] ) {
319 319
 								$have_non_network_modules = true;
320 320
 							}
321 321
 						}
@@ -335,9 +335,9 @@  discard block
 block discarded – undo
335 335
 						foreach ( $folder_modules as $module_file => $data ) {
336 336
 
337 337
 							$module_info[ $module_file ] = $data;
338
-							$module_info[ $module_file ]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
338
+							$module_info[ $module_file ][ 'is_uninstallable' ] = is_uninstallable_wordpoints_module( $module );
339 339
 
340
-							if ( ! $module_info[ $module_file ]['network'] ) {
340
+							if ( ! $module_info[ $module_file ][ 'network' ] ) {
341 341
 								$have_non_network_modules = true;
342 342
 							}
343 343
 						}
@@ -376,16 +376,16 @@  discard block
 block discarded – undo
376 376
 
377 377
 						foreach ( $module_info as $module ) {
378 378
 
379
-							if ( $module['is_uninstallable'] ) {
379
+							if ( $module[ 'is_uninstallable' ] ) {
380 380
 
381 381
 								// translators: 1. Module name; 2. Module author.
382
-								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em> (will also <strong>delete its data</strong>)', 'wordpoints' ), esc_html( $module['name'] ), esc_html( $module['author_name'] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
382
+								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em> (will also <strong>delete its data</strong>)', 'wordpoints' ), esc_html( $module[ 'name' ] ), esc_html( $module[ 'author_name' ] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
383 383
 								$data_to_delete = true;
384 384
 
385 385
 							} else {
386 386
 
387 387
 								// translators: 1. Module name; 2. Module author.
388
-								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em>', 'wordpoints' ), esc_html( $module['name'] ), esc_html( $module['author_name'] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
388
+								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em>', 'wordpoints' ), esc_html( $module[ 'name' ] ), esc_html( $module[ 'author_name' ] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
389 389
 							}
390 390
 						}
391 391
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -11,9 +11,11 @@
 block discarded – undo
11 11
 
12 12
 if ( isset( $_POST['clear-recent-list'] ) ) {
13 13
 	$action = 'clear-recent-list';
14
-} elseif ( isset( $_REQUEST['action'] ) && -1 !== (int) $_REQUEST['action'] ) { // WPCS: CSRF OK.
14
+} elseif ( isset( $_REQUEST['action'] ) && -1 !== (int) $_REQUEST['action'] ) {
15
+// WPCS: CSRF OK.
15 16
 	$action = sanitize_key( $_REQUEST['action'] );
16
-} elseif ( isset( $_REQUEST['action2'] ) && -1 !== (int) $_REQUEST['action2'] ) { // WPCS: CSRF OK.
17
+} elseif ( isset( $_REQUEST['action2'] ) && -1 !== (int) $_REQUEST['action2'] ) {
18
+// WPCS: CSRF OK.
17 19
 	$action = sanitize_key( $_REQUEST['action2'] );
18 20
 } else {
19 21
 	$action = '';
Please login to merge, or discard this patch.
src/admin/screens/configure-components.php 2 patches
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -17,13 +17,13 @@  discard block
 block discarded – undo
17 17
 // Show messages and errors.
18 18
 //
19 19
 
20
-if ( isset( $_GET['wordpoints_component'], $_GET['_wpnonce'] ) && $wordpoints_components->is_registered( sanitize_key( $_GET['wordpoints_component'] ) ) ) { // WPCS: CSRF OK.
20
+if ( isset( $_GET[ 'wordpoints_component' ], $_GET[ '_wpnonce' ] ) && $wordpoints_components->is_registered( sanitize_key( $_GET[ 'wordpoints_component' ] ) ) ) { // WPCS: CSRF OK.
21 21
 
22
-	$component = sanitize_key( $_GET['wordpoints_component'] );
22
+	$component = sanitize_key( $_GET[ 'wordpoints_component' ] );
23 23
 
24
-	if ( isset( $_GET['message'] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_message-{$component}" ) ) {
24
+	if ( isset( $_GET[ 'message' ] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_message-{$component}" ) ) {
25 25
 
26
-		switch ( (int) $_GET['message'] ) {
26
+		switch ( (int) $_GET[ 'message' ] ) {
27 27
 
28 28
 			case 1:
29 29
 				if ( $wordpoints_components->is_active( $component ) ) {
@@ -43,15 +43,15 @@  discard block
 block discarded – undo
43 43
 		if ( isset( $message ) ) {
44 44
 
45 45
 			wordpoints_show_admin_message(
46
-				esc_html( sprintf( $message, $components[ $component ]['name'] ) )
46
+				esc_html( sprintf( $message, $components[ $component ][ 'name' ] ) )
47 47
 				, 'success'
48 48
 				, array( 'dismissible' => true )
49 49
 			);
50 50
 		}
51 51
 
52
-	} elseif ( isset( $_GET['error'] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_error-{$component}" ) ) {
52
+	} elseif ( isset( $_GET[ 'error' ] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_error-{$component}" ) ) {
53 53
 
54
-		switch ( (int) $_GET['error'] ) {
54
+		switch ( (int) $_GET[ 'error' ] ) {
55 55
 
56 56
 			case 1:
57 57
 				if ( ! $wordpoints_components->is_active( $component ) ) {
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 		if ( isset( $error ) ) {
72 72
 
73 73
 			wordpoints_show_admin_error(
74
-				esc_html( sprintf( $error, $components[ $component ]['name'] ) )
74
+				esc_html( sprintf( $error, $components[ $component ][ 'name' ] ) )
75 75
 				, array( 'dismissible' => true )
76 76
 			);
77 77
 		}
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 
110 110
 		<?php
111 111
 
112
-		if ( $wordpoints_components->is_active( $component['slug'] ) ) {
112
+		if ( $wordpoints_components->is_active( $component[ 'slug' ] ) ) {
113 113
 
114 114
 			$action = 'deactivate';
115 115
 			$button = __( 'Deactivate', 'wordpoints' );
@@ -124,38 +124,38 @@  discard block
 block discarded – undo
124 124
 
125 125
 		<tr>
126 126
 			<td>
127
-				<?php if ( '' !== $component['component_uri'] ) : ?>
128
-					<a href="<?php echo esc_url( $component['component_uri'] ); ?>">
127
+				<?php if ( '' !== $component[ 'component_uri' ] ) : ?>
128
+					<a href="<?php echo esc_url( $component[ 'component_uri' ] ); ?>">
129 129
 				<?php endif; ?>
130
-					<?php echo esc_html( $component['name'] ); ?>
131
-				<?php if ( '' !== $component['component_uri'] ) : ?>
130
+					<?php echo esc_html( $component[ 'name' ] ); ?>
131
+				<?php if ( '' !== $component[ 'component_uri' ] ) : ?>
132 132
 					</a>
133 133
 				<?php endif; ?>
134 134
 			</td>
135 135
 			<td>
136
-				<?php echo wp_kses( $component['description'], 'wordpoints_component_description' ); ?>
137
-				<?php if ( '' !== $component['author'] ) : ?>
136
+				<?php echo wp_kses( $component[ 'description' ], 'wordpoints_component_description' ); ?>
137
+				<?php if ( '' !== $component[ 'author' ] ) : ?>
138 138
 					&nbsp;|&nbsp;
139 139
 					<?php
140 140
 					// translators: Author name.
141 141
 					echo esc_html( sprintf( __( 'By %s', 'wordpoints' ), '' /* This space intentionally left blank */ ) );
142 142
 					?>
143
-					<?php if ( '' !== $component['author_uri'] ) : ?>
144
-						<a href="<?php echo esc_url( $component['author_uri'] ); ?>">
143
+					<?php if ( '' !== $component[ 'author_uri' ] ) : ?>
144
+						<a href="<?php echo esc_url( $component[ 'author_uri' ] ); ?>">
145 145
 					<?php endif; ?>
146
-						<?php echo esc_html( $component['author'] ); ?>
147
-					<?php if ( '' !== $component['author_uri'] ) : ?>
146
+						<?php echo esc_html( $component[ 'author' ] ); ?>
147
+					<?php if ( '' !== $component[ 'author_uri' ] ) : ?>
148 148
 						</a>
149 149
 					<?php endif; ?>
150 150
 				<?php endif; ?>
151 151
 			</td>
152
-			<td><?php echo esc_html( $component['version'] ); ?></td>
152
+			<td><?php echo esc_html( $component[ 'version' ] ); ?></td>
153 153
 			<td>
154
-				<form method="post" name="wordpoints_components_form_<?php echo esc_attr( $component['slug'] ); ?>">
154
+				<form method="post" name="wordpoints_components_form_<?php echo esc_attr( $component[ 'slug' ] ); ?>">
155 155
 					<input type="hidden" name="wordpoints_component_action" value="<?php echo esc_attr( $action ); ?>" />
156
-					<input type="hidden" name="wordpoints_component" value="<?php echo esc_attr( $component['slug'] ); ?>" />
157
-					<?php wp_nonce_field( "wordpoints_{$action}_component-{$component['slug']}" ); ?>
158
-					<?php submit_button( $button, "secondary wordpoints-component-{$action}", "wordpoints-component-{$action}_{$component['slug']}", false ); ?>
156
+					<input type="hidden" name="wordpoints_component" value="<?php echo esc_attr( $component[ 'slug' ] ); ?>" />
157
+					<?php wp_nonce_field( "wordpoints_{$action}_component-{$component[ 'slug' ]}" ); ?>
158
+					<?php submit_button( $button, "secondary wordpoints-component-{$action}", "wordpoints-component-{$action}_{$component[ 'slug' ]}", false ); ?>
159 159
 				</form>
160 160
 			</td>
161 161
 		</tr>
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,8 @@
 block discarded – undo
17 17
 // Show messages and errors.
18 18
 //
19 19
 
20
-if ( isset( $_GET['wordpoints_component'], $_GET['_wpnonce'] ) && $wordpoints_components->is_registered( sanitize_key( $_GET['wordpoints_component'] ) ) ) { // WPCS: CSRF OK.
20
+if ( isset( $_GET['wordpoints_component'], $_GET['_wpnonce'] ) && $wordpoints_components->is_registered( sanitize_key( $_GET['wordpoints_component'] ) ) ) {
21
+// WPCS: CSRF OK.
21 22
 
22 23
 	$component = sanitize_key( $_GET['wordpoints_component'] );
23 24
 
Please login to merge, or discard this patch.
src/components/points/includes/class-wordpoints-points-hooks.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 	 */
111 111
 	public static function register( $class_name ) {
112 112
 
113
-		self::$classes[] = $class_name;
113
+		self::$classes[ ] = $class_name;
114 114
 	}
115 115
 
116 116
 	/**
@@ -286,10 +286,10 @@  discard block
 block discarded – undo
286 286
 
287 287
 			$args = $hook_type->get_options();
288 288
 
289
-			$args['_add']       = 'multi';
290
-			$args['_display']   = 'template';
291
-			$args['_multi_num'] = $hook_type->next_hook_id_number();
292
-			$args['_id_slug']   = $i;
289
+			$args[ '_add' ]       = 'multi';
290
+			$args[ '_display' ]   = 'template';
291
+			$args[ '_multi_num' ] = $hook_type->next_hook_id_number();
292
+			$args[ '_id_slug' ]   = $i;
293 293
 
294 294
 			$hook_type->set_options( $args );
295 295
 
@@ -338,11 +338,11 @@  discard block
 block discarded – undo
338 338
 
339 339
 			$options = $hook_type->get_options();
340 340
 
341
-			$options['_display'] = 'instance';
341
+			$options[ '_display' ] = 'instance';
342 342
 
343
-			unset( $options['_add'] );
343
+			unset( $options[ '_add' ] );
344 344
 
345
-			$options['_id_slug'] = $slug;
345
+			$options[ '_id_slug' ] = $slug;
346 346
 
347 347
 			$hook_type->set_options( $options );
348 348
 
@@ -596,15 +596,15 @@  discard block
 block discarded – undo
596 596
 
597 597
 						<p>
598 598
 							<label for="points-name-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html_x( 'Name:', 'points type', 'wordpoints' ); ?></label>
599
-							<input class="widefat" type="text" id="points-name-<?php echo esc_attr( $slug ); ?>" name="points-name" value="<?php echo esc_attr( $points_type['name'] ); ?>" />
599
+							<input class="widefat" type="text" id="points-name-<?php echo esc_attr( $slug ); ?>" name="points-name" value="<?php echo esc_attr( $points_type[ 'name' ] ); ?>" />
600 600
 						</p>
601 601
 						<p>
602 602
 							<label for="points-prefix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $prefix ); ?></label>
603
-							<input class="widefat" type="text" id="points-prefix-<?php echo esc_attr( $slug ); ?>" name="points-prefix" value="<?php echo esc_attr( $points_type['prefix'] ); ?>" />
603
+							<input class="widefat" type="text" id="points-prefix-<?php echo esc_attr( $slug ); ?>" name="points-prefix" value="<?php echo esc_attr( $points_type[ 'prefix' ] ); ?>" />
604 604
 						</p>
605 605
 						<p>
606 606
 							<label for="points-suffix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $suffix ); ?></label>
607
-							<input class="widefat" type="text" id="points-suffix-<?php echo esc_attr( $slug ); ?>" name="points-suffix" value="<?php echo esc_attr( $points_type['suffix'] ); ?>" />
607
+							<input class="widefat" type="text" id="points-suffix-<?php echo esc_attr( $slug ); ?>" name="points-suffix" value="<?php echo esc_attr( $points_type[ 'suffix' ] ); ?>" />
608 608
 						</p>
609 609
 
610 610
 						<?php
@@ -677,28 +677,28 @@  discard block
 block discarded – undo
677 677
 
678 678
 		$id_format = $hook_id;
679 679
 
680
-		$multi_number = ( isset( $options['_multi_num'] ) ) ? $options['_multi_num'] : '';
681
-		$add_new      = ( isset( $options['_add'] ) )       ? $options['_add']       : '';
680
+		$multi_number = ( isset( $options[ '_multi_num' ] ) ) ? $options[ '_multi_num' ] : '';
681
+		$add_new      = ( isset( $options[ '_add' ] ) ) ? $options[ '_add' ] : '';
682 682
 
683 683
 		// Prepare the URL query string.
684 684
 		$query_arg = array( 'edithook' => $id_format );
685 685
 
686 686
 		if ( $add_new ) {
687 687
 
688
-			$query_arg['addnew'] = 1;
688
+			$query_arg[ 'addnew' ] = 1;
689 689
 
690 690
 			if ( $multi_number ) {
691 691
 
692
-				$query_arg['num']  = $multi_number;
693
-				$query_arg['base'] = $id_base;
692
+				$query_arg[ 'num' ]  = $multi_number;
693
+				$query_arg[ 'base' ] = $id_base;
694 694
 			}
695 695
 
696 696
 		} else {
697 697
 
698
-			$query_arg['points_type'] = $points_type;
698
+			$query_arg[ 'points_type' ] = $points_type;
699 699
 		}
700 700
 
701
-		if ( isset( $options['_display'] ) && 'template' === $options['_display'] ) {
701
+		if ( isset( $options[ '_display' ] ) && 'template' === $options[ '_display' ] ) {
702 702
 
703 703
 			/*
704 704
 			 * We aren't outputting the form for a hook, but a template form for this
@@ -714,7 +714,7 @@  discard block
 block discarded – undo
714 714
 
715 715
 		?>
716 716
 
717
-		<div id="hook-<?php echo esc_html( $options['_id_slug'] ); ?>_<?php echo esc_attr( $id_format ); ?>" class="hook <?php echo esc_attr( $options['_classname'] ); ?>">
717
+		<div id="hook-<?php echo esc_html( $options[ '_id_slug' ] ); ?>_<?php echo esc_attr( $id_format ); ?>" class="hook <?php echo esc_attr( $options[ '_classname' ] ); ?>">
718 718
 		<div class="hook-top">
719 719
 			<div class="hook-title-action">
720 720
 				<a class="hook-action hide-if-no-js" href="#available-hooks"></a>
@@ -739,8 +739,8 @@  discard block
 block discarded – undo
739 739
 
740 740
 				<input type="hidden" name="hook-id" class="hook-id" value="<?php echo esc_attr( $id_format ); ?>" />
741 741
 				<input type="hidden" name="id_base" class="id_base" value="<?php echo esc_attr( $id_base ); ?>" />
742
-				<input type="hidden" name="hook-width" class="hook-width" value="<?php echo isset( $options['width'] ) ? esc_attr( $options['width'] ) : ''; ?>" />
743
-				<input type="hidden" name="hook-height" class="hook-height" value="<?php echo isset( $options['height'] ) ? esc_attr( $options['height'] ) : ''; ?>" />
742
+				<input type="hidden" name="hook-width" class="hook-width" value="<?php echo isset( $options[ 'width' ] ) ? esc_attr( $options[ 'width' ] ) : ''; ?>" />
743
+				<input type="hidden" name="hook-height" class="hook-height" value="<?php echo isset( $options[ 'height' ] ) ? esc_attr( $options[ 'height' ] ) : ''; ?>" />
744 744
 				<input type="hidden" name="hook_number" class="hook_number" value="<?php echo esc_attr( $number ); ?>" />
745 745
 				<input type="hidden" name="multi_number" class="multi_number" value="<?php echo esc_attr( $multi_number ); ?>" />
746 746
 				<input type="hidden" name="add_new" class="add_new" value="<?php echo esc_attr( $add_new ); ?>" />
@@ -760,8 +760,8 @@  discard block
 block discarded – undo
760 760
 		</div>
761 761
 
762 762
 		<div class="hook-description">
763
-			<?php if ( ! empty( $options['description'] ) ) : ?>
764
-				<?php echo esc_html( $options['description'] ); ?>
763
+			<?php if ( ! empty( $options[ 'description' ] ) ) : ?>
764
+				<?php echo esc_html( $options[ 'description' ] ); ?>
765 765
 			<?php endif; ?>
766 766
 		</div>
767 767
 		</div>
Please login to merge, or discard this patch.
src/components/points/includes/logs.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 
90 90
 	// The current user needs to be set dynamically, since it can change at times.
91 91
 	if ( 'current_user' === $query_slug ) {
92
-		$args['user_id'] = get_current_user_id();
92
+		$args[ 'user_id' ] = get_current_user_id();
93 93
 	}
94 94
 
95 95
 	/**
@@ -137,12 +137,12 @@  discard block
 block discarded – undo
137 137
 
138 138
 	$query_data = WordPoints_Points_Log_Queries::get_query_data( $query_slug );
139 139
 
140
-	if ( $query_data['cache_queries'] ) {
140
+	if ( $query_data[ 'cache_queries' ] ) {
141 141
 
142 142
 		$query->prime_cache(
143
-			$query_data['cache_key']
143
+			$query_data[ 'cache_key' ]
144 144
 			, null
145
-			, $query_data['network_wide']
145
+			, $query_data[ 'network_wide' ]
146 146
 		);
147 147
 	}
148 148
 
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
 
197 197
 	$args = array_merge( $defaults, $args );
198 198
 
199
-	if ( ! $args['datatable'] ) {
199
+	if ( ! $args[ 'datatable' ] ) {
200 200
 
201 201
 		_deprecated_argument(
202 202
 			__FUNCTION__
@@ -204,13 +204,13 @@  discard block
 block discarded – undo
204 204
 			, '$args["datatable"] is deprecated and should no longer be used. Use $args["paginate"] instead.'
205 205
 		);
206 206
 
207
-		$args['paginate'] = false;
207
+		$args[ 'paginate' ] = false;
208 208
 	}
209 209
 
210 210
 	$view = wordpoints_component( 'points' )
211 211
 		->get_sub_app( 'logs' )
212 212
 		->get_sub_app( 'views' )
213
-		->get( $args['view'], array( $logs_query, $args ) );
213
+		->get( $args[ 'view' ], array( $logs_query, $args ) );
214 214
 
215 215
 	if ( ! $view instanceof WordPoints_Points_Logs_View ) {
216 216
 		return;
@@ -284,10 +284,10 @@  discard block
 block discarded – undo
284 284
  */
285 285
 function wordpoints_points_logs_profile_edit( $text, $points, $points_type, $user_id, $log_type, $meta ) {
286 286
 
287
-	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta['user_id'] )->display_name, $meta['user_id'], 'display' );
287
+	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta[ 'user_id' ] )->display_name, $meta[ 'user_id' ], 'display' );
288 288
 
289 289
 	// translators: 1. User name; 2. Reason given.
290
-	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta['reason'] ) );
290
+	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta[ 'reason' ] ) );
291 291
 }
292 292
 
293 293
 /**
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
  */
300 300
 function wordpoints_points_logs_comment_disapprove( $text, $points, $points_type, $user_id, $log_type, $meta ) {
301 301
 
302
-	switch ( $meta['status'] ) {
302
+	switch ( $meta[ 'status' ] ) {
303 303
 
304 304
 		case 'spam':
305 305
 			$text = _x( 'Comment marked as spam.', 'points log description', 'wordpoints' );
@@ -325,9 +325,9 @@  discard block
 block discarded – undo
325 325
  */
326 326
 function wordpoints_points_logs_post_delete( $text, $points, $points_type, $user_id, $log_type, $meta ) {
327 327
 
328
-	if ( isset( $meta['post_type'] ) ) {
328
+	if ( isset( $meta[ 'post_type' ] ) ) {
329 329
 
330
-		$post_type = get_post_type_object( $meta['post_type'] );
330
+		$post_type = get_post_type_object( $meta[ 'post_type' ] );
331 331
 
332 332
 		if ( ! is_null( $post_type ) ) {
333 333
 
@@ -389,18 +389,18 @@  discard block
 block discarded – undo
389 389
 		'%user_id%',
390 390
 	);
391 391
 
392
-	if ( empty( $args['points_type'] ) ) {
392
+	if ( empty( $args[ 'points_type' ] ) ) {
393 393
 		$points_types = array_keys( wordpoints_get_points_types() );
394 394
 	} else {
395
-		$points_types = (array) $args['points_type'];
395
+		$points_types = (array) $args[ 'points_type' ];
396 396
 	}
397 397
 
398 398
 	foreach ( $points_types as $points_type ) {
399 399
 		foreach ( WordPoints_Points_Log_Queries::get_queries() as $query ) {
400 400
 
401
-			if ( ! empty( $query['cache_key'] ) ) {
401
+			if ( ! empty( $query[ 'cache_key' ] ) ) {
402 402
 
403
-				if ( $query['network_wide'] ) {
403
+				if ( $query[ 'network_wide' ] ) {
404 404
 					$group = 'wordpoints_network_points_logs_query';
405 405
 				} else {
406 406
 					$group = 'wordpoints_points_logs_query';
@@ -408,11 +408,11 @@  discard block
 block discarded – undo
408 408
 
409 409
 				$replace = array(
410 410
 					$points_type,
411
-					$args['user_id'],
411
+					$args[ 'user_id' ],
412 412
 				);
413 413
 
414 414
 				wp_cache_delete(
415
-					str_replace( $find, $replace, $query['cache_key'] )
415
+					str_replace( $find, $replace, $query[ 'cache_key' ] )
416 416
 					, $group
417 417
 				);
418 418
 			}
Please login to merge, or discard this patch.
src/components/points/includes/points.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -117,11 +117,11 @@  discard block
 block discarded – undo
117 117
  */
118 118
 function wordpoints_add_points_type( $settings ) {
119 119
 
120
-	if ( ! is_array( $settings ) || ! isset( $settings['name'] ) ) {
120
+	if ( ! is_array( $settings ) || ! isset( $settings[ 'name' ] ) ) {
121 121
 		return false;
122 122
 	}
123 123
 
124
-	$slug = $settings['name'];
124
+	$slug = $settings[ 'name' ];
125 125
 	$slug = preg_replace( '/\s+/', '-', $slug );
126 126
 	$slug = preg_replace( '/-+/', '-', $slug );
127 127
 	$slug = trim( $slug, '-' );
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
 
166 166
 	$points_types = wordpoints_get_points_types();
167 167
 
168
-	if ( ! is_array( $settings ) || ! isset( $points_types[ $slug ], $settings['name'] ) ) {
168
+	if ( ! is_array( $settings ) || ! isset( $points_types[ $slug ], $settings[ 'name' ] ) ) {
169 169
 		return false;
170 170
 	}
171 171
 
@@ -1094,13 +1094,13 @@  discard block
 block discarded – undo
1094 1094
 				, array( '%d' )
1095 1095
 			);
1096 1096
 
1097
-			if ( ! isset( $flushed['points_types'][ $log->points_type ], $flushed['user_ids'][ $log->user_id ] ) ) {
1097
+			if ( ! isset( $flushed[ 'points_types' ][ $log->points_type ], $flushed[ 'user_ids' ][ $log->user_id ] ) ) {
1098 1098
 				wordpoints_flush_points_logs_caches(
1099 1099
 					array( 'user_id' => $log->user_id, 'points_type' => $log->points_type )
1100 1100
 				);
1101 1101
 
1102
-				$flushed['points_types'][ $log->points_type ] = true;
1103
-				$flushed['user_ids'][ $log->user_id ] = true;
1102
+				$flushed[ 'points_types' ][ $log->points_type ] = true;
1103
+				$flushed[ 'user_ids' ][ $log->user_id ] = true;
1104 1104
 			}
1105 1105
 		}
1106 1106
 
@@ -1134,9 +1134,9 @@  discard block
 block discarded – undo
1134 1134
 		$cache = array( 'is_max' => false, 'top_users' => array() );
1135 1135
 	}
1136 1136
 
1137
-	$cached_users = count( $cache['top_users'] );
1137
+	$cached_users = count( $cache[ 'top_users' ] );
1138 1138
 
1139
-	if ( $num_users > $cached_users && ! $cache['is_max'] ) {
1139
+	if ( $num_users > $cached_users && ! $cache[ 'is_max' ] ) {
1140 1140
 
1141 1141
 		global $wpdb;
1142 1142
 
@@ -1185,17 +1185,17 @@  discard block
 block discarded – undo
1185 1185
 			return array();
1186 1186
 		}
1187 1187
 
1188
-		$cache['top_users'] = array_merge( $cache['top_users'], $top_users );
1188
+		$cache[ 'top_users' ] = array_merge( $cache[ 'top_users' ], $top_users );
1189 1189
 
1190
-		if ( count( $cache['top_users'] ) < $num_users ) {
1191
-			$cache['is_max'] = true;
1190
+		if ( count( $cache[ 'top_users' ] ) < $num_users ) {
1191
+			$cache[ 'is_max' ] = true;
1192 1192
 		}
1193 1193
 
1194 1194
 		wp_cache_set( $points_type, $cache, 'wordpoints_points_top_users' );
1195 1195
 
1196 1196
 	} // End if ( not cached ).
1197 1197
 
1198
-	return array_slice( $cache['top_users'], 0, $num_users );
1198
+	return array_slice( $cache[ 'top_users' ], 0, $num_users );
1199 1199
 }
1200 1200
 
1201 1201
 /**
@@ -1226,7 +1226,7 @@  discard block
 block discarded – undo
1226 1226
 	$points_type_name = wordpoints_get_points_type_setting( $points_type, 'name' );
1227 1227
 
1228 1228
 	if ( ! empty( $points_type_name ) ) {
1229
-		$column_headers['points'] = $points_type_name;
1229
+		$column_headers[ 'points' ] = $points_type_name;
1230 1230
 	}
1231 1231
 
1232 1232
 	/**
@@ -1250,9 +1250,9 @@  discard block
 block discarded – undo
1250 1250
 	<table class="wordpoints-points-top-users <?php echo esc_attr( implode( ' ', $extra_classes ) ); ?>">
1251 1251
 		<thead>
1252 1252
 			<tr>
1253
-				<th scope="col"><?php echo esc_html( $column_headers['position'] ); ?></th>
1254
-				<th scope="col"><?php echo esc_html( $column_headers['user'] ); ?></th>
1255
-				<th scope="col"><?php echo esc_html( $column_headers['points'] ); ?></th>
1253
+				<th scope="col"><?php echo esc_html( $column_headers[ 'position' ] ); ?></th>
1254
+				<th scope="col"><?php echo esc_html( $column_headers[ 'user' ] ); ?></th>
1255
+				<th scope="col"><?php echo esc_html( $column_headers[ 'points' ] ); ?></th>
1256 1256
 			</tr>
1257 1257
 		</thead>
1258 1258
 		<tbody>
@@ -1310,9 +1310,9 @@  discard block
 block discarded – undo
1310 1310
 		</tbody>
1311 1311
 		<tfoot>
1312 1312
 		<tr>
1313
-			<th scope="col"><?php echo esc_html( $column_headers['position'] ); ?></th>
1314
-			<th scope="col"><?php echo esc_html( $column_headers['user'] ); ?></th>
1315
-			<th scope="col"><?php echo esc_html( $column_headers['points'] ); ?></th>
1313
+			<th scope="col"><?php echo esc_html( $column_headers[ 'position' ] ); ?></th>
1314
+			<th scope="col"><?php echo esc_html( $column_headers[ 'user' ] ); ?></th>
1315
+			<th scope="col"><?php echo esc_html( $column_headers[ 'points' ] ); ?></th>
1316 1316
 		</tr>
1317 1317
 		</tfoot>
1318 1318
 	</table>
@@ -1351,7 +1351,7 @@  discard block
 block discarded – undo
1351 1351
 		$cache = wp_cache_get( $slug, 'wordpoints_points_top_users' );
1352 1352
 
1353 1353
 		// If there aren't fewer users than the cache holds, we don't need to clear it.
1354
-		if ( ! is_array( $cache ) || ! $cache['is_max'] ) {
1354
+		if ( ! is_array( $cache ) || ! $cache[ 'is_max' ] ) {
1355 1355
 			continue;
1356 1356
 		}
1357 1357
 
@@ -1376,7 +1376,7 @@  discard block
 block discarded – undo
1376 1376
 		$cache = wp_cache_get( $slug, 'wordpoints_points_top_users' );
1377 1377
 
1378 1378
 		// If this user isn't in the cache, we don't need to clear it.
1379
-		if ( ! is_array( $cache ) || ! in_array( (int) $user_id, array_map( 'intval', $cache['top_users'] ), true ) ) {
1379
+		if ( ! is_array( $cache ) || ! in_array( (int) $user_id, array_map( 'intval', $cache[ 'top_users' ] ), true ) ) {
1380 1380
 			continue;
1381 1381
 		}
1382 1382
 
Please login to merge, or discard this patch.
src/components/points/admin/includes/functions.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
  */
144 144
 function wordpoints_points_admin_screen_hooks() {
145 145
 
146
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
146
+	if ( isset( $_GET[ 'edithook' ] ) || isset( $_POST[ 'savehook' ] ) || isset( $_POST[ 'removehook' ] ) ) { // WPCS: CSRF OK.
147 147
 
148 148
 		// - We're doing this without AJAX (JS).
149 149
 
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
  */
207 207
 function wordpoints_no_js_points_hooks_save() {
208 208
 
209
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
209
+	if ( ! isset( $_POST[ 'savehook' ] ) && ! isset( $_POST[ 'removehook' ] ) ) { // WPCS: CSRF OK.
210 210
 		return;
211 211
 	}
212 212
 
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
  */
285 285
 function wordpoints_points_hook_description_form( $has_form, $instance, $hook ) {
286 286
 
287
-	$description = ( isset( $instance['_description'] ) ) ? $instance['_description'] : '';
287
+	$description = ( isset( $instance[ '_description' ] ) ) ? $instance[ '_description' ] : '';
288 288
 
289 289
 	?>
290 290
 
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
 			?>
351 351
 
352 352
 			<tr>
353
-				<th scope="row"><?php echo esc_html( $type['name'] ); ?></th>
353
+				<th scope="row"><?php echo esc_html( $type[ 'name' ] ); ?></th>
354 354
 				<td>
355 355
 					<input type="hidden" name="<?php echo esc_attr( "wordpoints_points_old-{$slug}" ); ?>" value="<?php echo esc_attr( $points ); ?>" />
356 356
 					<input type="number" name="<?php echo esc_attr( "wordpoints_points-{$slug}" ); ?>" value="<?php echo esc_attr( $points ); ?>" autocomplete="off" />
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
 		<tbody>
390 390
 		<?php foreach ( wordpoints_get_points_types() as $slug => $type ) : ?>
391 391
 			<tr>
392
-				<th scope="row" style="text-align: left;"><?php echo esc_html( $type['name'] ); ?></th>
392
+				<th scope="row" style="text-align: left;"><?php echo esc_html( $type[ 'name' ] ); ?></th>
393 393
 				<td style="text-align: right;"><?php wordpoints_display_points( $user->ID, $slug, 'profile_page' ); ?></td>
394 394
 			</tr>
395 395
 		<?php endforeach; ?>
@@ -419,7 +419,7 @@  discard block
 block discarded – undo
419 419
 	}
420 420
 
421 421
 	if (
422
-		! isset( $_POST['wordpoints_points_set_nonce'], $_POST['wordpoints_set_reason'] )
422
+		! isset( $_POST[ 'wordpoints_points_set_nonce' ], $_POST[ 'wordpoints_set_reason' ] )
423 423
 		|| ! wordpoints_verify_nonce( 'wordpoints_points_set_nonce', 'wordpoints_points_set_profile', null, 'post' )
424 424
 	) {
425 425
 		return;
@@ -444,7 +444,7 @@  discard block
 block discarded – undo
444 444
 				, 'profile_edit'
445 445
 				, array(
446 446
 					'user_id' => get_current_user_id(),
447
-					'reason' => sanitize_text_field( wp_unslash( $_POST['wordpoints_set_reason'] ) ),
447
+					'reason' => sanitize_text_field( wp_unslash( $_POST[ 'wordpoints_set_reason' ] ) ),
448 448
 				)
449 449
 			);
450 450
 		}
@@ -500,11 +500,11 @@  discard block
 block discarded – undo
500 500
 function wordpoints_points_admin_settings_save() {
501 501
 
502 502
 	if (
503
-		isset( $_POST['default_points_type'] )
503
+		isset( $_POST[ 'default_points_type' ] )
504 504
 		&& wordpoints_verify_nonce( 'wordpoints_default_points_type_nonce', 'wordpoints_default_points_type', null, 'post' )
505 505
 	) {
506 506
 
507
-		$points_type = sanitize_key( $_POST['default_points_type'] );
507
+		$points_type = sanitize_key( $_POST[ 'default_points_type' ] );
508 508
 
509 509
 		if ( '-1' === $points_type ) {
510 510
 
@@ -527,7 +527,7 @@  discard block
 block discarded – undo
527 527
 function wordpoints_points_admin_notices() {
528 528
 
529 529
 	if (
530
-		( ! isset( $_GET['page'] ) || 'wordpoints_points_types' !== $_GET['page'] ) // WPCS: CSRF OK.
530
+		( ! isset( $_GET[ 'page' ] ) || 'wordpoints_points_types' !== $_GET[ 'page' ] ) // WPCS: CSRF OK.
531 531
 		&& current_user_can( 'manage_wordpoints_points_types' )
532 532
 		&& ! wordpoints_get_points_types()
533 533
 	) {
Please login to merge, or discard this patch.
Braces   +9 added lines, -4 removed lines patch added patch discarded remove patch
@@ -143,7 +143,8 @@  discard block
 block discarded – undo
143 143
  */
144 144
 function wordpoints_points_admin_screen_hooks() {
145 145
 
146
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
146
+	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) {
147
+// WPCS: CSRF OK.
147 148
 
148 149
 		// - We're doing this without AJAX (JS).
149 150
 
@@ -206,7 +207,8 @@  discard block
 block discarded – undo
206 207
  */
207 208
 function wordpoints_no_js_points_hooks_save() {
208 209
 
209
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
210
+	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) {
211
+// WPCS: CSRF OK.
210 212
 		return;
211 213
 	}
212 214
 
@@ -290,9 +292,12 @@  discard block
 block discarded – undo
290 292
 
291 293
 	<?php if ( $has_form ) : ?>
292 294
 		<hr />
293
-	<?php else : ?>
295
+	<?php else {
296
+	: ?>
294 297
 		<br />
295
-	<?php endif; ?>
298
+	<?php endif;
299
+}
300
+?>
296 301
 
297 302
 	<div class="hook-instance-description">
298 303
 		<label for="<?php $hook->the_field_id( '_description' ); ?>"><?php echo esc_html_x( 'Description (optional):', 'points hook', 'wordpoints' ); ?></label>
Please login to merge, or discard this patch.
src/components/points/admin/screens/hooks-no-js-load.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -11,11 +11,11 @@  discard block
 block discarded – undo
11 11
 	return;
12 12
 }
13 13
 
14
-if ( ! isset( $_POST['hook-id'] ) ) {
14
+if ( ! isset( $_POST[ 'hook-id' ] ) ) {
15 15
 	return;
16 16
 }
17 17
 
18
-$hook_id = sanitize_key( $_POST['hook-id'] );
18
+$hook_id = sanitize_key( $_POST[ 'hook-id' ] );
19 19
 
20 20
 check_admin_referer( "save-delete-hook-{$hook_id}" );
21 21
 
@@ -25,12 +25,12 @@  discard block
 block discarded – undo
25 25
 	WordPoints_Points_Hooks::set_network_mode( true );
26 26
 }
27 27
 
28
-if ( ! isset( $_POST['points_type'], $_POST['id_base'] ) ) {
28
+if ( ! isset( $_POST[ 'points_type' ], $_POST[ 'id_base' ] ) ) {
29 29
 	return;
30 30
 }
31 31
 
32
-$points_type_id = sanitize_key( $_POST['points_type'] );
33
-$id_base        = sanitize_key( $_POST['id_base'] );
32
+$points_type_id = sanitize_key( $_POST[ 'points_type' ] );
33
+$id_base        = sanitize_key( $_POST[ 'id_base' ] );
34 34
 
35 35
 // These are the hooks grouped by points type.
36 36
 $points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
 $hook = WordPoints_Points_Hooks::get_handler_by_id_base( $id_base );
49 49
 
50
-if ( ! empty( $_POST['removehook'] ) ) {
50
+if ( ! empty( $_POST[ 'removehook' ] ) ) {
51 51
 
52 52
 	// - We are deleting an instance of a hook.
53 53
 
@@ -63,11 +63,11 @@  discard block
 block discarded – undo
63 63
 
64 64
 	$hook->delete_callback( $hook_id );
65 65
 
66
-} elseif ( ! empty( $_POST['savehook'] ) ) {
66
+} elseif ( ! empty( $_POST[ 'savehook' ] ) ) {
67 67
 
68 68
 	// - We are saving an instance of a hook.
69 69
 
70
-	$number = isset( $_POST['multi_number'] ) ? (int) $_POST['multi_number'] : '';
70
+	$number = isset( $_POST[ 'multi_number' ] ) ? (int) $_POST[ 'multi_number' ] : '';
71 71
 
72 72
 	if ( $number ) {
73 73
 
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 	// Add hook it to this points type.
102 102
 	if ( ! in_array( $hook_id, $points_type_hooks, true ) ) {
103 103
 
104
-		$points_type_hooks[] = $hook_id;
104
+		$points_type_hooks[ ] = $hook_id;
105 105
 		$points_types_hooks[ $points_type_id ] = $points_type_hooks;
106 106
 	}
107 107
 
Please login to merge, or discard this patch.
src/components/points/admin/screens/hooks-no-js.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -7,15 +7,15 @@  discard block
 block discarded – undo
7 7
  * @since 1.0.0
8 8
  */
9 9
 
10
-if ( ! isset( $_GET['edithook'] ) ) { // WPCS: CSRF OK.
10
+if ( ! isset( $_GET[ 'edithook' ] ) ) { // WPCS: CSRF OK.
11 11
 	return;
12 12
 }
13 13
 
14
-$hook_id = sanitize_key( $_GET['edithook'] );
14
+$hook_id = sanitize_key( $_GET[ 'edithook' ] );
15 15
 
16 16
 $points_types = wordpoints_get_points_types();
17 17
 
18
-if ( isset( $_GET['addnew'] ) ) { // WPCS: CSRF OK.
18
+if ( isset( $_GET[ 'addnew' ] ) ) { // WPCS: CSRF OK.
19 19
 
20 20
 	// - We are adding a new points hook.
21 21
 
@@ -33,11 +33,11 @@  discard block
 block discarded – undo
33 33
 		return;
34 34
 	}
35 35
 
36
-	if ( isset( $_GET['base'], $_GET['num'] ) ) { // WPCS: CSRF OK.
36
+	if ( isset( $_GET[ 'base' ], $_GET[ 'num' ] ) ) { // WPCS: CSRF OK.
37 37
 
38 38
 		// Copy minimal info from an existing instance of this hook to a new instance.
39 39
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
40
-			sanitize_key( $_GET['base'] ) // WPCS: CSRF OK.
40
+			sanitize_key( $_GET[ 'base' ] ) // WPCS: CSRF OK.
41 41
 		);
42 42
 
43 43
 		if ( ! $hook ) {
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 			return;
51 51
 		}
52 52
 
53
-		$multi_number = (int) $_GET['num'];
53
+		$multi_number = (int) $_GET[ 'num' ];
54 54
 		$number       = 0;
55 55
 		$hook_id      = $hook->get_id( $multi_number );
56 56
 		$_hook        = $hook;
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 
71 71
 	// We are editing an existing points hook.
72 72
 
73
-	$points_type = isset( $_GET['points_type'] ) ? sanitize_key( $_GET['points_type'] ) : '_inactive_hooks'; // WPCS: CSRF OK.
73
+	$points_type = isset( $_GET[ 'points_type' ] ) ? sanitize_key( $_GET[ 'points_type' ] ) : '_inactive_hooks'; // WPCS: CSRF OK.
74 74
 
75 75
 	$hook = WordPoints_Points_Hooks::get_handler( $hook_id );
76 76
 
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 			</div>
125 125
 			<br />
126 126
 			<div class="hook-control-actions">
127
-				<?php if ( isset( $_GET['addnew'] ) ) : // WPCS: CSRF OK. ?>
127
+				<?php if ( isset( $_GET[ 'addnew' ] ) ) : // WPCS: CSRF OK. ?>
128 128
 					<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_hooks' ) ); ?>" class="button alignleft"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></a>
129 129
 				<?php else :
130 130
 						submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
Please login to merge, or discard this patch.
Braces   +9 added lines, -4 removed lines patch added patch discarded remove patch
@@ -7,7 +7,8 @@  discard block
 block discarded – undo
7 7
  * @since 1.0.0
8 8
  */
9 9
 
10
-if ( ! isset( $_GET['edithook'] ) ) { // WPCS: CSRF OK.
10
+if ( ! isset( $_GET['edithook'] ) ) {
11
+// WPCS: CSRF OK.
11 12
 	return;
12 13
 }
13 14
 
@@ -15,7 +16,8 @@  discard block
 block discarded – undo
15 16
 
16 17
 $points_types = wordpoints_get_points_types();
17 18
 
18
-if ( isset( $_GET['addnew'] ) ) { // WPCS: CSRF OK.
19
+if ( isset( $_GET['addnew'] ) ) {
20
+// WPCS: CSRF OK.
19 21
 
20 22
 	// - We are adding a new points hook.
21 23
 
@@ -33,7 +35,8 @@  discard block
 block discarded – undo
33 35
 		return;
34 36
 	}
35 37
 
36
-	if ( isset( $_GET['base'], $_GET['num'] ) ) { // WPCS: CSRF OK.
38
+	if ( isset( $_GET['base'], $_GET['num'] ) ) {
39
+// WPCS: CSRF OK.
37 40
 
38 41
 		// Copy minimal info from an existing instance of this hook to a new instance.
39 42
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
@@ -126,8 +129,10 @@  discard block
 block discarded – undo
126 129
 			<div class="hook-control-actions">
127 130
 				<?php if ( isset( $_GET['addnew'] ) ) : // WPCS: CSRF OK. ?>
128 131
 					<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_hooks' ) ); ?>" class="button alignleft"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></a>
129
-				<?php else :
132
+				<?php else {
133
+	:
130 134
 						submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
135
+}
131 136
 					endif;
132 137
 
133 138
 					submit_button( __( 'Save Hook', 'wordpoints' ), 'button-primary alignright', 'savehook', false );
Please login to merge, or discard this patch.
src/components/points/admin/screens/hooks.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -43,18 +43,18 @@  discard block
 block discarded – undo
43 43
 		return;
44 44
 	}
45 45
 
46
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
46
+	if ( isset( $_GET[ 'message' ] ) && isset( $messages[ (int) $_GET[ 'message' ] ] ) ) { // WPCS: CSRF OK.
47 47
 
48 48
 		wordpoints_show_admin_message(
49
-			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
49
+			esc_html( $messages[ (int) $_GET[ 'message' ] ] ) // WPCS: CSRF OK.
50 50
 			, 'success'
51 51
 			, array( 'dismissible' => true )
52 52
 		);
53 53
 
54
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
54
+	} elseif ( isset( $_GET[ 'error' ] ) && isset( $errors[ (int) $_GET[ 'error' ] ] ) ) { // WPCS: CSRF OK.
55 55
 
56 56
 		wordpoints_show_admin_error(
57
-			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.
57
+			esc_html( $errors[ (int) $_GET[ 'error' ] ] ) // WPCS: CSRF OK.
58 58
 			, array( 'dismissible' => true )
59 59
 		);
60 60
 	}
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
 			foreach ( $points_types as $slug => $points_type ) {
125 125
 
126 126
 				$wrap_class = 'hooks-holder-wrap';
127
-				if ( ! empty( $points_type['class'] ) ) {
128
-					$wrap_class .= ' points-type-' . $points_type['class'];
127
+				if ( ! empty( $points_type[ 'class' ] ) ) {
128
+					$wrap_class .= ' points-type-' . $points_type[ 'class' ];
129 129
 				}
130 130
 
131 131
 				if ( $i ) {
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 				<div class="<?php echo esc_attr( $wrap_class ); ?>">
138 138
 					<div class="points-type-name">
139 139
 						<div class="points-type-name-arrow"><br /></div>
140
-						<h2><?php echo esc_html( $points_type['name'] ); ?><span class="spinner"></span></h2>
140
+						<h2><?php echo esc_html( $points_type[ 'name' ] ); ?><span class="spinner"></span></h2>
141 141
 					</div>
142 142
 					<div id="<?php echo esc_attr( $slug ); ?>" class="hooks-sortables">
143 143
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,8 @@  discard block
 block discarded – undo
43 43
 		return;
44 44
 	}
45 45
 
46
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
46
+	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) {
47
+// WPCS: CSRF OK.
47 48
 
48 49
 		wordpoints_show_admin_message(
49 50
 			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
@@ -51,7 +52,8 @@  discard block
 block discarded – undo
51 52
 			, array( 'dismissible' => true )
52 53
 		);
53 54
 
54
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
55
+	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) {
56
+// WPCS: CSRF OK.
55 57
 
56 58
 		wordpoints_show_admin_error(
57 59
 			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.
Please login to merge, or discard this patch.