WordPoints / wordpoints

src/components/points/classes/hook/extension/legacy/periods.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
 		$expected_settings = array( array( 'current:user' ) );
 
-		if ( ! isset( $settings['args'] ) || $settings['args'] !== $expected_settings ) {
+		if ( ! isset( $settings[ 'args' ] ) || $settings[ 'args' ] !== $expected_settings ) {
 			return false;
 		}
 

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

Spacing +5 added lines, -5 removed lines

@@ -65,8 +65,8 @@ 
 	 */
 	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' ];
 		}
 
 		$meta_queries = array(
@@ -85,14 +85,14 @@ 
 			$entities = $fire->event_args->get_signature_args();
 
 			if ( ! $entities ) {
-				$fire->data[ $this->slug ]['points_logs'] = array();
+				$fire->data[ $this->slug ][ 'points_logs' ] = array();
 				return array();
 			}
 
 			// Legacy hooks only ever related to a single entity.
 			$entity = reset( $entities );
 
-			$meta_queries[] = array(
+			$meta_queries[ ] = array(
 				'key'   => $meta_key,
 				'value' => $entity->get_the_id(),
 			);
@@ -117,7 +117,7 @@ 
 			$logs = array();
 		}
 
-		$fire->data[ $this->slug ]['points_logs'] = $logs;
+		$fire->data[ $this->slug ][ 'points_logs' ] = $logs;
 
 		return $logs;
 	}

src/components/points/classes/hook/reactor.php

Spacing +15 added lines, -15 removed lines

@@ -58,9 +58,9 @@ 
 	 */
 	public function get_settings_fields() {
 
-		$this->settings_fields['points']['label'] = _x( 'Points', 'form label', 'wordpoints' );
-		$this->settings_fields['log_text']['label'] = _x( 'Log Text', 'form label', 'wordpoints' );
-		$this->settings_fields['description']['label'] = _x( 'Description', 'form label', 'wordpoints' );
+		$this->settings_fields[ 'points' ][ 'label' ] = _x( 'Points', 'form label', 'wordpoints' );
+		$this->settings_fields[ 'log_text' ][ 'label' ] = _x( 'Log Text', 'form label', 'wordpoints' );
+		$this->settings_fields[ 'description' ][ 'label' ] = _x( 'Description', 'form label', 'wordpoints' );
 
 		return parent::get_settings_fields();
 	}
@@ -72,8 +72,8 @@ 
 
 		$data = parent::get_ui_script_data();
 
-		$data['target_label'] = __( 'Award To', 'wordpoints' );
-		$data['periods_label'] = __( 'Award each user no more than once in:', 'wordpoints' );
+		$data[ 'target_label' ] = __( 'Award To', 'wordpoints' );
+		$data[ 'periods_label' ] = __( 'Award each user no more than once in:', 'wordpoints' );
 
 		return $data;
 	}
@@ -87,19 +87,19 @@ 
 		WordPoints_Hook_Event_Args $event_args
 	) {
 
-		if ( ! isset( $settings['points'] ) || false === wordpoints_int( $settings['points'] ) ) {
+		if ( ! isset( $settings[ 'points' ] ) || false === wordpoints_int( $settings[ 'points' ] ) ) {
 			$validator->add_error( __( 'Points must be an integer.', 'wordpoints' ), 'points' );
 		}
 
-		if ( ! isset( $settings['points_type'] ) || ! wordpoints_is_points_type( $settings['points_type'] ) ) {
+		if ( ! isset( $settings[ 'points_type' ] ) || ! wordpoints_is_points_type( $settings[ 'points_type' ] ) ) {
 			$validator->add_error( __( 'Invalid points type.', 'wordpoints' ), 'points_type' );
 		}
 
-		if ( ! isset( $settings['description'] ) ) {
+		if ( ! isset( $settings[ 'description' ] ) ) {
 			$validator->add_error( __( 'Description is required.', 'wordpoints' ), 'description' );
 		}
 
-		if ( ! isset( $settings['log_text'] ) ) {
+		if ( ! isset( $settings[ 'log_text' ] ) ) {
 			$validator->add_error( __( 'Log Text is required.', 'wordpoints' ), 'log_text' );
 		}
 
@@ -116,10 +116,10 @@ 
 
 		parent::update_settings( $reaction, $settings );
 
-		$reaction->update_meta( 'points', $settings['points'] );
-		$reaction->update_meta( 'points_type', $settings['points_type'] );
-		$reaction->update_meta( 'description', $settings['description'] );
-		$reaction->update_meta( 'log_text', $settings['log_text'] );
+		$reaction->update_meta( 'points', $settings[ 'points' ] );
+		$reaction->update_meta( 'points_type', $settings[ 'points_type' ] );
+		$reaction->update_meta( 'description', $settings[ 'description' ] );
+		$reaction->update_meta( 'log_text', $settings[ 'log_text' ] );
 	}
 
 	/**
@@ -203,11 +203,11 @@ 
 	protected function get_hit_ids_to_be_reversed( WordPoints_Hook_Fire $fire ) {
 
 		// We closely integrate with the reversals extension to get the hit IDs.
-		if ( ! isset( $fire->data['reversals']['hit_ids'] ) ) {
+		if ( ! isset( $fire->data[ 'reversals' ][ 'hit_ids' ] ) ) {
 			return array();
 		}
 
-		return $fire->data['reversals']['hit_ids'];
+		return $fire->data[ 'reversals' ][ 'hit_ids' ];
 	}
 
 	/**

src/components/points/classes/hook/event/post/publish/legacy.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
 		$parsed = wordpoints_parse_dynamic_slug( $this->slug );
 
-		switch ( $parsed['dynamic'] ) {
+		switch ( $parsed[ 'dynamic' ] ) {
 
 			case 'post':
 				return __( 'Post deleted.', 'wordpoints' );

src/components/points/classes/legacy/hook/to/reaction/importer.php

Spacing +31 added lines, -31 removed lines

@@ -273,7 +273,7 @@ 
 		);
 
 		$reaction_settings = array(
-			'points'          => $legacy_instance['points'],
+			'points'          => $legacy_instance[ 'points' ],
 			'target'          => $this->target,
 			'reactor'         => 'points_legacy',
 			'event'           => $this->event_slug,
@@ -294,8 +294,8 @@ 
 			$order
 		);
 
-		if ( ! isset( $reaction_settings['log_text'] ) ) {
-			$reaction_settings['log_text'] = $this->get_log_text_for_instance(
+		if ( ! isset( $reaction_settings[ 'log_text' ] ) ) {
+			$reaction_settings[ 'log_text' ] = $this->get_log_text_for_instance(
 				$points_type
 			);
 		}
@@ -333,8 +333,8 @@ 
 
 		if (
 			$this->skip_non_reversing_hooks
-			&& isset( $this->legacy_instance['auto_reverse'] )
-			&& ! $this->legacy_instance['auto_reverse']
+			&& isset( $this->legacy_instance[ 'auto_reverse' ] )
+			&& ! $this->legacy_instance[ 'auto_reverse' ]
 		) {
 			return false;
 		}
@@ -359,17 +359,17 @@ 
 		}
 
 		if (
-			! isset( $meta['post_type'] )
-			&& isset( $this->legacy_instance['post_type'] )
+			! isset( $meta[ 'post_type' ] )
+			&& isset( $this->legacy_instance[ 'post_type' ] )
 		) {
-			$meta['post_type'] = $this->legacy_instance['post_type'];
+			$meta[ 'post_type' ] = $this->legacy_instance[ 'post_type' ];
 		}
 
 		if (
-			! isset( $meta['period'] )
-			&& isset( $this->legacy_instance['period'] )
+			! isset( $meta[ 'period' ] )
+			&& isset( $this->legacy_instance[ 'period' ] )
 		) {
-			$meta['period'] = $this->legacy_instance['period'];
+			$meta[ 'period' ] = $this->legacy_instance[ 'period' ];
 		}
 
 		return $this->legacy_handler->logs(
@@ -398,10 +398,10 @@ 
 
 		if ( 'wordpoints_periodic_points_hook' === $this->legacy_id_base ) {
 
-			$settings['points_legacy_periods'] = array(
+			$settings[ 'points_legacy_periods' ] = array(
 				'fire' => array(
 					array(
-						'length' => $this->legacy_instance['period'],
+						'length' => $this->legacy_instance[ 'period' ],
 						'args' => array( array( 'current:user' ) ),
 					),
 				),
@@ -411,21 +411,21 @@ 
 
 			// We do this even if reversals will be blocked, in case the blocking
 			// is ever removed for this reaction.
-			$settings['points_legacy_reversals'] = array(
+			$settings[ 'points_legacy_reversals' ] = array(
 				'toggle_off' => 'toggle_on',
 			);
 		}
 
 		if (
-			isset( $this->legacy_instance['auto_reverse'] )
-			&& ! $this->legacy_instance['auto_reverse']
+			isset( $this->legacy_instance[ 'auto_reverse' ] )
+			&& ! $this->legacy_instance[ 'auto_reverse' ]
 		) {
-			$settings['blocker']['toggle_off'] = true;
+			$settings[ 'blocker' ][ 'toggle_off' ] = true;
 		}
 
-		if ( isset( $this->legacy_instance['post_type'] ) ) {
+		if ( isset( $this->legacy_instance[ 'post_type' ] ) ) {
 
-			$post_type = $this->legacy_instance['post_type'];
+			$post_type = $this->legacy_instance[ 'post_type' ];
 
 			if ( 'ALL' === $post_type ) {
 
@@ -467,36 +467,36 @@ 
 	protected function format_settings_for_post_type( $post_type, $settings ) {
 
 		if (
-			'post_publish\post' === $settings['event']
-			|| 'points_legacy_post_publish\post' === $settings['event']
+			'post_publish\post' === $settings[ 'event' ]
+			|| 'points_legacy_post_publish\post' === $settings[ 'event' ]
 		) {
 
 			if ( 'attachment' === $post_type ) {
 
-				$settings['event'] = 'media_upload';
+				$settings[ 'event' ] = 'media_upload';
 
 			} else {
 
-				$settings['points_legacy_repeat_blocker'] = array(
+				$settings[ 'points_legacy_repeat_blocker' ] = array(
 					'toggle_on' => true,
 				);
 			}
 		}
 
-		$settings['event'] = str_replace(
+		$settings[ 'event' ] = str_replace(
 			'\post'
 			, '\\' . $post_type
-			, $settings['event']
+			, $settings[ 'event' ]
 		);
 
-		$settings['target'] = str_replace(
+		$settings[ 'target' ] = str_replace(
 			'\post'
 			, '\\' . $post_type
-			, $settings['target']
+			, $settings[ 'target' ]
 		);
 
-		$settings['log_text'] = $this->get_log_text_for_instance(
-			$settings['points_type']
+		$settings[ 'log_text' ] = $this->get_log_text_for_instance(
+			$settings[ 'points_type' ]
 			, array( 'post_type' => $post_type )
 		);
 
@@ -521,11 +521,11 @@ 
 			return false;
 		}
 
-		$this->imported_hooks[] = array(
+		$this->imported_hooks[ ] = array(
 			'order'       => $order,
 			'id_base'     => $this->legacy_id_base,
 			'instance'    => $this->legacy_instance,
-			'points_type' => $settings['points_type'],
+			'points_type' => $settings[ 'points_type' ],
 			'reaction_id' => $reaction->get_id(),
 		);
 

src/components/points/classes/widget/top/users.php

Spacing +9 added lines, -9 removed lines

@@ -50,8 +50,8 @@ 
 	 */
 	protected function verify_settings( $instance ) {
 
-		if ( empty( $instance['num_users'] ) ) {
-			$instance['num_users'] = $this->defaults['num_users'];
+		if ( empty( $instance[ 'num_users' ] ) ) {
+			$instance[ 'num_users' ] = $this->defaults[ 'num_users' ];
 		}
 
 		return parent::verify_settings( $instance );
@@ -64,8 +64,8 @@ 
 	protected function widget_body( $instance ) {
 
 		wordpoints_points_show_top_users(
-			$instance['num_users']
-			, $instance['points_type']
+			$instance[ 'num_users' ]
+			, $instance[ 'points_type' ]
 			, 'widget'
 		);
 	}
@@ -85,8 +85,8 @@ 
 
 		parent::update( $new_instance, $old_instance );
 
-		if ( ! wordpoints_posint( $this->instance['num_users'] ) ) {
-			$this->instance['num_users'] = $this->defaults['num_users'];
+		if ( ! wordpoints_posint( $this->instance[ 'num_users' ] ) ) {
+			$this->instance[ 'num_users' ] = $this->defaults[ 'num_users' ];
 		}
 
 		return $this->instance;
@@ -100,15 +100,15 @@ 
 
 		parent::form( $instance );
 
-		if ( ! wordpoints_posint( $this->instance['num_users'] ) ) {
-			$this->instance['num_users'] = $this->defaults['num_users'];
+		if ( ! wordpoints_posint( $this->instance[ 'num_users' ] ) ) {
+			$this->instance[ 'num_users' ] = $this->defaults[ 'num_users' ];
 		}
 
 		?>
 
 		<p>
 			<label for="<?php echo esc_attr( $this->get_field_id( 'num_users' ) ); ?>"><?php esc_html_e( 'Number of top users to show', 'wordpoints' ); ?></label>
-			<input type="number" min="1" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'num_users' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'num_users' ) ); ?>" value="<?php echo absint( $this->instance['num_users'] ); ?>" />
+			<input type="number" min="1" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'num_users' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'num_users' ) ); ?>" value="<?php echo absint( $this->instance[ 'num_users' ] ); ?>" />
 		</p>
 
 		<?php

src/components/ranks/includes/class-un-installer.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 	protected function before_update() {
 
 		if ( $this->network_wide ) {
-			unset( $this->updates['1_8_0'] );
+			unset( $this->updates[ '1_8_0' ] );
 		}
 	}
 

src/components/ranks/includes/class-wordpoints-rank-types.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 			return false;
 		}
 
-		$args['slug'] = $type;
+		$args[ 'slug' ] = $type;
 
 		self::$types[ $type ] = new $class( $args );
 

src/components/ranks/includes/integration/points/functions.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
 
 		$rank = wordpoints_get_formatted_user_rank(
 			get_current_user_id()
-			, "points_type-{$instance['points_type']}"
+			, "points_type-{$instance[ 'points_type' ]}"
 			, 'my-points-widget'
 			, array( 'widget_settings' => $instance )
 		);
@@ -76,18 +76,18 @@ 
  */
 function wordpoints_user_rank_shortcode_points_type_attr( $out, $pairs, $atts ) {
 
-	if ( empty( $out['rank_group'] ) ) {
+	if ( empty( $out[ 'rank_group' ] ) ) {
 
-		if ( isset( $atts['points_type'] ) ) {
+		if ( isset( $atts[ 'points_type' ] ) ) {
 
-			$out['rank_group'] = "points_type-{$atts['points_type']}";
+			$out[ 'rank_group' ] = "points_type-{$atts[ 'points_type' ]}";
 
 		} else {
 
 			$points_type = wordpoints_get_default_points_type();
 
 			if ( $points_type ) {
-				$out['rank_group'] = "points_type-{$points_type}";
+				$out[ 'rank_group' ] = "points_type-{$points_type}";
 			}
 		}
 	}
@@ -109,11 +109,11 @@ 
 		WordPoints_Rank_Groups::register_group(
 			"points_type-{$slug}"
 			, array(
-				'name' => $points_type['name'],
+				'name' => $points_type[ 'name' ],
 				'description' => sprintf(
 					// translators: Points type name.
 					__( 'This rank group is associated with the “%s” points type.', 'wordpoints' )
-					, $points_type['name']
+					, $points_type[ 'name' ]
 				),
 			)
 		);
@@ -170,7 +170,7 @@ 
 
 	?>
 
-	<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_ranks&tab=points_type-' . $points_type['slug'] ) ); ?>">
+	<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_ranks&tab=points_type-' . $points_type[ 'slug' ] ) ); ?>">
 		<?php esc_html_e( 'Go to the ranks for this points type.', 'wordpoints' ); ?>
 	</a>