WordPoints / hooks-api

src/includes/classes/hook/extension/reversals/legacy/points.php

Spacing +20 added lines, -20 removed lines

@@ -23,13 +23,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		if ( ! parent::should_hit( $fire ) ) {
+		if ( ! parent::should_hit($fire)) {
 
-			$logs = $this->get_points_logs_to_be_reversed( $fire );
+			$logs = $this->get_points_logs_to_be_reversed($fire);
 
-			return count( $logs ) > 0;
+			return count($logs) > 0;
 		}
 
 		return true;
@@ -38,16 +38,16 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function after_miss( WordPoints_Hook_Fire $fire ) {
+	public function after_miss(WordPoints_Hook_Fire $fire) {
 
-		parent::after_miss( $fire );
+		parent::after_miss($fire);
 
-		if ( ! $this->get_settings_from_fire( $fire ) ) {
+		if ( ! $this->get_settings_from_fire($fire)) {
 			return;
 		}
 
-		foreach ( $this->get_points_logs_to_be_reversed( $fire ) as $log ) {
-			wordpoints_add_points_log_meta( $log->id, 'auto_reversed', 0 );
+		foreach ($this->get_points_logs_to_be_reversed($fire) as $log) {
+			wordpoints_add_points_log_meta($log->id, 'auto_reversed', 0);
 		}
 	}
 
@@ -60,23 +60,23 @@ 
 	 *
 	 * @return array The points logs to be reversed.
 	 */
-	protected function get_points_logs_to_be_reversed( WordPoints_Hook_Fire $fire ) {
+	protected function get_points_logs_to_be_reversed(WordPoints_Hook_Fire $fire) {
 
-		if ( isset( $fire->data[ $this->slug ]['points_logs'] ) ) {
-			return $fire->data[ $this->slug ]['points_logs'];
+		if (isset($fire->data[$this->slug]['points_logs'])) {
+			return $fire->data[$this->slug]['points_logs'];
 		}
 
 		$entity = $fire->event_args->get_primary_arg();
 
-		if ( ! $entity ) {
-			$fire->data[ $this->slug ]['points_logs'] = array();
+		if ( ! $entity) {
+			$fire->data[$this->slug]['points_logs'] = array();
 			return array();
 		}
 
 		$slug = $entity->get_slug();
 
-		if ( ( $pos = strpos( $slug, '\\' ) ) ) {
-			$slug = substr( $slug, 0, $pos );
+		if (($pos = strpos($slug, '\\'))) {
+			$slug = substr($slug, 0, $pos);
 		}
 
 		$meta_queries = array(
@@ -92,9 +92,9 @@ 
 			)
 		);
 
-		$log_type = $fire->reaction->get_meta( 'legacy_log_type' );
+		$log_type = $fire->reaction->get_meta('legacy_log_type');
 
-		if ( ! $log_type ) {
+		if ( ! $log_type) {
 			$log_type = $fire->reaction->get_event_slug();
 		}
 
@@ -107,11 +107,11 @@ 
 
 		$logs = $query->get();
 
-		if ( ! $logs ) {
+		if ( ! $logs) {
 			$logs = array();
 		}
 
-		$fire->data[ $this->slug ]['points_logs'] = $logs;
+		$fire->data[$this->slug]['points_logs'] = $logs;
 
 		return $logs;
 	}

src/admin/includes/functions.php

Indentation +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
 		, 'WordPointsPointsTypesL10n'
 		, array(
 			'confirmDelete' => esc_html__( 'Are you sure that you want to delete this points type? This will delete all related logs and event hooks.', 'wordpoints' )
-			                   . ' ' . esc_html__( 'Once a points type has been deleted, you cannot bring it back.', 'wordpoints' ),
+							   . ' ' . esc_html__( 'Once a points type has been deleted, you cannot bring it back.', 'wordpoints' ),
 			'confirmTitle'  => esc_html__( 'Are you sure?', 'wordpoints' ),
 			'deleteText'    => esc_html__( 'Delete', 'wordpoints' ),
 			'cancelText'    => esc_html__( 'Cancel', 'wordpoints' ),

Spacing +94 added lines, -94 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');
 }
 
 /**
@@ -41,22 +41,22 @@ 
 	$wordpoints_menu = wordpoints_get_main_admin_menu();
 
 	/** @var WordPoints_Admin_Screens $admin_screens */
-	$admin_screens = wordpoints_apps()->get_sub_app( 'admin' )->get_sub_app(
+	$admin_screens = wordpoints_apps()->get_sub_app('admin')->get_sub_app(
 		'screen'
 	);
 
 	// Hooks page.
 	$id = add_submenu_page(
 		$wordpoints_menu
-		, __( 'WordPoints — Points Types', 'wordpoints' )
-		, __( 'Points Types', 'wordpoints' )
+		, __('WordPoints — Points Types', 'wordpoints')
+		, __('Points Types', 'wordpoints')
 		, 'manage_options'
 		, 'wordpoints_points_types'
-		, array( $admin_screens, 'display' )
+		, array($admin_screens, 'display')
 	);
 
-	if ( $id ) {
-		$admin_screens->register( $id, 'WordPoints_Admin_Screen_Points_Types' );
+	if ($id) {
+		$admin_screens->register($id, 'WordPoints_Admin_Screen_Points_Types');
 	}
 }
 
@@ -69,14 +69,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
 	);
 
@@ -84,22 +84,22 @@ 
 
 	wp_register_script(
 		'wordpoints-admin-points-types'
-		, $assets_url . '/js/points-types.js'
-		, array( 'backbone', 'jquery-ui-dialog', 'wp-util' )
+		, $assets_url.'/js/points-types.js'
+		, array('backbone', 'jquery-ui-dialog', 'wp-util')
 		, WORDPOINTS_VERSION
 	);
 
 	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
 	);
 
@@ -107,11 +107,11 @@ 
 		'wordpoints-admin-points-types'
 		, 'WordPointsPointsTypesL10n'
 		, array(
-			'confirmDelete' => esc_html__( 'Are you sure that you want to delete this points type? This will delete all related logs and event hooks.', 'wordpoints' )
-			                   . ' ' . esc_html__( 'Once a points type has been deleted, you cannot bring it back.', 'wordpoints' ),
-			'confirmTitle'  => esc_html__( 'Are you sure?', 'wordpoints' ),
-			'deleteText'    => esc_html__( 'Delete', 'wordpoints' ),
-			'cancelText'    => esc_html__( 'Cancel', 'wordpoints' ),
+			'confirmDelete' => esc_html__('Are you sure that you want to delete this points type? This will delete all related logs and event hooks.', 'wordpoints')
+			                   . ' '.esc_html__('Once a points type has been deleted, you cannot bring it back.', 'wordpoints'),
+			'confirmTitle'  => esc_html__('Are you sure?', 'wordpoints'),
+			'deleteText'    => esc_html__('Delete', 'wordpoints'),
+			'cancelText'    => esc_html__('Cancel', 'wordpoints'),
 		)
 	);
 
@@ -119,19 +119,19 @@ 
 		'wordpoints-hooks-views'
 		, 'WordPointsHooksAdminL10n'
 		, array(
-			'unexpectedError' => __( 'There was an unexpected error. Try reloading the page.', 'wordpoints' ),
-			'changesSaved'    => __( 'Your changes have been saved.', 'wordpoints' ),
+			'unexpectedError' => __('There was an unexpected error. Try reloading the page.', 'wordpoints'),
+			'changesSaved'    => __('Your changes have been saved.', 'wordpoints'),
 			/* translators: 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'),
 			// translators: form field
-			'cannotBeChanged' => __( '(cannot be changed)', 'wordpoints' ),
-			'fieldsInvalid'   => __( 'Error: the values of some fields are invalid. Please correct these and then try again.', 'wordpoints' ),
+			'cannotBeChanged' => __('(cannot be changed)', 'wordpoints'),
+			'fieldsInvalid'   => __('Error: the values of some fields are invalid. Please correct these and then try again.', 'wordpoints'),
 		)
 	);
 
@@ -143,10 +143,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">
@@ -165,16 +165,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>
@@ -222,15 +222,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
 	);
 
@@ -240,25 +240,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>
@@ -272,7 +272,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>
@@ -290,8 +290,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
 	);
 
@@ -301,7 +301,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>
@@ -319,12 +319,12 @@ 
 	$hooks = wordpoints_hooks();
 
 	$extensions_data = wordpoints_hooks_ui_get_script_data_from_objects(
-		$hooks->get_sub_app( 'extensions' )->get_all()
+		$hooks->get_sub_app('extensions')->get_all()
 		, 'extension'
 	);
 
 	$reactor_data = wordpoints_hooks_ui_get_script_data_from_objects(
-		$hooks->get_sub_app( 'reactors' )->get_all()
+		$hooks->get_sub_app('reactors')->get_all()
 		, 'reactor'
 	);
 
@@ -350,7 +350,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'
@@ -370,18 +370,18 @@ 
  *
  * @return array The data extracted from the objects.
  */
-function wordpoints_hooks_ui_get_script_data_from_objects( $objects, $type ) {
+function wordpoints_hooks_ui_get_script_data_from_objects($objects, $type) {
 
 	$data = array();
 
-	foreach ( $objects as $slug => $object ) {
+	foreach ($objects as $slug => $object) {
 
-		if ( $object instanceof WordPoints_Hook_UI_Script_Data_ProviderI ) {
-			$data[ $slug ] = $object->get_ui_script_data();
+		if ($object instanceof WordPoints_Hook_UI_Script_Data_ProviderI) {
+			$data[$slug] = $object->get_ui_script_data();
 		}
 
-		if ( wp_script_is( "wordpoints-hooks-{$type}-{$slug}", 'registered' ) ) {
-			wp_enqueue_script( "wordpoints-hooks-{$type}-{$slug}" );
+		if (wp_script_is("wordpoints-hooks-{$type}-{$slug}", 'registered')) {
+			wp_enqueue_script("wordpoints-hooks-{$type}-{$slug}");
 		}
 	}
 
@@ -402,31 +402,31 @@ 
 	$entities_data = array();
 
 	/** @var WordPoints_Class_Registry_Children $entity_children */
-	$entity_children = $entities->get_sub_app( 'children' );
+	$entity_children = $entities->get_sub_app('children');
 
 	/** @var WordPoints_Entity $entity */
-	foreach ( $entities->get_all() as $slug => $entity ) {
+	foreach ($entities->get_all() as $slug => $entity) {
 
 		$child_data = array();
 
 		/** @var WordPoints_EntityishI $child */
-		foreach ( $entity_children->get_children( $slug ) as $child_slug => $child ) {
+		foreach ($entity_children->get_children($slug) as $child_slug => $child) {
 
-			$child_data[ $child_slug ] = array(
+			$child_data[$child_slug] = array(
 				'slug'  => $child_slug,
 				'title' => $child->get_title(),
 			);
 
-			if ( $child instanceof WordPoints_Entity_Attr ) {
+			if ($child instanceof WordPoints_Entity_Attr) {
 
-				$child_data[ $child_slug ]['_type']     = 'attr';
-				$child_data[ $child_slug ]['data_type'] = $child->get_data_type();
+				$child_data[$child_slug]['_type']     = 'attr';
+				$child_data[$child_slug]['data_type'] = $child->get_data_type();
 
-			} elseif ( $child instanceof WordPoints_Entity_Relationship ) {
+			} elseif ($child instanceof WordPoints_Entity_Relationship) {
 
-				$child_data[ $child_slug ]['_type']     = 'relationship';
-				$child_data[ $child_slug ]['primary']   = $child->get_primary_entity_slug();
-				$child_data[ $child_slug ]['secondary'] = $child->get_related_entity_slug();
+				$child_data[$child_slug]['_type']     = 'relationship';
+				$child_data[$child_slug]['primary']   = $child->get_primary_entity_slug();
+				$child_data[$child_slug]['secondary'] = $child->get_related_entity_slug();
 			}
 
 			/**
@@ -437,14 +437,14 @@ 
 			 * @param array                $data  The data for the entity child.
 			 * @param WordPoints_Entityish $child The child's object.
 			 */
-			$child_data[ $child_slug ] = apply_filters(
+			$child_data[$child_slug] = apply_filters(
 				'wordpoints_hooks_ui_data_entity_child'
-				, $child_data[ $child_slug ]
+				, $child_data[$child_slug]
 				, $child
 			);
 		}
 
-		$entities_data[ $slug ] = array(
+		$entities_data[$slug] = array(
 			'slug'     => $slug,
 			'title'    => $entity->get_title(),
 			'children' => $child_data,
@@ -452,12 +452,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(),
@@ -465,7 +465,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -474,9 +474,9 @@ 
 		 * @param array             $data   The data for the entity.
 		 * @param WordPoints_Entity $entity The entity object.
 		 */
-		$entities_data[ $slug ] = apply_filters(
+		$entities_data[$slug] = apply_filters(
 			'wordpoints_hooks_ui_data_entity'
-			, $entities_data[ $slug ]
+			, $entities_data[$slug]
 			, $entity
 		);
 	}
@@ -496,19 +496,19 @@ 
 	// We want a list of the action types for each event. We can start with this list
 	// but it is indexed by action slug and then action type and then event slug, so
 	// we ned to do some processing.
-	$event_index = wordpoints_hooks()->get_sub_app( 'router' )->get_event_index();
+	$event_index = wordpoints_hooks()->get_sub_app('router')->get_event_index();
 
 	// We don't care about the action slugs, so first we get rid of that bottom level
 	// of the array.
-	$event_index = call_user_func_array( 'array_merge_recursive', $event_index );
+	$event_index = call_user_func_array('array_merge_recursive', $event_index);
 
 	$event_action_types = array();
 
 	// This leaves us the event indexed by action type. But we actually need to flip
 	// this, so that we have the action types indexed by event slug.
-	foreach ( $event_index as $action_type => $events ) {
-		foreach ( $events as $event => $unused ) {
-			$event_action_types[ $event ][ $action_type ] = true;
+	foreach ($event_index as $action_type => $events) {
+		foreach ($events as $event => $unused) {
+			$event_action_types[$event][$action_type] = true;
 		}
 	}
 
@@ -531,13 +531,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;
 	}
 
@@ -553,7 +553,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/action/post/depublish/delete.php

Spacing +3 added lines, -3 removed lines

@@ -19,13 +19,13 @@
 	 */
 	public function should_fire() {
 
-		$post = $this->get_arg_value( 'post' );
+		$post = $this->get_arg_value('post');
 
-		if ( ! $post ) {
+		if ( ! $post) {
 			return false;
 		}
 
-		if ( 'publish' !== get_post_status( $post ) ) {
+		if ('publish' !== get_post_status($post)) {
 			return false;
 		}
 

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

Spacing +77 added lines, -77 removed lines

@@ -38,11 +38,11 @@ 
 	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'),
 		);
 
 		/**
@@ -52,12 +52,12 @@ 
 		 *
 		 * @param string[] $periods The period titles, indexed by length in seconds.
 		 */
-		$periods = apply_filters( 'wordpoints_hooks_ui_data_periods', $periods );
+		$periods = apply_filters('wordpoints_hooks_ui_data_periods', $periods);
 
 		return array(
 			'periods' => $periods,
 			'l10n' => array(
-				'label' => __( 'Trigger reaction no more than once per:', 'wordpoints' ),
+				'label' => __('Trigger reaction no more than once per:', 'wordpoints'),
 			),
 		);
 	}
@@ -71,25 +71,25 @@ 
 	 *
 	 * @return array The validated periods.
 	 */
-	protected function validate_action_type_settings( $periods ) {
+	protected function validate_action_type_settings($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();
@@ -107,30 +107,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'
 			);
 
@@ -147,34 +147,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 arg hierarchy for period.', 'wordpoints' )
+					__('Invalid arg hierarchy for period.', 'wordpoints')
 				);
 			}
 
@@ -187,19 +187,19 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function should_hit( WordPoints_Hook_Fire $fire ) {
+	public function should_hit(WordPoints_Hook_Fire $fire) {
 
-		$periods = $this->get_settings_from_fire( $fire );
+		$periods = $this->get_settings_from_fire($fire);
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return true;
 		}
 
 		$this->event_args = $fire->event_args;
 		$this->action_type = $fire->action_type;
 
-		foreach ( $periods as $period ) {
-			if ( ! $this->has_period_ended( $period, $fire->reaction ) ) {
+		foreach ($periods as $period) {
+			if ( ! $this->has_period_ended($period, $fire->reaction)) {
 				return false;
 			}
 		}
@@ -223,24 +223,24 @@ 
 	) {
 
 		$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' );
-		$hit_time = strtotime( $period->date, $now );
+		$now = current_time('timestamp');
+		$hit_time = strtotime($period->date, $now);
 
-		if ( ! empty( $settings['relative'] ) ) {
-			return ( $hit_time < $now - $settings['length'] );
+		if ( ! empty($settings['relative'])) {
+			return ($hit_time < $now - $settings['length']);
 		} else {
 			return (
-				(int) ( $hit_time / $settings['length'] )
-				< (int) ( $now / $settings['length'] )
+				(int) ($hit_time / $settings['length'])
+				< (int) ($now / $settings['length'])
 			);
 		}
 	}
@@ -254,24 +254,24 @@ 
 	 *
 	 * @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();
 		}
 
-		ksort( $values );
+		ksort($values);
 
 		return $values;
 	}
@@ -285,11 +285,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_periods' );
+		$period = wp_cache_get($period_id, 'wordpoints_hook_periods');
 
-		if ( ! $period ) {
+		if ( ! $period) {
 
 			global $wpdb;
 
@@ -306,11 +306,11 @@ 
 				)
 			);
 
-			if ( ! $period ) {
+			if ( ! $period) {
 				return false;
 			}
 
-			wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+			wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 		}
 
 		return $period;
@@ -334,14 +334,14 @@ 
 
 		$reaction_guid = $reaction->get_guid();
 
-		$cache_key = wp_json_encode( $reaction_guid ) . "-{$signature}-{$this->action_type}";
+		$cache_key = wp_json_encode($reaction_guid)."-{$signature}-{$this->action_type}";
 
 		// 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_by_reaction' );
+		$period_id = wp_cache_get($cache_key, 'wordpoints_hook_period_ids_by_reaction');
 
 		// 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;
@@ -364,18 +364,18 @@ 
 				"
 				, $signature
 				, $reaction_guid['store']
-				, wp_json_encode( $reaction_guid['context_id'] )
+				, wp_json_encode($reaction_guid['context_id'])
 				, $reaction_guid['id']
 				, $this->action_type
 			)
 		);
 
-		if ( ! $period ) {
+		if ( ! $period) {
 			return false;
 		}
 
-		wp_cache_set( $cache_key, $period->id, 'wordpoints_hook_period_ids_by_reaction' );
-		wp_cache_set( $period->id, $period, 'wordpoints_hook_periods' );
+		wp_cache_set($cache_key, $period->id, 'wordpoints_hook_period_ids_by_reaction');
+		wp_cache_set($period->id, $period, 'wordpoints_hook_periods');
 
 		return $period;
 	}
@@ -383,21 +383,21 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function after_hit( WordPoints_Hook_Fire $fire ) {
+	public function after_hit(WordPoints_Hook_Fire $fire) {
 
-		$periods = $this->get_settings_from_fire( $fire );
+		$periods = $this->get_settings_from_fire($fire);
 
-		if ( empty( $periods ) ) {
+		if (empty($periods)) {
 			return;
 		}
 
 		$this->event_args = $fire->event_args;
 		$this->action_type = $fire->action_type;
 
-		foreach ( $periods as $settings ) {
+		foreach ($periods as $settings) {
 
 			$this->add_period(
-				$this->get_period_signature( $settings, $fire->reaction )
+				$this->get_period_signature($settings, $fire->reaction)
 				, $fire
 			);
 		}
@@ -422,14 +422,14 @@ 
 		WordPoints_Hook_ReactionI $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))
 		);
 	}
 
@@ -443,7 +443,7 @@ 
 	 *
 	 * @return false|object The period data, or false if not found.
 	 */
-	protected function add_period( $signature, WordPoints_Hook_Fire $fire ) {
+	protected function add_period($signature, WordPoints_Hook_Fire $fire) {
 
 		global $wpdb;
 
@@ -453,17 +453,17 @@ 
 				'hit_id' => $fire->hit_id,
 				'signature' => $signature,
 			)
-			, array( '%d', '%s' )
+			, array('%d', '%s')
 		);
 
-		if ( ! $inserted ) {
+		if ( ! $inserted) {
 			return false;
 		}
 
 		$period_id = $wpdb->insert_id;
 
 		wp_cache_set(
-			wp_json_encode( $fire->reaction->get_guid() ) . "-{$signature}-{$this->action_type}"
+			wp_json_encode($fire->reaction->get_guid())."-{$signature}-{$this->action_type}"
 			, $period_id
 			, 'wordpoints_hook_period_ids_by_reaction'
 		);

src/includes/classes/hook/reaction/store/options.php

Spacing +49 added lines, -49 removed lines

@@ -22,25 +22,25 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function reaction_exists( $id ) {
-		return (bool) $this->get_option( $this->get_settings_option_name( $id ) );
+	public function reaction_exists($id) {
+		return (bool) $this->get_option($this->get_settings_option_name($id));
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_reactions() {
-		return $this->create_reaction_objects( $this->get_reaction_index() );
+		return $this->create_reaction_objects($this->get_reaction_index());
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_reactions_to_event( $event_slug ) {
+	public function get_reactions_to_event($event_slug) {
 
 		$index = $this->get_reaction_index();
-		$index = wp_list_filter( $index, array( 'event' => $event_slug ) );
-		return $this->create_reaction_objects( $index );
+		$index = wp_list_filter($index, array('event' => $event_slug));
+		return $this->create_reaction_objects($index);
 	}
 
 	/**
@@ -59,9 +59,9 @@ 
 	 */
 	protected function get_reaction_index() {
 
-		$index = $this->get_option( $this->get_reaction_index_option_name() );
+		$index = $this->get_option($this->get_reaction_index_option_name());
 
-		if ( ! is_array( $index ) ) {
+		if ( ! is_array($index)) {
 			$index = array();
 		}
 
@@ -77,7 +77,7 @@ 
 	 *
 	 * @return bool Whether the index was updated successfully.
 	 */
-	protected function update_reaction_index( $index ) {
+	protected function update_reaction_index($index) {
 
 		return $this->update_option(
 			$this->get_reaction_index_option_name()
@@ -96,15 +96,15 @@ 
 	 *
 	 * @return string|false The event slug, or false if not found.
 	 */
-	public function get_reaction_event_from_index( $id ) {
+	public function get_reaction_event_from_index($id) {
 
 		$index = $this->get_reaction_index();
 
-		if ( ! isset( $index[ $id ]['event'] ) ) {
+		if ( ! isset($index[$id]['event'])) {
 			return false;
 		}
 
-		return $index[ $id ]['event'];
+		return $index[$id]['event'];
 	}
 
 	/**
@@ -119,17 +119,17 @@ 
 	 *
 	 * @return bool Whether the event was updated successfully.
 	 */
-	public function update_reaction_event_in_index( $id, $event ) {
+	public function update_reaction_event_in_index($id, $event) {
 
 		$index = $this->get_reaction_index();
 
-		if ( ! isset( $index[ $id ] ) ) {
+		if ( ! isset($index[$id])) {
 			return false;
 		}
 
-		$index[ $id ]['event'] = $event;
+		$index[$id]['event'] = $event;
 
-		return $this->update_reaction_index( $index );
+		return $this->update_reaction_index($index);
 	}
 
 	/**
@@ -141,15 +141,15 @@ 
 	 *
 	 * @return WordPoints_Hook_Reaction_Options[] The objects for the reactions.
 	 */
-	protected function create_reaction_objects( $index ) {
+	protected function create_reaction_objects($index) {
 
 		$reactions = array();
 
-		foreach ( $index as $reaction ) {
+		foreach ($index as $reaction) {
 
-			$object = $this->get_reaction( $reaction['id'] );
+			$object = $this->get_reaction($reaction['id']);
 
-			if ( ! $object ) {
+			if ( ! $object) {
 				continue;
 			}
 
@@ -162,52 +162,52 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function delete_reaction( $id ) {
+	public function delete_reaction($id) {
 
-		if ( ! $this->reaction_exists( $id ) ) {
+		if ( ! $this->reaction_exists($id)) {
 			return false;
 		}
 
-		$result = $this->delete_option( $this->get_settings_option_name( $id ) );
+		$result = $this->delete_option($this->get_settings_option_name($id));
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			return false;
 		}
 
 		$index = $this->get_reaction_index();
 
-		unset( $index[ $id ] );
+		unset($index[$id]);
 
-		return $this->update_reaction_index( $index );
+		return $this->update_reaction_index($index);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function _create_reaction( $event_slug ) {
+	protected function _create_reaction($event_slug) {
 
 		$index = $this->get_reaction_index();
 
-		$id = $this->get_next_id( $index );
+		$id = $this->get_next_id($index);
 
-		$settings = array( 'event' => $event_slug );
+		$settings = array('event' => $event_slug);
 
 		$result = $this->add_option(
-			$this->get_settings_option_name( $id )
+			$this->get_settings_option_name($id)
 			, $settings
 		);
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			return false;
 		}
 
-		$index[ $id ] = array( 'event' => $event_slug, 'id' => $id );
+		$index[$id] = array('event' => $event_slug, 'id' => $id);
 
-		if ( ! $this->update_reaction_index( $index ) ) {
+		if ( ! $this->update_reaction_index($index)) {
 			return false;
 		}
 
-		$this->update_option( $this->get_last_reaction_id_option_name(), $id );
+		$this->update_option($this->get_last_reaction_id_option_name(), $id);
 
 		return $id;
 	}
@@ -221,23 +221,23 @@ 
 	 *
 	 * @return int The ID to use for the next reaction.
 	 */
-	protected function get_next_id( $index ) {
+	protected function get_next_id($index) {
 
 		$id = 1;
 
-		$last_id = $this->get_option( $this->get_last_reaction_id_option_name() );
+		$last_id = $this->get_option($this->get_last_reaction_id_option_name());
 
-		if ( wordpoints_posint( $last_id ) ) {
+		if (wordpoints_posint($last_id)) {
 			$id = $last_id + 1;
 		}
 
 		// Double-check this against the index. This should avoid some issues from
 		// race conditions.
-		if ( ! empty( $index ) ) {
+		if ( ! empty($index)) {
 
-			$max = max( array_keys( $index ) );
+			$max = max(array_keys($index));
 
-			if ( $max >= $id ) {
+			if ($max >= $id) {
 				$id = $max + 1;
 			}
 		}
@@ -256,8 +256,8 @@ 
 	 *
 	 * @return mixed The option value, or false.
 	 */
-	public function get_option( $name ) {
-		return get_option( $name );
+	public function get_option($name) {
+		return get_option($name);
 	}
 
 	/**
@@ -270,8 +270,8 @@ 
 	 *
 	 * @return bool Whether the option was added successfully.
 	 */
-	protected function add_option( $name, $value ) {
-		return add_option( $name, $value );
+	protected function add_option($name, $value) {
+		return add_option($name, $value);
 	}
 
 	/**
@@ -286,8 +286,8 @@ 
 	 *
 	 * @return bool Whether the option was updated successfully.
 	 */
-	public function update_option( $name, $value ) {
-		return update_option( $name, $value );
+	public function update_option($name, $value) {
+		return update_option($name, $value);
 	}
 
 	/**
@@ -299,8 +299,8 @@ 
 	 *
 	 * @return bool Whether the option was deleted successfully.
 	 */
-	protected function delete_option( $name ) {
-		return delete_option( $name );
+	protected function delete_option($name) {
+		return delete_option($name);
 	}
 
 	/**
@@ -314,7 +314,7 @@ 
 	 *
 	 * @return string The name of the option where the settings are stored.
 	 */
-	public function get_settings_option_name( $id ) {
+	public function get_settings_option_name($id) {
 
 		$current_mode = wordpoints_hooks()->get_current_mode();
 

src/includes/classes/hook/condition/string/contains.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Contains', 'wordpoints' );
+		return __('Contains', 'wordpoints');
 	}
 
 	/**
@@ -28,7 +28,7 @@ 
 		return array(
 			'value' => array(
 				'type'     => 'text',
-				'label'    => __( 'Word or phrase (case sensitive)', 'wordpoints' ),
+				'label'    => __('Word or phrase (case sensitive)', 'wordpoints'),
 				'required' => true,
 			),
 		);
@@ -37,7 +37,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args) {
 
 		return false !== strpos(
 			$args->get_current()->get_the_value()

src/includes/classes/entity/user/role/name.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return _x( 'Name', 'user role', 'wordpoints' );
+		return _x('Name', 'user role', 'wordpoints');
 	}
 }
 

src/includes/classes/entity/term/id.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return _x( 'ID', 'taxonomy term', 'wordpoints' );
+		return _x('ID', 'taxonomy term', 'wordpoints');
 	}
 }
 // EOF

src/includes/classes/hook/extension.php

Spacing +17 added lines, -17 removed lines

@@ -57,14 +57,14 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings[ $this->slug ] ) ) {
+		if ( ! isset($settings[$this->slug])) {
 			return $settings;
 		}
 
-		if ( ! is_array( $settings[ $this->slug ] ) ) {
+		if ( ! is_array($settings[$this->slug])) {
 
 			$validator->add_error(
-				__( 'Invalid settings format.', 'wordpoints' )
+				__('Invalid settings format.', 'wordpoints')
 				, $this->slug
 			);
 
@@ -74,13 +74,13 @@ 
 		$this->validator = $validator;
 		$this->event_args = $event_args;
 
-		$this->validator->push_field( $this->slug );
+		$this->validator->push_field($this->slug);
 
-		foreach ( $settings[ $this->slug ] as $action_type => $action_type_settings ) {
+		foreach ($settings[$this->slug] as $action_type => $action_type_settings) {
 
-			$this->validator->push_field( $action_type );
+			$this->validator->push_field($action_type);
 
-			$settings[ $this->slug ][ $action_type ] = $this->validate_action_type_settings(
+			$settings[$this->slug][$action_type] = $this->validate_action_type_settings(
 				$action_type_settings
 			);
 
@@ -95,12 +95,12 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function update_settings( WordPoints_Hook_ReactionI $reaction, array $settings ) {
+	public function update_settings(WordPoints_Hook_ReactionI $reaction, array $settings) {
 
-		if ( isset( $settings[ $this->slug ] ) ) {
-			$reaction->update_meta( $this->slug, $settings[ $this->slug ] );
+		if (isset($settings[$this->slug])) {
+			$reaction->update_meta($this->slug, $settings[$this->slug]);
 		} else {
-			$reaction->delete_meta( $this->slug );
+			$reaction->delete_meta($this->slug);
 		}
 	}
 
@@ -113,7 +113,7 @@ 
 	 *
 	 * @return mixed The validated settings.
 	 */
-	protected function validate_action_type_settings( $settings ) {
+	protected function validate_action_type_settings($settings) {
 		return $settings;
 	}
 
@@ -130,16 +130,16 @@ 
 	 *
 	 * @return mixed The settings for the extension, or false if none.
 	 */
-	protected function get_settings_from_fire( WordPoints_Hook_Fire $fire ) {
+	protected function get_settings_from_fire(WordPoints_Hook_Fire $fire) {
 
-		$settings = $fire->reaction->get_meta( $this->slug );
+		$settings = $fire->reaction->get_meta($this->slug);
 
-		if ( ! is_array( $settings ) ) {
+		if ( ! is_array($settings)) {
 			return $settings;
 		}
 
-		if ( isset( $settings[ $fire->action_type ] ) ) {
-			return $settings[ $fire->action_type ];
+		if (isset($settings[$fire->action_type])) {
+			return $settings[$fire->action_type];
 		} else {
 			return false;
 		}