WordPoints / hooks-api

src/includes/classes/hooks.php

Spacing +34 added lines, -34 removed lines

@@ -46,13 +46,13 @@ 
 	protected function init() {
 
 		$sub_apps = $this->sub_apps;
-		$sub_apps->register( 'router', 'WordPoints_Hook_Router' );
-		$sub_apps->register( 'actions', 'WordPoints_Hook_Actions' );
-		$sub_apps->register( 'events', 'WordPoints_Hook_Events' );
-		$sub_apps->register( 'reactors', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'reaction_stores', 'WordPoints_Class_Registry_Children' );
-		$sub_apps->register( 'extensions', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'conditions', 'WordPoints_Class_Registry_Children' );
+		$sub_apps->register('router', 'WordPoints_Hook_Router');
+		$sub_apps->register('actions', 'WordPoints_Hook_Actions');
+		$sub_apps->register('events', 'WordPoints_Hook_Events');
+		$sub_apps->register('reactors', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('reaction_stores', 'WordPoints_Class_Registry_Children');
+		$sub_apps->register('extensions', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('conditions', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -75,8 +75,8 @@ 
 	 */
 	public function get_current_mode() {
 
-		if ( ! isset( $this->current_mode ) ) {
-			$this->current_mode = ( wordpoints_is_network_context() ? 'network' : 'standard' );
+		if ( ! isset($this->current_mode)) {
+			$this->current_mode = (wordpoints_is_network_context() ? 'network' : 'standard');
 		}
 
 		return $this->current_mode;
@@ -94,7 +94,7 @@ 
 	 *
 	 * @param string $mode The slug of the mode to set as the current mode.
 	 */
-	public function set_current_mode( $mode ) {
+	public function set_current_mode($mode) {
 		$this->current_mode = $mode;
 	}
 
@@ -107,19 +107,19 @@ 
 	 *
 	 * @return WordPoints_Hook_Reaction_StoreI|false The reaction storage object.
 	 */
-	public function get_reaction_store( $slug ) {
+	public function get_reaction_store($slug) {
 
 		$reaction_store = $this->reaction_stores->get(
 			$this->get_current_mode()
 			, $slug
 		);
 
-		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI ) {
+		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI) {
 			return false;
 		}
 
 		// Allowing access to stores out-of-context would lead to strange behavior.
-		if ( false === $reaction_store->get_context_id() ) {
+		if (false === $reaction_store->get_context_id()) {
 			return false;
 		}
 
@@ -142,19 +142,19 @@ 
 		$action_type
 	) {
 
-		foreach ( $this->reaction_stores->get_all() as $reaction_stores ) {
-			foreach ( $reaction_stores as $reaction_store ) {
+		foreach ($this->reaction_stores->get_all() as $reaction_stores) {
+			foreach ($reaction_stores as $reaction_store) {
 
-				if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI ) {
+				if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI) {
 					continue;
 				}
 
 				// Allowing access to stores out-of-context would lead to strange behavior.
-				if ( false === $reaction_store->get_context_id() ) {
+				if (false === $reaction_store->get_context_id()) {
 					continue;
 				}
 
-				foreach ( $reaction_store->get_reactions_to_event( $event_slug ) as $reaction ) {
+				foreach ($reaction_store->get_reactions_to_event($event_slug) as $reaction) {
 
 					$fire = new WordPoints_Hook_Fire(
 						$event_args
@@ -162,7 +162,7 @@ 
 						, $action_type
 					);
 
-					$this->fire_reaction( $fire );
+					$this->fire_reaction($fire);
 				}
 			}
 		}
@@ -175,32 +175,32 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	protected function fire_reaction( $fire ) {
+	protected function fire_reaction($fire) {
 
 		/** @var WordPoints_Hook_Reactor $reactor */
-		$reactor = $this->reactors->get( $fire->reaction->get_reactor_slug() );
+		$reactor = $this->reactors->get($fire->reaction->get_reactor_slug());
 
-		if ( ! in_array( $fire->action_type, $reactor->get_action_types(), true ) ) {
+		if ( ! in_array($fire->action_type, $reactor->get_action_types(), true)) {
 			return;
 		}
 
-		$validator = new WordPoints_Hook_Reaction_Validator( $fire->reaction, true );
+		$validator = new WordPoints_Hook_Reaction_Validator($fire->reaction, true);
 		$validator->validate();
 
-		if ( $validator->had_errors() ) {
+		if ($validator->had_errors()) {
 			return;
 		}
 
-		unset( $validator );
+		unset($validator);
 
 		/** @var WordPoints_Hook_Extension[] $extensions */
 		$extensions = $this->extensions->get_all();
 
-		foreach ( $extensions as $extension ) {
-			if ( ! $extension->should_hit( $fire ) ) {
-				foreach ( $extensions as $ext ) {
-					if ( $ext instanceof WordPoints_Hook_Extension_Miss_ListenerI ) {
-						$ext->after_miss( $fire );
+		foreach ($extensions as $extension) {
+			if ( ! $extension->should_hit($fire)) {
+				foreach ($extensions as $ext) {
+					if ($ext instanceof WordPoints_Hook_Extension_Miss_ListenerI) {
+						$ext->after_miss($fire);
 					}
 				}
 
@@ -210,11 +210,11 @@ 
 
 		$fire->hit();
 
-		$reactor->hit( $fire );
+		$reactor->hit($fire);
 
-		foreach ( $extensions as $extension ) {
-			if ( $extension instanceof WordPoints_Hook_Extension_Hit_ListenerI ) {
-				$extension->after_hit( $fire );
+		foreach ($extensions as $extension) {
+			if ($extension instanceof WordPoints_Hook_Extension_Hit_ListenerI) {
+				$extension->after_hit($fire);
 			}
 		}
 	}

src/includes/classes/hook/extension.php

Spacing +18 added lines, -18 removed lines

@@ -65,14 +65,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
 			);
 
@@ -82,13 +82,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
 			);
 
@@ -103,12 +103,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);
 		}
 	}
 
@@ -121,7 +121,7 @@ 
 	 *
 	 * @return mixed The validated settings.
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return $settings;
 	}
 
@@ -138,16 +138,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;
 		}
@@ -162,7 +162,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);
 
 	/**
 	 * Get the data the scripts need for the UI.

src/includes/classes/hook/extension/repeat/blocker.php

Spacing +5 added lines, -5 removed lines

@@ -22,22 +22,22 @@
 	/**
 	 * @since 1.0.0
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return (bool) $settings;
 	}
 
 	/**
 	 * @since 1.0.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 ( $fire->get_matching_hits_query()->count() > 0 ) {
+		if ($fire->get_matching_hits_query()->count() > 0) {
 			return false;
 		}
 

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

Spacing +17 added lines, -17 removed lines

@@ -32,36 +32,36 @@ 
 	/**
 	 * @since 1.0.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 1.0.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 1.0.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/includes/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

src/includes/classes/hook/router.php

Spacing +47 added lines, -47 removed lines

@@ -80,13 +80,13 @@ 
 	/**
 	 * @since 1.0.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];
 		}
 
@@ -103,9 +103,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;
 		}
 
@@ -113,7 +113,7 @@ 
 		// 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();
 
@@ -122,38 +122,38 @@ 
 			$this->actions = $hooks->actions;
 		}
 
-		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->events->args->get_children( $event_slug, array( $action_object ) );
+					$event_args = $this->events->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);
 				}
 			}
 		}
@@ -182,56 +182,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);
 	}
 
 	/**
@@ -241,20 +241,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);
 				}
 			}
 		}
@@ -269,8 +269,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;
 	}
 
 	/**
@@ -282,8 +282,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]);
 	}
 
 	/**
@@ -295,7 +295,7 @@ 
 	 */
 	public function get_event_index() {
 
-		if ( empty( $this->event_index ) ) {
+		if (empty($this->event_index)) {
 			wordpoints_hooks()->events;
 		}
 

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

Spacing +45 added lines, -45 removed lines

@@ -27,7 +27,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	protected $action_types = array( 'fire', 'toggle_on', 'toggle_off' );
+	protected $action_types = array('fire', 'toggle_on', 'toggle_off');
 
 	/**
 	 * @since 1.0.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 per:', 'wordpoints' );
+		$data['target_label'] = __('Award To', 'wordpoints');
+		$data['periods_label'] = __('Award each user no more than once per:', '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,58 +114,58 @@ 
 		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) {
 
-		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();
+		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) {
 
-		$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;
 		}
 
@@ -183,11 +183,11 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
-		$this->reverse_logs( $logs, $fire );
+		$this->reverse_logs($logs, $fire);
 	}
 
 	/**
@@ -199,10 +199,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();
 		}
 
@@ -217,28 +217,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()->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
@@ -249,11 +249,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/includes/classes/hook/reactor/points/legacy.php

Spacing +7 added lines, -7 removed lines

@@ -27,14 +27,14 @@ 
 		array $settings
 	) {
 
-		if ( isset( $settings['legacy_log_type'] ) ) {
+		if (isset($settings['legacy_log_type'])) {
 			$reaction->update_meta(
 				'legacy_log_type',
 				$settings['legacy_log_type']
 			);
 		}
 
-		parent::update_settings( $reaction, $settings );
+		parent::update_settings($reaction, $settings);
 	}
 
 	/**
@@ -52,9 +52,9 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
+	public function reverse_hit(WordPoints_Hook_Fire $fire) {
 
-		if ( isset( $fire->data['reversals_legacy_points']['points_logs'] ) ) {
+		if (isset($fire->data['reversals_legacy_points']['points_logs'])) {
 
 			$this->reverse_logs(
 				$fire->data['reversals_legacy_points']['points_logs']
@@ -62,17 +62,17 @@ 
 			);
 
 		} else {
-			parent::reverse_hit( $fire );
+			parent::reverse_hit($fire);
 		}
 	}
 
 	/**
 	 * @since 1.0.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['reversals_legacy_points']['hit_ids'] ) ) {
+		if ( ! isset($fire->data['reversals_legacy_points']['hit_ids'])) {
 			return array();
 		}
 

src/includes/classes/hook/event/media/upload.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	public function get_title() {
 
-		return __( 'Upload Media', 'wordpoints' );
+		return __('Upload Media', 'wordpoints');
 	}
 
 	/**
@@ -29,14 +29,14 @@ 
 	 */
 	public function get_description() {
 
-		return __( 'When a file is uploaded to the Media Library.', 'wordpoints' );
+		return __('When a file is uploaded to the Media Library.', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_reversal_text() {
-		return __( 'Media file deleted.', 'wordpoints' );
+		return __('Media file deleted.', 'wordpoints');
 	}
 }