WordPoints / hooks-api

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

Spacing +26 added lines, -26 removed lines

@@ -104,13 +104,13 @@ 
 	 * @param bool                            $fail_fast Whether to fail as soon as
 	 *                                                   the first error is found.
 	 */
-	public function __construct( $settings, WordPoints_Hook_Reactor $reactor, $fail_fast = false ) {
+	public function __construct($settings, WordPoints_Hook_Reactor $reactor, $fail_fast = false) {
 
 		$this->reactor = $reactor;
 		$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();
@@ -120,7 +120,7 @@ 
 
 			$this->settings = $settings;
 
-			if ( isset( $this->settings['event'] ) ) {
+			if (isset($this->settings['event'])) {
 				$this->event_slug = $this->settings['event'];
 			}
 		}
@@ -144,10 +144,10 @@ 
 			$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');
 			}
 
 			// From here on out we can collect errors as they come (unless we are
@@ -158,14 +158,14 @@ 
 				$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);
 
-			$this->settings = $this->reactor->validate_settings( $this->settings, $this, $this->event_args );
+			$this->settings = $this->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);
 			}
 
 			/**
@@ -175,12 +175,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;
@@ -208,17 +208,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;
 		}
 	}
@@ -231,7 +231,7 @@ 
 	 * @return bool Whether the validator found any errors.
 	 */
 	public function had_errors() {
-		return ! empty( $this->errors );
+		return ! empty($this->errors);
 	}
 
 	/**
@@ -252,7 +252,7 @@ 
 	 *
 	 * @param string $field The field.
 	 */
-	public function push_field( $field ) {
+	public function push_field($field) {
 		$this->field_stack[] = $field;
 	}
 
@@ -262,7 +262,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function pop_field() {
-		array_pop( $this->field_stack );
+		array_pop($this->field_stack);
 	}
 
 	/**
@@ -307,7 +307,7 @@ 
 	 */
 	public function get_id() {
 
-		if ( ! $this->reaction ) {
+		if ( ! $this->reaction) {
 			return false;
 		}
 
@@ -345,13 +345,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/entity/context/network.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 */
 	public function get_current_id() {
 
-		if ( ! is_multisite() ) {
+		if ( ! is_multisite()) {
 			return 1;
 		}
 

src/includes/classes/entity/context.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 
 		$this->slug = $slug;
 	}

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;

src/includes/classes/entity/context/site.php

Spacing +2 added lines, -2 removed lines

@@ -21,11 +21,11 @@
 	 */
 	public function get_current_id() {
 
-		if ( ! is_multisite() ) {
+		if ( ! is_multisite()) {
 			return 1;
 		}
 
-		if ( wordpoints_is_network_context() ) {
+		if (wordpoints_is_network_context()) {
 			return false;
 		}
 

src/includes/classes/class/registry/childreni.php

Spacing +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
 	 *
 	 * @return object[] The child objects, indexed by slug.
 	 */
-	public function get_children( $parent_slug );
+	public function get_children($parent_slug);
 
 	/**
 	 * Get the slugs of all of the classes that are children of a certain parent.
@@ -57,7 +57,7 @@ 
 	 *
 	 * @return string[] The child slugs.
 	 */
-	public function get_children_slugs( $parent_slug );
+	public function get_children_slugs($parent_slug);
 
 	/**
 	 * Get an object by its slug.
@@ -69,7 +69,7 @@ 
 	 *
 	 * @return object|false The object or false on failure.
 	 */
-	public function get( $parent_slug, $slug );
+	public function get($parent_slug, $slug);
 
 	/**
 	 * Register a type of object.
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return bool Whether the class was registered successfully.
 	 */
-	public function register( $parent_slug, $slug, $class, array $args = array() );
+	public function register($parent_slug, $slug, $class, array $args = array());
 
 	/**
 	 * Deregister a type of object.
@@ -93,7 +93,7 @@ 
 	 * @param string $parent_slug The group slug.
 	 * @param string $slug        The slug of the class to deregister.
 	 */
-	public function deregister( $parent_slug, $slug );
+	public function deregister($parent_slug, $slug);
 
 	/**
 	 * Deregister all children of a particular parent.
@@ -102,7 +102,7 @@ 
 	 *
 	 * @param string $parent_slug The group slug.
 	 */
-	public function deregister_children( $parent_slug );
+	public function deregister_children($parent_slug);
 
 	/**
 	 * Check if a type of object is registered by its slug.
@@ -117,7 +117,7 @@ 
 	 *
 	 * @return bool Whether the class is registered.
 	 */
-	public function is_registered( $parent_slug, $slug = null );
+	public function is_registered($parent_slug, $slug = null);
 }
 
 // EOF

src/includes/classes/class/registry/children.php

Spacing +27 added lines, -27 removed lines

@@ -33,12 +33,12 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_all( array $args = array() ) {
+	public function get_all(array $args = array()) {
 
 		$items = array();
 
-		foreach ( $this->classes as $parent_slug => $classes ) {
-			$items[ $parent_slug ] = WordPoints_Class_Registry::construct_with_args(
+		foreach ($this->classes as $parent_slug => $classes) {
+			$items[$parent_slug] = WordPoints_Class_Registry::construct_with_args(
 				$classes
 				, $args
 			);
@@ -51,20 +51,20 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_all_slugs() {
-		return array_map( 'array_keys', $this->classes );
+		return array_map('array_keys', $this->classes);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_children( $parent_slug, array $args = array() ) {
+	public function get_children($parent_slug, array $args = array()) {
 
 		$items = array();
 
-		if ( isset( $this->classes[ $parent_slug ] ) ) {
+		if (isset($this->classes[$parent_slug])) {
 
 			$items = WordPoints_Class_Registry::construct_with_args(
-				$this->classes[ $parent_slug ]
+				$this->classes[$parent_slug]
 				, $args
 			);
 		}
@@ -75,12 +75,12 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_children_slugs( $parent_slug ) {
+	public function get_children_slugs($parent_slug) {
 
 		$slugs = array();
 
-		if ( isset( $this->classes[ $parent_slug ] ) ) {
-			$slugs = array_keys( $this->classes[ $parent_slug ] );
+		if (isset($this->classes[$parent_slug])) {
+			$slugs = array_keys($this->classes[$parent_slug]);
 		}
 
 		return $slugs;
@@ -89,29 +89,29 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get( $parent_slug, $slug, array $args = array() ) {
+	public function get($parent_slug, $slug, array $args = array()) {
 
-		if ( ! isset( $this->classes[ $parent_slug ][ $slug ] ) ) {
+		if ( ! isset($this->classes[$parent_slug][$slug])) {
 			return false;
 		}
 
-		$class = $this->classes[ $parent_slug ][ $slug ];
+		$class = $this->classes[$parent_slug][$slug];
 
-		if ( empty( $args ) ) {
-			return new $class( $slug );
+		if (empty($args)) {
+			return new $class($slug);
 		} else {
-			array_unshift( $args, $slug );
+			array_unshift($args, $slug);
 
-			return wordpoints_construct_class_with_args( $class, $args );
+			return wordpoints_construct_class_with_args($class, $args);
 		}
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function register( $parent_slug, $slug, $class, array $args = array() ) {
+	public function register($parent_slug, $slug, $class, array $args = array()) {
 
-		$this->classes[ $parent_slug ][ $slug ] = $class;
+		$this->classes[$parent_slug][$slug] = $class;
 
 		return true;
 	}
@@ -119,26 +119,26 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister( $parent_slug, $slug ) {
-		unset( $this->classes[ $parent_slug ][ $slug ] );
+	public function deregister($parent_slug, $slug) {
+		unset($this->classes[$parent_slug][$slug]);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister_children( $parent_slug ) {
-		unset( $this->classes[ $parent_slug ] );
+	public function deregister_children($parent_slug) {
+		unset($this->classes[$parent_slug]);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function is_registered( $parent_slug, $slug = null ) {
+	public function is_registered($parent_slug, $slug = null) {
 
-		if ( isset( $slug ) ) {
-			return isset( $this->classes[ $parent_slug ][ $slug ] );
+		if (isset($slug)) {
+			return isset($this->classes[$parent_slug][$slug]);
 		} else {
-			return isset( $this->classes[ $parent_slug ] );
+			return isset($this->classes[$parent_slug]);
 		}
 	}
 }

src/includes/classes/class/registry.php

Spacing +23 added lines, -23 removed lines

@@ -30,46 +30,46 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_all( array $args = array() ) {
-		return self::construct_with_args( $this->classes, $args );
+	public function get_all(array $args = array()) {
+		return self::construct_with_args($this->classes, $args);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_all_slugs() {
-		return array_keys( $this->classes );
+		return array_keys($this->classes);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get( $slug, array $args = array() ) {
+	public function get($slug, array $args = array()) {
 
-		if ( ! isset( $this->classes[ $slug ] ) ) {
+		if ( ! isset($this->classes[$slug])) {
 			return false;
 		}
 
-		if ( ! empty( $args ) ) {
+		if ( ! empty($args)) {
 
-			array_unshift( $args, $slug );
+			array_unshift($args, $slug);
 
 			return wordpoints_construct_class_with_args(
-				$this->classes[ $slug ]
+				$this->classes[$slug]
 				, $args
 			);
 
 		} else {
-			return new $this->classes[ $slug ]( $slug );
+			return new $this->classes[$slug]($slug);
 		}
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function register( $slug, $class, array $args = array() ) {
+	public function register($slug, $class, array $args = array()) {
 
-		$this->classes[ $slug ] = $class;
+		$this->classes[$slug] = $class;
 
 		return true;
 	}
@@ -77,17 +77,17 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		unset( $this->classes[ $slug ] );
+		unset($this->classes[$slug]);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function is_registered( $slug ) {
+	public function is_registered($slug) {
 
-		return isset( $this->classes[ $slug ] );
+		return isset($this->classes[$slug]);
 	}
 
 	/**
@@ -101,24 +101,24 @@ 
 	 *
 	 * @return object[] An array of the constructed objects.
 	 */
-	public static function construct_with_args( array $classes, array $args ) {
+	public static function construct_with_args(array $classes, array $args) {
 
 		$objects = array();
 
-		if ( empty( $args ) ) {
+		if (empty($args)) {
 
-			foreach ( $classes as $slug => $class ) {
-				$objects[ $slug ] = new $class( $slug );
+			foreach ($classes as $slug => $class) {
+				$objects[$slug] = new $class($slug);
 			}
 
 		} else {
 
-			array_unshift( $args, null );
+			array_unshift($args, null);
 
-			foreach ( $classes as $slug => $class ) {
-				$objects[ $slug ] = wordpoints_construct_class_with_args(
+			foreach ($classes as $slug => $class) {
+				$objects[$slug] = wordpoints_construct_class_with_args(
 					$class
-					, array( $slug ) + $args
+					, array($slug) + $args
 				);
 			}
 		}

src/includes/classes/hook/arg.php

Spacing +9 added lines, -9 removed lines

@@ -70,14 +70,14 @@ 
 	 * @param string                  $slug   The arg slug.
 	 * @param WordPoints_Hook_ActionI $action The calling action's object.
 	 */
-	public function __construct( $slug, WordPoints_Hook_ActionI $action = null ) {
+	public function __construct($slug, WordPoints_Hook_ActionI $action = null) {
 
 		$this->slug = $slug;
 		$this->action = $action;
 
-		$parts = explode( ':', $slug, 2 );
+		$parts = explode(':', $slug, 2);
 
-		if ( isset( $parts[1] ) ) {
+		if (isset($parts[1])) {
 			$this->entity_slug = $parts[1];
 		} else {
 			$this->entity_slug = $slug;
@@ -119,11 +119,11 @@ 
 			$this->get_entity_slug()
 		);
 
-		if ( $entity instanceof WordPoints_Entity ) {
+		if ($entity instanceof WordPoints_Entity) {
 			$value = $this->get_value();
 
-			if ( $value ) {
-				$entity->set_the_value( $value );
+			if ($value) {
+				$entity->set_the_value($value);
 			}
 		}
 
@@ -139,8 +139,8 @@ 
 	 */
 	public function get_value() {
 
-		if ( $this->action instanceof WordPoints_Hook_ActionI ) {
-			return $this->action->get_arg_value( $this->slug );
+		if ($this->action instanceof WordPoints_Hook_ActionI) {
+			return $this->action->get_arg_value($this->slug);
 		} else {
 			return null;
 		}
@@ -157,7 +157,7 @@ 
 
 		$entity = $this->get_entity();
 
-		if ( ! $entity ) {
+		if ( ! $entity) {
 			return $this->slug;
 		}