WordPoints / wordpoints

src/components/points/classes/hook/extension/legacy/repeat/blocker.php

Spacing +14 added lines, -14 removed lines

@@ -27,39 +27,39 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return (bool) $settings;
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		$block_repeats = (bool) $this->get_settings_from_fire( $fire );
+		$block_repeats = (bool) $this->get_settings_from_fire($fire);
 
-		if ( ! $block_repeats ) {
+		if ( ! $block_repeats) {
 			return true;
 		}
 
-		if ( ! parent::should_hit( $fire ) ) {
+		if ( ! parent::should_hit($fire)) {
 			return false;
 		}
 
 		$meta_queries = array();
 
-		$meta_key = $fire->reaction->get_meta( 'legacy_meta_key' );
+		$meta_key = $fire->reaction->get_meta('legacy_meta_key');
 
-		if ( $meta_key ) {
+		if ($meta_key) {
 
 			$entities = $fire->event_args->get_signature_args();
 
-			if ( ! $entities ) {
+			if ( ! $entities) {
 				return true;
 			}
 
 			// Legacy hooks only ever related to a single entity.
-			$entity = reset( $entities );
+			$entity = reset($entities);
 
 			$meta_queries = array(
 				array(
@@ -70,16 +70,16 @@ 
 		}
 
 		$user = $fire->event_args->get_from_hierarchy(
-			$fire->reaction->get_meta( 'target' )
+			$fire->reaction->get_meta('target')
 		);
 
-		if ( ! $user instanceof WordPoints_Entity ) {
+		if ( ! $user instanceof WordPoints_Entity) {
 			return true;
 		}
 
-		$log_type = $fire->reaction->get_meta( 'legacy_log_type' );
+		$log_type = $fire->reaction->get_meta('legacy_log_type');
 
-		if ( ! $log_type ) {
+		if ( ! $log_type) {
 			$log_type = $fire->reaction->get_event_slug();
 		}
 
@@ -87,7 +87,7 @@ 
 			array(
 				'user_id'     => $user->get_the_id(),
 				'log_type'    => $log_type,
-				'points_type' => $fire->reaction->get_meta( 'points_type' ),
+				'points_type' => $fire->reaction->get_meta('points_type'),
 				'meta_query'  => $meta_queries,
 			)
 		);

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

Spacing +47 added lines, -47 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()
-			, $reaction->get_meta( 'points' )
-			, $reaction->get_meta( 'points_type' )
+			, $reaction->get_meta('points')
+			, $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 transaction that is being reversed, 2: the reason that this is being reversed. */
-			$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 transaction that is being reversed. */
-			$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);
 		}
 	}
 }

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 name of the post type
-					_x( '%s deleted.', 'post type', 'wordpoints' )
+					_x('%s deleted.', 'post type', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}

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

Spacing +8 added lines, -8 removed lines

@@ -27,21 +27,21 @@ 
 		array $settings
 	) {
 
-		if ( isset( $settings['legacy_log_type'] ) ) {
+		if (isset($settings['legacy_log_type'])) {
 			$reaction->update_meta(
 				'legacy_log_type',
 				$settings['legacy_log_type']
 			);
 		}
 
-		if ( isset( $settings['legacy_meta_key'] ) ) {
+		if (isset($settings['legacy_meta_key'])) {
 			$reaction->update_meta(
 				'legacy_meta_key',
 				$settings['legacy_meta_key']
 			);
 		}
 
-		parent::update_settings( $reaction, $settings );
+		parent::update_settings($reaction, $settings);
 	}
 
 	/**
@@ -59,9 +59,9 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
+	public function reverse_hit(WordPoints_Hook_Fire $fire) {
 
-		if ( isset( $fire->data['points_legacy_reversals']['points_logs'] ) ) {
+		if (isset($fire->data['points_legacy_reversals']['points_logs'])) {
 
 			$this->reverse_logs(
 				$fire->data['points_legacy_reversals']['points_logs']
@@ -69,17 +69,17 @@ 
 			);
 
 		} else {
-			parent::reverse_hit( $fire );
+			parent::reverse_hit($fire);
 		}
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	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 legacy reversals extension to get the IDs.
-		if ( ! isset( $fire->data['points_legacy_reversals']['hit_ids'] ) ) {
+		if ( ! isset($fire->data['points_legacy_reversals']['hit_ids'])) {
 			return array();
 		}
 

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 shortcircuit 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/includes/hooks/periodic.php

Spacing +52 added lines, -52 removed lines

@@ -24,7 +24,7 @@ 
 	 *
 	 * @type array $defaults
 	 */
-	protected $defaults = array( 'period' => DAY_IN_SECONDS, 'points' => 10 );
+	protected $defaults = array('period' => DAY_IN_SECONDS, 'points' => 10);
 
 	/**
 	 * Initialize the hook.
@@ -37,13 +37,13 @@ 
 	public function __construct() {
 
 		$this->init(
-			_x( 'Periodic Points', 'points hook name', 'wordpoints' )
-			, array( 'description' => __( 'Visiting the site at least once in a given time period.', 'wordpoints' ) )
+			_x('Periodic Points', 'points hook name', 'wordpoints')
+			, array('description' => __('Visiting the site at least once in a given time period.', 'wordpoints'))
 		);
 
-		add_action( 'init', array( $this, 'hook' ) );
+		add_action('init', array($this, 'hook'));
 
-		add_filter( 'wordpoints_points_log-periodic', array( $this, 'logs' ), 10, 6 );
+		add_filter('wordpoints_points_log-periodic', array($this, 'logs'), 10, 6);
 	}
 
 	/**
@@ -59,33 +59,33 @@ 
 
 		$user_id = get_current_user_id();
 
-		if ( ! $user_id ) {
+		if ( ! $user_id) {
 			return;
 		}
 
-		$last_visit = get_user_option( 'wordpoints_points_period_start', $user_id );
+		$last_visit = get_user_option('wordpoints_points_period_start', $user_id);
 
-		if ( ! is_array( $last_visit ) ) {
+		if ( ! is_array($last_visit)) {
 			$last_visit = array();
 		}
 
-		$now = current_time( 'timestamp' );
+		$now = current_time('timestamp');
 
 		$awarded_points = false;
 
-		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 ( ! $points_type ) {
+			if ( ! $points_type) {
 				continue;
 			}
 
 			if (
-				! isset( $last_visit[ $points_type ] )
-				|| (int) ( $last_visit[ $points_type ] / $instance['period'] ) < (int) ( $now / $instance['period'] )
+				! isset($last_visit[$points_type])
+				|| (int) ($last_visit[$points_type] / $instance['period']) < (int) ($now / $instance['period'])
 			) {
 
 				wordpoints_add_points(
@@ -93,23 +93,23 @@ 
 					, $instance['points']
 					, $points_type
 					, 'periodic'
-					, array( 'period' => $instance['period'] )
+					, array('period' => $instance['period'])
 				);
 
-				$last_visit[ $points_type ] = $now;
+				$last_visit[$points_type] = $now;
 
 				$awarded_points = true;
 			}
 		}
 
-		if ( $awarded_points ) {
+		if ($awarded_points) {
 
-			$global = ( ! is_multisite() || is_wordpoints_network_active() );
+			$global = ( ! is_multisite() || is_wordpoints_network_active());
 
 			update_user_option(
 				$user_id
 				, 'wordpoints_points_period_start'
-				, wp_slash( $last_visit )
+				, wp_slash($last_visit)
 				, $global
 			);
 		}
@@ -131,28 +131,28 @@ 
 	 *
 	 * @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) {
 
-		switch ( $meta['period'] ) {
+		switch ($meta['period']) {
 
 			case HOUR_IN_SECONDS:
-				$message = _x( 'Hourly points.', 'points log description', 'wordpoints' );
+				$message = _x('Hourly points.', 'points log description', 'wordpoints');
 			break;
 
 			case DAY_IN_SECONDS:
-				$message = _x( 'Daily points.', 'points log description', 'wordpoints' );
+				$message = _x('Daily points.', 'points log description', 'wordpoints');
 			break;
 
 			case WEEK_IN_SECONDS:
-				$message = _x( 'Weekly points.', 'points log description', 'wordpoints' );
+				$message = _x('Weekly points.', 'points log description', 'wordpoints');
 			break;
 
 			case 30 * DAY_IN_SECONDS:
-				$message = _x( 'Monthly points.', 'points log description', 'wordpoints' );
+				$message = _x('Monthly points.', 'points log description', 'wordpoints');
 			break;
 
 			default:
-				$message = _x( 'Periodic points.', 'points log description', 'wordpoints' );
+				$message = _x('Periodic points.', 'points log description', 'wordpoints');
 		}
 
 		return $message;
@@ -167,30 +167,30 @@ 
 	 *
 	 * @return string A description for the hook instance.
 	 */
-	protected function generate_description( $instance = array() ) {
+	protected function generate_description($instance = array()) {
 
-		if ( ! empty( $instance['period'] ) ) {
+		if ( ! empty($instance['period'])) {
 
-			switch ( $instance['period'] ) {
+			switch ($instance['period']) {
 
 				case HOUR_IN_SECONDS:
-					return __( 'Visiting the site at least once in an hour.', 'wordpoints' );
+					return __('Visiting the site at least once in an hour.', 'wordpoints');
 
 				case DAY_IN_SECONDS:
-					return __( 'Visiting the site at least once in a day.', 'wordpoints' );
+					return __('Visiting the site at least once in a day.', 'wordpoints');
 
 				case WEEK_IN_SECONDS:
-					return __( 'Visiting the site at least once in a week.', 'wordpoints' );
+					return __('Visiting the site at least once in a week.', 'wordpoints');
 
 				case 30 * DAY_IN_SECONDS:
-					return __( 'Visiting the site at least once in a month.', 'wordpoints' );
+					return __('Visiting the site at least once in a month.', 'wordpoints');
 
 				default:
-					return __( 'Visiting the site periodically.', 'wordpoints' );
+					return __('Visiting the site periodically.', 'wordpoints');
 			}
 		}
 
-		return parent::generate_description( $instance );
+		return parent::generate_description($instance);
 	}
 
 	/**
@@ -203,12 +203,12 @@ 
 	 *
 	 * @return array Settings to save.
 	 */
-	protected function update( $new_instance, $old_instance ) {
+	protected function update($new_instance, $old_instance) {
 
-		$new_instance = array_merge( $this->defaults, $old_instance, $new_instance );
+		$new_instance = array_merge($this->defaults, $old_instance, $new_instance);
 
-		wordpoints_posint( $new_instance['points'] );
-		wordpoints_posint( $new_instance['period'] );
+		wordpoints_posint($new_instance['points']);
+		wordpoints_posint($new_instance['period']);
 
 		return $new_instance;
 	}
@@ -222,25 +222,25 @@ 
 	 *
 	 * @return bool True.
 	 */
-	protected function form( $instance ) {
+	protected function form($instance) {
 
-		$instance = array_merge( $this->defaults, $instance );
+		$instance = array_merge($this->defaults, $instance);
 
 		$dropdown_args = array(
 			'selected' => $instance['period'],
-			'id'       => $this->get_field_id( 'period' ),
-			'name'     => $this->get_field_name( 'period' ),
+			'id'       => $this->get_field_id('period'),
+			'name'     => $this->get_field_name('period'),
 			'class'    => 'widefat wordpoints-append-to-hook-title',
 		);
 
-		$dropdown = new WordPoints_Dropdown_Builder( $this->get_periods(), $dropdown_args );
+		$dropdown = new WordPoints_Dropdown_Builder($this->get_periods(), $dropdown_args);
 
-		parent::form( $instance );
+		parent::form($instance);
 
 		?>
 
 		<p>
-			<label for="<?php $this->the_field_id( 'period' ); ?>"><?php echo esc_html_x( 'Period:', 'length of time', 'wordpoints' ); ?></label>
+			<label for="<?php $this->the_field_id('period'); ?>"><?php echo esc_html_x('Period:', 'length of time', 'wordpoints'); ?></label>
 			<?php $dropdown->display(); ?>
 		</p>
 
@@ -261,10 +261,10 @@ 
 	protected function get_periods() {
 
 		$periods = array(
-			HOUR_IN_SECONDS     => __( 'hourly',  'wordpoints' ),
-			DAY_IN_SECONDS      => __( 'daily',   'wordpoints' ),
-			WEEK_IN_SECONDS     => __( 'weekly',  'wordpoints' ),
-			30 * DAY_IN_SECONDS => __( 'monthly', 'wordpoints' ),
+			HOUR_IN_SECONDS     => __('hourly', 'wordpoints'),
+			DAY_IN_SECONDS      => __('daily', 'wordpoints'),
+			WEEK_IN_SECONDS     => __('weekly', 'wordpoints'),
+			30 * DAY_IN_SECONDS => __('monthly', 'wordpoints'),
 		);
 
 		/**
@@ -275,7 +275,7 @@ 
 		 * @param array $periods The default periods. Values are period names, keys
 		 *        length of periods in seconds.
 		 */
-		return apply_filters( 'wordpoints_points_periods', $periods );
+		return apply_filters('wordpoints_points_periods', $periods);
 	}
 
 } // class WordPoints_Periodic_Points_Hook

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: the post type name. */
-				'post_type_description' => __( 'New %s published.', 'wordpoints' ),
+				'post_type_description' => __('New %s published.', 'wordpoints'),
 				/* translators: %s will be a link to the post. */
-				'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 is the post type name, 2 is a link to the post. */
-				'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: %s is the name of a post type. */
-				'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: %s is the name of a post type. */
-				'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 is the post type name, 2 is the 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: %s will be the 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.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: the 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: %s will be the post's 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: %s is the name of a post type. */
-				'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: %s will be the post's 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: %s is the name of a post type. */
-				'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/registration.php

Spacing +10 added lines, -10 removed lines

@@ -23,7 +23,7 @@ 
 	 *
 	 * @type array $defaults
 	 */
-	protected $defaults = array( 'points' => 100 );
+	protected $defaults = array('points' => 100);
 
 	/**
 	 * Construct the class.
@@ -36,12 +36,12 @@ 
 	public function __construct() {
 
 		$this->init(
-			_x( 'Registration', 'points hook name', 'wordpoints' )
-			, array( 'description' => _x( 'Registering with the site.', 'points hook description', 'wordpoints' ) )
+			_x('Registration', 'points hook name', 'wordpoints')
+			, array('description' => _x('Registering with the site.', 'points hook description', 'wordpoints'))
 		);
 
-		add_action( 'user_register', array( $this, 'hook' ) );
-		add_filter( 'wordpoints_points_log-register', array( $this, 'logs' ) );
+		add_action('user_register', array($this, 'hook'));
+		add_filter('wordpoints_points_log-register', array($this, 'logs'));
 	}
 
 	/**
@@ -55,12 +55,12 @@ 
 	 *
 	 * @return void
 	 */
-	public function hook( $user_id ) {
+	public function hook($user_id) {
 
-		foreach ( $this->get_instances() as $number => $instance ) {
+		foreach ($this->get_instances() as $number => $instance) {
 
-			if ( isset( $instance['points'] ) ) {
-				wordpoints_add_points( $user_id, $instance['points'], $this->points_type( $number ), 'register' );
+			if (isset($instance['points'])) {
+				wordpoints_add_points($user_id, $instance['points'], $this->points_type($number), 'register');
 			}
 		}
 	}
@@ -76,7 +76,7 @@ 
 	 */
 	public function logs() {
 
-		return _x( 'Registration.', 'points log description', 'wordpoints' );
+		return _x('Registration.', 'points log description', 'wordpoints');
 	}
 
 } // class WordPoints_Registration_Points_Hook