WordPoints / hooks-api

src/admin/includes/functions.php

Spacing +82 added lines, -82 removed lines

@@ -16,16 +16,16 @@ 
  *
  * @param WordPoints_App $app The main WordPoints app.
  */
-function wordpoints_hooks_register_admin_apps( $app ) {
+function wordpoints_hooks_register_admin_apps($app) {
 
 	$apps = $app->sub_apps;
 
-	$apps->register( 'admin', 'WordPoints_App' );
+	$apps->register('admin', 'WordPoints_App');
 
 	/** @var WordPoints_App $admin */
-	$admin = $apps->get( 'admin' );
+	$admin = $apps->get('admin');
 
-	$admin->sub_apps->register( 'screen', 'WordPoints_Admin_Screens' );
+	$admin->sub_apps->register('screen', 'WordPoints_Admin_Screens');
 }
 
 /**
@@ -46,15 +46,15 @@ 
 	// 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_Admin_Screen_Points_Types' );
+	if ($id) {
+		$admin_screens->register($id, 'WordPoints_Admin_Screen_Points_Types');
 	}
 }
 
@@ -67,14 +67,14 @@ 
  */
 function wordpoints_hooks_admin_register_scripts() {
 
-	$assets_url = wordpoints_modules_url( '/assets', dirname( __FILE__ ) );
+	$assets_url = wordpoints_modules_url('/assets', dirname(__FILE__));
 
 	// CSS
 
 	wp_register_style(
 		'wordpoints-hooks-admin'
-		, $assets_url . '/css/hooks.css'
-		, array( 'dashicons', 'wp-jquery-ui-dialog' )
+		, $assets_url.'/css/hooks.css'
+		, array('dashicons', 'wp-jquery-ui-dialog')
 		, WORDPOINTS_VERSION
 	);
 
@@ -82,15 +82,15 @@ 
 
 	wp_register_script(
 		'wordpoints-hooks-models'
-		, $assets_url . '/js/hooks/models.js'
-		, array( 'backbone', 'jquery-ui-dialog', 'wp-util' )
+		, $assets_url.'/js/hooks/models.js'
+		, array('backbone', 'jquery-ui-dialog', 'wp-util')
 		, WORDPOINTS_VERSION
 	);
 
 	wp_register_script(
 		'wordpoints-hooks-views'
-		, $assets_url . '/js/hooks/views.js'
-		, array( 'wordpoints-hooks-models' )
+		, $assets_url.'/js/hooks/views.js'
+		, array('wordpoints-hooks-models')
 		, WORDPOINTS_VERSION
 	);
 
@@ -98,16 +98,16 @@ 
 		'wordpoints-hooks-views'
 		, 'WordPointsHooksAdminL10n'
 		, array(
-			'unexpectedError' => __( 'There was an unexpected error. Try reloading the page.', 'wordpoints' ),
-			'changesSaved'    => __( 'Your changes have been saved.', 'wordpoints' ),
+			'unexpectedError' => __('There was an unexpected error. Try reloading the page.', 'wordpoints'),
+			'changesSaved'    => __('Your changes have been saved.', 'wordpoints'),
 			/* translators: the name of the field that cannot be empty */
-			'emptyField'      => sprintf( __( '%s cannot be empty.', 'wordpoints' ), '{{ data.label }}' ),
-			'confirmDelete'   => __( 'Are you sure that you want to delete this reaction? This action cannot be undone.', 'wordpoints' ),
-			'confirmTitle'    => __( 'Are you sure?', 'wordpoints' ),
-			'deleteText'      => __( 'Delete', 'wordpoints' ),
-			'cancelText'      => __( 'Cancel', 'wordpoints' ),
-			'separator'       => __( ' » ', 'wordpoints' ),
-			'target_label'    => __( 'Target', 'wordpoints' ),
+			'emptyField'      => sprintf(__('%s cannot be empty.', 'wordpoints'), '{{ data.label }}'),
+			'confirmDelete'   => __('Are you sure that you want to delete this reaction? This action cannot be undone.', 'wordpoints'),
+			'confirmTitle'    => __('Are you sure?', 'wordpoints'),
+			'deleteText'      => __('Delete', 'wordpoints'),
+			'cancelText'      => __('Cancel', 'wordpoints'),
+			'separator'       => __(' » ', 'wordpoints'),
+			'target_label'    => __('Target', 'wordpoints'),
 		)
 	);
 
@@ -119,10 +119,10 @@ 
 			<div class="view">
 				<div class="title"></div>
 				<button type="button" class="edit button-secondary">
-					' . esc_html__( 'Edit', 'wordpoints' ) . '
+					' . esc_html__('Edit', 'wordpoints').'
 				</button>
 				<button type="button" class="close button-secondary">
-					' . esc_html__( 'Close', 'wordpoints' ) . '
+					' . esc_html__('Close', 'wordpoints').'
 				</button>
 			</div>
 			<div class="form">
@@ -141,16 +141,16 @@ 
 						</div>
 						<div class="action-buttons">
 							<button type="button" class="save button-primary" disabled>
-								' . esc_html__( 'Save', 'wordpoints' ) . '
+								' . esc_html__('Save', 'wordpoints').'
 							</button>
 							<button type="button" class="cancel button-secondary">
-								' . esc_html__( 'Cancel', 'wordpoints' ) . '
+								' . esc_html__('Cancel', 'wordpoints').'
 							</button>
 							<button type="button" class="close button-secondary">
-								' . esc_html__( 'Close', 'wordpoints' ) . '
+								' . esc_html__('Close', 'wordpoints').'
 							</button>
 							<button type="button" class="delete button-secondary">
-								' . esc_html__( 'Delete', 'wordpoints' ) . '
+								' . esc_html__('Delete', 'wordpoints').'
 							</button>
 						</div>
 					</div>
@@ -198,15 +198,15 @@ 
 
 	wp_register_script(
 		'wordpoints-hooks-reactor-points'
-		, $assets_url . '/js/hooks/reactors/points.js'
-		, array( 'wordpoints-hooks-views' )
+		, $assets_url.'/js/hooks/reactors/points.js'
+		, array('wordpoints-hooks-views')
 		, WORDPOINTS_VERSION
 	);
 
 	wp_register_script(
 		'wordpoints-hooks-extension-conditions'
-		, $assets_url . '/js/hooks/extensions/conditions.js'
-		, array( 'wordpoints-hooks-views' )
+		, $assets_url.'/js/hooks/extensions/conditions.js'
+		, array('wordpoints-hooks-views')
 		, WORDPOINTS_VERSION
 	);
 
@@ -216,25 +216,25 @@ 
 		, '
 			<script type="text/template" id="tmpl-wordpoints-hook-condition-groups">
 				<div class="conditions-title section-title">
-					<h4>' . esc_html__( 'Conditions', 'wordpoints' ) . '</h4>
+					<h4>' . esc_html__('Conditions', 'wordpoints').'</h4>
 					<button type="button" class="add-new button-secondary button-link">
-						<span class="screen-reader-text">' . esc_html__( 'Add New Condition', 'wordpoints' ) . '</span>
+						<span class="screen-reader-text">' . esc_html__('Add New Condition', 'wordpoints').'</span>
 						<span class="dashicons dashicons-plus"></span>
 					</button>
 				</div>
 				<div class="add-condition-form hidden">
 					<div class="no-conditions hidden">
-						' . esc_html__( 'No conditions available.', 'wordpoints' ) . '
+						' . esc_html__('No conditions available.', 'wordpoints').'
 					</div>
 					<div class="condition-selectors">
 						<div class="arg-selectors"></div>
 						<div class="condition-selector"></div>
 					</div>
-					<button type="button" class="confirm-add-new button-secondary" disabled aria-label="' . esc_attr__( 'Add Condition', 'wordpoints' ) . '">
-						' . esc_html_x( 'Add', 'reaction condition', 'wordpoints' ) . '
+					<button type="button" class="confirm-add-new button-secondary" disabled aria-label="' . esc_attr__('Add Condition', 'wordpoints').'">
+						' . esc_html_x('Add', 'reaction condition', 'wordpoints').'
 					</button>
-					<button type="button" class="cancel-add-new button-secondary" aria-label="' . esc_attr__( 'Cancel Adding New Condition', 'wordpoints' ) . '">
-						' . esc_html_x( 'Cancel', 'reaction condition', 'wordpoints' ) . '
+					<button type="button" class="cancel-add-new button-secondary" aria-label="' . esc_attr__('Cancel Adding New Condition', 'wordpoints').'">
+						' . esc_html_x('Cancel', 'reaction condition', 'wordpoints').'
 					</button>
 				</div>
 				<div class="condition-groups section-content"></div>
@@ -248,7 +248,7 @@ 
 				<div class="condition-controls">
 					<div class="condition-title"></div>
 					<button type="button" class="delete button-secondary button-link">
-						<span class="screen-reader-text">' . esc_html__( 'Remove Condition', 'wordpoints' ) . '</span>
+						<span class="screen-reader-text">' . esc_html__('Remove Condition', 'wordpoints').'</span>
 						<span class="dashicons dashicons-no"></span>
 					</button>
 				</div>
@@ -266,8 +266,8 @@ 
 
 	wp_register_script(
 		'wordpoints-hooks-extension-periods'
-		, $assets_url . '/js/hooks/extensions/periods.js'
-		, array( 'wordpoints-hooks-views' )
+		, $assets_url.'/js/hooks/extensions/periods.js'
+		, array('wordpoints-hooks-views')
 		, WORDPOINTS_VERSION
 	);
 
@@ -277,7 +277,7 @@ 
 		, '
 			<script type="text/template" id="tmpl-wordpoints-hook-periods">
 				<div class="periods-title section-title">
-					<h4>' . esc_html__( 'Rate Limit', 'wordpoints' ) . '</h4>
+					<h4>' . esc_html__('Rate Limit', 'wordpoints').'</h4>
 				</div>
 				<div class="periods section-content"></div>
 			</script>
@@ -295,27 +295,27 @@ 
 	$hooks = wordpoints_hooks();
 
 	$event_index = $hooks->router->get_event_index();
-	$event_index = call_user_func_array( 'array_merge_recursive', $event_index );
+	$event_index = call_user_func_array('array_merge_recursive', $event_index);
 
 	$event_action_types = array();
 
-	foreach ( $event_index as $action_type => $events ) {
+	foreach ($event_index as $action_type => $events) {
 		$event_action_types = array_merge_recursive(
 			$event_action_types
-			, array_fill_keys( array_keys( $events ), array( $action_type ) )
+			, array_fill_keys(array_keys($events), array($action_type))
 		);
 	}
 
 	$extensions_data = array();
 
-	foreach ( $hooks->extensions->get_all() as $slug => $extension ) {
+	foreach ($hooks->extensions->get_all() as $slug => $extension) {
 
-		if ( $extension instanceof WordPoints_Hook_Extension ) {
-			$extensions_data[ $slug ] = $extension->get_ui_script_data();
+		if ($extension instanceof WordPoints_Hook_Extension) {
+			$extensions_data[$slug] = $extension->get_ui_script_data();
 		}
 
-		if ( wp_script_is( "wordpoints-hooks-extension-{$slug}", 'registered' ) ) {
-			wp_enqueue_script( "wordpoints-hooks-extension-{$slug}" );
+		if (wp_script_is("wordpoints-hooks-extension-{$slug}", 'registered')) {
+			wp_enqueue_script("wordpoints-hooks-extension-{$slug}");
 		}
 	}
 
@@ -323,16 +323,16 @@ 
 
 	$reactor_data = array();
 
-	foreach ( $hooks->reactors->get_all() as $slug => $reactor ) {
+	foreach ($hooks->reactors->get_all() as $slug => $reactor) {
 
-		if ( $reactor instanceof WordPoints_Hook_Reactor ) {
-			$reactor_data[ $slug ] = $reactor->get_ui_script_data();
+		if ($reactor instanceof WordPoints_Hook_Reactor) {
+			$reactor_data[$slug] = $reactor->get_ui_script_data();
 		}
 
-		$reactor_data[ $slug ]['action_types'] = $action_types[ $slug ];
+		$reactor_data[$slug]['action_types'] = $action_types[$slug];
 
-		if ( wp_script_is( "wordpoints-hooks-reactor-{$slug}", 'registered' ) ) {
-			wp_enqueue_script( "wordpoints-hooks-reactor-{$slug}" );
+		if (wp_script_is("wordpoints-hooks-reactor-{$slug}", 'registered')) {
+			wp_enqueue_script("wordpoints-hooks-reactor-{$slug}");
 		}
 	}
 
@@ -344,28 +344,28 @@ 
 	$entity_children = $entities->children;
 
 	/** @var WordPoints_Entity $entity */
-	foreach ( $entities->get_all() as $slug => $entity ) {
+	foreach ($entities->get_all() as $slug => $entity) {
 
 		$child_data = array();
 
 		/** @var WordPoints_EntityishI $child */
-		foreach ( $entity_children->get_children( $slug ) as $child_slug => $child ) {
+		foreach ($entity_children->get_children($slug) as $child_slug => $child) {
 
-			$child_data[ $child_slug ] = array(
+			$child_data[$child_slug] = array(
 				'slug'  => $child_slug,
 				'title' => $child->get_title(),
 			);
 
-			if ( $child instanceof WordPoints_Entity_Attr ) {
+			if ($child instanceof WordPoints_Entity_Attr) {
 
-				$child_data[ $child_slug ]['_type'] = 'attr';
-				$child_data[ $child_slug ]['data_type']  = $child->get_data_type();
+				$child_data[$child_slug]['_type'] = 'attr';
+				$child_data[$child_slug]['data_type'] = $child->get_data_type();
 
-			} elseif ( $child instanceof WordPoints_Entity_Relationship ) {
+			} elseif ($child instanceof WordPoints_Entity_Relationship) {
 
-				$child_data[ $child_slug ]['_type']     = 'relationship';
-				$child_data[ $child_slug ]['primary']   = $child->get_primary_entity_slug();
-				$child_data[ $child_slug ]['secondary'] = $child->get_related_entity_slug();
+				$child_data[$child_slug]['_type']     = 'relationship';
+				$child_data[$child_slug]['primary']   = $child->get_primary_entity_slug();
+				$child_data[$child_slug]['secondary'] = $child->get_related_entity_slug();
 			}
 
 			/**
@@ -376,10 +376,10 @@ 
 			 * @param array                $data  The data for the entity child.
 			 * @param WordPoints_Entityish $child The child's object.
 			 */
-			$child_data[ $child_slug ] = apply_filters( 'wordpoints_hooks_ui_data_entity_child', $child_data[ $child_slug ], $child );
+			$child_data[$child_slug] = apply_filters('wordpoints_hooks_ui_data_entity_child', $child_data[$child_slug], $child);
 		}
 
-		$entities_data[ $slug ] = array(
+		$entities_data[$slug] = array(
 			'slug'     => $slug,
 			'title'    => $entity->get_title(),
 			'children' => $child_data,
@@ -387,12 +387,12 @@ 
 			'_type'    => 'entity',
 		);
 
-		if ( $entity instanceof WordPoints_Entity_EnumerableI ) {
+		if ($entity instanceof WordPoints_Entity_EnumerableI) {
 
 			$values = array();
 
-			foreach ( $entity->get_enumerated_values() as $value ) {
-				if ( $entity->set_the_value( $value ) ) {
+			foreach ($entity->get_enumerated_values() as $value) {
+				if ($entity->set_the_value($value)) {
 					$values[] = array(
 						'value' => $entity->get_the_id(),
 						'label' => $entity->get_the_human_id(),
@@ -400,7 +400,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -409,7 +409,7 @@ 
 		 * @param array             $data   The data for the entity.
 		 * @param WordPoints_Entity $entity The entity object.
 		 */
-		$entities_data[ $slug ] = apply_filters( 'wordpoints_hooks_ui_data_entity', $entities_data[ $slug ], $entity );
+		$entities_data[$slug] = apply_filters('wordpoints_hooks_ui_data_entity', $entities_data[$slug], $entity);
 	}
 
 	$data = array(
@@ -431,7 +431,7 @@ 
 	 *
 	 * @param array $data The data.
 	 */
-	$data = apply_filters( 'wordpoints_hooks_ui_data', $data );
+	$data = apply_filters('wordpoints_hooks_ui_data', $data);
 
 	wp_localize_script(
 		'wordpoints-hooks-models'
@@ -456,13 +456,13 @@ 
  *
  * @return string The HTML with templates appended.
  */
-function wordpoints_script_templates_filter( $html, $handle ) {
+function wordpoints_script_templates_filter($html, $handle) {
 
 	global $wp_scripts;
 
-	$templates = $wp_scripts->get_data( $handle, 'wordpoints-templates' );
+	$templates = $wp_scripts->get_data($handle, 'wordpoints-templates');
 
-	if ( $templates ) {
+	if ($templates) {
 		$html .= $templates;
 	}
 
@@ -478,7 +478,7 @@ 
  */
 function wordpoints_hooks_admin_ajax() {
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+	if (defined('DOING_AJAX') && DOING_AJAX) {
 		new WordPoints_Admin_Ajax_Hooks;
 	}
 }

src/admin/includes/classes/ajax/hooks.php

Indentation +4 added lines, -4 removed lines

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

Spacing +57 added lines, -57 removed lines

@@ -52,7 +52,7 @@ 
 	 *
 	 * @return array The hook reaction data extracted into an array.
 	 */
-	public static function prepare_hook_reaction( WordPoints_Hook_ReactionI $reaction ) {
+	public static function prepare_hook_reaction(WordPoints_Hook_ReactionI $reaction) {
 
 		return array_merge(
 			$reaction->get_all_meta()
@@ -60,8 +60,8 @@ 
 				'id' => $reaction->ID,
 				'event' => $reaction->get_event_slug(),
 				'reaction_store' => $reaction->get_store_slug(),
-				'nonce' => self::get_update_nonce( $reaction ),
-				'delete_nonce' => self::get_delete_nonce( $reaction ),
+				'nonce' => self::get_update_nonce($reaction),
+				'delete_nonce' => self::get_delete_nonce($reaction),
 			)
 		);
 	}
@@ -81,7 +81,7 @@ 
 		WordPoints_Hook_Reaction_StoreI $reaction_store
 	) {
 
-		return wp_create_nonce( self::get_create_nonce_action( $reaction_store ) );
+		return wp_create_nonce(self::get_create_nonce_action($reaction_store));
 	}
 
 	/**
@@ -99,9 +99,9 @@ 
 		WordPoints_Hook_Reaction_StoreI $reaction_store
 	) {
 
-		return 'wordpoints_create_hook_reaction|' . $reaction_store->get_slug()
-		       . '|' . wordpoints_hooks()->get_current_mode()
-		       . '|' . wp_json_encode( $reaction_store->get_context_id() );
+		return 'wordpoints_create_hook_reaction|'.$reaction_store->get_slug()
+		       . '|'.wordpoints_hooks()->get_current_mode()
+		       . '|'.wp_json_encode($reaction_store->get_context_id());
 	}
 
 	/**
@@ -113,9 +113,9 @@ 
 	 *
 	 * @return string A nonce for updating this reaction.
 	 */
-	public static function get_update_nonce( WordPoints_Hook_ReactionI $reaction ) {
+	public static function get_update_nonce(WordPoints_Hook_ReactionI $reaction) {
 
-		return wp_create_nonce( self::get_update_nonce_action( $reaction ) );
+		return wp_create_nonce(self::get_update_nonce_action($reaction));
 	}
 
 	/**
@@ -131,7 +131,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_update_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+		       . wp_json_encode($reaction->get_guid());
 	}
 
 	/**
@@ -143,9 +143,9 @@ 
 	 *
 	 * @return string A nonce for deleting this reaction.
 	 */
-	public static function get_delete_nonce( WordPoints_Hook_ReactionI $reaction ) {
+	public static function get_delete_nonce(WordPoints_Hook_ReactionI $reaction) {
 
-		return wp_create_nonce( self::get_delete_nonce_action( $reaction ) );
+		return wp_create_nonce(self::get_delete_nonce_action($reaction));
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 		return 'wordpoints_delete_hook_reaction|'
-		       . wp_json_encode( $reaction->get_guid() );
+		       . wp_json_encode($reaction->get_guid());
 	}
 
 	//
@@ -187,17 +187,17 @@ 
 
 		add_action(
 			'wp_ajax_wordpoints_admin_create_hook_reaction'
-			, array( $this, 'create_hook_reaction' )
+			, array($this, 'create_hook_reaction')
 		);
 
 		add_action(
 			'wp_ajax_wordpoints_admin_update_hook_reaction'
-			, array( $this, 'update_hook_reaction' )
+			, array($this, 'update_hook_reaction')
 		);
 
 		add_action(
 			'wp_ajax_wordpoints_admin_delete_hook_reaction'
-			, array( $this, 'delete_hook_reaction' )
+			, array($this, 'delete_hook_reaction')
 		);
 	}
 
@@ -212,11 +212,11 @@ 
 
 		$reaction_store = $this->get_reaction_store();
 
-		$this->verify_request( $this->get_create_nonce_action( $reaction_store ) );
+		$this->verify_request($this->get_create_nonce_action($reaction_store));
 
-		$reaction = $reaction_store->create_reaction( $this->get_data() );
+		$reaction = $reaction_store->create_reaction($this->get_data());
 
-		$this->send_json_result( $reaction, 'create' );
+		$this->send_json_result($reaction, 'create');
 	}
 
 	/**
@@ -231,14 +231,14 @@ 
 		$reaction_store = $this->get_reaction_store();
 		$reaction = $this->get_reaction();
 
-		$this->verify_request( $this->get_update_nonce_action( $reaction ) );
+		$this->verify_request($this->get_update_nonce_action($reaction));
 
 		$reaction = $reaction_store->update_reaction(
 			$reaction->ID
 			, $this->get_data()
 		);
 
-		$this->send_json_result( $reaction, 'update' );
+		$this->send_json_result($reaction, 'update');
 	}
 
 	/**
@@ -253,12 +253,12 @@ 
 		$reaction_store = $this->get_reaction_store();
 		$reaction = $this->get_reaction();
 
-		$this->verify_request( $this->get_delete_nonce_action( $reaction ) );
+		$this->verify_request($this->get_delete_nonce_action($reaction));
 
-		$result = $reaction_store->delete_reaction( $reaction->ID );
+		$result = $reaction_store->delete_reaction($reaction->ID);
 
-		if ( ! $result ) {
-			wp_send_json_error( array( 'message' => __( 'There was an error deleting the reaction. Please try again.', 'wordpoints' ) ) );
+		if ( ! $result) {
+			wp_send_json_error(array('message' => __('There was an error deleting the reaction. Please try again.', 'wordpoints')));
 		}
 
 		wp_send_json_success();
@@ -279,11 +279,11 @@ 
 	 *
 	 * @param string $debug_context Context sent with the message (for debugging).
 	 */
-	private function unexpected_error( $debug_context ) {
+	private function unexpected_error($debug_context) {
 
 		wp_send_json_error(
 			array(
-				'message' => __( 'There was an unexpected error. Try reloading the page.', 'wordpoints' ),
+				'message' => __('There was an unexpected error. Try reloading the page.', 'wordpoints'),
 				'debug'   => $debug_context,
 			)
 		);
@@ -299,8 +299,8 @@ 
 	 */
 	private function verify_user_can() {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_send_json_error( array( 'message' => __( 'You do not have permission to perform this action. Maybe you have been logged out?', 'wordpoints' ) ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_send_json_error(array('message' => __('You do not have permission to perform this action. Maybe you have been logged out?', 'wordpoints')));
 		}
 	}
 
@@ -313,14 +313,14 @@ 
 	 *
 	 * @param string $action The action the nonce should be for.
 	 */
-	private function verify_request( $action ) {
+	private function verify_request($action) {
 
 		if (
-			empty( $_POST['nonce'] )
-			|| ! wordpoints_verify_nonce( 'nonce', $action, null, 'post' )
+			empty($_POST['nonce'])
+			|| ! wordpoints_verify_nonce('nonce', $action, null, 'post')
 		) {
 			wp_send_json_error(
-				array( 'message' => __( 'Your security token for this action has expired. Refresh the page and try again.', 'wordpoints' ) )
+				array('message' => __('Your security token for this action has expired. Refresh the page and try again.', 'wordpoints'))
 			);
 		}
 	}
@@ -334,16 +334,16 @@ 
 	 */
 	protected function get_reaction_store() {
 
-		if ( ! isset( $_POST['reaction_store'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'reaction_store' );
+		if ( ! isset($_POST['reaction_store'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('reaction_store');
 		}
 
-		$reactor_slug = sanitize_key( $_POST['reaction_store'] ); // WPCS: CSRF OK.
+		$reactor_slug = sanitize_key($_POST['reaction_store']); // WPCS: CSRF OK.
 
-		$reaction_store = wordpoints_hooks()->get_reaction_store( $reactor_slug );
+		$reaction_store = wordpoints_hooks()->get_reaction_store($reactor_slug);
 
-		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI ) {
-			$this->unexpected_error( 'reaction_store_invalid' );
+		if ( ! $reaction_store instanceof WordPoints_Hook_Reaction_StoreI) {
+			$this->unexpected_error('reaction_store_invalid');
 		}
 
 		$this->reaction_store_slug = $reactor_slug;
@@ -361,16 +361,16 @@ 
 	 */
 	protected function get_reaction() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'id' );
+		if ( ! isset($_POST['id'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('id');
 		}
 
 		$reaction = $this->reaction_store->get_reaction(
-			wordpoints_int( $_POST['id'] ) // WPCS: CSRF OK.
+			wordpoints_int($_POST['id']) // WPCS: CSRF OK.
 		);
 
-		if ( ! $reaction ) {
-			wp_send_json_error( array( 'message' => __( 'The reaction ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints' ) ) );
+		if ( ! $reaction) {
+			wp_send_json_error(array('message' => __('The reaction ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints')));
 		}
 
 		return $reaction;
@@ -385,9 +385,9 @@ 
 	 */
 	protected function get_data() {
 
-		$data = wp_unslash( $_POST ); // WPCS: CSRF OK.
+		$data = wp_unslash($_POST); // WPCS: CSRF OK.
 
-		unset( $data['id'], $data['action'], $data['nonce'], $data['reaction_store'] );
+		unset($data['id'], $data['action'], $data['nonce'], $data['reaction_store']);
 
 		return $data;
 	}
@@ -400,30 +400,30 @@ 
 	 * @param mixed  $result The result of the action.
 	 * @param string $action The action being performed: 'create' or 'update'.
 	 */
-	private function send_json_result( $result, $action ) {
+	private function send_json_result($result, $action) {
 
-		if ( ! $result ) {
+		if ( ! $result) {
 
-			if ( 'create' === $action ) {
-				$message = __( 'There was an error adding the reaction. Please try again.', 'wordpoints' );
+			if ('create' === $action) {
+				$message = __('There was an error adding the reaction. Please try again.', 'wordpoints');
 			} else {
-				$message = __( 'There was an error updating the reaction. Please try again.', 'wordpoints' );
+				$message = __('There was an error updating the reaction. Please try again.', 'wordpoints');
 			}
 
-			wp_send_json_error( array( 'message' => $message ) );
+			wp_send_json_error(array('message' => $message));
 
-		} elseif ( $result instanceof WordPoints_Hook_Reaction_Validator ) {
+		} elseif ($result instanceof WordPoints_Hook_Reaction_Validator) {
 
-			wp_send_json_error( array( 'errors' => $result->get_errors() ) );
+			wp_send_json_error(array('errors' => $result->get_errors()));
 		}
 
 		$data = null;
 
-		if ( 'create' === $action ) {
-			$data = self::prepare_hook_reaction( $result );
+		if ('create' === $action) {
+			$data = self::prepare_hook_reaction($result);
 		}
 
-		wp_send_json_success( $data );
+		wp_send_json_success($data);
 	}
 }
 

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

Spacing +106 added lines, -106 removed lines

@@ -56,7 +56,7 @@ 
 	 * @since 1.0.0
 	 */
 	protected function get_title() {
-		return _x( 'Points Types', 'page title', 'wordpoints' );
+		return _x('Points Types', 'page title', 'wordpoints');
 	}
 
 	/**
@@ -66,8 +66,8 @@ 
 
 		parent::hooks();
 
-		add_action( 'add_meta_boxes', array( $this, 'add_points_settings_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_event_meta_boxes'));
 	}
 
 	/**
@@ -75,9 +75,9 @@ 
 	 */
 	public function enqueue_scripts() {
 
-		wp_enqueue_style( 'wordpoints-hooks-admin' );
+		wp_enqueue_style('wordpoints-hooks-admin');
 
-		wp_enqueue_script( 'postbox' );
+		wp_enqueue_script('postbox');
 
 		wordpoints_hooks_ui_setup_script_data();
 	}
@@ -97,7 +97,7 @@ 
 					.addClass( 'closed' );
 
 				postboxes.add_postbox_toggles(
-					<?php echo wp_json_encode( $this->id ); ?>
+					<?php echo wp_json_encode($this->id); ?>
 				);
 			} );
 		</script>
@@ -112,14 +112,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'
@@ -133,7 +133,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;
 		}
 
@@ -141,12 +141,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(
@@ -161,15 +161,15 @@ 
 		?>
 
 		<form method="post">
-			<?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
@@ -183,45 +183,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
@@ -235,28 +235,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"/>
@@ -273,17 +273,17 @@ 
 	 */
 	public function add_event_meta_boxes() {
 
-		if ( ! $this->current_points_type ) {
+		if ( ! $this->current_points_type) {
 			return;
 		}
 
 		/** @var WordPoints_Hook_EventI $event */
-		foreach ( $this->hooks->events->get_all() as $slug => $event ) {
+		foreach ($this->hooks->events->get_all() as $slug => $event) {
 
 			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'
@@ -304,54 +304,54 @@ 
 	 * @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();
 
-		$reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+		$reaction_store = wordpoints_hooks()->get_reaction_store('points');
 
-		foreach ( $reaction_store->get_reactions_to_event( $event_slug ) as $id => $reaction ) {
-			if ( $reaction->get_meta( 'points_type' ) === $this->current_points_type ) {
+		foreach ($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
 				);
 			}
 		}
 
-		$args = $this->hooks->events->args->get_children( $event_slug );
+		$args = $this->hooks->events->args->get_children($event_slug);
 
-		$event_data = array( 'args' => array() );
+		$event_data = array('args' => array());
 
-		foreach ( $args as $slug => $arg ) {
+		foreach ($args as $slug => $arg) {
 
-			$event_data['args'][ $slug ] = array(
+			$event_data['args'][$slug] = array(
 				'slug' => $slug,
 			);
 
-			if ( $arg instanceof WordPoints_Hook_Arg ) {
-				$event_data['args'][ $slug ]['title'] = $arg->get_title();
-				$event_data['args'][ $slug ]['is_stateful'] = $arg->is_stateful();
+			if ($arg instanceof WordPoints_Hook_Arg) {
+				$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( $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($reaction_store)); ?>"
 				data-wordpoints-hooks-reaction-store="points"
 				data-wordpoints-hooks-reactor="points">
 			</div>
@@ -366,7 +366,7 @@ 
 
 			<div class="controls">
 				<button type="button" class="button-primary add-reaction">
-					<?php esc_html_e( 'Add New', 'wordpoints' ); ?>
+					<?php esc_html_e('Add New', 'wordpoints'); ?>
 				</button>
 			</div>
 		</div>
@@ -386,19 +386,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;
 	}
@@ -410,7 +410,7 @@ 
 	 */
 	public function save_points_type() {
 
-		if ( ! current_user_can( 'manage_wordpoints_points_types' ) ) {
+		if ( ! current_user_can('manage_wordpoints_points_types')) {
 			return;
 		}
 
@@ -425,43 +425,43 @@ 
 
 			$settings = array();
 
-			$settings['name']   = trim( sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) ); // WPCS: CSRF OK
-			$settings['prefix'] = ltrim( sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) ); // WPCS: CSRF OK
-			$settings['suffix'] = rtrim( sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) ); // WPCS: CSRF OK
+			$settings['name']   = trim(sanitize_text_field(wp_unslash($_POST['points-name']))); // WPCS: CSRF OK
+			$settings['prefix'] = ltrim(sanitize_text_field(wp_unslash($_POST['points-prefix']))); // WPCS: CSRF OK
+			$settings['suffix'] = rtrim(sanitize_text_field(wp_unslash($_POST['points-suffix']))); // WPCS: CSRF OK
 
 			if (
-				isset( $_POST['points-slug'] )
-				&& wordpoints_verify_nonce( 'update_points_type', 'wordpoints_update_points_type-%s', array( 'points-slug' ), 'post' )
+				isset($_POST['points-slug'])
+				&& wordpoints_verify_nonce('update_points_type', 'wordpoints_update_points_type-%s', array('points-slug'), 'post')
 			) {
 
 				// - We are updating an existing points type.
 
-				$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')
 						, 'updated'
 					);
 
 					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')
 						, 'updated'
 					);
 
@@ -470,23 +470,23 @@ 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_update'
-						, __( 'Points type updated.', 'wordpoints' )
+						, __('Points type updated.', 'wordpoints')
 						, 'updated'
 					);
 				}
 
-			} elseif ( wordpoints_verify_nonce( 'add_new', 'wordpoints_add_new_points_type', null, 'post' ) ) {
+			} elseif (wordpoints_verify_nonce('add_new', 'wordpoints_add_new_points_type', null, 'post')) {
 
 				// - We are creating a new points type.
 
-				$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 {
@@ -496,26 +496,26 @@ 
 					add_settings_error(
 						''
 						, 'wordpoints_points_type_create'
-						, __( 'Points type created.', 'wordpoints' )
+						, __('Points type created.', 'wordpoints')
 						, 'updated'
 					);
 				}
 			}
 
 		} elseif (
-			! empty( $_POST['delete-points-type'] )
-			&& isset( $_POST['points-slug'] )
-			&& wordpoints_verify_nonce( 'delete-points-type-nonce', 'wordpoints_delete_points_type-%s', array( 'points-slug' ), 'post' )
+			! empty($_POST['delete-points-type'])
+			&& isset($_POST['points-slug'])
+			&& wordpoints_verify_nonce('delete-points-type-nonce', 'wordpoints_delete_points_type-%s', array('points-slug'), 'post')
 		) {
 
 			// - We are deleting a points type.
 
-			if ( wordpoints_delete_points_type( sanitize_key( $_POST['points-slug'] ) ) ) {
+			if (wordpoints_delete_points_type(sanitize_key($_POST['points-slug']))) {
 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Points type deleted.', 'wordpoints' )
+					, __('Points type deleted.', 'wordpoints')
 					, 'updated'
 				);
 
@@ -524,7 +524,7 @@ 
 				add_settings_error(
 					''
 					, 'wordpoints_points_type_delete'
-					, __( 'Error while deleting.', 'wordpoints' )
+					, __('Error while deleting.', 'wordpoints')
 				);
 			}
 		}
@@ -540,25 +540,25 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_admin_points_events_head' );
+		do_action('wordpoints_admin_points_events_head');
 
-		if ( is_network_admin() ) {
-			$title = __( 'Network Events', 'wordpoints' );
-			$description = __( 'Award points when various events happen on this network.', 'wordpoints' );
+		if (is_network_admin()) {
+			$title = __('Network Events', 'wordpoints');
+			$description = __('Award points when various events happen on this network.', 'wordpoints');
 		} else {
-			$title = __( 'Events', 'wordpoints' );
-			$description = __( 'Award points when various events happen on this site.', 'wordpoints' );
+			$title = __('Events', 'wordpoints');
+			$description = __('Award points when various events happen on this site.', 'wordpoints');
 		}
 
-		$points_type = wordpoints_get_points_type( $this->current_points_type );
+		$points_type = wordpoints_get_points_type($this->current_points_type);
 
 		?>
 
 		<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">
@@ -566,17 +566,17 @@ 
 					<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>
-								<p class="description"><?php echo esc_html( $description ); ?></p>
+								<h2><?php echo esc_html($title); ?></h2>
+								<p class="description"><?php echo esc_html($description); ?></p>
 							</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; ?>
 
@@ -595,7 +595,7 @@ 
 		 *
 		 * @since 1.0.0
 		 */
-		do_action( 'wordpoints_admin_points_events_foot' );
+		do_action('wordpoints_admin_points_events_foot');
 	}
 }
 

src/includes/functions.php

Spacing +129 added lines, -129 removed lines

@@ -33,15 +33,15 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $reactors The reactors registry.
  */
-function wordpoints_hook_reactors_init( $reactors ) {
+function wordpoints_hook_reactors_init($reactors) {
 
-	$reactors->register( 'points', 'WordPoints_Hook_Reactor_Points' );
+	$reactors->register('points', 'WordPoints_Hook_Reactor_Points');
 
 	$router = wordpoints_hooks()->router;
 
-	$router->add_action_type_to_reactor( 'fire', 'points', 'award' );
-	$router->add_action_type_to_reactor( 'toggle_on', 'points', 'award' );
-	$router->add_action_type_to_reactor( 'toggle_off', 'points', 'reverse' );
+	$router->add_action_type_to_reactor('fire', 'points', 'award');
+	$router->add_action_type_to_reactor('toggle_on', 'points', 'award');
+	$router->add_action_type_to_reactor('toggle_off', 'points', 'reverse');
 }
 
 /**
@@ -53,7 +53,7 @@ 
  *
  * @param WordPoints_Class_Registry_Children $reaction_stores The store registry.
  */
-function wordpoints_hook_reaction_stores_init( $reaction_stores ) {
+function wordpoints_hook_reaction_stores_init($reaction_stores) {
 
 	$reaction_stores->register(
 		'standard'
@@ -61,7 +61,7 @@ 
 		, 'WordPoints_Hook_Reaction_Store_Options'
 	);
 
-	if ( is_wordpoints_network_active() ) {
+	if (is_wordpoints_network_active()) {
 		$reaction_stores->register(
 			'network'
 			, 'points'
@@ -79,11 +79,11 @@ 
  *
  * @param WordPoints_Class_Registry_Persistent $extensions The extension registry.
  */
-function wordpoints_hook_extension_init( $extensions ) {
+function wordpoints_hook_extension_init($extensions) {
 
-	$extensions->register( 'blocker', 'WordPoints_Hook_Extension_Blocker' );
-	$extensions->register( 'conditions', 'WordPoints_Hook_Extension_Conditions' );
-	$extensions->register( 'periods', 'WordPoints_Hook_Extension_Periods' );
+	$extensions->register('blocker', 'WordPoints_Hook_Extension_Blocker');
+	$extensions->register('conditions', 'WordPoints_Hook_Extension_Conditions');
+	$extensions->register('periods', 'WordPoints_Hook_Extension_Periods');
 }
 
 /**
@@ -95,7 +95,7 @@ 
  *
  * @param WordPoints_Class_Registry_Children $conditions The conditions registry.
  */
-function wordpoints_hook_conditions_init( $conditions ) {
+function wordpoints_hook_conditions_init($conditions) {
 
 	$conditions->register(
 		'text'
@@ -131,7 +131,7 @@ 
  *
  * @param WordPoints_Hook_Actions $actions The action registry.
  */
-function wordpoints_hook_actions_init( $actions ) {
+function wordpoints_hook_actions_init($actions) {
 
 	$actions->register(
 		'comment_approve'
@@ -139,8 +139,8 @@ 
 		, array(
 			'action' => 'transition_comment_status',
 			'data'   => array(
-				'arg_index'    => array( 'comment' => 2 ),
-				'requirements' => array( 0 => 'approved' ),
+				'arg_index'    => array('comment' => 2),
+				'requirements' => array(0 => 'approved'),
 			),
 		)
 	);
@@ -151,7 +151,7 @@ 
 		, array(
 			'action' => 'wp_insert_comment',
 			'data'   => array(
-				'arg_index' => array( 'comment' => 1 ),
+				'arg_index' => array('comment' => 1),
 			),
 		)
 	);
@@ -162,8 +162,8 @@ 
 		, array(
 			'action' => 'transition_comment_status',
 			'data'   => array(
-				'arg_index' => array( 'comment' => 2 ),
-				'requirements' => array( 1 => 'approved' ),
+				'arg_index' => array('comment' => 2),
+				'requirements' => array(1 => 'approved'),
 			),
 		)
 	);
@@ -175,8 +175,8 @@ 
 		, array(
 			'action' => 'transition_post_status',
 			'data'   => array(
-				'arg_index' => array( 'post' => 2 ),
-				'requirements' => array( 0 => 'publish' ),
+				'arg_index' => array('post' => 2),
+				'requirements' => array(0 => 'publish'),
 			),
 		)
 	);
@@ -187,8 +187,8 @@ 
 		, array(
 			'action' => 'transition_post_status',
 			'data'   => array(
-				'arg_index' => array( 'post' => 2 ),
-				'requirements' => array( 1 => 'publish' ),
+				'arg_index' => array('post' => 2),
+				'requirements' => array(1 => 'publish'),
 			),
 		)
 	);
@@ -199,7 +199,7 @@ 
 		, array(
 			'action' => 'add_attachment',
 			'data'   => array(
-				'arg_index' => array( 'post\attachment' => 0 ),
+				'arg_index' => array('post\attachment' => 0),
 			),
 		)
 	);
@@ -210,7 +210,7 @@ 
 		, array(
 			'action' => 'delete_post',
 			'data'   => array(
-				'arg_index' => array( 'post' => 0 ),
+				'arg_index' => array('post' => 0),
 			),
 		)
 	);
@@ -221,7 +221,7 @@ 
 		, array(
 			'action' => 'user_register',
 			'data'   => array(
-				'arg_index' => array( 'user' => 0 ),
+				'arg_index' => array('user' => 0),
 			),
 		)
 	);
@@ -232,7 +232,7 @@ 
 		, array(
 			'action' => is_multisite() ? 'wpmu_delete_user' : 'delete_user',
 			'data'   => array(
-				'arg_index' => array( 'user' => 0 ),
+				'arg_index' => array('user' => 0),
 			),
 		)
 	);
@@ -255,7 +255,7 @@ 
  *
  * @param WordPoints_Hook_Events $events The event registry.
  */
-function wordpoints_hook_events_init( $events ) {
+function wordpoints_hook_events_init($events) {
 
 	$events->register(
 		'user_register'
@@ -285,7 +285,7 @@ 
 	);
 
 	// Register events for all of the public post types.
-	$post_types = get_post_types( array( 'public' => true ) );
+	$post_types = get_post_types(array('public' => true));
 
 	/**
 	 * Filter which post types to register hook events for.
@@ -294,20 +294,20 @@ 
 	 *
 	 * @param string[] The post type slugs ("names").
 	 */
-	$post_types = apply_filters( 'wordpoints_register_hook_events_for_post_types', $post_types );
+	$post_types = apply_filters('wordpoints_register_hook_events_for_post_types', $post_types);
 
-	foreach ( $post_types as $slug ) {
-		wordpoints_register_post_type_hook_events( $slug );
+	foreach ($post_types as $slug) {
+		wordpoints_register_post_type_hook_events($slug);
 	}
 
-	if ( is_multisite() ) {
+	if (is_multisite()) {
 
 		$event_slugs = array(
 			'user_visit',
 			'user_register',
 		);
 
-		foreach ( $event_slugs as $event_slug ) {
+		foreach ($event_slugs as $event_slug) {
 			// TODO network hooks
 			$events->args->register(
 				$event_slug
@@ -327,10 +327,10 @@ 
  *
  * @param WordPoints_App_Registry $entities The entities app.
  */
-function wordpoints_entities_app_init( $entities ) {
+function wordpoints_entities_app_init($entities) {
 
-	$entities->sub_apps->register( 'children', 'WordPoints_Class_Registry_Children' );
-	$entities->sub_apps->register( 'contexts', 'WordPoints_Class_Registry' );
+	$entities->sub_apps->register('children', 'WordPoints_Class_Registry_Children');
+	$entities->sub_apps->register('contexts', 'WordPoints_Class_Registry');
 }
 
 /**
@@ -342,10 +342,10 @@ 
  *
  * @param WordPoints_Class_Registry $contexts The entity context registry.
  */
-function wordpoints_entity_contexts_init( $contexts ) {
+function wordpoints_entity_contexts_init($contexts) {
 
-	$contexts->register( 'network', 'WordPoints_Entity_Context_Network' );
-	$contexts->register( 'site', 'WordPoints_Entity_Context_Site' );
+	$contexts->register('network', 'WordPoints_Entity_Context_Network');
+	$contexts->register('site', 'WordPoints_Entity_Context_Site');
 }
 
 /**
@@ -357,18 +357,18 @@ 
  *
  * @param WordPoints_App_Registry $entities The entities app.
  */
-function wordpoints_entities_init( $entities ) {
+function wordpoints_entities_init($entities) {
 
 	$children = $entities->children;
 
-	$entities->register( 'user', 'WordPoints_Entity_User' );
-	$children->register( 'user', 'roles', 'WordPoints_Entity_User_Roles' );
+	$entities->register('user', 'WordPoints_Entity_User');
+	$children->register('user', 'roles', 'WordPoints_Entity_User_Roles');
 
-	$entities->register( 'user_role', 'WordPoints_Entity_User_Role' );
-	$children->register( 'user_role', 'name', 'WordPoints_Entity_User_Role_Name' );
+	$entities->register('user_role', 'WordPoints_Entity_User_Role');
+	$children->register('user_role', 'name', 'WordPoints_Entity_User_Role_Name');
 
 	// Register entities for all of the public post types.
-	$post_types = get_post_types( array( 'public' => true ) );
+	$post_types = get_post_types(array('public' => true));
 
 	/**
 	 * Filter which post types to register entities for.
@@ -377,14 +377,14 @@ 
 	 *
 	 * @param string[] The post type slugs ("names").
 	 */
-	$post_types = apply_filters( 'wordpoints_register_entities_for_post_types', $post_types );
+	$post_types = apply_filters('wordpoints_register_entities_for_post_types', $post_types);
 
-	foreach ( $post_types as $slug ) {
-		wordpoints_register_post_type_entities( $slug );
+	foreach ($post_types as $slug) {
+		wordpoints_register_post_type_entities($slug);
 	}
 
 	// Register entities for all of the public taxonomies.
-	$taxonomies = get_taxonomies( array( 'public' => true ) );
+	$taxonomies = get_taxonomies(array('public' => true));
 
 	/**
 	 * Filter which taxonomies to register entities for.
@@ -393,10 +393,10 @@ 
 	 *
 	 * @param string[] The taxonomy slugs.
 	 */
-	$taxonomies = apply_filters( 'wordpoints_register_entities_for_taxonomies', $taxonomies );
+	$taxonomies = apply_filters('wordpoints_register_entities_for_taxonomies', $taxonomies);
 
-	foreach ( $taxonomies as $slug ) {
-		wordpoints_register_taxonomy_entities( $slug );
+	foreach ($taxonomies as $slug) {
+		wordpoints_register_taxonomy_entities($slug);
 	}
 }
 
@@ -407,28 +407,28 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_register_post_type_entities( $slug ) {
+function wordpoints_register_post_type_entities($slug) {
 
 	$entities = wordpoints_entities();
 	$children = $entities->children;
 
-	$entities->register( "post\\{$slug}", 'WordPoints_Entity_Post' );
-	$children->register( "post\\{$slug}", 'author', 'WordPoints_Entity_Post_Author' );
+	$entities->register("post\\{$slug}", 'WordPoints_Entity_Post');
+	$children->register("post\\{$slug}", 'author', 'WordPoints_Entity_Post_Author');
 
-	$supports = get_all_post_type_supports( $slug );
+	$supports = get_all_post_type_supports($slug);
 
-	if ( isset( $supports['editor'] ) ) {
-		$children->register( "post\\{$slug}", 'content', 'WordPoints_Entity_Post_Content' );
+	if (isset($supports['editor'])) {
+		$children->register("post\\{$slug}", 'content', 'WordPoints_Entity_Post_Content');
 	}
 
-	if ( isset( $supports['comments'] ) ) {
-		$entities->register( "comment\\{$slug}", 'WordPoints_Entity_Comment' );
-		$children->register( "comment\\{$slug}", "post\\{$slug}", 'WordPoints_Entity_Comment_Post' );
-		$children->register( "comment\\{$slug}", 'author', 'WordPoints_Entity_Comment_Author' );
+	if (isset($supports['comments'])) {
+		$entities->register("comment\\{$slug}", 'WordPoints_Entity_Comment');
+		$children->register("comment\\{$slug}", "post\\{$slug}", 'WordPoints_Entity_Comment_Post');
+		$children->register("comment\\{$slug}", 'author', 'WordPoints_Entity_Comment_Author');
 	}
 
-	foreach ( get_object_taxonomies( $slug ) as $taxonomy_slug ) {
-		$children->register( "post\\{$slug}", "terms\\{$taxonomy_slug}", 'WordPoints_Entity_Post_Terms' );
+	foreach (get_object_taxonomies($slug) as $taxonomy_slug) {
+		$children->register("post\\{$slug}", "terms\\{$taxonomy_slug}", 'WordPoints_Entity_Post_Terms');
 	}
 
 	/**
@@ -438,7 +438,7 @@ 
 	 *
 	 * @param string $slug The slug ("name") of the post type.
 	 */
-	do_action( 'wordpoints_register_post_type_entities', $slug );
+	do_action('wordpoints_register_post_type_entities', $slug);
 }
 
 /**
@@ -448,13 +448,13 @@ 
  *
  * @param string $slug The slug of the post type.
  */
-function wordpoints_register_post_type_hook_events( $slug ) {
+function wordpoints_register_post_type_hook_events($slug) {
 
 	$event_slugs = array();
 
 	$events = wordpoints_hooks()->events;
 
-	if ( 'attachment' === $slug ) {
+	if ('attachment' === $slug) {
 
 		$event_slugs[] = 'media_upload';
 
@@ -482,7 +482,7 @@ 
 			, array(
 				'actions' => array(
 					'toggle_on'  => 'post_publish',
-					'toggle_off' => array( 'post_depublish', 'post_delete' ),
+					'toggle_off' => array('post_depublish', 'post_delete'),
 				),
 				'args'    => array(
 					"post\\{$slug}" => 'WordPoints_Hook_Arg_Dynamic',
@@ -491,7 +491,7 @@ 
 		);
 	}
 
-	if ( post_type_supports( $slug, 'comments' ) ) {
+	if (post_type_supports($slug, 'comments')) {
 
 		$event_slugs[] = "comment_leave\\{$slug}";
 
@@ -500,7 +500,7 @@ 
 			, 'WordPoints_Hook_Event_Comment_Leave'
 			, array(
 				'actions' => array(
-					'toggle_on'  => array( 'comment_approve', 'comment_new' ),
+					'toggle_on'  => array('comment_approve', 'comment_new'),
 					'toggle_off' => 'comment_deapprove',
 				),
 				'args' => array(
@@ -510,8 +510,8 @@ 
 		);
 	}
 
-	if ( is_multisite() ) {
-		foreach ( $event_slugs as $event_slug ) {
+	if (is_multisite()) {
+		foreach ($event_slugs as $event_slug) {
 			$events->args->register(
 				$event_slug
 				, 'current:site'
@@ -527,7 +527,7 @@ 
 	 *
 	 * @param string $slug The slug ("name") of the post type.
 	 */
-	do_action( 'wordpoints_register_post_type_hook_events', $slug );
+	do_action('wordpoints_register_post_type_hook_events', $slug);
 }
 
 /**
@@ -537,13 +537,13 @@ 
  *
  * @param string $slug The slug of the taxonomy.
  */
-function wordpoints_register_taxonomy_entities( $slug ) {
+function wordpoints_register_taxonomy_entities($slug) {
 
 	$entities = wordpoints_entities();
 	$children = $entities->children;
 
-	$entities->register( "term\\{$slug}", 'WordPoints_Entity_Term' );
-	$children->register( "term\\{$slug}", 'id', 'WordPoints_Entity_Term_Id' );
+	$entities->register("term\\{$slug}", 'WordPoints_Entity_Term');
+	$children->register("term\\{$slug}", 'id', 'WordPoints_Entity_Term_Id');
 
 	/**
 	 * Fired when registering the entities for a taxonomy.
@@ -552,7 +552,7 @@ 
 	 *
 	 * @param string $slug The taxonomy's slug.
 	 */
-	do_action( 'wordpoints_register_taxonomy_entities', $slug );
+	do_action('wordpoints_register_taxonomy_entities', $slug);
 }
 
 /**
@@ -564,10 +564,10 @@ 
  *
  * @param WordPoints_Class_RegistryI $data_types The data types registry.
  */
-function wordpoints_data_types_init( $data_types ) {
+function wordpoints_data_types_init($data_types) {
 
-	$data_types->register( 'integer', 'WordPoints_Data_Type_Integer' );
-	$data_types->register( 'text', 'WordPoints_Data_Type_Text' );
+	$data_types->register('integer', 'WordPoints_Data_Type_Integer');
+	$data_types->register('text', 'WordPoints_Data_Type_Text');
 }
 
 /**
@@ -582,9 +582,9 @@ 
  *
  * @return bool Whether the user can view the points log.
  */
-function wordpoints_hooks_user_can_view_points_log( $can_view, $log ) {
+function wordpoints_hooks_user_can_view_points_log($can_view, $log) {
 
-	if ( ! $can_view ) {
+	if ( ! $can_view) {
 		return $can_view;
 	}
 
@@ -593,11 +593,11 @@ 
 	$event_slug = $log->log_type;
 
 	/** @var WordPoints_Hook_Arg $arg */
-	foreach ( wordpoints_hooks()->events->args->get_children( $event_slug ) as $slug => $arg ) {
+	foreach (wordpoints_hooks()->events->args->get_children($event_slug) as $slug => $arg) {
 
-		$value = wordpoints_get_points_log_meta( $log->id, $slug, true );
+		$value = wordpoints_get_points_log_meta($log->id, $slug, true);
 
-		if ( ! $value ) {
+		if ( ! $value) {
 			continue;
 		}
 
@@ -607,7 +607,7 @@ 
 			, $value
 		);
 
-		if ( ! $can_view ) {
+		if ( ! $can_view) {
 			break;
 		}
 	}
@@ -626,20 +626,20 @@ 
  *
  * @return bool Whether the user can view this entity.
  */
-function wordpoints_entity_user_can_view( $user_id, $entity_slug, $entity_id ) {
+function wordpoints_entity_user_can_view($user_id, $entity_slug, $entity_id) {
 
-	$entity = wordpoints_entities()->get( $entity_slug );
+	$entity = wordpoints_entities()->get($entity_slug);
 
 	// If this entity type is not found, we have no way of determining whether it is
 	// safe for the user to view it.
-	if ( ! ( $entity instanceof WordPoints_Entity ) ) {
+	if ( ! ($entity instanceof WordPoints_Entity)) {
 		return false;
 	}
 
 	$can_view = true;
 
-	if ( $entity instanceof WordPoints_Entity_Restricted_VisibilityI ) {
-		$can_view = $entity->user_can_view( $user_id, $entity_id );
+	if ($entity instanceof WordPoints_Entity_Restricted_VisibilityI) {
+		$can_view = $entity->user_can_view($user_id, $entity_id);
 	}
 
 	/**
@@ -670,8 +670,8 @@ 
  */
 function wordpoints_apps() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
-		WordPoints_App::$main = new WordPoints_App( 'apps' );
+	if ( ! isset(WordPoints_App::$main)) {
+		WordPoints_App::$main = new WordPoints_App('apps');
 	}
 
 	return WordPoints_App::$main;
@@ -686,7 +686,7 @@ 
  */
 function wordpoints_hooks() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
+	if ( ! isset(WordPoints_App::$main)) {
 		wordpoints_apps();
 	}
 
@@ -702,7 +702,7 @@ 
  */
 function wordpoints_entities() {
 
-	if ( ! isset( WordPoints_App::$main ) ) {
+	if ( ! isset(WordPoints_App::$main)) {
 		wordpoints_apps();
 	}
 
@@ -718,13 +718,13 @@ 
  *
  * @param WordPoints_App $app The main apps app.
  */
-function wordpoints_apps_init( $app ) {
+function wordpoints_apps_init($app) {
 
 	$apps = $app->sub_apps;
 
-	$apps->register( 'hooks', 'WordPoints_Hooks' );
-	$apps->register( 'entities', 'WordPoints_App_Registry' );
-	$apps->register( 'data_types', 'WordPoints_Class_Registry' );
+	$apps->register('hooks', 'WordPoints_Hooks');
+	$apps->register('entities', 'WordPoints_App_Registry');
+	$apps->register('data_types', 'WordPoints_Class_Registry');
 }
 
 /**
@@ -737,19 +737,19 @@ 
  *
  * @return object|false The constructed object, or false if to many args were passed.
  */
-function wordpoints_construct_class_with_args( $class_name, array $args ) {
+function wordpoints_construct_class_with_args($class_name, array $args) {
 
-	switch ( count( $args ) ) {
+	switch (count($args)) {
 		case 0:
 			return new $class_name();
 		case 1:
-			return new $class_name( $args[0] );
+			return new $class_name($args[0]);
 		case 2:
-			return new $class_name( $args[0], $args[1] );
+			return new $class_name($args[0], $args[1]);
 		case 3:
-			return new $class_name( $args[0], $args[1], $args[2] );
+			return new $class_name($args[0], $args[1], $args[2]);
 		case 4:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3] );
+			return new $class_name($args[0], $args[1], $args[2], $args[3]);
 		default:
 			return false;
 	}
@@ -771,13 +771,13 @@ 
  * @return array The slug parsed into the 'generic' and 'dynamic' portions. If the
  *               slug is not dynamic, the value of each of those keys will be false.
  */
-function wordpoints_parse_dynamic_slug( $slug ) {
+function wordpoints_parse_dynamic_slug($slug) {
 
-	$parsed = array( 'dynamic' => false, 'generic' => false );
+	$parsed = array('dynamic' => false, 'generic' => false);
 
-	$parts = explode( '\\', $slug, 2 );
+	$parts = explode('\\', $slug, 2);
 
-	if ( isset( $parts[1] ) ) {
+	if (isset($parts[1])) {
 		$parsed['dynamic'] = $parts[1];
 		$parsed['generic'] = $parts[0];
 	}
@@ -816,28 +816,28 @@ 
  *                     contexts, indexed by context slug, or false if any of the
  *                     contexts isn't current.
  */
-function wordpoints_entities_get_current_context_id( $slug ) {
+function wordpoints_entities_get_current_context_id($slug) {
 
 	$current_context = array();
 
 	/** @var WordPoints_Class_Registry $contexts */
 	$contexts = wordpoints_entities()->contexts;
 
-	while ( $slug ) {
+	while ($slug) {
 
-		$context = $contexts->get( $slug );
+		$context = $contexts->get($slug);
 
-		if ( ! $context instanceof WordPoints_Entity_Context ) {
+		if ( ! $context instanceof WordPoints_Entity_Context) {
 			return false;
 		}
 
 		$id = $context->get_current_id();
 
-		if ( false === $id ) {
+		if (false === $id) {
 			return false;
 		}
 
-		$current_context[ $slug ] = $id;
+		$current_context[$slug] = $id;
 
 		$slug = $context->get_parent_slug();
 	}
@@ -858,22 +858,22 @@ 
  */
 function wordpoints_is_network_context() {
 
-	if ( ! is_multisite() ) {
+	if ( ! is_multisite()) {
 		return false;
 	}
 
-	if ( is_network_admin() ) {
+	if (is_network_admin()) {
 		return true;
 	}
 
 	// See https://core.trac.wordpress.org/ticket/22589
 	if (
-		defined( 'DOING_AJAX' )
+		defined('DOING_AJAX')
 		&& DOING_AJAX
-		&& isset( $_SERVER['HTTP_REFERER'] )
+		&& isset($_SERVER['HTTP_REFERER'])
 		&& preg_match(
-			'#^' . preg_quote( network_admin_url(), '#' ) . '#i'
-			, esc_url_raw( wp_unslash( $_SERVER['HTTP_REFERER'] ) )
+			'#^'.preg_quote(network_admin_url(), '#').'#i'
+			, esc_url_raw(wp_unslash($_SERVER['HTTP_REFERER']))
 		)
 	) {
 		return true;
@@ -886,7 +886,7 @@ 
 	 *
 	 * @param bool $in_network_context Whether we are in network context.
 	 */
-	return apply_filters( 'wordpoints_is_network_context', false );
+	return apply_filters('wordpoints_is_network_context', false);
 }
 
 /**
@@ -900,21 +900,21 @@ 
  *
  * @return string The primary arg's GUID, JSON encoded.
  */
-function wordpoints_hooks_get_event_primary_arg_guid_json( WordPoints_Hook_Event_Args $event_args ) {
+function wordpoints_hooks_get_event_primary_arg_guid_json(WordPoints_Hook_Event_Args $event_args) {
 
 	$entity = $event_args->get_primary_arg();
 
-	if ( ! $entity ) {
+	if ( ! $entity) {
 		return '';
 	}
 
 	$the_guid = $entity->get_the_guid();
 
-	if ( ! $the_guid ) {
+	if ( ! $the_guid) {
 		return '';
 	}
 
-	return wp_json_encode( $the_guid );
+	return wp_json_encode($the_guid);
 }
 
 /**
@@ -927,7 +927,7 @@ 
  */
 function wordpoints_hooks_api_add_global_cache_groups() {
 
-	if ( function_exists( 'wp_cache_add_global_groups' ) ) {
+	if (function_exists('wp_cache_add_global_groups')) {
 
 		wp_cache_add_global_groups(
 			array(
@@ -953,8 +953,8 @@ 
  * @return string The escaped identifier. Already quoted, do not place within
  *                backticks.
  */
-function wordpoints_escape_mysql_identifier( $identifier ) {
-	return '`' . str_replace( '`', '``', $identifier ) . '`';
+function wordpoints_escape_mysql_identifier($identifier) {
+	return '`'.str_replace('`', '``', $identifier).'`';
 }
 
 // EOF

src/includes/classes/entity/attr/field.php

Spacing +2 added lines, -2 removed lines

@@ -37,8 +37,8 @@
 	/**
 	 * @since 1.0.0
 	 */
-	protected function get_attr_value_from_entity( WordPoints_Entity $entity ) {
-		return $entity->get_the_attr_value( $this->field );
+	protected function get_attr_value_from_entity(WordPoints_Entity $entity) {
+		return $entity->get_the_attr_value($this->field);
 	}
 	
 	/**

src/includes/classes/entity/stored/array.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	public function get_storage_info() {
 		return array(
 			'type' => 'array',
-			'info' => array( 'type' => 'method' ),
+			'info' => array('type' => 'method'),
 		);
 	}
 }

src/includes/classes/entity/comment/post.php

Spacing +2 added lines, -2 removed lines

@@ -34,8 +34,8 @@
 	/**
 	 * @since 1.0.0
 	 */
-	protected function get_related_entity_ids( WordPoints_Entity $entity ) {
-		return $entity->get_the_attr_value( $this->related_ids_field );
+	protected function get_related_entity_ids(WordPoints_Entity $entity) {
+		return $entity->get_the_attr_value($this->related_ids_field);
 	}
 
 	/**

src/includes/classes/entity/relationship/stored/field.php

Spacing +2 added lines, -2 removed lines

@@ -37,8 +37,8 @@
 	/**
 	 * @since 1.0.0
 	 */
-	protected function get_related_entity_ids( WordPoints_Entity $entity ) {
-		return $entity->get_the_attr_value( $this->related_ids_field );
+	protected function get_related_entity_ids(WordPoints_Entity $entity) {
+		return $entity->get_the_attr_value($this->related_ids_field);
 	}
 	
 	/**

src/includes/classes/entity/attr.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
 	 *
 	 * @return mixed The attribute value.
 	 */
-	abstract protected function get_attr_value_from_entity( WordPoints_Entity $entity );
+	abstract protected function get_attr_value_from_entity(WordPoints_Entity $entity);
 
 	/**
 	 * Get the data type of this attribute's values.
@@ -66,9 +66,9 @@ 
 	 *
 	 * @return bool Whether the value was set correctly.
 	 */
-	public function set_the_value_from_entity( WordPoints_Entity $entity ) {
+	public function set_the_value_from_entity(WordPoints_Entity $entity) {
 
-		$this->the_value = $this->get_attr_value_from_entity( $entity );
+		$this->the_value = $this->get_attr_value_from_entity($entity);
 
 		return true;
 	}