woocommerce / woocommerce-rest-api

src/Controllers/Version4/Utilities/SettingsTrait.php

Spacing +11 added lines, -11 removed lines

@@ -33,7 +33,7 @@ 
 	 * @return string|\WP_Error
 	 */
 	public function validate_setting_select_field( $value, $setting ) {
-		if ( array_key_exists( $value, $setting['options'] ) ) {
+		if ( array_key_exists( $value, $setting[ 'options' ] ) ) {
 			return $value;
 		} else {
 			return new \WP_Error( 'rest_setting_value_invalid', __( 'An invalid setting value was passed.', 'woocommerce' ), array( 'status' => 400 ) );
@@ -59,8 +59,8 @@ 
 
 		$final_values = array();
 		foreach ( $values as $value ) {
-			if ( array_key_exists( $value, $setting['options'] ) ) {
-				$final_values[] = $value;
+			if ( array_key_exists( $value, $setting[ 'options' ] ) ) {
+				$final_values[ ] = $value;
 			}
 		}
 
@@ -80,15 +80,15 @@ 
 			return new \WP_Error( 'rest_setting_value_invalid', __( 'An invalid setting value was passed.', 'woocommerce' ), array( 'status' => 400 ) );
 		}
 
-		$current = $setting['value'];
-		if ( isset( $values['width'] ) ) {
-			$current['width'] = intval( $values['width'] );
+		$current = $setting[ 'value' ];
+		if ( isset( $values[ 'width' ] ) ) {
+			$current[ 'width' ] = intval( $values[ 'width' ] );
 		}
-		if ( isset( $values['height'] ) ) {
-			$current['height'] = intval( $values['height'] );
+		if ( isset( $values[ 'height' ] ) ) {
+			$current[ 'height' ] = intval( $values[ 'height' ] );
 		}
-		if ( isset( $values['crop'] ) ) {
-			$current['crop'] = (bool) $values['crop'];
+		if ( isset( $values[ 'crop' ] ) ) {
+			$current[ 'crop' ] = (bool) $values[ 'crop' ];
 		}
 		return $current;
 	}
@@ -117,7 +117,7 @@ 
 		if ( in_array( $value, array( 'yes', 'no' ), true ) ) {
 			return $value;
 		} elseif ( empty( $value ) ) {
-			$value = isset( $setting['default'] ) ? $setting['default'] : 'no';
+			$value = isset( $setting[ 'default' ] ) ? $setting[ 'default' ] : 'no';
 			return $value;
 		} else {
 			return new \WP_Error( 'rest_setting_value_invalid', __( 'An invalid setting value was passed.', 'woocommerce' ), array( 'status' => 400 ) );

src/Controllers/Version4/Utilities/BatchTrait.php

Spacing +11 added lines, -11 removed lines

@@ -26,7 +26,7 @@ 
 		}
 
 		$batches  = [ 'create', 'update', 'delete' ];
-		$response = [];
+		$response = [ ];
 
 		foreach ( $batches as $batch ) {
 			$response[ $batch ] = $this->{"batch_$batch"}( $this->get_batch_of_items_from_request( $request, $batch ) );
@@ -164,11 +164,11 @@ 
 	 * @return array
 	 */
 	protected function get_default_params() {
-		$defaults = [];
+		$defaults = [ ];
 		$schema   = $this->get_public_item_schema();
-		foreach ( $schema['properties'] as $arg => $options ) {
-			if ( isset( $options['default'] ) ) {
-				$defaults[ $arg ] = $options['default'];
+		foreach ( $schema[ 'properties' ] as $arg => $options ) {
+			if ( isset( $options[ 'default' ] ) ) {
+				$defaults[ $arg ] = $options[ 'default' ];
 			}
 		}
 		return $defaults;
@@ -181,7 +181,7 @@ 
 	 * @return array Response data.
 	 */
 	protected function batch_create( $items ) {
-		$batch_response = [];
+		$batch_response = [ ];
 
 		foreach ( $items as $item ) {
 			$request = new \WP_REST_Request( 'POST' );
@@ -189,7 +189,7 @@ 
 			$request->set_body_params( $item );
 
 			$response         = $this->create_item( $request );
-			$batch_response[] = $this->format_response( 0, $response );
+			$batch_response[ ] = $this->format_response( 0, $response );
 		}
 
 		return $batch_response;
@@ -202,14 +202,14 @@ 
 	 * @return array Response data.
 	 */
 	protected function batch_update( $items ) {
-		$batch_response = [];
+		$batch_response = [ ];
 
 		foreach ( $items as $item ) {
 			$request = new \WP_REST_Request( 'PUT' );
 			$request->set_body_params( $item );
 
 			$response         = $this->update_item( $request );
-			$batch_response[] = $this->format_response( $item['id'], $response );
+			$batch_response[ ] = $this->format_response( $item[ 'id' ], $response );
 		}
 
 		return $batch_response;
@@ -222,7 +222,7 @@ 
 	 * @return array Response data.
 	 */
 	protected function batch_delete( $items ) {
-		$batch_response = [];
+		$batch_response = [ ];
 		$items          = wp_parse_id_list( $items );
 
 		foreach ( $items as $id ) {
@@ -234,7 +234,7 @@ 
 				]
 			);
 			$response         = $this->delete_item( $request );
-			$batch_response[] = $this->format_response( $id, $response );
+			$batch_response[ ] = $this->format_response( $id, $response );
 		}
 
 		return $batch_response;

src/Controllers/Version4/ShippingZoneMethods.php

Spacing +33 added lines, -33 removed lines

@@ -108,13 +108,13 @@ 
 	 * @return \WP_REST_Response|\WP_Error
 	 */
 	public function get_item( $request ) {
-		$zone = $this->get_zone( $request['zone_id'] );
+		$zone = $this->get_zone( $request[ 'zone_id' ] );
 
 		if ( is_wp_error( $zone ) ) {
 			return $zone;
 		}
 
-		$instance_id = (int) $request['instance_id'];
+		$instance_id = (int) $request[ 'instance_id' ];
 		$methods     = $zone->get_shipping_methods();
 		$method      = false;
 
@@ -141,7 +141,7 @@ 
 	 * @return \WP_REST_Response|\WP_Error
 	 */
 	public function get_items( $request ) {
-		$zone = $this->get_zone( $request['zone_id'] );
+		$zone = $this->get_zone( $request[ 'zone_id' ] );
 
 		if ( is_wp_error( $zone ) ) {
 			return $zone;
@@ -152,7 +152,7 @@ 
 
 		foreach ( $methods as $method_obj ) {
 			$method = $this->prepare_item_for_response( $method_obj, $request );
-			$data[] = $method;
+			$data[ ] = $method;
 		}
 
 		return rest_ensure_response( $data );
@@ -165,8 +165,8 @@ 
 	 * @return \WP_REST_Request|\WP_Error
 	 */
 	public function create_item( $request ) {
-		$method_id = $request['method_id'];
-		$zone      = $this->get_zone( $request['zone_id'] );
+		$method_id = $request[ 'method_id' ];
+		$zone      = $this->get_zone( $request[ 'zone_id' ] );
 		if ( is_wp_error( $zone ) ) {
 			return $zone;
 		}
@@ -201,13 +201,13 @@ 
 	 * @return \WP_Error|boolean
 	 */
 	public function delete_item( $request ) {
-		$zone = $this->get_zone( $request['zone_id'] );
+		$zone = $this->get_zone( $request[ 'zone_id' ] );
 		if ( is_wp_error( $zone ) ) {
 			return $zone;
 		}
 
-		$instance_id = (int) $request['instance_id'];
-		$force       = $request['force'];
+		$instance_id = (int) $request[ 'instance_id' ];
+		$force       = $request[ 'force' ];
 
 		// We don't support trashing for this type, error out.
 		if ( ! $force ) {
@@ -265,12 +265,12 @@ 
 	 * @return \WP_REST_Response|\WP_Error
 	 */
 	public function update_item( $request ) {
-		$zone = $this->get_zone( $request['zone_id'] );
+		$zone = $this->get_zone( $request[ 'zone_id' ] );
 		if ( is_wp_error( $zone ) ) {
 			return $zone;
 		}
 
-		$instance_id = (int) $request['instance_id'];
+		$instance_id = (int) $request[ 'instance_id' ];
 		$methods     = $zone->get_shipping_methods();
 		$method      = false;
 
@@ -307,16 +307,16 @@ 
 		global $wpdb;
 
 		// Update settings if present.
-		if ( isset( $request['settings'] ) ) {
+		if ( isset( $request[ 'settings' ] ) ) {
 			$method->init_instance_settings();
 			$instance_settings = $method->instance_settings;
 			$errors_found      = false;
 			foreach ( $method->get_instance_form_fields() as $key => $field ) {
-				if ( isset( $request['settings'][ $key ] ) ) {
-					if ( is_callable( array( $this, 'validate_setting_' . $field['type'] . '_field' ) ) ) {
-						$value = $this->{'validate_setting_' . $field['type'] . '_field'}( $request['settings'][ $key ], $field );
+				if ( isset( $request[ 'settings' ][ $key ] ) ) {
+					if ( is_callable( array( $this, 'validate_setting_' . $field[ 'type' ] . '_field' ) ) ) {
+						$value = $this->{'validate_setting_' . $field[ 'type' ] . '_field'}( $request[ 'settings' ][ $key ], $field );
 					} else {
-						$value = $this->validate_setting_text_field( $request['settings'][ $key ], $field );
+						$value = $this->validate_setting_text_field( $request[ 'settings' ][ $key ], $field );
 					}
 					if ( is_wp_error( $value ) ) {
 						$errors_found = true;
@@ -334,16 +334,16 @@ 
 		}
 
 		// Update order.
-		if ( isset( $request['order'] ) ) {
-			$wpdb->update( "{$wpdb->prefix}woocommerce_shipping_zone_methods", array( 'method_order' => absint( $request['order'] ) ), array( 'instance_id' => absint( $instance_id ) ) );
-			$method->method_order = absint( $request['order'] );
+		if ( isset( $request[ 'order' ] ) ) {
+			$wpdb->update( "{$wpdb->prefix}woocommerce_shipping_zone_methods", array( 'method_order' => absint( $request[ 'order' ] ) ), array( 'instance_id' => absint( $instance_id ) ) );
+			$method->method_order = absint( $request[ 'order' ] );
 		}
 
 		// Update if this method is enabled or not.
-		if ( isset( $request['enabled'] ) ) {
-			if ( $wpdb->update( "{$wpdb->prefix}woocommerce_shipping_zone_methods", array( 'is_enabled' => $request['enabled'] ), array( 'instance_id' => absint( $instance_id ) ) ) ) {
-				do_action( 'woocommerce_shipping_zone_method_status_toggled', $instance_id, $method->id, $request['zone_id'], $request['enabled'] );
-				$method->enabled = ( true === $request['enabled'] ? 'yes' : 'no' );
+		if ( isset( $request[ 'enabled' ] ) ) {
+			if ( $wpdb->update( "{$wpdb->prefix}woocommerce_shipping_zone_methods", array( 'is_enabled' => $request[ 'enabled' ] ), array( 'instance_id' => absint( $instance_id ) ) ) ) {
+				do_action( 'woocommerce_shipping_zone_method_status_toggled', $instance_id, $method->id, $request[ 'zone_id' ], $request[ 'enabled' ] );
+				$method->enabled = ( true === $request[ 'enabled' ] ? 'yes' : 'no' );
 			}
 		}
 
@@ -361,7 +361,7 @@ 
 		return array(
 			'id'                 => $object->instance_id,
 			'instance_id'        => $object->instance_id,
-			'title'              => $object->instance_settings['title'],
+			'title'              => $object->instance_settings[ 'title' ],
 			'order'              => $object->method_order,
 			'enabled'            => ( 'yes' === $object->enabled ),
 			'method_id'          => $object->id,
@@ -397,16 +397,16 @@ 
 		foreach ( $item->get_instance_form_fields() as $id => $field ) {
 			$data = array(
 				'id'          => $id,
-				'label'       => $field['title'],
-				'description' => empty( $field['description'] ) ? '' : $field['description'],
-				'type'        => $field['type'],
+				'label'       => $field[ 'title' ],
+				'description' => empty( $field[ 'description' ] ) ? '' : $field[ 'description' ],
+				'type'        => $field[ 'type' ],
 				'value'       => $item->instance_settings[ $id ],
-				'default'     => empty( $field['default'] ) ? '' : $field['default'],
-				'tip'         => empty( $field['description'] ) ? '' : $field['description'],
-				'placeholder' => empty( $field['placeholder'] ) ? '' : $field['placeholder'],
+				'default'     => empty( $field[ 'default' ] ) ? '' : $field[ 'default' ],
+				'tip'         => empty( $field[ 'description' ] ) ? '' : $field[ 'description' ],
+				'placeholder' => empty( $field[ 'placeholder' ] ) ? '' : $field[ 'placeholder' ],
 			);
-			if ( ! empty( $field['options'] ) ) {
-				$data['options'] = $field['options'];
+			if ( ! empty( $field[ 'options' ] ) ) {
+				$data[ 'options' ] = $field[ 'options' ];
 			}
 			$settings[ $id ] = $data;
 		}
@@ -421,7 +421,7 @@ 
 	 * @return array
 	 */
 	protected function prepare_links( $item, $request ) {
-		$base  = '/' . $this->namespace . '/' . $this->rest_base . '/' . $request['zone_id'];
+		$base  = '/' . $this->namespace . '/' . $this->rest_base . '/' . $request[ 'zone_id' ];
 		$links = array(
 			'self'       => array(
 				'href' => rest_url( $base . '/methods/' . $item->instance_id ),

src/Controllers/Version4/ProductAttributeTerms.php

Spacing +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
 	protected function update_term_meta_fields( $term, $request ) {
 		$id = (int) $term->term_id;
 
-		update_term_meta( $id, 'order_' . $this->taxonomy, $request['menu_order'] );
+		update_term_meta( $id, 'order_' . $this->taxonomy, $request[ 'menu_order' ] );
 
 		return true;
 	}

src/Controllers/Version4/Taxes.php

Spacing +40 added lines, -40 removed lines

@@ -111,21 +111,21 @@ 
 		global $wpdb;
 
 		$prepared_args           = array();
-		$prepared_args['order']  = $request['order'];
-		$prepared_args['number'] = $request['per_page'];
-		if ( ! empty( $request['offset'] ) ) {
-			$prepared_args['offset'] = $request['offset'];
+		$prepared_args[ 'order' ]  = $request[ 'order' ];
+		$prepared_args[ 'number' ] = $request[ 'per_page' ];
+		if ( ! empty( $request[ 'offset' ] ) ) {
+			$prepared_args[ 'offset' ] = $request[ 'offset' ];
 		} else {
-			$prepared_args['offset'] = ( $request['page'] - 1 ) * $prepared_args['number'];
+			$prepared_args[ 'offset' ] = ( $request[ 'page' ] - 1 ) * $prepared_args[ 'number' ];
 		}
 		$orderby_possibles        = array(
 			'id'    => 'tax_rate_id',
 			'order' => 'tax_rate_order',
 		);
-		$prepared_args['orderby'] = $orderby_possibles[ $request['orderby'] ];
-		$prepared_args['class']   = $request['class'];
-		$prepared_args['code']    = $request['code'];
-		$prepared_args['include'] = $request['include'];
+		$prepared_args[ 'orderby' ] = $orderby_possibles[ $request[ 'orderby' ] ];
+		$prepared_args[ 'class' ]   = $request[ 'class' ];
+		$prepared_args[ 'code' ]    = $request[ 'code' ];
+		$prepared_args[ 'include' ] = $request[ 'include' ];
 
 		/**
 		 * Filter arguments, before passing to $wpdb->get_results(), when querying taxes via the REST API.
@@ -142,13 +142,13 @@ 
 		";
 
 		// Filter by tax class.
-		if ( ! empty( $prepared_args['class'] ) ) {
-			$class  = 'standard' !== $prepared_args['class'] ? sanitize_title( $prepared_args['class'] ) : '';
+		if ( ! empty( $prepared_args[ 'class' ] ) ) {
+			$class  = 'standard' !== $prepared_args[ 'class' ] ? sanitize_title( $prepared_args[ 'class' ] ) : '';
 			$query .= " AND tax_rate_class = '$class'";
 		}
 
 		// Filter by tax code.
-		$tax_code_search = $prepared_args['code'];
+		$tax_code_search = $prepared_args[ 'code' ];
 		if ( $tax_code_search ) {
 			$tax_code_search = $wpdb->esc_like( $tax_code_search );
 			$tax_code_search = ' \'%' . $tax_code_search . '%\'';
@@ -156,17 +156,17 @@ 
 		}
 
 		// Filter by included tax rate IDs.
-		$included_taxes = $prepared_args['include'];
+		$included_taxes = $prepared_args[ 'include' ];
 		if ( ! empty( $included_taxes ) ) {
-			$included_taxes = implode( ',', $prepared_args['include'] );
+			$included_taxes = implode( ',', $prepared_args[ 'include' ] );
 			$query         .= " AND tax_rate_id IN ({$included_taxes})";
 		}
 
 		// Order tax rates.
-		$order_by = sprintf( ' ORDER BY %s', sanitize_key( $prepared_args['orderby'] ) );
+		$order_by = sprintf( ' ORDER BY %s', sanitize_key( $prepared_args[ 'orderby' ] ) );
 
 		// Pagination.
-		$pagination = sprintf( ' LIMIT %d, %d', $prepared_args['offset'], $prepared_args['number'] );
+		$pagination = sprintf( ' LIMIT %d, %d', $prepared_args[ 'offset' ], $prepared_args[ 'number' ] );
 
 		// Query taxes.
 		$results = $wpdb->get_results( $query . $order_by . $pagination ); // @codingStandardsIgnoreLine.
@@ -174,12 +174,12 @@ 
 		$taxes = array();
 		foreach ( $results as $tax ) {
 			$data    = $this->prepare_item_for_response( $tax, $request );
-			$taxes[] = $this->prepare_response_for_collection( $data );
+			$taxes[ ] = $this->prepare_response_for_collection( $data );
 		}
 
 		// Store pagination values for headers then unset for count query.
-		$per_page = (int) $prepared_args['number'];
-		$page     = ceil( ( ( (int) $prepared_args['offset'] ) / $per_page ) + 1 );
+		$per_page = (int) $prepared_args[ 'number' ];
+		$page     = ceil( ( ( (int) $prepared_args[ 'offset' ] ) / $per_page ) + 1 );
 
 		// Query only for ids.
 		$wpdb->get_results( str_replace( 'SELECT *', 'SELECT tax_rate_id', $query ) ); // @codingStandardsIgnoreLine.
@@ -202,7 +202,7 @@ 
 	 * @return object
 	 */
 	protected function create_or_update_tax( $request, $current = null ) {
-		$id          = absint( isset( $request['id'] ) ? $request['id'] : 0 );
+		$id          = absint( isset( $request[ 'id' ] ) ? $request[ 'id' ] : 0 );
 		$data        = array();
 		$fields      = array(
 			'tax_rate_country',
@@ -239,7 +239,7 @@ 
 					$data[ $field ] = absint( $request[ $key ] );
 					break;
 				case 'tax_rate_class':
-					$data[ $field ] = 'standard' !== $request['tax_rate_class'] ? $request['tax_rate_class'] : '';
+					$data[ $field ] = 'standard' !== $request[ 'tax_rate_class' ] ? $request[ 'tax_rate_class' ] : '';
 					break;
 				default:
 					$data[ $field ] = wc_clean( $request[ $key ] );
@@ -254,11 +254,11 @@ 
 		}
 
 		// Add locales.
-		if ( ! empty( $request['postcode'] ) ) {
-			\WC_Tax::_update_tax_rate_postcodes( $id, wc_clean( $request['postcode'] ) );
+		if ( ! empty( $request[ 'postcode' ] ) ) {
+			\WC_Tax::_update_tax_rate_postcodes( $id, wc_clean( $request[ 'postcode' ] ) );
 		}
-		if ( ! empty( $request['city'] ) ) {
-			\WC_Tax::_update_tax_rate_cities( $id, wc_clean( $request['city'] ) );
+		if ( ! empty( $request[ 'city' ] ) ) {
+			\WC_Tax::_update_tax_rate_cities( $id, wc_clean( $request[ 'city' ] ) );
 		}
 
 		return \WC_Tax::_get_tax_rate( $id, OBJECT );
@@ -271,7 +271,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function create_item( $request ) {
-		if ( ! empty( $request['id'] ) ) {
+		if ( ! empty( $request[ 'id' ] ) ) {
 			return new \WP_Error( 'woocommerce_rest_tax_exists', __( 'Cannot create existing resource.', 'woocommerce' ), array( 'status' => 400 ) );
 		}
 
@@ -304,7 +304,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$id      = (int) $request['id'];
+		$id      = (int) $request[ 'id' ];
 		$tax_obj = \WC_Tax::_get_tax_rate( $id, OBJECT );
 
 		if ( empty( $id ) || empty( $tax_obj ) ) {
@@ -324,7 +324,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function update_item( $request ) {
-		$id      = (int) $request['id'];
+		$id      = (int) $request[ 'id' ];
 		$tax_obj = \WC_Tax::_get_tax_rate( $id, OBJECT );
 
 		if ( empty( $id ) || empty( $tax_obj ) ) {
@@ -360,8 +360,8 @@ 
 	public function delete_item( $request ) {
 		global $wpdb;
 
-		$id    = (int) $request['id'];
-		$force = isset( $request['force'] ) ? (bool) $request['force'] : false;
+		$id    = (int) $request[ 'id' ];
+		$force = isset( $request[ 'force' ] ) ? (bool) $request[ 'force' ] : false;
 
 		// We don't support trashing for this type, error out.
 		if ( ! $force ) {
@@ -551,10 +551,10 @@ 
 	 */
 	public function get_collection_params() {
 		$params                       = array();
-		$params['context']            = $this->get_context_param();
-		$params['context']['default'] = 'view';
+		$params[ 'context' ]            = $this->get_context_param();
+		$params[ 'context' ][ 'default' ] = 'view';
 
-		$params['page'] = array(
+		$params[ 'page' ] = array(
 			'description'        => __( 'Current page of the collection.', 'woocommerce' ),
 			'type'               => 'integer',
 			'default'            => 1,
@@ -562,7 +562,7 @@ 
 			'validate_callback'  => 'rest_validate_request_arg',
 			'minimum'            => 1,
 		);
-		$params['per_page'] = array(
+		$params[ 'per_page' ] = array(
 			'description'        => __( 'Maximum number of items to be returned in result set.', 'woocommerce' ),
 			'type'               => 'integer',
 			'default'            => 10,
@@ -571,13 +571,13 @@ 
 			'sanitize_callback'  => 'absint',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['offset'] = array(
+		$params[ 'offset' ] = array(
 			'description'        => __( 'Offset the result set by a specific number of items.', 'woocommerce' ),
 			'type'               => 'integer',
 			'sanitize_callback'  => 'absint',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['order'] = array(
+		$params[ 'order' ] = array(
 			'default'            => 'asc',
 			'description'        => __( 'Order sort attribute ascending or descending.', 'woocommerce' ),
 			'enum'               => array( 'asc', 'desc' ),
@@ -585,7 +585,7 @@ 
 			'type'               => 'string',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['orderby'] = array(
+		$params[ 'orderby' ] = array(
 			'default'            => 'order',
 			'description'        => __( 'Sort collection by object attribute.', 'woocommerce' ),
 			'enum'               => array(
@@ -596,19 +596,19 @@ 
 			'type'               => 'string',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['class'] = array(
+		$params[ 'class' ] = array(
 			'description'        => __( 'Sort by tax class.', 'woocommerce' ),
 			'enum'               => array_merge( array( 'standard' ), \WC_Tax::get_tax_class_slugs() ),
 			'sanitize_callback'  => 'sanitize_title',
 			'type'               => 'string',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['code']    = array(
+		$params[ 'code' ] = array(
 			'description'       => __( 'Search by similar tax code.', 'woocommerce' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to items that have the specified rate ID(s) assigned.', 'woocommerce' ),
 			'type'              => 'array',
 			'items'             => array(

src/Controllers/Version4/SettingsOptions.php

Spacing +55 added lines, -55 removed lines

@@ -119,7 +119,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$setting = $this->get_setting( $request['group_id'], $request['id'] );
+		$setting = $this->get_setting( $request[ 'group_id' ], $request[ 'id' ] );
 
 		if ( is_wp_error( $setting ) ) {
 			return $setting;
@@ -138,7 +138,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_items( $request ) {
-		$settings = $this->get_group_settings( $request['group_id'] );
+		$settings = $this->get_group_settings( $request[ 'group_id' ] );
 
 		if ( is_wp_error( $settings ) ) {
 			return $settings;
@@ -149,8 +149,8 @@ 
 		foreach ( $settings as $setting_obj ) {
 			$setting = $this->prepare_item_for_response( $setting_obj, $request );
 			$setting = $this->prepare_response_for_collection( $setting );
-			if ( $this->is_setting_type_valid( $setting['type'] ) ) {
-				$data[] = $setting;
+			if ( $this->is_setting_type_valid( $setting[ 'type' ] ) ) {
+				$data[ ] = $setting;
 			}
 		}
 
@@ -176,26 +176,26 @@ 
 
 		$filtered_settings = array();
 		foreach ( $settings as $setting ) {
-			$option_key = $setting['option_key'];
+			$option_key = $setting[ 'option_key' ];
 			$setting    = $this->filter_setting( $setting );
-			$default    = isset( $setting['default'] ) ? $setting['default'] : '';
+			$default    = isset( $setting[ 'default' ] ) ? $setting[ 'default' ] : '';
 			// Get the option value.
 			if ( is_array( $option_key ) ) {
-				$option           = get_option( $option_key[0] );
-				$setting['value'] = isset( $option[ $option_key[1] ] ) ? $option[ $option_key[1] ] : $default;
+				$option           = get_option( $option_key[ 0 ] );
+				$setting[ 'value' ] = isset( $option[ $option_key[ 1 ] ] ) ? $option[ $option_key[ 1 ] ] : $default;
 			} else {
 				$admin_setting_value = \WC_Admin_Settings::get_option( $option_key, $default );
-				$setting['value']    = $admin_setting_value;
+				$setting[ 'value' ]    = $admin_setting_value;
 			}
 
-			if ( 'multi_select_countries' === $setting['type'] ) {
-				$setting['options'] = WC()->countries->get_countries();
-				$setting['type']    = 'multiselect';
-			} elseif ( 'single_select_country' === $setting['type'] ) {
-				$setting['type']    = 'select';
-				$setting['options'] = $this->get_countries_and_states();
-			} elseif ( 'single_select_page' === $setting['type'] ) {
-				$pages   = get_pages(
+			if ( 'multi_select_countries' === $setting[ 'type' ] ) {
+				$setting[ 'options' ] = WC()->countries->get_countries();
+				$setting[ 'type' ]    = 'multiselect';
+			} elseif ( 'single_select_country' === $setting[ 'type' ] ) {
+				$setting[ 'type' ]    = 'select';
+				$setting[ 'options' ] = $this->get_countries_and_states();
+			} elseif ( 'single_select_page' === $setting[ 'type' ] ) {
+				$pages = get_pages(
 					array(
 						'sort_column'  => 'menu_order',
 						'sort_order'   => 'ASC',
@@ -206,11 +206,11 @@ 
 				foreach ( $pages as $page ) {
 					$options[ $page->ID ] = ! empty( $page->post_title ) ? $page->post_title : '#' . $page->ID;
 				}
-				$setting['type']    = 'select';
-				$setting['options'] = $options;
+				$setting[ 'type' ]    = 'select';
+				$setting[ 'options' ] = $options;
 			}
 
-			$filtered_settings[] = $setting;
+			$filtered_settings[ ] = $setting;
 		}
 
 		return $filtered_settings;
@@ -267,9 +267,9 @@ 
 			return new \WP_Error( 'rest_setting_setting_invalid', __( 'Invalid setting.', 'woocommerce' ), array( 'status' => 404 ) );
 		}
 
-		$setting = $settings[ $array_key[0] ];
+		$setting = $settings[ $array_key[ 0 ] ];
 
-		if ( ! $this->is_setting_type_valid( $setting['type'] ) ) {
+		if ( ! $this->is_setting_type_valid( $setting[ 'type' ] ) ) {
 			return new \WP_Error( 'rest_setting_setting_invalid', __( 'Invalid setting.', 'woocommerce' ), array( 'status' => 404 ) );
 		}
 
@@ -277,7 +277,7 @@ 
 			return $setting;
 		}
 
-		$setting['group_id'] = $group_id;
+		$setting[ 'group_id' ] = $group_id;
 
 		return $setting;
 	}
@@ -319,32 +319,32 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function update_item( $request ) {
-		$setting = $this->get_setting( $request['group_id'], $request['id'] );
+		$setting = $this->get_setting( $request[ 'group_id' ], $request[ 'id' ] );
 
 		if ( is_wp_error( $setting ) ) {
 			return $setting;
 		}
 
-		if ( is_callable( array( $this, 'validate_setting_' . $setting['type'] . '_field' ) ) ) {
-			$value = $this->{'validate_setting_' . $setting['type'] . '_field'}( $request['value'], $setting );
+		if ( is_callable( array( $this, 'validate_setting_' . $setting[ 'type' ] . '_field' ) ) ) {
+			$value = $this->{'validate_setting_' . $setting[ 'type' ] . '_field'}( $request[ 'value' ], $setting );
 		} else {
-			$value = $this->validate_setting_text_field( $request['value'], $setting );
+			$value = $this->validate_setting_text_field( $request[ 'value' ], $setting );
 		}
 
 		if ( is_wp_error( $value ) ) {
 			return $value;
 		}
 
-		if ( is_array( $setting['option_key'] ) ) {
-			$setting['value']       = $value;
-			$option_key             = $setting['option_key'];
-			$prev                   = get_option( $option_key[0] );
-			$prev[ $option_key[1] ] = $request['value'];
-			update_option( $option_key[0], $prev );
+		if ( is_array( $setting[ 'option_key' ] ) ) {
+			$setting[ 'value' ]       = $value;
+			$option_key             = $setting[ 'option_key' ];
+			$prev                   = get_option( $option_key[ 0 ] );
+			$prev[ $option_key[ 1 ] ] = $request[ 'value' ];
+			update_option( $option_key[ 0 ], $prev );
 		} else {
 			$update_data                           = array();
-			$update_data[ $setting['option_key'] ] = $value;
-			$setting['value']                      = $value;
+			$update_data[ $setting[ 'option_key' ] ] = $value;
+			$setting[ 'value' ]                      = $value;
 			\WC_Admin_Settings::save_fields( array( $setting ), $update_data );
 		}
 
@@ -361,7 +361,7 @@ 
 	 * @return array Array of data in the correct format.
 	 */
 	protected function get_data_for_response( $object, $request ) {
-		unset( $object['option_key'] );
+		unset( $object[ 'option_key' ] );
 		return $this->filter_setting( $object );
 	}
 
@@ -373,10 +373,10 @@ 
 	 * @return array
 	 */
 	protected function prepare_links( $item, $request ) {
-		$base  = str_replace( '(?P<group_id>[\w-]+)', $request['group_id'], $this->rest_base );
+		$base  = str_replace( '(?P<group_id>[\w-]+)', $request[ 'group_id' ], $this->rest_base );
 		$links = array(
 			'self'       => array(
-				'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $base, $item['id'] ) ),
+				'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $base, $item[ 'id' ] ) ),
 			),
 			'collection' => array(
 				'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $base ) ),
@@ -400,11 +400,11 @@ 
 			array_flip( array_filter( array_keys( $setting ), array( $this, 'allowed_setting_keys' ) ) )
 		);
 
-		if ( empty( $setting['options'] ) ) {
-			unset( $setting['options'] );
+		if ( empty( $setting[ 'options' ] ) ) {
+			unset( $setting[ 'options' ] );
 		}
 
-		if ( 'image_width' === $setting['type'] ) {
+		if ( 'image_width' === $setting[ 'type' ] ) {
 			$setting = $this->cast_image_width( $setting );
 		}
 
@@ -423,9 +423,9 @@ 
 	public function cast_image_width( $setting ) {
 		foreach ( array( 'default', 'value' ) as $key ) {
 			if ( isset( $setting[ $key ] ) ) {
-				$setting[ $key ]['width']  = intval( $setting[ $key ]['width'] );
-				$setting[ $key ]['height'] = intval( $setting[ $key ]['height'] );
-				$setting[ $key ]['crop']   = (bool) $setting[ $key ]['crop'];
+				$setting[ $key ][ 'width' ]  = intval( $setting[ $key ][ 'width' ] );
+				$setting[ $key ][ 'height' ] = intval( $setting[ $key ][ 'height' ] );
+				$setting[ $key ][ 'crop' ]   = (bool) $setting[ $key ][ 'crop' ];
 			}
 		}
 		return $setting;
@@ -465,17 +465,17 @@ 
 	public function is_setting_type_valid( $type ) {
 		return in_array(
 			$type, array(
-				'text',         // Validates with validate_setting_text_field.
-				'email',        // Validates with validate_setting_text_field.
-				'number',       // Validates with validate_setting_text_field.
-				'color',        // Validates with validate_setting_text_field.
-				'password',     // Validates with validate_setting_text_field.
-				'textarea',     // Validates with validate_setting_textarea_field.
-				'select',       // Validates with validate_setting_select_field.
-				'multiselect',  // Validates with validate_setting_multiselect_field.
-				'radio',        // Validates with validate_setting_radio_field (-> validate_setting_select_field).
-				'checkbox',     // Validates with validate_setting_checkbox_field.
-				'image_width',  // Validates with validate_setting_image_width_field.
+				'text', // Validates with validate_setting_text_field.
+				'email', // Validates with validate_setting_text_field.
+				'number', // Validates with validate_setting_text_field.
+				'color', // Validates with validate_setting_text_field.
+				'password', // Validates with validate_setting_text_field.
+				'textarea', // Validates with validate_setting_textarea_field.
+				'select', // Validates with validate_setting_select_field.
+				'multiselect', // Validates with validate_setting_multiselect_field.
+				'radio', // Validates with validate_setting_radio_field (-> validate_setting_select_field).
+				'checkbox', // Validates with validate_setting_checkbox_field.
+				'image_width', // Validates with validate_setting_image_width_field.
 				'thumbnail_cropping', // Validates with validate_setting_text_field.
 			)
 		);

src/Controllers/Version4/PaymentGateways.php

Spacing +28 added lines, -28 removed lines

@@ -94,7 +94,7 @@ 
 			$payment_gateway->id = $payment_gateway_id;
 			$gateway             = $this->prepare_item_for_response( $payment_gateway, $request );
 			$gateway             = $this->prepare_response_for_collection( $gateway );
-			$response[]          = $gateway;
+			$response[ ]          = $gateway;
 		}
 		return rest_ensure_response( $response );
 	}
@@ -134,14 +134,14 @@ 
 		$settings = $gateway->settings;
 
 		// Update settings.
-		if ( isset( $request['settings'] ) ) {
+		if ( isset( $request[ 'settings' ] ) ) {
 			$errors_found = false;
 			foreach ( $gateway->form_fields as $key => $field ) {
-				if ( isset( $request['settings'][ $key ] ) ) {
-					if ( is_callable( array( $this, 'validate_setting_' . $field['type'] . '_field' ) ) ) {
-						$value = $this->{'validate_setting_' . $field['type'] . '_field'}( $request['settings'][ $key ], $field );
+				if ( isset( $request[ 'settings' ][ $key ] ) ) {
+					if ( is_callable( array( $this, 'validate_setting_' . $field[ 'type' ] . '_field' ) ) ) {
+						$value = $this->{'validate_setting_' . $field[ 'type' ] . '_field'}( $request[ 'settings' ][ $key ], $field );
 					} else {
-						$value = $this->validate_setting_text_field( $request['settings'][ $key ], $field );
+						$value = $this->validate_setting_text_field( $request[ 'settings' ][ $key ], $field );
 					}
 					if ( is_wp_error( $value ) ) {
 						$errors_found = true;
@@ -157,21 +157,21 @@ 
 		}
 
 		// Update if this method is enabled or not.
-		if ( isset( $request['enabled'] ) ) {
-			$settings['enabled'] = wc_bool_to_string( $request['enabled'] );
-			$gateway->enabled    = $settings['enabled'];
+		if ( isset( $request[ 'enabled' ] ) ) {
+			$settings[ 'enabled' ] = wc_bool_to_string( $request[ 'enabled' ] );
+			$gateway->enabled    = $settings[ 'enabled' ];
 		}
 
 		// Update title.
-		if ( isset( $request['title'] ) ) {
-			$settings['title'] = $request['title'];
-			$gateway->title    = $settings['title'];
+		if ( isset( $request[ 'title' ] ) ) {
+			$settings[ 'title' ] = $request[ 'title' ];
+			$gateway->title    = $settings[ 'title' ];
 		}
 
 		// Update description.
-		if ( isset( $request['description'] ) ) {
-			$settings['description'] = $request['description'];
-			$gateway->description    = $settings['description'];
+		if ( isset( $request[ 'description' ] ) ) {
+			$settings[ 'description' ] = $request[ 'description' ];
+			$gateway->description    = $settings[ 'description' ];
 		}
 
 		// Update options.
@@ -179,11 +179,11 @@ 
 		update_option( $gateway->get_option_key(), apply_filters( 'woocommerce_gateway_' . $gateway->id . '_settings_values', $settings, $gateway ) );
 
 		// Update order.
-		if ( isset( $request['order'] ) ) {
+		if ( isset( $request[ 'order' ] ) ) {
 			$order                 = (array) get_option( 'woocommerce_gateway_order' );
-			$order[ $gateway->id ] = $request['order'];
+			$order[ $gateway->id ] = $request[ 'order' ];
 			update_option( 'woocommerce_gateway_order', $order );
-			$gateway->order = absint( $request['order'] );
+			$gateway->order = absint( $request[ 'order' ] );
 		}
 
 		$gateway = $this->prepare_item_for_response( $gateway, $request );
@@ -200,7 +200,7 @@ 
 		$gateway          = null;
 		$payment_gateways = WC()->payment_gateways->payment_gateways();
 		foreach ( $payment_gateways as $payment_gateway_id => $payment_gateway ) {
-			if ( $request['id'] !== $payment_gateway_id ) {
+			if ( $request[ 'id' ] !== $payment_gateway_id ) {
 				continue;
 			}
 			$payment_gateway->id = $payment_gateway_id;
@@ -243,7 +243,7 @@ 
 		$gateway->init_form_fields();
 		foreach ( $gateway->form_fields as $id => $field ) {
 			// Make sure we at least have a title and type.
-			if ( empty( $field['title'] ) || empty( $field['type'] ) ) {
+			if ( empty( $field[ 'title' ] ) || empty( $field[ 'type' ] ) ) {
 				continue;
 			}
 
@@ -254,16 +254,16 @@ 
 
 			$data = array(
 				'id'          => $id,
-				'label'       => empty( $field['label'] ) ? $field['title'] : $field['label'],
-				'description' => empty( $field['description'] ) ? '' : $field['description'],
-				'type'        => $field['type'],
+				'label'       => empty( $field[ 'label' ] ) ? $field[ 'title' ] : $field[ 'label' ],
+				'description' => empty( $field[ 'description' ] ) ? '' : $field[ 'description' ],
+				'type'        => $field[ 'type' ],
 				'value'       => empty( $gateway->settings[ $id ] ) ? '' : $gateway->settings[ $id ],
-				'default'     => empty( $field['default'] ) ? '' : $field['default'],
-				'tip'         => empty( $field['description'] ) ? '' : $field['description'],
-				'placeholder' => empty( $field['placeholder'] ) ? '' : $field['placeholder'],
+				'default'     => empty( $field[ 'default' ] ) ? '' : $field[ 'default' ],
+				'tip'         => empty( $field[ 'description' ] ) ? '' : $field[ 'description' ],
+				'placeholder' => empty( $field[ 'placeholder' ] ) ? '' : $field[ 'placeholder' ],
 			);
-			if ( ! empty( $field['options'] ) ) {
-				$data['options'] = $field['options'];
+			if ( ! empty( $field[ 'options' ] ) ) {
+				$data[ 'options' ] = $field[ 'options' ];
 			}
 			$settings[ $id ] = $data;
 		}

src/Controllers/Version4/ProductVariations.php

Spacing +28 added lines, -28 removed lines

@@ -522,17 +522,17 @@ 
 		$params = parent::get_collection_params();
 
 		unset(
-			$params['in_stock'],
-			$params['type'],
-			$params['featured'],
-			$params['category'],
-			$params['tag'],
-			$params['shipping_class'],
-			$params['attribute'],
-			$params['attribute_term']
+			$params[ 'in_stock' ],
+			$params[ 'type' ],
+			$params[ 'featured' ],
+			$params[ 'category' ],
+			$params[ 'tag' ],
+			$params[ 'shipping_class' ],
+			$params[ 'attribute' ],
+			$params[ 'attribute_term' ]
 		);
 
-		$params['stock_status'] = array(
+		$params[ 'stock_status' ] = array(
 			'description'       => __( 'Limit result set to products with specified stock status.', 'woocommerce' ),
 			'type'              => 'string',
 			'enum'              => array_keys( wc_get_product_stock_status_options() ),
@@ -540,7 +540,7 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['search'] = array(
+		$params[ 'search' ] = array(
 			'description'       => __( 'Search by similar product name or sku.', 'woocommerce' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',
@@ -575,7 +575,7 @@ 
 		}
 
 		// Check if variation belongs to the correct parent product.
-		if ( $object && 0 !== $object->get_parent_id() && absint( $request['product_id'] ) !== $object->get_parent_id() ) {
+		if ( $object && 0 !== $object->get_parent_id() && absint( $request[ 'product_id' ] ) !== $object->get_parent_id() ) {
 			return new \WP_Error( 'woocommerce_rest_cannot_edit', __( 'Parent product does not match current variation.', 'woocommerce' ), array( 'status' => rest_authorization_required_code() ) );
 		}
 		return true;
@@ -653,7 +653,7 @@ 
 		$args = parent::prepare_objects_query( $request );
 
 		// Set post_status.
-		$args['post_status'] = $request['status'];
+		$args[ 'post_status' ] = $request[ 'status' ];
 
 		// Set custom args to handle later during clauses.
 		$custom_keys = array(
@@ -670,19 +670,19 @@ 
 		}
 
 		// Filter by tax class.
-		if ( ! empty( $request['tax_class'] ) ) {
-			$args['meta_query'] = $this->add_meta_query( // WPCS: slow query ok.
+		if ( ! empty( $request[ 'tax_class' ] ) ) {
+			$args[ 'meta_query' ] = $this->add_meta_query( // WPCS: slow query ok.
 				$args,
 				array(
 					'key'   => '_tax_class',
-					'value' => 'standard' !== $request['tax_class'] ? $request['tax_class'] : '',
+					'value' => 'standard' !== $request[ 'tax_class' ] ? $request[ 'tax_class' ] : '',
 				)
 			);
 		}
 
 		// Filter by on sale products.
-		if ( is_bool( $request['on_sale'] ) ) {
-			$on_sale_key = $request['on_sale'] ? 'post__in' : 'post__not_in';
+		if ( is_bool( $request[ 'on_sale' ] ) ) {
+			$on_sale_key = $request[ 'on_sale' ] ? 'post__in' : 'post__not_in';
 			$on_sale_ids = wc_get_product_ids_on_sale();
 
 			// Use 0 when there's no on sale products to avoid return all products.
@@ -692,17 +692,17 @@ 
 		}
 
 		// Force the post_type argument, since it's not a user input variable.
-		if ( ! empty( $request['sku'] ) ) {
-			$args['post_type'] = array( 'product', 'product_variation' );
+		if ( ! empty( $request[ 'sku' ] ) ) {
+			$args[ 'post_type' ] = array( 'product', 'product_variation' );
 		} else {
-			$args['post_type'] = $this->post_type;
+			$args[ 'post_type' ] = $this->post_type;
 		}
 
-		$args['post_parent'] = $request['product_id'];
+		$args[ 'post_parent' ] = $request[ 'product_id' ];
 
-		if ( ! empty( $request['search'] ) ) {
-			$args['search'] = $request['search'];
-			unset( $args['s'] );
+		if ( ! empty( $request[ 'search' ] ) ) {
+			$args[ 'search' ] = $request[ 'search' ];
+			unset( $args[ 's' ] );
 		}
 
 		return $args;
@@ -745,8 +745,8 @@ 
 	 * @return bool|\WP_Error|\WP_REST_Response
 	 */
 	public function delete_item( $request ) {
-		$force  = (bool) $request['force'];
-		$object = $this->get_object( (int) $request['id'] );
+		$force  = (bool) $request[ 'force' ];
+		$object = $this->get_object( (int) $request[ 'id' ] );
 		$result = false;
 
 		if ( ! $object || 0 === $object->get_id() ) {
@@ -854,7 +854,7 @@ 
 	protected function get_batch_of_items_from_request( $request, $batch_type ) {
 		$params     = $request->get_params();
 		$url_params = $request->get_url_params();
-		$product_id = $url_params['product_id'];
+		$product_id = $url_params[ 'product_id' ];
 
 		if ( ! isset( $params[ $batch_type ] ) ) {
 			return array();
@@ -884,7 +884,7 @@ 
 	 * @return array
 	 */
 	protected function prepare_links( $item, $request ) {
-		$product_id = (int) $request['product_id'];
+		$product_id = (int) $request[ 'product_id' ];
 		$base       = str_replace( '(?P<product_id>[\d]+)', $product_id, $this->rest_base );
 		$links      = array(
 			'self'       => array(

src/Controllers/Version4/Data/Currencies.php

Spacing +3 added lines, -3 removed lines

@@ -108,7 +108,7 @@ 
 		foreach ( array_keys( $currencies ) as $code ) {
 			$currency = $this->get_currency( $code, $request );
 			$response = $this->prepare_item_for_response( $currency, $request );
-			$data[]   = $this->prepare_response_for_collection( $response );
+			$data[ ]   = $this->prepare_response_for_collection( $response );
 		}
 
 		return rest_ensure_response( $data );
@@ -121,7 +121,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$data = $this->get_currency( strtoupper( $request['currency'] ), $request );
+		$data = $this->get_currency( strtoupper( $request[ 'currency' ] ), $request );
 		if ( empty( $data ) ) {
 			return new \WP_Error( 'woocommerce_rest_data_invalid_currency', __( 'There are no currencies matching these parameters.', 'woocommerce' ), array( 'status' => 404 ) );
 		}
@@ -158,7 +158,7 @@ 
 	 * @return array
 	 */
 	protected function prepare_links( $item, $request ) {
-		$code  = strtoupper( $item['code'] );
+		$code  = strtoupper( $item[ 'code' ] );
 		$links = array(
 			'self'       => array(
 				'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $code ) ),