WordPoints / wordpoints

src/components/points/admin/screens/hooks-no-js.php

Spacing +8 added lines, -8 removed lines

@@ -7,15 +7,15 @@ 
  * @since 1.0.0
  */
 
-if ( ! isset( $_GET['edithook'] ) ) { // WPCS: CSRF OK.
+if ( ! isset( $_GET[ 'edithook' ] ) ) { // WPCS: CSRF OK.
 	return;
 }
 
-$hook_id = sanitize_key( $_GET['edithook'] );
+$hook_id = sanitize_key( $_GET[ 'edithook' ] );
 
 $points_types = wordpoints_get_points_types();
 
-if ( isset( $_GET['addnew'] ) ) { // WPCS: CSRF OK.
+if ( isset( $_GET[ 'addnew' ] ) ) { // WPCS: CSRF OK.
 
 	// - We are adding a new points hook.
 
@@ -33,11 +33,11 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['base'], $_GET['num'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET[ 'base' ], $_GET[ 'num' ] ) ) { // WPCS: CSRF OK.
 
 		// Copy minimal info from an existing instance of this hook to a new instance.
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
-			sanitize_key( $_GET['base'] ) // WPCS: CSRF OK.
+			sanitize_key( $_GET[ 'base' ] ) // WPCS: CSRF OK.
 		);
 
 		if ( ! $hook ) {
@@ -50,7 +50,7 @@ 
 			return;
 		}
 
-		$multi_number = (int) $_GET['num'];
+		$multi_number = (int) $_GET[ 'num' ];
 		$number       = 0;
 		$hook_id      = $hook->get_id( $multi_number );
 		$_hook        = $hook;
@@ -70,7 +70,7 @@ 
 
 	// We are editing an existing points hook.
 
-	$points_type = isset( $_GET['points_type'] ) ? sanitize_key( $_GET['points_type'] ) : '_inactive_hooks'; // WPCS: CSRF OK.
+	$points_type = isset( $_GET[ 'points_type' ] ) ? sanitize_key( $_GET[ 'points_type' ] ) : '_inactive_hooks'; // WPCS: CSRF OK.
 
 	$hook = WordPoints_Points_Hooks::get_handler( $hook_id );
 
@@ -124,7 +124,7 @@ 
 			</div>
 			<br />
 			<div class="hook-control-actions">
-				<?php if ( isset( $_GET['addnew'] ) ) : // WPCS: CSRF OK. ?>
+				<?php if ( isset( $_GET[ 'addnew' ] ) ) : // WPCS: CSRF OK. ?>
 					<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>
 				<?php else :
 						submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );

Braces +9 added lines, -4 removed lines

@@ -7,7 +7,8 @@ 
  * @since 1.0.0
  */
 
-if ( ! isset( $_GET['edithook'] ) ) { // WPCS: CSRF OK.
+if ( ! isset( $_GET['edithook'] ) ) {
+// WPCS: CSRF OK.
 	return;
 }
 
@@ -15,7 +16,8 @@ 
 
 $points_types = wordpoints_get_points_types();
 
-if ( isset( $_GET['addnew'] ) ) { // WPCS: CSRF OK.
+if ( isset( $_GET['addnew'] ) ) {
+// WPCS: CSRF OK.
 
 	// - We are adding a new points hook.
 
@@ -33,7 +35,8 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['base'], $_GET['num'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['base'], $_GET['num'] ) ) {
+// WPCS: CSRF OK.
 
 		// Copy minimal info from an existing instance of this hook to a new instance.
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
@@ -126,8 +129,10 @@ 
 			<div class="hook-control-actions">
 				<?php if ( isset( $_GET['addnew'] ) ) : // WPCS: CSRF OK. ?>
 					<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>
-				<?php else :
+				<?php else {
+	:
 						submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
+}
 					endif;
 
 					submit_button( __( 'Save Hook', 'wordpoints' ), 'button-primary alignright', 'savehook', false );

src/components/points/admin/screens/hooks.php

Spacing +7 added lines, -7 removed lines

@@ -43,18 +43,18 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET[ 'message' ] ) && isset( $messages[ (int) $_GET[ 'message' ] ] ) ) { // WPCS: CSRF OK.
 
 		wordpoints_show_admin_message(
-			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
+			esc_html( $messages[ (int) $_GET[ 'message' ] ] ) // WPCS: CSRF OK.
 			, 'success'
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
+	} elseif ( isset( $_GET[ 'error' ] ) && isset( $errors[ (int) $_GET[ 'error' ] ] ) ) { // WPCS: CSRF OK.
 
 		wordpoints_show_admin_error(
-			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.
+			esc_html( $errors[ (int) $_GET[ 'error' ] ] ) // WPCS: CSRF OK.
 			, array( 'dismissible' => true )
 		);
 	}
@@ -124,8 +124,8 @@ 
 			foreach ( $points_types as $slug => $points_type ) {
 
 				$wrap_class = 'hooks-holder-wrap';
-				if ( ! empty( $points_type['class'] ) ) {
-					$wrap_class .= ' points-type-' . $points_type['class'];
+				if ( ! empty( $points_type[ 'class' ] ) ) {
+					$wrap_class .= ' points-type-' . $points_type[ 'class' ];
 				}
 
 				if ( $i ) {
@@ -137,7 +137,7 @@ 
 				<div class="<?php echo esc_attr( $wrap_class ); ?>">
 					<div class="points-type-name">
 						<div class="points-type-name-arrow"><br /></div>
-						<h2><?php echo esc_html( $points_type['name'] ); ?><span class="spinner"></span></h2>
+						<h2><?php echo esc_html( $points_type[ 'name' ] ); ?><span class="spinner"></span></h2>
 					</div>
 					<div id="<?php echo esc_attr( $slug ); ?>" class="hooks-sortables">
 

Braces +4 added lines, -2 removed lines

@@ -43,7 +43,8 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) {
+// WPCS: CSRF OK.
 
 		wordpoints_show_admin_message(
 			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
@@ -51,7 +52,8 @@ 
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
+	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) {
+// WPCS: CSRF OK.
 
 		wordpoints_show_admin_error(
 			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.

src/components/points/classes/shortcode.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 	 */
 	protected function verify_atts() {
 
-		if ( isset( $this->pairs['points_type'] ) ) {
+		if ( isset( $this->pairs[ 'points_type' ] ) ) {
 
 			$points_type = $this->get_points_type();
 
@@ -30,7 +30,7 @@ 
 				return $points_type;
 			}
 
-			$this->atts['points_type'] = $points_type;
+			$this->atts[ 'points_type' ] = $points_type;
 		}
 
 		return parent::verify_atts();
@@ -50,7 +50,7 @@ 
 	 */
 	protected function get_points_type() {
 
-		$points_type = $this->atts['points_type'];
+		$points_type = $this->atts[ 'points_type' ];
 
 		if ( ! wordpoints_is_points_type( $points_type ) ) {
 

src/components/points/classes/shortcode/logs.php

Spacing +12 added lines, -12 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	protected function verify_atts() {
 
-		if ( ! wordpoints_is_points_logs_query( $this->atts['query'] ) ) {
+		if ( ! wordpoints_is_points_logs_query( $this->atts[ 'query' ] ) ) {
 			return sprintf(
 				// translators: 1. Attribute name; 2. Shortcode name; 3. Example of proper usage.
 				__( 'The “%1$s” attribute of the %2$s shortcode must be the slug of a registered points log query. Example: %3$s.', 'wordpoints' )
@@ -50,17 +50,17 @@ 
 			);
 		}
 
-		if ( false === wordpoints_int( $this->atts['paginate'] ) ) {
-			$this->atts['paginate'] = 1;
+		if ( false === wordpoints_int( $this->atts[ 'paginate' ] ) ) {
+			$this->atts[ 'paginate' ] = 1;
 		}
 
 		// Back-compat. Needs to stay here "forever" for legacy installs.
-		if ( isset( $this->atts['datatables'] ) ) {
-			$this->atts['paginate'] = wordpoints_int( $this->atts['datatables'] );
+		if ( isset( $this->atts[ 'datatables' ] ) ) {
+			$this->atts[ 'paginate' ] = wordpoints_int( $this->atts[ 'datatables' ] );
 		}
 
-		if ( false === wordpoints_int( $this->atts['show_users'] ) ) {
-			$this->atts['show_users'] = 1;
+		if ( false === wordpoints_int( $this->atts[ 'show_users' ] ) ) {
+			$this->atts[ 'show_users' ] = 1;
 		}
 
 		return parent::verify_atts();
@@ -74,12 +74,12 @@ 
 
 		ob_start();
 		wordpoints_show_points_logs_query(
-			$this->atts['points_type']
-			, $this->atts['query']
+			$this->atts[ 'points_type' ]
+			, $this->atts[ 'query' ]
 			, array(
-				'paginate'   => $this->atts['paginate'],
-				'show_users' => $this->atts['show_users'],
-				'searchable' => $this->atts['searchable'],
+				'paginate'   => $this->atts[ 'paginate' ],
+				'show_users' => $this->atts[ 'show_users' ],
+				'searchable' => $this->atts[ 'searchable' ],
 			)
 		);
 

src/components/points/classes/shortcode/user/points.php

Spacing +2 added lines, -2 removed lines

@@ -37,8 +37,8 @@
 	protected function generate() {
 
 		return wordpoints_get_formatted_points(
-			$this->atts['user_id']
-			, $this->atts['points_type']
+			$this->atts[ 'user_id' ]
+			, $this->atts[ 'points_type' ]
 			, 'points-shortcode'
 		);
 	}

src/components/points/classes/shortcode/top/users.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 	 */
 	protected function verify_atts() {
 
-		if ( ! wordpoints_posint( $this->atts['users'] ) ) {
+		if ( ! wordpoints_posint( $this->atts[ 'users' ] ) ) {
 			return sprintf(
 				// translators: 1. Attribute name; 2. Shortcode name; 3. Example of proper usage.
 				__( 'The “%1$s” attribute of the %2$s shortcode must be a positive integer. Example: %3$s.', 'wordpoints' )
@@ -57,8 +57,8 @@ 
 
 		ob_start();
 		wordpoints_points_show_top_users(
-			$this->atts['users']
-			, $this->atts['points_type']
+			$this->atts[ 'users' ]
+			, $this->atts[ 'points_type' ]
 			, 'shortcode'
 		);
 

src/components/points/classes/logs/view.php

Spacing +2 added lines, -2 removed lines

@@ -145,7 +145,7 @@ 
 	 */
 	protected function get_logs() {
 
-		if ( $this->args['searchable'] ) {
+		if ( $this->args[ 'searchable' ] ) {
 
 			$search_term = $this->get_search_term();
 
@@ -161,7 +161,7 @@ 
 			}
 		}
 
-		if ( $this->args['paginate'] ) {
+		if ( $this->args[ 'paginate' ] ) {
 
 			$page = $this->get_page_number();
 			$per_page = $this->get_per_page();

src/components/points/classes/logs/viewing/restriction/wrapper.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 				continue;
 			}
 
-			$this->restrictions[] = $restriction;
+			$this->restrictions[ ] = $restriction;
 		}
 	}
 

src/components/points/classes/logs/viewing/restriction/hooks.php

Spacing +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
 				continue;
 			}
 
-			$descriptions[] = sprintf( $string, $entity->get_title() );
+			$descriptions[ ] = sprintf( $string, $entity->get_title() );
 		}
 
 		return $descriptions;