WordPoints / wordpoints

src/classes/components.php

Spacing +47 added lines, -47 removed lines

@@ -90,13 +90,13 @@ 
 	 */
 	public static function set_up() {
 
-		if ( isset( self::$instance ) ) {
+		if (isset(self::$instance)) {
 			return;
 		}
 
 		self::$instance = new WordPoints_Components();
 
-		add_action( 'plugins_loaded', array( self::$instance, 'load' ) );
+		add_action('plugins_loaded', array(self::$instance, 'load'));
 	}
 
 	/**
@@ -134,15 +134,15 @@ 
 		 * @since 1.0.0
 		 * @since 1.7.0 The components' code isn't loaded until after this hook.
 		 */
-		do_action( 'wordpoints_components_register' );
+		do_action('wordpoints_components_register');
 
-		foreach ( $this->get() as $component ) {
+		foreach ($this->get() as $component) {
 
-			if ( ! $this->is_active( $component['slug'] ) ) {
+			if ( ! $this->is_active($component['slug'])) {
 				continue;
 			}
 
-			include_once( $component['file'] );
+			include_once($component['file']);
 		}
 
 		/**
@@ -150,7 +150,7 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_components_loaded' );
+		do_action('wordpoints_components_loaded');
 	}
 
 	/**
@@ -165,7 +165,7 @@ 
 	 */
 	public function get() {
 
-		if ( ! isset( $this->registered ) ) {
+		if ( ! isset($this->registered)) {
 			return false;
 		}
 
@@ -181,13 +181,13 @@ 
 	 *
 	 * @return array|false The component, or false if it isn't registered.
 	 */
-	public function get_component( $slug ) {
+	public function get_component($slug) {
 
-		if ( ! isset( $this->registered[ $slug ] ) ) {
+		if ( ! isset($this->registered[$slug])) {
 			return false;
 		}
 
-		return $this->registered[ $slug ];
+		return $this->registered[$slug];
 	}
 
 	/**
@@ -215,9 +215,9 @@ 
 	 *
 	 * @return bool True if the component is registered, otherwise false.
 	 */
-	public function is_registered( $slug ) {
+	public function is_registered($slug) {
 
-		return isset( $this->registered[ $slug ] );
+		return isset($this->registered[$slug]);
 	}
 
 	/**
@@ -247,7 +247,7 @@ 
 	 *
 	 * @return bool True, or false if the component's slug has already been registered.
 	 */
-	public function register( $args ) {
+	public function register($args) {
 
 		$defaults = array(
 			'slug'          => '',
@@ -261,25 +261,25 @@ 
 			'un_installer'  => null,
 		);
 
-		$component = array_merge( $defaults, $args );
+		$component = array_merge($defaults, $args);
 
 		if (
-			empty( $component['name'] )
-			|| empty( $component['file'] )
-			|| empty( $component['slug'] )
-			|| $this->is_registered( $component['slug'] )
+			empty($component['name'])
+			|| empty($component['file'])
+			|| empty($component['slug'])
+			|| $this->is_registered($component['slug'])
 		) {
 			return false;
 		}
 
-		$this->registered[ $component['slug'] ] = array_intersect_key( $component, $defaults );
+		$this->registered[$component['slug']] = array_intersect_key($component, $defaults);
 
 		WordPoints_Installables::register(
 			'component'
 			, $component['slug']
 			, array(
-				'version'      => $this->registered[ $component['slug'] ]['version'],
-				'un_installer' => $this->registered[ $component['slug'] ]['un_installer'],
+				'version'      => $this->registered[$component['slug']]['version'],
+				'un_installer' => $this->registered[$component['slug']]['un_installer'],
 				'network_wide' => is_wordpoints_network_active(),
 			)
 		);
@@ -296,18 +296,18 @@ 
 	 *
 	 * @return bool True, even if the component isn't registered.
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		if ( isset( $this->registered[ $slug ] ) ) {
+		if (isset($this->registered[$slug])) {
 
 			/**
 			 * Component being deregistered.
 			 *
 			 * @since 1.0.0
 			 */
-			do_action( "wordpoints_component_deregister-{$slug}" );
+			do_action("wordpoints_component_deregister-{$slug}");
 
-			unset( $this->registered[ $slug ] );
+			unset($this->registered[$slug]);
 		}
 
 		return true;
@@ -330,24 +330,24 @@ 
 	 *
 	 * @return bool Whether the component was activated.
 	 */
-	public function activate( $slug ) {
+	public function activate($slug) {
 
-		if ( ! $this->is_registered( $slug ) ) {
+		if ( ! $this->is_registered($slug)) {
 			return false;
 		}
 
 		// If this component isn't already active, activate it.
-		if ( ! $this->is_active( $slug ) ) {
+		if ( ! $this->is_active($slug)) {
 
-			$this->active[ $slug ] = 1;
+			$this->active[$slug] = 1;
 
-			if ( ! wordpoints_update_maybe_network_option( 'wordpoints_active_components', $this->active ) ) {
+			if ( ! wordpoints_update_maybe_network_option('wordpoints_active_components', $this->active)) {
 				return false;
 			}
 
-			include_once( $this->registered[ $slug ]['file'] );
+			include_once($this->registered[$slug]['file']);
 
-			WordPoints_Installables::get_installer( 'component', $slug )->install(
+			WordPoints_Installables::get_installer('component', $slug)->install(
 				is_wordpoints_network_active()
 			);
 
@@ -356,7 +356,7 @@ 
 			 *
 			 * @since 1.0.0
 			 */
-			do_action( "wordpoints_component_activate-{$slug}" );
+			do_action("wordpoints_component_activate-{$slug}");
 		}
 
 		return true;
@@ -376,17 +376,17 @@ 
 	 *
 	 * @return bool Whether the component was deactivated.
 	 */
-	public function deactivate( $slug ) {
+	public function deactivate($slug) {
 
-		if ( ! $this->is_registered( $slug ) ) {
+		if ( ! $this->is_registered($slug)) {
 			return false;
 		}
 
-		if ( $this->is_active( $slug ) ) {
+		if ($this->is_active($slug)) {
 
-			unset( $this->active[ $slug ] );
+			unset($this->active[$slug]);
 
-			if ( ! wordpoints_update_maybe_network_option( 'wordpoints_active_components', $this->active ) ) {
+			if ( ! wordpoints_update_maybe_network_option('wordpoints_active_components', $this->active)) {
 				return false;
 			}
 
@@ -395,7 +395,7 @@ 
 			 *
 			 * @since 1.0.0
 			 */
-			do_action( "wordpoints_component_deactivate-{$slug}" );
+			do_action("wordpoints_component_deactivate-{$slug}");
 		}
 
 		return true;
@@ -413,11 +413,11 @@ 
 	 *
 	 * @return bool Whether the component is active.
 	 */
-	public function is_active( $slug ) {
+	public function is_active($slug) {
 
 		$this->get_active();
 
-		$is_active = isset( $this->active[ $slug ] );
+		$is_active = isset($this->active[$slug]);
 
 		/**
 		 * Is a component active?
@@ -427,7 +427,7 @@ 
 		 * @param bool   $is_active Whether the component is currently active.
 		 * @param string $slug      The component's slug.
 		 */
-		return apply_filters( 'wordpoints_component_active', $is_active, $slug );
+		return apply_filters('wordpoints_component_active', $is_active, $slug);
 	}
 
 	/**
@@ -438,7 +438,7 @@ 
 	 *
 	 * @param string $slug The component's slug.
 	 */
-	public function uninstall( $slug ) {
+	public function uninstall($slug) {
 
 		_deprecated_function(
 			__METHOD__
@@ -446,7 +446,7 @@ 
 			, 'WordPoints_Installables::uninstall()'
 		);
 
-		WordPoints_Installables::uninstall( 'component', $slug );
+		WordPoints_Installables::uninstall('component', $slug);
 	}
 
 	/**
@@ -493,7 +493,7 @@ 
 	 *
 	 * @return WordPoints_Un_Installer_Base|false The installer for the component.
 	 */
-	public function get_installer( $slug ) {
+	public function get_installer($slug) {
 
 		_deprecated_function(
 			__METHOD__
@@ -501,7 +501,7 @@ 
 			, 'WordPoints_Installables::get_installer()'
 		);
 
-		return WordPoints_Installables::get_installer( 'component', $slug );
+		return WordPoints_Installables::get_installer('component', $slug);
 	}
 }
 

src/classes/hook/extension.php

Spacing +24 added lines, -24 removed lines

@@ -57,25 +57,25 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings[ $this->slug ] ) ) {
+		if ( ! isset($settings[$this->slug])) {
 			return $settings;
 		}
 
 		$this->validator = $validator;
 		$this->event_args = $event_args;
 
-		$this->validator->push_field( $this->slug );
+		$this->validator->push_field($this->slug);
 
 		$validated_settings = $this->validate_extension_settings(
-			$settings[ $this->slug ]
+			$settings[$this->slug]
 		);
 
 		$this->validator->pop_field();
 
-		if ( null !== $validated_settings ) {
-			$settings[ $this->slug ] = $validated_settings;
+		if (null !== $validated_settings) {
+			$settings[$this->slug] = $validated_settings;
 		} else {
-			unset( $settings[ $this->slug ] );
+			unset($settings[$this->slug]);
 		}
 
 		return $settings;
@@ -95,27 +95,27 @@ 
 	 *
 	 * @return mixed The validated settings, or null if invalid.
 	 */
-	protected function validate_extension_settings( $settings ) {
+	protected function validate_extension_settings($settings) {
 
-		if ( ! is_array( $settings ) ) {
+		if ( ! is_array($settings)) {
 
-			$this->validator->add_error( __( 'Invalid settings format.', 'wordpoints' ) );
+			$this->validator->add_error(__('Invalid settings format.', 'wordpoints'));
 
 			return null;
 		}
 
-		foreach ( $settings as $action_type => $action_type_settings ) {
+		foreach ($settings as $action_type => $action_type_settings) {
 
-			$this->validator->push_field( $action_type );
+			$this->validator->push_field($action_type);
 
 			$validated_settings = $this->validate_action_type_settings(
 				$action_type_settings
 			);
 
-			if ( null !== $validated_settings ) {
-				$settings[ $action_type ] = $validated_settings;
+			if (null !== $validated_settings) {
+				$settings[$action_type] = $validated_settings;
 			} else {
-				unset( $settings[ $action_type ] );
+				unset($settings[$action_type]);
 			}
 
 			$this->validator->pop_field();
@@ -127,12 +127,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);
 		}
 	}
 
@@ -147,7 +147,7 @@ 
 	 *
 	 * @return mixed The validated settings, or null if invalid.
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return $settings;
 	}
 
@@ -164,16 +164,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;
 		}

src/classes/hook/extension/conditions.php

Spacing +55 added lines, -55 removed lines

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

src/classes/hook/event/args.php

Spacing +28 added lines, -28 removed lines

@@ -73,30 +73,30 @@ 
 	 *
 	 * @param WordPoints_Hook_ArgI[] $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->signature_args[ $slug ] = $entity;
+				$this->signature_args[$slug] = $entity;
 
 				// Back-compat. Some events have multiple "primary" args.
-				if ( ! $this->primary_arg_slug ) {
+				if ( ! $this->primary_arg_slug) {
 					$this->primary_arg_slug = $slug;
 				}
 			}
@@ -132,7 +132,7 @@ 
 	 */
 	public function get_signature_args() {
 
-		if ( ! $this->signature_args ) {
+		if ( ! $this->signature_args) {
 			return false;
 		}
 
@@ -156,11 +156,11 @@ 
 			, __CLASS__ . '::get_signature_args()'
 		);
 
-		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];
 	}
 
 	/**
@@ -172,7 +172,7 @@ 
 	 */
 	public function get_stateful_args() {
 
-		return array_diff_key( $this->entities, $this->signature_args );
+		return array_diff_key($this->entities, $this->signature_args);
 	}
 
 	/**
@@ -193,53 +193,53 @@ 
 	 *
 	 * @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 2.1.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(
 						// translators: Arg slug.
-						__( 'The “%s” arg is not registered for this event.', 'wordpoints' )
+						__('The “%s” arg is not registered for this event.', 'wordpoints')
 						, $child_slug
 					)
 				);
 
-			} elseif ( ! ( $this->current instanceof WordPoints_Entity_ParentI ) ) {
+			} elseif ( ! ($this->current instanceof WordPoints_Entity_ParentI)) {
 
 				$this->validator->add_error(
 					sprintf(
 						// translators: Arg name.
-						__( 'Cannot get descendant of %s: not a parent.', 'wordpoints' )
+						__('Cannot get descendant of %s: not a parent.', 'wordpoints')
 						, $this->current->get_title()
 					)
 				);
 
 			} 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(
 						sprintf(
 							// translators: 1. Arg slug; 2. Other arg slug.
-							__( '%1$s does not have a child “%2$s”.', 'wordpoints' )
+							__('%1$s does not have a child “%2$s”.', 'wordpoints')
 							, $this->current->get_slug()
 							, $child_slug
 						)
@@ -249,9 +249,9 @@ 
 
 			} // End if ( no current ) elseif ( current not parent ) else {}.
 
-		} elseif ( $this->push_on_descend ) {
+		} elseif ($this->push_on_descend) {
 
-			$this->validator->push_field( $child_slug );
+			$this->validator->push_field($child_slug);
 
 		} // End if ( ! $result ) elseif ( $this->push_on_descend ).
 
@@ -265,7 +265,7 @@ 
 
 		$ascended = parent::ascend();
 
-		if ( $ascended && $this->validator ) {
+		if ($ascended && $this->validator) {
 			$this->validator->pop_field();
 		}
 
@@ -275,10 +275,10 @@ 
 	/**
 	 * @since 2.1.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/classes/hook/event/post/publish.php

Spacing +15 added lines, -15 removed lines

@@ -26,20 +26,20 @@ 
 	 */
 	public function get_title() {
 
-		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
+		$parsed = wordpoints_parse_dynamic_slug($this->slug);
 
-		switch ( $parsed['dynamic'] ) {
+		switch ($parsed['dynamic']) {
 
 			case 'post':
-				return __( 'Publish Post', 'wordpoints' );
+				return __('Publish Post', 'wordpoints');
 
 			case 'page':
-				return __( 'Publish Page', 'wordpoints' );
+				return __('Publish Page', 'wordpoints');
 
 			default:
 				return sprintf(
 					// translators: Singular post type name.
-					__( 'Publish %s', 'wordpoints' )
+					__('Publish %s', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}
@@ -50,20 +50,20 @@ 
 	 */
 	public function get_description() {
 
-		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
+		$parsed = wordpoints_parse_dynamic_slug($this->slug);
 
-		switch ( $parsed['dynamic'] ) {
+		switch ($parsed['dynamic']) {
 
 			case 'post':
-				return __( 'When a Post is published.', 'wordpoints' );
+				return __('When a Post is published.', 'wordpoints');
 
 			case 'page':
-				return __( 'When a Page is published.', 'wordpoints' );
+				return __('When a Page is published.', 'wordpoints');
 
 			default:
 				return sprintf(
 					// translators: Singular post type name.
-					__( 'When a %s is published.', 'wordpoints' )
+					__('When a %s is published.', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}
@@ -74,20 +74,20 @@ 
 	 */
 	public function get_reversal_text() {
 
-		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
+		$parsed = wordpoints_parse_dynamic_slug($this->slug);
 
-		switch ( $parsed['dynamic'] ) {
+		switch ($parsed['dynamic']) {
 
 			case 'post':
-				return __( 'Post removed.', 'wordpoints' );
+				return __('Post removed.', 'wordpoints');
 
 			case 'page':
-				return __( 'Page removed.', 'wordpoints' );
+				return __('Page removed.', 'wordpoints');
 
 			default:
 				return sprintf(
 					// translators: Singular post type name.
-					_x( '%s removed.', 'post type', 'wordpoints' )
+					_x('%s removed.', 'post type', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}

src/classes/hook/event/comment/leave.php

Spacing +13 added lines, -13 removed lines

@@ -26,23 +26,23 @@ 
 	 */
 	public function get_title() {
 
-		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
+		$parsed = wordpoints_parse_dynamic_slug($this->slug);
 
-		switch ( $parsed['dynamic'] ) {
+		switch ($parsed['dynamic']) {
 
 			case 'post':
-				return __( 'Comment on a Post', 'wordpoints' );
+				return __('Comment on a Post', 'wordpoints');
 
 			case 'page':
-				return __( 'Comment on a Page', 'wordpoints' );
+				return __('Comment on a Page', 'wordpoints');
 
 			case 'attachment':
-				return __( 'Comment on a Media Upload', 'wordpoints' );
+				return __('Comment on a Media Upload', 'wordpoints');
 
 			default:
 				return sprintf(
 					// translators: Singular post type name.
-					__( 'Comment on a %s', 'wordpoints' )
+					__('Comment on a %s', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}
@@ -53,23 +53,23 @@ 
 	 */
 	public function get_description() {
 
-		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
+		$parsed = wordpoints_parse_dynamic_slug($this->slug);
 
-		switch ( $parsed['dynamic'] ) {
+		switch ($parsed['dynamic']) {
 
 			case 'post':
-				return __( 'When a user leaves a comment on a Post.', 'wordpoints' );
+				return __('When a user leaves a comment on a Post.', 'wordpoints');
 
 			case 'page':
-				return __( 'When a user leaves a comment on a Page.', 'wordpoints' );
+				return __('When a user leaves a comment on a Page.', 'wordpoints');
 
 			case 'attachment':
-				return __( 'When a user leaves a comment on a file uploaded to the Media Library.', 'wordpoints' );
+				return __('When a user leaves a comment on a file uploaded to the Media Library.', 'wordpoints');
 
 			default:
 				return sprintf(
 					// translators: Singular post type name.
-					__( 'When a user leaves a comment on a %s.', 'wordpoints' )
+					__('When a user leaves a comment on a %s.', 'wordpoints')
 					, $this->get_entity_title()
 				);
 		}
@@ -79,7 +79,7 @@ 
 	 * @since 2.1.0
 	 */
 	public function get_reversal_text() {
-		return __( 'Comment removed.', 'wordpoints' );
+		return __('Comment removed.', 'wordpoints');
 	}
 }
 

src/classes/hook/condition.php

Spacing +11 added lines, -11 removed lines

@@ -23,34 +23,34 @@ 
 		WordPoints_Hook_Reaction_Validator $validator
 	) {
 
-		if ( ! isset( $settings['value'] ) || '' === $settings['value'] ) {
+		if ( ! isset($settings['value']) || '' === $settings['value']) {
 
 			$settings_fields = $this->get_settings_fields();
 
 			$validator->add_error(
 				sprintf(
 					// translators: Form field name.
-					__( '%s is required.', 'wordpoints' )
+					__('%s is required.', 'wordpoints')
 					, $settings_fields['value']['label']
 				)
 				, 'value'
 			);
 
-		} elseif ( $arg instanceof WordPoints_Entity_Attr ) {
+		} elseif ($arg instanceof WordPoints_Entity_Attr) {
 
-			$data_types = wordpoints_apps()->get_sub_app( 'data_types' );
+			$data_types = wordpoints_apps()->get_sub_app('data_types');
 
-			$data_type = $data_types->get( $arg->get_data_type() );
+			$data_type = $data_types->get($arg->get_data_type());
 
 			// If this data type isn't recognized, that's probably OK. Validation is
 			// just to help the user know that they've made a mistake anyway.
-			if ( ! ( $data_type instanceof WordPoints_Data_TypeI ) ) {
+			if ( ! ($data_type instanceof WordPoints_Data_TypeI)) {
 				return $settings;
 			}
 
-			$validated_value = $data_type->validate_value( $settings['value'] );
+			$validated_value = $data_type->validate_value($settings['value']);
 
-			if ( is_wp_error( $validated_value ) ) {
+			if (is_wp_error($validated_value)) {
 
 				$settings_fields = $this->get_settings_fields();
 
@@ -67,13 +67,13 @@ 
 
 			$settings['value'] = $validated_value;
 
-		} elseif ( $arg instanceof WordPoints_Entity ) {
+		} elseif ($arg instanceof WordPoints_Entity) {
 
-			if ( ! $arg->exists( $settings['value'] ) ) {
+			if ( ! $arg->exists($settings['value'])) {
 				$validator->add_error(
 					sprintf(
 						// translators: 1. Singular item type name; 2. Item ID/slug.
-						__( '%1$s “%2$s” not found.', 'wordpoints' )
+						__('%1$s “%2$s” not found.', 'wordpoints')
 						, $arg->get_title()
 						, $settings['value']
 					)

src/classes/widget.php

Spacing +24 added lines, -24 removed lines

@@ -52,13 +52,13 @@ 
 	 * @param WP_Error|string $message The error message to display.
 	 * @param array           $args    Arguments for widget display.
 	 */
-	public function wordpoints_widget_error( $message, $args ) {
+	public function wordpoints_widget_error($message, $args) {
 
-		if ( ! current_user_can( 'edit_theme_options' ) ) {
+		if ( ! current_user_can('edit_theme_options')) {
 			return;
 		}
 
-		if ( is_wp_error( $message ) ) {
+		if (is_wp_error($message)) {
 			$message = $message->get_error_message();
 		}
 
@@ -73,8 +73,8 @@ 
 				echo wp_kses(
 					sprintf(
 						// translators: 1. Widget name; 2. Error message.
-						esc_html__( 'The “%1$s” widget could not be displayed because of an error: %2$s', 'wordpoints' )
-						, esc_html( $this->name )
+						esc_html__('The “%1$s” widget could not be displayed because of an error: %2$s', 'wordpoints')
+						, esc_html($this->name)
 						,  $message
 					)
 					, 'wordpoints_widget_error'
@@ -104,9 +104,9 @@ 
 	 *
 	 * @return array|WP_Error The verified settings.
 	 */
-	protected function verify_settings( $instance ) {
+	protected function verify_settings($instance) {
 
-		if ( ! isset( $instance['title'] ) ) {
+		if ( ! isset($instance['title'])) {
 			$instance['title'] = '';
 		}
 
@@ -121,12 +121,12 @@ 
 	 * @param array $args     Arguments for widget display.
 	 * @param array $instance The settings for this widget instance.
 	 */
-	public function widget( $args, $instance ) {
+	public function widget($args, $instance) {
 
-		$instance = $this->verify_settings( $instance );
+		$instance = $this->verify_settings($instance);
 
-		if ( is_wp_error( $instance ) ) {
-			$this->wordpoints_widget_error( $instance, $args );
+		if (is_wp_error($instance)) {
+			$this->wordpoints_widget_error($instance, $args);
 			return;
 		}
 
@@ -139,9 +139,9 @@ 
 		 *
 		 * @param string $title The widget title.
 		 */
-		$title = apply_filters( 'widget_title', $instance['title'] );
+		$title = apply_filters('widget_title', $instance['title']);
 
-		if ( ! empty( $title ) ) {
+		if ( ! empty($title)) {
 
 			echo $args['before_title'] . $title . $args['after_title']; // XSS OK, WPCS.
 		}
@@ -155,9 +155,9 @@ 
 		 *
 		 * @param array $instance The settings for this widget instance.
 		 */
-		do_action( "wordpoints_{$widget_slug}_widget_before", $instance );
+		do_action("wordpoints_{$widget_slug}_widget_before", $instance);
 
-		$this->widget_body( $instance );
+		$this->widget_body($instance);
 
 		/**
 		 * After a WordPoints widget.
@@ -166,7 +166,7 @@ 
 		 *
 		 * @param array $instance The settings for this widget instance.
 		 */
-		do_action( "wordpoints_{$widget_slug}_widget_after", $instance );
+		do_action("wordpoints_{$widget_slug}_widget_after", $instance);
 
 		echo $args['after_widget']; // XSS OK here too, WPCS.
 	}
@@ -186,14 +186,14 @@ 
 	 *
 	 * @param array $instance The settings of the widget instance to display.
 	 */
-	protected function widget_body( $instance ) {}
+	protected function widget_body($instance) {}
 
 	/**
 	 * @since 2.0.0
 	 */
-	public function update( $new_instance, $old_instance ) {
+	public function update($new_instance, $old_instance) {
 
-		$this->instance = array_merge( $this->defaults, $old_instance, $new_instance );
+		$this->instance = array_merge($this->defaults, $old_instance, $new_instance);
 
 		$this->update_title();
 
@@ -206,15 +206,15 @@ 
 	 * @since 2.0.0
 	 */
 	public function update_title() {
-		$this->instance['title'] = strip_tags( $this->instance['title'] );
+		$this->instance['title'] = strip_tags($this->instance['title']);
 	}
 
 	/**
 	 * @since 2.0.0
 	 */
-	public function form( $instance ) {
+	public function form($instance) {
 
-		$this->instance = array_merge( $this->defaults, $instance );
+		$this->instance = array_merge($this->defaults, $instance);
 
 		$this->form_title_field();
 
@@ -231,8 +231,8 @@ 
 		?>
 
 		<p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php echo esc_html_x( 'Title', 'form label', 'wordpoints' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>" value="<?php echo esc_attr( $this->instance['title'] ); ?>" />
+			<label for="<?php echo esc_attr($this->get_field_id('title')); ?>"><?php echo esc_html_x('Title', 'form label', 'wordpoints'); ?></label>
+			<input type="text" class="widefat" id="<?php echo esc_attr($this->get_field_id('title')); ?>" name="<?php echo esc_attr($this->get_field_name('title')); ?>" value="<?php echo esc_attr($this->instance['title']); ?>" />
 		</p>
 
 		<?php

src/classes/data/type/text.php

Spacing +3 added lines, -3 removed lines

@@ -20,13 +20,13 @@
 	/**
 	 * @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'
 				// translators: Form field name.
-				, __( '%s must be a text value.', 'wordpoints' )
+				, __('%s must be a text value.', 'wordpoints')
 			);
 		}