WordPoints / hooks-api

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

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
 	 *
 	 * @param string $event_slug The slug of the event this reaction is for.
 	 *
-	 * @return int|false The reaction ID, or false if not created.
+	 * @return integer The reaction ID, or false if not created.
 	 */
 	abstract protected function _create_reaction( $event_slug );
 }

Spacing +27 added lines, -27 removed lines

@@ -51,7 +51,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 		$this->slug = $slug;
 	}
 
@@ -66,33 +66,33 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_context_id() {
-		return wordpoints_entities_get_current_context_id( $this->context );
+		return wordpoints_entities_get_current_context_id($this->context);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_reaction( $id ) {
+	public function get_reaction($id) {
 
-		if ( ! $this->reaction_exists( $id ) ) {
+		if ( ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		return new $this->reaction_class( $id, $this );
+		return new $this->reaction_class($id, $this);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function create_reaction( array $settings ) {
-		return $this->create_or_update_reaction( $settings );
+	public function create_reaction(array $settings) {
+		return $this->create_or_update_reaction($settings);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_reaction( $id, array $settings ) {
-		return $this->create_or_update_reaction( $settings, $id );
+	public function update_reaction($id, array $settings) {
+		return $this->create_or_update_reaction($settings, $id);
 	}
 
 	/**
@@ -107,45 +107,45 @@ 
 	 *         The reaction object if created/updated successfully. False or a
 	 *         validator instance if not.
 	 */
-	protected function create_or_update_reaction( array $settings, $id = null ) {
+	protected function create_or_update_reaction(array $settings, $id = null) {
 
-		$is_new = ! isset( $id );
+		$is_new = ! isset($id);
 
-		if ( ! $is_new && ! $this->reaction_exists( $id ) ) {
+		if ( ! $is_new && ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		$validator = new WordPoints_Hook_Reaction_Validator( $settings );
+		$validator = new WordPoints_Hook_Reaction_Validator($settings);
 		$settings = $validator->validate();
 
-		if ( $validator->had_errors() ) {
+		if ($validator->had_errors()) {
 			return $validator;
 		}
 
-		if ( $is_new ) {
+		if ($is_new) {
 
-			$id = $this->_create_reaction( $settings['event'] );
+			$id = $this->_create_reaction($settings['event']);
 
-			if ( ! $id ) {
+			if ( ! $id) {
 				return false;
 			}
 		}
 
-		$reaction = $this->get_reaction( $id );
+		$reaction = $this->get_reaction($id);
 
-		$reaction->update_event_slug( $settings['event'] );
+		$reaction->update_event_slug($settings['event']);
 
-		unset( $settings['event'] );
+		unset($settings['event']);
 
-		$reaction->update_meta( 'reactor', $settings['reactor'] );
+		$reaction->update_meta('reactor', $settings['reactor']);
 
 		/** @var WordPoints_Hook_Reactor $reactor */
-		$reactor = wordpoints_hooks()->reactors->get( $settings['reactor'] );
-		$reactor->update_settings( $reaction, $settings );
+		$reactor = wordpoints_hooks()->reactors->get($settings['reactor']);
+		$reactor->update_settings($reaction, $settings);
 
 		/** @var WordPoints_Hook_Extension $extension */
-		foreach ( wordpoints_hooks()->extensions->get_all() as $extension ) {
-			$extension->update_settings( $reaction, $settings );
+		foreach (wordpoints_hooks()->extensions->get_all() as $extension) {
+			$extension->update_settings($reaction, $settings);
 		}
 
 		/**
@@ -155,7 +155,7 @@ 
 		 * @param array                     $settings The new settings for the reaction.
 		 * @param bool                      $is_new   Whether the reaction was just now created.
 		 */
-		do_action( 'wordpoints_hook_reaction_save', $reaction, $settings, $is_new );
+		do_action('wordpoints_hook_reaction_save', $reaction, $settings, $is_new);
 
 		return $reaction;
 	}
@@ -172,7 +172,7 @@ 
 	 *
 	 * @return int|false The reaction ID, or false if not created.
 	 */
-	abstract protected function _create_reaction( $event_slug );
+	abstract protected function _create_reaction($event_slug);
 }
 
 // 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->store->get_reaction_event_from_index( $this->ID );
+		return $this->store->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->store->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->store->get_option(
-			$this->store->get_settings_option_name( $this->ID )
+			$this->store->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->store->update_option(
-			$this->store->get_settings_option_name( $this->ID )
+			$this->store->get_settings_option_name($this->ID)
 			, $settings
 		);
 	}

src/includes/classes/hook/reaction/store/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/store/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_StoreI $store The storage object.
 	 */
-	public function __construct( $id, WordPoints_Hook_Reaction_StoreI $store );
+	public function __construct($id, WordPoints_Hook_Reaction_StoreI $store);
 
 	/**
 	 * Get a Globally Unique ID for this reaction.
@@ -67,7 +67,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.
@@ -100,7 +100,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.
@@ -114,7 +114,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.
@@ -126,7 +126,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.
@@ -137,7 +137,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/fire.php

Spacing +3 added lines, -3 removed lines

@@ -73,7 +73,7 @@ 
 		$this->firer      = $firer;
 		$this->event_args = $event_args;
 		$this->reaction   = $reaction;
-		$this->hit_logger = new WordPoints_Hook_Hit_Logger( $this );
+		$this->hit_logger = new WordPoints_Hook_Hit_Logger($this);
 	}
 
 	/**
@@ -85,11 +85,11 @@ 
 	 */
 	public function hit() {
 
-		if ( ! $this->hit_id ) {
+		if ( ! $this->hit_id) {
 
 			$this->hit_id = $this->hit_logger->log_hit();
 
-			if ( ! $this->hit_id ) {
+			if ( ! $this->hit_id) {
 				return false;
 			}
 		}

src/includes/classes/hook/hit/logger.php

Spacing +5 added lines, -5 removed lines

@@ -28,7 +28,7 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The fire that might be logged as a hit.
 	 */
-	public function __construct( WordPoints_Hook_Fire $fire ) {
+	public function __construct(WordPoints_Hook_Fire $fire) {
 
 		$this->fire = $fire;
 	}
@@ -44,7 +44,7 @@ 
 
 		global $wpdb;
 
-		$signature = wordpoints_hooks_get_event_primary_arg_guid_json( $this->fire->event_args );
+		$signature = wordpoints_hooks_get_event_primary_arg_guid_json($this->fire->event_args);
 
 		$inserted = $wpdb->insert(
 			$wpdb->wordpoints_hook_hits
@@ -54,13 +54,13 @@ 
 				'event' => $this->fire->reaction->get_event_slug(),
 				'reactor' => $this->fire->reaction->get_reactor_slug(),
 				'reaction_store' => $this->fire->reaction->get_store_slug(),
-				'reaction_context_id' => wp_json_encode( $this->fire->reaction->get_context_id() ),
+				'reaction_context_id' => wp_json_encode($this->fire->reaction->get_context_id()),
 				'reaction_id' => $this->fire->reaction->ID,
-				'date' => current_time( 'mysql' ),
+				'date' => current_time('mysql'),
 			)
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}
 

src/includes/constants.php

Spacing +3 added lines, -3 removed lines

@@ -9,8 +9,8 @@
 
 global $wpdb;
 
-$wpdb->wordpoints_hook_hits = $wpdb->base_prefix . 'wordpoints_hook_hits';
-$wpdb->wordpoints_hook_hitmeta = $wpdb->base_prefix . 'wordpoints_hook_hitmeta';
-$wpdb->wordpoints_hook_periods = $wpdb->base_prefix . 'wordpoints_hook_periods';
+$wpdb->wordpoints_hook_hits = $wpdb->base_prefix.'wordpoints_hook_hits';
+$wpdb->wordpoints_hook_hitmeta = $wpdb->base_prefix.'wordpoints_hook_hitmeta';
+$wpdb->wordpoints_hook_periods = $wpdb->base_prefix.'wordpoints_hook_periods';
 
 // EOF

src/includes/classes/db/query.php

Spacing +111 added lines, -111 removed lines

@@ -214,8 +214,8 @@ 
 	 *        @type array        $meta_query          See WP_Meta_Query.
 	 * }
 	 */
-	public function __construct( $args = array() ) {
-		$this->args = array_merge( $this->defaults, $args );
+	public function __construct($args = array()) {
+		$this->args = array_merge($this->defaults, $args);
 	}
 
 	/**
@@ -227,10 +227,10 @@ 
 	 *
 	 * @return mixed|null The query arg's value, or null if it isn't set.
 	 */
-	public function get_arg( $arg ) {
+	public function get_arg($arg) {
 
-		if ( isset( $this->args[ $arg ] ) ) {
-			return $this->args[ $arg ];
+		if (isset($this->args[$arg])) {
+			return $this->args[$arg];
 		} else {
 			return null;
 		}
@@ -246,9 +246,9 @@ 
 	 *
 	 * @param array $args A list of arguments to set and their values.
 	 */
-	public function set_args( array $args ) {
+	public function set_args(array $args) {
 
-		$this->args = array_merge( $this->args, $args );
+		$this->args = array_merge($this->args, $args);
 
 		$this->is_query_ready = false;
 	}
@@ -269,7 +269,7 @@ 
 
 		global $wpdb;
 
-		$count = (int) $wpdb->get_var( $this->get_sql( 'SELECT COUNT' ) ); // WPCS: unprepared SQL, cache OK
+		$count = (int) $wpdb->get_var($this->get_sql('SELECT COUNT')); // WPCS: unprepared SQL, cache OK
 
 		return $count;
 	}
@@ -284,20 +284,20 @@ 
 	 *
 	 * @return mixed The results of the query, or false on failure.
 	 */
-	public function get( $method = 'results' ) {
+	public function get($method = 'results') {
 
 		global $wpdb;
 
-		$methods = array( 'results', 'row', 'col', 'var' );
+		$methods = array('results', 'row', 'col', 'var');
 
-		if ( ! in_array( $method, $methods ) ) {
+		if ( ! in_array($method, $methods)) {
 
-			_doing_it_wrong( __METHOD__, esc_html( sprintf( 'WordPoints Debug Error: invalid get method %s, possible values are %s', $method, implode( ', ', $methods ) ) ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html(sprintf('WordPoints Debug Error: invalid get method %s, possible values are %s', $method, implode(', ', $methods))), '1.0.0');
 
 			return false;
 		}
 
-		$result = $wpdb->{"get_{$method}"}( $this->get_sql() );
+		$result = $wpdb->{"get_{$method}"}($this->get_sql());
 
 		return $result;
 	}
@@ -317,11 +317,11 @@ 
 	 *
 	 * @return string The SQL for the query.
 	 */
-	public function get_sql( $select_type = 'SELECT' ) {
+	public function get_sql($select_type = 'SELECT') {
 
 		$this->prepare_query();
 
-		$select = ( 'SELECT COUNT' === $select_type )
+		$select = ('SELECT COUNT' === $select_type)
 			? $this->select_count
 			: $this->select;
 
@@ -349,12 +349,12 @@ 
 	 *
 	 * @return string[] The valid columns.
 	 */
-	public function date_query_valid_columns_filter( $valid_columns ) {
+	public function date_query_valid_columns_filter($valid_columns) {
 
 		$valid_columns = array_merge(
 			$valid_columns
 			, array_keys(
-				wp_list_filter( $this->columns, array( 'is_date' => true ) )
+				wp_list_filter($this->columns, array('is_date' => true))
 			)
 		);
 
@@ -372,7 +372,7 @@ 
 	 */
 	protected function prepare_query() {
 
-		if ( ! $this->is_query_ready ) {
+		if ( ! $this->is_query_ready) {
 
 			$this->prepare_select();
 			$this->prepare_where();
@@ -390,26 +390,26 @@ 
 	 */
 	protected function prepare_select() {
 
-		$all_fields = array_keys( $this->columns );
+		$all_fields = array_keys($this->columns);
 		$fields = array();
 
-		if ( ! empty( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields'])) {
 
 			$fields = (array) $this->args['fields'];
-			$diff   = array_diff( $fields, $all_fields );
-			$fields = array_intersect( $all_fields, $fields );
+			$diff   = array_diff($fields, $all_fields);
+			$fields = array_intersect($all_fields, $fields);
 
-			if ( ! empty( $diff ) ) {
-				_doing_it_wrong( __METHOD__, esc_html( 'WordPoints Debug Error: invalid field(s) "' . implode( '", "', $diff ) . '" given' ), '1.0.0' );
+			if ( ! empty($diff)) {
+				_doing_it_wrong(__METHOD__, esc_html('WordPoints Debug Error: invalid field(s) "'.implode('", "', $diff).'" given'), '1.0.0');
 			}
 		}
 
 		// Pull all fields by default.
-		if ( empty( $fields ) ) {
+		if (empty($fields)) {
 			$fields = $all_fields;
 		}
 
-		$fields = implode( ', ', array_map( 'wordpoints_escape_mysql_identifier', $fields ) );
+		$fields = implode(', ', array_map('wordpoints_escape_mysql_identifier', $fields));
 
 		$this->select = "SELECT {$fields}";
 	}
@@ -424,13 +424,13 @@ 
 	 *
 	 * @return mixed The validated value, or false if invalid.
 	 */
-	protected function validate_value( $value, $validators ) {
+	protected function validate_value($value, $validators) {
 
-		foreach ( $validators as $validator ) {
+		foreach ($validators as $validator) {
 
-			$value = call_user_func_array( $validator, array( &$value ) );
+			$value = call_user_func_array($validator, array(&$value));
 
-			if ( false === $value ) {
+			if (false === $value) {
 				break;
 			}
 		}
@@ -448,14 +448,14 @@ 
 	 *
 	 * @return array The validated values, with any invalid ones removed.
 	 */
-	protected function validate_values( $values, $validators ) {
+	protected function validate_values($values, $validators) {
 
-		foreach ( $values as $index => $value ) {
+		foreach ($values as $index => $value) {
 
-			$value = $this->validate_value( $value, $validators );
+			$value = $this->validate_value($value, $validators);
 
-			if ( false === $value ) {
-				unset( $values[ $index ] );
+			if (false === $value) {
+				unset($values[$index]);
 			}
 		}
 
@@ -474,9 +474,9 @@ 
 	 *
 	 * @return int|false The validated value or false.
 	 */
-	protected function validate_unsigned_column( $value ) {
+	protected function validate_unsigned_column($value) {
 
-		if ( false !== wordpoints_int( $value ) && $value >= 0 ) {
+		if (false !== wordpoints_int($value) && $value >= 0) {
 			return $value;
 		}
 
@@ -492,14 +492,14 @@ 
 	 *
 	 * @return callable[] The validation functions.
 	 */
-	protected function get_validators_for_column( $data ) {
+	protected function get_validators_for_column($data) {
 
 		$validators = array();
 
 		// Default validators for integer columns.
-		if ( '%d' === $data['format'] ) {
-			if ( ! empty( $data['unsigned'] ) ) {
-				$validators[] = array( $this, 'validate_unsigned_column' );
+		if ('%d' === $data['format']) {
+			if ( ! empty($data['unsigned'])) {
+				$validators[] = array($this, 'validate_unsigned_column');
 			} else {
 				$validators[] = 'wordpoints_int';
 			}
@@ -516,15 +516,15 @@ 
 	 * @param string $column The column name.
 	 * @param array  $data   The column data.
 	 */
-	protected function prepare_column_where( $column, $data ) {
+	protected function prepare_column_where($column, $data) {
 
 		// If a single value has been supplied for the column, it takes precedence.
-		if ( isset( $this->args[ $column ] ) ) {
-			$this->prepare_column( $column, $data );
-		} elseif ( isset( $this->args[ "{$column}__in" ] ) ) {
-			$this->prepare_column__in( $column, $data );
-		} elseif ( isset( $this->args[ "{$column}__not_in" ] ) ) {
-			$this->prepare_column__in( $column, $data, 'NOT IN' );
+		if (isset($this->args[$column])) {
+			$this->prepare_column($column, $data);
+		} elseif (isset($this->args["{$column}__in"])) {
+			$this->prepare_column__in($column, $data);
+		} elseif (isset($this->args["{$column}__not_in"])) {
+			$this->prepare_column__in($column, $data, 'NOT IN');
 		}
 	}
 
@@ -536,29 +536,29 @@ 
 	 * @param string $column The name of the column
 	 * @param array  $data   The column data.
 	 */
-	protected function prepare_column( $column, $data ) {
+	protected function prepare_column($column, $data) {
 
 		global $wpdb;
 
 		if (
-			isset( $data['values'] )
-			&& ! in_array( $this->args[ $column ], $data['values'], true )
+			isset($data['values'])
+			&& ! in_array($this->args[$column], $data['values'], true)
 		) {
 			return;
 		}
 
 		$value = $this->validate_value(
-			$this->args[ $column ]
-			, $this->get_validators_for_column( $data )
+			$this->args[$column]
+			, $this->get_validators_for_column($data)
 		);
 
-		if ( false === $value ) {
+		if (false === $value) {
 			return;
 		}
 
-		$compare = $this->get_comparator_for_column( $column, $data );
+		$compare = $this->get_comparator_for_column($column, $data);
 
-		$column = wordpoints_escape_mysql_identifier( $column );
+		$column = wordpoints_escape_mysql_identifier($column);
 
 		$this->wheres[] = $wpdb->prepare( // WPCS: unprepared SQL OK.
 			"{$column} {$compare} {$data['format']}"
@@ -576,23 +576,23 @@ 
 	 *
 	 * @return string The comparator for the column.
 	 */
-	protected function get_comparator_for_column( $column, $data ) {
+	protected function get_comparator_for_column($column, $data) {
 
-		$comparisons = array( '=', '<', '>', '<>', '!=', '<=', '>=' );
+		$comparisons = array('=', '<', '>', '<>', '!=', '<=', '>=');
 
 		// MySQL doesn't support LIKE and NOT LIKE for int columns.
 		// See http://stackoverflow.com/q/8422455/1924128
-		if ( '%s' === $data['format'] ) {
-			$comparisons = array_merge( $comparisons, array( 'LIKE', 'NOT LIKE' ) );
+		if ('%s' === $data['format']) {
+			$comparisons = array_merge($comparisons, array('LIKE', 'NOT LIKE'));
 		}
 
 		$comparator = '=';
 
 		if (
-			isset( $this->args[ "{$column}__compare" ] )
-			&& in_array( $this->args[ "{$column}__compare" ], $comparisons, true )
+			isset($this->args["{$column}__compare"])
+			&& in_array($this->args["{$column}__compare"], $comparisons, true)
 		) {
-			$comparator = $this->args[ "{$column}__compare" ];
+			$comparator = $this->args["{$column}__compare"];
 		}
 
 		return $comparator;
@@ -607,36 +607,36 @@ 
 	 * @param array  $data   The column data.
 	 * @param string $type   The type of IN clause, IN or NOT IN.
 	 */
-	protected function prepare_column__in( $column, $data, $type = 'IN' ) {
+	protected function prepare_column__in($column, $data, $type = 'IN') {
 
-		$key = "{$column}__" . strtolower( str_replace( ' ', '_', $type ) );
+		$key = "{$column}__".strtolower(str_replace(' ', '_', $type));
 
-		if ( empty( $this->args[ $key ] ) || ! is_array( $this->args[ $key ] ) ) {
+		if (empty($this->args[$key]) || ! is_array($this->args[$key])) {
 			return;
 		}
 
-		$values = $this->args[ $key ];
+		$values = $this->args[$key];
 
-		if ( isset( $data['values'] ) ) {
-			$values = array_intersect( $values, $data['values'] );
+		if (isset($data['values'])) {
+			$values = array_intersect($values, $data['values']);
 		} else {
 			$values = $this->validate_values(
 				$values
-				, $this->get_validators_for_column( $data )
+				, $this->get_validators_for_column($data)
 			);
 		}
 
-		if ( empty( $values ) ) {
+		if (empty($values)) {
 			return;
 		}
 
-		$in = wordpoints_prepare__in( $values, $data['format'] );
+		$in = wordpoints_prepare__in($values, $data['format']);
 
-		if ( false === $in ) {
+		if (false === $in) {
 			return;
 		}
 
-		$column = wordpoints_escape_mysql_identifier( $column );
+		$column = wordpoints_escape_mysql_identifier($column);
 
 		$this->wheres[] = "{$column} {$type} ({$in})";
 	}
@@ -650,19 +650,19 @@ 
 
 		$this->wheres = array();
 
-		foreach ( $this->columns as $column => $data ) {
+		foreach ($this->columns as $column => $data) {
 
-			if ( ! empty( $data['is_date'] ) ) {
-				$this->prepare_date_where( $column );
+			if ( ! empty($data['is_date'])) {
+				$this->prepare_date_where($column);
 			} else {
-				$this->prepare_column_where( $column, $data );
+				$this->prepare_column_where($column, $data);
 			}
 		}
 
 		$this->prepare_meta_where();
 
-		if ( ! empty( $this->wheres ) ) {
-			$this->where = 'WHERE ' . implode( ' AND ', $this->wheres ) . "\n";
+		if ( ! empty($this->wheres)) {
+			$this->where = 'WHERE '.implode(' AND ', $this->wheres)."\n";
 		}
 	}
 
@@ -675,32 +675,32 @@ 
 
 		// MySQL doesn't allow for the offset without a limit, so if no limit is set
 		// we can ignore the start arg. See http://stackoverflow.com/a/271650/1924128
-		if ( ! isset( $this->args['limit'] ) ) {
+		if ( ! isset($this->args['limit'])) {
 			return;
 		}
 
-		foreach ( array( 'limit', 'start' ) as $key ) {
+		foreach (array('limit', 'start') as $key) {
 
 			// Save a backup of the arg value since wordpoints_int() is by reference.
-			$arg = $this->args[ $key ];
+			$arg = $this->args[$key];
 
-			if ( false === wordpoints_int( $this->args[ $key ] ) ) {
+			if (false === wordpoints_int($this->args[$key])) {
 
 				_doing_it_wrong(
 					__METHOD__
 					, sprintf(
 						"WordPoints Debug Error: '%s' must be a positive integer, %s given"
-						, esc_html( $key )
-						, esc_html( strval( $arg ) ? $arg : gettype( $arg ) )
+						, esc_html($key)
+						, esc_html(strval($arg) ? $arg : gettype($arg))
 					)
 					, '1.0.0'
 				);
 
-				$this->args[ $key ] = 0;
+				$this->args[$key] = 0;
 			}
 		}
 
-		if ( $this->args['limit'] > 0 && $this->args['start'] >= 0 ) {
+		if ($this->args['limit'] > 0 && $this->args['start'] >= 0) {
 			$this->limit = "LIMIT {$this->args['start']}, {$this->args['limit']}";
 		}
 	}
@@ -712,20 +712,20 @@ 
 	 */
 	protected function prepare_order_by() {
 
-		if ( empty( $this->args['order_by'] ) ) {
+		if (empty($this->args['order_by'])) {
 			return;
 		}
 
 		$order    = $this->args['order'];
 		$order_by = $this->args['order_by'];
 
-		if ( ! in_array( $order, array( 'DESC', 'ASC' ) ) ) {
+		if ( ! in_array($order, array('DESC', 'ASC'))) {
 
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: invalid 'order' \"{$order}\", possible values are DESC and ASC" ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: invalid 'order' \"{$order}\", possible values are DESC and ASC"), '1.0.0');
 			$order = 'DESC';
 		}
 
-		if ( 'meta_value' === $order_by ) {
+		if ('meta_value' === $order_by) {
 
 			global $wpdb;
 
@@ -733,9 +733,9 @@ 
 				$wpdb->{"{$this->meta_type}meta"}
 			);
 
-			if ( isset( $this->args['meta_type'] ) ) {
+			if (isset($this->args['meta_type'])) {
 
-				$meta_type = $this->meta_query->get_cast_for_type( $this->args['meta_type'] );
+				$meta_type = $this->meta_query->get_cast_for_type($this->args['meta_type']);
 				$order_by  = "CAST({$meta_table_name}.meta_value AS {$meta_type})";
 
 			} else {
@@ -743,13 +743,13 @@ 
 				$order_by = "{$meta_table_name}.meta_value";
 			}
 
-		} elseif ( isset( $this->columns[ $order_by ] ) ) {
+		} elseif (isset($this->columns[$order_by])) {
 
-			$order_by = wordpoints_escape_mysql_identifier( $order_by );
+			$order_by = wordpoints_escape_mysql_identifier($order_by);
 
 		} else {
 
-			_doing_it_wrong( __METHOD__, esc_html( "WordPoints Debug Error: invalid 'order_by' \"{$order_by}\", possible values are " . implode( ', ', array_keys( $this->columns ) ) ), '1.0.0' );
+			_doing_it_wrong(__METHOD__, esc_html("WordPoints Debug Error: invalid 'order_by' \"{$order_by}\", possible values are ".implode(', ', array_keys($this->columns))), '1.0.0');
 			return;
 		}
 
@@ -763,25 +763,25 @@ 
 	 *
 	 * @param string $column The name of the column.
 	 */
-	protected function prepare_date_where( $column ) {
+	protected function prepare_date_where($column) {
 
 		if (
-			empty( $this->args[ "{$column}_query" ] )
-			|| ! is_array( $this->args[ "{$column}_query" ] )
+			empty($this->args["{$column}_query"])
+			|| ! is_array($this->args["{$column}_query"])
 		) {
 			return;
 		}
 
-		add_filter( 'date_query_valid_columns', array( $this, 'date_query_valid_columns_filter' ) );
+		add_filter('date_query_valid_columns', array($this, 'date_query_valid_columns_filter'));
 
-		$date_query = new WP_Date_Query( $this->args[ "{$column}_query" ], $column );
+		$date_query = new WP_Date_Query($this->args["{$column}_query"], $column);
 		$date_query = $date_query->get_sql();
 
-		if ( ! empty( $date_query ) ) {
-			$this->wheres[] = ltrim( $date_query, ' AND' );
+		if ( ! empty($date_query)) {
+			$this->wheres[] = ltrim($date_query, ' AND');
 		}
 
-		remove_filter( 'date_query_valid_columns', array( $this, 'date_query_valid_columns_filter' ) );
+		remove_filter('date_query_valid_columns', array($this, 'date_query_valid_columns_filter'));
 	}
 
 	/**
@@ -791,7 +791,7 @@ 
 	 */
 	protected function prepare_meta_where() {
 
-		if ( empty( $this->meta_type ) ) {
+		if (empty($this->meta_type)) {
 			return;
 		}
 
@@ -806,12 +806,12 @@ 
 			)
 		);
 
-		if ( empty( $meta_args ) ) {
+		if (empty($meta_args)) {
 			return;
 		}
 
 		$this->meta_query = new WP_Meta_Query();
-		$this->meta_query->parse_query_vars( $meta_args );
+		$this->meta_query->parse_query_vars($meta_args);
 
 		$meta_query = $this->meta_query->get_sql(
 			$this->meta_type
@@ -820,11 +820,11 @@ 
 			, $this
 		);
 
-		if ( ! empty( $meta_query['where'] ) ) {
-			$this->wheres[] = ltrim( $meta_query['where'], ' AND' );
+		if ( ! empty($meta_query['where'])) {
+			$this->wheres[] = ltrim($meta_query['where'], ' AND');
 		}
 
-		$this->meta_join = $meta_query['join'] . "\n";
+		$this->meta_join = $meta_query['join']."\n";
 	}
 }