WordPoints / wordpoints

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,32 +147,32 @@ 
 	 * @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(
-					__( 'Unknown condition type “%s”.', 'wordpoints' )
+					__('Unknown condition type “%s”.', 'wordpoints')
 					, $settings['type']
 				)
 				, 'type'
@@ -181,17 +181,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
@@ -199,7 +199,7 @@ 
 			, $this->validator
 		);
 
-		list( $this->validator, $this->event_args ) = $backup;
+		list($this->validator, $this->event_args) = $backup;
 
 		$this->validator->pop_field();
 
@@ -209,11 +209,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;
 		}
 
@@ -235,35 +235,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;
 			}
 		}
@@ -280,13 +280,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/conditioni.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 	 *
 	 * @return bool Whether this condition is met.
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args );
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args);
 }
 
 // EOF

src/classes/hook/extensioni.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 	 *
 	 * @return bool Whether the target should be hit by this hook firing.
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire );
+	public function should_hit(WordPoints_Hook_Fire $fire);
 }
 
 // EOF

src/classes/hook/actions.php

Spacing +10 added lines, -10 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->router = wordpoints_hooks()->get_sub_app( 'router' );
+		$this->router = wordpoints_hooks()->get_sub_app('router');
 	}
 
 	/**
@@ -40,23 +40,23 @@ 
 	 *
 	 * @return object|false The action object, or false if not found.
 	 */
-	public function get( $slug, array $action_args = array(), array $args = array() ) {
+	public function get($slug, array $action_args = array(), array $args = array()) {
 
-		if ( ! isset( $this->classes[ $slug ] ) ) {
+		if ( ! isset($this->classes[$slug])) {
 			return false;
 		}
 
-		return new $this->classes[ $slug ]( $slug, $action_args, $args );
+		return new $this->classes[$slug]($slug, $action_args, $args);
 	}
 
 	/**
 	 * @since 2.1.0
 	 */
-	public function register( $slug, $class, array $args = array() ) {
+	public function register($slug, $class, array $args = array()) {
 
-		parent::register( $slug, $class, $args );
+		parent::register($slug, $class, $args);
 
-		$this->router->add_action( $slug, $args );
+		$this->router->add_action($slug, $args);
 
 		return true;
 	}
@@ -64,11 +64,11 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		parent::deregister( $slug );
+		parent::deregister($slug);
 
-		$this->router->remove_action( $slug );
+		$this->router->remove_action($slug);
 	}
 }
 

src/classes/hook/condition/string/contains.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	 * @since 2.1.0
 	 */
 	public function get_title() {
-		return __( 'Contains', 'wordpoints' );
+		return __('Contains', 'wordpoints');
 	}
 
 	/**
@@ -28,7 +28,7 @@ 
 		return array(
 			'value' => array(
 				'type'     => 'text',
-				'label'    => __( 'Word or phrase (case sensitive)', 'wordpoints' ),
+				'label'    => __('Word or phrase (case sensitive)', 'wordpoints'),
 				'required' => true,
 			),
 		);
@@ -37,7 +37,7 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args) {
 
 		return false !== strpos(
 			$args->get_current()->get_the_value()

src/classes/hook/condition/entity/array/contains.php

Spacing +35 added lines, -35 removed lines

@@ -47,7 +47,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->conditions_extension = wordpoints_hooks()->get_sub_app( 'extensions' )->get(
+		$this->conditions_extension = wordpoints_hooks()->get_sub_app('extensions')->get(
 			'conditions'
 		);
 	}
@@ -56,7 +56,7 @@ 
 	 * @since 2.1.0
 	 */
 	public function get_title() {
-		return __( 'Contains', 'wordpoints' );
+		return __('Contains', 'wordpoints');
 	}
 
 	/**
@@ -66,13 +66,13 @@ 
 		return array(
 			'min' => array(
 				'slug'    => 'min',
-				'label'   => __( 'Minimum number of items', 'wordpoints' ),
+				'label'   => __('Minimum number of items', 'wordpoints'),
 				'type'    => 'number',
 				'default' => 1,
 			),
 			'max' => array(
 				'slug'  => 'max',
-				'label' => __( 'Maximum number of items', 'wordpoints' ),
+				'label' => __('Maximum number of items', 'wordpoints'),
 				'type'  => 'number',
 			),
 		);
@@ -92,8 +92,8 @@ 
 
 		$this->validate_count();
 
-		if ( isset( $settings['conditions'] ) ) {
-			$this->validate_conditions( $arg );
+		if (isset($settings['conditions'])) {
+			$this->validate_conditions($arg);
 		}
 
 		return $this->settings;
@@ -107,31 +107,31 @@ 
 	protected function validate_count() {
 
 		if (
-			! empty( $this->settings['max'] )
-			&& ! wordpoints_posint( $this->settings['max'] )
+			! empty($this->settings['max'])
+			&& ! wordpoints_posint($this->settings['max'])
 		) {
 			$this->validator->add_error(
-				__( 'The maximum must be a positive integer.', 'wordpoints' )
+				__('The maximum must be a positive integer.', 'wordpoints')
 				, 'max'
 			);
 		}
 
-		if ( ! empty( $this->settings['min'] ) ) {
+		if ( ! empty($this->settings['min'])) {
 
-			if ( ! wordpoints_posint( $this->settings['min'] ) ) {
+			if ( ! wordpoints_posint($this->settings['min'])) {
 
 				$this->validator->add_error(
-					__( 'The minimum must be a positive integer.', 'wordpoints' )
+					__('The minimum must be a positive integer.', 'wordpoints')
 					, 'min'
 				);
 
 			} elseif (
-				! empty( $this->settings['max'] )
+				! empty($this->settings['max'])
 				&& $this->settings['max'] < $this->settings['min']
 			) {
 
 				$this->validator->add_error(
-					__( 'The minimum must be less than the maximum.', 'wordpoints' )
+					__('The minimum must be less than the maximum.', 'wordpoints')
 					, 'min'
 				);
 			}
@@ -145,24 +145,24 @@ 
 	 *
 	 * @param WordPoints_EntityishI $current_arg The current arg.
 	 */
-	protected function validate_conditions( $current_arg ) {
+	protected function validate_conditions($current_arg) {
 
-		$args = new WordPoints_Hook_Event_Args( array() );
+		$args = new WordPoints_Hook_Event_Args(array());
 
-		if ( $current_arg instanceof WordPoints_Entity_Array ) {
+		if ($current_arg instanceof WordPoints_Entity_Array) {
 
 			$entity = wordpoints_entities()->get(
 				$current_arg->get_entity_slug()
 			);
 
-			if ( $entity instanceof WordPoints_Entity ) {
-				$args->add_entity( $entity );
+			if ($entity instanceof WordPoints_Entity) {
+				$args->add_entity($entity);
 			}
 		}
 
-		$args->set_validator( $this->validator );
+		$args->set_validator($this->validator);
 
-		$this->validator->push_field( 'conditions' );
+		$this->validator->push_field('conditions');
 
 		$this->settings['conditions'] = $this->conditions_extension->validate_conditions(
 			$this->settings['conditions']
@@ -175,7 +175,7 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args) {
 
 		$this->settings = $settings;
 
@@ -183,16 +183,16 @@ 
 
 		$entities = array();
 
-		if ( $arg instanceof WordPoints_Entity_Array ) {
+		if ($arg instanceof WordPoints_Entity_Array) {
 
 			$entities = $arg->get_the_entities();
 
-			if ( isset( $this->settings['conditions'] ) ) {
-				$entities = $this->filter_entities( $entities );
+			if (isset($this->settings['conditions'])) {
+				$entities = $this->filter_entities($entities);
 			}
 		}
 
-		return $this->check_count( count( $entities ) );
+		return $this->check_count(count($entities));
 	}
 
 	/**
@@ -204,20 +204,20 @@ 
 	 *
 	 * @return WordPoints_Entity[] The entities that matched the sub-conditions.
 	 */
-	protected function filter_entities( $entities ) {
+	protected function filter_entities($entities) {
 
-		foreach ( $entities as $index => $entity ) {
+		foreach ($entities as $index => $entity) {
 
-			$event_args = new WordPoints_Hook_Event_Args( array() );
-			$event_args->add_entity( $entity );
+			$event_args = new WordPoints_Hook_Event_Args(array());
+			$event_args->add_entity($entity);
 
 			$matches = $this->conditions_extension->conditions_are_met(
 				$this->settings['conditions']
 				, $event_args
 			);
 
-			if ( ! $matches ) {
-				unset( $entities[ $index ] );
+			if ( ! $matches) {
+				unset($entities[$index]);
 			}
 		}
 
@@ -233,13 +233,13 @@ 
 	 *
 	 * @return bool Whether the count met the requirements.
 	 */
-	protected function check_count( $count ) {
+	protected function check_count($count) {
 
-		if ( isset( $this->settings['max'] ) && $count > $this->settings['max'] ) {
+		if (isset($this->settings['max']) && $count > $this->settings['max']) {
 			return false;
 		}
 
-		if ( isset( $this->settings['min'] ) && $count < $this->settings['min'] ) {
+		if (isset($this->settings['min']) && $count < $this->settings['min']) {
 			return false;
 		}
 

src/classes/hook/condition/equals.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	 * @since 2.1.0
 	 */
 	public function get_title() {
-		return __( 'Equals', 'wordpoints' );
+		return __('Equals', 'wordpoints');
 	}
 
 	/**
@@ -29,7 +29,7 @@ 
 		return array(
 			'value' => array(
 				'type' => 'text',
-				'label' => _x( 'Value', 'equals hook condition label', 'wordpoints' ),
+				'label' => _x('Value', 'equals hook condition label', 'wordpoints'),
 			),
 		);
 	}
@@ -37,7 +37,7 @@ 
 	/**
 	 * @since 2.1.0
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args) {
 
 		return $settings['value'] === $args->get_current()->get_the_value();
 	}

src/classes/hook/reactor.php

Spacing +8 added lines, -8 removed lines

@@ -111,21 +111,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');
 			}
 		}
 
@@ -135,8 +135,8 @@ 
 	/**
 	 * @since 2.1.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/classes/hook/fire.php

Spacing +4 added lines, -4 removed lines

@@ -88,11 +88,11 @@ 
 	 */
 	public function hit() {
 
-		if ( ! $this->hit_id ) {
+		if ( ! $this->hit_id) {
 
 			$this->hit_id = $this->log_hit();
 
-			if ( ! $this->hit_id ) {
+			if ( ! $this->hit_id) {
 				return false;
 			}
 		}
@@ -128,11 +128,11 @@ 
 					$this->reaction->get_context_id()
 				),
 				'reaction_id' => $this->reaction->get_id(),
-				'date' => current_time( 'mysql', true ),
+				'date' => current_time('mysql', true),
 			)
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}