WordPoints / hooks-api

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

Spacing +35 added lines, -35 removed lines

@@ -103,12 +103,12 @@ 
 	 * @param bool                            $fail_fast Whether to fail as soon as
 	 *                                                   the first error is found.
 	 */
-	public function __construct( $settings, $fail_fast = false ) {
+	public function __construct($settings, $fail_fast = false) {
 
 		$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();
@@ -119,11 +119,11 @@ 
 
 			$this->settings = $settings;
 
-			if ( isset( $this->settings['event'] ) ) {
+			if (isset($this->settings['event'])) {
 				$this->event_slug = $this->settings['event'];
 			}
 
-			if ( isset( $this->settings['reactor'] ) ) {
+			if (isset($this->settings['reactor'])) {
 				$this->reactor_slug = $this->settings['reactor'];
 			}
 		}
@@ -146,40 +146,40 @@ 
 			$fail_fast = $this->fail_fast;
 			$this->fail_fast = true;
 
-			$events = $this->hooks->get_sub_app( 'events' );
+			$events = $this->hooks->get_sub_app('events');
 
-			if ( ! isset( $this->event_slug ) ) {
-				$this->add_error( __( 'Event is missing.', 'wordpoints' ), 'event' );
-			} elseif ( ! $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 ( ! $events->is_registered($this->event_slug)) {
+				$this->add_error(__('Event is invalid.', 'wordpoints'), 'event');
 			}
 
-			$reactors = $this->hooks->get_sub_app( 'reactors' );
+			$reactors = $this->hooks->get_sub_app('reactors');
 
-			if ( ! isset( $this->reactor_slug ) ) {
-				$this->add_error( __( 'Reactor is missing.', 'wordpoints' ), 'reactor' );
-			} elseif ( ! $reactors->is_registered( $this->reactor_slug ) ) {
-				$this->add_error( __( 'Reactor is invalid.', 'wordpoints' ), 'reactor' );
+			if ( ! isset($this->reactor_slug)) {
+				$this->add_error(__('Reactor is missing.', 'wordpoints'), 'reactor');
+			} elseif ( ! $reactors->is_registered($this->reactor_slug)) {
+				$this->add_error(__('Reactor is invalid.', 'wordpoints'), 'reactor');
 			}
 
 			// From here on out we can collect errors as they come (unless we are
 			// supposed to fail fast).
 			$this->fail_fast = $fail_fast;
 
-			$event_args = $events->get_sub_app( 'args' )->get_children(
+			$event_args = $events->get_sub_app('args')->get_children(
 				$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);
 
-			$reactor = $reactors->get( $this->reactor_slug );
+			$reactor = $reactors->get($this->reactor_slug);
 
-			$this->settings = $reactor->validate_settings( $this->settings, $this, $this->event_args );
+			$this->settings = $reactor->validate_settings($this->settings, $this, $this->event_args);
 
 			/** @var WordPoints_Hook_ExtensionI $extension */
-			foreach ( $this->hooks->get_sub_app( 'extensions' )->get_all() as $extension ) {
-				$this->settings = $extension->validate_settings( $this->settings, $this, $this->event_args );
+			foreach ($this->hooks->get_sub_app('extensions')->get_all() as $extension) {
+				$this->settings = $extension->validate_settings($this->settings, $this, $this->event_args);
 			}
 
 			/**
@@ -189,12 +189,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;
@@ -222,17 +222,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 ( null !== $field ) {
+		if (null !== $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;
 		}
 	}
@@ -245,7 +245,7 @@ 
 	 * @return bool Whether the validator found any errors.
 	 */
 	public function had_errors() {
-		return ! empty( $this->errors );
+		return ! empty($this->errors);
 	}
 
 	/**
@@ -266,7 +266,7 @@ 
 	 *
 	 * @param string $field The field.
 	 */
-	public function push_field( $field ) {
+	public function push_field($field) {
 		$this->field_stack[] = $field;
 	}
 
@@ -276,7 +276,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function pop_field() {
-		array_pop( $this->field_stack );
+		array_pop($this->field_stack);
 	}
 
 	/**
@@ -321,7 +321,7 @@ 
 	 */
 	public function get_id() {
 
-		if ( ! $this->reaction ) {
+		if ( ! $this->reaction) {
 			return false;
 		}
 
@@ -359,13 +359,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];
 	}
 
 	/**

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

Spacing +27 added lines, -27 removed lines

@@ -55,7 +55,7 @@ 
 	 *
 	 * @param string $slug The slug of this store.
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 		$this->slug = $slug;
 	}
 
@@ -70,33 +70,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);
 	}
 
 	/**
@@ -111,45 +111,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_ReactorI $reactor */
-		$reactor = wordpoints_hooks()->get_sub_app( 'reactors' )->get( $settings['reactor'] );
-		$reactor->update_settings( $reaction, $settings );
+		$reactor = wordpoints_hooks()->get_sub_app('reactors')->get($settings['reactor']);
+		$reactor->update_settings($reaction, $settings);
 
 		/** @var WordPoints_Hook_ExtensionI $extension */
-		foreach ( wordpoints_hooks()->get_sub_app( 'extensions' )->get_all() as $extension ) {
-			$extension->update_settings( $reaction, $settings );
+		foreach (wordpoints_hooks()->get_sub_app('extensions')->get_all() as $extension) {
+			$extension->update_settings($reaction, $settings);
 		}
 
 		/**
@@ -159,7 +159,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;
 	}
@@ -176,7 +176,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/condition.php

Spacing +11 added lines, -11 removed lines

@@ -23,33 +23,33 @@ 
 		WordPoints_Hook_Reaction_Validator $validator
 	) {
 
-		if ( ! isset( $settings['value'] ) || '' === $settings['value'] ) {
+		if ( ! isset($settings['value']) || '' === $settings['value']) {
 
 			$settings_fields = $this->get_settings_fields();
 
 			$validator->add_error(
 				sprintf(
-					__( '%s is required.', 'wordpoints' )
+					__('%s is required.', 'wordpoints')
 					, $settings_fields['value']['label']
 				)
 				, 'value'
 			);
 
-		} elseif ( $arg instanceof WordPoints_Entity_Attr ) {
+		} elseif ($arg instanceof WordPoints_Entity_Attr) {
 
-			$data_types = wordpoints_apps()->get_sub_app( 'data_types' );
+			$data_types = wordpoints_apps()->get_sub_app('data_types');
 
-			$data_type = $data_types->get( $arg->get_data_type() );
+			$data_type = $data_types->get($arg->get_data_type());
 
 			// If this data type isn't recognized, that's probably OK. Validation is
 			// just to help the user know that they've made a mistake anyway.
-			if ( ! ( $data_type instanceof WordPoints_Data_TypeI ) ) {
+			if ( ! ($data_type instanceof WordPoints_Data_TypeI)) {
 				return $settings;
 			}
 
-			$validated_value = $data_type->validate_value( $settings['value'] );
+			$validated_value = $data_type->validate_value($settings['value']);
 
-			if ( is_wp_error( $validated_value ) ) {
+			if (is_wp_error($validated_value)) {
 
 				$settings_fields = $this->get_settings_fields();
 
@@ -66,13 +66,13 @@ 
 
 			$settings['value'] = $validated_value;
 
-		} elseif ( $arg instanceof WordPoints_Entity ) {
+		} elseif ($arg instanceof WordPoints_Entity) {
 
-			if ( ! $arg->exists( $settings['value'] ) ) {
+			if ( ! $arg->exists($settings['value'])) {
 				$validator->add_error(
 					sprintf(
 						// translators: 1. item type, 2. item ID/slug.
-						__( '%1$s “%2$s” not found.', 'wordpoints' )
+						__('%1$s “%2$s” not found.', 'wordpoints')
 						, $arg->get_title()
 						, $settings['value']
 					)

src/includes/classes/hook/settings.php

Spacing +31 added lines, -31 removed lines

@@ -8,13 +8,13 @@ 
  */
 
 interface WordPoints_Hook_Reaction_SettingI {
-	public function __construct( $slug );
+	public function __construct($slug);
 	public function get_type();
 	public function get_label();
 	public function is_required();
-	public function get( WordPoints_Hook_ReactionI $reaction );
-	public function validate( $value, WordPoints_Hook_Reaction_Validator $validator );
-	public function save( $value, WordPoints_Hook_ReactionI $reaction, $is_new );
+	public function get(WordPoints_Hook_ReactionI $reaction);
+	public function validate($value, WordPoints_Hook_Reaction_Validator $validator);
+	public function save($value, WordPoints_Hook_ReactionI $reaction, $is_new);
 }
 
 class WordPoints_Hook_Reaction_Setting implements WordPoints_Hook_Reaction_SettingI {
@@ -23,7 +23,7 @@ 
 	protected $type = 'hidden';
 	protected $required = true;
 
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 		$this->slug = $slug;
 	}
 
@@ -39,17 +39,17 @@ 
 		return $this->required;
 	}
 
-	public function get( WordPoints_Hook_ReactionI $reaction ) {
-		return $reaction->get_meta( $this->slug );
+	public function get(WordPoints_Hook_ReactionI $reaction) {
+		return $reaction->get_meta($this->slug);
 	}
 
-	public function validate( $value, WordPoints_Hook_Reaction_Validator $validator ) {
+	public function validate($value, WordPoints_Hook_Reaction_Validator $validator) {
 
-		if ( $this->is_required() && '' === trim( $value ) ) {
+		if ($this->is_required() && '' === trim($value)) {
 
 			$validator->add_error(
 				sprintf(
-					__( '%s cannot be empty.', 'wordpoints' )
+					__('%s cannot be empty.', 'wordpoints')
 					, $this->get_label()
 				)
 				, $this->slug
@@ -59,8 +59,8 @@ 
 		return $value;
 	}
 
-	public function save( $value, WordPoints_Hook_ReactionI $reaction, $is_new ) {
-		$reaction->update_meta( $this->slug, $value );
+	public function save($value, WordPoints_Hook_ReactionI $reaction, $is_new) {
+		$reaction->update_meta($this->slug, $value);
 	}
 }
 //
@@ -82,12 +82,12 @@ 
 
 		$fields = array();
 
-		foreach ( $this->settings as $slug => $class ) {
+		foreach ($this->settings as $slug => $class) {
 
 			/** @var WordPoints_Hook_Reaction_SettingI $setting */
-			$setting = new $class( $slug );
+			$setting = new $class($slug);
 
-			$fields[ $slug ] = array(
+			$fields[$slug] = array(
 				'type' => $setting->get_type(),
 				'label' => $setting->get_label(),
 				'required' => $setting->is_required(),
@@ -97,29 +97,29 @@ 
 		return $fields;
 	}
 
-	public function save_settings( $reaction, $settings, $is_new ) {
+	public function save_settings($reaction, $settings, $is_new) {
 
-		foreach ( $this->settings as $slug => $class ) {
+		foreach ($this->settings as $slug => $class) {
 
 			/** @var WordPoints_Hook_Reaction_SettingI $setting */
-			$setting = new $class( $slug );
+			$setting = new $class($slug);
 
-			$value = isset( $settings[ $slug ] ) ? $settings[ $slug ] : null;
+			$value = isset($settings[$slug]) ? $settings[$slug] : null;
 
-			$setting->save( $value, $reaction, $is_new );
+			$setting->save($value, $reaction, $is_new);
 		}
 	}
 
-	public function validate_settings( $settings, $validator ) {
+	public function validate_settings($settings, $validator) {
 
-		foreach ( $this->settings as $slug => $class ) {
+		foreach ($this->settings as $slug => $class) {
 
 			/** @var WordPoints_Hook_Reaction_SettingI $setting */
-			$setting = new $class( $slug );
+			$setting = new $class($slug);
 
-			$value = isset( $settings[ $slug ] ) ? $settings[ $slug ] : null;
+			$value = isset($settings[$slug]) ? $settings[$slug] : null;
 
-			$settings[ $slug ] = $setting->validate( $value, $validator );
+			$settings[$slug] = $setting->validate($value, $validator);
 		}
 
 		return $settings;
@@ -133,8 +133,8 @@ 
  *
  * @param WordPoints_Hooks $hooks
  */
-function wordpoints_hook_settings_app( $hooks ) {
-	$hooks->sub_apps()->register( 'settings', 'WordPoints_Hook_Settings' );
+function wordpoints_hook_settings_app($hooks) {
+	$hooks->sub_apps()->register('settings', 'WordPoints_Hook_Settings');
 }
 //add_action( 'wordpoints_hooks_init', 'wordpoints_hook_settings_app' );
 
@@ -145,12 +145,12 @@ 
  *
  * @param WordPoints_Hook_Settings $settings
  */
-function wordpoints_register_hook_settings( $settings ) {
+function wordpoints_register_hook_settings($settings) {
 
-	add_action( 'wordpoints_hook_reaction_save', array( $settings, 'save_settings' ), 10, 3 );
-	add_filter( 'wordpoints_hook_reaction_validate', array( $settings, 'validate_settings' ), 10, 2 );
+	add_action('wordpoints_hook_reaction_save', array($settings, 'save_settings'), 10, 3);
+	add_filter('wordpoints_hook_reaction_validate', array($settings, 'validate_settings'), 10, 2);
 
-	$settings->register( 'description', 'WordPoints_Hook_Reaction_Setting_Description' );
+	$settings->register('description', 'WordPoints_Hook_Reaction_Setting_Description');
 }
 //add_action( 'wordpoints_hook_settings_init', 'wordpoints_register_hook_settings' );
 

src/includes/classes/hook/condition/entity/array/contains.php

Spacing +35 added lines, -35 removed lines

@@ -47,7 +47,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->conditions_extension = wordpoints_hooks()->get_sub_app( 'extensions' )->get(
+		$this->conditions_extension = wordpoints_hooks()->get_sub_app('extensions')->get(
 			'conditions'
 		);
 	}
@@ -56,7 +56,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Contains', 'wordpoints' );
+		return __('Contains', 'wordpoints');
 	}
 
 	/**
@@ -66,13 +66,13 @@ 
 		return array(
 			'min' => array(
 				'slug'    => 'min',
-				'label'   => __( 'Minimum number of items', 'wordpoints' ),
+				'label'   => __('Minimum number of items', 'wordpoints'),
 				'type'    => 'number',
 				'default' => 1,
 			),
 			'max' => array(
 				'slug'  => 'max',
-				'label' => __( 'Maximum number of items', 'wordpoints' ),
+				'label' => __('Maximum number of items', 'wordpoints'),
 				'type'  => 'number',
 			),
 		);
@@ -92,8 +92,8 @@ 
 
 		$this->validate_count();
 
-		if ( isset( $settings['conditions'] ) ) {
-			$this->validate_conditions( $arg );
+		if (isset($settings['conditions'])) {
+			$this->validate_conditions($arg);
 		}
 
 		return $this->settings;
@@ -107,31 +107,31 @@ 
 	protected function validate_count() {
 
 		if (
-			! empty( $this->settings['max'] )
-			&& ! wordpoints_posint( $this->settings['max'] )
+			! empty($this->settings['max'])
+			&& ! wordpoints_posint($this->settings['max'])
 		) {
 			$this->validator->add_error(
-				__( 'The maximum must be a positive integer.', 'wordpoints' )
+				__('The maximum must be a positive integer.', 'wordpoints')
 				, 'max'
 			);
 		}
 
-		if ( ! empty( $this->settings['min'] ) ) {
+		if ( ! empty($this->settings['min'])) {
 
-			if ( ! wordpoints_posint( $this->settings['min'] ) ) {
+			if ( ! wordpoints_posint($this->settings['min'])) {
 
 				$this->validator->add_error(
-					__( 'The minimum must be a positive integer.', 'wordpoints' )
+					__('The minimum must be a positive integer.', 'wordpoints')
 					, 'min'
 				);
 
 			} elseif (
-				! empty( $this->settings['max'] )
+				! empty($this->settings['max'])
 				&& $this->settings['max'] < $this->settings['min']
 			) {
 
 				$this->validator->add_error(
-					__( 'The minimum must be less than the maximum.', 'wordpoints' )
+					__('The minimum must be less than the maximum.', 'wordpoints')
 					, 'min'
 				);
 			}
@@ -145,24 +145,24 @@ 
 	 *
 	 * @param WordPoints_EntityishI $current_arg The current arg.
 	 */
-	protected function validate_conditions( $current_arg ) {
+	protected function validate_conditions($current_arg) {
 
-		$args = new WordPoints_Hook_Event_Args( array() );
+		$args = new WordPoints_Hook_Event_Args(array());
 
-		if ( $current_arg instanceof WordPoints_Entity_Array ) {
+		if ($current_arg instanceof WordPoints_Entity_Array) {
 
 			$entity = wordpoints_entities()->get(
 				$current_arg->get_entity_slug()
 			);
 
-			if ( $entity instanceof WordPoints_Entity ) {
-				$args->add_entity( $entity );
+			if ($entity instanceof WordPoints_Entity) {
+				$args->add_entity($entity);
 			}
 		}
 
-		$args->set_validator( $this->validator );
+		$args->set_validator($this->validator);
 
-		$this->validator->push_field( 'conditions' );
+		$this->validator->push_field('conditions');
 
 		$this->settings['conditions'] = $this->conditions_extension->validate_conditions(
 			$this->settings['conditions']
@@ -175,7 +175,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args) {
 
 		$this->settings = $settings;
 
@@ -183,16 +183,16 @@ 
 
 		$entities = array();
 
-		if ( $arg instanceof WordPoints_Entity_Array ) {
+		if ($arg instanceof WordPoints_Entity_Array) {
 
 			$entities = $arg->get_the_entities();
 
-			if ( isset( $this->settings['conditions'] ) ) {
-				$entities = $this->filter_entities( $entities );
+			if (isset($this->settings['conditions'])) {
+				$entities = $this->filter_entities($entities);
 			}
 		}
 
-		return $this->check_count( count( $entities ) );
+		return $this->check_count(count($entities));
 	}
 
 	/**
@@ -204,20 +204,20 @@ 
 	 *
 	 * @return WordPoints_Entity[] The entities that matched the sub-conditions.
 	 */
-	protected function filter_entities( $entities ) {
+	protected function filter_entities($entities) {
 
-		foreach ( $entities as $index => $entity ) {
+		foreach ($entities as $index => $entity) {
 
-			$event_args = new WordPoints_Hook_Event_Args( array() );
-			$event_args->add_entity( $entity );
+			$event_args = new WordPoints_Hook_Event_Args(array());
+			$event_args->add_entity($entity);
 
 			$matches = $this->conditions_extension->conditions_are_met(
 				$this->settings['conditions']
 				, $event_args
 			);
 
-			if ( ! $matches ) {
-				unset( $entities[ $index ] );
+			if ( ! $matches) {
+				unset($entities[$index]);
 			}
 		}
 
@@ -233,13 +233,13 @@ 
 	 *
 	 * @return bool Whether the count met the requirements.
 	 */
-	protected function check_count( $count ) {
+	protected function check_count($count) {
 
-		if ( isset( $this->settings['max'] ) && $count > $this->settings['max'] ) {
+		if (isset($this->settings['max']) && $count > $this->settings['max']) {
 			return false;
 		}
 
-		if ( isset( $this->settings['min'] ) && $count < $this->settings['min'] ) {
+		if (isset($this->settings['min']) && $count < $this->settings['min']) {
 			return false;
 		}
 

src/includes/classes/hook/retroactive/query.php

Spacing +155 added lines, -155 removed lines

@@ -73,14 +73,14 @@ 
 
 
 interface WordPoints_Hook_Reactor_RetroactiveI extends WordPoints_Hook_Retroactive_Query_ModifierI {
-	public function retroactive_hit( $target, array $records, WordPoints_Hook_Event_RetroactiveI $event, WordPoints_Hook_ReactionI $reaction );
+	public function retroactive_hit($target, array $records, WordPoints_Hook_Event_RetroactiveI $event, WordPoints_Hook_ReactionI $reaction);
 }
 
 abstract class WordPoints_Hook_Reactor_Retroactive implements WordPoints_Hook_Reactor_RetroactiveI {
 
-	public function modify_retroactive_query( WordPoints_Hook_Retroactive_QueryI $query ) {
+	public function modify_retroactive_query(WordPoints_Hook_Retroactive_QueryI $query) {
 
-		$query->set_target( $query->get_reaction()->get_meta( 'target' ) );
+		$query->set_target($query->get_reaction()->get_meta('target'));
 	}
 
 	public function reverse_hits() {
@@ -97,17 +97,17 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function retroactive_hit( $target, array $records, WordPoints_Hook_Event_RetroactiveI $event, WordPoints_Hook_ReactionI $reaction ) {
+	public function retroactive_hit($target, array $records, WordPoints_Hook_Event_RetroactiveI $event, WordPoints_Hook_ReactionI $reaction) {
 
-		$count = count( $records );
+		$count = count($records);
 
 		wordpoints_alter_points(
 			$target
-			, $reaction->get_meta( 'points' ) * $count
-			, $reaction->get_meta( 'points_type' )
-			, 'retroactive_' . $reaction->get_event_slug()
-			, array( 'count' => $count )
-			, $event->get_retroactive_description( $reaction )
+			, $reaction->get_meta('points') * $count
+			, $reaction->get_meta('points_type')
+			, 'retroactive_'.$reaction->get_event_slug()
+			, array('count' => $count)
+			, $event->get_retroactive_description($reaction)
 		);
 	}
 
@@ -139,22 +139,22 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
 			wordpoints_alter_points(
 				$log->user_id
-				, - ( $log->points / wordpoints_get_points_log_meta( $log->id, 'count', true ) )
+				, -($log->points / wordpoints_get_points_log_meta($log->id, 'count', true))
 				, $log->points_type
 				, "reverse_{$event}"
-				, array( 'original_log_id' => $log->id )
+				, array('original_log_id' => $log->id)
 				, $args->get_description()
 			);
 
-			wordpoints_add_points_log_meta( $log->id, 'auto_reversed', $args->get_entity_id() );
+			wordpoints_add_points_log_meta($log->id, 'auto_reversed', $args->get_entity_id());
 		}
 	}
 }
@@ -196,13 +196,13 @@ 
 
 	protected $results;
 
-	public function __construct( WordPoints_Hook_ReactionI $reaction ) {
+	public function __construct(WordPoints_Hook_ReactionI $reaction) {
 
 		$this->reaction = $reaction;
-		$this->arg_hierarchy = new WordPoints_Hierarchy( 'sub_args' );
+		$this->arg_hierarchy = new WordPoints_Hierarchy('sub_args');
 		$this->hooks = wordpoints_hooks();
 		$this->entities = wordpoints_entities();
-		$this->validator = new WordPoints_Hook_Reaction_Validator( $reaction, true );
+		$this->validator = new WordPoints_Hook_Reaction_Validator($reaction, true);
 	}
 
 	public function get_reaction() {
@@ -215,11 +215,11 @@ 
 
 	public function get_results() {
 
-		if ( ! isset( $this->results ) ) {
+		if ( ! isset($this->results)) {
 
 			$this->execute();
 
-			if ( $this->validator->had_errors() ) {
+			if ($this->validator->had_errors()) {
 				$this->results = $this->validator;
 			}
 		}
@@ -236,8 +236,8 @@ 
 			$this->filter_results();
 			$this->group_results();
 
-		} catch ( WordPoints_Hook_Validator_Exception $e ) {
-			unset( $e );
+		} catch (WordPoints_Hook_Validator_Exception $e) {
+			unset($e);
 		}
 	}
 
@@ -245,33 +245,33 @@ 
 
 		$event_slug = $this->reaction->get_event_slug();
 
-		$events = $this->hooks->get_sub_app( 'events' );
-		$event  = $events->get( $event_slug );
+		$events = $this->hooks->get_sub_app('events');
+		$event  = $events->get($event_slug);
 
-		if ( ! ( $event instanceof WordPoints_Hook_Event_RetroactiveI ) ) {
-			$this->validator->add_error( 'invalid hook' );
+		if ( ! ($event instanceof WordPoints_Hook_Event_RetroactiveI)) {
+			$this->validator->add_error('invalid hook');
 		}
 
-		foreach ( $events->get_sub_app( 'args' )->get_children( $event_slug ) as $arg ) {
+		foreach ($events->get_sub_app('args')->get_children($event_slug) as $arg) {
 			$this->arg_hierarchy_push( // TODO
 				$arg
 			);
 		}
 
-		if ( $event instanceof WordPoints_Hook_Retroactive_Query_ModifierI ) {
-			$event->modify_retroactive_query( $this );
+		if ($event instanceof WordPoints_Hook_Retroactive_Query_ModifierI) {
+			$event->modify_retroactive_query($this);
 			$this->reset();
 		}
 
-		$reactor = $this->hooks->get_sub_app( 'reactors' )->get( $this->reaction->get_reactor_slug() );
+		$reactor = $this->hooks->get_sub_app('reactors')->get($this->reaction->get_reactor_slug());
 
-		$reactor->modify_retroactive_query( $this );
+		$reactor->modify_retroactive_query($this);
 		$this->reset();
 
-		foreach ( $this->hooks->get_sub_app( 'extensions' )->get_all() as $extension ) {
+		foreach ($this->hooks->get_sub_app('extensions')->get_all() as $extension) {
 
-			if ( $extension instanceof WordPoints_Hook_Retroactive_Query_ModifierI ) {
-				$extension->modify_retroactive_query( $this );
+			if ($extension instanceof WordPoints_Hook_Retroactive_Query_ModifierI) {
+				$extension->modify_retroactive_query($this);
 				$this->reset();
 			}
 		}
@@ -279,126 +279,126 @@ 
 
 	protected function perform_query() {
 
-		$this->queries = new WordPoints_Hierarchy( 'sub_queries' );
-		$this->consolidate_queries( array( $this->arg_hierarchy->get() ) );
-		unset( $this->query );
+		$this->queries = new WordPoints_Hierarchy('sub_queries');
+		$this->consolidate_queries(array($this->arg_hierarchy->get()));
+		unset($this->query);
 
 		$this->queries->reset();
 
 		// Find the tip of a query.
-		$this->results = $this->execute_queries( $this->queries->get() );
+		$this->results = $this->execute_queries($this->queries->get());
 
-		if ( is_wp_error( $this->results ) ) {
-			$this->validator->add_error( $this->results );
+		if (is_wp_error($this->results)) {
+			$this->validator->add_error($this->results);
 		}
 	}
 
-	protected function consolidate_queries( $arg_hierarchy, $storage_type = null ) {
+	protected function consolidate_queries($arg_hierarchy, $storage_type = null) {
 
-		foreach ( $arg_hierarchy as $data ) {
-			$this->consolidate_query( $data );
+		foreach ($arg_hierarchy as $data) {
+			$this->consolidate_query($data);
 		}
 	}
 
-	protected function consolidate_query( $data ) {
+	protected function consolidate_query($data) {
 
-		if ( $data['storage_info']['type'] !== $this->queries->get_field( 'slug' ) ) {
+		if ($data['storage_info']['type'] !== $this->queries->get_field('slug')) {
 
 			$parent_id = null;
-			if ( isset( $this->query ) ) {
+			if (isset($this->query)) {
 				$parent_id = $this->query->get_id();
 			}
 
-			$this->query = new WordPoints_Hierarchy( 'sub_args' );
+			$this->query = new WordPoints_Hierarchy('sub_args');
 
 			$this->queries->push(
 				$data['storage_info']['type']
-				, array( 'query' => $this->query, 'parent_id' => $parent_id )
+				, array('query' => $this->query, 'parent_id' => $parent_id)
 			);
 
 			$pushed_query = true;
 		}
 
-		if ( isset( $data['sub_args'] ) ) {
+		if (isset($data['sub_args'])) {
 			$sub_args = $data['sub_args'];
-			unset( $data['sub_args'] );
+			unset($data['sub_args']);
 		}
 
-		$this->query->push( $data['slug'], $data );
+		$this->query->push($data['slug'], $data);
 
-		if ( isset( $sub_args ) ) {
-			$this->consolidate_queries( $sub_args, $data['storage_info']['type'] );
+		if (isset($sub_args)) {
+			$this->consolidate_queries($sub_args, $data['storage_info']['type']);
 		}
 
 		$this->query->pop();
 
-		if ( ! empty( $pushed_query ) ) {
+		if ( ! empty($pushed_query)) {
 			$this->queries->pop();
-			$this->query = $this->queries->get_field( 'query' );
+			$this->query = $this->queries->get_field('query');
 		}
 	}
 
-	protected function execute_queries( $queries ) {
+	protected function execute_queries($queries) {
 
 		/** @var WordPoints_HierarchyI $query */
 		$query = $queries['query'];
 
-		if ( isset( $queries['sub_queries'] ) ) {
-			foreach ( $queries['sub_queries'] as $query_data ) {
+		if (isset($queries['sub_queries'])) {
+			foreach ($queries['sub_queries'] as $query_data) {
 
-				$results = $this->execute_queries( $query_data );
+				$results = $this->execute_queries($query_data);
 
-				if ( is_wp_error( $results ) ) {
+				if (is_wp_error($results)) {
 					return $results;
 				}
 
-				if ( empty( $results ) ) {
+				if (empty($results)) {
 					return array();
 				}
 
 				/** @var WordPoints_HierarchyI $child_query */
 				$child_query = $query_data['query'];
-				$query->go_to( $query_data['parent_id'] );
+				$query->go_to($query_data['parent_id']);
 
 				$condition = array(
 //					'field' => $child_query->get_field( 'slug' ),
 					'compare' => 'in',
 					'value' => wp_list_pluck(
 						$results
-						, $child_query->get_field( 'storage_info', 'meta', 'id_field' )
+						, $child_query->get_field('storage_info', 'meta', 'id_field')
 					),
 				);
 
-				$query->push_to( 'conditions', $condition );
+				$query->push_to('conditions', $condition);
 			}
 		}
 
 		$query->reset();
-		$storage_type = $query->get_field( 'storage_info', 'type' );
+		$storage_type = $query->get_field('storage_info', 'type');
 
 		// it is pretty stupid to wait until now to figure this out.
-		$executor = $this->hooks->get_sub_app( 'retroactive_query_executors' )->get( $storage_type );
+		$executor = $this->hooks->get_sub_app('retroactive_query_executors')->get($storage_type);
 
-		if ( ! $executor ) {
+		if ( ! $executor) {
 			$this->validator->add_error(
-				sprintf( 'unknown storage type "%s".', $storage_type )
+				sprintf('unknown storage type "%s".', $storage_type)
 			);
 		}
 
-		return $executor->execute( array( $query->get() ) );
+		return $executor->execute(array($query->get()));
 	}
 
 	protected function filter_results() {
 
-		$reactor = $this->hooks->get_sub_app( 'reactors' )->get( $this->reaction->get_reactor_slug() );
+		$reactor = $this->hooks->get_sub_app('reactors')->get($this->reaction->get_reactor_slug());
 
-		if ( $reactor instanceof WordPoints_Hook_Retroactive_Query_FilterI ) {
-			$reactor->filter_retroactive_query( $this );
+		if ($reactor instanceof WordPoints_Hook_Retroactive_Query_FilterI) {
+			$reactor->filter_retroactive_query($this);
 		}
 
-		foreach ( $this->hooks->get_sub_app( 'extensions' )->get_all() as $extension ) {
-			if ( $extension instanceof WordPoints_Hook_Retroactive_Query_FilterI ) {
-				$extension->filter_retroactive_query( $this );
+		foreach ($this->hooks->get_sub_app('extensions')->get_all() as $extension) {
+			if ($extension instanceof WordPoints_Hook_Retroactive_Query_FilterI) {
+				$extension->filter_retroactive_query($this);
 			}
 		}
 	}
@@ -407,27 +407,27 @@ 
 
 		$grouped_results = array();
 
-		foreach ( $this->results as $result ) {
-			$grouped_results[ $result->target ][] = $result;
+		foreach ($this->results as $result) {
+			$grouped_results[$result->target][] = $result;
 		}
 
 		$this->results = $grouped_results;
 	}
 
-	public function add_condition( array $condition ) {
+	public function add_condition(array $condition) {
 
-		$this->arg_hierarchy->push_to( 'conditions', $condition );
+		$this->arg_hierarchy->push_to('conditions', $condition);
 	}
 
-	public function select_value( $data = array() ) {
+	public function select_value($data = array()) {
 
-		$this->arg_hierarchy->set_field( 'select', $data );
+		$this->arg_hierarchy->set_field('select', $data);
 	}
 
-	public function set_target( $target_arg ) {
+	public function set_target($target_arg) {
 
-		foreach ( $target_arg as $arg_slug ) {
-			$this->arg_hierarchy_push( $arg_slug );
+		foreach ($target_arg as $arg_slug) {
+			$this->arg_hierarchy_push($arg_slug);
 		}
 
 //		$id = $this->arg_hierarchy->get_id();
@@ -437,10 +437,10 @@ 
 //			$field = $this->arg_hierarchy->get_field( 'storage_info', 'meta', 'field' );
 //			$this->arg_hierarchy->ascend();
 //		} else {
-			$field = $this->arg_hierarchy->get_field( 'storage_info', 'meta', 'id_field' );
+			$field = $this->arg_hierarchy->get_field('storage_info', 'meta', 'id_field');
 //		}
 
-		$this->select_value( array( 'field' => $field, 'as' => 'target' ) );
+		$this->select_value(array('field' => $field, 'as' => 'target'));
 
 //		$this->arg_hierarchy->go_to( $id );
 	}
@@ -452,40 +452,40 @@ 
 	 * @return WordPoints_EntityishI
 	 */
 	public function get_arg() {
-		return $this->arg_hierarchy->get_field( 'arg' );
+		return $this->arg_hierarchy->get_field('arg');
 	}
 
-	public function arg_hierarchy_push( $slug ) {
+	public function arg_hierarchy_push($slug) {
 
-		$current_slug = $this->arg_hierarchy->get_field( 'slug' );
+		$current_slug = $this->arg_hierarchy->get_field('slug');
 
-		if ( $current_slug === $slug && $this->arg_hierarchy->is_main() ) {
+		if ($current_slug === $slug && $this->arg_hierarchy->is_main()) {
 			return;
 		}
 
 //		if ( $current_slug !== $slug ) {
 
-			if ( empty( $current_slug ) ) {
-				$arg = $this->entities->get( $slug );
+			if (empty($current_slug)) {
+				$arg = $this->entities->get($slug);
 			} else {
 
 				// If this child exists, don't overwrite it, just descend into it.
-				if ( $this->arg_hierarchy->has_child( $slug ) ) {
-					$this->arg_hierarchy->descend( $slug );
+				if ($this->arg_hierarchy->has_child($slug)) {
+					$this->arg_hierarchy->descend($slug);
 					return;
 				}
 
-				$parent_arg = $this->arg_hierarchy->get_field( 'arg' );
+				$parent_arg = $this->arg_hierarchy->get_field('arg');
 
-				if ( $parent_arg instanceof WordPoints_Entity_ParentI ) {
-					$arg = $parent_arg->get_child( $slug );
+				if ($parent_arg instanceof WordPoints_Entity_ParentI) {
+					$arg = $parent_arg->get_child($slug);
 				} else {
 					return; // TODO
 				}
 			}
 
-			if ( $arg instanceof WordPoints_Entity_Array ) {
-				$arg = $this->entities->get( $arg->get_entity_slug() );
+			if ($arg instanceof WordPoints_Entity_Array) {
+				$arg = $this->entities->get($arg->get_entity_slug());
 			}
 
 			$data['arg'] = $arg;
@@ -493,7 +493,7 @@ 
 			// TODO check if storage type is recognized?
 			$data['storage_info'] = $arg->get_storage_info();
 
-			$this->arg_hierarchy->push( $slug, $data );
+			$this->arg_hierarchy->push($slug, $data);
 
 //		} else {
 //			$data = array();
@@ -510,11 +510,11 @@ 
 }
 
 interface WordPoints_Hook_Retroactive_Query_FilterI {
-	public function filter_retroactive_query( WordPoints_Hook_Retroactive_QueryI $query );
+	public function filter_retroactive_query(WordPoints_Hook_Retroactive_QueryI $query);
 }
 
 interface WordPoints_Hook_Retroactive_Query_ExecutorI {
-	public function execute( $query );
+	public function execute($query);
 }
 
 /*
@@ -543,48 +543,48 @@ 
 	protected $array;
 	protected $results;
 
-	public function execute( $query ) {
+	public function execute($query) {
 
-		foreach ( $query as $arg_data ) {
+		foreach ($query as $arg_data) {
 
 			$meta = $arg_data['storage_info']['meta'];
 
-			if ( isset( $meta['getter'] ) ) {
+			if (isset($meta['getter'])) {
 				$this->array = $meta['getter']();
 			}
 
-			if ( isset( $arg_data['conditions'] ) ) {
+			if (isset($arg_data['conditions'])) {
 
-				foreach ( $arg_data['conditions'] as $condition ) {
+				foreach ($arg_data['conditions'] as $condition) {
 
-					if ( ! isset( $condition['field'] ) ) {
-						if ( $arg_data['arg'] instanceof WordPoints_Entity_Attr ) {
+					if ( ! isset($condition['field'])) {
+						if ($arg_data['arg'] instanceof WordPoints_Entity_Attr) {
 							$condition['field'] = $meta['field'];
 						} else {
 							$condition['field'] = $meta['id_field'];
 						}
 					}
 
-					if ( ! isset( $condition['condition'] ) ) {
+					if ( ! isset($condition['condition'])) {
 						$condition['condition'] = '=';
 					}
 
 					$filter_args = array();
 
-					switch ( $condition['condition'] ) {
+					switch ($condition['condition']) {
 
 						case '=':
-							$filter_args[ $condition['field'] ] = $condition['value'];
+							$filter_args[$condition['field']] = $condition['value'];
 							break;
 
 						case 'in':
 							// TODO
 
 						default:
-							return new WP_Error( 'invalid condition type' );
+							return new WP_Error('invalid condition type');
 					}
 
-					$this->array = wp_list_filter( $this->array, $filter_args );
+					$this->array = wp_list_filter($this->array, $filter_args);
 				}
 			}
 
@@ -592,9 +592,9 @@ 
 //				$builder->add_field( $arg_data['select'] );
 //			}
 
-			if ( isset( $arg_data['sub_args'] ) ) {
+			if (isset($arg_data['sub_args'])) {
 				$this->parent_data = $arg_data;
-				$this->array = $this->execute( $arg_data['sub_args'] );
+				$this->array = $this->execute($arg_data['sub_args']);
 			}
 
 			$this->results = $this->array;
@@ -623,7 +623,7 @@ 
 
 	protected $entered_join;
 
-	public function execute( $query ) {
+	public function execute($query) {
 		// WE should be getting a hierarchy object here.
 		// This will let us loop over things easier.
 		// We'll also need to be marking the non-usable parts of the query.
@@ -633,69 +633,69 @@ 
 
 		$this->builder = new WordPoints_Query_Builder_DB_MySQL();
 
-		$this->build_query( $query );
+		$this->build_query($query);
 
 		$sql = $this->builder->get_query();
 
-		if ( is_wp_error( $sql ) ) {
+		if (is_wp_error($sql)) {
 			return $sql;
 		}
 
-		return $wpdb->get_results( $sql );
+		return $wpdb->get_results($sql);
 	}
 
-	protected function build_query( $args ) {
+	protected function build_query($args) {
 
-		foreach ( $args as $arg_data ) {
+		foreach ($args as $arg_data) {
 
 			$this->arg_data = $arg_data;
 
-			if ( isset( $arg_data['storage_info']['meta']['table_name'] ) ) {
-				$this->build_table_schema( $arg_data['storage_info']['meta'] );
+			if (isset($arg_data['storage_info']['meta']['table_name'])) {
+				$this->build_table_schema($arg_data['storage_info']['meta']);
 			}
 
-			if ( isset( $arg_data['conditions'] ) ) {
-				$this->build_conditions( $arg_data['conditions'] );
+			if (isset($arg_data['conditions'])) {
+				$this->build_conditions($arg_data['conditions']);
 			}
 
-			if ( isset( $arg_data['select'] ) ) {
-				$this->builder->add_field( $arg_data['select'] );
+			if (isset($arg_data['select'])) {
+				$this->builder->add_field($arg_data['select']);
 			}
 
-			if ( isset( $arg_data['sub_args'] ) ) {
+			if (isset($arg_data['sub_args'])) {
 
 				$this->grandparent_data = $this->parent_data;
 				$this->parent_data = $arg_data;
 
-				$this->build_query( $arg_data['sub_args'] );
+				$this->build_query($arg_data['sub_args']);
 
 				$this->parent_data = $this->grandparent_data;
 			}
 
-			if ( isset( $this->entered_join ) ) {
+			if (isset($this->entered_join)) {
 				$this->builder->exit_join();
 				$this->entered_join = null;
 			}
 		}
 	}
 
-	protected function build_table_schema( $db ) {
+	protected function build_table_schema($db) {
 
-		if ( $this->builder->get_table() ) {
+		if ($this->builder->get_table()) {
 
-			if ( $this->parent_data['arg'] instanceof WordPoints_Entity_Relationship ) {
+			if ($this->parent_data['arg'] instanceof WordPoints_Entity_Relationship) {
 				$primary_field = $this->parent_data['storage_info']['meta']['field'];
 				$join_field = $db['id_field'];
 				//$primary_field = $this->arg_data['arg']->get_secondary_field();
-			} elseif ( $this->arg_data['arg'] instanceof WordPoints_Entity_Relationship ) {
+			} elseif ($this->arg_data['arg'] instanceof WordPoints_Entity_Relationship) {
 				$join_field = $db['join_field'];
 				$primary_field = $this->parent_data['storage_info']['meta']['id_field'];
 			} else {
-				throw new WordPoints_Query_Builder_Exception( 'Houston, we have a problem.' );
+				throw new WordPoints_Query_Builder_Exception('Houston, we have a problem.');
 			}
 
-			if ( is_array( $primary_field ) && isset( $primary_field['table_name'] ) ) {
-				$this->builder->enter_join( $primary_field );
+			if (is_array($primary_field) && isset($primary_field['table_name'])) {
+				$this->builder->enter_join($primary_field);
 				$primary_field = $primary_field['on']['join_field'];
 			}
 
@@ -707,26 +707,26 @@ 
 				),
 			);
 
-			if ( isset( $db['join_where'] ) ) {
+			if (isset($db['join_where'])) {
 				$join['where'] = $db['join_where'];
 			}
 
-			$this->builder->enter_join( $join );
+			$this->builder->enter_join($join);
 
 			$this->entered_join = true;
 
 		} else {
-			$this->builder->set_table( $db['table_name'] );
-			$this->builder->add_field( $db['id_field'] );
+			$this->builder->set_table($db['table_name']);
+			$this->builder->add_field($db['id_field']);
 		}
 	}
 
-	protected function build_conditions( $conditions ) {
+	protected function build_conditions($conditions) {
 
 		// Join conditions should be pushed to the end.
 		$join_conditions = array();
 
-		foreach ( $conditions as $condition ) {
+		foreach ($conditions as $condition) {
 
 			// THis needs to be the identifier field for the type of arg that this
 			// field represents.
@@ -734,43 +734,43 @@ 
 			// However, for a relationship it could be something else.
 			// Ultimately, this will have to be left up to the arg object to
 			// decide.
-			if ( ! isset( $condition['field'] ) ) {
-				if ( $this->arg_data['arg'] instanceof WordPoints_Entity_Relationship ) {
+			if ( ! isset($condition['field'])) {
+				if ($this->arg_data['arg'] instanceof WordPoints_Entity_Relationship) {
 //					$condition['field'] = $this->arg_data['arg']->get_secondary_field();
 					$condition['field'] = $this->arg_data['storage_info']['meta']['field'];
 
-					if ( is_array( $condition['field'] && isset( $condition['field']['table_name'] ) ) ) {
+					if (is_array($condition['field'] && isset($condition['field']['table_name']))) {
 						$join_conditions[] = $condition;
 						continue;
 					}
-				} elseif ( $this->arg_data['arg'] instanceof WordPoints_Entity_Attr ) {
+				} elseif ($this->arg_data['arg'] instanceof WordPoints_Entity_Attr) {
 					$condition['field'] = $this->arg_data['arg']->get_field();
 				} else {
-					var_dump($this->arg_data['arg']);exit;
+					var_dump($this->arg_data['arg']); exit;
 				}
 			}
 
-			$this->builder->where( $condition );
+			$this->builder->where($condition);
 		}
 
-		$join_count = count( $join_conditions );
+		$join_count = count($join_conditions);
 		$i = 0;
 
-		foreach ( $join_conditions as $condition ) {
+		foreach ($join_conditions as $condition) {
 
 			$i++;
 
-			$this->builder->enter_join( $condition['field'] );
+			$this->builder->enter_join($condition['field']);
 
 			$condition['field'] = $condition['field']['on']['primary_field'];
 
-			$this->builder->where( $condition );
+			$this->builder->where($condition);
 
 			$this->builder->exit_join();
 
 			// If there are more join conditions, we need to leave a fresh, trailing
 			// join for the next one to join to.
-			if ( $i < $join_count ) {
+			if ($i < $join_count) {
 
 				// There is a current join, but it is already being used, so we
 				// need to exit it.

src/includes/classes/hook/events.php

Spacing +23 added lines, -23 removed lines

@@ -35,13 +35,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 
 		$hooks = wordpoints_hooks();
 
-		$this->router = $hooks->get_sub_app( 'router' );
+		$this->router = $hooks->get_sub_app('router');
 
-		parent::__construct( $slug );
+		parent::__construct($slug);
 	}
 
 	/**
@@ -49,7 +49,7 @@ 
 	 */
 	public function init() {
 
-		$this->sub_apps()->register( 'args', 'WordPoints_Class_Registry_Children' );
+		$this->sub_apps()->register('args', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -73,27 +73,27 @@ 
 	 *
 	 * @return bool Whether the event was registered.
 	 */
-	public function register( $slug, $class, array $args = array() ) {
+	public function register($slug, $class, array $args = array()) {
 
-		parent::register( $slug, $class, $args );
+		parent::register($slug, $class, $args);
 
-		if ( isset( $args['actions'] ) ) {
-			foreach ( $args['actions'] as $type => $actions ) {
-				foreach ( (array) $actions as $action_slug ) {
-					$this->router->add_event_to_action( $slug, $action_slug, $type );
+		if (isset($args['actions'])) {
+			foreach ($args['actions'] as $type => $actions) {
+				foreach ((array) $actions as $action_slug) {
+					$this->router->add_event_to_action($slug, $action_slug, $type);
 				}
 			}
 		}
 
-		if ( isset( $args['args'] ) ) {
-			$args_registry = $this->get_sub_app( 'args' );
+		if (isset($args['args'])) {
+			$args_registry = $this->get_sub_app('args');
 
-			foreach ( $args['args'] as $arg_slug => $class ) {
-				$args_registry->register( $slug, $arg_slug, $class );
+			foreach ($args['args'] as $arg_slug => $class) {
+				$args_registry->register($slug, $arg_slug, $class);
 			}
 		}
 
-		$this->event_data[ $slug ] = $args;
+		$this->event_data[$slug] = $args;
 
 		return true;
 	}
@@ -101,23 +101,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->get_sub_app( 'args' )->deregister_children( $slug );
+		$this->get_sub_app('args')->deregister_children($slug);
 
-		unset( $this->event_data[ $slug ] );
+		unset($this->event_data[$slug]);
 	}
 }
 

src/includes/classes/entity.php

Spacing +36 added lines, -36 removed lines

@@ -106,11 +106,11 @@ 
 	 *
 	 * @return mixed The entity, or false if not found.
 	 */
-	protected function get_entity( $id ) {
+	protected function get_entity($id) {
 
-		$entity = call_user_func( $this->getter, $id );
+		$entity = call_user_func($this->getter, $id);
 
-		if ( ! $this->is_entity( $entity ) ) {
+		if ( ! $this->is_entity($entity)) {
 			return false;
 		}
 
@@ -126,13 +126,13 @@ 
 	 *
 	 * @return bool Whether the passed value is an entity.
 	 */
-	protected function is_entity( $entity ) {
+	protected function is_entity($entity) {
 
-		if ( ! is_object( $entity ) && ! is_array( $entity ) ) {
+		if ( ! is_object($entity) && ! is_array($entity)) {
 			return false;
 		}
 
-		return (bool) $this->get_entity_id( $entity );
+		return (bool) $this->get_entity_id($entity);
 	}
 
 	/**
@@ -145,14 +145,14 @@ 
 	 *
 	 * @return mixed The value of the attribute of the entity.
 	 */
-	protected function get_attr_value( $entity, $attr ) {
+	protected function get_attr_value($entity, $attr) {
 
-		if ( is_array( $entity ) ) {
-			if ( isset( $entity[ $attr ] ) ) {
-				return $entity[ $attr ];
+		if (is_array($entity)) {
+			if (isset($entity[$attr])) {
+				return $entity[$attr];
 			}
 		} else {
-			if ( isset( $entity->{$attr} ) ) {
+			if (isset($entity->{$attr} )) {
 				return $entity->{$attr};
 			}
 		}
@@ -169,8 +169,8 @@ 
 	 *
 	 * @return mixed The ID of the entity.
 	 */
-	protected function get_entity_id( $entity ) {
-		return $this->get_attr_value( $entity, $this->get_id_field() );
+	protected function get_entity_id($entity) {
+		return $this->get_attr_value($entity, $this->get_id_field());
 	}
 
 	/**
@@ -186,8 +186,8 @@ 
 	 *
 	 * @return mixed The human ID of the entity.
 	 */
-	protected function get_entity_human_id( $entity ) {
-		return $this->get_attr_value( $entity, $this->human_id_field );
+	protected function get_entity_human_id($entity) {
+		return $this->get_attr_value($entity, $this->human_id_field);
 	}
 
 	//
@@ -229,15 +229,15 @@ 
 	 *
 	 * @return string|int|float|false The human identifier for the entity, or false.
 	 */
-	public function get_human_id( $id ) {
+	public function get_human_id($id) {
 
-		$entity = $this->get_entity( $id );
+		$entity = $this->get_entity($id);
 
-		if ( ! $entity ) {
+		if ( ! $entity) {
 			return false;
 		}
 
-		return $this->get_entity_human_id( $entity );
+		return $this->get_entity_human_id($entity);
 	}
 
 	/**
@@ -249,8 +249,8 @@ 
 	 *
 	 * @return bool Whether or not an entity with that ID exists.
 	 */
-	public function exists( $id ) {
-		return (bool) $this->get_entity( $id );
+	public function exists($id) {
+		return (bool) $this->get_entity($id);
 	}
 
 	/**
@@ -265,17 +265,17 @@ 
 	 *
 	 * @return WordPoints_Entityish|false The child's object, or false if not found.
 	 */
-	public function get_child( $child_slug ) {
+	public function get_child($child_slug) {
 
-		$children = wordpoints_entities()->get_sub_app( 'children' );
+		$children = wordpoints_entities()->get_sub_app('children');
 
-		$child = $children->get( $this->slug, $child_slug );
+		$child = $children->get($this->slug, $child_slug);
 
 		if (
-			isset( $this->the_value )
+			isset($this->the_value)
 			&& $child instanceof WordPoints_Entity_ChildI
 		) {
-			$child->set_the_value_from_entity( $this );
+			$child->set_the_value_from_entity($this);
 		}
 
 		return $child;
@@ -297,20 +297,20 @@ 
 	 *
 	 * @return bool Whether the value was set.
 	 */
-	public function set_the_value( $value ) {
+	public function set_the_value($value) {
 
 		$this->the_value = $this->the_entity = $this->the_context = null;
 
-		if ( $this->is_entity( $value ) ) {
+		if ($this->is_entity($value)) {
 
 			$entity = $value;
-			$value = $this->get_entity_id( $value );
+			$value = $this->get_entity_id($value);
 
 		} else {
 
-			$entity = $this->get_entity( $value );
+			$entity = $this->get_entity($value);
 
-			if ( ! $entity ) {
+			if ( ! $entity) {
 				return false;
 			}
 		}
@@ -333,8 +333,8 @@ 
 	 *
 	 * @return mixed The value of the attribute.
 	 */
-	public function get_the_attr_value( $attr ) {
-		return $this->get_attr_value( $this->the_entity, $attr );
+	public function get_the_attr_value($attr) {
+		return $this->get_attr_value($this->the_entity, $attr);
 	}
 
 	/**
@@ -358,7 +358,7 @@ 
 	 * @return string|int|float|null The human identifier for the entity, or null.
 	 */
 	public function get_the_human_id() {
-		return $this->get_entity_human_id( $this->the_entity );
+		return $this->get_entity_human_id($this->the_entity);
 	}
 
 	/**
@@ -392,11 +392,11 @@ 
 
 		$guid = $this->get_the_context();
 
-		if ( ! is_array( $guid ) ) {
+		if ( ! is_array($guid)) {
 			return $guid;
 		}
 
-		$guid = array( $this->slug => $this->get_the_id() ) + $guid;
+		$guid = array($this->slug => $this->get_the_id()) + $guid;
 
 		return $guid;
 	}

src/includes/functions.php

Spacing +135 added lines, -135 removed lines

@@ -21,7 +21,7 @@ 
 	// Just accessing this causes it to be initialized. We need to do that so
 	// the actions will be registered and hooked up. The rest of the API can be
 	// lazy-loaded as it is needed.
-	$hooks->get_sub_app( 'actions' );
+	$hooks->get_sub_app('actions');
 }
 
 /**
@@ -33,10 +33,10 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $reactors The reactors registry.
  */
-function wordpoints_hook_reactors_init( $reactors ) {
+function wordpoints_hook_reactors_init($reactors) {
 
-	$reactors->register( 'points', 'WordPoints_Hook_Reactor_Points' );
-	$reactors->register( 'points_legacy', 'WordPoints_Hook_Reactor_Points_Legacy' );
+	$reactors->register('points', 'WordPoints_Hook_Reactor_Points');
+	$reactors->register('points_legacy', 'WordPoints_Hook_Reactor_Points_Legacy');
 }
 
 /**
@@ -48,7 +48,7 @@ 
  *
  * @param WordPoints_Class_Registry_Children $reaction_stores The store registry.
  */
-function wordpoints_hook_reaction_stores_init( $reaction_stores ) {
+function wordpoints_hook_reaction_stores_init($reaction_stores) {
 
 	$reaction_stores->register(
 		'standard'
@@ -56,7 +56,7 @@ 
 		, 'WordPoints_Hook_Reaction_Store_Options'
 	);
 
-	if ( is_wordpoints_network_active() ) {
+	if (is_wordpoints_network_active()) {
 		$reaction_stores->register(
 			'network'
 			, 'points'
@@ -74,14 +74,14 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $extensions The extension registry.
  */
-function wordpoints_hook_extension_init( $extensions ) {
-
-	$extensions->register( 'blocker', 'WordPoints_Hook_Extension_Blocker' );
-	$extensions->register( 'repeat_blocker', 'WordPoints_Hook_Extension_Repeat_Blocker' );
-	$extensions->register( 'reversals', 'WordPoints_Hook_Extension_Reversals' );
-	$extensions->register( 'reversals_legacy_points', 'WordPoints_Hook_Extension_Reversals_Legacy_Points' );
-	$extensions->register( 'conditions', 'WordPoints_Hook_Extension_Conditions' );
-	$extensions->register( 'periods', 'WordPoints_Hook_Extension_Periods' );
+function wordpoints_hook_extension_init($extensions) {
+
+	$extensions->register('blocker', 'WordPoints_Hook_Extension_Blocker');
+	$extensions->register('repeat_blocker', 'WordPoints_Hook_Extension_Repeat_Blocker');
+	$extensions->register('reversals', 'WordPoints_Hook_Extension_Reversals');
+	$extensions->register('reversals_legacy_points', 'WordPoints_Hook_Extension_Reversals_Legacy_Points');
+	$extensions->register('conditions', 'WordPoints_Hook_Extension_Conditions');
+	$extensions->register('periods', 'WordPoints_Hook_Extension_Periods');
 }
 
 /**
@@ -93,7 +93,7 @@ 
  *
  * @param WordPoints_Class_Registry_Children $conditions The conditions registry.
  */
-function wordpoints_hook_conditions_init( $conditions ) {
+function wordpoints_hook_conditions_init($conditions) {
 
 	$conditions->register(
 		'text'
@@ -129,7 +129,7 @@ 
  *
  * @param WordPoints_Hook_Actions $actions The action registry.
  */
-function wordpoints_hook_actions_init( $actions ) {
+function wordpoints_hook_actions_init($actions) {
 
 	$actions->register(
 		'comment_approve'
@@ -137,8 +137,8 @@ 
 		, array(
 			'action' => 'transition_comment_status',
 			'data'   => array(
-				'arg_index'    => array( 'comment' => 2 ),
-				'requirements' => array( 0 => 'approved' ),
+				'arg_index'    => array('comment' => 2),
+				'requirements' => array(0 => 'approved'),
 			),
 		)
 	);
@@ -149,7 +149,7 @@ 
 		, array(
 			'action' => 'wp_insert_comment',
 			'data'   => array(
-				'arg_index' => array( 'comment' => 1 ),
+				'arg_index' => array('comment' => 1),
 			),
 		)
 	);
@@ -160,8 +160,8 @@ 
 		, array(
 			'action' => 'transition_comment_status',
 			'data'   => array(
-				'arg_index' => array( 'comment' => 2 ),
-				'requirements' => array( 1 => 'approved' ),
+				'arg_index' => array('comment' => 2),
+				'requirements' => array(1 => 'approved'),
 			),
 		)
 	);
@@ -173,8 +173,8 @@ 
 		, array(
 			'action' => 'transition_post_status',
 			'data'   => array(
-				'arg_index' => array( 'post' => 2 ),
-				'requirements' => array( 0 => 'publish' ),
+				'arg_index' => array('post' => 2),
+				'requirements' => array(0 => 'publish'),
 			),
 		)
 	);
@@ -185,8 +185,8 @@ 
 		, array(
 			'action' => 'transition_post_status',
 			'data'   => array(
-				'arg_index' => array( 'post' => 2 ),
-				'requirements' => array( 1 => 'publish' ),
+				'arg_index' => array('post' => 2),
+				'requirements' => array(1 => 'publish'),
 			),
 		)
 	);
@@ -197,7 +197,7 @@ 
 		, array(
 			'action' => 'delete_post',
 			'data'   => array(
-				'arg_index' => array( 'post' => 0 ),
+				'arg_index' => array('post' => 0),
 			),
 		)
 	);
@@ -208,7 +208,7 @@ 
 		, array(
 			'action' => 'add_attachment',
 			'data'   => array(
-				'arg_index' => array( 'post\attachment' => 0 ),
+				'arg_index' => array('post\attachment' => 0),
 			),
 		)
 	);
@@ -219,7 +219,7 @@ 
 		, array(
 			'action' => 'delete_post',
 			'data'   => array(
-				'arg_index' => array( 'post' => 0 ),
+				'arg_index' => array('post' => 0),
 			),
 		)
 	);
@@ -230,7 +230,7 @@ 
 		, array(
 			'action' => 'user_register',
 			'data'   => array(
-				'arg_index' => array( 'user' => 0 ),
+				'arg_index' => array('user' => 0),
 			),
 		)
 	);
@@ -241,7 +241,7 @@ 
 		, array(
 			'action' => is_multisite() ? 'wpmu_delete_user' : 'delete_user',
 			'data'   => array(
-				'arg_index' => array( 'user' => 0 ),
+				'arg_index' => array('user' => 0),
 			),
 		)
 	);
@@ -264,7 +264,7 @@ 
  *
  * @param WordPoints_Hook_Events $events The event registry.
  */
-function wordpoints_hook_events_init( $events ) {
+function wordpoints_hook_events_init($events) {
 
 	$events->register(
 		'user_register'
@@ -294,7 +294,7 @@ 
 	);
 
 	// Register events for all of the public post types.
-	$post_types = get_post_types( array( 'public' => true ) );
+	$post_types = get_post_types(array('public' => true));
 
 	/**
 	 * Filter which post types to register hook events for.
@@ -303,10 +303,10 @@ 
 	 *
 	 * @param string[] The post type slugs ("names").
 	 */
-	$post_types = apply_filters( 'wordpoints_register_hook_events_for_post_types', $post_types );
+	$post_types = apply_filters('wordpoints_register_hook_events_for_post_types', $post_types);
 
-	foreach ( $post_types as $slug ) {
-		wordpoints_register_post_type_hook_events( $slug );
+	foreach ($post_types as $slug) {
+		wordpoints_register_post_type_hook_events($slug);
 	}
 }
 
@@ -319,11 +319,11 @@ 
  *
  * @param WordPoints_App_Registry $entities The entities app.
  */
-function wordpoints_entities_app_init( $entities ) {
+function wordpoints_entities_app_init($entities) {
 
 	$sub_apps = $entities->sub_apps();
-	$sub_apps->register( 'children', 'WordPoints_Class_Registry_Children' );
-	$sub_apps->register( 'contexts', 'WordPoints_Class_Registry' );
+	$sub_apps->register('children', 'WordPoints_Class_Registry_Children');
+	$sub_apps->register('contexts', 'WordPoints_Class_Registry');
 }
 
 /**
@@ -335,10 +335,10 @@ 
  *
  * @param WordPoints_Class_RegistryI $contexts The entity context registry.
  */
-function wordpoints_entity_contexts_init( $contexts ) {
+function wordpoints_entity_contexts_init($contexts) {
 
-	$contexts->register( 'network', 'WordPoints_Entity_Context_Network' );
-	$contexts->register( 'site', 'WordPoints_Entity_Context_Site' );
+	$contexts->register('network', 'WordPoints_Entity_Context_Network');
+	$contexts->register('site', 'WordPoints_Entity_Context_Site');
 }
 
 /**
@@ -350,17 +350,17 @@ 
  *
  * @param WordPoints_App_Registry $entities The entities app.
  */
-function wordpoints_entities_init( $entities ) {
+function wordpoints_entities_init($entities) {
 
-	$children = $entities->get_sub_app( 'children' );
+	$children = $entities->get_sub_app('children');
 
-	$entities->register( 'user', 'WordPoints_Entity_User' );
-	$children->register( 'user', 'roles', 'WordPoints_Entity_User_Roles' );
+	$entities->register('user', 'WordPoints_Entity_User');
+	$children->register('user', 'roles', 'WordPoints_Entity_User_Roles');
 
-	$entities->register( 'user_role', 'WordPoints_Entity_User_Role' );
+	$entities->register('user_role', 'WordPoints_Entity_User_Role');
 
 	// Register entities for all of the public post types.
-	$post_types = get_post_types( array( 'public' => true ) );
+	$post_types = get_post_types(array('public' => true));
 
 	/**
 	 * Filter which post types to register entities for.
@@ -369,14 +369,14 @@ 
 	 *
 	 * @param string[] The post type slugs ("names").
 	 */
-	$post_types = apply_filters( 'wordpoints_register_entities_for_post_types', $post_types );
+	$post_types = apply_filters('wordpoints_register_entities_for_post_types', $post_types);
 
-	foreach ( $post_types as $slug ) {
-		wordpoints_register_post_type_entities( $slug );
+	foreach ($post_types as $slug) {
+		wordpoints_register_post_type_entities($slug);
 	}
 
 	// Register entities for all of the public taxonomies.
-	$taxonomies = get_taxonomies( array( 'public' => true ) );
+	$taxonomies = get_taxonomies(array('public' => true));
 
 	/**
 	 * Filter which taxonomies to register entities for.
@@ -385,10 +385,10 @@ 
 	 *
 	 * @param string[] The taxonomy slugs.
 	 */
-	$taxonomies = apply_filters( 'wordpoints_register_entities_for_taxonomies', $taxonomies );
+	$taxonomies = apply_filters('wordpoints_register_entities_for_taxonomies', $taxonomies);
 
-	foreach ( $taxonomies as $slug ) {
-		wordpoints_register_taxonomy_entities( $slug );
+	foreach ($taxonomies as $slug) {
+		wordpoints_register_taxonomy_entities($slug);
 	}
 }
 
@@ -399,28 +399,28 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_register_post_type_entities( $slug ) {
+function wordpoints_register_post_type_entities($slug) {
 
 	$entities = wordpoints_entities();
-	$children = $entities->get_sub_app( 'children' );
+	$children = $entities->get_sub_app('children');
 
-	$entities->register( "post\\{$slug}", 'WordPoints_Entity_Post' );
-	$children->register( "post\\{$slug}", 'author', 'WordPoints_Entity_Post_Author' );
+	$entities->register("post\\{$slug}", 'WordPoints_Entity_Post');
+	$children->register("post\\{$slug}", 'author', 'WordPoints_Entity_Post_Author');
 
-	$supports = get_all_post_type_supports( $slug );
+	$supports = get_all_post_type_supports($slug);
 
-	if ( isset( $supports['editor'] ) ) {
-		$children->register( "post\\{$slug}", 'content', 'WordPoints_Entity_Post_Content' );
+	if (isset($supports['editor'])) {
+		$children->register("post\\{$slug}", 'content', 'WordPoints_Entity_Post_Content');
 	}
 
-	if ( isset( $supports['comments'] ) ) {
-		$entities->register( "comment\\{$slug}", 'WordPoints_Entity_Comment' );
-		$children->register( "comment\\{$slug}", "post\\{$slug}", 'WordPoints_Entity_Comment_Post' );
-		$children->register( "comment\\{$slug}", 'author', 'WordPoints_Entity_Comment_Author' );
+	if (isset($supports['comments'])) {
+		$entities->register("comment\\{$slug}", 'WordPoints_Entity_Comment');
+		$children->register("comment\\{$slug}", "post\\{$slug}", 'WordPoints_Entity_Comment_Post');
+		$children->register("comment\\{$slug}", 'author', 'WordPoints_Entity_Comment_Author');
 	}
 
-	foreach ( get_object_taxonomies( $slug ) as $taxonomy_slug ) {
-		$children->register( "post\\{$slug}", "terms\\{$taxonomy_slug}", 'WordPoints_Entity_Post_Terms' );
+	foreach (get_object_taxonomies($slug) as $taxonomy_slug) {
+		$children->register("post\\{$slug}", "terms\\{$taxonomy_slug}", 'WordPoints_Entity_Post_Terms');
 	}
 
 	/**
@@ -430,7 +430,7 @@ 
 	 *
 	 * @param string $slug The slug ("name") of the post type.
 	 */
-	do_action( 'wordpoints_register_post_type_entities', $slug );
+	do_action('wordpoints_register_post_type_entities', $slug);
 }
 
 /**
@@ -440,11 +440,11 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_register_post_type_hook_events( $slug ) {
+function wordpoints_register_post_type_hook_events($slug) {
 
-	$events = wordpoints_hooks()->get_sub_app( 'events' );
+	$events = wordpoints_hooks()->get_sub_app('events');
 
-	if ( 'attachment' === $slug ) {
+	if ('attachment' === $slug) {
 
 		$events->register(
 			'media_upload'
@@ -468,7 +468,7 @@ 
 			, array(
 				'actions' => array(
 					'toggle_on'  => 'post_publish',
-					'toggle_off' => array( 'post_depublish', 'post_depublish_delete' ),
+					'toggle_off' => array('post_depublish', 'post_depublish_delete'),
 				),
 				'args'    => array(
 					"post\\{$slug}" => 'WordPoints_Hook_Arg_Dynamic',
@@ -477,14 +477,14 @@ 
 		);
 	}
 
-	if ( post_type_supports( $slug, 'comments' ) ) {
+	if (post_type_supports($slug, 'comments')) {
 
 		$events->register(
 			"comment_leave\\{$slug}"
 			, 'WordPoints_Hook_Event_Comment_Leave'
 			, array(
 				'actions' => array(
-					'toggle_on'  => array( 'comment_approve', 'comment_new' ),
+					'toggle_on'  => array('comment_approve', 'comment_new'),
 					'toggle_off' => 'comment_deapprove',
 				),
 				'args' => array(
@@ -501,7 +501,7 @@ 
 	 *
 	 * @param string $slug The slug ("name") of the post type.
 	 */
-	do_action( 'wordpoints_register_post_type_hook_events', $slug );
+	do_action('wordpoints_register_post_type_hook_events', $slug);
 }
 
 /**
@@ -511,13 +511,13 @@ 
  *
  * @param string $slug The slug of the taxonomy.
  */
-function wordpoints_register_taxonomy_entities( $slug ) {
+function wordpoints_register_taxonomy_entities($slug) {
 
 	$entities = wordpoints_entities();
-	$children = $entities->get_sub_app( 'children' );
+	$children = $entities->get_sub_app('children');
 
-	$entities->register( "term\\{$slug}", 'WordPoints_Entity_Term' );
-	$children->register( "term\\{$slug}", 'id', 'WordPoints_Entity_Term_Id' );
+	$entities->register("term\\{$slug}", 'WordPoints_Entity_Term');
+	$children->register("term\\{$slug}", 'id', 'WordPoints_Entity_Term_Id');
 
 	/**
 	 * Fired when registering the entities for a taxonomy.
@@ -526,7 +526,7 @@ 
 	 *
 	 * @param string $slug The taxonomy's slug.
 	 */
-	do_action( 'wordpoints_register_taxonomy_entities', $slug );
+	do_action('wordpoints_register_taxonomy_entities', $slug);
 }
 
 /**
@@ -538,10 +538,10 @@ 
  *
  * @param WordPoints_Class_RegistryI $data_types The data types registry.
  */
-function wordpoints_data_types_init( $data_types ) {
+function wordpoints_data_types_init($data_types) {
 
-	$data_types->register( 'integer', 'WordPoints_Data_Type_Integer' );
-	$data_types->register( 'text', 'WordPoints_Data_Type_Text' );
+	$data_types->register('integer', 'WordPoints_Data_Type_Integer');
+	$data_types->register('text', 'WordPoints_Data_Type_Text');
 }
 
 /**
@@ -556,9 +556,9 @@ 
  *
  * @return bool Whether the user can view the points log.
  */
-function wordpoints_hooks_user_can_view_points_log( $can_view, $log ) {
+function wordpoints_hooks_user_can_view_points_log($can_view, $log) {
 
-	if ( ! $can_view ) {
+	if ( ! $can_view) {
 		return $can_view;
 	}
 
@@ -567,11 +567,11 @@ 
 	$event_slug = $log->log_type;
 
 	/** @var WordPoints_Hook_ArgI $arg */
-	foreach ( wordpoints_hooks()->get_sub_app( 'events' )->get_sub_app( 'args' )->get_children( $event_slug ) as $slug => $arg ) {
+	foreach (wordpoints_hooks()->get_sub_app('events')->get_sub_app('args')->get_children($event_slug) as $slug => $arg) {
 
-		$value = wordpoints_get_points_log_meta( $log->id, $slug, true );
+		$value = wordpoints_get_points_log_meta($log->id, $slug, true);
 
-		if ( ! $value ) {
+		if ( ! $value) {
 			continue;
 		}
 
@@ -581,7 +581,7 @@ 
 			, $value
 		);
 
-		if ( ! $can_view ) {
+		if ( ! $can_view) {
 			break;
 		}
 	}
@@ -600,20 +600,20 @@ 
  *
  * @return bool Whether the user can view this entity.
  */
-function wordpoints_entity_user_can_view( $user_id, $entity_slug, $entity_id ) {
+function wordpoints_entity_user_can_view($user_id, $entity_slug, $entity_id) {
 
-	$entity = wordpoints_entities()->get( $entity_slug );
+	$entity = wordpoints_entities()->get($entity_slug);
 
 	// If this entity type is not found, we have no way of determining whether it is
 	// safe for the user to view it.
-	if ( ! ( $entity instanceof WordPoints_Entity ) ) {
+	if ( ! ($entity instanceof WordPoints_Entity)) {
 		return false;
 	}
 
 	$can_view = true;
 
-	if ( $entity instanceof WordPoints_Entity_Restricted_VisibilityI ) {
-		$can_view = $entity->user_can_view( $user_id, $entity_id );
+	if ($entity instanceof WordPoints_Entity_Restricted_VisibilityI) {
+		$can_view = $entity->user_can_view($user_id, $entity_id);
 	}
 
 	/**
@@ -644,8 +644,8 @@ 
  */
 function wordpoints_apps() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
-		WordPoints_App::$main = new WordPoints_App( 'apps' );
+	if ( ! isset(WordPoints_App::$main)) {
+		WordPoints_App::$main = new WordPoints_App('apps');
 	}
 
 	return WordPoints_App::$main;
@@ -660,11 +660,11 @@ 
  */
 function wordpoints_hooks() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
+	if ( ! isset(WordPoints_App::$main)) {
 		wordpoints_apps();
 	}
 
-	return WordPoints_App::$main->get_sub_app( 'hooks' );
+	return WordPoints_App::$main->get_sub_app('hooks');
 }
 
 /**
@@ -676,11 +676,11 @@ 
  */
 function wordpoints_entities() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
+	if ( ! isset(WordPoints_App::$main)) {
 		wordpoints_apps();
 	}
 
-	return WordPoints_App::$main->get_sub_app( 'entities' );
+	return WordPoints_App::$main->get_sub_app('entities');
 }
 
 /**
@@ -692,13 +692,13 @@ 
  *
  * @param WordPoints_App $app The main apps app.
  */
-function wordpoints_apps_init( $app ) {
+function wordpoints_apps_init($app) {
 
 	$apps = $app->sub_apps();
 
-	$apps->register( 'hooks', 'WordPoints_Hooks' );
-	$apps->register( 'entities', 'WordPoints_App_Registry' );
-	$apps->register( 'data_types', 'WordPoints_Class_Registry' );
+	$apps->register('hooks', 'WordPoints_Hooks');
+	$apps->register('entities', 'WordPoints_App_Registry');
+	$apps->register('data_types', 'WordPoints_Class_Registry');
 }
 
 /**
@@ -711,19 +711,19 @@ 
  *
  * @return object|false The constructed object, or false if to many args were passed.
  */
-function wordpoints_construct_class_with_args( $class_name, array $args ) {
+function wordpoints_construct_class_with_args($class_name, array $args) {
 
-	switch ( count( $args ) ) {
+	switch (count($args)) {
 		case 0:
 			return new $class_name();
 		case 1:
-			return new $class_name( $args[0] );
+			return new $class_name($args[0]);
 		case 2:
-			return new $class_name( $args[0], $args[1] );
+			return new $class_name($args[0], $args[1]);
 		case 3:
-			return new $class_name( $args[0], $args[1], $args[2] );
+			return new $class_name($args[0], $args[1], $args[2]);
 		case 4:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3] );
+			return new $class_name($args[0], $args[1], $args[2], $args[3]);
 		default:
 			return false;
 	}
@@ -745,13 +745,13 @@ 
  * @return array The slug parsed into the 'generic' and 'dynamic' portions. If the
  *               slug is not dynamic, the value of each of those keys will be false.
  */
-function wordpoints_parse_dynamic_slug( $slug ) {
+function wordpoints_parse_dynamic_slug($slug) {
 
-	$parsed = array( 'dynamic' => false, 'generic' => false );
+	$parsed = array('dynamic' => false, 'generic' => false);
 
-	$parts = explode( '\\', $slug, 2 );
+	$parts = explode('\\', $slug, 2);
 
-	if ( isset( $parts[1] ) ) {
+	if (isset($parts[1])) {
 		$parsed['dynamic'] = $parts[1];
 		$parsed['generic'] = $parts[0];
 	}
@@ -790,28 +790,28 @@ 
  *                     contexts, indexed by context slug, or false if any of the
  *                     contexts isn't current.
  */
-function wordpoints_entities_get_current_context_id( $slug ) {
+function wordpoints_entities_get_current_context_id($slug) {
 
 	$current_context = array();
 
 	/** @var WordPoints_Class_Registry $contexts */
-	$contexts = wordpoints_entities()->get_sub_app( 'contexts' );
+	$contexts = wordpoints_entities()->get_sub_app('contexts');
 
-	while ( $slug ) {
+	while ($slug) {
 
-		$context = $contexts->get( $slug );
+		$context = $contexts->get($slug);
 
-		if ( ! $context instanceof WordPoints_Entity_Context ) {
+		if ( ! $context instanceof WordPoints_Entity_Context) {
 			return false;
 		}
 
 		$id = $context->get_current_id();
 
-		if ( false === $id ) {
+		if (false === $id) {
 			return false;
 		}
 
-		$current_context[ $slug ] = $id;
+		$current_context[$slug] = $id;
 
 		$slug = $context->get_parent_slug();
 	}
@@ -832,22 +832,22 @@ 
  */
 function wordpoints_is_network_context() {
 
-	if ( ! is_multisite() ) {
+	if ( ! is_multisite()) {
 		return false;
 	}
 
-	if ( is_network_admin() ) {
+	if (is_network_admin()) {
 		return true;
 	}
 
 	// See https://core.trac.wordpress.org/ticket/22589
 	if (
-		defined( 'DOING_AJAX' )
+		defined('DOING_AJAX')
 		&& DOING_AJAX
-		&& isset( $_SERVER['HTTP_REFERER'] )
+		&& isset($_SERVER['HTTP_REFERER'])
 		&& preg_match(
-			'#^' . preg_quote( network_admin_url(), '#' ) . '#i'
-			, esc_url_raw( wp_unslash( $_SERVER['HTTP_REFERER'] ) )
+			'#^'.preg_quote(network_admin_url(), '#').'#i'
+			, esc_url_raw(wp_unslash($_SERVER['HTTP_REFERER']))
 		)
 	) {
 		return true;
@@ -860,7 +860,7 @@ 
 	 *
 	 * @param bool $in_network_context Whether we are in network context.
 	 */
-	return apply_filters( 'wordpoints_is_network_context', false );
+	return apply_filters('wordpoints_is_network_context', false);
 }
 
 /**
@@ -874,21 +874,21 @@ 
  *
  * @return string The primary arg's GUID, JSON encoded.
  */
-function wordpoints_hooks_get_event_primary_arg_guid_json( WordPoints_Hook_Event_Args $event_args ) {
+function wordpoints_hooks_get_event_primary_arg_guid_json(WordPoints_Hook_Event_Args $event_args) {
 
 	$entity = $event_args->get_primary_arg();
 
-	if ( ! $entity ) {
+	if ( ! $entity) {
 		return '';
 	}
 
 	$the_guid = $entity->get_the_guid();
 
-	if ( ! $the_guid ) {
+	if ( ! $the_guid) {
 		return '';
 	}
 
-	return wp_json_encode( $the_guid );
+	return wp_json_encode($the_guid);
 }
 
 /**
@@ -901,7 +901,7 @@ 
  */
 function wordpoints_hooks_api_add_global_cache_groups() {
 
-	if ( function_exists( 'wp_cache_add_global_groups' ) ) {
+	if (function_exists('wp_cache_add_global_groups')) {
 
 		wp_cache_add_global_groups(
 			array(
@@ -927,8 +927,8 @@ 
  * @return string The escaped identifier. Already quoted, do not place within
  *                backticks.
  */
-function wordpoints_escape_mysql_identifier( $identifier ) {
-	return '`' . str_replace( '`', '``', $identifier ) . '`';
+function wordpoints_escape_mysql_identifier($identifier) {
+	return '`'.str_replace('`', '``', $identifier).'`';
 }
 
 // EOF