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',
 				'order_by'      => '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];
 		}
 
 		return null;
@@ -356,7 +356,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;
 	}
 
@@ -369,9 +369,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;
 	}
 
 	/**
@@ -387,13 +387,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));
 	}
 
 	/**
@@ -415,34 +415,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;
 	}
@@ -457,22 +457,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.
@@ -488,13 +488,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);
 	}
 
 	/**
@@ -504,24 +504,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 ] ) ) {
+		if (isset($all_instances[$number])) {
 
-			unset( $all_instances[ $number ] );
+			unset($all_instances[$number]);
 
-			$this->_save_instances( $all_instances );
+			$this->_save_instances($all_instances);
 		}
 	}
 
@@ -534,27 +534,27 @@ 
 	 *
 	 * @return bool|null 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];
 		}
 
 		/**
@@ -567,11 +567,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.
@@ -583,7 +583,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;
 		}
@@ -606,9 +606,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 . ']');
 	}
 
 	/**
@@ -620,9 +620,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));
 	}
 
 	/**
@@ -640,9 +640,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);
 	}
 
 	/**
@@ -654,9 +654,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));
 	}
 
 	/**
@@ -669,15 +669,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.
@@ -688,7 +688,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);
 	}
 
 	/**
@@ -701,31 +701,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;
@@ -754,15 +754,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;
 		}
 
@@ -770,12 +770,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;
 	}
 
 	//
@@ -789,15 +789,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/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,31 +236,31 @@ 
 	 *
 	 * @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);
 
-		$post_type = get_post_field( 'post_type', $post_id );
+		$post_type = get_post_field('post_type', $post_id);
 
-		if ( $text && $post_type && post_type_exists( $post_type ) ) {
-			$args[] = get_post_type_object( $post_type )->labels->singular_name;
+		if ($text && $post_type && post_type_exists($post_type)) {
+			$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);
 	}
 
 	/**
@@ -272,17 +272,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);
 	}
 
 	/**
@@ -294,43 +294,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
@@ -348,9 +348,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');
 	}
 
 	/**
@@ -368,7 +368,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__
@@ -376,13 +376,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;
@@ -403,13 +403,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)
 		);
 	}
 
@@ -423,7 +423,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(
@@ -439,20 +439,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 +71 added lines, -71 removed lines

@@ -39,9 +39,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);
 }
 
 /**
@@ -55,9 +55,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);
 }
 
 /**
@@ -72,23 +72,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_id__not_in' => wordpoints_get_excluded_users( 'points_logs' ),
+		'user_id__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();
 	}
 
@@ -105,7 +105,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);
 }
 
 /**
@@ -125,19 +125,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']
@@ -180,9 +180,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;
 	}
 
@@ -194,9 +194,9 @@ 
 		'show_users' => true,
 	);
 
-	$args = array_merge( $defaults, $args );
+	$args = array_merge($defaults, $args);
 
-	if ( ! $args['datatable'] ) {
+	if ( ! $args['datatable']) {
 
 		_deprecated_argument(
 			__FUNCTION__
@@ -207,12 +207,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;
 	}
 
@@ -232,9 +232,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);
 }
 
 /**
@@ -253,7 +253,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,
@@ -267,7 +267,7 @@ 
 	 */
 	wordpoints_register_points_logs_query(
 		'network'
-		, array( 'blog_id' => false )
+		, array('blog_id' => false)
 		, array(
 			'network_wide'  => true,
 			'cache_queries' => true,
@@ -282,12 +282,12 @@ 
  *
  * @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');
 
 	// translators: 1. User name; 2. Reason given.
-	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']));
 }
 
 /**
@@ -297,20 +297,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;
@@ -323,23 +323,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)) {
 
 			return sprintf(
 				// translators: Singular post type name.
-				_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');
 }
 
 /**
@@ -355,10 +355,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)
 	);
 }
 
@@ -380,27 +380,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';
@@ -412,7 +412,7 @@ 
 				);
 
 				wp_cache_delete(
-					str_replace( $find, $replace, $query['cache_key'] )
+					str_replace($find, $replace, $query['cache_key'])
 					, $group
 				);
 			}
@@ -431,26 +431,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();
 	}
 
@@ -471,17 +471,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|null 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];
 	}
 
 	return null;
@@ -115,21 +115,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;
 	}
 
@@ -142,9 +142,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;
 	}
 
@@ -161,20 +161,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);
 }
 
 /**
@@ -189,11 +189,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;
 	}
 
@@ -205,48 +205,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'
@@ -279,19 +279,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}";
 
@@ -317,15 +317,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);
 }
 
 /**
@@ -367,9 +367,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;
 	}
 
@@ -381,7 +381,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);
 }
 
 /**
@@ -402,12 +402,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;
 	}
 
@@ -421,7 +421,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);
 }
 
 /**
@@ -438,15 +438,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);
 }
 
 /**
@@ -460,9 +460,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
 }
 
 /**
@@ -485,21 +485,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);
 }
 
 /**
@@ -521,19 +521,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);
 }
 
 /**
@@ -576,13 +576,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;
 	}
@@ -606,29 +606,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 {
 
@@ -647,10 +647,10 @@ 
 			)
 		);
 
-		wp_cache_delete( $user_id, 'user_meta' );
+		wp_cache_delete($user_id, 'user_meta');
 	}
 
-	if ( ! $result ) {
+	if ( ! $result) {
 		return false;
 	}
 
@@ -664,15 +664,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(
@@ -683,20 +683,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);
 			}
 
 			/**
@@ -712,7 +712,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 ).
@@ -730,9 +730,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;
@@ -758,9 +758,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);
 }
 
 /**
@@ -781,9 +781,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);
 }
 
 /**
@@ -795,9 +795,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';
 	}
 
@@ -822,23 +822,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;
 }
@@ -857,15 +857,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;
 }
@@ -885,23 +885,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;
 }
@@ -922,23 +922,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;
 }
@@ -950,7 +950,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;
 
@@ -965,15 +965,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');
 }
 
 /**
@@ -989,7 +989,7 @@ 
 		'wordpoints_default_points_type'
 	);
 
-	if ( ! wordpoints_is_points_type( $points_type ) ) {
+	if ( ! wordpoints_is_points_type($points_type)) {
 		return false;
 	}
 
@@ -1010,7 +1010,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.
@@ -1022,10 +1022,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;
@@ -1041,32 +1041,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
@@ -1076,31 +1076,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;
 			}
 		}
 
@@ -1122,35 +1122,35 @@ 
  * @return int[]|false The IDs of the users with the most points, or false if the
  *                     args are invalid.
  */
-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 false;
 	}
 
-	$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`
@@ -1175,27 +1175,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);
 }
 
 /**
@@ -1207,25 +1207,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;
 	}
 
@@ -1241,18 +1241,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>
@@ -1260,16 +1260,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(
@@ -1291,13 +1291,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
@@ -1310,9 +1310,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>
@@ -1332,9 +1332,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');
 }
 
 /**
@@ -1346,16 +1346,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');
 	}
 }
 
@@ -1369,18 +1369,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');
 	}
 }
 
@@ -1393,18 +1393,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];
 		} else {
 			return null;
 		}
@@ -122,13 +122,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]);
 	}
 
 	/**
@@ -140,17 +140,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;
 	}
@@ -166,21 +166,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;
 	}
@@ -199,30 +199,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;
 	}
@@ -240,23 +240,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;
 		}
 
@@ -264,12 +264,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;
 	}
@@ -283,45 +283,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;
@@ -338,17 +338,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);
 	}
 
 	/**
@@ -360,7 +360,7 @@ 
 	 */
 	public function get_ranks() {
 
-		return wordpoints_get_array_option( $this->option_name );
+		return wordpoints_get_array_option($this->option_name);
 	}
 
 	/**
@@ -374,7 +374,7 @@ 
 	 */
 	public function get_base_rank() {
 
-		return $this->get_rank( 0 );
+		return $this->get_rank(0);
 	}
 
 	/**
@@ -386,15 +386,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];
 	}
 
 	/**
@@ -406,10 +406,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
 		);
@@ -430,26 +430,26 @@ 
 	 *
 	 * @return int[] The list of ranks with the rank inserted.
 	 */
-	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
 			);
 		}
@@ -465,29 +465,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'
 		);
@@ -500,35 +500,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/functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -173,7 +173,7 @@ 
  *
  * @param mixed $maybe_int The value to convert to an integer.
  *
- * @return int|false False if $maybe_int couldn't reliably be converted to an integer.
+ * @return integer False if $maybe_int couldn't reliably be converted to an integer.
  */
 function wordpoints_int( &$maybe_int ) {
 
@@ -969,7 +969,7 @@ 
  * @since 1.0.1
  * @since 1.8.0 The $message now supports WP_Error objects.
  *
- * @param string|WP_Error $message The error message.
+ * @param string $message The error message.
  *
  * @return string The error message.
  */

Spacing +153 added lines, -153 removed lines

@@ -39,12 +39,12 @@ 
  *
  * @param bool $network_active Whether the plugin is being network activated.
  */
-function wordpoints_activate( $network_active ) {
+function wordpoints_activate($network_active) {
 
 	// The installer won't be registered yet.
 	wordpoints_register_installer();
 
-	WordPoints_Installables::install( 'plugin', 'wordpoints', $network_active );
+	WordPoints_Installables::install('plugin', 'wordpoints', $network_active);
 }
 
 /**
@@ -56,25 +56,25 @@ 
  */
 function wordpoints_breaking_update() {
 
-	if ( is_wordpoints_network_active() ) {
-		$wordpoints_data = get_site_option( 'wordpoints_data' );
+	if (is_wordpoints_network_active()) {
+		$wordpoints_data = get_site_option('wordpoints_data');
 	} else {
-		$wordpoints_data = get_option( 'wordpoints_data' );
+		$wordpoints_data = get_option('wordpoints_data');
 	}
 
 	// Normally this should never happen, because the plugins_loaded action won't
 	// run until the next request after WordPoints is activated and installs itself.
-	if ( empty( $wordpoints_data['version'] ) ) {
+	if (empty($wordpoints_data['version'])) {
 		return;
 	}
 
 	// The major version is determined by the first number, so we can just cast to
 	// an integer. IF the major versions are equal, we don't need to do anything.
-	if ( (int) WORDPOINTS_VERSION === (int) $wordpoints_data['version'] ) {
+	if ((int) WORDPOINTS_VERSION === (int) $wordpoints_data['version']) {
 		return;
 	}
 
-	$updater = new WordPoints_Breaking_Updater( 'wordpoints_breaking', WORDPOINTS_VERSION );
+	$updater = new WordPoints_Breaking_Updater('wordpoints_breaking', WORDPOINTS_VERSION);
 	$updater->update();
 }
 
@@ -88,27 +88,27 @@ 
  */
 function wordpoints_maintenance_shutdown_print_rand_str() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) {
+	if ( ! isset($_GET['wordpoints_module_check'])) {
 		return;
 	}
 
-	if ( is_network_admin() ) {
-		$nonce = get_site_option( 'wordpoints_module_check_nonce' );
+	if (is_network_admin()) {
+		$nonce = get_site_option('wordpoints_module_check_nonce');
 	} else {
-		$nonce = get_option( 'wordpoints_module_check_nonce' );
+		$nonce = get_option('wordpoints_module_check_nonce');
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
+	if ( ! $nonce || ! hash_equals($nonce, sanitize_key($_GET['wordpoints_module_check']))) {
 		return;
 	}
 
-	if ( is_network_admin() ) {
-		$rand_str = get_site_option( 'wordpoints_module_check_rand_str' );
+	if (is_network_admin()) {
+		$rand_str = get_site_option('wordpoints_module_check_rand_str');
 	} else {
-		$rand_str = get_option( 'wordpoints_module_check_rand_str' );
+		$rand_str = get_option('wordpoints_module_check_rand_str');
 	}
 
-	echo esc_html( $rand_str );
+	echo esc_html($rand_str);
 }
 
 /**
@@ -126,29 +126,29 @@ 
  *
  * @return array The modules whose compatibility is being checked.
  */
-function wordpoints_maintenance_filter_modules( $modules ) {
+function wordpoints_maintenance_filter_modules($modules) {
 
-	if ( ! isset( $_GET['check_module'], $_GET['wordpoints_module_check'] ) ) {
+	if ( ! isset($_GET['check_module'], $_GET['wordpoints_module_check'])) {
 		return $modules;
 	}
 
-	if ( is_network_admin() ) {
-		$nonce = get_site_option( 'wordpoints_module_check_nonce' );
+	if (is_network_admin()) {
+		$nonce = get_site_option('wordpoints_module_check_nonce');
 	} else {
-		$nonce = get_option( 'wordpoints_module_check_nonce' );
+		$nonce = get_option('wordpoints_module_check_nonce');
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
+	if ( ! $nonce || ! hash_equals($nonce, sanitize_key($_GET['wordpoints_module_check']))) {
 		return $modules;
 	}
 
 	$modules = explode(
 		','
-		, sanitize_text_field( wp_unslash( $_GET['check_module'] ) )
+		, sanitize_text_field(wp_unslash($_GET['check_module']))
 	);
 
-	if ( 'pre_site_option_wordpoints_sitewide_active_modules' === current_filter() ) {
-		$modules = array_flip( $modules );
+	if ('pre_site_option_wordpoints_sitewide_active_modules' === current_filter()) {
+		$modules = array_flip($modules);
 	}
 
 	return $modules;
@@ -175,16 +175,16 @@ 
  *
  * @return int|false False if $maybe_int couldn't reliably be converted to an integer.
  */
-function wordpoints_int( &$maybe_int ) {
+function wordpoints_int(&$maybe_int) {
 
-	$type = gettype( $maybe_int );
+	$type = gettype($maybe_int);
 
-	switch ( $type ) {
+	switch ($type) {
 
 		case 'integer': break;
 
 		case 'string':
-			if ( (string) (int) $maybe_int === $maybe_int ) {
+			if ((string) (int) $maybe_int === $maybe_int) {
 				$maybe_int = (int) $maybe_int;
 			} else {
 				$maybe_int = false;
@@ -192,7 +192,7 @@ 
 		break;
 
 		case 'double':
-			if ( (float) (int) $maybe_int === $maybe_int ) {
+			if ((float) (int) $maybe_int === $maybe_int) {
 				$maybe_int = (int) $maybe_int;
 			} else {
 				$maybe_int = false;
@@ -229,9 +229,9 @@ 
  *
  * @return int
  */
-function wordpoints_posint( &$maybe_int ) {
+function wordpoints_posint(&$maybe_int) {
 
-	$maybe_int = ( wordpoints_int( $maybe_int ) > 0 ) ? $maybe_int : false;
+	$maybe_int = (wordpoints_int($maybe_int) > 0) ? $maybe_int : false;
 	return $maybe_int;
 }
 
@@ -247,9 +247,9 @@ 
  *
  * @return int
  */
-function wordpoints_negint( &$maybe_int ) {
+function wordpoints_negint(&$maybe_int) {
 
-	$maybe_int = ( wordpoints_int( $maybe_int ) < 0 ) ? $maybe_int : false;
+	$maybe_int = (wordpoints_int($maybe_int) < 0) ? $maybe_int : false;
 	return $maybe_int;
 }
 
@@ -284,38 +284,38 @@ 
 	$request_type = 'get'
 ) {
 
-	if ( 'post' === $request_type ) {
+	if ('post' === $request_type) {
 		$request = $_POST; // WPCS: CSRF OK.
 	} else {
 		$request = $_GET;
 	}
 
-	if ( ! isset( $request[ $nonce_key ] ) ) {
+	if ( ! isset($request[$nonce_key])) {
 		return false;
 	}
 
-	if ( ! empty( $format_values ) ) {
+	if ( ! empty($format_values)) {
 
 		$values = array();
 
-		foreach ( (array) $format_values as $value ) {
+		foreach ((array) $format_values as $value) {
 
-			if ( ! isset( $request[ $value ] ) ) {
+			if ( ! isset($request[$value])) {
 				return false;
 			}
 
-			$values[] = $request[ $value ];
+			$values[] = $request[$value];
 		}
 
-		$action_format = vsprintf( $action_format, $values );
+		$action_format = vsprintf($action_format, $values);
 	}
 
 	$is_valid = wp_verify_nonce(
-		sanitize_key( $request[ $nonce_key ] )
-		, strip_tags( $action_format )
+		sanitize_key($request[$nonce_key])
+		, strip_tags($action_format)
 	);
 
-	if ( 1 === $is_valid || 2 === $is_valid ) {
+	if (1 === $is_valid || 2 === $is_valid) {
 		return $is_valid;
 	}
 
@@ -340,16 +340,16 @@ 
  *
  * @return WP_Error The sanitized error.
  */
-function wordpoints_sanitize_wp_error( $error ) {
+function wordpoints_sanitize_wp_error($error) {
 
 	$code = $error->get_error_code();
 
 	$error_data = $error->error_data;
 
-	if ( isset( $error_data[ $code ]['title'] ) ) {
+	if (isset($error_data[$code]['title'])) {
 
-		$error_data[ $code ]['title'] = wp_kses(
-			$error->error_data[ $code ]['title']
+		$error_data[$code]['title'] = wp_kses(
+			$error->error_data[$code]['title']
 			, 'wordpoints_sanitize_wp_error_title'
 		);
 
@@ -358,10 +358,10 @@ 
 
 	$all_errors = $error->errors;
 
-	foreach ( $all_errors as $code => $errors ) {
+	foreach ($all_errors as $code => $errors) {
 
-		foreach ( $errors as $key => $message ) {
-			$all_errors[ $code ][ $key ] = wp_kses(
+		foreach ($errors as $key => $message) {
+			$all_errors[$code][$key] = wp_kses(
 				$message
 				, 'wordpoints_sanitize_wp_error_message'
 			);
@@ -388,8 +388,8 @@ 
  * @return string The escaped identifier. Already quoted, do not place within
  *                backticks.
  */
-function wordpoints_escape_mysql_identifier( $identifier ) {
-	return '`' . str_replace( '`', '``', $identifier ) . '`';
+function wordpoints_escape_mysql_identifier($identifier) {
+	return '`' . str_replace('`', '``', $identifier) . '`';
 }
 
 //
@@ -412,16 +412,16 @@ 
  *
  * @return array The option value if it is an array, or an empty array if not.
  */
-function wordpoints_get_array_option( $option, $context = 'default' ) {
+function wordpoints_get_array_option($option, $context = 'default') {
 
-	switch ( $context ) {
+	switch ($context) {
 
 		case 'default':
-			$value = get_option( $option, array() );
+			$value = get_option($option, array());
 		break;
 
 		case 'site':
-			$value = get_site_option( $option, array() );
+			$value = get_site_option($option, array());
 		break;
 
 		case 'network':
@@ -431,15 +431,15 @@ 
 				, 'Using "network" as the value of $context is deprecated, use wordpoints_get_maybe_network_array_option() instead.'
 			);
 
-			$value = wordpoints_get_maybe_network_option( $option, array() );
+			$value = wordpoints_get_maybe_network_option($option, array());
 		break;
 
 		default:
-			_doing_it_wrong( __FUNCTION__, sprintf( 'Unknown option context "%s"', esc_html( $context ) ), '1.2.0' );
+			_doing_it_wrong(__FUNCTION__, sprintf('Unknown option context "%s"', esc_html($context)), '1.2.0');
 			$value = array();
 	}
 
-	if ( ! is_array( $value ) ) {
+	if ( ! is_array($value)) {
 		$value = array();
 	}
 
@@ -460,7 +460,7 @@ 
  *
  * @return mixed The option value if it exists, or $default (false by default).
  */
-function wordpoints_get_network_option( $option, $default = false ) {
+function wordpoints_get_network_option($option, $default = false) {
 
 	_deprecated_function(
 		__FUNCTION__
@@ -468,7 +468,7 @@ 
 		, 'wordpoints_get_maybe_network_option()'
 	);
 
-	return wordpoints_get_maybe_network_option( $option, null, $default );
+	return wordpoints_get_maybe_network_option($option, null, $default);
 }
 
 /**
@@ -487,7 +487,7 @@ 
  *
  * @return bool Whether the option was added successfully.
  */
-function wordpoints_add_network_option( $option, $value, $autoload = 'yes' ) {
+function wordpoints_add_network_option($option, $value, $autoload = 'yes') {
 
 	_deprecated_function(
 		__FUNCTION__
@@ -495,7 +495,7 @@ 
 		, 'wordpoints_add_maybe_network_option()'
 	);
 
-	return wordpoints_add_maybe_network_option( $option, $value, null, $autoload );
+	return wordpoints_add_maybe_network_option($option, $value, null, $autoload);
 }
 
 /**
@@ -512,7 +512,7 @@ 
  *
  * @return bool Whether the option was updated successfully.
  */
-function wordpoints_update_network_option( $option, $value ) {
+function wordpoints_update_network_option($option, $value) {
 
 	_deprecated_function(
 		__FUNCTION__
@@ -520,7 +520,7 @@ 
 		, 'wordpoints_update_maybe_network_option()'
 	);
 
-	return wordpoints_update_maybe_network_option( $option, $value );
+	return wordpoints_update_maybe_network_option($option, $value);
 }
 
 /**
@@ -536,7 +536,7 @@ 
  *
  * @return bool Whether the option was successfully deleted.
  */
-function wordpoints_delete_network_option( $option ) {
+function wordpoints_delete_network_option($option) {
 
 	_deprecated_function(
 		__FUNCTION__
@@ -544,7 +544,7 @@ 
 		, 'wordpoints_delete_maybe_network_option()'
 	);
 
-	return wordpoints_delete_maybe_network_option( $option );
+	return wordpoints_delete_maybe_network_option($option);
 }
 
 /**
@@ -563,11 +563,11 @@ 
  *
  * @return array The option value if it is an array, or an empty array if not.
  */
-function wordpoints_get_maybe_network_array_option( $option, $network = null ) {
+function wordpoints_get_maybe_network_array_option($option, $network = null) {
 
-	$value = wordpoints_get_maybe_network_option( $option, $network );
+	$value = wordpoints_get_maybe_network_option($option, $network);
 
-	if ( ! is_array( $value ) ) {
+	if ( ! is_array($value)) {
 		$value = array();
 	}
 
@@ -588,12 +588,12 @@ 
  *
  * @return mixed The option value if it exists, or $default (false by default).
  */
-function wordpoints_get_maybe_network_option( $option, $network = null, $default = false ) {
+function wordpoints_get_maybe_network_option($option, $network = null, $default = false) {
 
-	if ( $network || ( null === $network && is_wordpoints_network_active() ) ) {
-		return get_site_option( $option, $default );
+	if ($network || (null === $network && is_wordpoints_network_active())) {
+		return get_site_option($option, $default);
 	} else {
-		return get_option( $option, $default );
+		return get_option($option, $default);
 	}
 }
 
@@ -613,12 +613,12 @@ 
  *
  * @return bool Whether the option was added successfully.
  */
-function wordpoints_add_maybe_network_option( $option, $value, $network = null, $autoload = 'yes' ) {
+function wordpoints_add_maybe_network_option($option, $value, $network = null, $autoload = 'yes') {
 
-	if ( $network || ( null === $network && is_wordpoints_network_active() ) ) {
-		return add_site_option( $option, $value );
+	if ($network || (null === $network && is_wordpoints_network_active())) {
+		return add_site_option($option, $value);
 	} else {
-		return add_option( $option, $value, '', $autoload );
+		return add_option($option, $value, '', $autoload);
 	}
 }
 
@@ -636,12 +636,12 @@ 
  *
  * @return bool Whether the option was updated successfully.
  */
-function wordpoints_update_maybe_network_option( $option, $value, $network = null ) {
+function wordpoints_update_maybe_network_option($option, $value, $network = null) {
 
-	if ( $network || ( null === $network && is_wordpoints_network_active() ) ) {
-		return update_site_option( $option, $value );
+	if ($network || (null === $network && is_wordpoints_network_active())) {
+		return update_site_option($option, $value);
 	} else {
-		return update_option( $option, $value );
+		return update_option($option, $value);
 	}
 }
 
@@ -658,12 +658,12 @@ 
  *
  * @return bool Whether the option was successfully deleted.
  */
-function wordpoints_delete_maybe_network_option( $option, $network = null ) {
+function wordpoints_delete_maybe_network_option($option, $network = null) {
 
-	if ( $network || ( null === $network && is_wordpoints_network_active() ) ) {
-		return delete_site_option( $option );
+	if ($network || (null === $network && is_wordpoints_network_active())) {
+		return delete_site_option($option);
 	} else {
-		return delete_option( $option );
+		return delete_option($option);
 	}
 }
 
@@ -681,36 +681,36 @@ 
  *
  * @return string|false The comma-delimited string of values. False on failure.
  */
-function wordpoints_prepare__in( $_in, $format = '%s' ) {
+function wordpoints_prepare__in($_in, $format = '%s') {
 
 	global $wpdb;
 
 	// Validate $format.
-	$formats = array( '%d', '%s', '%f' );
+	$formats = array('%d', '%s', '%f');
 
-	if ( ! in_array( $format, $formats ) ) {
+	if ( ! in_array($format, $formats)) {
 
-		$format = esc_html( $format );
-		_doing_it_wrong( __FUNCTION__, esc_html( "WordPoints Debug Error: invalid format '{$format}', allowed values are %s, %d, and %f" ), '1.0.0' );
+		$format = esc_html($format);
+		_doing_it_wrong(__FUNCTION__, esc_html("WordPoints Debug Error: invalid format '{$format}', allowed values are %s, %d, and %f"), '1.0.0');
 
 		$format = '%s';
 	}
 
-	$_in = array_unique( $_in );
+	$_in = array_unique($_in);
 
-	$count = count( $_in );
+	$count = count($_in);
 
-	if ( 0 === $count ) {
+	if (0 === $count) {
 
-		_doing_it_wrong( __FUNCTION__, 'WordPoints Debug Error: empty array passed as first parameter', '1.0.0' );
+		_doing_it_wrong(__FUNCTION__, 'WordPoints Debug Error: empty array passed as first parameter', '1.0.0');
 
 		return false;
 	}
 
 	// String a bunch of format signs together, one for each value in $_in.
-	$in = $format . str_repeat( ",{$format}", $count - 1 );
+	$in = $format . str_repeat(",{$format}", $count - 1);
 
-	return $wpdb->prepare( $in, $_in ); // WPCS: unprepared SQL OK
+	return $wpdb->prepare($in, $_in); // WPCS: unprepared SQL OK
 }
 
 //
@@ -738,7 +738,7 @@ 
  * }
  * @param array $args Arguments to pass to get_post_types(). Default is array().
  */
-function wordpoints_list_post_types( $options, $args = array() ) {
+function wordpoints_list_post_types($options, $args = array()) {
 
 	$defaults = array(
 		'name'     => '',
@@ -748,18 +748,18 @@ 
 		'filter'   => null,
 	);
 
-	$options = array_merge( $defaults, $options );
+	$options = array_merge($defaults, $options);
 
-	echo '<select class="' . esc_attr( $options['class'] ) . '" name="' . esc_attr( $options['name'] ) . '" id="' . esc_attr( $options['id'] ) . '">';
-	echo '<option value="ALL"' . selected( $options['selected'], 'ALL', false ) . '>' . esc_html_x( 'Any', 'post type', 'wordpoints' ) . '</option>';
+	echo '<select class="' . esc_attr($options['class']) . '" name="' . esc_attr($options['name']) . '" id="' . esc_attr($options['id']) . '">';
+	echo '<option value="ALL"' . selected($options['selected'], 'ALL', false) . '>' . esc_html_x('Any', 'post type', 'wordpoints') . '</option>';
 
-	foreach ( get_post_types( $args, 'objects' ) as $post_type ) {
+	foreach (get_post_types($args, 'objects') as $post_type) {
 
-		if ( isset( $options['filter'] ) && ! call_user_func( $options['filter'], $post_type ) ) {
+		if (isset($options['filter']) && ! call_user_func($options['filter'], $post_type)) {
 			continue;
 		}
 
-		echo '<option value="' . esc_attr( $post_type->name ) . '"' . selected( $options['selected'], $post_type->name, false ) . '>' . esc_html( $post_type->label ) . '</option>';
+		echo '<option value="' . esc_attr($post_type->name) . '"' . selected($options['selected'], $post_type->name, false) . '>' . esc_html($post_type->label) . '</option>';
 	}
 
 	echo '</select>';
@@ -781,7 +781,7 @@ 
 	require_once ABSPATH . '/wp-admin/includes/plugin.php';
 
 	$network_active = is_plugin_active_for_network(
-		plugin_basename( WORDPOINTS_DIR . 'wordpoints.php' )
+		plugin_basename(WORDPOINTS_DIR . 'wordpoints.php')
 	);
 
 	/**
@@ -794,7 +794,7 @@ 
 	 *
 	 * @param bool $network_active Whether WordPoints is network activated.
 	 */
-	return apply_filters( 'is_wordpoints_network_active', $network_active );
+	return apply_filters('is_wordpoints_network_active', $network_active);
 }
 
 /**
@@ -810,7 +810,7 @@ 
  *
  * @return array An array of user IDs to exclude in the current $context.
  */
-function wordpoints_get_excluded_users( $context ) {
+function wordpoints_get_excluded_users($context) {
 
 	$user_ids = wordpoints_get_maybe_network_array_option(
 		'wordpoints_excluded_users'
@@ -826,7 +826,7 @@ 
 	 * @param int[]  $user_ids The ids of the user's to be excluded.
 	 * @param string $context  The context in which the function is being called.
 	 */
-	return apply_filters( 'wordpoints_excluded_users', $user_ids, $context );
+	return apply_filters('wordpoints_excluded_users', $user_ids, $context);
 }
 
 /**
@@ -842,18 +842,18 @@ 
  *
  * @return string The error message.
  */
-function wordpoints_shortcode_error( $message ) {
+function wordpoints_shortcode_error($message) {
 
 	if (
-		( get_post() && current_user_can( 'edit_post', get_the_ID() ) )
-		|| current_user_can( 'manage_options' )
+		(get_post() && current_user_can('edit_post', get_the_ID()))
+		|| current_user_can('manage_options')
 	) {
 
-		if ( is_wp_error( $message ) ) {
+		if (is_wp_error($message)) {
 			$message = $message->get_error_message();
 		}
 
-		return '<p class="wordpoints-shortcode-error">' . esc_html__( 'Shortcode error:', 'wordpoints' ) . ' ' . wp_kses( $message, 'wordpoints_shortcode_error' ) . '</p>';
+		return '<p class="wordpoints-shortcode-error">' . esc_html__('Shortcode error:', 'wordpoints') . ' ' . wp_kses($message, 'wordpoints_shortcode_error') . '</p>';
 	}
 
 	return '';
@@ -886,14 +886,14 @@ 
  * @param array $capabilities The capabilities to add as keys, WP core counterparts
  *                            as values.
  */
-function wordpoints_add_custom_caps( $capabilities ) {
+function wordpoints_add_custom_caps($capabilities) {
 
 	/** @var WP_Role $role */
-	foreach ( wp_roles()->role_objects as $role ) {
+	foreach (wp_roles()->role_objects as $role) {
 
-		foreach ( $capabilities as $custom_cap => $core_cap ) {
-			if ( $role->has_cap( $core_cap ) ) {
-				$role->add_cap( $custom_cap );
+		foreach ($capabilities as $custom_cap => $core_cap) {
+			if ($role->has_cap($core_cap)) {
+				$role->add_cap($custom_cap);
 			}
 		}
 	}
@@ -908,12 +908,12 @@ 
  *
  * @param string[] $capabilities The list of capabilities to remove.
  */
-function wordpoints_remove_custom_caps( $capabilities ) {
+function wordpoints_remove_custom_caps($capabilities) {
 
 	/** @var WP_Role $role */
-	foreach ( wp_roles()->role_objects as $role ) {
-		foreach ( $capabilities as $custom_cap ) {
-			$role->remove_cap( $custom_cap );
+	foreach (wp_roles()->role_objects as $role) {
+		foreach ($capabilities as $custom_cap) {
+			$role->remove_cap($custom_cap);
 		}
 	}
 }
@@ -931,13 +931,13 @@ 
  *
  * @return array The user's capabilities.
  */
-function wordpoints_map_custom_meta_caps( $caps, $cap, $user_id ) {
+function wordpoints_map_custom_meta_caps($caps, $cap, $user_id) {
 
-	switch ( $cap ) {
+	switch ($cap) {
 		case 'install_wordpoints_modules':
-			if ( defined( 'DISALLOW_FILE_MODS' ) && DISALLOW_FILE_MODS ) {
+			if (defined('DISALLOW_FILE_MODS') && DISALLOW_FILE_MODS) {
 				$caps[] = 'do_not_allow';
-			} elseif ( is_multisite() && ! is_super_admin( $user_id ) ) {
+			} elseif (is_multisite() && ! is_super_admin($user_id)) {
 				$caps[] = 'do_not_allow';
 			} else {
 				$caps[] = $cap;
@@ -959,9 +959,9 @@ 
  *
  * @return bool Whether the component was registered.
  */
-function wordpoints_component_register( $args ) {
+function wordpoints_component_register($args) {
 
-	return WordPoints_Components::instance()->register( $args );
+	return WordPoints_Components::instance()->register($args);
 }
 
 /**
@@ -975,9 +975,9 @@ 
  *
  * @return bool Whether the component is active.
  */
-function wordpoints_component_is_active( $slug ) {
+function wordpoints_component_is_active($slug) {
 
-	return WordPoints_Components::instance()->is_active( $slug );
+	return WordPoints_Components::instance()->is_active($slug);
 }
 
 /**
@@ -994,12 +994,12 @@ 
 	wordpoints_component_register(
 		array(
 			'slug'          => 'points',
-			'name'          => _x( 'Points', 'component name', 'wordpoints' ),
+			'name'          => _x('Points', 'component name', 'wordpoints'),
 			'version'       => WORDPOINTS_VERSION,
-			'author'        => _x( 'WordPoints', 'component author', 'wordpoints' ),
+			'author'        => _x('WordPoints', 'component author', 'wordpoints'),
 			'author_uri'    => 'https://wordpoints.org/',
 			'component_uri' => 'https://wordpoints.org/',
-			'description'   => __( 'Enables a points system for your site.', 'wordpoints' ),
+			'description'   => __('Enables a points system for your site.', 'wordpoints'),
 			'file'          => WORDPOINTS_DIR . 'components/points/points.php',
 			'un_installer'  => WORDPOINTS_DIR . 'components/points/includes/class-un-installer.php',
 		)
@@ -1018,12 +1018,12 @@ 
 	wordpoints_component_register(
 		array(
 			'slug'           => 'ranks',
-			'name'           => _x( 'Ranks', 'component name', 'wordpoints' ),
+			'name'           => _x('Ranks', 'component name', 'wordpoints'),
 			'version'        => WORDPOINTS_VERSION,
-			'author'         => _x( 'WordPoints', 'component author', 'wordpoints' ),
+			'author'         => _x('WordPoints', 'component author', 'wordpoints'),
 			'author_uri'     => 'https://wordpoints.org/',
 			'component_uri'  => 'https://wordpoints.org/',
-			'description'    => __( 'Assign users ranks based on their points levels.', 'wordpoints' ),
+			'description'    => __('Assign users ranks based on their points levels.', 'wordpoints'),
 			'file'           => WORDPOINTS_DIR . 'components/ranks/ranks.php',
 			'un_installer'   => WORDPOINTS_DIR . 'components/ranks/includes/class-un-installer.php',
 		)
@@ -1040,11 +1040,11 @@ 
  */
 function wordpoints_init_cache_groups() {
 
-	if ( function_exists( 'wp_cache_add_non_persistent_groups' ) ) {
-		wp_cache_add_non_persistent_groups( array( 'wordpoints_modules' ) );
+	if (function_exists('wp_cache_add_non_persistent_groups')) {
+		wp_cache_add_non_persistent_groups(array('wordpoints_modules'));
 	}
 
-	if ( function_exists( 'wp_cache_add_global_groups' ) ) {
+	if (function_exists('wp_cache_add_global_groups')) {
 
 		wp_cache_add_global_groups(
 			array(
@@ -1078,7 +1078,7 @@ 
  */
 function wordpoints_load_textdomain() {
 
-	load_plugin_textdomain( 'wordpoints', false, plugin_basename( WORDPOINTS_DIR ) . '/languages/' );
+	load_plugin_textdomain('wordpoints', false, plugin_basename(WORDPOINTS_DIR) . '/languages/');
 }
 
 /**
@@ -1096,8 +1096,8 @@ 
  *
  * @return string The hash.
  */
-function wordpoints_hash( $data ) {
-	return hash( 'sha256', $data );
+function wordpoints_hash($data) {
+	return hash('sha256', $data);
 }
 
 /**
@@ -1110,19 +1110,19 @@ 
  *
  * @return object|false The constructed object, or false if to many args were passed.
  */
-function wordpoints_construct_class_with_args( $class_name, array $args ) {
+function wordpoints_construct_class_with_args($class_name, array $args) {
 
-	switch ( count( $args ) ) {
+	switch (count($args)) {
 		case 0:
 			return new $class_name();
 		case 1:
-			return new $class_name( $args[0] );
+			return new $class_name($args[0]);
 		case 2:
-			return new $class_name( $args[0], $args[1] );
+			return new $class_name($args[0], $args[1]);
 		case 3:
-			return new $class_name( $args[0], $args[1], $args[2] );
+			return new $class_name($args[0], $args[1], $args[2]);
 		case 4:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3] );
+			return new $class_name($args[0], $args[1], $args[2], $args[3]);
 		default:
 			return false;
 	}
@@ -1137,13 +1137,13 @@ 
  *
  * @return bool Whether the function is disabled.
  */
-function wordpoints_is_function_disabled( $function ) {
+function wordpoints_is_function_disabled($function) {
 
-	if ( 'set_time_limit' === $function && ini_get( 'safe_mode' ) ) {
+	if ('set_time_limit' === $function && ini_get('safe_mode')) {
 		return true;
 	}
 
-	return in_array( $function, explode( ',', ini_get( 'disable_functions' ) ) );
+	return in_array($function, explode(',', ini_get('disable_functions')));
 }
 
 // EOF

src/components/points/points.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
  */
 include_once WORDPOINTS_DIR . 'components/points/includes/deprecated.php';
 
-if ( is_admin() ) {
+if (is_admin()) {
 
 	// We are on the administration side of the site.