WordPoints / wordpoints

src/components/points/includes/shortcodes.php

Spacing +8 added lines, -8 removed lines

@@ -19,7 +19,7 @@ 
 	/**
 	 * @since 2.3.0
 	 */
-	public function __construct( $atts, $content, $shortcode = null ) {
+	public function __construct($atts, $content, $shortcode = null) {
 
 		_deprecated_function(
 			__METHOD__
@@ -27,7 +27,7 @@ 
 			, 'WordPoints_Points_Shortcode_HTGP::__construct'
 		);
 
-		parent::__construct( $atts, $content, $shortcode );
+		parent::__construct($atts, $content, $shortcode);
 	}
 }
 
@@ -42,7 +42,7 @@ 
 	/**
 	 * @since 2.3.0
 	 */
-	public function __construct( $atts, $content, $shortcode = null ) {
+	public function __construct($atts, $content, $shortcode = null) {
 
 		_deprecated_function(
 			__METHOD__
@@ -50,7 +50,7 @@ 
 			, 'WordPoints_Points_Shortcode_User_Points::__construct'
 		);
 
-		parent::__construct( $atts, $content, $shortcode );
+		parent::__construct($atts, $content, $shortcode);
 	}
 }
 
@@ -65,7 +65,7 @@ 
 	/**
 	 * @since 2.3.0
 	 */
-	public function __construct( $atts, $content, $shortcode = null ) {
+	public function __construct($atts, $content, $shortcode = null) {
 
 		_deprecated_function(
 			__METHOD__
@@ -73,7 +73,7 @@ 
 			, 'WordPoints_Points_Shortcode_Logs::__construct'
 		);
 
-		parent::__construct( $atts, $content, $shortcode );
+		parent::__construct($atts, $content, $shortcode);
 	}
 }
 
@@ -88,7 +88,7 @@ 
 	/**
 	 * @since 2.3.0
 	 */
-	public function __construct( $atts, $content, $shortcode = null ) {
+	public function __construct($atts, $content, $shortcode = null) {
 
 		_deprecated_function(
 			__METHOD__
@@ -96,7 +96,7 @@ 
 			, 'WordPoints_Points_Shortcode_Top_Users::__construct'
 		);
 
-		parent::__construct( $atts, $content, $shortcode );
+		parent::__construct($atts, $content, $shortcode);
 	}
 }
 

src/components/points/includes/functions.php

Spacing +58 added lines, -58 removed lines

@@ -16,10 +16,10 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $reactors The reactors registry.
  */
-function wordpoints_points_hook_reactors_init( $reactors ) {
+function wordpoints_points_hook_reactors_init($reactors) {
 
-	$reactors->register( 'points', 'WordPoints_Points_Hook_Reactor' );
-	$reactors->register( 'points_legacy', 'WordPoints_Points_Hook_Reactor_Legacy' );
+	$reactors->register('points', 'WordPoints_Points_Hook_Reactor');
+	$reactors->register('points_legacy', 'WordPoints_Points_Hook_Reactor_Legacy');
 }
 
 /**
@@ -31,7 +31,7 @@ 
  *
  * @param WordPoints_Class_Registry_Children $reaction_stores The store registry.
  */
-function wordpoints_points_hook_reaction_stores_init( $reaction_stores ) {
+function wordpoints_points_hook_reaction_stores_init($reaction_stores) {
 
 	$reaction_stores->register(
 		'standard'
@@ -39,7 +39,7 @@ 
 		, 'WordPoints_Hook_Reaction_Store_Options'
 	);
 
-	if ( is_wordpoints_network_active() ) {
+	if (is_wordpoints_network_active()) {
 		$reaction_stores->register(
 			'network'
 			, 'points'
@@ -57,7 +57,7 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $extensions The extension registry.
  */
-function wordpoints_points_hook_extensions_init( $extensions ) {
+function wordpoints_points_hook_extensions_init($extensions) {
 
 	$extensions->register(
 		'points_legacy_reversals'
@@ -85,13 +85,13 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_points_register_legacy_post_publish_events( $slug ) {
+function wordpoints_points_register_legacy_post_publish_events($slug) {
 
-	if ( 'attachment' === $slug ) {
+	if ('attachment' === $slug) {
 		return;
 	}
 
-	$events = wordpoints_hooks()->get_sub_app( 'events' );
+	$events = wordpoints_hooks()->get_sub_app('events');
 
 	$events->register(
 		"points_legacy_post_publish\\{$slug}"
@@ -131,19 +131,19 @@ 
 	wp_register_style(
 		'wordpoints-points-logs'
 		, "{$assets_url}/css/points-logs{$suffix}.css"
-		, array( 'dashicons' )
+		, array('dashicons')
 		, WORDPOINTS_VERSION
 	);
 
 	$styles = wp_styles();
-	$rtl_styles = array( 'wordpoints-top-users', 'wordpoints-points-logs' );
+	$rtl_styles = array('wordpoints-top-users', 'wordpoints-points-logs');
 
-	foreach ( $rtl_styles as $rtl_style ) {
+	foreach ($rtl_styles as $rtl_style) {
 
-		$styles->add_data( $rtl_style, 'rtl', 'replace' );
+		$styles->add_data($rtl_style, 'rtl', 'replace');
 
-		if ( $suffix ) {
-			$styles->add_data( $rtl_style, 'suffix', $suffix );
+		if ($suffix) {
+			$styles->add_data($rtl_style, 'suffix', $suffix);
 		}
 	}
 }
@@ -160,7 +160,7 @@ 
 	return array(
 		'set_wordpoints_points'                  => 'manage_options',
 		'manage_network_wordpoints_points_hooks' => 'manage_network_options',
-		'manage_wordpoints_points_types'         => ( is_wordpoints_network_active() ) ? 'manage_network_options' : 'manage_options',
+		'manage_wordpoints_points_types'         => (is_wordpoints_network_active()) ? 'manage_network_options' : 'manage_options',
 	);
 }
 
@@ -178,23 +178,23 @@ 
  *
  * @return string $points formatted with prefix and suffix.
  */
-function wordpoints_format_points_filter( $formatted, $points, $type ) {
+function wordpoints_format_points_filter($formatted, $points, $type) {
 
-	$points_type = wordpoints_get_points_type( $type );
+	$points_type = wordpoints_get_points_type($type);
 
-	if ( isset( $points_type['prefix'] ) ) {
+	if (isset($points_type['prefix'])) {
 
-		if ( $points < 0 ) {
+		if ($points < 0) {
 
-			$points = abs( $points );
+			$points = abs($points);
 			$points_type['prefix'] = '-' . $points_type['prefix'];
 		}
 
-		$formatted = esc_html( $points_type['prefix'] . $points );
+		$formatted = esc_html($points_type['prefix'] . $points);
 	}
 
-	if ( isset( $points_type['suffix'] ) ) {
-		$formatted = $formatted . esc_html( $points_type['suffix'] );
+	if (isset($points_type['suffix'])) {
+		$formatted = $formatted . esc_html($points_type['suffix']);
 	}
 
 	return $formatted;
@@ -211,20 +211,20 @@ 
  * @param array $args The arguments for the dropdown {@see
  *        WordPoints_Dropdown_Builder::$args}.
  */
-function wordpoints_points_types_dropdown( array $args ) {
+function wordpoints_points_types_dropdown(array $args) {
 
 	$points_types = array();
 
-	foreach ( wordpoints_get_points_types() as $slug => $settings ) {
+	foreach (wordpoints_get_points_types() as $slug => $settings) {
 
-		$points_types[ $slug ] = $settings['name'];
+		$points_types[$slug] = $settings['name'];
 	}
 
-	if ( isset( $args['options'] ) && is_array( $args['options'] ) ) {
+	if (isset($args['options']) && is_array($args['options'])) {
 		$points_types = $args['options'] + $points_types;
 	}
 
-	$dropdown = new WordPoints_Dropdown_Builder( $points_types, $args );
+	$dropdown = new WordPoints_Dropdown_Builder($points_types, $args);
 
 	$dropdown->display();
 }
@@ -238,27 +238,27 @@ 
  *
  * @param int $user_id The ID of the user just deleted.
  */
-function wordpoints_delete_points_logs_for_user( $user_id ) {
+function wordpoints_delete_points_logs_for_user($user_id) {
 
 	global $wpdb;
 
-	$query_args = array( 'fields' => 'id', 'user_id' => $user_id );
+	$query_args = array('fields' => 'id', 'user_id' => $user_id);
 
 	// If the user is being deleted from all blogs on multisite.
-	if ( is_multisite() && ! get_userdata( $user_id ) ) {
+	if (is_multisite() && ! get_userdata($user_id)) {
 		$query_args['blog_id'] = 0;
 	}
 
 	// Delete log meta.
-	$query = new WordPoints_Points_Logs_Query( $query_args );
+	$query = new WordPoints_Points_Logs_Query($query_args);
 
-	foreach ( $query->get( 'col' ) as $log_id ) {
-		wordpoints_points_log_delete_all_metadata( $log_id );
+	foreach ($query->get('col') as $log_id) {
+		wordpoints_points_log_delete_all_metadata($log_id);
 	}
 
-	$where = array( 'user_id' => $user_id );
+	$where = array('user_id' => $user_id);
 
-	if ( ! isset( $query_args['blog_id'] ) ) {
+	if ( ! isset($query_args['blog_id'])) {
 		$where['blog_id'] = $wpdb->blogid;
 	}
 
@@ -269,7 +269,7 @@ 
 		,'%d'
 	);
 
-	wordpoints_flush_points_logs_caches( array( 'user_id' => $user_id ) );
+	wordpoints_flush_points_logs_caches(array('user_id' => $user_id));
 }
 
 /**
@@ -281,21 +281,21 @@ 
  *
  * @param int $blog_id The ID of the blog being deleted.
  */
-function wordpoints_delete_points_logs_for_blog( $blog_id ) {
+function wordpoints_delete_points_logs_for_blog($blog_id) {
 
 	global $wpdb;
 
 	// Delete log meta.
-	$query = new WordPoints_Points_Logs_Query( array( 'fields' => 'id' ) );
+	$query = new WordPoints_Points_Logs_Query(array('fields' => 'id'));
 
-	foreach ( $query->get( 'col' ) as $log_id ) {
-		wordpoints_points_log_delete_all_metadata( $log_id );
+	foreach ($query->get('col') as $log_id) {
+		wordpoints_points_log_delete_all_metadata($log_id);
 	}
 
 	// Now delete the logs.
 	$wpdb->delete(
 		$wpdb->wordpoints_points_logs
-		,array( 'blog_id' => $blog_id )
+		,array('blog_id' => $blog_id)
 		,'%d'
 	);
 
@@ -311,13 +311,13 @@ 
  *
  * @param string $points_type The type of points the settings are being shown for.
  */
-function wordpoints_points_settings_custom_meta_key_message( $points_type ) {
+function wordpoints_points_settings_custom_meta_key_message($points_type) {
 
-	$custom_key = wordpoints_get_points_type_setting( $points_type, 'meta_key' );
+	$custom_key = wordpoints_get_points_type_setting($points_type, 'meta_key');
 
-	if ( ! empty( $custom_key ) ) {
+	if ( ! empty($custom_key)) {
 		// translators: 1. Meta key.
-		echo '<p>' . esc_html( sprintf( __( 'This points type uses a custom meta key: %s', 'wordpoints' ), $custom_key ) ) . '</p>';
+		echo '<p>' . esc_html(sprintf(__('This points type uses a custom meta key: %s', 'wordpoints'), $custom_key)) . '</p>';
 	}
 }
 
@@ -330,20 +330,20 @@ 
  *
  * @param string $points_type The type of points whose logs are being displayed.
  */
-function wordpoints_points_logs_custom_meta_key_message( $points_type ) {
+function wordpoints_points_logs_custom_meta_key_message($points_type) {
 
-	if ( ! current_user_can( 'manage_options' ) ) {
+	if ( ! current_user_can('manage_options')) {
 		return;
 	}
 
-	$custom_key = wordpoints_get_points_type_setting( $points_type, 'meta_key' );
+	$custom_key = wordpoints_get_points_type_setting($points_type, 'meta_key');
 
-	if ( ! empty( $custom_key ) ) {
+	if ( ! empty($custom_key)) {
 		wordpoints_show_admin_message(
 			esc_html(
 				sprintf(
 					// translators: Meta key.
-					__( 'This points type uses a custom meta key (&#8220;%s&#8221;). If this key is also used by another plugin, changes made by it will not be logged. Only transactions performed by WordPoints are included in the logs.', 'wordpoints' )
+					__('This points type uses a custom meta key (&#8220;%s&#8221;). If this key is also used by another plugin, changes made by it will not be logged. Only transactions performed by WordPoints are included in the logs.', 'wordpoints')
 					, $custom_key
 				)
 			)
@@ -362,18 +362,18 @@ 
  */
 function wordpoints_points_add_global_cache_groups() {
 
-	if ( function_exists( 'wp_cache_add_global_groups' ) ) {
+	if (function_exists('wp_cache_add_global_groups')) {
 
 		$groups = array(
 			'wordpoints_network_points_logs_query',
 			'wordpoints_points_log_meta',
 		);
 
-		if ( is_wordpoints_network_active() ) {
+		if (is_wordpoints_network_active()) {
 			$groups[] = 'wordpoints_points_top_users';
 		}
 
-		wp_cache_add_global_groups( $groups );
+		wp_cache_add_global_groups($groups);
 	}
 }
 
@@ -387,13 +387,13 @@ 
 function wordpoints_register_points_widgets() {
 
 	// My points widget.
-	register_widget( 'WordPoints_Points_Widget_User_Points' );
+	register_widget('WordPoints_Points_Widget_User_Points');
 
 	// Top users widget.
-	register_widget( 'WordPoints_Points_Widget_Top_Users' );
+	register_widget('WordPoints_Points_Widget_Top_Users');
 
 	// Points logs widget.
-	register_widget( 'WordPoints_Points_Widget_Logs' );
+	register_widget('WordPoints_Points_Widget_Logs');
 }
 
 // EOF

src/components/points/classes/widget.php

Spacing +19 added lines, -19 removed lines

@@ -19,22 +19,22 @@ 
 	/**
 	 * @since 1.9.0
 	 */
-	protected function verify_settings( $instance ) {
+	protected function verify_settings($instance) {
 
-		if ( isset( $this->defaults['points_type'] ) ) {
+		if (isset($this->defaults['points_type'])) {
 
 			if (
-				empty( $instance['points_type'] )
-				|| ! wordpoints_is_points_type( $instance['points_type'] )
+				empty($instance['points_type'])
+				|| ! wordpoints_is_points_type($instance['points_type'])
 			) {
 				return new WP_Error(
 					'wordpoints_widget_invalid_points_type'
-					, esc_html__( 'Please select a valid points type.', 'wordpoints' )
+					, esc_html__('Please select a valid points type.', 'wordpoints')
 				);
 			}
 		}
 
-		return parent::verify_settings( $instance );
+		return parent::verify_settings($instance);
 	}
 
 	/**
@@ -48,17 +48,17 @@ 
 	 *
 	 * @param string $maybe_points_type The variable to insure is a points type.
 	 */
-	protected function make_a_points_type( &$maybe_points_type ) {
+	protected function make_a_points_type(&$maybe_points_type) {
 
-		if ( ! wordpoints_is_points_type( $maybe_points_type ) ) {
+		if ( ! wordpoints_is_points_type($maybe_points_type)) {
 
 			$maybe_points_type = wordpoints_get_default_points_type();
 
-			if ( ! $maybe_points_type ) {
+			if ( ! $maybe_points_type) {
 
 				$points_types = wordpoints_get_points_types();
 
-				$maybe_points_type = key( $points_types );
+				$maybe_points_type = key($points_types);
 			}
 		}
 	}
@@ -66,11 +66,11 @@ 
 	/**
 	 * @since 2.0.0
 	 */
-	public function update( $new_instance, $old_instance ) {
+	public function update($new_instance, $old_instance) {
 
-		parent::update( $new_instance, $old_instance );
+		parent::update($new_instance, $old_instance);
 
-		$this->make_a_points_type( $this->instance['points_type'] );
+		$this->make_a_points_type($this->instance['points_type']);
 
 		return $this->instance;
 	}
@@ -78,9 +78,9 @@ 
 	/**
 	 * @since 2.0.0
 	 */
-	public function form( $instance ) {
+	public function form($instance) {
 
-		parent::form( $instance );
+		parent::form($instance);
 
 		$this->form_points_type_field();
 
@@ -96,16 +96,16 @@ 
 
 		$dropdown_args = array(
 			'selected' => $this->instance['points_type'],
-			'id'       => $this->get_field_id( 'points_type' ),
-			'name'     => $this->get_field_name( 'points_type' ),
+			'id'       => $this->get_field_id('points_type'),
+			'name'     => $this->get_field_name('points_type'),
 			'class'    => 'widefat',
 		);
 
 		?>
 
 		<p>
-			<label for="<?php echo esc_attr( $dropdown_args['id'] ); ?>"><?php echo esc_html_x( 'Points type', 'form label', 'wordpoints' ); ?></label>
-			<?php wordpoints_points_types_dropdown( $dropdown_args ); ?>
+			<label for="<?php echo esc_attr($dropdown_args['id']); ?>"><?php echo esc_html_x('Points type', 'form label', 'wordpoints'); ?></label>
+			<?php wordpoints_points_types_dropdown($dropdown_args); ?>
 		</p>
 
 		<?php

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

Spacing +15 added lines, -15 removed lines

@@ -31,36 +31,36 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 
-		$period = parent::get_period_by_reaction( $settings, $reaction );
+		$period = parent::get_period_by_reaction($settings, $reaction);
 
-		if ( $period ) {
+		if ($period) {
 			return $period;
 		}
 
-		$expected_settings = array( array( 'current:user' ) );
+		$expected_settings = array(array('current:user'));
 
-		if ( ! isset( $settings['args'] ) || $settings['args'] !== $expected_settings ) {
+		if ( ! isset($settings['args']) || $settings['args'] !== $expected_settings) {
 			return false;
 		}
 
 		// Get the user ID.
-		$user = $this->event_args->get_from_hierarchy( array( 'current:user' ) );
+		$user = $this->event_args->get_from_hierarchy(array('current:user'));
 
-		if ( ! $user instanceof WordPoints_Entity ) {
+		if ( ! $user instanceof WordPoints_Entity) {
 			return false;
 		}
 
 		$user_id = $user->get_the_id();
 		$reaction_guid = $reaction->get_guid();
-		$points_type   = $reaction->get_meta( 'points_type' );
-		$log_type      = $reaction->get_meta( 'legacy_log_type' );
+		$points_type   = $reaction->get_meta('points_type');
+		$log_type      = $reaction->get_meta('legacy_log_type');
 
-		$cache_key = wp_json_encode( $reaction_guid ) . "-{$user_id}-{$points_type}-{$log_type}";
+		$cache_key = wp_json_encode($reaction_guid) . "-{$user_id}-{$points_type}-{$log_type}";
 
 		// Before we run the query, we try to lookup the period in the cache.
-		$period = wp_cache_get( $cache_key, 'wordpoints_points_legacy_hook_periods' );
+		$period = wp_cache_get($cache_key, 'wordpoints_points_legacy_hook_periods');
 
-		if ( $period ) {
+		if ($period) {
 			return $period;
 		}
 
@@ -74,15 +74,15 @@ 
 			)
 		);
 
-		$date = $query->get( 'var' );
+		$date = $query->get('var');
 
-		if ( ! $date ) {
+		if ( ! $date) {
 			return false;
 		}
 
-		$period = (object) array( 'date' => $date );
+		$period = (object) array('date' => $date);
 
-		wp_cache_set( $cache_key, $period, 'wordpoints_points_legacy_hook_periods' );
+		wp_cache_set($cache_key, $period, 'wordpoints_points_legacy_hook_periods');
 
 		return $period;
 	}

src/components/points/classes/shortcode.php

Spacing +8 added lines, -8 removed lines

@@ -22,11 +22,11 @@ 
 	 */
 	protected function verify_atts() {
 
-		if ( isset( $this->pairs['points_type'] ) ) {
+		if (isset($this->pairs['points_type'])) {
 
 			$points_type = $this->get_points_type();
 
-			if ( is_wp_error( $points_type ) ) {
+			if (is_wp_error($points_type)) {
 				return $points_type;
 			}
 
@@ -52,20 +52,20 @@ 
 
 		$points_type = $this->atts['points_type'];
 
-		if ( ! wordpoints_is_points_type( $points_type ) ) {
+		if ( ! wordpoints_is_points_type($points_type)) {
 
 			$points_type = wordpoints_get_default_points_type();
 
-			if ( ! $points_type ) {
+			if ( ! $points_type) {
 
 				$points_type = new WP_Error(
 					'wordpoints_shortcode_no_points_type'
 					, sprintf(
 						// translators: 1. Attribute name; 2. Shortcode name; 3. Example of proper usage.
-						__( 'The “%1$s” attribute of the %2$s shortcode must be the slug of a points type. Example: %3$s.', 'wordpoints' )
+						__('The “%1$s” attribute of the %2$s shortcode must be the slug of a points type. Example: %3$s.', 'wordpoints')
 						, 'points_type'
-						, '<code>[' . sanitize_key( $this->shortcode ) . ']</code>'
-						, '<code>[' . sanitize_key( $this->shortcode ) . ' points_type="points"]</code>'
+						, '<code>[' . sanitize_key($this->shortcode) . ']</code>'
+						, '<code>[' . sanitize_key($this->shortcode) . ' points_type="points"]</code>'
 					)
 				);
 			}
@@ -80,7 +80,7 @@ 
 		 * @param array           $atts        The shortcode attributes.
 		 * @param string          $shortcode   The shortcode.
 		 */
-		return apply_filters( 'wordpoints_shortcode_points_type', $points_type, $this->atts, $this->shortcode );
+		return apply_filters('wordpoints_shortcode_points_type', $points_type, $this->atts, $this->shortcode);
 	}
 }
 

src/components/points/classes/shortcode/top/users.php

Spacing +4 added lines, -4 removed lines

@@ -36,13 +36,13 @@
 	 */
 	protected function verify_atts() {
 
-		if ( ! wordpoints_posint( $this->atts['users'] ) ) {
+		if ( ! wordpoints_posint($this->atts['users'])) {
 			return sprintf(
 				// translators: 1. Attribute name; 2. Shortcode name; 3. Example of proper usage.
-				__( 'The “%1$s” attribute of the %2$s shortcode must be a positive integer. Example: %3$s.', 'wordpoints' )
+				__('The “%1$s” attribute of the %2$s shortcode must be a positive integer. Example: %3$s.', 'wordpoints')
 				, 'users'
-				, '<code>[' . sanitize_key( $this->shortcode ) . ']</code>'
-				, '<code>[' . sanitize_key( $this->shortcode ) . ' <b>users="10"</b> type="points"]</code>'
+				, '<code>[' . sanitize_key($this->shortcode) . ']</code>'
+				, '<code>[' . sanitize_key($this->shortcode) . ' <b>users="10"</b> type="points"]</code>'
 			);
 		}
 

src/components/points/classes/shortcode/htgp.php

Spacing +24 added lines, -24 removed lines

@@ -43,34 +43,34 @@ 
 		 * @param string[] $extra_classes The extra classes for the table element.
 		 * @param array    $atts          The arguments for table display from the shortcode.
 		 */
-		$extra_classes = apply_filters( 'wordpoints_how_to_get_points_table_extra_classes', array(), $this->atts );
+		$extra_classes = apply_filters('wordpoints_how_to_get_points_table_extra_classes', array(), $this->atts);
 
-		$points_heading = _x( 'Points', 'column name', 'wordpoints' );
+		$points_heading = _x('Points', 'column name', 'wordpoints');
 
 		$points_type_name = wordpoints_get_points_type_setting(
 			$this->atts['points_type']
 			, 'name'
 		);
 
-		if ( ! empty( $points_type_name ) ) {
+		if ( ! empty($points_type_name)) {
 			$points_heading = $points_type_name;
 		}
 
-		$html = '<table class="wordpoints-how-to-get-points ' . esc_attr( implode( ' ', $extra_classes ) ) . '">
+		$html = '<table class="wordpoints-how-to-get-points ' . esc_attr(implode(' ', $extra_classes)) . '">
 			<thead>
-				<tr><th style="padding-right: 10px">' . esc_html( $points_heading ) . '</th>
-				<th>' . esc_html_x( 'Action', 'column name', 'wordpoints' ) . '</th></tr>
+				<tr><th style="padding-right: 10px">' . esc_html($points_heading) . '</th>
+				<th>' . esc_html_x('Action', 'column name', 'wordpoints') . '</th></tr>
 			</thead>
 			<tbody>';
 
-		$html .= $this->list_reactions( 'points' );
+		$html .= $this->list_reactions('points');
 		$html .= $this->list_points_hooks();
 
-		if ( is_wordpoints_network_active() ) {
+		if (is_wordpoints_network_active()) {
 
-			WordPoints_Points_Hooks::set_network_mode( true );
+			WordPoints_Points_Hooks::set_network_mode(true);
 			$html .= $this->list_points_hooks();
-			WordPoints_Points_Hooks::set_network_mode( false );
+			WordPoints_Points_Hooks::set_network_mode(false);
 		}
 
 		$html .= '</tbody></table>';
@@ -90,7 +90,7 @@ 
 
 		$network = '';
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$network = 'network_';
 		}
 
@@ -100,17 +100,17 @@ 
 
 		$html = '';
 
-		foreach ( $hooks as $hook_id ) {
+		foreach ($hooks as $hook_id) {
 
-			$hook = WordPoints_Points_Hooks::get_handler( $hook_id );
+			$hook = WordPoints_Points_Hooks::get_handler($hook_id);
 
-			if ( ! $hook ) {
+			if ( ! $hook) {
 				continue;
 			}
 
-			$points = $hook->get_points( $network . $hook->get_number() );
+			$points = $hook->get_points($network . $hook->get_number());
 
-			if ( ! $points ) {
+			if ( ! $points) {
 				continue;
 			}
 
@@ -122,7 +122,7 @@ 
 
 			$html .= '<tr>
 				<td>' . $points . '</td>
-				<td>' . esc_html( $hook->get_description() ) . '</td>
+				<td>' . esc_html($hook->get_description()) . '</td>
 				</tr>';
 		}
 
@@ -142,21 +142,21 @@ 
 	 *
 	 * @return string The HTML for the table rows.
 	 */
-	protected function list_reactions( $store_slug ) {
+	protected function list_reactions($store_slug) {
 
 		$html = '';
 
-		foreach ( wordpoints_hooks()->get_reaction_stores( $store_slug ) as $store ) {
+		foreach (wordpoints_hooks()->get_reaction_stores($store_slug) as $store) {
 
-			foreach ( $store->get_reactions() as $reaction ) {
+			foreach ($store->get_reactions() as $reaction) {
 
-				if ( $reaction->get_meta( 'points_type' ) !== $this->atts['points_type'] ) {
+				if ($reaction->get_meta('points_type') !== $this->atts['points_type']) {
 					continue;
 				}
 
-				$points = $reaction->get_meta( 'points' );
+				$points = $reaction->get_meta('points');
 
-				if ( ! $points ) {
+				if ( ! $points) {
 					continue;
 				}
 
@@ -168,7 +168,7 @@ 
 
 				$html .= '<tr>
 					<td>' . $points . '</td>
-					<td>' . esc_html( $reaction->get_meta( 'description' ) ) . '</td>
+					<td>' . esc_html($reaction->get_meta('description')) . '</td>
 					</tr>';
 			}
 		}

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

Spacing +8 added lines, -8 removed lines

@@ -40,26 +40,26 @@
 	 */
 	protected function verify_atts() {
 
-		if ( ! wordpoints_is_points_logs_query( $this->atts['query'] ) ) {
+		if ( ! wordpoints_is_points_logs_query($this->atts['query'])) {
 			return sprintf(
 				// translators: 1. Attribute name; 2. Shortcode name; 3. Example of proper usage.
-				__( 'The “%1$s” attribute of the %2$s shortcode must be the slug of a registered points log query. Example: %3$s.', 'wordpoints' )
+				__('The “%1$s” attribute of the %2$s shortcode must be the slug of a registered points log query. Example: %3$s.', 'wordpoints')
 				, 'query'
-				, '<code>[' . sanitize_key( $this->shortcode ) . ']</code>'
-				, '<code>[' . sanitize_key( $this->shortcode ) . ' <b>query="default"</b> points_type="points"]</code>'
+				, '<code>[' . sanitize_key($this->shortcode) . ']</code>'
+				, '<code>[' . sanitize_key($this->shortcode) . ' <b>query="default"</b> points_type="points"]</code>'
 			);
 		}
 
-		if ( false === wordpoints_int( $this->atts['paginate'] ) ) {
+		if (false === wordpoints_int($this->atts['paginate'])) {
 			$this->atts['paginate'] = 1;
 		}
 
 		// Back-compat. Needs to stay here "forever" for legacy installs.
-		if ( isset( $this->atts['datatables'] ) ) {
-			$this->atts['paginate'] = wordpoints_int( $this->atts['datatables'] );
+		if (isset($this->atts['datatables'])) {
+			$this->atts['paginate'] = wordpoints_int($this->atts['datatables']);
 		}
 
-		if ( false === wordpoints_int( $this->atts['show_users'] ) ) {
+		if (false === wordpoints_int($this->atts['show_users'])) {
 			$this->atts['show_users'] = 1;
 		}
 

src/components/points/classes/widget/user/points.php

Spacing +39 added lines, -39 removed lines

@@ -30,47 +30,47 @@ 
 
 		parent::__construct(
 			'WordPoints_Points_Widget'
-			, __( 'WordPoints', 'wordpoints' )
+			, __('WordPoints', 'wordpoints')
 			, array(
-				'description' => __( 'Display the points of the current logged in user.', 'wordpoints' ),
+				'description' => __('Display the points of the current logged in user.', 'wordpoints'),
 				'wordpoints_hook_slug' => 'points',
 			)
 		);
 
 		$this->defaults = array(
-			'title'       => _x( 'My Points', 'widget title', 'wordpoints' ),
+			'title'       => _x('My Points', 'widget title', 'wordpoints'),
 			'points_type' => wordpoints_get_default_points_type(),
 			// translators: Number of points.
-			'text'        => sprintf( __( 'Points: %s', 'wordpoints' ), '%points%' ),
-			'alt_text'    => __( 'You must be logged in to view your points.', 'wordpoints' ),
+			'text'        => sprintf(__('Points: %s', 'wordpoints'), '%points%'),
+			'alt_text'    => __('You must be logged in to view your points.', 'wordpoints'),
 			'number_logs' => 5,
 		);
 
-		add_filter( 'wordpoints_points_widget_text', 'esc_html', 20 );
+		add_filter('wordpoints_points_widget_text', 'esc_html', 20);
 	}
 
 	/**
 	 * @since 1.9.0 As part of WordPoints_My_Points_Widget.
 	 * @since 2.3.0
 	 */
-	protected function verify_settings( $instance ) {
+	protected function verify_settings($instance) {
 
-		if ( ! is_user_logged_in() && empty( $instance['alt_text'] ) ) {
+		if ( ! is_user_logged_in() && empty($instance['alt_text'])) {
 			return new WP_Error;
 		}
 
 		if (
-			! isset( $instance['number_logs'] )
-			|| ! wordpoints_posint( $instance['number_logs'] )
+			! isset($instance['number_logs'])
+			|| ! wordpoints_posint($instance['number_logs'])
 		) {
 			$instance['number_logs'] = 0;
 		}
 
 		// In case the points type isn't set, we do this first.
-		$instance = parent::verify_settings( $instance );
+		$instance = parent::verify_settings($instance);
 
-		if ( ! is_wp_error( $instance ) && is_user_logged_in() && empty( $instance['text'] ) ) {
-			$instance['text'] = wordpoints_get_points_type_setting( $instance['points_type'], 'name' ) . ': %points%';
+		if ( ! is_wp_error($instance) && is_user_logged_in() && empty($instance['text'])) {
+			$instance['text'] = wordpoints_get_points_type_setting($instance['points_type'], 'name') . ': %points%';
 		}
 
 		return $instance;
@@ -80,14 +80,14 @@ 
 	 * @since 1.9.0 As part of WordPoints_My_Points_Widget.
 	 * @since 2.3.0
 	 */
-	protected function widget_body( $instance ) {
+	protected function widget_body($instance) {
 
-		if ( is_user_logged_in() ) {
+		if (is_user_logged_in()) {
 
 			$text = str_replace(
 				'%points%',
 				wordpoints_format_points(
-					wordpoints_get_points( get_current_user_id(), $instance['points_type'] ),
+					wordpoints_get_points(get_current_user_id(), $instance['points_type']),
 					$instance['points_type'],
 					'my_points_widget'
 				),
@@ -113,20 +113,20 @@ 
 		 * @param string $text The text for the widget set by the user.
 		 * @param array  $instance The settings for this instance of the widget.
 		 */
-		$text = apply_filters( 'wordpoints_points_widget_text', $text, $instance );
+		$text = apply_filters('wordpoints_points_widget_text', $text, $instance);
 
 		echo '<div class="wordpoints-points-widget-text">', $text, '</div><br />'; // XSS OK, WPCS
 
-		if ( is_user_logged_in() && 0 !== $instance['number_logs'] ) {
+		if (is_user_logged_in() && 0 !== $instance['number_logs']) {
 
-			$query_args = wordpoints_get_points_logs_query_args( $instance['points_type'], 'current_user' );
+			$query_args = wordpoints_get_points_logs_query_args($instance['points_type'], 'current_user');
 
 			$query_args['limit'] = $instance['number_logs'];
 
-			$logs_query = new WordPoints_Points_Logs_Query( $query_args );
-			$logs_query->prime_cache( 'current_user:%points_type%:%user_id%' );
+			$logs_query = new WordPoints_Points_Logs_Query($query_args);
+			$logs_query->prime_cache('current_user:%points_type%:%user_id%');
 
-			wordpoints_show_points_logs( $logs_query, array( 'paginate' => false, 'searchable' => false, 'show_users' => false ) );
+			wordpoints_show_points_logs($logs_query, array('paginate' => false, 'searchable' => false, 'show_users' => false));
 		}
 	}
 
@@ -141,14 +141,14 @@ 
 	 *
 	 * @return array The updated settings for the widget instance.
 	 */
-	public function update( $new_instance, $old_instance ) {
+	public function update($new_instance, $old_instance) {
 
-		parent::update( $new_instance, $old_instance );
+		parent::update($new_instance, $old_instance);
 
-		$this->instance['text']     = trim( $this->instance['text'] );
-		$this->instance['alt_text'] = trim( $this->instance['alt_text'] );
+		$this->instance['text']     = trim($this->instance['text']);
+		$this->instance['alt_text'] = trim($this->instance['alt_text']);
 
-		if ( ! wordpoints_posint( $this->instance['number_logs'] ) ) {
+		if ( ! wordpoints_posint($this->instance['number_logs'])) {
 			$this->instance['number_logs'] = 0;
 		}
 
@@ -159,17 +159,17 @@ 
 	 * @since 1.0.0 As part of WordPoints_My_Points_Widget.
 	 * @since 2.3.0
 	 */
-	public function form( $instance ) {
+	public function form($instance) {
 
-		parent::form( $instance );
+		parent::form($instance);
 
 		?>
 
 		<p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'text' ) ); ?>"><?php esc_html_e( 'Widget text', 'wordpoints' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'text' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'text' ) ); ?>" value="<?php echo esc_attr( $this->instance['text'] ); ?>" />
+			<label for="<?php echo esc_attr($this->get_field_id('text')); ?>"><?php esc_html_e('Widget text', 'wordpoints'); ?></label>
+			<input type="text" class="widefat" id="<?php echo esc_attr($this->get_field_id('text')); ?>" name="<?php echo esc_attr($this->get_field_name('text')); ?>" value="<?php echo esc_attr($this->instance['text']); ?>" />
 			<?php // translators: Placeholder name. ?>
-			<small><i><?php echo esc_html( sprintf( __( '%s will be replaced with the points of the logged in user', 'wordpoints' ), '%points%' ) ); ?></i></small>
+			<small><i><?php echo esc_html(sprintf(__('%s will be replaced with the points of the logged in user', 'wordpoints'), '%points%')); ?></i></small>
 			<?php
 
 			/**
@@ -179,19 +179,19 @@ 
 			 *
 			 * @param array $instance The settings of the current widget instance.
 			 */
-			do_action( 'wordpoints_my_points_widget_below_text_field', $this->instance );
+			do_action('wordpoints_my_points_widget_below_text_field', $this->instance);
 
 			?>
 		</p>
 		<p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'alt_text' ) ); ?>"><?php esc_html_e( 'Text if the user is not logged in', 'wordpoints' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'alt_text' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'alt_text' ) ); ?>" value="<?php echo esc_attr( $this->instance['alt_text'] ); ?>" />
-			<small><i><?php esc_html_e( 'Leave this field blank to hide the widget if the user is not logged in', 'wordpoints' ); ?></i></small>
+			<label for="<?php echo esc_attr($this->get_field_id('alt_text')); ?>"><?php esc_html_e('Text if the user is not logged in', 'wordpoints'); ?></label>
+			<input type="text" class="widefat" id="<?php echo esc_attr($this->get_field_id('alt_text')); ?>" name="<?php echo esc_attr($this->get_field_name('alt_text')); ?>" value="<?php echo esc_attr($this->instance['alt_text']); ?>" />
+			<small><i><?php esc_html_e('Leave this field blank to hide the widget if the user is not logged in', 'wordpoints'); ?></i></small>
 		</p>
 		<p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'number_logs' ) ); ?>"><?php esc_html_e( 'Number of latest log entries for this user to display', 'wordpoints' ); ?></label>
-			<input type="number" min="0" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'number_logs' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'number_logs' ) ); ?>" value="<?php echo esc_attr( $this->instance['number_logs'] ); ?>" />
-			<small><i><?php esc_html_e( 'Set this to 0 to keep from showing any logs', 'wordpoints' ); ?></i></small>
+			<label for="<?php echo esc_attr($this->get_field_id('number_logs')); ?>"><?php esc_html_e('Number of latest log entries for this user to display', 'wordpoints'); ?></label>
+			<input type="number" min="0" class="widefat" id="<?php echo esc_attr($this->get_field_id('number_logs')); ?>" name="<?php echo esc_attr($this->get_field_name('number_logs')); ?>" value="<?php echo esc_attr($this->instance['number_logs']); ?>" />
+			<small><i><?php esc_html_e('Set this to 0 to keep from showing any logs', 'wordpoints'); ?></i></small>
 		</p>
 
 		<?php