WordPoints / wordpoints

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

Spacing +131 added lines, -131 removed lines

@@ -78,7 +78,7 @@ 
 	 * @since 2.1.0
 	 */
 	protected function get_title() {
-		return _x( 'Points Types', 'page title', 'wordpoints' );
+		return _x('Points Types', 'page title', 'wordpoints');
 	}
 
 	/**
@@ -88,9 +88,9 @@ 
 
 		parent::hooks();
 
-		add_action( 'add_meta_boxes', array( $this, 'add_points_settings_meta_box' ) );
-		add_action( 'add_meta_boxes', array( $this, 'add_points_logs_meta_box' ) );
-		add_action( 'add_meta_boxes', array( $this, 'add_event_meta_boxes' ) );
+		add_action('add_meta_boxes', array($this, 'add_points_settings_meta_box'));
+		add_action('add_meta_boxes', array($this, 'add_points_logs_meta_box'));
+		add_action('add_meta_boxes', array($this, 'add_event_meta_boxes'));
 	}
 
 	/**
@@ -98,10 +98,10 @@ 
 	 */
 	public function enqueue_scripts() {
 
-		wp_enqueue_style( 'wordpoints-hooks-admin' );
+		wp_enqueue_style('wordpoints-hooks-admin');
 
-		wp_enqueue_script( 'postbox' );
-		wp_enqueue_script( 'wordpoints-admin-points-types' );
+		wp_enqueue_script('postbox');
+		wp_enqueue_script('wordpoints-admin-points-types');
 
 		wordpoints_hooks_ui_setup_script_data();
 	}
@@ -121,7 +121,7 @@ 
 					.addClass( 'closed' );
 
 				postboxes.add_postbox_toggles(
-					<?php echo wp_json_encode( $this->id ); ?>
+					<?php echo wp_json_encode($this->id); ?>
 				);
 			} );
 		</script>
@@ -136,14 +136,14 @@ 
 	 */
 	public function add_points_settings_meta_box() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
 		add_meta_box(
 			'settings'
-			, __( 'Settings', 'wordpoints' )
-			, array( $this, 'display_points_settings_meta_box' )
+			, __('Settings', 'wordpoints')
+			, array($this, 'display_points_settings_meta_box')
 			, $this->id
 			, 'side'
 			, 'default'
@@ -157,14 +157,14 @@ 
 	 */
 	public function add_points_logs_meta_box() {
 
-		if ( ! $this->current_points_type ) {
+		if ( ! $this->current_points_type) {
 			return;
 		}
 
 		add_meta_box(
 			'logs'
-			, __( 'Logs', 'wordpoints' )
-			, array( $this, 'display_points_logs_meta_box' )
+			, __('Logs', 'wordpoints')
+			, array($this, 'display_points_logs_meta_box')
 			, $this->id
 			, 'side'
 			, 'default'
@@ -178,7 +178,7 @@ 
 	 */
 	public function display_points_settings_meta_box() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
@@ -186,12 +186,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(
@@ -206,27 +206,27 @@ 
 		?>
 
 		<form method="post">
-			<?php if ( is_wordpoints_network_active() && ! is_network_admin() ) : ?>
+			<?php if (is_wordpoints_network_active() && ! is_network_admin()) : ?>
 				<div class="notice notice-info inline">
 					<p>
-						<?php if ( $slug ) : ?>
-							<?php esc_html_e( 'Changes to this points type&#8217;s settings will affect all sites on this network.', 'wordpoints' ) ?>
+						<?php if ($slug) : ?>
+							<?php esc_html_e('Changes to this points type&#8217;s settings will affect all sites on this network.', 'wordpoints') ?>
 						<?php else : ?>
-							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' ) ?>
+							<?php esc_html_e('The new points type will be global across all sites on this network.', 'wordpoints') ?>
 						<?php endif; ?>
 					</p>
 				</div>
 			<?php endif; ?>
 
-			<?php if ( $slug ) : ?>
+			<?php if ($slug) : ?>
 				<p>
 					<span class="wordpoints-points-slug">
 						<em>
-							<?php echo esc_html( sprintf( __( 'Slug: %s', 'wordpoints' ), $slug ) ); ?>
+							<?php echo esc_html(sprintf(__('Slug: %s', 'wordpoints'), $slug)); ?>
 						</em>
 					</span>
 				</p>
-				<?php wp_nonce_field( "wordpoints_update_points_type-$slug", 'update_points_type' ); ?>
+				<?php wp_nonce_field("wordpoints_update_points_type-$slug", 'update_points_type'); ?>
 			<?php endif; ?>
 
 			<?php
@@ -240,45 +240,45 @@ 
 			 *
 			 * @param string $points_type The slug of the points type.
 			 */
-			do_action( 'wordpoints_points_type_form_top', $slug );
+			do_action('wordpoints_points_type_form_top', $slug);
 
-			if ( $add_new ) {
+			if ($add_new) {
 
 				// Mark the prefix and suffix optional on the add new form.
-				$prefix = _x( 'Prefix (optional):', 'points type', 'wordpoints' );
-				$suffix = _x( 'Suffix (optional):', 'points type', 'wordpoints' );
+				$prefix = _x('Prefix (optional):', 'points type', 'wordpoints');
+				$suffix = _x('Suffix (optional):', 'points type', 'wordpoints');
 
 			} else {
 
-				$prefix = _x( 'Prefix:', 'points type', 'wordpoints' );
-				$suffix = _x( 'Suffix:', 'points type', 'wordpoints' );
+				$prefix = _x('Prefix:', 'points type', 'wordpoints');
+				$suffix = _x('Suffix:', 'points type', 'wordpoints');
 			}
 
 			?>
 
 			<p>
 				<label
-					for="points-name-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html_x( 'Name:', 'points type', 'wordpoints' ); ?></label>
+					for="points-name-<?php echo esc_attr($slug); ?>"><?php echo esc_html_x('Name:', 'points type', 'wordpoints'); ?></label>
 				<input class="widefat" type="text"
-				       id="points-name-<?php echo esc_attr( $slug ); ?>"
+				       id="points-name-<?php echo esc_attr($slug); ?>"
 				       name="points-name"
-				       value="<?php echo esc_attr( $points_type['name'] ); ?>"/>
+				       value="<?php echo esc_attr($points_type['name']); ?>"/>
 			</p>
 			<p>
 				<label
-					for="points-prefix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $prefix ); ?></label>
+					for="points-prefix-<?php echo esc_attr($slug); ?>"><?php echo esc_html($prefix); ?></label>
 				<input class="widefat" type="text"
-				       id="points-prefix-<?php echo esc_attr( $slug ); ?>"
+				       id="points-prefix-<?php echo esc_attr($slug); ?>"
 				       name="points-prefix"
-				       value="<?php echo esc_attr( $points_type['prefix'] ); ?>"/>
+				       value="<?php echo esc_attr($points_type['prefix']); ?>"/>
 			</p>
 			<p>
 				<label
-					for="points-suffix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $suffix ); ?></label>
+					for="points-suffix-<?php echo esc_attr($slug); ?>"><?php echo esc_html($suffix); ?></label>
 				<input class="widefat" type="text"
-				       id="points-suffix-<?php echo esc_attr( $slug ); ?>"
+				       id="points-suffix-<?php echo esc_attr($slug); ?>"
 				       name="points-suffix"
-				       value="<?php echo esc_attr( $points_type['suffix'] ); ?>"/>
+				       value="<?php echo esc_attr($points_type['suffix']); ?>"/>
 			</p>
 
 			<?php
@@ -292,28 +292,28 @@ 
 			 *
 			 * @param string $points_type The slug of the points type.
 			 */
-			do_action( 'wordpoints_points_type_form_bottom', $slug );
+			do_action('wordpoints_points_type_form_bottom', $slug);
 
 			?>
 
 			<input type="hidden" name="points-slug"
-			       value="<?php echo esc_attr( $slug ); ?>"/>
+			       value="<?php echo esc_attr($slug); ?>"/>
 			<input type="hidden" name="add_new" class="add_new"
-			       value="<?php echo esc_attr( $add_new ); ?>"/>
+			       value="<?php echo esc_attr($add_new); ?>"/>
 
 			<div class="hook-control-actions">
 				<div class="alignleft">
 					<?php
 
-					if ( ! $add_new ) {
-						wp_nonce_field( "wordpoints_delete_points_type-{$slug}", 'delete-points-type-nonce' );
-						submit_button( _x( 'Delete', 'points type', 'wordpoints' ), 'delete', 'delete-points-type', false, array( 'id' => "delete_points_type-{$slug}" ) );
+					if ( ! $add_new) {
+						wp_nonce_field("wordpoints_delete_points_type-{$slug}", 'delete-points-type-nonce');
+						submit_button(_x('Delete', 'points type', 'wordpoints'), 'delete', 'delete-points-type', false, array('id' => "delete_points_type-{$slug}"));
 					}
 
 					?>
 				</div>
 				<div class="alignright">
-					<?php submit_button( _x( 'Save', 'points type', 'wordpoints' ), 'button-primary hook-control-save right', 'save-points-type', false, array( 'id' => "points-{$slug}-save" ) ); ?>
+					<?php submit_button(_x('Save', 'points type', 'wordpoints'), 'button-primary hook-control-save right', 'save-points-type', false, array('id' => "points-{$slug}-save")); ?>
 					<span class="spinner"></span>
 				</div>
 				<br class="clear"/>
@@ -332,8 +332,8 @@ 
 
 		?>
 
-		<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_logs&tab=' . $this->current_points_type ) ); ?>">
-			<?php esc_html_e( 'Go to the logs for this points type.', 'wordpoints' ); ?>
+		<a href="<?php echo esc_url(self_admin_url('admin.php?page=wordpoints_points_logs&tab=' . $this->current_points_type)); ?>">
+			<?php esc_html_e('Go to the logs for this points type.', 'wordpoints'); ?>
 		</a>
 
 		<?php
@@ -346,22 +346,22 @@ 
 	 */
 	public function add_event_meta_boxes() {
 
-		if ( ! $this->current_points_type ) {
+		if ( ! $this->current_points_type) {
 			return;
 		}
 
-		$events_app       = $this->hooks->get_sub_app( 'events' );
-		$this->event_args = $events_app->get_sub_app( 'args' );
+		$events_app       = $this->hooks->get_sub_app('events');
+		$this->event_args = $events_app->get_sub_app('args');
 
-		$this->reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+		$this->reaction_store = wordpoints_hooks()->get_reaction_store('points');
 
-		if ( ! $this->reaction_store ) {
+		if ( ! $this->reaction_store) {
 			return;
 		}
 
 		/** @var WordPoints_Hook_ReactorI $reactor */
-		$reactor = $this->hooks->get_sub_app( 'reactors' )->get( 'points' );
-		$reactor_action_types = array_fill_keys( $reactor->get_action_types(), true );
+		$reactor = $this->hooks->get_sub_app('reactors')->get('points');
+		$reactor_action_types = array_fill_keys($reactor->get_action_types(), true);
 
 		$event_action_types = wordpoints_hooks_ui_get_script_data_event_action_types();
 
@@ -386,11 +386,11 @@ 
 			, $this->current_points_type
 		);
 
-		foreach ( $events as $slug => $event ) {
+		foreach ($events as $slug => $event) {
 
 			if (
 				! array_intersect_key(
-					$event_action_types[ $slug ]
+					$event_action_types[$slug]
 					, $reactor_action_types
 				)
 			) {
@@ -400,7 +400,7 @@ 
 			add_meta_box(
 				"{$this->current_points_type}-{$slug}"
 				, $event->get_title()
-				, array( $this, 'display_event_meta_box' )
+				, array($this, 'display_event_meta_box')
 				, $this->id
 				, 'events'
 				, 'default'
@@ -421,50 +421,50 @@ 
 	 * @param array $points_type The points type this meta-box relates to.
 	 * @param array $meta_box    The data the meta-box was created with.
 	 */
-	public function display_event_meta_box( $points_type, $meta_box ) {
+	public function display_event_meta_box($points_type, $meta_box) {
 
 		$event_slug = $meta_box['args']['slug'];
 
 		$data = array();
 
-		foreach ( $this->reaction_store->get_reactions_to_event( $event_slug ) as $id => $reaction ) {
-			if ( $reaction->get_meta( 'points_type' ) === $this->current_points_type ) {
+		foreach ($this->reaction_store->get_reactions_to_event($event_slug) as $id => $reaction) {
+			if ($reaction->get_meta('points_type') === $this->current_points_type) {
 				$data[] = WordPoints_Admin_Ajax_Hooks::prepare_hook_reaction(
 					$reaction
 				);
 			}
 		}
 
-		$event_data = array( 'args' => array() );
+		$event_data = array('args' => array());
 
-		foreach ( $this->event_args->get_children( $event_slug ) as $slug => $arg ) {
+		foreach ($this->event_args->get_children($event_slug) as $slug => $arg) {
 
-			$event_data['args'][ $slug ] = array(
+			$event_data['args'][$slug] = array(
 				'slug' => $slug,
 			);
 
-			if ( $arg instanceof WordPoints_Hook_ArgI ) {
-				$event_data['args'][ $slug ]['title'] = $arg->get_title();
-				$event_data['args'][ $slug ]['is_stateful'] = $arg->is_stateful();
+			if ($arg instanceof WordPoints_Hook_ArgI) {
+				$event_data['args'][$slug]['title'] = $arg->get_title();
+				$event_data['args'][$slug]['is_stateful'] = $arg->is_stateful();
 			}
 		}
 
 		?>
 
 		<script>
-			WordPointsHooksAdminData.events[<?php echo wp_json_encode( $event_slug ); ?>] = <?php echo wp_json_encode( $event_data ); ?>;
-			WordPointsHooksAdminData.reactions[<?php echo wp_json_encode( $event_slug ); ?>] = <?php echo wp_json_encode( $data ); ?>;
+			WordPointsHooksAdminData.events[<?php echo wp_json_encode($event_slug); ?>] = <?php echo wp_json_encode($event_data); ?>;
+			WordPointsHooksAdminData.reactions[<?php echo wp_json_encode($event_slug); ?>] = <?php echo wp_json_encode($data); ?>;
 		</script>
 
 		<div class="wordpoints-hook-reaction-group-container">
 			<p class="description wordpoints-hook-reaction-group-description">
-				<?php echo esc_html( $meta_box['args']['event']->get_description() ); ?>
+				<?php echo esc_html($meta_box['args']['event']->get_description()); ?>
 			</p>
 
 			<div class="wordpoints-hook-reaction-group"
-				data-wordpoints-hooks-hook-event="<?php echo esc_attr( $event_slug ); ?>"
-				data-wordpoints-hooks-points-type="<?php echo esc_attr( $this->current_points_type ); ?>"
-				data-wordpoints-hooks-create-nonce="<?php echo esc_attr( WordPoints_Admin_Ajax_Hooks::get_create_nonce( $this->reaction_store ) ); ?>"
+				data-wordpoints-hooks-hook-event="<?php echo esc_attr($event_slug); ?>"
+				data-wordpoints-hooks-points-type="<?php echo esc_attr($this->current_points_type); ?>"
+				data-wordpoints-hooks-create-nonce="<?php echo esc_attr(WordPoints_Admin_Ajax_Hooks::get_create_nonce($this->reaction_store)); ?>"
 				data-wordpoints-hooks-reaction-store="points"
 				data-wordpoints-hooks-reactor="points">
 			</div>
@@ -479,7 +479,7 @@ 
 
 			<div class="controls">
 				<button type="button" class="button-primary add-reaction">
-					<?php esc_html_e( 'Add New Reaction', 'wordpoints' ); ?>
+					<?php esc_html_e('Add New Reaction', 'wordpoints'); ?>
 				</button>
 			</div>
 		</div>
@@ -499,19 +499,19 @@ 
 		// Show a tab for each points type.
 		$tabs = array();
 
-		foreach ( $points_types as $slug => $settings ) {
-			$tabs[ $slug ] = $settings['name'];
+		foreach ($points_types as $slug => $settings) {
+			$tabs[$slug] = $settings['name'];
 		}
 
-		$tabs['add-new'] = __( 'Add New', 'wordpoints' );
+		$tabs['add-new'] = __('Add New', 'wordpoints');
 
-		$tab = wordpoints_admin_get_current_tab( $tabs );
+		$tab = wordpoints_admin_get_current_tab($tabs);
 
-		if ( 'add-new' !== $tab ) {
+		if ('add-new' !== $tab) {
 			$this->current_points_type = $tab;
 		}
 
-		do_action( 'add_meta_boxes', $this->id );
+		do_action('add_meta_boxes', $this->id);
 
 		$this->tabs = $tabs;
 	}
@@ -523,19 +523,19 @@ 
 	 */
 	public function save_points_type() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
-		if ( isset( $_POST['save-points-type'] ) ) { // WPCS: CSRF OK
+		if (isset($_POST['save-points-type'])) { // WPCS: CSRF OK
 
-			if ( ! empty( $_POST['add_new'] ) ) { // WPCS: CSRF OK
+			if ( ! empty($_POST['add_new'])) { // WPCS: CSRF OK
 				$this->add_points_type();
 			} else {
 				$this->update_points_type();
 			}
 
-		} elseif ( ! empty( $_POST['delete-points-type'] ) ) { // WPCS: CSRF OK
+		} elseif ( ! empty($_POST['delete-points-type'])) { // WPCS: CSRF OK
 
 			$this->delete_points_type();
 		}
@@ -552,21 +552,21 @@ 
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) { // WPCS: CSRF OK
+		if (isset($_POST['points-name'])) { // WPCS: CSRF OK
 			$settings['name'] = trim(
-				sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) // WPCS: CSRF OK
+				sanitize_text_field(wp_unslash($_POST['points-name'])) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) { // WPCS: CSRF OK
+		if (isset($_POST['points-prefix'])) { // WPCS: CSRF OK
 			$settings['prefix'] = ltrim(
-				sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) // WPCS: CSRF OK
+				sanitize_text_field(wp_unslash($_POST['points-prefix'])) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) { // WPCS: CSRF OK
+		if (isset($_POST['points-suffix'])) { // WPCS: CSRF OK
 			$settings['suffix'] = rtrim(
-				sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) // WPCS: CSRF OK
+				sanitize_text_field(wp_unslash($_POST['points-suffix'])) // WPCS: CSRF OK
 			);
 		}
 
@@ -584,52 +584,52 @@ 
 			! wordpoints_verify_nonce(
 				'update_points_type'
 				, 'wordpoints_update_points_type-%s'
-				, array( 'points-slug' )
+				, array('points-slug')
 				, 'post'
 			)
-			|| ! isset( $_POST['points-slug'] )
+			|| ! isset($_POST['points-slug'])
 		) {
 			return;
 		}
 
 		$settings = $this->get_points_type_settings();
 
-		if ( empty( $settings['name'] ) ) {
+		if (empty($settings['name'])) {
 
 			add_settings_error(
 				'points-name'
 				, 'wordpoints_points_type_update'
-				, __( 'Error: points type name cannot be empty.', 'wordpoints' )
+				, __('Error: points type name cannot be empty.', 'wordpoints')
 			);
 
 			return;
 		}
 
-		$points_type = sanitize_key( $_POST['points-slug'] );
+		$points_type = sanitize_key($_POST['points-slug']);
 
-		$old_settings = wordpoints_get_points_type( $points_type );
+		$old_settings = wordpoints_get_points_type($points_type);
 
-		if ( false === $old_settings ) {
+		if (false === $old_settings) {
 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_update'
-				, __( 'Error: failed updating points type.', 'wordpoints' )
+				, __('Error: failed updating points type.', 'wordpoints')
 			);
 
 			return;
 		}
 
-		if ( is_array( $old_settings ) ) {
-			$settings = array_merge( $old_settings, $settings );
+		if (is_array($old_settings)) {
+			$settings = array_merge($old_settings, $settings);
 		}
 
-		if ( ! wordpoints_update_points_type( $points_type, $settings ) ) {
+		if ( ! wordpoints_update_points_type($points_type, $settings)) {
 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_update'
-				, __( 'Error: failed updating points type.', 'wordpoints' )
+				, __('Error: failed updating points type.', 'wordpoints')
 			);
 
 		} else {
@@ -637,7 +637,7 @@ 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_update'
-				, __( 'Points type updated.', 'wordpoints' )
+				, __('Points type updated.', 'wordpoints')
 				, 'updated'
 			);
 		}
@@ -663,14 +663,14 @@ 
 
 		$settings = $this->get_points_type_settings();
 
-		$slug = wordpoints_add_points_type( $settings );
+		$slug = wordpoints_add_points_type($settings);
 
-		if ( ! $slug ) {
+		if ( ! $slug) {
 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_create'
-				, __( 'Please choose a unique name for this points type.', 'wordpoints' )
+				, __('Please choose a unique name for this points type.', 'wordpoints')
 			);
 
 		} else {
@@ -680,7 +680,7 @@ 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_create'
-				, __( 'Points type created.', 'wordpoints' )
+				, __('Points type created.', 'wordpoints')
 				, 'updated'
 			);
 		}
@@ -697,22 +697,22 @@ 
 			wordpoints_verify_nonce(
 				'delete-points-type-nonce'
 				, 'wordpoints_delete_points_type-%s'
-				, array( 'points-slug' )
+				, array('points-slug')
 				, 'post'
 			)
-			&& isset( $_POST['points-slug'] )
+			&& isset($_POST['points-slug'])
 		) {
 
 			if (
 				wordpoints_delete_points_type(
-					sanitize_key( $_POST['points-slug'] )
+					sanitize_key($_POST['points-slug'])
 				)
 			) {
 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Points type deleted.', 'wordpoints' )
+					, __('Points type deleted.', 'wordpoints')
 					, 'updated'
 				);
 
@@ -721,7 +721,7 @@ 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Error while deleting.', 'wordpoints' )
+					, __('Error while deleting.', 'wordpoints')
 				);
 			}
 		}
@@ -737,18 +737,18 @@ 
 		 *
 		 * @since 2.1.0
 		 */
-		do_action( 'wordpoints_admin_points_events_head' );
+		do_action('wordpoints_admin_points_events_head');
 
-		if ( is_network_admin() ) {
-			$title = __( 'Network Events', 'wordpoints' );
-			$description = __( 'Add reactions to these events to award points whenever they take place on this network.', 'wordpoints' );
+		if (is_network_admin()) {
+			$title = __('Network Events', 'wordpoints');
+			$description = __('Add reactions to these events to award points whenever they take place on this network.', 'wordpoints');
 		} else {
-			$title = __( 'Events', 'wordpoints' );
-			$description = __( 'Add reactions to these events to award points whenever they take place on this site.', 'wordpoints' );
+			$title = __('Events', 'wordpoints');
+			$description = __('Add reactions to these events to award points whenever they take place on this site.', 'wordpoints');
 		}
 
-		if ( isset( $this->current_points_type ) ) {
-			$points_type = wordpoints_get_points_type( $this->current_points_type );
+		if (isset($this->current_points_type)) {
+			$points_type = wordpoints_get_points_type($this->current_points_type);
 			$points_type['slug'] = $this->current_points_type;
 		} else {
 			$points_type = false;
@@ -759,8 +759,8 @@ 
 		<div class="wordpoints-points-type-meta-box-wrap">
 
 				<form>
-					<?php wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false ); ?>
-					<?php wp_nonce_field( 'meta-box-order', 'meta-box-order-nonce', false ); ?>
+					<?php wp_nonce_field('closedpostboxes', 'closedpostboxesnonce', false); ?>
+					<?php wp_nonce_field('meta-box-order', 'meta-box-order-nonce', false); ?>
 				</form>
 
 				<div id="poststuff">
@@ -768,22 +768,22 @@ 
 					<div id="post-body" class="metabox-holder columns-<?php echo 1 === (int) get_current_screen()->get_columns() ? '1' : '2'; ?>">
 
 						<div id="postbox-container-1" class="postbox-container">
-							<?php do_meta_boxes( $this->id, 'side', $points_type ); ?>
+							<?php do_meta_boxes($this->id, 'side', $points_type); ?>
 						</div>
 
-						<?php if ( isset( $this->current_points_type ) ) : ?>
+						<?php if (isset($this->current_points_type)) : ?>
 							<div class="wordpoints-hook-events-heading">
-								<h2><?php echo esc_html( $title ); ?></h2>
+								<h2><?php echo esc_html($title); ?></h2>
 								<p class="description">
-									<?php echo esc_html( $description ); ?>
+									<?php echo esc_html($description); ?>
 									<?php
 
 									echo wp_kses(
 										sprintf(
-											__( 'You can learn more about how they work from <a href="%s">the user guide on WordPoints.org</a>.', 'wordpoints' )
+											__('You can learn more about how they work from <a href="%s">the user guide on WordPoints.org</a>.', 'wordpoints')
 											, 'https://wordpoints.org/user-guide/points-reactions/'
 										)
-										, array( 'a' => array( 'href' => true, 'target' => true ) )
+										, array('a' => array('href' => true, 'target' => true))
 									);
 
 									?>
@@ -791,7 +791,7 @@ 
 							</div>
 
 							<div id="postbox-container-2" class="postbox-container">
-								<?php do_meta_boxes( $this->id, 'events', $points_type ); ?>
+								<?php do_meta_boxes($this->id, 'events', $points_type); ?>
 							</div>
 						<?php endif; ?>
 
@@ -810,7 +810,7 @@ 
 		 *
 		 * @since 2.1.0
 		 */
-		do_action( 'wordpoints_admin_points_events_foot' );
+		do_action('wordpoints_admin_points_events_foot');
 	}
 }
 

Braces +5 added lines, -2 removed lines

@@ -211,9 +211,12 @@
 					<p>
 						<?php if ( $slug ) : ?>
 							<?php esc_html_e( 'Changes to this points type&#8217;s settings will affect all sites on this network.', 'wordpoints' ) ?>
-						<?php else : ?>
+						<?php else {
+	: ?>
 							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' ) ?>
-						<?php endif; ?>
+						<?php endif;
+}
+?>
 					</p>
 				</div>
 			<?php endif; ?>

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

Spacing +23 added lines, -23 removed lines

@@ -13,58 +13,58 @@
 $screen->add_help_tab(
 	array(
 		'id'      => 'overview',
-		'title'   => esc_html__( 'Overview', 'wordpoints' ),
+		'title'   => esc_html__('Overview', 'wordpoints'),
 		'content' =>
-			'<p>' . esc_html__( 'Points Hooks let you award users points by &#8220;hooking into&#8221; different actions. They can be hooked to any points type that you have created.', 'wordpoints' ) . '</p>
-			<p>' . esc_html__( 'To link a hook to a points type, click on the hook&#8217;s title bar and select a points type, or drag and drop the hook title bars into the desired points type. By default, only the first points type area is expanded. To populate additional points types, click on their title bars to expand them.', 'wordpoints' ) . '</p>
-			<p>' . esc_html__( 'The Available Hooks section contains all the hooks you can choose from. Once you add a hook into a points type, it will open to allow you to configure its settings. When you are happy with the hook settings, click the Save button and the hook will begin awarding points. If you click Delete, it will remove the hook.', 'wordpoints' ) . '</p>',
+			'<p>' . esc_html__('Points Hooks let you award users points by &#8220;hooking into&#8221; different actions. They can be hooked to any points type that you have created.', 'wordpoints') . '</p>
+			<p>' . esc_html__('To link a hook to a points type, click on the hook&#8217;s title bar and select a points type, or drag and drop the hook title bars into the desired points type. By default, only the first points type area is expanded. To populate additional points types, click on their title bars to expand them.', 'wordpoints') . '</p>
+			<p>' . esc_html__('The Available Hooks section contains all the hooks you can choose from. Once you add a hook into a points type, it will open to allow you to configure its settings. When you are happy with the hook settings, click the Save button and the hook will begin awarding points. If you click Delete, it will remove the hook.', 'wordpoints') . '</p>',
 	)
 );
 
 $screen->add_help_tab(
 	array(
 		'id'      => 'removing-reusing',
-		'title'   => esc_html__( 'Removing and Reusing', 'wordpoints' ),
+		'title'   => esc_html__('Removing and Reusing', 'wordpoints'),
 		'content' =>
-			'<p>' . esc_html__( 'If you want to remove the hook but save its setting for possible future use, just drag it into the Inactive Hooks area. You can add them back anytime from there.', 'wordpoints' ) . '</p>
-			<p>' . esc_html__( 'Hooks may be used multiple times.', 'wordpoints' ) . '</p>
-			<p>' . esc_html__( 'Enabling Accessibility Mode, via Screen Options, allows you to use Add and Edit buttons instead of using drag and drop.', 'wordpoints' ) . '</p>',
+			'<p>' . esc_html__('If you want to remove the hook but save its setting for possible future use, just drag it into the Inactive Hooks area. You can add them back anytime from there.', 'wordpoints') . '</p>
+			<p>' . esc_html__('Hooks may be used multiple times.', 'wordpoints') . '</p>
+			<p>' . esc_html__('Enabling Accessibility Mode, via Screen Options, allows you to use Add and Edit buttons instead of using drag and drop.', 'wordpoints') . '</p>',
 	)
 );
 
 $screen->set_help_sidebar(
-	'<p><strong>' . esc_html__( 'For more information:', 'wordpoints' ) . '</strong></p>
-	<p><a href="https://wordpoints.org/user-guide/points-hooks/">' . esc_html__( 'Documentation on Points Hooks', 'wordpoints' ) . '</a></p>
-	<p><a href="https://wordpress.org/support/plugin/wordpoints">' . esc_html__( 'Support Forums', 'wordpoints' ) . '</a></p>'
+	'<p><strong>' . esc_html__('For more information:', 'wordpoints') . '</strong></p>
+	<p><a href="https://wordpoints.org/user-guide/points-hooks/">' . esc_html__('Documentation on Points Hooks', 'wordpoints') . '</a></p>
+	<p><a href="https://wordpress.org/support/plugin/wordpoints">' . esc_html__('Support Forums', 'wordpoints') . '</a></p>'
 );
 
-$accessibility_mode = get_user_setting( 'wordpoints_points_hooks_access' );
+$accessibility_mode = get_user_setting('wordpoints_points_hooks_access');
 
 if (
-	isset( $_GET['accessibility-mode'], $_GET['wordpoints-accessiblity-nonce'] )
-	&& wordpoints_verify_nonce( 'wordpoints-accessiblity-nonce', 'wordpoints_points_hooks_accessiblity' )
+	isset($_GET['accessibility-mode'], $_GET['wordpoints-accessiblity-nonce'])
+	&& wordpoints_verify_nonce('wordpoints-accessiblity-nonce', 'wordpoints_points_hooks_accessiblity')
 ) {
 
-	$accessibility_mode = ( 'on' === sanitize_key( $_GET['accessibility-mode'] ) ) ? 'on' : 'off';
-	set_user_setting( 'wordpoints_points_hooks_access', $accessibility_mode );
+	$accessibility_mode = ('on' === sanitize_key($_GET['accessibility-mode'])) ? 'on' : 'off';
+	set_user_setting('wordpoints_points_hooks_access', $accessibility_mode);
 }
 
 // Enqueue needed scripts and styles.
-if ( 'on' === $accessibility_mode ) {
+if ('on' === $accessibility_mode) {
 
-	add_filter( 'admin_body_class', 'wordpoints_points_hooks_access_body_class' );
+	add_filter('admin_body_class', 'wordpoints_points_hooks_access_body_class');
 
 } else {
 
-	wp_enqueue_style( 'wp-jquery-ui-dialog' );
+	wp_enqueue_style('wp-jquery-ui-dialog');
 
-	wp_enqueue_script( 'wordpoints-admin-points-hooks' );
+	wp_enqueue_script('wordpoints-admin-points-hooks');
 
-	if ( wp_is_mobile() ) {
-		wp_enqueue_script( 'jquery-touch-punch' );
+	if (wp_is_mobile()) {
+		wp_enqueue_script('jquery-touch-punch');
 	}
 }
 
-wp_enqueue_style( 'wordpoints-admin-points-hooks' );
+wp_enqueue_style('wordpoints-admin-points-hooks');
 
 // EOF

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

Spacing +43 added lines, -43 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">
@@ -175,7 +175,7 @@ 
 							<?php
 						}
 
-						WordPoints_Points_Hooks::list_by_points_type( $slug );
+						WordPoints_Points_Hooks::list_by_points_type($slug);
 
 						?>
 
@@ -189,16 +189,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">
@@ -238,13 +238,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>
@@ -256,18 +256,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>

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

Spacing +18 added lines, -18 removed lines

@@ -7,17 +7,17 @@ 
  * @since 1.0.0
  */
 
-if ( is_network_admin() ) {
-	$title = __( 'WordPoints — Network Points Logs', 'wordpoints' );
+if (is_network_admin()) {
+	$title = __('WordPoints — Network Points Logs', 'wordpoints');
 } else {
-	$title = __( 'WordPoints — Points Logs', 'wordpoints' );
+	$title = __('WordPoints — Points Logs', 'wordpoints');
 }
 
 ?>
 
 <div class="wrap">
-	<h1><?php echo esc_html( $title ); ?></h1>
-	<p class="wordpoints-admin-panel-desc"><?php esc_html_e( 'View recent points transactions.', 'wordpoints' ); ?></p>
+	<h1><?php echo esc_html($title); ?></h1>
+	<p class="wordpoints-admin-panel-desc"><?php esc_html_e('View recent points transactions.', 'wordpoints'); ?></p>
 
 	<?php
 
@@ -26,16 +26,16 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_points_logs' );
+	do_action('wordpoints_admin_points_logs');
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( empty( $points_types ) ) {
+	if (empty($points_types)) {
 
 		wordpoints_show_admin_error(
 			sprintf(
-				__( 'You need to <a href="%s">create a type of points</a> before you can use this page.', 'wordpoints' )
-				, esc_url( self_admin_url( 'admin.php?page=wordpoints_points_types' ) )
+				__('You need to <a href="%s">create a type of points</a> before you can use this page.', 'wordpoints')
+				, esc_url(self_admin_url('admin.php?page=wordpoints_points_types'))
 			)
 		);
 
@@ -44,20 +44,20 @@ 
 		// Show a tab for each points type.
 		$tabs = array();
 
-		foreach ( $points_types as $slug => $settings ) {
+		foreach ($points_types as $slug => $settings) {
 
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[$slug] = $settings['name'];
 		}
 
-		wordpoints_admin_show_tabs( $tabs, false );
+		wordpoints_admin_show_tabs($tabs, false);
 
-		if ( is_network_admin() ) {
+		if (is_network_admin()) {
 			$query = 'network';
 		} else {
 			$query = 'default';
 		}
 
-		$current_type = wordpoints_admin_get_current_tab( $tabs );
+		$current_type = wordpoints_admin_get_current_tab($tabs);
 
 		/**
 		 * At the top of one of the tabs on the points logs admin panel.
@@ -67,10 +67,10 @@ 
 		 * @param string $points_type The points type the current tab is for.
 		 * @param string $query       The current logs query being performed.
 		 */
-		do_action( 'wordpoints_admin_points_logs_tab', $current_type, $query );
+		do_action('wordpoints_admin_points_logs_tab', $current_type, $query);
 
 		// Get and display the logs based on current points type.
-		wordpoints_show_points_logs_query( $current_type, $query );
+		wordpoints_show_points_logs_query($current_type, $query);
 
 		/**
 		 * At the bottom of one of the tabs on the points logs admin panel.
@@ -80,7 +80,7 @@ 
 		 * @param string $points_type The points type the current tab is for.
 		 * @param string $query       The current logs query being performed.
 		 */
-		do_action( 'wordpoints_admin_points_logs_tab_after', $current_type, $query );
+		do_action('wordpoints_admin_points_logs_tab_after', $current_type, $query);
 
 	} // End if ( empty( $points_types ) ) else.
 
@@ -89,7 +89,7 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	do_action( 'wordpoints_admin_points_logs_after' );
+	do_action('wordpoints_admin_points_logs_after');
 
 	?>
 

src/components/points/admin/screens/hooks-no-js-load.php

Spacing +38 added lines, -38 removed lines

@@ -7,122 +7,122 @@
  * @since 1.2.0
  */
 
-if ( ! current_user_can( 'manage_options' ) ) {
+if ( ! current_user_can('manage_options')) {
 	return;
 }
 
-if ( ! isset( $_POST['hook-id'] ) ) {
+if ( ! isset($_POST['hook-id'])) {
 	return;
 }
 
-$hook_id = sanitize_key( $_POST['hook-id'] );
+$hook_id = sanitize_key($_POST['hook-id']);
 
-check_admin_referer( "save-delete-hook-{$hook_id}" );
+check_admin_referer("save-delete-hook-{$hook_id}");
 
-$redirect_url = self_admin_url( 'admin.php?page=wordpoints_points_hooks' );
+$redirect_url = self_admin_url('admin.php?page=wordpoints_points_hooks');
 
-if ( is_network_admin() ) {
-	WordPoints_Points_Hooks::set_network_mode( true );
+if (is_network_admin()) {
+	WordPoints_Points_Hooks::set_network_mode(true);
 }
 
-if ( ! isset( $_POST['points_type'], $_POST['id_base'] ) ) {
+if ( ! isset($_POST['points_type'], $_POST['id_base'])) {
 	return;
 }
 
-$points_type_id = sanitize_key( $_POST['points_type'] );
-$id_base        = sanitize_key( $_POST['id_base'] );
+$points_type_id = sanitize_key($_POST['points_type']);
+$id_base        = sanitize_key($_POST['id_base']);
 
 // These are the hooks grouped by points type.
 $points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
 
-if ( empty( $points_types_hooks ) ) {
+if (empty($points_types_hooks)) {
 	$points_types_hooks = WordPoints_Points_Hooks::get_defaults();
 }
 
-if ( isset( $points_types_hooks[ $points_type_id ] ) ) {
-	$points_type_hooks = $points_types_hooks[ $points_type_id ];
+if (isset($points_types_hooks[$points_type_id])) {
+	$points_type_hooks = $points_types_hooks[$points_type_id];
 } else {
 	$points_type_hooks = array();
 }
 
-$hook = WordPoints_Points_Hooks::get_handler_by_id_base( $id_base );
+$hook = WordPoints_Points_Hooks::get_handler_by_id_base($id_base);
 
-if ( ! empty( $_POST['removehook'] ) ) {
+if ( ! empty($_POST['removehook'])) {
 
 	// - We are deleting an instance of a hook.
 
-	if ( ! in_array( $hook_id, $points_type_hooks, true ) ) {
+	if ( ! in_array($hook_id, $points_type_hooks, true)) {
 
 		// The hook isn't hooked to this points type, give an error.
-		wp_safe_redirect( $redirect_url . '&error=0' );
+		wp_safe_redirect($redirect_url . '&error=0');
 		exit;
 	}
 
 	// Remove the hook from this points type.
-	$points_types_hooks[ $points_type_id ] = array_diff( $points_type_hooks, array( $hook_id ) );
+	$points_types_hooks[$points_type_id] = array_diff($points_type_hooks, array($hook_id));
 
-	$hook->delete_callback( $hook_id );
+	$hook->delete_callback($hook_id);
 
-} elseif ( ! empty( $_POST['savehook'] ) ) {
+} elseif ( ! empty($_POST['savehook'])) {
 
 	// - We are saving an instance of a hook.
 
-	$number = isset( $_POST['multi_number'] ) ? (int) $_POST['multi_number'] : '';
+	$number = isset($_POST['multi_number']) ? (int) $_POST['multi_number'] : '';
 
-	if ( $number ) {
+	if ($number) {
 
 		// Search the POST for the instance settings.
-		foreach ( $_POST as $key => $val ) {
+		foreach ($_POST as $key => $val) {
 
-			if ( is_array( $val ) && preg_match( '/__i__|%i%/', key( $val ) ) ) {
+			if (is_array($val) && preg_match('/__i__|%i%/', key($val))) {
 
-				$new_instance = array_shift( $val );
+				$new_instance = array_shift($val);
 				break;
 			}
 		}
 
 	} else {
 
-		if ( isset( $_POST[ 'hook-' . $id_base ] ) && is_array( $_POST[ 'hook-' . $id_base ] ) ) {
-			$new_instance = wp_unslash( reset( $_POST[ 'hook-' . $id_base ] ) ); // WPCS sanitization OK.
+		if (isset($_POST['hook-' . $id_base]) && is_array($_POST['hook-' . $id_base])) {
+			$new_instance = wp_unslash(reset($_POST['hook-' . $id_base])); // WPCS sanitization OK.
 		}
 
-		$number = $hook->get_number_by_id( $hook_id );
+		$number = $hook->get_number_by_id($hook_id);
 	}
 
-	if ( ! isset( $new_instance ) || ! is_array( $new_instance ) ) {
+	if ( ! isset($new_instance) || ! is_array($new_instance)) {
 
 		$new_instance = array();
 	}
 
 	// Update the hook.
-	$hook->update_callback( $new_instance, $number );
+	$hook->update_callback($new_instance, $number);
 
 	// Add hook it to this points type.
-	if ( ! in_array( $hook_id, $points_type_hooks ) ) {
+	if ( ! in_array($hook_id, $points_type_hooks)) {
 
 		$points_type_hooks[] = $hook_id;
-		$points_types_hooks[ $points_type_id ] = $points_type_hooks;
+		$points_types_hooks[$points_type_id] = $points_type_hooks;
 	}
 
 	// Remove from old points type if it has changed.
-	$old_points_type = WordPoints_Points_Hooks::get_points_type( $hook_id );
+	$old_points_type = WordPoints_Points_Hooks::get_points_type($hook_id);
 
-	if ( $old_points_type && $old_points_type !== $points_type_id && is_array( $points_types_hooks[ $old_points_type ] ) ) {
+	if ($old_points_type && $old_points_type !== $points_type_id && is_array($points_types_hooks[$old_points_type])) {
 
-		$points_types_hooks[ $old_points_type ] = array_diff( $points_types_hooks[ $old_points_type ], array( $hook_id ) );
+		$points_types_hooks[$old_points_type] = array_diff($points_types_hooks[$old_points_type], array($hook_id));
 	}
 
 } else {
 
-	wp_safe_redirect( $redirect_url . '&error=0' );
+	wp_safe_redirect($redirect_url . '&error=0');
 	exit;
 
 } // End if ( deleting hook ) elseif ( saving hook ) else.
 
-WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
+WordPoints_Points_Hooks::save_points_types_hooks($points_types_hooks);
 
-wp_safe_redirect( $redirect_url . '&message=0' );
+wp_safe_redirect($redirect_url . '&message=0');
 exit;
 
 // EOF

src/components/points/admin/screens/hooks-no-js.php

Spacing +38 added lines, -38 removed lines

@@ -7,44 +7,44 @@ 
  * @since 1.0.0
  */
 
-if ( ! isset( $_GET['edithook'] ) ) {
+if ( ! isset($_GET['edithook'])) {
 	return;
 }
 
-$hook_id = sanitize_key( $_GET['edithook'] );
+$hook_id = sanitize_key($_GET['edithook']);
 
 $points_types = wordpoints_get_points_types();
 
-if ( isset( $_GET['addnew'] ) ) {
+if (isset($_GET['addnew'])) {
 
 	// - We are adding a new points hook.
 
 	$points_type = wordpoints_get_default_points_type();
 
 	// Default to the first points type.
-	if ( ! $points_type ) {
-		reset( $points_types );
-		$points_type = key( $points_types );
+	if ( ! $points_type) {
+		reset($points_types);
+		$points_type = key($points_types);
 	}
 
-	if ( ! $points_type ) {
+	if ( ! $points_type) {
 
-		wordpoints_show_admin_error( esc_html__( 'You need to add a points type before you can add any hooks.', 'wordpoints' ) );
+		wordpoints_show_admin_error(esc_html__('You need to add a points type before you can add any hooks.', 'wordpoints'));
 		return;
 	}
 
-	if ( isset( $_GET['base'], $_GET['num'] ) ) {
+	if (isset($_GET['base'], $_GET['num'])) {
 
 		// Copy minimal info from an existing instance of this hook to a new instance.
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
-			sanitize_key( $_GET['base'] )
+			sanitize_key($_GET['base'])
 		);
 
-		if ( ! $hook ) {
+		if ( ! $hook) {
 
 			wordpoints_show_admin_error(
-				esc_html__( 'Unable to add the points hook, please try again.', 'wordpoints' )
-				, array( 'dismissible' => true )
+				esc_html__('Unable to add the points hook, please try again.', 'wordpoints')
+				, array('dismissible' => true)
 			);
 
 			return;
@@ -52,15 +52,15 @@ 
 
 		$multi_number = (int) $_GET['num'];
 		$number       = 0;
-		$hook_id      = $hook->get_id( $multi_number );
+		$hook_id      = $hook->get_id($multi_number);
 		$_hook        = $hook;
 		$id_base      = $hook->get_id_base();
 
 	} else {
 
 		wordpoints_show_admin_error(
-			esc_html__( 'Unable to add the points hook, please try again.', 'wordpoints' )
-			, array( 'dismissible' => true )
+			esc_html__('Unable to add the points hook, please try again.', 'wordpoints')
+			, array('dismissible' => true)
 		);
 
 		return;
@@ -70,42 +70,42 @@ 
 
 	// We are editing an existing points hook.
 
-	$points_type = isset( $_GET['points_type'] ) ? sanitize_key( $_GET['points_type'] ) : '_inactive_hooks';
+	$points_type = isset($_GET['points_type']) ? sanitize_key($_GET['points_type']) : '_inactive_hooks';
 
-	$hook = WordPoints_Points_Hooks::get_handler( $hook_id );
+	$hook = WordPoints_Points_Hooks::get_handler($hook_id);
 
-	if ( ! $hook ) {
+	if ( ! $hook) {
 
-		wordpoints_show_admin_error( esc_html__( 'The hook you have asked to edit could not be found. Please go back and try again.', 'wordpoints' ) );
+		wordpoints_show_admin_error(esc_html__('The hook you have asked to edit could not be found. Please go back and try again.', 'wordpoints'));
 		return;
 	}
 
 	$id_base = $hook->get_id_base();
 	$multi_number = 0;
-	$number = $hook->get_number_by_id( $hook_id );
+	$number = $hook->get_number_by_id($hook_id);
 
 } // End if ( adding hook ) else { updating hook }.
 
-$name = esc_html( $hook->get_name() );
+$name = esc_html($hook->get_name());
 
 // Show the hook form.
 
 ?>
 
 <div class="wrap">
-	<h1><?php esc_html_e( 'Points Hooks', 'wordpoints' ); ?></h1>
-	<div class="edithook" style="width:<?php echo absint( $hook->get_option( 'width' ) ); ?>px">
-		<h2><?php echo esc_html( sprintf( _x( 'Hook %s', 'hook name', 'wordpoints' ), $name ) ); ?></h2>
+	<h1><?php esc_html_e('Points Hooks', 'wordpoints'); ?></h1>
+	<div class="edithook" style="width:<?php echo absint($hook->get_option('width')); ?>px">
+		<h2><?php echo esc_html(sprintf(_x('Hook %s', 'hook name', 'wordpoints'), $name)); ?></h2>
 
-		<form action="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_hooks' ) ); ?>" method="post">
+		<form action="<?php echo esc_url(self_admin_url('admin.php?page=wordpoints_points_hooks')); ?>" method="post">
 			<div class="hook-inside">
-				<?php $hook->form_callback( $number ); ?>
+				<?php $hook->form_callback($number); ?>
 			</div>
 
-			<p class="describe"><?php esc_html_e( 'Select the points type to attach this hook to.', 'wordpoints' ); ?></p>
+			<p class="describe"><?php esc_html_e('Select the points type to attach this hook to.', 'wordpoints'); ?></p>
 			<div class="hook-position">
 				<label for="points_type" class="screen-reader-text">
-					<?php esc_html_e( 'Points Type:', 'wordpoints' ); ?>
+					<?php esc_html_e('Points Type:', 'wordpoints'); ?>
 				</label>
 				<?php
 
@@ -115,7 +115,7 @@ 
 							'name'     => 'points_type',
 							'id'       => 'points_type',
 							'class'    => 'widefat',
-							'options'  => array( '_inactive_hooks' => __( 'Inactive Hooks', 'wordpoints' ) ),
+							'options'  => array('_inactive_hooks' => __('Inactive Hooks', 'wordpoints')),
 						)
 					);
 
@@ -123,19 +123,19 @@ 
 			</div>
 			<br />
 			<div class="hook-control-actions">
-				<?php if ( isset( $_GET['addnew'] ) ) : ?>
-					<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_hooks' ) ); ?>" class="button alignleft"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></a>
+				<?php if (isset($_GET['addnew'])) : ?>
+					<a href="<?php echo esc_url(self_admin_url('admin.php?page=wordpoints_points_hooks')); ?>" class="button alignleft"><?php esc_html_e('Cancel', 'wordpoints'); ?></a>
 				<?php else :
-						submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
+						submit_button(_x('Delete', 'points hook', 'wordpoints'), 'button alignleft', 'removehook', false);
 					endif;
 
-					submit_button( __( 'Save Hook', 'wordpoints' ), 'button-primary alignright', 'savehook', false );
+					submit_button(__('Save Hook', 'wordpoints'), 'button-primary alignright', 'savehook', false);
 				?>
 
-				<input type="hidden" name="hook-id" class="hook-id" value="<?php echo esc_attr( $hook_id ); ?>" />
-				<input type="hidden" name="id_base" class="id_base" value="<?php echo esc_attr( $id_base ); ?>" />
-				<input type="hidden" name="multi_number" class="multi_number" value="<?php echo esc_attr( $multi_number ); ?>" />
-				<?php wp_nonce_field( "save-delete-hook-{$hook_id}" ); ?>
+				<input type="hidden" name="hook-id" class="hook-id" value="<?php echo esc_attr($hook_id); ?>" />
+				<input type="hidden" name="id_base" class="id_base" value="<?php echo esc_attr($id_base); ?>" />
+				<input type="hidden" name="multi_number" class="multi_number" value="<?php echo esc_attr($multi_number); ?>" />
+				<?php wp_nonce_field("save-delete-hook-{$hook_id}"); ?>
 				<br class="clear" />
 			</div>
 		</form>

Braces +3 added lines, -1 removed lines

@@ -125,8 +125,10 @@
 			<div class="hook-control-actions">
 				<?php if ( isset( $_GET['addnew'] ) ) : ?>
 					<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_hooks' ) ); ?>" class="button alignleft"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></a>
-				<?php else :
+				<?php else {
+	:
 						submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
+}
 					endif;
 
 					submit_button( __( 'Save Hook', 'wordpoints' ), 'button-primary alignright', 'savehook', false );

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

Spacing +16 added lines, -16 removed lines

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

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

Spacing +79 added lines, -79 removed lines

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

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

Indentation +3 added lines, -3 removed lines

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

Spacing +89 added lines, -89 removed lines

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

Braces +5 added lines, -2 removed lines

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