WordPoints / wordpoints

src/classes/class/registryi.php

Spacing +4 added lines, -4 removed lines

@@ -50,7 +50,7 @@ 
 	 *
 	 * @return object|false The object or false if it is not registered.
 	 */
-	public function get( $slug );
+	public function get($slug);
 
 	/**
 	 * Register a type of object.
@@ -63,7 +63,7 @@ 
 	 *
 	 * @return bool Whether the class was registered successfully.
 	 */
-	public function register( $slug, $class, array $args = array() );
+	public function register($slug, $class, array $args = array());
 
 	/**
 	 * Deregister a type of object
@@ -72,7 +72,7 @@ 
 	 *
 	 * @param string $slug The slug of the class to deregister.
 	 */
-	public function deregister( $slug );
+	public function deregister($slug);
 
 	/**
 	 * Check if a type of object is registered by its slug.
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return bool Whether the class is registered.
 	 */
-	public function is_registered( $slug );
+	public function is_registered($slug);
 }
 
 // EOF

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 	 * @since 2.2.0
 	 *
 	 * @param object $log The object for the points log the restriction is for.
+	 * @return void
 	 */
 	public function __construct( $log );
 

src/classes/class/registry.php

Spacing +29 added lines, -29 removed lines

@@ -30,46 +30,46 @@ 
 	/**
 	 * @since 2.1.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 2.1.0
 	 */
 	public function get_all_slugs() {
-		return array_keys( $this->classes );
+		return array_keys($this->classes);
 	}
 
 	/**
 	 * @since 2.1.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 2.1.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 2.1.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		unset( $this->classes[ $slug ] );
+		unset($this->classes[$slug]);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function is_registered( $slug ) {
+	public function is_registered($slug) {
 
-		return isset( $this->classes[ $slug ] );
+		return isset($this->classes[$slug]);
 	}
 
 	/**
@@ -114,39 +114,39 @@ 
 		array $args = array()
 	) {
 
-		$pass_slugs = ( ! isset( $args['pass_slugs'] ) || $args['pass_slugs'] );
+		$pass_slugs = ( ! isset($args['pass_slugs']) || $args['pass_slugs']);
 
 		$objects = array();
 
-		if ( empty( $construct_with_args ) ) {
+		if (empty($construct_with_args)) {
 
-			if ( $pass_slugs ) {
-				foreach ( $classes as $slug => $class ) {
-					$objects[ $slug ] = new $class( $slug );
+			if ($pass_slugs) {
+				foreach ($classes as $slug => $class) {
+					$objects[$slug] = new $class($slug);
 				}
 			} else {
-				foreach ( $classes as $slug => $class ) {
-					$objects[ $slug ] = new $class();
+				foreach ($classes as $slug => $class) {
+					$objects[$slug] = new $class();
 				}
 			}
 
 		} else {
 
-			if ( $pass_slugs ) {
+			if ($pass_slugs) {
 
-				array_unshift( $construct_with_args, null );
+				array_unshift($construct_with_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 ) + $construct_with_args
+						, array($slug) + $construct_with_args
 					);
 				}
 
 			} else {
 
-				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
 						, $construct_with_args
 					);

src/classes/entity.php

Spacing +51 added lines, -51 removed lines

@@ -108,11 +108,11 @@ 
 	 *
 	 * @return mixed The entity, or false if not found.
 	 */
-	protected function get_entity( $id ) {
+	protected function get_entity($id) {
 
-		$entity = call_user_func( $this->getter, $id );
+		$entity = call_user_func($this->getter, $id);
 
-		if ( ! $this->is_entity( $entity ) ) {
+		if ( ! $this->is_entity($entity)) {
 			return false;
 		}
 
@@ -128,13 +128,13 @@ 
 	 *
 	 * @return bool Whether the passed value is an entity.
 	 */
-	protected function is_entity( $entity ) {
+	protected function is_entity($entity) {
 
-		if ( ! is_object( $entity ) && ! is_array( $entity ) ) {
+		if ( ! is_object($entity) && ! is_array($entity)) {
 			return false;
 		}
 
-		return (bool) $this->get_entity_id( $entity );
+		return (bool) $this->get_entity_id($entity);
 	}
 
 	/**
@@ -151,16 +151,16 @@ 
 	 *
 	 * @return mixed The entity, or false if not found.
 	 */
-	protected function get_entity_from_context( $id, $context ) {
+	protected function get_entity_from_context($id, $context) {
 
 		/** @var WordPoints_Entity_Contexts $contexts */
-		$contexts = wordpoints_entities()->get_sub_app( 'contexts' );
+		$contexts = wordpoints_entities()->get_sub_app('contexts');
 
-		if ( ! $contexts->switch_to( $context ) ) {
+		if ( ! $contexts->switch_to($context)) {
 			return false;
 		}
 
-		$entity = $this->get_entity( $id );
+		$entity = $this->get_entity($id);
 
 		$contexts->switch_back();
 
@@ -176,13 +176,13 @@ 
 	 *
 	 * @return bool Whether the passed value is a GUID for an entity of this type.
 	 */
-	protected function is_guid( $guid ) {
+	protected function is_guid($guid) {
 
-		if ( ! is_array( $guid ) ) {
+		if ( ! is_array($guid)) {
 			return false;
 		}
 
-		return isset( $guid[ $this->get_slug() ], $guid[ $this->get_context() ] );
+		return isset($guid[$this->get_slug()], $guid[$this->get_context()]);
 	}
 
 	/**
@@ -197,15 +197,15 @@ 
 	 *         @type array      $context The entity context.
 	 * }
 	 */
-	protected function split_guid( $guid ) {
+	protected function split_guid($guid) {
 
 		$slug    = $this->get_slug();
-		$id      = $guid[ $slug ];
+		$id      = $guid[$slug];
 		$context = $guid;
 
-		unset( $context[ $slug ] );
+		unset($context[$slug]);
 
-		return array( 'id' => $id, 'context' => $context );
+		return array('id' => $id, 'context' => $context);
 	}
 
 	/**
@@ -218,14 +218,14 @@ 
 	 *
 	 * @return mixed The value of the attribute of the entity.
 	 */
-	protected function get_attr_value( $entity, $attr ) {
+	protected function get_attr_value($entity, $attr) {
 
-		if ( is_array( $entity ) ) {
-			if ( isset( $entity[ $attr ] ) ) {
-				return $entity[ $attr ];
+		if (is_array($entity)) {
+			if (isset($entity[$attr])) {
+				return $entity[$attr];
 			}
 		} else {
-			if ( isset( $entity->{$attr} ) ) {
+			if (isset($entity->{$attr} )) {
 				return $entity->{$attr};
 			}
 		}
@@ -242,8 +242,8 @@ 
 	 *
 	 * @return mixed The ID of the entity.
 	 */
-	protected function get_entity_id( $entity ) {
-		return $this->get_attr_value( $entity, $this->get_id_field() );
+	protected function get_entity_id($entity) {
+		return $this->get_attr_value($entity, $this->get_id_field());
 	}
 
 	/**
@@ -259,8 +259,8 @@ 
 	 *
 	 * @return mixed The human ID of the entity.
 	 */
-	protected function get_entity_human_id( $entity ) {
-		return $this->get_attr_value( $entity, $this->human_id_field );
+	protected function get_entity_human_id($entity) {
+		return $this->get_attr_value($entity, $this->human_id_field);
 	}
 
 	//
@@ -302,15 +302,15 @@ 
 	 *
 	 * @return string|int|float|false The human identifier for the entity, or false.
 	 */
-	public function get_human_id( $id ) {
+	public function get_human_id($id) {
 
-		$entity = $this->get_entity( $id );
+		$entity = $this->get_entity($id);
 
-		if ( ! $entity ) {
+		if ( ! $entity) {
 			return false;
 		}
 
-		return $this->get_entity_human_id( $entity );
+		return $this->get_entity_human_id($entity);
 	}
 
 	/**
@@ -322,8 +322,8 @@ 
 	 *
 	 * @return bool Whether or not an entity with that ID exists.
 	 */
-	public function exists( $id ) {
-		return (bool) $this->get_entity( $id );
+	public function exists($id) {
+		return (bool) $this->get_entity($id);
 	}
 
 	/**
@@ -338,17 +338,17 @@ 
 	 *
 	 * @return WordPoints_Entityish|false The child's object, or false if not found.
 	 */
-	public function get_child( $child_slug ) {
+	public function get_child($child_slug) {
 
-		$children = wordpoints_entities()->get_sub_app( 'children' );
+		$children = wordpoints_entities()->get_sub_app('children');
 
-		$child = $children->get( $this->slug, $child_slug );
+		$child = $children->get($this->slug, $child_slug);
 
 		if (
-			isset( $this->the_value )
+			isset($this->the_value)
 			&& $child instanceof WordPoints_Entity_ChildI
 		) {
-			$child->set_the_value_from_entity( $this );
+			$child->set_the_value_from_entity($this);
 		}
 
 		return $child;
@@ -371,33 +371,33 @@ 
 	 *
 	 * @return bool Whether the value was set.
 	 */
-	public function set_the_value( $value ) {
+	public function set_the_value($value) {
 
 		$this->the_value = $this->the_entity = $this->the_context = null;
 
-		if ( $this->is_entity( $value ) ) {
+		if ($this->is_entity($value)) {
 
 			$entity = $value;
-			$value = $this->get_entity_id( $value );
+			$value = $this->get_entity_id($value);
 
-		} elseif ( $this->is_guid( $value ) ) {
+		} elseif ($this->is_guid($value)) {
 
-			$guid = $this->split_guid( $value );
+			$guid = $this->split_guid($value);
 			$context = $guid['context'];
 			$value   = $guid['id'];
 
-			$entity = $this->get_entity_from_context( $guid['id'], $context );
+			$entity = $this->get_entity_from_context($guid['id'], $context);
 
 		} else {
 
-			$entity = $this->get_entity( $value );
+			$entity = $this->get_entity($value);
 		}
 
-		if ( ! $entity ) {
+		if ( ! $entity) {
 			return false;
 		}
 
-		if ( ! isset( $context ) ) {
+		if ( ! isset($context)) {
 			$context = wordpoints_entities_get_current_context_id(
 				$this->get_context()
 			);
@@ -419,8 +419,8 @@ 
 	 *
 	 * @return mixed The value of the attribute.
 	 */
-	public function get_the_attr_value( $attr ) {
-		return $this->get_attr_value( $this->the_entity, $attr );
+	public function get_the_attr_value($attr) {
+		return $this->get_attr_value($this->the_entity, $attr);
 	}
 
 	/**
@@ -444,7 +444,7 @@ 
 	 * @return string|int|float|null The human identifier for the entity, or null.
 	 */
 	public function get_the_human_id() {
-		return $this->get_entity_human_id( $this->the_entity );
+		return $this->get_entity_human_id($this->the_entity);
 	}
 
 	/**
@@ -478,11 +478,11 @@ 
 
 		$guid = $this->get_the_context();
 
-		if ( ! is_array( $guid ) ) {
+		if ( ! is_array($guid)) {
 			return $guid;
 		}
 
-		$guid = array( $this->slug => $this->get_the_id() ) + $guid;
+		$guid = array($this->slug => $this->get_the_id()) + $guid;
 
 		return $guid;
 	}

Doc Comments +1 added lines, -1 removed lines

@@ -430,7 +430,7 @@
 	 *
 	 * @since 2.1.0
 	 *
-	 * @return mixed The ID of the entity.
+	 * @return integer The ID of the entity.
 	 */
 	public function get_the_id() {
 		return $this->get_the_value();

src/classes/data/typei.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	 *
 	 * @return mixed|WP_Error The validated value or a WP_Error on failure.
 	 */
-	public function validate_value( $value );
+	public function validate_value($value);
 }
 
 // EOF

src/classes/data/type/integer.php

Spacing +4 added lines, -4 removed lines

@@ -17,14 +17,14 @@
 	/**
 	 * @since 2.1.0
 	 */
-	public function validate_value( $value ) {
+	public function validate_value($value) {
 
-		wordpoints_int( $value );
+		wordpoints_int($value);
 
-		if ( false === $value ) {
+		if (false === $value) {
 			return new WP_Error(
 				'not_integer'
-				, __( '%s must be an integer.', 'wordpoints' )
+				, __('%s must be an integer.', 'wordpoints')
 			);
 		}
 

src/classes/data/type/text.php

Spacing +3 added lines, -3 removed lines

@@ -20,12 +20,12 @@
 	/**
 	 * @since 2.1.0
 	 */
-	public function validate_value( $value ) {
+	public function validate_value($value) {
 
-		if ( ! is_string( $value ) ) {
+		if ( ! is_string($value)) {
 			return new WP_Error(
 				'not_string'
-				, __( '%s must be a text value.', 'wordpoints' )
+				, __('%s must be a text value.', 'wordpoints')
 			);
 		}
 

src/classes/hook/reactori.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 	 *
 	 * @param WordPoints_Hook_Fire $fire The hook fire object.
 	 */
-	public function hit( WordPoints_Hook_Fire $fire );
+	public function hit(WordPoints_Hook_Fire $fire);
 }
 
 // EOF

src/classes/hook/router.php

Spacing +51 added lines, -51 removed lines

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

src/classes/hook/extension.php

Spacing +17 added lines, -17 removed lines

@@ -57,14 +57,14 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings[ $this->slug ] ) ) {
+		if ( ! isset($settings[$this->slug])) {
 			return $settings;
 		}
 
-		if ( ! is_array( $settings[ $this->slug ] ) ) {
+		if ( ! is_array($settings[$this->slug])) {
 
 			$validator->add_error(
-				__( 'Invalid settings format.', 'wordpoints' )
+				__('Invalid settings format.', 'wordpoints')
 				, $this->slug
 			);
 
@@ -74,13 +74,13 @@ 
 		$this->validator = $validator;
 		$this->event_args = $event_args;
 
-		$this->validator->push_field( $this->slug );
+		$this->validator->push_field($this->slug);
 
-		foreach ( $settings[ $this->slug ] as $action_type => $action_type_settings ) {
+		foreach ($settings[$this->slug] as $action_type => $action_type_settings) {
 
-			$this->validator->push_field( $action_type );
+			$this->validator->push_field($action_type);
 
-			$settings[ $this->slug ][ $action_type ] = $this->validate_action_type_settings(
+			$settings[$this->slug][$action_type] = $this->validate_action_type_settings(
 				$action_type_settings
 			);
 
@@ -95,12 +95,12 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function update_settings( WordPoints_Hook_ReactionI $reaction, array $settings ) {
+	public function update_settings(WordPoints_Hook_ReactionI $reaction, array $settings) {
 
-		if ( isset( $settings[ $this->slug ] ) ) {
-			$reaction->update_meta( $this->slug, $settings[ $this->slug ] );
+		if (isset($settings[$this->slug])) {
+			$reaction->update_meta($this->slug, $settings[$this->slug]);
 		} else {
-			$reaction->delete_meta( $this->slug );
+			$reaction->delete_meta($this->slug);
 		}
 	}
 
@@ -113,7 +113,7 @@ 
 	 *
 	 * @return mixed The validated settings.
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return $settings;
 	}
 
@@ -130,16 +130,16 @@ 
 	 *
 	 * @return mixed The settings for the extension, or false if none.
 	 */
-	protected function get_settings_from_fire( WordPoints_Hook_Fire $fire ) {
+	protected function get_settings_from_fire(WordPoints_Hook_Fire $fire) {
 
-		$settings = $fire->reaction->get_meta( $this->slug );
+		$settings = $fire->reaction->get_meta($this->slug);
 
-		if ( ! is_array( $settings ) ) {
+		if ( ! is_array($settings)) {
 			return $settings;
 		}
 
-		if ( isset( $settings[ $fire->action_type ] ) ) {
-			return $settings[ $fire->action_type ];
+		if (isset($settings[$fire->action_type])) {
+			return $settings[$fire->action_type];
 		} else {
 			return false;
 		}