WordPoints / wordpoints

src/admin/admin.php

Spacing +4 added lines, -4 removed lines

@@ -17,14 +17,14 @@ 
  *
  * @since 2.1.0
  */
-require_once( WORDPOINTS_DIR . '/admin/includes/functions.php' );
+require_once(WORDPOINTS_DIR . '/admin/includes/functions.php');
 
 /**
  * Admin-side hooks.
  *
  * @since 2.1.0
  */
-require_once( WORDPOINTS_DIR . '/admin/includes/filters.php' );
+require_once(WORDPOINTS_DIR . '/admin/includes/filters.php');
 
 /**
  * Screen: Configuration.
@@ -33,7 +33,7 @@ 
  */
 include_once WORDPOINTS_DIR . 'admin/screens/configure.php';
 
-WordPoints_Class_Autoloader::register_dir( WORDPOINTS_DIR . 'admin/includes' );
-WordPoints_Class_Autoloader::register_dir( WORDPOINTS_DIR . 'admin/classes' );
+WordPoints_Class_Autoloader::register_dir(WORDPOINTS_DIR . 'admin/includes');
+WordPoints_Class_Autoloader::register_dir(WORDPOINTS_DIR . 'admin/classes');
 
 // EOF

src/admin/classes/screen.php

Spacing +5 added lines, -5 removed lines

@@ -62,8 +62,8 @@ 
 	public function hooks() {
 
 		/* Load the JavaScript needed for the settings screen. */
-		add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
-		add_action( 'admin_print_footer_scripts', array( $this, 'footer_scripts' ) );
+		add_action('admin_enqueue_scripts', array($this, 'enqueue_scripts'));
+		add_action('admin_print_footer_scripts', array($this, 'footer_scripts'));
 	}
 
 	/**
@@ -98,12 +98,12 @@ 
 
 		<div class="wrap">
 
-			<h1><?php echo esc_html( $this->get_title() ); ?></h1>
+			<h1><?php echo esc_html($this->get_title()); ?></h1>
 
 			<?php settings_errors(); ?>
 
-			<?php if ( ! empty( $this->tabs ) ) : ?>
-				<?php wordpoints_admin_show_tabs( $this->tabs, false ); ?>
+			<?php if ( ! empty($this->tabs)) : ?>
+				<?php wordpoints_admin_show_tabs($this->tabs, false); ?>
 			<?php endif; ?>
 
 			<?php $this->display_content(); ?>

src/admin/classes/ajax/hooks.php

Indentation +4 added lines, -4 removed lines

@@ -100,8 +100,8 @@ 
 	) {
 
 		return 'wordpoints_create_hook_reaction|' . $reaction_store->get_slug()
-		       . '|' . wordpoints_hooks()->get_current_mode()
-		       . '|' . wp_json_encode( $reaction_store->get_context_id() );
+			   . '|' . wordpoints_hooks()->get_current_mode()
+			   . '|' . wp_json_encode( $reaction_store->get_context_id() );
 	}
 
 	/**
@@ -131,7 +131,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_update_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+			   . wp_json_encode( $reaction->get_guid() );
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_delete_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+			   . wp_json_encode( $reaction->get_guid() );
 	}
 
 	//

Spacing +55 added lines, -55 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->get_id(),
 				'event' => $reaction->get_event_slug(),
 				'reaction_store' => $reaction->get_store_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),
 			)
 		);
 	}
@@ -81,7 +81,7 @@ 
 		WordPoints_Hook_Reaction_StoreI $reaction_store
 	) {
 
-		return wp_create_nonce( self::get_create_nonce_action( $reaction_store ) );
+		return wp_create_nonce(self::get_create_nonce_action($reaction_store));
 	}
 
 	/**
@@ -101,7 +101,7 @@ 
 
 		return 'wordpoints_create_hook_reaction|' . $reaction_store->get_slug()
 		       . '|' . wordpoints_hooks()->get_current_mode()
-		       . '|' . wp_json_encode( $reaction_store->get_context_id() );
+		       . '|' . wp_json_encode($reaction_store->get_context_id());
 	}
 
 	/**
@@ -113,9 +113,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));
 	}
 
 	/**
@@ -131,7 +131,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_update_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+		       . wp_json_encode($reaction->get_guid());
 	}
 
 	/**
@@ -143,9 +143,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));
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_delete_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+		       . wp_json_encode($reaction->get_guid());
 	}
 
 	//
@@ -187,17 +187,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')
 		);
 	}
 
@@ -212,11 +212,11 @@ 
 
 		$reaction_store = $this->get_reaction_store();
 
-		$this->verify_request( $this->get_create_nonce_action( $reaction_store ) );
+		$this->verify_request($this->get_create_nonce_action($reaction_store));
 
-		$reaction = $reaction_store->create_reaction( $this->get_data() );
+		$reaction = $reaction_store->create_reaction($this->get_data());
 
-		$this->send_json_result( $reaction, 'create' );
+		$this->send_json_result($reaction, 'create');
 	}
 
 	/**
@@ -231,14 +231,14 @@ 
 		$reaction_store = $this->get_reaction_store();
 		$reaction = $this->get_reaction();
 
-		$this->verify_request( $this->get_update_nonce_action( $reaction ) );
+		$this->verify_request($this->get_update_nonce_action($reaction));
 
 		$reaction = $reaction_store->update_reaction(
 			$reaction->get_id()
 			, $this->get_data()
 		);
 
-		$this->send_json_result( $reaction, 'update' );
+		$this->send_json_result($reaction, 'update');
 	}
 
 	/**
@@ -253,12 +253,12 @@ 
 		$reaction_store = $this->get_reaction_store();
 		$reaction = $this->get_reaction();
 
-		$this->verify_request( $this->get_delete_nonce_action( $reaction ) );
+		$this->verify_request($this->get_delete_nonce_action($reaction));
 
-		$result = $reaction_store->delete_reaction( $reaction->get_id() );
+		$result = $reaction_store->delete_reaction($reaction->get_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();
@@ -279,11 +279,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,
 			)
 		);
@@ -299,8 +299,8 @@ 
 	 */
 	private function verify_user_can() {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_send_json_error( array( 'message' => __( 'Sorry, you are not allowed to perform this action. Maybe you have been logged out?', 'wordpoints' ) ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_send_json_error(array('message' => __('Sorry, you are not allowed to perform this action. Maybe you have been logged out?', 'wordpoints')));
 		}
 	}
 
@@ -313,14 +313,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'))
 			);
 		}
 	}
@@ -334,16 +334,16 @@ 
 	 */
 	protected function get_reaction_store() {
 
-		if ( ! isset( $_POST['reaction_store'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'reaction_store' );
+		if ( ! isset($_POST['reaction_store'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('reaction_store');
 		}
 
-		$reactor_slug = sanitize_key( $_POST['reaction_store'] ); // WPCS: CSRF OK.
+		$reactor_slug = sanitize_key($_POST['reaction_store']); // WPCS: CSRF OK.
 
-		$reaction_store = wordpoints_hooks()->get_reaction_store( $reactor_slug );
+		$reaction_store = wordpoints_hooks()->get_reaction_store($reactor_slug);
 
-		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI ) {
-			$this->unexpected_error( 'reaction_store_invalid' );
+		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI) {
+			$this->unexpected_error('reaction_store_invalid');
 		}
 
 		$this->reaction_store_slug = $reactor_slug;
@@ -361,16 +361,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->reaction_store->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;
@@ -385,9 +385,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['reaction_store'] );
+		unset($data['id'], $data['action'], $data['nonce'], $data['reaction_store']);
 
 		return $data;
 	}
@@ -400,30 +400,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/classes/screens.php

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
 	 * @since 2.1.0
 	 */
 	public function __construct() {
-		add_action( 'current_screen', array( $this, 'set_current_screen' ) );
+		add_action('current_screen', array($this, 'set_current_screen'));
 	}
 
 	/**
@@ -37,23 +37,23 @@ 
 	 *
 	 * @param WP_Screen $current_screen The WP_Screen object for the current screen.
 	 */
-	public function set_current_screen( $current_screen ) {
+	public function set_current_screen($current_screen) {
 
 		$screen_id = $current_screen->id;
 
-		if ( is_network_admin() ) {
-			$screen_id = substr( $screen_id, 0, -8 /* -network */ );
+		if (is_network_admin()) {
+			$screen_id = substr($screen_id, 0, -8 /* -network */);
 		}
 
-		$screen = $this->get( $screen_id );
+		$screen = $this->get($screen_id);
 
-		if ( ! ( $screen instanceof WordPoints_Admin_Screen ) ) {
+		if ( ! ($screen instanceof WordPoints_Admin_Screen)) {
 			return;
 		}
 
 		$this->current_screen = $screen;
 
-		add_action( "load-{$screen_id}", array( $this->current_screen, 'load' ) );
+		add_action("load-{$screen_id}", array($this->current_screen, 'load'));
 	}
 
 	/**

src/admin/screens/module-install.php

Spacing +4 added lines, -4 removed lines

@@ -9,7 +9,7 @@ 
 
 global $tab;
 
-if ( empty( $tab ) ) {
+if (empty($tab)) {
 	$tab = 'upload';
 }
 
@@ -25,7 +25,7 @@ 
 });
 </script>
 <div class="wrap">
-	<h1><?php esc_html_e( 'Install Modules', 'wordpoints' ); ?></h1>
+	<h1><?php esc_html_e('Install Modules', 'wordpoints'); ?></h1>
 	<?php
 
 	/**
@@ -35,7 +35,7 @@ 
 	 *
 	 * @since 1.1.0
 	 */
-	do_action( 'wordpoints_install_modules_screen' );
+	do_action('wordpoints_install_modules_screen');
 
 	?>
 	<br class="clear" />
@@ -49,7 +49,7 @@ 
 	 *
 	 * @since 1.1.0
 	 */
-	do_action( "wordpoints_install_modules-{$tab}" );
+	do_action("wordpoints_install_modules-{$tab}");
 
 	?>
 </div>

src/admin/screens/configure-components-load.php

Spacing +14 added lines, -14 removed lines

@@ -7,45 +7,45 @@ 
  * @since 1.1.0
  */
 
-if ( ! current_user_can( 'manage_options' ) ) {
+if ( ! current_user_can('manage_options')) {
 	return;
 }
 
 if (
 	'components' !== wordpoints_admin_get_current_tab()
-	|| ! isset( $_POST['wordpoints_component'], $_POST['wordpoints_component_action'], $_POST['_wpnonce'] )
+	|| ! isset($_POST['wordpoints_component'], $_POST['wordpoints_component_action'], $_POST['_wpnonce'])
 ) {
 	return;
 }
 
 $components = WordPoints_Components::instance();
-$component  = sanitize_key( $_POST['wordpoints_component'] );
+$component  = sanitize_key($_POST['wordpoints_component']);
 
-$action = sanitize_key( $_POST['wordpoints_component_action'] );
+$action = sanitize_key($_POST['wordpoints_component_action']);
 
-check_admin_referer( "wordpoints_{$action}_component-{$component}" );
+check_admin_referer("wordpoints_{$action}_component-{$component}");
 
-switch ( $action ) {
+switch ($action) {
 
 	case 'activate':
-		if ( $components->activate( $component ) ) {
+		if ($components->activate($component)) {
 
-			$message = array( 'message' => 1 );
+			$message = array('message' => 1);
 
 		} else {
 
-			$message = array( 'error' => 1 );
+			$message = array('error' => 1);
 		}
 	break;
 
 	case 'deactivate':
-		if ( $components->deactivate( $component ) ) {
+		if ($components->deactivate($component)) {
 
-			$message = array( 'message' => 2 );
+			$message = array('message' => 2);
 
 		} else {
 
-			$message = array( 'error' => 2 );
+			$message = array('error' => 2);
 		}
 	break;
 
@@ -58,9 +58,9 @@ 
 			'page'                 => 'wordpoints_configure',
 			'tab'                  => 'components',
 			'wordpoints_component' => $component,
-			'_wpnonce'             => wp_create_nonce( 'wordpoints_component_' . key( $message ) . "-{$component}" ),
+			'_wpnonce'             => wp_create_nonce('wordpoints_component_' . key($message) . "-{$component}"),
 		)
-		, self_admin_url( 'admin.php' )
+		, self_admin_url('admin.php')
 	)
 );
 

src/admin/screens/configure-settings-load.php

Spacing +7 added lines, -7 removed lines

@@ -12,18 +12,18 @@
 $screen->add_help_tab(
 	array(
 		'id'      => 'overview',
-		'title'   => __( 'Overview', 'wordpoints' ),
+		'title'   => __('Overview', 'wordpoints'),
 		'content' =>
-			'<p>' . esc_html__( 'WordPoints doesn&#8217;t have a lot of configuration options, which makes it simple and easy to use.', 'wordpoints' ) . '</p>
-			<p>' . esc_html__( 'The settings provided below are optional, and are mainly for convenience', 'wordpoints' ) . '</p>
-			<p>' . esc_html__( 'If you need more help getting started with WordPoints, see the links in the help sidebar.', 'wordpoints' ) . '</p>',
+			'<p>' . esc_html__('WordPoints doesn&#8217;t have a lot of configuration options, which makes it simple and easy to use.', 'wordpoints') . '</p>
+			<p>' . esc_html__('The settings provided below are optional, and are mainly for convenience', 'wordpoints') . '</p>
+			<p>' . esc_html__('If you need more help getting started with WordPoints, see the links in the help sidebar.', 'wordpoints') . '</p>',
 	)
 );
 
 $screen->set_help_sidebar(
-	'<p><strong>' . esc_html__( 'For more information:', 'wordpoints' ) . '</strong></p>
-	<p><a href="https://wordpoints.org/user-guide/">' . esc_html__( 'User Guide', 'wordpoints' ) . '</a></p>
-	<p><a href="https://wordpress.org/support/plugin/wordpoints">' . esc_html__( 'Support Forums', 'wordpoints' ) . '</a></p>'
+	'<p><strong>' . esc_html__('For more information:', 'wordpoints') . '</strong></p>
+	<p><a href="https://wordpoints.org/user-guide/">' . esc_html__('User Guide', 'wordpoints') . '</a></p>
+	<p><a href="https://wordpress.org/support/plugin/wordpoints">' . esc_html__('Support Forums', 'wordpoints') . '</a></p>'
 );
 
 // EOF

src/admin/screens/configure-settings.php

Spacing +18 added lines, -18 removed lines

@@ -7,24 +7,24 @@ 
  * @since 1.0.0
  */
 
-if ( ! current_user_can( 'manage_options' ) ) {
+if ( ! current_user_can('manage_options')) {
 	return;
 }
 
 if (
-	isset( $_POST['wordpoints_settings_nonce'], $_POST['excluded_users'] )
-	&& wordpoints_verify_nonce( 'wordpoints_settings_nonce', 'wordpoints_settings_submit', null, 'post' )
+	isset($_POST['wordpoints_settings_nonce'], $_POST['excluded_users'])
+	&& wordpoints_verify_nonce('wordpoints_settings_nonce', 'wordpoints_settings_submit', null, 'post')
 ) {
 
 	// - The form has been submitted.
 
-	$excluded_users = preg_replace( '/\s/', '', sanitize_text_field( wp_unslash( $_POST['excluded_users'] ) ) );
+	$excluded_users = preg_replace('/\s/', '', sanitize_text_field(wp_unslash($_POST['excluded_users'])));
 
-	if ( ! empty( $excluded_users ) ) {
+	if ( ! empty($excluded_users)) {
 
-		$excluded_users = explode( ',', $excluded_users );
+		$excluded_users = explode(',', $excluded_users);
 
-		$excluded_users = array_unique( array_filter( $excluded_users, 'wordpoints_posint' ) );
+		$excluded_users = array_unique(array_filter($excluded_users, 'wordpoints_posint'));
 	}
 
 	wordpoints_update_maybe_network_option(
@@ -39,18 +39,18 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_settings_update' );
+	do_action('wordpoints_admin_settings_update');
 
 	wordpoints_show_admin_message(
-		esc_html__( 'Settings updated.', 'wordpoints' )
+		esc_html__('Settings updated.', 'wordpoints')
 		, 'success'
-		, array( 'dismissible' => true )
+		, array('dismissible' => true)
 	);
 }
 
 ?>
 
-<p><?php esc_html_e( 'Configure WordPoints to your liking.', 'wordpoints' ); ?></p>
+<p><?php esc_html_e('Configure WordPoints to your liking.', 'wordpoints'); ?></p>
 <form id="wordpoints-settings" method="post">
 
 	<?php
@@ -60,20 +60,20 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_settings_top' );
+	do_action('wordpoints_admin_settings_top');
 
 	?>
 
-	<h3><?php esc_html_e( 'Excluded Users', 'wordpoints' ); ?></h3>
-	<p><?php esc_html_e( 'Enter the IDs of users to exclude from leader boards, logs, etc. This may be useful if you use certain accounts for testing.', 'wordpoints' ); ?></p>
+	<h3><?php esc_html_e('Excluded Users', 'wordpoints'); ?></h3>
+	<p><?php esc_html_e('Enter the IDs of users to exclude from leader boards, logs, etc. This may be useful if you use certain accounts for testing.', 'wordpoints'); ?></p>
 	<table class="form-table">
 		<tbody>
 			<tr>
 				<th>
-					<label for="excluded_users"><?php esc_html_e( 'Excluded Users', 'wordpoints' ); ?></label>
+					<label for="excluded_users"><?php esc_html_e('Excluded Users', 'wordpoints'); ?></label>
 				</th>
 				<td>
-					<input type="text" name="excluded_users" id="excluded_users" value="<?php echo esc_attr( implode( ', ', wordpoints_get_maybe_network_array_option( 'wordpoints_excluded_users' ) ) ); ?>" />
+					<input type="text" name="excluded_users" id="excluded_users" value="<?php echo esc_attr(implode(', ', wordpoints_get_maybe_network_array_option('wordpoints_excluded_users'))); ?>" />
 				</td>
 			</tr>
 		</tbody>
@@ -86,10 +86,10 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_settings_bottom' );
+	do_action('wordpoints_admin_settings_bottom');
 
 	?>
 
-	<?php wp_nonce_field( 'wordpoints_settings_submit', 'wordpoints_settings_nonce' ); ?>
+	<?php wp_nonce_field('wordpoints_settings_submit', 'wordpoints_settings_nonce'); ?>
 	<?php submit_button(); ?>
 </form>

src/classes/entity.php

Doc Comments +1 added lines, -1 removed lines

@@ -430,7 +430,7 @@
 	 *
 	 * @since 2.1.0
 	 *
-	 * @return mixed The ID of the entity.
+	 * @return integer The ID of the entity.
 	 */
 	public function get_the_id() {
 		return $this->get_the_value();

Spacing +51 added lines, -51 removed lines

@@ -108,11 +108,11 @@ 
 	 *
 	 * @return mixed The entity, or false if not found.
 	 */
-	protected function get_entity( $id ) {
+	protected function get_entity($id) {
 
-		$entity = call_user_func( $this->getter, $id );
+		$entity = call_user_func($this->getter, $id);
 
-		if ( ! $this->is_entity( $entity ) ) {
+		if ( ! $this->is_entity($entity)) {
 			return false;
 		}
 
@@ -128,13 +128,13 @@ 
 	 *
 	 * @return bool Whether the passed value is an entity.
 	 */
-	protected function is_entity( $entity ) {
+	protected function is_entity($entity) {
 
-		if ( ! is_object( $entity ) && ! is_array( $entity ) ) {
+		if ( ! is_object($entity) && ! is_array($entity)) {
 			return false;
 		}
 
-		return (bool) $this->get_entity_id( $entity );
+		return (bool) $this->get_entity_id($entity);
 	}
 
 	/**
@@ -151,16 +151,16 @@ 
 	 *
 	 * @return mixed The entity, or false if not found.
 	 */
-	protected function get_entity_from_context( $id, $context ) {
+	protected function get_entity_from_context($id, $context) {
 
 		/** @var WordPoints_Entity_Contexts $contexts */
-		$contexts = wordpoints_entities()->get_sub_app( 'contexts' );
+		$contexts = wordpoints_entities()->get_sub_app('contexts');
 
-		if ( ! $contexts->switch_to( $context ) ) {
+		if ( ! $contexts->switch_to($context)) {
 			return false;
 		}
 
-		$entity = $this->get_entity( $id );
+		$entity = $this->get_entity($id);
 
 		$contexts->switch_back();
 
@@ -176,13 +176,13 @@ 
 	 *
 	 * @return bool Whether the passed value is a GUID for an entity of this type.
 	 */
-	protected function is_guid( $guid ) {
+	protected function is_guid($guid) {
 
-		if ( ! is_array( $guid ) ) {
+		if ( ! is_array($guid)) {
 			return false;
 		}
 
-		return isset( $guid[ $this->get_slug() ], $guid[ $this->get_context() ] );
+		return isset($guid[$this->get_slug()], $guid[$this->get_context()]);
 	}
 
 	/**
@@ -197,15 +197,15 @@ 
 	 *         @type array      $context The entity context.
 	 * }
 	 */
-	protected function split_guid( $guid ) {
+	protected function split_guid($guid) {
 
 		$slug    = $this->get_slug();
-		$id      = $guid[ $slug ];
+		$id      = $guid[$slug];
 		$context = $guid;
 
-		unset( $context[ $slug ] );
+		unset($context[$slug]);
 
-		return array( 'id' => $id, 'context' => $context );
+		return array('id' => $id, 'context' => $context);
 	}
 
 	/**
@@ -218,14 +218,14 @@ 
 	 *
 	 * @return mixed The value of the attribute of the entity.
 	 */
-	protected function get_attr_value( $entity, $attr ) {
+	protected function get_attr_value($entity, $attr) {
 
-		if ( is_array( $entity ) ) {
-			if ( isset( $entity[ $attr ] ) ) {
-				return $entity[ $attr ];
+		if (is_array($entity)) {
+			if (isset($entity[$attr])) {
+				return $entity[$attr];
 			}
 		} else {
-			if ( isset( $entity->{$attr} ) ) {
+			if (isset($entity->{$attr} )) {
 				return $entity->{$attr};
 			}
 		}
@@ -242,8 +242,8 @@ 
 	 *
 	 * @return mixed The ID of the entity.
 	 */
-	protected function get_entity_id( $entity ) {
-		return $this->get_attr_value( $entity, $this->get_id_field() );
+	protected function get_entity_id($entity) {
+		return $this->get_attr_value($entity, $this->get_id_field());
 	}
 
 	/**
@@ -259,8 +259,8 @@ 
 	 *
 	 * @return mixed The human ID of the entity.
 	 */
-	protected function get_entity_human_id( $entity ) {
-		return $this->get_attr_value( $entity, $this->human_id_field );
+	protected function get_entity_human_id($entity) {
+		return $this->get_attr_value($entity, $this->human_id_field);
 	}
 
 	//
@@ -302,15 +302,15 @@ 
 	 *
 	 * @return string|int|float|false The human identifier for the entity, or false.
 	 */
-	public function get_human_id( $id ) {
+	public function get_human_id($id) {
 
-		$entity = $this->get_entity( $id );
+		$entity = $this->get_entity($id);
 
-		if ( ! $entity ) {
+		if ( ! $entity) {
 			return false;
 		}
 
-		return $this->get_entity_human_id( $entity );
+		return $this->get_entity_human_id($entity);
 	}
 
 	/**
@@ -322,8 +322,8 @@ 
 	 *
 	 * @return bool Whether or not an entity with that ID exists.
 	 */
-	public function exists( $id ) {
-		return (bool) $this->get_entity( $id );
+	public function exists($id) {
+		return (bool) $this->get_entity($id);
 	}
 
 	/**
@@ -338,17 +338,17 @@ 
 	 *
 	 * @return WordPoints_Entityish|false The child's object, or false if not found.
 	 */
-	public function get_child( $child_slug ) {
+	public function get_child($child_slug) {
 
-		$children = wordpoints_entities()->get_sub_app( 'children' );
+		$children = wordpoints_entities()->get_sub_app('children');
 
-		$child = $children->get( $this->slug, $child_slug );
+		$child = $children->get($this->slug, $child_slug);
 
 		if (
-			isset( $this->the_value )
+			isset($this->the_value)
 			&& $child instanceof WordPoints_Entity_ChildI
 		) {
-			$child->set_the_value_from_entity( $this );
+			$child->set_the_value_from_entity($this);
 		}
 
 		return $child;
@@ -371,35 +371,35 @@ 
 	 *
 	 * @return bool Whether the value was set.
 	 */
-	public function set_the_value( $value ) {
+	public function set_the_value($value) {
 
 		$this->the_value = null;
 		$this->the_entity = null;
 		$this->the_context = null;
 
-		if ( $this->is_entity( $value ) ) {
+		if ($this->is_entity($value)) {
 
 			$entity = $value;
-			$value = $this->get_entity_id( $value );
+			$value = $this->get_entity_id($value);
 
-		} elseif ( $this->is_guid( $value ) ) {
+		} elseif ($this->is_guid($value)) {
 
-			$guid = $this->split_guid( $value );
+			$guid = $this->split_guid($value);
 			$context = $guid['context'];
 			$value   = $guid['id'];
 
-			$entity = $this->get_entity_from_context( $guid['id'], $context );
+			$entity = $this->get_entity_from_context($guid['id'], $context);
 
 		} else {
 
-			$entity = $this->get_entity( $value );
+			$entity = $this->get_entity($value);
 		}
 
-		if ( ! $entity ) {
+		if ( ! $entity) {
 			return false;
 		}
 
-		if ( ! isset( $context ) ) {
+		if ( ! isset($context)) {
 			$context = wordpoints_entities_get_current_context_id(
 				$this->get_context()
 			);
@@ -421,8 +421,8 @@ 
 	 *
 	 * @return mixed The value of the attribute.
 	 */
-	public function get_the_attr_value( $attr ) {
-		return $this->get_attr_value( $this->the_entity, $attr );
+	public function get_the_attr_value($attr) {
+		return $this->get_attr_value($this->the_entity, $attr);
 	}
 
 	/**
@@ -446,7 +446,7 @@ 
 	 * @return string|int|float|null The human identifier for the entity, or null.
 	 */
 	public function get_the_human_id() {
-		return $this->get_entity_human_id( $this->the_entity );
+		return $this->get_entity_human_id($this->the_entity);
 	}
 
 	/**
@@ -480,11 +480,11 @@ 
 
 		$guid = $this->get_the_context();
 
-		if ( ! is_array( $guid ) ) {
+		if ( ! is_array($guid)) {
 			return $guid;
 		}
 
-		$guid = array( $this->slug => $this->get_the_id() ) + $guid;
+		$guid = array($this->slug => $this->get_the_id()) + $guid;
 
 		return $guid;
 	}