WordPoints / hooks-api

src/includes/classes/index.php

Spacing +107 added lines, -107 removed lines

@@ -10,113 +10,113 @@
  */
 
 // auto-generated {
-require_once( dirname( __FILE__ ) . 'app.php' );
-require_once( dirname( __FILE__ ) . 'app/registry.php' );
-require_once( dirname( __FILE__ ) . 'class/autoloader.php' );
-require_once( dirname( __FILE__ ) . 'class/registry.php' );
-require_once( dirname( __FILE__ ) . 'class/registry/children.php' );
-require_once( dirname( __FILE__ ) . 'class/registry/childreni.php' );
-require_once( dirname( __FILE__ ) . 'class/registry/persistent.php' );
-require_once( dirname( __FILE__ ) . 'class/registryi.php' );
-require_once( dirname( __FILE__ ) . 'data/type.php' );
-require_once( dirname( __FILE__ ) . 'data/type/integer.php' );
-require_once( dirname( __FILE__ ) . 'data/type/text.php' );
-require_once( dirname( __FILE__ ) . 'data/typei.php' );
-require_once( dirname( __FILE__ ) . 'db/query.php' );
-require_once( dirname( __FILE__ ) . 'entity.php' );
-require_once( dirname( __FILE__ ) . 'entity/array.php' );
-require_once( dirname( __FILE__ ) . 'entity/attr.php' );
-require_once( dirname( __FILE__ ) . 'entity/attr/field.php' );
-require_once( dirname( __FILE__ ) . 'entity/childi.php' );
-require_once( dirname( __FILE__ ) . 'entity/comment.php' );
-require_once( dirname( __FILE__ ) . 'entity/comment/author.php' );
-require_once( dirname( __FILE__ ) . 'entity/comment/post.php' );
-require_once( dirname( __FILE__ ) . 'entity/context.php' );
-require_once( dirname( __FILE__ ) . 'entity/context/network.php' );
-require_once( dirname( __FILE__ ) . 'entity/context/site.php' );
-require_once( dirname( __FILE__ ) . 'entity/enumerablei.php' );
-require_once( dirname( __FILE__ ) . 'entity/hierarchy.php' );
-require_once( dirname( __FILE__ ) . 'entity/hierarchyi.php' );
-require_once( dirname( __FILE__ ) . 'entity/parenti.php' );
-require_once( dirname( __FILE__ ) . 'entity/post.php' );
-require_once( dirname( __FILE__ ) . 'entity/post/author.php' );
-require_once( dirname( __FILE__ ) . 'entity/post/content.php' );
-require_once( dirname( __FILE__ ) . 'entity/post/terms.php' );
-require_once( dirname( __FILE__ ) . 'entity/post/type.php' );
-require_once( dirname( __FILE__ ) . 'entity/post/type/name.php' );
-require_once( dirname( __FILE__ ) . 'entity/post/type/relationship.php' );
-require_once( dirname( __FILE__ ) . 'entity/relationship.php' );
-require_once( dirname( __FILE__ ) . 'entity/relationship/dynamic.php' );
-require_once( dirname( __FILE__ ) . 'entity/relationship/stored/field.php' );
-require_once( dirname( __FILE__ ) . 'entity/restricted/visibilityi.php' );
-require_once( dirname( __FILE__ ) . 'entity/site.php' );
-require_once( dirname( __FILE__ ) . 'entity/stored/array.php' );
-require_once( dirname( __FILE__ ) . 'entity/stored/db/table.php' );
-require_once( dirname( __FILE__ ) . 'entity/term.php' );
-require_once( dirname( __FILE__ ) . 'entity/term/id.php' );
-require_once( dirname( __FILE__ ) . 'entity/user.php' );
-require_once( dirname( __FILE__ ) . 'entity/user/role.php' );
-require_once( dirname( __FILE__ ) . 'entity/user/role/name.php' );
-require_once( dirname( __FILE__ ) . 'entity/user/roles.php' );
-require_once( dirname( __FILE__ ) . 'entityish.php' );
-require_once( dirname( __FILE__ ) . 'entityish/storedi.php' );
-require_once( dirname( __FILE__ ) . 'entityishi.php' );
-require_once( dirname( __FILE__ ) . 'hierarchy.php' );
-require_once( dirname( __FILE__ ) . 'hook/action.php' );
-require_once( dirname( __FILE__ ) . 'hook/action/comment/new.php' );
-require_once( dirname( __FILE__ ) . 'hook/action/post/publish.php' );
-require_once( dirname( __FILE__ ) . 'hook/actioni.php' );
-require_once( dirname( __FILE__ ) . 'hook/actions.php' );
-require_once( dirname( __FILE__ ) . 'hook/arg.php' );
-require_once( dirname( __FILE__ ) . 'hook/arg/current/post.php' );
-require_once( dirname( __FILE__ ) . 'hook/arg/current/site.php' );
-require_once( dirname( __FILE__ ) . 'hook/arg/current/user.php' );
-require_once( dirname( __FILE__ ) . 'hook/arg/dynamic.php' );
-require_once( dirname( __FILE__ ) . 'hook/condition.php' );
-require_once( dirname( __FILE__ ) . 'hook/condition/entity/array/contains.php' );
-require_once( dirname( __FILE__ ) . 'hook/condition/equals.php' );
-require_once( dirname( __FILE__ ) . 'hook/condition/string/contains.php' );
-require_once( dirname( __FILE__ ) . 'hook/conditioni.php' );
-require_once( dirname( __FILE__ ) . 'hook/event.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/args.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/comment/leave.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/dynamic.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/media/upload.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/post/publish.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/user/register.php' );
-require_once( dirname( __FILE__ ) . 'hook/event/user/visit.php' );
-require_once( dirname( __FILE__ ) . 'hook/eventi.php' );
-require_once( dirname( __FILE__ ) . 'hook/events.php' );
-require_once( dirname( __FILE__ ) . 'hook/extension.php' );
-require_once( dirname( __FILE__ ) . 'hook/extension/blocker.php' );
-require_once( dirname( __FILE__ ) . 'hook/extension/conditions.php' );
-require_once( dirname( __FILE__ ) . 'hook/extension/periods.php' );
-require_once( dirname( __FILE__ ) . 'hook/fire.php' );
-require_once( dirname( __FILE__ ) . 'hook/hit/logger.php' );
-require_once( dirname( __FILE__ ) . 'hook/hit/query.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction/options.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction/store.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction/store/options.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction/store/options/network.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction/storei.php' );
-require_once( dirname( __FILE__ ) . 'hook/reaction/validator.php' );
-require_once( dirname( __FILE__ ) . 'hook/reactioni.php' );
-require_once( dirname( __FILE__ ) . 'hook/reactor.php' );
-require_once( dirname( __FILE__ ) . 'hook/reactor/points.php' );
-require_once( dirname( __FILE__ ) . 'hook/retroactive/conditions.php' );
-require_once( dirname( __FILE__ ) . 'hook/retroactive/query.php' );
-require_once( dirname( __FILE__ ) . 'hook/retroactive/query/modifieri.php' );
-require_once( dirname( __FILE__ ) . 'hook/retroactive/queryable.php' );
-require_once( dirname( __FILE__ ) . 'hook/retroactive/queryi.php' );
-require_once( dirname( __FILE__ ) . 'hook/router.php' );
-require_once( dirname( __FILE__ ) . 'hook/settings.php' );
-require_once( dirname( __FILE__ ) . 'hook/settingsi.php' );
-require_once( dirname( __FILE__ ) . 'hook/validator/exception.php' );
-require_once( dirname( __FILE__ ) . 'hooks.php' );
-require_once( dirname( __FILE__ ) . 'query/builder/db/mysql.php' );
-require_once( dirname( __FILE__ ) . 'spec.php' );
-require_once( dirname( __FILE__ ) . 'specedi.php' );
+require_once(dirname(__FILE__).'app.php');
+require_once(dirname(__FILE__).'app/registry.php');
+require_once(dirname(__FILE__).'class/autoloader.php');
+require_once(dirname(__FILE__).'class/registry.php');
+require_once(dirname(__FILE__).'class/registry/children.php');
+require_once(dirname(__FILE__).'class/registry/childreni.php');
+require_once(dirname(__FILE__).'class/registry/persistent.php');
+require_once(dirname(__FILE__).'class/registryi.php');
+require_once(dirname(__FILE__).'data/type.php');
+require_once(dirname(__FILE__).'data/type/integer.php');
+require_once(dirname(__FILE__).'data/type/text.php');
+require_once(dirname(__FILE__).'data/typei.php');
+require_once(dirname(__FILE__).'db/query.php');
+require_once(dirname(__FILE__).'entity.php');
+require_once(dirname(__FILE__).'entity/array.php');
+require_once(dirname(__FILE__).'entity/attr.php');
+require_once(dirname(__FILE__).'entity/attr/field.php');
+require_once(dirname(__FILE__).'entity/childi.php');
+require_once(dirname(__FILE__).'entity/comment.php');
+require_once(dirname(__FILE__).'entity/comment/author.php');
+require_once(dirname(__FILE__).'entity/comment/post.php');
+require_once(dirname(__FILE__).'entity/context.php');
+require_once(dirname(__FILE__).'entity/context/network.php');
+require_once(dirname(__FILE__).'entity/context/site.php');
+require_once(dirname(__FILE__).'entity/enumerablei.php');
+require_once(dirname(__FILE__).'entity/hierarchy.php');
+require_once(dirname(__FILE__).'entity/hierarchyi.php');
+require_once(dirname(__FILE__).'entity/parenti.php');
+require_once(dirname(__FILE__).'entity/post.php');
+require_once(dirname(__FILE__).'entity/post/author.php');
+require_once(dirname(__FILE__).'entity/post/content.php');
+require_once(dirname(__FILE__).'entity/post/terms.php');
+require_once(dirname(__FILE__).'entity/post/type.php');
+require_once(dirname(__FILE__).'entity/post/type/name.php');
+require_once(dirname(__FILE__).'entity/post/type/relationship.php');
+require_once(dirname(__FILE__).'entity/relationship.php');
+require_once(dirname(__FILE__).'entity/relationship/dynamic.php');
+require_once(dirname(__FILE__).'entity/relationship/stored/field.php');
+require_once(dirname(__FILE__).'entity/restricted/visibilityi.php');
+require_once(dirname(__FILE__).'entity/site.php');
+require_once(dirname(__FILE__).'entity/stored/array.php');
+require_once(dirname(__FILE__).'entity/stored/db/table.php');
+require_once(dirname(__FILE__).'entity/term.php');
+require_once(dirname(__FILE__).'entity/term/id.php');
+require_once(dirname(__FILE__).'entity/user.php');
+require_once(dirname(__FILE__).'entity/user/role.php');
+require_once(dirname(__FILE__).'entity/user/role/name.php');
+require_once(dirname(__FILE__).'entity/user/roles.php');
+require_once(dirname(__FILE__).'entityish.php');
+require_once(dirname(__FILE__).'entityish/storedi.php');
+require_once(dirname(__FILE__).'entityishi.php');
+require_once(dirname(__FILE__).'hierarchy.php');
+require_once(dirname(__FILE__).'hook/action.php');
+require_once(dirname(__FILE__).'hook/action/comment/new.php');
+require_once(dirname(__FILE__).'hook/action/post/publish.php');
+require_once(dirname(__FILE__).'hook/actioni.php');
+require_once(dirname(__FILE__).'hook/actions.php');
+require_once(dirname(__FILE__).'hook/arg.php');
+require_once(dirname(__FILE__).'hook/arg/current/post.php');
+require_once(dirname(__FILE__).'hook/arg/current/site.php');
+require_once(dirname(__FILE__).'hook/arg/current/user.php');
+require_once(dirname(__FILE__).'hook/arg/dynamic.php');
+require_once(dirname(__FILE__).'hook/condition.php');
+require_once(dirname(__FILE__).'hook/condition/entity/array/contains.php');
+require_once(dirname(__FILE__).'hook/condition/equals.php');
+require_once(dirname(__FILE__).'hook/condition/string/contains.php');
+require_once(dirname(__FILE__).'hook/conditioni.php');
+require_once(dirname(__FILE__).'hook/event.php');
+require_once(dirname(__FILE__).'hook/event/args.php');
+require_once(dirname(__FILE__).'hook/event/comment/leave.php');
+require_once(dirname(__FILE__).'hook/event/dynamic.php');
+require_once(dirname(__FILE__).'hook/event/media/upload.php');
+require_once(dirname(__FILE__).'hook/event/post/publish.php');
+require_once(dirname(__FILE__).'hook/event/user/register.php');
+require_once(dirname(__FILE__).'hook/event/user/visit.php');
+require_once(dirname(__FILE__).'hook/eventi.php');
+require_once(dirname(__FILE__).'hook/events.php');
+require_once(dirname(__FILE__).'hook/extension.php');
+require_once(dirname(__FILE__).'hook/extension/blocker.php');
+require_once(dirname(__FILE__).'hook/extension/conditions.php');
+require_once(dirname(__FILE__).'hook/extension/periods.php');
+require_once(dirname(__FILE__).'hook/fire.php');
+require_once(dirname(__FILE__).'hook/hit/logger.php');
+require_once(dirname(__FILE__).'hook/hit/query.php');
+require_once(dirname(__FILE__).'hook/reaction.php');
+require_once(dirname(__FILE__).'hook/reaction/options.php');
+require_once(dirname(__FILE__).'hook/reaction/store.php');
+require_once(dirname(__FILE__).'hook/reaction/store/options.php');
+require_once(dirname(__FILE__).'hook/reaction/store/options/network.php');
+require_once(dirname(__FILE__).'hook/reaction/storei.php');
+require_once(dirname(__FILE__).'hook/reaction/validator.php');
+require_once(dirname(__FILE__).'hook/reactioni.php');
+require_once(dirname(__FILE__).'hook/reactor.php');
+require_once(dirname(__FILE__).'hook/reactor/points.php');
+require_once(dirname(__FILE__).'hook/retroactive/conditions.php');
+require_once(dirname(__FILE__).'hook/retroactive/query.php');
+require_once(dirname(__FILE__).'hook/retroactive/query/modifieri.php');
+require_once(dirname(__FILE__).'hook/retroactive/queryable.php');
+require_once(dirname(__FILE__).'hook/retroactive/queryi.php');
+require_once(dirname(__FILE__).'hook/router.php');
+require_once(dirname(__FILE__).'hook/settings.php');
+require_once(dirname(__FILE__).'hook/settingsi.php');
+require_once(dirname(__FILE__).'hook/validator/exception.php');
+require_once(dirname(__FILE__).'hooks.php');
+require_once(dirname(__FILE__).'query/builder/db/mysql.php');
+require_once(dirname(__FILE__).'spec.php');
+require_once(dirname(__FILE__).'specedi.php');
 // }
 
 // EOF

src/includes/classes/hook/reactor.php

Spacing +12 added lines, -12 removed lines

@@ -131,21 +131,21 @@ 
 	) {
 
 		if (
-			empty( $settings['target'] )
-			|| ! is_array( $settings['target'] )
+			empty($settings['target'])
+			|| ! is_array($settings['target'])
 		) {
 
-			$validator->add_error( __( 'Invalid target.', 'wordpoints' ), 'target' );
+			$validator->add_error(__('Invalid target.', 'wordpoints'), 'target');
 
 		} else {
 
-			$target = $event_args->get_from_hierarchy( $settings['target'] );
+			$target = $event_args->get_from_hierarchy($settings['target']);
 
 			if (
 				! $target instanceof WordPoints_Entity
-				|| ! in_array( $target->get_slug(), (array) $this->arg_types )
+				|| ! in_array($target->get_slug(), (array) $this->arg_types)
 			) {
-				$validator->add_error( __( 'Invalid target.', 'wordpoints' ), 'target' );
+				$validator->add_error(__('Invalid target.', 'wordpoints'), 'target');
 			}
 		}
 
@@ -155,8 +155,8 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_settings( WordPoints_Hook_ReactionI $reaction, array $settings ) {
-		$reaction->update_meta( 'target', $settings['target'] );
+	public function update_settings(WordPoints_Hook_ReactionI $reaction, array $settings) {
+		$reaction->update_meta('target', $settings['target']);
 	}
 
 	/**
@@ -167,11 +167,11 @@ 
 	 * @param string               $hit_type The type of hit to perform.
 	 * @param WordPoints_Hook_Fire $fire     The hook fire object.
 	 */
-	public function hit( $hit_type, WordPoints_Hook_Fire $fire ) {
+	public function hit($hit_type, WordPoints_Hook_Fire $fire) {
 
-		if ( in_array( $hit_type, (array) $this->hit_types ) ) {
+		if (in_array($hit_type, (array) $this->hit_types)) {
 
-			call_user_func( array( $this, $hit_type ), $fire );
+			call_user_func(array($this, $hit_type), $fire);
 
 		} else {
 
@@ -183,7 +183,7 @@ 
 			 * @param string               $hit_type The type of hit to perform.
 			 * @param WordPoints_Hook_Fire $fire     The hook fire object.
 			 */
-			do_action( "wordpoints_hook_reactor_hit-{$this->slug}", $hit_type, $fire );
+			do_action("wordpoints_hook_reactor_hit-{$this->slug}", $hit_type, $fire);
 		}
 	}
 }

src/includes/classes/hook/router.php

Spacing +70 added lines, -70 removed lines

@@ -86,13 +86,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];
 		}
 
@@ -109,9 +109,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;
 		}
 
@@ -119,7 +119,7 @@ 
 		// 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->actions ) ) {
+		if ( ! isset($this->actions)) {
 
 			$hooks = wordpoints_hooks();
 
@@ -127,38 +127,38 @@ 
 			$this->actions = $hooks->actions;
 		}
 
-		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->events->args->get_children( $event_slug, array( $action_object ) );
+					$event_args = $this->events->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->fire_event( $type, $event_slug, $event_args );
+					$this->fire_event($type, $event_slug, $event_args);
 				}
 			}
 		}
@@ -182,19 +182,19 @@ 
 
 		$hooks = wordpoints_hooks();
 
-		foreach ( $hooks->reaction_stores->get_all() as $reaction_stores ) {
-			foreach ( $reaction_stores as $reaction_store ) {
+		foreach ($hooks->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(
 						$action_type
@@ -202,7 +202,7 @@ 
 						, $reaction
 					);
 
-					$this->fire_reaction( $fire );
+					$this->fire_reaction($fire);
 				}
 			}
 		}
@@ -215,32 +215,32 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	protected function fire_reaction( $fire ) {
+	protected function fire_reaction($fire) {
 
 		$hooks = wordpoints_hooks();
 
 		$reactor_slug = $fire->reaction->get_reactor_slug();
 
-		if ( ! isset( $this->reactor_index[ $reactor_slug ][ $fire->action_type ] ) ) {
+		if ( ! isset($this->reactor_index[$reactor_slug][$fire->action_type])) {
 			return;
 		}
 
-		$hit_type = $this->reactor_index[ $reactor_slug ][ $fire->action_type ];
+		$hit_type = $this->reactor_index[$reactor_slug][$fire->action_type];
 
-		$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_Extension[] $extensions */
 		$extensions = $hooks->extensions->get_all();
 
-		foreach ( $extensions as $extension ) {
-			if ( ! $extension->should_hit( $fire ) ) {
+		foreach ($extensions as $extension) {
+			if ( ! $extension->should_hit($fire)) {
 				return;
 			}
 		}
@@ -248,12 +248,12 @@ 
 		$fire->hit();
 
 		/** @var WordPoints_Hook_Reactor $reactor */
-		$reactor = $hooks->reactors->get( $reactor_slug );
+		$reactor = $hooks->reactors->get($reactor_slug);
 
-		$reactor->hit( $hit_type, $fire );
+		$reactor->hit($hit_type, $fire);
 
-		foreach ( $extensions as $extension ) {
-			$extension->after_hit( $fire );
+		foreach ($extensions as $extension) {
+			$extension->after_hit($fire);
 		}
 	}
 
@@ -280,56 +280,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);
 	}
 
 	/**
@@ -339,20 +339,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);
 				}
 			}
 		}
@@ -367,8 +367,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;
 	}
 
 	/**
@@ -380,8 +380,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]);
 	}
 
 	/**
@@ -394,8 +394,8 @@ 
 	 * @param string $hit_type     The type of hit the reactor should perform when
 	 *                             hit by this type of event.
 	 */
-	public function add_action_type_to_reactor( $action_type, $reactor_slug, $hit_type ) {
-		$this->reactor_index[ $reactor_slug ][ $action_type ] = $hit_type;
+	public function add_action_type_to_reactor($action_type, $reactor_slug, $hit_type) {
+		$this->reactor_index[$reactor_slug][$action_type] = $hit_type;
 	}
 
 	/**
@@ -406,8 +406,8 @@ 
 	 * @param string $action_type  The slug of the action type.
 	 * @param string $reactor_slug The slug of the reactor.
 	 */
-	public function remove_action_type_from_reactor( $action_type, $reactor_slug ) {
-		unset( $this->reactor_index[ $reactor_slug ][ $action_type ] );
+	public function remove_action_type_from_reactor($action_type, $reactor_slug) {
+		unset($this->reactor_index[$reactor_slug][$action_type]);
 	}
 
 	/**
@@ -419,7 +419,7 @@ 
 	 */
 	public function get_event_index() {
 
-		if ( empty( $this->reactor_index ) ) {
+		if (empty($this->reactor_index)) {
 			wordpoints_hooks()->events;
 		}
 
@@ -435,7 +435,7 @@ 
 	 */
 	public function get_reactor_index() {
 
-		if ( empty( $this->reactor_index ) ) {
+		if (empty($this->reactor_index)) {
 			wordpoints_hooks()->reactors;
 		}
 

src/includes/classes/hook/reaction.php

Spacing +5 added lines, -5 removed lines

@@ -35,18 +35,18 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	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;
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
-		if ( 'ID' === $var ) {
+		if ('ID' === $var) {
 			return $this->ID;
 		}
 
@@ -69,7 +69,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_reactor_slug() {
-		return $this->get_meta( 'reactor' ); // TODO remove this method?
+		return $this->get_meta('reactor'); // TODO remove this method?
 	}
 
 	/**

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

Spacing +156 added lines, -156 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,32 +245,32 @@ 
 
 		$event_slug = $this->reaction->get_event_slug();
 
-		$event = $this->hooks->events->get( $event_slug );
+		$event = $this->hooks->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 ( $this->hooks->events->args->get_children( $event_slug ) as $arg ) {
+		foreach ($this->hooks->events->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->reactors->get( $this->reaction->get_reactor_slug() );
+		$reactor = $this->hooks->reactors->get($this->reaction->get_reactor_slug());
 
-		$reactor->modify_retroactive_query( $this );
+		$reactor->modify_retroactive_query($this);
 		$this->reset();
 
-		foreach ( $this->hooks->extensions->get_all() as $extension ) {
+		foreach ($this->hooks->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();
 			}
 		}
@@ -278,125 +278,125 @@ 
 
 	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');
 
-		$executor = $this->hooks->retroactive_query_executors->get( $storage_type );
+		$executor = $this->hooks->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->reactors->get( $this->reaction->get_reactor_slug() );
+		$reactor = $this->hooks->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->extensions->get_all() as $extension ) {
-			if ( $extension instanceof WordPoints_Hook_Retroactive_Query_FilterI ) {
-				$extension->filter_retroactive_query( $this );
+		foreach ($this->hooks->extensions->get_all() as $extension) {
+			if ($extension instanceof WordPoints_Hook_Retroactive_Query_FilterI) {
+				$extension->filter_retroactive_query($this);
 			}
 		}
 	}
@@ -405,27 +405,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();
@@ -435,10 +435,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 );
 	}
@@ -450,40 +450,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;
@@ -491,7 +491,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();
@@ -508,11 +508,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);
 }
 
 /*
@@ -541,48 +541,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);
 				}
 			}
 
@@ -590,9 +590,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;
@@ -602,7 +602,7 @@ 
 	}
 }
 
-wordpoints_hooks()->retroactive_query_executors->register( 'array', 'WordPoints_Hook_Retroactive_Query_Executor_Array' );
+wordpoints_hooks()->retroactive_query_executors->register('array', 'WordPoints_Hook_Retroactive_Query_Executor_Array');
 
 class WordPoints_Hook_Retroactive_Query_Executor_MySQL
 	implements WordPoints_Hook_Retroactive_Query_ExecutorI {
@@ -621,7 +621,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.
@@ -631,69 +631,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'];
 			}
 
@@ -705,26 +705,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.
@@ -732,43 +732,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.
@@ -784,7 +784,7 @@ 
 }
 
 
-wordpoints_hooks()->retroactive_query_executors->register( 'db', 'WordPoints_Hook_Retroactive_Query_Executor_MySQL' );
+wordpoints_hooks()->retroactive_query_executors->register('db', 'WordPoints_Hook_Retroactive_Query_Executor_MySQL');
 
 
 // EOF

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

Spacing +37 added lines, -37 removed lines

@@ -27,7 +27,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	protected $hit_types = array( 'award', 'reverse' );
+	protected $hit_types = array('award', 'reverse');
 
 	/**
 	 * @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,12 +114,12 @@ 
 		array $settings
 	) {
 
-		parent::update_settings( $reaction, $settings );
+		parent::update_settings($reaction, $settings);
 
-		$reaction->update_meta( 'points', $settings['points'] );
-		$reaction->update_meta( 'points_type', $settings['points_type'] );
-		$reaction->update_meta( 'description', $settings['description'] );
-		$reaction->update_meta( 'log_text', $settings['log_text'] );
+		$reaction->update_meta('points', $settings['points']);
+		$reaction->update_meta('points_type', $settings['points_type']);
+		$reaction->update_meta('description', $settings['description']);
+		$reaction->update_meta('log_text', $settings['log_text']);
 	}
 
 	/**
@@ -129,31 +129,31 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	protected function award( WordPoints_Hook_Fire $fire ) {
+	protected function award(WordPoints_Hook_Fire $fire) {
 
 		$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();
 
-		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')
 		);
 	}
 
@@ -164,7 +164,7 @@ 
 	 *
 	 * @param WordPoints_Hook_Fire $fire The fire object.
 	 */
-	protected function reverse( WordPoints_Hook_Fire $fire ) {
+	protected function reverse(WordPoints_Hook_Fire $fire) {
 
 		$meta_queries = array(
 			array(
@@ -175,7 +175,7 @@ 
 			),
 		);
 
-		foreach ( $fire->event_args->get_entities() as $slug => $entity ) {
+		foreach ($fire->event_args->get_entities() as $slug => $entity) {
 
 			$meta_queries[] = array(
 				'key'   => $slug,
@@ -192,35 +192,35 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
 		global $wpdb;
 
-		add_filter( 'wordpoints_points_log', '__return_false' );
+		add_filter('wordpoints_points_log', '__return_false');
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
 			wordpoints_alter_points(
 				$log->user_id
 				, -$log->points
 				, $log->points_type
 				, "reverse-{$log->log_type}"
-				, array( 'original_log_id' => $log->id )
+				, array('original_log_id' => $log->id)
 			);
 
-			wordpoints_points_log_delete_all_metadata( $log->id );
+			wordpoints_points_log_delete_all_metadata($log->id);
 
 			// Now delete the log.
 			$wpdb->delete(
 				$wpdb->wordpoints_points_logs
-				, array( 'id' => $log->id )
+				, array('id' => $log->id)
 				, '%d'
 			); // WPCS: cache OK, cleaned by wordpoints_alter_points().
 		}
 
-		remove_filter( 'wordpoints_points_log', '__return_false' );
+		remove_filter('wordpoints_points_log', '__return_false');
 	}
 }
 

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

Spacing +27 added lines, -27 removed lines

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

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

Spacing +32 added lines, -32 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,16 +146,16 @@ 
 			$fail_fast = $this->fail_fast;
 			$this->fail_fast = true;
 
-			if ( ! isset( $this->event_slug ) ) {
-				$this->add_error( __( 'Event is missing.', 'wordpoints' ), 'event' );
-			} elseif ( ! $this->hooks->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 ( ! $this->hooks->events->is_registered($this->event_slug)) {
+				$this->add_error(__('Event is invalid.', 'wordpoints'), 'event');
 			}
 
-			if ( ! isset( $this->reactor_slug ) ) {
-				$this->add_error( __( 'Reactor is missing.', 'wordpoints' ), 'reactor' );
-			} elseif ( ! $this->hooks->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 ( ! $this->hooks->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
@@ -166,16 +166,16 @@ 
 				$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 = $this->hooks->reactors->get( $this->reactor_slug );
+			$reactor = $this->hooks->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_Extension $extension */
-			foreach ( $this->hooks->extensions->get_all() as $extension ) {
-				$this->settings = $extension->validate_settings( $this->settings, $this, $this->event_args );
+			foreach ($this->hooks->extensions->get_all() as $extension) {
+				$this->settings = $extension->validate_settings($this->settings, $this, $this->event_args);
 			}
 
 			/**
@@ -185,12 +185,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;
@@ -218,17 +218,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;
 		}
 	}
@@ -241,7 +241,7 @@ 
 	 * @return bool Whether the validator found any errors.
 	 */
 	public function had_errors() {
-		return ! empty( $this->errors );
+		return ! empty($this->errors);
 	}
 
 	/**
@@ -262,7 +262,7 @@ 
 	 *
 	 * @param string $field The field.
 	 */
-	public function push_field( $field ) {
+	public function push_field($field) {
 		$this->field_stack[] = $field;
 	}
 
@@ -272,7 +272,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function pop_field() {
-		array_pop( $this->field_stack );
+		array_pop($this->field_stack);
 	}
 
 	/**
@@ -317,7 +317,7 @@ 
 	 */
 	public function get_id() {
 
-		if ( ! $this->reaction ) {
+		if ( ! $this->reaction) {
 			return false;
 		}
 
@@ -355,13 +355,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/storei.php

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,7 @@ 
 	 *
 	 * @param string $slug The slug of this store.
 	 */
-	public function __construct( $slug );
+	public function __construct($slug);
 
 	/**
 	 * Get the slug of this reaction store.
@@ -59,7 +59,7 @@ 
 	 *
 	 * @return bool Whether the reaction exists.
 	 */
-	public function reaction_exists( $id );
+	public function reaction_exists($id);
 
 	/**
 	 * Get an reaction object.
@@ -70,7 +70,7 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI|false The reaction, or false if nonexistent.
 	 */
-	public function get_reaction( $id );
+	public function get_reaction($id);
 
 	/**
 	 * Create an reaction.
@@ -83,7 +83,7 @@ 
 	 *         The reaction object if created successfully. False or a validator
 	 *         instance if not.
 	 */
-	public function create_reaction( array $settings );
+	public function create_reaction(array $settings);
 
 	/**
 	 * Update an reaction.
@@ -97,7 +97,7 @@ 
 	 *         The reaction object if updated successfully. False or a validator
 	 *         instance if not.
 	 */
-	public function update_reaction( $id, array $settings );
+	public function update_reaction($id, array $settings);
 
 	/**
 	 * Delete an reaction.
@@ -108,7 +108,7 @@ 
 	 *
 	 * @return bool Whether the reaction was deleted successfully.
 	 */
-	public function delete_reaction( $id );
+	public function delete_reaction($id);
 
 	/**
 	 * Get all hook reactions for the reactor.
@@ -131,7 +131,7 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI[]
 	 */
-	public function get_reactions_to_event( $event_slug );
+	public function get_reactions_to_event($event_slug);
 }
 
 // EOF