woocommerce / woocommerce-rest-api

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-rest-api' ), 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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(

unit-tests/Bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
 
 			\WC_Install::install();
 
-			$GLOBALS['wp_roles'] = null; // WPCS: override ok.
+			$GLOBALS[ 'wp_roles' ] = null; // WPCS: override ok.
 			\wp_roles();
 		} );
 	}

unit-tests/Helpers/ProductHelper.php

Spacing +6 added lines, -6 removed lines

@@ -129,13 +129,13 @@ 
 		$attribute_data = self::create_attribute( 'size', array( 'small', 'large' ) ); // Create all attribute related things.
 		$attributes     = array();
 		$attribute      = new WC_Product_Attribute();
-		$attribute->set_id( $attribute_data['attribute_id'] );
-		$attribute->set_name( $attribute_data['attribute_taxonomy'] );
-		$attribute->set_options( $attribute_data['term_ids'] );
+		$attribute->set_id( $attribute_data[ 'attribute_id' ] );
+		$attribute->set_name( $attribute_data[ 'attribute_taxonomy' ] );
+		$attribute->set_options( $attribute_data[ 'term_ids' ] );
 		$attribute->set_position( 1 );
 		$attribute->set_visible( true );
 		$attribute->set_variation( true );
-		$attributes[] = $attribute;
+		$attributes[ ] = $attribute;
 
 		$product->set_attributes( $attributes );
 		$product->save();
@@ -246,9 +246,9 @@ 
 
 			if ( ! $result ) {
 				$result               = wp_insert_term( $term, $attribute->slug );
-				$return['term_ids'][] = $result['term_id'];
+				$return[ 'term_ids' ][ ] = $result[ 'term_id' ];
 			} else {
-				$return['term_ids'][] = $result['term_id'];
+				$return[ 'term_ids' ][ ] = $result[ 'term_id' ];
 			}
 		}
 

unit-tests/Helpers/OrderHelper.php

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
 
 		// Delete all products in the order.
 		foreach ( $order->get_items() as $item ) {
-			\Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::delete_product( $item['product_id'] );
+			\Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::delete_product( $item[ 'product_id' ] );
 		}
 
 		ShippingHelper::delete_simple_flat_rate();
@@ -65,7 +65,7 @@ 
 			'total'         => '',
 		);
 
-		$_SERVER['REMOTE_ADDR'] = '127.0.0.1'; // Required, else wc_create_order throws an exception
+		$_SERVER[ 'REMOTE_ADDR' ] = '127.0.0.1'; // Required, else wc_create_order throws an exception
 		$order                  = wc_create_order( $order_data );
 
 		// Add order products
@@ -113,7 +113,7 @@ 
 
 		// Set payment gateway
 		$payment_gateways = WC()->payment_gateways->payment_gateways();
-		$order->set_payment_method( $payment_gateways['bacs'] );
+		$order->set_payment_method( $payment_gateways[ 'bacs' ] );
 
 		// Set totals
 		$order->set_shipping_total( 10 );

unit-tests/Helpers/SettingsHelper.php

Spacing +5 added lines, -5 removed lines

@@ -30,26 +30,26 @@ 
 	 * @return array
 	 */
 	public static function register_groups( $groups ) {
-		$groups[] = array(
+		$groups[ ] = array(
 			'id'          => 'test',
 			'bad'         => 'value',
 			'label'       => 'Test extension',
 			'description' => 'My awesome test settings.',
 			'option_key'  => '',
 		);
-		$groups[] = array(
+		$groups[ ] = array(
 			'id'          => 'sub-test',
 			'parent_id'   => 'test',
 			'label'       => 'Sub test',
 			'description' => '',
 			'option_key'  => '',
 		);
-		$groups[] = array(
+		$groups[ ] = array(
 			'id'         => 'coupon-data',
 			'label'      => 'Coupon data',
 			'option_key' => '',
 		);
-		$groups[] = array(
+		$groups[ ] = array(
 			'id'         => 'invalid',
 			'option_key' => '',
 		);
@@ -64,7 +64,7 @@ 
 	 * @return array
 	 */
 	public static function register_test_settings( $settings ) {
-		$settings[] = array(
+		$settings[ ] = array(
 			'id'          => 'woocommerce_shop_page_display',
 			'label'       => 'Shop page display',
 			'description' => 'This controls what is shown on the product archive.',

unit-tests/AbstractRestApiTest.php

Spacing +7 added lines, -7 removed lines

@@ -38,14 +38,14 @@ 
 	 *
 	 * @var array Keys are property names, values are supported context.
 	 */
-	protected $properties = [];
+	protected $properties = [ ];
 
 	/**
 	 * Routes that this endpoint creates.
 	 *
 	 * @var array
 	 */
-	protected $routes = [];
+	protected $routes = [ ];
 
 	/**
 	 * User variable.
@@ -93,10 +93,10 @@ 
 	 * @return void
 	 */
 	public function test_schema_properties() {
-		$request    = new \WP_REST_Request( 'OPTIONS', $this->routes[0] );
+		$request    = new \WP_REST_Request( 'OPTIONS', $this->routes[ 0 ] );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( count( array_keys( $this->properties ) ), count( $properties ), print_r( array_diff( array_keys( $properties ), array_keys( $this->properties ) ), true ) );
 
@@ -136,7 +136,7 @@ 
 	 * @param array   $params Request body or query.
 	 * @return object
 	 */
-	protected function do_request( $endpoint, $type = 'GET', $params = [] ) {
+	protected function do_request( $endpoint, $type = 'GET', $params = [ ] ) {
 		$request = new \WP_REST_Request( $type, untrailingslashit( $endpoint ) );
 		'GET' === $type ? $request->set_query_params( $params ) : $request->set_body_params( $params );
 		$response = $this->server->dispatch( $request );
@@ -194,8 +194,8 @@ 
 			$this->assertArrayHasKey( 'id', $meta );
 			$this->assertArrayHasKey( 'key', $meta );
 			$this->assertArrayHasKey( 'value', $meta );
-			$this->assertEquals( $expected_meta_data[ $key ]['key'], $meta['key'] );
-			$this->assertEquals( $expected_meta_data[ $key ]['value'], $meta['value'] );
+			$this->assertEquals( $expected_meta_data[ $key ][ 'key' ], $meta[ 'key' ] );
+			$this->assertEquals( $expected_meta_data[ $key ][ 'value' ], $meta[ 'value' ] );
 		}
 	}
 

unit-tests/Tests/Version3/reports-products-totals.php

Spacing +3 added lines, -3 removed lines

@@ -48,14 +48,14 @@ 
 				'hide_empty' => false,
 			)
 		);
-		$data     = array();
+		$data = array();
 
 		foreach ( $terms as $product_type ) {
 			if ( ! isset( $types[ $product_type->name ] ) ) {
 				continue;
 			}
 
-			$data[] = array(
+			$data[ ] = array(
 				'slug'  => $product_type->name,
 				'name'  => $types[ $product_type->name ],
 				'total' => (int) $product_type->count,
@@ -89,7 +89,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v3/reports/products/totals' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 3, count( $properties ) );
 		$this->assertArrayHasKey( 'slug', $properties );

unit-tests/Tests/Version3/shipping-zones.php

Spacing +50 added lines, -50 removed lines

@@ -27,7 +27,7 @@ 
 				'role' => 'administrator',
 			)
 		);
-		$this->zones    = array();
+		$this->zones = array();
 	}
 
 	/**
@@ -44,7 +44,7 @@ 
 		$zone->set_locations( $locations );
 		$zone->save();
 
-		$this->zones[] = $zone;
+		$this->zones[ ] = $zone;
 
 		return $zone;
 	}
@@ -79,13 +79,13 @@ 
 		$this->assertEquals( count( $data ), 1 );
 		$this->assertContains(
 			array(
-				'id'     => $data[0]['id'],
+				'id'     => $data[ 0 ][ 'id' ],
 				'name'   => 'Locations not covered by your other zones',
 				'order'  => 0,
 				'_links' => array(
 					'self'        => array(
 						array(
-							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[0]['id'] ),
+							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[ 0 ][ 'id' ] ),
 						),
 					),
 					'collection'  => array(
@@ -95,7 +95,7 @@ 
 					),
 					'describedby' => array(
 						array(
-							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[0]['id'] . '/locations' ),
+							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[ 0 ][ 'id' ] . '/locations' ),
 						),
 					),
 				),
@@ -113,13 +113,13 @@ 
 		$this->assertEquals( count( $data ), 2 );
 		$this->assertContains(
 			array(
-				'id'     => $data[1]['id'],
+				'id'     => $data[ 1 ][ 'id' ],
 				'name'   => 'Zone 1',
 				'order'  => 0,
 				'_links' => array(
 					'self'        => array(
 						array(
-							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[1]['id'] ),
+							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[ 1 ][ 'id' ] ),
 						),
 					),
 					'collection'  => array(
@@ -129,7 +129,7 @@ 
 					),
 					'describedby' => array(
 						array(
-							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[1]['id'] . '/locations' ),
+							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[ 1 ][ 'id' ] . '/locations' ),
 						),
 					),
 				),
@@ -175,10 +175,10 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v3/shipping/zones' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 		$this->assertEquals( 3, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
-		$this->assertTrue( $properties['id']['readonly'] );
+		$this->assertTrue( $properties[ 'id' ][ 'readonly' ] );
 		$this->assertArrayHasKey( 'name', $properties );
 		$this->assertArrayHasKey( 'order', $properties );
 	}
@@ -204,13 +204,13 @@ 
 		$this->assertEquals( 201, $response->get_status() );
 		$this->assertEquals(
 			array(
-				'id'     => $data['id'],
+				'id'     => $data[ 'id' ],
 				'name'   => 'Test Zone',
 				'order'  => 1,
 				'_links' => array(
 					'self'        => array(
 						array(
-							'href' => rest_url( '/wc/v3/shipping/zones/' . $data['id'] ),
+							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[ 'id' ] ),
 						),
 					),
 					'collection'  => array(
@@ -220,7 +220,7 @@ 
 					),
 					'describedby' => array(
 						array(
-							'href' => rest_url( '/wc/v3/shipping/zones/' . $data['id'] . '/locations' ),
+							'href' => rest_url( '/wc/v3/shipping/zones/' . $data[ 'id' ] . '/locations' ),
 						),
 					),
 				),
@@ -597,16 +597,16 @@ 
 		foreach ( $method->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'       => $method->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;
 		}
@@ -616,7 +616,7 @@ 
 		$expected = array(
 			'id'                 => $instance_id,
 			'instance_id'        => $instance_id,
-			'title'              => $method->instance_settings['title'],
+			'title'              => $method->instance_settings[ 'title' ],
 			'order'              => $method->method_order,
 			'enabled'            => ( 'yes' === $method->enabled ),
 			'method_id'          => $method->id,
@@ -702,12 +702,12 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertArrayHasKey( 'title', $data['settings'] );
-		$this->assertEquals( 'Flat rate', $data['settings']['title']['value'] );
-		$this->assertArrayHasKey( 'tax_status', $data['settings'] );
-		$this->assertEquals( 'taxable', $data['settings']['tax_status']['value'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '0', $data['settings']['cost']['value'] );
+		$this->assertArrayHasKey( 'title', $data[ 'settings' ] );
+		$this->assertEquals( 'Flat rate', $data[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertArrayHasKey( 'tax_status', $data[ 'settings' ] );
+		$this->assertEquals( 'taxable', $data[ 'settings' ][ 'tax_status' ][ 'value' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '0', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 
 		// Update a single value
 		$request = new WP_REST_Request( 'POST', '/wc/v3/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
@@ -721,12 +721,12 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertArrayHasKey( 'title', $data['settings'] );
-		$this->assertEquals( 'Flat rate', $data['settings']['title']['value'] );
-		$this->assertArrayHasKey( 'tax_status', $data['settings'] );
-		$this->assertEquals( 'taxable', $data['settings']['tax_status']['value'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '5', $data['settings']['cost']['value'] );
+		$this->assertArrayHasKey( 'title', $data[ 'settings' ] );
+		$this->assertEquals( 'Flat rate', $data[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertArrayHasKey( 'tax_status', $data[ 'settings' ] );
+		$this->assertEquals( 'taxable', $data[ 'settings' ][ 'tax_status' ][ 'value' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '5', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 
 		// Test multiple settings
 		$request = new WP_REST_Request( 'POST', '/wc/v3/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
@@ -741,12 +741,12 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertArrayHasKey( 'title', $data['settings'] );
-		$this->assertEquals( 'Flat rate', $data['settings']['title']['value'] );
-		$this->assertArrayHasKey( 'tax_status', $data['settings'] );
-		$this->assertEquals( 'none', $data['settings']['tax_status']['value'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '10', $data['settings']['cost']['value'] );
+		$this->assertArrayHasKey( 'title', $data[ 'settings' ] );
+		$this->assertEquals( 'Flat rate', $data[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertArrayHasKey( 'tax_status', $data[ 'settings' ] );
+		$this->assertEquals( 'none', $data[ 'settings' ][ 'tax_status' ][ 'value' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '10', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 
 		// Test bogus
 		$request = new WP_REST_Request( 'POST', '/wc/v3/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
@@ -762,8 +762,8 @@ 
 		$this->assertEquals( 400, $response->get_status() );
 
 		// Test other parameters
-		$this->assertTrue( $data['enabled'] );
-		$this->assertEquals( 1, $data['order'] );
+		$this->assertTrue( $data[ 'enabled' ] );
+		$this->assertEquals( 1, $data[ 'order' ] );
 
 		$request = new WP_REST_Request( 'POST', '/wc/v3/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
 		$request->set_body_params(
@@ -775,10 +775,10 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertFalse( $data['enabled'] );
-		$this->assertEquals( 2, $data['order'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '10', $data['settings']['cost']['value'] );
+		$this->assertFalse( $data[ 'enabled' ] );
+		$this->assertEquals( 2, $data[ 'order' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '10', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 	}
 
 	/**
@@ -800,10 +800,10 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertFalse( $data['enabled'] );
-		$this->assertEquals( 2, $data['order'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '0', $data['settings']['cost']['value'] );
+		$this->assertFalse( $data[ 'enabled' ] );
+		$this->assertEquals( 2, $data[ 'order' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '0', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 	}
 
 	/**

unit-tests/Tests/Version3/reports-orders-totals.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
 				continue;
 			}
 
-			$data[] = array(
+			$data[ ] = array(
 				'slug'  => str_replace( 'wc-', '', $slug ),
 				'name'  => $name,
 				'total' => (int) $totals->$slug,
@@ -82,7 +82,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v3/reports/orders/totals' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 3, count( $properties ) );
 		$this->assertArrayHasKey( 'slug', $properties );