WordPoints / wordpoints

src/classes/hook/reaction/store.php

Doc Comments +1 added lines, -1 removed lines

@@ -190,7 +190,7 @@
 	 *
 	 * @param string $event_slug The slug of the event this reaction is for.
 	 *
-	 * @return int|false The reaction ID, or false if not created.
+	 * @return integer The reaction ID, or false if not created.
 	 */
 	abstract protected function _create_reaction( $event_slug );
 }

Spacing +27 added lines, -27 removed lines

@@ -63,7 +63,7 @@ 
 	 * @param string $slug      The slug of this store.
 	 * @param string $mode_slug The slug of the mode this store relates to.
 	 */
-	public function __construct( $slug, $mode_slug ) {
+	public function __construct($slug, $mode_slug) {
 		$this->slug      = $slug;
 		$this->mode_slug = $mode_slug;
 	}
@@ -86,33 +86,33 @@ 
 	 * @since 2.1.0
 	 */
 	public function get_context_id() {
-		return wordpoints_entities_get_current_context_id( $this->context );
+		return wordpoints_entities_get_current_context_id($this->context);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function get_reaction( $id ) {
+	public function get_reaction($id) {
 
-		if ( ! $this->reaction_exists( $id ) ) {
+		if ( ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		return new $this->reaction_class( $id, $this );
+		return new $this->reaction_class($id, $this);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function create_reaction( array $settings ) {
-		return $this->create_or_update_reaction( $settings );
+	public function create_reaction(array $settings) {
+		return $this->create_or_update_reaction($settings);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function update_reaction( $id, array $settings ) {
-		return $this->create_or_update_reaction( $settings, $id );
+	public function update_reaction($id, array $settings) {
+		return $this->create_or_update_reaction($settings, $id);
 	}
 
 	/**
@@ -127,45 +127,45 @@ 
 	 *         The reaction object if created/updated successfully. False or a
 	 *         validator instance if not.
 	 */
-	protected function create_or_update_reaction( array $settings, $id = null ) {
+	protected function create_or_update_reaction(array $settings, $id = null) {
 
-		$is_new = ! isset( $id );
+		$is_new = ! isset($id);
 
-		if ( ! $is_new && ! $this->reaction_exists( $id ) ) {
+		if ( ! $is_new && ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		$validator = new WordPoints_Hook_Reaction_Validator( $settings );
+		$validator = new WordPoints_Hook_Reaction_Validator($settings);
 		$settings = $validator->validate();
 
-		if ( $validator->had_errors() ) {
+		if ($validator->had_errors()) {
 			return $validator;
 		}
 
-		if ( $is_new ) {
+		if ($is_new) {
 
-			$id = $this->_create_reaction( $settings['event'] );
+			$id = $this->_create_reaction($settings['event']);
 
-			if ( ! $id ) {
+			if ( ! $id) {
 				return false;
 			}
 		}
 
-		$reaction = $this->get_reaction( $id );
+		$reaction = $this->get_reaction($id);
 
-		$reaction->update_event_slug( $settings['event'] );
+		$reaction->update_event_slug($settings['event']);
 
-		unset( $settings['event'] );
+		unset($settings['event']);
 
-		$reaction->update_meta( 'reactor', $settings['reactor'] );
+		$reaction->update_meta('reactor', $settings['reactor']);
 
 		/** @var WordPoints_Hook_ReactorI $reactor */
-		$reactor = wordpoints_hooks()->get_sub_app( 'reactors' )->get( $settings['reactor'] );
-		$reactor->update_settings( $reaction, $settings );
+		$reactor = wordpoints_hooks()->get_sub_app('reactors')->get($settings['reactor']);
+		$reactor->update_settings($reaction, $settings);
 
 		/** @var WordPoints_Hook_ExtensionI $extension */
-		foreach ( wordpoints_hooks()->get_sub_app( 'extensions' )->get_all() as $extension ) {
-			$extension->update_settings( $reaction, $settings );
+		foreach (wordpoints_hooks()->get_sub_app('extensions')->get_all() as $extension) {
+			$extension->update_settings($reaction, $settings);
 		}
 
 		/**
@@ -175,7 +175,7 @@ 
 		 * @param array                     $settings The new settings for the reaction.
 		 * @param bool                      $is_new   Whether the reaction was just now created.
 		 */
-		do_action( 'wordpoints_hook_reaction_save', $reaction, $settings, $is_new );
+		do_action('wordpoints_hook_reaction_save', $reaction, $settings, $is_new);
 
 		return $reaction;
 	}
@@ -192,7 +192,7 @@ 
 	 *
 	 * @return int|false The reaction ID, or false if not created.
 	 */
-	abstract protected function _create_reaction( $event_slug );
+	abstract protected function _create_reaction($event_slug);
 }
 
 // EOF

src/components/points/includes/class-un-installer.php

Doc Comments +1 added lines

@@ -1073,6 +1073,7 @@
 	 *
 	 * @since 2.1.4
 	 *
+	 * @param string[] $post_types
 	 * @return array The slugs of the events.
 	 */
 	protected function _2_1_4_get_reversal_log_types( $post_types ) {

Spacing +181 added lines, -181 removed lines

@@ -27,16 +27,16 @@ 
 	 * @since 1.8.0
 	 */
 	protected $updates = array(
-		'1.2.0'  => array( 'single' => true, /*     -     */ 'network' => true ),
-		'1.4.0'  => array( 'single' => true, 'site' => true, 'network' => true ),
+		'1.2.0'  => array('single' => true, /*     -     */ 'network' => true),
+		'1.4.0'  => array('single' => true, 'site' => true, 'network' => true),
 		'1.5.0'  => array( /*      -      */ 'site' => true  /*      -      */ ),
-		'1.5.1'  => array( 'single' => true, /*     -     */ 'network' => true ),
+		'1.5.1'  => array('single' => true, /*     -     */ 'network' => true),
 		'1.8.0'  => array( /*      -      */ 'site' => true  /*      -      */ ),
-		'1.9.0'  => array( 'single' => true, 'site' => true, 'network' => true ),
-		'1.10.0' => array( 'single' => true, /*     -     */ 'network' => true ),
-		'2.0.0'  => array( 'single' => true, /*     -     */ 'network' => true ),
-		'2.1.0'  => array( 'single' => true, /*     -     */ 'network' => true ),
-		'2.1.4'  => array( 'single' => true, 'site' => true, /*      -      */ ),
+		'1.9.0'  => array('single' => true, 'site' => true, 'network' => true),
+		'1.10.0' => array('single' => true, /*     -     */ 'network' => true),
+		'2.0.0'  => array('single' => true, /*     -     */ 'network' => true),
+		'2.1.0'  => array('single' => true, /*     -     */ 'network' => true),
+		'2.1.4'  => array('single' => true, 'site' => true, /*      -      */),
 	);
 
 	/**
@@ -143,26 +143,26 @@ 
 
 		parent::before_update();
 
-		if ( 1 === version_compare( '1.4.0', $this->updating_from ) ) {
-			add_filter( 'wordpoints_points_hook_update_callback', array( $this, '_1_4_0_clean_hook_settings' ), 10, 4 );
+		if (1 === version_compare('1.4.0', $this->updating_from)) {
+			add_filter('wordpoints_points_hook_update_callback', array($this, '_1_4_0_clean_hook_settings'), 10, 4);
 		}
 
-		if ( 1 === version_compare( '1.5.0', $this->updating_from ) ) {
+		if (1 === version_compare('1.5.0', $this->updating_from)) {
 
-			if ( ! $this->network_wide ) {
-				unset( $this->updates['1_5_0'] );
+			if ( ! $this->network_wide) {
+				unset($this->updates['1_5_0']);
 			}
 		}
 
-		if ( $this->network_wide ) {
-			unset( $this->updates['1_8_0'] );
+		if ($this->network_wide) {
+			unset($this->updates['1_8_0']);
 		}
 
-		if ( 1 === version_compare( '1.9.0', $this->updating_from ) ) {
+		if (1 === version_compare('1.9.0', $this->updating_from)) {
 
 			// If we're updating to 1.4.0, we initialize the hooks early, because
 			// we use them during the update.
-			remove_action( 'wordpoints_modules_loaded', array( 'WordPoints_Points_Hooks', 'initialize_hooks' ) );
+			remove_action('wordpoints_modules_loaded', array('WordPoints_Points_Hooks', 'initialize_hooks'));
 
 			WordPoints_Points_Hooks::register(
 				'WordPoints_Comment_Removed_Points_Hook'
@@ -177,7 +177,7 @@ 
 			// Default to network mode off during the tests, but save the current
 			// mode so we can restore it afterward.
 			$this->points_hooks_network_mode = WordPoints_Points_Hooks::get_network_mode();
-			WordPoints_Points_Hooks::set_network_mode( false );
+			WordPoints_Points_Hooks::set_network_mode(false);
 		}
 	}
 
@@ -186,11 +186,11 @@ 
 	 */
 	protected function after_update() {
 
-		if ( isset( $this->points_hooks_network_mode ) ) {
+		if (isset($this->points_hooks_network_mode)) {
 
-			WordPoints_Points_Hooks::set_network_mode( $this->points_hooks_network_mode );
+			WordPoints_Points_Hooks::set_network_mode($this->points_hooks_network_mode);
 
-			remove_filter( 'wordpoints_points_hook_update_callback', array( $this, '_1_4_0_clean_hook_settings' ), 10 );
+			remove_filter('wordpoints_points_hook_update_callback', array($this, '_1_4_0_clean_hook_settings'), 10);
 		}
 	}
 
@@ -203,11 +203,11 @@ 
 		 * Regenerate the custom caps every time on multisite, because they depend on
 		 * network activation status.
 		 */
-		if ( 'site' === $this->context ) {
-			wordpoints_remove_custom_caps( $this->custom_caps_keys );
+		if ('site' === $this->context) {
+			wordpoints_remove_custom_caps($this->custom_caps_keys);
 		}
 
-		wordpoints_add_custom_caps( $this->custom_caps );
+		wordpoints_add_custom_caps($this->custom_caps);
 	}
 
 	/**
@@ -237,7 +237,7 @@ 
 
 		parent::install_single();
 
-		add_option( 'wordpoints_default_points_type', '' );
+		add_option('wordpoints_default_points_type', '');
 
 		$this->disable_legacy_hooks();
 	}
@@ -317,18 +317,18 @@ 
 			"
 		); // WPCS: cache pass.
 
-		if ( $log_ids && is_array( $log_ids ) ) {
+		if ($log_ids && is_array($log_ids)) {
 
 			$wpdb->query( // WPCS: unprepared SQL OK
 				"
 					DELETE
 					FROM {$wpdb->wordpoints_points_logs}
-					WHERE `id` IN (" . implode( ',', array_map( 'absint', $log_ids ) ) . ')
+					WHERE `id` IN (" . implode(',', array_map('absint', $log_ids)) . ')
 				'
 			); // WPCS: cache pass (points logs weren't cached until 1.5.0).
 
-			foreach ( $log_ids as $log_id ) {
-				wordpoints_points_log_delete_all_metadata( $log_id );
+			foreach ($log_ids as $log_id) {
+				wordpoints_points_log_delete_all_metadata($log_id);
 			}
 		}
 	}
@@ -353,11 +353,11 @@ 
 			"
 		); // WPCS: cache pass.
 
-		$hook = WordPoints_Points_Hooks::get_handler_by_id_base( 'wordpoints_post_points_hook' );
+		$hook = WordPoints_Points_Hooks::get_handler_by_id_base('wordpoints_post_points_hook');
 
-		if ( $post_ids && is_array( $post_ids ) && $hook ) {
-			foreach ( $post_ids as $post_id ) {
-				$hook->clean_logs_on_post_deletion( $post_id );
+		if ($post_ids && is_array($post_ids) && $hook) {
+			foreach ($post_ids as $post_id) {
+				$hook->clean_logs_on_post_deletion($post_id);
 			}
 		}
 	}
@@ -382,11 +382,11 @@ 
 			"
 		); // WPCS: cache pass.
 
-		$hook = WordPoints_Points_Hooks::get_handler_by_id_base( 'wordpoints_comment_points_hook' );
+		$hook = WordPoints_Points_Hooks::get_handler_by_id_base('wordpoints_comment_points_hook');
 
-		if ( $comment_ids && is_array( $comment_ids ) && $hook ) {
-			foreach ( $comment_ids as $comment_id ) {
-				$hook->clean_logs_on_comment_deletion( $comment_id );
+		if ($comment_ids && is_array($comment_ids) && $hook) {
+			foreach ($comment_ids as $comment_id) {
+				$hook->clean_logs_on_comment_deletion($comment_id);
 			}
 		}
 	}
@@ -398,14 +398,14 @@ 
 	 */
 	protected function update_network_to_1_4_0() {
 
-		if ( $this->network_wide ) {
+		if ($this->network_wide) {
 
 			// Split the network-wide points hooks.
 			$network_mode = WordPoints_Points_Hooks::get_network_mode();
-			WordPoints_Points_Hooks::set_network_mode( true );
+			WordPoints_Points_Hooks::set_network_mode(true);
 			$this->_1_4_0_split_post_hooks();
 			$this->_1_4_0_split_comment_hooks();
-			WordPoints_Points_Hooks::set_network_mode( $network_mode );
+			WordPoints_Points_Hooks::set_network_mode($network_mode);
 		}
 	}
 
@@ -471,9 +471,9 @@ 
 	 * @param string $key       The settings key for the hook that holds the points.
 	 * @param string $split_key The settings key for points that is being split.
 	 */
-	protected function _1_4_0_split_points_hooks( $hook, $new_hook, $key, $split_key ) {
+	protected function _1_4_0_split_points_hooks($hook, $new_hook, $key, $split_key) {
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$hook_type = 'network';
 			$network_ = 'network_';
 		} else {
@@ -481,58 +481,58 @@ 
 			$network_ = '';
 		}
 
-		$new_hook = WordPoints_Points_Hooks::get_handler_by_id_base( $new_hook );
-		$hook = WordPoints_Points_Hooks::get_handler_by_id_base( $hook );
+		$new_hook = WordPoints_Points_Hooks::get_handler_by_id_base($new_hook);
+		$hook = WordPoints_Points_Hooks::get_handler_by_id_base($hook);
 
 		$points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
-		$instances = $hook->get_instances( $hook_type );
+		$instances = $hook->get_instances($hook_type);
 
 		// Loop through all of the post hook instances.
-		foreach ( $instances as $number => $settings ) {
+		foreach ($instances as $number => $settings) {
 
 			// Don't split the hook if it is just a placeholder, or it's already split.
-			if ( 0 === (int) $number || ! isset( $settings[ $key ], $settings[ $split_key ] ) ) {
+			if (0 === (int) $number || ! isset($settings[$key], $settings[$split_key])) {
 				continue;
 			}
 
-			if ( ! isset( $settings['post_type'] ) ) {
+			if ( ! isset($settings['post_type'])) {
 				$settings['post_type'] = 'ALL';
 			}
 
 			// If the trash points are set, create a post delete points hook instead.
-			if ( isset( $settings[ $split_key ] ) && wordpoints_posint( $settings[ $split_key ] ) ) {
+			if (isset($settings[$split_key]) && wordpoints_posint($settings[$split_key])) {
 
 				$new_hook->update_callback(
 					array(
-						'points'    => $settings[ $split_key ],
+						'points'    => $settings[$split_key],
 						'post_type' => $settings['post_type'],
 					)
 					, $new_hook->next_hook_id_number()
 				);
 
 				// Make sure the correct points type is retrieved for network hooks.
-				$points_type = $hook->points_type( $network_ . $number );
+				$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 ] ) ) {
+			if (isset($settings[$key]) && wordpoints_posint($settings[$key])) {
 
-				$settings['points'] = $settings[ $key ];
+				$settings['points'] = $settings[$key];
 
-				$hook->update_callback( $settings, $number );
+				$hook->update_callback($settings, $number);
 
 			} else {
 
 				// If not, delete this instance.
-				$hook->delete_callback( $hook->get_id( $number ) );
+				$hook->delete_callback($hook->get_id($number));
 			}
 
 		} // End foreach ( $instances ).
 
-		WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
+		WordPoints_Points_Hooks::save_points_types_hooks($points_types_hooks);
 	}
 
 	/**
@@ -551,12 +551,12 @@ 
 	 *
 	 * @return array The filtered instance settings.
 	 */
-	public function _1_4_0_clean_hook_settings( $instance, $new_instance, $old_instance, $hook ) {
+	public function _1_4_0_clean_hook_settings($instance, $new_instance, $old_instance, $hook) {
 
-		if ( $hook instanceof WordPoints_Post_Points_Hook ) {
-			unset( $instance['trash'], $instance['publish'] );
-		} elseif ( $hook instanceof WordPoints_Comment_Points_Hook ) {
-			unset( $instance['approve'], $instance['disapprove'] );
+		if ($hook instanceof WordPoints_Post_Points_Hook) {
+			unset($instance['trash'], $instance['publish']);
+		} elseif ($hook instanceof WordPoints_Comment_Points_Hook) {
+			unset($instance['approve'], $instance['disapprove']);
 		}
 
 		return $instance;
@@ -585,11 +585,11 @@ 
 			"
 		); // WPCS: cache pass.
 
-		$hook = WordPoints_Points_Hooks::get_handler_by_id_base( 'wordpoints_comment_points_hook' );
+		$hook = WordPoints_Points_Hooks::get_handler_by_id_base('wordpoints_comment_points_hook');
 
-		if ( $post_ids && is_array( $post_ids ) && $hook ) {
-			foreach ( $post_ids as $post_id ) {
-				$hook->clean_logs_on_post_deletion( $post_id );
+		if ($post_ids && is_array($post_ids) && $hook) {
+			foreach ($post_ids as $post_id) {
+				$hook->clean_logs_on_post_deletion($post_id);
 			}
 		}
 	}
@@ -604,7 +604,7 @@ 
 	 */
 	protected function update_site_to_1_5_0() {
 
-		wordpoints_add_custom_caps( $this->custom_caps );
+		wordpoints_add_custom_caps($this->custom_caps);
 	}
 
 	/**
@@ -616,18 +616,18 @@ 
 
 		global $wpdb;
 
-		if ( empty( $wpdb->charset ) ) {
+		if (empty($wpdb->charset)) {
 			return;
 		}
 
 		$charset_collate = " CHARACTER SET {$wpdb->charset}";
 
-		if ( ! empty( $wpdb->collate ) ) {
+		if ( ! empty($wpdb->collate)) {
 			$charset_collate .= " COLLATE {$wpdb->collate}";
 		}
 
-		$wpdb->query( "ALTER TABLE {$wpdb->wordpoints_points_logs} CONVERT TO {$charset_collate}" ); // WPCS: unprepared SQL, cache pass.
-		$wpdb->query( "ALTER TABLE {$wpdb->wordpoints_points_log_meta} CONVERT TO {$charset_collate}" ); // WPCS: unprepared SQL, cache pass.
+		$wpdb->query("ALTER TABLE {$wpdb->wordpoints_points_logs} CONVERT TO {$charset_collate}"); // WPCS: unprepared SQL, cache pass.
+		$wpdb->query("ALTER TABLE {$wpdb->wordpoints_points_log_meta} CONVERT TO {$charset_collate}"); // WPCS: unprepared SQL, cache pass.
 	}
 
 	/**
@@ -655,14 +655,14 @@ 
 	 */
 	protected function update_network_to_1_9_0() {
 
-		if ( $this->network_wide ) {
+		if ($this->network_wide) {
 
 			// Combine the network-wide points hooks.
 			$network_mode = WordPoints_Points_Hooks::get_network_mode();
-			WordPoints_Points_Hooks::set_network_mode( true );
-			$this->_1_9_0_combine_hooks( 'comment', 'comment_removed' );
-			$this->_1_9_0_combine_hooks( 'post', 'post_delete' );
-			WordPoints_Points_Hooks::set_network_mode( $network_mode );
+			WordPoints_Points_Hooks::set_network_mode(true);
+			$this->_1_9_0_combine_hooks('comment', 'comment_removed');
+			$this->_1_9_0_combine_hooks('post', 'post_delete');
+			WordPoints_Points_Hooks::set_network_mode($network_mode);
 		}
 	}
 
@@ -672,8 +672,8 @@ 
 	 * @since 1.9.0
 	 */
 	protected function update_site_to_1_9_0() {
-		$this->_1_9_0_combine_hooks( 'comment', 'comment_removed' );
-		$this->_1_9_0_combine_hooks( 'post', 'post_delete' );
+		$this->_1_9_0_combine_hooks('comment', 'comment_removed');
+		$this->_1_9_0_combine_hooks('post', 'post_delete');
 	}
 
 	/**
@@ -682,8 +682,8 @@ 
 	 * @since 1.9.0
 	 */
 	protected function update_single_to_1_9_0() {
-		$this->_1_9_0_combine_hooks( 'comment', 'comment_removed' );
-		$this->_1_9_0_combine_hooks( 'post', 'post_delete' );
+		$this->_1_9_0_combine_hooks('comment', 'comment_removed');
+		$this->_1_9_0_combine_hooks('post', 'post_delete');
 	}
 
 	/**
@@ -694,7 +694,7 @@ 
 	 * @param string $type         The primary hook type that awards the points.
 	 * @param string $reverse_type The counterpart hook type that reverses points.
 	 */
-	protected function _1_9_0_combine_hooks( $type, $reverse_type ) {
+	protected function _1_9_0_combine_hooks($type, $reverse_type) {
 
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
 			"wordpoints_{$type}_points_hook"
@@ -703,7 +703,7 @@ 
 			"wordpoints_{$reverse_type}_points_hook"
 		);
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$hook_type = 'network';
 			$network_ = 'network_';
 		} else {
@@ -711,11 +711,11 @@ 
 			$network_ = '';
 		}
 
-		$hook_instances = $hook->get_instances( $hook_type );
-		$hook_reverse_instances = $reverse_hook->get_instances( $hook_type );
+		$hook_instances = $hook->get_instances($hook_type);
+		$hook_reverse_instances = $reverse_hook->get_instances($hook_type);
 
-		$default_points = ( 'post' === $hook_type ) ? 20 : 10;
-		$defaults = array( 'points' => $default_points, 'post_type' => 'ALL' );
+		$default_points = ('post' === $hook_type) ? 20 : 10;
+		$defaults = array('points' => $default_points, 'post_type' => 'ALL');
 
 		// Get the hooks into an array that is indexed by post type and the
 		// number of points. This allows us to easily check for any counterparts when
@@ -723,53 +723,53 @@ 
 		// is doing something crazy like multiple hooks for the same post type.
 		$hook_instances_indexed = array();
 
-		foreach ( $hook_instances as $number => $instance ) {
+		foreach ($hook_instances as $number => $instance) {
 
-			$instance = array_merge( $defaults, $instance );
+			$instance = array_merge($defaults, $instance);
 
 			$hook_instances_indexed
-				[ $hook->points_type( $network_ . $number ) ]
-				[ $instance['post_type'] ]
-				[ $instance['points'] ]
+				[$hook->points_type($network_ . $number)]
+				[$instance['post_type']]
+				[$instance['points']]
 				[] = $number;
 		}
 
-		foreach ( $hook_reverse_instances as $number => $instance ) {
+		foreach ($hook_reverse_instances as $number => $instance) {
 
-			$instance = array_merge( $defaults, $instance );
+			$instance = array_merge($defaults, $instance);
 
-			$points_type = $reverse_hook->points_type( $network_ . $number );
+			$points_type = $reverse_hook->points_type($network_ . $number);
 
 			// We use empty() instead of isset() because array_pop() below may leave
 			// us with an empty array as the value.
-			if ( empty( $hook_instances_indexed[ $points_type ][ $instance['post_type'] ][ $instance['points'] ] ) ) {
+			if (empty($hook_instances_indexed[$points_type][$instance['post_type']][$instance['points']])) {
 				continue;
 			}
 
 			$comment_instance_number = array_pop(
-				$hook_instances_indexed[ $points_type ][ $instance['post_type'] ][ $instance['points'] ]
+				$hook_instances_indexed[$points_type][$instance['post_type']][$instance['points']]
 			);
 
 			// We need to unset this instance from the list of hook instances. It
 			// is expected for it to be automatically reversed, and that is the
 			// default setting. If we don't unset it here it will get auto-reversal
 			// turned off below, which isn't what we want.
-			unset( $hook_instances[ $comment_instance_number ] );
+			unset($hook_instances[$comment_instance_number]);
 
 			// Now we can just delete this reverse hook instance.
 			$reverse_hook->delete_callback(
-				$reverse_hook->get_id( $number )
+				$reverse_hook->get_id($number)
 			);
 		}
 
 		// Any hooks left in the array are not paired with a reverse type hook, and
 		// aren't expected to auto-reverse, so we need to turn their auto-reversal
 		// setting off.
-		if ( ! empty( $hook_instances ) ) {
+		if ( ! empty($hook_instances)) {
 
-			foreach ( $hook_instances as $number => $instance ) {
+			foreach ($hook_instances as $number => $instance) {
 				$instance['auto_reverse'] = 0;
-				$hook->update_callback( $instance, $number );
+				$hook->update_callback($instance, $number);
 			}
 
 			// We add a flag to the database so we'll know to enable legacy features.
@@ -782,7 +782,7 @@ 
 		// Now we check if there are any unpaired reverse type hooks. If there are
 		// we'll set this flag in the database that will keep some legacy features
 		// enabled.
-		if ( $reverse_hook->get_instances( $hook_type ) ) {
+		if ($reverse_hook->get_instances($hook_type)) {
 			update_site_option(
 				"wordpoints_{$reverse_type}_hook_legacy"
 				, true
@@ -796,7 +796,7 @@ 
 	 * @since 1.10.0
 	 */
 	protected function update_network_to_1_10_0() {
-		$this->_1_10_0_delete_post_title_points_log_meta( true );
+		$this->_1_10_0_delete_post_title_points_log_meta(true);
 	}
 
 	/**
@@ -816,28 +816,28 @@ 
 	 * @param bool $network_wide Whether to delete all of the metadata for the whole
 	 *                           network, or just the current site (default).
 	 */
-	protected function _1_10_0_delete_post_title_points_log_meta( $network_wide = false ) {
+	protected function _1_10_0_delete_post_title_points_log_meta($network_wide = false) {
 
 		$query_args = array(
 			'log_type'   => 'post_delete',
 			'meta_query' => array(
-				array( 'key' => 'post_title', 'compare' => 'EXISTS' ),
+				array('key' => 'post_title', 'compare' => 'EXISTS'),
 			),
 		);
 
-		if ( $network_wide ) {
+		if ($network_wide) {
 			$query_args['blog_id'] = false;
 		}
 
-		$query = new WordPoints_Points_Logs_Query( $query_args );
+		$query = new WordPoints_Points_Logs_Query($query_args);
 
 		$logs = $query->get();
 
-		foreach ( $logs as $log ) {
-			wordpoints_delete_points_log_meta( $log->id, 'post_title' );
+		foreach ($logs as $log) {
+			wordpoints_delete_points_log_meta($log->id, 'post_title');
 		}
 
-		wordpoints_regenerate_points_logs( $logs );
+		wordpoints_regenerate_points_logs($logs);
 	}
 
 	/**
@@ -875,7 +875,7 @@ 
 			"
 		); // WPCS: cache pass.
 
-		$this->maybe_update_tables_to_utf8mb4( 'global' );
+		$this->maybe_update_tables_to_utf8mb4('global');
 	}
 
 	/**
@@ -893,7 +893,7 @@ 
 	 * @since 2.1.0
 	 */
 	protected function update_network_to_2_1_0() {
-		add_site_option( 'wordpoints_disabled_points_hooks_edit_points_types', true );
+		add_site_option('wordpoints_disabled_points_hooks_edit_points_types', true);
 	}
 
 	/**
@@ -902,7 +902,7 @@ 
 	 * @since 2.1.0
 	 */
 	protected function update_single_to_2_1_0() {
-		add_option( 'wordpoints_disabled_points_hooks_edit_points_types', true );
+		add_option('wordpoints_disabled_points_hooks_edit_points_types', true);
 	}
 
 	/**
@@ -922,44 +922,44 @@ 
 	protected function update_single_to_2_1_4() {
 
 		$post_types = $this->_2_1_4_get_post_types();
-		$reversal_log_types = $this->_2_1_4_get_reversal_log_types( $post_types );
+		$reversal_log_types = $this->_2_1_4_get_reversal_log_types($post_types);
 		$hits = $this->_2_1_4_get_hits_with_multiple_logs();
 
 		$logs_to_delete = array();
 
-		foreach ( $hits as $hit ) {
+		foreach ($hits as $hit) {
 
-			$log_ids = explode( ',', $hit->log_ids );
+			$log_ids = explode(',', $hit->log_ids);
 
 			// If there weren't exactly two of them, we don't know what to do.
-			if ( count( $log_ids ) !== 2 ) {
+			if (count($log_ids) !== 2) {
 				continue;
 			}
 
 			$query = new WordPoints_Points_Logs_Query(
 				array(
 					'id__in'       => $log_ids,
-					'log_type__in' => array_keys( $reversal_log_types ),
+					'log_type__in' => array_keys($reversal_log_types),
 				)
 			);
 
 			// And if they aren't both the correct types, we don't know what to do.
-			if ( $query->count() !== 2 ) {
+			if ($query->count() !== 2) {
 				continue;
 			}
 
-			$original_log_ids = $this->_2_1_4_get_original_log_ids( $log_ids );
+			$original_log_ids = $this->_2_1_4_get_original_log_ids($log_ids);
 
-			$post_publish_log_id = min( array_keys( $original_log_ids ) );
-			$post_update_log_id = max( array_keys( $original_log_ids ) );
-			$post_publish_reverse_log_id = $original_log_ids[ $post_publish_log_id ];
-			$post_update_reverse_log_id = $original_log_ids[ $post_update_log_id ];
+			$post_publish_log_id = min(array_keys($original_log_ids));
+			$post_update_log_id = max(array_keys($original_log_ids));
+			$post_publish_reverse_log_id = $original_log_ids[$post_publish_log_id];
+			$post_update_reverse_log_id = $original_log_ids[$post_update_log_id];
 
 			$query = new WordPoints_Points_Logs_Query(
-				array( 'id__in' => array( $post_publish_reverse_log_id ) )
+				array('id__in' => array($post_publish_reverse_log_id))
 			);
 
-			$post_publish_reverse_log = $query->get( 'row' );
+			$post_publish_reverse_log = $query->get('row');
 
 			$post_type = str_replace(
 				'reverse-post_publish\\'
@@ -979,7 +979,7 @@ 
 				, true
 			);
 
-			if ( $post_id !== $post_id_2 ) {
+			if ($post_id !== $post_id_2) {
 				continue;
 			}
 
@@ -988,9 +988,9 @@ 
 			$logs_to_delete[] = $post_update_log_id;
 
 			// Give the user their points back, as they were removed in error.
-			$this->_2_1_4_revert_log( $post_publish_reverse_log );
+			$this->_2_1_4_revert_log($post_publish_reverse_log);
 
-			$this->_2_1_4_mark_unreversed( $post_publish_log_id );
+			$this->_2_1_4_mark_unreversed($post_publish_log_id);
 
 			// Now clean up any later updates.
 			$logs_to_delete = $this->_2_1_4_clean_other_logs(
@@ -1002,43 +1002,43 @@ 
 		} // End foreach ( $hits ).
 
 		// Now the legacy logs.
-		$legacy_logs = $this->_2_1_4_get_legacy_reactor_logs( $post_types );
+		$legacy_logs = $this->_2_1_4_get_legacy_reactor_logs($post_types);
 		$post_ids = $this->_2_1_4_get_legacy_points_hook_post_ids();
 
-		foreach ( $post_ids as $post_id ) {
+		foreach ($post_ids as $post_id) {
 
-			if ( ! isset( $legacy_logs[ $post_id ] ) ) {
+			if ( ! isset($legacy_logs[$post_id])) {
 				continue;
 			}
 
-			array_map( array( $this, '_2_1_4_revert_log' ), $legacy_logs[ $post_id ] );
+			array_map(array($this, '_2_1_4_revert_log'), $legacy_logs[$post_id]);
 
 			$logs_to_delete = array_merge(
 				$logs_to_delete
-				, wp_list_pluck( $legacy_logs[ $post_id ], 'id' )
+				, wp_list_pluck($legacy_logs[$post_id], 'id')
 			);
 
-			unset( $legacy_logs[ $post_id ] );
+			unset($legacy_logs[$post_id]);
 		}
 
-		foreach ( $legacy_logs as $logs ) {
-			if ( count( $logs ) > 1 ) {
+		foreach ($legacy_logs as $logs) {
+			if (count($logs) > 1) {
 
 				// The first one is the original, so keep it.
-				unset( $logs[0] );
+				unset($logs[0]);
 
-				array_map( array( $this, '_2_1_4_revert_log' ), $logs );
+				array_map(array($this, '_2_1_4_revert_log'), $logs);
 
 				$logs_to_delete = array_merge(
 					$logs_to_delete
-					, wp_list_pluck( $logs, 'id' )
+					, wp_list_pluck($logs, 'id')
 				);
 			}
 		}
 
 		// Now delete the logs.
-		if ( $logs_to_delete ) {
-			$this->_2_1_4_delete_logs( $logs_to_delete );
+		if ($logs_to_delete) {
+			$this->_2_1_4_delete_logs($logs_to_delete);
 		}
 	}
 
@@ -1051,7 +1051,7 @@ 
 	 */
 	protected function _2_1_4_get_post_types() {
 
-		$post_types = get_post_types( array( 'public' => true ) );
+		$post_types = get_post_types(array('public' => true));
 
 		/**
 		 * Filter which post types to register hook events for.
@@ -1075,12 +1075,12 @@ 
 	 *
 	 * @return array The slugs of the events.
 	 */
-	protected function _2_1_4_get_reversal_log_types( $post_types ) {
+	protected function _2_1_4_get_reversal_log_types($post_types) {
 
 		$event_slugs = array();
 
-		foreach ( $post_types as $slug ) {
-			$event_slugs[ "reverse-post_publish\\{$slug}" ] = true;
+		foreach ($post_types as $slug) {
+			$event_slugs["reverse-post_publish\\{$slug}"] = true;
 		}
 
 		return $event_slugs;
@@ -1124,11 +1124,11 @@ 
 	 *
 	 * @return array The IDs of the original logs.
 	 */
-	protected function _2_1_4_get_original_log_ids( $log_ids ) {
+	protected function _2_1_4_get_original_log_ids($log_ids) {
 
 		$original_log_ids = array();
 
-		foreach ( $log_ids as $log_id ) {
+		foreach ($log_ids as $log_id) {
 
 			$original_log_id = wordpoints_get_points_log_meta(
 				$log_id
@@ -1136,7 +1136,7 @@ 
 				, true
 			);
 
-			$original_log_ids[ $original_log_id ] = $log_id;
+			$original_log_ids[$original_log_id] = $log_id;
 		}
 
 		return $original_log_ids;
@@ -1151,13 +1151,13 @@ 
 	 *
 	 * @param int[] $log_ids The IDs of the logs to delete.
 	 */
-	protected function _2_1_4_delete_logs( $log_ids ) {
+	protected function _2_1_4_delete_logs($log_ids) {
 
 		$hits_to_delete = array();
 
 		global $wpdb;
 
-		foreach ( $log_ids as $log_id ) {
+		foreach ($log_ids as $log_id) {
 
 			$hit_id = wordpoints_get_points_log_meta(
 				$log_id
@@ -1165,26 +1165,26 @@ 
 				, true
 			);
 
-			if ( $hit_id ) {
+			if ($hit_id) {
 				$hits_to_delete[] = $hit_id;
 			}
 
-			wordpoints_points_log_delete_all_metadata( $log_id );
+			wordpoints_points_log_delete_all_metadata($log_id);
 		}
 
 		$wpdb->query( // WPCS: unprepared SQL OK.
 			"
 				DELETE
 				FROM `{$wpdb->wordpoints_points_logs}`
-				WHERE `id` IN (" . wordpoints_prepare__in( $log_ids, '%d' ) . ')
+				WHERE `id` IN (" . wordpoints_prepare__in($log_ids, '%d') . ')
 			'
 		);
 
 		wordpoints_flush_points_logs_caches();
 
 		// Now delete the hits.
-		if ( $hits_to_delete ) {
-			$this->_2_1_4_delete_hits( $hits_to_delete );
+		if ($hits_to_delete) {
+			$this->_2_1_4_delete_hits($hits_to_delete);
 		}
 	}
 
@@ -1195,11 +1195,11 @@ 
 	 *
 	 * @param int[] $hit_ids The IDs of the hits to delete.
 	 */
-	protected function _2_1_4_delete_hits( $hit_ids ) {
+	protected function _2_1_4_delete_hits($hit_ids) {
 
 		global $wpdb;
 
-		foreach ( $hit_ids as $hit_id ) {
+		foreach ($hit_ids as $hit_id) {
 
 			$hit_ids[] = get_metadata(
 				'wordpoints_hook_hit'
@@ -1208,14 +1208,14 @@ 
 				, true
 			);
 
-			delete_metadata( 'wordpoints_hook_hit', $hit_id, '', '', true );
+			delete_metadata('wordpoints_hook_hit', $hit_id, '', '', true);
 		}
 
 		$wpdb->query( // WPCS: unprepared SQL OK.
 			"
 				DELETE
 				FROM `{$wpdb->wordpoints_hook_hits}`
-				WHERE `id` IN (" . wordpoints_prepare__in( $hit_ids, '%d' ) . ')
+				WHERE `id` IN (" . wordpoints_prepare__in($hit_ids, '%d') . ')
 			'
 		); // WPCS: cache OK.
 	}
@@ -1231,14 +1231,14 @@ 
 	 *
 	 * @return int[] The list of logs to be deleted.
 	 */
-	protected function _2_1_4_clean_other_logs( $post_id, $post_type, $logs_to_delete ) {
+	protected function _2_1_4_clean_other_logs($post_id, $post_type, $logs_to_delete) {
 
 		$query = new WordPoints_Points_Logs_Query(
 			array(
 				'order'        => 'ASC',
 				'orderby'      => 'id',
 				'id__not_in'   => $logs_to_delete,
-				'log_type__in' => array( 'post_publish\\' . $post_type ),
+				'log_type__in' => array('post_publish\\' . $post_type),
 				'meta_query'   => array(
 					array(
 						'key'   => 'post\\' . $post_type,
@@ -1257,7 +1257,7 @@ 
 				'meta_query' => array(
 					array(
 						'key'     => 'original_log_id',
-						'value'   => wp_list_pluck( $other_logs, 'id' ),
+						'value'   => wp_list_pluck($other_logs, 'id'),
 						'compare' => 'IN',
 					),
 				),
@@ -1266,7 +1266,7 @@ 
 
 		$reversal_logs = array();
 
-		foreach ( $query->get() as $log ) {
+		foreach ($query->get() as $log) {
 
 			$original_log_id = wordpoints_get_points_log_meta(
 				$log->id
@@ -1274,20 +1274,20 @@ 
 				, true
 			);
 
-			$reversal_logs[ $original_log_id ] = $log;
+			$reversal_logs[$original_log_id] = $log;
 		}
 
-		foreach ( $other_logs as $index => $log ) {
+		foreach ($other_logs as $index => $log) {
 
 			// If this is a log that was reversed within less than a second or so
 			// of its occurrence, it is almost certainly the result of another
 			// update, and it is just cluttering things up.
 			if (
-				isset( $reversal_logs[ $log->id ] )
-				&& strtotime( $reversal_logs[ $log->id ]->date ) - strtotime( $log->date ) < 2
+				isset($reversal_logs[$log->id])
+				&& strtotime($reversal_logs[$log->id]->date) - strtotime($log->date) < 2
 			) {
 				$logs_to_delete[] = $log->id;
-				$logs_to_delete[] = $reversal_logs[ $log->id ]->id;
+				$logs_to_delete[] = $reversal_logs[$log->id]->id;
 			}
 		}
 
@@ -1301,9 +1301,9 @@ 
 	 *
 	 * @param object $log The points log object.
 	 */
-	protected function _2_1_4_revert_log( $log ) {
+	protected function _2_1_4_revert_log($log) {
 
-		add_filter( 'wordpoints_points_log', '__return_false' );
+		add_filter('wordpoints_points_log', '__return_false');
 
 		wordpoints_alter_points(
 			$log->user_id
@@ -1312,7 +1312,7 @@ 
 			, $log->log_type
 		);
 
-		remove_filter( 'wordpoints_points_log', '__return_false' );
+		remove_filter('wordpoints_points_log', '__return_false');
 	}
 
 	/**
@@ -1322,13 +1322,13 @@ 
 	 *
 	 * @param int $log_id The ID of the log to mark as unreversed.
 	 */
-	protected function _2_1_4_mark_unreversed( $log_id ) {
+	protected function _2_1_4_mark_unreversed($log_id) {
 
-		wordpoints_delete_points_log_meta( $log_id, 'auto_reversed' );
+		wordpoints_delete_points_log_meta($log_id, 'auto_reversed');
 
-		$hit_id = wordpoints_get_points_log_meta( $log_id, 'hook_hit_id', true );
+		$hit_id = wordpoints_get_points_log_meta($log_id, 'hook_hit_id', true);
 
-		delete_metadata( 'wordpoints_hook_hit', $hit_id, 'reverse_fired' );
+		delete_metadata('wordpoints_hook_hit', $hit_id, 'reverse_fired');
 	}
 
 	/**
@@ -1340,21 +1340,21 @@ 
 	 *
 	 * @return array[] The logs, grouped by post ID.
 	 */
-	protected function _2_1_4_get_legacy_reactor_logs( $post_types ) {
+	protected function _2_1_4_get_legacy_reactor_logs($post_types) {
 
 		$legacy_log_types = array();
 
-		foreach ( $post_types as $post_type ) {
+		foreach ($post_types as $post_type) {
 			$legacy_log_types[] = "points_legacy_post_publish\\{$post_type}";
 		}
 
 		$query = new WordPoints_Points_Logs_Query(
-			array( 'log_type__in' => $legacy_log_types, 'order' => 'ASC' )
+			array('log_type__in' => $legacy_log_types, 'order' => 'ASC')
 		);
 
 		$legacy_logs = array();
 
-		foreach ( $query->get() as $legacy_log ) {
+		foreach ($query->get() as $legacy_log) {
 
 			$post_type = str_replace(
 				'points_legacy_post_publish\\'
@@ -1368,7 +1368,7 @@ 
 				, true
 			);
 
-			$legacy_logs[ $post_id ][] = $legacy_log;
+			$legacy_logs[$post_id][] = $legacy_log;
 		}
 
 		return $legacy_logs;

src/components/points/includes/class-wordpoints-points-hook.php

Doc Comments +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
 	 *
 	 * @param int|string $number The hook number or ID to get the points for.
 	 *
-	 * @return int|false The number of points for this instance, or false.
+	 * @return integer The number of points for this instance, or false.
 	 */
 	public function get_points( $number = null ) {
 

Spacing +103 added lines, -103 removed lines

@@ -105,18 +105,18 @@ 
 	 *
 	 * @return int|false The number of points for this instance, or false.
 	 */
-	public function get_points( $number = null ) {
+	public function get_points($number = null) {
 
-		if ( isset( $number ) ) {
-			$number = $this->get_number_by_id( $number );
+		if (isset($number)) {
+			$number = $this->get_number_by_id($number);
 		} else {
 			$number = $this->number;
 		}
 
 		$instances = $this->get_instances();
 
-		if ( isset( $instances[ $number ]['points'] ) ) {
-			return $instances[ $number ]['points'];
+		if (isset($instances[$number]['points'])) {
+			return $instances[$number]['points'];
 		}
 
 		return false;
@@ -147,12 +147,12 @@ 
 	 *
 	 * @return array|false Settings to save, or false to cancel saving.
 	 */
-	protected function update( $new_instance, $old_instance ) {
+	protected function update($new_instance, $old_instance) {
 
-		$new_instance = array_merge( array( 'points' => 0 ), $old_instance, $new_instance );
+		$new_instance = array_merge(array('points' => 0), $old_instance, $new_instance);
 
-		if ( false === wordpoints_posint( $new_instance['points'] ) ) {
-			if ( isset( $this->defaults['points'] ) ) {
+		if (false === wordpoints_posint($new_instance['points'])) {
+			if (isset($this->defaults['points'])) {
 				$new_instance['points'] = $this->defaults['points'];
 			} else {
 				return false;
@@ -178,11 +178,11 @@ 
 	 *
 	 * @return bool Whether the hook has a form.
 	 */
-	protected function form( $instance ) {
+	protected function form($instance) {
 
-		if ( ! isset( $instance['points'] ) ) {
+		if ( ! isset($instance['points'])) {
 
-			if ( isset( $this->defaults['points'] ) ) {
+			if (isset($this->defaults['points'])) {
 				$instance['points'] = $this->defaults['points'];
 			} else {
 				$instance['points'] = 0;
@@ -192,8 +192,8 @@ 
 		?>
 
 		<p>
-			<label for="<?php $this->the_field_id( 'points' ); ?>"><?php echo esc_html( $this->options['points_label'] ); ?></label>
-			<input class="widefat" name="<?php $this->the_field_name( 'points' ); ?>" id="<?php $this->the_field_id( 'points' ); ?>" type="number" min="0" value="<?php echo wordpoints_posint( $instance['points'] ); ?>" />
+			<label for="<?php $this->the_field_id('points'); ?>"><?php echo esc_html($this->options['points_label']); ?></label>
+			<input class="widefat" name="<?php $this->the_field_name('points'); ?>" id="<?php $this->the_field_id('points'); ?>" type="number" min="0" value="<?php echo wordpoints_posint($instance['points']); ?>" />
 		</p>
 
 		<?php
@@ -210,7 +210,7 @@ 
 	 *
 	 * @return string The hook instance's description.
 	 */
-	protected function generate_description( $instance = array() ) {
+	protected function generate_description($instance = array()) {
 
 		return $this->options['description'];
 	}
@@ -243,9 +243,9 @@ 
 	 *
 	 * @return string The unique ID for the instance.
 	 */
-	final public function get_id( $number = null ) {
+	final public function get_id($number = null) {
 
-		if ( ! isset( $number ) ) {
+		if ( ! isset($number)) {
 			$number = $this->number;
 		}
 
@@ -275,16 +275,16 @@ 
 	 *
 	 * @param int|string $instance_id The number or ID of an instance.
 	 */
-	final public function set_number( $instance_id ) {
+	final public function set_number($instance_id) {
 
-		$this->number = $this->get_number_by_id( $instance_id );
+		$this->number = $this->get_number_by_id($instance_id);
 
-		if ( '__i__' === $this->number ) {
+		if ('__i__' === $this->number) {
 			return;
-		} elseif ( '0' === $this->number ) {
+		} elseif ('0' === $this->number) {
 			$this->number = 0;
 		} else {
-			wordpoints_posint( $this->number );
+			wordpoints_posint($this->number);
 		}
 	}
 
@@ -298,9 +298,9 @@ 
 	 * @return string The number for the hook instance. Prefixed with 'network_' for
 	 *                network hooks.
 	 */
-	final public function get_number_by_id( $id ) {
+	final public function get_number_by_id($id) {
 
-		return str_replace( $this->id_base . '-', '', $id );
+		return str_replace($this->id_base . '-', '', $id);
 	}
 
 	/**
@@ -338,11 +338,11 @@ 
 	 *
 	 * @return mixed The option, or null if it doesn't exist.
 	 */
-	final public function get_option( $option ) {
+	final public function get_option($option) {
 
-		if ( isset( $this->options[ $option ] ) ) {
+		if (isset($this->options[$option])) {
 
-			return $this->options[ $option ];
+			return $this->options[$option];
 		}
 	}
 
@@ -354,7 +354,7 @@ 
 	 *
 	 * @param array $options The options for the hook.
 	 */
-	final public function set_options( array $options ) {
+	final public function set_options(array $options) {
 		$this->options = $options;
 	}
 
@@ -367,9 +367,9 @@ 
 	 * @param string $option The index for the option.
 	 * @param mixed  $value  The value to assign to this option.
 	 */
-	final public function set_option( $option, $value ) {
+	final public function set_option($option, $value) {
 
-		$this->options[ $option ] = $value;
+		$this->options[$option] = $value;
 	}
 
 	/**
@@ -385,13 +385,13 @@ 
 	 */
 	final public function next_hook_id_number() {
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$type = 'network';
 		} else {
 			$type = 'standard';
 		}
 
-		return 1 + max( array_keys( $this->get_instances( $type ) ) + array( 0 ) );
+		return 1 + max(array_keys($this->get_instances($type)) + array(0));
 	}
 
 	/**
@@ -413,34 +413,34 @@ 
 	 *
 	 * @return array The saved instances of this hook.
 	 */
-	final public function get_instances( $type = 'all' ) {
+	final public function get_instances($type = 'all') {
 
-		switch ( $type ) {
+		switch ($type) {
 
 			case 'standard':
-				$instances = wordpoints_get_array_option( $this->option_name );
+				$instances = wordpoints_get_array_option($this->option_name);
 			break;
 
 			case 'network':
-				if ( is_multisite() ) {
-					$instances = wordpoints_get_array_option( $this->option_name, 'site' );
+				if (is_multisite()) {
+					$instances = wordpoints_get_array_option($this->option_name, 'site');
 				} else {
 					$instances = array();
 				}
 			break;
 
 			case 'all':
-				$instances = wordpoints_get_array_option( $this->option_name );
+				$instances = wordpoints_get_array_option($this->option_name);
 
-				if ( is_multisite() ) {
-					foreach ( wordpoints_get_array_option( $this->option_name, 'site' ) as $number => $instance ) {
-						$instances[ 'network_' . $number ] = $instance;
+				if (is_multisite()) {
+					foreach (wordpoints_get_array_option($this->option_name, 'site') as $number => $instance) {
+						$instances['network_' . $number] = $instance;
 					}
 				}
 			break;
 		}
 
-		unset( $instances['__i__'] );
+		unset($instances['__i__']);
 
 		return $instances;
 	}
@@ -455,22 +455,22 @@ 
 	 * @param array $new_instance The new instance of this hooks settings.
 	 * @param int   $number       The ID number for this hook.
 	 */
-	final public function update_callback( $new_instance, $number ) {
+	final public function update_callback($new_instance, $number) {
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$type = 'network';
 		} else {
 			$type = 'standard';
 		}
 
 		// Get all saved instances of this points hook.
-		$all_instances = $this->get_instances( $type );
+		$all_instances = $this->get_instances($type);
 
-		$this->set_number( $number );
+		$this->set_number($number);
 
-		$old_instance = isset( $all_instances[ $this->number ] ) ? $all_instances[ $this->number ] : array();
+		$old_instance = isset($all_instances[$this->number]) ? $all_instances[$this->number] : array();
 
-		$instance = $this->update( $new_instance, $old_instance );
+		$instance = $this->update($new_instance, $old_instance);
 
 		/**
 		 * Filter a points hook's settings before saving.
@@ -486,13 +486,13 @@ 
 		 * @param array                  $old_instance The old instance of the hook.
 		 * @param WordPoints_Points_Hook $hook         The hook object.
 		 */
-		$instance = apply_filters( 'wordpoints_points_hook_update_callback', $instance, $new_instance, $old_instance, $this );
+		$instance = apply_filters('wordpoints_points_hook_update_callback', $instance, $new_instance, $old_instance, $this);
 
-		if ( false !== $instance ) {
-			$all_instances[ $this->number ] = $instance;
+		if (false !== $instance) {
+			$all_instances[$this->number] = $instance;
 		}
 
-		$this->_save_instances( $all_instances );
+		$this->_save_instances($all_instances);
 	}
 
 	/**
@@ -502,24 +502,24 @@ 
 	 *
 	 * @param string $hook_id The ID of the instance to delete.
 	 */
-	final public function delete_callback( $hook_id ) {
+	final public function delete_callback($hook_id) {
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$type = 'network';
 		} else {
 			$type = 'standard';
 		}
 
 		// Get all saved instances of this points hook.
-		$all_instances = $this->get_instances( $type );
+		$all_instances = $this->get_instances($type);
 
-		$number = $this->get_number_by_id( $hook_id );
+		$number = $this->get_number_by_id($hook_id);
 
-		if ( isset( $all_instances[ $number ] ) && $hook_id = $this->get_id( $number ) ) {
+		if (isset($all_instances[$number]) && $hook_id = $this->get_id($number)) {
 
-			unset( $all_instances[ $number ] );
+			unset($all_instances[$number]);
 
-			$this->_save_instances( $all_instances );
+			$this->_save_instances($all_instances);
 		}
 	}
 
@@ -532,27 +532,27 @@ 
 	 *
 	 * @return bool|void Whether the form was displayed.
 	 */
-	final public function form_callback( $number ) {
+	final public function form_callback($number) {
 
-		$this->set_number( $number );
+		$this->set_number($number);
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$type = 'network';
 		} else {
 			$type = 'standard';
 		}
 
-		$all_instances = $this->get_instances( $type );
+		$all_instances = $this->get_instances($type);
 
-		if ( 0 === $this->number || '0' === $this->number ) {
+		if (0 === $this->number || '0' === $this->number) {
 
 			// We echo out a form where 'number' can be set later.
-			$this->set_number( '__i__' );
+			$this->set_number('__i__');
 			$instance = array();
 
 		} else {
 
-			$instance = $all_instances[ $this->number ];
+			$instance = $all_instances[$this->number];
 		}
 
 		/**
@@ -565,11 +565,11 @@ 
 		 * @param array                  $instance The settings for this instance.
 		 * @param WordPoints_Points_Hook $hook     The hook object.
 		 */
-		$instance = apply_filters( 'wordpoints_points_hook_form_callback', $instance, $this );
+		$instance = apply_filters('wordpoints_points_hook_form_callback', $instance, $this);
 
-		if ( false !== $instance ) {
+		if (false !== $instance) {
 
-			$has_form = $this->form( $instance );
+			$has_form = $this->form($instance);
 
 			/**
 			 * Inside the points hook form.
@@ -581,7 +581,7 @@ 
 			 * @param array                  $instance Settings for this instance.
 			 * @param WordPoints_Points_Hook $hook     The hook object.
 			 */
-			do_action( 'wordpoints_in_points_hook_form', $has_form, $instance, $this );
+			do_action('wordpoints_in_points_hook_form', $has_form, $instance, $this);
 
 			return $has_form;
 		}
@@ -602,9 +602,9 @@ 
 	 *
 	 * @return string Name attribute for $field_name.
 	 */
-	final public function get_field_name( $field_name ) {
+	final public function get_field_name($field_name) {
 
-		return esc_attr( 'hook-' . $this->id_base . '[' . $this->number . '][' . $field_name . ']' );
+		return esc_attr('hook-' . $this->id_base . '[' . $this->number . '][' . $field_name . ']');
 	}
 
 	/**
@@ -616,9 +616,9 @@ 
 	 *
 	 * @param string $field_name The field name.
 	 */
-	final public function the_field_name( $field_name ) {
+	final public function the_field_name($field_name) {
 
-		echo esc_attr( $this->get_field_name( $field_name ) );
+		echo esc_attr($this->get_field_name($field_name));
 	}
 
 	/**
@@ -636,9 +636,9 @@ 
 	 *
 	 * @return string ID attribute for $field_name.
 	 */
-	final public function get_field_id( $field_name ) {
+	final public function get_field_id($field_name) {
 
-		return esc_attr( 'hook-' . $this->id_base . '-' . $this->number . '-' . $field_name );
+		return esc_attr('hook-' . $this->id_base . '-' . $this->number . '-' . $field_name);
 	}
 
 	/**
@@ -650,9 +650,9 @@ 
 	 *
 	 * @param string $field_name The field name.
 	 */
-	final public function the_field_id( $field_name ) {
+	final public function the_field_id($field_name) {
 
-		echo esc_attr( $this->get_field_id( $field_name ) );
+		echo esc_attr($this->get_field_id($field_name));
 	}
 
 	/**
@@ -665,15 +665,15 @@ 
 	 *
 	 * @return string The hook's description.
 	 */
-	final public function get_description( $type = 'any' ) {
+	final public function get_description($type = 'any') {
 
 		$instances = $this->get_instances();
 
-		if ( 'generated' !== $type && ! empty( $instances[ $this->number ]['_description'] ) ) {
-			return $instances[ $this->number ]['_description'];
+		if ('generated' !== $type && ! empty($instances[$this->number]['_description'])) {
+			return $instances[$this->number]['_description'];
 		}
 
-		$instance = ( isset( $instances[ $this->number ] ) ) ? $instances[ $this->number ] : array();
+		$instance = (isset($instances[$this->number])) ? $instances[$this->number] : array();
 
 		/**
 		 * Filter the description for a points hook.
@@ -684,7 +684,7 @@ 
 		 * @param WordPoints_Points_Hook $hook        The points hook object.
 		 * @param array                  $instance    The settings for this instance.
 		 */
-		return apply_filters( 'wordpoints_points_hook_description', $this->generate_description( $instance ), $this, $instance );
+		return apply_filters('wordpoints_points_hook_description', $this->generate_description($instance), $this, $instance);
 	}
 
 	/**
@@ -697,31 +697,31 @@ 
 	 *
 	 * @return string|false The type of points, or false if none found.
 	 */
-	final public function points_type( $number = null ) {
+	final public function points_type($number = null) {
 
 		$network_mode = false;
 
-		if ( ! isset( $number ) ) {
+		if ( ! isset($number)) {
 
 			$number = $this->number;
 
-		} elseif ( is_string( $number ) && 'network_' === substr( $number, 0, 8 ) ) {
+		} elseif (is_string($number) && 'network_' === substr($number, 0, 8)) {
 
 			$network_mode = true;
-			$number = (int) substr( $number, 8 );
+			$number = (int) substr($number, 8);
 		}
 
 		$current_mode = WordPoints_Points_Hooks::get_network_mode();
 
-		if ( $current_mode !== $network_mode ) {
-			WordPoints_Points_Hooks::set_network_mode( $network_mode );
+		if ($current_mode !== $network_mode) {
+			WordPoints_Points_Hooks::set_network_mode($network_mode);
 		}
 
-		$points_type = WordPoints_Points_Hooks::get_points_type( $this->get_id( $number ) );
+		$points_type = WordPoints_Points_Hooks::get_points_type($this->get_id($number));
 
 		// Reset network mode if it was changed.
-		if ( $current_mode !== $network_mode ) {
-			WordPoints_Points_Hooks::set_network_mode( $current_mode );
+		if ($current_mode !== $network_mode) {
+			WordPoints_Points_Hooks::set_network_mode($current_mode);
 		}
 
 		return $points_type;
@@ -750,15 +750,15 @@ 
 	 *              overriding that function, you can ignore this option.
 	 * }
 	 */
-	final protected function init( $name, array $options = array() ) {
+	final protected function init($name, array $options = array()) {
 
-		$this->id_base     = strtolower( get_class( $this ) );
+		$this->id_base     = strtolower(get_class($this));
 		$this->name        = $name;
 		$this->option_name = 'wordpoints_hook-' . $this->id_base;
 
 		// Option names can only be 191 characters long.
-		if ( isset( $this->option_name{191} ) ) {
-			_doing_it_wrong( __METHOD__, sprintf( 'Points hook class names cannot be longer than 175 characters, %s is %s character(s) too long.', esc_html( $this->id_base ), (int) strlen( $this->id_base ) - 175 ), '1.5.0' );
+		if (isset($this->option_name{191} )) {
+			_doing_it_wrong(__METHOD__, sprintf('Points hook class names cannot be longer than 175 characters, %s is %s character(s) too long.', esc_html($this->id_base), (int) strlen($this->id_base) - 175), '1.5.0');
 			return;
 		}
 
@@ -766,12 +766,12 @@ 
 			array(
 				'width'        => 250,
 				'description'  => '',
-				'points_label' => __( 'Points:', 'wordpoints' ),
+				'points_label' => __('Points:', 'wordpoints'),
 			)
 			, $options
 		);
 
-		$this->options['_classname']   = $this->option_name;
+		$this->options['_classname'] = $this->option_name;
 	}
 
 	//
@@ -785,15 +785,15 @@ 
 	 *
 	 * @param array $instances All settings, indexed by instance number.
 	 */
-	final private function _save_instances( $instances ) {
+	final private function _save_instances($instances) {
 
 		// This needs to start at 1.
-		unset( $instances[0] );
+		unset($instances[0]);
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
-			update_site_option( $this->option_name, $instances );
+		if (WordPoints_Points_Hooks::get_network_mode()) {
+			update_site_option($this->option_name, $instances);
 		} else {
-			update_option( $this->option_name, $instances );
+			update_option($this->option_name, $instances);
 		}
 	}
 

src/components/points/includes/class-wordpoints-points-logs-query.php

Doc Comments +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
 	 *
 	 * @param string $arg The query arg whose value to retrieve.
 	 *
-	 * @return mixed|void The query arg's value, or nothing if it isn't set.
+	 * @return string The query arg's value, or nothing if it isn't set.
 	 */
 	public function get_arg( $arg ) {
 

Spacing +149 added lines, -149 removed lines

@@ -38,7 +38,7 @@ 
 	 *
 	 * @type array $_fields
 	 */
-	private $_fields = array( 'id', 'user_id', 'log_type', 'text', 'points', 'points_type', 'blog_id', 'site_id', 'date' );
+	private $_fields = array('id', 'user_id', 'log_type', 'text', 'points', 'points_type', 'blog_id', 'site_id', 'date');
 
 	/**
 	 * Whether the query is ready for execution.
@@ -220,7 +220,7 @@ 
 	 *        @type array        $meta_query          See WP_Meta_Query.
 	 * }
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 
 		global $wpdb;
 
@@ -252,7 +252,7 @@ 
 			'date_query'          => null,
 		);
 
-		$this->_args = wp_parse_args( $args, $defaults );
+		$this->_args = wp_parse_args($args, $defaults);
 
 	} // public function __construct()
 
@@ -265,10 +265,10 @@ 
 	 *
 	 * @return mixed|void The query arg's value, or nothing if it isn't set.
 	 */
-	public function get_arg( $arg ) {
+	public function get_arg($arg) {
 
-		if ( isset( $this->_args[ $arg ] ) ) {
-			return $this->_args[ $arg ];
+		if (isset($this->_args[$arg])) {
+			return $this->_args[$arg];
 		}
 	}
 
@@ -279,9 +279,9 @@ 
 	 *
 	 * @param array $args A list of arguments to set and their values.
 	 */
-	public function set_args( array $args ) {
+	public function set_args(array $args) {
 
-		$this->_args = array_merge( $this->_args, $args );
+		$this->_args = array_merge($this->_args, $args);
 
 		$this->_query_ready = false;
 		$this->_cache_query_hash = null;
@@ -303,20 +303,20 @@ 
 	 */
 	public function count() {
 
-		if ( $this->_is_cached_query ) {
-			$cache = $this->_cache_get( 'count' );
+		if ($this->_is_cached_query) {
+			$cache = $this->_cache_get('count');
 
-			if ( false !== $cache ) {
+			if (false !== $cache) {
 				return $cache;
 			}
 		}
 
 		$this->_select_type = 'SELECT COUNT';
 
-		$count = (int) $this->_get( 'var' );
+		$count = (int) $this->_get('var');
 
-		if ( $this->_is_cached_query ) {
-			$this->_cache_set( $count, 'count' );
+		if ($this->_is_cached_query) {
+			$this->_cache_set($count, 'count');
 		}
 
 		return $count;
@@ -334,34 +334,34 @@ 
 	 *
 	 * @return mixed The results of the query, or false on failure.
 	 */
-	public function get( $method = 'results' ) {
+	public function get($method = 'results') {
 
-		$methods = array( 'results', 'row', 'col', 'var' );
+		$methods = array('results', 'row', 'col', 'var');
 
-		if ( ! in_array( $method, $methods ) ) {
+		if ( ! in_array($method, $methods)) {
 
-			_doing_it_wrong( __METHOD__, esc_html( sprintf( 'WordPoints Debug Error: invalid get method %s, possible values are %s', $method, implode( ', ', $methods ) ) ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html(sprintf('WordPoints Debug Error: invalid get method %s, possible values are %s', $method, implode(', ', $methods))), '1.0.0');
 
 			return false;
 		}
 
-		if ( $this->_is_cached_query ) {
-			$cache = $this->_cache_get( "get_{$method}" );
+		if ($this->_is_cached_query) {
+			$cache = $this->_cache_get("get_{$method}");
 
-			if ( false !== $cache ) {
+			if (false !== $cache) {
 				return $cache;
 			}
 		}
 
 		$this->_select_type = 'SELECT';
 
-		$result = $this->_get( $method );
+		$result = $this->_get($method);
 
-		if ( $this->_is_cached_query ) {
-			$this->_cache_set( $result, "get_{$method}" );
+		if ($this->_is_cached_query) {
+			$this->_cache_set($result, "get_{$method}");
 
-			if ( 'results' === $method || 'col' === $method ) {
-				$this->_cache_set( count( $result ), 'count' );
+			if ('results' === $method || 'col' === $method) {
+				$this->_cache_set(count($result), 'count');
 			}
 		}
 
@@ -388,20 +388,20 @@ 
 	 *
 	 * @return object[]|false The logs for this page, or false if $page or $per_page is invalid.
 	 */
-	public function get_page( $page, $per_page = 25 ) {
+	public function get_page($page, $per_page = 25) {
 
-		if ( ! wordpoints_posint( $page ) || ! wordpoints_posint( $per_page ) ) {
+		if ( ! wordpoints_posint($page) || ! wordpoints_posint($per_page)) {
 			return false;
 		}
 
-		$start = ( $page - 1 ) * $per_page;
+		$start = ($page - 1) * $per_page;
 
 		// First try the main cache.
-		if ( $this->_is_cached_query ) {
+		if ($this->_is_cached_query) {
 
-			$cache = $this->_cache_get( 'get_results' );
+			$cache = $this->_cache_get('get_results');
 
-			if ( false !== $cache ) {
+			if (false !== $cache) {
 				return array_slice(
 					$cache
 					, $start - $this->_args['start']
@@ -415,11 +415,11 @@ 
 
 		$this->_args['start'] += $start;
 
-		if ( $this->_args['limit'] ) {
+		if ($this->_args['limit']) {
 			$this->_args['limit'] -= $start;
 		}
 
-		if ( ! $this->_args['limit'] || $this->_args['limit'] > $per_page ) {
+		if ( ! $this->_args['limit'] || $this->_args['limit'] > $per_page) {
 			$this->_args['limit'] = $per_page;
 		}
 
@@ -428,7 +428,7 @@ 
 
 		$this->_select_type = 'SELECT';
 
-		unset( $this->_cache_query_hash );
+		unset($this->_cache_query_hash);
 
 		$results = $this->get();
 
@@ -439,7 +439,7 @@ 
 		$this->_limit = '';
 		$this->_prepare_limit();
 
-		unset( $this->_cache_query_hash );
+		unset($this->_cache_query_hash);
 
 		return $results;
 	}
@@ -460,9 +460,9 @@ 
 	 *
 	 * @return string The SQL for the query.
 	 */
-	public function get_sql( $select_type = null ) {
+	public function get_sql($select_type = null) {
 
-		if ( isset( $select_type ) ) {
+		if (isset($select_type)) {
 			$this->_select_type = $select_type;
 		}
 
@@ -512,9 +512,9 @@ 
 	 * @param string $deprecated Deprecated; no longer used.
 	 * @param bool   $network    Whether this is a network-wide query.
 	 */
-	public function prime_cache( $key = 'default:%points_type%', $deprecated = null, $network = false ) {
+	public function prime_cache($key = 'default:%points_type%', $deprecated = null, $network = false) {
 
-		if ( ! is_null( $deprecated ) ) {
+		if ( ! is_null($deprecated)) {
 			_deprecated_argument(
 				__METHOD__
 				, '1.9.0'
@@ -536,7 +536,7 @@ 
 			, $key
 		);
 
-		if ( $network ) {
+		if ($network) {
 			$this->_cache_group = 'wordpoints_network_points_logs_query';
 		} else {
 			$this->_cache_group = 'wordpoints_points_logs_query';
@@ -556,7 +556,7 @@ 
 	 *
 	 * @return string[] The valid columns.
 	 */
-	public function date_query_valid_columns_filter( $valid_columns ) {
+	public function date_query_valid_columns_filter($valid_columns) {
 
 		$valid_columns[] = 'date';
 
@@ -572,9 +572,9 @@ 
 	 *
 	 * @return string The correct meta table ID column, if the key is wordpoints_points_log_.
 	 */
-	public function meta_query_meta_table_id_filter( $key ) {
+	public function meta_query_meta_table_id_filter($key) {
 
-		if ( 'wordpoints_points_log__id' === $key ) {
+		if ('wordpoints_points_log__id' === $key) {
 			$key = 'log_id';
 		}
 
@@ -598,7 +598,7 @@ 
 
 		$this->_prepare_query();
 
-		$select = ( 'SELECT COUNT' === $this->_select_type ) ? $this->_select_count : $this->_select;
+		$select = ('SELECT COUNT' === $this->_select_type) ? $this->_select_count : $this->_select;
 
 		return $select
 			. "\nFROM `{$wpdb->wordpoints_points_logs}`\n"
@@ -619,13 +619,13 @@ 
 	 *
 	 * @return mixed The query results.
 	 */
-	private function _get( $method ) {
+	private function _get($method) {
 
 		global $wpdb;
 
 		$get = "get_{$method}";
 
-		return $wpdb->$get( $this->_get_sql() );
+		return $wpdb->$get($this->_get_sql());
 	}
 
 	/**
@@ -638,29 +638,29 @@ 
 	 *
 	 * @return mixed Cached value, or false if none.
 	 */
-	private function _cache_get( $type = null ) {
+	private function _cache_get($type = null) {
 
-		$cache = wp_cache_get( $this->_cache_key, $this->_cache_group );
+		$cache = wp_cache_get($this->_cache_key, $this->_cache_group);
 
-		if ( ! is_array( $cache ) ) {
+		if ( ! is_array($cache)) {
 			return false;
 		}
 
 		$this->_calc_cache_query_hash();
 
-		if ( ! isset( $cache[ $this->_cache_query_hash ] ) ) {
+		if ( ! isset($cache[$this->_cache_query_hash])) {
 			return false;
 		}
 
-		if ( isset( $type ) ) {
-			if ( isset( $cache[ $this->_cache_query_hash ][ $type ] ) ) {
-				return $cache[ $this->_cache_query_hash ][ $type ];
+		if (isset($type)) {
+			if (isset($cache[$this->_cache_query_hash][$type])) {
+				return $cache[$this->_cache_query_hash][$type];
 			} else {
 				return false;
 			}
 		}
 
-		return $cache[ $this->_cache_query_hash ];
+		return $cache[$this->_cache_query_hash];
 	}
 
 	/**
@@ -672,26 +672,26 @@ 
 	 * @param string $type  Optionally specify a results type to cache. Default is
 	 *                      null, or all types.
 	 */
-	private function _cache_set( $value, $type = null ) {
+	private function _cache_set($value, $type = null) {
 
-		$cache = wp_cache_get( $this->_cache_key, $this->_cache_group );
+		$cache = wp_cache_get($this->_cache_key, $this->_cache_group);
 
 		$this->_calc_cache_query_hash();
 
 		if (
-			! isset( $cache[ $this->_cache_query_hash ] )
-			|| ! is_array( $cache[ $this->_cache_query_hash ] )
+			! isset($cache[$this->_cache_query_hash])
+			|| ! is_array($cache[$this->_cache_query_hash])
 		) {
-			$cache[ $this->_cache_query_hash ] = array();
+			$cache[$this->_cache_query_hash] = array();
 		}
 
-		if ( isset( $type ) ) {
-			$cache[ $this->_cache_query_hash ][ $type ] = $value;
+		if (isset($type)) {
+			$cache[$this->_cache_query_hash][$type] = $value;
 		} else {
-			$cache[ $this->_cache_query_hash ] = $value;
+			$cache[$this->_cache_query_hash] = $value;
 		}
 
-		wp_cache_set( $this->_cache_key, $cache, $this->_cache_group );
+		wp_cache_set($this->_cache_key, $cache, $this->_cache_group);
 	}
 
 	/**
@@ -701,8 +701,8 @@ 
 	 */
 	private function _calc_cache_query_hash() {
 
-		if ( ! isset( $this->_cache_query_hash ) ) {
-			$this->_cache_query_hash = wordpoints_hash( $this->get_sql() );
+		if ( ! isset($this->_cache_query_hash)) {
+			$this->_cache_query_hash = wordpoints_hash($this->get_sql());
 		}
 	}
 
@@ -713,7 +713,7 @@ 
 	 */
 	private function _prepare_query() {
 
-		if ( ! $this->_query_ready ) {
+		if ( ! $this->_query_ready) {
 
 			$this->_prepare_select();
 			$this->_prepare_where();
@@ -734,22 +734,22 @@ 
 		// Pull all fields by default.
 		$fields = $this->_fields;
 
-		if ( 'all' !== $this->_args['fields'] ) {
+		if ('all' !== $this->_args['fields']) {
 
 			$_fields = (array) $this->_args['fields'];
-			$diff    = array_diff( $_fields, $this->_fields );
-			$_fields = array_intersect( $this->_fields, $_fields );
+			$diff    = array_diff($_fields, $this->_fields);
+			$_fields = array_intersect($this->_fields, $_fields);
 
-			if ( ! empty( $diff ) ) {
-				_doing_it_wrong( __METHOD__, esc_html( 'WordPoints Debug Error: invalid field(s) "' . implode( '", "', $diff ) . '" given' ), '1.0.0' );
+			if ( ! empty($diff)) {
+				_doing_it_wrong(__METHOD__, esc_html('WordPoints Debug Error: invalid field(s) "' . implode('", "', $diff) . '" given'), '1.0.0');
 			}
 
-			if ( ! empty( $_fields ) ) {
+			if ( ! empty($_fields)) {
 				$fields = $_fields;
 			}
 		}
 
-		$fields = '`' . implode( '`, `', $fields ) . '`';
+		$fields = '`' . implode('`, `', $fields) . '`';
 
 		$this->_select = "SELECT {$fields}";
 	}
@@ -763,8 +763,8 @@ 
 
 		$this->_wheres = array();
 
-		$this->_prepare_posint__in( $this->_args['id__in'], 'id' );
-		$this->_prepare_posint__in( $this->_args['id__not_in'], 'id', 'NOT IN' );
+		$this->_prepare_posint__in($this->_args['id__in'], 'id');
+		$this->_prepare_posint__in($this->_args['id__not_in'], 'id', 'NOT IN');
 		$this->_prepare_user_where();
 		$this->_prepare_points_type_where();
 		$this->_prepare_log_type_where();
@@ -774,9 +774,9 @@ 
 		$this->_prepare_date_where();
 		$this->_prepare_meta_where();
 
-		if ( ! empty( $this->_wheres ) ) {
+		if ( ! empty($this->_wheres)) {
 
-			$this->_where = 'WHERE ' . implode( ' AND ', $this->_wheres ) . "\n";
+			$this->_where = 'WHERE ' . implode(' AND ', $this->_wheres) . "\n";
 		}
 
 	} // function _prepare_where()
@@ -790,31 +790,31 @@ 
 	 */
 	private function _prepare_limit() {
 
-		if ( ! isset( $this->_args['limit'] ) ) {
+		if ( ! isset($this->_args['limit'])) {
 			return;
 		}
 
-		foreach ( array( 'limit', 'start' ) as $var ) {
+		foreach (array('limit', 'start') as $var) {
 
-			$_var = $this->_args[ $var ];
+			$_var = $this->_args[$var];
 
-			if ( false === wordpoints_int( $this->_args[ $var ] ) ) {
+			if (false === wordpoints_int($this->_args[$var])) {
 
 				_doing_it_wrong(
 					__METHOD__
 					, sprintf(
 						"WordPoints Debug Error: '%s' must be a positive integer, %s given"
-						, esc_html( $var )
-						, esc_html( strval( $_var ) ? $_var : gettype( $_var ) )
+						, esc_html($var)
+						, esc_html(strval($_var) ? $_var : gettype($_var))
 					)
 					, '1.0.0'
 				);
 
-				$this->_args[ $var ] = 0;
+				$this->_args[$var] = 0;
 			}
 		}
 
-		if ( $this->_args['limit'] > 0 && $this->_args['start'] >= 0 ) {
+		if ($this->_args['limit'] > 0 && $this->_args['start'] >= 0) {
 			$this->_limit = "LIMIT {$this->_args['start']}, {$this->_args['limit']}";
 		}
 	}
@@ -833,21 +833,21 @@ 
 		$order    = $this->_args['order'];
 		$order_by = $this->_args['orderby'];
 
-		if ( 'none' === $order_by ) {
+		if ('none' === $order_by) {
 			return;
 		}
 
-		if ( ! in_array( $order, array( 'DESC', 'ASC' ) ) ) {
+		if ( ! in_array($order, array('DESC', 'ASC'))) {
 
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: invalid 'order' \"{$order}\", possible values are DESC and ASC" ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: invalid 'order' \"{$order}\", possible values are DESC and ASC"), '1.0.0');
 			$order = 'DESC';
 		}
 
-		if ( 'meta_value' === $order_by ) {
+		if ('meta_value' === $order_by) {
 
-			if ( isset( $this->_args['meta_type'] ) ) {
+			if (isset($this->_args['meta_type'])) {
 
-				$meta_type = $this->meta_query->get_cast_for_type( $this->_args['meta_type'] );
+				$meta_type = $this->meta_query->get_cast_for_type($this->_args['meta_type']);
 				$order_by  = "CAST({$wpdb->wordpoints_points_log_meta}.meta_value AS {$meta_type})";
 
 			} else {
@@ -855,9 +855,9 @@ 
 				$order_by = "{$wpdb->wordpoints_points_log_meta}.meta_value";
 			}
 
-		} elseif ( ! in_array( $order_by, $this->_fields ) ) {
+		} elseif ( ! in_array($order_by, $this->_fields)) {
 
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: invalid 'orderby' \"{$order_by}\", possible values are " . implode( ', ', $this->_fields ) ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: invalid 'orderby' \"{$order_by}\", possible values are " . implode(', ', $this->_fields)), '1.0.0');
 			return;
 		}
 
@@ -876,13 +876,13 @@ 
 	 * @param string $type   The type of IN condition: 'IN' or 'NOT IN'.
 	 * @param string $format The format for the values in $_in ('%s', '%d', '%f').
 	 */
-	private function _prepare__in( $_in, $column, $type = 'IN', $format = '%s' ) {
+	private function _prepare__in($_in, $column, $type = 'IN', $format = '%s') {
 
-		if ( ! empty( $_in ) ) {
+		if ( ! empty($_in)) {
 
-			$in = wordpoints_prepare__in( $_in, $format );
+			$in = wordpoints_prepare__in($_in, $format);
 
-			if ( $in ) {
+			if ($in) {
 				$this->_wheres[] = "{$column} {$type} ({$in})";
 			}
 		}
@@ -899,10 +899,10 @@ 
 	 * @param string $column The column to search.
 	 * @param string $type   The type of IN condition: 'IN' or 'NOT IN'.
 	 */
-	private function _prepare_posint__in( $in, $column, $type = 'IN' ) {
+	private function _prepare_posint__in($in, $column, $type = 'IN') {
 
-		$in = array_filter( array_map( 'wordpoints_posint', $in ) );
-		$this->_prepare__in( $in, $column, $type, '%d' );
+		$in = array_filter(array_map('wordpoints_posint', $in));
+		$this->_prepare__in($in, $column, $type, '%d');
 	}
 
 	/**
@@ -914,14 +914,14 @@ 
 
 		global $wpdb;
 
-		if ( wordpoints_posint( $this->_args['user_id'] ) ) {
+		if (wordpoints_posint($this->_args['user_id'])) {
 
-			$this->_wheres[] = $wpdb->prepare( '`user_id` = %d', $this->_args['user_id'] );
+			$this->_wheres[] = $wpdb->prepare('`user_id` = %d', $this->_args['user_id']);
 
 		} else {
 
-			$this->_prepare_posint__in( $this->_args['user__in'], 'user_id' );
-			$this->_prepare_posint__in( $this->_args['user__not_in'], 'user_id', 'NOT IN' );
+			$this->_prepare_posint__in($this->_args['user__in'], 'user_id');
+			$this->_prepare_posint__in($this->_args['user__not_in'], 'user_id', 'NOT IN');
 		}
 	}
 
@@ -934,22 +934,22 @@ 
 
 		global $wpdb;
 
-		if ( wordpoints_is_points_type( $this->_args['points_type'] ) ) {
+		if (wordpoints_is_points_type($this->_args['points_type'])) {
 
-			$this->_wheres[] = $wpdb->prepare( '`points_type` = %s', $this->_args['points_type'] );
+			$this->_wheres[] = $wpdb->prepare('`points_type` = %s', $this->_args['points_type']);
 
 		} else {
 
-			$points_types = array_keys( wordpoints_get_points_types() );
+			$points_types = array_keys(wordpoints_get_points_types());
 
-			if ( is_array( $this->_args['points_type__in'] ) ) {
+			if (is_array($this->_args['points_type__in'])) {
 
-				$this->_prepare__in( array_intersect( $this->_args['points_type__in'], $points_types ), 'points_type' );
+				$this->_prepare__in(array_intersect($this->_args['points_type__in'], $points_types), 'points_type');
 			}
 
-			if ( is_array( $this->_args['points_type__not_in'] ) ) {
+			if (is_array($this->_args['points_type__not_in'])) {
 
-				$this->_prepare__in( array_intersect( $this->_args['points_type__not_in'], $points_types ), 'points_type', 'NOT IN' );
+				$this->_prepare__in(array_intersect($this->_args['points_type__not_in'], $points_types), 'points_type', 'NOT IN');
 			}
 		}
 	}
@@ -963,14 +963,14 @@ 
 
 		global $wpdb;
 
-		if ( ! empty( $this->_args['log_type'] ) ) {
+		if ( ! empty($this->_args['log_type'])) {
 
-			$this->_wheres[] = $wpdb->prepare( '`log_type` = %s', $this->_args['log_type'] );
+			$this->_wheres[] = $wpdb->prepare('`log_type` = %s', $this->_args['log_type']);
 
 		} else {
 
-			$this->_prepare__in( $this->_args['log_type__in'], 'log_type' );
-			$this->_prepare__in( $this->_args['log_type__not_in'], 'log_type', 'NOT IN' );
+			$this->_prepare__in($this->_args['log_type__in'], 'log_type');
+			$this->_prepare__in($this->_args['log_type__not_in'], 'log_type', 'NOT IN');
 		}
 	}
 
@@ -983,26 +983,26 @@ 
 
 		global $wpdb;
 
-		if ( ! isset( $this->_args['points'] ) ) {
+		if ( ! isset($this->_args['points'])) {
 			return;
 		}
 
 		$_points = $this->_args['points'];
 
-		if ( ! wordpoints_int( $this->_args['points'] ) ) {
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: 'points' must be an integer, " . gettype( $_points ) . ' given' ), '1.0.0' );
+		if ( ! wordpoints_int($this->_args['points'])) {
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: 'points' must be an integer, " . gettype($_points) . ' given'), '1.0.0');
 			return;
 		}
 
-		$comparisons = array( '=', '<', '>', '<>', '!=', '<=', '>=' );
+		$comparisons = array('=', '<', '>', '<>', '!=', '<=', '>=');
 
-		if ( ! in_array( $this->_args['points__compare'], $comparisons ) ) {
+		if ( ! in_array($this->_args['points__compare'], $comparisons)) {
 
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: invalid 'points__compare' {$this->_args['points__compare']}, possible values are " . implode( ', ', $comparisons ) ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: invalid 'points__compare' {$this->_args['points__compare']}, possible values are " . implode(', ', $comparisons)), '1.0.0');
 			$this->_args['points__compare'] = '=';
 		}
 
-		$this->_wheres[] = $wpdb->prepare( "`points` {$this->_args['points__compare']} %d", $this->_args['points'] ); // WPCS: unprepared SQL OK
+		$this->_wheres[] = $wpdb->prepare("`points` {$this->_args['points__compare']} %d", $this->_args['points']); // WPCS: unprepared SQL OK
 	}
 
 	/**
@@ -1014,19 +1014,19 @@ 
 
 		global $wpdb;
 
-		if ( empty( $this->_args['text'] ) ) {
+		if (empty($this->_args['text'])) {
 			return;
 		}
 
-		$comparisons = array( '=', '<>', '!=', 'LIKE', 'NOT LIKE' );
+		$comparisons = array('=', '<>', '!=', 'LIKE', 'NOT LIKE');
 
-		if ( ! in_array( $this->_args['text__compare'], $comparisons ) ) {
+		if ( ! in_array($this->_args['text__compare'], $comparisons)) {
 
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: invalid 'text__compare' {$this->_args['text__compare']}, possible values are " . implode( ', ', $comparisons ) ), '1.6.0' );
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: invalid 'text__compare' {$this->_args['text__compare']}, possible values are " . implode(', ', $comparisons)), '1.6.0');
 			$this->_args['text__compare'] = '=';
 		}
 
-		$this->_wheres[] = $wpdb->prepare( "`text` {$this->_args['text__compare']} %s", $this->_args['text'] ); // WPCS: unprepared SQL OK
+		$this->_wheres[] = $wpdb->prepare("`text` {$this->_args['text__compare']} %s", $this->_args['text']); // WPCS: unprepared SQL OK
 	}
 
 	/**
@@ -1038,22 +1038,22 @@ 
 
 		global $wpdb;
 
-		if ( ! is_multisite() ) {
+		if ( ! is_multisite()) {
 			return;
 		}
 
-		if ( wordpoints_posint( $this->_args['site_id'] ) ) {
-			$this->_wheres[] = $wpdb->prepare( '`site_id` = %d', $this->_args['site_id'] );
+		if (wordpoints_posint($this->_args['site_id'])) {
+			$this->_wheres[] = $wpdb->prepare('`site_id` = %d', $this->_args['site_id']);
 		}
 
-		if ( ! empty( $this->_args['blog__in'] ) || ! empty( $this->_args['blog__not_in'] ) ) {
+		if ( ! empty($this->_args['blog__in']) || ! empty($this->_args['blog__not_in'])) {
 
-			$this->_prepare_posint__in( $this->_args['blog__in'], 'blog_id' );
-			$this->_prepare_posint__in( $this->_args['blog__not_in'], 'blog_id', 'NOT IN' );
+			$this->_prepare_posint__in($this->_args['blog__in'], 'blog_id');
+			$this->_prepare_posint__in($this->_args['blog__not_in'], 'blog_id', 'NOT IN');
 
-		} elseif ( wordpoints_posint( $this->_args['blog_id'] ) ) {
+		} elseif (wordpoints_posint($this->_args['blog_id'])) {
 
-			$this->_wheres[] = $wpdb->prepare( '`blog_id` = %d', $this->_args['blog_id'] );
+			$this->_wheres[] = $wpdb->prepare('`blog_id` = %d', $this->_args['blog_id']);
 		}
 	}
 
@@ -1064,21 +1064,21 @@ 
 	 */
 	private function _prepare_date_where() {
 
-		if ( empty( $this->_args['date_query'] ) || ! is_array( $this->_args['date_query'] ) ) {
+		if (empty($this->_args['date_query']) || ! is_array($this->_args['date_query'])) {
 			return;
 		}
 
-		add_filter( 'date_query_valid_columns', array( $this, 'date_query_valid_columns_filter' ) );
+		add_filter('date_query_valid_columns', array($this, 'date_query_valid_columns_filter'));
 
-		$date_query = new WP_Date_Query( $this->_args['date_query'], 'date' );
+		$date_query = new WP_Date_Query($this->_args['date_query'], 'date');
 		$date_query = $date_query->get_sql();
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 
-			$this->_wheres[] = ltrim( $date_query, ' AND' );
+			$this->_wheres[] = ltrim($date_query, ' AND');
 		}
 
-		remove_filter( 'date_query_valid_columns', array( $this, 'date_query_valid_columns_filter' ) );
+		remove_filter('date_query_valid_columns', array($this, 'date_query_valid_columns_filter'));
 	}
 
 	/**
@@ -1101,19 +1101,19 @@ 
 			)
 		);
 
-		if ( empty( $meta_args ) ) {
+		if (empty($meta_args)) {
 			return;
 		}
 
 		$this->meta_query = new WP_Meta_Query();
-		$this->meta_query->parse_query_vars( $meta_args );
+		$this->meta_query->parse_query_vars($meta_args);
 
-		add_filter( 'sanitize_key', array( $this, 'meta_query_meta_table_id_filter' ) );
-		$meta_query = $this->meta_query->get_sql( 'wordpoints_points_log_', $wpdb->wordpoints_points_logs, 'id', $this );
-		remove_filter( 'sanitize_key', array( $this, 'meta_query_meta_table_id_filter' ) );
+		add_filter('sanitize_key', array($this, 'meta_query_meta_table_id_filter'));
+		$meta_query = $this->meta_query->get_sql('wordpoints_points_log_', $wpdb->wordpoints_points_logs, 'id', $this);
+		remove_filter('sanitize_key', array($this, 'meta_query_meta_table_id_filter'));
 
-		if ( ! empty( $meta_query['where'] ) ) {
-			$this->_wheres[] = ltrim( $meta_query['where'], ' AND' );
+		if ( ! empty($meta_query['where'])) {
+			$this->_wheres[] = ltrim($meta_query['where'], ' AND');
 		}
 
 		$this->_meta_join = $meta_query['join'] . "\n";

src/components/points/includes/hooks/abstracts/post-type.php

Doc Comments +1 added lines, -1 removed lines

@@ -421,7 +421,7 @@
 	 * @since 2.0.0
 	 *
 	 * @param string $key       The log meta key to match on.
-	 * @param mixed  $value     The log meta value to match on.
+	 * @param integer  $value     The log meta value to match on.
 	 * @param string $new_key   The key for the new meta value to add.
 	 * @param mixed  $new_value The new meta value to add. If falsey, new meta isn't added.
 	 */

Spacing +81 added lines, -81 removed lines

@@ -41,17 +41,17 @@ 
 	/**
 	 * @since 1.9.0
 	 */
-	public function __construct( $title, $args ) {
+	public function __construct($title, $args) {
 
-		$this->init( $title, $args );
+		$this->init($title, $args);
 
-		add_filter( "wordpoints_points_log-{$this->log_type}", array( $this, 'logs' ), 10, 6 );
-		add_filter( "wordpoints_points_log-reverse_{$this->log_type}", array( $this, 'logs' ), 10, 6 );
+		add_filter("wordpoints_points_log-{$this->log_type}", array($this, 'logs'), 10, 6);
+		add_filter("wordpoints_points_log-reverse_{$this->log_type}", array($this, 'logs'), 10, 6);
 
-		add_action( 'delete_post', array( $this, 'clean_logs_on_post_deletion' ), 15 );
+		add_action('delete_post', array($this, 'clean_logs_on_post_deletion'), 15);
 
-		if ( ! $this->uses_points_logs_viewing_restriction_api ) {
-			add_filter( "wordpoints_user_can_view_points_log-{$this->log_type}", array( $this, 'user_can_view' ), 10, 3 );
+		if ( ! $this->uses_points_logs_viewing_restriction_api) {
+			add_filter("wordpoints_user_can_view_points_log-{$this->log_type}", array($this, 'user_can_view'), 10, 3);
 		}
 	}
 
@@ -65,14 +65,14 @@ 
 	 *
 	 * @return bool Whether this post type matches and points should be awarded.
 	 */
-	public function is_matching_post_type( $post_type, $instance_post_type ) {
+	public function is_matching_post_type($post_type, $instance_post_type) {
 
 		return (
 			$instance_post_type === $post_type
 			|| (
 				'ALL' === $instance_post_type
-				&& post_type_exists( $post_type )
-				&& get_post_type_object( $post_type )->public
+				&& post_type_exists($post_type)
+				&& get_post_type_object($post_type)->public
 			)
 		);
 	}
@@ -86,27 +86,27 @@ 
 	 *
 	 * @return bool True if auto-reversals should be done; false otherwise.
 	 */
-	protected function should_do_auto_reversals_for_post_type( $post_type ) {
+	protected function should_do_auto_reversals_for_post_type($post_type) {
 
 		// Check if this hook type allows auto-reversals to be disabled.
-		if ( ! $this->get_option( 'disable_auto_reverse_label' ) ) {
+		if ( ! $this->get_option('disable_auto_reverse_label')) {
 			return true;
 		}
 
 		$instances = $this->get_instances();
 
-		foreach ( $instances as $instance ) {
+		foreach ($instances as $instance) {
 
-			$instance = array_merge( $this->defaults, $instance );
+			$instance = array_merge($this->defaults, $instance);
 
-			if ( $post_type === $instance['post_type'] ) {
-				return ! empty( $instance['auto_reverse'] );
-			} elseif ( 'ALL' === $instance['post_type'] ) {
+			if ($post_type === $instance['post_type']) {
+				return ! empty($instance['auto_reverse']);
+			} elseif ('ALL' === $instance['post_type']) {
 				$all_posts_instance = $instance;
 			}
 		}
 
-		return ! empty( $all_posts_instance['auto_reverse'] );
+		return ! empty($all_posts_instance['auto_reverse']);
 	}
 
 	/**
@@ -118,7 +118,7 @@ 
 	 *
 	 * @return array The logs to reverse.
 	 */
-	protected function get_logs_to_auto_reverse( array $meta_query ) {
+	protected function get_logs_to_auto_reverse(array $meta_query) {
 
 		$query = new WordPoints_Points_Logs_Query(
 			array(
@@ -135,7 +135,7 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return array();
 		}
 
@@ -149,17 +149,17 @@ 
 	 *
 	 * @param object $log The log to reverse.
 	 */
-	protected function auto_reverse_log( $log ) {
+	protected function auto_reverse_log($log) {
 
 		wordpoints_alter_points(
 			$log->user_id
 			, -$log->points
 			, $log->points_type
 			, "reverse_{$this->log_type}"
-			, array( 'original_log_id' => $log->id )
+			, array('original_log_id' => $log->id)
 		);
 
-		wordpoints_add_points_log_meta( $log->id, 'auto_reversed', true );
+		wordpoints_add_points_log_meta($log->id, 'auto_reversed', true);
 	}
 
 	/**
@@ -176,47 +176,47 @@ 
 	 *
 	 * @return string
 	 */
-	public function logs( $text, $points, $points_type, $user_id, $log_type, $meta ) {
+	public function logs($text, $points, $points_type, $user_id, $log_type, $meta) {
 
 		$reverse = '';
 
-		if ( "reverse_{$this->log_type}" === $log_type ) {
+		if ("reverse_{$this->log_type}" === $log_type) {
 			$reverse = '_reverse';
 		}
 
 		$post = false;
 
-		if ( '' === $reverse && isset( $meta['post_id'] ) ) {
-			$post = get_post( $meta['post_id'] );
+		if ('' === $reverse && isset($meta['post_id'])) {
+			$post = get_post($meta['post_id']);
 		}
 
-		if ( $post ) {
+		if ($post) {
 
 			// This post exists, so we should include the permalink in the log text.
-			$text = $this->log_with_post_title_link( $post->ID, $reverse );
+			$text = $this->log_with_post_title_link($post->ID, $reverse);
 
 		} else {
 
 			// This post doesn't exist; we probably can't use the title.
-			$text = $this->get_option( 'log_text_no_post_title' . $reverse );
+			$text = $this->get_option('log_text_no_post_title' . $reverse);
 
 			if (
-				$this->get_option( 'log_text_post_type' . $reverse )
-				&& isset( $meta['post_type'] )
-				&& post_type_exists( $meta['post_type'] )
+				$this->get_option('log_text_post_type' . $reverse)
+				&& isset($meta['post_type'])
+				&& post_type_exists($meta['post_type'])
 			) {
 
 				// We do know the type of post though, so include that in the log.
 				$text = sprintf(
-					$this->get_option( 'log_text_post_type' . $reverse )
-					, get_post_type_object( $meta['post_type'] )->labels->singular_name
+					$this->get_option('log_text_post_type' . $reverse)
+					, get_post_type_object($meta['post_type'])->labels->singular_name
 				);
 
-			} elseif ( isset( $meta['post_title'] ) ) {
+			} elseif (isset($meta['post_title'])) {
 
 				// If the title is saved as metadata, then we can use it.
 				$text = sprintf(
-					$this->get_option( 'log_text_post_title' . $reverse )
+					$this->get_option('log_text_post_title' . $reverse)
 					, $meta['post_title']
 				);
 			}
@@ -236,33 +236,33 @@ 
 	 *
 	 * @return string The text for the log entry.
 	 */
-	protected function log_with_post_title_link( $post_id, $reverse = '', $url = null ) {
+	protected function log_with_post_title_link($post_id, $reverse = '', $url = null) {
 
-		if ( ! isset( $url ) ) {
-			$url = get_permalink( $post_id );
+		if ( ! isset($url)) {
+			$url = get_permalink($post_id);
 		}
 
-		$post_title = get_the_title( $post_id );
+		$post_title = get_the_title($post_id);
 
 		$args = array();
 
-		$args[] = '<a href="' . esc_url( $url ) . '">'
-			. ( $post_title ? $post_title : _x( '(no title)', 'post title', 'wordpoints' ) )
+		$args[] = '<a href="' . esc_url($url) . '">'
+			. ($post_title ? $post_title : _x('(no title)', 'post title', 'wordpoints'))
 			. '</a>';
 
-		$text = $this->get_option( 'log_text_post_title_and_type' . $reverse );
+		$text = $this->get_option('log_text_post_title_and_type' . $reverse);
 
 		if (
 			$text
-			&& ( $post_type = get_post_field( 'post_type', $post_id ) )
-			&& post_type_exists( $post_type )
+			&& ($post_type = get_post_field('post_type', $post_id))
+			&& post_type_exists($post_type)
 		) {
-			$args[] = get_post_type_object( $post_type )->labels->singular_name;
+			$args[] = get_post_type_object($post_type)->labels->singular_name;
 		} else {
-			$text = $this->get_option( 'log_text_post_title' . $reverse );
+			$text = $this->get_option('log_text_post_title' . $reverse);
 		}
 
-		return vsprintf( $text, $args );
+		return vsprintf($text, $args);
 	}
 
 	/**
@@ -274,17 +274,17 @@ 
 	 *
 	 * @return string A description for the hook instance.
 	 */
-	protected function generate_description( $instance = array() ) {
+	protected function generate_description($instance = array()) {
 
-		if ( ! empty( $instance['post_type'] ) && 'ALL' !== $instance['post_type'] ) {
-			$post_type = get_post_type_object( $instance['post_type'] );
+		if ( ! empty($instance['post_type']) && 'ALL' !== $instance['post_type']) {
+			$post_type = get_post_type_object($instance['post_type']);
 
-			if ( $post_type ) {
-				return sprintf( $this->get_option( 'post_type_description' ), $post_type->labels->singular_name );
+			if ($post_type) {
+				return sprintf($this->get_option('post_type_description'), $post_type->labels->singular_name);
 			}
 		}
 
-		return parent::generate_description( $instance );
+		return parent::generate_description($instance);
 	}
 
 	/**
@@ -296,43 +296,43 @@ 
 	 *
 	 * @return bool True.
 	 */
-	protected function form( $instance ) {
+	protected function form($instance) {
 
-		$instance = array_merge( $this->defaults, $instance );
+		$instance = array_merge($this->defaults, $instance);
 
 		?>
 
 		<p>
-			<label for="<?php $this->the_field_id( 'post_type' ); ?>"><?php esc_html_e( 'Select post type:', 'wordpoints' ); ?></label>
+			<label for="<?php $this->the_field_id('post_type'); ?>"><?php esc_html_e('Select post type:', 'wordpoints'); ?></label>
 			<?php
 
 			wordpoints_list_post_types(
 				array(
 					'selected' => $instance['post_type'],
-					'id'       => $this->get_field_id( 'post_type' ),
-					'name'     => $this->get_field_name( 'post_type' ),
+					'id'       => $this->get_field_id('post_type'),
+					'name'     => $this->get_field_name('post_type'),
 					'class'    => 'widefat wordpoints-append-to-hook-title',
-					'filter'   => $this->get_option( 'post_type_filter' ),
+					'filter'   => $this->get_option('post_type_filter'),
 				)
-				, array( 'public' => true )
+				, array('public' => true)
 			);
 
 			?>
 		</p>
 
-		<?php parent::form( $instance ); ?>
+		<?php parent::form($instance); ?>
 
 		<?php
 
-		$disable_label = $this->get_option( 'disable_auto_reverse_label' );
+		$disable_label = $this->get_option('disable_auto_reverse_label');
 
-		if ( $disable_label ) {
+		if ($disable_label) {
 
 			?>
 
 			<p>
-				<input class="widefat" name="<?php $this->the_field_name( 'auto_reverse' ); ?>" id="<?php $this->the_field_id( 'auto_reverse' ); ?>" type="checkbox" value="1" <?php checked( '1', $instance['auto_reverse'] ); ?> />
-				<label for="<?php $this->the_field_id( 'auto_reverse' ); ?>"><?php echo esc_html( $disable_label ); ?></label>
+				<input class="widefat" name="<?php $this->the_field_name('auto_reverse'); ?>" id="<?php $this->the_field_id('auto_reverse'); ?>" type="checkbox" value="1" <?php checked('1', $instance['auto_reverse']); ?> />
+				<label for="<?php $this->the_field_id('auto_reverse'); ?>"><?php echo esc_html($disable_label); ?></label>
 			</p>
 
 		<?php
@@ -350,9 +350,9 @@ 
 	 *
 	 * @return bool True if the post type supports comments, false otherwise.
 	 */
-	public function post_type_supports_comments( $post_type ) {
+	public function post_type_supports_comments($post_type) {
 
-		return post_type_supports( $post_type->name, 'comments' );
+		return post_type_supports($post_type->name, 'comments');
 	}
 
 	/**
@@ -370,7 +370,7 @@ 
 	 *
 	 * @return bool Whether the user can view this log.
 	 */
-	public function user_can_view( $can_view, $log, $user_id ) {
+	public function user_can_view($can_view, $log, $user_id) {
 
 		_deprecated_function(
 			__METHOD__
@@ -378,13 +378,13 @@ 
 			, 'the points logs viewing restrictions API'
 		);
 
-		if ( $can_view ) {
+		if ($can_view) {
 
 			$restriction = new WordPoints_Points_Logs_Viewing_Restriction_Read_Post(
 				$log
 			);
 
-			$can_view = $restriction->user_can( $user_id );
+			$can_view = $restriction->user_can($user_id);
 		}
 
 		return $can_view;
@@ -405,13 +405,13 @@ 
 	 *
 	 * @param int $post_id The ID of the post being deleted.
 	 */
-	public function clean_logs_on_post_deletion( $post_id ) {
+	public function clean_logs_on_post_deletion($post_id) {
 
 		$this->clean_logs(
 			'post_id'
 			, $post_id
 			, 'post_type'
-			, get_post_field( 'post_type', $post_id )
+			, get_post_field('post_type', $post_id)
 		);
 	}
 
@@ -425,7 +425,7 @@ 
 	 * @param string $new_key   The key for the new meta value to add.
 	 * @param mixed  $new_value The new meta value to add. If falsey, new meta isn't added.
 	 */
-	protected function clean_logs( $key, $value, $new_key, $new_value ) {
+	protected function clean_logs($key, $value, $new_key, $new_value) {
 
 		$logs_query = new WordPoints_Points_Logs_Query(
 			array(
@@ -441,20 +441,20 @@ 
 
 		$logs = $logs_query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
-			wordpoints_delete_points_log_meta( $log->id, $key );
+			wordpoints_delete_points_log_meta($log->id, $key);
 
-			if ( $new_value ) {
-				wordpoints_add_points_log_meta( $log->id, $new_key, $new_value );
+			if ($new_value) {
+				wordpoints_add_points_log_meta($log->id, $new_key, $new_value);
 			}
 		}
 
-		wordpoints_regenerate_points_logs( $logs );
+		wordpoints_regenerate_points_logs($logs);
 	}
 }
 

src/components/points/includes/logs.php

Doc Comments +11 added lines, -1 removed lines

@@ -294,7 +294,7 @@ 
  *        'default', which will return a list of the most recent points logs, with
  *        users excluded according to the general settings.
  *
- * @return WordPoints_Points_Logs_Query|false Logs query instance, or false.
+ * @return WordPoints_Points_Logs_Query Logs query instance, or false.
  */
 function wordpoints_get_points_logs_query( $points_type, $query_slug = 'default' ) {
 
@@ -466,6 +466,11 @@ 
  * @since 1.9.0
  *
  * @WordPress\action wordpoints_points_log-comment_disapprove
+ * @param string $text
+ * @param integer $points
+ * @param string $points_type
+ * @param integer $user_id
+ * @param string $log_type
  */
 function wordpoints_points_logs_comment_disapprove( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
@@ -492,6 +497,11 @@ 
  * @since 1.9.0
  *
  * @WordPress\action wordpoints_points_log-post_delete
+ * @param string $text
+ * @param integer $points
+ * @param string $points_type
+ * @param integer $user_id
+ * @param string $log_type
  */
 function wordpoints_points_logs_post_delete( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 

Spacing +88 added lines, -88 removed lines

@@ -55,7 +55,7 @@ 
 	 */
 	private static function init() {
 
-		if ( self::$initialized ) {
+		if (self::$initialized) {
 			return;
 		}
 
@@ -67,10 +67,10 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_register_points_logs_queries' );
+		do_action('wordpoints_register_points_logs_queries');
 
 		// Make sure that the default query is registered.
-		self::register_query( 'default', array(), array( 'cache_queries' => true ) );
+		self::register_query('default', array(), array('cache_queries' => true));
 
 		self::$initialized = true;
 	}
@@ -89,9 +89,9 @@ 
 	 *
 	 * @return bool Whether the query was registered.
 	 */
-	public static function register_query( $slug, array $args, array $data = array() ) {
+	public static function register_query($slug, array $args, array $data = array()) {
 
-		if ( empty( $slug ) || isset( self::$queries[ $slug ] ) ) {
+		if (empty($slug) || isset(self::$queries[$slug])) {
 			return false;
 		}
 
@@ -101,8 +101,8 @@ 
 			'network_wide'  => false,
 		);
 
-		self::$queries[ $slug ] = array_merge( $defaults, $data );
-		self::$queries[ $slug ]['args'] = $args;
+		self::$queries[$slug] = array_merge($defaults, $data);
+		self::$queries[$slug]['args'] = $args;
 
 		return true;
 	}
@@ -116,11 +116,11 @@ 
 	 *
 	 * @return bool Whether $slug is the slug of a registered query.
 	 */
-	public static function is_query( $slug ) {
+	public static function is_query($slug) {
 
 		self::init();
 
-		return isset( self::$queries[ $slug ] );
+		return isset(self::$queries[$slug]);
 	}
 
 	/**
@@ -146,9 +146,9 @@ 
 	 *
 	 * @return array The query's args.
 	 */
-	public static function get_query_args( $query_slug ) {
+	public static function get_query_args($query_slug) {
 
-		return self::get_query_data( $query_slug, 'args' );
+		return self::get_query_data($query_slug, 'args');
 	}
 
 	/**
@@ -162,16 +162,16 @@ 
 	 *
 	 * @return mixed The data, or null if not found.
 	 */
-	public static function get_query_data( $query_slug, $data = null ) {
+	public static function get_query_data($query_slug, $data = null) {
 
 		self::init();
 
-		if ( isset( self::$queries[ $query_slug ] ) ) {
+		if (isset(self::$queries[$query_slug])) {
 
-			if ( empty( $data ) ) {
-				return self::$queries[ $query_slug ];
-			} elseif ( isset( self::$queries[ $query_slug ][ $data ] ) ) {
-				return self::$queries[ $query_slug ][ $data ];
+			if (empty($data)) {
+				return self::$queries[$query_slug];
+			} elseif (isset(self::$queries[$query_slug][$data])) {
+				return self::$queries[$query_slug][$data];
 			}
 		}
 	}
@@ -210,9 +210,9 @@ 
  *
  * @return bool Whether the query was registered.
  */
-function wordpoints_register_points_logs_query( $slug, array $args, array $data = array() ) {
+function wordpoints_register_points_logs_query($slug, array $args, array $data = array()) {
 
-	return WordPoints_Points_Log_Queries::register_query( $slug, $args, $data );
+	return WordPoints_Points_Log_Queries::register_query($slug, $args, $data);
 }
 
 /**
@@ -226,9 +226,9 @@ 
  *
  * @return bool Whether $slug is the slug of a registered query.
  */
-function wordpoints_is_points_logs_query( $slug ) {
+function wordpoints_is_points_logs_query($slug) {
 
-	return WordPoints_Points_Log_Queries::is_query( $slug );
+	return WordPoints_Points_Log_Queries::is_query($slug);
 }
 
 /**
@@ -243,23 +243,23 @@ 
  *
  * @return array|false The args for the query, or false on failure.
  */
-function wordpoints_get_points_logs_query_args( $points_type, $query_slug = 'default' ) {
+function wordpoints_get_points_logs_query_args($points_type, $query_slug = 'default') {
 
-	$args = WordPoints_Points_Log_Queries::get_query_args( $query_slug );
+	$args = WordPoints_Points_Log_Queries::get_query_args($query_slug);
 
-	if ( is_null( $args ) || ! wordpoints_is_points_type( $points_type ) ) {
+	if (is_null($args) || ! wordpoints_is_points_type($points_type)) {
 		return false;
 	}
 
 	$defaults = array(
-		'user__not_in' => wordpoints_get_excluded_users( 'points_logs' ),
+		'user__not_in' => wordpoints_get_excluded_users('points_logs'),
 		'points_type'  => $points_type,
 	);
 
-	$args = array_merge( $defaults, $args );
+	$args = array_merge($defaults, $args);
 
 	// The current user needs to be set dynamically, since it can change at times.
-	if ( 'current_user' === $query_slug ) {
+	if ('current_user' === $query_slug) {
 		$args['user_id'] = get_current_user_id();
 	}
 
@@ -276,7 +276,7 @@ 
 	 * @param string $query_slug  The slug for the query.
 	 * @param string $points_type The points type the query is being made for.
 	 */
-	return apply_filters( 'wordpoints_points_logs_query_args', $args, $query_slug, $points_type );
+	return apply_filters('wordpoints_points_logs_query_args', $args, $query_slug, $points_type);
 }
 
 /**
@@ -296,19 +296,19 @@ 
  *
  * @return WordPoints_Points_Logs_Query|false Logs query instance, or false.
  */
-function wordpoints_get_points_logs_query( $points_type, $query_slug = 'default' ) {
+function wordpoints_get_points_logs_query($points_type, $query_slug = 'default') {
 
-	$args = wordpoints_get_points_logs_query_args( $points_type, $query_slug );
+	$args = wordpoints_get_points_logs_query_args($points_type, $query_slug);
 
-	if ( ! $args ) {
+	if ( ! $args) {
 		return false;
 	}
 
-	$query = new WordPoints_Points_Logs_Query( $args );
+	$query = new WordPoints_Points_Logs_Query($args);
 
-	$query_data = WordPoints_Points_Log_Queries::get_query_data( $query_slug );
+	$query_data = WordPoints_Points_Log_Queries::get_query_data($query_slug);
 
-	if ( $query_data['cache_queries'] ) {
+	if ($query_data['cache_queries']) {
 
 		$query->prime_cache(
 			$query_data['cache_key']
@@ -351,9 +351,9 @@ 
  *
  * @return void
  */
-function wordpoints_show_points_logs( $logs_query, array $args = array() ) {
+function wordpoints_show_points_logs($logs_query, array $args = array()) {
 
-	if ( ! $logs_query instanceof WordPoints_Points_Logs_Query ) {
+	if ( ! $logs_query instanceof WordPoints_Points_Logs_Query) {
 		return;
 	}
 
@@ -365,9 +365,9 @@ 
 		'show_users' => true,
 	);
 
-	$args = array_merge( $defaults, $args );
+	$args = array_merge($defaults, $args);
 
-	if ( ! $args['datatable'] ) {
+	if ( ! $args['datatable']) {
 
 		_deprecated_argument(
 			__FUNCTION__
@@ -378,12 +378,12 @@ 
 		$args['paginate'] = false;
 	}
 
-	$view = wordpoints_component( 'points' )
-		->get_sub_app( 'logs' )
-		->get_sub_app( 'views' )
-		->get( $args['view'], array( $logs_query, $args ) );
+	$view = wordpoints_component('points')
+		->get_sub_app('logs')
+		->get_sub_app('views')
+		->get($args['view'], array($logs_query, $args));
 
-	if ( ! $view instanceof WordPoints_Points_Logs_View ) {
+	if ( ! $view instanceof WordPoints_Points_Logs_View) {
 		return;
 	}
 
@@ -403,9 +403,9 @@ 
  * @param string $query_slug  The query to use to display the logs.
  * @param array  $args        Arguments to pass to wordpoints_show_points_logs().
  */
-function wordpoints_show_points_logs_query( $points_type, $query_slug = 'default', array $args = array() ) {
+function wordpoints_show_points_logs_query($points_type, $query_slug = 'default', array $args = array()) {
 
-	wordpoints_show_points_logs( wordpoints_get_points_logs_query( $points_type, $query_slug ), $args );
+	wordpoints_show_points_logs(wordpoints_get_points_logs_query($points_type, $query_slug), $args);
 }
 
 /**
@@ -424,7 +424,7 @@ 
 	 */
 	wordpoints_register_points_logs_query(
 		'current_user'
-		, array( 'user_id' => get_current_user_id() )
+		, array('user_id' => get_current_user_id())
 		, array(
 			'cache_key'     => 'current_user:%points_type%:%user_id%',
 			'cache_queries' => true,
@@ -438,7 +438,7 @@ 
 	 */
 	wordpoints_register_points_logs_query(
 		'network'
-		, array( 'blog_id' => false )
+		, array('blog_id' => false)
 		, array(
 			'network_wide'  => true,
 			'cache_queries' => true,
@@ -453,11 +453,11 @@ 
  *
  * @WordPress\action wordpoints_points_log-profile_edit
  */
-function wordpoints_points_logs_profile_edit( $text, $points, $points_type, $user_id, $log_type, $meta ) {
+function wordpoints_points_logs_profile_edit($text, $points, $points_type, $user_id, $log_type, $meta) {
 
-	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta['user_id'] )->display_name, $meta['user_id'], 'display' );
+	$user_name = sanitize_user_field('display_name', get_userdata($meta['user_id'])->display_name, $meta['user_id'], 'display');
 
-	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta['reason'] ) );
+	return sprintf(_x('Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints'), $user_name, esc_html($meta['reason']));
 }
 
 /**
@@ -467,20 +467,20 @@ 
  *
  * @WordPress\action wordpoints_points_log-comment_disapprove
  */
-function wordpoints_points_logs_comment_disapprove( $text, $points, $points_type, $user_id, $log_type, $meta ) {
+function wordpoints_points_logs_comment_disapprove($text, $points, $points_type, $user_id, $log_type, $meta) {
 
-	switch ( $meta['status'] ) {
+	switch ($meta['status']) {
 
 		case 'spam':
-			$text = _x( 'Comment marked as spam.', 'points log description', 'wordpoints' );
+			$text = _x('Comment marked as spam.', 'points log description', 'wordpoints');
 		break;
 
 		case 'trash':
-			$text = _x( 'Comment moved to trash.', 'points log description', 'wordpoints' );
+			$text = _x('Comment moved to trash.', 'points log description', 'wordpoints');
 		break;
 
 		default:
-			$text = _x( 'Comment unapproved.', 'points log description', 'wordpoints' );
+			$text = _x('Comment unapproved.', 'points log description', 'wordpoints');
 	}
 
 	return $text;
@@ -493,23 +493,23 @@ 
  *
  * @WordPress\action wordpoints_points_log-post_delete
  */
-function wordpoints_points_logs_post_delete( $text, $points, $points_type, $user_id, $log_type, $meta ) {
+function wordpoints_points_logs_post_delete($text, $points, $points_type, $user_id, $log_type, $meta) {
 
-	if ( isset( $meta['post_type'] ) ) {
+	if (isset($meta['post_type'])) {
 
-		$post_type = get_post_type_object( $meta['post_type'] );
+		$post_type = get_post_type_object($meta['post_type']);
 
-		if ( ! is_null( $post_type ) ) {
+		if ( ! is_null($post_type)) {
 
 			/* translators: the post type's name. */
 			return sprintf(
-				_x( '%s deleted.', 'points log description', 'wordpoints' )
+				_x('%s deleted.', 'points log description', 'wordpoints')
 				, $post_type->labels->singular_name
 			);
 		}
 	}
 
-	return _x( 'Post deleted.', 'points log description', 'wordpoints' );
+	return _x('Post deleted.', 'points log description', 'wordpoints');
 }
 
 /**
@@ -525,10 +525,10 @@ 
  * @param int    $points      The number of points. Not used.
  * @param string $points_type The type of points being awarded.
  */
-function wordpoints_clean_points_logs_cache( $user_id, $points, $points_type ) {
+function wordpoints_clean_points_logs_cache($user_id, $points, $points_type) {
 
 	wordpoints_flush_points_logs_caches(
-		array( 'user_id' => $user_id, 'points_type' => $points_type )
+		array('user_id' => $user_id, 'points_type' => $points_type)
 	);
 }
 
@@ -550,27 +550,27 @@ 
  *        @type int             $user_id     Only clear the cache for this user.
  * }
  */
-function wordpoints_flush_points_logs_caches( $args = array() ) {
+function wordpoints_flush_points_logs_caches($args = array()) {
 
-	$args = array_merge( array( 'points_type' => false, 'user_id' => 0 ), $args );
+	$args = array_merge(array('points_type' => false, 'user_id' => 0), $args);
 
 	$find = array(
 		'%points_type%',
 		'%user_id%',
 	);
 
-	if ( empty( $args['points_type'] ) ) {
-		$points_types = array_keys( wordpoints_get_points_types() );
+	if (empty($args['points_type'])) {
+		$points_types = array_keys(wordpoints_get_points_types());
 	} else {
 		$points_types = (array) $args['points_type'];
 	}
 
-	foreach ( $points_types as $points_type ) {
-		foreach ( WordPoints_Points_Log_Queries::get_queries() as $query ) {
+	foreach ($points_types as $points_type) {
+		foreach (WordPoints_Points_Log_Queries::get_queries() as $query) {
 
-			if ( ! empty( $query['cache_key'] ) ) {
+			if ( ! empty($query['cache_key'])) {
 
-				if ( $query['network_wide'] ) {
+				if ($query['network_wide']) {
 					$group = 'wordpoints_network_points_logs_query';
 				} else {
 					$group = 'wordpoints_points_logs_query';
@@ -582,7 +582,7 @@ 
 				);
 
 				wp_cache_delete(
-					str_replace( $find, $replace, $query['cache_key'] )
+					str_replace($find, $replace, $query['cache_key'])
 					, $group
 				);
 			}
@@ -601,26 +601,26 @@ 
  *
  * @return bool Whether the user can view this points log entry.
  */
-function wordpoints_user_can_view_points_log( $user_id, $log ) {
+function wordpoints_user_can_view_points_log($user_id, $log) {
 
 	// We do this just once here for optimization, as otherwise it would run 3 times.
-	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);
 	}
 
 	/** @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');
 
-	$can_view = $viewing_restrictions->get_restriction( $log )
-		->user_can( $user_id );
+	$can_view = $viewing_restrictions->get_restriction($log)
+		->user_can($user_id);
 
-	if ( $can_view ) {
-		$can_view = $viewing_restrictions->apply_legacy_filters( $user_id, $log );
+	if ($can_view) {
+		$can_view = $viewing_restrictions->apply_legacy_filters($user_id, $log);
 	}
 
-	if ( isset( $switched ) ) {
+	if (isset($switched)) {
 		restore_current_blog();
 	}
 
@@ -641,17 +641,17 @@ 
  *
  * @return bool Whether the user can view the points log.
  */
-function wordpoints_hooks_user_can_view_points_log( $can_view, $user_id, $log ) {
+function wordpoints_hooks_user_can_view_points_log($can_view, $user_id, $log) {
 
-	_deprecated_function( __FUNCTION__, '2.2.0' );
+	_deprecated_function(__FUNCTION__, '2.2.0');
 
-	if ( ! $can_view ) {
+	if ( ! $can_view) {
 		return $can_view;
 	}
 
-	$restriction = new WordPoints_Points_Logs_Viewing_Restriction_Hooks( $log );
+	$restriction = new WordPoints_Points_Logs_Viewing_Restriction_Hooks($log);
 
-	return $restriction->user_can( $user_id );
+	return $restriction->user_can($user_id);
 }
 
 // EOF

src/components/points/includes/points.php

Doc Comments +1 added lines, -1 removed lines

@@ -363,7 +363,7 @@
  *
  * @param string $type The slug for a points type.
  *
- * @return int|false The minimum for this type of points. False if $type is bad.
+ * @return integer The minimum for this type of points. False if $type is bad.
  */
 function wordpoints_get_points_minimum( $type ) {
 

Spacing +228 added lines, -228 removed lines

@@ -19,11 +19,11 @@ 
  *
  * @return bool Whether a points type with the given slug exists.
  */
-function wordpoints_is_points_type( $slug ) {
+function wordpoints_is_points_type($slug) {
 
 	$points_types = wordpoints_get_points_types();
 
-	return isset( $points_types[ $slug ] );
+	return isset($points_types[$slug]);
 }
 
 /**
@@ -55,7 +55,7 @@ 
  */
 function wordpoints_get_points_types() {
 
-	return wordpoints_get_maybe_network_array_option( 'wordpoints_points_types' );
+	return wordpoints_get_maybe_network_array_option('wordpoints_points_types');
 }
 
 /**
@@ -67,15 +67,15 @@ 
  *
  * @return array|false An array of settings for this points type. False on failure.
  */
-function wordpoints_get_points_type( $slug ) {
+function wordpoints_get_points_type($slug) {
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( ! isset( $points_types[ $slug ] ) ) {
+	if ( ! isset($points_types[$slug])) {
 		return false;
 	}
 
-	return $points_types[ $slug ];
+	return $points_types[$slug];
 }
 
 /**
@@ -91,12 +91,12 @@ 
  *
  * @return string|void The value of the setting if it exists, otherwise null.
  */
-function wordpoints_get_points_type_setting( $slug, $setting ) {
+function wordpoints_get_points_type_setting($slug, $setting) {
 
-	$points_type = wordpoints_get_points_type( $slug );
+	$points_type = wordpoints_get_points_type($slug);
 
-	if ( isset( $points_type[ $setting ] ) ) {
-		return $points_type[ $setting ];
+	if (isset($points_type[$setting])) {
+		return $points_type[$setting];
 	}
 }
 
@@ -113,21 +113,21 @@ 
  *
  * @return string|false The slug on success. False on failure.
  */
-function wordpoints_add_points_type( $settings ) {
+function wordpoints_add_points_type($settings) {
 
-	if ( ! is_array( $settings ) || ! isset( $settings['name'] ) ) {
+	if ( ! is_array($settings) || ! isset($settings['name'])) {
 		return false;
 	}
 
 	$slug = $settings['name'];
-	$slug = preg_replace( '/\s+/', '-', $slug );
-	$slug = preg_replace( '/-+/', '-', $slug );
-	$slug = trim( $slug, '-' );
-	$slug = sanitize_key( $slug );
+	$slug = preg_replace('/\s+/', '-', $slug);
+	$slug = preg_replace('/-+/', '-', $slug);
+	$slug = trim($slug, '-');
+	$slug = sanitize_key($slug);
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( empty( $slug ) || isset( $points_types[ $slug ] ) ) {
+	if (empty($slug) || isset($points_types[$slug])) {
 		return false;
 	}
 
@@ -140,9 +140,9 @@ 
 	 * @param string $slug     The slug for this points type.
 	 * @param bool   $is_new   Whether this points type is new, or being updated.
 	 */
-	$points_types[ $slug ] = apply_filters( 'wordpoints_points_settings', $settings, $slug, true );
+	$points_types[$slug] = apply_filters('wordpoints_points_settings', $settings, $slug, true);
 
-	if ( ! wordpoints_update_maybe_network_option( 'wordpoints_points_types', $points_types ) ) {
+	if ( ! wordpoints_update_maybe_network_option('wordpoints_points_types', $points_types)) {
 		return false;
 	}
 
@@ -159,20 +159,20 @@ 
  *
  * @return bool True, or false on failure, or if this points type does not exist.
  */
-function wordpoints_update_points_type( $slug, $settings ) {
+function wordpoints_update_points_type($slug, $settings) {
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( ! is_array( $settings ) || ! isset( $points_types[ $slug ], $settings['name'] ) ) {
+	if ( ! is_array($settings) || ! isset($points_types[$slug], $settings['name'])) {
 		return false;
 	}
 
 	/**
 	 * @see wordpoints_add_points_type()
 	 */
-	$points_types[ $slug ] = apply_filters( 'wordpoints_points_settings', $settings, $slug, false );
+	$points_types[$slug] = apply_filters('wordpoints_points_settings', $settings, $slug, false);
 
-	return wordpoints_update_maybe_network_option( 'wordpoints_points_types', $points_types );
+	return wordpoints_update_maybe_network_option('wordpoints_points_types', $points_types);
 }
 
 /**
@@ -187,11 +187,11 @@ 
  *
  * @return bool Whether the points type was deleted successfully.
  */
-function wordpoints_delete_points_type( $slug ) {
+function wordpoints_delete_points_type($slug) {
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( ! isset( $points_types[ $slug ] ) ) {
+	if ( ! isset($points_types[$slug])) {
 		return false;
 	}
 
@@ -203,48 +203,48 @@ 
 	 * @param string $slug      The slug of the points type being deleted.
 	 * @param array  $settings The settings of the points type being deleted.
 	 */
-	do_action( 'wordpoints_delete_points_type', $slug, $points_types[ $slug ] );
+	do_action('wordpoints_delete_points_type', $slug, $points_types[$slug]);
 
-	$meta_key = wordpoints_get_points_user_meta_key( $slug );
+	$meta_key = wordpoints_get_points_user_meta_key($slug);
 
 	global $wpdb;
 
 	// Delete log meta for this points type.
 	$query = new WordPoints_Points_Logs_Query(
-		array( 'field' => 'id', 'points_type' => $slug )
+		array('field' => 'id', 'points_type' => $slug)
 	);
 
-	$log_ids = $query->get( 'col' );
+	$log_ids = $query->get('col');
 
-	foreach ( $log_ids as $log_id ) {
-		wordpoints_points_log_delete_all_metadata( $log_id );
+	foreach ($log_ids as $log_id) {
+		wordpoints_points_log_delete_all_metadata($log_id);
 	}
 
 	// Delete logs for this points type.
-	$wpdb->delete( $wpdb->wordpoints_points_logs, array( 'points_type' => $slug ) );
+	$wpdb->delete($wpdb->wordpoints_points_logs, array('points_type' => $slug));
 
-	wordpoints_flush_points_logs_caches( array( 'points_type' => $slug ) );
+	wordpoints_flush_points_logs_caches(array('points_type' => $slug));
 
 	// Delete all user points of this type.
-	delete_metadata( 'user', 0, wp_slash( $meta_key ), '', true );
+	delete_metadata('user', 0, wp_slash($meta_key), '', true);
 
 	// Delete hooks associated with this points type.
 	$points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
 
-	unset( $points_types_hooks[ $slug ] );
+	unset($points_types_hooks[$slug]);
 
-	WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
+	WordPoints_Points_Hooks::save_points_types_hooks($points_types_hooks);
 
 	// Delete reactions associated with this points type.
-	foreach ( wordpoints_hooks()->get_reaction_stores( 'points' ) as $reaction_store ) {
-		foreach ( $reaction_store->get_reactions() as $reaction ) {
-			if ( $slug === $reaction->get_meta( 'points_type' ) ) {
-				$reaction_store->delete_reaction( $reaction->get_id() );
+	foreach (wordpoints_hooks()->get_reaction_stores('points') as $reaction_store) {
+		foreach ($reaction_store->get_reactions() as $reaction) {
+			if ($slug === $reaction->get_meta('points_type')) {
+				$reaction_store->delete_reaction($reaction->get_id());
 			}
 		}
 	}
 
-	unset( $points_types[ $slug ] );
+	unset($points_types[$slug]);
 
 	wordpoints_update_maybe_network_option(
 		'wordpoints_points_types'
@@ -277,19 +277,19 @@ 
  *
  * @return string|false The user meta meta_key for a points type, or false.
  */
-function wordpoints_get_points_user_meta_key( $points_type ) {
+function wordpoints_get_points_user_meta_key($points_type) {
 
-	if ( ! wordpoints_is_points_type( $points_type ) ) {
+	if ( ! wordpoints_is_points_type($points_type)) {
 		return false;
 	}
 
-	$setting = wordpoints_get_points_type_setting( $points_type, 'meta_key' );
+	$setting = wordpoints_get_points_type_setting($points_type, 'meta_key');
 
-	if ( ! empty( $setting ) ) {
+	if ( ! empty($setting)) {
 
 		$meta_key = $setting;
 
-	} elseif ( ! is_multisite() || is_wordpoints_network_active() ) {
+	} elseif ( ! is_multisite() || is_wordpoints_network_active()) {
 
 		$meta_key = "wordpoints_points-{$points_type}";
 
@@ -315,15 +315,15 @@ 
  *
  * @return int|false The user's points, or false on failure.
  */
-function wordpoints_get_points( $user_id, $type ) {
+function wordpoints_get_points($user_id, $type) {
 
-	if ( ! wordpoints_posint( $user_id ) || ! wordpoints_is_points_type( $type ) ) {
+	if ( ! wordpoints_posint($user_id) || ! wordpoints_is_points_type($type)) {
 		return false;
 	}
 
-	$points = get_user_meta( $user_id, wordpoints_get_points_user_meta_key( $type ), true );
+	$points = get_user_meta($user_id, wordpoints_get_points_user_meta_key($type), true);
 
-	return (int) wordpoints_int( $points );
+	return (int) wordpoints_int($points);
 }
 
 /**
@@ -365,9 +365,9 @@ 
  *
  * @return int|false The minimum for this type of points. False if $type is bad.
  */
-function wordpoints_get_points_minimum( $type ) {
+function wordpoints_get_points_minimum($type) {
 
-	if ( ! wordpoints_is_points_type( $type ) ) {
+	if ( ! wordpoints_is_points_type($type)) {
 		return false;
 	}
 
@@ -379,7 +379,7 @@ 
 	 * @param int    $minimum The minimum number of points.
 	 * @param string $type    The points type slug.
 	 */
-	return apply_filters( 'wordpoints_points_minimum', 0, $type );
+	return apply_filters('wordpoints_points_minimum', 0, $type);
 }
 
 /**
@@ -400,12 +400,12 @@ 
  *
  * @return string The integer value of $points formatted for display.
  */
-function wordpoints_format_points( $points, $type, $context ) {
+function wordpoints_format_points($points, $type, $context) {
 
 	$_points = $points;
-	wordpoints_int( $_points );
+	wordpoints_int($_points);
 
-	if ( false === $_points || ! wordpoints_is_points_type( $type ) ) {
+	if (false === $_points || ! wordpoints_is_points_type($type)) {
 		return (string) $points;
 	}
 
@@ -419,7 +419,7 @@ 
 	 * @param string $type      The type of points.
 	 * @param string $context   The context in which the points will be displayed.
 	 */
-	return apply_filters( 'wordpoints_format_points', $_points, $_points, $type, $context );
+	return apply_filters('wordpoints_format_points', $_points, $_points, $type, $context);
 }
 
 /**
@@ -436,15 +436,15 @@ 
  *
  * @return string|false The user's points formatted for display, or false on failure.
  */
-function wordpoints_get_formatted_points( $user_id, $type, $context ) {
+function wordpoints_get_formatted_points($user_id, $type, $context) {
 
-	$points = wordpoints_get_points( $user_id, $type );
+	$points = wordpoints_get_points($user_id, $type);
 
-	if ( false === $points ) {
+	if (false === $points) {
 		return false;
 	}
 
-	return wordpoints_format_points( $points, $type, $context );
+	return wordpoints_format_points($points, $type, $context);
 }
 
 /**
@@ -458,9 +458,9 @@ 
  * @param string $type    The type of points to display.
  * @param string $context The context in which the points will be displayed.
  */
-function wordpoints_display_points( $user_id, $type, $context ) {
+function wordpoints_display_points($user_id, $type, $context) {
 
-	echo wordpoints_get_formatted_points( $user_id, $type, $context ); // XSS OK, WPCS
+	echo wordpoints_get_formatted_points($user_id, $type, $context); // XSS OK, WPCS
 }
 
 /**
@@ -483,21 +483,21 @@ 
  *
  * @return int|false False on failure.
  */
-function wordpoints_get_points_above_minimum( $user_id, $type ) {
+function wordpoints_get_points_above_minimum($user_id, $type) {
 
-	$minimum = wordpoints_get_points_minimum( $type );
+	$minimum = wordpoints_get_points_minimum($type);
 
-	if ( false === $minimum ) {
+	if (false === $minimum) {
 		return false;
 	}
 
-	$points = wordpoints_get_points( $user_id, $type );
+	$points = wordpoints_get_points($user_id, $type);
 
-	if ( false === $points ) {
+	if (false === $points) {
 		return false;
 	}
 
-	return max( 0, $points - $minimum );
+	return max(0, $points - $minimum);
 }
 
 /**
@@ -519,19 +519,19 @@ 
  *
  * @return bool Whether the transaction was successful.
  */
-function wordpoints_set_points( $user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '' ) {
+function wordpoints_set_points($user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '') {
 
-	if ( false === wordpoints_int( $points ) ) {
+	if (false === wordpoints_int($points)) {
 		return false;
 	}
 
-	$current = wordpoints_get_points( $user_id, $points_type );
+	$current = wordpoints_get_points($user_id, $points_type);
 
-	if ( false === $current ) {
+	if (false === $current) {
 		return false;
 	}
 
-	return wordpoints_alter_points( $user_id, $points - $current, $points_type, $log_type, $meta, $log_text );
+	return wordpoints_alter_points($user_id, $points - $current, $points_type, $log_type, $meta, $log_text);
 }
 
 /**
@@ -574,13 +574,13 @@ 
  * @return int|bool On success, the log ID if the transaction is logged, or true if
  *                  it is not. False on failure.
  */
-function wordpoints_alter_points( $user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '' ) {
+function wordpoints_alter_points($user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '') {
 
 	if (
-		! wordpoints_posint( $user_id )
-		|| ! wordpoints_int( $points )
-		|| ! wordpoints_is_points_type( $points_type )
-		|| empty( $log_type )
+		! wordpoints_posint($user_id)
+		|| ! wordpoints_int($points)
+		|| ! wordpoints_is_points_type($points_type)
+		|| empty($log_type)
 	) {
 		return false;
 	}
@@ -604,29 +604,29 @@ 
 	 * @param string $log_type    The type of transaction.
 	 * @param array  $meta        Metadata for the transaction.
 	 */
-	$points = apply_filters( 'wordpoints_alter_points', $points, $points_type, $user_id, $log_type, $meta );
+	$points = apply_filters('wordpoints_alter_points', $points, $points_type, $user_id, $log_type, $meta);
 
-	if ( 0 === wordpoints_int( $points ) ) {
+	if (0 === wordpoints_int($points)) {
 		return true;
-	} elseif ( false === $points ) {
+	} elseif (false === $points) {
 		return false;
 	}
 
 	// Get the current points so we can check this won't go below the minimum.
-	$current_points = wordpoints_get_points( $user_id, $points_type );
-	$minimum = wordpoints_get_points_minimum( $points_type );
+	$current_points = wordpoints_get_points($user_id, $points_type);
+	$minimum = wordpoints_get_points_minimum($points_type);
 
-	if ( ( $current_points + $points ) < $minimum ) {
+	if (($current_points + $points) < $minimum) {
 
 		// The total was less than the minimum, set the number to the minimum.
 		$points = $minimum - $current_points;
 	}
 
-	$meta_key = wordpoints_get_points_user_meta_key( $points_type );
+	$meta_key = wordpoints_get_points_user_meta_key($points_type);
 
-	if ( '' === get_user_meta( $user_id, $meta_key, true ) ) {
+	if ('' === get_user_meta($user_id, $meta_key, true)) {
 
-		$result = add_user_meta( $user_id, wp_slash( $meta_key ), (int) $points, true );
+		$result = add_user_meta($user_id, wp_slash($meta_key), (int) $points, true);
 
 	} else {
 
@@ -645,10 +645,10 @@ 
 			)
 		);
 
-		wp_cache_delete( $user_id, 'user_meta' );
+		wp_cache_delete($user_id, 'user_meta');
 	}
 
-	if ( ! $result ) {
+	if ( ! $result) {
 		return false;
 	}
 
@@ -662,15 +662,15 @@ 
 	 * @param string $log_type        The type of transaction.
 	 * @param array  $meta            The metadata for this transaction.
 	 */
-	$log_transaction = apply_filters( 'wordpoints_points_log', true, $user_id, $points, $points_type, $log_type, $meta );
+	$log_transaction = apply_filters('wordpoints_points_log', true, $user_id, $points, $points_type, $log_type, $meta);
 
 	$log_id = false;
-	if ( $log_transaction ) {
+	if ($log_transaction) {
 
-		$log_text = wordpoints_render_points_log_text( $user_id, $points, $points_type, $log_type, $meta, $log_text );
+		$log_text = wordpoints_render_points_log_text($user_id, $points, $points_type, $log_type, $meta, $log_text);
 
-		if ( 'utf8' === $wpdb->get_col_charset( $wpdb->wordpoints_points_logs, 'text' ) ) {
-			$log_text = wp_encode_emoji( $log_text );
+		if ('utf8' === $wpdb->get_col_charset($wpdb->wordpoints_points_logs, 'text')) {
+			$log_text = wp_encode_emoji($log_text);
 		}
 
 		$result = $wpdb->insert(
@@ -681,20 +681,20 @@ 
 				'points_type' => $points_type,
 				'log_type'    => $log_type,
 				'text'        => $log_text,
-				'date'        => current_time( 'mysql', 1 ),
+				'date'        => current_time('mysql', 1),
 				'site_id'     => $wpdb->siteid,
 				'blog_id'     => $wpdb->blogid,
 			),
-			array( '%d', '%d', '%s', '%s', '%s', '%s', '%d', '%d' )
+			array('%d', '%d', '%s', '%s', '%s', '%s', '%d', '%d')
 		);
 
-		if ( false !== $result ) {
+		if (false !== $result) {
 
 			$log_id = (int) $wpdb->insert_id;
 
-			foreach ( $meta as $meta_key => $meta_value ) {
+			foreach ($meta as $meta_key => $meta_value) {
 
-				wordpoints_add_points_log_meta( $log_id, $meta_key, $meta_value );
+				wordpoints_add_points_log_meta($log_id, $meta_key, $meta_value);
 			}
 
 			/**
@@ -710,7 +710,7 @@ 
 			 * @param array  $meta        Metadata for the transaction.
 			 * @param int    $log_id      The ID of the transaction log entry.
 			 */
-			do_action( 'wordpoints_points_log', $user_id, $points, $points_type, $log_type, $meta, $log_id );
+			do_action('wordpoints_points_log', $user_id, $points, $points_type, $log_type, $meta, $log_id);
 		}
 
 	}  // End if ( $log_transaction ).
@@ -728,9 +728,9 @@ 
 	 * @param array  $meta        Metadata for the transaction.
 	 * @param int|false $log_id   The ID of the transaction log, or false if not logged.
 	 */
-	do_action( 'wordpoints_points_altered', $user_id, $points, $points_type, $log_type, $meta, $log_id );
+	do_action('wordpoints_points_altered', $user_id, $points, $points_type, $log_type, $meta, $log_id);
 
-	if ( $log_id ) {
+	if ($log_id) {
 		return $log_id;
 	} else {
 		return true;
@@ -756,9 +756,9 @@ 
  *
  * @return bool Whether the points were added successfully.
  */
-function wordpoints_add_points( $user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '' ) {
+function wordpoints_add_points($user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '') {
 
-	return wordpoints_alter_points( $user_id, wordpoints_posint( $points ), $points_type, $log_type, $meta, $log_text );
+	return wordpoints_alter_points($user_id, wordpoints_posint($points), $points_type, $log_type, $meta, $log_text);
 }
 
 /**
@@ -779,9 +779,9 @@ 
  *
  * @return bool Whether the points were subtracted successfully.
  */
-function wordpoints_subtract_points( $user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '' ) {
+function wordpoints_subtract_points($user_id, $points, $points_type, $log_type, $meta = array(), $log_text = '') {
 
-	return wordpoints_alter_points( $user_id, -wordpoints_posint( $points ), $points_type, $log_type, $meta, $log_text );
+	return wordpoints_alter_points($user_id, -wordpoints_posint($points), $points_type, $log_type, $meta, $log_text);
 }
 
 /**
@@ -793,9 +793,9 @@ 
  *
  * @return string The corrected column name.
  */
-function _wordpoints_points_log_meta_column( $column ) {
+function _wordpoints_points_log_meta_column($column) {
 
-	if ( 'wordpoints_points_log_id' === $column ) {
+	if ('wordpoints_points_log_id' === $column) {
 		$column = 'log_id';
 	}
 
@@ -820,23 +820,23 @@ 
  *
  * @return bool Whether the metadata was added successfully.
  */
-function wordpoints_add_points_log_meta( $log_id, $meta_key, $meta_value, $unique = false ) {
+function wordpoints_add_points_log_meta($log_id, $meta_key, $meta_value, $unique = false) {
 
 	global $wpdb;
 
-	add_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	add_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 	$wpdb->wordpoints_points_logmeta = $wpdb->wordpoints_points_log_meta;
 
 	$result = add_metadata(
 		'wordpoints_points_log'
 		, $log_id
-		, wp_slash( $meta_key )
-		, wp_slash( $meta_value )
+		, wp_slash($meta_key)
+		, wp_slash($meta_value)
 		, $unique
 	);
 
-	unset( $wpdb->wordpoints_points_logmeta );
-	remove_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	unset($wpdb->wordpoints_points_logmeta);
+	remove_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 
 	return $result;
 }
@@ -855,15 +855,15 @@ 
  *
  * @return mixed The meta key, or null on failure.
  */
-function wordpoints_get_points_log_meta( $log_id, $meta_key = '', $single = false ) {
+function wordpoints_get_points_log_meta($log_id, $meta_key = '', $single = false) {
 
 	global $wpdb;
 
-	add_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	add_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 	$wpdb->wordpoints_points_logmeta = $wpdb->wordpoints_points_log_meta;
-	$result = get_metadata( 'wordpoints_points_log', $log_id, $meta_key, $single );
-	unset( $wpdb->wordpoints_points_logmeta );
-	remove_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	$result = get_metadata('wordpoints_points_log', $log_id, $meta_key, $single);
+	unset($wpdb->wordpoints_points_logmeta);
+	remove_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 
 	return $result;
 }
@@ -883,23 +883,23 @@ 
  *
  * @return bool Whether any rows were updated.
  */
-function wordpoints_update_points_log_meta( $log_id, $meta_key, $meta_value, $previous = null ) {
+function wordpoints_update_points_log_meta($log_id, $meta_key, $meta_value, $previous = null) {
 
 	global $wpdb;
 
-	add_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	add_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 	$wpdb->wordpoints_points_logmeta = $wpdb->wordpoints_points_log_meta;
 
 	$result = update_metadata(
 		'wordpoints_points_log'
 		, $log_id
-		, wp_slash( $meta_key )
-		, wp_slash( $meta_value )
+		, wp_slash($meta_key)
+		, wp_slash($meta_value)
 		, $previous
 	);
 
-	unset( $wpdb->wordpoints_points_logmeta );
-	remove_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	unset($wpdb->wordpoints_points_logmeta);
+	remove_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 
 	return $result;
 }
@@ -920,23 +920,23 @@ 
  *
  * @return bool Whether any rows where deleted.
  */
-function wordpoints_delete_points_log_meta( $log_id, $meta_key = '', $meta_value = null, $delete_all = false ) {
+function wordpoints_delete_points_log_meta($log_id, $meta_key = '', $meta_value = null, $delete_all = false) {
 
 	global $wpdb;
 
-	add_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	add_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 	$wpdb->wordpoints_points_logmeta = $wpdb->wordpoints_points_log_meta;
 
 	$result = delete_metadata(
 		'wordpoints_points_log'
 		, $log_id
-		, wp_slash( $meta_key )
-		, wp_slash( $meta_value )
+		, wp_slash($meta_key)
+		, wp_slash($meta_value)
 		, $delete_all
 	);
 
-	unset( $wpdb->wordpoints_points_logmeta );
-	remove_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	unset($wpdb->wordpoints_points_logmeta);
+	remove_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 
 	return $result;
 }
@@ -948,7 +948,7 @@ 
  *
  * @param int $log_id The ID of the points log whose metadata to delete.
  */
-function wordpoints_points_log_delete_all_metadata( $log_id ) {
+function wordpoints_points_log_delete_all_metadata($log_id) {
 
 	global $wpdb;
 
@@ -963,15 +963,15 @@ 
 		)
 	); // WPCS: cache pass.
 
-	add_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	add_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 	$wpdb->wordpoints_points_logmeta = $wpdb->wordpoints_points_log_meta;
 
-	foreach ( $meta_ids as $mid ) {
-		delete_metadata_by_mid( 'wordpoints_points_log', $mid );
+	foreach ($meta_ids as $mid) {
+		delete_metadata_by_mid('wordpoints_points_log', $mid);
 	}
 
-	unset( $wpdb->wordpoints_points_logmeta );
-	remove_filter( 'sanitize_key', '_wordpoints_points_log_meta_column' );
+	unset($wpdb->wordpoints_points_logmeta);
+	remove_filter('sanitize_key', '_wordpoints_points_log_meta_column');
 }
 
 /**
@@ -987,7 +987,7 @@ 
 		'wordpoints_default_points_type'
 	);
 
-	if ( ! wordpoints_is_points_type( $points_type ) ) {
+	if ( ! wordpoints_is_points_type($points_type)) {
 		return false;
 	}
 
@@ -1008,7 +1008,7 @@ 
  *
  * @return string The log text.
  */
-function wordpoints_render_points_log_text( $user_id, $points, $points_type, $log_type, $meta, $default_text = '' ) {
+function wordpoints_render_points_log_text($user_id, $points, $points_type, $log_type, $meta, $default_text = '') {
 
 	/**
 	 * The text for a points log entry.
@@ -1020,10 +1020,10 @@ 
 	 * @param string $log_type    The type of transaction being logged.
 	 * @param array  $meta        The metadata for this transaction.
 	 */
-	$text = apply_filters( "wordpoints_points_log-{$log_type}", $default_text, $user_id, $points, $points_type, $log_type, $meta );
+	$text = apply_filters("wordpoints_points_log-{$log_type}", $default_text, $user_id, $points, $points_type, $log_type, $meta);
 
-	if ( empty( $text ) ) {
-		$text = _x( '(no description)', 'points log', 'wordpoints' );
+	if (empty($text)) {
+		$text = _x('(no description)', 'points log', 'wordpoints');
 	}
 
 	return $text;
@@ -1039,32 +1039,32 @@ 
  *
  * @return void
  */
-function wordpoints_regenerate_points_logs( $logs ) {
+function wordpoints_regenerate_points_logs($logs) {
 
-	if ( empty( $logs ) || ! is_array( $logs ) ) {
+	if (empty($logs) || ! is_array($logs)) {
 		return;
 	}
 
-	if ( ! is_object( current( $logs ) ) ) {
+	if ( ! is_object(current($logs))) {
 
-		_deprecated_argument( __FUNCTION__, '1.6.0', 'The first parameter should be an array of log objects, not log IDs.' );
+		_deprecated_argument(__FUNCTION__, '1.6.0', 'The first parameter should be an array of log objects, not log IDs.');
 
-		$logs = new WordPoints_Points_Logs_Query( array( 'id__in' => $logs ) );
+		$logs = new WordPoints_Points_Logs_Query(array('id__in' => $logs));
 		$logs = $logs->get();
 
-		if ( ! is_array( $logs ) ) {
+		if ( ! is_array($logs)) {
 			return;
 		}
 	}
 
 	global $wpdb;
 
-	$flushed = array( 'points_types' => array(), 'user_ids' => array() );
+	$flushed = array('points_types' => array(), 'user_ids' => array());
 
-	foreach ( $logs as $log ) {
+	foreach ($logs as $log) {
 
-		$meta = wordpoints_get_points_log_meta( $log->id );
-		$meta = wp_list_pluck( $meta, 0 );
+		$meta = wordpoints_get_points_log_meta($log->id);
+		$meta = wp_list_pluck($meta, 0);
 
 		$new_log_text = wordpoints_render_points_log_text(
 			$log->user_id
@@ -1074,31 +1074,31 @@ 
 			, $meta
 		);
 
-		if ( $new_log_text !== $log->text ) {
+		if ($new_log_text !== $log->text) {
 
-			if ( ! isset( $is_utf8 ) ) {
-				$is_utf8 = 'utf8' === $wpdb->get_col_charset( $wpdb->wordpoints_points_logs, 'text' );
+			if ( ! isset($is_utf8)) {
+				$is_utf8 = 'utf8' === $wpdb->get_col_charset($wpdb->wordpoints_points_logs, 'text');
 			}
 
-			if ( $is_utf8 ) {
-				$new_log_text = wp_encode_emoji( $new_log_text );
+			if ($is_utf8) {
+				$new_log_text = wp_encode_emoji($new_log_text);
 			}
 
 			$wpdb->update(
 				$wpdb->wordpoints_points_logs
-				, array( 'text' => $new_log_text )
-				, array( 'id' => $log->id )
-				, array( '%s' )
-				, array( '%d' )
+				, array('text' => $new_log_text)
+				, array('id' => $log->id)
+				, array('%s')
+				, array('%d')
 			);
 
-			if ( ! isset( $flushed['points_types'][ $log->points_type ], $flushed['user_ids'][ $log->user_id ] ) ) {
+			if ( ! isset($flushed['points_types'][$log->points_type], $flushed['user_ids'][$log->user_id])) {
 				wordpoints_flush_points_logs_caches(
-					array( 'user_id' => $log->user_id, 'points_type' => $log->points_type )
+					array('user_id' => $log->user_id, 'points_type' => $log->points_type)
 				);
 
-				$flushed['points_types'][ $log->points_type ] = true;
-				$flushed['user_ids'][ $log->user_id ] = true;
+				$flushed['points_types'][$log->points_type] = true;
+				$flushed['user_ids'][$log->user_id] = true;
 			}
 		}
 
@@ -1118,35 +1118,35 @@ 
  *
  * @return int[] The IDs of the users with the most points.
  */
-function wordpoints_points_get_top_users( $num_users, $points_type ) {
+function wordpoints_points_get_top_users($num_users, $points_type) {
 
-	if ( ! wordpoints_posint( $num_users ) || ! wordpoints_is_points_type( $points_type ) ) {
+	if ( ! wordpoints_posint($num_users) || ! wordpoints_is_points_type($points_type)) {
 		return;
 	}
 
-	$cache = wp_cache_get( $points_type, 'wordpoints_points_top_users' );
+	$cache = wp_cache_get($points_type, 'wordpoints_points_top_users');
 
-	if ( ! is_array( $cache ) ) {
-		$cache = array( 'is_max' => false, 'top_users' => array() );
+	if ( ! is_array($cache)) {
+		$cache = array('is_max' => false, 'top_users' => array());
 	}
 
-	$cached_users = count( $cache['top_users'] );
+	$cached_users = count($cache['top_users']);
 
-	if ( $num_users > $cached_users && ! $cache['is_max'] ) {
+	if ($num_users > $cached_users && ! $cache['is_max']) {
 
 		global $wpdb;
 
-		$excluded = wordpoints_get_excluded_users( 'top_users' );
+		$excluded = wordpoints_get_excluded_users('top_users');
 
 		$exclude_users = '';
-		if ( ! empty( $excluded ) ) {
-			$exclude_users = 'WHERE `ID` NOT IN (' . wordpoints_prepare__in( $excluded, '%d' ) . ')';
+		if ( ! empty($excluded)) {
+			$exclude_users = 'WHERE `ID` NOT IN (' . wordpoints_prepare__in($excluded, '%d') . ')';
 		}
 
 		$multisite_join = '';
-		if ( is_multisite() && ! is_wordpoints_network_active() ) {
+		if (is_multisite() && ! is_wordpoints_network_active()) {
 
-			$prefix = $wpdb->get_blog_prefix( get_current_blog_id() );
+			$prefix = $wpdb->get_blog_prefix(get_current_blog_id());
 
 			$multisite_join = "
 					INNER JOIN `{$wpdb->usermeta}` AS `cap`
@@ -1171,27 +1171,27 @@ 
 					ORDER BY COALESCE(CONVERT(`meta`.`meta_value`, SIGNED INTEGER), 0) DESC
 					LIMIT %d,%d
 				",
-				wordpoints_get_points_user_meta_key( $points_type ),
+				wordpoints_get_points_user_meta_key($points_type),
 				$cached_users,
 				$num_users
 			)
 		);
 
-		if ( ! is_array( $top_users ) ) {
+		if ( ! is_array($top_users)) {
 			return array();
 		}
 
-		$cache['top_users'] = array_merge( $cache['top_users'], $top_users );
+		$cache['top_users'] = array_merge($cache['top_users'], $top_users);
 
-		if ( count( $cache['top_users'] ) < $num_users ) {
+		if (count($cache['top_users']) < $num_users) {
 			$cache['is_max'] = true;
 		}
 
-		wp_cache_set( $points_type, $cache, 'wordpoints_points_top_users' );
+		wp_cache_set($points_type, $cache, 'wordpoints_points_top_users');
 
 	} // End if ( not cached ).
 
-	return array_slice( $cache['top_users'], 0, $num_users );
+	return array_slice($cache['top_users'], 0, $num_users);
 }
 
 /**
@@ -1203,25 +1203,25 @@ 
  * @param string $points_type The type of points.
  * @param string $context     The context in which the table is being displayed.
  */
-function wordpoints_points_show_top_users( $num_users, $points_type, $context = 'default' ) {
+function wordpoints_points_show_top_users($num_users, $points_type, $context = 'default') {
 
-	wp_enqueue_style( 'wordpoints-top-users' );
+	wp_enqueue_style('wordpoints-top-users');
 
-	$top_users = wordpoints_points_get_top_users( $num_users, $points_type );
+	$top_users = wordpoints_points_get_top_users($num_users, $points_type);
 
-	if ( ! $top_users ) {
+	if ( ! $top_users) {
 		return;
 	}
 
 	$column_headers = array(
-		'position' => _x( 'Position', 'top users table heading', 'wordpoints' ),
-		'user'     => _x( 'User', 'top users table heading', 'wordpoints' ),
-		'points'   => _x( 'Points', 'top users table heading', 'wordpoints' ),
+		'position' => _x('Position', 'top users table heading', 'wordpoints'),
+		'user'     => _x('User', 'top users table heading', 'wordpoints'),
+		'points'   => _x('Points', 'top users table heading', 'wordpoints'),
 	);
 
-	$points_type_name = wordpoints_get_points_type_setting( $points_type, 'name' );
+	$points_type_name = wordpoints_get_points_type_setting($points_type, 'name');
 
-	if ( ! empty( $points_type_name ) ) {
+	if ( ! empty($points_type_name)) {
 		$column_headers['points'] = $points_type_name;
 	}
 
@@ -1237,18 +1237,18 @@ 
 	$extra_classes = apply_filters(
 		'wordpoints_points_top_users_table_extra_classes'
 		, array()
-		, compact( 'num_users', 'points_type', 'context' )
+		, compact('num_users', 'points_type', 'context')
 		, $top_users
 	);
 
 	?>
 
-	<table class="wordpoints-points-top-users <?php echo esc_attr( implode( ' ', $extra_classes ) ); ?>">
+	<table class="wordpoints-points-top-users <?php echo esc_attr(implode(' ', $extra_classes)); ?>">
 		<thead>
 			<tr>
-				<th scope="col"><?php echo esc_html( $column_headers['position'] ); ?></th>
-				<th scope="col"><?php echo esc_html( $column_headers['user'] ); ?></th>
-				<th scope="col"><?php echo esc_html( $column_headers['points'] ); ?></th>
+				<th scope="col"><?php echo esc_html($column_headers['position']); ?></th>
+				<th scope="col"><?php echo esc_html($column_headers['user']); ?></th>
+				<th scope="col"><?php echo esc_html($column_headers['points']); ?></th>
 			</tr>
 		</thead>
 		<tbody>
@@ -1256,16 +1256,16 @@ 
 
 			$position = 1;
 
-			foreach ( $top_users as $user_id ) {
+			foreach ($top_users as $user_id) {
 
-				$user = get_userdata( $user_id );
+				$user = get_userdata($user_id);
 
 				?>
 
 				<tr class="top-<?php echo (int) $position; ?>">
-					<td><?php echo esc_html( number_format_i18n( $position ) ); ?></td>
+					<td><?php echo esc_html(number_format_i18n($position)); ?></td>
 					<td>
-						<?php echo get_avatar( $user_id, 32 ); ?>
+						<?php echo get_avatar($user_id, 32); ?>
 						<?php
 
 						$name = sanitize_user_field(
@@ -1287,13 +1287,13 @@ 
 						 * @param string $points_type The points type the table is for.
 						 * @param string $context     The context in which the table is being displayed.
 						 */
-						$name = apply_filters( 'wordpoints_points_top_users_username', $name, $user_id, $points_type, "top_users_{$context}" );
+						$name = apply_filters('wordpoints_points_top_users_username', $name, $user_id, $points_type, "top_users_{$context}");
 
-						echo wp_kses( $name, 'wordpoints_top_users_username' );
+						echo wp_kses($name, 'wordpoints_top_users_username');
 
 						?>
 					</td>
-					<td><?php wordpoints_display_points( $user_id, $points_type, "top_users_{$context}" ); ?></td>
+					<td><?php wordpoints_display_points($user_id, $points_type, "top_users_{$context}"); ?></td>
 				</tr>
 
 				<?php
@@ -1306,9 +1306,9 @@ 
 		</tbody>
 		<tfoot>
 		<tr>
-			<th scope="col"><?php echo esc_html( $column_headers['position'] ); ?></th>
-			<th scope="col"><?php echo esc_html( $column_headers['user'] ); ?></th>
-			<th scope="col"><?php echo esc_html( $column_headers['points'] ); ?></th>
+			<th scope="col"><?php echo esc_html($column_headers['position']); ?></th>
+			<th scope="col"><?php echo esc_html($column_headers['user']); ?></th>
+			<th scope="col"><?php echo esc_html($column_headers['points']); ?></th>
 		</tr>
 		</tfoot>
 	</table>
@@ -1328,9 +1328,9 @@ 
  * @param int    $points      The number of points. Not used.
  * @param string $points_type The type of points being awarded.
  */
-function wordpoints_clean_points_top_users_cache( $user_id, $points, $points_type ) {
+function wordpoints_clean_points_top_users_cache($user_id, $points, $points_type) {
 
-	wp_cache_delete( $points_type, 'wordpoints_points_top_users' );
+	wp_cache_delete($points_type, 'wordpoints_points_top_users');
 }
 
 /**
@@ -1342,16 +1342,16 @@ 
  */
 function wordpoints_clean_points_top_users_cache_user_register() {
 
-	foreach ( wordpoints_get_points_types() as $slug => $unused ) {
+	foreach (wordpoints_get_points_types() as $slug => $unused) {
 
-		$cache = wp_cache_get( $slug, 'wordpoints_points_top_users' );
+		$cache = wp_cache_get($slug, 'wordpoints_points_top_users');
 
 		// If there aren't fewer users than the cache holds, we don't need to clear it.
-		if ( ! is_array( $cache ) || ! $cache['is_max'] ) {
+		if ( ! is_array($cache) || ! $cache['is_max']) {
 			continue;
 		}
 
-		wp_cache_delete( $slug, 'wordpoints_points_top_users' );
+		wp_cache_delete($slug, 'wordpoints_points_top_users');
 	}
 }
 
@@ -1365,18 +1365,18 @@ 
  *
  * @param int $user_id The ID of the user who was deleted.
  */
-function wordpoints_clean_points_top_users_cache_user_deleted( $user_id ) {
+function wordpoints_clean_points_top_users_cache_user_deleted($user_id) {
 
-	foreach ( wordpoints_get_points_types() as $slug => $unused ) {
+	foreach (wordpoints_get_points_types() as $slug => $unused) {
 
-		$cache = wp_cache_get( $slug, 'wordpoints_points_top_users' );
+		$cache = wp_cache_get($slug, 'wordpoints_points_top_users');
 
 		// If this user isn't in the cache, we don't need to clear it.
-		if ( ! is_array( $cache ) || ! in_array( $user_id, $cache['top_users'] ) ) {
+		if ( ! is_array($cache) || ! in_array($user_id, $cache['top_users'])) {
 			continue;
 		}
 
-		wp_cache_delete( $slug, 'wordpoints_points_top_users' );
+		wp_cache_delete($slug, 'wordpoints_points_top_users');
 	}
 }
 
@@ -1389,18 +1389,18 @@ 
  */
 function wordpoints_register_points_hooks() {
 
-	WordPoints_Points_Hooks::register( 'WordPoints_Comment_Received_Points_Hook' );
-	WordPoints_Points_Hooks::register( 'WordPoints_Comment_Points_Hook' );
-	WordPoints_Points_Hooks::register( 'WordPoints_Periodic_Points_Hook' );
-	WordPoints_Points_Hooks::register( 'WordPoints_Post_Points_Hook' );
-	WordPoints_Points_Hooks::register( 'WordPoints_Registration_Points_Hook' );
+	WordPoints_Points_Hooks::register('WordPoints_Comment_Received_Points_Hook');
+	WordPoints_Points_Hooks::register('WordPoints_Comment_Points_Hook');
+	WordPoints_Points_Hooks::register('WordPoints_Periodic_Points_Hook');
+	WordPoints_Points_Hooks::register('WordPoints_Post_Points_Hook');
+	WordPoints_Points_Hooks::register('WordPoints_Registration_Points_Hook');
 
-	if ( get_site_option( 'wordpoints_post_delete_hook_legacy' ) ) {
-		WordPoints_Points_Hooks::register( 'WordPoints_Post_Delete_Points_Hook' );
+	if (get_site_option('wordpoints_post_delete_hook_legacy')) {
+		WordPoints_Points_Hooks::register('WordPoints_Post_Delete_Points_Hook');
 	}
 
-	if ( get_site_option( 'wordpoints_comment_removed_hook_legacy' ) ) {
-		WordPoints_Points_Hooks::register( 'WordPoints_Comment_Removed_Points_Hook' );
+	if (get_site_option('wordpoints_comment_removed_hook_legacy')) {
+		WordPoints_Points_Hooks::register('WordPoints_Comment_Removed_Points_Hook');
 	}
 }
 

src/components/ranks/includes/class-wordpoints-rank-group.php

Doc Comments +1 added lines, -1 removed lines

@@ -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 +86 added lines, -86 removed lines

@@ -52,7 +52,7 @@ 
 	 *
 	 * @type string[] $types
 	 */
-	private $types = array( 'base' => 'base' );
+	private $types = array('base' => 'base');
 
 	/**
 	 * Construct the group with its slug and other data.
@@ -67,7 +67,7 @@ 
 	 *        @type string $description A description of this group.
 	 * }
 	 */
-	public function __construct( $slug, $data ) {
+	public function __construct($slug, $data) {
 
 		$this->slug = $slug;
 		$this->data = $data;
@@ -79,12 +79,12 @@ 
 	 *
 	 * @since 1.7.0
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( isset( $this->$key ) ) {
+		if (isset($this->$key)) {
 			return $this->$key;
-		} elseif ( isset( $this->data[ $key ] ) ) {
-			return $this->data[ $key ];
+		} elseif (isset($this->data[$key])) {
+			return $this->data[$key];
 		}
 	}
 
@@ -120,13 +120,13 @@ 
 	 *
 	 * @return bool Whether the rank type was registered for this group.
 	 */
-	public function has_type( $type ) {
+	public function has_type($type) {
 
-		if ( ! WordPoints_Rank_Types::is_type_registered( $type ) ) {
+		if ( ! WordPoints_Rank_Types::is_type_registered($type)) {
 			return false;
 		}
 
-		return isset( $this->types[ $type ] );
+		return isset($this->types[$type]);
 	}
 
 	/**
@@ -138,17 +138,17 @@ 
 	 *
 	 * @return bool Whether the type was added successfully.
 	 */
-	public function add_type( $type ) {
+	public function add_type($type) {
 
-		if ( ! WordPoints_Rank_Types::is_type_registered( $type ) ) {
+		if ( ! WordPoints_Rank_Types::is_type_registered($type)) {
 			return false;
 		}
 
-		if ( isset( $this->types[ $type ] ) ) {
+		if (isset($this->types[$type])) {
 			return false;
 		}
 
-		$this->types[ $type ] = $type;
+		$this->types[$type] = $type;
 
 		return true;
 	}
@@ -164,21 +164,21 @@ 
 	 *
 	 * @return bool Whether the type was removed successfully.
 	 */
-	public function remove_type( $type ) {
+	public function remove_type($type) {
 
-		if ( 'base' === $type ) {
+		if ('base' === $type) {
 			return false;
 		}
 
-		if ( ! WordPoints_Rank_Types::is_type_registered( $type ) ) {
+		if ( ! WordPoints_Rank_Types::is_type_registered($type)) {
 			return false;
 		}
 
-		if ( ! isset( $this->types[ $type ] ) ) {
+		if ( ! isset($this->types[$type])) {
 			return false;
 		}
 
-		unset( $this->types[ $type ] );
+		unset($this->types[$type]);
 
 		return true;
 	}
@@ -197,30 +197,30 @@ 
 	 *
 	 * @return bool Whether the rank was added successfully.
 	 */
-	public function add_rank( $rank_id, $position ) {
+	public function add_rank($rank_id, $position) {
 
 		if (
-			! wordpoints_posint( $rank_id )
-			|| false === wordpoints_int( $position )
+			! wordpoints_posint($rank_id)
+			|| false === wordpoints_int($position)
 			|| $position < 0
 		) {
 			return false;
 		}
 
 		// Don't allow a rank to be added to this group more than once.
-		if ( false !== $this->get_rank_position( $rank_id ) ) {
+		if (false !== $this->get_rank_position($rank_id)) {
 			return false;
 		}
 
-		$ranks = $this->_insert_rank( $this->get_ranks(), $rank_id, $position );
+		$ranks = $this->_insert_rank($this->get_ranks(), $rank_id, $position);
 
-		if ( ! $this->save_ranks( $ranks ) ) {
+		if ( ! $this->save_ranks($ranks)) {
 			return false;
 		}
 
 		// If there is a rank before this one, check if any of the users who have it
 		// can increase to this new one.
-		$this->_maybe_increase_users_with_previous_rank( $rank_id );
+		$this->_maybe_increase_users_with_previous_rank($rank_id);
 
 		return true;
 	}
@@ -238,23 +238,23 @@ 
 	 *
 	 * @return bool Whether the rank was moved successfully.
 	 */
-	public function move_rank( $rank_id, $position ) {
+	public function move_rank($rank_id, $position) {
 
-		$current_position = $this->get_rank_position( $rank_id );
+		$current_position = $this->get_rank_position($rank_id);
 
-		if ( $current_position === $position ) {
+		if ($current_position === $position) {
 			return false;
 		}
 
 		$ranks = $this->get_ranks();
 
-		unset( $ranks[ $current_position ] );
+		unset($ranks[$current_position]);
 
-		ksort( $ranks );
+		ksort($ranks);
 
-		$ranks = $this->_insert_rank( array_values( $ranks ), $rank_id, $position );
+		$ranks = $this->_insert_rank(array_values($ranks), $rank_id, $position);
 
-		if ( ! $this->save_ranks( $ranks ) ) {
+		if ( ! $this->save_ranks($ranks)) {
 			return false;
 		}
 
@@ -262,12 +262,12 @@ 
 		// rank's position.
 		$this->_move_users_from_rank_to_rank(
 			$rank_id
-			, $this->get_rank( $current_position - 1 )
+			, $this->get_rank($current_position - 1)
 		);
 
 		// The users of the rank previous to this one's new position should maybe be
 		// increased to this rank.
-		$this->_maybe_increase_users_with_previous_rank( $rank_id );
+		$this->_maybe_increase_users_with_previous_rank($rank_id);
 
 		return true;
 	}
@@ -281,45 +281,45 @@ 
 	 *
 	 * @return bool Whether the rank was removed successfully.
 	 */
-	public function remove_rank( $rank_id ) {
+	public function remove_rank($rank_id) {
 
 		global $wpdb;
 
-		$position = $this->get_rank_position( $rank_id );
+		$position = $this->get_rank_position($rank_id);
 
-		if ( false === $position ) {
+		if (false === $position) {
 			return false;
 		}
 
 		$ranks = $this->get_ranks();
 
-		unset( $ranks[ $position ] );
+		unset($ranks[$position]);
 
-		if ( ! $this->save_ranks( $ranks ) ) {
+		if ( ! $this->save_ranks($ranks)) {
 			return false;
 		}
 
 		// Assign the previous rank to users who have this rank.
-		if ( isset( $ranks[ $position - 1 ] ) ) {
+		if (isset($ranks[$position - 1])) {
 
-			$this->_move_users_from_rank_to_rank( $rank_id, $ranks[ $position - 1 ] );
+			$this->_move_users_from_rank_to_rank($rank_id, $ranks[$position - 1]);
 
 		} else {
 
 			// If there is no previous rank, just delete the rows.
 			$wpdb->delete(
 				$wpdb->wordpoints_user_ranks
-				, array( 'rank_id' => $rank_id )
+				, array('rank_id' => $rank_id)
 				, '%d'
 			);
 
-			$group_ranks = wp_cache_get( $this->slug, 'wordpoints_user_ranks' );
-			unset( $group_ranks[ $rank_id ] );
-			wp_cache_set( $this->slug, $group_ranks, 'wordpoints_user_ranks' );
+			$group_ranks = wp_cache_get($this->slug, 'wordpoints_user_ranks');
+			unset($group_ranks[$rank_id]);
+			wp_cache_set($this->slug, $group_ranks, 'wordpoints_user_ranks');
 
-			unset( $group_ranks );
+			unset($group_ranks);
 
-			wp_cache_delete( $rank_id, 'wordpoints_users_with_rank' );
+			wp_cache_delete($rank_id, 'wordpoints_users_with_rank');
 		}
 
 		return true;
@@ -336,17 +336,17 @@ 
 	 *
 	 * @return bool Whether the ranks were saved successfully.
 	 */
-	public function save_ranks( $rank_ids ) {
+	public function save_ranks($rank_ids) {
 
-		if ( count( array_unique( $rank_ids ) ) !== count( $rank_ids ) ) {
+		if (count(array_unique($rank_ids)) !== count($rank_ids)) {
 			return false;
 		}
 
-		ksort( $rank_ids );
+		ksort($rank_ids);
 
-		$rank_ids = array_values( $rank_ids );
+		$rank_ids = array_values($rank_ids);
 
-		return update_option( $this->option_name, $rank_ids );
+		return update_option($this->option_name, $rank_ids);
 	}
 
 	/**
@@ -358,7 +358,7 @@ 
 	 */
 	public function get_ranks() {
 
-		return wordpoints_get_array_option( $this->option_name );
+		return wordpoints_get_array_option($this->option_name);
 	}
 
 	/**
@@ -372,7 +372,7 @@ 
 	 */
 	public function get_base_rank() {
 
-		return $this->get_rank( 0 );
+		return $this->get_rank(0);
 	}
 
 	/**
@@ -384,15 +384,15 @@ 
 	 *
 	 * @return int|false The ID of the rank, or false if not found.
 	 */
-	public function get_rank( $position ) {
+	public function get_rank($position) {
 
 		$ranks = $this->get_ranks();
 
-		if ( ! isset( $ranks[ $position ] ) ) {
+		if ( ! isset($ranks[$position])) {
 			return false;
 		}
 
-		return $ranks[ $position ];
+		return $ranks[$position];
 	}
 
 	/**
@@ -404,10 +404,10 @@ 
 	 *
 	 * @return int|false The rank's position, or false.
 	 */
-	public function get_rank_position( $rank_id ) {
+	public function get_rank_position($rank_id) {
 
 		return array_search(
-			wordpoints_posint( $rank_id )
+			wordpoints_posint($rank_id)
 			, $this->get_ranks()
 			, true
 		);
@@ -428,26 +428,26 @@ 
 	 *
 	 * @return int[] The list of ranks with the rank insterted.
 	 */
-	private function _insert_rank( $ranks, $rank_id, $position ) {
+	private function _insert_rank($ranks, $rank_id, $position) {
 
-		$count = count( $ranks );
+		$count = count($ranks);
 
-		if ( $count === $position ) {
+		if ($count === $position) {
 
-			$ranks[ $position ] = $rank_id;
+			$ranks[$position] = $rank_id;
 
-		} elseif ( $count < $position ) {
+		} elseif ($count < $position) {
 
 			$ranks[] = $rank_id;
 
 		} else {
 
-			$lower_ranks  = array_slice( $ranks, 0, $position, true );
-			$higher_ranks = array_slice( $ranks, $position, null, true );
+			$lower_ranks  = array_slice($ranks, 0, $position, true);
+			$higher_ranks = array_slice($ranks, $position, null, true);
 
 			$ranks = array_merge(
 				$lower_ranks
-				, array( $position => $rank_id )
+				, array($position => $rank_id)
 				, $higher_ranks
 			);
 		}
@@ -463,29 +463,29 @@ 
 	 * @param int $rank_from_id The ID of the rank the users have now.
 	 * @param int $rank_to_id   The ID of the rank to give the users instead.
 	 */
-	private function _move_users_from_rank_to_rank( $rank_from_id, $rank_to_id ) {
+	private function _move_users_from_rank_to_rank($rank_from_id, $rank_to_id) {
 
 		global $wpdb;
 
-		if ( ! wordpoints_posint( $rank_from_id ) || ! wordpoints_posint( $rank_to_id ) ) {
+		if ( ! wordpoints_posint($rank_from_id) || ! wordpoints_posint($rank_to_id)) {
 			return;
 		}
 
-		$group_ranks = wp_cache_get( $this->slug, 'wordpoints_user_ranks' );
+		$group_ranks = wp_cache_get($this->slug, 'wordpoints_user_ranks');
 
-		unset( $group_ranks[ $rank_from_id ], $group_ranks[ $rank_to_id ] );
+		unset($group_ranks[$rank_from_id], $group_ranks[$rank_to_id]);
 
-		wp_cache_set( $this->slug, $group_ranks, 'wordpoints_user_ranks' );
+		wp_cache_set($this->slug, $group_ranks, 'wordpoints_user_ranks');
 
-		unset( $group_ranks );
+		unset($group_ranks);
 
-		wp_cache_delete( $rank_from_id, 'wordpoints_users_with_rank' );
-		wp_cache_delete( $rank_to_id, 'wordpoints_users_with_rank' );
+		wp_cache_delete($rank_from_id, 'wordpoints_users_with_rank');
+		wp_cache_delete($rank_to_id, 'wordpoints_users_with_rank');
 
 		$wpdb->update(
 			$wpdb->wordpoints_user_ranks
-			, array( 'rank_id' => $rank_to_id )
-			, array( 'rank_id' => $rank_from_id )
+			, array('rank_id' => $rank_to_id)
+			, array('rank_id' => $rank_from_id)
 			, '%d'
 			, '%d'
 		);
@@ -498,35 +498,35 @@ 
 	 *
 	 * @param int $rank_id A rank ID.
 	 */
-	private function _maybe_increase_users_with_previous_rank( $rank_id ) {
+	private function _maybe_increase_users_with_previous_rank($rank_id) {
 
-		$rank = wordpoints_get_rank( $rank_id );
+		$rank = wordpoints_get_rank($rank_id);
 		$previous_rank = $rank->get_adjacent( -1 );
 
-		if ( ! $previous_rank ) {
+		if ( ! $previous_rank) {
 			return;
 		}
 
-		$users = wordpoints_get_users_with_rank( $previous_rank->ID );
+		$users = wordpoints_get_users_with_rank($previous_rank->ID);
 
-		if ( empty( $users ) ) {
+		if (empty($users)) {
 			return;
 		}
 
-		$rank_type = WordPoints_Rank_Types::get_type( $rank->type );
+		$rank_type = WordPoints_Rank_Types::get_type($rank->type);
 
-		foreach ( $users as $user_id ) {
+		foreach ($users as $user_id) {
 
 			$new_rank = $rank_type->maybe_increase_user_rank(
 				$user_id
 				, $previous_rank
 			);
 
-			if ( $new_rank->ID === $previous_rank->ID ) {
+			if ($new_rank->ID === $previous_rank->ID) {
 				continue;
 			}
 
-			wordpoints_update_user_rank( $user_id, $new_rank->ID );
+			wordpoints_update_user_rank($user_id, $new_rank->ID);
 		}
 	}
 }

src/includes/class-un-installer-base.php

Doc Comments +1 added lines, -1 removed lines

@@ -697,7 +697,7 @@
 	 * @since 2.0.0
 	 *
 	 * @param string $option The name of the option to set.
-	 * @param mixed  $value  The value of the option.
+	 * @param string  $value  The value of the option.
 	 */
 	private function _set_option( $option, $value = true ) {
 

Spacing +268 added lines, -268 removed lines

@@ -285,21 +285,21 @@ 
 	 * @param string $slug    The slug of the entity.
 	 * @param string $version The current code version of the entity.
 	 */
-	public function __construct( $slug = null, $version = null ) {
+	public function __construct($slug = null, $version = null) {
 
-		if ( ! isset( $slug ) ) {
-			_doing_it_wrong( __METHOD__, 'The $slug parameter is required.', '2.0.0' );
+		if ( ! isset($slug)) {
+			_doing_it_wrong(__METHOD__, 'The $slug parameter is required.', '2.0.0');
 		}
 
-		if ( ! isset( $version ) ) {
-			_doing_it_wrong( __METHOD__, 'The $version parameter is required.', '2.0.0' );
+		if ( ! isset($version)) {
+			_doing_it_wrong(__METHOD__, 'The $version parameter is required.', '2.0.0');
 		}
 
 		$this->slug = $slug;
 		$this->version = $version;
 
-		if ( isset( $this->option_prefix ) ) {
-			_deprecated_argument( __METHOD__, '2.0.0', 'The $option_prefix property is deprecated.' );
+		if (isset($this->option_prefix)) {
+			_deprecated_argument(__METHOD__, '2.0.0', 'The $option_prefix property is deprecated.');
 		}
 	}
 
@@ -310,7 +310,7 @@ 
 	 *
 	 * @param bool $network Whether the install should be network-wide on multisite.
 	 */
-	public function install( $network ) {
+	public function install($network) {
 
 		$this->action = 'install';
 
@@ -320,7 +320,7 @@ 
 
 		$hooks = wordpoints_hooks();
 		$hooks_mode = $hooks->get_current_mode();
-		$hooks->set_current_mode( 'standard' );
+		$hooks->set_current_mode('standard');
 
 		$this->before_install();
 
@@ -329,38 +329,38 @@ 
 		 *
 		 * @since 1.8.0
 		 */
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH . 'wp-admin/includes/upgrade.php');
 
-		if ( is_multisite() ) {
+		if (is_multisite()) {
 
 			$this->context = 'network';
-			$hooks->set_current_mode( 'network' );
+			$hooks->set_current_mode('network');
 
 			$this->install_network();
 
 			$this->context = 'site';
-			$hooks->set_current_mode( 'standard' );
+			$hooks->set_current_mode('standard');
 
-			if ( $network ) {
+			if ($network) {
 
 				$this->set_network_installed();
 
 				$skip_per_site_install = $this->skip_per_site_install();
 
-				if ( ! ( $skip_per_site_install & self::SKIP_INSTALL ) ) {
+				if ( ! ($skip_per_site_install & self::SKIP_INSTALL)) {
 
 					$ms_switched_state = new WordPoints_Multisite_Switched_State();
 					$ms_switched_state->backup();
 
-					foreach ( $this->get_all_site_ids() as $blog_id ) {
-						switch_to_blog( $blog_id );
+					foreach ($this->get_all_site_ids() as $blog_id) {
+						switch_to_blog($blog_id);
 						$this->install_site();
 					}
 
 					$ms_switched_state->restore();
 				}
 
-				if ( $skip_per_site_install & self::REQUIRES_MANUAL_INSTALL ) {
+				if ($skip_per_site_install & self::REQUIRES_MANUAL_INSTALL) {
 
 					// We'll check this later and let the user know that per-site
 					// install was skipped.
@@ -380,7 +380,7 @@ 
 
 		} // End if ( is_multisite() ) else.
 
-		$hooks->set_current_mode( $hooks_mode );
+		$hooks->set_current_mode($hooks_mode);
 	}
 
 	/**
@@ -390,7 +390,7 @@ 
 	 *
 	 * @param int $site_id The ID of the site to install on.
 	 */
-	public function install_on_site( $site_id ) {
+	public function install_on_site($site_id) {
 
 		$this->action = 'install';
 		$this->network_wide = true;
@@ -399,7 +399,7 @@ 
 
 		$hooks = wordpoints_hooks();
 		$hooks_mode = $hooks->get_current_mode();
-		$hooks->set_current_mode( 'standard' );
+		$hooks->set_current_mode('standard');
 
 		$this->before_install();
 
@@ -408,15 +408,15 @@ 
 		 *
 		 * @since 1.8.0
 		 */
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH . 'wp-admin/includes/upgrade.php');
 
 		$this->context = 'site';
 
-		switch_to_blog( $site_id );
+		switch_to_blog($site_id);
 		$this->install_site();
 		restore_current_blog();
 
-		$hooks->set_current_mode( $hooks_mode );
+		$hooks->set_current_mode($hooks_mode);
 	}
 
 	/**
@@ -435,13 +435,13 @@ 
 
 		$hooks = wordpoints_hooks();
 		$hooks_mode = $hooks->get_current_mode();
-		$hooks->set_current_mode( 'standard' );
+		$hooks->set_current_mode('standard');
 
 		$this->before_uninstall();
 
-		if ( is_multisite() ) {
+		if (is_multisite()) {
 
-			if ( $this->do_per_site_uninstall() ) {
+			if ($this->do_per_site_uninstall()) {
 
 				$this->context = 'site';
 
@@ -450,22 +450,22 @@ 
 
 				$site_ids = $this->get_installed_site_ids();
 
-				if ( ! $this->is_network_installed() ) {
-					$site_ids = $this->validate_site_ids( $site_ids );
+				if ( ! $this->is_network_installed()) {
+					$site_ids = $this->validate_site_ids($site_ids);
 				}
 
-				foreach ( $site_ids as $blog_id ) {
-					switch_to_blog( $blog_id );
+				foreach ($site_ids as $blog_id) {
+					switch_to_blog($blog_id);
 					$this->uninstall_site();
 				}
 
 				$ms_switched_state->restore();
 
-				unset( $ms_switched_state, $site_ids );
+				unset($ms_switched_state, $site_ids);
 			}
 
 			$this->context = 'network';
-			$hooks->set_current_mode( 'network' );
+			$hooks->set_current_mode('network');
 
 			$this->uninstall_network();
 
@@ -473,7 +473,7 @@ 
 
 			// If WordPoints is being uninstalled, the options will already have been
 			// deleted, and calling these methods will actually create them again.
-			if ( 'wordpoints' !== $this->slug ) {
+			if ('wordpoints' !== $this->slug) {
 				$this->unset_network_installed();
 				$this->unset_network_install_skipped();
 				$this->unset_network_update_skipped();
@@ -486,7 +486,7 @@ 
 
 		} // End if ( is_multisite() ) else.
 
-		$hooks->set_current_mode( $hooks_mode );
+		$hooks->set_current_mode($hooks_mode);
 	}
 
 	/**
@@ -499,24 +499,24 @@ 
 	 * @param bool   $network Whether the entity is network active. Defaults to the
 	 *                        state of WordPoints itself.
 	 */
-	protected function prepare_to_update( $from, $to, $network ) {
+	protected function prepare_to_update($from, $to, $network) {
 
 		$this->action = 'update';
 
-		if ( null === $network ) {
+		if (null === $network) {
 			$network = is_wordpoints_network_active();
 		}
 
 		$this->network_wide = $network;
-		$this->updating_from = ( null === $from ) ? $this->get_db_version() : $from;
-		$this->updating_to   = ( null === $to ) ? $this->version : $to;
+		$this->updating_from = (null === $from) ? $this->get_db_version() : $from;
+		$this->updating_to   = (null === $to) ? $this->version : $to;
 
 		$updates = array();
 
-		foreach ( $this->updates as $version => $types ) {
+		foreach ($this->updates as $version => $types) {
 
-			if ( version_compare( $from, $version, '<' ) ) {
-				$updates[ str_replace( array( '.', '-' ), '_', $version ) ] = $types;
+			if (version_compare($from, $version, '<')) {
+				$updates[str_replace(array('.', '-'), '_', $version)] = $types;
 			}
 		}
 
@@ -533,11 +533,11 @@ 
 	 * @param bool   $network Whether the entity is network active. Defaults to the
 	 *                        state of WordPoints itself.
 	 */
-	public function update( $from = null, $to = null, $network = null ) {
+	public function update($from = null, $to = null, $network = null) {
 
-		$this->prepare_to_update( $from, $to, $network );
+		$this->prepare_to_update($from, $to, $network);
 
-		if ( empty( $this->updates ) ) {
+		if (empty($this->updates)) {
 			return;
 		}
 
@@ -545,7 +545,7 @@ 
 
 		$hooks = wordpoints_hooks();
 		$hooks_mode = $hooks->get_current_mode();
-		$hooks->set_current_mode( 'standard' );
+		$hooks->set_current_mode('standard');
 
 		$this->before_update();
 
@@ -554,32 +554,32 @@ 
 		 *
 		 * @since 1.8.0
 		 */
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH . 'wp-admin/includes/upgrade.php');
 
-		if ( is_multisite() ) {
+		if (is_multisite()) {
 
 			$this->context = 'network';
-			$hooks->set_current_mode( 'network' );
+			$hooks->set_current_mode('network');
 
-			$this->update_( 'network', $this->get_updates_for( 'network' ) );
+			$this->update_('network', $this->get_updates_for('network'));
 
 			$this->context = 'site';
-			$hooks->set_current_mode( 'standard' );
+			$hooks->set_current_mode('standard');
 
-			if ( $this->network_wide ) {
+			if ($this->network_wide) {
 
-				$updates = $this->get_updates_for( 'site' );
+				$updates = $this->get_updates_for('site');
 
-				if ( $updates ) {
+				if ($updates) {
 
-					if ( $this->do_per_site_update() ) {
+					if ($this->do_per_site_update()) {
 
 						$ms_switched_state = new WordPoints_Multisite_Switched_State();
 						$ms_switched_state->backup();
 
-						foreach ( $this->get_installed_site_ids() as $blog_id ) {
-							switch_to_blog( $blog_id );
-							$this->update_( 'site', $updates );
+						foreach ($this->get_installed_site_ids() as $blog_id) {
+							switch_to_blog($blog_id);
+							$this->update_('site', $updates);
 						}
 
 						$ms_switched_state->restore();
@@ -594,19 +594,19 @@ 
 
 			} else {
 
-				$this->update_( 'site', $this->get_updates_for( 'site' ) );
+				$this->update_('site', $this->get_updates_for('site'));
 			}
 
 		} else {
 
 			$this->context = 'single';
-			$this->update_( 'single', $this->get_updates_for( 'single' ) );
+			$this->update_('single', $this->get_updates_for('single'));
 
 		} // End if ( is_multisite() ) else.
 
 		$this->after_update();
 
-		$hooks->set_current_mode( $hooks_mode );
+		$hooks->set_current_mode($hooks_mode);
 	}
 
 	//
@@ -620,10 +620,10 @@ 
 	 */
 	protected function no_interruptions() {
 
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! wordpoints_is_function_disabled( 'set_time_limit' ) ) {
-			set_time_limit( 0 );
+		if ( ! wordpoints_is_function_disabled('set_time_limit')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -638,7 +638,7 @@ 
 	 */
 	protected function skip_per_site_install() {
 
-		return ( wp_is_large_network() )
+		return (wp_is_large_network())
 			? self::SKIP_INSTALL | self::REQUIRES_MANUAL_INSTALL
 			: self::DO_INSTALL;
 	}
@@ -673,9 +673,9 @@ 
 	 */
 	protected function get_all_site_ids() {
 
-		$site_ids = get_site_transient( 'wordpoints_all_site_ids' );
+		$site_ids = get_site_transient('wordpoints_all_site_ids');
 
-		if ( ! $site_ids ) {
+		if ( ! $site_ids) {
 
 			$site_ids = get_sites(
 				array(
@@ -685,7 +685,7 @@ 
 				)
 			);
 
-			set_site_transient( 'wordpoints_all_site_ids', $site_ids, 2 * MINUTE_IN_SECONDS );
+			set_site_transient('wordpoints_all_site_ids', $site_ids, 2 * MINUTE_IN_SECONDS);
 		}
 
 		return $site_ids;
@@ -699,11 +699,11 @@ 
 	 * @param string $option The name of the option to set.
 	 * @param mixed  $value  The value of the option.
 	 */
-	private function _set_option( $option, $value = true ) {
+	private function _set_option($option, $value = true) {
 
-		if ( isset( $this->option_prefix ) ) {
+		if (isset($this->option_prefix)) {
 
-			update_site_option( "{$this->option_prefix}{$option}", $value );
+			update_site_option("{$this->option_prefix}{$option}", $value);
 
 		} else {
 
@@ -712,9 +712,9 @@ 
 				, 'site'
 			);
 
-			$data[ $this->type ][ $this->slug ] = $value;
+			$data[$this->type][$this->slug] = $value;
 
-			update_site_option( "wordpoints_{$option}", $data );
+			update_site_option("wordpoints_{$option}", $data);
 		}
 	}
 
@@ -725,11 +725,11 @@ 
 	 *
 	 * @param string $option The name of the option to delete.
 	 */
-	private function _unset_option( $option ) {
+	private function _unset_option($option) {
 
-		if ( isset( $this->option_prefix ) ) {
+		if (isset($this->option_prefix)) {
 
-			delete_site_option( "{$this->option_prefix}{$option}" );
+			delete_site_option("{$this->option_prefix}{$option}");
 
 		} else {
 
@@ -738,9 +738,9 @@ 
 				, 'site'
 			);
 
-			unset( $data[ $this->type ][ $this->slug ] );
+			unset($data[$this->type][$this->slug]);
 
-			update_site_option( "wordpoints_{$option}", $data );
+			update_site_option("wordpoints_{$option}", $data);
 		}
 	}
 
@@ -753,9 +753,9 @@ 
 	 */
 	protected function is_network_installed() {
 
-		if ( isset( $this->option_prefix ) ) {
+		if (isset($this->option_prefix)) {
 
-			return (bool) get_site_option( "{$this->option_prefix}network_installed" );
+			return (bool) get_site_option("{$this->option_prefix}network_installed");
 
 		} else {
 
@@ -764,7 +764,7 @@ 
 				, 'site'
 			);
 
-			return isset( $network_installed[ $this->type ][ $this->slug ] );
+			return isset($network_installed[$this->type][$this->slug]);
 		}
 	}
 
@@ -774,7 +774,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function set_network_installed() {
-		$this->_set_option( 'network_installed' );
+		$this->_set_option('network_installed');
 	}
 
 	/**
@@ -783,7 +783,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function unset_network_installed() {
-		$this->_unset_option( 'network_installed' );
+		$this->_unset_option('network_installed');
 	}
 
 	/**
@@ -792,7 +792,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function set_network_install_skipped() {
-		$this->_set_option( 'network_install_skipped' );
+		$this->_set_option('network_install_skipped');
 	}
 
 	/**
@@ -801,7 +801,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function unset_network_install_skipped() {
-		$this->_unset_option( 'network_install_skipped' );
+		$this->_unset_option('network_install_skipped');
 	}
 
 	/**
@@ -810,7 +810,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function set_network_update_skipped() {
-		$this->_set_option( 'network_update_skipped', $this->updating_from );
+		$this->_set_option('network_update_skipped', $this->updating_from);
 	}
 
 	/**
@@ -819,7 +819,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function unset_network_update_skipped() {
-		$this->_unset_option( 'network_update_skipped' );
+		$this->_unset_option('network_update_skipped');
 	}
 
 	/**
@@ -833,11 +833,11 @@ 
 	 */
 	protected function do_per_site_uninstall() {
 
-		if ( wp_is_large_network() ) {
+		if (wp_is_large_network()) {
 
-			if ( $this->is_network_installed() ) {
+			if ($this->is_network_installed()) {
 				return false;
-			} elseif ( count( $this->get_installed_site_ids() ) > 10000 ) {
+			} elseif (count($this->get_installed_site_ids()) > 10000) {
 				return false;
 			}
 		}
@@ -856,7 +856,7 @@ 
 	 */
 	protected function do_per_site_update() {
 
-		if ( $this->is_network_installed() && wp_is_large_network() ) {
+		if ($this->is_network_installed() && wp_is_large_network()) {
 			return false;
 		}
 
@@ -872,11 +872,11 @@ 
 	 */
 	private function _get_installed_site_ids_option_name() {
 
-		if ( isset( $this->option_prefix ) ) {
+		if (isset($this->option_prefix)) {
 			$option_prefix = $this->option_prefix;
-		} elseif ( 'wordpoints' === $this->slug ) {
+		} elseif ('wordpoints' === $this->slug) {
 			$option_prefix = 'wordpoints_';
-		} elseif ( 'component' === $this->type ) {
+		} elseif ('component' === $this->type) {
 			$option_prefix = "wordpoints_{$this->slug}_";
 		} else {
 			$option_prefix = "wordpoints_{$this->type}_{$this->slug}_";
@@ -894,7 +894,7 @@ 
 	 */
 	protected function get_installed_site_ids() {
 
-		if ( $this->is_network_installed() ) {
+		if ($this->is_network_installed()) {
 			$sites = $this->get_all_site_ids();
 		} else {
 			$sites = wordpoints_get_array_option(
@@ -913,18 +913,18 @@ 
 	 *
 	 * @param int $id The ID of the site to add. Defaults to the current site's ID.
 	 */
-	protected function add_installed_site_id( $id = null ) {
+	protected function add_installed_site_id($id = null) {
 
-		if ( empty( $id ) ) {
+		if (empty($id)) {
 			$id = get_current_blog_id();
 		}
 
 		$option_name = $this->_get_installed_site_ids_option_name();
 
-		$sites = wordpoints_get_array_option( $option_name, 'site' );
+		$sites = wordpoints_get_array_option($option_name, 'site');
 		$sites[] = $id;
 
-		update_site_option( $option_name, $sites );
+		update_site_option($option_name, $sites);
 	}
 
 	/**
@@ -933,7 +933,7 @@ 
 	 * @since 2.0.0
 	 */
 	protected function delete_installed_site_ids() {
-		delete_site_option( $this->_get_installed_site_ids_option_name() );
+		delete_site_option($this->_get_installed_site_ids_option_name());
 	}
 
 	/**
@@ -945,9 +945,9 @@ 
 	 *
 	 * @return array The validated site IDs.
 	 */
-	protected function validate_site_ids( $site_ids ) {
+	protected function validate_site_ids($site_ids) {
 
-		if ( empty( $site_ids ) || ! is_array( $site_ids ) ) {
+		if (empty($site_ids) || ! is_array($site_ids)) {
 			return array();
 		}
 
@@ -972,20 +972,20 @@ 
 	 */
 	protected function get_db_version() {
 
-		if ( 'network' === $this->context ) {
-			$wordpoints_data = wordpoints_get_array_option( 'wordpoints_data', 'site' );
+		if ('network' === $this->context) {
+			$wordpoints_data = wordpoints_get_array_option('wordpoints_data', 'site');
 		} else {
-			$wordpoints_data = wordpoints_get_array_option( 'wordpoints_data' );
+			$wordpoints_data = wordpoints_get_array_option('wordpoints_data');
 		}
 
-		if ( 'wordpoints' === $this->slug ) {
+		if ('wordpoints' === $this->slug) {
 
-			if ( isset( $wordpoints_data['version'] ) ) {
+			if (isset($wordpoints_data['version'])) {
 				return $wordpoints_data['version'];
 			}
 
-		} elseif ( isset( $wordpoints_data[ "{$this->type}s" ][ $this->slug ]['version'] ) ) {
-			return $wordpoints_data[ "{$this->type}s" ][ $this->slug ]['version'];
+		} elseif (isset($wordpoints_data["{$this->type}s"][$this->slug]['version'])) {
+			return $wordpoints_data["{$this->type}s"][$this->slug]['version'];
 		}
 
 		return false;
@@ -998,28 +998,28 @@ 
 	 *
 	 * @param string $version The version of the entity.
 	 */
-	protected function set_db_version( $version = null ) {
+	protected function set_db_version($version = null) {
 
-		if ( null === $version ) {
+		if (null === $version) {
 			$version = $this->version;
 		}
 
-		if ( 'network' === $this->context ) {
-			$wordpoints_data = wordpoints_get_array_option( 'wordpoints_data', 'site' );
+		if ('network' === $this->context) {
+			$wordpoints_data = wordpoints_get_array_option('wordpoints_data', 'site');
 		} else {
-			$wordpoints_data = wordpoints_get_array_option( 'wordpoints_data' );
+			$wordpoints_data = wordpoints_get_array_option('wordpoints_data');
 		}
 
-		if ( 'wordpoints' === $this->slug ) {
+		if ('wordpoints' === $this->slug) {
 			$wordpoints_data['version'] = $version;
 		} else {
-			$wordpoints_data[ "{$this->type}s" ][ $this->slug ]['version'] = $version;
+			$wordpoints_data["{$this->type}s"][$this->slug]['version'] = $version;
 		}
 
-		if ( 'network' === $this->context ) {
-			update_site_option( 'wordpoints_data', $wordpoints_data );
+		if ('network' === $this->context) {
+			update_site_option('wordpoints_data', $wordpoints_data);
 		} else {
-			update_option( 'wordpoints_data', $wordpoints_data );
+			update_option('wordpoints_data', $wordpoints_data);
 		}
 	}
 
@@ -1031,22 +1031,22 @@ 
 	protected function unset_db_version() {
 
 		// The whole option will be deleted when WordPoints is uninstalled.
-		if ( 'wordpoints' === $this->slug ) {
+		if ('wordpoints' === $this->slug) {
 			return;
 		}
 
-		if ( 'network' === $this->context ) {
-			$wordpoints_data = wordpoints_get_array_option( 'wordpoints_data', 'site' );
+		if ('network' === $this->context) {
+			$wordpoints_data = wordpoints_get_array_option('wordpoints_data', 'site');
 		} else {
-			$wordpoints_data = wordpoints_get_array_option( 'wordpoints_data' );
+			$wordpoints_data = wordpoints_get_array_option('wordpoints_data');
 		}
 
-		unset( $wordpoints_data[ "{$this->type}s" ][ $this->slug ] );
+		unset($wordpoints_data["{$this->type}s"][$this->slug]);
 
-		if ( 'network' === $this->context ) {
-			update_site_option( 'wordpoints_data', $wordpoints_data );
+		if ('network' === $this->context) {
+			update_site_option('wordpoints_data', $wordpoints_data);
 		} else {
-			update_option( 'wordpoints_data', $wordpoints_data );
+			update_option('wordpoints_data', $wordpoints_data);
 		}
 	}
 
@@ -1061,19 +1061,19 @@ 
 	 * @param string $component The component's slug.
 	 * @param string $version   The installed component version.
 	 */
-	protected function set_component_version( $component, $version ) {
+	protected function set_component_version($component, $version) {
 
-		_deprecated_function( __METHOD__, '2.0.0', '::set_db_version()' );
+		_deprecated_function(__METHOD__, '2.0.0', '::set_db_version()');
 
 		$wordpoints_data = wordpoints_get_maybe_network_array_option(
 			'wordpoints_data'
 		);
 
-		if ( empty( $wordpoints_data['components'][ $component ]['version'] ) ) {
-			$wordpoints_data['components'][ $component ]['version'] = $version;
+		if (empty($wordpoints_data['components'][$component]['version'])) {
+			$wordpoints_data['components'][$component]['version'] = $version;
 		}
 
-		wordpoints_update_maybe_network_option( 'wordpoints_data', $wordpoints_data );
+		wordpoints_update_maybe_network_option('wordpoints_data', $wordpoints_data);
 	}
 
 	/**
@@ -1083,14 +1083,14 @@ 
 	 */
 	protected function maybe_load_custom_caps() {
 
-		if ( empty( $this->custom_caps_getter ) ) {
+		if (empty($this->custom_caps_getter)) {
 			return;
 		}
 
-		$this->custom_caps = call_user_func( $this->custom_caps_getter );
-		$this->custom_caps_keys = array_keys( $this->custom_caps );
+		$this->custom_caps = call_user_func($this->custom_caps_getter);
+		$this->custom_caps_keys = array_keys($this->custom_caps);
 
-		if ( 'uninstall' === $this->action ) {
+		if ('uninstall' === $this->action) {
 			$this->uninstall['local']['custom_caps'] = true;
 		}
 	}
@@ -1101,7 +1101,7 @@ 
 	 * @since 1.8.0
 	 */
 	protected function before_install() {
-		$this->map_shortcuts( 'schema' );
+		$this->map_shortcuts('schema');
 		$this->maybe_load_custom_caps();
 	}
 
@@ -1112,8 +1112,8 @@ 
 	 */
 	protected function install_custom_caps() {
 
-		if ( ! empty( $this->custom_caps ) ) {
-			wordpoints_add_custom_caps( $this->custom_caps );
+		if ( ! empty($this->custom_caps)) {
+			wordpoints_add_custom_caps($this->custom_caps);
 		}
 	}
 
@@ -1126,8 +1126,8 @@ 
 
 		$schema = $this->get_db_schema();
 
-		if ( ! empty( $schema ) ) {
-			dbDelta( $schema );
+		if ( ! empty($schema)) {
+			dbDelta($schema);
 		}
 	}
 
@@ -1140,7 +1140,7 @@ 
 	 */
 	public function get_db_schema() {
 
-		if ( ! isset( $this->schema[ $this->context ]['tables'] ) ) {
+		if ( ! isset($this->schema[$this->context]['tables'])) {
 			return '';
 		}
 
@@ -1150,16 +1150,16 @@ 
 
 		$charset_collate = $wpdb->get_charset_collate();
 
-		if ( 'site' === $this->context ) {
+		if ('site' === $this->context) {
 			$prefix = $wpdb->prefix;
 		} else {
 			$prefix = $wpdb->base_prefix;
 		}
 
-		foreach ( $this->schema[ $this->context ]['tables'] as $table_name => $table_schema ) {
+		foreach ($this->schema[$this->context]['tables'] as $table_name => $table_schema) {
 
-			$table_name   = str_replace( '`', '``', $table_name );
-			$table_schema = trim( $table_schema );
+			$table_name   = str_replace('`', '``', $table_name);
+			$table_schema = trim($table_schema);
 
 			$schema .= "CREATE TABLE {$prefix}{$table_name} (
 				{$table_schema}
@@ -1179,31 +1179,31 @@ 
 		$this->maybe_load_custom_caps();
 
 		$this->prepare_uninstall_list_tables();
-		$this->prepare_uninstall_non_per_site_items( 'meta_boxes' );
-		$this->map_uninstall_shortcut( 'widgets', 'options', array( 'prefix' => 'widget_' ) );
-		$this->map_uninstall_shortcut( 'points_hooks', 'options', array( 'prefix' => 'wordpoints_hook-' ) );
+		$this->prepare_uninstall_non_per_site_items('meta_boxes');
+		$this->map_uninstall_shortcut('widgets', 'options', array('prefix' => 'widget_'));
+		$this->map_uninstall_shortcut('points_hooks', 'options', array('prefix' => 'wordpoints_hook-'));
 
 		// Add any tables to uninstall based on the db schema.
-		foreach ( $this->schema as $context => $schema ) {
+		foreach ($this->schema as $context => $schema) {
 
-			if ( ! isset( $schema['tables'] ) ) {
+			if ( ! isset($schema['tables'])) {
 				continue;
 			}
 
-			if ( ! isset( $this->uninstall[ $context ]['tables'] ) ) {
-				$this->uninstall[ $context ]['tables'] = array();
+			if ( ! isset($this->uninstall[$context]['tables'])) {
+				$this->uninstall[$context]['tables'] = array();
 			}
 
-			$this->uninstall[ $context ]['tables'] = array_unique(
+			$this->uninstall[$context]['tables'] = array_unique(
 				array_merge(
-					$this->uninstall[ $context ]['tables']
-					, array_keys( $schema['tables'] )
+					$this->uninstall[$context]['tables']
+					, array_keys($schema['tables'])
 				)
 			);
 		}
 
 		// This *must* happen *after* the schema and list tables args are parsed.
-		$this->map_shortcuts( 'uninstall' );
+		$this->map_shortcuts('uninstall');
 	}
 
 	/**
@@ -1237,7 +1237,7 @@ 
 	 */
 	protected function prepare_uninstall_list_tables() {
 
-		if ( ! isset( $this->uninstall['list_tables'] ) ) {
+		if ( ! isset($this->uninstall['list_tables'])) {
 			return;
 		}
 
@@ -1248,17 +1248,17 @@ 
 		);
 
 		// Loop through all of the list table screens.
-		foreach ( $this->uninstall['list_tables'] as $screen_id => $args ) {
+		foreach ($this->uninstall['list_tables'] as $screen_id => $args) {
 
 			// Back-compat for pre-2.1.0.
-			if ( isset( $args['parent'] ) && '_page' === substr( $args['parent'], -5 /* _page */ ) ) {
-				$args['parent'] = substr( $args['parent'], 0, -5 );
+			if (isset($args['parent']) && '_page' === substr($args['parent'], -5 /* _page */)) {
+				$args['parent'] = substr($args['parent'], 0, -5);
 			}
 
-			$this->uninstall['universal']['list_tables'][ $screen_id ] = $args;
+			$this->uninstall['universal']['list_tables'][$screen_id] = $args;
 		}
 
-		$this->prepare_uninstall_non_per_site_items( 'list_tables' );
+		$this->prepare_uninstall_non_per_site_items('list_tables');
 	}
 
 	/**
@@ -1276,22 +1276,22 @@ 
 	 *
 	 * @param string $items_key The key name of the items in the uninstall[*] arrays.
 	 */
-	protected function prepare_uninstall_non_per_site_items( $items_key ) {
+	protected function prepare_uninstall_non_per_site_items($items_key) {
 
-		if ( isset( $this->uninstall['universal'][ $items_key ] ) ) {
+		if (isset($this->uninstall['universal'][$items_key])) {
 
-			$this->uninstall['global'][ $items_key ]
-				= $this->uninstall['universal'][ $items_key ];
+			$this->uninstall['global'][$items_key]
+				= $this->uninstall['universal'][$items_key];
 
-			unset( $this->uninstall['universal'][ $items_key ] );
+			unset($this->uninstall['universal'][$items_key]);
 		}
 
-		if ( isset( $this->uninstall['site'][ $items_key ] ) ) {
+		if (isset($this->uninstall['site'][$items_key])) {
 
-			$this->uninstall['network'][ $items_key ]
-				= $this->uninstall['site'][ $items_key ];
+			$this->uninstall['network'][$items_key]
+				= $this->uninstall['site'][$items_key];
 
-			unset( $this->uninstall['site'][ $items_key ] );
+			unset($this->uninstall['site'][$items_key]);
 		}
 	}
 
@@ -1328,20 +1328,20 @@ 
 	 *                             adding to them to the canonical array.
 	 * }
 	 */
-	protected function map_uninstall_shortcut( $shortcut, $canonical, $args ) {
+	protected function map_uninstall_shortcut($shortcut, $canonical, $args) {
 
-		$args = array_merge( array( 'prefix' => '' ), $args );
+		$args = array_merge(array('prefix' => ''), $args);
 
-		$types = array( 'single', 'site', 'network', 'local', 'global', 'universal' );
+		$types = array('single', 'site', 'network', 'local', 'global', 'universal');
 
-		foreach ( $types as $type ) {
+		foreach ($types as $type) {
 
-			if ( ! isset( $this->uninstall[ $type ][ $shortcut ] ) ) {
+			if ( ! isset($this->uninstall[$type][$shortcut])) {
 				continue;
 			}
 
-			foreach ( $this->uninstall[ $type ][ $shortcut ] as $slug ) {
-				$this->uninstall[ $type ][ $canonical ][] = $args['prefix'] . $slug;
+			foreach ($this->uninstall[$type][$shortcut] as $slug) {
+				$this->uninstall[$type][$canonical][] = $args['prefix'] . $slug;
 			}
 		}
 	}
@@ -1361,28 +1361,28 @@ 
 	 *
 	 * @param string $type The type of shortcuts to map. Corresponds to a member var.
 	 */
-	protected function map_shortcuts( $type ) {
+	protected function map_shortcuts($type) {
 
 		// shortcut => canonicals
 		$map = array(
-			'local'     => array( 'single', 'site', /*  -  */ ),
-			'global'    => array( 'single', /* - */ 'network' ),
-			'universal' => array( 'single', 'site', 'network' ),
+			'local'     => array('single', 'site', /*  -  */),
+			'global'    => array('single', /* - */ 'network'),
+			'universal' => array('single', 'site', 'network'),
 		);
 
 		$this->$type = array_merge(
 			array_fill_keys(
-				array( 'single', 'site', 'network', 'local', 'global', 'universal' )
+				array('single', 'site', 'network', 'local', 'global', 'universal')
 				, array()
 			)
 			, $this->$type
 		);
 
-		foreach ( $map as $shortcut => $canonicals ) {
-			foreach ( $canonicals as $canonical ) {
-				$this->{$type}[ $canonical ] = array_merge_recursive(
-					$this->{$type}[ $canonical ]
-					, $this->{$type}[ $shortcut ]
+		foreach ($map as $shortcut => $canonicals) {
+			foreach ($canonicals as $canonical) {
+				$this->{$type}[$canonical] = array_merge_recursive(
+					$this->{$type}[$canonical]
+					, $this->{$type}[$shortcut]
 				);
 			}
 		}
@@ -1413,9 +1413,9 @@ 
 	 *
 	 * @return array The versions that request this type of update.
 	 */
-	protected function get_updates_for( $type ) {
+	protected function get_updates_for($type) {
 
-		return array_keys( wp_list_filter( $this->updates, array( $type => true ) ) );
+		return array_keys(wp_list_filter($this->updates, array($type => true)));
 	}
 
 	/**
@@ -1426,9 +1426,9 @@ 
 	 * @param string $type     The type of update to run.
 	 * @param array  $versions The versions to run this type of update for.
 	 */
-	protected function update_( $type, $versions ) {
+	protected function update_($type, $versions) {
 
-		foreach ( $versions as $version ) {
+		foreach ($versions as $version) {
 			$this->{"update_{$type}_to_{$version}"}();
 		}
 	}
@@ -1440,22 +1440,22 @@ 
 	 *
 	 * @param string $type The type of tables to update.
 	 */
-	protected function maybe_update_tables_to_utf8mb4( $type ) {
+	protected function maybe_update_tables_to_utf8mb4($type) {
 
 		global $wpdb;
 
-		if ( 'utf8mb4' !== $wpdb->charset ) {
+		if ('utf8mb4' !== $wpdb->charset) {
 			return;
 		}
 
-		if ( 'global' === $type || 'network' === $type ) {
+		if ('global' === $type || 'network' === $type) {
 			$prefix = $wpdb->base_prefix;
 		} else {
 			$prefix = $wpdb->prefix;
 		}
 
-		foreach ( $this->schema[ $type ]['tables'] as $table_name => $schema ) {
-			maybe_convert_table_to_utf8mb4( $prefix . $table_name );
+		foreach ($this->schema[$type]['tables'] as $table_name => $schema) {
+			maybe_convert_table_to_utf8mb4($prefix . $table_name);
 		}
 	}
 
@@ -1466,9 +1466,9 @@ 
 	 *
 	 * @param string $type The type of uninstallation to perform.
 	 */
-	protected function uninstall_( $type ) {
+	protected function uninstall_($type) {
 
-		if ( empty( $this->uninstall[ $type ] ) ) {
+		if (empty($this->uninstall[$type])) {
 			return;
 		}
 
@@ -1481,35 +1481,35 @@ 
 				'meta_boxes'   => array(),
 				'list_tables'  => array(),
 			)
-			, $this->uninstall[ $type ]
+			, $this->uninstall[$type]
 		);
 
-		if ( ! empty( $uninstall['custom_caps'] ) ) {
-			$this->uninstall_custom_caps( $this->custom_caps_keys );
+		if ( ! empty($uninstall['custom_caps'])) {
+			$this->uninstall_custom_caps($this->custom_caps_keys);
 		}
 
-		foreach ( $uninstall['user_meta'] as $meta_key ) {
-			$this->uninstall_metadata( 'user', $meta_key );
+		foreach ($uninstall['user_meta'] as $meta_key) {
+			$this->uninstall_metadata('user', $meta_key);
 		}
 
-		foreach ( $uninstall['options'] as $option ) {
-			$this->uninstall_option( $option );
+		foreach ($uninstall['options'] as $option) {
+			$this->uninstall_option($option);
 		}
 
-		foreach ( $uninstall['tables'] as $table ) {
-			$this->uninstall_table( $table );
+		foreach ($uninstall['tables'] as $table) {
+			$this->uninstall_table($table);
 		}
 
-		foreach ( $uninstall['comment_meta'] as $meta_key ) {
-			$this->uninstall_metadata( 'comment', $meta_key );
+		foreach ($uninstall['comment_meta'] as $meta_key) {
+			$this->uninstall_metadata('comment', $meta_key);
 		}
 
-		foreach ( $uninstall['meta_boxes'] as $screen_id => $args ) {
-			$this->uninstall_meta_boxes( $screen_id, $args );
+		foreach ($uninstall['meta_boxes'] as $screen_id => $args) {
+			$this->uninstall_meta_boxes($screen_id, $args);
 		}
 
-		foreach ( $uninstall['list_tables'] as $screen_id => $args ) {
-			$this->uninstall_list_table( $screen_id, $args );
+		foreach ($uninstall['list_tables'] as $screen_id => $args) {
+			$this->uninstall_list_table($screen_id, $args);
 		}
 	}
 
@@ -1520,9 +1520,9 @@ 
 	 *
 	 * @param string[] $caps The capabilities to uninstall.
 	 */
-	protected function uninstall_custom_caps( $caps ) {
+	protected function uninstall_custom_caps($caps) {
 
-		wordpoints_remove_custom_caps( $caps );
+		wordpoints_remove_custom_caps($caps);
 	}
 
 	/**
@@ -1533,17 +1533,17 @@ 
 	 * @param string $type The type of metadata to uninstall, e.g., 'user', 'post'.
 	 * @param string $key  The metadata key to delete.
 	 */
-	protected function uninstall_metadata( $type, $key ) {
+	protected function uninstall_metadata($type, $key) {
 
-		if ( 'user' === $type && 'site' === $this->context ) {
+		if ('user' === $type && 'site' === $this->context) {
 			$key = $GLOBALS['wpdb']->get_blog_prefix() . $key;
 		}
 
-		if ( false !== strpos( $key, '%' ) ) {
+		if (false !== strpos($key, '%')) {
 
 			global $wpdb;
 
-			$table = wordpoints_escape_mysql_identifier( _get_meta_table( $type ) );
+			$table = wordpoints_escape_mysql_identifier(_get_meta_table($type));
 
 			$keys = $wpdb->get_col( // WPCS: unprepared SQL OK.
 				$wpdb->prepare( // WPCS: unprepared SQL OK.
@@ -1557,11 +1557,11 @@ 
 			); // WPCS: cache pass.
 
 		} else {
-			$keys = array( $key );
+			$keys = array($key);
 		}
 
-		foreach ( $keys as $key ) {
-			delete_metadata( $type, 0, wp_slash( $key ), '', true );
+		foreach ($keys as $key) {
+			delete_metadata($type, 0, wp_slash($key), '', true);
 		}
 	}
 
@@ -1595,25 +1595,25 @@ 
 	 *                                'meta-box-order'.
 	 * }
 	 */
-	protected function uninstall_meta_boxes( $screen_id, $args ) {
+	protected function uninstall_meta_boxes($screen_id, $args) {
 
 		$defaults = array(
 			'parent' => 'wordpoints',
-			'options' => array( 'closedpostboxes', 'metaboxhidden', 'meta-box-order' ),
+			'options' => array('closedpostboxes', 'metaboxhidden', 'meta-box-order'),
 		);
 
-		$args = array_merge( $defaults, $args );
-		$args['options'] = array_merge( $defaults['options'], $args['options'] );
+		$args = array_merge($defaults, $args);
+		$args['options'] = array_merge($defaults['options'], $args['options']);
 
 		// Each user gets to set the options to their liking.
-		foreach ( $args['options'] as $option ) {
+		foreach ($args['options'] as $option) {
 
 			$this->uninstall_metadata(
 				'user'
 				, "{$option}_{$args['parent']}_page_{$screen_id}"
 			);
 
-			if ( 'network' === $this->context ) {
+			if ('network' === $this->context) {
 				$this->uninstall_metadata(
 					'user'
 					, "{$option}_{$args['parent']}_page_{$screen_id}-network"
@@ -1654,20 +1654,20 @@ 
 	 *                                'per_page'.
 	 * }
 	 */
-	protected function uninstall_list_table( $screen_id, $args ) {
+	protected function uninstall_list_table($screen_id, $args) {
 
 		$defaults = array(
 			'parent' => 'wordpoints',
-			'options' => array( 'per_page' ),
+			'options' => array('per_page'),
 		);
 
-		$args = array_merge( $defaults, $args );
+		$args = array_merge($defaults, $args);
 
 		$parent = $network_parent = $args['parent'];
 
 		// The parent page is usually the same on a multisite site, but we need to
 		// handle the special case of the modules screen.
-		if ( 'wordpoints_modules' === $screen_id && is_multisite() ) {
+		if ('wordpoints_modules' === $screen_id && is_multisite()) {
 			$parent = 'toplevel';
 		}
 
@@ -1676,23 +1676,23 @@ 
 		// Each user can hide specific columns of the table.
 		$meta_keys[] = "manage{$parent}_page_{$screen_id}columnshidden";
 
-		if ( 'network' === $this->context ) {
+		if ('network' === $this->context) {
 			$meta_keys[] = "manage{$network_parent}_page_{$screen_id}-networkcolumnshidden";
 		}
 
 		// Loop through each of the other options provided by this list table.
-		foreach ( $args['options'] as $option ) {
+		foreach ($args['options'] as $option) {
 
 			// Each user gets to set the options to their liking.
 			$meta_keys[]  = "{$parent}_page_{$screen_id}_{$option}";
 
-			if ( 'network' === $this->context ) {
+			if ('network' === $this->context) {
 				$meta_keys[] = "{$network_parent}_page_{$screen_id}_network_{$option}";
 			}
 		}
 
-		foreach ( $meta_keys as $meta_key ) {
-			$this->uninstall_metadata( 'user', $meta_key );
+		foreach ($meta_keys as $meta_key) {
+			$this->uninstall_metadata('user', $meta_key);
 		}
 	}
 
@@ -1707,14 +1707,14 @@ 
 	 *
 	 * @param string $option The option to uninstall.
 	 */
-	protected function uninstall_option( $option ) {
+	protected function uninstall_option($option) {
 
-		if ( 'network' === $this->context ) {
-			$this->uninstall_network_option( $option );
+		if ('network' === $this->context) {
+			$this->uninstall_network_option($option);
 			return;
 		}
 
-		if ( false !== strpos( $option, '%' ) ) {
+		if (false !== strpos($option, '%')) {
 
 			global $wpdb;
 
@@ -1730,10 +1730,10 @@ 
 			); // WPCS: cache pass.
 
 		} else {
-			$options = array( $option );
+			$options = array($option);
 		}
 
-		array_map( 'delete_option', $options );
+		array_map('delete_option', $options);
 	}
 
 	/**
@@ -1745,9 +1745,9 @@ 
 	 *
 	 * @param string $option The network option to uninstall.
 	 */
-	protected function uninstall_network_option( $option ) {
+	protected function uninstall_network_option($option) {
 
-		if ( false !== strpos( $option, '%' ) ) {
+		if (false !== strpos($option, '%')) {
 
 			global $wpdb;
 
@@ -1765,10 +1765,10 @@ 
 			); // WPCS: cache pass.
 
 		} else {
-			$options = array( $option );
+			$options = array($option);
 		}
 
-		array_map( 'delete_site_option', $options );
+		array_map('delete_site_option', $options);
 	}
 
 	/**
@@ -1778,9 +1778,9 @@ 
 	 *
 	 * @param string $id_base The base ID of the widget to uninstall (class name).
 	 */
-	protected function uninstall_widget( $id_base ) {
+	protected function uninstall_widget($id_base) {
 
-		$this->uninstall_option( "widget_{$id_base}" );
+		$this->uninstall_option("widget_{$id_base}");
 	}
 
 	/**
@@ -1790,9 +1790,9 @@ 
 	 *
 	 * @param string $id_base The base ID (class) of the points hook to uninstall.
 	 */
-	protected function uninstall_points_hook( $id_base ) {
+	protected function uninstall_points_hook($id_base) {
 
-		$this->uninstall_option( "wordpoints_hook-{$id_base}" );
+		$this->uninstall_option("wordpoints_hook-{$id_base}");
 	}
 
 	/**
@@ -1802,20 +1802,20 @@ 
 	 *
 	 * @param string $table The name of the table to uninstall, sans DB prefix.
 	 */
-	protected function uninstall_table( $table ) {
+	protected function uninstall_table($table) {
 
 		global $wpdb;
 
 		// Null will use the current blog's prefix, 0 the base prefix.
-		if ( 'site' === $this->context ) {
+		if ('site' === $this->context) {
 			$site_id = null;
 		} else {
 			$site_id = 0;
 		}
 
-		$table = str_replace( '`', '``', $table );
+		$table = str_replace('`', '``', $table);
 
-		$wpdb->query( 'DROP TABLE IF EXISTS `' . $wpdb->get_blog_prefix( $site_id ) . $table . '`' ); // WPCS: unprepared SQL, cache pass.
+		$wpdb->query('DROP TABLE IF EXISTS `' . $wpdb->get_blog_prefix($site_id) . $table . '`'); // WPCS: unprepared SQL, cache pass.
 	}
 
 	//
@@ -1835,7 +1835,7 @@ 
 
 		$this->install_db_schema();
 
-		if ( $this->network_wide ) {
+		if ($this->network_wide) {
 			$this->set_db_version();
 		}
 	}
@@ -1851,11 +1851,11 @@ 
 	 */
 	protected function install_site() {
 
-		if ( isset( $this->schema['site'] ) ) {
+		if (isset($this->schema['site'])) {
 			$this->install_db_schema();
 		}
 
-		if ( ! $this->network_wide ) {
+		if ( ! $this->network_wide) {
 			$this->set_db_version();
 		}
 
@@ -1885,10 +1885,10 @@ 
 	 */
 	protected function load_base_dependencies() {
 
-		require_once dirname( __FILE__ ) . '/constants.php';
+		require_once dirname(__FILE__) . '/constants.php';
 		require_once WORDPOINTS_DIR . '/classes/class/autoloader.php';
 
-		WordPoints_Class_Autoloader::register_dir( WORDPOINTS_DIR . '/classes' );
+		WordPoints_Class_Autoloader::register_dir(WORDPOINTS_DIR . '/classes');
 
 		require_once WORDPOINTS_DIR . '/includes/functions.php';
 		require_once WORDPOINTS_DIR . '/includes/apps.php';
@@ -1915,8 +1915,8 @@ 
 	 */
 	protected function uninstall_network() {
 
-		if ( ! empty( $this->uninstall['network'] ) ) {
-			$this->uninstall_( 'network' );
+		if ( ! empty($this->uninstall['network'])) {
+			$this->uninstall_('network');
 		}
 
 		$this->unset_db_version();
@@ -1933,8 +1933,8 @@ 
 	 */
 	protected function uninstall_site() {
 
-		if ( ! empty( $this->uninstall['site'] ) ) {
-			$this->uninstall_( 'site' );
+		if ( ! empty($this->uninstall['site'])) {
+			$this->uninstall_('site');
 		}
 
 		$this->unset_db_version();
@@ -1951,8 +1951,8 @@ 
 	 */
 	protected function uninstall_single() {
 
-		if ( ! empty( $this->uninstall['single'] ) ) {
-			$this->uninstall_( 'single' );
+		if ( ! empty($this->uninstall['single'])) {
+			$this->uninstall_('single');
 		}
 
 		$this->unset_db_version();