WordPoints / hooks-api

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

Spacing +21 added lines, -21 removed lines

@@ -45,16 +45,16 @@ 
 	 *
 	 * @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();
 
-			if ( $entity instanceof WordPoints_Entity ) {
-				$this->entities[ $arg->get_slug() ] = $entity;
+			if ($entity instanceof WordPoints_Entity) {
+				$this->entities[$arg->get_slug()] = $entity;
 			}
 		}
 	}
@@ -66,54 +66,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;
@@ -126,7 +126,7 @@ 
 
 		$ascended = parent::ascend();
 
-		if ( $ascended && $this->validator ) {
+		if ($ascended && $this->validator) {
 			$this->validator->pop_field();
 		}
 
@@ -136,10 +136,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/hook/event/comment/leave.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Comment', 'wordpoints' );
+		return __('Comment', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'When a user leaves a reply to a Post or other type of content.', 'wordpoints' );
+		return __('When a user leaves a reply to a Post or other type of content.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/event/post/publish.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Post Publish', 'wordpoints' );
+		return __('Post Publish', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'When a Post, Page, or other type of content is published.', 'wordpoints' );
+		return __('When a Post, Page, or other type of content is published.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/event/user/register.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Register', 'wordpoints' );
+		return __('Register', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'Registering.', 'wordpoints' );
+		return __('Registering.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/event/user/visit.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Visit', 'wordpoints' );
+		return __('Visit', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'When a logged-in user or guest visits the site.', 'wordpoints' );
+		return __('When a logged-in user or guest visits the site.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/eventi.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	/**
 	 * @param string $slug The event slug.
 	 */
-	public function __construct( $slug );
+	public function __construct($slug);
 
 	/**
 	 * Get the event slug.

src/includes/classes/hook/events.php

Spacing +21 added lines, -21 removed lines

@@ -37,13 +37,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 
 		$hooks = wordpoints_hooks();
 
 		$this->router = $hooks->router;
 
-		parent::__construct( $slug );
+		parent::__construct($slug);
 	}
 
 	/**
@@ -51,7 +51,7 @@ 
 	 */
 	public function init() {
 
-		$this->sub_apps->register( 'args', 'WordPoints_Class_Registry_Children' );
+		$this->sub_apps->register('args', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -75,29 +75,29 @@ 
 	 *
 	 * @return bool Whether the event was registered.
 	 */
-	public function register( $slug, $class, array $args = array() ) {
+	public function register($slug, $class, array $args = array()) {
 
 		// TODO shoudl this be required?
-		if ( ! isset( $args['actions'] ) ) {
+		if ( ! isset($args['actions'])) {
 			return false;
 		}
 
-		parent::register( $slug, $class, $args );
+		parent::register($slug, $class, $args);
 
-		foreach ( $args['actions'] as $type => $actions ) {
-			foreach ( (array) $actions as $action_slug ) {
-				$this->router->add_event_to_action( $slug, $action_slug, $type );
+		foreach ($args['actions'] as $type => $actions) {
+			foreach ((array) $actions as $action_slug) {
+				$this->router->add_event_to_action($slug, $action_slug, $type);
 			}
 		}
 
 		// TODO should this be required?
-		if ( isset( $args['args'] ) ) {
-			foreach ( $args['args'] as $arg_slug => $class ) {
-				$this->args->register( $slug, $arg_slug, $class );
+		if (isset($args['args'])) {
+			foreach ($args['args'] as $arg_slug => $class) {
+				$this->args->register($slug, $arg_slug, $class);
 			}
 		}
 
-		$this->event_data[ $slug ] = $args;
+		$this->event_data[$slug] = $args;
 
 		return true;
 	}
@@ -105,23 +105,23 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		if ( ! $this->is_registered( $slug ) ) {
+		if ( ! $this->is_registered($slug)) {
 			return;
 		}
 
-		parent::deregister( $slug );
+		parent::deregister($slug);
 
-		foreach ( (array) $this->event_data[ $slug ]['actions'] as $type => $actions ) {
-			foreach ( (array) $actions as $action_slug ) {
-				$this->router->remove_event_from_action( $slug, $action_slug, $type );
+		foreach ((array) $this->event_data[$slug]['actions'] as $type => $actions) {
+			foreach ((array) $actions as $action_slug) {
+				$this->router->remove_event_from_action($slug, $action_slug, $type);
 			}
 		}
 
-		$this->args->deregister_children( $slug );
+		$this->args->deregister_children($slug);
 
-		unset( $this->event_data[ $slug ] );
+		unset($this->event_data[$slug]);
 	}
 }
 

src/includes/classes/hook/extension.php

Spacing +7 added lines, -7 removed lines

@@ -65,15 +65,15 @@ 
 		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 );
-		$settings[ $this->slug ] = $this->{"validate_{$this->slug}"}( $settings[ $this->slug ] );
+		$this->validator->push_field($this->slug);
+		$settings[$this->slug] = $this->{"validate_{$this->slug}"}($settings[$this->slug]);
 		$this->validator->pop_field();
 
 		return $settings;
@@ -82,12 +82,12 @@ 
 	/**
 	 * @since 1.0.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);
 		}
 	}
 

src/includes/classes/hook/extension/periods.php

Spacing +53 added lines, -53 removed lines

@@ -28,27 +28,27 @@ 
 	 *
 	 * @return array The validated periods.
 	 */
-	protected function validate_periods( $periods ) {
+	protected function validate_periods($periods) {
 
-		if ( ! is_array( $periods ) ) {
+		if ( ! is_array($periods)) {
 
 			$this->validator->add_error(
-				__( 'Periods do not match expected format.', 'wordpoints' )
+				__('Periods do not match expected format.', 'wordpoints')
 			);
 
 			return array();
 		}
 
-		foreach ( $periods as $index => $period ) {
+		foreach ($periods as $index => $period) {
 
-			$this->validator->push_field( $index );
+			$this->validator->push_field($index);
 
-			$period = $this->validate_period( $period );
+			$period = $this->validate_period($period);
 
-			if ( $period ) {
-				$periods[ $index ] = $period;
+			if ($period) {
+				$periods[$index] = $period;
 			} else {
-				unset( $periods[ $index ] );
+				unset($periods[$index]);
 			}
 
 			$this->validator->pop_field();
@@ -66,56 +66,56 @@ 
 	 *
 	 * @return array|false The validated period, or false if invalid.
 	 */
-	protected function validate_period( $period ) {
+	protected function validate_period($period) {
 
-		if ( ! is_array( $period ) ) {
+		if ( ! is_array($period)) {
 			$this->validator->add_error(
-				__( 'Period does not match expected format.', 'wordpoints' )
+				__('Period does not match expected format.', 'wordpoints')
 			);
 
 			return false;
 		}
 
-		if ( ! isset( $period['args'] ) ) {
+		if ( ! isset($period['args'])) {
 			// TODO is there a default period args?
 			$period['args'] = array();
 		}
 
-		if ( ! is_array( $period['args'] ) ) {
+		if ( ! is_array($period['args'])) {
 			return false;
 		}
 
-		if ( ! $this->event_args->get_from_hierarchy( $period['args'] ) ) {
+		if ( ! $this->event_args->get_from_hierarchy($period['args'])) {
 			$this->validator->add_error(
-				__( 'Invalid period.', 'wordpoints' ) // TODO better error message
+				__('Invalid period.', 'wordpoints') // TODO better error message
 				, 'args'
 			);
 
 			return false;
 		}
 
-		if ( ! isset( $period['settings'] ) ) {
+		if ( ! isset($period['settings'])) {
 			$this->validator->add_error(
-				__( 'Period settings are missing.', 'wordpoints' )
+				__('Period settings are missing.', 'wordpoints')
 			);
 
 			return false;
 		}
 
-		if ( ! isset( $period['settings']['length'] ) ) {
+		if ( ! isset($period['settings']['length'])) {
 			$this->validator->add_error(
-				__( 'Period length setting is missing.', 'wordpoints' )
+				__('Period length setting is missing.', 'wordpoints')
 				, 'settings'
 			);
 
 			return false;
 		}
 
-		$this->validator->push_field( 'settings' );
+		$this->validator->push_field('settings');
 
-		if ( false === wordpoints_posint( $period['settings']['length'] ) ) {
+		if (false === wordpoints_posint($period['settings']['length'])) {
 			$this->validator->add_error(
-				__( 'Period length must be a positive integer.', 'wordpoints' )
+				__('Period length must be a positive integer.', 'wordpoints')
 				, 'length'
 			);
 
@@ -130,11 +130,11 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function should_hit( WordPoints_Hook_Reaction_Validator $reaction, WordPoints_Hook_Event_Args $event_args ) {
+	public function should_hit(WordPoints_Hook_Reaction_Validator $reaction, WordPoints_Hook_Event_Args $event_args) {
 
-		$periods = $reaction->get_meta( 'periods' );
+		$periods = $reaction->get_meta('periods');
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return true;
 		}
 
@@ -142,8 +142,8 @@ 
 
 		$this->event_args = $event_args;
 
-		foreach ( $periods as $period ) {
-			if ( ! $this->has_period_ended( $period, $reaction_id ) ) {
+		foreach ($periods as $period) {
+			if ( ! $this->has_period_ended($period, $reaction_id)) {
 				return false;
 			}
 		}
@@ -163,20 +163,20 @@ 
 	 *
 	 * @return bool Whether the period has ended.
 	 */
-	final private function has_period_ended( array $settings, $reaction_id ) {
+	final private function has_period_ended(array $settings, $reaction_id) {
 
 		$period = $this->get_period_by_reaction(
-			$this->get_arg_values( $settings['args'] )
+			$this->get_arg_values($settings['args'])
 			, $reaction_id
 		);
 
 		// If the period isn't found, we know that we can still fire.
-		if ( ! $period ) {
+		if ( ! $period) {
 			return true;
 		}
 
-		if ( ! empty( $settings['settings']['absolute'] ) ) {
-			return ( $period->expiration < current_time( 'timestamp' ) );
+		if ( ! empty($settings['settings']['absolute'])) {
+			return ($period->expiration < current_time('timestamp'));
 		} else {
 			return false;
 		}
@@ -191,18 +191,18 @@ 
 	 *
 	 * @return array The arg values.
 	 */
-	final private function get_arg_values( array $period_args ) {
+	final private function get_arg_values(array $period_args) {
 
 		$values = array();
 
-		foreach ( $period_args as $arg_slug => $sub_args ) {
+		foreach ($period_args as $arg_slug => $sub_args) {
 
-			$this->event_args->descend( $arg_slug );
+			$this->event_args->descend($arg_slug);
 
-			if ( is_array( $sub_args ) ) {
-				$values = array_merge( $values, $this->get_arg_values( $sub_args ) );
+			if (is_array($sub_args)) {
+				$values = array_merge($values, $this->get_arg_values($sub_args));
 			} else {
-				$values[ $arg_slug ] = $this->event_args->get_current()->get_the_value();
+				$values[$arg_slug] = $this->event_args->get_current()->get_the_value();
 			}
 
 			$this->event_args->ascend();
@@ -220,11 +220,11 @@ 
 	 *
 	 * @return object|false The period data, or false if not found.
 	 */
-	final private function get_period( $period_id ) {
+	final private function get_period($period_id) {
 
-		$period = wp_cache_get( $period_id, 'wordpoints_hook_period' );
+		$period = wp_cache_get($period_id, 'wordpoints_hook_period');
 
-		if ( ! $period ) {
+		if ( ! $period) {
 
 			global $wpdb;
 
@@ -239,12 +239,12 @@ 
 				)
 			);
 
-			if ( ! $period ) {
+			if ( ! $period) {
 				return false;
 			}
 
-			wp_cache_set( $period->hook_id . $period->arg_hash, $period->id, 'wordpoints_hook_period_ids' );
-			wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+			wp_cache_set($period->hook_id.$period->arg_hash, $period->id, 'wordpoints_hook_period_ids');
+			wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 		}
 
 		return $period;
@@ -260,19 +260,19 @@ 
 	 *
 	 * @return object|false The period data, or false if not found.
 	 */
-	final private function get_period_by_reaction( array $args, $reaction_id ) {
+	final private function get_period_by_reaction(array $args, $reaction_id) {
 
 		// The periods for a reaction are differentiated by a hash of specific args.
-		$arg_hash = md5( serialize( $args ) );
+		$arg_hash = md5(serialize($args));
 
 		$cache_key = "{$this->slug}-{$reaction_id}-{$arg_hash}";
 
 		// Before we run the query, we try to lookup the ID in the cache.
-		$period_id = wp_cache_get( $cache_key, 'wordpoints_hook_period_ids' );
+		$period_id = wp_cache_get($cache_key, 'wordpoints_hook_period_ids');
 
 		// If we found it, we can retrieve the period by ID instead.
-		if ( $period_id ) {
-			return $this->get_period( $period_id );
+		if ($period_id) {
+			return $this->get_period($period_id);
 		}
 
 		global $wpdb;
@@ -291,12 +291,12 @@ 
 			)
 		);
 
-		if ( ! $period ) {
+		if ( ! $period) {
 			return false;
 		}
 
-		wp_cache_set( $cache_key, $period->id, 'wordpoints_hook_period_ids' );
-		wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+		wp_cache_set($cache_key, $period->id, 'wordpoints_hook_period_ids');
+		wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 
 		return $period;
 	}