WordPoints / wordpoints

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

Spacing +10 added lines, -10 removed lines

@@ -77,7 +77,7 @@ 
 	 */
 	public function __construct( array $args ) {
 
-		$this->slug = $args['slug'];
+		$this->slug = $args[ 'slug' ];
 	}
 
 	//
@@ -193,10 +193,10 @@ 
 
 			// If we aren't using placeholders, calculate the value. Hidden fields
 			// never use placeholders.
-			if ( ! $args['placeholders'] || 'hidden' === $field['type'] ) {
+			if ( ! $args[ 'placeholders' ] || 'hidden' === $field[ 'type' ] ) {
 
 				// Default to the default value.
-				$value = $field['default'];
+				$value = $field[ 'default' ];
 
 				// If the value is set use that instead.
 				if ( isset( $meta[ $name ] ) ) {
@@ -204,21 +204,21 @@ 
 				}
 			}
 
-			switch ( $field['type'] ) {
+			switch ( $field[ 'type' ] ) {
 
 				case 'hidden':
 				case 'number':
 				case 'text':
-					if ( isset( $field['label'] ) ) {
+					if ( isset( $field[ 'label' ] ) ) {
 						?><p class="description description-thin"><label><?php
-						echo esc_html( $field['label'] );
+						echo esc_html( $field[ 'label' ] );
 					}
 
 					?>
 					<input
-						type="<?php echo esc_attr( $field['type'] ); ?>"
+						type="<?php echo esc_attr( $field[ 'type' ] ); ?>"
 						name="<?php echo esc_attr( $name ); ?>"
-						<?php if ( $args['placeholders'] && 'hidden' !== $field['type'] ) : ?>
+						<?php if ( $args[ 'placeholders' ] && 'hidden' !== $field[ 'type' ] ) : ?>
 						value="<% if ( typeof <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); // WPCS: XSS OK ?> !== "undefined" ) { print( <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); // WPCS: XSS OK ?> ); } %>"
 						<?php else : ?>
 						value="<?php echo esc_attr( $value ); ?>"
@@ -227,7 +227,7 @@ 
 					/>
 					<?php
 
-					if ( isset( $field['label'] ) ) {
+					if ( isset( $field[ 'label' ] ) ) {
 						?></label></p><?php
 					}
 				break;
@@ -237,7 +237,7 @@ 
 						__METHOD__
 						, sprintf(
 							'WordPoints Error: Unknown field type "%s".'
-							, esc_html( $field['type'] )
+							, esc_html( $field[ 'type' ] )
 						)
 						, '1.7.0'
 					);

src/classes/db/query.php

Spacing +40 added lines, -40 removed lines

@@ -237,12 +237,12 @@ 
 			if ( isset( $args[ $arg ] ) ) {
 
 				_deprecated_argument(
-					esc_html( "{$data['class']}::__construct" )
-					, esc_html( $data['version'] )
-					, esc_html( "{$arg} is deprecated, use {$data['replacement']} instead" )
+					esc_html( "{$data[ 'class' ]}::__construct" )
+					, esc_html( $data[ 'version' ] )
+					, esc_html( "{$arg} is deprecated, use {$data[ 'replacement' ]} instead" )
 				);
 
-				$args[ $data['replacement'] ] = $args[ $arg ];
+				$args[ $data[ 'replacement' ] ] = $args[ $arg ];
 
 				unset( $args[ $arg ] );
 			}
@@ -265,12 +265,12 @@ 
 		if ( isset( $this->deprecated_args[ $arg ] ) ) {
 
 			_deprecated_argument(
-				esc_html( "{$this->deprecated_args[ $arg ]['class']}::get_arg" )
-				, esc_html( $this->deprecated_args[ $arg ]['version'] )
-				, esc_html( "{$arg} is deprecated, use {$this->deprecated_args[ $arg ]['replacement']} instead" )
+				esc_html( "{$this->deprecated_args[ $arg ][ 'class' ]}::get_arg" )
+				, esc_html( $this->deprecated_args[ $arg ][ 'version' ] )
+				, esc_html( "{$arg} is deprecated, use {$this->deprecated_args[ $arg ][ 'replacement' ]} instead" )
 			);
 
-			$arg = $this->deprecated_args[ $arg ]['replacement'];
+			$arg = $this->deprecated_args[ $arg ][ 'replacement' ];
 		}
 
 		if ( isset( $this->args[ $arg ] ) ) {
@@ -298,12 +298,12 @@ 
 			if ( isset( $args[ $arg ] ) ) {
 
 				_deprecated_argument(
-					esc_html( "{$data['class']}::set_args" )
-					, esc_html( $data['version'] )
-					, esc_html( "{$arg} is deprecated, use {$data['replacement']} instead" )
+					esc_html( "{$data[ 'class' ]}::set_args" )
+					, esc_html( $data[ 'version' ] )
+					, esc_html( "{$arg} is deprecated, use {$data[ 'replacement' ]} instead" )
 				);
 
-				$args[ $data['replacement'] ] = $args[ $arg ];
+				$args[ $data[ 'replacement' ] ] = $args[ $arg ];
 
 				unset( $args[ $arg ] );
 			}
@@ -456,9 +456,9 @@ 
 		$all_fields = array_keys( $this->columns );
 		$fields = array();
 
-		if ( ! empty( $this->args['fields'] ) ) {
+		if ( ! empty( $this->args[ 'fields' ] ) ) {
 
-			$fields = (array) $this->args['fields'];
+			$fields = (array) $this->args[ 'fields' ];
 			$diff   = array_diff( $fields, $all_fields );
 			$fields = array_intersect( $all_fields, $fields );
 
@@ -560,11 +560,11 @@ 
 		$validators = array();
 
 		// Default validators for integer columns.
-		if ( '%d' === $data['format'] ) {
-			if ( ! empty( $data['unsigned'] ) ) {
-				$validators[] = array( $this, 'validate_unsigned_column' );
+		if ( '%d' === $data[ 'format' ] ) {
+			if ( ! empty( $data[ 'unsigned' ] ) ) {
+				$validators[ ] = array( $this, 'validate_unsigned_column' );
 			} else {
-				$validators[] = 'wordpoints_int';
+				$validators[ ] = 'wordpoints_int';
 			}
 		}
 
@@ -604,8 +604,8 @@ 
 		global $wpdb;
 
 		if (
-			isset( $data['values'] )
-			&& ! in_array( $this->args[ $column ], $data['values'], true )
+			isset( $data[ 'values' ] )
+			&& ! in_array( $this->args[ $column ], $data[ 'values' ], true )
 		) {
 			return;
 		}
@@ -623,8 +623,8 @@ 
 
 		$column = wordpoints_escape_mysql_identifier( $column );
 
-		$this->wheres[] = $wpdb->prepare( // WPCS: unprepared SQL OK.
-			"{$column} {$compare} {$data['format']}"
+		$this->wheres[ ] = $wpdb->prepare( // WPCS: unprepared SQL OK.
+			"{$column} {$compare} {$data[ 'format' ]}"
 			, $value
 		);
 	}
@@ -645,7 +645,7 @@ 
 
 		// MySQL doesn't support LIKE and NOT LIKE for int columns.
 		// See https://stackoverflow.com/q/8422455/1924128
-		if ( '%s' === $data['format'] ) {
+		if ( '%s' === $data[ 'format' ] ) {
 			$comparisons = array_merge( $comparisons, array( 'LIKE', 'NOT LIKE' ) );
 		}
 
@@ -680,8 +680,8 @@ 
 
 		$values = $this->args[ $key ];
 
-		if ( isset( $data['values'] ) ) {
-			$values = array_intersect( $values, $data['values'] );
+		if ( isset( $data[ 'values' ] ) ) {
+			$values = array_intersect( $values, $data[ 'values' ] );
 		} else {
 			$values = $this->validate_values(
 				$values
@@ -693,7 +693,7 @@ 
 			return;
 		}
 
-		$in = wordpoints_prepare__in( $values, $data['format'] );
+		$in = wordpoints_prepare__in( $values, $data[ 'format' ] );
 
 		if ( false === $in ) {
 			return;
@@ -701,7 +701,7 @@ 
 
 		$column = wordpoints_escape_mysql_identifier( $column );
 
-		$this->wheres[] = "{$column} {$type} ({$in})";
+		$this->wheres[ ] = "{$column} {$type} ({$in})";
 	}
 
 	/**
@@ -715,7 +715,7 @@ 
 
 		foreach ( $this->columns as $column => $data ) {
 
-			if ( ! empty( $data['is_date'] ) ) {
+			if ( ! empty( $data[ 'is_date' ] ) ) {
 				$this->prepare_date_where( $column );
 			} else {
 				$this->prepare_column_where( $column, $data );
@@ -738,7 +738,7 @@ 
 
 		// MySQL doesn't allow for the offset without a limit, so if no limit is set
 		// we can ignore the start arg. See https://stackoverflow.com/a/271650/1924128
-		if ( ! isset( $this->args['limit'] ) ) {
+		if ( ! isset( $this->args[ 'limit' ] ) ) {
 			return;
 		}
 
@@ -763,8 +763,8 @@ 
 			}
 		}
 
-		if ( $this->args['limit'] > 0 && $this->args['start'] >= 0 ) {
-			$this->limit = "LIMIT {$this->args['start']}, {$this->args['limit']}";
+		if ( $this->args[ 'limit' ] > 0 && $this->args[ 'start' ] >= 0 ) {
+			$this->limit = "LIMIT {$this->args[ 'start' ]}, {$this->args[ 'limit' ]}";
 		}
 	}
 
@@ -775,12 +775,12 @@ 
 	 */
 	protected function prepare_order_by() {
 
-		if ( empty( $this->args['order_by'] ) ) {
+		if ( empty( $this->args[ 'order_by' ] ) ) {
 			return;
 		}
 
-		$order    = $this->args['order'];
-		$order_by = $this->args['order_by'];
+		$order    = $this->args[ 'order' ];
+		$order_by = $this->args[ 'order_by' ];
 
 		if ( ! in_array( $order, array( 'DESC', 'ASC' ), true ) ) {
 
@@ -796,9 +796,9 @@ 
 				$wpdb->{"{$this->meta_type}meta"}
 			);
 
-			if ( isset( $this->args['meta_type'] ) ) {
+			if ( isset( $this->args[ 'meta_type' ] ) ) {
 
-				$meta_type = $this->meta_query->get_cast_for_type( $this->args['meta_type'] );
+				$meta_type = $this->meta_query->get_cast_for_type( $this->args[ 'meta_type' ] );
 				$order_by  = "CAST({$meta_table_name}.meta_value AS {$meta_type})";
 
 			} else {
@@ -841,7 +841,7 @@ 
 		$date_query = $date_query->get_sql();
 
 		if ( ! empty( $date_query ) ) {
-			$this->wheres[] = ltrim( $date_query, ' AND' );
+			$this->wheres[ ] = ltrim( $date_query, ' AND' );
 		}
 
 		remove_filter( 'date_query_valid_columns', array( $this, 'date_query_valid_columns_filter' ) );
@@ -883,11 +883,11 @@ 
 			, $this
 		);
 
-		if ( ! empty( $meta_query['where'] ) ) {
-			$this->wheres[] = ltrim( $meta_query['where'], ' AND' );
+		if ( ! empty( $meta_query[ 'where' ] ) ) {
+			$this->wheres[ ] = ltrim( $meta_query[ 'where' ], ' AND' );
 		}
 
-		$this->meta_join = $meta_query['join'] . "\n";
+		$this->meta_join = $meta_query[ 'join' ] . "\n";
 	}
 }
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -474,7 +474,7 @@ 
 
 		$fields = implode( ', ', array_map( 'wordpoints_escape_mysql_identifier', $fields ) );
 
-		$this->select = "SELECT {$fields}";
+		$this->select = "select {$fields}";
 	}
 
 	/**
@@ -764,7 +764,7 @@ 
 		}
 
 		if ( $this->args['limit'] > 0 && $this->args['start'] >= 0 ) {
-			$this->limit = "LIMIT {$this->args['start']}, {$this->args['limit']}";
+			$this->limit = "limit {$this->args['start']}, {$this->args['limit']}";
 		}
 	}
 

src/classes/entity.php

Spacing +3 added lines, -3 removed lines

@@ -385,10 +385,10 @@
 		} elseif ( $this->is_guid( $value ) ) {
 
 			$guid = $this->split_guid( $value );
-			$context = $guid['context'];
-			$value   = $guid['id'];
+			$context = $guid[ 'context' ];
+			$value   = $guid[ 'id' ];
 
-			$entity = $this->get_entity_from_context( $guid['id'], $context );
+			$entity = $this->get_entity_from_context( $guid[ 'id' ], $context );
 
 		} else {
 

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/reactor.php

Spacing +4 added lines, -4 removed lines

@@ -111,15 +111,15 @@ 
 	) {
 
 		if (
-			empty( $settings['target'] )
-			|| ! is_array( $settings['target'] )
+			empty( $settings[ 'target' ] )
+			|| ! is_array( $settings[ 'target' ] )
 		) {
 
 			$validator->add_error( __( 'Invalid target.', 'wordpoints' ), 'target' );
 
 		} else {
 
-			$target = $event_args->get_from_hierarchy( $settings['target'] );
+			$target = $event_args->get_from_hierarchy( $settings[ 'target' ] );
 
 			if (
 				! $target instanceof WordPoints_Entity
@@ -136,7 +136,7 @@ 
 	 * @since 2.1.0
 	 */
 	public function update_settings( WordPoints_Hook_ReactionI $reaction, array $settings ) {
-		$reaction->update_meta( 'target', $settings['target'] );
+		$reaction->update_meta( 'target', $settings[ 'target' ] );
 	}
 }