WordPoints / wordpoints

src/components/points/includes/deprecated.php

Spacing +82 added lines, -82 removed lines

@@ -22,11 +22,11 @@ 
  *
  * @return string
  */
-function wordpoints_points_top_shortcode( $atts ) {
+function wordpoints_points_top_shortcode($atts) {
 
-	_deprecated_function( __FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()' );
+	_deprecated_function(__FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()');
 
-	return WordPoints_Shortcodes::do_shortcode( $atts, null, 'wordpoints_points_top' );
+	return WordPoints_Shortcodes::do_shortcode($atts, null, 'wordpoints_points_top');
 }
 
 /**
@@ -51,11 +51,11 @@ 
  *
  * @return string
  */
-function wordpoints_points_logs_shortcode( $atts ) {
+function wordpoints_points_logs_shortcode($atts) {
 
-	_deprecated_function( __FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()' );
+	_deprecated_function(__FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()');
 
-	return WordPoints_Shortcodes::do_shortcode( $atts, null, 'wordpoints_points_logs' );
+	return WordPoints_Shortcodes::do_shortcode($atts, null, 'wordpoints_points_logs');
 }
 
 /**
@@ -76,11 +76,11 @@ 
  *
  * @return string The points for the user.
  */
-function wordpoints_points_shortcode( $atts ) {
+function wordpoints_points_shortcode($atts) {
 
-	_deprecated_function( __FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()' );
+	_deprecated_function(__FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()');
 
-	return WordPoints_Shortcodes::do_shortcode( $atts, null, 'wordpoints_points' );
+	return WordPoints_Shortcodes::do_shortcode($atts, null, 'wordpoints_points');
 }
 
 /**
@@ -97,11 +97,11 @@ 
  *
  * @return string A list of points hooks describing how the user can earn points.
  */
-function wordpoints_how_to_get_points_shortcode( $atts ) {
+function wordpoints_how_to_get_points_shortcode($atts) {
 
-	_deprecated_function( __FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()' );
+	_deprecated_function(__FUNCTION__, '1.8.0', 'WordPoints_Shortcodes::do_shortcode()');
 
-	return WordPoints_Shortcodes::do_shortcode( $atts, null, 'wordpoints_how_to_get_points' );
+	return WordPoints_Shortcodes::do_shortcode($atts, null, 'wordpoints_how_to_get_points');
 }
 
 /**
@@ -123,7 +123,7 @@ 
 	 *
 	 * @type array $defaults
 	 */
-	protected $defaults = array( 'points' => 10, 'post_type' => 'ALL' );
+	protected $defaults = array('points' => 10, 'post_type' => 'ALL');
 
 	/**
 	 * Initialize the hook.
@@ -132,23 +132,23 @@ 
 	 */
 	public function __construct() {
 
-		if ( ! get_site_option( 'wordpoints_comment_removed_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_comment_removed_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
 		$this->init(
-			_x( 'Comment Removed', 'points hook name', 'wordpoints' )
+			_x('Comment Removed', 'points hook name', 'wordpoints')
 			, array(
-				'description' => __( 'Comment removed from the site.', 'wordpoints' ),
+				'description' => __('Comment removed from the site.', 'wordpoints'),
 				// translators: Singular post type name.
-				'post_type_description' => __( 'Comment on a %s removed from the site.', 'wordpoints' ),
-				'post_type_filter' => array( $this, 'post_type_supports_comments' ),
-				'points_label' => __( 'Points subtracted if comment removed:', 'wordpoints' ),
+				'post_type_description' => __('Comment on a %s removed from the site.', 'wordpoints'),
+				'post_type_filter' => array($this, 'post_type_supports_comments'),
+				'points_label' => __('Points subtracted if comment removed:', 'wordpoints'),
 			)
 		);
 
-		add_action( 'transition_comment_status', array( $this, 'hook' ), 10, 3 );
-		add_filter( 'wordpoints_points_log-comment_disapprove', array( $this, 'logs' ), 10, 6 );
+		add_action('transition_comment_status', array($this, 'hook'), 10, 3);
+		add_filter('wordpoints_points_log-comment_disapprove', array($this, 'logs'), 10, 6);
 	}
 
 	/**
@@ -166,36 +166,36 @@ 
 	 *
 	 * @return void
 	 */
-	public function hook( $new_status, $old_status, $comment ) {
+	public function hook($new_status, $old_status, $comment) {
 
-		if ( ! get_site_option( 'wordpoints_comment_removed_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_comment_removed_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
-		if ( ! $comment->user_id || $old_status === $new_status ) {
+		if ( ! $comment->user_id || $old_status === $new_status) {
 			return;
 		}
 
-		$post = get_post( $comment->comment_post_ID );
+		$post = get_post($comment->comment_post_ID);
 
-		if ( 'approved' === $old_status ) {
+		if ('approved' === $old_status) {
 
-			foreach ( $this->get_instances() as $number => $instance ) {
+			foreach ($this->get_instances() as $number => $instance) {
 
-				$instance = array_merge( $this->defaults, $instance );
+				$instance = array_merge($this->defaults, $instance);
 
-				if ( ! $this->is_matching_post_type( $post->post_type, $instance['post_type'] ) ) {
+				if ( ! $this->is_matching_post_type($post->post_type, $instance['post_type'])) {
 					continue;
 				}
 
-				$points_type = $this->points_type( $number );
+				$points_type = $this->points_type($number);
 
-				wordpoints_subtract_points( $comment->user_id, $instance['points'], $points_type, 'comment_disapprove', array( 'status' => $new_status ) );
+				wordpoints_subtract_points($comment->user_id, $instance['points'], $points_type, 'comment_disapprove', array('status' => $new_status));
 
 				update_comment_meta(
 					$comment->comment_ID
-					, wp_slash( "wordpoints_last_status-{$points_type}" )
-					, wp_slash( $new_status )
+					, wp_slash("wordpoints_last_status-{$points_type}")
+					, wp_slash($new_status)
 				);
 			}
 		}
@@ -220,17 +220,17 @@ 
 	 *
 	 * @return void
 	 */
-	public function new_comment_hook( $comment_id, $comment ) {
+	public function new_comment_hook($comment_id, $comment) {
 
-		if ( ! get_site_option( 'wordpoints_comment_removed_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_comment_removed_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
-		if ( 0 === (int) $comment->user_id ) {
+		if (0 === (int) $comment->user_id) {
 			return;
 		}
 
-		switch ( $comment->comment_approved ) {
+		switch ($comment->comment_approved) {
 
 			// Comment hasn't been approved yet.
 			case 0: return;
@@ -244,7 +244,7 @@ 
 				$old_status = 'approved';
 		}
 
-		$this->hook( $new_status, $old_status, $comment );
+		$this->hook($new_status, $old_status, $comment);
 	}
 
 	/**
@@ -263,10 +263,10 @@ 
 	 *
 	 * @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) {
 
-		if ( ! get_site_option( 'wordpoints_comment_removed_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0', 'wordpoints_points_logs_comment_disapprove' );
+		if ( ! get_site_option('wordpoints_comment_removed_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0', 'wordpoints_points_logs_comment_disapprove');
 		}
 
 		return wordpoints_points_logs_comment_disapprove(
@@ -288,15 +288,15 @@ 
 	 *
 	 * @return int|false The number of points, or false.
 	 */
-	public function get_points( $number = null ) {
+	public function get_points($number = null) {
 
-		if ( ! get_site_option( 'wordpoints_comment_removed_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_comment_removed_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
-		$points = parent::get_points( $number );
+		$points = parent::get_points($number);
 
-		if ( $points ) {
+		if ($points) {
 			$points = -$points;
 		}
 
@@ -322,7 +322,7 @@ 
 	 *
 	 * @type array $defaults
 	 */
-	protected $defaults = array( 'points' => 20, 'post_type' => 'ALL' );
+	protected $defaults = array('points' => 20, 'post_type' => 'ALL');
 
 	//
 	// Public Methods.
@@ -335,22 +335,22 @@ 
 	 */
 	public function __construct() {
 
-		if ( ! get_site_option( 'wordpoints_post_delete_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_post_delete_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
 		$this->init(
-			_x( 'Post Delete', 'points hook name', 'wordpoints' )
+			_x('Post Delete', 'points hook name', 'wordpoints')
 			,array(
-				'description'  => __( 'A post is permanently deleted.', 'wordpoints' ),
-				'points_label' => __( 'Points removed when deleted:', 'wordpoints' ),
+				'description'  => __('A post is permanently deleted.', 'wordpoints'),
+				'points_label' => __('Points removed when deleted:', 'wordpoints'),
 				// translators: Singular post type name.
-				'post_type_description' => __( '%s permanently deleted.', 'wordpoints' ),
+				'post_type_description' => __('%s permanently deleted.', 'wordpoints'),
 			)
 		);
 
-		add_action( 'delete_post', array( $this, 'hook' ) );
-		add_filter( 'wordpoints_points_log-post_delete', array( $this, 'logs' ), 10, 6 );
+		add_action('delete_post', array($this, 'hook'));
+		add_filter('wordpoints_points_log-post_delete', array($this, 'logs'), 10, 6);
 	}
 
 	/**
@@ -362,30 +362,30 @@ 
 	 *
 	 * @param int $post_id The post's ID.
 	 */
-	public function hook( $post_id ) {
+	public function hook($post_id) {
 
-		if ( ! get_site_option( 'wordpoints_post_delete_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_post_delete_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
-		$post = get_post( $post_id, OBJECT, 'display' );
+		$post = get_post($post_id, OBJECT, 'display');
 
-		foreach ( $this->get_instances() as $number => $instance ) {
+		foreach ($this->get_instances() as $number => $instance) {
 
-			$instance = array_merge( $this->defaults, $instance );
+			$instance = array_merge($this->defaults, $instance);
 
 			if (
-				$this->is_matching_post_type( $post->post_type, $instance['post_type'] )
+				$this->is_matching_post_type($post->post_type, $instance['post_type'])
 				&& 'auto-draft' !== $post->post_status
-				&& __( 'Auto Draft', 'default' ) !== $post->post_title
+				&& __('Auto Draft', 'default') !== $post->post_title
 			) {
 
 				wordpoints_alter_points(
 					$post->post_author
 					, -$instance['points']
-					, $this->points_type( $number )
+					, $this->points_type($number)
 					, 'post_delete'
-					, array( 'post_title' => $post->post_title, 'post_type' => $post->post_type )
+					, array('post_title' => $post->post_title, 'post_type' => $post->post_type)
 				);
 			}
 		}
@@ -409,10 +409,10 @@ 
 	 *
 	 * @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) {
 
-		if ( ! get_site_option( 'wordpoints_post_delete_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0', 'wordpoints_points_logs_post_delete' );
+		if ( ! get_site_option('wordpoints_post_delete_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0', 'wordpoints_points_logs_post_delete');
 		}
 
 		return wordpoints_points_logs_post_delete(
@@ -434,15 +434,15 @@ 
 	 *
 	 * @return int|false The number of points, or false.
 	 */
-	public function get_points( $number = null ) {
+	public function get_points($number = null) {
 
-		if ( ! get_site_option( 'wordpoints_post_delete_hook_legacy' ) ) {
-			_deprecated_function( __METHOD__, '1.9.0' );
+		if ( ! get_site_option('wordpoints_post_delete_hook_legacy')) {
+			_deprecated_function(__METHOD__, '1.9.0');
 		}
 
-		$points = parent::get_points( $number );
+		$points = parent::get_points($number);
 
-		if ( $points ) {
+		if ($points) {
 			$points = -$points;
 		}
 
@@ -461,7 +461,7 @@ 
  */
 function wordpoints_points_get_db_schema() {
 
-	_deprecated_function( __FUNCTION__, '2.0.0' );
+	_deprecated_function(__FUNCTION__, '2.0.0');
 
 	global $wpdb;
 
@@ -500,16 +500,16 @@ 
  *
  * @param int $blog_id The ID of the new site.
  */
-function wordpoints_points_add_custom_caps_to_new_sites( $blog_id ) {
+function wordpoints_points_add_custom_caps_to_new_sites($blog_id) {
 
-	_deprecated_function( __FUNCTION__, '2.0.0' );
+	_deprecated_function(__FUNCTION__, '2.0.0');
 
-	if ( ! is_wordpoints_network_active() ) {
+	if ( ! is_wordpoints_network_active()) {
 		return;
 	}
 
-	switch_to_blog( $blog_id );
-	wordpoints_add_custom_caps( wordpoints_points_get_custom_caps() );
+	switch_to_blog($blog_id);
+	wordpoints_add_custom_caps(wordpoints_points_get_custom_caps());
 	restore_current_blog();
 }
 

src/components/points/includes/hooks/comment.php

Spacing +17 added lines, -17 removed lines

@@ -36,29 +36,29 @@ 
 	public function __construct() {
 
 		parent::__construct(
-			_x( 'Comment', 'points hook name', 'wordpoints' )
+			_x('Comment', 'points hook name', 'wordpoints')
 			, array(
-				'description' => __( 'Leaving a new comment.', 'wordpoints' ),
+				'description' => __('Leaving a new comment.', 'wordpoints'),
 				// translators: Singular post type name.
-				'post_type_description' => __( 'Leaving a new comment on a %s.', 'wordpoints' ),
+				'post_type_description' => __('Leaving a new comment on a %s.', 'wordpoints'),
 				// translators: Post title.
-				'log_text_post_title' => _x( 'Comment on %s.', 'points log description', 'wordpoints' ),
-				'log_text_no_post_title' => _x( 'Comment', 'points log description', 'wordpoints' ),
+				'log_text_post_title' => _x('Comment on %s.', 'points log description', 'wordpoints'),
+				'log_text_no_post_title' => _x('Comment', 'points log description', 'wordpoints'),
 				// translators: Singular post type name.
-				'log_text_post_type' => _x( 'Comment on a %s.', 'points log description', 'wordpoints' ),
+				'log_text_post_type' => _x('Comment on a %s.', 'points log description', 'wordpoints'),
 				// translators: Post title.
-				'log_text_post_title_reverse' => _x( 'Comment on %s removed.', 'points log description', 'wordpoints' ),
-				'log_text_no_post_title_reverse' => _x( 'Comment removed.', 'points log description', 'wordpoints' ),
+				'log_text_post_title_reverse' => _x('Comment on %s removed.', 'points log description', 'wordpoints'),
+				'log_text_no_post_title_reverse' => _x('Comment removed.', 'points log description', 'wordpoints'),
 				// translators: Singular post type name.
-				'log_text_post_type_reverse' => _x( 'Comment on a %s removed.', 'points log description', 'wordpoints' ),
+				'log_text_post_type_reverse' => _x('Comment on a %s removed.', 'points log description', 'wordpoints'),
 				'last_status_meta_key' => 'wordpoints_last_status',
 			)
 		);
 
-		if ( get_site_option( 'wordpoints_comment_hook_legacy' ) ) {
+		if (get_site_option('wordpoints_comment_hook_legacy')) {
 			$this->set_option(
 				'disable_auto_reverse_label'
-				, __( 'Revoke the points if the comment is removed.', 'wordpoints' )
+				, __('Revoke the points if the comment is removed.', 'wordpoints')
 			);
 		}
 	}
@@ -90,19 +90,19 @@ 
 	 *
 	 * @return bool Whether points have been awarded.
 	 */
-	public function awarded_points_already( $comment_id, $points_type ) {
+	public function awarded_points_already($comment_id, $points_type) {
 
-		_deprecated_function( __METHOD__, '1.8.0' );
+		_deprecated_function(__METHOD__, '1.8.0');
 
-		$meta_key = $this->get_last_status_comment_meta_key( $points_type );
+		$meta_key = $this->get_last_status_comment_meta_key($points_type);
 
-		$last_status = get_comment_meta( $comment_id, $meta_key, true );
+		$last_status = get_comment_meta($comment_id, $meta_key, true);
 
-		if ( 'approved' !== $last_status ) {
+		if ('approved' !== $last_status) {
 
 			update_comment_meta(
 				$comment_id
-				, wp_slash( $meta_key )
+				, wp_slash($meta_key)
 				, 'approved'
 				, $last_status
 			);

src/components/points/includes/hooks/post.php

Spacing +39 added lines, -39 removed lines

@@ -52,37 +52,37 @@ 
 	public function __construct() {
 
 		parent::__construct(
-			_x( 'Post Publish', 'points hook name', 'wordpoints' )
+			_x('Post Publish', 'points hook name', 'wordpoints')
 			, array(
-				'description' => __( 'New post published.', 'wordpoints' ),
+				'description' => __('New post published.', 'wordpoints'),
 				// translators: Singular post type name.
-				'post_type_description' => __( 'New %s published.', 'wordpoints' ),
+				'post_type_description' => __('New %s published.', 'wordpoints'),
 				// translators: Post title.
-				'log_text_post_title' => _x( 'Post %s published.', 'points log description', 'wordpoints' ),
+				'log_text_post_title' => _x('Post %s published.', 'points log description', 'wordpoints'),
 				// translators: 1. Singular post type name; 2. Post title.
-				'log_text_post_title_and_type' => _x( '%2$s %1$s published.', 'points log description', 'wordpoints' ),
+				'log_text_post_title_and_type' => _x('%2$s %1$s published.', 'points log description', 'wordpoints'),
 				// translators: Singular post type name.
-				'log_text_post_type' => _x( '%s published.', 'points log description', 'wordpoints' ),
-				'log_text_no_post_title' => _x( 'Post published.', 'points log description', 'wordpoints' ),
+				'log_text_post_type' => _x('%s published.', 'points log description', 'wordpoints'),
+				'log_text_no_post_title' => _x('Post published.', 'points log description', 'wordpoints'),
 				// translators: Singular post type name.
-				'log_text_post_type_reverse' => _x( '%s deleted.', 'points log description', 'wordpoints' ),
+				'log_text_post_type_reverse' => _x('%s deleted.', 'points log description', 'wordpoints'),
 				// translators: 1. Singular post type name; 2. Post title.
-				'log_text_post_title_and_type_reverse' => _x( '%1$s “%2$s” deleted.', 'points log description', 'wordpoints' ),
+				'log_text_post_title_and_type_reverse' => _x('%1$s “%2$s” deleted.', 'points log description', 'wordpoints'),
 				// translators: Post title.
-				'log_text_post_title_reverse' => _x( 'Post “%s” deleted.', 'points log description', 'wordpoints' ),
-				'log_text_no_post_title_reverse' => _x( 'Post deleted.', 'points log description', 'wordpoints' ),
+				'log_text_post_title_reverse' => _x('Post “%s” deleted.', 'points log description', 'wordpoints'),
+				'log_text_no_post_title_reverse' => _x('Post deleted.', 'points log description', 'wordpoints'),
 			)
 		);
 
-		if ( get_site_option( 'wordpoints_post_hook_legacy' ) ) {
+		if (get_site_option('wordpoints_post_hook_legacy')) {
 			$this->set_option(
 				'disable_auto_reverse_label'
-				, __( 'Revoke the points if the post is permanently deleted.', 'wordpoints' )
+				, __('Revoke the points if the post is permanently deleted.', 'wordpoints')
 			);
 		}
 
-		add_action( 'transition_post_status', array( $this, 'hook' ), 10, 3 );
-		add_action( 'delete_post', array( $this, 'reverse_hook' ) );
+		add_action('transition_post_status', array($this, 'hook'), 10, 3);
+		add_action('delete_post', array($this, 'reverse_hook'));
 	}
 
 	/**
@@ -98,29 +98,29 @@ 
 	 *
 	 * @return void
 	 */
-	public function hook( $new_status, $old_status, $post ) {
+	public function hook($new_status, $old_status, $post) {
 
-		if ( 'publish' !== $new_status ) {
+		if ('publish' !== $new_status) {
 			return;
 		}
 
-		foreach ( $this->get_instances() as $number => $instance ) {
+		foreach ($this->get_instances() as $number => $instance) {
 
-			$instance = array_merge( $this->defaults, $instance );
+			$instance = array_merge($this->defaults, $instance);
 
-			$points_type = $this->points_type( $number );
+			$points_type = $this->points_type($number);
 
 			if (
-				$this->is_matching_post_type( $post->post_type, $instance['post_type'] )
-				&& ! $this->awarded_points_already( $post->ID, $points_type )
+				$this->is_matching_post_type($post->post_type, $instance['post_type'])
+				&& ! $this->awarded_points_already($post->ID, $points_type)
 			) {
 
-				if ( isset( $instance['publish'] ) ) {
-					_deprecated_argument( __METHOD__, '1.4.0', 'The "publish" hook setting is no longer used to hold the value for the points. Use "points" instead.' );
+				if (isset($instance['publish'])) {
+					_deprecated_argument(__METHOD__, '1.4.0', 'The "publish" hook setting is no longer used to hold the value for the points. Use "points" instead.');
 					$instance['points'] = $instance['publish'];
 				}
 
-				if ( ! isset( $instance['points'] ) ) {
+				if ( ! isset($instance['points'])) {
 					continue;
 				}
 
@@ -129,7 +129,7 @@ 
 					, $instance['points']
 					, $points_type
 					, 'post_publish'
-					, array( 'post_id' => $post->ID )
+					, array('post_id' => $post->ID)
 				);
 			}
 		}
@@ -147,11 +147,11 @@ 
 	 *
 	 * @return void
 	 */
-	public function publish_hook( $new_status, $old_status, $post ) {
+	public function publish_hook($new_status, $old_status, $post) {
 
-		_deprecated_function( __METHOD__, '2.0.0', __CLASS__ . '::hook' );
+		_deprecated_function(__METHOD__, '2.0.0', __CLASS__ . '::hook');
 
-		$this->hook( $new_status, $old_status, $post );
+		$this->hook($new_status, $old_status, $post);
 	}
 
 	/**
@@ -163,10 +163,10 @@ 
 	 *
 	 * @param int $post_id The ID of the post being deleted.
 	 */
-	public function reverse_hook( $post_id ) {
+	public function reverse_hook($post_id) {
 
-		$post_type = get_post_field( 'post_type', $post_id );
-		if ( ! $this->should_do_auto_reversals_for_post_type( $post_type ) ) {
+		$post_type = get_post_field('post_type', $post_id);
+		if ( ! $this->should_do_auto_reversals_for_post_type($post_type)) {
 			return;
 		}
 
@@ -177,8 +177,8 @@ 
 			)
 		);
 
-		foreach ( $logs as $log ) {
-			$this->auto_reverse_log( $log );
+		foreach ($logs as $log) {
+			$this->auto_reverse_log($log);
 		}
 	}
 
@@ -197,11 +197,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function publish_logs( $text, $points, $points_type, $user_id, $log_type, $meta ) {
+	public function publish_logs($text, $points, $points_type, $user_id, $log_type, $meta) {
 
-		_deprecated_function( __METHOD__, '2.0.0', __CLASS__ . '::logs' );
+		_deprecated_function(__METHOD__, '2.0.0', __CLASS__ . '::logs');
 
-		return $this->logs( $text, $points, $points_type, $user_id, $log_type, $meta );
+		return $this->logs($text, $points, $points_type, $user_id, $log_type, $meta);
 	}
 
 	/**
@@ -214,7 +214,7 @@ 
 	 *
 	 * @return bool Whether points have been awarded for publishing this post before.
 	 */
-	public function awarded_points_already( $post_id, $points_type ) {
+	public function awarded_points_already($post_id, $points_type) {
 
 		$query = new WordPoints_Points_Logs_Query(
 			array(
@@ -226,7 +226,7 @@ 
 			)
 		);
 
-		return ( $query->count() > 0 );
+		return ($query->count() > 0);
 	}
 
 } // class WordPoints_Post_Points_Hook

src/components/points/includes/hooks/comment-received.php

Spacing +14 added lines, -14 removed lines

@@ -33,21 +33,21 @@ 
 	public function __construct() {
 
 		parent::__construct(
-			_x( 'Comment Received', 'points hook name', 'wordpoints' )
+			_x('Comment Received', 'points hook name', 'wordpoints')
 			, array(
-				'description' => __( 'Receiving a comment.', 'wordpoints' ),
+				'description' => __('Receiving a comment.', 'wordpoints'),
 				// translators: Singular post type name.
-				'post_type_description' => __( 'Receiving a comment on a %s.', 'wordpoints' ),
+				'post_type_description' => __('Receiving a comment on a %s.', 'wordpoints'),
 				// translators: Post title.
-				'log_text_post_title' => _x( 'Received a comment on %s.', 'points log description', 'wordpoints' ),
-				'log_text_no_post_title' => _x( 'Received a comment.', 'points log description', 'wordpoints' ),
+				'log_text_post_title' => _x('Received a comment on %s.', 'points log description', 'wordpoints'),
+				'log_text_no_post_title' => _x('Received a comment.', 'points log description', 'wordpoints'),
 				// translators: Post title.
-				'log_text_post_title_reverse' => _x( 'Comment received on %s removed.', 'points log description', 'wordpoints' ),
-				'log_text_no_post_title_reverse' => _x( 'Comment received removed.', 'points log description', 'wordpoints' ),
+				'log_text_post_title_reverse' => _x('Comment received on %s removed.', 'points log description', 'wordpoints'),
+				'log_text_no_post_title_reverse' => _x('Comment received removed.', 'points log description', 'wordpoints'),
 				// translators: Singular post type name.
-				'log_text_post_type' => _x( 'Received a comment on a %s.', 'points log description', 'wordpoints' ),
+				'log_text_post_type' => _x('Received a comment on a %s.', 'points log description', 'wordpoints'),
 				// translators: Singular post type name.
-				'log_text_post_type_reverse' => _x( 'Comment received on a %s removed.', 'points log description', 'wordpoints' ),
+				'log_text_post_type_reverse' => _x('Comment received on a %s removed.', 'points log description', 'wordpoints'),
 			)
 		);
 	}
@@ -55,15 +55,15 @@ 
 	/**
 	 * @since 1.8.0
 	 */
-	protected function shortcircuit_hook( $new_status, $old_status, $comment ) {
+	protected function shortcircuit_hook($new_status, $old_status, $comment) {
 
-		if ( parent::shortcircuit_hook( $new_status, $old_status, $comment ) ) {
+		if (parent::shortcircuit_hook($new_status, $old_status, $comment)) {
 			return true;
 		}
 
-		$post = get_post( $comment->comment_post_ID );
+		$post = get_post($comment->comment_post_ID);
 
-		if ( $post && (int) $post->post_author === (int) $comment->user_id ) {
+		if ($post && (int) $post->post_author === (int) $comment->user_id) {
 			return true;
 		}
 
@@ -73,7 +73,7 @@ 
 	/**
 	 * @since 1.8.0
 	 */
-	protected function select_user_to_award( $comment, $post ) {
+	protected function select_user_to_award($comment, $post) {
 		return $post->post_author;
 	}
 

src/components/points/includes/hooks/abstracts/comment-approved.php

Spacing +53 added lines, -53 removed lines

@@ -30,21 +30,21 @@ 
 	/**
 	 * @since 1.8.0
 	 */
-	public function __construct( $title, $args ) {
+	public function __construct($title, $args) {
 
 		$defaults = array(
-			'post_type_filter' => array( $this, 'post_type_supports_comments' ),
+			'post_type_filter' => array($this, 'post_type_supports_comments'),
 		);
 
-		$args = array_merge( $defaults, $args );
+		$args = array_merge($defaults, $args);
 
-		parent::__construct( $title, $args );
+		parent::__construct($title, $args);
 
-		add_action( 'transition_comment_status', array( $this, 'hook' ), 10, 3 );
-		add_action( 'transition_comment_status', array( $this, 'reverse_hook' ), 10, 3 );
-		add_action( 'wp_insert_comment', array( $this, 'new_comment_hook' ), 10, 2 );
+		add_action('transition_comment_status', array($this, 'hook'), 10, 3);
+		add_action('transition_comment_status', array($this, 'reverse_hook'), 10, 3);
+		add_action('wp_insert_comment', array($this, 'new_comment_hook'), 10, 2);
 
-		add_action( 'delete_comment', array( $this, 'clean_logs_on_comment_deletion' ) );
+		add_action('delete_comment', array($this, 'clean_logs_on_comment_deletion'));
 	}
 
 	/**
@@ -62,46 +62,46 @@ 
 	 *
 	 * @return void
 	 */
-	public function hook( $new_status, $old_status, $comment ) {
+	public function hook($new_status, $old_status, $comment) {
 
-		if ( $this->shortcircuit_hook( $new_status, $old_status, $comment ) ) {
+		if ($this->shortcircuit_hook($new_status, $old_status, $comment)) {
 			return;
 		}
 
-		$post = get_post( $comment->comment_post_ID );
+		$post = get_post($comment->comment_post_ID);
 
-		if ( ! $post ) {
+		if ( ! $post) {
 			return;
 		}
 
-		foreach ( $this->get_instances() as $number => $instance ) {
+		foreach ($this->get_instances() as $number => $instance) {
 
-			$instance = array_merge( $this->defaults, $instance );
+			$instance = array_merge($this->defaults, $instance);
 
-			$points_type = $this->points_type( $number );
+			$points_type = $this->points_type($number);
 
-			$meta_key = $this->get_last_status_comment_meta_key( $points_type );
+			$meta_key = $this->get_last_status_comment_meta_key($points_type);
 
-			$last_status = get_comment_meta( $comment->comment_ID, $meta_key, true );
+			$last_status = get_comment_meta($comment->comment_ID, $meta_key, true);
 
 			if (
 				'approved' === $last_status
-				|| ! $this->is_matching_post_type( $post->post_type, $instance['post_type'] )
+				|| ! $this->is_matching_post_type($post->post_type, $instance['post_type'])
 			) {
 				continue;
 			}
 
 			wordpoints_add_points(
-				$this->select_user_to_award( $comment, $post )
+				$this->select_user_to_award($comment, $post)
 				, $instance['points']
 				, $points_type
 				, $this->log_type
-				, array( 'comment_id' => $comment->comment_ID )
+				, array('comment_id' => $comment->comment_ID)
 			);
 
 			update_comment_meta(
 				$comment->comment_ID
-				, wp_slash( $meta_key )
+				, wp_slash($meta_key)
 				, 'approved'
 				, $last_status
 			);
@@ -119,9 +119,9 @@ 
 	 *
 	 * @return bool Whether the hook should short-circuit and not award any points.
 	 */
-	protected function shortcircuit_hook( $new_status, $old_status, $comment ) {
+	protected function shortcircuit_hook($new_status, $old_status, $comment) {
 
-		if ( 'approved' !== $new_status || $old_status === $new_status ) {
+		if ('approved' !== $new_status || $old_status === $new_status) {
 			return true;
 		}
 
@@ -147,9 +147,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function new_comment_hook( $comment_id, $comment ) {
+	public function new_comment_hook($comment_id, $comment) {
 
-		switch ( $comment->comment_approved ) {
+		switch ($comment->comment_approved) {
 
 			// Comment hasn't been approved yet.
 			case 0: return;
@@ -166,7 +166,7 @@ 
 				$old_status = 'approved';
 		}
 
-		$this->hook( $new_status, $old_status, $comment );
+		$this->hook($new_status, $old_status, $comment);
 	}
 
 	/**
@@ -182,14 +182,14 @@ 
 	 *
 	 * @return void
 	 */
-	public function reverse_hook( $new_status, $old_status, $comment ) {
+	public function reverse_hook($new_status, $old_status, $comment) {
 
-		if ( 'approved' !== $old_status || $old_status === $new_status ) {
+		if ('approved' !== $old_status || $old_status === $new_status) {
 			return;
 		}
 
-		$post_type = get_post_field( 'post_type', $comment->comment_post_ID );
-		if ( ! $this->should_do_auto_reversals_for_post_type( $post_type ) ) {
+		$post_type = get_post_field('post_type', $comment->comment_post_ID);
+		if ( ! $this->should_do_auto_reversals_for_post_type($post_type)) {
 			return;
 		}
 
@@ -200,17 +200,17 @@ 
 			)
 		);
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
-			$meta_key = $this->get_last_status_comment_meta_key( $log->points_type );
+			$meta_key = $this->get_last_status_comment_meta_key($log->points_type);
 
-			if ( 'approved' !== get_comment_meta( $comment->comment_ID, $meta_key, true ) ) {
+			if ('approved' !== get_comment_meta($comment->comment_ID, $meta_key, true)) {
 				continue;
 			}
 
-			$this->auto_reverse_log( $log );
+			$this->auto_reverse_log($log);
 
-			delete_comment_meta( $comment->comment_ID, wp_slash( $meta_key ) );
+			delete_comment_meta($comment->comment_ID, wp_slash($meta_key));
 		}
 	}
 
@@ -228,7 +228,7 @@ 
 	 *
 	 * @return int The ID of the user the points should be awarded to.
 	 */
-	protected function select_user_to_award( $comment, $post ) {
+	protected function select_user_to_award($comment, $post) {
 		return $comment->user_id;
 	}
 
@@ -248,30 +248,30 @@ 
 	 *
 	 * @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';
 		}
 
 		$comment = false;
 
-		if ( isset( $meta['comment_id'] ) ) {
-			$comment = get_comment( $meta['comment_id'] );
+		if (isset($meta['comment_id'])) {
+			$comment = get_comment($meta['comment_id']);
 		}
 
-		if ( ! $comment ) {
+		if ( ! $comment) {
 
-			$text = parent::logs( $text, $points, $points_type, $user_id, $log_type, $meta );
+			$text = parent::logs($text, $points, $points_type, $user_id, $log_type, $meta);
 
 		} else {
 
 			$text = $this->log_with_post_title_link(
 				$comment->comment_post_ID
 				, $reverse
-				, get_comment_link( $comment )
+				, get_comment_link($comment)
 			);
 		}
 
@@ -287,11 +287,11 @@ 
 	 *
 	 * @return string The meta key where the last status is stored.
 	 */
-	protected function get_last_status_comment_meta_key( $points_type ) {
+	protected function get_last_status_comment_meta_key($points_type) {
 
-		$meta_key = $this->get_option( 'last_status_meta_key' );
+		$meta_key = $this->get_option('last_status_meta_key');
 
-		if ( empty( $meta_key ) ) {
+		if (empty($meta_key)) {
 			$meta_key = "wordpoints_last_status-{$this->log_type}";
 		}
 
@@ -313,16 +313,16 @@ 
 	 *
 	 * @param int $comment_id The ID of the comment being deleted.
 	 */
-	public function clean_logs_on_comment_deletion( $comment_id ) {
+	public function clean_logs_on_comment_deletion($comment_id) {
 
 		$post_id = false;
-		$comment = get_comment( $comment_id );
+		$comment = get_comment($comment_id);
 
-		if ( $comment ) {
+		if ($comment) {
 			$post_id = $comment->comment_post_ID;
 		}
 
-		$this->clean_logs( 'comment_id', $comment_id, 'post_id', $post_id );
+		$this->clean_logs('comment_id', $comment_id, 'post_id', $post_id);
 	}
 
 	/**
@@ -340,7 +340,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__
@@ -348,13 +348,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;

src/components/points/admin/screens/logs.php

Spacing +18 added lines, -18 removed lines

@@ -7,17 +7,17 @@ 
  * @since 1.0.0
  */
 
-if ( is_network_admin() ) {
-	$title = __( 'WordPoints — Network Points Logs', 'wordpoints' );
+if (is_network_admin()) {
+	$title = __('WordPoints — Network Points Logs', 'wordpoints');
 } else {
-	$title = __( 'WordPoints — Points Logs', 'wordpoints' );
+	$title = __('WordPoints — Points Logs', 'wordpoints');
 }
 
 ?>
 
 <div class="wrap">
-	<h1><?php echo esc_html( $title ); ?></h1>
-	<p class="wordpoints-admin-panel-desc"><?php esc_html_e( 'View recent points transactions.', 'wordpoints' ); ?></p>
+	<h1><?php echo esc_html($title); ?></h1>
+	<p class="wordpoints-admin-panel-desc"><?php esc_html_e('View recent points transactions.', 'wordpoints'); ?></p>
 
 	<?php
 
@@ -26,17 +26,17 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_points_logs' );
+	do_action('wordpoints_admin_points_logs');
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( empty( $points_types ) ) {
+	if (empty($points_types)) {
 
 		wordpoints_show_admin_error(
 			sprintf(
 				// translators: URL of Points Types admin screen.
-				__( 'You need to <a href="%s">create a type of points</a> before you can use this page.', 'wordpoints' )
-				, esc_url( self_admin_url( 'admin.php?page=wordpoints_points_types' ) )
+				__('You need to <a href="%s">create a type of points</a> before you can use this page.', 'wordpoints')
+				, esc_url(self_admin_url('admin.php?page=wordpoints_points_types'))
 			)
 		);
 
@@ -45,20 +45,20 @@ 
 		// Show a tab for each points type.
 		$tabs = array();
 
-		foreach ( $points_types as $slug => $settings ) {
+		foreach ($points_types as $slug => $settings) {
 
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[$slug] = $settings['name'];
 		}
 
-		wordpoints_admin_show_tabs( $tabs, false );
+		wordpoints_admin_show_tabs($tabs, false);
 
-		if ( is_network_admin() ) {
+		if (is_network_admin()) {
 			$query = 'network';
 		} else {
 			$query = 'default';
 		}
 
-		$current_type = wordpoints_admin_get_current_tab( $tabs );
+		$current_type = wordpoints_admin_get_current_tab($tabs);
 
 		/**
 		 * At the top of one of the tabs on the points logs admin panel.
@@ -68,10 +68,10 @@ 
 		 * @param string $points_type The points type the current tab is for.
 		 * @param string $query       The current logs query being performed.
 		 */
-		do_action( 'wordpoints_admin_points_logs_tab', $current_type, $query );
+		do_action('wordpoints_admin_points_logs_tab', $current_type, $query);
 
 		// Get and display the logs based on current points type.
-		wordpoints_show_points_logs_query( $current_type, $query );
+		wordpoints_show_points_logs_query($current_type, $query);
 
 		/**
 		 * At the bottom of one of the tabs on the points logs admin panel.
@@ -81,7 +81,7 @@ 
 		 * @param string $points_type The points type the current tab is for.
 		 * @param string $query       The current logs query being performed.
 		 */
-		do_action( 'wordpoints_admin_points_logs_tab_after', $current_type, $query );
+		do_action('wordpoints_admin_points_logs_tab_after', $current_type, $query);
 
 	} // End if ( empty( $points_types ) ) else.
 
@@ -90,7 +90,7 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_points_logs_after' );
+	do_action('wordpoints_admin_points_logs_after');
 
 	?>
 

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

Spacing +54 added lines, -54 removed lines

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

src/components/points/classes/hook/reactor.php

Spacing +49 added lines, -49 removed lines

@@ -27,7 +27,7 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	protected $action_types = array( 'fire', 'toggle_on', 'toggle_off' );
+	protected $action_types = array('fire', 'toggle_on', 'toggle_off');
 
 	/**
 	 * @since 2.1.0
@@ -58,9 +58,9 @@ 
 	 */
 	public function get_settings_fields() {
 
-		$this->settings_fields['points']['label'] = _x( 'Points', 'form label', 'wordpoints' );
-		$this->settings_fields['log_text']['label'] = _x( 'Log Text', 'form label', 'wordpoints' );
-		$this->settings_fields['description']['label'] = _x( 'Description', 'form label', 'wordpoints' );
+		$this->settings_fields['points']['label'] = _x('Points', 'form label', 'wordpoints');
+		$this->settings_fields['log_text']['label'] = _x('Log Text', 'form label', 'wordpoints');
+		$this->settings_fields['description']['label'] = _x('Description', 'form label', 'wordpoints');
 
 		return parent::get_settings_fields();
 	}
@@ -72,8 +72,8 @@ 
 
 		$data = parent::get_ui_script_data();
 
-		$data['target_label'] = __( 'Award To', 'wordpoints' );
-		$data['periods_label'] = __( 'Award each user no more than once in:', 'wordpoints' );
+		$data['target_label'] = __('Award To', 'wordpoints');
+		$data['periods_label'] = __('Award each user no more than once in:', 'wordpoints');
 
 		return $data;
 	}
@@ -87,23 +87,23 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings['points'] ) || false === wordpoints_int( $settings['points'] ) ) {
-			$validator->add_error( __( 'Points must be an integer.', 'wordpoints' ), 'points' );
+		if ( ! isset($settings['points']) || false === wordpoints_int($settings['points'])) {
+			$validator->add_error(__('Points must be an integer.', 'wordpoints'), 'points');
 		}
 
-		if ( ! isset( $settings['points_type'] ) || ! wordpoints_is_points_type( $settings['points_type'] ) ) {
-			$validator->add_error( __( 'Invalid points type.', 'wordpoints' ), 'points_type' );
+		if ( ! isset($settings['points_type']) || ! wordpoints_is_points_type($settings['points_type'])) {
+			$validator->add_error(__('Invalid points type.', 'wordpoints'), 'points_type');
 		}
 
-		if ( ! isset( $settings['description'] ) ) {
-			$validator->add_error( __( 'Description is required.', 'wordpoints' ), 'description' );
+		if ( ! isset($settings['description'])) {
+			$validator->add_error(__('Description is required.', 'wordpoints'), 'description');
 		}
 
-		if ( ! isset( $settings['log_text'] ) ) {
-			$validator->add_error( __( 'Log Text is required.', 'wordpoints' ), 'log_text' );
+		if ( ! isset($settings['log_text'])) {
+			$validator->add_error(__('Log Text is required.', 'wordpoints'), 'log_text');
 		}
 
-		return parent::validate_settings( $settings, $validator, $event_args );
+		return parent::validate_settings($settings, $validator, $event_args);
 	}
 
 	/**
@@ -114,59 +114,59 @@ 
 		array $settings
 	) {
 
-		parent::update_settings( $reaction, $settings );
+		parent::update_settings($reaction, $settings);
 
-		$reaction->update_meta( 'points', $settings['points'] );
-		$reaction->update_meta( 'points_type', $settings['points_type'] );
-		$reaction->update_meta( 'description', $settings['description'] );
-		$reaction->update_meta( 'log_text', $settings['log_text'] );
+		$reaction->update_meta('points', $settings['points']);
+		$reaction->update_meta('points_type', $settings['points_type']);
+		$reaction->update_meta('description', $settings['description']);
+		$reaction->update_meta('log_text', $settings['log_text']);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function hit( WordPoints_Hook_Fire $fire ) {
+	public function hit(WordPoints_Hook_Fire $fire) {
 
-		if ( 'toggle_off' === $fire->action_type ) {
-			$this->reverse_hit( $fire );
+		if ('toggle_off' === $fire->action_type) {
+			$this->reverse_hit($fire);
 			return;
 		}
 
 		$reaction = $fire->reaction;
 
 		$target = $fire->event_args->get_from_hierarchy(
-			$reaction->get_meta( 'target' )
+			$reaction->get_meta('target')
 		);
 
-		if ( ! $target instanceof WordPoints_Entity ) {
+		if ( ! $target instanceof WordPoints_Entity) {
 			return;
 		}
 
-		$meta = array( 'hook_hit_id' => $fire->hit_id );
+		$meta = array('hook_hit_id' => $fire->hit_id);
 
-		foreach ( $fire->event_args->get_entities() as $entity ) {
-			$meta[ $entity->get_slug() ] = $entity->get_the_id();
-			$meta[ $entity->get_slug() . '_guid' ] = $entity->get_the_guid();
+		foreach ($fire->event_args->get_entities() as $entity) {
+			$meta[$entity->get_slug()] = $entity->get_the_id();
+			$meta[$entity->get_slug() . '_guid'] = $entity->get_the_guid();
 		}
 
 		wordpoints_alter_points(
 			$target->get_the_id()
-			, $this->get_points_to_award( $fire )
-			, $reaction->get_meta( 'points_type' )
+			, $this->get_points_to_award($fire)
+			, $reaction->get_meta('points_type')
 			, $reaction->get_event_slug()
 			, $meta
-			, $reaction->get_meta( 'log_text' )
+			, $reaction->get_meta('log_text')
 		);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
+	public function reverse_hit(WordPoints_Hook_Fire $fire) {
 
-		$hit_ids = $this->get_hit_ids_to_be_reversed( $fire );
+		$hit_ids = $this->get_hit_ids_to_be_reversed($fire);
 
-		if ( empty( $hit_ids ) ) {
+		if (empty($hit_ids)) {
 			return;
 		}
 
@@ -184,11 +184,11 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
-		$this->reverse_logs( $logs, $fire );
+		$this->reverse_logs($logs, $fire);
 	}
 
 	/**
@@ -200,10 +200,10 @@ 
 	 *
 	 * @return array The IDs of the hits to be reversed.
 	 */
-	protected function get_hit_ids_to_be_reversed( WordPoints_Hook_Fire $fire ) {
+	protected function get_hit_ids_to_be_reversed(WordPoints_Hook_Fire $fire) {
 
 		// We closely integrate with the reversals extension to get the hit IDs.
-		if ( ! isset( $fire->data['reversals']['hit_ids'] ) ) {
+		if ( ! isset($fire->data['reversals']['hit_ids'])) {
 			return array();
 		}
 
@@ -218,28 +218,28 @@ 
 	 * @param object[]             $logs The logs to reverse.
 	 * @param WordPoints_Hook_Fire $fire The fire object.
 	 */
-	protected function reverse_logs( $logs, WordPoints_Hook_Fire $fire ) {
+	protected function reverse_logs($logs, WordPoints_Hook_Fire $fire) {
 
-		$event = wordpoints_hooks()->get_sub_app( 'events' )->get(
+		$event = wordpoints_hooks()->get_sub_app('events')->get(
 			$fire->reaction->get_event_slug()
 		);
 
-		if ( $event instanceof WordPoints_Hook_Event_ReversingI ) {
+		if ($event instanceof WordPoints_Hook_Event_ReversingI) {
 
 			// translators: 1. Log text for reversed transaction; 2. Reason for reversal.
-			$template = __( 'Reversed “%1$s” (%2$s)', 'wordpoints' );
+			$template = __('Reversed “%1$s” (%2$s)', 'wordpoints');
 
 			$event_description = $event->get_reversal_text();
 
 		} else {
 
 			// translators: 1. Log text for reversed transaction.
-			$template = __( 'Reversed “%1$s”', 'wordpoints' );
+			$template = __('Reversed “%1$s”', 'wordpoints');
 
 			$event_description = '';
 		}
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
 			$log_id = wordpoints_alter_points(
 				$log->user_id
@@ -250,11 +250,11 @@ 
 					'original_log_id' => $log->id,
 					'hook_hit_id'     => $fire->hit_id,
 				)
-				, sprintf( $template, $log->text, $event_description )
+				, sprintf($template, $log->text, $event_description)
 			);
 
 			// Mark the old log as reversed by this one.
-			wordpoints_update_points_log_meta( $log->id, 'auto_reversed', $log_id );
+			wordpoints_update_points_log_meta($log->id, 'auto_reversed', $log_id);
 		}
 	}
 
@@ -267,9 +267,9 @@ 
 	 *
 	 * @return int The number of points to award.
 	 */
-	protected function get_points_to_award( WordPoints_Hook_Fire $fire ) {
+	protected function get_points_to_award(WordPoints_Hook_Fire $fire) {
 
-		$points = $fire->reaction->get_meta( 'points' );
+		$points = $fire->reaction->get_meta('points');
 
 		/**
 		 * Filters the number of points to award in the Points hook reactor.

src/components/points/classes/hook/event/post/publish/legacy.php

Spacing +6 added lines, -6 removed lines

@@ -24,7 +24,7 @@ 
 	public function get_title() {
 
 		// translators: Hook event title.
-		return sprintf( __( '%s (Legacy)', 'wordpoints' ), parent::get_title() );
+		return sprintf(__('%s (Legacy)', 'wordpoints'), parent::get_title());
 	}
 
 	/**
@@ -45,20 +45,20 @@ 
 	 */
 	public function get_reversal_text() {
 
-		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
+		$parsed = wordpoints_parse_dynamic_slug($this->slug);
 
-		switch ( $parsed['dynamic'] ) {
+		switch ($parsed['dynamic']) {
 
 			case 'post':
-				return __( 'Post deleted.', 'wordpoints' );
+				return __('Post deleted.', 'wordpoints');
 
 			case 'page':
-				return __( 'Page deleted.', 'wordpoints' );
+				return __('Page deleted.', 'wordpoints');
 
 			default:
 				return sprintf(
 					// translators: Singular post type name.
-					_x( '%s deleted.', 'post type', 'wordpoints' )
+					_x('%s deleted.', 'post type', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}