WordPoints / hooks-api

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

Spacing +7 added lines, -7 removed lines

@@ -17,21 +17,21 @@
 	/**
 	 * @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 ( $hooks->reactors->get_all() as $reactor ) {
+		foreach ($hooks->reactors->get_all() as $reactor) {
 
-			if ( ! ( $reactor instanceof WordPoints_Hook_Reactor_ReverseI ) ) {
+			if ( ! ($reactor instanceof WordPoints_Hook_Reactor_ReverseI)) {
 				continue;
 			}
 
-			$reactor->reverse_hits( $event_slug, $event_args );
+			$reactor->reverse_hits($event_slug, $event_args);
 
-			foreach ( $hooks->extensions->get_all() as $extension ) {
-				if ( $extension instanceof WordPoints_Hook_Extension_ReverseI ) {
-					$extension->after_reverse( $event_slug, $event_args, $reactor );
+			foreach ($hooks->extensions->get_all() as $extension) {
+				if ($extension instanceof WordPoints_Hook_Extension_ReverseI) {
+					$extension->after_reverse($event_slug, $event_args, $reactor);
 				}
 			}
 		}

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

Spacing +7 added lines, -7 removed lines

@@ -17,21 +17,21 @@
 	/**
 	 * @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 ( $hooks->reactors->get_all() as $reactor ) {
+		foreach ($hooks->reactors->get_all() as $reactor) {
 
-			if ( ! ( $reactor instanceof WordPoints_Hook_Reactor_SpamI ) ) {
+			if ( ! ($reactor instanceof WordPoints_Hook_Reactor_SpamI)) {
 				continue;
 			}
 
-			$reactor->spam_hits( $event_slug, $event_args );
+			$reactor->spam_hits($event_slug, $event_args);
 
-			foreach ( $hooks->extensions->get_all() as $extension ) {
-				if ( $extension instanceof WordPoints_Hook_Extension_SpamI ) {
-					$extension->after_spam( $event_slug, $event_args, $reactor );
+			foreach ($hooks->extensions->get_all() as $extension) {
+				if ($extension instanceof WordPoints_Hook_Extension_SpamI) {
+					$extension->after_spam($event_slug, $event_args, $reactor);
 				}
 			}
 		}

src/includes/classes/hook/fireri.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	 * @param string                     $event_slug The event slug.
 	 * @param WordPoints_Hook_Event_Args $event_args The event args.
 	 */
-	public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args );
+	public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args);
 }
 
 // EOF

src/includes/classes/hook/reaction.php

Spacing +4 added lines, -4 removed lines

@@ -53,9 +53,9 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $id, WordPoints_Hook_Reaction_StorageI $storage ) {
+	public function __construct($id, WordPoints_Hook_Reaction_StorageI $storage) {
 
-		$this->ID           = wordpoints_int( $id );
+		$this->ID           = wordpoints_int($id);
 		$this->storage      = $storage;
 		$this->reactor_slug = $this->storage->get_reactor_slug();
 		$this->network_wide = $this->storage->is_network_wide();
@@ -64,9 +64,9 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
-		if ( 'ID' === $var ) {
+		if ('ID' === $var) {
 			return $this->ID;
 		}
 

src/includes/classes/hook/reaction/options.php

Spacing +20 added lines, -20 removed lines

@@ -25,13 +25,13 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_event_slug() {
-		return $this->storage->get_reaction_event_from_index( $this->ID );
+		return $this->storage->get_reaction_event_from_index($this->ID);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_event_slug( $event_slug ) {
+	public function update_event_slug($event_slug) {
 		return $this->storage->update_reaction_event_in_index(
 			$this->ID
 			, $event_slug
@@ -41,63 +41,63 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_meta( $key ) {
+	public function get_meta($key) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) || ! isset( $settings[ $key ] ) ) {
+		if ( ! is_array($settings) || ! isset($settings[$key])) {
 			return false;
 		}
 
-		return $settings[ $key ];
+		return $settings[$key];
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function add_meta( $key, $value ) {
+	public function add_meta($key, $value) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) || isset( $settings[ $key ] ) ) {
+		if ( ! is_array($settings) || isset($settings[$key])) {
 			return false;
 		}
 
-		$settings[ $key ] = $value;
+		$settings[$key] = $value;
 
-		return $this->update_settings( $settings );
+		return $this->update_settings($settings);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_meta( $key, $value ) {
+	public function update_meta($key, $value) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) ) {
+		if ( ! is_array($settings)) {
 			return false;
 		}
 
-		$settings[ $key ] = $value;
+		$settings[$key] = $value;
 
-		return $this->update_settings( $settings );
+		return $this->update_settings($settings);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function delete_meta( $key ) {
+	public function delete_meta($key) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) || ! isset( $settings[ $key ] ) ) {
+		if ( ! is_array($settings) || ! isset($settings[$key])) {
 			return false;
 		}
 
-		unset( $settings[ $key ] );
+		unset($settings[$key]);
 
-		return $this->update_settings( $settings );
+		return $this->update_settings($settings);
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 	protected function get_settings() {
 
 		return $this->storage->get_option(
-			$this->storage->get_settings_option_name( $this->ID )
+			$this->storage->get_settings_option_name($this->ID)
 		);
 	}
 
@@ -130,10 +130,10 @@ 
 	 *
 	 * @return bool Whether the settings were updated successfully.
 	 */
-	protected function update_settings( $settings ) {
+	protected function update_settings($settings) {
 
 		return $this->storage->update_option(
-			$this->storage->get_settings_option_name( $this->ID )
+			$this->storage->get_settings_option_name($this->ID)
 			, $settings
 		);
 	}

src/includes/classes/hook/reaction/storage/options.php

Spacing +43 added lines, -43 removed lines

@@ -22,25 +22,25 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function reaction_exists( $id ) {
-		return (bool) $this->get_option( $this->get_settings_option_name( $id ) );
+	public function reaction_exists($id) {
+		return (bool) $this->get_option($this->get_settings_option_name($id));
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_reactions() {
-		return $this->create_reaction_objects( $this->get_reaction_index() );
+		return $this->create_reaction_objects($this->get_reaction_index());
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_reactions_to_event( $event_slug ) {
+	public function get_reactions_to_event($event_slug) {
 
 		$index = $this->get_reaction_index();
-		$index = wp_list_filter( $index, array( 'event' => $event_slug ) );
-		return $this->create_reaction_objects( $index );
+		$index = wp_list_filter($index, array('event' => $event_slug));
+		return $this->create_reaction_objects($index);
 	}
 
 	/**
@@ -59,9 +59,9 @@ 
 	 */
 	protected function get_reaction_index() {
 
-		$index = $this->get_option( $this->get_reaction_index_option_name() );
+		$index = $this->get_option($this->get_reaction_index_option_name());
 
-		if ( ! is_array( $index ) ) {
+		if ( ! is_array($index)) {
 			$index = array();
 		}
 
@@ -77,7 +77,7 @@ 
 	 *
 	 * @return bool Whether the index was updated successfully.
 	 */
-	protected function update_reaction_index( $index ) {
+	protected function update_reaction_index($index) {
 
 		return $this->update_option(
 			$this->get_reaction_index_option_name()
@@ -96,15 +96,15 @@ 
 	 *
 	 * @return string|false The event slug, or false if not found.
 	 */
-	public function get_reaction_event_from_index( $id ) {
+	public function get_reaction_event_from_index($id) {
 
 		$index = $this->get_reaction_index();
 
-		if ( ! isset( $index[ $id ]['event'] ) ) {
+		if ( ! isset($index[$id]['event'])) {
 			return false;
 		}
 
-		return $index[ $id ]['event'];
+		return $index[$id]['event'];
 	}
 
 	/**
@@ -119,17 +119,17 @@ 
 	 *
 	 * @return bool Whether the event was updated successfully.
 	 */
-	public function update_reaction_event_in_index( $id, $event ) {
+	public function update_reaction_event_in_index($id, $event) {
 
 		$index = $this->get_reaction_index();
 
-		if ( ! isset( $index[ $id ] ) ) {
+		if ( ! isset($index[$id])) {
 			return false;
 		}
 
-		$index[ $id ]['event'] = $event;
+		$index[$id]['event'] = $event;
 
-		return $this->update_reaction_index( $index );
+		return $this->update_reaction_index($index);
 	}
 
 	/**
@@ -141,15 +141,15 @@ 
 	 *
 	 * @return WordPoints_Hook_Reaction_Options[] The objects for the reactions.
 	 */
-	protected function create_reaction_objects( $index ) {
+	protected function create_reaction_objects($index) {
 
 		$reactions = array();
 
-		foreach ( $index as $reaction ) {
+		foreach ($index as $reaction) {
 
-			$object = $this->get_reaction( $reaction['id'] );
+			$object = $this->get_reaction($reaction['id']);
 
-			if ( ! $object ) {
+			if ( ! $object) {
 				continue;
 			}
 
@@ -162,53 +162,53 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function delete_reaction( $id ) {
+	public function delete_reaction($id) {
 
-		if ( ! $this->reaction_exists( $id ) ) {
+		if ( ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		$result = $this->delete_option( $this->get_settings_option_name( $id ) );
+		$result = $this->delete_option($this->get_settings_option_name($id));
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			return false;
 		}
 
 		$index = $this->get_reaction_index();
 
-		unset( $index[ $id ] );
+		unset($index[$id]);
 
-		return $this->update_reaction_index( $index );
+		return $this->update_reaction_index($index);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function _create_reaction( $event_slug ) {
+	protected function _create_reaction($event_slug) {
 
 		$index = $this->get_reaction_index();
 
 		$id = 1;
 
 		// TODO this is fragile when the newest reaction gets deleted.
-		if ( ! empty( $index ) ) {
-			$id = 1 + max( array_keys( $index ) );
+		if ( ! empty($index)) {
+			$id = 1 + max(array_keys($index));
 		}
 
-		$settings = array( 'event' => $event_slug );
+		$settings = array('event' => $event_slug);
 
 		$result = $this->add_option(
-			$this->get_settings_option_name( $id )
+			$this->get_settings_option_name($id)
 			, $settings
 		);
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			return false;
 		}
 
-		$index[ $id ] = array( 'event' => $event_slug, 'id' => $id );
+		$index[$id] = array('event' => $event_slug, 'id' => $id);
 
-		if ( ! $this->update_reaction_index( $index ) ) {
+		if ( ! $this->update_reaction_index($index)) {
 			return false;
 		}
 
@@ -226,8 +226,8 @@ 
 	 *
 	 * @return mixed The option value, or false.
 	 */
-	public function get_option( $name ) {
-		return get_option( $name );
+	public function get_option($name) {
+		return get_option($name);
 	}
 
 	/**
@@ -240,8 +240,8 @@ 
 	 *
 	 * @return bool Whether the option was added successfully.
 	 */
-	protected function add_option( $name, $value ) {
-		return add_option( $name, $value );
+	protected function add_option($name, $value) {
+		return add_option($name, $value);
 	}
 
 	/**
@@ -256,8 +256,8 @@ 
 	 *
 	 * @return bool Whether the option was updated successfully.
 	 */
-	public function update_option( $name, $value ) {
-		return update_option( $name, $value );
+	public function update_option($name, $value) {
+		return update_option($name, $value);
 	}
 
 	/**
@@ -269,8 +269,8 @@ 
 	 *
 	 * @return bool Whether the option was deleted successfully.
 	 */
-	protected function delete_option( $name ) {
-		return delete_option( $name );
+	protected function delete_option($name) {
+		return delete_option($name);
 	}
 
 	/**
@@ -282,7 +282,7 @@ 
 	 *
 	 * @return string The name of the option where the settings are stored.
 	 */
-	public function get_settings_option_name( $id ) {
+	public function get_settings_option_name($id) {
 		return "wordpoints_{$this->reactor_slug}_hook_reaction-{$id}";
 	}
 

src/includes/classes/hook/reaction/storage/options/network.php

Spacing +8 added lines, -8 removed lines

@@ -22,29 +22,29 @@
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_option( $name ) {
-		return get_site_option( $name );
+	public function get_option($name) {
+		return get_site_option($name);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function add_option( $name, $value ) {
-		return add_site_option( $name, $value );
+	protected function add_option($name, $value) {
+		return add_site_option($name, $value);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_option( $name, $value ) {
-		return update_site_option( $name, $value );
+	public function update_option($name, $value) {
+		return update_site_option($name, $value);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function delete_option( $name ) {
-		return delete_site_option( $name );
+	protected function delete_option($name) {
+		return delete_site_option($name);
 	}
 }
 

src/includes/classes/hook/reaction/storagei.php

Spacing +7 added lines, -7 removed lines

@@ -25,7 +25,7 @@ 
 	 * @param string $reactor_slug The slug of the reactor the reactions belong to.
 	 * @param bool   $network_wide Whether this object will store network-wide reactions.
 	 */
-	public function __construct( $reactor_slug, $network_wide );
+	public function __construct($reactor_slug, $network_wide);
 
 	/**
 	 * Get the slug of the reactor this object stores reactions for.
@@ -54,7 +54,7 @@ 
 	 *
 	 * @return bool Whether the reaction exists.
 	 */
-	public function reaction_exists( $id );
+	public function reaction_exists($id);
 
 	/**
 	 * Get an reaction object.
@@ -65,7 +65,7 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI|false The reaction, or false if nonexistent.
 	 */
-	public function get_reaction( $id );
+	public function get_reaction($id);
 
 	/**
 	 * Create an reaction.
@@ -78,7 +78,7 @@ 
 	 *         The reaction object if created successfully. False or a validator
 	 *         instance if not.
 	 */
-	public function create_reaction( array $settings );
+	public function create_reaction(array $settings);
 
 	/**
 	 * Update an reaction.
@@ -92,7 +92,7 @@ 
 	 *         The reaction object if updated successfully. False or a validator
 	 *         instance if not.
 	 */
-	public function update_reaction( $id, array $settings );
+	public function update_reaction($id, array $settings);
 
 	/**
 	 * Delete an reaction.
@@ -103,7 +103,7 @@ 
 	 *
 	 * @return bool Whether the reaction was deleted successfully.
 	 */
-	public function delete_reaction( $id );
+	public function delete_reaction($id);
 
 	/**
 	 * Get all hook reactions for the reactor.
@@ -126,7 +126,7 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI[]
 	 */
-	public function get_reactions_to_event( $event_slug );
+	public function get_reactions_to_event($event_slug);
 }
 
 // EOF

src/includes/classes/hook/reaction/validator.php

Spacing +26 added lines, -26 removed lines

@@ -104,13 +104,13 @@ 
 	 * @param bool                            $fail_fast Whether to fail as soon as
 	 *                                                   the first error is found.
 	 */
-	public function __construct( $settings, WordPoints_Hook_Reactor $reactor, $fail_fast = false ) {
+	public function __construct($settings, WordPoints_Hook_Reactor $reactor, $fail_fast = false) {
 
 		$this->reactor = $reactor;
 		$this->fail_fast = $fail_fast;
 		$this->hooks = wordpoints_hooks();
 
-		if ( $settings instanceof WordPoints_Hook_ReactionI ) {
+		if ($settings instanceof WordPoints_Hook_ReactionI) {
 
 			$this->reaction   = $settings;
 			$this->settings   = $this->reaction->get_all_meta();
@@ -120,7 +120,7 @@ 
 
 			$this->settings = $settings;
 
-			if ( isset( $this->settings['event'] ) ) {
+			if (isset($this->settings['event'])) {
 				$this->event_slug = $this->settings['event'];
 			}
 		}
@@ -144,10 +144,10 @@ 
 			$fail_fast = $this->fail_fast;
 			$this->fail_fast = true;
 
-			if ( ! isset( $this->event_slug ) ) {
-				$this->add_error( __( 'Event is missing.', 'wordpoints' ), 'event' );
-			} elseif ( ! $this->hooks->events->is_registered( $this->event_slug ) ) {
-				$this->add_error( __( 'Event is invalid.', 'wordpoints' ), 'event' );
+			if ( ! isset($this->event_slug)) {
+				$this->add_error(__('Event is missing.', 'wordpoints'), 'event');
+			} elseif ( ! $this->hooks->events->is_registered($this->event_slug)) {
+				$this->add_error(__('Event is invalid.', 'wordpoints'), 'event');
 			}
 
 			// From here on out we can collect errors as they come (unless we are
@@ -158,14 +158,14 @@ 
 				$this->event_slug
 			);
 
-			$this->event_args = new WordPoints_Hook_Event_Args( $event_args );
-			$this->event_args->set_validator( $this );
+			$this->event_args = new WordPoints_Hook_Event_Args($event_args);
+			$this->event_args->set_validator($this);
 
-			$this->settings = $this->reactor->validate_settings( $this->settings, $this, $this->event_args );
+			$this->settings = $this->reactor->validate_settings($this->settings, $this, $this->event_args);
 
 			/** @var WordPoints_Hook_Extension $extension */
-			foreach ( $this->hooks->extensions->get_all() as $extension ) {
-				$this->settings = $extension->validate_settings( $this->settings, $this, $this->event_args );
+			foreach ($this->hooks->extensions->get_all() as $extension) {
+				$this->settings = $extension->validate_settings($this->settings, $this, $this->event_args);
 			}
 
 			/**
@@ -175,12 +175,12 @@ 
 			 * @param WordPoints_Hook_Reaction_Validator $validator The validator object.
 			 * @param WordPoints_Hook_Event_Args         $args      The event args object.
 			 */
-			$this->settings = apply_filters( 'wordpoints_hook_reaction_validate', $this->settings, $this, $this->event_args );
+			$this->settings = apply_filters('wordpoints_hook_reaction_validate', $this->settings, $this, $this->event_args);
 
-		} catch ( WordPoints_Hook_Validator_Exception $e ) {
+		} catch (WordPoints_Hook_Validator_Exception $e) {
 
 			// Do nothing.
-			unset( $e );
+			unset($e);
 		}
 
 		return $this->settings;
@@ -208,17 +208,17 @@ 
 	 * @throws WordPoints_Hook_Validator_Exception If the validator is configured to
 	 *                                             fail as soon as an error is found.
 	 */
-	public function add_error( $message, $field = null ) {
+	public function add_error($message, $field = null) {
 
 		$field_stack = $this->field_stack;
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$field_stack[] = $field;
 		}
 
-		$this->errors[] = array( 'message' => $message, 'field' => $field_stack );
+		$this->errors[] = array('message' => $message, 'field' => $field_stack);
 
-		if ( $this->fail_fast ) {
+		if ($this->fail_fast) {
 			throw new WordPoints_Hook_Validator_Exception;
 		}
 	}
@@ -231,7 +231,7 @@ 
 	 * @return bool Whether the validator found any errors.
 	 */
 	public function had_errors() {
-		return ! empty( $this->errors );
+		return ! empty($this->errors);
 	}
 
 	/**
@@ -252,7 +252,7 @@ 
 	 *
 	 * @param string $field The field.
 	 */
-	public function push_field( $field ) {
+	public function push_field($field) {
 		$this->field_stack[] = $field;
 	}
 
@@ -262,7 +262,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function pop_field() {
-		array_pop( $this->field_stack );
+		array_pop($this->field_stack);
 	}
 
 	/**
@@ -307,7 +307,7 @@ 
 	 */
 	public function get_id() {
 
-		if ( ! $this->reaction ) {
+		if ( ! $this->reaction) {
 			return false;
 		}
 
@@ -345,13 +345,13 @@ 
 	 *
 	 * @return mixed The meta value.
 	 */
-	public function get_meta( $key ) {
+	public function get_meta($key) {
 
-		if ( ! isset( $this->settings[ $key ] ) ) {
+		if ( ! isset($this->settings[$key])) {
 			return null;
 		}
 
-		return $this->settings[ $key ];
+		return $this->settings[$key];
 	}
 
 	/**