WordPoints / hooks-api

src/includes/classes/hook/event/user/visit.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Visit', 'wordpoints' );
+		return __('Visit', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'When a logged-in user or guest visits the site.', 'wordpoints' );
+		return __('When a logged-in user or guest visits the site.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/eventi.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	/**
 	 * @param string $slug The event slug.
 	 */
-	public function __construct( $slug );
+	public function __construct($slug);
 
 	/**
 	 * Get the event slug.

src/includes/classes/hook/events.php

Spacing +21 added lines, -21 removed lines

@@ -37,13 +37,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 
 		$hooks = wordpoints_hooks();
 
 		$this->router = $hooks->router;
 
-		parent::__construct( $slug );
+		parent::__construct($slug);
 	}
 
 	/**
@@ -51,7 +51,7 @@ 
 	 */
 	public function init() {
 
-		$this->sub_apps->register( 'args', 'WordPoints_Class_Registry_Children' );
+		$this->sub_apps->register('args', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -75,29 +75,29 @@ 
 	 *
 	 * @return bool Whether the event was registered.
 	 */
-	public function register( $slug, $class, array $args = array() ) {
+	public function register($slug, $class, array $args = array()) {
 
 		// TODO shoudl this be required?
-		if ( ! isset( $args['actions'] ) ) {
+		if ( ! isset($args['actions'])) {
 			return false;
 		}
 
-		parent::register( $slug, $class, $args );
+		parent::register($slug, $class, $args);
 
-		foreach ( $args['actions'] as $type => $actions ) {
-			foreach ( (array) $actions as $action_slug ) {
-				$this->router->add_event_to_action( $slug, $action_slug, $type );
+		foreach ($args['actions'] as $type => $actions) {
+			foreach ((array) $actions as $action_slug) {
+				$this->router->add_event_to_action($slug, $action_slug, $type);
 			}
 		}
 
 		// TODO should this be required?
-		if ( isset( $args['args'] ) ) {
-			foreach ( $args['args'] as $arg_slug => $class ) {
-				$this->args->register( $slug, $arg_slug, $class );
+		if (isset($args['args'])) {
+			foreach ($args['args'] as $arg_slug => $class) {
+				$this->args->register($slug, $arg_slug, $class);
 			}
 		}
 
-		$this->event_data[ $slug ] = $args;
+		$this->event_data[$slug] = $args;
 
 		return true;
 	}
@@ -105,23 +105,23 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		if ( ! $this->is_registered( $slug ) ) {
+		if ( ! $this->is_registered($slug)) {
 			return;
 		}
 
-		parent::deregister( $slug );
+		parent::deregister($slug);
 
-		foreach ( (array) $this->event_data[ $slug ]['actions'] as $type => $actions ) {
-			foreach ( (array) $actions as $action_slug ) {
-				$this->router->remove_event_from_action( $slug, $action_slug, $type );
+		foreach ((array) $this->event_data[$slug]['actions'] as $type => $actions) {
+			foreach ((array) $actions as $action_slug) {
+				$this->router->remove_event_from_action($slug, $action_slug, $type);
 			}
 		}
 
-		$this->args->deregister_children( $slug );
+		$this->args->deregister_children($slug);
 
-		unset( $this->event_data[ $slug ] );
+		unset($this->event_data[$slug]);
 	}
 }
 

src/includes/classes/hook/extension.php

Spacing +7 added lines, -7 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);
 		}
 	}
 

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/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}";
 	}