WordPoints / hooks-api

src/includes/classes/hook/reactioni.php

Spacing +5 added lines, -5 removed lines

@@ -57,7 +57,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.
@@ -90,7 +90,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.
@@ -104,7 +104,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.
@@ -116,7 +116,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.
@@ -127,7 +127,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/hooks.php

Spacing +36 added lines, -36 removed lines

@@ -40,13 +40,13 @@ 
 	protected function init() {
 
 		$sub_apps = $this->sub_apps;
-		$sub_apps->register( 'router', 'WordPoints_Hook_Router' );
-		$sub_apps->register( 'actions', 'WordPoints_Hook_Actions' );
-		$sub_apps->register( 'events', 'WordPoints_Hook_Events' );
-		$sub_apps->register( 'reactors', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'reaction_stores', 'WordPoints_Class_Registry_Children' );
-		$sub_apps->register( 'extensions', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'conditions', 'WordPoints_Class_Registry_Children' );
+		$sub_apps->register('router', 'WordPoints_Hook_Router');
+		$sub_apps->register('actions', 'WordPoints_Hook_Actions');
+		$sub_apps->register('events', 'WordPoints_Hook_Events');
+		$sub_apps->register('reactors', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('reaction_stores', 'WordPoints_Class_Registry_Children');
+		$sub_apps->register('extensions', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('conditions', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -69,8 +69,8 @@ 
 	 */
 	public function get_current_mode() {
 
-		if ( ! isset( $this->current_mode ) ) {
-			$this->current_mode = ( wordpoints_is_network_context() ? 'network' : 'standard' );
+		if ( ! isset($this->current_mode)) {
+			$this->current_mode = (wordpoints_is_network_context() ? 'network' : 'standard');
 		}
 
 		return $this->current_mode;
@@ -88,7 +88,7 @@ 
 	 *
 	 * @param string $mode The slug of the mode to set as the current mode.
 	 */
-	public function set_current_mode( $mode ) {
+	public function set_current_mode($mode) {
 		$this->current_mode = $mode;
 	}
 
@@ -101,19 +101,19 @@ 
 	 *
 	 * @return WordPoints_Hook_Reaction_StoreI|false The reaction storage object.
 	 */
-	public function get_reaction_store( $slug ) {
+	public function get_reaction_store($slug) {
 
-		$reaction_store = $this->get_sub_app( 'reaction_stores' )->get(
+		$reaction_store = $this->get_sub_app('reaction_stores')->get(
 			$this->get_current_mode()
 			, $slug
 		);
 
-		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI ) {
+		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI) {
 			return false;
 		}
 
 		// Allowing access to stores out-of-context would lead to strange behavior.
-		if ( false === $reaction_store->get_context_id() ) {
+		if (false === $reaction_store->get_context_id()) {
 			return false;
 		}
 
@@ -136,19 +136,19 @@ 
 		$action_type
 	) {
 
-		foreach ( $this->get_sub_app( 'reaction_stores' )->get_all() as $reaction_stores ) {
-			foreach ( $reaction_stores as $reaction_store ) {
+		foreach ($this->get_sub_app('reaction_stores')->get_all() as $reaction_stores) {
+			foreach ($reaction_stores as $reaction_store) {
 
-				if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI ) {
+				if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI) {
 					continue;
 				}
 
 				// Allowing access to stores out-of-context would lead to strange behavior.
-				if ( false === $reaction_store->get_context_id() ) {
+				if (false === $reaction_store->get_context_id()) {
 					continue;
 				}
 
-				foreach ( $reaction_store->get_reactions_to_event( $event_slug ) as $reaction ) {
+				foreach ($reaction_store->get_reactions_to_event($event_slug) as $reaction) {
 
 					$fire = new WordPoints_Hook_Fire(
 						$event_args
@@ -156,7 +156,7 @@ 
 						, $action_type
 					);
 
-					$this->fire_reaction( $fire );
+					$this->fire_reaction($fire);
 				}
 			}
 		}
@@ -169,32 +169,32 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	protected function fire_reaction( $fire ) {
+	protected function fire_reaction($fire) {
 
 		/** @var WordPoints_Hook_ReactorI $reactor */
-		$reactor = $this->get_sub_app( 'reactors' )->get( $fire->reaction->get_reactor_slug() );
+		$reactor = $this->get_sub_app('reactors')->get($fire->reaction->get_reactor_slug());
 
-		if ( ! in_array( $fire->action_type, $reactor->get_action_types(), true ) ) {
+		if ( ! in_array($fire->action_type, $reactor->get_action_types(), true)) {
 			return;
 		}
 
-		$validator = new WordPoints_Hook_Reaction_Validator( $fire->reaction, true );
+		$validator = new WordPoints_Hook_Reaction_Validator($fire->reaction, true);
 		$validator->validate();
 
-		if ( $validator->had_errors() ) {
+		if ($validator->had_errors()) {
 			return;
 		}
 
-		unset( $validator );
+		unset($validator);
 
 		/** @var WordPoints_Hook_ExtensionI[] $extensions */
-		$extensions = $this->get_sub_app( 'extensions' )->get_all();
+		$extensions = $this->get_sub_app('extensions')->get_all();
 
-		foreach ( $extensions as $extension ) {
-			if ( ! $extension->should_hit( $fire ) ) {
-				foreach ( $extensions as $ext ) {
-					if ( $ext instanceof WordPoints_Hook_Extension_Miss_ListenerI ) {
-						$ext->after_miss( $fire );
+		foreach ($extensions as $extension) {
+			if ( ! $extension->should_hit($fire)) {
+				foreach ($extensions as $ext) {
+					if ($ext instanceof WordPoints_Hook_Extension_Miss_ListenerI) {
+						$ext->after_miss($fire);
 					}
 				}
 
@@ -204,11 +204,11 @@ 
 
 		$fire->hit();
 
-		$reactor->hit( $fire );
+		$reactor->hit($fire);
 
-		foreach ( $extensions as $extension ) {
-			if ( $extension instanceof WordPoints_Hook_Extension_Hit_ListenerI ) {
-				$extension->after_hit( $fire );
+		foreach ($extensions as $extension) {
+			if ($extension instanceof WordPoints_Hook_Extension_Hit_ListenerI) {
+				$extension->after_hit($fire);
 			}
 		}
 	}

src/includes/classes/app.php

Spacing +13 added lines, -13 removed lines

@@ -94,16 +94,16 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug, $parent = null ) {
+	public function __construct($slug, $parent = null) {
 
 		$this->slug = $slug;
 		$this->full_slug = $slug;
 
-		if ( $parent instanceof WordPoints_App ) {
+		if ($parent instanceof WordPoints_App) {
 			$this->parent = $parent;
 
-			if ( 'apps' !== $this->parent->full_slug ) {
-				$this->full_slug = $this->parent->full_slug . '-' . $this->full_slug;
+			if ('apps' !== $this->parent->full_slug) {
+				$this->full_slug = $this->parent->full_slug.'-'.$this->full_slug;
 			}
 		}
 
@@ -121,21 +121,21 @@ 
 	 *
 	 * @return null|object|WordPoints_App|WordPoints_Class_RegistryI|WordPoints_Class_Registry_ChildrenI The sub app, or null if not found.
 	 */
-	public function get_sub_app( $slug ) {
+	public function get_sub_app($slug) {
 
-		$sub_app = $this->sub_apps->get( $slug, array( $this ) );
+		$sub_app = $this->sub_apps->get($slug, array($this));
 
-		if ( ! $sub_app ) {
+		if ( ! $sub_app) {
 			return null;
 		}
 
 		if (
-			empty( $this->did_init[ $slug ] )
+			empty($this->did_init[$slug])
 			&& ! self::$main->silent
-			&& $this->should_do_registry_init( $sub_app )
+			&& $this->should_do_registry_init($sub_app)
 		) {
 
-			$this->did_init[ $slug ] = true;
+			$this->did_init[$slug] = true;
 
 			/**
 			 * Initialization of an app registry.
@@ -147,7 +147,7 @@ 
 			 * @param WordPoints_Class_RegistryI|WordPoints_Class_Registry_ChildrenI
 			 *        $registry The registry object.
 			 */
-			do_action( "wordpoints_init_app_registry-{$this->full_slug}-{$slug}", $sub_app );
+			do_action("wordpoints_init_app_registry-{$this->full_slug}-{$slug}", $sub_app);
 		}
 
 		return $sub_app;
@@ -173,7 +173,7 @@ 
 	 *
 	 * @return bool Whether to call the init action or not.
 	 */
-	protected function should_do_registry_init( $registry ) {
+	protected function should_do_registry_init($registry) {
 		return (
 		   $registry instanceof WordPoints_Class_RegistryI
 		   || $registry instanceof WordPoints_Class_Registry_ChildrenI
@@ -197,7 +197,7 @@ 
 		 *
 		 * @param WordPoints_App $app The app object.
 		 */
-		do_action( "wordpoints_init_app-{$this->full_slug}", $this );
+		do_action("wordpoints_init_app-{$this->full_slug}", $this);
 	}
 }
 

src/includes/classes/hook/reaction.php

Spacing +3 added lines, -3 removed lines

@@ -40,9 +40,9 @@ 
 	 * @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) {
 
-		$this->ID    = wordpoints_int( $id );
+		$this->ID    = wordpoints_int($id);
 		$this->store = $store;
 	}
 
@@ -69,7 +69,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_reactor_slug() {
-		return $this->get_meta( 'reactor' );
+		return $this->get_meta('reactor');
 	}
 
 	/**

src/includes/classes/hook/router.php

Spacing +51 added lines, -51 removed lines

@@ -89,13 +89,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __call( $name, $args ) {
+	public function __call($name, $args) {
 
-		$this->route_action( $name, $args );
+		$this->route_action($name, $args);
 
 		// Return the first value, in case it is hooked to a filter.
 		$return = null;
-		if ( isset( $args[0] ) ) {
+		if (isset($args[0])) {
 			$return = $args[0];
 		}
 
@@ -112,9 +112,9 @@ 
 	 *                     action name and the priority.
 	 * @param array  $args The args the action was fired with.
 	 */
-	protected function route_action( $name, $args ) {
+	protected function route_action($name, $args) {
 
-		if ( ! isset( $this->action_index[ $name ] ) ) {
+		if ( ! isset($this->action_index[$name])) {
 			return;
 		}
 
@@ -122,48 +122,48 @@ 
 		// registry attempts to access the router in its own constructor. The result
 		// of attempting to do this before the router itself has been fully
 		// constructed is that the events registry gets null instead of the router.
-		if ( ! isset( $this->hooks ) ) {
+		if ( ! isset($this->hooks)) {
 
 			$hooks = wordpoints_hooks();
 
 			$this->hooks   = $hooks;
-			$this->events  = $hooks->get_sub_app( 'events' );
-			$this->actions = $hooks->get_sub_app( 'actions' );
-			$this->event_args = $this->events->get_sub_app( 'args' );
+			$this->events  = $hooks->get_sub_app('events');
+			$this->actions = $hooks->get_sub_app('actions');
+			$this->event_args = $this->events->get_sub_app('args');
 		}
 
-		foreach ( $this->action_index[ $name ]['actions'] as $slug => $data ) {
+		foreach ($this->action_index[$name]['actions'] as $slug => $data) {
 
-			if ( ! isset( $this->event_index[ $slug ] ) ) {
+			if ( ! isset($this->event_index[$slug])) {
 				continue;
 			}
 
-			$action_object = $this->actions->get( $slug, $args, $data );
+			$action_object = $this->actions->get($slug, $args, $data);
 
-			if ( ! ( $action_object instanceof WordPoints_Hook_ActionI ) ) {
+			if ( ! ($action_object instanceof WordPoints_Hook_ActionI)) {
 				continue;
 			}
 
-			if ( ! $action_object->should_fire() ) {
+			if ( ! $action_object->should_fire()) {
 				continue;
 			}
 
-			foreach ( $this->event_index[ $slug ] as $type => $events ) {
-				foreach ( $events as $event_slug => $unused ) {
+			foreach ($this->event_index[$slug] as $type => $events) {
+				foreach ($events as $event_slug => $unused) {
 
-					if ( ! $this->events->is_registered( $event_slug ) ) {
+					if ( ! $this->events->is_registered($event_slug)) {
 						continue;
 					}
 
-					$event_args = $this->event_args->get_children( $event_slug, array( $action_object ) );
+					$event_args = $this->event_args->get_children($event_slug, array($action_object));
 
-					if ( empty( $event_args ) ) {
+					if (empty($event_args)) {
 						continue;
 					}
 
-					$event_args = new WordPoints_Hook_Event_Args( $event_args );
+					$event_args = new WordPoints_Hook_Event_Args($event_args);
 
-					$this->hooks->fire( $event_slug, $event_args, $type );
+					$this->hooks->fire($event_slug, $event_args, $type);
 				}
 			}
 		}
@@ -192,56 +192,56 @@ 
 	 *        }
 	 * }
 	 */
-	public function add_action( $slug, array $args ) {
+	public function add_action($slug, array $args) {
 
 		$priority = 10;
-		if ( isset( $args['priority'] ) ) {
+		if (isset($args['priority'])) {
 			$priority = $args['priority'];
 		}
 
-		if ( ! isset( $args['action'] ) ) {
+		if ( ! isset($args['action'])) {
 			return;
 		}
 
 		$method = "{$args['action']},{$priority}";
 
-		$this->action_index[ $method ]['actions'][ $slug ] = array();
+		$this->action_index[$method]['actions'][$slug] = array();
 
 		$arg_number = 1;
 
-		if ( isset( $args['data'] ) ) {
+		if (isset($args['data'])) {
 
-			if ( isset( $args['data']['arg_index'] ) ) {
-				$arg_number = 1 + max( $args['data']['arg_index'] );
+			if (isset($args['data']['arg_index'])) {
+				$arg_number = 1 + max($args['data']['arg_index']);
 			}
 
-			if ( isset( $args['data']['requirements'] ) ) {
-				$requirements = 1 + max( array_keys( $args['data']['requirements'] ) );
+			if (isset($args['data']['requirements'])) {
+				$requirements = 1 + max(array_keys($args['data']['requirements']));
 
-				if ( $requirements > $arg_number ) {
+				if ($requirements > $arg_number) {
 					$arg_number = $requirements;
 				}
 			}
 
-			$this->action_index[ $method ]['actions'][ $slug ] = $args['data'];
+			$this->action_index[$method]['actions'][$slug] = $args['data'];
 		}
 
-		if ( isset( $args['arg_number'] ) ) {
+		if (isset($args['arg_number'])) {
 			$arg_number = $args['arg_number'];
 		}
 
 		// If this action is already being routed, and will have enough args, we
 		// don't need to hook to it again.
 		if (
-			isset( $this->action_index[ $method ]['arg_number'] )
-			&& $this->action_index[ $method ]['arg_number'] >= $arg_number
+			isset($this->action_index[$method]['arg_number'])
+			&& $this->action_index[$method]['arg_number'] >= $arg_number
 		) {
 			return;
 		}
 
-		$this->action_index[ $method ]['arg_number'] = $arg_number;
+		$this->action_index[$method]['arg_number'] = $arg_number;
 
-		add_action( $args['action'], array( $this, $method ), $priority, $arg_number );
+		add_action($args['action'], array($this, $method), $priority, $arg_number);
 	}
 
 	/**
@@ -251,20 +251,20 @@ 
 	 *
 	 * @param string $slug The action slug.
 	 */
-	public function remove_action( $slug ) {
+	public function remove_action($slug) {
 
-		foreach ( $this->action_index as $method => $data ) {
-			if ( isset( $data['actions'][ $slug ] ) ) {
+		foreach ($this->action_index as $method => $data) {
+			if (isset($data['actions'][$slug])) {
 
-				unset( $this->action_index[ $method ]['actions'][ $slug ] );
+				unset($this->action_index[$method]['actions'][$slug]);
 
-				if ( empty( $this->action_index[ $method ]['actions'] ) ) {
+				if (empty($this->action_index[$method]['actions'])) {
 
-					unset( $this->action_index[ $method ] );
+					unset($this->action_index[$method]);
 
-					list( $action, $priority ) = explode( ',', $method );
+					list($action, $priority) = explode(',', $method);
 
-					remove_action( $action, array( $this, $method ), $priority );
+					remove_action($action, array($this, $method), $priority);
 				}
 			}
 		}
@@ -279,8 +279,8 @@ 
 	 * @param string $action_slug The slug of the action.
 	 * @param string $action_type The type of action. Default is 'fire'.
 	 */
-	public function add_event_to_action( $event_slug, $action_slug, $action_type = 'fire' ) {
-		$this->event_index[ $action_slug ][ $action_type ][ $event_slug ] = true;
+	public function add_event_to_action($event_slug, $action_slug, $action_type = 'fire') {
+		$this->event_index[$action_slug][$action_type][$event_slug] = true;
 	}
 
 	/**
@@ -292,8 +292,8 @@ 
 	 * @param string $action_slug The slug of the action.
 	 * @param string $action_type The type of action. Default is 'fire'.
 	 */
-	public function remove_event_from_action( $event_slug, $action_slug, $action_type = 'fire' ) {
-		unset( $this->event_index[ $action_slug ][ $action_type ][ $event_slug ] );
+	public function remove_event_from_action($event_slug, $action_slug, $action_type = 'fire') {
+		unset($this->event_index[$action_slug][$action_type][$event_slug]);
 	}
 
 	/**
@@ -305,8 +305,8 @@ 
 	 */
 	public function get_event_index() {
 
-		if ( empty( $this->event_index ) ) {
-			wordpoints_hooks()->get_sub_app( 'events' );
+		if (empty($this->event_index)) {
+			wordpoints_hooks()->get_sub_app('events');
 		}
 
 		return $this->event_index;

src/includes/classes/hook/extension/conditions.php

Spacing +55 added lines, -55 removed lines

@@ -34,7 +34,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function __construct() {
-		$this->conditions = wordpoints_hooks()->get_sub_app( 'conditions' );
+		$this->conditions = wordpoints_hooks()->get_sub_app('conditions');
 	}
 
 	/**
@@ -44,14 +44,14 @@ 
 
 		$conditions_data = array();
 
-		foreach ( $this->conditions->get_all() as $data_type => $conditions ) {
-			foreach ( $conditions as $slug => $condition ) {
+		foreach ($this->conditions->get_all() as $data_type => $conditions) {
+			foreach ($conditions as $slug => $condition) {
 
-				if ( ! ( $condition instanceof WordPoints_Hook_ConditionI ) ) {
+				if ( ! ($condition instanceof WordPoints_Hook_ConditionI)) {
 					continue;
 				}
 
-				$conditions_data[ $data_type ][ $slug ] = array(
+				$conditions_data[$data_type][$slug] = array(
 					'slug'      => $slug,
 					'data_type' => $data_type,
 					'title'     => $condition->get_title(),
@@ -60,14 +60,14 @@ 
 			}
 		}
 
-		return array( 'conditions' => $conditions_data );
+		return array('conditions' => $conditions_data);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function validate_action_type_settings( $settings ) {
-		return $this->validate_conditions( $settings );
+	protected function validate_action_type_settings($settings) {
+		return $this->validate_conditions($settings);
 	}
 
 	/**
@@ -80,36 +80,36 @@ 
 	 *
 	 * @return array The validated settings.
 	 */
-	public function validate_conditions( $conditions, WordPoints_Hook_Event_Args $event_args = null ) {
+	public function validate_conditions($conditions, WordPoints_Hook_Event_Args $event_args = null) {
 
-		if ( $event_args ) {
+		if ($event_args) {
 			$this->event_args = $event_args;
 			$this->validator = $event_args->get_validator();
 		}
 
-		if ( ! is_array( $conditions ) ) {
+		if ( ! is_array($conditions)) {
 
 			$this->validator->add_error(
-				__( 'Conditions do not match expected format.', 'wordpoints' )
+				__('Conditions do not match expected format.', 'wordpoints')
 			);
 
 			return array();
 		}
 
-		foreach ( $conditions as $arg_slug => $sub_args ) {
+		foreach ($conditions as $arg_slug => $sub_args) {
 
-			if ( '_conditions' === $arg_slug ) {
+			if ('_conditions' === $arg_slug) {
 
-				$this->validator->push_field( $arg_slug );
+				$this->validator->push_field($arg_slug);
 
-				foreach ( $sub_args as $index => $settings ) {
+				foreach ($sub_args as $index => $settings) {
 
-					$this->validator->push_field( $index );
+					$this->validator->push_field($index);
 
-					$condition = $this->validate_condition( $settings );
+					$condition = $this->validate_condition($settings);
 
-					if ( $condition ) {
-						$sub_args[ $index ] = $condition;
+					if ($condition) {
+						$sub_args[$index] = $condition;
 					}
 
 					$this->validator->pop_field();
@@ -119,18 +119,18 @@ 
 
 			} else {
 
-				if ( ! $this->event_args->descend( $arg_slug ) ) {
+				if ( ! $this->event_args->descend($arg_slug)) {
 					continue;
 				}
 
-				$sub_args = $this->validate_action_type_settings( $sub_args );
+				$sub_args = $this->validate_action_type_settings($sub_args);
 
-				$conditions[ $arg_slug ] = $sub_args;
+				$conditions[$arg_slug] = $sub_args;
 
 				$this->event_args->ascend();
 			}
 
-			$conditions[ $arg_slug ] = $sub_args;
+			$conditions[$arg_slug] = $sub_args;
 		}
 
 		return $conditions;
@@ -146,32 +146,32 @@ 
 	 * @return array|false The validated conditions settings, or false if unable to
 	 *                     validate.
 	 */
-	protected function validate_condition( $settings ) {
+	protected function validate_condition($settings) {
 
-		if ( ! isset( $settings['type'] ) ) {
-			$this->validator->add_error( __( 'Condition type is missing.', 'wordpoints' ) );
+		if ( ! isset($settings['type'])) {
+			$this->validator->add_error(__('Condition type is missing.', 'wordpoints'));
 			return false;
 		}
 
 		$arg = $this->event_args->get_current();
 
-		$data_type = $this->get_data_type( $arg );
+		$data_type = $this->get_data_type($arg);
 
-		if ( ! $data_type ) {
+		if ( ! $data_type) {
 			$this->validator->add_error(
-				__( 'This type of condition does not work for the selected attribute.', 'wordpoints' )
+				__('This type of condition does not work for the selected attribute.', 'wordpoints')
 			);
 
 			return false;
 		}
 
-		$condition = $this->conditions->get( $data_type, $settings['type'] );
+		$condition = $this->conditions->get($data_type, $settings['type']);
 
-		if ( ! $condition ) {
+		if ( ! $condition) {
 
 			$this->validator->add_error(
 				sprintf(
-					__( 'Unknown condition type “%s”.', 'wordpoints' )
+					__('Unknown condition type “%s”.', 'wordpoints')
 					, $settings['type']
 				)
 				, 'type'
@@ -180,17 +180,17 @@ 
 			return false;
 		}
 
-		if ( ! isset( $settings['settings'] ) ) {
-			$this->validator->add_error( __( 'Condition settings are missing.', 'wordpoints' ) );
+		if ( ! isset($settings['settings'])) {
+			$this->validator->add_error(__('Condition settings are missing.', 'wordpoints'));
 			return false;
 		}
 
-		$this->validator->push_field( 'settings' );
+		$this->validator->push_field('settings');
 
 		// The condition may call this object's validate_settings() method to
 		// validate some sub-conditions. When that happens, these properties will be
 		// reset, so we need to back up their values and then restore them below.
-		$backup = array( $this->validator, $this->event_args );
+		$backup = array($this->validator, $this->event_args);
 
 		$settings['settings'] = $condition->validate_settings(
 			$arg
@@ -198,7 +198,7 @@ 
 			, $this->validator
 		);
 
-		list( $this->validator, $this->event_args ) = $backup;
+		list($this->validator, $this->event_args) = $backup;
 
 		$this->validator->pop_field();
 
@@ -208,11 +208,11 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		$conditions = $this->get_settings_from_fire( $fire );
+		$conditions = $this->get_settings_from_fire($fire);
 
-		if ( $conditions && ! $this->conditions_are_met( $conditions, $fire->event_args ) ) {
+		if ($conditions && ! $this->conditions_are_met($conditions, $fire->event_args)) {
 			return false;
 		}
 
@@ -234,35 +234,35 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		foreach ( $conditions as $arg_slug => $sub_args ) {
+		foreach ($conditions as $arg_slug => $sub_args) {
 
-			$event_args->descend( $arg_slug );
+			$event_args->descend($arg_slug);
 
-			if ( isset( $sub_args['_conditions'] ) ) {
+			if (isset($sub_args['_conditions'])) {
 
-				foreach ( $sub_args['_conditions'] as $settings ) {
+				foreach ($sub_args['_conditions'] as $settings) {
 
 					$condition = $this->conditions->get(
-						$this->get_data_type( $event_args->get_current() )
+						$this->get_data_type($event_args->get_current())
 						, $settings['type']
 					);
 
-					$is_met = $condition->is_met( $settings['settings'], $event_args );
+					$is_met = $condition->is_met($settings['settings'], $event_args);
 
-					if ( ! $is_met ) {
+					if ( ! $is_met) {
 						$event_args->ascend();
 						return false;
 					}
 				}
 
-				unset( $sub_args['_conditions'] );
+				unset($sub_args['_conditions']);
 			}
 
-			$are_met = $this->conditions_are_met( $sub_args, $event_args );
+			$are_met = $this->conditions_are_met($sub_args, $event_args);
 
 			$event_args->ascend();
 
-			if ( ! $are_met ) {
+			if ( ! $are_met) {
 				return false;
 			}
 		}
@@ -279,13 +279,13 @@ 
 	 *
 	 * @return string|false The data type, or false.
 	 */
-	protected function get_data_type( $arg ) {
+	protected function get_data_type($arg) {
 
-		if ( $arg instanceof WordPoints_Entity_Attr ) {
+		if ($arg instanceof WordPoints_Entity_Attr) {
 			$data_type = $arg->get_data_type();
-		} elseif ( $arg instanceof WordPoints_Entity_Array ) {
+		} elseif ($arg instanceof WordPoints_Entity_Array) {
 			$data_type = 'entity_array';
-		} elseif ( $arg instanceof WordPoints_Entity ) {
+		} elseif ($arg instanceof WordPoints_Entity) {
 			$data_type = 'entity';
 		} else {
 			$data_type = false;

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

Spacing +46 added lines, -46 removed lines

@@ -27,7 +27,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	protected $action_types = array( 'fire', 'toggle_on', 'toggle_off' );
+	protected $action_types = array('fire', 'toggle_on', 'toggle_off');
 
 	/**
 	 * @since 1.0.0
@@ -58,9 +58,9 @@ 
 	 */
 	public function get_settings_fields() {
 
-		$this->settings_fields['points']['label'] = _x( 'Points', 'form label', 'wordpoints' );
-		$this->settings_fields['log_text']['label'] = _x( 'Log Text', 'form label', 'wordpoints' );
-		$this->settings_fields['description']['label'] = _x( 'Description', 'form label', 'wordpoints' );
+		$this->settings_fields['points']['label'] = _x('Points', 'form label', 'wordpoints');
+		$this->settings_fields['log_text']['label'] = _x('Log Text', 'form label', 'wordpoints');
+		$this->settings_fields['description']['label'] = _x('Description', 'form label', 'wordpoints');
 
 		return parent::get_settings_fields();
 	}
@@ -72,8 +72,8 @@ 
 
 		$data = parent::get_ui_script_data();
 
-		$data['target_label'] = __( 'Award To', 'wordpoints' );
-		$data['periods_label'] = __( 'Award each user no more than once per:', 'wordpoints' );
+		$data['target_label'] = __('Award To', 'wordpoints');
+		$data['periods_label'] = __('Award each user no more than once per:', 'wordpoints');
 
 		return $data;
 	}
@@ -87,23 +87,23 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings['points'] ) || false === wordpoints_int( $settings['points'] ) ) {
-			$validator->add_error( __( 'Points must be an integer.', 'wordpoints' ), 'points' );
+		if ( ! isset($settings['points']) || false === wordpoints_int($settings['points'])) {
+			$validator->add_error(__('Points must be an integer.', 'wordpoints'), 'points');
 		}
 
-		if ( ! isset( $settings['points_type'] ) || ! wordpoints_is_points_type( $settings['points_type'] ) ) {
-			$validator->add_error( __( 'Invalid points type.', 'wordpoints' ), 'points_type' );
+		if ( ! isset($settings['points_type']) || ! wordpoints_is_points_type($settings['points_type'])) {
+			$validator->add_error(__('Invalid points type.', 'wordpoints'), 'points_type');
 		}
 
-		if ( ! isset( $settings['description'] ) ) {
-			$validator->add_error( __( 'Description is required.', 'wordpoints' ), 'description' );
+		if ( ! isset($settings['description'])) {
+			$validator->add_error(__('Description is required.', 'wordpoints'), 'description');
 		}
 
-		if ( ! isset( $settings['log_text'] ) ) {
-			$validator->add_error( __( 'Log Text is required.', 'wordpoints' ), 'log_text' );
+		if ( ! isset($settings['log_text'])) {
+			$validator->add_error(__('Log Text is required.', 'wordpoints'), 'log_text');
 		}
 
-		return parent::validate_settings( $settings, $validator, $event_args );
+		return parent::validate_settings($settings, $validator, $event_args);
 	}
 
 	/**
@@ -114,58 +114,58 @@ 
 		array $settings
 	) {
 
-		parent::update_settings( $reaction, $settings );
+		parent::update_settings($reaction, $settings);
 
-		$reaction->update_meta( 'points', $settings['points'] );
-		$reaction->update_meta( 'points_type', $settings['points_type'] );
-		$reaction->update_meta( 'description', $settings['description'] );
-		$reaction->update_meta( 'log_text', $settings['log_text'] );
+		$reaction->update_meta('points', $settings['points']);
+		$reaction->update_meta('points_type', $settings['points_type']);
+		$reaction->update_meta('description', $settings['description']);
+		$reaction->update_meta('log_text', $settings['log_text']);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function hit( WordPoints_Hook_Fire $fire ) {
+	public function hit(WordPoints_Hook_Fire $fire) {
 
-		if ( 'toggle_off' === $fire->action_type ) {
-			$this->reverse_hit( $fire );
+		if ('toggle_off' === $fire->action_type) {
+			$this->reverse_hit($fire);
 			return;
 		}
 
 		$reaction = $fire->reaction;
 
 		$target = $fire->event_args->get_from_hierarchy(
-			$reaction->get_meta( 'target' )
+			$reaction->get_meta('target')
 		);
 
-		if ( ! $target instanceof WordPoints_Entity ) {
+		if ( ! $target instanceof WordPoints_Entity) {
 			return;
 		}
 
-		$meta = array( 'hook_hit_id' => $fire->hit_id );
+		$meta = array('hook_hit_id' => $fire->hit_id);
 
-		foreach ( $fire->event_args->get_entities() as $entity ) {
-			$meta[ $entity->get_slug() ] = $entity->get_the_id();
+		foreach ($fire->event_args->get_entities() as $entity) {
+			$meta[$entity->get_slug()] = $entity->get_the_id();
 		}
 
 		wordpoints_alter_points(
 			$target->get_the_id()
-			, $reaction->get_meta( 'points' )
-			, $reaction->get_meta( 'points_type' )
+			, $reaction->get_meta('points')
+			, $reaction->get_meta('points_type')
 			, $reaction->get_event_slug()
 			, $meta
-			, $reaction->get_meta( 'log_text' )
+			, $reaction->get_meta('log_text')
 		);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
+	public function reverse_hit(WordPoints_Hook_Fire $fire) {
 
-		$hit_ids = $this->get_hit_ids_to_be_reversed( $fire );
+		$hit_ids = $this->get_hit_ids_to_be_reversed($fire);
 
-		if ( empty( $hit_ids ) ) {
+		if (empty($hit_ids)) {
 			return;
 		}
 
@@ -183,11 +183,11 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
-		$this->reverse_logs( $logs, $fire );
+		$this->reverse_logs($logs, $fire);
 	}
 
 	/**
@@ -199,10 +199,10 @@ 
 	 *
 	 * @return array The IDs of the hits to be reversed.
 	 */
-	protected function get_hit_ids_to_be_reversed( WordPoints_Hook_Fire $fire ) {
+	protected function get_hit_ids_to_be_reversed(WordPoints_Hook_Fire $fire) {
 
 		// We closely integrate with the reversals extension to get the hit IDs.
-		if ( ! isset( $fire->data['reversals']['hit_ids'] ) ) {
+		if ( ! isset($fire->data['reversals']['hit_ids'])) {
 			return array();
 		}
 
@@ -217,28 +217,28 @@ 
 	 * @param object[]             $logs The logs to reverse.
 	 * @param WordPoints_Hook_Fire $fire The fire object.
 	 */
-	protected function reverse_logs( $logs, WordPoints_Hook_Fire $fire ) {
+	protected function reverse_logs($logs, WordPoints_Hook_Fire $fire) {
 
-		$event = wordpoints_hooks()->get_sub_app( 'events' )->get(
+		$event = wordpoints_hooks()->get_sub_app('events')->get(
 			$fire->reaction->get_event_slug()
 		);
 
-		if ( $event instanceof WordPoints_Hook_Event_ReversingI ) {
+		if ($event instanceof WordPoints_Hook_Event_ReversingI) {
 
 			/* translators: 1: log text for transaction that is being reversed, 2: the reason that this is being reversed. */
-			$template = __( 'Reversed “%1$s” (%2$s)', 'wordpoints' );
+			$template = __('Reversed “%1$s” (%2$s)', 'wordpoints');
 
 			$event_description = $event->get_reversal_text();
 
 		} else {
 
 			/* translators: 1: log text for transaction that is being reversed. */
-			$template = __( 'Reversed “%1$s”', 'wordpoints' );
+			$template = __('Reversed “%1$s”', 'wordpoints');
 
 			$event_description = '';
 		}
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
 			$log_id = wordpoints_alter_points(
 				$log->user_id
@@ -249,11 +249,11 @@ 
 					'original_log_id' => $log->id,
 					'hook_hit_id'     => $fire->hit_id,
 				)
-				, sprintf( $template, $log->text, $event_description )
+				, sprintf($template, $log->text, $event_description)
 			);
 
 			// Mark the old log as reversed by this one.
-			wordpoints_update_points_log_meta( $log->id, 'auto_reversed', $log_id );
+			wordpoints_update_points_log_meta($log->id, 'auto_reversed', $log_id);
 		}
 	}
 }

src/includes/classes/hook/actions.php

Spacing +10 added lines, -10 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->router = wordpoints_hooks()->get_sub_app( 'router' );
+		$this->router = wordpoints_hooks()->get_sub_app('router');
 	}
 
 	/**
@@ -40,23 +40,23 @@ 
 	 *
 	 * @return object|false The action object, or false if not found.
 	 */
-	public function get( $slug, array $action_args = array(), array $args = array() ) {
+	public function get($slug, array $action_args = array(), array $args = array()) {
 
-		if ( ! isset( $this->classes[ $slug ] ) ) {
+		if ( ! isset($this->classes[$slug])) {
 			return false;
 		}
 
-		return new $this->classes[ $slug ]( $slug, $action_args, $args );
+		return new $this->classes[$slug]($slug, $action_args, $args);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	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);
 
-		$this->router->add_action( $slug, $args );
+		$this->router->add_action($slug, $args);
 
 		return true;
 	}
@@ -64,11 +64,11 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		parent::deregister( $slug );
+		parent::deregister($slug);
 
-		$this->router->remove_action( $slug );
+		$this->router->remove_action($slug);
 	}
 }
 

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];
 	}
 
 	/**