WordPoints / hooks-api

src/includes/classes/hook/reactor/points.php

Spacing +36 added lines, -36 removed lines

@@ -53,9 +53,9 @@ 
 	 */
 	public function get_settings_fields() {
 
-		$this->settings_fields['points']['label'] = _x( 'Points', 'form label', 'wordpoints' );
-		$this->settings_fields['log_text']['label'] = _x( 'Log Text', 'form label', 'wordpoints' );
-		$this->settings_fields['description']['label'] = _x( 'Description', 'form label', 'wordpoints' );
+		$this->settings_fields['points']['label'] = _x('Points', 'form label', 'wordpoints');
+		$this->settings_fields['log_text']['label'] = _x('Log Text', 'form label', 'wordpoints');
+		$this->settings_fields['description']['label'] = _x('Description', 'form label', 'wordpoints');
 
 		return parent::get_settings_fields();
 	}
@@ -67,8 +67,8 @@ 
 
 		$data = parent::get_ui_script_data();
 
-		$data['target_label'] = __( 'Award To', 'wordpoints' );
-		$data['periods_label'] = __( 'Award each user no more than once per:', 'wordpoints' );
+		$data['target_label'] = __('Award To', 'wordpoints');
+		$data['periods_label'] = __('Award each user no more than once per:', 'wordpoints');
 
 		return $data;
 	}
@@ -82,23 +82,23 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings['points'] ) || false === wordpoints_int( $settings['points'] ) ) {
-			$validator->add_error( __( 'Points must be an integer.', 'wordpoints' ), 'points' );
+		if ( ! isset($settings['points']) || false === wordpoints_int($settings['points'])) {
+			$validator->add_error(__('Points must be an integer.', 'wordpoints'), 'points');
 		}
 
-		if ( ! isset( $settings['points_type'] ) || ! wordpoints_is_points_type( $settings['points_type'] ) ) {
-			$validator->add_error( __( 'Invalid points type.', 'wordpoints' ), 'points_type' );
+		if ( ! isset($settings['points_type']) || ! wordpoints_is_points_type($settings['points_type'])) {
+			$validator->add_error(__('Invalid points type.', 'wordpoints'), 'points_type');
 		}
 
-		if ( ! isset( $settings['description'] ) ) {
-			$validator->add_error( __( 'Description is required.', 'wordpoints' ), 'description' );
+		if ( ! isset($settings['description'])) {
+			$validator->add_error(__('Description is required.', 'wordpoints'), 'description');
 		}
 
-		if ( ! isset( $settings['log_text'] ) ) {
-			$validator->add_error( __( 'Log Text is required.', 'wordpoints' ), 'log_text' );
+		if ( ! isset($settings['log_text'])) {
+			$validator->add_error(__('Log Text is required.', 'wordpoints'), 'log_text');
 		}
 
-		return parent::validate_settings( $settings, $validator, $event_args );
+		return parent::validate_settings($settings, $validator, $event_args);
 	}
 
 	/**
@@ -109,49 +109,49 @@ 
 		array $settings
 	) {
 
-		parent::update_settings( $reaction, $settings );
+		parent::update_settings($reaction, $settings);
 
-		$reaction->update_meta( 'points', $settings['points'] );
-		$reaction->update_meta( 'points_type', $settings['points_type'] );
-		$reaction->update_meta( 'description', $settings['description'] );
-		$reaction->update_meta( 'log_text', $settings['log_text'] );
+		$reaction->update_meta('points', $settings['points']);
+		$reaction->update_meta('points_type', $settings['points_type']);
+		$reaction->update_meta('description', $settings['description']);
+		$reaction->update_meta('log_text', $settings['log_text']);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function hit( WordPoints_Hook_Fire $fire ) {
+	public function hit(WordPoints_Hook_Fire $fire) {
 
 		$reaction = $fire->reaction;
 
 		$target = $fire->event_args->get_from_hierarchy(
-			$reaction->get_meta( 'target' )
+			$reaction->get_meta('target')
 		);
 
-		if ( ! $target instanceof WordPoints_Entity ) {
+		if ( ! $target instanceof WordPoints_Entity) {
 			return;
 		}
 
 		$meta = array();
 
-		foreach ( $fire->event_args->get_entities() as $entity ) {
-			$meta[ $entity->get_slug() ] = $entity->get_the_id();
+		foreach ($fire->event_args->get_entities() as $entity) {
+			$meta[$entity->get_slug()] = $entity->get_the_id();
 		}
 
 		wordpoints_alter_points(
 			$target->get_the_id()
-			, $reaction->get_meta( 'points' )
-			, $reaction->get_meta( 'points_type' )
+			, $reaction->get_meta('points')
+			, $reaction->get_meta('points_type')
 			, $reaction->get_event_slug()
 			, $meta
-			, $reaction->get_meta( 'log_text' )
+			, $reaction->get_meta('log_text')
 		);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function reverse_hit( WordPoints_Hook_Fire $fire ) {
+	public function reverse_hit(WordPoints_Hook_Fire $fire) {
 
 		$meta_queries = array(
 			array(
@@ -162,7 +162,7 @@ 
 			),
 		);
 
-		foreach ( $fire->event_args->get_entities() as $slug => $entity ) {
+		foreach ($fire->event_args->get_entities() as $slug => $entity) {
 
 			$meta_queries[] = array(
 				'key'   => $slug,
@@ -179,35 +179,35 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			return;
 		}
 
 		global $wpdb;
 
-		add_filter( 'wordpoints_points_log', '__return_false' );
+		add_filter('wordpoints_points_log', '__return_false');
 
-		foreach ( $logs as $log ) {
+		foreach ($logs as $log) {
 
 			wordpoints_alter_points(
 				$log->user_id
 				, -$log->points
 				, $log->points_type
 				, "reverse-{$log->log_type}"
-				, array( 'original_log_id' => $log->id )
+				, array('original_log_id' => $log->id)
 			);
 
-			wordpoints_points_log_delete_all_metadata( $log->id );
+			wordpoints_points_log_delete_all_metadata($log->id);
 
 			// Now delete the log.
 			$wpdb->delete(
 				$wpdb->wordpoints_points_logs
-				, array( 'id' => $log->id )
+				, array('id' => $log->id)
 				, '%d'
 			); // WPCS: cache OK, cleaned by wordpoints_alter_points().
 		}
 
-		remove_filter( 'wordpoints_points_log', '__return_false' );
+		remove_filter('wordpoints_points_log', '__return_false');
 	}
 }
 

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

Indentation +5 added lines, -5 removed lines

@@ -96,9 +96,9 @@ 
 	) {
 
 		return 'wordpoints_create_hook_reaction'
-		       . '|' . $reactor->get_slug()
-		       . '|' . wordpoints_hooks()->get_current_mode()
-		       . '|' . wp_json_encode( $reactor->reactions->get_context_id() );
+			   . '|' . $reactor->get_slug()
+			   . '|' . wordpoints_hooks()->get_current_mode()
+			   . '|' . wp_json_encode( $reactor->reactions->get_context_id() );
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_update_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+			   . wp_json_encode( $reaction->get_guid() );
 	}
 
 	/**
@@ -158,7 +158,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_delete_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+			   . wp_json_encode( $reaction->get_guid() );
 	}
 
 	//

Spacing +58 added lines, -58 removed lines

@@ -52,7 +52,7 @@ 
 	 *
 	 * @return array The hook reaction data extracted into an array.
 	 */
-	public static function prepare_hook_reaction( WordPoints_Hook_ReactionI $reaction ) {
+	public static function prepare_hook_reaction(WordPoints_Hook_ReactionI $reaction) {
 
 		return array_merge(
 			$reaction->get_all_meta()
@@ -60,8 +60,8 @@ 
 				'id' => $reaction->ID,
 				'event' => $reaction->get_event_slug(),
 				'reactor' => $reaction->get_reactor_slug(),
-				'nonce' => self::get_update_nonce( $reaction ),
-				'delete_nonce' => self::get_delete_nonce( $reaction ),
+				'nonce' => self::get_update_nonce($reaction),
+				'delete_nonce' => self::get_delete_nonce($reaction),
 			)
 		);
 	}
@@ -76,9 +76,9 @@ 
 	 *
 	 * @return string A nonce for creating a new reaction to this reactor.
 	 */
-	public static function get_create_nonce( WordPoints_Hook_Reactor $reactor ) {
+	public static function get_create_nonce(WordPoints_Hook_Reactor $reactor) {
 
-		return wp_create_nonce( self::get_create_nonce_action( $reactor ) );
+		return wp_create_nonce(self::get_create_nonce_action($reactor));
 	}
 
 	/**
@@ -96,9 +96,9 @@ 
 	) {
 
 		return 'wordpoints_create_hook_reaction'
-		       . '|' . $reactor->get_slug()
-		       . '|' . wordpoints_hooks()->get_current_mode()
-		       . '|' . wp_json_encode( $reactor->reactions->get_context_id() );
+		       . '|'.$reactor->get_slug()
+		       . '|'.wordpoints_hooks()->get_current_mode()
+		       . '|'.wp_json_encode($reactor->reactions->get_context_id());
 	}
 
 	/**
@@ -110,9 +110,9 @@ 
 	 *
 	 * @return string A nonce for updating this reaction.
 	 */
-	public static function get_update_nonce( WordPoints_Hook_ReactionI $reaction ) {
+	public static function get_update_nonce(WordPoints_Hook_ReactionI $reaction) {
 
-		return wp_create_nonce( self::get_update_nonce_action( $reaction ) );
+		return wp_create_nonce(self::get_update_nonce_action($reaction));
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_update_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+		       . wp_json_encode($reaction->get_guid());
 	}
 
 	/**
@@ -140,9 +140,9 @@ 
 	 *
 	 * @return string A nonce for deleting this reaction.
 	 */
-	public static function get_delete_nonce( WordPoints_Hook_ReactionI $reaction ) {
+	public static function get_delete_nonce(WordPoints_Hook_ReactionI $reaction) {
 
-		return wp_create_nonce( self::get_delete_nonce_action( $reaction ) );
+		return wp_create_nonce(self::get_delete_nonce_action($reaction));
 	}
 
 	/**
@@ -158,7 +158,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_delete_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+		       . wp_json_encode($reaction->get_guid());
 	}
 
 	//
@@ -184,17 +184,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')
 		);
 	}
 
@@ -209,11 +209,11 @@ 
 
 		$reactor = $this->get_reactor();
 
-		$this->verify_request( $this->get_create_nonce_action( $reactor ) );
+		$this->verify_request($this->get_create_nonce_action($reactor));
 
-		$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');
 	}
 
 	/**
@@ -228,14 +228,14 @@ 
 		$reactor  = $this->get_reactor();
 		$reaction = $this->get_reaction();
 
-		$this->verify_request( $this->get_update_nonce_action( $reaction ) );
+		$this->verify_request($this->get_update_nonce_action($reaction));
 
 		$reaction = $reactor->reactions->update_reaction(
 			$reaction->ID
 			, $this->get_data()
 		);
 
-		$this->send_json_result( $reaction, 'update' );
+		$this->send_json_result($reaction, 'update');
 	}
 
 	/**
@@ -250,12 +250,12 @@ 
 		$reactor  = $this->get_reactor();
 		$reaction = $this->get_reaction();
 
-		$this->verify_request( $this->get_delete_nonce_action( $reaction ) );
+		$this->verify_request($this->get_delete_nonce_action($reaction));
 
-		$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();
@@ -276,11 +276,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,
 			)
 		);
@@ -296,8 +296,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')));
 		}
 	}
 
@@ -310,14 +310,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'))
 			);
 		}
 	}
@@ -331,16 +331,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;
@@ -358,16 +358,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;
@@ -382,9 +382,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;
 	}
@@ -397,30 +397,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);
 	}
 }
 

src/admin/includes/classes/screen/points/types.php

Spacing +106 added lines, -106 removed lines

@@ -56,7 +56,7 @@ 
 	 * @since 1.0.0
 	 */
 	protected function get_title() {
-		return _x( 'Points Types', 'page title', 'wordpoints' );
+		return _x('Points Types', 'page title', 'wordpoints');
 	}
 
 	/**
@@ -66,8 +66,8 @@ 
 
 		parent::hooks();
 
-		add_action( 'add_meta_boxes', array( $this, 'add_points_settings_meta_box' ) );
-		add_action( 'add_meta_boxes', array( $this, 'add_event_meta_boxes' ) );
+		add_action('add_meta_boxes', array($this, 'add_points_settings_meta_box'));
+		add_action('add_meta_boxes', array($this, 'add_event_meta_boxes'));
 	}
 
 	/**
@@ -75,9 +75,9 @@ 
 	 */
 	public function enqueue_scripts() {
 
-		wp_enqueue_style( 'wordpoints-hooks-admin' );
+		wp_enqueue_style('wordpoints-hooks-admin');
 
-		wp_enqueue_script( 'postbox' );
+		wp_enqueue_script('postbox');
 
 		wordpoints_hooks_ui_setup_script_data();
 	}
@@ -97,7 +97,7 @@ 
 					.addClass( 'closed' );
 
 				postboxes.add_postbox_toggles(
-					<?php echo wp_json_encode( $this->id ); ?>
+					<?php echo wp_json_encode($this->id); ?>
 				);
 			} );
 		</script>
@@ -112,14 +112,14 @@ 
 	 */
 	public function add_points_settings_meta_box() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
 		add_meta_box(
 			'settings'
-			, __( 'Settings', 'wordpoints' )
-			, array( $this, 'display_points_settings_meta_box' )
+			, __('Settings', 'wordpoints')
+			, array($this, 'display_points_settings_meta_box')
 			, $this->id
 			, 'side'
 			, 'default'
@@ -133,7 +133,7 @@ 
 	 */
 	public function display_points_settings_meta_box() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
@@ -141,12 +141,12 @@ 
 
 		$add_new = 0;
 
-		$points_type = wordpoints_get_points_type( $slug );
+		$points_type = wordpoints_get_points_type($slug);
 
-		if ( ! $points_type ) {
+		if ( ! $points_type) {
 
 			$points_type = array();
-			$add_new     = wp_create_nonce( 'wordpoints_add_new_points_type' );
+			$add_new     = wp_create_nonce('wordpoints_add_new_points_type');
 		}
 
 		$points_type = array_merge(
@@ -161,15 +161,15 @@ 
 		?>
 
 		<form method="post">
-			<?php if ( $slug ) : ?>
+			<?php if ($slug) : ?>
 				<p>
 					<span class="wordpoints-points-slug">
 						<em>
-							<?php echo esc_html( sprintf( __( 'Slug: %s', 'wordpoints' ), $slug ) ); ?>
+							<?php echo esc_html(sprintf(__('Slug: %s', 'wordpoints'), $slug)); ?>
 						</em>
 					</span>
 				</p>
-				<?php wp_nonce_field( "wordpoints_update_points_type-$slug", 'update_points_type' ); ?>
+				<?php wp_nonce_field("wordpoints_update_points_type-$slug", 'update_points_type'); ?>
 			<?php endif; ?>
 
 			<?php
@@ -183,45 +183,45 @@ 
 			 *
 			 * @param string $points_type The slug of the points type.
 			 */
-			do_action( 'wordpoints_points_type_form_top', $slug );
+			do_action('wordpoints_points_type_form_top', $slug);
 
-			if ( $add_new ) {
+			if ($add_new) {
 
 				// Mark the prefix and suffix optional on the add new form.
-				$prefix = _x( 'Prefix (optional):', 'points type', 'wordpoints' );
-				$suffix = _x( 'Suffix (optional):', 'points type', 'wordpoints' );
+				$prefix = _x('Prefix (optional):', 'points type', 'wordpoints');
+				$suffix = _x('Suffix (optional):', 'points type', 'wordpoints');
 
 			} else {
 
-				$prefix = _x( 'Prefix:', 'points type', 'wordpoints' );
-				$suffix = _x( 'Suffix:', 'points type', 'wordpoints' );
+				$prefix = _x('Prefix:', 'points type', 'wordpoints');
+				$suffix = _x('Suffix:', 'points type', 'wordpoints');
 			}
 
 			?>
 
 			<p>
 				<label
-					for="points-name-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html_x( 'Name:', 'points type', 'wordpoints' ); ?></label>
+					for="points-name-<?php echo esc_attr($slug); ?>"><?php echo esc_html_x('Name:', 'points type', 'wordpoints'); ?></label>
 				<input class="widefat" type="text"
-				       id="points-name-<?php echo esc_attr( $slug ); ?>"
+				       id="points-name-<?php echo esc_attr($slug); ?>"
 				       name="points-name"
-				       value="<?php echo esc_attr( $points_type['name'] ); ?>"/>
+				       value="<?php echo esc_attr($points_type['name']); ?>"/>
 			</p>
 			<p>
 				<label
-					for="points-prefix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $prefix ); ?></label>
+					for="points-prefix-<?php echo esc_attr($slug); ?>"><?php echo esc_html($prefix); ?></label>
 				<input class="widefat" type="text"
-				       id="points-prefix-<?php echo esc_attr( $slug ); ?>"
+				       id="points-prefix-<?php echo esc_attr($slug); ?>"
 				       name="points-prefix"
-				       value="<?php echo esc_attr( $points_type['prefix'] ); ?>"/>
+				       value="<?php echo esc_attr($points_type['prefix']); ?>"/>
 			</p>
 			<p>
 				<label
-					for="points-suffix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $suffix ); ?></label>
+					for="points-suffix-<?php echo esc_attr($slug); ?>"><?php echo esc_html($suffix); ?></label>
 				<input class="widefat" type="text"
-				       id="points-suffix-<?php echo esc_attr( $slug ); ?>"
+				       id="points-suffix-<?php echo esc_attr($slug); ?>"
 				       name="points-suffix"
-				       value="<?php echo esc_attr( $points_type['suffix'] ); ?>"/>
+				       value="<?php echo esc_attr($points_type['suffix']); ?>"/>
 			</p>
 
 			<?php
@@ -235,28 +235,28 @@ 
 			 *
 			 * @param string $points_type The slug of the points type.
 			 */
-			do_action( 'wordpoints_points_type_form_bottom', $slug );
+			do_action('wordpoints_points_type_form_bottom', $slug);
 
 			?>
 
 			<input type="hidden" name="points-slug"
-			       value="<?php echo esc_attr( $slug ); ?>"/>
+			       value="<?php echo esc_attr($slug); ?>"/>
 			<input type="hidden" name="add_new" class="add_new"
-			       value="<?php echo esc_attr( $add_new ); ?>"/>
+			       value="<?php echo esc_attr($add_new); ?>"/>
 
 			<div class="hook-control-actions">
 				<div class="alignleft">
 					<?php
 
-					if ( ! $add_new ) {
-						wp_nonce_field( "wordpoints_delete_points_type-{$slug}", 'delete-points-type-nonce' );
-						submit_button( _x( 'Delete', 'points type', 'wordpoints' ), 'delete', 'delete-points-type', false, array( 'id' => "delete_points_type-{$slug}" ) );
+					if ( ! $add_new) {
+						wp_nonce_field("wordpoints_delete_points_type-{$slug}", 'delete-points-type-nonce');
+						submit_button(_x('Delete', 'points type', 'wordpoints'), 'delete', 'delete-points-type', false, array('id' => "delete_points_type-{$slug}"));
 					}
 
 					?>
 				</div>
 				<div class="alignright">
-					<?php submit_button( _x( 'Save', 'points type', 'wordpoints' ), 'button-primary hook-control-save right', 'save-points-type', false, array( 'id' => "points-{$slug}-save" ) ); ?>
+					<?php submit_button(_x('Save', 'points type', 'wordpoints'), 'button-primary hook-control-save right', 'save-points-type', false, array('id' => "points-{$slug}-save")); ?>
 					<span class="spinner"></span>
 				</div>
 				<br class="clear"/>
@@ -273,17 +273,17 @@ 
 	 */
 	public function add_event_meta_boxes() {
 
-		if ( ! $this->current_points_type ) {
+		if ( ! $this->current_points_type) {
 			return;
 		}
 
 		/** @var WordPoints_Hook_EventI $event */
-		foreach ( $this->hooks->events->get_all() as $slug => $event ) {
+		foreach ($this->hooks->events->get_all() as $slug => $event) {
 
 			add_meta_box(
 				"{$this->current_points_type}-{$slug}"
 				, $event->get_title()
-				, array( $this, 'display_event_meta_box' )
+				, array($this, 'display_event_meta_box')
 				, $this->id
 				, 'events'
 				, 'default'
@@ -304,55 +304,55 @@ 
 	 * @param array $points_type The points type this meta-box relates to.
 	 * @param array $meta_box    The data the meta-box was created with.
 	 */
-	public function display_event_meta_box( $points_type, $meta_box ) {
+	public function display_event_meta_box($points_type, $meta_box) {
 
 		$event_slug = $meta_box['args']['slug'];
 
 		/** @var WordPoints_Hook_Reactor $points_reactor */
-		$points_reactor = $this->hooks->reactors->get( 'points' );
+		$points_reactor = $this->hooks->reactors->get('points');
 
 		$data = array();
 
-		foreach ( $points_reactor->reactions->get_reactions_to_event( $event_slug ) as $id => $reaction ) {
-			if ( $reaction->get_meta( 'points_type' ) === $this->current_points_type ) {
+		foreach ($points_reactor->reactions->get_reactions_to_event($event_slug) as $id => $reaction) {
+			if ($reaction->get_meta('points_type') === $this->current_points_type) {
 				$data[] = WordPoints_Admin_Ajax_Hooks::prepare_hook_reaction(
 					$reaction
 				);
 			}
 		}
 
-		$args = $this->hooks->events->args->get_children( $event_slug );
+		$args = $this->hooks->events->args->get_children($event_slug);
 
-		$event_data = array( 'args' => array() );
+		$event_data = array('args' => array());
 
-		foreach ( $args as $slug => $arg ) {
+		foreach ($args as $slug => $arg) {
 
-			$event_data['args'][ $slug ] = array(
+			$event_data['args'][$slug] = array(
 				'slug' => $slug,
 			);
 
-			if ( $arg instanceof WordPoints_Hook_Arg ) {
-				$event_data['args'][ $slug ]['title'] = $arg->get_title();
-				$event_data['args'][ $slug ]['is_stateful'] = $arg->is_stateful();
+			if ($arg instanceof WordPoints_Hook_Arg) {
+				$event_data['args'][$slug]['title'] = $arg->get_title();
+				$event_data['args'][$slug]['is_stateful'] = $arg->is_stateful();
 			}
 		}
 
 		?>
 
 		<script>
-			WordPointsHooksAdminData.events[<?php echo wp_json_encode( $event_slug ); ?>] = <?php echo wp_json_encode( $event_data ); ?>;
-			WordPointsHooksAdminData.reactions[<?php echo wp_json_encode( $event_slug ); ?>] = <?php echo wp_json_encode( $data ); ?>;
+			WordPointsHooksAdminData.events[<?php echo wp_json_encode($event_slug); ?>] = <?php echo wp_json_encode($event_data); ?>;
+			WordPointsHooksAdminData.reactions[<?php echo wp_json_encode($event_slug); ?>] = <?php echo wp_json_encode($data); ?>;
 		</script>
 
 		<div class="wordpoints-hook-reaction-group-container">
 			<p class="description wordpoints-hook-reaction-group-description">
-				<?php echo esc_html( $meta_box['args']['event']->get_description() ); ?>
+				<?php echo esc_html($meta_box['args']['event']->get_description()); ?>
 			</p>
 
 			<div class="wordpoints-hook-reaction-group"
-				data-wordpoints-hooks-hook-event="<?php echo esc_attr( $event_slug ); ?>"
-				data-wordpoints-hooks-points-type="<?php echo esc_attr( $this->current_points_type ); ?>"
-				data-wordpoints-hooks-create-nonce="<?php echo esc_attr( WordPoints_Admin_Ajax_Hooks::get_create_nonce( $points_reactor ) ); ?>"
+				data-wordpoints-hooks-hook-event="<?php echo esc_attr($event_slug); ?>"
+				data-wordpoints-hooks-points-type="<?php echo esc_attr($this->current_points_type); ?>"
+				data-wordpoints-hooks-create-nonce="<?php echo esc_attr(WordPoints_Admin_Ajax_Hooks::get_create_nonce($points_reactor)); ?>"
 				data-wordpoints-hooks-reactor="points">
 			</div>
 
@@ -366,7 +366,7 @@ 
 
 			<div class="controls">
 				<button type="button" class="button-primary add-reaction">
-					<?php esc_html_e( 'Add New', 'wordpoints' ); ?>
+					<?php esc_html_e('Add New', 'wordpoints'); ?>
 				</button>
 			</div>
 		</div>
@@ -386,19 +386,19 @@ 
 		// Show a tab for each points type.
 		$tabs = array();
 
-		foreach ( $points_types as $slug => $settings ) {
-			$tabs[ $slug ] = $settings['name'];
+		foreach ($points_types as $slug => $settings) {
+			$tabs[$slug] = $settings['name'];
 		}
 
-		$tabs['add-new'] = __( 'Add New', 'wordpoints' );
+		$tabs['add-new'] = __('Add New', 'wordpoints');
 
-		$tab = wordpoints_admin_get_current_tab( $tabs );
+		$tab = wordpoints_admin_get_current_tab($tabs);
 
-		if ( 'add-new' !== $tab ) {
+		if ('add-new' !== $tab) {
 			$this->current_points_type = $tab;
 		}
 
-		do_action( 'add_meta_boxes', $this->id );
+		do_action('add_meta_boxes', $this->id);
 
 		$this->tabs = $tabs;
 	}
@@ -410,7 +410,7 @@ 
 	 */
 	public function save_points_type() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
@@ -425,43 +425,43 @@ 
 
 			$settings = array();
 
-			$settings['name']   = trim( sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) ); // WPCS: CSRF OK
-			$settings['prefix'] = ltrim( sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) ); // WPCS: CSRF OK
-			$settings['suffix'] = rtrim( sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) ); // WPCS: CSRF OK
+			$settings['name']   = trim(sanitize_text_field(wp_unslash($_POST['points-name']))); // WPCS: CSRF OK
+			$settings['prefix'] = ltrim(sanitize_text_field(wp_unslash($_POST['points-prefix']))); // WPCS: CSRF OK
+			$settings['suffix'] = rtrim(sanitize_text_field(wp_unslash($_POST['points-suffix']))); // WPCS: CSRF OK
 
 			if (
-				isset( $_POST['points-slug'] )
-				&& wordpoints_verify_nonce( 'update_points_type', 'wordpoints_update_points_type-%s', array( 'points-slug' ), 'post' )
+				isset($_POST['points-slug'])
+				&& wordpoints_verify_nonce('update_points_type', 'wordpoints_update_points_type-%s', array('points-slug'), 'post')
 			) {
 
 				// - We are updating an existing points type.
 
-				$points_type = sanitize_key( $_POST['points-slug'] );
+				$points_type = sanitize_key($_POST['points-slug']);
 
-				$old_settings = wordpoints_get_points_type( $points_type );
+				$old_settings = wordpoints_get_points_type($points_type);
 
-				if ( false === $old_settings ) {
+				if (false === $old_settings) {
 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_update'
-						, __( 'Error: failed updating points type.', 'wordpoints' )
+						, __('Error: failed updating points type.', 'wordpoints')
 						, 'updated'
 					);
 
 					return;
 				}
 
-				if ( is_array( $old_settings ) ) {
-					$settings = array_merge( $old_settings, $settings );
+				if (is_array($old_settings)) {
+					$settings = array_merge($old_settings, $settings);
 				}
 
-				if ( ! wordpoints_update_points_type( $points_type, $settings ) ) {
+				if ( ! wordpoints_update_points_type($points_type, $settings)) {
 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_update'
-						, __( 'Error: failed updating points type.', 'wordpoints' )
+						, __('Error: failed updating points type.', 'wordpoints')
 						, 'updated'
 					);
 
@@ -470,23 +470,23 @@ 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_update'
-						, __( 'Points type updated.', 'wordpoints' )
+						, __('Points type updated.', 'wordpoints')
 						, 'updated'
 					);
 				}
 
-			} elseif ( wordpoints_verify_nonce( 'add_new', 'wordpoints_add_new_points_type', null, 'post' ) ) {
+			} elseif (wordpoints_verify_nonce('add_new', 'wordpoints_add_new_points_type', null, 'post')) {
 
 				// - We are creating a new points type.
 
-				$slug = wordpoints_add_points_type( $settings );
+				$slug = wordpoints_add_points_type($settings);
 
-				if ( ! $slug ) {
+				if ( ! $slug) {
 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_create'
-						, __( 'Please choose a unique name for this points type.', 'wordpoints' )
+						, __('Please choose a unique name for this points type.', 'wordpoints')
 					);
 
 				} else {
@@ -496,26 +496,26 @@ 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_create'
-						, __( 'Points type created.', 'wordpoints' )
+						, __('Points type created.', 'wordpoints')
 						, 'updated'
 					);
 				}
 			}
 
 		} elseif (
-			! empty( $_POST['delete-points-type'] )
-			&& isset( $_POST['points-slug'] )
-			&& wordpoints_verify_nonce( 'delete-points-type-nonce', 'wordpoints_delete_points_type-%s', array( 'points-slug' ), 'post' )
+			! empty($_POST['delete-points-type'])
+			&& isset($_POST['points-slug'])
+			&& wordpoints_verify_nonce('delete-points-type-nonce', 'wordpoints_delete_points_type-%s', array('points-slug'), 'post')
 		) {
 
 			// - We are deleting a points type.
 
-			if ( wordpoints_delete_points_type( sanitize_key( $_POST['points-slug'] ) ) ) {
+			if (wordpoints_delete_points_type(sanitize_key($_POST['points-slug']))) {
 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Points type deleted.', 'wordpoints' )
+					, __('Points type deleted.', 'wordpoints')
 					, 'updated'
 				);
 
@@ -524,7 +524,7 @@ 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Error while deleting.', 'wordpoints' )
+					, __('Error while deleting.', 'wordpoints')
 				);
 			}
 		}
@@ -540,25 +540,25 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_admin_points_events_head' );
+		do_action('wordpoints_admin_points_events_head');
 
-		if ( is_network_admin() ) {
-			$title = __( 'Network Events', 'wordpoints' );
-			$description = __( 'Award points when various events happen on this network.', 'wordpoints' );
+		if (is_network_admin()) {
+			$title = __('Network Events', 'wordpoints');
+			$description = __('Award points when various events happen on this network.', 'wordpoints');
 		} else {
-			$title = __( 'Events', 'wordpoints' );
-			$description = __( 'Award points when various events happen on this site.', 'wordpoints' );
+			$title = __('Events', 'wordpoints');
+			$description = __('Award points when various events happen on this site.', 'wordpoints');
 		}
 
-		$points_type = wordpoints_get_points_type( $this->current_points_type );
+		$points_type = wordpoints_get_points_type($this->current_points_type);
 
 		?>
 
 		<div class="wordpoints-points-type-meta-box-wrap">
 
 				<form>
-					<?php wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false ); ?>
-					<?php wp_nonce_field( 'meta-box-order', 'meta-box-order-nonce', false ); ?>
+					<?php wp_nonce_field('closedpostboxes', 'closedpostboxesnonce', false); ?>
+					<?php wp_nonce_field('meta-box-order', 'meta-box-order-nonce', false); ?>
 				</form>
 
 				<div id="poststuff">
@@ -566,17 +566,17 @@ 
 					<div id="post-body" class="metabox-holder columns-<?php echo 1 === (int) get_current_screen()->get_columns() ? '1' : '2'; ?>">
 
 						<div id="postbox-container-1" class="postbox-container">
-							<?php do_meta_boxes( $this->id, 'side', $points_type ); ?>
+							<?php do_meta_boxes($this->id, 'side', $points_type); ?>
 						</div>
 
-						<?php if ( isset( $this->current_points_type ) ) : ?>
+						<?php if (isset($this->current_points_type)) : ?>
 							<div class="wordpoints-hook-events-heading">
-								<h2><?php echo esc_html( $title ); ?></h2>
-								<p class="description"><?php echo esc_html( $description ); ?></p>
+								<h2><?php echo esc_html($title); ?></h2>
+								<p class="description"><?php echo esc_html($description); ?></p>
 							</div>
 
 							<div id="postbox-container-2" class="postbox-container">
-								<?php do_meta_boxes( $this->id, 'events', $points_type ); ?>
+								<?php do_meta_boxes($this->id, 'events', $points_type); ?>
 							</div>
 						<?php endif; ?>
 
@@ -595,7 +595,7 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_admin_points_events_foot' );
+		do_action('wordpoints_admin_points_events_foot');
 	}
 }
 

src/includes/classes/hook/reaction/store.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
 	 *
 	 * @param string $event_slug The slug of the event this reaction is for.
 	 *
-	 * @return int|false The reaction ID, or false if not created.
+	 * @return integer The reaction ID, or false if not created.
 	 */
 	abstract protected function _create_reaction( $event_slug );
 }

Spacing +25 added lines, -25 removed lines

@@ -60,7 +60,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug, WordPoints_Hook_Reactor $reactor ) {
+	public function __construct($slug, WordPoints_Hook_Reactor $reactor) {
 
 		$this->slug = $slug;
 		$this->reactor = $reactor;
@@ -84,33 +84,33 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_context_id() {
-		return wordpoints_entities_get_current_context_id( $this->context );
+		return wordpoints_entities_get_current_context_id($this->context);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_reaction( $id ) {
+	public function get_reaction($id) {
 
-		if ( ! $this->reaction_exists( $id ) ) {
+		if ( ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		return new $this->reaction_class( $id, $this );
+		return new $this->reaction_class($id, $this);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function create_reaction( array $settings ) {
-		return $this->create_or_update_reaction( $settings );
+	public function create_reaction(array $settings) {
+		return $this->create_or_update_reaction($settings);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_reaction( $id, array $settings ) {
-		return $this->create_or_update_reaction( $settings, $id );
+	public function update_reaction($id, array $settings) {
+		return $this->create_or_update_reaction($settings, $id);
 	}
 
 	/**
@@ -125,41 +125,41 @@ 
 	 *         The reaction object if created/updated successfully. False or a
 	 *         validator instance if not.
 	 */
-	protected function create_or_update_reaction( array $settings, $id = null ) {
+	protected function create_or_update_reaction(array $settings, $id = null) {
 
-		$is_new = ! isset( $id );
+		$is_new = ! isset($id);
 
-		if ( ! $is_new && ! $this->reaction_exists( $id ) ) {
+		if ( ! $is_new && ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		$validator = new WordPoints_Hook_Reaction_Validator( $settings, $this->reactor );
+		$validator = new WordPoints_Hook_Reaction_Validator($settings, $this->reactor);
 		$settings = $validator->validate();
 
-		if ( $validator->had_errors() ) {
+		if ($validator->had_errors()) {
 			return $validator;
 		}
 
-		if ( $is_new ) {
+		if ($is_new) {
 
-			$id = $this->_create_reaction( $settings['event'] );
+			$id = $this->_create_reaction($settings['event']);
 
-			if ( ! $id ) {
+			if ( ! $id) {
 				return false;
 			}
 		}
 
-		$reaction = $this->get_reaction( $id );
+		$reaction = $this->get_reaction($id);
 
-		$reaction->update_event_slug( $settings['event'] );
+		$reaction->update_event_slug($settings['event']);
 
-		unset( $settings['event'] );
+		unset($settings['event']);
 
-		$this->reactor->update_settings( $reaction, $settings );
+		$this->reactor->update_settings($reaction, $settings);
 
 		/** @var WordPoints_Hook_Extension $extension */
-		foreach ( wordpoints_hooks()->extensions->get_all() as $extension ) {
-			$extension->update_settings( $reaction, $settings );
+		foreach (wordpoints_hooks()->extensions->get_all() as $extension) {
+			$extension->update_settings($reaction, $settings);
 		}
 
 		/**
@@ -169,7 +169,7 @@ 
 		 * @param array                     $settings The new settings for the reaction.
 		 * @param bool                      $is_new   Whether the reaction was just now created.
 		 */
-		do_action( 'wordpoints_hook_reaction_save', $reaction, $settings, $is_new );
+		do_action('wordpoints_hook_reaction_save', $reaction, $settings, $is_new);
 
 		return $reaction;
 	}
@@ -186,7 +186,7 @@ 
 	 *
 	 * @return int|false The reaction ID, or false if not created.
 	 */
-	abstract protected function _create_reaction( $event_slug );
+	abstract protected function _create_reaction($event_slug);
 }
 
 // EOF

src/includes/actions.php

Spacing +15 added lines, -15 removed lines

@@ -7,26 +7,26 @@
  * @since 1.0.0
  */
 
-add_action( 'wordpoints_init_app-apps', 'wordpoints_apps_init' );
-add_action( 'wordpoints_init_app-entities', 'wordpoints_entities_app_init' );
+add_action('wordpoints_init_app-apps', 'wordpoints_apps_init');
+add_action('wordpoints_init_app-entities', 'wordpoints_entities_app_init');
 
-add_action( 'wordpoints_init_app_registry-apps-entities', 'wordpoints_entities_init' );
-add_action( 'wordpoints_init_app_registry-entities-contexts', 'wordpoints_entity_contexts_init' );
+add_action('wordpoints_init_app_registry-apps-entities', 'wordpoints_entities_init');
+add_action('wordpoints_init_app_registry-entities-contexts', 'wordpoints_entity_contexts_init');
 
-add_action( 'wordpoints_init_app_registry-apps-data_types', 'wordpoints_data_types_init' );
+add_action('wordpoints_init_app_registry-apps-data_types', 'wordpoints_data_types_init');
 
-add_action( 'wordpoints_init_app_registry-hooks-firers', 'wordpoints_hook_firers_init' );
-add_action( 'wordpoints_init_app_registry-hooks-events', 'wordpoints_hook_events_init' );
-add_action( 'wordpoints_init_app_registry-hooks-actions', 'wordpoints_hook_actions_init' );
-add_action( 'wordpoints_init_app_registry-hooks-reactors', 'wordpoints_hook_reactors_init' );
-add_action( 'wordpoints_init_app_registry-hooks-reaction_stores', 'wordpoints_hook_reaction_stores_init' );
-add_action( 'wordpoints_init_app_registry-hooks-extensions', 'wordpoints_hook_extension_init' );
-add_action( 'wordpoints_init_app_registry-hooks-conditions', 'wordpoints_hook_conditions_init' );
+add_action('wordpoints_init_app_registry-hooks-firers', 'wordpoints_hook_firers_init');
+add_action('wordpoints_init_app_registry-hooks-events', 'wordpoints_hook_events_init');
+add_action('wordpoints_init_app_registry-hooks-actions', 'wordpoints_hook_actions_init');
+add_action('wordpoints_init_app_registry-hooks-reactors', 'wordpoints_hook_reactors_init');
+add_action('wordpoints_init_app_registry-hooks-reaction_stores', 'wordpoints_hook_reaction_stores_init');
+add_action('wordpoints_init_app_registry-hooks-extensions', 'wordpoints_hook_extension_init');
+add_action('wordpoints_init_app_registry-hooks-conditions', 'wordpoints_hook_conditions_init');
 
-add_action( 'wordpoints_modules_loaded', 'wordpoints_init_hooks' );
+add_action('wordpoints_modules_loaded', 'wordpoints_init_hooks');
 
-add_action( 'init', 'wordpoints_hooks_api_add_global_cache_groups', 5 );
+add_action('init', 'wordpoints_hooks_api_add_global_cache_groups', 5);
 
-add_filter( 'wordpoints_user_can_view_points_log', 'wordpoints_hooks_user_can_view_points_log' );
+add_filter('wordpoints_user_can_view_points_log', 'wordpoints_hooks_user_can_view_points_log');
 
 // EOF

src/includes/classes/hook/reaction.php

Spacing +4 added lines, -4 removed lines

@@ -35,18 +35,18 @@
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $id, WordPoints_Hook_Reaction_StoreI $store ) {
+	public function __construct($id, WordPoints_Hook_Reaction_StoreI $store) {
 
-		$this->ID    = wordpoints_int( $id );
+		$this->ID    = wordpoints_int($id);
 		$this->store = $store;
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
-		if ( 'ID' === $var ) {
+		if ('ID' === $var) {
 			return $this->ID;
 		}
 

src/includes/classes/hook/reaction/options.php

Spacing +20 added lines, -20 removed lines

@@ -25,13 +25,13 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_event_slug() {
-		return $this->store->get_reaction_event_from_index( $this->ID );
+		return $this->store->get_reaction_event_from_index($this->ID);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_event_slug( $event_slug ) {
+	public function update_event_slug($event_slug) {
 		return $this->store->update_reaction_event_in_index(
 			$this->ID
 			, $event_slug
@@ -41,63 +41,63 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_meta( $key ) {
+	public function get_meta($key) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) || ! isset( $settings[ $key ] ) ) {
+		if ( ! is_array($settings) || ! isset($settings[$key])) {
 			return false;
 		}
 
-		return $settings[ $key ];
+		return $settings[$key];
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function add_meta( $key, $value ) {
+	public function add_meta($key, $value) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) || isset( $settings[ $key ] ) ) {
+		if ( ! is_array($settings) || isset($settings[$key])) {
 			return false;
 		}
 
-		$settings[ $key ] = $value;
+		$settings[$key] = $value;
 
-		return $this->update_settings( $settings );
+		return $this->update_settings($settings);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_meta( $key, $value ) {
+	public function update_meta($key, $value) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) ) {
+		if ( ! is_array($settings)) {
 			return false;
 		}
 
-		$settings[ $key ] = $value;
+		$settings[$key] = $value;
 
-		return $this->update_settings( $settings );
+		return $this->update_settings($settings);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function delete_meta( $key ) {
+	public function delete_meta($key) {
 
 		$settings = $this->get_settings();
 
-		if ( ! is_array( $settings ) || ! isset( $settings[ $key ] ) ) {
+		if ( ! is_array($settings) || ! isset($settings[$key])) {
 			return false;
 		}
 
-		unset( $settings[ $key ] );
+		unset($settings[$key]);
 
-		return $this->update_settings( $settings );
+		return $this->update_settings($settings);
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 	protected function get_settings() {
 
 		return $this->store->get_option(
-			$this->store->get_settings_option_name( $this->ID )
+			$this->store->get_settings_option_name($this->ID)
 		);
 	}
 
@@ -130,10 +130,10 @@ 
 	 *
 	 * @return bool Whether the settings were updated successfully.
 	 */
-	protected function update_settings( $settings ) {
+	protected function update_settings($settings) {
 
 		return $this->store->update_option(
-			$this->store->get_settings_option_name( $this->ID )
+			$this->store->get_settings_option_name($this->ID)
 			, $settings
 		);
 	}

src/includes/classes/hook/reaction/store/options.php

Spacing +43 added lines, -43 removed lines

@@ -22,25 +22,25 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function reaction_exists( $id ) {
-		return (bool) $this->get_option( $this->get_settings_option_name( $id ) );
+	public function reaction_exists($id) {
+		return (bool) $this->get_option($this->get_settings_option_name($id));
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_reactions() {
-		return $this->create_reaction_objects( $this->get_reaction_index() );
+		return $this->create_reaction_objects($this->get_reaction_index());
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_reactions_to_event( $event_slug ) {
+	public function get_reactions_to_event($event_slug) {
 
 		$index = $this->get_reaction_index();
-		$index = wp_list_filter( $index, array( 'event' => $event_slug ) );
-		return $this->create_reaction_objects( $index );
+		$index = wp_list_filter($index, array('event' => $event_slug));
+		return $this->create_reaction_objects($index);
 	}
 
 	/**
@@ -59,9 +59,9 @@ 
 	 */
 	protected function get_reaction_index() {
 
-		$index = $this->get_option( $this->get_reaction_index_option_name() );
+		$index = $this->get_option($this->get_reaction_index_option_name());
 
-		if ( ! is_array( $index ) ) {
+		if ( ! is_array($index)) {
 			$index = array();
 		}
 
@@ -77,7 +77,7 @@ 
 	 *
 	 * @return bool Whether the index was updated successfully.
 	 */
-	protected function update_reaction_index( $index ) {
+	protected function update_reaction_index($index) {
 
 		return $this->update_option(
 			$this->get_reaction_index_option_name()
@@ -96,15 +96,15 @@ 
 	 *
 	 * @return string|false The event slug, or false if not found.
 	 */
-	public function get_reaction_event_from_index( $id ) {
+	public function get_reaction_event_from_index($id) {
 
 		$index = $this->get_reaction_index();
 
-		if ( ! isset( $index[ $id ]['event'] ) ) {
+		if ( ! isset($index[$id]['event'])) {
 			return false;
 		}
 
-		return $index[ $id ]['event'];
+		return $index[$id]['event'];
 	}
 
 	/**
@@ -119,17 +119,17 @@ 
 	 *
 	 * @return bool Whether the event was updated successfully.
 	 */
-	public function update_reaction_event_in_index( $id, $event ) {
+	public function update_reaction_event_in_index($id, $event) {
 
 		$index = $this->get_reaction_index();
 
-		if ( ! isset( $index[ $id ] ) ) {
+		if ( ! isset($index[$id])) {
 			return false;
 		}
 
-		$index[ $id ]['event'] = $event;
+		$index[$id]['event'] = $event;
 
-		return $this->update_reaction_index( $index );
+		return $this->update_reaction_index($index);
 	}
 
 	/**
@@ -141,15 +141,15 @@ 
 	 *
 	 * @return WordPoints_Hook_Reaction_Options[] The objects for the reactions.
 	 */
-	protected function create_reaction_objects( $index ) {
+	protected function create_reaction_objects($index) {
 
 		$reactions = array();
 
-		foreach ( $index as $reaction ) {
+		foreach ($index as $reaction) {
 
-			$object = $this->get_reaction( $reaction['id'] );
+			$object = $this->get_reaction($reaction['id']);
 
-			if ( ! $object ) {
+			if ( ! $object) {
 				continue;
 			}
 
@@ -162,53 +162,53 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function delete_reaction( $id ) {
+	public function delete_reaction($id) {
 
-		if ( ! $this->reaction_exists( $id ) ) {
+		if ( ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		$result = $this->delete_option( $this->get_settings_option_name( $id ) );
+		$result = $this->delete_option($this->get_settings_option_name($id));
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			return false;
 		}
 
 		$index = $this->get_reaction_index();
 
-		unset( $index[ $id ] );
+		unset($index[$id]);
 
-		return $this->update_reaction_index( $index );
+		return $this->update_reaction_index($index);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function _create_reaction( $event_slug ) {
+	protected function _create_reaction($event_slug) {
 
 		$index = $this->get_reaction_index();
 
 		$id = 1;
 
 		// TODO this is fragile when the newest reaction gets deleted.
-		if ( ! empty( $index ) ) {
-			$id = 1 + max( array_keys( $index ) );
+		if ( ! empty($index)) {
+			$id = 1 + max(array_keys($index));
 		}
 
-		$settings = array( 'event' => $event_slug );
+		$settings = array('event' => $event_slug);
 
 		$result = $this->add_option(
-			$this->get_settings_option_name( $id )
+			$this->get_settings_option_name($id)
 			, $settings
 		);
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			return false;
 		}
 
-		$index[ $id ] = array( 'event' => $event_slug, 'id' => $id );
+		$index[$id] = array('event' => $event_slug, 'id' => $id);
 
-		if ( ! $this->update_reaction_index( $index ) ) {
+		if ( ! $this->update_reaction_index($index)) {
 			return false;
 		}
 
@@ -226,8 +226,8 @@ 
 	 *
 	 * @return mixed The option value, or false.
 	 */
-	public function get_option( $name ) {
-		return get_option( $name );
+	public function get_option($name) {
+		return get_option($name);
 	}
 
 	/**
@@ -240,8 +240,8 @@ 
 	 *
 	 * @return bool Whether the option was added successfully.
 	 */
-	protected function add_option( $name, $value ) {
-		return add_option( $name, $value );
+	protected function add_option($name, $value) {
+		return add_option($name, $value);
 	}
 
 	/**
@@ -256,8 +256,8 @@ 
 	 *
 	 * @return bool Whether the option was updated successfully.
 	 */
-	public function update_option( $name, $value ) {
-		return update_option( $name, $value );
+	public function update_option($name, $value) {
+		return update_option($name, $value);
 	}
 
 	/**
@@ -269,8 +269,8 @@ 
 	 *
 	 * @return bool Whether the option was deleted successfully.
 	 */
-	protected function delete_option( $name ) {
-		return delete_option( $name );
+	protected function delete_option($name) {
+		return delete_option($name);
 	}
 
 	/**
@@ -282,7 +282,7 @@ 
 	 *
 	 * @return string The name of the option where the settings are stored.
 	 */
-	public function get_settings_option_name( $id ) {
+	public function get_settings_option_name($id) {
 		return "wordpoints_{$this->reactor_slug}_hook_reaction-{$id}";
 	}
 

src/includes/classes/hook/reaction/store/options/network.php

Spacing +8 added lines, -8 removed lines

@@ -22,29 +22,29 @@
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_option( $name ) {
-		return get_site_option( $name );
+	public function get_option($name) {
+		return get_site_option($name);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function add_option( $name, $value ) {
-		return add_site_option( $name, $value );
+	protected function add_option($name, $value) {
+		return add_site_option($name, $value);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_option( $name, $value ) {
-		return update_site_option( $name, $value );
+	public function update_option($name, $value) {
+		return update_site_option($name, $value);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function delete_option( $name ) {
-		return delete_site_option( $name );
+	protected function delete_option($name) {
+		return delete_site_option($name);
 	}
 }