WordPoints / wordpoints

src/classes/data/type/text.php

Spacing +3 added lines, -3 removed lines

@@ -20,12 +20,12 @@
 	/**
 	 * @since 2.1.0
 	 */
-	public function validate_value( $value ) {
+	public function validate_value($value) {
 
-		if ( ! is_string( $value ) ) {
+		if ( ! is_string($value)) {
 			return new WP_Error(
 				'not_string'
-				, __( '%s must be a text value.', 'wordpoints' )
+				, __('%s must be a text value.', 'wordpoints')
 			);
 		}
 

src/classes/hook/reactori.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	public function hit( WordPoints_Hook_Fire $fire );
+	public function hit(WordPoints_Hook_Fire $fire);
 }
 
 // EOF

src/classes/hook/router.php

Spacing +51 added lines, -51 removed lines

@@ -89,13 +89,13 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function __call( $name, $args ) {
+	public function __call($name, $args) {
 
-		$this->route_action( $name, $args );
+		$this->route_action($name, $args);
 
 		// Return the first value, in case it is hooked to a filter.
 		$return = null;
-		if ( isset( $args[0] ) ) {
+		if (isset($args[0])) {
 			$return = $args[0];
 		}
 
@@ -112,9 +112,9 @@ 
 	 *                     action name and the priority.
 	 * @param array  $args The args the action was fired with.
 	 */
-	protected function route_action( $name, $args ) {
+	protected function route_action($name, $args) {
 
-		if ( ! isset( $this->action_index[ $name ] ) ) {
+		if ( ! isset($this->action_index[$name])) {
 			return;
 		}
 
@@ -122,48 +122,48 @@ 
 		// registry attempts to access the router in its own constructor. The result
 		// of attempting to do this before the router itself has been fully
 		// constructed is that the events registry gets null instead of the router.
-		if ( ! isset( $this->hooks ) ) {
+		if ( ! isset($this->hooks)) {
 
 			$hooks = wordpoints_hooks();
 
 			$this->hooks   = $hooks;
-			$this->events  = $hooks->get_sub_app( 'events' );
-			$this->actions = $hooks->get_sub_app( 'actions' );
-			$this->event_args = $this->events->get_sub_app( 'args' );
+			$this->events  = $hooks->get_sub_app('events');
+			$this->actions = $hooks->get_sub_app('actions');
+			$this->event_args = $this->events->get_sub_app('args');
 		}
 
-		foreach ( $this->action_index[ $name ]['actions'] as $slug => $data ) {
+		foreach ($this->action_index[$name]['actions'] as $slug => $data) {
 
-			if ( ! isset( $this->event_index[ $slug ] ) ) {
+			if ( ! isset($this->event_index[$slug])) {
 				continue;
 			}
 
-			$action_object = $this->actions->get( $slug, $args, $data );
+			$action_object = $this->actions->get($slug, $args, $data);
 
-			if ( ! ( $action_object instanceof WordPoints_Hook_ActionI ) ) {
+			if ( ! ($action_object instanceof WordPoints_Hook_ActionI)) {
 				continue;
 			}
 
-			if ( ! $action_object->should_fire() ) {
+			if ( ! $action_object->should_fire()) {
 				continue;
 			}
 
-			foreach ( $this->event_index[ $slug ] as $type => $events ) {
-				foreach ( $events as $event_slug => $unused ) {
+			foreach ($this->event_index[$slug] as $type => $events) {
+				foreach ($events as $event_slug => $unused) {
 
-					if ( ! $this->events->is_registered( $event_slug ) ) {
+					if ( ! $this->events->is_registered($event_slug)) {
 						continue;
 					}
 
-					$event_args = $this->event_args->get_children( $event_slug, array( $action_object ) );
+					$event_args = $this->event_args->get_children($event_slug, array($action_object));
 
-					if ( empty( $event_args ) ) {
+					if (empty($event_args)) {
 						continue;
 					}
 
-					$event_args = new WordPoints_Hook_Event_Args( $event_args );
+					$event_args = new WordPoints_Hook_Event_Args($event_args);
 
-					$this->hooks->fire( $event_slug, $event_args, $type );
+					$this->hooks->fire($event_slug, $event_args, $type);
 				}
 			}
 		}
@@ -192,56 +192,56 @@ 
 	 *        }
 	 * }
 	 */
-	public function add_action( $slug, array $args ) {
+	public function add_action($slug, array $args) {
 
 		$priority = 10;
-		if ( isset( $args['priority'] ) ) {
+		if (isset($args['priority'])) {
 			$priority = $args['priority'];
 		}
 
-		if ( ! isset( $args['action'] ) ) {
+		if ( ! isset($args['action'])) {
 			return;
 		}
 
 		$method = "{$args['action']},{$priority}";
 
-		$this->action_index[ $method ]['actions'][ $slug ] = array();
+		$this->action_index[$method]['actions'][$slug] = array();
 
 		$arg_number = 1;
 
-		if ( isset( $args['data'] ) ) {
+		if (isset($args['data'])) {
 
-			if ( isset( $args['data']['arg_index'] ) ) {
-				$arg_number = 1 + max( $args['data']['arg_index'] );
+			if (isset($args['data']['arg_index'])) {
+				$arg_number = 1 + max($args['data']['arg_index']);
 			}
 
-			if ( isset( $args['data']['requirements'] ) ) {
-				$requirements = 1 + max( array_keys( $args['data']['requirements'] ) );
+			if (isset($args['data']['requirements'])) {
+				$requirements = 1 + max(array_keys($args['data']['requirements']));
 
-				if ( $requirements > $arg_number ) {
+				if ($requirements > $arg_number) {
 					$arg_number = $requirements;
 				}
 			}
 
-			$this->action_index[ $method ]['actions'][ $slug ] = $args['data'];
+			$this->action_index[$method]['actions'][$slug] = $args['data'];
 		}
 
-		if ( isset( $args['arg_number'] ) ) {
+		if (isset($args['arg_number'])) {
 			$arg_number = $args['arg_number'];
 		}
 
 		// If this action is already being routed, and will have enough args, we
 		// don't need to hook to it again.
 		if (
-			isset( $this->action_index[ $method ]['arg_number'] )
-			&& $this->action_index[ $method ]['arg_number'] >= $arg_number
+			isset($this->action_index[$method]['arg_number'])
+			&& $this->action_index[$method]['arg_number'] >= $arg_number
 		) {
 			return;
 		}
 
-		$this->action_index[ $method ]['arg_number'] = $arg_number;
+		$this->action_index[$method]['arg_number'] = $arg_number;
 
-		add_action( $args['action'], array( $this, $method ), $priority, $arg_number );
+		add_action($args['action'], array($this, $method), $priority, $arg_number);
 	}
 
 	/**
@@ -251,20 +251,20 @@ 
 	 *
 	 * @param string $slug The action slug.
 	 */
-	public function remove_action( $slug ) {
+	public function remove_action($slug) {
 
-		foreach ( $this->action_index as $method => $data ) {
-			if ( isset( $data['actions'][ $slug ] ) ) {
+		foreach ($this->action_index as $method => $data) {
+			if (isset($data['actions'][$slug])) {
 
-				unset( $this->action_index[ $method ]['actions'][ $slug ] );
+				unset($this->action_index[$method]['actions'][$slug]);
 
-				if ( empty( $this->action_index[ $method ]['actions'] ) ) {
+				if (empty($this->action_index[$method]['actions'])) {
 
-					unset( $this->action_index[ $method ] );
+					unset($this->action_index[$method]);
 
-					list( $action, $priority ) = explode( ',', $method );
+					list($action, $priority) = explode(',', $method);
 
-					remove_action( $action, array( $this, $method ), $priority );
+					remove_action($action, array($this, $method), $priority);
 				}
 			}
 		}
@@ -279,8 +279,8 @@ 
 	 * @param string $action_slug The slug of the action.
 	 * @param string $action_type The type of action. Default is 'fire'.
 	 */
-	public function add_event_to_action( $event_slug, $action_slug, $action_type = 'fire' ) {
-		$this->event_index[ $action_slug ][ $action_type ][ $event_slug ] = true;
+	public function add_event_to_action($event_slug, $action_slug, $action_type = 'fire') {
+		$this->event_index[$action_slug][$action_type][$event_slug] = true;
 	}
 
 	/**
@@ -292,8 +292,8 @@ 
 	 * @param string $action_slug The slug of the action.
 	 * @param string $action_type The type of action. Default is 'fire'.
 	 */
-	public function remove_event_from_action( $event_slug, $action_slug, $action_type = 'fire' ) {
-		unset( $this->event_index[ $action_slug ][ $action_type ][ $event_slug ] );
+	public function remove_event_from_action($event_slug, $action_slug, $action_type = 'fire') {
+		unset($this->event_index[$action_slug][$action_type][$event_slug]);
 	}
 
 	/**
@@ -305,8 +305,8 @@ 
 	 */
 	public function get_event_index() {
 
-		if ( empty( $this->event_index ) ) {
-			wordpoints_hooks()->get_sub_app( 'events' );
+		if (empty($this->event_index)) {
+			wordpoints_hooks()->get_sub_app('events');
 		}
 
 		return $this->event_index;

src/classes/hook/extension.php

Spacing +17 added lines, -17 removed lines

@@ -57,14 +57,14 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings[ $this->slug ] ) ) {
+		if ( ! isset($settings[$this->slug])) {
 			return $settings;
 		}
 
-		if ( ! is_array( $settings[ $this->slug ] ) ) {
+		if ( ! is_array($settings[$this->slug])) {
 
 			$validator->add_error(
-				__( 'Invalid settings format.', 'wordpoints' )
+				__('Invalid settings format.', 'wordpoints')
 				, $this->slug
 			);
 
@@ -74,13 +74,13 @@ 
 		$this->validator = $validator;
 		$this->event_args = $event_args;
 
-		$this->validator->push_field( $this->slug );
+		$this->validator->push_field($this->slug);
 
-		foreach ( $settings[ $this->slug ] as $action_type => $action_type_settings ) {
+		foreach ($settings[$this->slug] as $action_type => $action_type_settings) {
 
-			$this->validator->push_field( $action_type );
+			$this->validator->push_field($action_type);
 
-			$settings[ $this->slug ][ $action_type ] = $this->validate_action_type_settings(
+			$settings[$this->slug][$action_type] = $this->validate_action_type_settings(
 				$action_type_settings
 			);
 
@@ -95,12 +95,12 @@ 
 	/**
 	 * @since 2.1.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);
 		}
 	}
 
@@ -113,7 +113,7 @@ 
 	 *
 	 * @return mixed The validated settings.
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return $settings;
 	}
 
@@ -130,16 +130,16 @@ 
 	 *
 	 * @return mixed The settings for the extension, or false if none.
 	 */
-	protected function get_settings_from_fire( WordPoints_Hook_Fire $fire ) {
+	protected function get_settings_from_fire(WordPoints_Hook_Fire $fire) {
 
-		$settings = $fire->reaction->get_meta( $this->slug );
+		$settings = $fire->reaction->get_meta($this->slug);
 
-		if ( ! is_array( $settings ) ) {
+		if ( ! is_array($settings)) {
 			return $settings;
 		}
 
-		if ( isset( $settings[ $fire->action_type ] ) ) {
-			return $settings[ $fire->action_type ];
+		if (isset($settings[$fire->action_type])) {
+			return $settings[$fire->action_type];
 		} else {
 			return false;
 		}

src/classes/hook/extension/reversals.php

Spacing +17 added lines, -17 removed lines

@@ -32,36 +32,36 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		if ( ! $this->get_settings_from_fire( $fire ) ) {
+		if ( ! $this->get_settings_from_fire($fire)) {
 			return true;
 		}
 
-		$ids = $this->get_hits_to_be_reversed( $fire );
+		$ids = $this->get_hits_to_be_reversed($fire);
 
-		return count( $ids ) > 0;
+		return count($ids) > 0;
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function after_hit( WordPoints_Hook_Fire $fire ) {
+	public function after_hit(WordPoints_Hook_Fire $fire) {
 
-		if ( ! $this->get_settings_from_fire( $fire ) ) {
+		if ( ! $this->get_settings_from_fire($fire)) {
 			return;
 		}
 
-		foreach ( $this->get_hits_to_be_reversed( $fire ) as $id ) {
-			add_metadata( 'wordpoints_hook_hit', $id, 'reverse_fired', true );
+		foreach ($this->get_hits_to_be_reversed($fire) as $id) {
+			add_metadata('wordpoints_hook_hit', $id, 'reverse_fired', true);
 		}
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function after_miss( WordPoints_Hook_Fire $fire ) {
-		$this->after_hit( $fire );
+	public function after_miss(WordPoints_Hook_Fire $fire) {
+		$this->after_hit($fire);
 	}
 
 	/**
@@ -73,12 +73,12 @@ 
 	 *
 	 * @return array The IDs the hits to be reversed.
 	 */
-	protected function get_hits_to_be_reversed( WordPoints_Hook_Fire $fire ) {
+	protected function get_hits_to_be_reversed(WordPoints_Hook_Fire $fire) {
 
 		// We cache these so that we don't run the query both before and after the
 		// fire.
-		if ( isset( $fire->data[ $this->slug ]['hit_ids'] ) ) {
-			return $fire->data[ $this->slug ]['hit_ids'];
+		if (isset($fire->data[$this->slug]['hit_ids'])) {
+			return $fire->data[$this->slug]['hit_ids'];
 		}
 
 		$query = $fire->get_matching_hits_query();
@@ -86,19 +86,19 @@ 
 		$query->set_args(
 			array(
 				'fields'       => 'id',
-				'action_type'  => $this->get_settings_from_fire( $fire ),
+				'action_type'  => $this->get_settings_from_fire($fire),
 				'meta_key'     => 'reverse_fired',
 				'meta_compare' => 'NOT EXISTS',
 			)
 		);
 
-		$ids = $query->get( 'col' );
+		$ids = $query->get('col');
 
-		if ( ! $ids ) {
+		if ( ! $ids) {
 			$ids = array();
 		}
 
-		$fire->data[ $this->slug ]['hit_ids'] = $ids;
+		$fire->data[$this->slug]['hit_ids'] = $ids;
 
 		return $ids;
 	}

src/classes/hook/extension/blocker.php

Spacing +3 added lines, -3 removed lines

@@ -24,15 +24,15 @@
 	/**
 	 * @since 2.1.0
 	 */
-	public function validate_action_type_settings( $settings ) {
+	public function validate_action_type_settings($settings) {
 		return (bool) $settings;
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
-		return ! $this->get_settings_from_fire( $fire );
+	public function should_hit(WordPoints_Hook_Fire $fire) {
+		return ! $this->get_settings_from_fire($fire);
 	}
 }
 

src/classes/hook/extension/periods.php

Spacing +79 added lines, -79 removed lines

@@ -39,12 +39,12 @@ 
 	public function get_ui_script_data() {
 
 		$period_units = array(
-			1                   => __( 'Seconds', 'wordpoints' ),
-			MINUTE_IN_SECONDS   => __( 'Minutes', 'wordpoints' ),
-			HOUR_IN_SECONDS     => __( 'Hours',   'wordpoints' ),
-			DAY_IN_SECONDS      => __( 'Days',    'wordpoints' ),
-			WEEK_IN_SECONDS     => __( 'Weeks',   'wordpoints' ),
-			30 * DAY_IN_SECONDS => __( 'Months',  'wordpoints' ),
+			1                   => __('Seconds', 'wordpoints'),
+			MINUTE_IN_SECONDS   => __('Minutes', 'wordpoints'),
+			HOUR_IN_SECONDS     => __('Hours', 'wordpoints'),
+			DAY_IN_SECONDS      => __('Days', 'wordpoints'),
+			WEEK_IN_SECONDS     => __('Weeks', 'wordpoints'),
+			30 * DAY_IN_SECONDS => __('Months', 'wordpoints'),
 		);
 
 		/**
@@ -62,7 +62,7 @@ 
 		return array(
 			'period_units' => $period_units,
 			'l10n' => array(
-				'label' => __( 'Trigger reaction no more than once in:', 'wordpoints' ),
+				'label' => __('Trigger reaction no more than once in:', 'wordpoints'),
 			),
 		);
 	}
@@ -76,25 +76,25 @@ 
 	 *
 	 * @return array The validated periods.
 	 */
-	protected function validate_action_type_settings( $periods ) {
+	protected function validate_action_type_settings($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();
@@ -112,30 +112,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'
 			);
 
@@ -152,34 +152,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 arg hierarchy for period.', 'wordpoints' )
+					__('Invalid arg hierarchy for period.', 'wordpoints')
 				);
 			}
 
@@ -192,19 +192,19 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		$periods = $this->get_settings_from_fire( $fire );
+		$periods = $this->get_settings_from_fire($fire);
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return true;
 		}
 
 		$this->event_args = $fire->event_args;
 		$this->action_type = $fire->action_type;
 
-		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;
 			}
 		}
@@ -227,27 +227,27 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 
-		$period = $this->get_period_by_reaction( $settings, $reaction );
+		$period = $this->get_period_by_reaction($settings, $reaction);
 
 		// If the period isn't found, we know that we can still fire.
-		if ( ! $period ) {
+		if ( ! $period) {
 			return true;
 		}
 
-		$now = current_time( 'timestamp', true );
-		$hit_time = strtotime( $period->date, $now );
+		$now = current_time('timestamp', true);
+		$hit_time = strtotime($period->date, $now);
 
-		if ( ! empty( $settings['relative'] ) ) {
+		if ( ! empty($settings['relative'])) {
 
-			return ( $now > $hit_time + $settings['length'] );
+			return ($now > $hit_time + $settings['length']);
 
 		} else {
 
-			$offset = get_option( 'gmt_offset' ) * HOUR_IN_SECONDS;
+			$offset = get_option('gmt_offset') * HOUR_IN_SECONDS;
 
 			return (
-				(int) ( ( $hit_time + $offset ) / $settings['length'] )
-				< (int) ( ( $now + $offset ) / $settings['length'] )
+				(int) (($hit_time + $offset) / $settings['length'])
+				< (int) (($now + $offset) / $settings['length'])
 			);
 		}
 	}
@@ -261,24 +261,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;
 	}
@@ -292,11 +292,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_periods' );
+		$period = wp_cache_get($period_id, 'wordpoints_hook_periods');
 
-		if ( ! $period ) {
+		if ( ! $period) {
 
 			global $wpdb;
 
@@ -313,11 +313,11 @@ 
 				)
 			);
 
-			if ( ! $period ) {
+			if ( ! $period) {
 				return false;
 			}
 
-			wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+			wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 		}
 
 		return $period;
@@ -338,17 +338,17 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 
-		$signature = $this->get_period_signature( $settings, $reaction );
+		$signature = $this->get_period_signature($settings, $reaction);
 		$reaction_guid = $reaction->get_guid();
 
-		$cache_key = wp_json_encode( $reaction_guid ) . "-{$signature}-{$this->action_type}";
+		$cache_key = wp_json_encode($reaction_guid) . "-{$signature}-{$this->action_type}";
 
 		// 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_by_reaction' );
+		$period_id = wp_cache_get($cache_key, 'wordpoints_hook_period_ids_by_reaction');
 
 		// 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;
@@ -373,18 +373,18 @@ 
 				, $signature
 				, $reaction_guid['mode']
 				, $reaction_guid['store']
-				, wp_json_encode( $reaction_guid['context_id'] )
+				, wp_json_encode($reaction_guid['context_id'])
 				, $reaction_guid['id']
 				, $this->action_type
 			)
 		);
 
-		if ( ! $period ) {
+		if ( ! $period) {
 			return false;
 		}
 
-		wp_cache_set( $cache_key, $period->id, 'wordpoints_hook_period_ids_by_reaction' );
-		wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+		wp_cache_set($cache_key, $period->id, 'wordpoints_hook_period_ids_by_reaction');
+		wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 
 		return $period;
 	}
@@ -392,21 +392,21 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function after_hit( WordPoints_Hook_Fire $fire ) {
+	public function after_hit(WordPoints_Hook_Fire $fire) {
 
-		$periods = $this->get_settings_from_fire( $fire );
+		$periods = $this->get_settings_from_fire($fire);
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return;
 		}
 
 		$this->event_args = $fire->event_args;
 		$this->action_type = $fire->action_type;
 
-		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
 			);
 		}
@@ -431,14 +431,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))
 		);
 	}
 
@@ -452,7 +452,7 @@ 
 	 *
 	 * @return false|object The period data, or false if not found.
 	 */
-	protected function add_period( $signature, WordPoints_Hook_Fire $fire ) {
+	protected function add_period($signature, WordPoints_Hook_Fire $fire) {
 
 		global $wpdb;
 
@@ -462,17 +462,17 @@ 
 				'hit_id' => $fire->hit_id,
 				'signature' => $signature,
 			)
-			, array( '%d', '%s' )
+			, array('%d', '%s')
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}
 
 		$period_id = $wpdb->insert_id;
 
 		wp_cache_set(
-			wp_json_encode( $fire->reaction->get_guid() ) . "-{$signature}-{$this->action_type}"
+			wp_json_encode($fire->reaction->get_guid()) . "-{$signature}-{$this->action_type}"
 			, $period_id
 			, 'wordpoints_hook_period_ids_by_reaction'
 		);

src/classes/hook/extension/hit/listeneri.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,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);
 }
 
 // EOF

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 	 * @since 2.2.0
 	 *
 	 * @param object $log The object for the points log the restriction is for.
+	 * @return void
 	 */
 	public function __construct( $log );
 

src/classes/hook/extension/miss/listeneri.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 *
 	 * @param WordPoints_Hook_Fire $fire The fire that was a miss.
 	 */
-	public function after_miss( WordPoints_Hook_Fire $fire );
+	public function after_miss(WordPoints_Hook_Fire $fire);
 }
 
 // EOF

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 	 * @since 2.2.0
 	 *
 	 * @param object $log The object for the points log the restriction is for.
+	 * @return void
 	 */
 	public function __construct( $log );