WordPoints / hooks-api

src/admin/includes/actions.php

Spacing +7 added lines, -7 removed lines

@@ -7,17 +7,17 @@
  * @since 1.0.0
  */
 
-add_action( 'wordpoints_init_app-apps', 'wordpoints_hooks_register_admin_apps' );
+add_action('wordpoints_init_app-apps', 'wordpoints_hooks_register_admin_apps');
 
-add_action( 'admin_menu', 'wordpoints_hooks_api_admin_menu' );
+add_action('admin_menu', 'wordpoints_hooks_api_admin_menu');
 
-if ( is_wordpoints_network_active() ) {
-	add_action( 'network_admin_menu', 'wordpoints_hooks_api_admin_menu' );
+if (is_wordpoints_network_active()) {
+	add_action('network_admin_menu', 'wordpoints_hooks_api_admin_menu');
 }
 
-add_action( 'admin_init', 'wordpoints_hooks_admin_register_scripts' );
-add_action( 'admin_init', 'wordpoints_hooks_admin_ajax' );
+add_action('admin_init', 'wordpoints_hooks_admin_register_scripts');
+add_action('admin_init', 'wordpoints_hooks_admin_ajax');
 
-add_filter( 'script_loader_tag', 'wordpoints_script_templates_filter', 10, 2 );
+add_filter('script_loader_tag', 'wordpoints_script_templates_filter', 10, 2);
 
 // EOF

src/admin/includes/classes/screen.php

Spacing +5 added lines, -5 removed lines

@@ -62,8 +62,8 @@ 
 	public function hooks() {
 
 		/* Load the JavaScript needed for the settings screen. */
-		add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
-		add_action( "admin_footer-{$this->id}", array( $this, 'footer_scripts' ) );
+		add_action('admin_enqueue_scripts', array($this, 'enqueue_scripts'));
+		add_action("admin_footer-{$this->id}", array($this, 'footer_scripts'));
 	}
 
 	/**
@@ -98,12 +98,12 @@ 
 
 		<div class="wrap">
 
-			<h1><?php echo esc_html( $this->get_title() ); ?></h1>
+			<h1><?php echo esc_html($this->get_title()); ?></h1>
 
 			<?php settings_errors(); ?>
 
-			<?php if ( ! empty( $this->tabs ) ) : ?>
-				<?php wordpoints_admin_show_tabs( $this->tabs, false ); ?>
+			<?php if ( ! empty($this->tabs)) : ?>
+				<?php wordpoints_admin_show_tabs($this->tabs, false); ?>
 			<?php endif; ?>
 
 			<?php $this->display_content(); ?>

src/admin/includes/classes/screens.php

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function __construct() {
-		add_action( 'current_screen', array( $this, 'set_current_screen' ) );
+		add_action('current_screen', array($this, 'set_current_screen'));
 	}
 
 	/**
@@ -37,23 +37,23 @@ 
 	 *
 	 * @param WP_Screen $current_screen The WP_Screen object for the current screen.
 	 */
-	public function set_current_screen( $current_screen ) {
+	public function set_current_screen($current_screen) {
 
 		$screen_id = $current_screen->id;
 
-		if ( is_network_admin() ) {
-			$screen_id = substr( $screen_id, 0, -8 /* -network */ );
+		if (is_network_admin()) {
+			$screen_id = substr($screen_id, 0, -8 /* -network */);
 		}
 
-		$screen = $this->get( $screen_id );
+		$screen = $this->get($screen_id);
 
-		if ( ! ( $screen instanceof WordPoints_Admin_Screen ) ) {
+		if ( ! ($screen instanceof WordPoints_Admin_Screen)) {
 			return;
 		}
 
 		$this->current_screen = $screen;
 
-		add_action( "load-{$screen_id}", array( $this->current_screen, 'load' ) );
+		add_action("load-{$screen_id}", array($this->current_screen, 'load'));
 	}
 
 	/**

src/admin/includes/functions.php

Spacing +78 added lines, -78 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,15 +98,15 @@ 
 		'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' ),
+			'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'),
 		)
 	);
 
@@ -118,10 +118,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">
@@ -140,16 +140,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>
@@ -197,15 +197,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
 	);
 
@@ -215,25 +215,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>
@@ -247,7 +247,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>
@@ -265,8 +265,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
 	);
 
@@ -276,7 +276,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,32 +295,32 @@ 
 
 	$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}");
 		}
 	}
 
 	$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 ] = array(
+		if ($reactor instanceof WordPoints_Hook_Reactor) {
+			$reactor_data[$slug] = array(
 				'slug'         => $reactor->get_slug(),
 				'fields'       => $reactor->get_settings_fields(),
 				'arg_types'    => $reactor->get_arg_types(),
-				'target_label' => __( 'Award To', 'wordpoints' ),
+				'target_label' => __('Award To', 'wordpoints'),
 			);
 		}
 
-		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}");
 		}
 	}
 
@@ -332,28 +332,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();
 			}
 
 			/**
@@ -364,10 +364,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,
@@ -375,12 +375,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(),
@@ -388,7 +388,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -397,7 +397,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(
@@ -418,7 +418,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'
@@ -443,13 +443,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;
 	}
 
@@ -465,7 +465,7 @@ 
  */
 function wordpoints_hooks_admin_ajax() {
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+	if (defined('DOING_AJAX') && DOING_AJAX) {
 		new WordPoints_Admin_Ajax_Hooks;
 	}
 }

src/includes/classes/hook/firer.php

Spacing +10 added lines, -10 removed lines

@@ -17,14 +17,14 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function do_event( $event_slug, WordPoints_Hook_Event_Args $event_args ) {
+	public function do_event($event_slug, WordPoints_Hook_Event_Args $event_args) {
 
 		$hooks = wordpoints_hooks();
 
 		/** @var WordPoints_Hook_Reactor $reactor */
-		foreach ( $hooks->reactors->get_all() as $reactor ) {
+		foreach ($hooks->reactors->get_all() as $reactor) {
 
-			foreach ( $reactor->get_all_reactions_to_event( $event_slug ) as $reaction ) {
+			foreach ($reactor->get_all_reactions_to_event($event_slug) as $reaction) {
 
 				$validator = new WordPoints_Hook_Reaction_Validator(
 					$reaction
@@ -34,26 +34,26 @@ 
 
 				$validator->validate();
 
-				if ( $validator->had_errors() ) {
+				if ($validator->had_errors()) {
 					continue;
 				}
 
-				$event_args->set_validator( $validator );
+				$event_args->set_validator($validator);
 				$reaction = $validator;
 
 				/** @var WordPoints_Hook_Extension[] $extensions */
 				$extensions = $hooks->extensions->get_all();
 
-				foreach ( $extensions as $extension ) {
-					if ( ! $extension->should_hit( $reaction, $event_args ) ) {
+				foreach ($extensions as $extension) {
+					if ( ! $extension->should_hit($reaction, $event_args)) {
 						continue 2;
 					}
 				}
 
-				$reactor->hit( $event_args, $reaction );
+				$reactor->hit($event_args, $reaction);
 
-				foreach ( $extensions as $extension ) {
-					$extension->after_hit( $reaction, $event_args );
+				foreach ($extensions as $extension) {
+					$extension->after_hit($reaction, $event_args);
 				}
 			}
 		}

src/includes/classes/hook/extension/periods.php

Spacing +69 added lines, -69 removed lines

@@ -25,18 +25,18 @@ 
 	public function get_ui_script_data() {
 
 		$periods = array(
-			MINUTE_IN_SECONDS   => __( 'Minute', 'wordpoints' ),
-			HOUR_IN_SECONDS     => __( 'Hour',   'wordpoints' ),
-			DAY_IN_SECONDS      => __( 'Day',    'wordpoints' ),
-			WEEK_IN_SECONDS     => __( 'Week',   'wordpoints' ),
-			30 * DAY_IN_SECONDS => __( 'Month',  'wordpoints' ),
+			MINUTE_IN_SECONDS   => __('Minute', 'wordpoints'),
+			HOUR_IN_SECONDS     => __('Hour', 'wordpoints'),
+			DAY_IN_SECONDS      => __('Day', 'wordpoints'),
+			WEEK_IN_SECONDS     => __('Week', 'wordpoints'),
+			30 * DAY_IN_SECONDS => __('Month', 'wordpoints'),
 		);
 
 		return array(
 			'periods' => $periods,
 			'l10n' => array(
 				// TODO this should be supplied per-reactor
-				'label' => __( 'Award each user no more than once per:', 'wordpoints' ),
+				'label' => __('Award each user no more than once per:', 'wordpoints'),
 			),
 		);
 	}
@@ -50,25 +50,25 @@ 
 	 *
 	 * @return array The validated periods.
 	 */
-	protected function validate_periods( $periods ) {
+	protected function validate_periods($periods) {
 
-		if ( ! is_array( $periods ) ) {
+		if ( ! is_array($periods)) {
 
 			$this->validator->add_error(
-				__( 'Periods do not match expected format.', 'wordpoints' )
+				__('Periods do not match expected format.', 'wordpoints')
 			);
 
 			return array();
 		}
 
-		foreach ( $periods as $index => $period ) {
+		foreach ($periods as $index => $period) {
 
-			$this->validator->push_field( $index );
+			$this->validator->push_field($index);
 
-			$period = $this->validate_period( $period );
+			$period = $this->validate_period($period);
 
-			if ( $period ) {
-				$periods[ $index ] = $period;
+			if ($period) {
+				$periods[$index] = $period;
 			}
 
 			$this->validator->pop_field();
@@ -86,30 +86,30 @@ 
 	 *
 	 * @return array|false The validated period, or false if invalid.
 	 */
-	protected function validate_period( $period ) {
+	protected function validate_period($period) {
 
-		if ( ! is_array( $period ) ) {
+		if ( ! is_array($period)) {
 			$this->validator->add_error(
-				__( 'Period does not match expected format.', 'wordpoints' )
+				__('Period does not match expected format.', 'wordpoints')
 			);
 
 			return false;
 		}
 
-		if ( isset( $period['args'] ) ) {
-			$this->validate_period_args( $period['args'] );
+		if (isset($period['args'])) {
+			$this->validate_period_args($period['args']);
 		}
 
-		if ( ! isset( $period['length'] ) ) {
+		if ( ! isset($period['length'])) {
 
 			$this->validator->add_error(
-				__( 'Period length setting is missing.', 'wordpoints' )
+				__('Period length setting is missing.', 'wordpoints')
 			);
 
-		} elseif ( false === wordpoints_posint( $period['length'] ) ) {
+		} elseif (false === wordpoints_posint($period['length'])) {
 
 			$this->validator->add_error(
-				__( 'Period length must be a positive integer.', 'wordpoints' )
+				__('Period length must be a positive integer.', 'wordpoints')
 				, 'length'
 			);
 
@@ -126,34 +126,34 @@ 
 	 *
 	 * @param mixed $args The args the period is related to.
 	 */
-	protected function validate_period_args( $args ) {
+	protected function validate_period_args($args) {
 
-		if ( ! is_array( $args ) ) {
+		if ( ! is_array($args)) {
 
 			$this->validator->add_error(
-				__( 'Period does not match expected format.', 'wordpoints' )
+				__('Period does not match expected format.', 'wordpoints')
 				, 'args'
 			);
 
 			return;
 		}
 
-		$this->validator->push_field( 'args' );
+		$this->validator->push_field('args');
 
-		foreach ( $args as $index => $hierarchy ) {
+		foreach ($args as $index => $hierarchy) {
 
-			$this->validator->push_field( $index );
+			$this->validator->push_field($index);
 
-			if ( ! is_array( $hierarchy ) ) {
+			if ( ! is_array($hierarchy)) {
 
 				$this->validator->add_error(
-					__( 'Period does not match expected format.', 'wordpoints' )
+					__('Period does not match expected format.', 'wordpoints')
 				);
 
-			} elseif ( ! $this->event_args->get_from_hierarchy( $hierarchy ) ) {
+			} elseif ( ! $this->event_args->get_from_hierarchy($hierarchy)) {
 
 				$this->validator->add_error(
-					__( 'Invalid period.', 'wordpoints' ) // TODO better error message
+					__('Invalid period.', 'wordpoints') // TODO better error message
 				);
 			}
 
@@ -171,16 +171,16 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		$periods = $reaction->get_meta( 'periods' );
+		$periods = $reaction->get_meta('periods');
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return true;
 		}
 
 		$this->event_args = $event_args;
 
-		foreach ( $periods as $period ) {
-			if ( ! $this->has_period_ended( $period, $reaction ) ) {
+		foreach ($periods as $period) {
+			if ( ! $this->has_period_ended($period, $reaction)) {
 				return false;
 			}
 		}
@@ -204,23 +204,23 @@ 
 	) {
 
 		$period = $this->get_period_by_reaction(
-			$this->get_period_signature( $settings, $reaction )
+			$this->get_period_signature($settings, $reaction)
 			, $reaction
 		);
 
 		// If the period isn't found, we know that we can still fire.
-		if ( ! $period ) {
+		if ( ! $period) {
 			return true;
 		}
 
-		$now = current_time( 'timestamp' );
+		$now = current_time('timestamp');
 
-		if ( ! empty( $settings['relative'] ) ) {
-			return ( $period->hit_time < $now - $settings['length'] );
+		if ( ! empty($settings['relative'])) {
+			return ($period->hit_time < $now - $settings['length']);
 		} else {
 			return (
-				(int) ( $period->hit_time / $settings['length'] )
-				< (int) ( $now / $settings['length'] )
+				(int) ($period->hit_time / $settings['length'])
+				< (int) ($now / $settings['length'])
 			);
 		}
 	}
@@ -234,21 +234,21 @@ 
 	 *
 	 * @return array The arg values.
 	 */
-	protected function get_arg_values( array $period_args ) {
+	protected function get_arg_values(array $period_args) {
 
 		$values = array();
 
-		foreach ( $period_args as $arg_hierarchy ) {
+		foreach ($period_args as $arg_hierarchy) {
 
 			$arg = $this->event_args->get_from_hierarchy(
 				$arg_hierarchy
 			);
 
-			if ( ! $arg instanceof WordPoints_EntityishI ) {
+			if ( ! $arg instanceof WordPoints_EntityishI) {
 				continue;
 			}
 
-			$values[ implode( '.', $arg_hierarchy ) ] = $arg->get_the_value();
+			$values[implode('.', $arg_hierarchy)] = $arg->get_the_value();
 		}
 
 		return $values;
@@ -263,11 +263,11 @@ 
 	 *
 	 * @return object|false The period data, or false if not found.
 	 */
-	protected function get_period( $period_id ) {
+	protected function get_period($period_id) {
 
-		$period = wp_cache_get( $period_id, 'wordpoints_hook_period' );
+		$period = wp_cache_get($period_id, 'wordpoints_hook_period');
 
-		if ( ! $period ) {
+		if ( ! $period) {
 
 			global $wpdb;
 
@@ -282,7 +282,7 @@ 
 				)
 			);
 
-			if ( ! $period ) {
+			if ( ! $period) {
 				return false;
 			}
 
@@ -292,7 +292,7 @@ 
 				, 'wordpoints_hook_period_ids'
 			);
 
-			wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+			wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 		}
 
 		return $period;
@@ -319,11 +319,11 @@ 
 		$cache_key = "{$reaction_id}-{$signature}";
 
 		// Before we run the query, we try to lookup the ID in the cache.
-		$period_id = wp_cache_get( $cache_key, 'wordpoints_hook_period_ids' );
+		$period_id = wp_cache_get($cache_key, 'wordpoints_hook_period_ids');
 
 		// If we found it, we can retrieve the period by ID instead.
-		if ( $period_id ) {
-			return $this->get_period( $period_id );
+		if ($period_id) {
+			return $this->get_period($period_id);
 		}
 
 		global $wpdb;
@@ -342,12 +342,12 @@ 
 			)
 		);
 
-		if ( ! $period ) {
+		if ( ! $period) {
 			return false;
 		}
 
-		wp_cache_set( $cache_key, $period->id, 'wordpoints_hook_period_ids' );
-		wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+		wp_cache_set($cache_key, $period->id, 'wordpoints_hook_period_ids');
+		wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 
 		return $period;
 	}
@@ -360,18 +360,18 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		$periods = $reaction->get_meta( 'periods' );
+		$periods = $reaction->get_meta('periods');
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return;
 		}
 
 		$this->event_args = $event_args;
 
-		foreach ( $periods as $settings ) {
+		foreach ($periods as $settings) {
 
 			$this->add_period(
-				$this->get_period_signature( $settings, $reaction )
+				$this->get_period_signature($settings, $reaction)
 				, $reaction
 			);
 		}
@@ -396,14 +396,14 @@ 
 		WordPoints_Hook_Reaction_Validator $reaction
 	) {
 
-		if ( isset( $settings['args'] ) ) {
+		if (isset($settings['args'])) {
 			$period_args = $settings['args'];
 		} else {
-			$period_args = array( $reaction->get_meta( 'target' ) );
+			$period_args = array($reaction->get_meta('target'));
 		}
 
 		return wordpoints_hash(
-			wp_json_encode( $this->get_arg_values( $period_args ) )
+			wp_json_encode($this->get_arg_values($period_args))
 		);
 	}
 
@@ -431,12 +431,12 @@ 
 			, array(
 				'reaction_id' => $reaction_id,
 				'signature'   => $signature,
-				'hit_time'    => current_time( 'timestamp' ),
+				'hit_time'    => current_time('timestamp'),
 			)
-			, array( '%d', '%s', '%d' )
+			, array('%d', '%s', '%d')
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}
 

src/includes/classes/hook/reaction/validator.php

Spacing +26 added lines, -26 removed lines

@@ -104,13 +104,13 @@ 
 	 * @param bool                            $fail_fast Whether to fail as soon as
 	 *                                                   the first error is found.
 	 */
-	public function __construct( $settings, WordPoints_Hook_Reactor $reactor, $fail_fast = false ) {
+	public function __construct($settings, WordPoints_Hook_Reactor $reactor, $fail_fast = false) {
 
 		$this->reactor = $reactor;
 		$this->fail_fast = $fail_fast;
 		$this->hooks = wordpoints_hooks();
 
-		if ( $settings instanceof WordPoints_Hook_ReactionI ) {
+		if ($settings instanceof WordPoints_Hook_ReactionI) {
 
 			$this->reaction   = $settings;
 			$this->settings   = $this->reaction->get_all_meta();
@@ -120,7 +120,7 @@ 
 
 			$this->settings = $settings;
 
-			if ( isset( $this->settings['event'] ) ) {
+			if (isset($this->settings['event'])) {
 				$this->event_slug = $this->settings['event'];
 			}
 		}
@@ -144,10 +144,10 @@ 
 			$fail_fast = $this->fail_fast;
 			$this->fail_fast = true;
 
-			if ( ! isset( $this->event_slug ) ) {
-				$this->add_error( __( 'Event is missing.', 'wordpoints' ), 'event' );
-			} elseif ( ! $this->hooks->events->is_registered( $this->event_slug ) ) {
-				$this->add_error( __( 'Event is invalid.', 'wordpoints' ), 'event' );
+			if ( ! isset($this->event_slug)) {
+				$this->add_error(__('Event is missing.', 'wordpoints'), 'event');
+			} elseif ( ! $this->hooks->events->is_registered($this->event_slug)) {
+				$this->add_error(__('Event is invalid.', 'wordpoints'), 'event');
 			}
 
 			// From here on out we can collect errors as they come (unless we are
@@ -158,14 +158,14 @@ 
 				$this->event_slug
 			);
 
-			$this->event_args = new WordPoints_Hook_Event_Args( $event_args );
-			$this->event_args->set_validator( $this );
+			$this->event_args = new WordPoints_Hook_Event_Args($event_args);
+			$this->event_args->set_validator($this);
 
-			$this->settings = $this->reactor->validate_settings( $this->settings, $this, $this->event_args );
+			$this->settings = $this->reactor->validate_settings($this->settings, $this, $this->event_args);
 
 			/** @var WordPoints_Hook_Extension $extension */
-			foreach ( $this->hooks->extensions->get_all() as $extension ) {
-				$this->settings = $extension->validate_settings( $this->settings, $this, $this->event_args );
+			foreach ($this->hooks->extensions->get_all() as $extension) {
+				$this->settings = $extension->validate_settings($this->settings, $this, $this->event_args);
 			}
 
 			/**
@@ -175,12 +175,12 @@ 
 			 * @param WordPoints_Hook_Reaction_Validator $validator The validator object.
 			 * @param WordPoints_Hook_Event_Args         $args      The event args object.
 			 */
-			$this->settings = apply_filters( 'wordpoints_hook_reaction_validate', $this->settings, $this, $this->event_args );
+			$this->settings = apply_filters('wordpoints_hook_reaction_validate', $this->settings, $this, $this->event_args);
 
-		} catch ( WordPoints_Hook_Validator_Exception $e ) {
+		} catch (WordPoints_Hook_Validator_Exception $e) {
 
 			// Do nothing.
-			unset( $e );
+			unset($e);
 		}
 
 		return $this->settings;
@@ -208,17 +208,17 @@ 
 	 * @throws WordPoints_Hook_Validator_Exception If the validator is configured to
 	 *                                             fail as soon as an error is found.
 	 */
-	public function add_error( $message, $field = null ) {
+	public function add_error($message, $field = null) {
 
 		$field_stack = $this->field_stack;
 
-		if ( null !== $field ) {
+		if (null !== $field) {
 			$field_stack[] = $field;
 		}
 
-		$this->errors[] = array( 'message' => $message, 'field' => $field_stack );
+		$this->errors[] = array('message' => $message, 'field' => $field_stack);
 
-		if ( $this->fail_fast ) {
+		if ($this->fail_fast) {
 			throw new WordPoints_Hook_Validator_Exception;
 		}
 	}
@@ -231,7 +231,7 @@ 
 	 * @return bool Whether the validator found any errors.
 	 */
 	public function had_errors() {
-		return ! empty( $this->errors );
+		return ! empty($this->errors);
 	}
 
 	/**
@@ -252,7 +252,7 @@ 
 	 *
 	 * @param string $field The field.
 	 */
-	public function push_field( $field ) {
+	public function push_field($field) {
 		$this->field_stack[] = $field;
 	}
 
@@ -262,7 +262,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function pop_field() {
-		array_pop( $this->field_stack );
+		array_pop($this->field_stack);
 	}
 
 	/**
@@ -307,7 +307,7 @@ 
 	 */
 	public function get_id() {
 
-		if ( ! $this->reaction ) {
+		if ( ! $this->reaction) {
 			return false;
 		}
 
@@ -345,13 +345,13 @@ 
 	 *
 	 * @return mixed The meta value.
 	 */
-	public function get_meta( $key ) {
+	public function get_meta($key) {
 
-		if ( ! isset( $this->settings[ $key ] ) ) {
+		if ( ! isset($this->settings[$key])) {
 			return null;
 		}
 
-		return $this->settings[ $key ];
+		return $this->settings[$key];
 	}
 
 	/**

src/includes/classes/hooks.php

Spacing +13 added lines, -13 removed lines

@@ -46,13 +46,13 @@ 
 	protected function init() {
 
 		$sub_apps = $this->sub_apps;
-		$sub_apps->register( 'router', 'WordPoints_Hook_Router' );
-		$sub_apps->register( 'actions', 'WordPoints_Hook_Actions' );
-		$sub_apps->register( 'events', 'WordPoints_Hook_Events' );
-		$sub_apps->register( 'firers', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'reactors', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'extensions', 'WordPoints_Class_Registry_Persistent' );
-		$sub_apps->register( 'conditions', 'WordPoints_Class_Registry_Children' );
+		$sub_apps->register('router', 'WordPoints_Hook_Router');
+		$sub_apps->register('actions', 'WordPoints_Hook_Actions');
+		$sub_apps->register('events', 'WordPoints_Hook_Events');
+		$sub_apps->register('firers', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('reactors', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('extensions', 'WordPoints_Class_Registry_Persistent');
+		$sub_apps->register('conditions', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -66,18 +66,18 @@ 
 	 */
 	public function get_network_mode() {
 
-		if ( ! isset( $this->network_mode ) ) {
+		if ( ! isset($this->network_mode)) {
 			$this->network_mode = is_network_admin();
 
 			// See https://core.trac.wordpress.org/ticket/22589
 			if (
-				defined( 'DOING_AJAX' )
+				defined('DOING_AJAX')
 				&& DOING_AJAX
 				&& is_multisite()
-				&& 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']))
 				)
 			) {
 				$this->network_mode = true;
@@ -121,7 +121,7 @@ 
 	 *
 	 * @param bool $on Whether network-wide mode should be on (or off).
 	 */
-	public function _set_network_mode( $on = true ) {
+	public function _set_network_mode($on = true) {
 		$this->network_mode = (bool) $on;
 	}
 }