WordPoints / hooks-api

src/includes/classes/hook/retroactive/query/modifieri.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
  */
 
 interface WordPoints_Hook_Retroactive_Query_ModifierI {
-	public function modify_retroactive_query( WordPoints_Hook_Retroactive_QueryI $query );
+	public function modify_retroactive_query(WordPoints_Hook_Retroactive_QueryI $query);
 }
 
 // EOF

src/includes/classes/hook/retroactive/queryi.php

Spacing +4 added lines, -4 removed lines

@@ -50,7 +50,7 @@ 
 
 interface WordPoints_Hook_Retroactive_QueryI {
 
-	public function __construct( WordPoints_Hook_ReactionI $reaction );
+	public function __construct(WordPoints_Hook_ReactionI $reaction);
 
 	/**
 	 *
@@ -61,13 +61,13 @@ 
 	public function get_reaction();
 	public function get_validator();
 
-	public function arg_hierarchy_push( $slug );
+	public function arg_hierarchy_push($slug);
 	public function arg_hierarchy_pop();
 	public function get_arg();
 
-	public function set_target( $target_arg );
+	public function set_target($target_arg);
 	public function select_value();
-	public function add_condition( array $condition );
+	public function add_condition(array $condition);
 
 	public function get_results();
 }

src/includes/classes/hook/router.php

Spacing +48 added lines, -48 removed lines

@@ -80,13 +80,13 @@ 
 	/**
 	 * @since 1.0.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];
 		}
 
@@ -103,13 +103,13 @@ 
 	 *                     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;
 		}
 
-		if ( ! isset( $this->actions ) ) {
+		if ( ! isset($this->actions)) {
 
 			$hooks = wordpoints_hooks();
 
@@ -118,41 +118,41 @@ 
 			$this->firers  = $hooks->firers;
 		}
 
-		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->events->args->get_children( $event_slug, array( $action_object ) );
+					$event_args = $this->events->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);
 
-					$firer = $this->firers->get( $type );
+					$firer = $this->firers->get($type);
 
-					if ( $firer instanceof WordPoints_Hook_FirerI ) {
-						$firer->do_event( $event_slug, $event_args );
+					if ($firer instanceof WordPoints_Hook_FirerI) {
+						$firer->do_event($event_slug, $event_args);
 					}
 				}
 			}
@@ -182,56 +182,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);
 	}
 
 	/**
@@ -241,20 +241,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);
 				}
 			}
 		}
@@ -269,8 +269,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;
 	}
 
 	/**
@@ -282,8 +282,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]);
 	}
 }
 

src/includes/constants.php

Spacing +1 added lines, -1 removed lines

@@ -9,6 +9,6 @@
 
 global $wpdb;
 
-$wpdb->wordpoints_hook_periods = $wpdb->base_prefix . 'wordpoints_hook_periods';
+$wpdb->wordpoints_hook_periods = $wpdb->base_prefix.'wordpoints_hook_periods';
 
 // EOF

src/includes/classes/data/type.php

Spacing +1 added lines, -1 removed lines

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

src/includes/classes/entity/comment/author.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Author', 'wordpoints' );
+		return __('Author', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/action/comment/new.php

Spacing +2 added lines, -2 removed lines

@@ -19,11 +19,11 @@
 	 */
 	public function should_fire() {
 
-		if ( ! isset( $this->args[1]->comment_approved ) ) {
+		if ( ! isset($this->args[1]->comment_approved)) {
 			return false;
 		}
 
-		if ( 1 !== (int) $this->args[1]->comment_approved ) {
+		if (1 !== (int) $this->args[1]->comment_approved) {
 			return false;
 		}
 

src/admin/admin.php

Spacing +3 added lines, -3 removed lines

@@ -12,17 +12,17 @@
  *
  * @since 1.0.0
  */
-require_once( dirname( __FILE__ ) . '/includes/functions.php' );
+require_once(dirname(__FILE__).'/includes/functions.php');
 
 /**
  * The admin-side actions and filters.
  *
  * @since 1.0.0
  */
-require_once( dirname( __FILE__ ) . '/includes/actions.php' );
+require_once(dirname(__FILE__).'/includes/actions.php');
 
 WordPoints_Class_Autoloader::register_dir(
-	dirname( __FILE__ ) . '/includes/classes'
+	dirname(__FILE__).'/includes/classes'
 	, 'WordPoints_Admin_'
 );
 

src/admin/includes/classes/ajax/hooks.php

Spacing +42 added lines, -42 removed lines

@@ -52,7 +52,7 @@ 
 	 *
 	 * @return array The hook reaction data extracted into an array.
 	 */
-	public static function prepare_hook_reaction( $reaction ) {
+	public static function prepare_hook_reaction($reaction) {
 
 		$reactor = $reaction->get_reactor_slug();
 
@@ -95,17 +95,17 @@ 
 
 		add_action(
 			'wp_ajax_wordpoints_admin_create_hook_reaction'
-			, array( $this, 'create_hook_reaction' )
+			, array($this, 'create_hook_reaction')
 		);
 
 		add_action(
 			'wp_ajax_wordpoints_admin_update_hook_reaction'
-			, array( $this, 'update_hook_reaction' )
+			, array($this, 'update_hook_reaction')
 		);
 
 		add_action(
 			'wp_ajax_wordpoints_admin_delete_hook_reaction'
-			, array( $this, 'delete_hook_reaction' )
+			, array($this, 'delete_hook_reaction')
 		);
 	}
 
@@ -124,9 +124,9 @@ 
 			"wordpoints_create_hook_reaction|{$this->reactor_slug}"
 		);
 
-		$reaction = $reactor->reactions->create_reaction( $this->get_data() );
+		$reaction = $reactor->reactions->create_reaction($this->get_data());
 
-		$this->send_json_result( $reaction, 'create' );
+		$this->send_json_result($reaction, 'create');
 	}
 
 	/**
@@ -150,7 +150,7 @@ 
 			, $this->get_data()
 		);
 
-		$this->send_json_result( $reaction, 'update' );
+		$this->send_json_result($reaction, 'update');
 	}
 
 	/**
@@ -169,10 +169,10 @@ 
 			"wordpoints_delete_hook_reaction|{$this->reactor_slug}|{$reaction->ID}"
 		);
 
-		$result = $reactor->reactions->delete_reaction( $reaction->ID );
+		$result = $reactor->reactions->delete_reaction($reaction->ID);
 
-		if ( ! $result ) {
-			wp_send_json_error( array( 'message' => __( 'There was an error deleting the reaction. Please try again.', 'wordpoints' ) ) );
+		if ( ! $result) {
+			wp_send_json_error(array('message' => __('There was an error deleting the reaction. Please try again.', 'wordpoints')));
 		}
 
 		wp_send_json_success();
@@ -193,11 +193,11 @@ 
 	 *
 	 * @param string $debug_context Context sent with the message (for debugging).
 	 */
-	private function unexpected_error( $debug_context ) {
+	private function unexpected_error($debug_context) {
 
 		wp_send_json_error(
 			array(
-				'message' => __( 'There was an unexpected error. Try reloading the page.', 'wordpoints' ),
+				'message' => __('There was an unexpected error. Try reloading the page.', 'wordpoints'),
 				'debug'   => $debug_context,
 			)
 		);
@@ -213,8 +213,8 @@ 
 	 */
 	private function verify_user_can() {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_send_json_error( array( 'message' => __( 'You do not have permission to perform this action. Maybe you have been logged out?', 'wordpoints' ) ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_send_json_error(array('message' => __('You do not have permission to perform this action. Maybe you have been logged out?', 'wordpoints')));
 		}
 	}
 
@@ -227,14 +227,14 @@ 
 	 *
 	 * @param string $action The action the nonce should be for.
 	 */
-	private function verify_request( $action ) {
+	private function verify_request($action) {
 
 		if (
-			empty( $_POST['nonce'] )
-			|| ! wordpoints_verify_nonce( 'nonce', $action, null, 'post' )
+			empty($_POST['nonce'])
+			|| ! wordpoints_verify_nonce('nonce', $action, null, 'post')
 		) {
 			wp_send_json_error(
-				array( 'message' => __( 'Your security token for this action has expired. Refresh the page and try again.', 'wordpoints' ) )
+				array('message' => __('Your security token for this action has expired. Refresh the page and try again.', 'wordpoints'))
 			);
 		}
 	}
@@ -248,16 +248,16 @@ 
 	 */
 	protected function get_reactor() {
 
-		if ( ! isset( $_POST['reactor'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'reactor' );
+		if ( ! isset($_POST['reactor'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('reactor');
 		}
 
-		$reactor_slug = sanitize_key( $_POST['reactor'] ); // WPCS: CSRF OK.
+		$reactor_slug = sanitize_key($_POST['reactor']); // WPCS: CSRF OK.
 
-		$reactor = wordpoints_hooks()->reactors->get( $reactor_slug );
+		$reactor = wordpoints_hooks()->reactors->get($reactor_slug);
 
-		if ( ! $reactor instanceof WordPoints_Hook_Reactor ) {
-			$this->unexpected_error( 'reactor_invalid' );
+		if ( ! $reactor instanceof WordPoints_Hook_Reactor) {
+			$this->unexpected_error('reactor_invalid');
 		}
 
 		$this->reactor_slug = $reactor_slug;
@@ -275,16 +275,16 @@ 
 	 */
 	protected function get_reaction() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'id' );
+		if ( ! isset($_POST['id'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('id');
 		}
 
 		$reaction = $this->reactor->reactions->get_reaction(
-			wordpoints_int( $_POST['id'] ) // WPCS: CSRF OK.
+			wordpoints_int($_POST['id']) // WPCS: CSRF OK.
 		);
 
-		if ( ! $reaction ) {
-			wp_send_json_error( array( 'message' => __( 'The reaction ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints' ) ) );
+		if ( ! $reaction) {
+			wp_send_json_error(array('message' => __('The reaction ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints')));
 		}
 
 		return $reaction;
@@ -299,9 +299,9 @@ 
 	 */
 	protected function get_data() {
 
-		$data = wp_unslash( $_POST ); // WPCS: CSRF OK.
+		$data = wp_unslash($_POST); // WPCS: CSRF OK.
 
-		unset( $data['id'], $data['action'], $data['nonce'], $data['reactor'] );
+		unset($data['id'], $data['action'], $data['nonce'], $data['reactor']);
 
 		return $data;
 	}
@@ -314,30 +314,30 @@ 
 	 * @param mixed  $result The result of the action.
 	 * @param string $action The action being performed: 'create' or 'update'.
 	 */
-	private function send_json_result( $result, $action ) {
+	private function send_json_result($result, $action) {
 
-		if ( ! $result ) {
+		if ( ! $result) {
 
-			if ( 'create' === $action ) {
-				$message = __( 'There was an error adding the reaction. Please try again.', 'wordpoints' );
+			if ('create' === $action) {
+				$message = __('There was an error adding the reaction. Please try again.', 'wordpoints');
 			} else {
-				$message = __( 'There was an error updating the reaction. Please try again.', 'wordpoints' );
+				$message = __('There was an error updating the reaction. Please try again.', 'wordpoints');
 			}
 
-			wp_send_json_error( array( 'message' => $message ) );
+			wp_send_json_error(array('message' => $message));
 
-		} elseif ( $result instanceof WordPoints_Hook_Reaction_Validator ) {
+		} elseif ($result instanceof WordPoints_Hook_Reaction_Validator) {
 
-			wp_send_json_error( array( 'errors' => $result->get_errors() ) );
+			wp_send_json_error(array('errors' => $result->get_errors()));
 		}
 
 		$data = null;
 
-		if ( 'create' === $action ) {
-			$data = self::prepare_hook_reaction( $result );
+		if ('create' === $action) {
+			$data = self::prepare_hook_reaction($result);
 		}
 
-		wp_send_json_success( $data );
+		wp_send_json_success($data);
 	}
 }