WordPoints / wordpoints

src/components/points/admin/includes/filters.php

Spacing +16 added lines, -16 removed lines

@@ -7,34 +7,34 @@
  * @since 2.1.0
  */
 
-add_action( 'init', 'wordpoints_points_admin_register_scripts' );
+add_action('init', 'wordpoints_points_admin_register_scripts');
 
-add_action( 'admin_menu', 'wordpoints_points_admin_menu' );
+add_action('admin_menu', 'wordpoints_points_admin_menu');
 
-if ( is_wordpoints_network_active() ) {
-	add_action( 'network_admin_menu', 'wordpoints_points_admin_menu' );
+if (is_wordpoints_network_active()) {
+	add_action('network_admin_menu', 'wordpoints_points_admin_menu');
 }
 
-add_action( 'load-wordpoints_page_wordpoints_points_hooks', 'wordpoints_admin_points_hooks_help' );
+add_action('load-wordpoints_page_wordpoints_points_hooks', 'wordpoints_admin_points_hooks_help');
 
-add_action( 'load-wordpoints_page_wordpoints_points_hooks', 'wordpoints_no_js_points_hooks_save' );
+add_action('load-wordpoints_page_wordpoints_points_hooks', 'wordpoints_no_js_points_hooks_save');
 
-add_action( 'screen_settings', 'wordpoints_admin_points_hooks_screen_options', 10, 2 );
+add_action('screen_settings', 'wordpoints_admin_points_hooks_screen_options', 10, 2);
 
-add_action( 'wordpoints_in_points_hook_form', 'wordpoints_points_hook_description_form', 10, 3 );
+add_action('wordpoints_in_points_hook_form', 'wordpoints_points_hook_description_form', 10, 3);
 
-add_action( 'personal_options', 'wordpoints_points_profile_options', 20 );
+add_action('personal_options', 'wordpoints_points_profile_options', 20);
 
-add_action( 'personal_options_update', 'wordpoints_points_profile_options_update' );
-add_action( 'edit_user_profile_update', 'wordpoints_points_profile_options_update' );
+add_action('personal_options_update', 'wordpoints_points_profile_options_update');
+add_action('edit_user_profile_update', 'wordpoints_points_profile_options_update');
 
-add_action( 'wordpoints_admin_settings_top', 'wordpoints_points_admin_settings' );
+add_action('wordpoints_admin_settings_top', 'wordpoints_points_admin_settings');
 
-add_action( 'wordpoints_admin_settings_update', 'wordpoints_points_admin_settings_save' );
+add_action('wordpoints_admin_settings_update', 'wordpoints_points_admin_settings_save');
 
-add_action( 'admin_notices', 'wordpoints_points_admin_notices' );
+add_action('admin_notices', 'wordpoints_points_admin_notices');
 
-add_action( 'wp_ajax_wordpoints-points-hooks-order', 'wordpoints_ajax_points_hooks_order' );
-add_action( 'wp_ajax_save-wordpoints-points-hook', 'wordpoints_ajax_save_points_hook' );
+add_action('wp_ajax_wordpoints-points-hooks-order', 'wordpoints_ajax_points_hooks_order');
+add_action('wp_ajax_save-wordpoints-points-hook', 'wordpoints_ajax_save_points_hook');
 
 // EOF

src/components/points/admin/includes/ajax.php

Spacing +79 added lines, -79 removed lines

@@ -16,61 +16,61 @@ 
  */
 function wordpoints_ajax_points_hooks_order() {
 
-	if ( check_ajax_referer( 'save-network-wordpoints-points-hooks', 'savehooks', false ) ) {
+	if (check_ajax_referer('save-network-wordpoints-points-hooks', 'savehooks', false)) {
 
-		if ( ! current_user_can( 'manage_network_wordpoints_points_hooks' ) ) {
-			wp_die( -1, '', array( 'response' => 403 ) );
+		if ( ! current_user_can('manage_network_wordpoints_points_hooks')) {
+			wp_die( -1, '', array('response' => 403) );
 		}
 
 		// Saving network hooks order, turn on network mode.
-		WordPoints_Points_Hooks::set_network_mode( true );
+		WordPoints_Points_Hooks::set_network_mode(true);
 
-	} elseif ( check_ajax_referer( 'save-wordpoints-points-hooks', 'savehooks', false ) ) {
+	} elseif (check_ajax_referer('save-wordpoints-points-hooks', 'savehooks', false)) {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_die( -1, '', array( 'response' => 403 ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_die( -1, '', array('response' => 403) );
 		}
 
 	} else {
 
 		// CSRF attack (or, more probably, the user left the browser open too long).
-		wp_die( -1, '', array( 'response' => 403 ) );
+		wp_die( -1, '', array('response' => 403) );
 	}
 
 	// Save hooks order for all points types.
-	if ( ! isset( $_POST['points_types'] ) || ! is_array( $_POST['points_types'] ) ) {
-		wp_die( -1, '', array( 'response' => 400 ) );
+	if ( ! isset($_POST['points_types']) || ! is_array($_POST['points_types'])) {
+		wp_die( -1, '', array('response' => 400) );
 	}
 
 	$points_types_hooks = array();
 
-	foreach ( array_keys( wordpoints_get_points_types() ) as $points_type ) {
+	foreach (array_keys(wordpoints_get_points_types()) as $points_type) {
 
-		if ( empty( $_POST['points_types'][ $points_type ] ) ) {
+		if (empty($_POST['points_types'][$points_type])) {
 			continue;
 		}
 
-		$hooks = sanitize_text_field( wp_unslash( $_POST['points_types'][ $points_type ] ) );
+		$hooks = sanitize_text_field(wp_unslash($_POST['points_types'][$points_type]));
 
 		$points_type_hooks = array();
 
-		$hooks = explode( ',', $hooks );
+		$hooks = explode(',', $hooks);
 
-		foreach ( $hooks as $order => $hook_id ) {
+		foreach ($hooks as $order => $hook_id) {
 
-			if ( false === strpos( $hook_id, 'hook-' ) ) {
+			if (false === strpos($hook_id, 'hook-')) {
 				continue;
 			}
 
-			$points_type_hooks[ $order ] = substr( $hook_id, strpos( $hook_id, '_' ) + 1 );
+			$points_type_hooks[$order] = substr($hook_id, strpos($hook_id, '_') + 1);
 		}
 
-		$points_types_hooks[ $points_type ] = $points_type_hooks;
+		$points_types_hooks[$points_type] = $points_type_hooks;
 	}
 
-	WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
+	WordPoints_Points_Hooks::save_points_types_hooks($points_types_hooks);
 
-	wp_die( 1, '', array( 'response' => 200 ) );
+	wp_die(1, '', array('response' => 200));
 }
 
 /**
@@ -83,30 +83,30 @@ 
  */
 function wordpoints_ajax_save_points_hook() {
 
-	if ( check_ajax_referer( 'save-network-wordpoints-points-hooks', 'savehooks', false ) ) {
+	if (check_ajax_referer('save-network-wordpoints-points-hooks', 'savehooks', false)) {
 
-		if ( ! current_user_can( 'manage_network_wordpoints_points_hooks' ) ) {
-			wp_die( -1, '', array( 'response' => 403 ) );
+		if ( ! current_user_can('manage_network_wordpoints_points_hooks')) {
+			wp_die( -1, '', array('response' => 403) );
 		}
 
 		// Saving network hooks, turn on network mode.
-		WordPoints_Points_Hooks::set_network_mode( true );
+		WordPoints_Points_Hooks::set_network_mode(true);
 
-	} elseif ( check_ajax_referer( 'save-wordpoints-points-hooks', 'savehooks', false ) ) {
+	} elseif (check_ajax_referer('save-wordpoints-points-hooks', 'savehooks', false)) {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_die( -1, '', array( 'response' => 403 ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_die( -1, '', array('response' => 403) );
 		}
 
 	} else {
 
 		// CSRF attack (or, more probably the user left the browser open too long).
-		wp_die( -1, '', array( 'response' => 403 ) );
+		wp_die( -1, '', array('response' => 403) );
 	}
 
-	$error = __( 'An error has occurred. Please reload the page and try again.', 'wordpoints' );
+	$error = __('An error has occurred. Please reload the page and try again.', 'wordpoints');
 
-	if ( isset( $_POST['points-slug'] ) ) {
+	if (isset($_POST['points-slug'])) {
 
 		// - We are saving the settings for a points type.
 
@@ -117,55 +117,55 @@ 
 			, 'Performing CRUD actions on points types using this function is deprecated.'
 		);
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
-			wp_die( -1, '', array( 'response' => 403 ) );
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
+			wp_die( -1, '', array('response' => 403) );
 		}
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) {
-			$settings['name'] = trim( sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) );
+		if (isset($_POST['points-name'])) {
+			$settings['name'] = trim(sanitize_text_field(wp_unslash($_POST['points-name'])));
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) {
-			$settings['prefix'] = ltrim( sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) );
+		if (isset($_POST['points-prefix'])) {
+			$settings['prefix'] = ltrim(sanitize_text_field(wp_unslash($_POST['points-prefix'])));
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) {
-			$settings['suffix'] = rtrim( sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) );
+		if (isset($_POST['points-suffix'])) {
+			$settings['suffix'] = rtrim(sanitize_text_field(wp_unslash($_POST['points-suffix'])));
 		}
 
-		$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 ) {
-			wp_die( -1, '', array( 'response' => 200 ) );
+		if (false === $old_settings) {
+			wp_die( -1, '', array('response' => 200) );
 		}
 
-		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)) {
 
 			// If this fails, show the user a message along with the form.
-			echo '<p>' . esc_html__( 'An error has occurred. Please try again.', 'wordpoints' ) . '</p>';
+			echo '<p>' . esc_html__('An error has occurred. Please try again.', 'wordpoints') . '</p>';
 
-			WordPoints_Points_Hooks::points_type_form( $points_type, 'none' );
+			WordPoints_Points_Hooks::points_type_form($points_type, 'none');
 		}
 
 	} else {
 
 		// - We are creating/updating/deleting an instance of a hook.
 
-		if ( ! isset( $_POST['id_base'], $_POST['hook-id'], $_POST['points_type'], $_POST['hook_number'] ) ) {
-			wp_die( -1, '', array( 'response' => 400 ) );
+		if ( ! isset($_POST['id_base'], $_POST['hook-id'], $_POST['points_type'], $_POST['hook_number'])) {
+			wp_die( -1, '', array('response' => 400) );
 		}
 
-		$id_base        = sanitize_key( $_POST['id_base'] );
-		$hook_id        = sanitize_key( $_POST['hook-id'] );
-		$points_type_id = sanitize_key( $_POST['points_type'] );
+		$id_base        = sanitize_key($_POST['id_base']);
+		$hook_id        = sanitize_key($_POST['hook-id']);
+		$points_type_id = sanitize_key($_POST['points_type']);
 		$number         = (int) $_POST['hook_number'];
 
 		/*
@@ -174,48 +174,48 @@ 
 		 * created hook, the ID won't have been set when the form was output, so
 		 * 'hook-id' will be empty, and we'll get the ID from 'multi_number'.
 		 */
-		if ( ! $number ) {
+		if ( ! $number) {
 
 			// This holds the ID number if the hook is brand new.
-			if ( ! isset( $_POST['multi_number'] ) || ! wordpoints_posint( $_POST['multi_number'] ) ) {
-				wp_die( '<p>' . esc_html( $error ) . '</p>', '', array( 'response' => 400 ) );
+			if ( ! isset($_POST['multi_number']) || ! wordpoints_posint($_POST['multi_number'])) {
+				wp_die('<p>' . esc_html($error) . '</p>', '', array('response' => 400));
 			}
 
 			$number  = (int) $_POST['multi_number'];
 			$hook_id = $id_base . '-' . $number;
 		}
 
-		$hook = WordPoints_Points_Hooks::get_handler( $hook_id );
+		$hook = WordPoints_Points_Hooks::get_handler($hook_id);
 
 		$settings = false;
 
-		if ( isset( $_POST[ 'hook-' . $id_base ] ) && is_array( $_POST[ 'hook-' . $id_base ] ) ) {
-			$settings = wp_unslash( $_POST[ 'hook-' . $id_base ] ); // WPCS: sanitization OK.
+		if (isset($_POST['hook-' . $id_base]) && is_array($_POST['hook-' . $id_base])) {
+			$settings = wp_unslash($_POST['hook-' . $id_base]); // WPCS: sanitization OK.
 		}
 
 		$points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
 
 		// Get the hooks for this points type.
-		$points_type_hooks = ( isset( $points_types_hooks[ $points_type_id ] ) ) ? $points_types_hooks[ $points_type_id ] : array();
+		$points_type_hooks = (isset($points_types_hooks[$points_type_id])) ? $points_types_hooks[$points_type_id] : array();
 
-		if ( ! empty( $_POST['delete_hook'] ) ) {
+		if ( ! empty($_POST['delete_hook'])) {
 
 			// - We are deleting a hook instance.
 
-			if ( false === $hook ) {
-				wp_die( '<p>' . esc_html( $error ) . '</p>', '', array( 'response' => 400 ) );
+			if (false === $hook) {
+				wp_die('<p>' . esc_html($error) . '</p>', '', array('response' => 400));
 			}
 
-			$hook->delete_callback( $number );
+			$hook->delete_callback($number);
 
 			// Remove this instance of the hook, and reset the positions (keys).
-			$points_types_hooks[ $points_type_id ] = array_diff( $points_type_hooks, array( $hook_id ) );
+			$points_types_hooks[$points_type_id] = array_diff($points_type_hooks, array($hook_id));
 
-			WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
+			WordPoints_Points_Hooks::save_points_types_hooks($points_types_hooks);
 
-			wp_die( esc_html( "deleted:{$hook_id}" ), '', array( 'response' => 200 ) );
+			wp_die(esc_html("deleted:{$hook_id}"), '', array('response' => 200));
 
-		} elseif ( $settings && false === $hook ) {
+		} elseif ($settings && false === $hook) {
 
 			// - We are creating a new a new instance of a hook.
 
@@ -224,36 +224,36 @@ 
 			 * since the hook is new, it hasn't been assigned an ID yet, so we can't
 			 * just get it from the array of hooks by ID.
 			 */
-			$hook = WordPoints_Points_Hooks::get_handler_by_id_base( $id_base );
+			$hook = WordPoints_Points_Hooks::get_handler_by_id_base($id_base);
 
-			$new_instance = reset( $settings );
+			$new_instance = reset($settings);
 
 			// Save the points types-hooks associations.
-			$points_type_hooks[] = $hook->get_id( $number );
-			$points_types_hooks[ $points_type_id ] = $points_type_hooks;
-			WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
+			$points_type_hooks[] = $hook->get_id($number);
+			$points_types_hooks[$points_type_id] = $points_type_hooks;
+			WordPoints_Points_Hooks::save_points_types_hooks($points_types_hooks);
 
 		} else {
 
 			// - We are updating the settings for an instance of a hook.
 
-			if ( false === $hook ) {
-				wp_die( '<p>' . esc_html( $error ) . '</p>', '', array( 'response' => 400 ) );
+			if (false === $hook) {
+				wp_die('<p>' . esc_html($error) . '</p>', '', array('response' => 400));
 			}
 
-			$new_instance = ( ! empty( $settings ) ) ? reset( $settings ) : array();
+			$new_instance = ( ! empty($settings)) ? reset($settings) : array();
 
 		} // End if ( deleting ) elseif ( creating ) else { updating }.
 
-		$hook->update_callback( wp_unslash( $new_instance ), $number );
+		$hook->update_callback(wp_unslash($new_instance), $number);
 
-		if ( empty( $_POST['add_new'] ) ) {
-			$hook->form_callback( $number );
+		if (empty($_POST['add_new'])) {
+			$hook->form_callback($number);
 		}
 
 	} // End if ( saving points type ) else { deleting/creation/updating hook }.
 
-	wp_die( '', '', array( 'response' => 200 ) );
+	wp_die('', '', array('response' => 200));
 }
 
 // EOF

src/components/points/admin/includes/functions.php

Indentation +3 added lines, -3 removed lines

@@ -246,9 +246,9 @@
 			}
 
 			$screen_options = '<p><a id="access-on" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=on">'
-			                  . esc_html__( 'Enable accessibility mode', 'wordpoints' )
-			                  . '</a><a id="access-off" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=off">'
-			                  . esc_html__( 'Disable accessibility mode', 'wordpoints' ) . "</a></p>\n";
+							  . esc_html__( 'Enable accessibility mode', 'wordpoints' )
+							  . '</a><a id="access-off" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=off">'
+							  . esc_html__( 'Disable accessibility mode', 'wordpoints' ) . "</a></p>\n";
 			break;
 	}
 

Braces +5 added lines, -2 removed lines

@@ -290,9 +290,12 @@
 
 	<?php if ( $has_form ) : ?>
 		<hr />
-	<?php else : ?>
+	<?php else {
+	: ?>
 		<br />
-	<?php endif; ?>
+	<?php endif;
+}
+?>
 
 	<div class="hook-instance-description">
 		<label for="<?php $hook->the_field_id( '_description' ); ?>"><?php echo esc_html_x( 'Description (optional):', 'points hook', 'wordpoints' ); ?></label>

Spacing +90 added lines, -90 removed lines

@@ -25,15 +25,15 @@ 
 	wp_register_style(
 		'wordpoints-admin-points-hooks'
 		, "{$assets_url}/css/hooks{$suffix}.css"
-		, array( 'dashicons' )
+		, array('dashicons')
 		, WORDPOINTS_VERSION
 	);
 
 	$styles = wp_styles();
-	$styles->add_data( 'wordpoints-admin-points-hooks', 'rtl', 'replace' );
+	$styles->add_data('wordpoints-admin-points-hooks', 'rtl', 'replace');
 
-	if ( $suffix ) {
-		$styles->add_data( 'wordpoints-admin-points-hooks', 'suffix', $suffix );
+	if ($suffix) {
+		$styles->add_data('wordpoints-admin-points-hooks', 'suffix', $suffix);
 	}
 
 	// JS
@@ -41,7 +41,7 @@ 
 	wp_register_script(
 		'wordpoints-admin-points-types'
 		, "{$assets_url}/js/points-types{$suffix}.js"
-		, array( 'backbone', 'jquery-ui-dialog', 'wp-util' )
+		, array('backbone', 'jquery-ui-dialog', 'wp-util')
 		, WORDPOINTS_VERSION
 	);
 
@@ -49,25 +49,25 @@ 
 		'wordpoints-admin-points-types'
 		, 'WordPointsPointsTypesL10n'
 		, array(
-			'confirmDelete' => esc_html__( 'Are you sure that you want to delete this points type? This will delete all logs, event reactions, and other data associated with this points type.', 'wordpoints' )
-				. ' ' . esc_html__( 'Once a points type has been deleted, you cannot bring it back.', 'wordpoints' ),
-			'confirmTitle'  => esc_html__( 'Are you sure?', 'wordpoints' ),
-			'deleteText'    => esc_html__( 'Delete', 'wordpoints' ),
-			'cancelText'    => esc_html__( 'Cancel', 'wordpoints' ),
+			'confirmDelete' => esc_html__('Are you sure that you want to delete this points type? This will delete all logs, event reactions, and other data associated with this points type.', 'wordpoints')
+				. ' ' . esc_html__('Once a points type has been deleted, you cannot bring it back.', 'wordpoints'),
+			'confirmTitle'  => esc_html__('Are you sure?', 'wordpoints'),
+			'deleteText'    => esc_html__('Delete', 'wordpoints'),
+			'cancelText'    => esc_html__('Cancel', 'wordpoints'),
 		)
 	);
 
 	wp_register_script(
 		'wordpoints-hooks-reactor-points'
 		, "{$assets_url}/js/hooks/reactors/points{$manifested_suffix}.js"
-		, array( 'wordpoints-hooks-views' )
+		, array('wordpoints-hooks-views')
 		, WORDPOINTS_VERSION
 	);
 
 	wp_register_script(
 		'wordpoints-admin-points-hooks'
 		, "{$assets_url}/js/hooks{$suffix}.js"
-		, array( 'jquery', 'jquery-ui-droppable', 'jquery-ui-sortable', 'jquery-ui-dialog' )
+		, array('jquery', 'jquery-ui-droppable', 'jquery-ui-sortable', 'jquery-ui-dialog')
 		, WORDPOINTS_VERSION
 	);
 }
@@ -85,22 +85,22 @@ 
 	$wordpoints_menu = wordpoints_get_main_admin_menu();
 
 	/** @var WordPoints_Admin_Screens $admin_screens */
-	$admin_screens = wordpoints_apps()->get_sub_app( 'admin' )->get_sub_app(
+	$admin_screens = wordpoints_apps()->get_sub_app('admin')->get_sub_app(
 		'screen'
 	);
 
 	// Hooks page.
 	$id = add_submenu_page(
 		$wordpoints_menu
-		, __( 'WordPoints — Points Types', 'wordpoints' )
-		, __( 'Points Types', 'wordpoints' )
+		, __('WordPoints — Points Types', 'wordpoints')
+		, __('Points Types', 'wordpoints')
 		, 'manage_options'
 		, 'wordpoints_points_types'
-		, array( $admin_screens, 'display' )
+		, array($admin_screens, 'display')
 	);
 
-	if ( $id ) {
-		$admin_screens->register( $id, 'WordPoints_Points_Admin_Screen_Points_Types' );
+	if ($id) {
+		$admin_screens->register($id, 'WordPoints_Points_Admin_Screen_Points_Types');
 	}
 
 	// Remove the old hooks screen if not needed.
@@ -113,12 +113,12 @@ 
 
 	// If all of the registered hooks have been imported and disabled, then there is
 	// no need to keep the old hooks screen.
-	if ( array_diff_key( $hooks, $disabled_hooks ) ) {
+	if (array_diff_key($hooks, $disabled_hooks)) {
 		// Legacy hooks page.
 		add_submenu_page(
 			$wordpoints_menu
-			,__( 'WordPoints — Points Hooks', 'wordpoints' )
-			,__( 'Points Hooks', 'wordpoints' )
+			,__('WordPoints — Points Hooks', 'wordpoints')
+			,__('Points Hooks', 'wordpoints')
 			,'manage_options'
 			,'wordpoints_points_hooks'
 			,'wordpoints_points_admin_screen_hooks'
@@ -128,8 +128,8 @@ 
 	// Logs page.
 	add_submenu_page(
 		$wordpoints_menu
-		,__( 'WordPoints — Points Logs', 'wordpoints' )
-		,__( 'Points Logs', 'wordpoints' )
+		,__('WordPoints — Points Logs', 'wordpoints')
+		,__('Points Logs', 'wordpoints')
 		,'manage_options'
 		,'wordpoints_points_logs'
 		,'wordpoints_points_admin_screen_logs'
@@ -143,7 +143,7 @@ 
  */
 function wordpoints_points_admin_screen_hooks() {
 
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if (isset($_GET['edithook']) || isset($_POST['savehook']) || isset($_POST['removehook'])) { // WPCS: CSRF OK.
 
 		// - We're doing this without AJAX (JS).
 
@@ -206,7 +206,7 @@ 
  */
 function wordpoints_no_js_points_hooks_save() {
 
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset($_POST['savehook']) && ! isset($_POST['removehook'])) { // WPCS: CSRF OK.
 		return;
 	}
 
@@ -230,25 +230,25 @@ 
  *
  * @return string Options for this screen.
  */
-function wordpoints_admin_points_hooks_screen_options( $screen_options, $screen ) {
+function wordpoints_admin_points_hooks_screen_options($screen_options, $screen) {
 
 	$path = 'admin.php?page=wordpoints_points_hooks';
 
-	switch ( $screen->id ) {
+	switch ($screen->id) {
 
 		case 'wordpoints_page_wordpoints_points_hooks':
-			$url = admin_url( $path );
+			$url = admin_url($path);
 			// Fall through.
 
 		case 'wordpoints_page_wordpoints_points_hooks-network':
-			if ( ! isset( $url ) ) {
-				$url = network_admin_url( $path );
+			if ( ! isset($url)) {
+				$url = network_admin_url($path);
 			}
 
-			$screen_options = '<p><a id="access-on" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=on">'
-			                  . esc_html__( 'Enable accessibility mode', 'wordpoints' )
-			                  . '</a><a id="access-off" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=off">'
-			                  . esc_html__( 'Disable accessibility mode', 'wordpoints' ) . "</a></p>\n";
+			$screen_options = '<p><a id="access-on" href="' . esc_url(wp_nonce_url($url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce')) . '&amp;accessibility-mode=on">'
+			                  . esc_html__('Enable accessibility mode', 'wordpoints')
+			                  . '</a><a id="access-off" href="' . esc_url(wp_nonce_url($url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce')) . '&amp;accessibility-mode=off">'
+			                  . esc_html__('Disable accessibility mode', 'wordpoints') . "</a></p>\n";
 			break;
 	}
 
@@ -266,7 +266,7 @@ 
  *
  * @return string The classes, with 'wordpoints_hooks_access' added.
  */
-function wordpoints_points_hooks_access_body_class( $classes ) {
+function wordpoints_points_hooks_access_body_class($classes) {
 
 	return "{$classes} wordpoints_hooks_access ";
 }
@@ -282,29 +282,29 @@ 
  * @param array                  $instance The settings for this hook instance.
  * @param WordPoints_Points_Hook $hook     The points hook object.
  */
-function wordpoints_points_hook_description_form( $has_form, $instance, $hook ) {
+function wordpoints_points_hook_description_form($has_form, $instance, $hook) {
 
-	$description = ( isset( $instance['_description'] ) ) ? $instance['_description'] : '';
+	$description = (isset($instance['_description'])) ? $instance['_description'] : '';
 
 	?>
 
-	<?php if ( $has_form ) : ?>
+	<?php if ($has_form) : ?>
 		<hr />
 	<?php else : ?>
 		<br />
 	<?php endif; ?>
 
 	<div class="hook-instance-description">
-		<label for="<?php $hook->the_field_id( '_description' ); ?>"><?php echo esc_html_x( 'Description (optional):', 'points hook', 'wordpoints' ); ?></label>
-		<input type="text" id="<?php $hook->the_field_id( '_description' ); ?>" name="<?php $hook->the_field_name( '_description' ); ?>" class="widefat" value="<?php echo esc_attr( $description ); ?>" />
+		<label for="<?php $hook->the_field_id('_description'); ?>"><?php echo esc_html_x('Description (optional):', 'points hook', 'wordpoints'); ?></label>
+		<input type="text" id="<?php $hook->the_field_id('_description'); ?>" name="<?php $hook->the_field_name('_description'); ?>" class="widefat" value="<?php echo esc_attr($description); ?>" />
 		<p class="description">
 			<?php
 
 			echo esc_html(
 				sprintf(
 					// translators: Default points hook description.
-					_x( 'Default: %s', 'points hook description', 'wordpoints' )
-					, $hook->get_description( 'generated' )
+					_x('Default: %s', 'points hook description', 'wordpoints')
+					, $hook->get_description('generated')
 				)
 			);
 
@@ -326,44 +326,44 @@ 
  *
  * @param WP_User $user The user object for the user being edited.
  */
-function wordpoints_points_profile_options( $user ) {
+function wordpoints_points_profile_options($user) {
 
-	if ( current_user_can( 'set_wordpoints_points', $user->ID ) ) {
+	if (current_user_can('set_wordpoints_points', $user->ID)) {
 
 		?>
 
 		</table>
 
-		<h2><?php esc_html_e( 'WordPoints', 'wordpoints' ); ?></h2>
-		<p><?php esc_html_e( "If you would like to change the value for a type of points, enter the desired value in the text field, and check the checkbox beside it. If you don't check the checkbox, the change will not be saved. To provide a reason for the change, fill out the text field below.", 'wordpoints' ); ?></p>
-		<lable><?php esc_html_e( 'Reason', 'wordpoints' ); ?> <input type="text" name="wordpoints_set_reason" />
+		<h2><?php esc_html_e('WordPoints', 'wordpoints'); ?></h2>
+		<p><?php esc_html_e("If you would like to change the value for a type of points, enter the desired value in the text field, and check the checkbox beside it. If you don't check the checkbox, the change will not be saved. To provide a reason for the change, fill out the text field below.", 'wordpoints'); ?></p>
+		<lable><?php esc_html_e('Reason', 'wordpoints'); ?> <input type="text" name="wordpoints_set_reason" />
 		<table class="form-table">
 
 		<?php
 
-		wp_nonce_field( 'wordpoints_points_set_profile', 'wordpoints_points_set_nonce' );
+		wp_nonce_field('wordpoints_points_set_profile', 'wordpoints_points_set_nonce');
 
-		foreach ( wordpoints_get_points_types() as $slug => $type ) {
+		foreach (wordpoints_get_points_types() as $slug => $type) {
 
-			$points = wordpoints_get_points( $user->ID, $slug );
+			$points = wordpoints_get_points($user->ID, $slug);
 
 			?>
 
 			<tr>
-				<th scope="row"><?php echo esc_html( $type['name'] ); ?></th>
+				<th scope="row"><?php echo esc_html($type['name']); ?></th>
 				<td>
-					<input type="hidden" name="<?php echo esc_attr( "wordpoints_points_old-{$slug}" ); ?>" value="<?php echo esc_attr( $points ); ?>" />
-					<input type="number" name="<?php echo esc_attr( "wordpoints_points-{$slug}" ); ?>" value="<?php echo esc_attr( $points ); ?>" autocomplete="off" />
-					<input type="checkbox" value="1" name="<?php echo esc_attr( "wordpoints_points_set-{$slug}" ); ?>" />
+					<input type="hidden" name="<?php echo esc_attr("wordpoints_points_old-{$slug}"); ?>" value="<?php echo esc_attr($points); ?>" />
+					<input type="number" name="<?php echo esc_attr("wordpoints_points-{$slug}"); ?>" value="<?php echo esc_attr($points); ?>" autocomplete="off" />
+					<input type="checkbox" value="1" name="<?php echo esc_attr("wordpoints_points_set-{$slug}"); ?>" />
 					<?php // translators: Number of points. ?>
-					<span><?php echo esc_html( sprintf( __( '(current: %s)', 'wordpoints' ), $points ) ); ?></span>
+					<span><?php echo esc_html(sprintf(__('(current: %s)', 'wordpoints'), $points)); ?></span>
 				</td>
 			</tr>
 
 			<?php
 		}
 
-	} elseif ( defined( 'IS_PROFILE_PAGE' ) && IS_PROFILE_PAGE ) {
+	} elseif (defined('IS_PROFILE_PAGE') && IS_PROFILE_PAGE) {
 
 		/**
 		 * My points admin profile heading.
@@ -377,20 +377,20 @@ 
 		 *
 		 * @param string $heading The text for the heading.
 		 */
-		$heading = apply_filters( 'wordpoints_profile_points_heading', __( 'My Points', 'wordpoints' ) );
+		$heading = apply_filters('wordpoints_profile_points_heading', __('My Points', 'wordpoints'));
 
 		?>
 
 		</table>
 
-		<h2><?php echo esc_html( $heading ); ?></h2>
+		<h2><?php echo esc_html($heading); ?></h2>
 
 		<table>
 		<tbody>
-		<?php foreach ( wordpoints_get_points_types() as $slug => $type ) : ?>
+		<?php foreach (wordpoints_get_points_types() as $slug => $type) : ?>
 			<tr>
-				<th scope="row" style="text-align: left;"><?php echo esc_html( $type['name'] ); ?></th>
-				<td style="text-align: right;"><?php wordpoints_display_points( $user->ID, $slug, 'profile_page' ); ?></td>
+				<th scope="row" style="text-align: left;"><?php echo esc_html($type['name']); ?></th>
+				<td style="text-align: right;"><?php wordpoints_display_points($user->ID, $slug, 'profile_page'); ?></td>
 			</tr>
 		<?php endforeach; ?>
 		</tbody>
@@ -412,39 +412,39 @@ 
  *
  * @return void
  */
-function wordpoints_points_profile_options_update( $user_id ) {
+function wordpoints_points_profile_options_update($user_id) {
 
-	if ( ! current_user_can( 'set_wordpoints_points', $user_id ) ) {
+	if ( ! current_user_can('set_wordpoints_points', $user_id)) {
 		return;
 	}
 
 	if (
-		! isset( $_POST['wordpoints_points_set_nonce'], $_POST['wordpoints_set_reason'] )
-		|| ! wordpoints_verify_nonce( 'wordpoints_points_set_nonce', 'wordpoints_points_set_profile', null, 'post' )
+		! isset($_POST['wordpoints_points_set_nonce'], $_POST['wordpoints_set_reason'])
+		|| ! wordpoints_verify_nonce('wordpoints_points_set_nonce', 'wordpoints_points_set_profile', null, 'post')
 	) {
 		return;
 	}
 
-	foreach ( wordpoints_get_points_types() as $slug => $type ) {
+	foreach (wordpoints_get_points_types() as $slug => $type) {
 
 		if (
 			isset(
-				$_POST[ "wordpoints_points_set-{$slug}" ]
-				, $_POST[ "wordpoints_points-{$slug}" ]
-				, $_POST[ "wordpoints_points_old-{$slug}" ]
+				$_POST["wordpoints_points_set-{$slug}"]
+				, $_POST["wordpoints_points-{$slug}"]
+				, $_POST["wordpoints_points_old-{$slug}"]
 			)
-			&& false !== wordpoints_int( $_POST[ "wordpoints_points-{$slug}" ] )
-			&& false !== wordpoints_int( $_POST[ "wordpoints_points_old-{$slug}" ] )
+			&& false !== wordpoints_int($_POST["wordpoints_points-{$slug}"])
+			&& false !== wordpoints_int($_POST["wordpoints_points_old-{$slug}"])
 		) {
 
 			wordpoints_alter_points(
 				$user_id
-				, (int) $_POST[ "wordpoints_points-{$slug}" ] - (int) $_POST[ "wordpoints_points_old-{$slug}" ]
+				, (int) $_POST["wordpoints_points-{$slug}"] - (int) $_POST["wordpoints_points_old-{$slug}"]
 				, $slug
 				, 'profile_edit'
 				, array(
 					'user_id' => get_current_user_id(),
-					'reason' => sanitize_text_field( wp_unslash( $_POST['wordpoints_set_reason'] ) ),
+					'reason' => sanitize_text_field(wp_unslash($_POST['wordpoints_set_reason'])),
 				)
 			);
 		}
@@ -466,22 +466,22 @@ 
 		'selected'         => wordpoints_get_default_points_type(),
 		'id'               => 'default_points_type',
 		'name'             => 'default_points_type',
-		'show_option_none' => __( 'No default', 'wordpoints' ),
+		'show_option_none' => __('No default', 'wordpoints'),
 	);
 
 	?>
 
-	<h3><?php esc_html_e( 'Default Points Type', 'wordpoints' ); ?></h3>
-	<p><?php esc_html_e( 'You can optionally set one points type to be the default. The default points type will, for example, be used by shortcodes when no type is specified. This is also useful if you only have one type of points.', 'wordpoints' ); ?></p>
+	<h3><?php esc_html_e('Default Points Type', 'wordpoints'); ?></h3>
+	<p><?php esc_html_e('You can optionally set one points type to be the default. The default points type will, for example, be used by shortcodes when no type is specified. This is also useful if you only have one type of points.', 'wordpoints'); ?></p>
 	<table class="form-table">
 		<tbody>
 			<tr>
 				<th>
-					<label for="default_points_type"><?php esc_html_e( 'Default', 'wordpoints' ); ?></label>
+					<label for="default_points_type"><?php esc_html_e('Default', 'wordpoints'); ?></label>
 				</th>
 				<td>
-					<?php wordpoints_points_types_dropdown( $dropdown_args ); ?>
-					<?php wp_nonce_field( 'wordpoints_default_points_type', 'wordpoints_default_points_type_nonce' ); ?>
+					<?php wordpoints_points_types_dropdown($dropdown_args); ?>
+					<?php wp_nonce_field('wordpoints_default_points_type', 'wordpoints_default_points_type_nonce'); ?>
 				</td>
 			</tr>
 		</tbody>
@@ -500,19 +500,19 @@ 
 function wordpoints_points_admin_settings_save() {
 
 	if (
-		isset( $_POST['default_points_type'] )
-		&& wordpoints_verify_nonce( 'wordpoints_default_points_type_nonce', 'wordpoints_default_points_type', null, 'post' )
+		isset($_POST['default_points_type'])
+		&& wordpoints_verify_nonce('wordpoints_default_points_type_nonce', 'wordpoints_default_points_type', null, 'post')
 	) {
 
-		$points_type = sanitize_key( $_POST['default_points_type'] );
+		$points_type = sanitize_key($_POST['default_points_type']);
 
-		if ( '-1' === $points_type ) {
+		if ('-1' === $points_type) {
 
-			wordpoints_update_maybe_network_option( 'wordpoints_default_points_type', '' );
+			wordpoints_update_maybe_network_option('wordpoints_default_points_type', '');
 
-		} elseif ( wordpoints_is_points_type( $points_type ) ) {
+		} elseif (wordpoints_is_points_type($points_type)) {
 
-			wordpoints_update_maybe_network_option( 'wordpoints_default_points_type', $points_type );
+			wordpoints_update_maybe_network_option('wordpoints_default_points_type', $points_type);
 		}
 	}
 }
@@ -527,16 +527,16 @@ 
 function wordpoints_points_admin_notices() {
 
 	if (
-		( ! isset( $_GET['page'] ) || 'wordpoints_points_types' !== $_GET['page'] )
-		&& current_user_can( 'manage_wordpoints_points_types' )
+		( ! isset($_GET['page']) || 'wordpoints_points_types' !== $_GET['page'])
+		&& current_user_can('manage_wordpoints_points_types')
 		&& ! wordpoints_get_points_types()
 	) {
 
 		wordpoints_show_admin_message(
 			sprintf(
 				// translators: URL of Points Types admin screen.
-				__( 'Welcome to WordPoints! Get started by <a href="%s">creating a points type</a>.', 'wordpoints' )
-				, esc_url( self_admin_url( 'admin.php?page=wordpoints_points_types' ) )
+				__('Welcome to WordPoints! Get started by <a href="%s">creating a points type</a>.', 'wordpoints')
+				, esc_url(self_admin_url('admin.php?page=wordpoints_points_types'))
 			)
 			, 'info'
 		);

src/components/ranks/ranks.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
  */
 include_once WORDPOINTS_DIR . 'components/ranks/includes/deprecated.php';
 
-if ( wordpoints_component_is_active( 'points' ) ) {
+if (wordpoints_component_is_active('points')) {
 
 	/**
 	 * Ranks integration with the Points component.
@@ -49,7 +49,7 @@ 
 	include_once WORDPOINTS_DIR . 'components/ranks/includes/integration/points.php';
 }
 
-if ( is_admin() ) {
+if (is_admin()) {
 
 	/**
 	 * Administration-side rank code.

src/components/ranks/includes/filters.php

Spacing +3 added lines, -3 removed lines

@@ -7,10 +7,10 @@
  * @since 2.1.0
  */
 
-add_action( 'wordpoints_modules_loaded', 'WordPoints_Rank_Types::init' );
+add_action('wordpoints_modules_loaded', 'WordPoints_Rank_Types::init');
 
-add_action( 'wordpoints_ranks_register', 'wordpoints_register_core_ranks' );
+add_action('wordpoints_ranks_register', 'wordpoints_register_core_ranks');
 
-WordPoints_Shortcodes::register( 'wordpoints_user_rank', 'WordPoints_User_Rank_Shortcode' );
+WordPoints_Shortcodes::register('wordpoints_user_rank', 'WordPoints_User_Rank_Shortcode');
 
 // EOF

src/components/ranks/includes/shortcodes.php

Spacing +2 added lines, -2 removed lines

@@ -19,10 +19,10 @@
 	 */
 	protected function verify_atts() {
 
-		if ( isset( $this->pairs['rank_group'] ) && empty( $this->atts['rank_group'] ) ) {
+		if (isset($this->pairs['rank_group']) && empty($this->atts['rank_group'])) {
 
 			return sprintf(
-				__( 'The “%1$s” attribute of the %2$s shortcode must be a rank group slug.', 'wordpoints' )
+				__('The “%1$s” attribute of the %2$s shortcode must be a rank group slug.', 'wordpoints')
 				,'rank_group'
 				, "<code>[{$this->shortcode}]</code>"
 			);

src/components/ranks/includes/deprecated.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
  */
 function wordpoints_ranks_get_db_schema() {
 
-	_deprecated_function( __FUNCTION__, '2.0.0' );
+	_deprecated_function(__FUNCTION__, '2.0.0');
 
 	global $wpdb;
 

src/components/ranks/includes/rank-types/base.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 	 *
 	 * @return array|false The validated metadata or false if it should't be saved.
 	 */
-	public function validate_rank_meta( array $meta ) {
+	public function validate_rank_meta(array $meta) {
 		return $meta;
 	}
 
@@ -52,7 +52,7 @@ 
 	 *
 	 * @return bool Whether the user meets the requirements for this rank.
 	 */
-	protected function can_transition_user_rank( $user_id, $rank, array $args ) {
+	protected function can_transition_user_rank($user_id, $rank, array $args) {
 		return true;
 	}
 }

src/components/ranks/includes/class-wordpoints-rank.php

Spacing +30 added lines, -30 removed lines

@@ -75,17 +75,17 @@ 
 	 *
 	 * @param int|WordPoints_Rank $id The ID of a rank.
 	 */
-	public function __construct( $id ) {
+	public function __construct($id) {
 
-		if ( is_a( $id, __CLASS__ ) ) {
-			$this->init( $id->data );
+		if (is_a($id, __CLASS__)) {
+			$this->init($id->data);
 			return;
 		}
 
-		$id = wordpoints_int( $id );
+		$id = wordpoints_int($id);
 
-		if ( $id ) {
-			$this->init( self::get_data( $id ) );
+		if ($id) {
+			$this->init(self::get_data($id));
 		}
 	}
 
@@ -94,13 +94,13 @@ 
 	 *
 	 * @since 1.7.0
 	 */
-	public function __isset( $key ) {
+	public function __isset($key) {
 
-		if ( isset( $this->data->$key ) ) {
+		if (isset($this->data->$key)) {
 			return true;
 		}
 
-		return metadata_exists( 'wordpoints_rank', $this->ID, $key );
+		return metadata_exists('wordpoints_rank', $this->ID, $key);
 	}
 
 	/**
@@ -108,14 +108,14 @@ 
 	 *
 	 * @since 1.7.0
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( 'ID' === $key ) {
+		if ('ID' === $key) {
 			return $this->ID;
-		} elseif ( isset( $this->data->$key ) ) {
+		} elseif (isset($this->data->$key)) {
 			$value = $this->data->$key;
 		} else {
-			$value = wordpoints_get_rank_meta( $this->ID, $key, true );
+			$value = wordpoints_get_rank_meta($this->ID, $key, true);
 		}
 
 		return $value;
@@ -126,9 +126,9 @@ 
 	 *
 	 * @since 1.7.0
 	 */
-	public function __set( $key, $value ) {
+	public function __set($key, $value) {
 
-		if ( 'ID' !== $key && 'data' !== $key ) {
+		if ('ID' !== $key && 'data' !== $key) {
 
 			_doing_it_wrong(
 				__METHOD__
@@ -145,9 +145,9 @@ 
 	 *
 	 * @param object $data The data for a rank.
 	 */
-	public function init( $data ) {
+	public function init($data) {
 
-		if ( ! isset( $data->id ) ) {
+		if ( ! isset($data->id)) {
 			return;
 		}
 
@@ -164,7 +164,7 @@ 
 	 */
 	public function exists() {
 
-		return ! empty( $this->ID );
+		return ! empty($this->ID);
 	}
 
 	/**
@@ -181,23 +181,23 @@ 
 	 *
 	 * @return WordPoints_Rank|false The adjacent rank, or false.
 	 */
-	public function get_adjacent( $relative_position ) {
+	public function get_adjacent($relative_position) {
 
-		if ( 0 === wordpoints_int( $relative_position ) ) {
+		if (0 === wordpoints_int($relative_position)) {
 			return $this;
 		}
 
-		$group = WordPoints_Rank_Groups::get_group( $this->rank_group );
+		$group = WordPoints_Rank_Groups::get_group($this->rank_group);
 
-		$position = $group->get_rank_position( $this->ID );
+		$position = $group->get_rank_position($this->ID);
 
-		$adjacent_rank_id = $group->get_rank( $position + $relative_position );
+		$adjacent_rank_id = $group->get_rank($position + $relative_position);
 
-		if ( ! $adjacent_rank_id ) {
+		if ( ! $adjacent_rank_id) {
 			return false;
 		}
 
-		return new WordPoints_Rank( $adjacent_rank_id );
+		return new WordPoints_Rank($adjacent_rank_id);
 	}
 
 	//
@@ -213,11 +213,11 @@ 
 	 *
 	 * @return object|false The rank's data, or false if not found.
 	 */
-	public static function get_data( $id ) {
+	public static function get_data($id) {
 
-		$rank_data = wp_cache_get( $id, 'wordpoints_ranks' );
+		$rank_data = wp_cache_get($id, 'wordpoints_ranks');
 
-		if ( false !== $rank_data ) {
+		if (false !== $rank_data) {
 			return $rank_data;
 		}
 
@@ -234,11 +234,11 @@ 
 			)
 		);
 
-		if ( null === $rank_data ) {
+		if (null === $rank_data) {
 			return false;
 		}
 
-		wp_cache_add( $rank_data->id, $rank_data, 'wordpoints_ranks' );
+		wp_cache_add($rank_data->id, $rank_data, 'wordpoints_ranks');
 
 		return $rank_data;
 	}