WordPoints / wordpoints

src/components/points/includes/hooks/registration.php

Spacing +10 added lines, -10 removed lines

@@ -23,7 +23,7 @@ 
 	 *
 	 * @type array $defaults
 	 */
-	protected $defaults = array( 'points' => 100 );
+	protected $defaults = array('points' => 100);
 
 	/**
 	 * Construct the class.
@@ -36,12 +36,12 @@ 
 	public function __construct() {
 
 		$this->init(
-			_x( 'Registration', 'points hook name', 'wordpoints' )
-			, array( 'description' => _x( 'Registering with the site.', 'points hook description', 'wordpoints' ) )
+			_x('Registration', 'points hook name', 'wordpoints')
+			, array('description' => _x('Registering with the site.', 'points hook description', 'wordpoints'))
 		);
 
-		add_action( 'user_register', array( $this, 'hook' ) );
-		add_filter( 'wordpoints_points_log-register', array( $this, 'logs' ) );
+		add_action('user_register', array($this, 'hook'));
+		add_filter('wordpoints_points_log-register', array($this, 'logs'));
 	}
 
 	/**
@@ -55,12 +55,12 @@ 
 	 *
 	 * @return void
 	 */
-	public function hook( $user_id ) {
+	public function hook($user_id) {
 
-		foreach ( $this->get_instances() as $number => $instance ) {
+		foreach ($this->get_instances() as $number => $instance) {
 
-			if ( isset( $instance['points'] ) ) {
-				wordpoints_add_points( $user_id, $instance['points'], $this->points_type( $number ), 'register' );
+			if (isset($instance['points'])) {
+				wordpoints_add_points($user_id, $instance['points'], $this->points_type($number), 'register');
 			}
 		}
 	}
@@ -76,7 +76,7 @@ 
 	 */
 	public function logs() {
 
-		return _x( 'Registration.', 'points log description', 'wordpoints' );
+		return _x('Registration.', 'points log description', 'wordpoints');
 	}
 
 } // class WordPoints_Registration_Points_Hook

src/components/points/includes/apps.php

Spacing +14 added lines, -14 removed lines

@@ -16,11 +16,11 @@ 
  *
  * @param WordPoints_App $components The components app.
  */
-function wordpoints_points_components_app_init( $components ) {
+function wordpoints_points_components_app_init($components) {
 
 	$apps = $components->sub_apps();
 
-	$apps->register( 'points', 'WordPoints_App' );
+	$apps->register('points', 'WordPoints_App');
 }
 
 /**
@@ -32,11 +32,11 @@ 
  *
  * @param WordPoints_App $app The points app.
  */
-function wordpoints_points_apps_init( $app ) {
+function wordpoints_points_apps_init($app) {
 
 	$apps = $app->sub_apps();
 
-	$apps->register( 'logs', 'WordPoints_App' );
+	$apps->register('logs', 'WordPoints_App');
 }
 
 /**
@@ -48,12 +48,12 @@ 
  *
  * @param WordPoints_App $app The points logs app.
  */
-function wordpoints_points_logs_apps_init( $app ) {
+function wordpoints_points_logs_apps_init($app) {
 
 	$apps = $app->sub_apps();
 
-	$apps->register( 'views', 'WordPoints_Class_Registry' );
-	$apps->register( 'viewing_restrictions', 'WordPoints_Points_Logs_Viewing_Restrictions' );
+	$apps->register('views', 'WordPoints_Class_Registry');
+	$apps->register('viewing_restrictions', 'WordPoints_Points_Logs_Viewing_Restrictions');
 }
 
 /**
@@ -65,9 +65,9 @@ 
  *
  * @param WordPoints_Class_RegistryI $views The points log views registry.
  */
-function wordpoints_points_logs_views_init( $views ) {
+function wordpoints_points_logs_views_init($views) {
 
-	$views->register( 'table', 'WordPoints_Points_Logs_View_Table' );
+	$views->register('table', 'WordPoints_Points_Logs_View_Table');
 }
 
 /**
@@ -79,13 +79,13 @@ 
  *
  * @param WordPoints_Points_Logs_Viewing_Restrictions $restrictions The registry.
  */
-function wordpoints_points_logs_viewing_restrictions_init( $restrictions ) {
+function wordpoints_points_logs_viewing_restrictions_init($restrictions) {
 
-	$restrictions->register( 'all', 'hooks', 'WordPoints_Points_Logs_Viewing_Restriction_Hooks' );
+	$restrictions->register('all', 'hooks', 'WordPoints_Points_Logs_Viewing_Restriction_Hooks');
 
-	$restrictions->register( 'comment_approve', 'read_comment', 'WordPoints_Points_Logs_Viewing_Restriction_Read_Comment_Post' );
-	$restrictions->register( 'comment_received', 'read_comment', 'WordPoints_Points_Logs_Viewing_Restriction_Read_Comment_Post' );
-	$restrictions->register( 'post_publish', 'read_post', 'WordPoints_Points_Logs_Viewing_Restriction_Read_Post' );
+	$restrictions->register('comment_approve', 'read_comment', 'WordPoints_Points_Logs_Viewing_Restriction_Read_Comment_Post');
+	$restrictions->register('comment_received', 'read_comment', 'WordPoints_Points_Logs_Viewing_Restriction_Read_Comment_Post');
+	$restrictions->register('post_publish', 'read_post', 'WordPoints_Points_Logs_Viewing_Restriction_Read_Post');
 }
 
 // EOF

src/components/points/admin/admin.php

Spacing +2 added lines, -2 removed lines

@@ -22,14 +22,14 @@
  *
  * @since 2.1.0
  */
-require_once( WORDPOINTS_DIR . '/components/points/admin/includes/functions.php' );
+require_once(WORDPOINTS_DIR . '/components/points/admin/includes/functions.php');
 
 /**
  * Admin-side hooks.
  *
  * @since 2.1.0
  */
-require_once( WORDPOINTS_DIR . '/components/points/admin/includes/filters.php' );
+require_once(WORDPOINTS_DIR . '/components/points/admin/includes/filters.php');
 
 WordPoints_Class_Autoloader::register_dir(
 	WORDPOINTS_DIR . '/components/points/admin/classes'

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

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; ?>

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,32 +206,32 @@ 
 		?>
 
 		<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
 
 							// translators: Points type slug.
-							echo esc_html( sprintf( __( 'Slug: %s', 'wordpoints' ), $slug ) );
+							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
@@ -245,45 +245,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
@@ -297,28 +297,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"/>
@@ -337,8 +337,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
@@ -351,22 +351,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();
 
@@ -391,11 +391,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
 				)
 			) {
@@ -405,7 +405,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'
@@ -426,50 +426,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>
@@ -484,7 +484,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>
@@ -504,19 +504,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;
 	}
@@ -528,19 +528,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();
 		}
@@ -557,21 +557,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
 			);
 		}
 
@@ -589,52 +589,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 {
@@ -642,7 +642,7 @@ 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_update'
-				, __( 'Points type updated.', 'wordpoints' )
+				, __('Points type updated.', 'wordpoints')
 				, 'updated'
 			);
 		}
@@ -668,14 +668,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 {
@@ -685,7 +685,7 @@ 
 			add_settings_error(
 				''
 				, 'wordpoints_points_type_create'
-				, __( 'Points type created.', 'wordpoints' )
+				, __('Points type created.', 'wordpoints')
 				, 'updated'
 			);
 		}
@@ -702,22 +702,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'
 				);
 
@@ -726,7 +726,7 @@ 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Error while deleting.', 'wordpoints' )
+					, __('Error while deleting.', 'wordpoints')
 				);
 			}
 		}
@@ -742,18 +742,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;
@@ -764,8 +764,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">
@@ -773,23 +773,23 @@ 
 					<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(
 											// translators: URL of points reactions user docs on WordPoints.org.
-											__( '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))
 									);
 
 									?>
@@ -797,7 +797,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; ?>
 
@@ -816,7 +816,7 @@ 
 		 *
 		 * @since 2.1.0
 		 */
-		do_action( 'wordpoints_admin_points_events_foot' );
+		do_action('wordpoints_admin_points_events_foot');
 	}
 }
 

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-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

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 );

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,43 +70,43 @@ 
 
 	// 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">
+	<h1><?php esc_html_e('Points Hooks', 'wordpoints'); ?></h1>
+	<div class="edithook" style="width:<?php echo absint($hook->get_option('width')); ?>px">
 		<?php // translators: Hook name. ?>
-		<h2><?php echo esc_html( sprintf( __( 'Hook %s', 'wordpoints' ), $name ) ); ?></h2>
+		<h2><?php echo esc_html(sprintf(__('Hook %s', '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
 
@@ -116,7 +116,7 @@ 
 							'name'     => 'points_type',
 							'id'       => 'points_type',
 							'class'    => 'widefat',
-							'options'  => array( '_inactive_hooks' => __( 'Inactive Hooks', 'wordpoints' ) ),
+							'options'  => array('_inactive_hooks' => __('Inactive Hooks', 'wordpoints')),
 						)
 					);
 
@@ -124,19 +124,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>

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