WordPoints / wordpoints

src/components/ranks/classes/rank/group.php

Doc Comments +2 added lines, -2 removed lines

@@ -368,7 +368,7 @@ 
 	 *
 	 * @since 1.7.0
 	 *
-	 * @return int|false The ID of the rank, or false if not found.
+	 * @return integer The ID of the rank, or false if not found.
 	 */
 	public function get_base_rank() {
 
@@ -402,7 +402,7 @@ 
 	 *
 	 * @param int $rank_id The ID of the rank to get the position of.
 	 *
-	 * @return int|false The rank's position, or false.
+	 * @return integer The rank's position, or false.
 	 */
 	public function get_rank_position( $rank_id ) {
 

Spacing +1 added lines, -1 removed lines

@@ -440,7 +440,7 @@
 
 		} elseif ( $count < $position ) {
 
-			$ranks[] = $rank_id;
+			$ranks[ ] = $rank_id;
 
 		} else {
 

src/components/points/admin/classes/screen/points/types.php

Spacing +30 added lines, -30 removed lines

@@ -302,7 +302,7 @@ 
 					type="text"
 					id="points-name-<?php echo esc_attr( $slug ); ?>"
 					name="points-name"
-					value="<?php echo esc_attr( $points_type['name'] ); ?>"
+					value="<?php echo esc_attr( $points_type[ 'name' ] ); ?>"
 					<?php if ( ! $slug ) : ?>
 					autofocus
 					<?php endif; ?>
@@ -315,7 +315,7 @@ 
 					type="text"
 					id="points-prefix-<?php echo esc_attr( $slug ); ?>"
 					name="points-prefix"
-					value="<?php echo esc_attr( $points_type['prefix'] ); ?>"
+					value="<?php echo esc_attr( $points_type[ 'prefix' ] ); ?>"
 				/>
 			</p>
 			<p>
@@ -325,7 +325,7 @@ 
 					type="text"
 					id="points-suffix-<?php echo esc_attr( $slug ); ?>"
 					name="points-suffix"
-					value="<?php echo esc_attr( $points_type['suffix'] ); ?>"
+					value="<?php echo esc_attr( $points_type[ 'suffix' ] ); ?>"
 				/>
 			</p>
 
@@ -547,13 +547,13 @@ 
 	 */
 	public function display_event_meta_box( $points_type, $meta_box ) {
 
-		$event_slug = $meta_box['args']['slug'];
+		$event_slug = $meta_box[ 'args' ][ 'slug' ];
 
 		$data = array();
 
 		foreach ( $this->reaction_store->get_reactions_to_event( $event_slug ) as $id => $reaction ) {
 			if ( $reaction->get_meta( 'points_type' ) === $this->current_points_type ) {
-				$data[] = WordPoints_Admin_Ajax_Hooks::prepare_hook_reaction(
+				$data[ ] = WordPoints_Admin_Ajax_Hooks::prepare_hook_reaction(
 					$reaction
 				);
 			}
@@ -567,13 +567,13 @@ 
 
 		foreach ( $this->event_args->get_children( $event_slug ) as $slug => $arg ) {
 
-			$event_data['args'][ $slug ] = array(
+			$event_data[ 'args' ][ $slug ] = array(
 				'slug' => $slug,
 			);
 
 			if ( $arg instanceof WordPoints_Hook_ArgI ) {
-				$event_data['args'][ $slug ]['title']       = $arg->get_title();
-				$event_data['args'][ $slug ]['is_stateful'] = $arg->is_stateful();
+				$event_data[ 'args' ][ $slug ][ 'title' ]       = $arg->get_title();
+				$event_data[ 'args' ][ $slug ][ 'is_stateful' ] = $arg->is_stateful();
 			}
 		}
 
@@ -586,7 +586,7 @@ 
 
 		<div class="wordpoints-hook-reaction-group-container">
 			<p class="description wordpoints-hook-reaction-group-description">
-				<?php echo esc_html( $meta_box['args']['event']->get_description() ); ?>
+				<?php echo esc_html( $meta_box[ 'args' ][ 'event' ]->get_description() ); ?>
 			</p>
 
 			<div class="wordpoints-hook-reaction-group"
@@ -628,10 +628,10 @@ 
 		$tabs = array();
 
 		foreach ( $points_types as $slug => $settings ) {
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[ $slug ] = $settings[ 'name' ];
 		}
 
-		$tabs['add-new'] = __( 'Add New', 'wordpoints' );
+		$tabs[ 'add-new' ] = __( 'Add New', 'wordpoints' );
 
 		$tab = wordpoints_admin_get_current_tab( $tabs );
 
@@ -655,15 +655,15 @@ 
 			return;
 		}
 
-		if ( isset( $_POST['save-points-type'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST[ 'save-points-type' ] ) ) { // WPCS: CSRF OK
 
-			if ( ! empty( $_POST['add_new'] ) ) { // WPCS: CSRF OK
+			if ( ! empty( $_POST[ 'add_new' ] ) ) { // WPCS: CSRF OK
 				$this->add_points_type();
 			} else {
 				$this->update_points_type();
 			}
 
-		} elseif ( ! empty( $_POST['delete-points-type'] ) ) { // WPCS: CSRF OK
+		} elseif ( ! empty( $_POST[ 'delete-points-type' ] ) ) { // WPCS: CSRF OK
 
 			$this->delete_points_type();
 		}
@@ -680,21 +680,21 @@ 
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) { // WPCS: CSRF OK
-			$settings['name'] = trim(
-				sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) // WPCS: CSRF OK
+		if ( isset( $_POST[ 'points-name' ] ) ) { // WPCS: CSRF OK
+			$settings[ 'name' ] = trim(
+				sanitize_text_field( wp_unslash( $_POST[ 'points-name' ] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) { // WPCS: CSRF OK
-			$settings['prefix'] = ltrim(
-				sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) // WPCS: CSRF OK
+		if ( isset( $_POST[ 'points-prefix' ] ) ) { // WPCS: CSRF OK
+			$settings[ 'prefix' ] = ltrim(
+				sanitize_text_field( wp_unslash( $_POST[ 'points-prefix' ] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) { // WPCS: CSRF OK
-			$settings['suffix'] = rtrim(
-				sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) // WPCS: CSRF OK
+		if ( isset( $_POST[ 'points-suffix' ] ) ) { // WPCS: CSRF OK
+			$settings[ 'suffix' ] = rtrim(
+				sanitize_text_field( wp_unslash( $_POST[ 'points-suffix' ] ) ) // WPCS: CSRF OK
 			);
 		}
 
@@ -715,14 +715,14 @@ 
 				, array( 'points-slug' )
 				, 'post'
 			)
-			|| ! isset( $_POST['points-slug'] )
+			|| ! isset( $_POST[ 'points-slug' ] )
 		) {
 			return;
 		}
 
 		$settings = $this->get_points_type_settings();
 
-		if ( empty( $settings['name'] ) ) {
+		if ( empty( $settings[ 'name' ] ) ) {
 
 			add_settings_error(
 				'points-name'
@@ -733,7 +733,7 @@ 
 			return;
 		}
 
-		$points_type = sanitize_key( $_POST['points-slug'] );
+		$points_type = sanitize_key( $_POST[ 'points-slug' ] );
 
 		$old_settings = wordpoints_get_points_type( $points_type );
 
@@ -803,7 +803,7 @@ 
 
 		} else {
 
-			$_GET['tab'] = $slug;
+			$_GET[ 'tab' ] = $slug;
 
 			add_settings_error(
 				''
@@ -828,12 +828,12 @@ 
 				, array( 'points-slug' )
 				, 'post'
 			)
-			&& isset( $_POST['points-slug'] )
+			&& isset( $_POST[ 'points-slug' ] )
 		) {
 
 			if (
 				wordpoints_delete_points_type(
-					sanitize_key( $_POST['points-slug'] )
+					sanitize_key( $_POST[ 'points-slug' ] )
 				)
 			) {
 
@@ -926,7 +926,7 @@ 
 
 		if ( isset( $this->current_points_type ) ) {
 			$points_type         = wordpoints_get_points_type( $this->current_points_type );
-			$points_type['slug'] = $this->current_points_type;
+			$points_type[ 'slug' ] = $this->current_points_type;
 		} else {
 			$points_type = false;
 		}

Braces +17 added lines, -8 removed lines

@@ -245,8 +245,11 @@ 
 					<p>
 						<?php if ( $slug ) : ?>
 							<?php esc_html_e( 'Changes to this points type&#8217;s settings will affect all sites on this network.', 'wordpoints' ); ?>
-						<?php else : ?>
-							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' ); ?>
+						<?php else {
+	: ?>
+							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' );
+}
+?>
 						<?php endif; ?>
 					</p>
 				</div>
@@ -655,15 +658,18 @@ 
 			return;
 		}
 
-		if ( isset( $_POST['save-points-type'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['save-points-type'] ) ) {
+// WPCS: CSRF OK
 
-			if ( ! empty( $_POST['add_new'] ) ) { // WPCS: CSRF OK
+			if ( ! empty( $_POST['add_new'] ) ) {
+// WPCS: CSRF OK
 				$this->add_points_type();
 			} else {
 				$this->update_points_type();
 			}
 
-		} elseif ( ! empty( $_POST['delete-points-type'] ) ) { // WPCS: CSRF OK
+		} elseif ( ! empty( $_POST['delete-points-type'] ) ) {
+// WPCS: CSRF OK
 
 			$this->delete_points_type();
 		}
@@ -680,19 +686,22 @@ 
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['points-name'] ) ) {
+// WPCS: CSRF OK
 			$settings['name'] = trim(
 				sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['points-prefix'] ) ) {
+// WPCS: CSRF OK
 			$settings['prefix'] = ltrim(
 				sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['points-suffix'] ) ) {
+// WPCS: CSRF OK
 			$settings['suffix'] = rtrim(
 				sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) // WPCS: CSRF OK
 			);

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 );
 
@@ -130,7 +130,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 : ?>
 					<?php submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false ); ?>

Braces +11 added lines, -5 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(
@@ -132,8 +135,11 @@ 
 			<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 submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false ); ?>
+				<?php else {
+	: ?>
+					<?php submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
+}
+?>
 				<?php endif; ?>
 
 				<?php submit_button( __( 'Save Hook', 'wordpoints' ), 'button-primary alignright', 'savehook', false ); ?>

src/components/points/classes/hook/reactor.php

Spacing +15 added lines, -15 removed lines

@@ -58,9 +58,9 @@ 
 	 */
 	public function get_settings_fields() {
 
-		$this->settings_fields['points']['label']      = _x( 'Points', 'form label', 'wordpoints' );
-		$this->settings_fields['log_text']['label']    = _x( 'Log Text', 'form label', 'wordpoints' );
-		$this->settings_fields['description']['label'] = _x( 'Description', 'form label', 'wordpoints' );
+		$this->settings_fields[ 'points' ][ 'label' ]      = _x( 'Points', 'form label', 'wordpoints' );
+		$this->settings_fields[ 'log_text' ][ 'label' ]    = _x( 'Log Text', 'form label', 'wordpoints' );
+		$this->settings_fields[ 'description' ][ 'label' ] = _x( 'Description', 'form label', 'wordpoints' );
 
 		return parent::get_settings_fields();
 	}
@@ -72,8 +72,8 @@ 
 
 		$data = parent::get_ui_script_data();
 
-		$data['target_label']  = __( 'Award To', 'wordpoints' );
-		$data['periods_label'] = __( 'Award each user no more than once in:', 'wordpoints' );
+		$data[ 'target_label' ]  = __( 'Award To', 'wordpoints' );
+		$data[ 'periods_label' ] = __( 'Award each user no more than once in:', 'wordpoints' );
 
 		return $data;
 	}
@@ -87,19 +87,19 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings['points'] ) || false === wordpoints_int( $settings['points'] ) ) {
+		if ( ! isset( $settings[ 'points' ] ) || false === wordpoints_int( $settings[ 'points' ] ) ) {
 			$validator->add_error( __( 'Points must be an integer.', 'wordpoints' ), 'points' );
 		}
 
-		if ( ! isset( $settings['points_type'] ) || ! wordpoints_is_points_type( $settings['points_type'] ) ) {
+		if ( ! isset( $settings[ 'points_type' ] ) || ! wordpoints_is_points_type( $settings[ 'points_type' ] ) ) {
 			$validator->add_error( __( 'Invalid points type.', 'wordpoints' ), 'points_type' );
 		}
 
-		if ( ! isset( $settings['description'] ) ) {
+		if ( ! isset( $settings[ 'description' ] ) ) {
 			$validator->add_error( __( 'Description is required.', 'wordpoints' ), 'description' );
 		}
 
-		if ( ! isset( $settings['log_text'] ) ) {
+		if ( ! isset( $settings[ 'log_text' ] ) ) {
 			$validator->add_error( __( 'Log Text is required.', 'wordpoints' ), 'log_text' );
 		}
 
@@ -116,10 +116,10 @@ 
 
 		parent::update_settings( $reaction, $settings );
 
-		$reaction->update_meta( 'points', $settings['points'] );
-		$reaction->update_meta( 'points_type', $settings['points_type'] );
-		$reaction->update_meta( 'description', $settings['description'] );
-		$reaction->update_meta( 'log_text', $settings['log_text'] );
+		$reaction->update_meta( 'points', $settings[ 'points' ] );
+		$reaction->update_meta( 'points_type', $settings[ 'points_type' ] );
+		$reaction->update_meta( 'description', $settings[ 'description' ] );
+		$reaction->update_meta( 'log_text', $settings[ 'log_text' ] );
 	}
 
 	/**
@@ -203,11 +203,11 @@ 
 	protected function get_hit_ids_to_be_reversed( WordPoints_Hook_Fire $fire ) {
 
 		// We closely integrate with the reversals extension to get the hit IDs.
-		if ( ! isset( $fire->data['reversals']['hit_ids'] ) ) {
+		if ( ! isset( $fire->data[ 'reversals' ][ 'hit_ids' ] ) ) {
 			return array();
 		}
 
-		return $fire->data['reversals']['hit_ids'];
+		return $fire->data[ 'reversals' ][ 'hit_ids' ];
 	}
 
 	/**

src/components/points/classes/log/queries.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 		);
 
 		self::$queries[ $slug ]         = array_merge( $defaults, $data );
-		self::$queries[ $slug ]['args'] = $args;
+		self::$queries[ $slug ][ 'args' ] = $args;
 
 		return true;
 	}

src/components/points/classes/legacy/hook/to/reaction/importer.php

Spacing +31 added lines, -31 removed lines

@@ -273,7 +273,7 @@ 
 		);
 
 		$reaction_settings = array(
-			'points'          => $legacy_instance['points'],
+			'points'          => $legacy_instance[ 'points' ],
 			'target'          => $this->target,
 			'reactor'         => 'points_legacy',
 			'event'           => $this->event_slug,
@@ -294,8 +294,8 @@ 
 			$order
 		);
 
-		if ( ! isset( $reaction_settings['log_text'] ) ) {
-			$reaction_settings['log_text'] = $this->get_log_text_for_instance(
+		if ( ! isset( $reaction_settings[ 'log_text' ] ) ) {
+			$reaction_settings[ 'log_text' ] = $this->get_log_text_for_instance(
 				$points_type
 			);
 		}
@@ -333,8 +333,8 @@ 
 
 		if (
 			$this->skip_non_reversing_hooks
-			&& isset( $this->legacy_instance['auto_reverse'] )
-			&& ! $this->legacy_instance['auto_reverse']
+			&& isset( $this->legacy_instance[ 'auto_reverse' ] )
+			&& ! $this->legacy_instance[ 'auto_reverse' ]
 		) {
 			return false;
 		}
@@ -359,17 +359,17 @@ 
 		}
 
 		if (
-			! isset( $meta['post_type'] )
-			&& isset( $this->legacy_instance['post_type'] )
+			! isset( $meta[ 'post_type' ] )
+			&& isset( $this->legacy_instance[ 'post_type' ] )
 		) {
-			$meta['post_type'] = $this->legacy_instance['post_type'];
+			$meta[ 'post_type' ] = $this->legacy_instance[ 'post_type' ];
 		}
 
 		if (
-			! isset( $meta['period'] )
-			&& isset( $this->legacy_instance['period'] )
+			! isset( $meta[ 'period' ] )
+			&& isset( $this->legacy_instance[ 'period' ] )
 		) {
-			$meta['period'] = $this->legacy_instance['period'];
+			$meta[ 'period' ] = $this->legacy_instance[ 'period' ];
 		}
 
 		return $this->legacy_handler->logs(
@@ -398,10 +398,10 @@ 
 
 		if ( 'wordpoints_periodic_points_hook' === $this->legacy_id_base ) {
 
-			$settings['points_legacy_periods'] = array(
+			$settings[ 'points_legacy_periods' ] = array(
 				'fire' => array(
 					array(
-						'length' => $this->legacy_instance['period'],
+						'length' => $this->legacy_instance[ 'period' ],
 						'args'   => array( array( 'current:user' ) ),
 					),
 				),
@@ -411,21 +411,21 @@ 
 
 			// We do this even if reversals will be blocked, in case the blocking
 			// is ever removed for this reaction.
-			$settings['points_legacy_reversals'] = array(
+			$settings[ 'points_legacy_reversals' ] = array(
 				'toggle_off' => 'toggle_on',
 			);
 		}
 
 		if (
-			isset( $this->legacy_instance['auto_reverse'] )
-			&& ! $this->legacy_instance['auto_reverse']
+			isset( $this->legacy_instance[ 'auto_reverse' ] )
+			&& ! $this->legacy_instance[ 'auto_reverse' ]
 		) {
-			$settings['blocker']['toggle_off'] = true;
+			$settings[ 'blocker' ][ 'toggle_off' ] = true;
 		}
 
-		if ( isset( $this->legacy_instance['post_type'] ) ) {
+		if ( isset( $this->legacy_instance[ 'post_type' ] ) ) {
 
-			$post_type = $this->legacy_instance['post_type'];
+			$post_type = $this->legacy_instance[ 'post_type' ];
 
 			if ( 'ALL' === $post_type ) {
 
@@ -467,36 +467,36 @@ 
 	protected function format_settings_for_post_type( $post_type, $settings ) {
 
 		if (
-			'post_publish\post' === $settings['event']
-			|| 'points_legacy_post_publish\post' === $settings['event']
+			'post_publish\post' === $settings[ 'event' ]
+			|| 'points_legacy_post_publish\post' === $settings[ 'event' ]
 		) {
 
 			if ( 'attachment' === $post_type ) {
 
-				$settings['event'] = 'media_upload';
+				$settings[ 'event' ] = 'media_upload';
 
 			} else {
 
-				$settings['points_legacy_repeat_blocker'] = array(
+				$settings[ 'points_legacy_repeat_blocker' ] = array(
 					'toggle_on' => true,
 				);
 			}
 		}
 
-		$settings['event'] = str_replace(
+		$settings[ 'event' ] = str_replace(
 			'\post'
 			, '\\' . $post_type
-			, $settings['event']
+			, $settings[ 'event' ]
 		);
 
-		$settings['target'] = str_replace(
+		$settings[ 'target' ] = str_replace(
 			'\post'
 			, '\\' . $post_type
-			, $settings['target']
+			, $settings[ 'target' ]
 		);
 
-		$settings['log_text'] = $this->get_log_text_for_instance(
-			$settings['points_type']
+		$settings[ 'log_text' ] = $this->get_log_text_for_instance(
+			$settings[ 'points_type' ]
 			, array( 'post_type' => $post_type )
 		);
 
@@ -521,11 +521,11 @@ 
 			return false;
 		}
 
-		$this->imported_hooks[] = array(
+		$this->imported_hooks[ ] = array(
 			'order'       => $order,
 			'id_base'     => $this->legacy_id_base,
 			'instance'    => $this->legacy_instance,
-			'points_type' => $settings['points_type'],
+			'points_type' => $settings[ 'points_type' ],
 			'reaction_id' => $reaction->get_id(),
 		);
 

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

Spacing +17 added lines, -17 removed lines

@@ -72,22 +72,22 @@ 
 	 */
 	protected function verify_settings( $instance ) {
 
-		if ( ! is_user_logged_in() && empty( $instance['alt_text'] ) ) {
+		if ( ! is_user_logged_in() && empty( $instance[ 'alt_text' ] ) ) {
 			return new WP_Error();
 		}
 
 		if (
-			! isset( $instance['number_logs'] )
-			|| ! wordpoints_posint( $instance['number_logs'] )
+			! isset( $instance[ 'number_logs' ] )
+			|| ! wordpoints_posint( $instance[ 'number_logs' ] )
 		) {
-			$instance['number_logs'] = 0;
+			$instance[ 'number_logs' ] = 0;
 		}
 
 		// In case the points type isn't set, we do this first.
 		$instance = WordPoints_Points_Widget::verify_settings( $instance );
 
-		if ( ! is_wp_error( $instance ) && is_user_logged_in() && empty( $instance['text'] ) ) {
-			$instance['text'] = wordpoints_get_points_type_setting( $instance['points_type'], 'name' ) . ': %points%';
+		if ( ! is_wp_error( $instance ) && is_user_logged_in() && empty( $instance[ 'text' ] ) ) {
+			$instance[ 'text' ] = wordpoints_get_points_type_setting( $instance[ 'points_type' ], 'name' ) . ': %points%';
 		}
 
 		return $instance;
@@ -104,16 +104,16 @@ 
 			$text = str_replace(
 				'%points%',
 				wordpoints_format_points(
-					wordpoints_get_points( get_current_user_id(), $instance['points_type'] ),
-					$instance['points_type'],
+					wordpoints_get_points( get_current_user_id(), $instance[ 'points_type' ] ),
+					$instance[ 'points_type' ],
 					'my_points_widget'
 				),
-				$instance['text']
+				$instance[ 'text' ]
 			);
 
 		} else {
 
-			$text = $instance['alt_text'];
+			$text = $instance[ 'alt_text' ];
 		}
 
 		/**
@@ -134,7 +134,7 @@ 
 
 		echo '<div class="wordpoints-points-widget-text">', $text, '</div><br />'; // XSS OK, WPCS
 
-		if ( is_user_logged_in() && 0 !== $instance['number_logs'] ) {
+		if ( is_user_logged_in() && 0 !== $instance[ 'number_logs' ] ) {
 			parent::widget_body( $instance );
 		}
 	}
@@ -152,12 +152,12 @@ 
 	 */
 	public function update( $new_instance, $old_instance ) {
 
-		$this->defaults['number_logs'] = 0;
+		$this->defaults[ 'number_logs' ] = 0;
 
 		parent::update( $new_instance, $old_instance );
 
-		$this->instance['text']     = trim( $this->instance['text'] );
-		$this->instance['alt_text'] = trim( $this->instance['alt_text'] );
+		$this->instance[ 'text' ]     = trim( $this->instance[ 'text' ] );
+		$this->instance[ 'alt_text' ] = trim( $this->instance[ 'alt_text' ] );
 
 		return $this->instance;
 	}
@@ -174,7 +174,7 @@ 
 
 		<p>
 			<label for="<?php echo esc_attr( $this->get_field_id( 'text' ) ); ?>"><?php esc_html_e( 'Widget text', 'wordpoints' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'text' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'text' ) ); ?>" value="<?php echo esc_attr( $this->instance['text'] ); ?>" />
+			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'text' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'text' ) ); ?>" value="<?php echo esc_attr( $this->instance[ 'text' ] ); ?>" />
 			<small>
 				<i>
 					<?php
@@ -200,12 +200,12 @@ 
 		</p>
 		<p>
 			<label for="<?php echo esc_attr( $this->get_field_id( 'alt_text' ) ); ?>"><?php esc_html_e( 'Text if the user is not logged in', 'wordpoints' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'alt_text' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'alt_text' ) ); ?>" value="<?php echo esc_attr( $this->instance['alt_text'] ); ?>" />
+			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'alt_text' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'alt_text' ) ); ?>" value="<?php echo esc_attr( $this->instance[ 'alt_text' ] ); ?>" />
 			<small><i><?php esc_html_e( 'Leave this field blank to hide the widget if the user is not logged in', 'wordpoints' ); ?></i></small>
 		</p>
 		<p>
 			<label for="<?php echo esc_attr( $this->get_field_id( 'number_logs' ) ); ?>"><?php esc_html_e( 'Number of latest log entries for this user to display', 'wordpoints' ); ?></label>
-			<input type="number" min="0" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'number_logs' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'number_logs' ) ); ?>" value="<?php echo esc_attr( $this->instance['number_logs'] ); ?>" />
+			<input type="number" min="0" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'number_logs' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'number_logs' ) ); ?>" value="<?php echo esc_attr( $this->instance[ 'number_logs' ] ); ?>" />
 			<small><i><?php esc_html_e( 'Set this to 0 to keep from showing any logs', 'wordpoints' ); ?></i></small>
 		</p>
 

src/components/points/classes/updater/2/4/0/condition/contains.php

Spacing +10 added lines, -10 removed lines

@@ -80,9 +80,9 @@ 
 				continue;
 			}
 
-			if ( isset( $sub_args['_conditions'] ) && '}' === substr( $arg_slug, -1 ) ) {
-				$sub_args['_conditions'] = $this->process_conditions(
-					$sub_args['_conditions']
+			if ( isset( $sub_args[ '_conditions' ] ) && '}' === substr( $arg_slug, -1 ) ) {
+				$sub_args[ '_conditions' ] = $this->process_conditions(
+					$sub_args[ '_conditions' ]
 				);
 			}
 
@@ -105,9 +105,9 @@ 
 
 		foreach ( $conditions as $index => $condition ) {
 
-			if ( 'contains' === $condition['type'] ) {
-				$condition['settings'] = $this->process_condition(
-					$condition['settings']
+			if ( 'contains' === $condition[ 'type' ] ) {
+				$condition[ 'settings' ] = $this->process_condition(
+					$condition[ 'settings' ]
 				);
 			}
 
@@ -128,15 +128,15 @@ 
 	 */
 	protected function process_condition( $settings ) {
 
-		if ( isset( $settings['max'] ) && empty( $settings['max'] ) ) {
+		if ( isset( $settings[ 'max' ] ) && empty( $settings[ 'max' ] ) ) {
 
-			$settings['max'] = 0;
+			$settings[ 'max' ] = 0;
 
 			$this->changed = true;
 		}
 
-		if ( isset( $settings['conditions'] ) ) {
-			$settings['conditions'] = $this->walk_args( $settings['conditions'] );
+		if ( isset( $settings[ 'conditions' ] ) ) {
+			$settings[ 'conditions' ] = $this->walk_args( $settings[ 'conditions' ] );
 		}
 
 		return $settings;

src/components/points/classes/updater/1/4/0/hooks.php

Spacing +11 added lines, -11 removed lines

@@ -61,10 +61,10 @@ 
 
 		foreach ( $this->hooks as $data ) {
 			$this->split_hooks(
-				$data['hook'],
-				$data['new_hook'],
-				$data['key'],
-				$data['split_key']
+				$data[ 'hook' ],
+				$data[ 'new_hook' ],
+				$data[ 'key' ],
+				$data[ 'split_key' ]
 			);
 		}
 
@@ -107,8 +107,8 @@ 
 				continue;
 			}
 
-			if ( ! isset( $settings['post_type'] ) ) {
-				$settings['post_type'] = 'ALL';
+			if ( ! isset( $settings[ 'post_type' ] ) ) {
+				$settings[ 'post_type' ] = 'ALL';
 			}
 
 			// If the trash points are set, create a post delete points hook instead.
@@ -117,7 +117,7 @@ 
 				$new_hook->update_callback(
 					array(
 						'points'    => $settings[ $split_key ],
-						'post_type' => $settings['post_type'],
+						'post_type' => $settings[ 'post_type' ],
 					)
 					, $new_hook->next_hook_id_number()
 				);
@@ -126,13 +126,13 @@ 
 				$points_type = $hook->points_type( $network_ . $number );
 
 				// Add this instance to the points-types-hooks list.
-				$points_types_hooks[ $points_type ][] = $new_hook->get_id( $number );
+				$points_types_hooks[ $points_type ][ ] = $new_hook->get_id( $number );
 			}
 
 			// If the publish points are set, update the settings of the hook.
 			if ( isset( $settings[ $key ] ) && wordpoints_posint( $settings[ $key ] ) ) {
 
-				$settings['points'] = $settings[ $key ];
+				$settings[ 'points' ] = $settings[ $key ];
 
 				$hook->update_callback( $settings, $number );
 
@@ -166,9 +166,9 @@ 
 	public function clean_hook_settings( $instance, $new_instance, $old_instance, $hook ) {
 
 		if ( $hook instanceof WordPoints_Post_Points_Hook ) {
-			unset( $instance['trash'], $instance['publish'] );
+			unset( $instance[ 'trash' ], $instance[ 'publish' ] );
 		} elseif ( $hook instanceof WordPoints_Comment_Points_Hook ) {
-			unset( $instance['approve'], $instance['disapprove'] );
+			unset( $instance[ 'approve' ], $instance[ 'disapprove' ] );
 		}
 
 		return $instance;