WordPoints / hooks-api

src/includes/functions.php

Spacing +114 added lines, -114 removed lines

@@ -33,9 +33,9 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $reactors The reactors registry.
  */
-function wordpoints_hook_reactors_init( $reactors ) {
+function wordpoints_hook_reactors_init($reactors) {
 
-	$reactors->register( 'points', 'WordPoints_Hook_Reactor_Points' );
+	$reactors->register('points', 'WordPoints_Hook_Reactor_Points');
 }
 
 /**
@@ -47,10 +47,10 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $extensions The extension registry.
  */
-function wordpoints_hook_extension_init( $extensions ) {
+function wordpoints_hook_extension_init($extensions) {
 
-	$extensions->register( 'conditions', 'WordPoints_Hook_Extension_Conditions' );
-	$extensions->register( 'periods', 'WordPoints_Hook_Extension_Periods' );
+	$extensions->register('conditions', 'WordPoints_Hook_Extension_Conditions');
+	$extensions->register('periods', 'WordPoints_Hook_Extension_Periods');
 }
 
 /**
@@ -62,7 +62,7 @@ 
  *
  * @param WordPoints_Class_Registry_Children $conditions The conditions registry.
  */
-function wordpoints_hook_conditions_init( $conditions ) {
+function wordpoints_hook_conditions_init($conditions) {
 
 	$conditions->register(
 		'text'
@@ -98,7 +98,7 @@ 
  *
  * @param WordPoints_Hook_Actions $actions The action registry.
  */
-function wordpoints_hook_actions_init( $actions ) {
+function wordpoints_hook_actions_init($actions) {
 
 	$actions->register(
 		'comment_approve'
@@ -106,8 +106,8 @@ 
 		, array(
 			'action' => 'transition_comment_status',
 			'data'   => array(
-				'arg_index'    => array( 'comment' => 2 ),
-				'requirements' => array( 0 => 'approved' ),
+				'arg_index'    => array('comment' => 2),
+				'requirements' => array(0 => 'approved'),
 			),
 		)
 	);
@@ -118,7 +118,7 @@ 
 		, array(
 			'action' => 'wp_insert_comment',
 			'data'   => array(
-				'arg_index' => array( 'comment' => 1 ),
+				'arg_index' => array('comment' => 1),
 			),
 		)
 	);
@@ -129,8 +129,8 @@ 
 		, array(
 			'action' => 'transition_comment_status',
 			'data'   => array(
-				'arg_index' => array( 'comment' => 2 ),
-				'requirements' => array( 1 => 'approved' ),
+				'arg_index' => array('comment' => 2),
+				'requirements' => array(1 => 'approved'),
 			),
 		)
 	);
@@ -142,8 +142,8 @@ 
 		, array(
 			'action' => 'transition_post_status',
 			'data'   => array(
-				'arg_index' => array( 'post' => 2 ),
-				'requirements' => array( 0 => 'publish' ),
+				'arg_index' => array('post' => 2),
+				'requirements' => array(0 => 'publish'),
 			),
 		)
 	);
@@ -154,7 +154,7 @@ 
 		, array(
 			'action' => 'add_attachment',
 			'data'   => array(
-				'arg_index' => array( 'post\attachment' => 0 ),
+				'arg_index' => array('post\attachment' => 0),
 			),
 		)
 	);
@@ -165,7 +165,7 @@ 
 		, array(
 			'action' => 'post_delete',
 			'data'   => array(
-				'arg_index' => array( 'post' => 0 ),
+				'arg_index' => array('post' => 0),
 			),
 		)
 	);
@@ -176,7 +176,7 @@ 
 		, array(
 			'action' => 'user_register',
 			'data'   => array(
-				'arg_index' => array( 'user' => 0 ),
+				'arg_index' => array('user' => 0),
 			),
 		)
 	);
@@ -187,7 +187,7 @@ 
 		, array(
 			'action' => is_multisite() ? 'wpmu_delete_user' : 'delete_user',
 			'data'   => array(
-				'arg_index' => array( 'user' => 0 ),
+				'arg_index' => array('user' => 0),
 			),
 		)
 	);
@@ -210,7 +210,7 @@ 
  *
  * @param WordPoints_Hook_Events $events The event registry.
  */
-function wordpoints_hook_events_init( $events ) {
+function wordpoints_hook_events_init($events) {
 
 	$events->register(
 		'user_register'
@@ -240,7 +240,7 @@ 
 	);
 
 	// Register events for all of the public post types.
-	$post_types = get_post_types( array( 'public' => true ) );
+	$post_types = get_post_types(array('public' => true));
 
 	/**
 	 * Filter which post types to register hook events for.
@@ -249,20 +249,20 @@ 
 	 *
 	 * @param string[] The post type slugs ("names").
 	 */
-	$post_types = apply_filters( 'wordpoints_register_hook_events_for_post_types', $post_types );
+	$post_types = apply_filters('wordpoints_register_hook_events_for_post_types', $post_types);
 
-	foreach ( $post_types as $slug ) {
-		wordpoints_register_post_type_hook_events( $slug );
+	foreach ($post_types as $slug) {
+		wordpoints_register_post_type_hook_events($slug);
 	}
 
-	if ( is_multisite() ) {
+	if (is_multisite()) {
 
 		$event_slugs = array(
 			'user_visit',
 			'user_register',
 		);
 
-		foreach ( $event_slugs as $event_slug ) {
+		foreach ($event_slugs as $event_slug) {
 			// TODO network hooks
 			$events->args->register(
 				$event_slug
@@ -282,11 +282,11 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $firers The firer registry.
  */
-function wordpoints_hook_firers_init( $firers ) {
+function wordpoints_hook_firers_init($firers) {
 
-	$firers->register( 'fire', 'WordPoints_Hook_Firer' );
-	$firers->register( 'reverse', 'WordPoints_Hook_Firer_Reverse' );
-	$firers->register( 'spam', 'WordPoints_Hook_Firer_Spam' );
+	$firers->register('fire', 'WordPoints_Hook_Firer');
+	$firers->register('reverse', 'WordPoints_Hook_Firer_Reverse');
+	$firers->register('spam', 'WordPoints_Hook_Firer_Spam');
 }
 
 /**
@@ -298,10 +298,10 @@ 
  *
  * @param WordPoints_App_Registry $entities The entities app.
  */
-function wordpoints_entities_app_init( $entities ) {
+function wordpoints_entities_app_init($entities) {
 
-	$entities->sub_apps->register( 'children', 'WordPoints_Class_Registry_Children' );
-	$entities->sub_apps->register( 'contexts', 'WordPoints_Class_Registry' );
+	$entities->sub_apps->register('children', 'WordPoints_Class_Registry_Children');
+	$entities->sub_apps->register('contexts', 'WordPoints_Class_Registry');
 }
 
 /**
@@ -313,10 +313,10 @@ 
  *
  * @param WordPoints_Class_Registry $contexts The entity context registry.
  */
-function wordpoints_entity_contexts_init( $contexts ) {
+function wordpoints_entity_contexts_init($contexts) {
 
-	$contexts->register( 'network', 'WordPoints_Entity_Context_Network' );
-	$contexts->register( 'site', 'WordPoints_Entity_Context_Site' );
+	$contexts->register('network', 'WordPoints_Entity_Context_Network');
+	$contexts->register('site', 'WordPoints_Entity_Context_Site');
 }
 
 /**
@@ -328,18 +328,18 @@ 
  *
  * @param WordPoints_App_Registry $entities The entities app.
  */
-function wordpoints_entities_init( $entities ) {
+function wordpoints_entities_init($entities) {
 
 	$children = $entities->children;
 
-	$entities->register( 'user', 'WordPoints_Entity_User' );
-	$children->register( 'user', 'roles', 'WordPoints_Entity_User_Roles' );
+	$entities->register('user', 'WordPoints_Entity_User');
+	$children->register('user', 'roles', 'WordPoints_Entity_User_Roles');
 
-	$entities->register( 'user_role', 'WordPoints_Entity_User_Role' );
-	$children->register( 'user_role', 'name', 'WordPoints_Entity_User_Role_Name' );
+	$entities->register('user_role', 'WordPoints_Entity_User_Role');
+	$children->register('user_role', 'name', 'WordPoints_Entity_User_Role_Name');
 
 	// Register entities for all of the public post types.
-	$post_types = get_post_types( array( 'public' => true ) );
+	$post_types = get_post_types(array('public' => true));
 
 	/**
 	 * Filter which post types to register entities for.
@@ -348,14 +348,14 @@ 
 	 *
 	 * @param string[] The post type slugs ("names").
 	 */
-	$post_types = apply_filters( 'wordpoints_register_entities_for_post_types', $post_types );
+	$post_types = apply_filters('wordpoints_register_entities_for_post_types', $post_types);
 
-	foreach ( $post_types as $slug ) {
-		wordpoints_register_post_type_entities( $slug );
+	foreach ($post_types as $slug) {
+		wordpoints_register_post_type_entities($slug);
 	}
 
 	// Register entities for all of the public taxonomies.
-	$taxonomies = get_taxonomies( array( 'public' => true ) );
+	$taxonomies = get_taxonomies(array('public' => true));
 
 	/**
 	 * Filter which taxonomies to register entities for.
@@ -364,10 +364,10 @@ 
 	 *
 	 * @param string[] The taxonomy slugs.
 	 */
-	$taxonomies = apply_filters( 'wordpoints_register_entities_for_taxonomies', $taxonomies );
+	$taxonomies = apply_filters('wordpoints_register_entities_for_taxonomies', $taxonomies);
 
-	foreach ( $taxonomies as $slug ) {
-		wordpoints_register_taxonomy_entities( $slug );
+	foreach ($taxonomies as $slug) {
+		wordpoints_register_taxonomy_entities($slug);
 	}
 }
 
@@ -378,28 +378,28 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_register_post_type_entities( $slug ) {
+function wordpoints_register_post_type_entities($slug) {
 
 	$entities = wordpoints_entities();
 	$children = $entities->children;
 
-	$entities->register( "post\\{$slug}", 'WordPoints_Entity_Post' );
-	$children->register( "post\\{$slug}", 'author', 'WordPoints_Entity_Post_Author' );
+	$entities->register("post\\{$slug}", 'WordPoints_Entity_Post');
+	$children->register("post\\{$slug}", 'author', 'WordPoints_Entity_Post_Author');
 
-	$supports = get_all_post_type_supports( $slug );
+	$supports = get_all_post_type_supports($slug);
 
-	if ( isset( $supports['editor'] ) ) {
-		$children->register( "post\\{$slug}", 'content', 'WordPoints_Entity_Post_Content' );
+	if (isset($supports['editor'])) {
+		$children->register("post\\{$slug}", 'content', 'WordPoints_Entity_Post_Content');
 	}
 
-	if ( isset( $supports['comments'] ) ) {
-		$entities->register( "comment\\{$slug}", 'WordPoints_Entity_Comment' );
-		$children->register( "comment\\{$slug}", "post\\{$slug}", 'WordPoints_Entity_Comment_Post' );
-		$children->register( "comment\\{$slug}", 'author', 'WordPoints_Entity_Comment_Author' );
+	if (isset($supports['comments'])) {
+		$entities->register("comment\\{$slug}", 'WordPoints_Entity_Comment');
+		$children->register("comment\\{$slug}", "post\\{$slug}", 'WordPoints_Entity_Comment_Post');
+		$children->register("comment\\{$slug}", 'author', 'WordPoints_Entity_Comment_Author');
 	}
 
-	foreach ( get_object_taxonomies( $slug ) as $taxonomy_slug ) {
-		$children->register( "post\\{$slug}", "terms\\{$taxonomy_slug}", 'WordPoints_Entity_Post_Terms' );
+	foreach (get_object_taxonomies($slug) as $taxonomy_slug) {
+		$children->register("post\\{$slug}", "terms\\{$taxonomy_slug}", 'WordPoints_Entity_Post_Terms');
 	}
 
 	/**
@@ -409,7 +409,7 @@ 
 	 *
 	 * @param string $slug The slug ("name") of the post type.
 	 */
-	do_action( 'wordpoints_register_post_type_entities', $slug );
+	do_action('wordpoints_register_post_type_entities', $slug);
 }
 
 /**
@@ -419,13 +419,13 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_register_post_type_hook_events( $slug ) {
+function wordpoints_register_post_type_hook_events($slug) {
 
 	$event_slugs = array();
 
 	$events = wordpoints_hooks()->events;
 
-	if ( 'attachment' === $slug ) {
+	if ('attachment' === $slug) {
 
 		$event_slugs[] = 'media_upload';
 
@@ -462,7 +462,7 @@ 
 		);
 	}
 
-	if ( post_type_supports( $slug, 'comments' ) ) {
+	if (post_type_supports($slug, 'comments')) {
 
 		$event_slugs[] = "comment_leave\\{$slug}";
 
@@ -471,7 +471,7 @@ 
 			, 'WordPoints_Hook_Event_Comment_Leave'
 			, array(
 				'actions' => array(
-					'fire' => array( 'comment_approve', 'comment_new' ),
+					'fire' => array('comment_approve', 'comment_new'),
 					'reverse' => 'comment_deapprove',
 					'spam' => 'comment_spam',
 				),
@@ -482,8 +482,8 @@ 
 		);
 	}
 
-	if ( is_multisite() ) {
-		foreach ( $event_slugs as $event_slug ) {
+	if (is_multisite()) {
+		foreach ($event_slugs as $event_slug) {
 			$events->args->register(
 				$event_slug
 				, 'current:site'
@@ -499,7 +499,7 @@ 
 	 *
 	 * @param string $slug The slug ("name") of the post type.
 	 */
-	do_action( 'wordpoints_register_post_type_hook_events', $slug );
+	do_action('wordpoints_register_post_type_hook_events', $slug);
 }
 
 /**
@@ -509,13 +509,13 @@ 
  *
  * @param string $slug The slug of the taxonomy.
  */
-function wordpoints_register_taxonomy_entities( $slug ) {
+function wordpoints_register_taxonomy_entities($slug) {
 
 	$entities = wordpoints_entities();
 	$children = $entities->children;
 
-	$entities->register( "term\\{$slug}", 'WordPoints_Entity_Term' );
-	$children->register( "term\\{$slug}", 'id', 'WordPoints_Entity_Term_Id' );
+	$entities->register("term\\{$slug}", 'WordPoints_Entity_Term');
+	$children->register("term\\{$slug}", 'id', 'WordPoints_Entity_Term_Id');
 
 	/**
 	 * Fired when registering the entities for a taxonomy.
@@ -524,7 +524,7 @@ 
 	 *
 	 * @param string $slug The taxonomy's slug.
 	 */
-	do_action( 'wordpoints_register_taxonomy_entities', $slug );
+	do_action('wordpoints_register_taxonomy_entities', $slug);
 }
 
 /**
@@ -536,10 +536,10 @@ 
  *
  * @param WordPoints_Class_RegistryI $data_types The data types registry.
  */
-function wordpoints_data_types_init( $data_types ) {
+function wordpoints_data_types_init($data_types) {
 
-	$data_types->register( 'integer', 'WordPoints_Data_Type_Integer' );
-	$data_types->register( 'text', 'WordPoints_Data_Type_Text' );
+	$data_types->register('integer', 'WordPoints_Data_Type_Integer');
+	$data_types->register('text', 'WordPoints_Data_Type_Text');
 }
 
 /**
@@ -554,9 +554,9 @@ 
  *
  * @return bool Whether the user can view the points log.
  */
-function wordpoints_hooks_user_can_view_points_log( $can_view, $log ) {
+function wordpoints_hooks_user_can_view_points_log($can_view, $log) {
 
-	if ( ! $can_view ) {
+	if ( ! $can_view) {
 		return $can_view;
 	}
 
@@ -565,11 +565,11 @@ 
 	$event_slug = $log->log_type;
 
 	/** @var WordPoints_Hook_Arg $arg */
-	foreach ( wordpoints_hooks()->events->args->get_children( $event_slug ) as $slug => $arg ) {
+	foreach (wordpoints_hooks()->events->args->get_children($event_slug) as $slug => $arg) {
 
-		$value = wordpoints_get_points_log_meta( $log->id, $slug, true );
+		$value = wordpoints_get_points_log_meta($log->id, $slug, true);
 
-		if ( ! $value ) {
+		if ( ! $value) {
 			continue;
 		}
 
@@ -579,7 +579,7 @@ 
 			, $value
 		);
 
-		if ( ! $can_view ) {
+		if ( ! $can_view) {
 			break;
 		}
 	}
@@ -598,20 +598,20 @@ 
  *
  * @return bool Whether the user can view this entity.
  */
-function wordpoints_entity_user_can_view( $user_id, $entity_slug, $entity_id ) {
+function wordpoints_entity_user_can_view($user_id, $entity_slug, $entity_id) {
 
-	$entity = wordpoints_entities()->get( $entity_slug );
+	$entity = wordpoints_entities()->get($entity_slug);
 
 	// If this entity type is not found, we have no way of determining whether it is
 	// safe for the user to view it.
-	if ( ! ( $entity instanceof WordPoints_Entity ) ) {
+	if ( ! ($entity instanceof WordPoints_Entity)) {
 		return false;
 	}
 
 	$can_view = true;
 
-	if ( $entity instanceof WordPoints_Entity_Restricted_VisibilityI ) {
-		$can_view = $entity->user_can_view( $user_id, $entity_id );
+	if ($entity instanceof WordPoints_Entity_Restricted_VisibilityI) {
+		$can_view = $entity->user_can_view($user_id, $entity_id);
 	}
 
 	/**
@@ -642,8 +642,8 @@ 
  */
 function wordpoints_apps() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
-		WordPoints_App::$main = new WordPoints_App( 'apps' );
+	if ( ! isset(WordPoints_App::$main)) {
+		WordPoints_App::$main = new WordPoints_App('apps');
 	}
 
 	return WordPoints_App::$main;
@@ -658,7 +658,7 @@ 
  */
 function wordpoints_hooks() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
+	if ( ! isset(WordPoints_App::$main)) {
 		wordpoints_apps();
 	}
 
@@ -674,7 +674,7 @@ 
  */
 function wordpoints_entities() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
+	if ( ! isset(WordPoints_App::$main)) {
 		wordpoints_apps();
 	}
 
@@ -690,13 +690,13 @@ 
  *
  * @param WordPoints_App $app The main apps app.
  */
-function wordpoints_apps_init( $app ) {
+function wordpoints_apps_init($app) {
 
 	$apps = $app->sub_apps;
 
-	$apps->register( 'hooks', 'WordPoints_Hooks' );
-	$apps->register( 'entities', 'WordPoints_App_Registry' );
-	$apps->register( 'data_types', 'WordPoints_Class_Registry' );
+	$apps->register('hooks', 'WordPoints_Hooks');
+	$apps->register('entities', 'WordPoints_App_Registry');
+	$apps->register('data_types', 'WordPoints_Class_Registry');
 }
 
 /**
@@ -709,19 +709,19 @@ 
  *
  * @return object|false The constructed object, or false if to many args were passed.
  */
-function wordpoints_construct_class_with_args( $class_name, array $args ) {
+function wordpoints_construct_class_with_args($class_name, array $args) {
 
-	switch ( count( $args ) ) {
+	switch (count($args)) {
 		case 0:
 			return new $class_name();
 		case 1:
-			return new $class_name( $args[0] );
+			return new $class_name($args[0]);
 		case 2:
-			return new $class_name( $args[0], $args[1] );
+			return new $class_name($args[0], $args[1]);
 		case 3:
-			return new $class_name( $args[0], $args[1], $args[2] );
+			return new $class_name($args[0], $args[1], $args[2]);
 		case 4:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3] );
+			return new $class_name($args[0], $args[1], $args[2], $args[3]);
 		default:
 			return false;
 	}
@@ -743,13 +743,13 @@ 
  * @return array The slug parsed into the 'generic' and 'dynamic' portions. If the
  *               slug is not dynamic, the value of each of those keys will be false.
  */
-function wordpoints_parse_dynamic_slug( $slug ) {
+function wordpoints_parse_dynamic_slug($slug) {
 
-	$parsed = array( 'dynamic' => false, 'generic' => false );
+	$parsed = array('dynamic' => false, 'generic' => false);
 
-	$parts = explode( '\\', $slug, 2 );
+	$parts = explode('\\', $slug, 2);
 
-	if ( isset( $parts[1] ) ) {
+	if (isset($parts[1])) {
 		$parsed['dynamic'] = $parts[1];
 		$parsed['generic'] = $parts[0];
 	}
@@ -786,43 +786,43 @@ 
  * @return array|false The IDs of the context(s) you passed in, indexed by context
  *                     slug, or false if any of the contexts isn't current.
  */
-function wordpoints_entities_get_current_context_id( array $slugs ) {
+function wordpoints_entities_get_current_context_id(array $slugs) {
 
 	$current_context = array();
 
 	/** @var WordPoints_Class_Registry $contexts */
 	$contexts = wordpoints_entities()->contexts;
 
-	foreach ( $slugs as $slug ) {
+	foreach ($slugs as $slug) {
 
-		$context = $contexts->get( $slug );
+		$context = $contexts->get($slug);
 
-		if ( ! $context instanceof WordPoints_Entity_Context ) {
+		if ( ! $context instanceof WordPoints_Entity_Context) {
 			return false;
 		}
 
 		$id = $context->get_current_id();
 
-		if ( false === $id ) {
+		if (false === $id) {
 			return false;
 		}
 
-		$current_context[ $slug ] = $id;
+		$current_context[$slug] = $id;
 	}
 
 	return $current_context;
 }
 
-function wordpoints_hooks_get_event_signature( WordPoints_Hook_Event_Args $event_args ) {
+function wordpoints_hooks_get_event_signature(WordPoints_Hook_Event_Args $event_args) {
 
 	$entity = $event_args->get_primary_arg();
 
 	// TODO what if GUID isn't set?
-	if ( ! $entity ) {
-		return str_repeat( '-', 64 );
+	if ( ! $entity) {
+		return str_repeat('-', 64);
 	}
 
-	return wordpoints_hash( wp_json_encode( $entity->get_the_guid() ) );
+	return wordpoints_hash(wp_json_encode($entity->get_the_guid()));
 }
 
 // EOF

src/includes/classes/hook/reactor.php

Spacing +22 added lines, -22 removed lines

@@ -97,29 +97,29 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
 		$network_mode = wordpoints_hooks()->get_network_mode();
 
-		switch ( $var ) {
+		switch ($var) {
 			case 'reactions':
 				$var = $network_mode ? 'network_reactions' : 'standard_reactions';
 				// fall through
 
 			case 'standard_reactions':
 			case 'network_reactions':
-				if ( $network_mode && 'standard_reactions' === $var ) {
+				if ($network_mode && 'standard_reactions' === $var) {
 					return null;
 				}
 
-				if ( 'network_reactions' === $var && ! $this->is_network_wide() ) {
+				if ('network_reactions' === $var && ! $this->is_network_wide()) {
 					return null;
 				}
 
-				if ( ! isset( $this->$var ) ) {
-					if ( isset( $this->{"{$var}_class"} ) ) {
+				if ( ! isset($this->$var)) {
+					if (isset($this->{"{$var}_class"} )) {
 						$this->$var = new $this->{"{$var}_class"}(
-							substr( $var, 0, -10 /* _reactions */ )
+							substr($var, 0, -10 /* _reactions */)
 							, $this
 						);
 					}
@@ -194,21 +194,21 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI[] All of the reaction objects.
 	 */
-	public function get_all_reactions_to_event( $event_slug ) {
+	public function get_all_reactions_to_event($event_slug) {
 
 		$reactions = array();
 
-		foreach ( array( 'standard', 'network' ) as $store ) {
+		foreach (array('standard', 'network') as $store) {
 
 			$storage = $this->{"{$store}_reactions"};
 
-			if ( ! $storage instanceof WordPoints_Hook_Reaction_StorageI ) {
+			if ( ! $storage instanceof WordPoints_Hook_Reaction_StorageI) {
 				continue;
 			}
 
 			$reactions = array_merge(
 				$reactions
-				, $storage->get_reactions_to_event( $event_slug )
+				, $storage->get_reactions_to_event($event_slug)
 			);
 		}
 
@@ -230,15 +230,15 @@ 
 
 		$reactions = array();
 
-		foreach ( array( 'standard', 'network' ) as $store ) {
+		foreach (array('standard', 'network') as $store) {
 
 			$storage = $this->{"{$store}_reactions"};
 
-			if ( ! $storage instanceof WordPoints_Hook_Reaction_StorageI ) {
+			if ( ! $storage instanceof WordPoints_Hook_Reaction_StorageI) {
 				continue;
 			}
 
-			$reactions = array_merge( $reactions, $storage->get_reactions() );
+			$reactions = array_merge($reactions, $storage->get_reactions());
 		}
 
 		return $reactions;
@@ -254,21 +254,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');
 			}
 		}
 
@@ -278,8 +278,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']);
 	}
 
 	/**

src/includes/classes/hook/reaction.php

Spacing +4 added lines, -4 removed lines

@@ -35,18 +35,18 @@
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $id, WordPoints_Hook_Reaction_StorageI $storage ) {
+	public function __construct($id, WordPoints_Hook_Reaction_StorageI $storage) {
 
-		$this->ID      = wordpoints_int( $id );
+		$this->ID      = wordpoints_int($id);
 		$this->storage = $storage;
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
-		if ( 'ID' === $var ) {
+		if ('ID' === $var) {
 			return $this->ID;
 		}
 

src/includes/classes/hook/reaction/storage/options/network.php

Spacing +9 added lines, -9 removed lines

@@ -17,7 +17,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	protected $context = array( 'network' );
+	protected $context = array('network');
 
 	/**
 	 * @since 1.0.0
@@ -27,29 +27,29 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_option( $name ) {
-		return get_site_option( $name );
+	public function get_option($name) {
+		return get_site_option($name);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function add_option( $name, $value ) {
-		return add_site_option( $name, $value );
+	protected function add_option($name, $value) {
+		return add_site_option($name, $value);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_option( $name, $value ) {
-		return update_site_option( $name, $value );
+	public function update_option($name, $value) {
+		return update_site_option($name, $value);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function delete_option( $name ) {
-		return delete_site_option( $name );
+	protected function delete_option($name) {
+		return delete_site_option($name);
 	}
 }
 

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

Spacing +26 added lines, -26 removed lines

@@ -44,7 +44,7 @@ 
 	 *
 	 * @var string[]
 	 */
-	protected $context = array( 'network', 'site' );
+	protected $context = array('network', 'site');
 
 	/**
 	 * The name of the class to use for reaction objects.
@@ -60,7 +60,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug, WordPoints_Hook_Reactor $reactor ) {
+	public function __construct($slug, WordPoints_Hook_Reactor $reactor) {
 
 		$this->slug = $slug;
 		$this->reactor = $reactor;
@@ -84,33 +84,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);
 	}
 
 	/**
@@ -125,41 +125,41 @@ 
 	 *         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, $this->reactor );
+		$validator = new WordPoints_Hook_Reaction_Validator($settings, $this->reactor);
 		$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']);
 
-		$this->reactor->update_settings( $reaction, $settings );
+		$this->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);
 		}
 
 		/**
@@ -169,7 +169,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;
 	}
@@ -186,7 +186,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/storagei.php

Spacing +7 added lines, -7 removed lines

@@ -25,7 +25,7 @@ 
 	 * @param string                  $slug    The slug of this storage group.
 	 * @param WordPoints_Hook_Reactor $reactor The reactor the reactions belong to.
 	 */
-	public function __construct( $slug, WordPoints_Hook_Reactor $reactor );
+	public function __construct($slug, WordPoints_Hook_Reactor $reactor);
 
 	/**
 	 * Get the slug of this reaction storage group.
@@ -69,7 +69,7 @@ 
 	 *
 	 * @return bool Whether the reaction exists.
 	 */
-	public function reaction_exists( $id );
+	public function reaction_exists($id);
 
 	/**
 	 * Get an reaction object.
@@ -80,7 +80,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.
@@ -93,7 +93,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.
@@ -107,7 +107,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.
@@ -118,7 +118,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.
@@ -141,7 +141,7 @@ 
 	 *
 	 * @return WordPoints_Hook_ReactionI[]
 	 */
-	public function get_reactions_to_event( $event_slug );
+	public function get_reactions_to_event($event_slug);
 }
 
 // EOF

src/includes/classes/hook/firer.php

Spacing +10 added lines, -10 removed lines

@@ -17,14 +17,14 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) {
+	public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) {
 
 		$hooks = wordpoints_hooks();
 
 		/** @var WordPoints_Hook_Reactor $reactor */
-		foreach ( $hooks->reactors->get_all() as $reactor ) {
+		foreach ($hooks->reactors->get_all() as $reactor) {
 
-			foreach ( $reactor->get_all_reactions_to_event( $event_slug ) as $reaction ) {
+			foreach ($reactor->get_all_reactions_to_event($event_slug) as $reaction) {
 
 				$validator = new WordPoints_Hook_Reaction_Validator(
 					$reaction
@@ -34,25 +34,25 @@ 
 
 				$validator->validate();
 
-				if ( $validator->had_errors() ) {
+				if ($validator->had_errors()) {
 					continue;
 				}
 
-				unset( $validator );
+				unset($validator);
 
 				/** @var WordPoints_Hook_Extension[] $extensions */
 				$extensions = $hooks->extensions->get_all();
 
-				foreach ( $extensions as $extension ) {
-					if ( ! $extension->should_hit( $reaction, $event_args ) ) {
+				foreach ($extensions as $extension) {
+					if ( ! $extension->should_hit($reaction, $event_args)) {
 						continue 2;
 					}
 				}
 
-				$reactor->hit( $event_args, $reaction );
+				$reactor->hit($event_args, $reaction);
 
-				foreach ( $extensions as $extension ) {
-					$extension->after_hit( $reaction, $event_args );
+				foreach ($extensions as $extension) {
+					$extension->after_hit($reaction, $event_args);
 				}
 			}
 		}

src/includes/classes/hook/event/args.php

Spacing +26 added lines, -26 removed lines

@@ -63,22 +63,22 @@ 
 	 *
 	 * @param WordPoints_Hook_Arg[] $args The hook args.
 	 */
-	public function __construct( array $args ) {
+	public function __construct(array $args) {
 
 		parent::__construct();
 
-		foreach ( $args as $arg ) {
+		foreach ($args as $arg) {
 
 			$entity = $arg->get_entity();
 			$slug = $arg->get_slug();
 
-			if ( ! $entity instanceof WordPoints_Entity ) {
+			if ( ! $entity instanceof WordPoints_Entity) {
 				continue;
 			}
 
-			$this->entities[ $slug ] = $entity;
+			$this->entities[$slug] = $entity;
 
-			if ( ! $arg->is_stateful() ) {
+			if ( ! $arg->is_stateful()) {
 				// If any of the args aren't stateful the event isn't repeatable.
 				$this->is_repeatable = false;
 				$this->primary_arg_slug = $slug;
@@ -115,11 +115,11 @@ 
 	 */
 	public function get_primary_arg() {
 
-		if ( ! $this->primary_arg_slug ) {
+		if ( ! $this->primary_arg_slug) {
 			return false;
 		}
 
-		return $this->entities[ $this->primary_arg_slug ];
+		return $this->entities[$this->primary_arg_slug];
 	}
 
 	/**
@@ -133,8 +133,8 @@ 
 
 		$stateful = $this->entities;
 
-		if ( $this->primary_arg_slug ) {
-			unset( $stateful[ $this->primary_arg_slug ] );
+		if ($this->primary_arg_slug) {
+			unset($stateful[$this->primary_arg_slug]);
 		}
 
 		return $stateful;
@@ -147,54 +147,54 @@ 
 	 *
 	 * @param WordPoints_Hook_Reaction_Validator $validator The validator.
 	 */
-	public function set_validator( WordPoints_Hook_Reaction_Validator $validator ) {
+	public function set_validator(WordPoints_Hook_Reaction_Validator $validator) {
 		$this->validator = $validator;
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function descend( $child_slug ) {
+	public function descend($child_slug) {
 
-		$result = parent::descend( $child_slug );
+		$result = parent::descend($child_slug);
 
 		// Just in case no validator has been set.
-		if ( ! $this->validator ) {
+		if ( ! $this->validator) {
 			return $result;
 		}
 
-		if ( ! $result ) {
+		if ( ! $result) {
 
-			if ( ! isset( $this->current ) ) {
+			if ( ! isset($this->current)) {
 
 				$this->validator->add_error(
 					sprintf(
-						__( 'The %s arg is not registered for this event.', 'wordpoints' ) // TODO message
+						__('The %s arg is not registered for this event.', 'wordpoints') // TODO message
 						, $child_slug
 					)
 				);
 
-			} elseif ( ! ( $this->current instanceof WordPoints_Entity_ParentI ) ) {
+			} elseif ( ! ($this->current instanceof WordPoints_Entity_ParentI)) {
 
 				$this->validator->add_error(
-					__( 'Cannot get descendant of %s: not a parent.', 'wordpoints' ) // TODO message
+					__('Cannot get descendant of %s: not a parent.', 'wordpoints') // TODO message
 				);
 
 			} else {
 
-				$child_arg = $this->current->get_child( $child_slug );
+				$child_arg = $this->current->get_child($child_slug);
 
-				if ( ! $child_arg ) {
+				if ( ! $child_arg) {
 					$this->validator->add_error(
-						__( '%s does not have a child "%s".', 'wordpoints' ) // TODO message
+						__('%s does not have a child "%s".', 'wordpoints') // TODO message
 						, $this->push_on_descend ? $child_slug : null
 					);
 				}
 			}
 
-		} elseif ( $this->push_on_descend ) {
+		} elseif ($this->push_on_descend) {
 
-			$this->validator->push_field( $child_slug );
+			$this->validator->push_field($child_slug);
 		}
 
 		return $result;
@@ -207,7 +207,7 @@ 
 
 		$ascended = parent::ascend();
 
-		if ( $ascended && $this->validator ) {
+		if ($ascended && $this->validator) {
 			$this->validator->pop_field();
 		}
 
@@ -217,10 +217,10 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_from_hierarchy( array $hierarchy ) {
+	public function get_from_hierarchy(array $hierarchy) {
 
 		$this->push_on_descend = false;
-		$entityish = parent::get_from_hierarchy( $hierarchy );
+		$entityish = parent::get_from_hierarchy($hierarchy);
 		$this->push_on_descend = true;
 
 		return $entityish;