WordPoints / hooks-api

src/includes/classes/hook/extension.php

Spacing +10 added lines, -10 removed lines

@@ -65,15 +65,15 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings[ $this->slug ] ) ) {
+		if ( ! isset($settings[$this->slug])) {
 			return $settings;
 		}
 
 		$this->validator = $validator;
 		$this->event_args = $event_args;
 
-		$this->validator->push_field( $this->slug );
-		$settings[ $this->slug ] = $this->{"validate_{$this->slug}"}( $settings[ $this->slug ] );
+		$this->validator->push_field($this->slug);
+		$settings[$this->slug] = $this->{"validate_{$this->slug}"}($settings[$this->slug]);
 		$this->validator->pop_field();
 
 		return $settings;
@@ -82,12 +82,12 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_settings( WordPoints_Hook_ReactionI $reaction, array $settings ) {
+	public function update_settings(WordPoints_Hook_ReactionI $reaction, array $settings) {
 
-		if ( isset( $settings[ $this->slug ] ) ) {
-			$reaction->update_meta( $this->slug, $settings[ $this->slug ] );
+		if (isset($settings[$this->slug])) {
+			$reaction->update_meta($this->slug, $settings[$this->slug]);
 		} else {
-			$reaction->delete_meta( $this->slug );
+			$reaction->delete_meta($this->slug);
 		}
 	}
 
@@ -100,7 +100,7 @@ 
 	 *
 	 * @return bool Whether the target should be hit by this hook firing.
 	 */
-	abstract public function should_hit( WordPoints_Hook_Fire $fire );
+	abstract public function should_hit(WordPoints_Hook_Fire $fire);
 
 	/**
 	 * After a reaction has hit the target.
@@ -109,7 +109,7 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	public function after_hit( WordPoints_Hook_Fire $fire ) {}
+	public function after_hit(WordPoints_Hook_Fire $fire) {}
 
 	/**
 	 * Called after a reverse action is called.
@@ -118,7 +118,7 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The reverse fire object.
 	 */
-	public function after_reverse( WordPoints_Hook_Fire $fire ) {}
+	public function after_reverse(WordPoints_Hook_Fire $fire) {}
 
 	/**
 	 * Get the data the scripts need for the UI.

src/includes/classes/hook/extension/conditions.php

Spacing +51 added lines, -51 removed lines

@@ -42,14 +42,14 @@ 
 
 		$conditions_data = array();
 
-		foreach ( $this->conditions->get_all() as $data_type => $conditions ) {
-			foreach ( $conditions as $slug => $condition ) {
+		foreach ($this->conditions->get_all() as $data_type => $conditions) {
+			foreach ($conditions as $slug => $condition) {
 
-				if ( ! ( $condition instanceof WordPoints_Hook_Condition ) ) {
+				if ( ! ($condition instanceof WordPoints_Hook_Condition)) {
 					continue;
 				}
 
-				$conditions_data[ $data_type ][ $slug ] = array(
+				$conditions_data[$data_type][$slug] = array(
 					'slug'      => $slug,
 					'data_type' => $data_type,
 					'title'     => $condition->get_title(),
@@ -58,7 +58,7 @@ 
 			}
 		}
 
-		return array( 'conditions' => $conditions_data );
+		return array('conditions' => $conditions_data);
 	}
 
 	/**
@@ -70,31 +70,31 @@ 
 	 *
 	 * @return array The validated settings.
 	 */
-	protected function validate_conditions( $args ) {
+	protected function validate_conditions($args) {
 
-		if ( ! is_array( $args ) ) {
+		if ( ! is_array($args)) {
 
 			$this->validator->add_error(
-				__( 'Conditions do not match expected format.', 'wordpoints' )
+				__('Conditions do not match expected format.', 'wordpoints')
 			);
 
 			return array();
 		}
 
-		foreach ( $args as $arg_slug => $sub_args ) {
+		foreach ($args as $arg_slug => $sub_args) {
 
-			if ( '_conditions' === $arg_slug ) {
+			if ('_conditions' === $arg_slug) {
 
-				$this->validator->push_field( $arg_slug );
+				$this->validator->push_field($arg_slug);
 
-				foreach ( $sub_args as $index => $settings ) {
+				foreach ($sub_args as $index => $settings) {
 
-					$this->validator->push_field( $index );
+					$this->validator->push_field($index);
 
-					$condition = $this->validate_condition( $settings );
+					$condition = $this->validate_condition($settings);
 
-					if ( $condition ) {
-						$sub_args[ $index ] = $condition;
+					if ($condition) {
+						$sub_args[$index] = $condition;
 					}
 
 					$this->validator->pop_field();
@@ -104,18 +104,18 @@ 
 
 			} else {
 
-				if ( ! $this->event_args->descend( $arg_slug ) ) {
+				if ( ! $this->event_args->descend($arg_slug)) {
 					continue;
 				}
 
-				$sub_args = $this->validate_conditions( $sub_args );
+				$sub_args = $this->validate_conditions($sub_args);
 
-				$args[ $arg_slug ] = $sub_args;
+				$args[$arg_slug] = $sub_args;
 
 				$this->event_args->ascend();
 			}
 
-			$args[ $arg_slug ] = $sub_args;
+			$args[$arg_slug] = $sub_args;
 		}
 
 		return $args;
@@ -131,32 +131,32 @@ 
 	 * @return array|false The validated conditions settings, or false if unable to
 	 *                     validate.
 	 */
-	protected function validate_condition( $settings ) {
+	protected function validate_condition($settings) {
 
-		if ( ! isset( $settings['type'] ) ) {
-			$this->validator->add_error( __( 'Condition type is missing.', 'wordpoints' ) );
+		if ( ! isset($settings['type'])) {
+			$this->validator->add_error(__('Condition type is missing.', 'wordpoints'));
 			return false;
 		}
 
 		$arg = $this->event_args->get_current();
 
-		$data_type = $this->get_data_type( $arg );
+		$data_type = $this->get_data_type($arg);
 
-		if ( ! $data_type ) {
+		if ( ! $data_type) {
 			$this->validator->add_error(
-				__( 'This type of condition does not work for the selected attribute.', 'wordpoints' )
+				__('This type of condition does not work for the selected attribute.', 'wordpoints')
 			);
 
 			return false;
 		}
 
-		$condition = wordpoints_hooks()->conditions->get( $data_type, $settings['type'] );
+		$condition = wordpoints_hooks()->conditions->get($data_type, $settings['type']);
 
-		if ( ! $condition ) {
+		if ( ! $condition) {
 
 			$this->validator->add_error(
 				sprintf(
-					__( 'Unknown condition type “%s”.', 'wordpoints' )
+					__('Unknown condition type “%s”.', 'wordpoints')
 					, $settings['type']
 				)
 				, 'type'
@@ -165,17 +165,17 @@ 
 			return false;
 		}
 
-		if ( ! isset( $settings['settings'] ) ) {
-			$this->validator->add_error( __( 'Condition settings are missing.', 'wordpoints' ) );
+		if ( ! isset($settings['settings'])) {
+			$this->validator->add_error(__('Condition settings are missing.', 'wordpoints'));
 			return false;
 		}
 
-		$this->validator->push_field( 'settings' );
+		$this->validator->push_field('settings');
 
 		// The condition may call this object's validate_settings() method to
 		// validate some sub-conditions. When that happens, these properties will be
 		// reset, so we need to back up their values and then restore them below.
-		$backup = array( $this->validator, $this->event_args );
+		$backup = array($this->validator, $this->event_args);
 
 		$settings['settings'] = $condition->validate_settings(
 			$arg
@@ -183,7 +183,7 @@ 
 			, $this->validator
 		);
 
-		list( $this->validator, $this->event_args ) = $backup;
+		list($this->validator, $this->event_args) = $backup;
 
 		$this->validator->pop_field();
 
@@ -193,11 +193,11 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		$conditions = $fire->reaction->get_meta( 'conditions' );
+		$conditions = $fire->reaction->get_meta('conditions');
 
-		if ( $conditions && ! $this->conditions_are_met( $conditions, $fire->event_args ) ) {
+		if ($conditions && ! $this->conditions_are_met($conditions, $fire->event_args)) {
 			return false;
 		}
 
@@ -219,35 +219,35 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		foreach ( $conditions as $arg_slug => $sub_args ) {
+		foreach ($conditions as $arg_slug => $sub_args) {
 
-			$event_args->descend( $arg_slug );
+			$event_args->descend($arg_slug);
 
-			if ( isset( $sub_args['_conditions'] ) ) {
+			if (isset($sub_args['_conditions'])) {
 
-				foreach ( $sub_args['_conditions'] as $settings ) {
+				foreach ($sub_args['_conditions'] as $settings) {
 
 					$condition = $this->conditions->get(
-						$this->get_data_type( $event_args->get_current() )
+						$this->get_data_type($event_args->get_current())
 						, $settings['type']
 					);
 
-					$is_met = $condition->is_met( $settings['settings'], $event_args );
+					$is_met = $condition->is_met($settings['settings'], $event_args);
 
-					if ( ! $is_met ) {
+					if ( ! $is_met) {
 						$event_args->ascend();
 						return false;
 					}
 				}
 
-				unset( $sub_args['_conditions'] );
+				unset($sub_args['_conditions']);
 			}
 
-			$are_met = $this->conditions_are_met( $sub_args, $event_args );
+			$are_met = $this->conditions_are_met($sub_args, $event_args);
 
 			$event_args->ascend();
 
-			if ( ! $are_met ) {
+			if ( ! $are_met) {
 				return false;
 			}
 		}
@@ -264,13 +264,13 @@ 
 	 *
 	 * @return string|false The data type, or false.
 	 */
-	protected function get_data_type( $arg ) {
+	protected function get_data_type($arg) {
 
-		if ( $arg instanceof WordPoints_Entity_Attr ) {
+		if ($arg instanceof WordPoints_Entity_Attr) {
 			$data_type = $arg->get_data_type();
-		} elseif ( $arg instanceof WordPoints_Entity_Array ) {
+		} elseif ($arg instanceof WordPoints_Entity_Array) {
 			$data_type = 'entity_array';
-		} elseif ( $arg instanceof WordPoints_Entity ) {
+		} elseif ($arg instanceof WordPoints_Entity) {
 			$data_type = 'entity';
 		} else {
 			$data_type = false;

src/includes/classes/hook/extension/periods.php

Spacing +74 added lines, -74 removed lines

@@ -25,11 +25,11 @@ 
 	public function get_ui_script_data() {
 
 		$periods = array(
-			MINUTE_IN_SECONDS   => __( 'Minute', 'wordpoints' ),
-			HOUR_IN_SECONDS     => __( 'Hour',   'wordpoints' ),
-			DAY_IN_SECONDS      => __( 'Day',    'wordpoints' ),
-			WEEK_IN_SECONDS     => __( 'Week',   'wordpoints' ),
-			30 * DAY_IN_SECONDS => __( 'Month',  'wordpoints' ),
+			MINUTE_IN_SECONDS   => __('Minute', 'wordpoints'),
+			HOUR_IN_SECONDS     => __('Hour', 'wordpoints'),
+			DAY_IN_SECONDS      => __('Day', 'wordpoints'),
+			WEEK_IN_SECONDS     => __('Week', 'wordpoints'),
+			30 * DAY_IN_SECONDS => __('Month', 'wordpoints'),
 		);
 
 		/**
@@ -39,13 +39,13 @@ 
 		 *
 		 * @param string[] $periods The period titles, indexed by length in seconds.
 		 */
-		$periods = apply_filters( 'wordpoints_hooks_ui_data_periods', $periods );
+		$periods = apply_filters('wordpoints_hooks_ui_data_periods', $periods);
 
 		return array(
 			'periods' => $periods,
 			'l10n' => array(
 				// TODO this should be supplied per-reactor
-				'label' => __( 'Award each user no more than once per:', 'wordpoints' ),
+				'label' => __('Award each user no more than once per:', 'wordpoints'),
 			),
 		);
 	}
@@ -59,25 +59,25 @@ 
 	 *
 	 * @return array The validated periods.
 	 */
-	protected function validate_periods( $periods ) {
+	protected function validate_periods($periods) {
 
-		if ( ! is_array( $periods ) ) {
+		if ( ! is_array($periods)) {
 
 			$this->validator->add_error(
-				__( 'Periods do not match expected format.', 'wordpoints' )
+				__('Periods do not match expected format.', 'wordpoints')
 			);
 
 			return array();
 		}
 
-		foreach ( $periods as $index => $period ) {
+		foreach ($periods as $index => $period) {
 
-			$this->validator->push_field( $index );
+			$this->validator->push_field($index);
 
-			$period = $this->validate_period( $period );
+			$period = $this->validate_period($period);
 
-			if ( $period ) {
-				$periods[ $index ] = $period;
+			if ($period) {
+				$periods[$index] = $period;
 			}
 
 			$this->validator->pop_field();
@@ -95,30 +95,30 @@ 
 	 *
 	 * @return array|false The validated period, or false if invalid.
 	 */
-	protected function validate_period( $period ) {
+	protected function validate_period($period) {
 
-		if ( ! is_array( $period ) ) {
+		if ( ! is_array($period)) {
 			$this->validator->add_error(
-				__( 'Period does not match expected format.', 'wordpoints' )
+				__('Period does not match expected format.', 'wordpoints')
 			);
 
 			return false;
 		}
 
-		if ( isset( $period['args'] ) ) {
-			$this->validate_period_args( $period['args'] );
+		if (isset($period['args'])) {
+			$this->validate_period_args($period['args']);
 		}
 
-		if ( ! isset( $period['length'] ) ) {
+		if ( ! isset($period['length'])) {
 
 			$this->validator->add_error(
-				__( 'Period length setting is missing.', 'wordpoints' )
+				__('Period length setting is missing.', 'wordpoints')
 			);
 
-		} elseif ( false === wordpoints_posint( $period['length'] ) ) {
+		} elseif (false === wordpoints_posint($period['length'])) {
 
 			$this->validator->add_error(
-				__( 'Period length must be a positive integer.', 'wordpoints' )
+				__('Period length must be a positive integer.', 'wordpoints')
 				, 'length'
 			);
 
@@ -135,34 +135,34 @@ 
 	 *
 	 * @param mixed $args The args the period is related to.
 	 */
-	protected function validate_period_args( $args ) {
+	protected function validate_period_args($args) {
 
-		if ( ! is_array( $args ) ) {
+		if ( ! is_array($args)) {
 
 			$this->validator->add_error(
-				__( 'Period does not match expected format.', 'wordpoints' )
+				__('Period does not match expected format.', 'wordpoints')
 				, 'args'
 			);
 
 			return;
 		}
 
-		$this->validator->push_field( 'args' );
+		$this->validator->push_field('args');
 
-		foreach ( $args as $index => $hierarchy ) {
+		foreach ($args as $index => $hierarchy) {
 
-			$this->validator->push_field( $index );
+			$this->validator->push_field($index);
 
-			if ( ! is_array( $hierarchy ) ) {
+			if ( ! is_array($hierarchy)) {
 
 				$this->validator->add_error(
-					__( 'Period does not match expected format.', 'wordpoints' )
+					__('Period does not match expected format.', 'wordpoints')
 				);
 
-			} elseif ( ! $this->event_args->get_from_hierarchy( $hierarchy ) ) {
+			} elseif ( ! $this->event_args->get_from_hierarchy($hierarchy)) {
 
 				$this->validator->add_error(
-					__( 'Invalid period.', 'wordpoints' ) // TODO better error message
+					__('Invalid period.', 'wordpoints') // TODO better error message
 				);
 			}
 
@@ -175,18 +175,18 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		$periods = $fire->reaction->get_meta( 'periods' );
+		$periods = $fire->reaction->get_meta('periods');
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return true;
 		}
 
 		$this->event_args = $fire->event_args;
 
-		foreach ( $periods as $period ) {
-			if ( ! $this->has_period_ended( $period, $fire->reaction ) ) {
+		foreach ($periods as $period) {
+			if ( ! $this->has_period_ended($period, $fire->reaction)) {
 				return false;
 			}
 		}
@@ -210,24 +210,24 @@ 
 	) {
 
 		$period = $this->get_period_by_reaction(
-			$this->get_period_signature( $settings, $reaction )
+			$this->get_period_signature($settings, $reaction)
 			, $reaction
 		);
 
 		// If the period isn't found, we know that we can still fire.
-		if ( ! $period ) {
+		if ( ! $period) {
 			return true;
 		}
 
-		$now = current_time( 'timestamp' );
-		$hit_time = strtotime( $period->hit_time, $now );
+		$now = current_time('timestamp');
+		$hit_time = strtotime($period->hit_time, $now);
 
-		if ( ! empty( $settings['relative'] ) ) {
-			return ( $hit_time < $now - $settings['length'] );
+		if ( ! empty($settings['relative'])) {
+			return ($hit_time < $now - $settings['length']);
 		} else {
 			return (
-				(int) ( $hit_time / $settings['length'] )
-				< (int) ( $now / $settings['length'] )
+				(int) ($hit_time / $settings['length'])
+				< (int) ($now / $settings['length'])
 			);
 		}
 	}
@@ -241,24 +241,24 @@ 
 	 *
 	 * @return array The arg values.
 	 */
-	protected function get_arg_values( array $period_args ) {
+	protected function get_arg_values(array $period_args) {
 
 		$values = array();
 
-		foreach ( $period_args as $arg_hierarchy ) {
+		foreach ($period_args as $arg_hierarchy) {
 
 			$arg = $this->event_args->get_from_hierarchy(
 				$arg_hierarchy
 			);
 
-			if ( ! $arg instanceof WordPoints_EntityishI ) {
+			if ( ! $arg instanceof WordPoints_EntityishI) {
 				continue;
 			}
 
-			$values[ implode( '.', $arg_hierarchy ) ] = $arg->get_the_value();
+			$values[implode('.', $arg_hierarchy)] = $arg->get_the_value();
 		}
 
-		ksort( $values );
+		ksort($values);
 
 		return $values;
 	}
@@ -272,11 +272,11 @@ 
 	 *
 	 * @return object|false The period data, or false if not found.
 	 */
-	protected function get_period( $period_id ) {
+	protected function get_period($period_id) {
 
-		$period = wp_cache_get( $period_id, 'wordpoints_hook_period' );
+		$period = wp_cache_get($period_id, 'wordpoints_hook_period');
 
-		if ( ! $period ) {
+		if ( ! $period) {
 
 			global $wpdb;
 
@@ -291,7 +291,7 @@ 
 				)
 			);
 
-			if ( ! $period ) {
+			if ( ! $period) {
 				return false;
 			}
 
@@ -301,7 +301,7 @@ 
 				, 'wordpoints_hook_period_ids'
 			);
 
-			wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+			wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 		}
 
 		return $period;
@@ -328,11 +328,11 @@ 
 		$cache_key = "{$reaction_id}-{$signature}";
 
 		// Before we run the query, we try to lookup the ID in the cache.
-		$period_id = wp_cache_get( $cache_key, 'wordpoints_hook_period_ids' );
+		$period_id = wp_cache_get($cache_key, 'wordpoints_hook_period_ids');
 
 		// If we found it, we can retrieve the period by ID instead.
-		if ( $period_id ) {
-			return $this->get_period( $period_id );
+		if ($period_id) {
+			return $this->get_period($period_id);
 		}
 
 		global $wpdb;
@@ -353,12 +353,12 @@ 
 			)
 		);
 
-		if ( ! $period ) {
+		if ( ! $period) {
 			return false;
 		}
 
-		wp_cache_set( $cache_key, $period->id, 'wordpoints_hook_period_ids' );
-		wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+		wp_cache_set($cache_key, $period->id, 'wordpoints_hook_period_ids');
+		wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 
 		return $period;
 	}
@@ -366,20 +366,20 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function after_hit( WordPoints_Hook_Fire $fire ) {
+	public function after_hit(WordPoints_Hook_Fire $fire) {
 
-		$periods = $fire->reaction->get_meta( 'periods' );
+		$periods = $fire->reaction->get_meta('periods');
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return;
 		}
 
 		$this->event_args = $fire->event_args;
 
-		foreach ( $periods as $settings ) {
+		foreach ($periods as $settings) {
 
 			$this->add_period(
-				$this->get_period_signature( $settings, $fire->reaction )
+				$this->get_period_signature($settings, $fire->reaction)
 				, $fire->reaction
 			);
 		}
@@ -404,14 +404,14 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 
-		if ( isset( $settings['args'] ) ) {
+		if (isset($settings['args'])) {
 			$period_args = $settings['args'];
 		} else {
-			$period_args = array( $reaction->get_meta( 'target' ) );
+			$period_args = array($reaction->get_meta('target'));
 		}
 
 		return wordpoints_hash(
-			wp_json_encode( $this->get_arg_values( $period_args ) )
+			wp_json_encode($this->get_arg_values($period_args))
 		);
 	}
 
@@ -439,12 +439,12 @@ 
 			, array(
 				'reaction_id' => $reaction_id,
 				'signature'   => $signature,
-				'hit_time'    => current_time( 'mysql' ),
+				'hit_time'    => current_time('mysql'),
 			)
-			, array( '%d', '%s', '%s' )
+			, array('%d', '%s', '%s')
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}
 

src/includes/classes/hook/fire.php

Spacing +6 added lines, -6 removed lines

@@ -84,7 +84,7 @@ 
 		$this->firer      = $firer;
 		$this->event_args = $event_args;
 		$this->reaction   = $reaction;
-		$this->hit_logger = new WordPoints_Hook_Hit_Logger( $this );
+		$this->hit_logger = new WordPoints_Hook_Hit_Logger($this);
 		$this->supersedes = $supersedes;
 	}
 
@@ -97,11 +97,11 @@ 
 	 */
 	public function hit() {
 
-		if ( ! $this->hit_id ) {
+		if ( ! $this->hit_id) {
 
 			$this->hit_id = $this->hit_logger->log_hit();
 
-			if ( ! $this->hit_id ) {
+			if ( ! $this->hit_id) {
 				return false;
 			}
 		}
@@ -120,7 +120,7 @@ 
 	 */
 	public function get_superseded_hit() {
 
-		if ( isset( $this->supersedes ) ) {
+		if (isset($this->supersedes)) {
 			return $this->supersedes;
 		}
 
@@ -140,7 +140,7 @@ 
 					AND `superseded_by` IS NULL
 				"
 				, $this->firer->get_slug()
-				, wordpoints_hooks_get_event_signature( $this->event_args )
+				, wordpoints_hooks_get_event_signature($this->event_args)
 				, $this->reaction->get_event_slug()
 				, $this->reaction->get_reactor_slug()
 				, $this->reaction->get_storage_group_slug()
@@ -148,7 +148,7 @@ 
 			)
 		);
 
-		$this->supersedes = ( $hit ) ? $hit : false;
+		$this->supersedes = ($hit) ? $hit : false;
 
 		return $this->supersedes;
 	}

src/includes/classes/hook/firer.php

Spacing +12 added lines, -12 removed lines

@@ -26,7 +26,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 		$this->slug = $slug;
 	}
 
@@ -40,14 +40,14 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) {
+	public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) {
 
 		$hooks = wordpoints_hooks();
 
 		/** @var WordPoints_Hook_Reactor $reactor */
-		foreach ( $hooks->reactors->get_all() as $reactor ) {
+		foreach ($hooks->reactors->get_all() as $reactor) {
 
-			foreach ( $reactor->get_all_reactions_to_event( $event_slug ) as $reaction ) {
+			foreach ($reactor->get_all_reactions_to_event($event_slug) as $reaction) {
 
 				$validator = new WordPoints_Hook_Reaction_Validator(
 					$reaction
@@ -57,29 +57,29 @@ 
 
 				$validator->validate();
 
-				if ( $validator->had_errors() ) {
+				if ($validator->had_errors()) {
 					continue;
 				}
 
-				unset( $validator );
+				unset($validator);
 
-				$fire = new WordPoints_Hook_Fire( $this, $event_args, $reaction );
+				$fire = new WordPoints_Hook_Fire($this, $event_args, $reaction);
 
 				/** @var WordPoints_Hook_Extension[] $extensions */
 				$extensions = $hooks->extensions->get_all();
 
-				foreach ( $extensions as $extension ) {
-					if ( ! $extension->should_hit( $fire ) ) {
+				foreach ($extensions as $extension) {
+					if ( ! $extension->should_hit($fire)) {
 						continue 2;
 					}
 				}
 
 				$fire->hit();
 
-				$reactor->hit( $fire );
+				$reactor->hit($fire);
 
-				foreach ( $extensions as $extension ) {
-					$extension->after_hit( $fire );
+				foreach ($extensions as $extension) {
+					$extension->after_hit($fire);
 				}
 			}
 		}

src/includes/classes/hook/firer/reverse.php

Spacing +15 added lines, -15 removed lines

@@ -17,39 +17,39 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) {
+	public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) {
 
 		$hooks = wordpoints_hooks();
 
-		foreach ( $this->get_hits( $event_slug, $event_args ) as $hit ) {
+		foreach ($this->get_hits($event_slug, $event_args) as $hit) {
 
-			$reactor = $hooks->reactors->get( $hit->reactor );
+			$reactor = $hooks->reactors->get($hit->reactor);
 
-			if ( ! $reactor instanceof WordPoints_Hook_Reactor ) {
+			if ( ! $reactor instanceof WordPoints_Hook_Reactor) {
 				continue;
 			}
 
-			$reactions = $reactor->get_reaction_group( $hit->reaction_type );
+			$reactions = $reactor->get_reaction_group($hit->reaction_type);
 
-			if ( ! $reactions ) {
+			if ( ! $reactions) {
 				continue;
 			}
 
-			$reaction = $reactions->get_reaction( $hit->reaction_id );
+			$reaction = $reactions->get_reaction($hit->reaction_id);
 
-			if ( ! $reaction ) {
+			if ( ! $reaction) {
 				continue;
 			}
 
-			$fire = new WordPoints_Hook_Fire( $this, $event_args, $reaction, $hit );
+			$fire = new WordPoints_Hook_Fire($this, $event_args, $reaction, $hit);
 
 			$fire->hit();
 
-			$reactor->reverse_hit( $fire );
+			$reactor->reverse_hit($fire);
 
 			/** @var WordPoints_Hook_Extension $extension */
-			foreach ( $hooks->extensions->get_all() as $extension ) {
-				$extension->after_reverse( $fire );
+			foreach ($hooks->extensions->get_all() as $extension) {
+				$extension->after_reverse($fire);
 			}
 		}
 	}
@@ -64,7 +64,7 @@ 
 	 *
 	 * @return object[] The data for each hit from the hit logs database table.
 	 */
-	protected function get_hits( $event_slug, WordPoints_Hook_Event_Args $event_args ) {
+	protected function get_hits($event_slug, WordPoints_Hook_Event_Args $event_args) {
 
 		global $wpdb;
 
@@ -78,12 +78,12 @@ 
 					AND `event` = %s
 					AND `superseded_by` IS NULL
 				"
-				, wordpoints_hooks_get_event_signature( $event_args )
+				, wordpoints_hooks_get_event_signature($event_args)
 				, $event_slug
 			)
 		);
 
-		if ( ! is_array( $hits ) ) {
+		if ( ! is_array($hits)) {
 			return array();
 		}
 

src/includes/classes/hook/hit/logger.php

Spacing +9 added lines, -9 removed lines

@@ -26,7 +26,7 @@ 
 	/**
 	 * @param WordPoints_Hook_Fire $fire The fire that might be logged as a hit.
 	 */
-	public function __construct( WordPoints_Hook_Fire $fire ) {
+	public function __construct(WordPoints_Hook_Fire $fire) {
 
 		$this->fire = $fire;
 	}
@@ -42,7 +42,7 @@ 
 
 		global $wpdb;
 
-		$signature = wordpoints_hooks_get_event_signature( $this->fire->event_args );
+		$signature = wordpoints_hooks_get_event_signature($this->fire->event_args);
 
 		$inserted = $wpdb->insert(
 			$wpdb->wordpoints_hook_hits
@@ -53,11 +53,11 @@ 
 				'reactor' => $this->fire->reaction->get_reactor_slug(),
 				'reaction_type' => $this->fire->reaction->get_storage_group_slug(),
 				'reaction_id' => $this->fire->reaction->ID,
-				'date' => current_time( 'mysql' ),
+				'date' => current_time('mysql'),
 			)
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}
 
@@ -65,13 +65,13 @@ 
 
 		$supersedes = $this->fire->get_superseded_hit();
 
-		if ( $supersedes ) {
+		if ($supersedes) {
 			$wpdb->update(
 				$wpdb->wordpoints_hook_hits
-				, array( 'superseded_by' => $hit_id )
-				, array( 'id' => $supersedes->id )
-				, array( '%d' )
-				, array( '%d' )
+				, array('superseded_by' => $hit_id)
+				, array('id' => $supersedes->id)
+				, array('%d')
+				, array('%d')
 			);
 		}
 

src/includes/classes/hook/reactor.php

Spacing +26 added lines, -26 removed lines

@@ -53,9 +53,9 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
-		if ( 'reactions' !== $var ) {
+		if ('reactions' !== $var) {
 			return null;
 		}
 
@@ -63,7 +63,7 @@ 
 			wordpoints_hooks()->get_current_mode()
 		);
 
-		if ( ! $reaction_group ) {
+		if ( ! $reaction_group) {
 			return null;
 		}
 
@@ -79,20 +79,20 @@ 
 	 *
 	 * @return WordPoints_Hook_Reaction_StorageI|false The reaction storage object.
 	 */
-	public function get_reaction_group( $slug ) {
+	public function get_reaction_group($slug) {
 
 		$reaction_group = wordpoints_hooks()->reaction_groups->get(
 			$this->slug
 			, $slug
-			, array( $this )
+			, array($this)
 		);
 
-		if ( ! $reaction_group instanceof WordPoints_Hook_Reaction_StorageI ) {
+		if ( ! $reaction_group instanceof WordPoints_Hook_Reaction_StorageI) {
 			return false;
 		}
 
 		// Allowing access to groups out-of-context would lead to strange behavior.
-		if ( false === $reaction_group->get_context_id() ) {
+		if (false === $reaction_group->get_context_id()) {
 			return false;
 		}
 
@@ -166,7 +166,7 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI[] All of the reaction objects.
 	 */
-	public function get_all_reactions_to_event( $event_slug ) {
+	public function get_all_reactions_to_event($event_slug) {
 
 		$reactions = array();
 
@@ -174,17 +174,17 @@ 
 			$this->slug
 		);
 
-		foreach ( $slugs as $slug ) {
+		foreach ($slugs as $slug) {
 
-			$storage = $this->get_reaction_group( $slug );
+			$storage = $this->get_reaction_group($slug);
 
-			if ( ! $storage ) {
+			if ( ! $storage) {
 				continue;
 			}
 
 			$reactions = array_merge(
 				$reactions
-				, $storage->get_reactions_to_event( $event_slug )
+				, $storage->get_reactions_to_event($event_slug)
 			);
 		}
 
@@ -213,15 +213,15 @@ 
 			$this->slug
 		);
 
-		foreach ( $slugs as $slug ) {
+		foreach ($slugs as $slug) {
 
-			$storage = $this->get_reaction_group( $slug );
+			$storage = $this->get_reaction_group($slug);
 
-			if ( ! $storage ) {
+			if ( ! $storage) {
 				continue;
 			}
 
-			$reactions = array_merge( $reactions, $storage->get_reactions() );
+			$reactions = array_merge($reactions, $storage->get_reactions());
 		}
 
 		return $reactions;
@@ -237,21 +237,21 @@ 
 	) {
 
 		if (
-			empty( $settings['target'] )
-			|| ! is_array( $settings['target'] )
+			empty($settings['target'])
+			|| ! is_array($settings['target'])
 		) {
 
-			$validator->add_error( __( 'Invalid target.', 'wordpoints' ), 'target' );
+			$validator->add_error(__('Invalid target.', 'wordpoints'), 'target');
 
 		} else {
 
-			$target = $event_args->get_from_hierarchy( $settings['target'] );
+			$target = $event_args->get_from_hierarchy($settings['target']);
 
 			if (
 				! $target instanceof WordPoints_Entity
-				|| ! in_array( $target->get_slug(), (array) $this->arg_types )
+				|| ! in_array($target->get_slug(), (array) $this->arg_types)
 			) {
-				$validator->add_error( __( 'Invalid target.', 'wordpoints' ), 'target' );
+				$validator->add_error(__('Invalid target.', 'wordpoints'), 'target');
 			}
 		}
 
@@ -261,8 +261,8 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_settings( WordPoints_Hook_ReactionI $reaction, array $settings ) {
-		$reaction->update_meta( 'target', $settings['target'] );
+	public function update_settings(WordPoints_Hook_ReactionI $reaction, array $settings) {
+		$reaction->update_meta('target', $settings['target']);
 	}
 
 	/**
@@ -272,7 +272,7 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	abstract public function hit( WordPoints_Hook_Fire $fire );
+	abstract public function hit(WordPoints_Hook_Fire $fire);
 
 	/**
 	 * Reverses all hits matching this event and args.
@@ -281,7 +281,7 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The reverse fire object.
 	 */
-	abstract public function reverse_hit( WordPoints_Hook_Fire $fire );
+	abstract public function reverse_hit(WordPoints_Hook_Fire $fire);
 }
 
 // EOF

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

Spacing +34 added lines, -34 removed lines

@@ -53,9 +53,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();
 	}
@@ -69,23 +69,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);
 	}
 
 	/**
@@ -96,49 +96,49 @@ 
 		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 1.0.0
 	 */
-	public function hit( WordPoints_Hook_Fire $fire ) {
+	public function hit(WordPoints_Hook_Fire $fire) {
 
 		$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();
 
-		foreach ( $fire->event_args->get_entities() as $entity ) {
-			$meta[ $entity->get_slug() ] = $entity->get_the_id();
+		foreach ($fire->event_args->get_entities() as $entity) {
+			$meta[$entity->get_slug()] = $entity->get_the_id();
 		}
 
 		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 1.0.0
 	 */
-	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
+	public function reverse_hit(WordPoints_Hook_Fire $fire) {
 
 		$meta_queries = array(
 			array(
@@ -149,7 +149,7 @@ 
 			),
 		);
 
-		foreach ( $fire->event_args->get_entities() as $slug => $entity ) {
+		foreach ($fire->event_args->get_entities() as $slug => $entity) {
 
 			$meta_queries[] = array(
 				'key'   => $slug,
@@ -166,35 +166,35 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
 		global $wpdb;
 
-		add_filter( 'wordpoints_points_log', '__return_false' );
+		add_filter('wordpoints_points_log', '__return_false');
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
 			wordpoints_alter_points(
 				$log->user_id
 				, -$log->points
 				, $log->points_type
 				, "reverse-{$log->log_type}"
-				, array( 'original_log_id' => $log->id )
+				, array('original_log_id' => $log->id)
 			);
 
-			wordpoints_points_log_delete_all_metadata( $log->id );
+			wordpoints_points_log_delete_all_metadata($log->id);
 
 			// Now delete the log.
 			$wpdb->delete(
 				$wpdb->wordpoints_points_logs
-				, array( 'id' => $log->id )
+				, array('id' => $log->id)
 				, '%d'
 			); // WPCS: cache OK, cleaned by wordpoints_alter_points().
 		}
 
-		remove_filter( 'wordpoints_points_log', '__return_false' );
+		remove_filter('wordpoints_points_log', '__return_false');
 	}
 }