WordPoints / hooks-api

src/admin/includes/functions.php

Spacing +89 added lines, -89 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,22 +82,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
 	);
 
@@ -105,11 +105,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'),
 		)
 	);
 
@@ -117,19 +117,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'),
 		)
 	);
 
@@ -141,10 +141,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">
@@ -163,16 +163,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>
@@ -220,15 +220,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
 	);
 
@@ -238,25 +238,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>
@@ -270,7 +270,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>
@@ -288,8 +288,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
 	);
 
@@ -299,7 +299,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>
@@ -348,7 +348,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'
@@ -368,18 +368,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}");
 		}
 	}
 
@@ -403,28 +403,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();
 			}
 
 			/**
@@ -435,14 +435,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,
@@ -450,12 +450,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(),
@@ -463,7 +463,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -472,9 +472,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
 		);
 	}
@@ -498,15 +498,15 @@ 
 
 	// 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;
 		}
 	}
 
@@ -529,13 +529,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;
 	}
 
@@ -551,7 +551,7 @@ 
  */
 function wordpoints_hooks_admin_ajax() {
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+	if (defined('DOING_AJAX') && DOING_AJAX) {
 		new WordPoints_Admin_Ajax_Hooks;
 	}
 }