WordPoints / wordpoints

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;

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

Spacing +24 added lines, -24 removed lines

@@ -61,7 +61,7 @@ 
 			);
 
 			if ( ! empty( $points_type_name ) ) {
-				$this->column_headings['points'] = $points_type_name;
+				$this->column_headings[ 'points' ] = $points_type_name;
 			}
 		}
 
@@ -69,7 +69,7 @@ 
 
 		foreach ( array( 'searchable', 'paginate' ) as $arg ) {
 			if ( $this->args[ $arg ] ) {
-				$extra_classes[] = $arg;
+				$extra_classes[ ] = $arg;
 			}
 		}
 
@@ -92,19 +92,19 @@ 
 		?>
 
 		<div class="wordpoints-points-logs-wrapper">
-			<?php if ( $this->args['searchable'] ) : ?>
+			<?php if ( $this->args[ 'searchable' ] ) : ?>
 				<?php $this->search_box(); ?>
 			<?php endif; ?>
 
 			<table class="wordpoints-points-logs widefat <?php echo esc_attr( implode( ' ', $extra_classes ) ); ?>">
 				<thead>
 					<tr>
-						<?php if ( $this->args['show_users'] ) : ?>
-							<th scope="col"><?php echo esc_html( $this->column_headings['user'] ); ?></th>
+						<?php if ( $this->args[ 'show_users' ] ) : ?>
+							<th scope="col"><?php echo esc_html( $this->column_headings[ 'user' ] ); ?></th>
 						<?php endif; ?>
-						<th scope="col"><?php echo esc_html( $this->column_headings['points'] ); ?></th>
-						<th scope="col"><?php echo esc_html( $this->column_headings['description'] ); ?></th>
-						<th scope="col"><?php echo esc_html( $this->column_headings['time'] ); ?></th>
+						<th scope="col"><?php echo esc_html( $this->column_headings[ 'points' ] ); ?></th>
+						<th scope="col"><?php echo esc_html( $this->column_headings[ 'description' ] ); ?></th>
+						<th scope="col"><?php echo esc_html( $this->column_headings[ 'time' ] ); ?></th>
 					</tr>
 				</thead>
 				<tbody>
@@ -118,7 +118,7 @@ 
 
 		?>
 		<tr>
-			<td colspan="<?php echo ( $this->args['show_users'] ) ? 4 : 3; ?>">
+			<td colspan="<?php echo ( $this->args[ 'show_users' ] ) ? 4 : 3; ?>">
 				<?php esc_html_e( 'No matching logs found.', 'wordpoints' ); ?>
 			</td>
 		</tr>
@@ -135,7 +135,7 @@ 
 		?>
 
 		<tr class="wordpoints-log-id-<?php echo (int) $log->id; ?> <?php echo ( $this->i % 2 ) ? 'odd' : 'even'; ?>">
-			<?php if ( $this->args['show_users'] ) : ?>
+			<?php if ( $this->args[ 'show_users' ] ) : ?>
 				<td>
 					<?php echo get_avatar( $user->ID, 32 ); ?>
 					<?php
@@ -197,17 +197,17 @@ 
 				</tbody>
 				<tfoot>
 					<tr>
-						<?php if ( $this->args['show_users'] ) : ?>
-							<th scope="col"><?php echo esc_html( $this->column_headings['user'] ); ?></th>
+						<?php if ( $this->args[ 'show_users' ] ) : ?>
+							<th scope="col"><?php echo esc_html( $this->column_headings[ 'user' ] ); ?></th>
 						<?php endif; ?>
-						<th scope="col"><?php echo esc_html( $this->column_headings['points'] ); ?></th>
-						<th scope="col"><?php echo esc_html( $this->column_headings['description'] ); ?></th>
-						<th scope="col"><?php echo esc_html( $this->column_headings['time'] ); ?></th>
+						<th scope="col"><?php echo esc_html( $this->column_headings[ 'points' ] ); ?></th>
+						<th scope="col"><?php echo esc_html( $this->column_headings[ 'description' ] ); ?></th>
+						<th scope="col"><?php echo esc_html( $this->column_headings[ 'time' ] ); ?></th>
 					</tr>
 				</tfoot>
 			</table>
 
-			<?php if ( $this->args['paginate'] ) : ?>
+			<?php if ( $this->args[ 'paginate' ] ) : ?>
 				<?php $this->pagination(); ?>
 			<?php endif; ?>
 		</div>
@@ -222,10 +222,10 @@ 
 
 		$search_term = '';
 
-		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST[ 'wordpoints_points_logs_search' ] ) ) { // WPCS: CSRF OK
 			$search_term = trim(
 				sanitize_text_field(
-					wp_unslash( $_POST['wordpoints_points_logs_search'] ) // WPCS: CSRF OK
+					wp_unslash( $_POST[ 'wordpoints_points_logs_search' ] ) // WPCS: CSRF OK
 				)
 			);
 
@@ -243,10 +243,10 @@ 
 		$page = 1;
 
 		if (
-			isset( $_GET['wordpoints_points_logs_page'] ) // WPCS: CSRF OK.
-			&& wordpoints_posint( $_GET['wordpoints_points_logs_page'] ) // WPCS: CSRF OK.
+			isset( $_GET[ 'wordpoints_points_logs_page' ] ) // WPCS: CSRF OK.
+			&& wordpoints_posint( $_GET[ 'wordpoints_points_logs_page' ] ) // WPCS: CSRF OK.
 		) {
-			$page = (int) $_GET['wordpoints_points_logs_page'];
+			$page = (int) $_GET[ 'wordpoints_points_logs_page' ];
 		}
 
 		return $page;
@@ -260,10 +260,10 @@ 
 		$per_page = 25;
 
 		if (
-			isset( $_GET['wordpoints_points_logs_per_page'] ) // WPCS: CSRF OK.
-			&& wordpoints_posint( $_GET['wordpoints_points_logs_per_page'] ) // WPCS: CSRF OK.
+			isset( $_GET[ 'wordpoints_points_logs_per_page' ] ) // WPCS: CSRF OK.
+			&& wordpoints_posint( $_GET[ 'wordpoints_points_logs_per_page' ] ) // WPCS: CSRF OK.
 		) {
-			$per_page = (int) $_GET['wordpoints_points_logs_per_page'];
+			$per_page = (int) $_GET[ 'wordpoints_points_logs_per_page' ];
 		}
 
 		return $per_page;

Braces +2 added lines, -1 removed lines

@@ -222,7 +222,8 @@
 
 		$search_term = '';
 
-		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) {
+// WPCS: CSRF OK
 			$search_term = trim(
 				sanitize_text_field(
 					wp_unslash( $_POST['wordpoints_points_logs_search'] ) // WPCS: CSRF OK

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/hook/reactor/legacy.php

Spacing +9 added lines, -9 removed lines

@@ -27,17 +27,17 @@ 
 		array $settings
 	) {
 
-		if ( isset( $settings['legacy_log_type'] ) ) {
+		if ( isset( $settings[ 'legacy_log_type' ] ) ) {
 			$reaction->update_meta(
 				'legacy_log_type',
-				$settings['legacy_log_type']
+				$settings[ 'legacy_log_type' ]
 			);
 		}
 
-		if ( isset( $settings['legacy_meta_key'] ) ) {
+		if ( isset( $settings[ 'legacy_meta_key' ] ) ) {
 			$reaction->update_meta(
 				'legacy_meta_key',
-				$settings['legacy_meta_key']
+				$settings[ 'legacy_meta_key' ]
 			);
 		}
 
@@ -51,7 +51,7 @@ 
 
 		$data = parent::get_ui_script_data();
 
-		$data['reversals_extension_slug'] = 'points_legacy_reversals';
+		$data[ 'reversals_extension_slug' ] = 'points_legacy_reversals';
 
 		return $data;
 	}
@@ -61,10 +61,10 @@ 
 	 */
 	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
 
-		if ( isset( $fire->data['points_legacy_reversals']['points_logs'] ) ) {
+		if ( isset( $fire->data[ 'points_legacy_reversals' ][ 'points_logs' ] ) ) {
 
 			$this->reverse_logs(
-				$fire->data['points_legacy_reversals']['points_logs']
+				$fire->data[ 'points_legacy_reversals' ][ 'points_logs' ]
 				, $fire
 			);
 
@@ -79,11 +79,11 @@ 
 	protected function get_hit_ids_to_be_reversed( WordPoints_Hook_Fire $fire ) {
 
 		// We closely integrate with the legacy reversals extension to get the IDs.
-		if ( ! isset( $fire->data['points_legacy_reversals']['hit_ids'] ) ) {
+		if ( ! isset( $fire->data[ 'points_legacy_reversals' ][ 'hit_ids' ] ) ) {
 			return array();
 		}
 
-		return $fire->data['points_legacy_reversals']['hit_ids'];
+		return $fire->data[ 'points_legacy_reversals' ][ 'hit_ids' ];
 	}
 }
 

src/components/points/classes/hook/extension/legacy/periods.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
 		$expected_settings = array( array( 'current:user' ) );
 
-		if ( ! isset( $settings['args'] ) || $settings['args'] !== $expected_settings ) {
+		if ( ! isset( $settings[ 'args' ] ) || $settings[ 'args' ] !== $expected_settings ) {
 			return false;
 		}
 

src/components/points/classes/hook/extension/legacy/reversals.php

Spacing +5 added lines, -5 removed lines

@@ -65,8 +65,8 @@ 
 	 */
 	protected function get_points_logs_to_be_reversed( WordPoints_Hook_Fire $fire ) {
 
-		if ( isset( $fire->data[ $this->slug ]['points_logs'] ) ) {
-			return $fire->data[ $this->slug ]['points_logs'];
+		if ( isset( $fire->data[ $this->slug ][ 'points_logs' ] ) ) {
+			return $fire->data[ $this->slug ][ 'points_logs' ];
 		}
 
 		$meta_queries = array(
@@ -85,14 +85,14 @@ 
 			$entities = $fire->event_args->get_signature_args();
 
 			if ( ! $entities ) {
-				$fire->data[ $this->slug ]['points_logs'] = array();
+				$fire->data[ $this->slug ][ 'points_logs' ] = array();
 				return array();
 			}
 
 			// Legacy hooks only ever related to a single entity.
 			$entity = reset( $entities );
 
-			$meta_queries[] = array(
+			$meta_queries[ ] = array(
 				'key'   => $meta_key,
 				'value' => $entity->get_the_id(),
 			);
@@ -117,7 +117,7 @@ 
 			$logs = array();
 		}
 
-		$fire->data[ $this->slug ]['points_logs'] = $logs;
+		$fire->data[ $this->slug ][ 'points_logs' ] = $logs;
 
 		return $logs;
 	}

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/hook/event/post/publish/legacy.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
 		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
 
-		switch ( $parsed['dynamic'] ) {
+		switch ( $parsed[ 'dynamic' ] ) {
 
 			case 'post':
 				return __( 'Post deleted.', 'wordpoints' );

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(),
 		);