WordPoints / wordpoints

src/classes/hook/reaction/validator.php

Spacing +7 added lines, -7 removed lines

@@ -119,12 +119,12 @@ 
 
 			$this->settings = $settings;
 
-			if ( isset( $this->settings['event'] ) ) {
-				$this->event_slug = $this->settings['event'];
+			if ( isset( $this->settings[ 'event' ] ) ) {
+				$this->event_slug = $this->settings[ 'event' ];
 			}
 
-			if ( isset( $this->settings['reactor'] ) ) {
-				$this->reactor_slug = $this->settings['reactor'];
+			if ( isset( $this->settings[ 'reactor' ] ) ) {
+				$this->reactor_slug = $this->settings[ 'reactor' ];
 			}
 		}
 	}
@@ -229,10 +229,10 @@ 
 		$field_stack = $this->field_stack;
 
 		if ( null !== $field ) {
-			$field_stack[] = $field;
+			$field_stack[ ] = $field;
 		}
 
-		$this->errors[] = array( 'message' => $message, 'field' => $field_stack );
+		$this->errors[ ] = array( 'message' => $message, 'field' => $field_stack );
 
 		if ( $this->fail_fast ) {
 			throw new WordPoints_Hook_Validator_Exception;
@@ -269,7 +269,7 @@ 
 	 * @param string $field The field.
 	 */
 	public function push_field( $field ) {
-		$this->field_stack[] = $field;
+		$this->field_stack[ ] = $field;
 	}
 
 	/**

src/classes/hook/action.php

Spacing +7 added lines, -7 removed lines

@@ -88,12 +88,12 @@ 
 		$this->slug = $slug;
 		$this->args = $action_args;
 
-		if ( isset( $args['requirements'] ) ) {
-			$this->requirements = $args['requirements'];
+		if ( isset( $args[ 'requirements' ] ) ) {
+			$this->requirements = $args[ 'requirements' ];
 		}
 
-		if ( isset( $args['arg_index'] ) ) {
-			$this->arg_index = $args['arg_index'];
+		if ( isset( $args[ 'arg_index' ] ) ) {
+			$this->arg_index = $args[ 'arg_index' ];
 		}
 	}
 
@@ -113,9 +113,9 @@ 
 
 			$comparator = '=';
 
-			if ( is_array( $value ) && isset( $value['comparator'], $value['value'] ) ) {
-				$comparator = $value['comparator'];
-				$value = $value['value'];
+			if ( is_array( $value ) && isset( $value[ 'comparator' ], $value[ 'value' ] ) ) {
+				$comparator = $value[ 'comparator' ];
+				$value = $value[ 'value' ];
 			}
 
 			switch ( $comparator ) {

src/classes/hook/router.php

Spacing +23 added lines, -23 removed lines

@@ -95,8 +95,8 @@ 
 
 		// Return the first value, in case it is hooked to a filter.
 		$return = null;
-		if ( isset( $args[0] ) ) {
-			$return = $args[0];
+		if ( isset( $args[ 0 ] ) ) {
+			$return = $args[ 0 ];
 		}
 
 		return $return;
@@ -132,7 +132,7 @@ 
 			$this->event_args = $this->events->get_sub_app( 'args' );
 		}
 
-		foreach ( $this->action_index[ $name ]['actions'] as $slug => $data ) {
+		foreach ( $this->action_index[ $name ][ 'actions' ] as $slug => $data ) {
 
 			if ( ! isset( $this->event_index[ $slug ] ) ) {
 				continue;
@@ -195,53 +195,53 @@ 
 	public function add_action( $slug, array $args ) {
 
 		$priority = 10;
-		if ( isset( $args['priority'] ) ) {
-			$priority = $args['priority'];
+		if ( isset( $args[ 'priority' ] ) ) {
+			$priority = $args[ 'priority' ];
 		}
 
-		if ( ! isset( $args['action'] ) ) {
+		if ( ! isset( $args[ 'action' ] ) ) {
 			return;
 		}
 
-		$method = "{$args['action']},{$priority}";
+		$method = "{$args[ 'action' ]},{$priority}";
 
-		$this->action_index[ $method ]['actions'][ $slug ] = array();
+		$this->action_index[ $method ][ 'actions' ][ $slug ] = array();
 
 		$arg_number = 1;
 
-		if ( isset( $args['data'] ) ) {
+		if ( isset( $args[ 'data' ] ) ) {
 
-			if ( isset( $args['data']['arg_index'] ) ) {
-				$arg_number = 1 + max( $args['data']['arg_index'] );
+			if ( isset( $args[ 'data' ][ 'arg_index' ] ) ) {
+				$arg_number = 1 + max( $args[ 'data' ][ 'arg_index' ] );
 			}
 
-			if ( isset( $args['data']['requirements'] ) ) {
-				$requirements = 1 + max( array_keys( $args['data']['requirements'] ) );
+			if ( isset( $args[ 'data' ][ 'requirements' ] ) ) {
+				$requirements = 1 + max( array_keys( $args[ 'data' ][ 'requirements' ] ) );
 
 				if ( $requirements > $arg_number ) {
 					$arg_number = $requirements;
 				}
 			}
 
-			$this->action_index[ $method ]['actions'][ $slug ] = $args['data'];
+			$this->action_index[ $method ][ 'actions' ][ $slug ] = $args[ 'data' ];
 		}
 
-		if ( isset( $args['arg_number'] ) ) {
-			$arg_number = $args['arg_number'];
+		if ( isset( $args[ 'arg_number' ] ) ) {
+			$arg_number = $args[ 'arg_number' ];
 		}
 
 		// If this action is already being routed, and will have enough args, we
 		// don't need to hook to it again.
 		if (
-			isset( $this->action_index[ $method ]['arg_number'] )
-			&& $this->action_index[ $method ]['arg_number'] >= $arg_number
+			isset( $this->action_index[ $method ][ 'arg_number' ] )
+			&& $this->action_index[ $method ][ 'arg_number' ] >= $arg_number
 		) {
 			return;
 		}
 
-		$this->action_index[ $method ]['arg_number'] = $arg_number;
+		$this->action_index[ $method ][ 'arg_number' ] = $arg_number;
 
-		add_action( $args['action'], array( $this, $method ), $priority, $arg_number );
+		add_action( $args[ 'action' ], array( $this, $method ), $priority, $arg_number );
 	}
 
 	/**
@@ -254,11 +254,11 @@ 
 	public function remove_action( $slug ) {
 
 		foreach ( $this->action_index as $method => $data ) {
-			if ( isset( $data['actions'][ $slug ] ) ) {
+			if ( isset( $data[ 'actions' ][ $slug ] ) ) {
 
-				unset( $this->action_index[ $method ]['actions'][ $slug ] );
+				unset( $this->action_index[ $method ][ 'actions' ][ $slug ] );
 
-				if ( empty( $this->action_index[ $method ]['actions'] ) ) {
+				if ( empty( $this->action_index[ $method ][ 'actions' ] ) ) {
 
 					unset( $this->action_index[ $method ] );
 

src/classes/hook/action/post/type.php

Spacing +5 added lines, -5 removed lines

@@ -50,23 +50,23 @@ 
 
 		$parts = wordpoints_parse_dynamic_slug( $this->slug );
 
-		if ( ! $parts['dynamic'] ) {
+		if ( ! $parts[ 'dynamic' ] ) {
 			return false;
 		}
 
 		$this->post_hierarchy = str_replace(
 			'\\post'
-			, '\\' . $parts['dynamic']
+			, '\\' . $parts[ 'dynamic' ]
 			, $this->post_hierarchy
 		);
 
-		$entity = wordpoints_entities()->get( $this->post_hierarchy[0] );
+		$entity = wordpoints_entities()->get( $this->post_hierarchy[ 0 ] );
 
 		if ( ! $entity instanceof WordPoints_Entity ) {
 			return false;
 		}
 
-		$entity->set_the_value( $this->get_arg_value( $this->post_hierarchy[0] ) );
+		$entity->set_the_value( $this->get_arg_value( $this->post_hierarchy[ 0 ] ) );
 
 		if ( 1 === count( $this->post_hierarchy ) ) {
 
@@ -83,7 +83,7 @@ 
 			}
 		}
 
-		if ( $parts['dynamic'] !== $post_entity->get_the_attr_value( 'post_type' ) ) {
+		if ( $parts[ 'dynamic' ] !== $post_entity->get_the_attr_value( 'post_type' ) ) {
 			return false;
 		}
 

src/classes/hook/action/comment/new.php

Spacing +2 added lines, -2 removed lines

@@ -25,11 +25,11 @@
 	 */
 	public function should_fire() {
 
-		if ( ! isset( $this->args[1]->comment_approved ) ) {
+		if ( ! isset( $this->args[ 1 ]->comment_approved ) ) {
 			return false;
 		}
 
-		if ( 1 !== (int) $this->args[1]->comment_approved ) {
+		if ( 1 !== (int) $this->args[ 1 ]->comment_approved ) {
 			return false;
 		}
 

src/classes/hook/arg.php

Spacing +2 added lines, -2 removed lines

@@ -77,8 +77,8 @@
 
 		$parts = explode( ':', $slug, 2 );
 
-		if ( isset( $parts[1] ) ) {
-			$this->entity_slug = $parts[1];
+		if ( isset( $parts[ 1 ] ) ) {
+			$this->entity_slug = $parts[ 1 ];
 		} else {
 			$this->entity_slug = $slug;
 		}

src/classes/hook/extension/periods.php

Spacing +18 added lines, -18 removed lines

@@ -41,10 +41,10 @@ 
 		$period_units = array(
 			1                   => __( 'Seconds', 'wordpoints' ),
 			MINUTE_IN_SECONDS   => __( 'Minutes', 'wordpoints' ),
-			HOUR_IN_SECONDS     => __( 'Hours',   'wordpoints' ),
-			DAY_IN_SECONDS      => __( 'Days',    'wordpoints' ),
-			WEEK_IN_SECONDS     => __( 'Weeks',   'wordpoints' ),
-			30 * DAY_IN_SECONDS => __( 'Months',  'wordpoints' ),
+			HOUR_IN_SECONDS     => __( 'Hours', 'wordpoints' ),
+			DAY_IN_SECONDS      => __( 'Days', 'wordpoints' ),
+			WEEK_IN_SECONDS     => __( 'Weeks', 'wordpoints' ),
+			30 * DAY_IN_SECONDS => __( 'Months', 'wordpoints' ),
 		);
 
 		/**
@@ -122,17 +122,17 @@ 
 			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' )
 			);
 
-		} elseif ( false === wordpoints_posint( $period['length'] ) ) {
+		} elseif ( false === wordpoints_posint( $period[ 'length' ] ) ) {
 
 			$this->validator->add_error(
 				__( 'Period length must be a positive integer.', 'wordpoints' )
@@ -237,17 +237,17 @@ 
 		$now = current_time( 'timestamp', true );
 		$hit_time = strtotime( $period->date, $now );
 
-		if ( ! empty( $settings['relative'] ) ) {
+		if ( ! empty( $settings[ 'relative' ] ) ) {
 
-			return ( $now > $hit_time + $settings['length'] );
+			return ( $now > $hit_time + $settings[ 'length' ] );
 
 		} else {
 
 			$offset = get_option( 'gmt_offset' ) * HOUR_IN_SECONDS;
 
 			return (
-				(int) ( ( $hit_time + $offset ) / $settings['length'] )
-				< (int) ( ( $now + $offset ) / $settings['length'] )
+				(int) ( ( $hit_time + $offset ) / $settings[ 'length' ] )
+				< (int) ( ( $now + $offset ) / $settings[ 'length' ] )
 			);
 		}
 	}
@@ -371,10 +371,10 @@ 
 					LIMIT 1
 				"
 				, $signature
-				, $reaction_guid['mode']
-				, $reaction_guid['store']
-				, wp_json_encode( $reaction_guid['context_id'] )
-				, $reaction_guid['id']
+				, $reaction_guid[ 'mode' ]
+				, $reaction_guid[ 'store' ]
+				, wp_json_encode( $reaction_guid[ 'context_id' ] )
+				, $reaction_guid[ 'id' ]
 				, $this->action_type
 			)
 		);
@@ -431,8 +431,8 @@ 
 		WordPoints_Hook_ReactionI $reaction
 	) {
 
-		if ( isset( $settings['args'] ) ) {
-			$period_args = $settings['args'];
+		if ( isset( $settings[ 'args' ] ) ) {
+			$period_args = $settings[ 'args' ];
 		} else {
 			$period_args = array( $reaction->get_meta( 'target' ) );
 		}

src/classes/hook/extension/conditions.php

Spacing +11 added lines, -11 removed lines

@@ -155,7 +155,7 @@ 
 	 */
 	protected function validate_condition( $settings ) {
 
-		if ( ! isset( $settings['type'] ) ) {
+		if ( ! isset( $settings[ 'type' ] ) ) {
 			$this->validator->add_error( __( 'Condition type is missing.', 'wordpoints' ) );
 			return false;
 		}
@@ -172,7 +172,7 @@ 
 			return false;
 		}
 
-		$condition = $this->conditions->get( $data_type, $settings['type'] );
+		$condition = $this->conditions->get( $data_type, $settings[ 'type' ] );
 
 		if ( ! $condition ) {
 
@@ -180,7 +180,7 @@ 
 				sprintf(
 					// translators: Condition type slug.
 					__( 'Unknown condition type “%s”.', 'wordpoints' )
-					, $settings['type']
+					, $settings[ 'type' ]
 				)
 				, 'type'
 			);
@@ -188,7 +188,7 @@ 
 			return false;
 		}
 
-		if ( ! isset( $settings['settings'] ) ) {
+		if ( ! isset( $settings[ 'settings' ] ) ) {
 			$this->validator->add_error( __( 'Condition settings are missing.', 'wordpoints' ) );
 			return false;
 		}
@@ -200,9 +200,9 @@ 
 		// reset, so we need to back up their values and then restore them below.
 		$backup = array( $this->validator, $this->event_args );
 
-		$settings['settings'] = $condition->validate_settings(
+		$settings[ 'settings' ] = $condition->validate_settings(
 			$arg
-			, $settings['settings']
+			, $settings[ 'settings' ]
 			, $this->validator
 		);
 
@@ -246,16 +246,16 @@ 
 
 			$event_args->descend( $arg_slug );
 
-			if ( isset( $sub_args['_conditions'] ) ) {
+			if ( isset( $sub_args[ '_conditions' ] ) ) {
 
-				foreach ( $sub_args['_conditions'] as $settings ) {
+				foreach ( $sub_args[ '_conditions' ] as $settings ) {
 
 					$condition = $this->conditions->get(
 						$this->get_data_type( $event_args->get_current() )
-						, $settings['type']
+						, $settings[ 'type' ]
 					);
 
-					$is_met = $condition->is_met( $settings['settings'], $event_args );
+					$is_met = $condition->is_met( $settings[ 'settings' ], $event_args );
 
 					if ( ! $is_met ) {
 						$event_args->ascend();
@@ -263,7 +263,7 @@ 
 					}
 				}
 
-				unset( $sub_args['_conditions'] );
+				unset( $sub_args[ '_conditions' ] );
 			}
 
 			$are_met = $this->conditions_are_met( $sub_args, $event_args );

src/classes/hook/extension/reversals.php

Spacing +3 added lines, -3 removed lines

@@ -77,8 +77,8 @@ 
 
 		// We cache these so that we don't run the query both before and after the
 		// fire.
-		if ( isset( $fire->data[ $this->slug ]['hit_ids'] ) ) {
-			return $fire->data[ $this->slug ]['hit_ids'];
+		if ( isset( $fire->data[ $this->slug ][ 'hit_ids' ] ) ) {
+			return $fire->data[ $this->slug ][ 'hit_ids' ];
 		}
 
 		$query = $fire->get_matching_hits_query();
@@ -98,7 +98,7 @@ 
 			$ids = array();
 		}
 
-		$fire->data[ $this->slug ]['hit_ids'] = $ids;
+		$fire->data[ $this->slug ][ 'hit_ids' ] = $ids;
 
 		return $ids;
 	}