WordPoints / hooks-api

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

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/reactioni.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
 	 * @param int                               $id      The ID of a hook reaction.
 	 * @param WordPoints_Hook_Reaction_StorageI $storage The storage object.
 	 */
-	public function __construct( $id, WordPoints_Hook_Reaction_StorageI $storage );
+	public function __construct($id, WordPoints_Hook_Reaction_StorageI $storage);
 
 	/**
 	 * Check whether this reaction is network-wide.
@@ -54,7 +54,7 @@ 
 	 *
 	 * @return bool Whether the event was updated successfully.
 	 */
-	public function update_event_slug( $event_slug );
+	public function update_event_slug($event_slug);
 
 	/**
 	 * Get the slug of the reactor this reaction is for.
@@ -74,7 +74,7 @@ 
 	 *
 	 * @return mixed|false The meta value, or false if not found.
 	 */
-	public function get_meta( $key );
+	public function get_meta($key);
 
 	/**
 	 * Add a piece of metadata for this reaction.
@@ -88,7 +88,7 @@ 
 	 *
 	 * @return bool Whether the metadata was added successfully.
 	 */
-	public function add_meta( $key, $value );
+	public function add_meta($key, $value);
 
 	/**
 	 * Update a piece of metadata for this reaction.
@@ -100,7 +100,7 @@ 
 	 *
 	 * @return bool Whether the metadata was updated successfully.
 	 */
-	public function update_meta( $key, $value );
+	public function update_meta($key, $value);
 
 	/**
 	 * Delete a piece of metadata for this reaction.
@@ -111,7 +111,7 @@ 
 	 *
 	 * @return bool Whether the metadata was deleted successfully.
 	 */
-	public function delete_meta( $key );
+	public function delete_meta($key);
 
 	/**
 	 * Get all of the metadata for this reaction.

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

Spacing +33 added lines, -33 removed lines

@@ -55,9 +55,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();
 	}
@@ -71,23 +71,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);
 	}
 
 	/**
@@ -98,12 +98,12 @@ 
 		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']);
 	}
 
 	/**
@@ -115,33 +115,33 @@ 
 	) {
 
 		$target = $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 ( $event_args->get_entities() as $entity ) {
-			$meta[ $entity->get_slug() ] = $entity->get_the_id();
+		foreach ($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 spam_hits( $event_slug, WordPoints_Hook_Event_Args $event_args ) {
+	public function spam_hits($event_slug, WordPoints_Hook_Event_Args $event_args) {
 
 		$meta_queries = array(
 			array(
@@ -152,7 +152,7 @@ 
 			),
 		);
 
-		foreach ( $event_args->get_entities() as $slug => $entity ) {
+		foreach ($event_args->get_entities() as $slug => $entity) {
 
 			$meta_queries[] = array(
 				'key'   => $slug,
@@ -169,35 +169,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
 				, "spam-{$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');
 	}
 }