WordPoints / wordpoints

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

Spacing +12 added lines, -12 removed lines

@@ -43,17 +43,17 @@ 
 	 * @param object                                        $log          Points log.
 	 * @param WordPoints_Points_Logs_Viewing_RestrictionI[] $restrictions Restrictions.
 	 */
-	public function __construct( $log, array $restrictions = array() ) {
+	public function __construct($log, array $restrictions = array()) {
 
 		$this->log = $log;
 
-		foreach ( $restrictions as $restriction ) {
+		foreach ($restrictions as $restriction) {
 
-			if ( ! $restriction instanceof WordPoints_Points_Logs_Viewing_RestrictionI ) {
+			if ( ! $restriction instanceof WordPoints_Points_Logs_Viewing_RestrictionI) {
 				continue;
 			}
 
-			if ( ! $restriction->applies() ) {
+			if ( ! $restriction->applies()) {
 				continue;
 			}
 
@@ -68,7 +68,7 @@ 
 
 		$descriptions = array();
 
-		foreach ( $this->restrictions as $restriction ) {
+		foreach ($this->restrictions as $restriction) {
 
 			$descriptions = array_merge(
 				$descriptions
@@ -83,28 +83,28 @@ 
 	 * @since 2.2.0
 	 */
 	public function applies() {
-		return ! empty( $this->restrictions );
+		return ! empty($this->restrictions);
 	}
 
 	/**
 	 * @since 2.2.0
 	 */
-	public function user_can( $user_id ) {
+	public function user_can($user_id) {
 
-		if ( $this->log->blog_id && get_current_blog_id() !== $this->log->blog_id ) {
-			$switched = switch_to_blog( $this->log->blog_id );
+		if ($this->log->blog_id && get_current_blog_id() !== $this->log->blog_id) {
+			$switched = switch_to_blog($this->log->blog_id);
 		}
 
 		$user_can = true;
 
-		foreach ( $this->restrictions as $restriction ) {
-			if ( ! $restriction->user_can( $user_id ) ) {
+		foreach ($this->restrictions as $restriction) {
+			if ( ! $restriction->user_can($user_id)) {
 				$user_can = false;
 				break;
 			}
 		}
 
-		if ( isset( $switched ) ) {
+		if (isset($switched)) {
 			restore_current_blog();
 		}
 

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

Spacing +24 added lines, -24 removed lines

@@ -32,24 +32,24 @@ 
 	/**
 	 * @since 2.2.0
 	 */
-	public function __construct( $log ) {
+	public function __construct($log) {
 
-		$events = wordpoints_hooks()->get_sub_app( 'events' );
+		$events = wordpoints_hooks()->get_sub_app('events');
 
 		$log_id = $log->id;
 		$event_slug = $log->log_type;
-		$is_reversal = ( 'reverse-' === substr( $log->log_type, 0, 8 ) );
+		$is_reversal = ('reverse-' === substr($log->log_type, 0, 8));
 
-		if ( $is_reversal ) {
-			$event_slug = substr( $log->log_type, 8 );
+		if ($is_reversal) {
+			$event_slug = substr($log->log_type, 8);
 		}
 
 		// If this is not a log from an event, this doesn't apply.
-		if ( ! $events->is_registered( $event_slug ) ) {
+		if ( ! $events->is_registered($event_slug)) {
 			return;
 		}
 
-		if ( $is_reversal ) {
+		if ($is_reversal) {
 			$log_id = wordpoints_get_points_log_meta(
 				$log_id
 				, 'original_log_id'
@@ -58,31 +58,31 @@ 
 		}
 
 		/** @var WordPoints_Entity_Restrictions $entity_restrictions */
-		$entity_restrictions = wordpoints_entities()->get_sub_app( 'restrictions' );
+		$entity_restrictions = wordpoints_entities()->get_sub_app('restrictions');
 
 		/** @var WordPoints_Hook_ArgI $arg */
-		foreach ( $events->get_sub_app( 'args' )->get_children( $event_slug ) as $slug => $arg ) {
+		foreach ($events->get_sub_app('args')->get_children($event_slug) as $slug => $arg) {
 
-			$value = wordpoints_get_points_log_meta( $log_id, "{$slug}_guid", true );
+			$value = wordpoints_get_points_log_meta($log_id, "{$slug}_guid", true);
 
-			if ( ! $value ) {
-				$value = wordpoints_get_points_log_meta( $log_id, $slug, true );
+			if ( ! $value) {
+				$value = wordpoints_get_points_log_meta($log_id, $slug, true);
 
 				// If we don't find the value it may mean that a new arg has been
 				// registered or something. Just skip over it.
-				if ( ! $value ) {
+				if ( ! $value) {
 					continue;
 				}
 			}
 
 			$entity_slug = $arg->get_entity_slug();
-			$restriction = $entity_restrictions->get( $value, $entity_slug, 'view' );
+			$restriction = $entity_restrictions->get($value, $entity_slug, 'view');
 
-			if ( ! $restriction->applies() ) {
+			if ( ! $restriction->applies()) {
 				continue;
 			}
 
-			$this->restrictions[ $entity_slug ] = $restriction;
+			$this->restrictions[$entity_slug] = $restriction;
 		}
 	}
 
@@ -101,15 +101,15 @@ 
 
 		$descriptions = array();
 
-		foreach ( $this->restrictions as $entity_slug => $restriction ) {
+		foreach ($this->restrictions as $entity_slug => $restriction) {
 
-			$entity = $entities->get( $entity_slug );
+			$entity = $entities->get($entity_slug);
 
-			if ( ! $entity instanceof WordPoints_Entity ) {
+			if ( ! $entity instanceof WordPoints_Entity) {
 				continue;
 			}
 
-			$descriptions[] = sprintf( $string, $entity->get_title() );
+			$descriptions[] = sprintf($string, $entity->get_title());
 		}
 
 		return $descriptions;
@@ -118,10 +118,10 @@ 
 	/**
 	 * @since 2.2.0
 	 */
-	public function user_can( $user_id ) {
+	public function user_can($user_id) {
 
-		foreach ( $this->restrictions as $restriction ) {
-			if ( ! $restriction->user_can( $user_id ) ) {
+		foreach ($this->restrictions as $restriction) {
+			if ( ! $restriction->user_can($user_id)) {
 				return false;
 			}
 		}
@@ -133,7 +133,7 @@ 
 	 * @since 2.2.0
 	 */
 	public function applies() {
-		return ! empty( $this->restrictions );
+		return ! empty($this->restrictions);
 	}
 }
 

src/components/points/classes/logs/viewing/restriction/read/post.php

Spacing +8 added lines, -8 removed lines

@@ -45,13 +45,13 @@ 
 	/**
 	 * @since 2.2.0
 	 */
-	public function __construct( $log ) {
+	public function __construct($log) {
 
 		$this->log = $log;
 
 		$this->post_id = $this->get_post_id();
 
-		if ( ! $this->post_id ) {
+		if ( ! $this->post_id) {
 
 			// We remove any reference to the post from the log when it is deleted,
 			// so it is OK to show it publicly.
@@ -60,10 +60,10 @@ 
 		} else {
 
 			$post_status = get_post_status_object(
-				get_post_status( $this->post_id )
+				get_post_status($this->post_id)
 			);
 
-			if ( $post_status && $post_status->public ) {
+			if ($post_status && $post_status->public) {
 				$this->is_public = true;
 			}
 		}
@@ -73,20 +73,20 @@ 
 	 * @since 2.2.0
 	 */
 	public function get_description() {
-		return __( 'This log entry is only visible to users who can view the post.', 'wordpoints' );
+		return __('This log entry is only visible to users who can view the post.', 'wordpoints');
 	}
 
 	/**
 	 * @since 2.2.0
 	 */
-	public function user_can( $user_id ) {
+	public function user_can($user_id) {
 
-		if ( $this->is_public ) {
+		if ($this->is_public) {
 			return true;
 		}
 
 		// If the post doesn't have a public status, fall back to the caps API.
-		return user_can( $user_id, 'read_post', $this->post_id );
+		return user_can($user_id, 'read_post', $this->post_id);
 	}
 
 	/**

src/components/points/classes/logs/viewing/restrictions.php

Spacing +17 added lines, -17 removed lines

@@ -31,21 +31,21 @@ 
 	 *
 	 * @return WordPoints_Points_Logs_Viewing_RestrictionI Restriction for this log.
 	 */
-	public function get_restriction( $log ) {
+	public function get_restriction($log) {
 
-		if ( $log->blog_id && get_current_blog_id() !== $log->blog_id ) {
-			$switched = switch_to_blog( $log->blog_id );
+		if ($log->blog_id && get_current_blog_id() !== $log->blog_id) {
+			$switched = switch_to_blog($log->blog_id);
 		}
 
 		$restrictions = array();
 
-		foreach ( array( 'all', $log->log_type ) as $slug ) {
+		foreach (array('all', $log->log_type) as $slug) {
 
-			if ( isset( $this->classes[ $slug ] ) ) {
+			if (isset($this->classes[$slug])) {
 
 				$restrictions = array_merge(
 					$restrictions
-					, array_values( $this->get_children( $slug, array( $log ) ) )
+					, array_values($this->get_children($slug, array($log)))
 				);
 			}
 		}
@@ -55,7 +55,7 @@ 
 			, $restrictions
 		);
 
-		if ( isset( $switched ) ) {
+		if (isset($switched)) {
 			restore_current_blog();
 		}
 
@@ -72,18 +72,18 @@ 
 	 *
 	 * @return bool Whether the legacy filters say the user can view the points log.
 	 */
-	public function apply_legacy_filters( $user_id, $log ) {
+	public function apply_legacy_filters($user_id, $log) {
 
-		if ( $log->blog_id && get_current_blog_id() !== $log->blog_id ) {
-			$switched = switch_to_blog( $log->blog_id );
+		if ($log->blog_id && get_current_blog_id() !== $log->blog_id) {
+			$switched = switch_to_blog($log->blog_id);
 		}
 
 		$current_user = wp_get_current_user();
 
 		// Back-compat for WordPoints pre-2.1.0, when the below filter assumed that
 		// the user in question was the current user.
-		if ( $user_id !== $current_user->ID ) {
-			wp_set_current_user( $user_id );
+		if ($user_id !== $current_user->ID) {
+			wp_set_current_user($user_id);
 		}
 
 		/**
@@ -104,15 +104,15 @@ 
 		 */
 		$can_view = apply_filters_deprecated(
 			"wordpoints_user_can_view_points_log-{$log->log_type}"
-			, array( true, $log, $user_id )
+			, array(true, $log, $user_id)
 			, '2.2.0'
 			, false
 			, 'Use the points logs viewing restrictions API instead.'
 		);
 
 		// Restore the current user after the temporary override above.
-		if ( $user_id !== $current_user->ID ) {
-			wp_set_current_user( $current_user->ID );
+		if ($user_id !== $current_user->ID) {
+			wp_set_current_user($current_user->ID);
 		}
 
 		/**
@@ -127,13 +127,13 @@ 
 		 */
 		$can_view = apply_filters_deprecated(
 			'wordpoints_user_can_view_points_log'
-			, array( $can_view, $user_id, $log )
+			, array($can_view, $user_id, $log)
 			, '2.2.0'
 			, false
 			, 'Use the points logs viewing restrictions API instead.'
 		);
 
-		if ( isset( $switched ) ) {
+		if (isset($switched)) {
 			restore_current_blog();
 		}
 

src/components/points/classes/logs/viewing/restrictioni.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 	 *
 	 * @param object $log The object for the points log the restriction is for.
 	 */
-	public function __construct( $log );
+	public function __construct($log);
 
 	/**
 	 * Get a description of this restriction.
@@ -48,7 +48,7 @@ 
 	 *
 	 * @return bool Whether the user can view the points log.
 	 */
-	public function user_can( $user_id );
+	public function user_can($user_id);
 }
 
 // EOF

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 	 * @since 2.2.0
 	 *
 	 * @param object $log The object for the points log the restriction is for.
+	 * @return void
 	 */
 	public function __construct( $log );
 

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

Spacing +54 added lines, -54 removed lines

@@ -40,35 +40,35 @@ 
 	 */
 	protected function before() {
 
-		wp_enqueue_style( 'wordpoints-points-logs' );
+		wp_enqueue_style('wordpoints-points-logs');
 
-		$this->current_time = current_time( 'timestamp', true );
+		$this->current_time = current_time('timestamp', true);
 
 		$this->column_headings = array(
-			'user'        => _x( 'User', 'points logs table heading', 'wordpoints' ),
-			'points'      => _x( 'Points', 'points logs table heading', 'wordpoints' ),
-			'description' => _x( 'Description', 'points logs table heading', 'wordpoints' ),
-			'time'        => _x( 'Time', 'points logs table heading', 'wordpoints' ),
+			'user'        => _x('User', 'points logs table heading', 'wordpoints'),
+			'points'      => _x('Points', 'points logs table heading', 'wordpoints'),
+			'description' => _x('Description', 'points logs table heading', 'wordpoints'),
+			'time'        => _x('Time', 'points logs table heading', 'wordpoints'),
 		);
 
-		$points_type = $this->logs_query->get_arg( 'points_type' );
+		$points_type = $this->logs_query->get_arg('points_type');
 
-		if ( ! empty( $points_type ) ) {
+		if ( ! empty($points_type)) {
 
 			$points_type_name = wordpoints_get_points_type_setting(
 				$points_type
 				, 'name'
 			);
 
-			if ( ! empty( $points_type_name ) ) {
+			if ( ! empty($points_type_name)) {
 				$this->column_headings['points'] = $points_type_name;
 			}
 		}
 
 		$extra_classes = array();
 
-		foreach ( array( 'searchable', 'paginate' ) as $arg ) {
-			if ( $this->args[ $arg ] ) {
+		foreach (array('searchable', 'paginate') as $arg) {
+			if ($this->args[$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 ) ); ?>">
+			<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,8 +118,8 @@ 
 
 		?>
 		<tr>
-			<td colspan="<?php echo ( $this->args['show_users'] ) ? 4 : 3; ?>">
-				<?php esc_html_e( 'No matching logs found.', 'wordpoints' ); ?>
+			<td colspan="<?php echo ($this->args['show_users']) ? 4 : 3; ?>">
+				<?php esc_html_e('No matching logs found.', 'wordpoints'); ?>
 			</td>
 		</tr>
 		<?php
@@ -128,16 +128,16 @@ 
 	/**
 	 * @since 2.2.0
 	 */
-	protected function log( $log ) {
+	protected function log($log) {
 
-		$user = get_userdata( $log->user_id );
+		$user = get_userdata($log->user_id);
 
 		?>
 
-		<tr class="wordpoints-log-id-<?php echo (int) $log->id; ?> <?php echo ( $this->i % 2 ) ? 'odd' : 'even'; ?>">
-			<?php if ( $this->args['show_users'] ) : ?>
+		<tr class="wordpoints-log-id-<?php echo (int) $log->id; ?> <?php echo ($this->i % 2) ? 'odd' : 'even'; ?>">
+			<?php if ($this->args['show_users']) : ?>
 				<td>
-					<?php echo get_avatar( $user->ID, 32 ); ?>
+					<?php echo get_avatar($user->ID, 32); ?>
 					<?php
 
 					/**
@@ -155,33 +155,33 @@ 
 					 */
 					$name = apply_filters(
 						'wordpoints_points_logs_table_username'
-						, sanitize_user_field( 'display_name', $user->display_name, $log->user_id, 'display' )
+						, sanitize_user_field('display_name', $user->display_name, $log->user_id, 'display')
 						, $user
 						, $log
 					);
 
-					echo wp_kses( $name, 'wordpoints_points_logs_username' );
+					echo wp_kses($name, 'wordpoints_points_logs_username');
 
 					?>
 				</td>
 			<?php endif; ?>
-			<td><?php echo wordpoints_format_points( $log->points, $log->points_type, 'logs' ); ?></td>
+			<td><?php echo wordpoints_format_points($log->points, $log->points_type, 'logs'); ?></td>
 			<td>
 				<div class="wordpoints-log-text">
-					<?php echo wp_kses( $log->text, 'wordpoints_points_log' ); ?>
+					<?php echo wp_kses($log->text, 'wordpoints_points_log'); ?>
 				</div>
-				<?php if ( $this->restriction->applies() ) : ?>
+				<?php if ($this->restriction->applies()) : ?>
 					<div class="wordpoints-log-viewing-restrictions">
-						<?php foreach ( (array) $this->restriction->get_description() as $description ) : ?>
+						<?php foreach ((array) $this->restriction->get_description() as $description) : ?>
 							<div class="wordpoints-log-viewing-restriction">
-								<?php echo esc_html( $description ); ?>
+								<?php echo esc_html($description); ?>
 							</div>
 						<?php endforeach; ?>
 					</div>
 				<?php endif; ?>
 			</td>
-			<td title="<?php echo esc_attr( $log->date ); ?> <?php /* translators: coordinated universal time */ esc_attr_e( 'UTC', 'wordpoints' ); ?>">
-				<?php echo esc_html( human_time_diff( strtotime( $log->date ), $this->current_time ) ); ?>
+			<td title="<?php echo esc_attr($log->date); ?> <?php /* translators: coordinated universal time */ esc_attr_e('UTC', 'wordpoints'); ?>">
+				<?php echo esc_html(human_time_diff(strtotime($log->date), $this->current_time)); ?>
 			</td>
 		</tr>
 
@@ -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,8 +243,8 @@ 
 		$page = 1;
 
 		if (
-			isset( $_GET['wordpoints_points_logs_page'] )
-			&& wordpoints_posint( $_GET['wordpoints_points_logs_page'] )
+			isset($_GET['wordpoints_points_logs_page'])
+			&& wordpoints_posint($_GET['wordpoints_points_logs_page'])
 		) {
 			$page = (int) $_GET['wordpoints_points_logs_page'];
 		}
@@ -260,8 +260,8 @@ 
 		$per_page = 25;
 
 		if (
-			isset( $_GET['wordpoints_points_logs_per_page'] )
-			&& wordpoints_posint( $_GET['wordpoints_points_logs_per_page'] )
+			isset($_GET['wordpoints_points_logs_per_page'])
+			&& wordpoints_posint($_GET['wordpoints_points_logs_per_page'])
 		) {
 			$per_page = (int) $_GET['wordpoints_points_logs_per_page'];
 		}
@@ -278,13 +278,13 @@ 
 
 		?>
 
-		<?php if ( ! empty( $search_term ) ) : ?>
+		<?php if ( ! empty($search_term)) : ?>
 			<div class="wordpoints-points-logs-searching">
 				<?php
 
 				echo esc_html(
 					sprintf(
-						__( 'Searching for &#8220;%s&#8221;', 'wordpoints' )
+						__('Searching for &#8220;%s&#8221;', 'wordpoints')
 						, $search_term
 					)
 				);
@@ -294,20 +294,20 @@ 
 		<?php endif; ?>
 
 		<div class="wordpoints-points-logs-search">
-			<form method="POST" action="<?php echo esc_url( remove_query_arg( 'wordpoints_points_logs_page' ) ); ?>">
+			<form method="POST" action="<?php echo esc_url(remove_query_arg('wordpoints_points_logs_page')); ?>">
 				<label class="screen-reader-text" for="wordpoints_points_logs_search">
-					<?php esc_html_e( 'Search Logs:', 'wordpoints' ); ?>
+					<?php esc_html_e('Search Logs:', 'wordpoints'); ?>
 				</label>
 				<input
 					type="text"
 					name="wordpoints_points_logs_search"
 					id="wordpoints_points_logs_search"
-					value="<?php echo esc_attr( $search_term ); ?>"
+					value="<?php echo esc_attr($search_term); ?>"
 				/>
 				<input
 					name=""
 					class="button"
-					value="<?php esc_attr_e( 'Search Logs', 'wordpoints' ); ?>"
+					value="<?php esc_attr_e('Search Logs', 'wordpoints'); ?>"
 					type="submit"
 				/>
 			</form>
@@ -325,9 +325,9 @@ 
 
 		echo paginate_links( // XSS pass WPCS.
 			array(
-				'base'     => add_query_arg( '%_%', '' ),
+				'base'     => add_query_arg('%_%', ''),
 				'format'   => 'wordpoints_points_logs_page=%#%',
-				'total'    => ceil( $this->logs_query->count() / $per_page ),
+				'total'    => ceil($this->logs_query->count() / $per_page),
 				'current'  => $this->get_page_number(),
 				'add_args' => array(
 					'wordpoints_points_logs_per_page' => $per_page,

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

Spacing +21 added lines, -21 removed lines

@@ -109,7 +109,7 @@ 
 
 		$this->slug       = $slug;
 		$this->logs_query = $logs_query;
-		$this->args       = array_merge( $this->defaults, $args );
+		$this->args       = array_merge($this->defaults, $args);
 	}
 
 	/**
@@ -123,7 +123,7 @@ 
 
 		$this->before();
 
-		if ( empty( $this->logs ) ) {
+		if (empty($this->logs)) {
 			$this->no_logs();
 		} else {
 			$this->logs();
@@ -145,28 +145,28 @@ 
 	 */
 	protected function get_logs() {
 
-		if ( $this->args['searchable'] ) {
+		if ($this->args['searchable']) {
 
 			$search_term = $this->get_search_term();
 
-			if ( $search_term ) {
+			if ($search_term) {
 
 				global $wpdb;
 
-				$escaped_search_term = $wpdb->esc_like( $search_term );
+				$escaped_search_term = $wpdb->esc_like($search_term);
 
 				$this->logs_query->set_args(
-					array( 'text' => "%{$escaped_search_term}%" )
+					array('text' => "%{$escaped_search_term}%")
 				);
 			}
 		}
 
-		if ( $this->args['paginate'] ) {
+		if ($this->args['paginate']) {
 
 			$page = $this->get_page_number();
 			$per_page = $this->get_per_page();
 
-			$logs = $this->logs_query->get_page( $page, $per_page );
+			$logs = $this->logs_query->get_page($page, $per_page);
 
 		} else {
 
@@ -187,44 +187,44 @@ 
 	protected function logs() {
 
 		/** @var WordPoints_Points_Logs_Viewing_Restrictions $viewing_restrictions */
-		$viewing_restrictions = wordpoints_component( 'points' )
-			->get_sub_app( 'logs' )
-			->get_sub_app( 'viewing_restrictions' );
+		$viewing_restrictions = wordpoints_component('points')
+			->get_sub_app('logs')
+			->get_sub_app('viewing_restrictions');
 
 		$current_user_id = get_current_user_id();
 
-		if ( is_multisite() ) {
+		if (is_multisite()) {
 			$ms_switched_state = new WordPoints_Multisite_Switched_State();
 			$current_site_id   = $ms_switched_state->backup();
 		}
 
 		$this->i = 0;
 
-		foreach ( $this->logs as $log ) {
+		foreach ($this->logs as $log) {
 
-			if ( isset( $current_site_id ) && $current_site_id !== $log->blog_id ) {
-				switch_to_blog( $log->blog_id );
+			if (isset($current_site_id) && $current_site_id !== $log->blog_id) {
+				switch_to_blog($log->blog_id);
 				$current_site_id = $log->blog_id;
 			}
 
-			$this->restriction = $viewing_restrictions->get_restriction( $log );
+			$this->restriction = $viewing_restrictions->get_restriction($log);
 
-			if ( ! $this->restriction->user_can( $current_user_id ) ) {
+			if ( ! $this->restriction->user_can($current_user_id)) {
 				continue;
 			}
 
 			if (
-				! $viewing_restrictions->apply_legacy_filters( $current_user_id, $log )
+				! $viewing_restrictions->apply_legacy_filters($current_user_id, $log)
 			) {
 				continue;
 			}
 
 			$this->i++;
 
-			$this->log( $log );
+			$this->log($log);
 		}
 
-		if ( isset( $ms_switched_state ) ) {
+		if (isset($ms_switched_state)) {
 			$ms_switched_state->restore();
 		}
 	}
@@ -254,7 +254,7 @@ 
 	 *
 	 * @param object $log The log object.
 	 */
-	abstract protected function log( $log );
+	abstract protected function log($log);
 
 	/**
 	 * Displays content after the logs loop.

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

Spacing +52 added lines, -52 removed lines

@@ -203,24 +203,24 @@ 
 			$this->legacy_id_base
 		);
 
-		if ( ! $this->legacy_handler ) {
+		if ( ! $this->legacy_handler) {
 			return;
 		}
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$this->hook_mode = 'network';
 		} else {
 			$this->hook_mode = 'standard';
 		}
 
 		$imported_option = 'wordpoints_imported_points_hooks';
-		$this->imported_hooks = $this->get_array_option( $imported_option );
+		$this->imported_hooks = $this->get_array_option($imported_option);
 
 		$this->points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
 
-		$this->reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+		$this->reaction_store = wordpoints_hooks()->get_reaction_store('points');
 
-		if ( ! $this->reaction_store ) {
+		if ( ! $this->reaction_store) {
 			return;
 		}
 
@@ -228,24 +228,24 @@ 
 			$this->hook_mode
 		);
 
-		foreach ( $this->legacy_instances as $number => $legacy_instance ) {
-			$this->import_instance( $legacy_instance, $number );
+		foreach ($this->legacy_instances as $number => $legacy_instance) {
+			$this->import_instance($legacy_instance, $number);
 		}
 
 		// This means that all of the instances were imported successfully.
-		if ( empty( $this->legacy_instances ) ) {
+		if (empty($this->legacy_instances)) {
 
 			$option = 'wordpoints_legacy_points_hooks_disabled';
 
-			$disabled = $this->get_array_option( $option );
-			$disabled[ $this->legacy_id_base ] = true;
+			$disabled = $this->get_array_option($option);
+			$disabled[$this->legacy_id_base] = true;
 
-			$this->update_option( $option, $disabled );
+			$this->update_option($option, $disabled);
 		}
 
-		WordPoints_Points_Hooks::save_points_types_hooks( $this->points_types_hooks );
+		WordPoints_Points_Hooks::save_points_types_hooks($this->points_types_hooks);
 
-		$this->update_option( $imported_option, $this->imported_hooks );
+		$this->update_option($imported_option, $this->imported_hooks);
 	}
 
 	/**
@@ -258,15 +258,15 @@ 
 	 *
 	 * @return bool Whether the instance was successfully converted to a reaction.
 	 */
-	protected function import_instance( $legacy_instance, $number ) {
+	protected function import_instance($legacy_instance, $number) {
 
 		$this->legacy_instance = $legacy_instance;
 
-		if ( ! $this->should_import_instance() ) {
+		if ( ! $this->should_import_instance()) {
 			return false;
 		}
 
-		$this->legacy_handler->set_number( $number );
+		$this->legacy_handler->set_number($number);
 
 		$points_type = $this->legacy_handler->points_type(
 			'network' === $this->hook_mode ? "network_{$number}" : $number
@@ -285,7 +285,7 @@ 
 
 		$order = array_search(
 			$this->legacy_handler->get_id()
-			, $this->points_types_hooks[ $points_type ]
+			, $this->points_types_hooks[$points_type]
 			, true
 		);
 
@@ -294,21 +294,21 @@ 
 			$order
 		);
 
-		if ( ! isset( $reaction_settings['log_text'] ) ) {
+		if ( ! isset($reaction_settings['log_text'])) {
 			$reaction_settings['log_text'] = $this->get_log_text_for_instance(
 				$points_type
 			);
 		}
 
-		if ( ! $this->create_reaction( $reaction_settings, $order ) ) {
+		if ( ! $this->create_reaction($reaction_settings, $order)) {
 			return false;
 		}
 
-		$this->legacy_handler->delete_callback( $number );
+		$this->legacy_handler->delete_callback($number);
 
 		unset(
-			$this->points_types_hooks[ $points_type ][ $order ]
-			, $this->legacy_instances[ $number ]
+			$this->points_types_hooks[$points_type][$order]
+			, $this->legacy_instances[$number]
 		);
 
 		return true;
@@ -324,16 +324,16 @@ 
 	protected function should_import_instance() {
 
 		// We ignore certain settings...
-		$ignored = array( '_description' => true );
+		$ignored = array('_description' => true);
 
 		// But if there are any other unexpected ones, we really can't safely import.
-		if ( array_diff_key( $this->legacy_instance, $this->expected_settings, $ignored ) ) {
+		if (array_diff_key($this->legacy_instance, $this->expected_settings, $ignored)) {
 			return false;
 		}
 
 		if (
 			$this->skip_non_reversing_hooks
-			&& isset( $this->legacy_instance['auto_reverse'] )
+			&& isset($this->legacy_instance['auto_reverse'])
 			&& ! $this->legacy_instance['auto_reverse']
 		) {
 			return false;
@@ -352,22 +352,22 @@ 
 	 *
 	 * @return string The log text derived from this instance.
 	 */
-	protected function get_log_text_for_instance( $points_type, $meta = array() ) {
+	protected function get_log_text_for_instance($points_type, $meta = array()) {
 
-		if ( ! method_exists( $this->legacy_handler, 'logs' ) ) {
-			return $this->legacy_handler->get_option( 'description' );
+		if ( ! method_exists($this->legacy_handler, 'logs')) {
+			return $this->legacy_handler->get_option('description');
 		}
 
 		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'];
 		}
 
 		if (
-			! isset( $meta['period'] )
-			&& isset( $this->legacy_instance['period'] )
+			! isset($meta['period'])
+			&& isset($this->legacy_instance['period'])
 		) {
 			$meta['period'] = $this->legacy_instance['period'];
 		}
@@ -394,15 +394,15 @@ 
 	 *
 	 * @return array The modified settings.
 	 */
-	protected function filter_reaction_settings( $settings, $order ) {
+	protected function filter_reaction_settings($settings, $order) {
 
-		if ( 'wordpoints_periodic_points_hook' === $this->legacy_id_base ) {
+		if ('wordpoints_periodic_points_hook' === $this->legacy_id_base) {
 
 			$settings['points_legacy_periods'] = array(
 				'fire' => array(
 					array(
 						'length' => $this->legacy_instance['period'],
-						'args' => array( array( 'current:user' ) ),
+						'args' => array(array('current:user')),
 					),
 				),
 			);
@@ -417,23 +417,23 @@ 
 		}
 
 		if (
-			isset( $this->legacy_instance['auto_reverse'] )
+			isset($this->legacy_instance['auto_reverse'])
 			&& ! $this->legacy_instance['auto_reverse']
 		) {
 			$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'];
 
-			if ( 'ALL' === $post_type ) {
+			if ('ALL' === $post_type) {
 
-				$post_type_slugs = get_post_types( array( 'public' => true ) );
+				$post_type_slugs = get_post_types(array('public' => true));
 
-				$post_type = array_pop( $post_type_slugs );
+				$post_type = array_pop($post_type_slugs);
 
-				foreach ( $post_type_slugs as $post_type_slug ) {
+				foreach ($post_type_slugs as $post_type_slug) {
 
 					$this->create_reaction(
 						$this->format_settings_for_post_type(
@@ -464,14 +464,14 @@ 
 	 *
 	 * @return array The settings modified for this particular post type.
 	 */
-	protected function format_settings_for_post_type( $post_type, $settings ) {
+	protected function format_settings_for_post_type($post_type, $settings) {
 
 		if (
 			'post_publish\post' === $settings['event']
 			|| 'points_legacy_post_publish\post' === $settings['event']
 		) {
 
-			if ( 'attachment' === $post_type ) {
+			if ('attachment' === $post_type) {
 
 				$settings['event'] = 'media_upload';
 
@@ -497,7 +497,7 @@ 
 
 		$settings['log_text'] = $this->get_log_text_for_instance(
 			$settings['points_type']
-			, array( 'post_type' => $post_type )
+			, array('post_type' => $post_type)
 		);
 
 		return $settings;
@@ -513,11 +513,11 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI|false The reaction, or false.
 	 */
-	protected function create_reaction( $settings, $order ) {
+	protected function create_reaction($settings, $order) {
 
-		$reaction = $this->reaction_store->create_reaction( $settings );
+		$reaction = $this->reaction_store->create_reaction($settings);
 
-		if ( ! $reaction instanceof WordPoints_Hook_ReactionI ) {
+		if ( ! $reaction instanceof WordPoints_Hook_ReactionI) {
 			return false;
 		}
 
@@ -542,12 +542,12 @@ 
 	 *
 	 * @return bool Whether the option was updated successfully.
 	 */
-	protected function update_option( $option, $value ) {
+	protected function update_option($option, $value) {
 
-		if ( 'network' === $this->hook_mode ) {
-			return update_site_option( $option, $value );
+		if ('network' === $this->hook_mode) {
+			return update_site_option($option, $value);
 		} else {
-			return update_option( $option, $value );
+			return update_option($option, $value);
 		}
 	}
 
@@ -560,7 +560,7 @@ 
 	 *
 	 * @return array The option value.
 	 */
-	protected function get_array_option( $option ) {
+	protected function get_array_option($option) {
 
 		return wordpoints_get_maybe_network_array_option(
 			$option

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

Spacing +21 added lines, -21 removed lines

@@ -23,16 +23,16 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
 		// Normally we wouldn't let the hit occur if there is nothing to reverse.
-		if ( ! parent::should_hit( $fire ) ) {
+		if ( ! parent::should_hit($fire)) {
 
 			// But in this case we want to check the points logs as well.
-			$logs = $this->get_points_logs_to_be_reversed( $fire );
+			$logs = $this->get_points_logs_to_be_reversed($fire);
 
 			// If there are logs to reverse, we'll let the hit happen.
-			return count( $logs ) > 0;
+			return count($logs) > 0;
 		}
 
 		return true;
@@ -41,16 +41,16 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function after_miss( WordPoints_Hook_Fire $fire ) {
+	public function after_miss(WordPoints_Hook_Fire $fire) {
 
-		parent::after_miss( $fire );
+		parent::after_miss($fire);
 
-		if ( ! $this->get_settings_from_fire( $fire ) ) {
+		if ( ! $this->get_settings_from_fire($fire)) {
 			return;
 		}
 
-		foreach ( $this->get_points_logs_to_be_reversed( $fire ) as $log ) {
-			wordpoints_add_points_log_meta( $log->id, 'auto_reversed', 0 );
+		foreach ($this->get_points_logs_to_be_reversed($fire) as $log) {
+			wordpoints_add_points_log_meta($log->id, 'auto_reversed', 0);
 		}
 	}
 
@@ -63,10 +63,10 @@ 
 	 *
 	 * @return array The points logs to be reversed.
 	 */
-	protected function get_points_logs_to_be_reversed( WordPoints_Hook_Fire $fire ) {
+	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(
@@ -78,19 +78,19 @@ 
 			),
 		);
 
-		$meta_key = $fire->reaction->get_meta( 'legacy_meta_key' );
+		$meta_key = $fire->reaction->get_meta('legacy_meta_key');
 
-		if ( $meta_key ) {
+		if ($meta_key) {
 
 			$entities = $fire->event_args->get_signature_args();
 
-			if ( ! $entities ) {
-				$fire->data[ $this->slug ]['points_logs'] = array();
+			if ( ! $entities) {
+				$fire->data[$this->slug]['points_logs'] = array();
 				return array();
 			}
 
 			// Legacy hooks only ever related to a single entity.
-			$entity = reset( $entities );
+			$entity = reset($entities);
 
 			$meta_queries[] = array(
 				'key'   => $meta_key,
@@ -98,9 +98,9 @@ 
 			);
 		}
 
-		$log_type = $fire->reaction->get_meta( 'legacy_log_type' );
+		$log_type = $fire->reaction->get_meta('legacy_log_type');
 
-		if ( ! $log_type ) {
+		if ( ! $log_type) {
 			$log_type = $fire->reaction->get_event_slug();
 		}
 
@@ -113,11 +113,11 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			$logs = array();
 		}
 
-		$fire->data[ $this->slug ]['points_logs'] = $logs;
+		$fire->data[$this->slug]['points_logs'] = $logs;
 
 		return $logs;
 	}