WordPoints / wordpoints

src/admin/includes/functions.php

Spacing +187 added lines, -187 removed lines

@@ -16,16 +16,16 @@ 
  *
  * @param WordPoints_App $app The main WordPoints app.
  */
-function wordpoints_hooks_register_admin_apps( $app ) {
+function wordpoints_hooks_register_admin_apps($app) {
 
 	$apps = $app->sub_apps();
 
-	$apps->register( 'admin', 'WordPoints_App' );
+	$apps->register('admin', 'WordPoints_App');
 
 	/** @var WordPoints_App $admin */
-	$admin = $apps->get( 'admin' );
+	$admin = $apps->get('admin');
 
-	$admin->sub_apps()->register( 'screen', 'WordPoints_Admin_Screens' );
+	$admin->sub_apps()->register('screen', 'WordPoints_Admin_Screens');
 }
 
 /**
@@ -48,7 +48,7 @@ 
 	 * the modules screen should be the main one (the configure menu is only for the
 	 * network admin when network active).
 	 */
-	if ( is_wordpoints_network_active() && 'admin_menu' === current_filter() ) {
+	if (is_wordpoints_network_active() && 'admin_menu' === current_filter()) {
 		$slug = 'wordpoints_modules';
 	}
 
@@ -66,18 +66,18 @@ 
 function wordpoints_admin_menu() {
 
 	$main_menu  = wordpoints_get_main_admin_menu();
-	$wordpoints = __( 'WordPoints', 'wordpoints' );
+	$wordpoints = __('WordPoints', 'wordpoints');
 
 	/*
 	 * The settings page is always the main menu, except when the plugin is network
 	 * active on multisite. Then it is only the main menu when in the network admin.
 	 */
-	if ( 'wordpoints_configure' === $main_menu ) {
+	if ('wordpoints_configure' === $main_menu) {
 
 		// Main page.
 		add_menu_page(
 			$wordpoints
-			,esc_html( $wordpoints )
+			,esc_html($wordpoints)
 			,'manage_options'
 			,'wordpoints_configure'
 			,'wordpoints_admin_screen_configure'
@@ -86,8 +86,8 @@ 
 		// Settings page.
 		add_submenu_page(
 			'wordpoints_configure'
-			,__( 'WordPoints — Configure', 'wordpoints' )
-			,esc_html__( 'Configure', 'wordpoints' )
+			,__('WordPoints — Configure', 'wordpoints')
+			,esc_html__('Configure', 'wordpoints')
 			,'manage_options'
 			,'wordpoints_configure'
 			,'wordpoints_admin_screen_configure'
@@ -103,7 +103,7 @@ 
 		// Main page.
 		add_menu_page(
 			$wordpoints
-			,esc_html( $wordpoints )
+			,esc_html($wordpoints)
 			,'activate_wordpoints_modules'
 			,'wordpoints_modules'
 			,'wordpoints_admin_screen_modules'
@@ -114,8 +114,8 @@ 
 	// Modules page.
 	add_submenu_page(
 		$main_menu
-		,__( 'WordPoints — Modules', 'wordpoints' )
-		,esc_html__( 'Modules', 'wordpoints' )
+		,__('WordPoints — Modules', 'wordpoints')
+		,esc_html__('Modules', 'wordpoints')
 		,'activate_wordpoints_modules'
 		,'wordpoints_modules'
 		,'wordpoints_admin_screen_modules'
@@ -124,8 +124,8 @@ 
 	// Module install page.
 	add_submenu_page(
 		'_wordpoints_modules' // Fake menu.
-		,__( 'WordPoints — Install Modules', 'wordpoints' )
-		,esc_html__( 'Install Modules', 'wordpoints' )
+		,__('WordPoints — Install Modules', 'wordpoints')
+		,esc_html__('Install Modules', 'wordpoints')
 		,'install_wordpoints_modules'
 		,'wordpoints_install_modules'
 		,'wordpoints_admin_screen_install_modules'
@@ -253,15 +253,15 @@ 
 	wp_register_style(
 		'wordpoints-hooks-admin'
 		, "{$assets_url}/css/hooks{$suffix}.css"
-		, array( 'dashicons', 'wp-jquery-ui-dialog' )
+		, array('dashicons', 'wp-jquery-ui-dialog')
 		, WORDPOINTS_VERSION
 	);
 
 	$styles = wp_styles();
-	$styles->add_data( 'wordpoints-hooks-admin', 'rtl', 'replace' );
+	$styles->add_data('wordpoints-hooks-admin', 'rtl', 'replace');
 
-	if ( $suffix ) {
-		$styles->add_data( 'wordpoints-hooks-admin', 'suffix', $suffix );
+	if ($suffix) {
+		$styles->add_data('wordpoints-hooks-admin', 'suffix', $suffix);
 	}
 
 	// JS
@@ -269,21 +269,21 @@ 
 	wp_register_script(
 		'wordpoints-admin-dismiss-notice'
 		, "{$assets_url}/js/dismiss-notice{$suffix}.js"
-		, array( 'jquery', 'wp-util' )
+		, array('jquery', 'wp-util')
 		, WORDPOINTS_VERSION
 	);
 
 	wp_register_script(
 		'wordpoints-hooks-models'
 		, "{$assets_url}/js/hooks/models{$manifested_suffix}.js"
-		, array( 'backbone', 'jquery-ui-dialog', 'wp-util' )
+		, array('backbone', 'jquery-ui-dialog', 'wp-util')
 		, WORDPOINTS_VERSION
 	);
 
 	wp_register_script(
 		'wordpoints-hooks-views'
 		, "{$assets_url}/js/hooks/views{$manifested_suffix}.js"
-		, array( 'wordpoints-hooks-models' )
+		, array('wordpoints-hooks-models')
 		, WORDPOINTS_VERSION
 	);
 
@@ -291,19 +291,19 @@ 
 		'wordpoints-hooks-views'
 		, 'WordPointsHooksAdminL10n'
 		, array(
-			'unexpectedError' => __( 'There was an unexpected error. Try reloading the page.', 'wordpoints' ),
-			'changesSaved'    => __( 'Your changes have been saved.', 'wordpoints' ),
+			'unexpectedError' => __('There was an unexpected error. Try reloading the page.', 'wordpoints'),
+			'changesSaved'    => __('Your changes have been saved.', 'wordpoints'),
 			// translators: Form field name.
-			'emptyField'      => sprintf( __( '%s cannot be empty.', 'wordpoints' ), '{{ data.label }}' ),
-			'confirmDelete'   => __( 'Are you sure that you want to delete this reaction? This action cannot be undone.', 'wordpoints' ),
-			'confirmTitle'    => __( 'Are you sure?', 'wordpoints' ),
-			'deleteText'      => __( 'Delete', 'wordpoints' ),
-			'cancelText'      => __( 'Cancel', 'wordpoints' ),
+			'emptyField'      => sprintf(__('%s cannot be empty.', 'wordpoints'), '{{ data.label }}'),
+			'confirmDelete'   => __('Are you sure that you want to delete this reaction? This action cannot be undone.', 'wordpoints'),
+			'confirmTitle'    => __('Are you sure?', 'wordpoints'),
+			'deleteText'      => __('Delete', 'wordpoints'),
+			'cancelText'      => __('Cancel', 'wordpoints'),
 			'separator'       => is_rtl() ? ' « ' : ' » ',
-			'target_label'    => __( 'Target', 'wordpoints' ),
+			'target_label'    => __('Target', 'wordpoints'),
 			// translators: Form field.
-			'cannotBeChanged' => __( '(cannot be changed)', 'wordpoints' ),
-			'fieldsInvalid'   => __( 'Error: the values of some fields are invalid. Please correct these and then try again.', 'wordpoints' ),
+			'cannotBeChanged' => __('(cannot be changed)', 'wordpoints'),
+			'fieldsInvalid'   => __('Error: the values of some fields are invalid. Please correct these and then try again.', 'wordpoints'),
 		)
 	);
 
@@ -315,10 +315,10 @@ 
 			<div class="view">
 				<div class="title"></div>
 				<button type="button" class="edit button-secondary">
-					' . esc_html__( 'Edit', 'wordpoints' ) . '
+					' . esc_html__('Edit', 'wordpoints') . '
 				</button>
 				<button type="button" class="close button-secondary">
-					' . esc_html__( 'Close', 'wordpoints' ) . '
+					' . esc_html__('Close', 'wordpoints') . '
 				</button>
 			</div>
 			<div class="form">
@@ -337,16 +337,16 @@ 
 						</div>
 						<div class="action-buttons">
 							<button type="button" class="save button-primary" disabled>
-								' . esc_html__( 'Save', 'wordpoints' ) . '
+								' . esc_html__('Save', 'wordpoints') . '
 							</button>
 							<button type="button" class="cancel button-secondary">
-								' . esc_html__( 'Cancel', 'wordpoints' ) . '
+								' . esc_html__('Cancel', 'wordpoints') . '
 							</button>
 							<button type="button" class="close button-secondary">
-								' . esc_html__( 'Close', 'wordpoints' ) . '
+								' . esc_html__('Close', 'wordpoints') . '
 							</button>
 							<button type="button" class="delete button-secondary">
-								' . esc_html__( 'Delete', 'wordpoints' ) . '
+								' . esc_html__('Delete', 'wordpoints') . '
 							</button>
 						</div>
 					</div>
@@ -395,7 +395,7 @@ 
 	wp_register_script(
 		'wordpoints-hooks-extension-conditions'
 		, "{$assets_url}/js/hooks/extensions/conditions{$manifested_suffix}.js"
-		, array( 'wordpoints-hooks-views' )
+		, array('wordpoints-hooks-views')
 		, WORDPOINTS_VERSION
 	);
 
@@ -405,25 +405,25 @@ 
 		, '
 			<script type="text/template" id="tmpl-wordpoints-hook-condition-groups">
 				<div class="conditions-title section-title">
-					<h4>' . esc_html__( 'Conditions', 'wordpoints' ) . '</h4>
+					<h4>' . esc_html__('Conditions', 'wordpoints') . '</h4>
 					<button type="button" class="add-new button-secondary button-link">
-						<span class="screen-reader-text">' . esc_html__( 'Add New Condition', 'wordpoints' ) . '</span>
+						<span class="screen-reader-text">' . esc_html__('Add New Condition', 'wordpoints') . '</span>
 						<span class="dashicons dashicons-plus"></span>
 					</button>
 				</div>
 				<div class="add-condition-form hidden">
 					<div class="no-conditions hidden">
-						' . esc_html__( 'No conditions available.', 'wordpoints' ) . '
+						' . esc_html__('No conditions available.', 'wordpoints') . '
 					</div>
 					<div class="condition-selectors">
 						<div class="arg-selectors"></div>
 						<div class="condition-selector"></div>
 					</div>
-					<button type="button" class="confirm-add-new button-secondary" disabled aria-label="' . esc_attr__( 'Add Condition', 'wordpoints' ) . '">
-						' . esc_html_x( 'Add', 'reaction condition', 'wordpoints' ) . '
+					<button type="button" class="confirm-add-new button-secondary" disabled aria-label="' . esc_attr__('Add Condition', 'wordpoints') . '">
+						' . esc_html_x('Add', 'reaction condition', 'wordpoints') . '
 					</button>
-					<button type="button" class="cancel-add-new button-secondary" aria-label="' . esc_attr__( 'Cancel Adding New Condition', 'wordpoints' ) . '">
-						' . esc_html_x( 'Cancel', 'reaction condition', 'wordpoints' ) . '
+					<button type="button" class="cancel-add-new button-secondary" aria-label="' . esc_attr__('Cancel Adding New Condition', 'wordpoints') . '">
+						' . esc_html_x('Cancel', 'reaction condition', 'wordpoints') . '
 					</button>
 				</div>
 				<div class="condition-groups section-content"></div>
@@ -437,7 +437,7 @@ 
 				<div class="condition-controls">
 					<div class="condition-title"></div>
 					<button type="button" class="delete button-secondary button-link">
-						<span class="screen-reader-text">' . esc_html__( 'Remove Condition', 'wordpoints' ) . '</span>
+						<span class="screen-reader-text">' . esc_html__('Remove Condition', 'wordpoints') . '</span>
 						<span class="dashicons dashicons-no"></span>
 					</button>
 				</div>
@@ -456,7 +456,7 @@ 
 	wp_register_script(
 		'wordpoints-hooks-extension-periods'
 		, "{$assets_url}/js/hooks/extensions/periods{$manifested_suffix}.js"
-		, array( 'wordpoints-hooks-views' )
+		, array('wordpoints-hooks-views')
 		, WORDPOINTS_VERSION
 	);
 
@@ -467,7 +467,7 @@ 
 			<script type="text/template" id="tmpl-wordpoints-hook-periods">
 				<div class="wordpoints-hook-periods">
 					<div class="periods-title section-title">
-						<h4>' . esc_html__( 'Rate Limit', 'wordpoints' ) . '</h4>
+						<h4>' . esc_html__('Rate Limit', 'wordpoints') . '</h4>
 					</div>
 					<div class="periods section-content"></div>
 				</div>
@@ -480,11 +480,11 @@ 
 						<p class="description description-thin">
 							<legend>{{ data.length_in_units_label }}</legend>
 							<label>
-								<span class="screen-reader-text">' . esc_html__( 'Time Units:', 'wordpoints' ) . '</span>
+								<span class="screen-reader-text">' . esc_html__('Time Units:', 'wordpoints') . '</span>
 								<select class="widefat wordpoints-hook-period-sync wordpoints-hook-period-units"></select>
 							</label>
 							<label>
-								<span class="screen-reader-text">' . esc_html__( 'Number:', 'wordpoints' ) . '</span>
+								<span class="screen-reader-text">' . esc_html__('Number:', 'wordpoints') . '</span>
 								<input type="number" value="{{ data.length_in_units }}" class="widefat wordpoints-hook-period-sync wordpoints-hook-period-length-in-units" />
 							</label>
 						</p>
@@ -505,12 +505,12 @@ 
 	$hooks = wordpoints_hooks();
 
 	$extensions_data = wordpoints_hooks_ui_get_script_data_from_objects(
-		$hooks->get_sub_app( 'extensions' )->get_all()
+		$hooks->get_sub_app('extensions')->get_all()
 		, 'extension'
 	);
 
 	$reactor_data = wordpoints_hooks_ui_get_script_data_from_objects(
-		$hooks->get_sub_app( 'reactors' )->get_all()
+		$hooks->get_sub_app('reactors')->get_all()
 		, 'reactor'
 	);
 
@@ -536,7 +536,7 @@ 
 	 *
 	 * @param array $data The data.
 	 */
-	$data = apply_filters( 'wordpoints_hooks_ui_data', $data );
+	$data = apply_filters('wordpoints_hooks_ui_data', $data);
 
 	wp_localize_script(
 		'wordpoints-hooks-models'
@@ -556,18 +556,18 @@ 
  *
  * @return array The data extracted from the objects.
  */
-function wordpoints_hooks_ui_get_script_data_from_objects( $objects, $type ) {
+function wordpoints_hooks_ui_get_script_data_from_objects($objects, $type) {
 
 	$data = array();
 
-	foreach ( $objects as $slug => $object ) {
+	foreach ($objects as $slug => $object) {
 
-		if ( $object instanceof WordPoints_Hook_UI_Script_Data_ProviderI ) {
-			$data[ $slug ] = $object->get_ui_script_data();
+		if ($object instanceof WordPoints_Hook_UI_Script_Data_ProviderI) {
+			$data[$slug] = $object->get_ui_script_data();
 		}
 
-		if ( wp_script_is( "wordpoints-hooks-{$type}-{$slug}", 'registered' ) ) {
-			wp_enqueue_script( "wordpoints-hooks-{$type}-{$slug}" );
+		if (wp_script_is("wordpoints-hooks-{$type}-{$slug}", 'registered')) {
+			wp_enqueue_script("wordpoints-hooks-{$type}-{$slug}");
 		}
 	}
 
@@ -588,31 +588,31 @@ 
 	$entities_data = array();
 
 	/** @var WordPoints_Class_Registry_Children $entity_children */
-	$entity_children = $entities->get_sub_app( 'children' );
+	$entity_children = $entities->get_sub_app('children');
 
 	/** @var WordPoints_Entity $entity */
-	foreach ( $entities->get_all() as $slug => $entity ) {
+	foreach ($entities->get_all() as $slug => $entity) {
 
 		$child_data = array();
 
 		/** @var WordPoints_EntityishI $child */
-		foreach ( $entity_children->get_children( $slug ) as $child_slug => $child ) {
+		foreach ($entity_children->get_children($slug) as $child_slug => $child) {
 
-			$child_data[ $child_slug ] = array(
+			$child_data[$child_slug] = array(
 				'slug'  => $child_slug,
 				'title' => $child->get_title(),
 			);
 
-			if ( $child instanceof WordPoints_Entity_Attr ) {
+			if ($child instanceof WordPoints_Entity_Attr) {
 
-				$child_data[ $child_slug ]['_type']     = 'attr';
-				$child_data[ $child_slug ]['data_type'] = $child->get_data_type();
+				$child_data[$child_slug]['_type']     = 'attr';
+				$child_data[$child_slug]['data_type'] = $child->get_data_type();
 
-			} elseif ( $child instanceof WordPoints_Entity_Relationship ) {
+			} elseif ($child instanceof WordPoints_Entity_Relationship) {
 
-				$child_data[ $child_slug ]['_type']     = 'relationship';
-				$child_data[ $child_slug ]['primary']   = $child->get_primary_entity_slug();
-				$child_data[ $child_slug ]['secondary'] = $child->get_related_entity_slug();
+				$child_data[$child_slug]['_type']     = 'relationship';
+				$child_data[$child_slug]['primary']   = $child->get_primary_entity_slug();
+				$child_data[$child_slug]['secondary'] = $child->get_related_entity_slug();
 			}
 
 			/**
@@ -623,14 +623,14 @@ 
 			 * @param array                $data  The data for the entity child.
 			 * @param WordPoints_Entityish $child The child's object.
 			 */
-			$child_data[ $child_slug ] = apply_filters(
+			$child_data[$child_slug] = apply_filters(
 				'wordpoints_hooks_ui_data_entity_child'
-				, $child_data[ $child_slug ]
+				, $child_data[$child_slug]
 				, $child
 			);
 		}
 
-		$entities_data[ $slug ] = array(
+		$entities_data[$slug] = array(
 			'slug'     => $slug,
 			'title'    => $entity->get_title(),
 			'children' => $child_data,
@@ -638,12 +638,12 @@ 
 			'_type'    => 'entity',
 		);
 
-		if ( $entity instanceof WordPoints_Entity_EnumerableI ) {
+		if ($entity instanceof WordPoints_Entity_EnumerableI) {
 
 			$values = array();
 
-			foreach ( $entity->get_enumerated_values() as $value ) {
-				if ( $entity->set_the_value( $value ) ) {
+			foreach ($entity->get_enumerated_values() as $value) {
+				if ($entity->set_the_value($value)) {
 					$values[] = array(
 						'value' => $entity->get_the_id(),
 						'label' => $entity->get_the_human_id(),
@@ -651,7 +651,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -660,9 +660,9 @@ 
 		 * @param array             $data   The data for the entity.
 		 * @param WordPoints_Entity $entity The entity object.
 		 */
-		$entities_data[ $slug ] = apply_filters(
+		$entities_data[$slug] = apply_filters(
 			'wordpoints_hooks_ui_data_entity'
-			, $entities_data[ $slug ]
+			, $entities_data[$slug]
 			, $entity
 		);
 
@@ -683,19 +683,19 @@ 
 	// We want a list of the action types for each event. We can start with this list
 	// but it is indexed by action slug and then action type and then event slug, so
 	// we ned to do some processing.
-	$event_index = wordpoints_hooks()->get_sub_app( 'router' )->get_event_index();
+	$event_index = wordpoints_hooks()->get_sub_app('router')->get_event_index();
 
 	// We don't care about the action slugs, so first we get rid of that bottom level
 	// of the array.
-	$event_index = call_user_func_array( 'array_merge_recursive', $event_index );
+	$event_index = call_user_func_array('array_merge_recursive', $event_index);
 
 	$event_action_types = array();
 
 	// This leaves us the event indexed by action type. But we actually need to flip
 	// this, so that we have the action types indexed by event slug.
-	foreach ( $event_index as $action_type => $events ) {
-		foreach ( $events as $event => $unused ) {
-			$event_action_types[ $event ][ $action_type ] = true;
+	foreach ($event_index as $action_type => $events) {
+		foreach ($events as $event => $unused) {
+			$event_action_types[$event][$action_type] = true;
 		}
 	}
 
@@ -718,13 +718,13 @@ 
  *
  * @return string The HTML with templates appended.
  */
-function wordpoints_script_templates_filter( $html, $handle ) {
+function wordpoints_script_templates_filter($html, $handle) {
 
 	global $wp_scripts;
 
-	$templates = $wp_scripts->get_data( $handle, 'wordpoints-templates' );
+	$templates = $wp_scripts->get_data($handle, 'wordpoints-templates');
 
-	if ( $templates ) {
+	if ($templates) {
 		$html .= $templates;
 	}
 
@@ -742,9 +742,9 @@ 
  * @param string $message The text for the error message.
  * @param array  $args    Other optional arguments.
  */
-function wordpoints_show_admin_error( $message, array $args = array() ) {
+function wordpoints_show_admin_error($message, array $args = array()) {
 
-	wordpoints_show_admin_message( $message, 'error', $args );
+	wordpoints_show_admin_message($message, 'error', $args);
 }
 
 /**
@@ -774,16 +774,16 @@ 
  *                                  Required when $dismissible is true.
  * }
  */
-function wordpoints_show_admin_message( $message, $type = 'success', array $args = array() ) {
+function wordpoints_show_admin_message($message, $type = 'success', array $args = array()) {
 
 	$defaults = array(
 		'dismissible' => false,
 		'option'      => '',
 	);
 
-	$args = array_merge( $defaults, $args );
+	$args = array_merge($defaults, $args);
 
-	if ( isset( $args['dismissable'] ) ) {
+	if (isset($args['dismissable'])) {
 
 		$args['dismissible'] = $args['dismissable'];
 
@@ -794,7 +794,7 @@ 
 		);
 	}
 
-	if ( 'updated' === $type ) {
+	if ('updated' === $type) {
 
 		$type = 'success';
 
@@ -805,27 +805,27 @@ 
 		);
 	}
 
-	if ( $args['dismissible'] && $args['option'] ) {
-		wp_enqueue_script( 'wordpoints-admin-dismiss-notice' );
+	if ($args['dismissible'] && $args['option']) {
+		wp_enqueue_script('wordpoints-admin-dismiss-notice');
 	}
 
 	?>
 
 	<div
-		class="notice notice-<?php echo sanitize_html_class( $type, 'success' ); ?><?php echo ( $args['dismissible'] ) ? ' is-dismissible' : ''; ?>"
-		<?php if ( $args['dismissible'] && $args['option'] ) : ?>
-			data-nonce="<?php echo esc_attr( wp_create_nonce( "wordpoints_dismiss_notice-{$args['option']}" ) ); ?>"
-			data-option="<?php echo esc_attr( $args['option'] ); ?>"
+		class="notice notice-<?php echo sanitize_html_class($type, 'success'); ?><?php echo ($args['dismissible']) ? ' is-dismissible' : ''; ?>"
+		<?php if ($args['dismissible'] && $args['option']) : ?>
+			data-nonce="<?php echo esc_attr(wp_create_nonce("wordpoints_dismiss_notice-{$args['option']}")); ?>"
+			data-option="<?php echo esc_attr($args['option']); ?>"
 		<?php endif; ?>
 		>
 		<p>
-			<?php echo wp_kses( $message, 'wordpoints_admin_message' ); ?>
+			<?php echo wp_kses($message, 'wordpoints_admin_message'); ?>
 		</p>
-		<?php if ( $args['dismissible'] && $args['option'] ) : ?>
+		<?php if ($args['dismissible'] && $args['option']) : ?>
 			<form method="post" class="wordpoints-notice-dismiss-form" style="padding-bottom: 5px;">
-				<input type="hidden" name="wordpoints_notice" value="<?php echo esc_html( $args['option'] ); ?>" />
-				<?php wp_nonce_field( "wordpoints_dismiss_notice-{$args['option']}" ); ?>
-				<?php submit_button( __( 'Hide This Notice', 'wordpoints' ), 'secondary', 'wordpoints_dismiss_notice', false ); ?>
+				<input type="hidden" name="wordpoints_notice" value="<?php echo esc_html($args['option']); ?>" />
+				<?php wp_nonce_field("wordpoints_dismiss_notice-{$args['option']}"); ?>
+				<?php submit_button(__('Hide This Notice', 'wordpoints'), 'secondary', 'wordpoints_dismiss_notice', false); ?>
 			</form>
 		<?php endif; ?>
 	</div>
@@ -843,19 +843,19 @@ 
  *
  * @return string
  */
-function wordpoints_admin_get_current_tab( array $tabs = null ) {
+function wordpoints_admin_get_current_tab(array $tabs = null) {
 
 	$tab = '';
 
-	if ( isset( $_GET['tab'] ) ) {
+	if (isset($_GET['tab'])) {
 
-		$tab = sanitize_key( $_GET['tab'] );
+		$tab = sanitize_key($_GET['tab']);
 	}
 
-	if ( isset( $tabs ) && ! isset( $tabs[ $tab ] ) ) {
+	if (isset($tabs) && ! isset($tabs[$tab])) {
 
-		reset( $tabs );
-		$tab = key( $tabs );
+		reset($tabs);
+		$tab = key($tabs);
 	}
 
 	return $tab;
@@ -872,29 +872,29 @@ 
  * @param bool     $show_heading Whether to show an <h1> element using the current
  *                               tab. Default is true.
  */
-function wordpoints_admin_show_tabs( $tabs, $show_heading = true ) {
+function wordpoints_admin_show_tabs($tabs, $show_heading = true) {
 
-	$current = wordpoints_admin_get_current_tab( $tabs );
+	$current = wordpoints_admin_get_current_tab($tabs);
 
-	if ( $show_heading ) {
+	if ($show_heading) {
 
 		// translators: Current tab name.
-		echo '<h1>', esc_html( sprintf( __( 'WordPoints — %s', 'wordpoints' ), $tabs[ $current ] ) ), '</h1>';
+		echo '<h1>', esc_html(sprintf(__('WordPoints — %s', 'wordpoints'), $tabs[$current])), '</h1>';
 	}
 
 	echo '<h2 class="nav-tab-wrapper">';
 
 	$page = '';
 
-	if ( isset( $_GET['page'] ) ) {
-		$page = sanitize_key( $_GET['page'] );
+	if (isset($_GET['page'])) {
+		$page = sanitize_key($_GET['page']);
 	}
 
-	foreach ( $tabs as $tab => $name ) {
+	foreach ($tabs as $tab => $name) {
 
-		$class = ( $tab === $current ) ? ' nav-tab-active' : '';
+		$class = ($tab === $current) ? ' nav-tab-active' : '';
 
-		echo '<a class="nav-tab ', sanitize_html_class( $class ), '" href="?page=', rawurlencode( $page ), '&amp;tab=', rawurlencode( $tab ), '">', esc_html( $name ), '</a>';
+		echo '<a class="nav-tab ', sanitize_html_class($class), '" href="?page=', rawurlencode($page), '&amp;tab=', rawurlencode($tab), '">', esc_html($name), '</a>';
 	}
 
 	echo '</h2>';
@@ -918,12 +918,12 @@ 
 	</style>
 
 	<div class="upload-plugin wordpoints-upload-module">
-		<p class="install-help"><?php esc_html_e( 'If you have a module in a .zip format, you may install it by uploading it here.', 'wordpoints' ); ?></p>
-		<form method="post" enctype="multipart/form-data" class="wp-upload-form" action="<?php echo esc_url( self_admin_url( 'update.php?action=upload-wordpoints-module' ) ); ?>">
-			<?php wp_nonce_field( 'wordpoints-module-upload' ); ?>
-			<label class="screen-reader-text" for="modulezip"><?php esc_html_e( 'Module zip file', 'wordpoints' ); ?></label>
+		<p class="install-help"><?php esc_html_e('If you have a module in a .zip format, you may install it by uploading it here.', 'wordpoints'); ?></p>
+		<form method="post" enctype="multipart/form-data" class="wp-upload-form" action="<?php echo esc_url(self_admin_url('update.php?action=upload-wordpoints-module')); ?>">
+			<?php wp_nonce_field('wordpoints-module-upload'); ?>
+			<label class="screen-reader-text" for="modulezip"><?php esc_html_e('Module zip file', 'wordpoints'); ?></label>
 			<input type="file" id="modulezip" name="modulezip" />
-			<?php submit_button( __( 'Install Now', 'wordpoints' ), 'button', 'install-module-submit', false ); ?>
+			<?php submit_button(__('Install Now', 'wordpoints'), 'button', 'install-module-submit', false); ?>
 		</form>
 	</div>
 
@@ -941,15 +941,15 @@ 
 
 	global $title, $parent_file, $submenu_file;
 
-	if ( ! current_user_can( 'install_wordpoints_modules' ) ) {
-		wp_die( esc_html__( 'Sorry, you are not allowed to install WordPoints modules on this site.', 'wordpoints' ), '', array( 'response' => 403 ) );
+	if ( ! current_user_can('install_wordpoints_modules')) {
+		wp_die(esc_html__('Sorry, you are not allowed to install WordPoints modules on this site.', 'wordpoints'), '', array('response' => 403));
 	}
 
-	check_admin_referer( 'wordpoints-module-upload' );
+	check_admin_referer('wordpoints-module-upload');
 
-	$file_upload = new File_Upload_Upgrader( 'modulezip', 'package' );
+	$file_upload = new File_Upload_Upgrader('modulezip', 'package');
 
-	$title = esc_html__( 'Upload WordPoints Module', 'wordpoints' );
+	$title = esc_html__('Upload WordPoints Module', 'wordpoints');
 	$parent_file  = 'admin.php';
 	$submenu_file = 'admin.php';
 
@@ -959,17 +959,17 @@ 
 		new WordPoints_Module_Installer_Skin(
 			array(
 				// translators: File name.
-				'title' => sprintf( esc_html__( 'Installing Module from uploaded file: %s', 'wordpoints' ), esc_html( basename( $file_upload->filename ) ) ),
+				'title' => sprintf(esc_html__('Installing Module from uploaded file: %s', 'wordpoints'), esc_html(basename($file_upload->filename))),
 				'nonce' => 'wordpoints-module-upload',
-				'url'   => add_query_arg( array( 'package' => $file_upload->id ), self_admin_url( 'update.php?action=upload-wordpoints-module' ) ),
+				'url'   => add_query_arg(array('package' => $file_upload->id), self_admin_url('update.php?action=upload-wordpoints-module')),
 				'type'  => 'upload',
 			)
 		)
 	);
 
-	$result = $upgrader->install( $file_upload->package );
+	$result = $upgrader->install($file_upload->package);
 
-	if ( $result || is_wp_error( $result ) ) {
+	if ($result || is_wp_error($result)) {
 		$file_upload->cleanup();
 	}
 
@@ -993,11 +993,11 @@ 
  *
  * @return string|WP_Error The filtered module source.
  */
-function wordpoints_plugin_upload_error_filter( $source ) {
+function wordpoints_plugin_upload_error_filter($source) {
 
 	static $_source;
 
-	if ( ! isset( $_source ) ) {
+	if ( ! isset($_source)) {
 
 		$_source = $source;
 
@@ -1006,33 +1006,33 @@ 
 		global $wp_filesystem;
 
 		if (
-			! is_wp_error( $_source )
-			&& is_wp_error( $source )
+			! is_wp_error($_source)
+			&& is_wp_error($source)
 			&& 'incompatible_archive_no_plugins' === $source->get_error_code()
 		) {
 
 			$working_directory = str_replace(
 				$wp_filesystem->wp_content_dir()
-				, trailingslashit( WP_CONTENT_DIR )
+				, trailingslashit(WP_CONTENT_DIR)
 				, $_source
 			);
 
-			if ( is_dir( $working_directory ) ) {
+			if (is_dir($working_directory)) {
 
-				$files = glob( $working_directory . '*.php' );
+				$files = glob($working_directory . '*.php');
 
-				if ( is_array( $files ) ) {
+				if (is_array($files)) {
 
 					// Check if the folder contains a module.
-					foreach ( $files as $file ) {
+					foreach ($files as $file) {
 
-						$info = wordpoints_get_module_data( $file, false, false );
+						$info = wordpoints_get_module_data($file, false, false);
 
-						if ( ! empty( $info['name'] ) ) {
+						if ( ! empty($info['name'])) {
 							$source = new WP_Error(
 								'wordpoints_module_archive_not_plugin'
 								, $source->get_error_message()
-								, __( 'This appears to be a WordPoints module archive. Try installing it on the WordPoints module install screen instead.', 'wordpoints' )
+								, __('This appears to be a WordPoints module archive. Try installing it on the WordPoints module install screen instead.', 'wordpoints')
 							);
 
 							break;
@@ -1042,7 +1042,7 @@ 
 			}
 		}
 
-		unset( $_source );
+		unset($_source);
 
 	} // End if ( ! isset( $_source ) ) else.
 
@@ -1062,16 +1062,16 @@ 
 
 	<div style="height: 120px;border: none;padding: 1px 12px;background-color: #fff;border-left: 4px solid rgb(122, 208, 58);box-shadow: 0 1px 1px 0 rgba(0, 0, 0, 0.1);margin-top: 50px;">
 		<div style="width:48%;float:left;">
-			<h3><?php esc_html_e( 'Like this plugin?', 'wordpoints' ); ?></h3>
+			<h3><?php esc_html_e('Like this plugin?', 'wordpoints'); ?></h3>
 			<?php // translators: URL for leaving a review of WordPoints on WordPress.org. ?>
-			<p><?php echo wp_kses( sprintf( __( 'If you think WordPoints is great, let everyone know by giving it a <a href="%s">5 star rating</a>.', 'wordpoints' ), 'https://wordpress.org/support/view/plugin-reviews/wordpoints?rate=5#postform' ), array( 'a' => array( 'href' => true ) ) ); ?></p>
-			<p><?php esc_html_e( 'If you don&#8217;t think this plugin deserves 5 stars, please let us know how we can improve it.', 'wordpoints' ); ?></p>
+			<p><?php echo wp_kses(sprintf(__('If you think WordPoints is great, let everyone know by giving it a <a href="%s">5 star rating</a>.', 'wordpoints'), 'https://wordpress.org/support/view/plugin-reviews/wordpoints?rate=5#postform'), array('a' => array('href' => true))); ?></p>
+			<p><?php esc_html_e('If you don&#8217;t think this plugin deserves 5 stars, please let us know how we can improve it.', 'wordpoints'); ?></p>
 		</div>
 		<div style="width:48%;float:left;">
-			<h3><?php esc_html_e( 'Need help?', 'wordpoints' ); ?></h3>
+			<h3><?php esc_html_e('Need help?', 'wordpoints'); ?></h3>
 			<?php // translators: URL of WordPoints plugin support forums WordPress.org. ?>
-			<p><?php echo wp_kses( sprintf( __( 'Post your feature request or support question in the <a href="%s">support forums</a>', 'wordpoints' ), 'https://wordpress.org/support/plugin/wordpoints' ), array( 'a' => array( 'href' => true ) ) ); ?></p>
-			<p><em><?php esc_html_e( 'Thank you for using WordPoints!', 'wordpoints' ); ?></em></p>
+			<p><?php echo wp_kses(sprintf(__('Post your feature request or support question in the <a href="%s">support forums</a>', 'wordpoints'), 'https://wordpress.org/support/plugin/wordpoints'), array('a' => array('href' => true))); ?></p>
+			<p><em><?php esc_html_e('Thank you for using WordPoints!', 'wordpoints'); ?></em></p>
 		</div>
 	</div>
 
@@ -1089,19 +1089,19 @@ 
 
 	wordpoints_delete_admin_notice_option();
 
-	if ( current_user_can( 'activate_wordpoints_modules' ) ) {
+	if (current_user_can('activate_wordpoints_modules')) {
 
-		if ( is_network_admin() ) {
+		if (is_network_admin()) {
 
-			$deactivated_modules = get_site_option( 'wordpoints_breaking_deactivated_modules' );
+			$deactivated_modules = get_site_option('wordpoints_breaking_deactivated_modules');
 
-			if ( is_array( $deactivated_modules ) ) {
+			if (is_array($deactivated_modules)) {
 				wordpoints_show_admin_error(
 					sprintf(
 						// translators: 1. Plugin version; 2. List of modules.
-						__( 'WordPoints has deactivated the following modules because of incompatibilities with WordPoints %1$s: %2$s', 'wordpoints' )
+						__('WordPoints has deactivated the following modules because of incompatibilities with WordPoints %1$s: %2$s', 'wordpoints')
 						, WORDPOINTS_VERSION
-						, implode( ', ', $deactivated_modules )
+						, implode(', ', $deactivated_modules)
 					)
 					, array(
 						'dismissible' => true,
@@ -1110,15 +1110,15 @@ 
 				);
 			}
 
-			$incompatible_modules = get_site_option( 'wordpoints_incompatible_modules' );
+			$incompatible_modules = get_site_option('wordpoints_incompatible_modules');
 
-			if ( is_array( $incompatible_modules ) ) {
+			if (is_array($incompatible_modules)) {
 				wordpoints_show_admin_error(
 					sprintf(
 						// translators: 1. Plugin version; 2. List of modules.
-						__( 'WordPoints has deactivated the following network-active modules because of incompatibilities with WordPoints %1$s: %2$s', 'wordpoints' )
+						__('WordPoints has deactivated the following network-active modules because of incompatibilities with WordPoints %1$s: %2$s', 'wordpoints')
 						, WORDPOINTS_VERSION
-						, implode( ', ', $incompatible_modules )
+						, implode(', ', $incompatible_modules)
 					)
 					, array(
 						'dismissible' => true,
@@ -1129,15 +1129,15 @@ 
 
 		} else {
 
-			$incompatible_modules = get_option( 'wordpoints_incompatible_modules' );
+			$incompatible_modules = get_option('wordpoints_incompatible_modules');
 
-			if ( is_array( $incompatible_modules ) ) {
+			if (is_array($incompatible_modules)) {
 				wordpoints_show_admin_error(
 					sprintf(
 						// translators: 1. Plugin version; 2. List of modules.
-						__( 'WordPoints has deactivated the following modules on this site because of incompatibilities with WordPoints %1$s: %2$s', 'wordpoints' )
+						__('WordPoints has deactivated the following modules on this site because of incompatibilities with WordPoints %1$s: %2$s', 'wordpoints')
 						, WORDPOINTS_VERSION
-						, implode( ', ', $incompatible_modules )
+						, implode(', ', $incompatible_modules)
 					)
 					, array(
 						'dismissible' => true,
@@ -1164,23 +1164,23 @@ 
 	$is_notice_dismissed = wordpoints_verify_nonce(
 		'_wpnonce'
 		, 'wordpoints_dismiss_notice-%s'
-		, array( 'wordpoints_notice' )
+		, array('wordpoints_notice')
 		, 'post'
 	);
 
-	if ( $is_notice_dismissed && isset( $_POST['wordpoints_notice'] ) ) {
+	if ($is_notice_dismissed && isset($_POST['wordpoints_notice'])) {
 
-		$option = sanitize_key( $_POST['wordpoints_notice'] );
+		$option = sanitize_key($_POST['wordpoints_notice']);
 
-		if ( ! is_network_admin() && 'wordpoints_incompatible_modules' === $option ) {
-			delete_option( $option );
+		if ( ! is_network_admin() && 'wordpoints_incompatible_modules' === $option) {
+			delete_option($option);
 		} else {
-			wordpoints_delete_maybe_network_option( $option );
+			wordpoints_delete_maybe_network_option($option);
 		}
 	}
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		wp_die( '', 200 );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		wp_die('', 200);
 	}
 }
 
@@ -1197,14 +1197,14 @@ 
  *
  * @return mixed The option value, sanitized if it is for one of the plugin's screens.
  */
-function wordpoints_admin_set_screen_option( $sanitized, $option, $value ) {
+function wordpoints_admin_set_screen_option($sanitized, $option, $value) {
 
-	switch ( $option ) {
+	switch ($option) {
 
 		case 'wordpoints_page_wordpoints_modules_per_page':
 		case 'wordpoints_page_wordpoints_modules_network_per_page':
 		case 'toplevel_page_wordpoints_modules_per_page':
-			$sanitized = wordpoints_posint( $value );
+			$sanitized = wordpoints_posint($value);
 			break;
 	}
 
@@ -1225,18 +1225,18 @@ 
  */
 function wordpoints_admin_ajax_breaking_module_check() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) {
-		wp_die( '', 400 );
+	if ( ! isset($_GET['wordpoints_module_check'])) {
+		wp_die('', 400);
 	}
 
-	if ( is_network_admin() ) {
-		$nonce = get_site_option( 'wordpoints_module_check_nonce' );
+	if (is_network_admin()) {
+		$nonce = get_site_option('wordpoints_module_check_nonce');
 	} else {
-		$nonce = get_option( 'wordpoints_module_check_nonce' );
+		$nonce = get_option('wordpoints_module_check_nonce');
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
-		wp_die( '', 403 );
+	if ( ! $nonce || ! hash_equals($nonce, sanitize_key($_GET['wordpoints_module_check']))) {
+		wp_die('', 403);
 	}
 
 	// The list table constructor calls WP_Screen::get(), which expects this.
@@ -1244,7 +1244,7 @@ 
 
 	wordpoints_admin_screen_modules();
 
-	wp_die( '', 200 );
+	wp_die('', 200);
 }
 
 /**
@@ -1256,7 +1256,7 @@ 
  */
 function wordpoints_hooks_admin_ajax() {
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+	if (defined('DOING_AJAX') && DOING_AJAX) {
 		new WordPoints_Admin_Ajax_Hooks;
 	}
 }

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

Spacing +25 added lines, -25 removed lines

@@ -38,13 +38,13 @@ 
 	 *
 	 * @since 1.7.0
 	 */
-	public function __construct( array $args ) {
+	public function __construct(array $args) {
 
-		parent::__construct( $args );
+		parent::__construct($args);
 
-		$this->name = _x( 'Points', 'rank type', 'wordpoints' );
+		$this->name = _x('Points', 'rank type', 'wordpoints');
 
-		if ( ! isset( $args['points_type'] ) ) {
+		if ( ! isset($args['points_type'])) {
 			_doing_it_wrong(
 				__METHOD__
 				, 'WordPoints Error: The "points_type" argument is required.'
@@ -53,10 +53,10 @@ 
 			return;
 		}
 
-		$this->meta_fields['points']['label'] = _x( 'Points', 'form label', 'wordpoints' );
+		$this->meta_fields['points']['label'] = _x('Points', 'form label', 'wordpoints');
 		$this->meta_fields['points_type']['default'] = $args['points_type'];
 
-		add_action( 'wordpoints_points_altered', array( $this, 'hook' ), 10, 3 );
+		add_action('wordpoints_points_altered', array($this, 'hook'), 10, 3);
 	}
 
 	/**
@@ -66,7 +66,7 @@ 
 	 */
 	public function destruct() {
 
-		remove_action( 'wordpoints_points_altered', array( $this, 'hook' ), 10 );
+		remove_action('wordpoints_points_altered', array($this, 'hook'), 10);
 	}
 
 	/**
@@ -78,47 +78,47 @@ 
 	 * @param int    $points      The number of points.
 	 * @param string $points_type The type of points.
 	 */
-	public function hook( $user_id, $points, $points_type ) {
+	public function hook($user_id, $points, $points_type) {
 
-		if ( $points_type !== $this->meta_fields['points_type']['default'] ) {
+		if ($points_type !== $this->meta_fields['points_type']['default']) {
 			return;
 		}
 
-		$this->maybe_transition_user_ranks( $user_id, $points > 0 );
+		$this->maybe_transition_user_ranks($user_id, $points > 0);
 	}
 
 	/**
 	 * @since 1.7.0
 	 */
-	public function validate_rank_meta( array $meta ) {
+	public function validate_rank_meta(array $meta) {
 
-		if ( ! isset( $meta['points'] ) || false === wordpoints_int( $meta['points'] ) ) {
+		if ( ! isset($meta['points']) || false === wordpoints_int($meta['points'])) {
 			return new WP_Error(
 				'wordpoints_points_rank_type_invalid_points'
-				, __( 'The amount of points is required, and must be a valid number.', 'wordpoints' )
-				, array( 'field' => 'points' )
+				, __('The amount of points is required, and must be a valid number.', 'wordpoints')
+				, array('field' => 'points')
 			);
 		}
 
 		if (
-			! isset( $meta['points_type'] )
-			|| ! wordpoints_is_points_type( $meta['points_type'] )
+			! isset($meta['points_type'])
+			|| ! wordpoints_is_points_type($meta['points_type'])
 		) {
 			return false;
 		}
 
-		$minimum = wordpoints_get_points_minimum( $meta['points_type'] );
+		$minimum = wordpoints_get_points_minimum($meta['points_type']);
 
-		if ( $meta['points'] < $minimum ) {
+		if ($meta['points'] < $minimum) {
 
 			return new WP_Error(
 				'wordpoints_points_rank_type_points_less_than_minimum'
 				, sprintf(
 					// translators: Minimum number of points.
-					__( 'The number of points must be more than the minimum (%s).', 'wordpoints' )
-					, wordpoints_format_points( $minimum, $meta['points_type'], 'points_rank_error' )
+					__('The number of points must be more than the minimum (%s).', 'wordpoints')
+					, wordpoints_format_points($minimum, $meta['points_type'], 'points_rank_error')
 				)
-				, array( 'field' => 'points' )
+				, array('field' => 'points')
 			);
 		}
 
@@ -141,15 +141,15 @@ 
 	 *
 	 * @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) {
 
-		if ( $rank->points_type !== $this->meta_fields['points_type']['default'] ) {
+		if ($rank->points_type !== $this->meta_fields['points_type']['default']) {
 			return false;
 		}
 
-		$user_points = wordpoints_get_points( $user_id, $rank->points_type );
+		$user_points = wordpoints_get_points($user_id, $rank->points_type);
 
-		if ( $rank->points > $user_points ) {
+		if ($rank->points > $user_points) {
 			return false;
 		}
 

src/components/ranks/includes/integration/points/functions.php

Spacing +18 added lines, -18 removed lines

@@ -14,18 +14,18 @@ 
  *
  * @WordPress\filter wordpoints_points_widget_text 30 After esc_html().
  */
-function wordpoints_ranks_points_widget_text_filter( $text, $instance ) {
+function wordpoints_ranks_points_widget_text_filter($text, $instance) {
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 
 		$rank = wordpoints_get_formatted_user_rank(
 			get_current_user_id()
 			, "points_type-{$instance['points_type']}"
 			, 'my-points-widget'
-			, array( 'widget_settings' => $instance )
+			, array('widget_settings' => $instance)
 		);
 
-		$text = str_replace( '%rank%', $rank, $text );
+		$text = str_replace('%rank%', $rank, $text);
 	}
 
 	return $text;
@@ -43,7 +43,7 @@ 
 	?>
 	<br />
 	<?php // translators: Placeholder name. ?>
-	<small><i><?php echo esc_html( sprintf( __( '%s will be replaced with the rank of the logged in user', 'wordpoints' ), '%rank%' ) ); ?></i></small>
+	<small><i><?php echo esc_html(sprintf(__('%s will be replaced with the rank of the logged in user', 'wordpoints'), '%rank%')); ?></i></small>
 	<?php
 }
 
@@ -54,7 +54,7 @@ 
  *
  * @WordPress\filter wordpoints_points_top_users_username
  */
-function wordpoints_ranks_points_top_users_username_filter( $name, $user_id, $points_type, $context ) {
+function wordpoints_ranks_points_top_users_username_filter($name, $user_id, $points_type, $context) {
 
 	$rank = wordpoints_get_formatted_user_rank(
 		$user_id
@@ -74,11 +74,11 @@ 
  *
  * @WordPress\filter shortcode_atts_wordpoints_user_rank
  */
-function wordpoints_user_rank_shortcode_points_type_attr( $out, $pairs, $atts ) {
+function wordpoints_user_rank_shortcode_points_type_attr($out, $pairs, $atts) {
 
-	if ( empty( $out['rank_group'] ) ) {
+	if (empty($out['rank_group'])) {
 
-		if ( isset( $atts['points_type'] ) ) {
+		if (isset($atts['points_type'])) {
 
 			$out['rank_group'] = "points_type-{$atts['points_type']}";
 
@@ -86,7 +86,7 @@ 
 
 			$points_type = wordpoints_get_default_points_type();
 
-			if ( $points_type ) {
+			if ($points_type) {
 				$out['rank_group'] = "points_type-{$points_type}";
 			}
 		}
@@ -104,7 +104,7 @@ 
  */
 function wordpoints_register_points_ranks() {
 
-	foreach ( wordpoints_get_points_types() as $slug => $points_type ) {
+	foreach (wordpoints_get_points_types() as $slug => $points_type) {
 
 		WordPoints_Rank_Groups::register_group(
 			"points_type-{$slug}"
@@ -112,7 +112,7 @@ 
 				'name' => $points_type['name'],
 				'description' => sprintf(
 					// translators: Points type name.
-					__( 'This rank group is associated with the &#8220;%s&#8221; points type.', 'wordpoints' )
+					__('This rank group is associated with the &#8220;%s&#8221; points type.', 'wordpoints')
 					, $points_type['name']
 				),
 			)
@@ -121,7 +121,7 @@ 
 		WordPoints_Rank_Types::register_type(
 			"points-{$slug}"
 			, 'WordPoints_Points_Rank_Type'
-			, array( 'points_type' => $slug )
+			, array('points_type' => $slug)
 		);
 
 		WordPoints_Rank_Groups::register_type_for_group(
@@ -145,13 +145,13 @@ 
 
 	$tab = wordpoints_admin_get_current_tab();
 
-	if ( 'add-new' === $tab ) {
+	if ('add-new' === $tab) {
 		return;
 	}
 
 	add_meta_box(
 		'ranks'
-		, _x( 'Ranks', 'Points Types screen meta box title', 'wordpoints' )
+		, _x('Ranks', 'Points Types screen meta box title', 'wordpoints')
 		, 'wordpoints_ranks_display_points_types_meta_box_ranks'
 		, 'wordpoints_page_wordpoints_points_types'
 		, 'side'
@@ -166,12 +166,12 @@ 
  *
  * @param array $points_type The data for the points type being edited.
  */
-function wordpoints_ranks_display_points_types_meta_box_ranks( $points_type ) {
+function wordpoints_ranks_display_points_types_meta_box_ranks($points_type) {
 
 	?>
 
-	<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_ranks&tab=points_type-' . $points_type['slug'] ) ); ?>">
-		<?php esc_html_e( 'Go to the ranks for this points type.', 'wordpoints' ); ?>
+	<a href="<?php echo esc_url(self_admin_url('admin.php?page=wordpoints_ranks&tab=points_type-' . $points_type['slug'])); ?>">
+		<?php esc_html_e('Go to the ranks for this points type.', 'wordpoints'); ?>
 	</a>
 
 	<?php

src/components/ranks/admin/screens/ranks.php

Spacing +34 added lines, -34 removed lines

@@ -9,25 +9,25 @@ 
 
 $rank_groups = WordPoints_Rank_Groups::get();
 
-if ( empty( $rank_groups ) ) {
+if (empty($rank_groups)) {
 
 	?>
 	<div class="wrap">
-		<?php wordpoints_show_admin_error( esc_html__( 'No rank groups are currently available.', 'wordpoints' ) ); ?>
+		<?php wordpoints_show_admin_error(esc_html__('No rank groups are currently available.', 'wordpoints')); ?>
 	</div>
 	<?php
 
 	return;
 }
 
-$rank_group = $rank_groups[ wordpoints_admin_get_current_tab( $rank_groups ) ];
+$rank_group = $rank_groups[wordpoints_admin_get_current_tab($rank_groups)];
 
 ?>
 
 <div class="wrap">
-	<h1><?php esc_html_e( 'Ranks', 'wordpoints' ); ?></h1>
+	<h1><?php esc_html_e('Ranks', 'wordpoints'); ?></h1>
 
-	<?php wordpoints_admin_show_tabs( wp_list_pluck( $rank_groups, 'name' ), false ) ?>
+	<?php wordpoints_admin_show_tabs(wp_list_pluck($rank_groups, 'name'), false) ?>
 
 	<div id="ranks-error-message" class="notice notice-error hidden">
 		<p></p>
@@ -42,34 +42,34 @@ 
 	 *
 	 * @param WordPoints_Rank_Group $rank_group The rank group being displayed.
 	 */
-	do_action( 'wordpoints_ranks_admin_screen_top', $rank_group );
+	do_action('wordpoints_ranks_admin_screen_top', $rank_group);
 
 	?>
 
 	<div class="wordpoints-rank-group-container">
-		<p class="description group-description"><?php echo esc_html( $rank_group->description ); ?></p>
+		<p class="description group-description"><?php echo esc_html($rank_group->description); ?></p>
 		<ul
-			id="wordpoints-rank-group_<?php echo esc_attr( $rank_group->slug ); ?>"
+			id="wordpoints-rank-group_<?php echo esc_attr($rank_group->slug); ?>"
 			class="wordpoints-rank-group"
-			data-slug="<?php echo esc_attr( $rank_group->slug ); ?>"
-			data-nonce="<?php echo esc_attr( wp_create_nonce( "wordpoints_get_ranks-{$rank_group->slug}" ) ); ?>"
+			data-slug="<?php echo esc_attr($rank_group->slug); ?>"
+			data-nonce="<?php echo esc_attr(wp_create_nonce("wordpoints_get_ranks-{$rank_group->slug}")); ?>"
 		>
 		</ul>
 		<div class="spinner-overlay" style="display: block;">
 			<span class="spinner is-active"></span>
 		</div>
 		<div class="controls">
-			<button class="add-rank button-primary"><?php esc_html_e( 'Add Rank', 'wordpoints' ); ?></button>
+			<button class="add-rank button-primary"><?php esc_html_e('Add Rank', 'wordpoints'); ?></button>
 			<label class="wordpoints-rank-types">
-				<?php esc_html_e( 'Rank Type:', 'wordpoints' ); ?>
+				<?php esc_html_e('Rank Type:', 'wordpoints'); ?>
 				<select>
-					<?php foreach ( $rank_group->get_types() as $rank_type ) : ?>
-						<?php if ( 'base' !== $rank_type ) : ?>
+					<?php foreach ($rank_group->get_types() as $rank_type) : ?>
+						<?php if ('base' !== $rank_type) : ?>
 							<option
-								value="<?php echo esc_attr( $rank_type ); ?>"
-								data-nonce="<?php echo esc_attr( wp_create_nonce( "wordpoints_create_rank|{$rank_group->slug}|{$rank_type}" ) ); ?>"
+								value="<?php echo esc_attr($rank_type); ?>"
+								data-nonce="<?php echo esc_attr(wp_create_nonce("wordpoints_create_rank|{$rank_group->slug}|{$rank_type}")); ?>"
 							>
-								<?php echo esc_html( WordPoints_Rank_Types::get_type( $rank_type )->get_name() ); ?>
+								<?php echo esc_html(WordPoints_Rank_Types::get_type($rank_type)->get_name()); ?>
 							</option>
 						<?php endif; ?>
 					<?php endforeach; ?>
@@ -86,13 +86,13 @@ 
 	 *
 	 * @param WordPoints_Rank_Group $rank_group The rank group being displayed.
 	 */
-	do_action( 'wordpoints_ranks_admin_screen_bottom', $rank_group );
+	do_action('wordpoints_ranks_admin_screen_bottom', $rank_group);
 
 	?>
 </div>
 
-<?php foreach ( $rank_group->get_types() as $rank_type ) : ?>
-	<script type="text/template" class="rank-template_<?php echo sanitize_html_class( $rank_type ); ?>">
+<?php foreach ($rank_group->get_types() as $rank_type) : ?>
+	<script type="text/template" class="rank-template_<?php echo sanitize_html_class($rank_type); ?>">
 		<div class="view">
 			<div>
 				<%- name %>
@@ -100,31 +100,31 @@ 
 
 				$field = key(
 					wp_list_filter(
-						WordPoints_Rank_Types::get_type( $rank_type )->get_meta_fields()
-						, array( 'in_title' => true )
+						WordPoints_Rank_Types::get_type($rank_type)->get_meta_fields()
+						, array('in_title' => true)
 					)
 				);
 
 				?>
-				<?php if ( $field ) : ?>
-					<span><% if ( typeof <?php echo preg_replace( '/[^a-z0-9_]/i', '', $field ); // WPCS: XSS OK ?> !== "undefined" ) { print( <?php echo preg_replace( '/[^a-z0-9_]/i', '', $field ); // WPCS: XSS OK ?> ); } %></span>
+				<?php if ($field) : ?>
+					<span><% if ( typeof <?php echo preg_replace('/[^a-z0-9_]/i', '', $field); // WPCS: XSS OK ?> !== "undefined" ) { print( <?php echo preg_replace('/[^a-z0-9_]/i', '', $field); // WPCS: XSS OK ?> ); } %></span>
 				<?php endif; ?>
 			</div>
-			<a class="edit"><?php echo esc_html_x( 'Edit', 'rank', 'wordpoints' ); ?></a>
-			<a class="close"><?php esc_html_e( 'Close', 'wordpoints' ); ?></a>
+			<a class="edit"><?php echo esc_html_x('Edit', 'rank', 'wordpoints'); ?></a>
+			<a class="close"><?php esc_html_e('Close', 'wordpoints'); ?></a>
 		</div>
 		<form>
 			<div class="fields">
 				<p class="description description-thin">
 				<label>
-					<?php echo esc_html_x( 'Title', 'rank', 'wordpoints' ); ?>
+					<?php echo esc_html_x('Title', 'rank', 'wordpoints'); ?>
 					<input class="widefat" type="text" value="<%- name %>" name="name" />
 				</label>
 				</p>
-				<input type="hidden" name="type" value="<?php echo esc_attr( $rank_type ); ?>" />
+				<input type="hidden" name="type" value="<?php echo esc_attr($rank_type); ?>" />
 				<input type="hidden" name="order" value="<%- order %>" />
 				<input type="hidden" name="nonce" value="<%- nonce %>" />
-				<?php WordPoints_Rank_Types::get_type( $rank_type )->display_rank_meta_form_fields( array(), array( 'placeholders' => true ) ); ?>
+				<?php WordPoints_Rank_Types::get_type($rank_type)->display_rank_meta_form_fields(array(), array('placeholders' => true)); ?>
 			</div>
 			<div class="messages">
 				<div class="success"></div>
@@ -135,11 +135,11 @@ 
 					<span class="spinner is-active"></span>
 				</div>
 				<div class="action-buttons">
-					<button class="save button-primary" disabled><?php esc_html_e( 'Save', 'wordpoints' ); ?></button>
-					<button class="cancel button-secondary"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></button>
-					<button class="close button-secondary"><?php esc_html_e( 'Close', 'wordpoints' ); ?></button>
-					<?php if ( 'base' !== $rank_type ) : ?>
-						<button class="delete button-secondary"><?php esc_html_e( 'Delete', 'wordpoints' ); ?></button>
+					<button class="save button-primary" disabled><?php esc_html_e('Save', 'wordpoints'); ?></button>
+					<button class="cancel button-secondary"><?php esc_html_e('Cancel', 'wordpoints'); ?></button>
+					<button class="close button-secondary"><?php esc_html_e('Close', 'wordpoints'); ?></button>
+					<?php if ('base' !== $rank_type) : ?>
+						<button class="delete button-secondary"><?php esc_html_e('Delete', 'wordpoints'); ?></button>
 					<?php endif; ?>
 				</div>
 			</div>

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

Spacing +73 added lines, -73 removed lines

@@ -66,8 +66,8 @@ 
 		$rank_groups = WordPoints_Rank_Groups::get();
 		$ranks = array();
 
-		foreach ( $rank_groups as $group ) {
-			$ranks[ $group->slug ] = self::prepare_group_ranks( $group );
+		foreach ($rank_groups as $group) {
+			$ranks[$group->slug] = self::prepare_group_ranks($group);
 		}
 
 		return $ranks;
@@ -82,20 +82,20 @@ 
 	 *
 	 * @return array The ranks of this group, prepared for sending to the JS.
 	 */
-	public static function prepare_group_ranks( $group ) {
+	public static function prepare_group_ranks($group) {
 
 		$rank_ids = $group->get_ranks();
 		$ranks = array();
 
-		foreach ( $rank_ids as $order => $rank_id ) {
+		foreach ($rank_ids as $order => $rank_id) {
 
-			$rank = wordpoints_get_rank( $rank_id );
+			$rank = wordpoints_get_rank($rank_id);
 
-			if ( ! $rank ) {
+			if ( ! $rank) {
 				continue;
 			}
 
-			$ranks[] = self::_prepare_rank( $rank );
+			$ranks[] = self::_prepare_rank($rank);
 		}
 
 		return $ranks;
@@ -112,8 +112,8 @@ 
 	 */
 	public function __construct() {
 
-		if ( isset( self::$instance ) ) {
-			_doing_it_wrong( __METHOD__, 'Class should only be constructed once.', '1.7.0' );
+		if (isset(self::$instance)) {
+			_doing_it_wrong(__METHOD__, 'Class should only be constructed once.', '1.7.0');
 		}
 
 		self::$instance = $this;
@@ -128,10 +128,10 @@ 
 	 */
 	public function hooks() {
 
-		add_action( 'wp_ajax_wordpoints_admin_get_ranks', array( $this, 'get_ranks' ) );
-		add_action( 'wp_ajax_wordpoints_admin_create_rank', array( $this, 'create_rank' ) );
-		add_action( 'wp_ajax_wordpoints_admin_update_rank', array( $this, 'update_rank' ) );
-		add_action( 'wp_ajax_wordpoints_admin_delete_rank', array( $this, 'delete_rank' ) );
+		add_action('wp_ajax_wordpoints_admin_get_ranks', array($this, 'get_ranks'));
+		add_action('wp_ajax_wordpoints_admin_create_rank', array($this, 'create_rank'));
+		add_action('wp_ajax_wordpoints_admin_update_rank', array($this, 'update_rank'));
+		add_action('wp_ajax_wordpoints_admin_delete_rank', array($this, 'delete_rank'));
 	}
 
 	/**
@@ -145,9 +145,9 @@ 
 
 		$group = $this->_get_group();
 
-		$this->_verify_request( "wordpoints_get_ranks-{$group->slug}" );
+		$this->_verify_request("wordpoints_get_ranks-{$group->slug}");
 
-		wp_send_json_success( self::prepare_group_ranks( $group ) );
+		wp_send_json_success(self::prepare_group_ranks($group));
 	}
 
 	/**
@@ -162,7 +162,7 @@ 
 		$group = $this->_get_group();
 		$type = $this->_get_rank_type()->get_slug();
 
-		$this->_verify_request( "wordpoints_create_rank|{$group->slug}|{$type}" );
+		$this->_verify_request("wordpoints_create_rank|{$group->slug}|{$type}");
 
 		// Attempt to save the rank.
 		$result = wordpoints_add_rank(
@@ -173,7 +173,7 @@ 
 			, $this->_get_rank_meta()
 		);
 
-		$this->_send_json_result( $result, 'create' );
+		$this->_send_json_result($result, 'create');
 	}
 
 	/**
@@ -194,8 +194,8 @@ 
 
 		$type = $this->_get_rank_type()->get_slug();
 
-		if ( $type !== $rank->type ) {
-			wp_send_json_error( array( 'message' => __( 'This rank does not match any rank in the database, perhaps it was deleted. Refresh the page to update the list of ranks.', 'wordpoints' ) ) );
+		if ($type !== $rank->type) {
+			wp_send_json_error(array('message' => __('This rank does not match any rank in the database, perhaps it was deleted. Refresh the page to update the list of ranks.', 'wordpoints')));
 		}
 
 		$result = wordpoints_update_rank(
@@ -207,7 +207,7 @@ 
 			, $this->_get_rank_meta()
 		);
 
-		$this->_send_json_result( $result, 'update' );
+		$this->_send_json_result($result, 'update');
 	}
 
 	/**
@@ -226,10 +226,10 @@ 
 			"wordpoints_delete_rank|{$group->slug}|{$rank->ID}"
 		);
 
-		$result = wordpoints_delete_rank( $rank->ID );
+		$result = wordpoints_delete_rank($rank->ID);
 
-		if ( ! $result ) {
-			wp_send_json_error( array( 'message' => __( 'There was an error deleting the rank. Please try again.', 'wordpoints' ) ) );
+		if ( ! $result) {
+			wp_send_json_error(array('message' => __('There was an error deleting the rank. Please try again.', 'wordpoints')));
 		}
 
 		wp_send_json_success();
@@ -250,11 +250,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,
 			)
 		);
@@ -270,8 +270,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')));
 		}
 	}
 
@@ -284,14 +284,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'))
 			);
 		}
 	}
@@ -305,14 +305,14 @@ 
 	 */
 	private function _get_group() {
 
-		if ( ! isset( $_POST['group'] ) ) { // WPCS: CSRF OK.
-			$this->_unexpected_error( 'group' );
+		if ( ! isset($_POST['group'])) { // WPCS: CSRF OK.
+			$this->_unexpected_error('group');
 		}
 
-		$group = WordPoints_Rank_Groups::get_group( sanitize_key( $_POST['group'] ) ); // WPCS: CSRF OK.
+		$group = WordPoints_Rank_Groups::get_group(sanitize_key($_POST['group'])); // WPCS: CSRF OK.
 
-		if ( ! $group ) {
-			wp_send_json_error( array( 'message' => __( 'The rank group passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints' ) ) );
+		if ( ! $group) {
+			wp_send_json_error(array('message' => __('The rank group passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints')));
 		}
 
 		return $group;
@@ -327,14 +327,14 @@ 
 	 */
 	private function _get_rank() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
-			$this->_unexpected_error( 'id' );
+		if ( ! isset($_POST['id'])) { // WPCS: CSRF OK.
+			$this->_unexpected_error('id');
 		}
 
-		$rank = wordpoints_get_rank( wordpoints_int( $_POST['id'] ) ); // WPCS: CSRF OK.
+		$rank = wordpoints_get_rank(wordpoints_int($_POST['id'])); // WPCS: CSRF OK.
 
-		if ( ! $rank ) {
-			wp_send_json_error( array( 'message' => __( 'The rank ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints' ) ) );
+		if ( ! $rank) {
+			wp_send_json_error(array('message' => __('The rank ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints')));
 		}
 
 		return $rank;
@@ -351,14 +351,14 @@ 
 
 		$name = '';
 
-		if ( ! empty( $_POST['name'] ) ) { // WPCS: CSRF OK.
-			$name = sanitize_text_field( wp_unslash( $_POST['name'] ) ); // WPCS: CSRF OK.
+		if ( ! empty($_POST['name'])) { // WPCS: CSRF OK.
+			$name = sanitize_text_field(wp_unslash($_POST['name'])); // WPCS: CSRF OK.
 		}
 
-		if ( ! $name ) {
+		if ( ! $name) {
 			wp_send_json_error(
 				array(
-					'message' => __( 'Please enter a name for this rank.', 'wordpoints' ),
+					'message' => __('Please enter a name for this rank.', 'wordpoints'),
 					'field'   => 'name',
 				)
 			);
@@ -376,17 +376,17 @@ 
 	 */
 	private function _get_rank_type() {
 
-		if ( empty( $_POST['type'] ) ) { // WPCS: CSRF OK.
-			$this->_unexpected_error( 'type' );
+		if (empty($_POST['type'])) { // WPCS: CSRF OK.
+			$this->_unexpected_error('type');
 		}
 
-		$type = sanitize_text_field( wp_unslash( $_POST['type'] ) ); // WPCS: CSRF OK.
+		$type = sanitize_text_field(wp_unslash($_POST['type'])); // WPCS: CSRF OK.
 
-		if ( ! WordPoints_Rank_Types::is_type_registered( $type ) ) {
-			wp_send_json_error( array( 'message' => __( 'That rank type was not recognized. It may no longer be available. Try reloading the page.', 'wordpoints' ) ) );
+		if ( ! WordPoints_Rank_Types::is_type_registered($type)) {
+			wp_send_json_error(array('message' => __('That rank type was not recognized. It may no longer be available. Try reloading the page.', 'wordpoints')));
 		}
 
-		$this->rank_type = WordPoints_Rank_Types::get_type( $type );
+		$this->rank_type = WordPoints_Rank_Types::get_type($type);
 
 		return $this->rank_type;
 	}
@@ -401,10 +401,10 @@ 
 	private function _get_rank_position() {
 
 		if (
-			! isset( $_POST['order'] ) // WPCS: CSRF OK.
-			|| false === wordpoints_int( $_POST['order'] ) // WPCS: CSRF OK.
+			! isset($_POST['order']) // WPCS: CSRF OK.
+			|| false === wordpoints_int($_POST['order']) // WPCS: CSRF OK.
 		) {
-			$this->_unexpected_error( 'order' );
+			$this->_unexpected_error('order');
 		}
 
 		return (int) $_POST['order'];
@@ -420,7 +420,7 @@ 
 	private function _get_rank_meta() {
 
 		return array_intersect_key(
-			wp_unslash( $_POST ) // WPCS: CSRF OK.
+			wp_unslash($_POST) // WPCS: CSRF OK.
 			, $this->rank_type->get_meta_fields()
 		);
 	}
@@ -433,19 +433,19 @@ 
 	 * @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 rank. Please try again.', 'wordpoints' );
+			if ('create' === $action) {
+				$message = __('There was an error adding the rank. Please try again.', 'wordpoints');
 			} else {
-				$message = __( 'There was an error updating the rank. Please try again.', 'wordpoints' );
+				$message = __('There was an error updating the rank. Please try again.', 'wordpoints');
 			}
 
-			wp_send_json_error( array( 'message' => $message ) );
+			wp_send_json_error(array('message' => $message));
 
-		} elseif ( is_wp_error( $result ) ) {
+		} elseif (is_wp_error($result)) {
 
 			wp_send_json_error(
 				array(
@@ -457,11 +457,11 @@ 
 
 		$data = null;
 
-		if ( 'create' === $action ) {
-			$data = self::_prepare_rank( wordpoints_get_rank( $result ) );
+		if ('create' === $action) {
+			$data = self::_prepare_rank(wordpoints_get_rank($result));
 		}
 
-		wp_send_json_success( $data );
+		wp_send_json_success($data);
 	}
 
 	//
@@ -477,15 +477,15 @@ 
 	 *
 	 * @return array The rank data extracted into an array.
 	 */
-	private static function _prepare_rank( $rank ) {
+	private static function _prepare_rank($rank) {
 
 		$name = $rank->name;
-		if ( empty( $name ) ) {
-			$name = __( '(no title)', 'wordpoints' );
+		if (empty($name)) {
+			$name = __('(no title)', 'wordpoints');
 		}
 
-		$order = WordPoints_Rank_Groups::get_group( $rank->rank_group )
-			->get_rank_position( $rank->ID );
+		$order = WordPoints_Rank_Groups::get_group($rank->rank_group)
+			->get_rank_position($rank->ID);
 
 		$prepared_rank = array(
 			'id'    => $rank->ID,
@@ -500,10 +500,10 @@ 
 			),
 		);
 
-		$rank_type = WordPoints_Rank_Types::get_type( $rank->type );
+		$rank_type = WordPoints_Rank_Types::get_type($rank->type);
 
-		foreach ( $rank_type->get_meta_fields() as $field => $data ) {
-			$prepared_rank[ $field ] = $rank->{$field};
+		foreach ($rank_type->get_meta_fields() as $field => $data) {
+			$prepared_rank[$field] = $rank->{$field};
 		}
 
 		return $prepared_rank;

src/components/points/includes/class-wordpoints-points-hooks.php

Spacing +125 added lines, -125 removed lines

@@ -98,7 +98,7 @@ 
 	 * @deprecated 2.1.0
 	 */
 	public static function init() {
-		_deprecated_function( __METHOD__, '2.1.0' );
+		_deprecated_function(__METHOD__, '2.1.0');
 	}
 
 	/**
@@ -108,7 +108,7 @@ 
 	 *
 	 * @param string $class_name A 'WordPoints_Points_Hook' class name.
 	 */
-	public static function register( $class_name ) {
+	public static function register($class_name) {
 
 		self::$classes[] = $class_name;
 	}
@@ -127,14 +127,14 @@ 
 		 *
 		 * @since 1.4.0
 		 */
-		do_action( 'wordpoints_points_hooks_register' );
+		do_action('wordpoints_points_hooks_register');
 
-		$classes = array_unique( self::$classes );
+		$classes = array_unique(self::$classes);
 
-		foreach ( $classes as $class_name ) {
+		foreach ($classes as $class_name) {
 
 			$hook_type = new $class_name();
-			self::$hook_types[ $hook_type->get_id_base() ] = $hook_type;
+			self::$hook_types[$hook_type->get_id_base()] = $hook_type;
 		}
 
 		/**
@@ -142,7 +142,7 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_points_hooks_registered' );
+		do_action('wordpoints_points_hooks_registered');
 	}
 
 	/**
@@ -166,25 +166,25 @@ 
 	 *
 	 * @return WordPoints_Points_Hook|false The hook object, or false for invalid ID.
 	 */
-	public static function get_handler( $hook_id ) {
+	public static function get_handler($hook_id) {
 
-		list( $hook_type, $id_number ) = explode( '-', $hook_id );
+		list($hook_type, $id_number) = explode('-', $hook_id);
 
-		$hook_type = self::get_handler_by_id_base( $hook_type );
+		$hook_type = self::get_handler_by_id_base($hook_type);
 
-		if ( false === $hook_type ) {
+		if (false === $hook_type) {
 			return false;
 		}
 
-		$type = ( self::$network_mode ) ? 'network' : 'standard';
+		$type = (self::$network_mode) ? 'network' : 'standard';
 
-		$instances = $hook_type->get_instances( $type );
+		$instances = $hook_type->get_instances($type);
 
-		if ( ! isset( $instances[ $id_number ] ) ) {
+		if ( ! isset($instances[$id_number])) {
 			return false;
 		}
 
-		$hook_type->set_number( $id_number );
+		$hook_type->set_number($id_number);
 
 		return $hook_type;
 	}
@@ -201,13 +201,13 @@ 
 	 *
 	 * @return WordPoints_Points_Hook|false False if no handler found.
 	 */
-	public static function get_handler_by_id_base( $id_base ) {
+	public static function get_handler_by_id_base($id_base) {
 
-		if ( ! isset( self::$hook_types[ $id_base ] ) ) {
+		if ( ! isset(self::$hook_types[$id_base])) {
 			return false;
 		}
 
-		return self::$hook_types[ $id_base ];
+		return self::$hook_types[$id_base];
 	}
 
 	/**
@@ -221,35 +221,35 @@ 
 	 *                                 installed. Only needed if on multisite. If
 	 *                                 omitted, the current site ID is used.
 	 */
-	public static function uninstall_hook_types( $hook_types, array $site_ids = null ) {
+	public static function uninstall_hook_types($hook_types, array $site_ids = null) {
 
-		_deprecated_function( __METHOD__, '2.0.0', 'WordPoints_Un_Installer_Base::$uninstall' );
+		_deprecated_function(__METHOD__, '2.0.0', 'WordPoints_Un_Installer_Base::$uninstall');
 
 		$hook_types = (array) $hook_types;
 
-		if ( is_multisite() ) {
+		if (is_multisite()) {
 
-			foreach ( $hook_types as $hook_type ) {
-				delete_site_option( "wordpoints_hook-{$hook_type}" );
+			foreach ($hook_types as $hook_type) {
+				delete_site_option("wordpoints_hook-{$hook_type}");
 			}
 
-			if ( ! isset( $site_ids ) ) {
-				$site_ids = array( get_current_blog_id() );
+			if ( ! isset($site_ids)) {
+				$site_ids = array(get_current_blog_id());
 			}
 
-			foreach ( $site_ids as $site_id ) {
+			foreach ($site_ids as $site_id) {
 
-				switch_to_blog( $site_id );
-				foreach ( $hook_types as $hook_type ) {
-					delete_option( "wordpoints_hook-{$hook_type}" );
+				switch_to_blog($site_id);
+				foreach ($hook_types as $hook_type) {
+					delete_option("wordpoints_hook-{$hook_type}");
 				}
 				restore_current_blog();
 			}
 
 		} else {
 
-			foreach ( $hook_types as $hook_type ) {
-				delete_option( "wordpoints_hook-{$hook_type}" );
+			foreach ($hook_types as $hook_type) {
+				delete_option("wordpoints_hook-{$hook_type}");
 			}
 		}
 	}
@@ -266,7 +266,7 @@ 
 
 		// Sort the hooks by name.
 		$hook_types = self::$hook_types;
-		uasort( $hook_types, array( __CLASS__, '_sort_name_callback' ) );
+		uasort($hook_types, array(__CLASS__, '_sort_name_callback'));
 
 		$disabled_hooks = wordpoints_get_maybe_network_array_option(
 			'wordpoints_legacy_points_hooks_disabled'
@@ -276,9 +276,9 @@ 
 		$i = 0;
 
 		// Display a representative for each hook type.
-		foreach ( $hook_types as $id_base => $hook_type ) {
+		foreach ($hook_types as $id_base => $hook_type) {
 
-			if ( isset( $disabled_hooks[ $id_base ] ) ) {
+			if (isset($disabled_hooks[$id_base])) {
 				continue;
 			}
 
@@ -291,16 +291,16 @@ 
 			$args['_multi_num'] = $hook_type->next_hook_id_number();
 			$args['_id_slug']   = $i;
 
-			$hook_type->set_options( $args );
+			$hook_type->set_options($args);
 
-			self::_list_hook( $hook_type->get_id( 0 ), $hook_type );
+			self::_list_hook($hook_type->get_id(0), $hook_type);
 		}
 
 		// If there were none, give the user a message.
-		if ( empty( $hook_types ) ) {
+		if (empty($hook_types)) {
 
 			echo '<div class="wordpoints-no-hooks">'
-				. esc_html__( 'There are no points hooks currently available.', 'wordpoints' )
+				. esc_html__('There are no points hooks currently available.', 'wordpoints')
 				. '</div>';
 		}
 	}
@@ -318,21 +318,21 @@ 
 	 *
 	 * @return void
 	 */
-	public static function list_by_points_type( $slug ) {
+	public static function list_by_points_type($slug) {
 
-		if ( '_inactive_hooks' !== $slug && ! wordpoints_is_points_type( $slug ) ) {
+		if ('_inactive_hooks' !== $slug && ! wordpoints_is_points_type($slug)) {
 			return;
 		}
 
-		$points_type_hooks = self::get_points_type_hooks( $slug );
+		$points_type_hooks = self::get_points_type_hooks($slug);
 
-		foreach ( $points_type_hooks as $hook_id ) {
+		foreach ($points_type_hooks as $hook_id) {
 
-			list( $hook_type ) = explode( '-', $hook_id );
+			list($hook_type) = explode('-', $hook_id);
 
-			$hook_type = self::get_handler_by_id_base( $hook_type );
+			$hook_type = self::get_handler_by_id_base($hook_type);
 
-			if ( false === $hook_type ) {
+			if (false === $hook_type) {
 				continue;
 			}
 
@@ -340,13 +340,13 @@ 
 
 			$options['_display'] = 'instance';
 
-			unset( $options['_add'] );
+			unset($options['_add']);
 
 			$options['_id_slug'] = $slug;
 
-			$hook_type->set_options( $options );
+			$hook_type->set_options($options);
 
-			self::_list_hook( $hook_id, $hook_type, $slug );
+			self::_list_hook($hook_id, $hook_type, $slug);
 		}
 	}
 
@@ -362,9 +362,9 @@ 
 	 *
 	 * @param bool $on Whether to turn network mode on or off.
 	 */
-	public static function set_network_mode( $on ) {
+	public static function set_network_mode($on) {
 
-		if ( $on !== self::$network_mode ) {
+		if ($on !== self::$network_mode) {
 			self::$network_mode = (bool) $on;
 		}
 	}
@@ -408,12 +408,12 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_points_type_hooks( $slug ) {
+	public static function get_points_type_hooks($slug) {
 
 		$points_types_hooks = self::get_points_types_hooks();
 
-		if ( isset( $points_types_hooks[ $slug ] ) && is_array( $points_types_hooks[ $slug ] ) ) {
-			$points_type_hooks = $points_types_hooks[ $slug ];
+		if (isset($points_types_hooks[$slug]) && is_array($points_types_hooks[$slug])) {
+			$points_type_hooks = $points_types_hooks[$slug];
 		} else {
 			$points_type_hooks = array();
 		}
@@ -428,12 +428,12 @@ 
 	 *
 	 * @param array $points_types_hooks The list of points types and their hooks.
 	 */
-	public static function save_points_types_hooks( array $points_types_hooks ) {
+	public static function save_points_types_hooks(array $points_types_hooks) {
 
-		if ( self::$network_mode ) {
-			update_site_option( 'wordpoints_points_types_hooks', $points_types_hooks );
+		if (self::$network_mode) {
+			update_site_option('wordpoints_points_types_hooks', $points_types_hooks);
 		} else {
-			update_option( 'wordpoints_points_types_hooks', $points_types_hooks );
+			update_option('wordpoints_points_types_hooks', $points_types_hooks);
 		}
 	}
 
@@ -446,11 +446,11 @@ 
 	 *
 	 * @return string|false The points type for the hook. False if not found.
 	 */
-	public static function get_points_type( $hook_id ) {
+	public static function get_points_type($hook_id) {
 
-		foreach ( self::get_points_types_hooks() as $points_type => $hooks ) {
+		foreach (self::get_points_types_hooks() as $points_type => $hooks) {
 
-			if ( in_array( $hook_id, $hooks ) ) {
+			if (in_array($hook_id, $hooks)) {
 				return $points_type;
 			}
 		}
@@ -469,9 +469,9 @@ 
 
 		$defaults = array();
 
-		foreach ( wordpoints_get_points_types() as $slug => $settings ) {
+		foreach (wordpoints_get_points_types() as $slug => $settings) {
 
-			$defaults[ $slug ] = array();
+			$defaults[$slug] = array();
 		}
 
 		return $defaults;
@@ -496,7 +496,7 @@ 
 	 * @param string $slug The slug for this type of points.
 	 * @param string $wrap Whether to wrap the form inputs in a "widget" or not.
 	 */
-	public static function points_type_form( $slug = null, $wrap = 'hook' ) {
+	public static function points_type_form($slug = null, $wrap = 'hook') {
 
 		_deprecated_function(
 			__METHOD__
@@ -506,12 +506,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(
@@ -523,11 +523,11 @@ 
 			,$points_type
 		);
 
-		if ( ! isset( $slug ) && 'hook' === $wrap ) {
+		if ( ! isset($slug) && 'hook' === $wrap) {
 			$wrap = 'hook-content';
 		}
 
-		switch ( $wrap ) {
+		switch ($wrap) {
 
 			case 'hook':
 				$hook_wrap = true;
@@ -546,25 +546,25 @@ 
 
 		?>
 
-		<?php if ( $hook_wrap ) : ?>
+		<?php if ($hook_wrap) : ?>
 			<div class="hook points-settings">
 				<div class="hook-top">
 					<div class="hook-title-action">
 						<a class="hook-action hide-if-no-js" href="#available-hooks"></a>
 					</div>
-					<div class="hook-title"><h3><?php esc_html_e( 'Settings', 'wordpoints' ); ?><span class="in-hook-title"></span></h3></div>
+					<div class="hook-title"><h3><?php esc_html_e('Settings', 'wordpoints'); ?><span class="in-hook-title"></span></h3></div>
 				</div>
 
 				<div class="hook-inside">
 		<?php endif; ?>
 
-			<?php if ( $hook_content_wrap ) : ?>
+			<?php if ($hook_content_wrap) : ?>
 				<form method="post">
 					<div class="hook-content">
 			<?php endif; ?>
 
-						<?php if ( $slug ) : ?>
-						<p><span class="wordpoints-points-slug"><em><?php esc_html_e( 'Slug', 'wordpoints' ); ?>: <?php echo esc_html( $slug ); ?></em></span></p>
+						<?php if ($slug) : ?>
+						<p><span class="wordpoints-points-slug"><em><?php esc_html_e('Slug', 'wordpoints'); ?>: <?php echo esc_html($slug); ?></em></span></p>
 						<?php endif; ?>
 
 						<?php
@@ -578,33 +578,33 @@ 
 						 *
 						 * @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 ( 'hook-content' === $wrap ) {
+						if ('hook-content' === $wrap) {
 
 							// 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>
-							<input class="widefat" type="text" id="points-name-<?php echo esc_attr( $slug ); ?>" name="points-name" value="<?php echo esc_attr( $points_type['name'] ); ?>" />
+							<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); ?>" name="points-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>
-							<input class="widefat" type="text" id="points-prefix-<?php echo esc_attr( $slug ); ?>" name="points-prefix" value="<?php echo esc_attr( $points_type['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); ?>" name="points-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>
-							<input class="widefat" type="text" id="points-suffix-<?php echo esc_attr( $slug ); ?>" name="points-suffix" value="<?php echo esc_attr( $points_type['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); ?>" name="points-suffix" value="<?php echo esc_attr($points_type['suffix']); ?>" />
 						</p>
 
 						<?php
@@ -618,30 +618,30 @@ 
 						 *
 						 * @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);
 
 						?>
 
-			<?php if ( $hook_content_wrap ) : ?>
+			<?php if ($hook_content_wrap) : ?>
 					</div>
 
-					<input type="hidden" name="points-slug" value="<?php echo esc_attr( $slug ); ?>" />
-					<input type="hidden" name="add_new" class="add_new" value="<?php echo esc_attr( $add_new ); ?>" />
+					<input type="hidden" name="points-slug" value="<?php echo esc_attr($slug); ?>" />
+					<input type="hidden" name="add_new" class="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}"));
 							}
 
 							?>
-							<a class="hook-control-close" href="#close"><?php esc_html_e( 'Close', 'wordpoints' ); ?></a>
+							<a class="hook-control-close" href="#close"><?php esc_html_e('Close', 'wordpoints'); ?></a>
 						</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" />
@@ -649,7 +649,7 @@ 
 				</form>
 			<?php endif; ?>
 
-		<?php if ( $hook_wrap ) : ?>
+		<?php if ($hook_wrap) : ?>
 				</div>
 			</div>
 
@@ -669,25 +669,25 @@ 
 	 * @param WordPoints_Points_Hook $hook        A points hook object.
 	 * @param string                 $points_type The slug for a points type.
 	 */
-	private static function _list_hook( $hook_id, $hook, $points_type = null ) {
+	private static function _list_hook($hook_id, $hook, $points_type = null) {
 
-		$number  = $hook->get_number_by_id( $hook_id );
+		$number  = $hook->get_number_by_id($hook_id);
 		$id_base = $hook->get_id_base();
 		$options = $hook->get_options();
 
 		$id_format = $hook_id;
 
-		$multi_number = ( isset( $options['_multi_num'] ) ) ? $options['_multi_num'] : '';
-		$add_new      = ( isset( $options['_add'] ) )       ? $options['_add']       : '';
+		$multi_number = (isset($options['_multi_num'])) ? $options['_multi_num'] : '';
+		$add_new      = (isset($options['_add'])) ? $options['_add'] : '';
 
 		// Prepare the URL query string.
-		$query_arg = array( 'edithook' => $id_format );
+		$query_arg = array('edithook' => $id_format);
 
-		if ( $add_new ) {
+		if ($add_new) {
 
 			$query_arg['addnew'] = 1;
 
-			if ( $multi_number ) {
+			if ($multi_number) {
 
 				$query_arg['num']  = $multi_number;
 				$query_arg['base'] = $id_base;
@@ -698,7 +698,7 @@ 
 			$query_arg['points_type'] = $points_type;
 		}
 
-		if ( isset( $options['_display'] ) && 'template' === $options['_display'] ) {
+		if (isset($options['_display']) && 'template' === $options['_display']) {
 
 			/*
 			 * We aren't outputting the form for a hook, but a template form for this
@@ -714,17 +714,17 @@ 
 
 		?>
 
-		<div id="hook-<?php echo esc_html( $options['_id_slug'] ); ?>_<?php echo esc_attr( $id_format ); ?>" class="hook <?php echo esc_attr( $options['_classname'] ); ?>">
+		<div id="hook-<?php echo esc_html($options['_id_slug']); ?>_<?php echo esc_attr($id_format); ?>" class="hook <?php echo esc_attr($options['_classname']); ?>">
 		<div class="hook-top">
 			<div class="hook-title-action">
 				<a class="hook-action hide-if-no-js" href="#available-hooks"></a>
-				<a class="hook-control-edit hide-if-js" href="<?php echo esc_url( add_query_arg( $query_arg ) ); ?>">
-					<span class="edit"><?php echo esc_html_x( 'Edit', 'hook', 'wordpoints' ); ?></span>
-					<span class="add"><?php echo esc_html_x( 'Add', 'hook', 'wordpoints' ); ?></span>
-					<span class="screen-reader-text"><?php echo esc_html( strip_tags( $hook->get_name() ) ); ?></span>
+				<a class="hook-control-edit hide-if-js" href="<?php echo esc_url(add_query_arg($query_arg)); ?>">
+					<span class="edit"><?php echo esc_html_x('Edit', 'hook', 'wordpoints'); ?></span>
+					<span class="add"><?php echo esc_html_x('Add', 'hook', 'wordpoints'); ?></span>
+					<span class="screen-reader-text"><?php echo esc_html(strip_tags($hook->get_name())); ?></span>
 				</a>
 			</div>
-			<div class="hook-title"><h3><?php echo esc_html( strip_tags( $hook->get_name() ) ) ?><span class="in-hook-title"></span></h3></div>
+			<div class="hook-title"><h3><?php echo esc_html(strip_tags($hook->get_name())) ?><span class="in-hook-title"></span></h3></div>
 		</div>
 
 		<div class="hook-inside">
@@ -732,26 +732,26 @@ 
 				<div class="hook-content">
 					<?php
 
-					$has_form = $hook->form_callback( $number );
+					$has_form = $hook->form_callback($number);
 
 					?>
 				</div>
 
-				<input type="hidden" name="hook-id" class="hook-id" value="<?php echo esc_attr( $id_format ); ?>" />
-				<input type="hidden" name="id_base" class="id_base" value="<?php echo esc_attr( $id_base ); ?>" />
-				<input type="hidden" name="hook-width" class="hook-width" value="<?php echo isset( $options['width'] ) ? esc_attr( $options['width'] ) : ''; ?>" />
-				<input type="hidden" name="hook-height" class="hook-height" value="<?php echo isset( $options['height'] ) ? esc_attr( $options['height'] ) : ''; ?>" />
-				<input type="hidden" name="hook_number" class="hook_number" value="<?php echo esc_attr( $number ); ?>" />
-				<input type="hidden" name="multi_number" class="multi_number" value="<?php echo esc_attr( $multi_number ); ?>" />
-				<input type="hidden" name="add_new" class="add_new" value="<?php echo esc_attr( $add_new ); ?>" />
+				<input type="hidden" name="hook-id" class="hook-id" value="<?php echo esc_attr($id_format); ?>" />
+				<input type="hidden" name="id_base" class="id_base" value="<?php echo esc_attr($id_base); ?>" />
+				<input type="hidden" name="hook-width" class="hook-width" value="<?php echo isset($options['width']) ? esc_attr($options['width']) : ''; ?>" />
+				<input type="hidden" name="hook-height" class="hook-height" value="<?php echo isset($options['height']) ? esc_attr($options['height']) : ''; ?>" />
+				<input type="hidden" name="hook_number" class="hook_number" value="<?php echo esc_attr($number); ?>" />
+				<input type="hidden" name="multi_number" class="multi_number" value="<?php echo esc_attr($multi_number); ?>" />
+				<input type="hidden" name="add_new" class="add_new" value="<?php echo esc_attr($add_new); ?>" />
 
 				<div class="hook-control-actions">
 					<div class="alignleft">
-						<a class="hook-control-remove" href="#remove"><?php esc_html_e( 'Delete', 'wordpoints' ); ?></a> |
-						<a class="hook-control-close" href="#close"><?php esc_html_e( 'Close', 'wordpoints' ); ?></a>
+						<a class="hook-control-remove" href="#remove"><?php esc_html_e('Delete', 'wordpoints'); ?></a> |
+						<a class="hook-control-close" href="#close"><?php esc_html_e('Close', 'wordpoints'); ?></a>
 					</div>
-					<div class="alignright<?php echo ( false === $has_form ) ? ' hook-control-noform' : ''; ?>">
-						<?php submit_button( __( 'Save', 'wordpoints' ), 'button-primary hook-control-save right', 'savehook', false, array( 'id' => "hook-{$id_format}-savehook" ) ); ?>
+					<div class="alignright<?php echo (false === $has_form) ? ' hook-control-noform' : ''; ?>">
+						<?php submit_button(__('Save', 'wordpoints'), 'button-primary hook-control-save right', 'savehook', false, array('id' => "hook-{$id_format}-savehook")); ?>
 						<span class="spinner"></span>
 					</div>
 					<br class="clear" />
@@ -760,8 +760,8 @@ 
 		</div>
 
 		<div class="hook-description">
-			<?php if ( ! empty( $options['description'] ) ) : ?>
-				<?php echo esc_html( $options['description'] ); ?>
+			<?php if ( ! empty($options['description'])) : ?>
+				<?php echo esc_html($options['description']); ?>
 			<?php endif; ?>
 		</div>
 		</div>
@@ -781,9 +781,9 @@ 
 	 *
 	 * @return int
 	 */
-	private static function _sort_name_callback( $a, $b ) {
+	private static function _sort_name_callback($a, $b) {
 
-		return strnatcasecmp( $a->get_name(), $b->get_name() );
+		return strnatcasecmp($a->get_name(), $b->get_name());
 	}
 
 } // class WordPoints_Points_Hooks

src/components/points/includes/class-wordpoints-points-logs-query.php

Spacing +85 added lines, -85 removed lines

@@ -27,15 +27,15 @@ 
 	 * @since 2.3.0
 	 */
 	protected $columns = array(
-		'id'          => array( 'format' => '%d', 'unsigned' => true ),
-		'user_id'     => array( 'format' => '%d', 'unsigned' => true ),
-		'log_type'    => array( 'format' => '%s' ),
-		'points'      => array( 'format' => '%d' ),
-		'points_type' => array( 'format' => '%s' ),
-		'text'        => array( 'format' => '%s' ),
-		'blog_id'     => array( 'format' => '%d', 'unsigned' => true ),
-		'site_id'     => array( 'format' => '%d', 'unsigned' => true ),
-		'date'        => array( 'format' => '%s', 'is_date' => true ),
+		'id'          => array('format' => '%d', 'unsigned' => true),
+		'user_id'     => array('format' => '%d', 'unsigned' => true),
+		'log_type'    => array('format' => '%s'),
+		'points'      => array('format' => '%d'),
+		'points_type' => array('format' => '%s'),
+		'text'        => array('format' => '%s'),
+		'blog_id'     => array('format' => '%d', 'unsigned' => true),
+		'site_id'     => array('format' => '%d', 'unsigned' => true),
+		'date'        => array('format' => '%s', 'is_date' => true),
 	);
 
 	/**
@@ -47,11 +47,11 @@ 
 	 * @since 2.3.0
 	 */
 	protected $deprecated_args = array(
-		'orderby'      => array( 'replacement' => 'order_by', 'version' => '2.3.0', 'class' => __CLASS__ ),
-		'user__in'     => array( 'replacement' => 'user_id__in', 'version' => '2.3.0', 'class' => __CLASS__ ),
-		'user__not_in' => array( 'replacement' => 'user_id__not_in', 'version' => '2.3.0', 'class' => __CLASS__ ),
-		'blog__in'     => array( 'replacement' => 'blog_id__in', 'version' => '2.3.0', 'class' => __CLASS__ ),
-		'blog__not_in' => array( 'replacement' => 'blog_id__not_in', 'version' => '2.3.0', 'class' => __CLASS__ ),
+		'orderby'      => array('replacement' => 'order_by', 'version' => '2.3.0', 'class' => __CLASS__),
+		'user__in'     => array('replacement' => 'user_id__in', 'version' => '2.3.0', 'class' => __CLASS__),
+		'user__not_in' => array('replacement' => 'user_id__not_in', 'version' => '2.3.0', 'class' => __CLASS__),
+		'blog__in'     => array('replacement' => 'blog_id__in', 'version' => '2.3.0', 'class' => __CLASS__),
+		'blog__not_in' => array('replacement' => 'blog_id__not_in', 'version' => '2.3.0', 'class' => __CLASS__),
 	);
 
 	//
@@ -177,7 +177,7 @@ 
 	 *        @type array        $meta_query          See WP_Meta_Query.
 	 * }
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 
 		global $wpdb;
 
@@ -191,21 +191,21 @@ 
 		$this->defaults['text__compare'] = 'LIKE';
 
 		// Back-compat for pre-2.3.0, in case an object or string is passed.
-		$args = wp_parse_args( $args );
-		$args = $this->convert_deprecated_arg_values( $args );
+		$args = wp_parse_args($args);
+		$args = $this->convert_deprecated_arg_values($args);
 
-		parent::__construct( $args );
+		parent::__construct($args);
 
-		if ( is_multisite() ) {
-			foreach ( array( 'blog', 'site' ) as $arg ) {
+		if (is_multisite()) {
+			foreach (array('blog', 'site') as $arg) {
 
 				if (
 					// Support passing these as null to override the defaults.
-					! array_key_exists( "{$arg}_id", $this->args )
-					&& ! isset( $this->args[ "{$arg}_id__in" ] )
-					&& ! isset( $this->args[ "{$arg}_id__not_in" ] )
+					! array_key_exists("{$arg}_id", $this->args)
+					&& ! isset($this->args["{$arg}_id__in"])
+					&& ! isset($this->args["{$arg}_id__not_in"])
 				) {
-					$this->args[ "{$arg}_id" ] = $wpdb->{"{$arg}id"};
+					$this->args["{$arg}_id"] = $wpdb->{"{$arg}id"};
 				}
 			}
 		}
@@ -220,31 +220,31 @@ 
 	 *
 	 * @return array $args The args, with any values converted as needed.
 	 */
-	protected function convert_deprecated_arg_values( $args ) {
+	protected function convert_deprecated_arg_values($args) {
 
 		// Back-compat for pre-2.3.0, when the fields arg supported 'all'.
-		if ( isset( $args['fields'] ) && 'all' === $args['fields'] ) {
+		if (isset($args['fields']) && 'all' === $args['fields']) {
 
 			_deprecated_argument(
 				__METHOD__
 				, '2.3.0'
-				, esc_html( "Passing 'fields' => 'all' is deprecated, just omit the 'fields' arg instead, since all fields returned by default." )
+				, esc_html("Passing 'fields' => 'all' is deprecated, just omit the 'fields' arg instead, since all fields returned by default.")
 			);
 
-			unset( $args['fields'] );
+			unset($args['fields']);
 		}
 
 		// Back-compat for pre-2.3.0, when the orderby arg supported 'none'.
-		if ( isset( $args['orderby'] ) && 'none' === $args['orderby'] ) {
+		if (isset($args['orderby']) && 'none' === $args['orderby']) {
 
 			_deprecated_argument(
 				__METHOD__
 				, '2.3.0'
-				, esc_html( "Passing 'orderby' => 'none' is deprecated, pass 'order_by' => null instead." )
+				, esc_html("Passing 'orderby' => 'none' is deprecated, pass 'order_by' => null instead.")
 			);
 
 			$args['order_by'] = null;
-			unset( $args['orderby'] );
+			unset($args['orderby']);
 
 			return $args;
 		}
@@ -256,11 +256,11 @@ 
 	 * @since 1.6.0
 	 * @since 2.3.0 Now returns $this.
 	 */
-	public function set_args( array $args ) {
+	public function set_args(array $args) {
 
 		$this->_cache_query_hash = null;
 
-		return parent::set_args( $this->convert_deprecated_arg_values( $args ) );
+		return parent::set_args($this->convert_deprecated_arg_values($args));
 	}
 
 	/**
@@ -279,18 +279,18 @@ 
 	 */
 	public function count() {
 
-		if ( $this->_is_cached_query ) {
-			$cache = $this->_cache_get( 'count' );
+		if ($this->_is_cached_query) {
+			$cache = $this->_cache_get('count');
 
-			if ( false !== $cache ) {
+			if (false !== $cache) {
 				return $cache;
 			}
 		}
 
 		$count = parent::count();
 
-		if ( $this->_is_cached_query ) {
-			$this->_cache_set( $count, 'count' );
+		if ($this->_is_cached_query) {
+			$this->_cache_set($count, 'count');
 		}
 
 		return $count;
@@ -308,23 +308,23 @@ 
 	 *
 	 * @return mixed The results of the query, or false on failure.
 	 */
-	public function get( $method = 'results' ) {
+	public function get($method = 'results') {
 
-		if ( $this->_is_cached_query ) {
-			$cache = $this->_cache_get( "get_{$method}" );
+		if ($this->_is_cached_query) {
+			$cache = $this->_cache_get("get_{$method}");
 
-			if ( false !== $cache ) {
+			if (false !== $cache) {
 				return $cache;
 			}
 		}
 
-		$result = parent::get( $method );
+		$result = parent::get($method);
 
-		if ( $this->_is_cached_query ) {
-			$this->_cache_set( $result, "get_{$method}" );
+		if ($this->_is_cached_query) {
+			$this->_cache_set($result, "get_{$method}");
 
-			if ( 'results' === $method || 'col' === $method ) {
-				$this->_cache_set( count( $result ), 'count' );
+			if ('results' === $method || 'col' === $method) {
+				$this->_cache_set(count($result), 'count');
 			}
 		}
 
@@ -351,20 +351,20 @@ 
 	 *
 	 * @return object[]|false The logs for this page, or false if $page or $per_page is invalid.
 	 */
-	public function get_page( $page, $per_page = 25 ) {
+	public function get_page($page, $per_page = 25) {
 
-		if ( ! wordpoints_posint( $page ) || ! wordpoints_posint( $per_page ) ) {
+		if ( ! wordpoints_posint($page) || ! wordpoints_posint($per_page)) {
 			return false;
 		}
 
-		$start = ( $page - 1 ) * $per_page;
+		$start = ($page - 1) * $per_page;
 
 		// First try the main cache.
-		if ( $this->_is_cached_query ) {
+		if ($this->_is_cached_query) {
 
-			$cache = $this->_cache_get( 'get_results' );
+			$cache = $this->_cache_get('get_results');
 
-			if ( false !== $cache ) {
+			if (false !== $cache) {
 				return array_slice(
 					$cache
 					, $start - $this->args['start']
@@ -378,18 +378,18 @@ 
 
 		$this->args['start'] += $start;
 
-		if ( ! empty( $this->args['limit'] ) ) {
+		if ( ! empty($this->args['limit'])) {
 			$this->args['limit'] -= $start;
 		}
 
-		if ( empty( $this->args['limit'] ) || $this->args['limit'] > $per_page ) {
+		if (empty($this->args['limit']) || $this->args['limit'] > $per_page) {
 			$this->args['limit'] = $per_page;
 		}
 
 		// Regenerate the query limit after changing the start and limit args.
 		$this->prepare_limit();
 
-		unset( $this->_cache_query_hash );
+		unset($this->_cache_query_hash);
 
 		$results = $this->get();
 
@@ -400,7 +400,7 @@ 
 		$this->limit = '';
 		$this->prepare_limit();
 
-		unset( $this->_cache_query_hash );
+		unset($this->_cache_query_hash);
 
 		return $results;
 	}
@@ -448,9 +448,9 @@ 
 	 * @param string $deprecated Deprecated; no longer used.
 	 * @param bool   $network    Whether this is a network-wide query.
 	 */
-	public function prime_cache( $key = 'default:%points_type%', $deprecated = null, $network = false ) {
+	public function prime_cache($key = 'default:%points_type%', $deprecated = null, $network = false) {
 
-		if ( ! is_null( $deprecated ) ) {
+		if ( ! is_null($deprecated)) {
 			_deprecated_argument(
 				__METHOD__
 				, '1.9.0'
@@ -466,13 +466,13 @@ 
 				'%user_id%',
 			)
 			, array(
-				isset( $this->args['points_type'] ) ? $this->args['points_type'] : '',
-				isset( $this->args['user_id'] ) ? $this->args['user_id'] : 0,
+				isset($this->args['points_type']) ? $this->args['points_type'] : '',
+				isset($this->args['user_id']) ? $this->args['user_id'] : 0,
 			)
 			, $key
 		);
 
-		if ( $network ) {
+		if ($network) {
 			$this->_cache_group = 'wordpoints_network_points_logs_query';
 		} else {
 			$this->_cache_group = 'wordpoints_points_logs_query';
@@ -488,9 +488,9 @@ 
 	 *
 	 * @return string The correct meta table ID column, if the key is wordpoints_points_log_.
 	 */
-	public function meta_query_meta_table_id_filter( $key ) {
+	public function meta_query_meta_table_id_filter($key) {
 
-		if ( 'wordpoints_points_log__id' === $key ) {
+		if ('wordpoints_points_log__id' === $key) {
 			$key = 'log_id';
 		}
 
@@ -506,9 +506,9 @@ 
 	 */
 	protected function prepare_meta_where() {
 
-		add_filter( 'sanitize_key', array( $this, 'meta_query_meta_table_id_filter' ) );
+		add_filter('sanitize_key', array($this, 'meta_query_meta_table_id_filter'));
 		parent::prepare_meta_where();
-		remove_filter( 'sanitize_key', array( $this, 'meta_query_meta_table_id_filter' ) );
+		remove_filter('sanitize_key', array($this, 'meta_query_meta_table_id_filter'));
 	}
 
 	//
@@ -525,29 +525,29 @@ 
 	 *
 	 * @return mixed Cached value, or false if none.
 	 */
-	private function _cache_get( $type = null ) {
+	private function _cache_get($type = null) {
 
-		$cache = wp_cache_get( $this->_cache_key, $this->_cache_group );
+		$cache = wp_cache_get($this->_cache_key, $this->_cache_group);
 
-		if ( ! is_array( $cache ) ) {
+		if ( ! is_array($cache)) {
 			return false;
 		}
 
 		$this->_calc_cache_query_hash();
 
-		if ( ! isset( $cache[ $this->_cache_query_hash ] ) ) {
+		if ( ! isset($cache[$this->_cache_query_hash])) {
 			return false;
 		}
 
-		if ( isset( $type ) ) {
-			if ( isset( $cache[ $this->_cache_query_hash ][ $type ] ) ) {
-				return $cache[ $this->_cache_query_hash ][ $type ];
+		if (isset($type)) {
+			if (isset($cache[$this->_cache_query_hash][$type])) {
+				return $cache[$this->_cache_query_hash][$type];
 			} else {
 				return false;
 			}
 		}
 
-		return $cache[ $this->_cache_query_hash ];
+		return $cache[$this->_cache_query_hash];
 	}
 
 	/**
@@ -559,26 +559,26 @@ 
 	 * @param string $type  Optionally specify a results type to cache. Default is
 	 *                      null, or all types.
 	 */
-	private function _cache_set( $value, $type = null ) {
+	private function _cache_set($value, $type = null) {
 
-		$cache = wp_cache_get( $this->_cache_key, $this->_cache_group );
+		$cache = wp_cache_get($this->_cache_key, $this->_cache_group);
 
 		$this->_calc_cache_query_hash();
 
 		if (
-			! isset( $cache[ $this->_cache_query_hash ] )
-			|| ! is_array( $cache[ $this->_cache_query_hash ] )
+			! isset($cache[$this->_cache_query_hash])
+			|| ! is_array($cache[$this->_cache_query_hash])
 		) {
-			$cache[ $this->_cache_query_hash ] = array();
+			$cache[$this->_cache_query_hash] = array();
 		}
 
-		if ( isset( $type ) ) {
-			$cache[ $this->_cache_query_hash ][ $type ] = $value;
+		if (isset($type)) {
+			$cache[$this->_cache_query_hash][$type] = $value;
 		} else {
-			$cache[ $this->_cache_query_hash ] = $value;
+			$cache[$this->_cache_query_hash] = $value;
 		}
 
-		wp_cache_set( $this->_cache_key, $cache, $this->_cache_group );
+		wp_cache_set($this->_cache_key, $cache, $this->_cache_group);
 	}
 
 	/**
@@ -588,8 +588,8 @@ 
 	 */
 	private function _calc_cache_query_hash() {
 
-		if ( ! isset( $this->_cache_query_hash ) ) {
-			$this->_cache_query_hash = wordpoints_hash( $this->get_sql() );
+		if ( ! isset($this->_cache_query_hash)) {
+			$this->_cache_query_hash = wordpoints_hash($this->get_sql());
 		}
 	}
 

src/components/points/classes/logs/viewing/restriction/read/comment/post.php

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
 	 * @since 2.2.0
 	 */
 	public function get_description() {
-		return __( 'This log entry is only visible to users who can view the post that the comment is on.', 'wordpoints' );
+		return __('This log entry is only visible to users who can view the post that the comment is on.', 'wordpoints');
 	}
 
 	/**
@@ -33,10 +33,10 @@ 
 			, true
 		);
 
-		if ( $comment_id ) {
-			$comment = get_comment( $comment_id );
+		if ($comment_id) {
+			$comment = get_comment($comment_id);
 
-			if ( $comment ) {
+			if ($comment) {
 				return $comment->comment_post_ID;
 			}
 		}

src/components/points/admin/screens/hooks.php

Spacing +45 added lines, -45 removed lines

@@ -12,57 +12,57 @@ 
 
 // These messages/errors are used upon redirection from the non-JS version.
 $messages = array(
-	__( 'Changes saved.', 'wordpoints' ),
+	__('Changes saved.', 'wordpoints'),
 );
 
 $errors = array(
-	__( 'Error while saving.', 'wordpoints' ),
-	__( 'Error in displaying the hooks settings form.', 'wordpoints' ),
+	__('Error while saving.', 'wordpoints'),
+	__('Error in displaying the hooks settings form.', 'wordpoints'),
 	'', // Back-compat for pre-2.1.0.
-	__( 'Error while deleting.', 'wordpoints' ),
+	__('Error while deleting.', 'wordpoints'),
 );
 
-if ( is_network_admin() ) {
-	$title = _x( 'Network Points Hooks', 'page title', 'wordpoints' );
+if (is_network_admin()) {
+	$title = _x('Network Points Hooks', 'page title', 'wordpoints');
 } else {
-	$title = _x( 'Points Hooks', 'page title', 'wordpoints' );
+	$title = _x('Points Hooks', 'page title', 'wordpoints');
 }
 
 ?>
 
 <div class="wrap">
-	<h1><?php echo esc_html( $title ); ?></h1>
+	<h1><?php echo esc_html($title); ?></h1>
 
 	<?php
 
-	if ( empty( $points_types ) && ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+	if (empty($points_types) && ! current_user_can('manage_wordpoints_points_types')) {
 
-		wordpoints_show_admin_error( esc_html__( 'No points types have been created yet. Only network administrators can create points types.', 'wordpoints' ) );
+		wordpoints_show_admin_error(esc_html__('No points types have been created yet. Only network administrators can create points types.', 'wordpoints'));
 
 		echo '</div>';
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) {
+	if (isset($_GET['message']) && isset($messages[(int) $_GET['message']])) {
 
 		wordpoints_show_admin_message(
-			esc_html( $messages[ (int) $_GET['message'] ] )
+			esc_html($messages[(int) $_GET['message']])
 			, 'success'
-			, array( 'dismissible' => true )
+			, array('dismissible' => true)
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) {
+	} elseif (isset($_GET['error']) && isset($errors[(int) $_GET['error']])) {
 
 		wordpoints_show_admin_error(
-			esc_html( $errors[ (int) $_GET['error'] ] )
-			, array( 'dismissible' => true )
+			esc_html($errors[(int) $_GET['error']])
+			, array('dismissible' => true)
 		);
 	}
 
-	if ( is_network_admin() && current_user_can( 'manage_network_wordpoints_points_hooks' ) ) {
+	if (is_network_admin() && current_user_can('manage_network_wordpoints_points_hooks')) {
 
 		// Display network wide hooks.
-		WordPoints_Points_Hooks::set_network_mode( true );
+		WordPoints_Points_Hooks::set_network_mode(true);
 	}
 
 	/**
@@ -70,7 +70,7 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_points_hooks_head' );
+	do_action('wordpoints_admin_points_hooks_head');
 
 	?>
 
@@ -79,10 +79,10 @@ 
 			<div id="available-hooks" class="hooks-holder-wrap hooks-holder-wrap">
 				<div class="points-type-name">
 					<div class="points-type-name-arrow"><br /></div>
-					<h2><?php esc_html_e( 'Available Hooks', 'wordpoints' ); ?> <span id="removing-hook"><?php echo esc_html_x( 'Deactivate', 'removing-hook', 'wordpoints' ); ?> <span></span></span></h2>
+					<h2><?php esc_html_e('Available Hooks', 'wordpoints'); ?> <span id="removing-hook"><?php echo esc_html_x('Deactivate', 'removing-hook', 'wordpoints'); ?> <span></span></span></h2>
 				</div>
 				<div class="hook-holder hook-holder">
-					<p class="description"><?php esc_html_e( 'Drag hooks from here to a points type on the right to activate them. Drag hooks back here to deactivate them and delete their settings.', 'wordpoints' ); ?></p>
+					<p class="description"><?php esc_html_e('Drag hooks from here to a points type on the right to activate them. Drag hooks back here to deactivate them and delete their settings.', 'wordpoints'); ?></p>
 					<div id="hook-list">
 						<?php WordPoints_Points_Hooks::list_hooks(); ?>
 					</div>
@@ -94,7 +94,7 @@ 
 			<div class="hooks-holder-wrap inactive-points-type">
 				<div class="points-type-name">
 					<div class="points-type-name-arrow"><br /></div>
-					<h2><?php esc_html_e( 'Inactive Hooks', 'wordpoints' ); ?>
+					<h2><?php esc_html_e('Inactive Hooks', 'wordpoints'); ?>
 						<span class="spinner"></span>
 					</h2>
 				</div>
@@ -102,10 +102,10 @@ 
 					<div id="_inactive_hooks" class="hooks-sortables">
 						<div class="points-type-description">
 							<p class="description">
-								<?php esc_html_e( 'Drag hooks here to remove them from the points type but keep their settings.', 'wordpoints' ); ?>
+								<?php esc_html_e('Drag hooks here to remove them from the points type but keep their settings.', 'wordpoints'); ?>
 							</p>
 						</div>
-						<?php WordPoints_Points_Hooks::list_by_points_type( '_inactive_hooks' ); ?>
+						<?php WordPoints_Points_Hooks::list_by_points_type('_inactive_hooks'); ?>
 					</div>
 					<div class="clear"></div>
 				</div>
@@ -121,31 +121,31 @@ 
 
 			$i = 0;
 
-			foreach ( $points_types as $slug => $points_type ) {
+			foreach ($points_types as $slug => $points_type) {
 
 				$wrap_class = 'hooks-holder-wrap';
-				if ( ! empty( $points_type['class'] ) ) {
+				if ( ! empty($points_type['class'])) {
 					$wrap_class .= ' points-type-' . $points_type['class'];
 				}
 
-				if ( $i ) {
+				if ($i) {
 					$wrap_class .= ' closed';
 				}
 
 				?>
 
-				<div class="<?php echo esc_attr( $wrap_class ); ?>">
+				<div class="<?php echo esc_attr($wrap_class); ?>">
 					<div class="points-type-name">
 						<div class="points-type-name-arrow"><br /></div>
-						<h2><?php echo esc_html( $points_type['name'] ); ?><span class="spinner"></span></h2>
+						<h2><?php echo esc_html($points_type['name']); ?><span class="spinner"></span></h2>
 					</div>
-					<div id="<?php echo esc_attr( $slug ); ?>" class="hooks-sortables">
+					<div id="<?php echo esc_attr($slug); ?>" class="hooks-sortables">
 
 						<?php
 
 						if (
-							get_site_option( 'wordpoints_disabled_points_hooks_edit_points_types' )
-							&& current_user_can( 'manage_wordpoints_points_types' )
+							get_site_option('wordpoints_disabled_points_hooks_edit_points_types')
+							&& current_user_can('manage_wordpoints_points_types')
 						) {
 							?>
 							<div class="notice notice-info inline notice-alt">
@@ -155,7 +155,7 @@ 
 									echo wp_kses_data(
 										sprintf(
 											// translators: URL of Points Types admin screen.
-											__( 'You can edit this points type&#8219;s settings on the <a href="%s">Points Types screen</a>.', 'wordpoints' )
+											__('You can edit this points type&#8219;s settings on the <a href="%s">Points Types screen</a>.', 'wordpoints')
 											, esc_attr(
 												esc_url(
 													self_admin_url(
@@ -173,7 +173,7 @@ 
 							<?php
 						}
 
-						WordPoints_Points_Hooks::list_by_points_type( $slug );
+						WordPoints_Points_Hooks::list_by_points_type($slug);
 
 						?>
 
@@ -187,16 +187,16 @@ 
 			} // End foreach ( $points_types ).
 
 			if (
-				get_site_option( 'wordpoints_disabled_points_hooks_edit_points_types' )
-				&& current_user_can( 'manage_wordpoints_points_types' )
+				get_site_option('wordpoints_disabled_points_hooks_edit_points_types')
+				&& current_user_can('manage_wordpoints_points_types')
 			) {
 
 				?>
 
-				<div class="hooks-holder-wrap new-points-type <?php echo ( $i > 0 ) ? 'closed' : ''; ?>">
+				<div class="hooks-holder-wrap new-points-type <?php echo ($i > 0) ? 'closed' : ''; ?>">
 					<div class="points-type-name">
 						<div class="points-type-name-arrow"><br /></div>
-						<h2><?php esc_html_e( 'Add New Points Type', 'wordpoints' ); ?><span class="spinner"></span></h2>
+						<h2><?php esc_html_e('Add New Points Type', 'wordpoints'); ?><span class="spinner"></span></h2>
 					</div>
 					<div class="wordpoints-points-add-new hooks-sortables hook">
 						<div class="notice notice-info inline notice-alt">
@@ -206,7 +206,7 @@ 
 								echo wp_kses_data(
 									sprintf(
 										// translators: URL of Points Types admin screen.
-										__( 'You can create new points types on the <a href="%s">Points Types screen</a>.', 'wordpoints' )
+										__('You can create new points types on the <a href="%s">Points Types screen</a>.', 'wordpoints')
 										, esc_attr(
 											esc_url(
 												self_admin_url(
@@ -234,13 +234,13 @@ 
 	<form method="post">
 		<?php
 
-		if ( WordPoints_Points_Hooks::get_network_mode() ) {
+		if (WordPoints_Points_Hooks::get_network_mode()) {
 			$field = 'save-network-wordpoints-points-hooks';
 		} else {
 			$field = 'save-wordpoints-points-hooks';
 		}
 
-		wp_nonce_field( $field, '_wpnonce_hooks', false );
+		wp_nonce_field($field, '_wpnonce_hooks', false);
 
 		?>
 	</form>
@@ -252,18 +252,18 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_points_hooks_foot' );
+	do_action('wordpoints_admin_points_hooks_foot');
 
 	?>
 
 	<br class="clear" />
 
 	<div class="hooks-chooser">
-		<h3><?php esc_html_e( 'Choose a points type:', 'wordpoints' ); ?></h3>
+		<h3><?php esc_html_e('Choose a points type:', 'wordpoints'); ?></h3>
 		<ul class="hooks-chooser-points-types"></ul>
 		<div class="hooks-chooser-actions">
-			<button class="button-secondary"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></button>
-			<button class="button-primary"><?php esc_html_e( 'Add Hook', 'wordpoints' ); ?></button>
+			<button class="button-secondary"><?php esc_html_e('Cancel', 'wordpoints'); ?></button>
+			<button class="button-primary"><?php esc_html_e('Add Hook', 'wordpoints'); ?></button>
 		</div>
 	</div>
 </div>