WordPoints / wordpoints

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

Spacing +39 added lines, -39 removed lines

@@ -75,7 +75,7 @@ 
 	 *        @type string $slug The slug of this rank type.
 	 * }
 	 */
-	public function __construct( array $args ) {
+	public function __construct(array $args) {
 
 		$this->slug = $args['slug'];
 	}
@@ -109,7 +109,7 @@ 
 	 * @return array|WP_Error|false The validated metadata, or false or a WP_Error
 	 *                              if it should't be saved.
 	 */
-	abstract public function validate_rank_meta( array $meta );
+	abstract public function validate_rank_meta(array $meta);
 
 	/**
 	 * Determine if a user meets the requirements for a rank of this type.
@@ -126,7 +126,7 @@ 
 	 *
 	 * @return bool Whether the user meets the requirements for this rank.
 	 */
-	abstract protected function can_transition_user_rank( $user_id, $rank, array $args );
+	abstract protected function can_transition_user_rank($user_id, $rank, array $args);
 
 	//
 	// Final Public Methods.
@@ -185,47 +185,47 @@ 
 		array $args = array()
 	) {
 
-		$args = array_merge( array( 'placeholders' => false ), $args );
+		$args = array_merge(array('placeholders' => false), $args);
 
-		foreach ( $this->meta_fields as $name => $field ) {
+		foreach ($this->meta_fields as $name => $field) {
 
 			// If we aren't using placeholders, calculate the value. Hidden fields
 			// never use placeholders.
-			if ( ! $args['placeholders'] || 'hidden' === $field['type'] ) {
+			if ( ! $args['placeholders'] || 'hidden' === $field['type']) {
 
 				// Default to the default value.
 				$value = $field['default'];
 
 				// If the value is set use that instead.
-				if ( isset( $meta[ $name ] ) ) {
-					$value = $meta[ $name ];
+				if (isset($meta[$name])) {
+					$value = $meta[$name];
 				}
 			}
 
-			switch ( $field['type'] ) {
+			switch ($field['type']) {
 
 				case 'hidden':
 				case 'number':
 				case 'text':
-					if ( isset( $field['label'] ) ) {
+					if (isset($field['label'])) {
 						?><p class="description description-thin"><label><?php
-						echo esc_html( $field['label'] );
+						echo esc_html($field['label']);
 					}
 
 					?>
 					<input
-						type="<?php echo esc_attr( $field['type'] ); ?>"
-						name="<?php echo esc_attr( $name ); ?>"
-						<?php if ( $args['placeholders'] && 'hidden' !== $field['type'] ) : ?>
-						value="<% if ( typeof <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); // WPCS: XSS OK ?> !== "undefined" ) { print( <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); // WPCS: XSS OK ?> ); } %>"
+						type="<?php echo esc_attr($field['type']); ?>"
+						name="<?php echo esc_attr($name); ?>"
+						<?php if ($args['placeholders'] && 'hidden' !== $field['type']) : ?>
+						value="<% if ( typeof <?php echo preg_replace('/[^a-z0-9_]/i', '', $name); // WPCS: XSS OK ?> !== "undefined" ) { print( <?php echo preg_replace('/[^a-z0-9_]/i', '', $name); // WPCS: XSS OK ?> ); } %>"
 						<?php else : ?>
-						value="<?php echo esc_attr( $value ); ?>"
+						value="<?php echo esc_attr($value); ?>"
 						<?php endif; ?>
 						class="widefat"
 					/>
 					<?php
 
-					if ( isset( $field['label'] ) ) {
+					if (isset($field['label'])) {
 						?></label></p><?php
 					}
 				break;
@@ -235,7 +235,7 @@ 
 						__METHOD__
 						, sprintf(
 							'WordPoints Error: Unknown field type "%s".'
-							, esc_html( $field['type'] )
+							, esc_html($field['type'])
 						)
 						, '1.7.0'
 					);
@@ -256,21 +256,21 @@ 
 	 *
 	 * @return WordPoints_Rank The highest rank that the user can be increased to.
 	 */
-	final public function maybe_increase_user_rank( $user_id, $rank, array $args = array() ) {
+	final public function maybe_increase_user_rank($user_id, $rank, array $args = array()) {
 
-		$next_rank = $rank->get_adjacent( 1 );
+		$next_rank = $rank->get_adjacent(1);
 
-		if ( ! $next_rank ) {
+		if ( ! $next_rank) {
 			return $rank;
 		}
 
-		$rank_type = WordPoints_Rank_Types::get_type( $next_rank->type );
+		$rank_type = WordPoints_Rank_Types::get_type($next_rank->type);
 
-		if ( ! $rank_type->can_transition_user_rank( $user_id, $next_rank, $args ) ) {
+		if ( ! $rank_type->can_transition_user_rank($user_id, $next_rank, $args)) {
 			return $rank;
 		}
 
-		return $this->maybe_increase_user_rank( $user_id, $next_rank, $args );
+		return $this->maybe_increase_user_rank($user_id, $next_rank, $args);
 	}
 
 	/**
@@ -284,21 +284,21 @@ 
 	 *
 	 * @return WordPoints_Rank The highest rank that the user can have.
 	 */
-	final public function maybe_decrease_user_rank( $user_id, $rank, array $args = array() ) {
+	final public function maybe_decrease_user_rank($user_id, $rank, array $args = array()) {
 
-		$rank_type = WordPoints_Rank_Types::get_type( $rank->type );
+		$rank_type = WordPoints_Rank_Types::get_type($rank->type);
 
-		if ( $rank_type->can_transition_user_rank( $user_id, $rank, $args ) ) {
+		if ($rank_type->can_transition_user_rank($user_id, $rank, $args)) {
 			return $rank;
 		}
 
 		$previous_rank = $rank->get_adjacent( -1 );
 
-		if ( ! $previous_rank ) {
+		if ( ! $previous_rank) {
 			return $rank;
 		}
 
-		return $this->maybe_decrease_user_rank( $user_id, $previous_rank, $args );
+		return $this->maybe_decrease_user_rank($user_id, $previous_rank, $args);
 	}
 
 	//
@@ -322,28 +322,28 @@ 
 
 		$groups = WordPoints_Rank_Groups::get();
 
-		foreach ( $groups as $group_slug => $group ) {
+		foreach ($groups as $group_slug => $group) {
 
-			if ( ! WordPoints_Rank_Groups::is_type_registered_for_group( $this->slug, $group_slug ) ) {
+			if ( ! WordPoints_Rank_Groups::is_type_registered_for_group($this->slug, $group_slug)) {
 				continue;
 			}
 
-			$rank_id = wordpoints_get_user_rank( $user_id, $group_slug );
+			$rank_id = wordpoints_get_user_rank($user_id, $group_slug);
 
-			$rank = wordpoints_get_rank( $rank_id );
+			$rank = wordpoints_get_rank($rank_id);
 
-			if ( ! $rank ) {
+			if ( ! $rank) {
 				continue;
 			}
 
-			if ( $increase ) {
-				$new_rank = $this->maybe_increase_user_rank( $user_id, $rank, $args );
+			if ($increase) {
+				$new_rank = $this->maybe_increase_user_rank($user_id, $rank, $args);
 			} else {
-				$new_rank = $this->maybe_decrease_user_rank( $user_id, $rank, $args );
+				$new_rank = $this->maybe_decrease_user_rank($user_id, $rank, $args);
 			}
 
-			if ( $new_rank !== $rank ) {
-				wordpoints_update_user_rank( $user_id, $new_rank->ID );
+			if ($new_rank !== $rank) {
+				wordpoints_update_user_rank($user_id, $new_rank->ID);
 			}
 		}
 	}

Braces +5 added lines, -2 removed lines

@@ -218,8 +218,11 @@
 						name="<?php echo esc_attr( $name ); ?>"
 						<?php if ( $args['placeholders'] && 'hidden' !== $field['type'] ) : ?>
 						value="<% if ( typeof <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); // WPCS: XSS OK ?> !== "undefined" ) { print( <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); // WPCS: XSS OK ?> ); } %>"
-						<?php else : ?>
-						value="<?php echo esc_attr( $value ); ?>"
+						<?php else {
+	: ?>
+						value="<?php echo esc_attr( $value );
+}
+?>"
 						<?php endif; ?>
 						class="widefat"
 					/>

src/components/ranks/includes/integration/points.php

Spacing +2 added lines, -2 removed lines

@@ -12,13 +12,13 @@
  *
  * @since 2.1.0
  */
-include_once( WORDPOINTS_DIR . '/components/ranks/includes/integration/points/functions.php' );
+include_once(WORDPOINTS_DIR . '/components/ranks/includes/integration/points/functions.php');
 
 /**
  * Actions and filters to integrate with the points component.
  *
  * @since 2.1.0
  */
-include_once( WORDPOINTS_DIR . '/components/ranks/includes/integration/points/filters.php' );
+include_once(WORDPOINTS_DIR . '/components/ranks/includes/integration/points/filters.php');
 
 // EOF

src/components/ranks/includes/integration/points/filters.php

Spacing +6 added lines, -6 removed lines

@@ -7,15 +7,15 @@
  * @since 2.1.0
  */
 
-add_filter( 'wordpoints_points_widget_text', 'wordpoints_ranks_points_widget_text_filter', 30, 2 );
-add_action( 'wordpoints_my_points_widget_below_text_field', 'wordpoints_ranks_my_points_widget_below_text_field' );
+add_filter('wordpoints_points_widget_text', 'wordpoints_ranks_points_widget_text_filter', 30, 2);
+add_action('wordpoints_my_points_widget_below_text_field', 'wordpoints_ranks_my_points_widget_below_text_field');
 
-add_filter( 'wordpoints_points_top_users_username', 'wordpoints_ranks_points_top_users_username_filter', 10, 4 );
+add_filter('wordpoints_points_top_users_username', 'wordpoints_ranks_points_top_users_username_filter', 10, 4);
 
-add_filter( 'shortcode_atts_wordpoints_user_rank', 'wordpoints_user_rank_shortcode_points_type_attr', 10, 3 );
+add_filter('shortcode_atts_wordpoints_user_rank', 'wordpoints_user_rank_shortcode_points_type_attr', 10, 3);
 
-add_action( 'wordpoints_ranks_register', 'wordpoints_register_points_ranks' );
+add_action('wordpoints_ranks_register', 'wordpoints_register_points_ranks');
 
-add_action( 'add_meta_boxes', 'wordpoints_ranks_add_points_types_meta_box_ranks', 20 );
+add_action('add_meta_boxes', 'wordpoints_ranks_add_points_types_meta_box_ranks', 20);
 
 // EOF

src/components/ranks/includes/rank-types.php

Spacing +2 added lines, -2 removed lines

@@ -8,13 +8,13 @@
  * @deprecated 2.2.0
  */
 
-_deprecated_file( __FILE__, '2.2.0' );
+_deprecated_file(__FILE__, '2.2.0');
 
 /**
  * The base rank type class.
  *
  * @since 1.7.0
  */
-include_once( WORDPOINTS_DIR . '/components/ranks/includes/rank-types/base.php' );
+include_once(WORDPOINTS_DIR . '/components/ranks/includes/rank-types/base.php');
 
 // EOF

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

Spacing +24 added lines, -24 removed lines

@@ -32,9 +32,9 @@ 
 	 *
 	 * @return bool Whether the group is registered.
 	 */
-	public static function is_group_registered( $slug ) {
+	public static function is_group_registered($slug) {
 
-		return isset( self::$groups[ $slug ] );
+		return isset(self::$groups[$slug]);
 	}
 
 	/**
@@ -47,17 +47,17 @@ 
 	 *
 	 * @return bool True if registered, false if already registered.
 	 */
-	public static function register_group( $slug, $data ) {
+	public static function register_group($slug, $data) {
 
-		if ( self::is_group_registered( $slug ) ) {
+		if (self::is_group_registered($slug)) {
 			return false;
 		}
 
-		self::$groups[ $slug ] = new WordPoints_Rank_Group( $slug, $data );
+		self::$groups[$slug] = new WordPoints_Rank_Group($slug, $data);
 
 		// If this is a brand new group, create the base rank.
-		if ( ! self::$groups[ $slug ]->get_base_rank() ) {
-			wordpoints_add_rank( '', 'base', $slug, 0 );
+		if ( ! self::$groups[$slug]->get_base_rank()) {
+			wordpoints_add_rank('', 'base', $slug, 0);
 		}
 
 		return true;
@@ -72,13 +72,13 @@ 
 	 *
 	 * @return bool True if deregistered, false if not registered.
 	 */
-	public static function deregister_group( $slug ) {
+	public static function deregister_group($slug) {
 
-		if ( ! self::is_group_registered( $slug ) ) {
+		if ( ! self::is_group_registered($slug)) {
 			return false;
 		}
 
-		unset( self::$groups[ $slug ] );
+		unset(self::$groups[$slug]);
 
 		return true;
 	}
@@ -92,7 +92,7 @@ 
 	 */
 	public static function get() {
 
-		if ( ! did_action( 'wordpoints_ranks_register' ) ) {
+		if ( ! did_action('wordpoints_ranks_register')) {
 			_doing_it_wrong(
 				__METHOD__
 				, 'Rank groups should not be retreived until after they are all registered.'
@@ -112,13 +112,13 @@ 
 	 *
 	 * @return WordPoints_Rank_Group|false The handler object, or false.
 	 */
-	public static function get_group( $group ) {
+	public static function get_group($group) {
 
-		if ( ! self::is_group_registered( $group ) ) {
+		if ( ! self::is_group_registered($group)) {
 			return false;
 		}
 
-		return self::$groups[ $group ];
+		return self::$groups[$group];
 	}
 
 	/**
@@ -135,13 +135,13 @@ 
 	 *
 	 * @return bool Whether the rank type is registered for the group.
 	 */
-	public static function is_type_registered_for_group( $type, $group ) {
+	public static function is_type_registered_for_group($type, $group) {
 
-		if ( ! self::is_group_registered( $group ) ) {
+		if ( ! self::is_group_registered($group)) {
 			return false;
 		}
 
-		return self::$groups[ $group ]->has_type( $type );
+		return self::$groups[$group]->has_type($type);
 	}
 
 	/**
@@ -156,13 +156,13 @@ 
 	 *
 	 * @return bool True on success. False if the group is invalid.
 	 */
-	public static function register_type_for_group( $type, $group ) {
+	public static function register_type_for_group($type, $group) {
 
-		if ( ! self::is_group_registered( $group ) ) {
+		if ( ! self::is_group_registered($group)) {
 			return false;
 		}
 
-		return self::$groups[ $group ]->add_type( $type );
+		return self::$groups[$group]->add_type($type);
 	}
 
 	/**
@@ -176,17 +176,17 @@ 
 	 * @return bool True on success. False if the group is invalid, or the rank type
 	 *              isn't registered for it.
 	 */
-	public static function deregister_type_for_group( $type, $group ) {
+	public static function deregister_type_for_group($type, $group) {
 
-		if ( ! self::is_group_registered( $group ) ) {
+		if ( ! self::is_group_registered($group)) {
 			return false;
 		}
 
-		if ( ! self::is_type_registered_for_group( $type, $group ) ) {
+		if ( ! self::is_type_registered_for_group($type, $group)) {
 			return false;
 		}
 
-		return self::$groups[ $group ]->remove_type( $type );
+		return self::$groups[$group]->remove_type($type);
 	}
 }
 

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

Spacing +19 added lines, -19 removed lines

@@ -35,9 +35,9 @@ 
 	 *
 	 * @return bool Whether this rank type is registered.
 	 */
-	public static function is_type_registered( $type ) {
+	public static function is_type_registered($type) {
 
-		return isset( self::$types[ $type ] );
+		return isset(self::$types[$type]);
 	}
 
 	/**
@@ -54,15 +54,15 @@ 
 	 *
 	 * @return bool Whether the rank type was registered.
 	 */
-	public static function register_type( $type, $class, array $args = array() ) {
+	public static function register_type($type, $class, array $args = array()) {
 
-		if ( self::is_type_registered( $type ) ) {
+		if (self::is_type_registered($type)) {
 			return false;
 		}
 
 		$args['slug'] = $type;
 
-		self::$types[ $type ] = new $class( $args );
+		self::$types[$type] = new $class($args);
 
 		return true;
 	}
@@ -78,23 +78,23 @@ 
 	 *
 	 * @return bool Whether the rank type was deregistered.
 	 */
-	public static function deregister_type( $type ) {
+	public static function deregister_type($type) {
 
-		if ( 'base' === $type ) {
+		if ('base' === $type) {
 			return false;
 		}
 
-		if ( ! self::is_type_registered( $type ) ) {
+		if ( ! self::is_type_registered($type)) {
 			return false;
 		}
 
-		self::$types[ $type ]->destruct();
+		self::$types[$type]->destruct();
 
-		foreach ( WordPoints_Rank_Groups::get() as $group ) {
-			$group->remove_type( $type );
+		foreach (WordPoints_Rank_Groups::get() as $group) {
+			$group->remove_type($type);
 		}
 
-		unset( self::$types[ $type ] );
+		unset(self::$types[$type]);
 
 		return true;
 	}
@@ -108,7 +108,7 @@ 
 	 */
 	public static function get() {
 
-		if ( ! did_action( 'wordpoints_ranks_register' ) ) {
+		if ( ! did_action('wordpoints_ranks_register')) {
 			_doing_it_wrong(
 				__METHOD__
 				, 'Ranks should not be retreived until after they are all registered.'
@@ -128,13 +128,13 @@ 
 	 *
 	 * @return WordPoints_Rank_Type|false The hander object, or false.
 	 */
-	public static function get_type( $type ) {
+	public static function get_type($type) {
 
-		if ( ! self::is_type_registered( $type ) ) {
+		if ( ! self::is_type_registered($type)) {
 			return false;
 		}
 
-		return self::$types[ $type ];
+		return self::$types[$type];
 	}
 
 	/**
@@ -146,21 +146,21 @@ 
 	 */
 	public static function init() {
 
-		self::register_type( 'base', 'WordPoints_Base_Rank_Type' );
+		self::register_type('base', 'WordPoints_Base_Rank_Type');
 
 		/**
 		 * Ranks should not register later than this action.
 		 *
 		 * @since 1.7.0
 		 */
-		do_action( 'wordpoints_ranks_register' );
+		do_action('wordpoints_ranks_register');
 
 		/**
 		 * All ranks are registered.
 		 *
 		 * @since 1.7.0
 		 */
-		do_action( 'wordpoints_ranks_registered' );
+		do_action('wordpoints_ranks_registered');
 	}
 }
 

src/components/ranks/includes/ranks.php

Spacing +132 added lines, -132 removed lines

@@ -26,27 +26,27 @@ 
  *
  * @return int|false|WP_Error The ID of the inserted rank, or false or WP_Error on failure.
  */
-function wordpoints_add_rank( $name, $type, $group, $position, array $meta = array() ) {
+function wordpoints_add_rank($name, $type, $group, $position, array $meta = array()) {
 
 	global $wpdb;
 
-	if ( ! WordPoints_Rank_Groups::is_type_registered_for_group( $type, $group ) ) {
+	if ( ! WordPoints_Rank_Groups::is_type_registered_for_group($type, $group)) {
 		return false;
 	}
 
-	$rank_type = WordPoints_Rank_Types::get_type( $type );
+	$rank_type = WordPoints_Rank_Types::get_type($type);
 
-	if ( ! $rank_type ) {
+	if ( ! $rank_type) {
 		return false;
 	}
 
-	$meta = $rank_type->validate_rank_meta( $meta );
-	if ( false === $meta || is_wp_error( $meta ) ) {
+	$meta = $rank_type->validate_rank_meta($meta);
+	if (false === $meta || is_wp_error($meta)) {
 		return $meta;
 	}
 
-	if ( 'utf8' === $wpdb->get_col_charset( $wpdb->wordpoints_ranks, 'name' ) ) {
-		$name = wp_encode_emoji( $name );
+	if ('utf8' === $wpdb->get_col_charset($wpdb->wordpoints_ranks, 'name')) {
+		$name = wp_encode_emoji($name);
 	}
 
 	$inserted = $wpdb->insert(
@@ -60,17 +60,17 @@ 
 		)
 	);
 
-	if ( ! $inserted ) {
+	if ( ! $inserted) {
 		return false;
 	}
 
 	$rank_id = (int) $wpdb->insert_id;
 
-	foreach ( $meta as $meta_key => $meta_value ) {
-		wordpoints_add_rank_meta( $rank_id, $meta_key, $meta_value );
+	foreach ($meta as $meta_key => $meta_value) {
+		wordpoints_add_rank_meta($rank_id, $meta_key, $meta_value);
 	}
 
-	WordPoints_Rank_Groups::get_group( $group )->add_rank( $rank_id, $position );
+	WordPoints_Rank_Groups::get_group($group)->add_rank($rank_id, $position);
 
 	/**
 	 * Perform actions when a rank is added.
@@ -79,7 +79,7 @@ 
 	 *
 	 * @param int $rank_id The ID of the rank that was added.
 	 */
-	do_action( 'wordpoints_add_rank', $rank_id );
+	do_action('wordpoints_add_rank', $rank_id);
 
 	return $rank_id;
 }
@@ -93,13 +93,13 @@ 
  *
  * @return bool True if the rank was deleted, false otherwise.
  */
-function wordpoints_delete_rank( $id ) {
+function wordpoints_delete_rank($id) {
 
 	global $wpdb;
 
-	$rank = wordpoints_get_rank( $id );
+	$rank = wordpoints_get_rank($id);
 
-	if ( ! $rank ) {
+	if ( ! $rank) {
 		return false;
 	}
 
@@ -110,15 +110,15 @@ 
 	 *
 	 * @param WordPoints_Rank $rank The rank that is being deleted.
 	 */
-	do_action( 'wordpoints_pre_delete_rank', $rank );
+	do_action('wordpoints_pre_delete_rank', $rank);
 
 	$deleted = $wpdb->delete(
 		$wpdb->wordpoints_ranks
-		, array( 'id' => $id )
-		, array( '%d' )
+		, array('id' => $id)
+		, array('%d')
 	);
 
-	if ( ! $deleted ) {
+	if ( ! $deleted) {
 		return false;
 	}
 
@@ -133,12 +133,12 @@ 
 		)
 	); // WPCS: cache pass.
 
-	WordPoints_Rank_Groups::get_group( $rank->rank_group )->remove_rank( $id );
+	WordPoints_Rank_Groups::get_group($rank->rank_group)->remove_rank($id);
 
-	wp_cache_delete( $id, 'wordpoints_ranks' );
+	wp_cache_delete($id, 'wordpoints_ranks');
 
-	foreach ( $rank_meta_ids as $mid ) {
-		delete_metadata_by_mid( 'wordpoints_rank', $mid );
+	foreach ($rank_meta_ids as $mid) {
+		delete_metadata_by_mid('wordpoints_rank', $mid);
 	}
 
 	/**
@@ -148,7 +148,7 @@ 
 	 *
 	 * @param int $rank_id The ID of the rank that was deleted.
 	 */
-	do_action( 'wordpoints_delete_rank', $id );
+	do_action('wordpoints_delete_rank', $id);
 
 	return true;
 }
@@ -170,28 +170,28 @@ 
  *
  * @return bool|WP_Error True the rank was updated successfully, or false/WP_Error on failure.
  */
-function wordpoints_update_rank( $id, $name, $type, $group, $position, array $meta = array() ) {
+function wordpoints_update_rank($id, $name, $type, $group, $position, array $meta = array()) {
 
 	global $wpdb;
 
-	$rank = wordpoints_get_rank( $id );
+	$rank = wordpoints_get_rank($id);
 
-	if ( ! $rank ) {
+	if ( ! $rank) {
 		return false;
 	}
 
-	if ( ! WordPoints_Rank_Groups::is_type_registered_for_group( $type, $group ) ) {
+	if ( ! WordPoints_Rank_Groups::is_type_registered_for_group($type, $group)) {
 		return false;
 	}
 
-	$rank_type = WordPoints_Rank_Types::get_type( $type );
+	$rank_type = WordPoints_Rank_Types::get_type($type);
 
-	if ( ! $rank_type ) {
+	if ( ! $rank_type) {
 		return false;
 	}
 
-	$meta = $rank_type->validate_rank_meta( $meta );
-	if ( false === $meta || is_wp_error( $meta ) ) {
+	$meta = $rank_type->validate_rank_meta($meta);
+	if (false === $meta || is_wp_error($meta)) {
 		return $meta;
 	}
 
@@ -207,50 +207,50 @@ 
 	 * @param int             $position The new position this rank should have in the group.
 	 * @param array           $meta     The new metadata for the rank.
 	 */
-	do_action( 'wordpoints_pre_update_rank', $rank, $name, $type, $group, $position, $meta );
+	do_action('wordpoints_pre_update_rank', $rank, $name, $type, $group, $position, $meta);
 
-	if ( 'utf8' === $wpdb->get_col_charset( $wpdb->wordpoints_ranks, 'name' ) ) {
-		$name = wp_encode_emoji( $name );
+	if ('utf8' === $wpdb->get_col_charset($wpdb->wordpoints_ranks, 'name')) {
+		$name = wp_encode_emoji($name);
 	}
 
 	$updated = $wpdb->update(
 		$wpdb->wordpoints_ranks
-		, array( 'name' => $name, 'type' => $type, 'rank_group' => $group )
-		, array( 'id' => $id )
+		, array('name' => $name, 'type' => $type, 'rank_group' => $group)
+		, array('id' => $id)
 		, '%s'
 		, '%d'
 	);
 
-	if ( false === $updated ) {
+	if (false === $updated) {
 		return false;
 	}
 
-	wp_cache_delete( $id, 'wordpoints_ranks' );
+	wp_cache_delete($id, 'wordpoints_ranks');
 
-	foreach ( $meta as $meta_key => $meta_value ) {
-		wordpoints_update_rank_meta( $id, $meta_key, $meta_value );
+	foreach ($meta as $meta_key => $meta_value) {
+		wordpoints_update_rank_meta($id, $meta_key, $meta_value);
 	}
 
-	$rank_group = WordPoints_Rank_Groups::get_group( $group );
+	$rank_group = WordPoints_Rank_Groups::get_group($group);
 
-	if ( $rank->rank_group !== $group ) {
+	if ($rank->rank_group !== $group) {
 
-		$previous_group = WordPoints_Rank_Groups::get_group( $rank->rank_group );
-		if ( $previous_group ) {
-			$previous_group->remove_rank( $rank->ID );
+		$previous_group = WordPoints_Rank_Groups::get_group($rank->rank_group);
+		if ($previous_group) {
+			$previous_group->remove_rank($rank->ID);
 		}
 
-		$rank_group->add_rank( $rank->ID, $position );
+		$rank_group->add_rank($rank->ID, $position);
 
 	} else {
 
-		if ( $position !== $rank_group->get_rank_position( $rank->ID ) ) {
-			$rank_group->move_rank( $rank->ID, $position );
+		if ($position !== $rank_group->get_rank_position($rank->ID)) {
+			$rank_group->move_rank($rank->ID, $position);
 		} else {
 			// If the position doesn't change, we still need refresh the ranks of
 			// users who have this rank, if the metadata or type has changed.
-			if ( $meta || $type !== $rank->type ) {
-				wordpoints_refresh_rank_users( $rank->ID );
+			if ($meta || $type !== $rank->type) {
+				wordpoints_refresh_rank_users($rank->ID);
 			}
 		}
 	}
@@ -262,7 +262,7 @@ 
 	 *
 	 * @param int $rank_id The ID of the rank that was updated.
 	 */
-	do_action( 'wordpoints_update_rank', $id );
+	do_action('wordpoints_update_rank', $id);
 
 	return true;
 }
@@ -276,11 +276,11 @@ 
  *
  * @return WordPoints_Rank|false The rank object, or false if it doesn't exist.
  */
-function wordpoints_get_rank( $id ) {
+function wordpoints_get_rank($id) {
 
-	$rank = new WordPoints_Rank( $id );
+	$rank = new WordPoints_Rank($id);
 
-	if ( ! $rank->exists() ) {
+	if ( ! $rank->exists()) {
 		return false;
 	}
 
@@ -302,11 +302,11 @@ 
  *
  * @return string The integer value of $points formatted for display.
  */
-function wordpoints_format_rank( $rank_id, $context, array $args = array() ) {
+function wordpoints_format_rank($rank_id, $context, array $args = array()) {
 
-	$rank = wordpoints_get_rank( $rank_id );
+	$rank = wordpoints_get_rank($rank_id);
 
-	if ( ! $rank ) {
+	if ( ! $rank) {
 		return false;
 	}
 
@@ -326,7 +326,7 @@ 
 	 *        @type int $user_id The ID of the user the rank is being displayed with.
 	 * }
 	 */
-	return apply_filters( 'wordpoints_format_rank', $formatted, $rank, $context, $args );
+	return apply_filters('wordpoints_format_rank', $formatted, $rank, $context, $args);
 }
 
 //
@@ -348,13 +348,13 @@ 
  *
  * @return int|bool The meta ID on success, false on failure.
  */
-function wordpoints_add_rank_meta( $rank_id, $meta_key, $meta_value, $unique = false ) {
+function wordpoints_add_rank_meta($rank_id, $meta_key, $meta_value, $unique = false) {
 
 	return add_metadata(
 		'wordpoints_rank'
 		, $rank_id
-		, wp_slash( $meta_key )
-		, wp_slash( $meta_value )
+		, wp_slash($meta_key)
+		, wp_slash($meta_value)
 		, $unique
 	);
 }
@@ -375,13 +375,13 @@ 
  * @return int|bool Meta ID if the key didn't exist, true on successful update,
  *                  false on failure.
  */
-function wordpoints_update_rank_meta( $rank_id, $meta_key, $meta_value, $prev_value = '' ) {
+function wordpoints_update_rank_meta($rank_id, $meta_key, $meta_value, $prev_value = '') {
 
 	return update_metadata(
 		'wordpoints_rank'
 		, $rank_id
-		, wp_slash( $meta_key )
-		, wp_slash( $meta_value )
+		, wp_slash($meta_key)
+		, wp_slash($meta_value)
 		, $prev_value
 	);
 }
@@ -401,13 +401,13 @@ 
  *
  * @return bool True on successful delete, false on failure.
  */
-function wordpoints_delete_rank_meta( $rank_id, $meta_key, $meta_value = '', $delete_all = false ) {
+function wordpoints_delete_rank_meta($rank_id, $meta_key, $meta_value = '', $delete_all = false) {
 
 	return delete_metadata(
 		'wordpoints_rank'
 		, $rank_id
-		, wp_slash( $meta_key )
-		, wp_slash( $meta_value )
+		, wp_slash($meta_key)
+		, wp_slash($meta_value)
 		, $delete_all
 	);
 }
@@ -425,9 +425,9 @@ 
  *
  * @return string|array Single metadata value or array of metadata values.
  */
-function wordpoints_get_rank_meta( $rank_id, $meta_key = '', $single = false ) {
+function wordpoints_get_rank_meta($rank_id, $meta_key = '', $single = false) {
 
-	return get_metadata( 'wordpoints_rank', $rank_id, $meta_key, $single );
+	return get_metadata('wordpoints_rank', $rank_id, $meta_key, $single);
 }
 
 //
@@ -444,30 +444,30 @@ 
  *
  * @return int|false The ID of the rank this user has, or false for invalid args.
  */
-function wordpoints_get_user_rank( $user_id, $group ) {
+function wordpoints_get_user_rank($user_id, $group) {
 
 	global $wpdb;
 
-	if ( ! wordpoints_posint( $user_id ) ) {
+	if ( ! wordpoints_posint($user_id)) {
 		return false;
 	}
 
-	$rank_group = WordPoints_Rank_Groups::get_group( $group );
+	$rank_group = WordPoints_Rank_Groups::get_group($group);
 
-	if ( ! $rank_group ) {
+	if ( ! $rank_group) {
 		return false;
 	}
 
-	$group_ranks = wp_cache_get( $group, 'wordpoints_user_ranks' );
+	$group_ranks = wp_cache_get($group, 'wordpoints_user_ranks');
 
-	foreach ( (array) $group_ranks as $_rank_id => $user_ids ) {
-		if ( isset( $user_ids[ $user_id ] ) ) {
+	foreach ((array) $group_ranks as $_rank_id => $user_ids) {
+		if (isset($user_ids[$user_id])) {
 			  $rank_id = $_rank_id;
 			  break;
 		}
 	}
 
-	if ( ! isset( $rank_id ) ) {
+	if ( ! isset($rank_id)) {
 
 		$rank_id = $wpdb->get_var(
 			$wpdb->prepare(
@@ -488,13 +488,13 @@ 
 			)
 		);
 
-		if ( ! $rank_id ) {
+		if ( ! $rank_id) {
 			$rank_id = $rank_group->get_base_rank();
 		}
 
-		$group_ranks[ $rank_id ][ $user_id ] = $user_id;
+		$group_ranks[$rank_id][$user_id] = $user_id;
 
-		wp_cache_set( $group, $group_ranks, 'wordpoints_user_ranks' );
+		wp_cache_set($group, $group_ranks, 'wordpoints_user_ranks');
 	}
 
 	return (int) $rank_id;
@@ -512,17 +512,17 @@ 
  *
  * @return string|false The rank of this user formatted for dispay, or false.
  */
-function wordpoints_get_formatted_user_rank( $user_id, $group, $context, array $args = array() ) {
+function wordpoints_get_formatted_user_rank($user_id, $group, $context, array $args = array()) {
 
-	$rank_id = wordpoints_get_user_rank( $user_id, $group );
+	$rank_id = wordpoints_get_user_rank($user_id, $group);
 
-	if ( ! $rank_id ) {
+	if ( ! $rank_id) {
 		return false;
 	}
 
-	$args = array_merge( $args, array( 'user_id' => $user_id ) );
+	$args = array_merge($args, array('user_id' => $user_id));
 
-	return wordpoints_format_rank( $rank_id, $context, $args );
+	return wordpoints_format_rank($rank_id, $context, $args);
 }
 
 /**
@@ -535,29 +535,29 @@ 
  *
  * @return bool True if the update was successful. False otherwise.
  */
-function wordpoints_update_user_rank( $user_id, $rank_id ) {
+function wordpoints_update_user_rank($user_id, $rank_id) {
 
 	global $wpdb;
 
-	if ( ! wordpoints_posint( $rank_id ) || ! wordpoints_posint( $user_id ) ) {
+	if ( ! wordpoints_posint($rank_id) || ! wordpoints_posint($user_id)) {
 		return false;
 	}
 
-	$rank = wordpoints_get_rank( $rank_id );
+	$rank = wordpoints_get_rank($rank_id);
 
-	if ( ! $rank ) {
+	if ( ! $rank) {
 		return false;
 	}
 
-	$old_rank_id = wordpoints_get_user_rank( $user_id, $rank->rank_group );
+	$old_rank_id = wordpoints_get_user_rank($user_id, $rank->rank_group);
 
-	if ( $rank_id === $old_rank_id ) {
+	if ($rank_id === $old_rank_id) {
 		return true;
 	}
 
-	$old_rank = wordpoints_get_rank( $old_rank_id );
+	$old_rank = wordpoints_get_rank($old_rank_id);
 
-	switch ( $old_rank->type ) {
+	switch ($old_rank->type) {
 
 		case 'base':
 			// If this is a base rank, it's possible that the user will not have
@@ -577,7 +577,7 @@ 
 
 			// If the user rank isn't in the database, we can't run an update query,
 			// and need to do this insert instead.
-			if ( ! $has_rank ) {
+			if ( ! $has_rank) {
 
 				// This user doesn't yet have a rank in this group.
 				$result = $wpdb->insert(
@@ -598,7 +598,7 @@ 
 		default:
 			$result = $wpdb->update(
 				$wpdb->wordpoints_user_ranks
-				, array( 'rank_id' => $rank_id )
+				, array('rank_id' => $rank_id)
 				, array(
 					'user_id' => $user_id,
 					'rank_id' => $old_rank_id,
@@ -609,22 +609,22 @@ 
 
 	} // End switch ( $old_rank->type ).
 
-	if ( false === $result ) {
+	if (false === $result) {
 		return false;
 	}
 
-	$group_ranks = wp_cache_get( $rank->rank_group, 'wordpoints_user_ranks' );
+	$group_ranks = wp_cache_get($rank->rank_group, 'wordpoints_user_ranks');
 
-	foreach ( $group_ranks as $_rank_id => $user_ids ) {
-		unset( $group_ranks[ $_rank_id ][ $user_id ] );
+	foreach ($group_ranks as $_rank_id => $user_ids) {
+		unset($group_ranks[$_rank_id][$user_id]);
 	}
 
-	wp_cache_set( $rank->rank_group, $group_ranks, 'wordpoints_user_ranks' );
+	wp_cache_set($rank->rank_group, $group_ranks, 'wordpoints_user_ranks');
 
-	unset( $group_ranks );
+	unset($group_ranks);
 
-	wp_cache_delete( $rank_id, 'wordpoints_users_with_rank' );
-	wp_cache_delete( $old_rank_id, 'wordpoints_users_with_rank' );
+	wp_cache_delete($rank_id, 'wordpoints_users_with_rank');
+	wp_cache_delete($old_rank_id, 'wordpoints_users_with_rank');
 
 	/**
 	 * Perform actions when a user rank is updated.
@@ -635,7 +635,7 @@ 
 	 * @param int $new_rank_id The ID of the new rank the user has.
 	 * @param int $old_rank_id The ID of the old rank the user used to have.
 	 */
-	do_action( 'wordpoints_update_user_rank', $user_id, $rank_id, $old_rank_id );
+	do_action('wordpoints_update_user_rank', $user_id, $rank_id, $old_rank_id);
 
 	return true;
 }
@@ -649,19 +649,19 @@ 
  *
  * @return int[]|false Array of user IDs or false if the $rank_id is invalid.
  */
-function wordpoints_get_users_with_rank( $rank_id ) {
+function wordpoints_get_users_with_rank($rank_id) {
 
 	global $wpdb;
 
-	$rank = wordpoints_get_rank( $rank_id );
+	$rank = wordpoints_get_rank($rank_id);
 
-	if ( ! $rank ) {
+	if ( ! $rank) {
 		return false;
 	}
 
-	$user_ids = wp_cache_get( $rank_id, 'wordpoints_users_with_rank' );
+	$user_ids = wp_cache_get($rank_id, 'wordpoints_users_with_rank');
 
-	if ( false === $user_ids ) {
+	if (false === $user_ids) {
 
 		$user_ids = $wpdb->get_col(
 			$wpdb->prepare(
@@ -674,7 +674,7 @@ 
 			)
 		);
 
-		if ( 'base' === $rank->type ) {
+		if ('base' === $rank->type) {
 
 			$other_user_ids = $wpdb->get_col(
 				$wpdb->prepare(
@@ -693,10 +693,10 @@ 
 				)
 			);
 
-			$user_ids = array_merge( $user_ids, $other_user_ids );
+			$user_ids = array_merge($user_ids, $other_user_ids);
 		}
 
-		wp_cache_set( $rank_id, $user_ids, 'wordpoints_users_with_rank' );
+		wp_cache_set($rank_id, $user_ids, 'wordpoints_users_with_rank');
 
 	} // End if ( not cached ).
 
@@ -712,59 +712,59 @@ 
  *
  * @param int $rank_id The ID of the rank to refresh.
  */
-function wordpoints_refresh_rank_users( $rank_id ) {
+function wordpoints_refresh_rank_users($rank_id) {
 
-	$rank = wordpoints_get_rank( $rank_id );
+	$rank = wordpoints_get_rank($rank_id);
 
-	if ( ! $rank || 'base' === $rank->type ) {
+	if ( ! $rank || 'base' === $rank->type) {
 		return;
 	}
 
 	$prev_rank = $rank->get_adjacent( -1 );
 
-	if ( ! $prev_rank ) {
+	if ( ! $prev_rank) {
 		return;
 	}
 
 	// Get a list of users who have this rank.
-	$users = wordpoints_get_users_with_rank( $rank->ID );
+	$users = wordpoints_get_users_with_rank($rank->ID);
 
 	// Also get users who have the previous rank.
-	$prev_rank_users = wordpoints_get_users_with_rank( $prev_rank->ID );
+	$prev_rank_users = wordpoints_get_users_with_rank($prev_rank->ID);
 
 	// If there are some users who have this rank, check if any of them need to
 	// decrease to that rank.
-	if ( ! empty( $users ) ) {
+	if ( ! empty($users)) {
 
-		$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_decrease_user_rank( $user_id, $rank );
+			$new_rank = $rank_type->maybe_decrease_user_rank($user_id, $rank);
 
-			if ( $new_rank->ID === $rank->ID ) {
+			if ($new_rank->ID === $rank->ID) {
 				continue;
 			}
 
-			wordpoints_update_user_rank( $user_id, $new_rank->ID );
+			wordpoints_update_user_rank($user_id, $new_rank->ID);
 		}
 	}
 
 	// If there were some users with the previous rank, check if any of them can now
 	// increase to this rank.
-	if ( ! empty( $prev_rank_users ) ) {
+	if ( ! empty($prev_rank_users)) {
 
-		$rank_type = WordPoints_Rank_Types::get_type( $rank->type );
+		$rank_type = WordPoints_Rank_Types::get_type($rank->type);
 
-		foreach ( $prev_rank_users as $user_id ) {
+		foreach ($prev_rank_users as $user_id) {
 
-			$new_rank = $rank_type->maybe_increase_user_rank( $user_id, $prev_rank );
+			$new_rank = $rank_type->maybe_increase_user_rank($user_id, $prev_rank);
 
-			if ( $new_rank->ID === $prev_rank->ID ) {
+			if ($new_rank->ID === $prev_rank->ID) {
 				continue;
 			}
 
-			wordpoints_update_user_rank( $user_id, $new_rank->ID );
+			wordpoints_update_user_rank($user_id, $new_rank->ID);
 		}
 	}
 }

src/components/ranks/admin/admin.php

Spacing +3 added lines, -3 removed lines

@@ -12,20 +12,20 @@
  *
  * @since 1.7.0
  */
-include_once( WORDPOINTS_DIR . 'components/ranks/admin/includes/ajax.php' );
+include_once(WORDPOINTS_DIR . 'components/ranks/admin/includes/ajax.php');
 
 /**
  * Admin-side functions of the ranks component.
  *
  * @since 2.1.0
  */
-include_once( WORDPOINTS_DIR . 'components/ranks/admin/includes/functions.php' );
+include_once(WORDPOINTS_DIR . 'components/ranks/admin/includes/functions.php');
 
 /**
  * Admin-side actin and filter hooks of the ranks component.
  *
  * @since 2.1.0
  */
-include_once( WORDPOINTS_DIR . 'components/ranks/admin/includes/filters.php' );
+include_once(WORDPOINTS_DIR . 'components/ranks/admin/includes/filters.php');
 
 // EOF

src/components/ranks/admin/includes/filters.php

Spacing +3 added lines, -3 removed lines

@@ -7,11 +7,11 @@
  * @since 2.1.0
  */
 
-add_action( 'init', 'wordpoints_ranks_admin_register_scripts' );
+add_action('init', 'wordpoints_ranks_admin_register_scripts');
 
-add_action( 'admin_menu', 'wordpoints_ranks_admin_menu' );
+add_action('admin_menu', 'wordpoints_ranks_admin_menu');
 
-add_action( 'load-wordpoints_page_wordpoints_ranks', 'wordpoints_ranks_admin_screen_load' );
+add_action('load-wordpoints_page_wordpoints_ranks', 'wordpoints_ranks_admin_screen_load');
 
 
 // EOF