woocommerce / woocommerce-rest-api

src/Controllers/Version4/PaymentGateways.php

Spacing +29 added lines, -29 removed lines

@@ -123,7 +123,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 );
 	}
@@ -163,14 +163,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;
@@ -186,21 +186,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.
@@ -208,11 +208,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 );
@@ -229,7 +229,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;
@@ -259,7 +259,7 @@ 
 			'settings'           => $this->get_settings( $gateway ),
 		);
 
-		$context = ! empty( $request['context'] ) ? $request['context'] : 'view';
+		$context = ! empty( $request[ 'context' ] ) ? $request[ 'context' ] : 'view';
 		$data    = $this->add_additional_fields_to_object( $item, $request );
 		$data    = $this->filter_response_by_context( $data, $context );
 
@@ -288,7 +288,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;
 			}
 
@@ -299,16 +299,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 +106 added lines, -106 removed lines

@@ -155,14 +155,14 @@ 
 	 * @return \WP_Error|boolean
 	 */
 	public function update_item_permissions_check( $request ) {
-		$object = $this->get_object( (int) $request['id'] );
+		$object = $this->get_object( (int) $request[ 'id' ] );
 
 		if ( $object && 0 !== $object->get_id() && ! wc_rest_check_post_permissions( $this->post_type, 'edit', $object->get_id() ) ) {
 			return new \WP_Error( 'woocommerce_rest_cannot_edit', __( 'Sorry, you are not allowed to edit this resource.', 'woocommerce' ), array( 'status' => rest_authorization_required_code() ) );
 		}
 
 		// 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() ) );
 		}
 
@@ -177,7 +177,7 @@ 
 	 * @return \WP_REST_Response
 	 */
 	public function prepare_object_for_response( $object, $request ) {
-		$context = ! empty( $request['context'] ) ? $request['context'] : 'view';
+		$context = ! empty( $request[ 'context' ] ) ? $request[ 'context' ] : 'view';
 		$data    = array(
 			'id'                    => $object->get_id(),
 			'name'                  => $object->get_name( $context ),
@@ -227,7 +227,7 @@ 
 			'meta_data'             => $object->get_meta_data(),
 		);
 
-		$context  = ! empty( $request['context'] ) ? $request['context'] : 'view';
+		$context  = ! empty( $request[ 'context' ] ) ? $request[ 'context' ] : 'view';
 		$data     = $this->add_additional_fields_to_object( $data, $request );
 		$data     = $this->filter_response_by_context( $data, $context );
 		$response = rest_ensure_response( $data );
@@ -292,10 +292,10 @@ 
 	 * @return WC_Product_Variation
 	 */
 	protected function set_variation_image( $variation, $image ) {
-		$attachment_id = isset( $image['id'] ) ? absint( $image['id'] ) : 0;
+		$attachment_id = isset( $image[ 'id' ] ) ? absint( $image[ 'id' ] ) : 0;
 
-		if ( 0 === $attachment_id && isset( $image['src'] ) ) {
-			$upload = wc_rest_upload_image_from_url( esc_url_raw( $image['src'] ) );
+		if ( 0 === $attachment_id && isset( $image[ 'src' ] ) ) {
+			$upload = wc_rest_upload_image_from_url( esc_url_raw( $image[ 'src' ] ) );
 
 			if ( is_wp_error( $upload ) ) {
 				if ( ! apply_filters( 'woocommerce_rest_suppress_image_upload_error', false, $upload, $variation->get_id(), array( $image ) ) ) {
@@ -314,16 +314,16 @@ 
 		$variation->set_image_id( $attachment_id );
 
 		// Set the image alt if present.
-		if ( ! empty( $image['alt'] ) ) {
-			update_post_meta( $attachment_id, '_wp_attachment_image_alt', wc_clean( $image['alt'] ) );
+		if ( ! empty( $image[ 'alt' ] ) ) {
+			update_post_meta( $attachment_id, '_wp_attachment_image_alt', wc_clean( $image[ 'alt' ] ) );
 		}
 
 		// Set the image name if present.
-		if ( ! empty( $image['name'] ) ) {
+		if ( ! empty( $image[ 'name' ] ) ) {
 			wp_update_post(
 				array(
 					'ID'         => $attachment_id,
-					'post_title' => $image['name'],
+					'post_title' => $image[ 'name' ],
 				)
 			);
 		}
@@ -342,17 +342,17 @@ 
 		$args = parent::prepare_objects_query( $request );
 
 		// Set post_status.
-		$args['post_status'] = $request['status'];
+		$args[ 'post_status' ] = $request[ 'status' ];
 
 		// Filter by sku.
-		if ( ! empty( $request['sku'] ) ) {
-			$skus = explode( ',', $request['sku'] );
+		if ( ! empty( $request[ 'sku' ] ) ) {
+			$skus = explode( ',', $request[ 'sku' ] );
 			// Include the current string as a SKU too.
 			if ( 1 < count( $skus ) ) {
-				$skus[] = $request['sku'];
+				$skus[ ] = $request[ 'sku' ];
 			}
 
-			$args['meta_query'] = $this->add_meta_query( // WPCS: slow query ok.
+			$args[ 'meta_query' ] = $this->add_meta_query( // WPCS: slow query ok.
 				$args,
 				array(
 					'key'     => '_sku',
@@ -363,35 +363,35 @@ 
 		}
 
 		// 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' ] : '',
 				)
 			);
 		}
 
 		// Price filter.
-		if ( ! empty( $request['min_price'] ) || ! empty( $request['max_price'] ) ) {
-			$args['meta_query'] = $this->add_meta_query( $args, wc_get_min_max_price_meta_query( $request ) );  // WPCS: slow query ok.
+		if ( ! empty( $request[ 'min_price' ] ) || ! empty( $request[ 'max_price' ] ) ) {
+			$args[ 'meta_query' ] = $this->add_meta_query( $args, wc_get_min_max_price_meta_query( $request ) ); // WPCS: slow query ok.
 		}
 
 		// Filter product based on stock_status.
-		if ( ! empty( $request['stock_status'] ) ) {
-			$args['meta_query'] = $this->add_meta_query( // WPCS: slow query ok.
+		if ( ! empty( $request[ 'stock_status' ] ) ) {
+			$args[ 'meta_query' ] = $this->add_meta_query( // WPCS: slow query ok.
 				$args,
 				array(
 					'key'   => '_stock_status',
-					'value' => $request['stock_status'],
+					'value' => $request[ 'stock_status' ],
 				)
 			);
 		}
 
 		// 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.
@@ -401,17 +401,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;
@@ -425,58 +425,58 @@ 
 	 * @return \WP_Error|WC_Data
 	 */
 	protected function prepare_object_for_database( $request, $creating = false ) {
-		if ( isset( $request['id'] ) ) {
-			$variation = wc_get_product( absint( $request['id'] ) );
+		if ( isset( $request[ 'id' ] ) ) {
+			$variation = wc_get_product( absint( $request[ 'id' ] ) );
 		} else {
 			$variation = new \WC_Product_Variation();
 		}
 
-		$variation->set_parent_id( absint( $request['product_id'] ) );
+		$variation->set_parent_id( absint( $request[ 'product_id' ] ) );
 
 		// Status.
-		if ( isset( $request['status'] ) ) {
-			$variation->set_status( get_post_status_object( $request['status'] ) ? $request['status'] : 'draft' );
+		if ( isset( $request[ 'status' ] ) ) {
+			$variation->set_status( get_post_status_object( $request[ 'status' ] ) ? $request[ 'status' ] : 'draft' );
 		}
 
 		// SKU.
-		if ( isset( $request['sku'] ) ) {
-			$variation->set_sku( wc_clean( $request['sku'] ) );
+		if ( isset( $request[ 'sku' ] ) ) {
+			$variation->set_sku( wc_clean( $request[ 'sku' ] ) );
 		}
 
 		// Thumbnail.
-		if ( isset( $request['image'] ) ) {
-			if ( is_array( $request['image'] ) ) {
-				$variation = $this->set_variation_image( $variation, $request['image'] );
+		if ( isset( $request[ 'image' ] ) ) {
+			if ( is_array( $request[ 'image' ] ) ) {
+				$variation = $this->set_variation_image( $variation, $request[ 'image' ] );
 			} else {
 				$variation->set_image_id( '' );
 			}
 		}
 
 		// Virtual variation.
-		if ( isset( $request['virtual'] ) ) {
-			$variation->set_virtual( $request['virtual'] );
+		if ( isset( $request[ 'virtual' ] ) ) {
+			$variation->set_virtual( $request[ 'virtual' ] );
 		}
 
 		// Downloadable variation.
-		if ( isset( $request['downloadable'] ) ) {
-			$variation->set_downloadable( $request['downloadable'] );
+		if ( isset( $request[ 'downloadable' ] ) ) {
+			$variation->set_downloadable( $request[ 'downloadable' ] );
 		}
 
 		// Downloads.
 		if ( $variation->get_downloadable() ) {
 			// Downloadable files.
-			if ( isset( $request['downloads'] ) && is_array( $request['downloads'] ) ) {
-				$variation = $this->save_downloadable_files( $variation, $request['downloads'] );
+			if ( isset( $request[ 'downloads' ] ) && is_array( $request[ 'downloads' ] ) ) {
+				$variation = $this->save_downloadable_files( $variation, $request[ 'downloads' ] );
 			}
 
 			// Download limit.
-			if ( isset( $request['download_limit'] ) ) {
-				$variation->set_download_limit( $request['download_limit'] );
+			if ( isset( $request[ 'download_limit' ] ) ) {
+				$variation->set_download_limit( $request[ 'download_limit' ] );
 			}
 
 			// Download expiry.
-			if ( isset( $request['download_expiry'] ) ) {
-				$variation->set_download_expiry( $request['download_expiry'] );
+			if ( isset( $request[ 'download_expiry' ] ) ) {
+				$variation->set_download_expiry( $request[ 'download_expiry' ] );
 			}
 		}
 
@@ -484,24 +484,24 @@ 
 		$variation = $this->save_product_shipping_data( $variation, $request );
 
 		// Stock handling.
-		if ( isset( $request['manage_stock'] ) ) {
-			$variation->set_manage_stock( $request['manage_stock'] );
+		if ( isset( $request[ 'manage_stock' ] ) ) {
+			$variation->set_manage_stock( $request[ 'manage_stock' ] );
 		}
 
-		if ( isset( $request['stock_status'] ) ) {
-			$variation->set_stock_status( $request['stock_status'] );
+		if ( isset( $request[ 'stock_status' ] ) ) {
+			$variation->set_stock_status( $request[ 'stock_status' ] );
 		}
 
-		if ( isset( $request['backorders'] ) ) {
-			$variation->set_backorders( $request['backorders'] );
+		if ( isset( $request[ 'backorders' ] ) ) {
+			$variation->set_backorders( $request[ 'backorders' ] );
 		}
 
 		if ( $variation->get_manage_stock() ) {
-			if ( isset( $request['stock_quantity'] ) ) {
-				$variation->set_stock_quantity( $request['stock_quantity'] );
-			} elseif ( isset( $request['inventory_delta'] ) ) {
+			if ( isset( $request[ 'stock_quantity' ] ) ) {
+				$variation->set_stock_quantity( $request[ 'stock_quantity' ] );
+			} elseif ( isset( $request[ 'inventory_delta' ] ) ) {
 				$stock_quantity  = wc_stock_amount( $variation->get_stock_quantity() );
-				$stock_quantity += wc_stock_amount( $request['inventory_delta'] );
+				$stock_quantity += wc_stock_amount( $request[ 'inventory_delta' ] );
 				$variation->set_stock_quantity( $stock_quantity );
 			}
 		} else {
@@ -510,43 +510,43 @@ 
 		}
 
 		// Regular Price.
-		if ( isset( $request['regular_price'] ) ) {
-			$variation->set_regular_price( $request['regular_price'] );
+		if ( isset( $request[ 'regular_price' ] ) ) {
+			$variation->set_regular_price( $request[ 'regular_price' ] );
 		}
 
 		// Sale Price.
-		if ( isset( $request['sale_price'] ) ) {
-			$variation->set_sale_price( $request['sale_price'] );
+		if ( isset( $request[ 'sale_price' ] ) ) {
+			$variation->set_sale_price( $request[ 'sale_price' ] );
 		}
 
-		if ( isset( $request['date_on_sale_from'] ) ) {
-			$variation->set_date_on_sale_from( $request['date_on_sale_from'] );
+		if ( isset( $request[ 'date_on_sale_from' ] ) ) {
+			$variation->set_date_on_sale_from( $request[ 'date_on_sale_from' ] );
 		}
 
-		if ( isset( $request['date_on_sale_from_gmt'] ) ) {
-			$variation->set_date_on_sale_from( $request['date_on_sale_from_gmt'] ? strtotime( $request['date_on_sale_from_gmt'] ) : null );
+		if ( isset( $request[ 'date_on_sale_from_gmt' ] ) ) {
+			$variation->set_date_on_sale_from( $request[ 'date_on_sale_from_gmt' ] ? strtotime( $request[ 'date_on_sale_from_gmt' ] ) : null );
 		}
 
-		if ( isset( $request['date_on_sale_to'] ) ) {
-			$variation->set_date_on_sale_to( $request['date_on_sale_to'] );
+		if ( isset( $request[ 'date_on_sale_to' ] ) ) {
+			$variation->set_date_on_sale_to( $request[ 'date_on_sale_to' ] );
 		}
 
-		if ( isset( $request['date_on_sale_to_gmt'] ) ) {
-			$variation->set_date_on_sale_to( $request['date_on_sale_to_gmt'] ? strtotime( $request['date_on_sale_to_gmt'] ) : null );
+		if ( isset( $request[ 'date_on_sale_to_gmt' ] ) ) {
+			$variation->set_date_on_sale_to( $request[ 'date_on_sale_to_gmt' ] ? strtotime( $request[ 'date_on_sale_to_gmt' ] ) : null );
 		}
 
 		// Tax class.
-		if ( isset( $request['tax_class'] ) ) {
-			$variation->set_tax_class( $request['tax_class'] );
+		if ( isset( $request[ 'tax_class' ] ) ) {
+			$variation->set_tax_class( $request[ 'tax_class' ] );
 		}
 
 		// Description.
-		if ( isset( $request['description'] ) ) {
-			$variation->set_description( wp_kses_post( $request['description'] ) );
+		if ( isset( $request[ 'description' ] ) ) {
+			$variation->set_description( wp_kses_post( $request[ 'description' ] ) );
 		}
 
 		// Update taxonomies.
-		if ( isset( $request['attributes'] ) ) {
+		if ( isset( $request[ 'attributes' ] ) ) {
 			$attributes = array();
 			$parent     = wc_get_product( $variation->get_parent_id() );
 
@@ -561,16 +561,16 @@ 
 
 			$parent_attributes = $parent->get_attributes();
 
-			foreach ( $request['attributes'] as $attribute ) {
+			foreach ( $request[ 'attributes' ] as $attribute ) {
 				$attribute_id   = 0;
 				$attribute_name = '';
 
 				// Check ID for global attributes or name for product attributes.
-				if ( ! empty( $attribute['id'] ) ) {
-					$attribute_id   = absint( $attribute['id'] );
+				if ( ! empty( $attribute[ 'id' ] ) ) {
+					$attribute_id   = absint( $attribute[ 'id' ] );
 					$attribute_name = wc_attribute_taxonomy_name_by_id( $attribute_id );
-				} elseif ( ! empty( $attribute['name'] ) ) {
-					$attribute_name = sanitize_title( $attribute['name'] );
+				} elseif ( ! empty( $attribute[ 'name' ] ) ) {
+					$attribute_name = sanitize_title( $attribute[ 'name' ] );
 				}
 
 				if ( ! $attribute_id && ! $attribute_name ) {
@@ -582,7 +582,7 @@ 
 				}
 
 				$attribute_key   = sanitize_title( $parent_attributes[ $attribute_name ]->get_name() );
-				$attribute_value = isset( $attribute['option'] ) ? wc_clean( stripslashes( $attribute['option'] ) ) : '';
+				$attribute_value = isset( $attribute[ 'option' ] ) ? wc_clean( stripslashes( $attribute[ 'option' ] ) ) : '';
 
 				if ( $parent_attributes[ $attribute_name ]->is_taxonomy() ) {
 					// If dealing with a taxonomy, we need to get the slug from the name posted to the API.
@@ -602,14 +602,14 @@ 
 		}
 
 		// Menu order.
-		if ( $request['menu_order'] ) {
-			$variation->set_menu_order( $request['menu_order'] );
+		if ( $request[ 'menu_order' ] ) {
+			$variation->set_menu_order( $request[ 'menu_order' ] );
 		}
 
 		// Meta data.
-		if ( is_array( $request['meta_data'] ) ) {
-			foreach ( $request['meta_data'] as $meta ) {
-				$variation->update_meta_data( $meta['key'], $meta['value'], isset( $meta['id'] ) ? $meta['id'] : '' );
+		if ( is_array( $request[ 'meta_data' ] ) ) {
+			foreach ( $request[ 'meta_data' ] as $meta ) {
+				$variation->update_meta_data( $meta[ 'key' ], $meta[ 'value' ], isset( $meta[ 'id' ] ) ? $meta[ 'id' ] : '' );
 			}
 		}
 
@@ -644,8 +644,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() ) {
@@ -753,14 +753,14 @@ 
 	public function batch_items( $request ) {
 		$items       = array_filter( $request->get_params() );
 		$params      = $request->get_url_params();
-		$product_id  = $params['product_id'];
+		$product_id  = $params[ 'product_id' ];
 		$body_params = array();
 
 		foreach ( array( 'update', 'create', 'delete' ) as $batch_type ) {
 			if ( ! empty( $items[ $batch_type ] ) ) {
 				$injected_items = array();
 				foreach ( $items[ $batch_type ] as $item ) {
-					$injected_items[] = is_array( $item ) ? array_merge(
+					$injected_items[ ] = is_array( $item ) ? array_merge(
 						array(
 							'product_id' => $product_id,
 						), $item
@@ -784,7 +784,7 @@ 
 	 * @return array                   Links for the given post.
 	 */
 	protected function prepare_links( $object, $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(
@@ -1182,17 +1182,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() ),
@@ -1200,7 +1200,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',

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 ) );
 		}
@@ -170,7 +170,7 @@ 
 	 * @return array Links for the given currency.
 	 */
 	protected function prepare_links( $item ) {
-		$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 ) ),

src/Controllers/Version4/Data/DownloadIPs.php

Spacing +5 added lines, -5 removed lines

@@ -55,13 +55,13 @@ 
 	public function get_items( $request ) {
 		global $wpdb;
 
-		if ( isset( $request['match'] ) ) {
+		if ( isset( $request[ 'match' ] ) ) {
 			$downloads = $wpdb->get_results(
 				$wpdb->prepare(
 					"SELECT DISTINCT( user_ip_address ) FROM {$wpdb->prefix}wc_download_log
 					WHERE user_ip_address LIKE %s
 					LIMIT 10",
-					$request['match'] . '%'
+					$request[ 'match' ] . '%'
 				)
 			);
 		} else {
@@ -73,7 +73,7 @@ 
 		if ( ! empty( $downloads ) ) {
 			foreach ( $downloads as $download ) {
 				$response = $this->prepare_item_for_response( $download, $request );
-				$data[]   = $this->prepare_response_for_collection( $response );
+				$data[ ]   = $this->prepare_response_for_collection( $response );
 			}
 		}
 
@@ -127,8 +127,8 @@ 
 	 */
 	public function get_collection_params() {
 		$params            = array();
-		$params['context'] = $this->get_context_param( array( 'default' => 'view' ) );
-		$params['match']   = array(
+		$params[ 'context' ] = $this->get_context_param( array( 'default' => 'view' ) );
+		$params[ 'match' ]   = array(
 			'description'       => __( 'A partial IP address can be passed and matching results will be returned.', 'woocommerce' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',

src/Controllers/Version4/Data/Countries.php

Spacing +5 added lines, -5 removed lines

@@ -87,13 +87,13 @@ 
 		$local_states = array();
 		if ( isset( $states[ $country_code ] ) ) {
 			foreach ( $states[ $country_code ] as $state_code => $state_name ) {
-				$local_states[] = array(
+				$local_states[ ] = array(
 					'code' => $state_code,
 					'name' => $state_name,
 				);
 			}
 		}
-		$country['states'] = $local_states;
+		$country[ 'states' ] = $local_states;
 		return $country;
 	}
 
@@ -111,7 +111,7 @@ 
 		foreach ( array_keys( $countries ) as $country_code ) {
 			$country  = $this->get_country( $country_code, $request );
 			$response = $this->prepare_item_for_response( $country, $request );
-			$data[]   = $this->prepare_response_for_collection( $response );
+			$data[ ]   = $this->prepare_response_for_collection( $response );
 		}
 
 		return rest_ensure_response( $data );
@@ -125,7 +125,7 @@ 
 	 * @return \WP_Error\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$data = $this->get_country( strtoupper( $request['location'] ), $request );
+		$data = $this->get_country( strtoupper( $request[ 'location' ] ), $request );
 		if ( empty( $data ) ) {
 			return new \WP_Error( 'woocommerce_rest_data_invalid_location', __( 'There are no locations matching these parameters.', 'woocommerce' ), array( 'status' => 404 ) );
 		}
@@ -166,7 +166,7 @@ 
 	 * @return array Links for the given country.
 	 */
 	protected function prepare_links( $item ) {
-		$country_code = strtolower( $item['code'] );
+		$country_code = strtolower( $item[ 'code' ] );
 		$links        = array(
 			'self'       => array(
 				'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $country_code ) ),

src/Controllers/Version4/Data/Continents.php

Spacing +9 added lines, -9 removed lines

@@ -86,11 +86,11 @@ 
 
 		$continent = array(
 			'code' => $continent_code,
-			'name' => $continent_list['name'],
+			'name' => $continent_list[ 'name' ],
 		);
 
 		$local_countries = array();
-		foreach ( $continent_list['countries'] as $country_code ) {
+		foreach ( $continent_list[ 'countries' ] as $country_code ) {
 			if ( isset( $countries[ $country_code ] ) ) {
 				$country = array(
 					'code' => $country_code,
@@ -119,13 +119,13 @@ 
 				$local_states = array();
 				if ( isset( $states[ $country_code ] ) ) {
 					foreach ( $states[ $country_code ] as $state_code => $state_name ) {
-						$local_states[] = array(
+						$local_states[ ] = array(
 							'code' => $state_code,
 							'name' => $state_name,
 						);
 					}
 				}
-				$country['states'] = $local_states;
+				$country[ 'states' ] = $local_states;
 
 				// Allow only desired keys (e.g. filter out tax rates).
 				$allowed = array(
@@ -142,11 +142,11 @@ 
 				);
 				$country = array_intersect_key( $country, array_flip( $allowed ) );
 
-				$local_countries[] = $country;
+				$local_countries[ ] = $country;
 			}
 		}
 
-		$continent['countries'] = $local_countries;
+		$continent[ 'countries' ] = $local_countries;
 		return $continent;
 	}
 
@@ -164,7 +164,7 @@ 
 		foreach ( array_keys( $continents ) as $continent_code ) {
 			$continent = $this->get_continent( $continent_code, $request );
 			$response  = $this->prepare_item_for_response( $continent, $request );
-			$data[]    = $this->prepare_response_for_collection( $response );
+			$data[ ]    = $this->prepare_response_for_collection( $response );
 		}
 
 		return rest_ensure_response( $data );
@@ -178,7 +178,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$data = $this->get_continent( strtoupper( $request['location'] ), $request );
+		$data = $this->get_continent( strtoupper( $request[ 'location' ] ), $request );
 		if ( empty( $data ) ) {
 			return new \WP_Error( 'woocommerce_rest_data_invalid_location', __( 'There are no locations matching these parameters.', 'woocommerce' ), array( 'status' => 404 ) );
 		}
@@ -219,7 +219,7 @@ 
 	 * @return array Links for the given continent.
 	 */
 	protected function prepare_links( $item ) {
-		$continent_code = strtolower( $item['code'] );
+		$continent_code = strtolower( $item[ 'code' ] );
 		$links          = array(
 			'self'       => array(
 				'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $continent_code ) ),

src/Controllers/Version4/NetworkOrders.php

Spacing +12 added lines, -12 removed lines

@@ -46,31 +46,31 @@ 
 	public function get_public_item_schema() {
 		$schema = parent::get_public_item_schema();
 
-		$schema['properties']['blog']              = array(
+		$schema[ 'properties' ][ 'blog' ] = array(
 			'description' => __( 'Blog id of the record on the multisite.', 'woocommerce' ),
 			'type'        => 'integer',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['edit_url']          = array(
+		$schema[ 'properties' ][ 'edit_url' ] = array(
 			'description' => __( 'URL to edit the order', 'woocommerce' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['customer'][]        = array(
+		$schema[ 'properties' ][ 'customer' ][ ] = array(
 			'description' => __( 'Name of the customer for the order', 'woocommerce' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['status_name'][]     = array(
+		$schema[ 'properties' ][ 'status_name' ][ ] = array(
 			'description' => __( 'Order Status', 'woocommerce' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['formatted_total'][] = array(
+		$schema[ 'properties' ][ 'formatted_total' ][ ] = array(
 			'description' => __( 'Order total formatted for locale', 'woocommerce' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
@@ -133,14 +133,14 @@ 
 		remove_filter( 'woocommerce_rest_orders_prepare_object_query', array( $this, 'network_orders_filter_args' ) );
 
 		foreach ( $items->data as &$current_order ) {
-			$order = wc_get_order( $current_order['id'] );
+			$order = wc_get_order( $current_order[ 'id' ] );
 
-			$current_order['blog']     = get_blog_details( get_current_blog_id() );
-			$current_order['edit_url'] = get_admin_url( $blog_id, 'post.php?post=' . absint( $order->get_id() ) . '&action=edit' );
+			$current_order[ 'blog' ]     = get_blog_details( get_current_blog_id() );
+			$current_order[ 'edit_url' ] = get_admin_url( $blog_id, 'post.php?post=' . absint( $order->get_id() ) . '&action=edit' );
 			/* translators: 1: first name 2: last name */
-			$current_order['customer']        = trim( sprintf( _x( '%1$s %2$s', 'full name', 'woocommerce' ), $order->get_billing_first_name(), $order->get_billing_last_name() ) );
-			$current_order['status_name']     = wc_get_order_status_name( $order->get_status() );
-			$current_order['formatted_total'] = $order->get_formatted_order_total();
+			$current_order[ 'customer' ]        = trim( sprintf( _x( '%1$s %2$s', 'full name', 'woocommerce' ), $order->get_billing_first_name(), $order->get_billing_last_name() ) );
+			$current_order[ 'status_name' ]     = wc_get_order_status_name( $order->get_status() );
+			$current_order[ 'formatted_total' ] = $order->get_formatted_order_total();
 		}
 
 		restore_current_blog();
@@ -156,7 +156,7 @@ 
 	 * @return array
 	 */
 	public function network_orders_filter_args( $args ) {
-		$args['post_status'] = array(
+		$args[ 'post_status' ] = array(
 			'wc-on-hold',
 			'wc-processing',
 		);

src/Controllers/Version4/AbstractObjectsController.php

Spacing +63 added lines, -63 removed lines

@@ -60,7 +60,7 @@ 
 	 * @return \WP_Error|boolean
 	 */
 	public function get_item_permissions_check( $request ) {
-		$object = $this->get_object( (int) $request['id'] );
+		$object = $this->get_object( (int) $request[ 'id' ] );
 
 		if ( $object && 0 !== $object->get_id() && ! wc_rest_check_post_permissions( $this->post_type, 'read', $object->get_id() ) ) {
 			return new \WP_Error( 'woocommerce_rest_cannot_view', __( 'Sorry, you cannot view this resource.', 'woocommerce' ), array( 'status' => rest_authorization_required_code() ) );
@@ -90,7 +90,7 @@ 
 	 * @return \WP_Error|boolean
 	 */
 	public function update_item_permissions_check( $request ) {
-		$object = $this->get_object( (int) $request['id'] );
+		$object = $this->get_object( (int) $request[ 'id' ] );
 
 		if ( $object && 0 !== $object->get_id() && ! wc_rest_check_post_permissions( $this->post_type, 'edit', $object->get_id() ) ) {
 			return new \WP_Error( 'woocommerce_rest_cannot_edit', __( 'Sorry, you are not allowed to edit this resource.', 'woocommerce' ), array( 'status' => rest_authorization_required_code() ) );
@@ -106,7 +106,7 @@ 
 	 * @return bool|\WP_Error
 	 */
 	public function delete_item_permissions_check( $request ) {
-		$object = $this->get_object( (int) $request['id'] );
+		$object = $this->get_object( (int) $request[ 'id' ] );
 
 		if ( $object && 0 !== $object->get_id() && ! wc_rest_check_post_permissions( $this->post_type, 'delete', $object->get_id() ) ) {
 			return new \WP_Error( 'woocommerce_rest_cannot_delete', __( 'Sorry, you are not allowed to delete this resource.', 'woocommerce' ), array( 'status' => rest_authorization_required_code() ) );
@@ -173,7 +173,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$object = $this->get_object( (int) $request['id'] );
+		$object = $this->get_object( (int) $request[ 'id' ] );
 
 		if ( ! $object || 0 === $object->get_id() ) {
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_id", __( 'Invalid ID.', 'woocommerce' ), array( 'status' => 404 ) );
@@ -218,7 +218,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function create_item( $request ) {
-		if ( ! empty( $request['id'] ) ) {
+		if ( ! empty( $request[ 'id' ] ) ) {
 			/* translators: %s: post type */
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_exists", sprintf( __( 'Cannot create existing %s.', 'woocommerce' ), $this->post_type ), array( 'status' => 400 ) );
 		}
@@ -264,7 +264,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function update_item( $request ) {
-		$object = $this->get_object( (int) $request['id'] );
+		$object = $this->get_object( (int) $request[ 'id' ] );
 
 		if ( ! $object || 0 === $object->get_id() ) {
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_id", __( 'Invalid ID.', 'woocommerce' ), array( 'status' => 404 ) );
@@ -307,42 +307,42 @@ 
 	 */
 	protected function prepare_objects_query( $request ) {
 		$args                        = array();
-		$args['offset']              = $request['offset'];
-		$args['order']               = $request['order'];
-		$args['orderby']             = $request['orderby'];
-		$args['paged']               = $request['page'];
-		$args['post__in']            = $request['include'];
-		$args['post__not_in']        = $request['exclude'];
-		$args['posts_per_page']      = $request['per_page'];
-		$args['name']                = $request['slug'];
-		$args['post_parent__in']     = $request['parent'];
-		$args['post_parent__not_in'] = $request['parent_exclude'];
-		$args['s']                   = $request['search'];
-		$args['fields']              = 'ids';
-
-		if ( 'date' === $args['orderby'] ) {
-			$args['orderby'] = 'date ID';
+		$args[ 'offset' ]              = $request[ 'offset' ];
+		$args[ 'order' ]               = $request[ 'order' ];
+		$args[ 'orderby' ]             = $request[ 'orderby' ];
+		$args[ 'paged' ]               = $request[ 'page' ];
+		$args[ 'post__in' ]            = $request[ 'include' ];
+		$args[ 'post__not_in' ]        = $request[ 'exclude' ];
+		$args[ 'posts_per_page' ]      = $request[ 'per_page' ];
+		$args[ 'name' ]                = $request[ 'slug' ];
+		$args[ 'post_parent__in' ]     = $request[ 'parent' ];
+		$args[ 'post_parent__not_in' ] = $request[ 'parent_exclude' ];
+		$args[ 's' ]                   = $request[ 'search' ];
+		$args[ 'fields' ]              = 'ids';
+
+		if ( 'date' === $args[ 'orderby' ] ) {
+			$args[ 'orderby' ] = 'date ID';
 		}
 
-		$args['date_query'] = array();
+		$args[ 'date_query' ] = array();
 
 		// Set before into date query. Date query must be specified as an array of an array.
-		if ( isset( $request['before'] ) ) {
-			$args['date_query'][0]['before'] = $request['before'];
+		if ( isset( $request[ 'before' ] ) ) {
+			$args[ 'date_query' ][ 0 ][ 'before' ] = $request[ 'before' ];
 		}
 
 		// Set after into date query. Date query must be specified as an array of an array.
-		if ( isset( $request['after'] ) ) {
-			$args['date_query'][0]['after'] = $request['after'];
+		if ( isset( $request[ 'after' ] ) ) {
+			$args[ 'date_query' ][ 0 ][ 'after' ] = $request[ 'after' ];
 		}
 
 		// Set date query colummn. Defaults to post_date.
-		if ( isset( $request['date_column'] ) && ! empty( $args['date_query'][0] ) ) {
-			$args['date_query'][0]['column'] = 'post_' . $request['date_column'];
+		if ( isset( $request[ 'date_column' ] ) && ! empty( $args[ 'date_query' ][ 0 ] ) ) {
+			$args[ 'date_query' ][ 0 ][ 'column' ] = 'post_' . $request[ 'date_column' ];
 		}
 
 		// Force the post_type argument, since it's not a user input variable.
-		$args['post_type'] = $this->post_type;
+		$args[ 'post_type' ] = $this->post_type;
 
 		/**
 		 * Filter the query arguments for a request.
@@ -372,7 +372,7 @@ 
 		$total_posts = $query->found_posts;
 		if ( $total_posts < 1 ) {
 			// Out-of-bounds, run the query again without LIMIT for total count.
-			unset( $query_args['paged'] );
+			unset( $query_args[ 'paged' ] );
 			$count_query = new \WP_Query();
 			$count_query->query( $query_args );
 			$total_posts = $count_query->found_posts;
@@ -381,7 +381,7 @@ 
 		return array(
 			'objects' => array_map( array( $this, 'get_object' ), $result ),
 			'total'   => (int) $total_posts,
-			'pages'   => (int) ceil( $total_posts / (int) $query->query_vars['posts_per_page'] ),
+			'pages'   => (int) ceil( $total_posts / (int) $query->query_vars[ 'posts_per_page' ] ),
 		);
 	}
 
@@ -396,20 +396,20 @@ 
 		$query_results = $this->get_objects( $query_args );
 
 		$objects = array();
-		foreach ( $query_results['objects'] as $object ) {
+		foreach ( $query_results[ 'objects' ] as $object ) {
 			if ( ! wc_rest_check_post_permissions( $this->post_type, 'read', $object->get_id() ) ) {
 				continue;
 			}
 
 			$data      = $this->prepare_object_for_response( $object, $request );
-			$objects[] = $this->prepare_response_for_collection( $data );
+			$objects[ ] = $this->prepare_response_for_collection( $data );
 		}
 
-		$page      = (int) $query_args['paged'];
-		$max_pages = $query_results['pages'];
+		$page      = (int) $query_args[ 'paged' ];
+		$max_pages = $query_results[ 'pages' ];
 
 		$response = rest_ensure_response( $objects );
-		$response->header( 'X-WP-Total', $query_results['total'] );
+		$response->header( 'X-WP-Total', $query_results[ 'total' ] );
 		$response->header( 'X-WP-TotalPages', (int) $max_pages );
 
 		$base          = $this->rest_base;
@@ -419,8 +419,8 @@ 
 			preg_match( '/\(\?P<[^>]+>.*\)/', $base, $attrib_names, PREG_OFFSET_CAPTURE );
 			foreach ( $attrib_names as $attrib_name_match ) {
 				$beginning_offset = strlen( $attrib_prefix );
-				$attrib_name_end  = strpos( $attrib_name_match[0], '>', $attrib_name_match[1] );
-				$attrib_name      = substr( $attrib_name_match[0], $beginning_offset, $attrib_name_end - $beginning_offset );
+				$attrib_name_end  = strpos( $attrib_name_match[ 0 ], '>', $attrib_name_match[ 1 ] );
+				$attrib_name      = substr( $attrib_name_match[ 0 ], $beginning_offset, $attrib_name_end - $beginning_offset );
 				if ( isset( $request[ $attrib_name ] ) ) {
 					$base = str_replace( "(?P<$attrib_name>[\d]+)", $request[ $attrib_name ], $base );
 				}
@@ -452,8 +452,8 @@ 
 	 * @return \WP_REST_Response|\WP_Error
 	 */
 	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() ) {
@@ -563,10 +563,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,
@@ -575,7 +575,7 @@ 
 			'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,
@@ -585,28 +585,28 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['search'] = array(
+		$params[ 'search' ] = array(
 			'description'       => __( 'Limit results to those matching a string.', 'woocommerce' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_text_field',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['after'] = array(
+		$params[ 'after' ] = array(
 			'description'       => __( 'Limit response to resources created after a given ISO8601 compliant date.', 'woocommerce' ),
 			'type'              => 'string',
 			'format'            => 'date-time',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['before'] = array(
+		$params[ 'before' ] = array(
 			'description'       => __( 'Limit response to resources created before a given ISO8601 compliant date.', 'woocommerce' ),
 			'type'              => 'string',
 			'format'            => 'date-time',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['date_column'] = array(
+		$params[ 'date_column' ] = array(
 			'description'       => __( 'When limiting response using after/before, which date column to compare against.', 'woocommerce' ),
 			'type'              => 'string',
 			'default'           => 'date',
@@ -619,14 +619,14 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['modified_before'] = array(
+		$params[ 'modified_before' ] = array(
 			'description'       => __( 'Limit response to resources modified before a given ISO8601 compliant date.', 'woocommerce' ),
 			'type'              => 'string',
 			'format'            => 'date-time',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['exclude'] = array(
+		$params[ 'exclude' ] = array(
 			'description'       => __( 'Ensure result set excludes specific IDs.', 'woocommerce' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -636,7 +636,7 @@ 
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
 
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to specific ids.', 'woocommerce' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -646,14 +646,14 @@ 
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
 
-		$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(
 			'description'       => __( 'Order sort attribute ascending or descending.', 'woocommerce' ),
 			'type'              => 'string',
 			'default'           => 'desc',
@@ -661,7 +661,7 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['orderby'] = array(
+		$params[ 'orderby' ] = array(
 			'description'       => __( 'Sort collection by object attribute.', 'woocommerce' ),
 			'type'              => 'string',
 			'default'           => 'date',
@@ -677,7 +677,7 @@ 
 		);
 
 		if ( $this->hierarchical ) {
-			$params['parent'] = array(
+			$params[ 'parent' ] = array(
 				'description'       => __( 'Limit result set to those of particular parent IDs.', 'woocommerce' ),
 				'type'              => 'array',
 				'items'             => array(
@@ -687,7 +687,7 @@ 
 				'default'           => array(),
 			);
 
-			$params['parent_exclude'] = array(
+			$params[ 'parent_exclude' ] = array(
 				'description'       => __( 'Limit result set to all items except those of a particular parent ID.', 'woocommerce' ),
 				'type'              => 'array',
 				'items'             => array(
@@ -738,14 +738,14 @@ 
 			}
 		}
 
-		$query_args['ignore_sticky_posts'] = true;
+		$query_args[ 'ignore_sticky_posts' ] = true;
 
-		if ( 'include' === $query_args['orderby'] ) {
-			$query_args['orderby'] = 'post__in';
-		} elseif ( 'id' === $query_args['orderby'] ) {
-			$query_args['orderby'] = 'ID'; // ID must be capitalized.
-		} elseif ( 'slug' === $query_args['orderby'] ) {
-			$query_args['orderby'] = 'name';
+		if ( 'include' === $query_args[ 'orderby' ] ) {
+			$query_args[ 'orderby' ] = 'post__in';
+		} elseif ( 'id' === $query_args[ 'orderby' ] ) {
+			$query_args[ 'orderby' ] = 'ID'; // ID must be capitalized.
+		} elseif ( 'slug' === $query_args[ 'orderby' ] ) {
+			$query_args[ 'orderby' ] = 'name';
 		}
 
 		return $query_args;

src/Controllers/Version4/AbstractTermsContoller.php

Spacing +72 added lines, -72 removed lines

@@ -248,7 +248,7 @@ 
 		}
 
 		// Check permissions for a single term.
-		$id = intval( $request['id'] );
+		$id = intval( $request[ 'id' ] );
 		if ( $id ) {
 			$term = get_term( $id, $taxonomy );
 
@@ -271,32 +271,32 @@ 
 	public function get_items( $request ) {
 		$taxonomy      = $this->get_taxonomy( $request );
 		$prepared_args = array(
-			'exclude'    => $request['exclude'],
-			'include'    => $request['include'],
-			'order'      => $request['order'],
-			'orderby'    => $request['orderby'],
-			'product'    => $request['product'],
-			'hide_empty' => $request['hide_empty'],
-			'number'     => $request['per_page'],
-			'search'     => $request['search'],
-			'slug'       => $request['slug'],
+			'exclude'    => $request[ 'exclude' ],
+			'include'    => $request[ 'include' ],
+			'order'      => $request[ 'order' ],
+			'orderby'    => $request[ 'orderby' ],
+			'product'    => $request[ 'product' ],
+			'hide_empty' => $request[ 'hide_empty' ],
+			'number'     => $request[ 'per_page' ],
+			'search'     => $request[ 'search' ],
+			'slug'       => $request[ 'slug' ],
 		);
 
-		if ( ! empty( $request['offset'] ) ) {
-			$prepared_args['offset'] = $request['offset'];
+		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' ];
 		}
 
 		$taxonomy_obj = get_taxonomy( $taxonomy );
 
-		if ( $taxonomy_obj->hierarchical && isset( $request['parent'] ) ) {
-			if ( 0 === $request['parent'] ) {
+		if ( $taxonomy_obj->hierarchical && isset( $request[ 'parent' ] ) ) {
+			if ( 0 === $request[ 'parent' ] ) {
 				// Only query top-level terms.
-				$prepared_args['parent'] = 0;
+				$prepared_args[ 'parent' ] = 0;
 			} else {
-				if ( $request['parent'] ) {
-					$prepared_args['parent'] = $request['parent'];
+				if ( $request[ 'parent' ] ) {
+					$prepared_args[ 'parent' ] = $request[ 'parent' ];
 				}
 			}
 		}
@@ -315,20 +315,20 @@ 
 		 */
 		$prepared_args = apply_filters( "woocommerce_rest_{$taxonomy}_query", $prepared_args, $request );
 
-		if ( ! empty( $prepared_args['product'] ) ) {
+		if ( ! empty( $prepared_args[ 'product' ] ) ) {
 			$query_result = $this->get_terms_for_product( $prepared_args, $request );
 			$total_terms  = $this->total_terms;
 		} else {
 			$query_result = get_terms( $taxonomy, $prepared_args );
 
 			$count_args = $prepared_args;
-			unset( $count_args['number'] );
-			unset( $count_args['offset'] );
+			unset( $count_args[ 'number' ] );
+			unset( $count_args[ 'offset' ] );
 			$total_terms = wp_count_terms( $taxonomy, $count_args );
 
 			// Ensure we don't return results when offset is out of bounds.
 			// See https://core.trac.wordpress.org/ticket/35935.
-			if ( $prepared_args['offset'] && $prepared_args['offset'] >= $total_terms ) {
+			if ( $prepared_args[ 'offset' ] && $prepared_args[ 'offset' ] >= $total_terms ) {
 				$query_result = array();
 			}
 
@@ -340,20 +340,20 @@ 
 		$response = array();
 		foreach ( $query_result as $term ) {
 			$data       = $this->prepare_item_for_response( $term, $request );
-			$response[] = $this->prepare_response_for_collection( $data );
+			$response[ ] = $this->prepare_response_for_collection( $data );
 		}
 
 		$response = rest_ensure_response( $response );
 
 		// 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 );
 
 		$response->header( 'X-WP-Total', (int) $total_terms );
 		$max_pages = ceil( $total_terms / $per_page );
 		$response->header( 'X-WP-TotalPages', (int) $max_pages );
 
-		$base  = str_replace( '(?P<attribute_id>[\d]+)', $request['attribute_id'], $this->rest_base );
+		$base = str_replace( '(?P<attribute_id>[\d]+)', $request[ 'attribute_id' ], $this->rest_base );
 		$base = add_query_arg( $request->get_query_params(), rest_url( '/' . $this->namespace . '/' . $base ) );
 		if ( $page > 1 ) {
 			$prev_page = $page - 1;
@@ -380,21 +380,21 @@ 
 	 */
 	public function create_item( $request ) {
 		$taxonomy = $this->get_taxonomy( $request );
-		$name     = $request['name'];
+		$name     = $request[ 'name' ];
 		$args     = array();
 		$schema   = $this->get_item_schema();
 
-		if ( ! empty( $schema['properties']['description'] ) && isset( $request['description'] ) ) {
-			$args['description'] = $request['description'];
+		if ( ! empty( $schema[ 'properties' ][ 'description' ] ) && isset( $request[ 'description' ] ) ) {
+			$args[ 'description' ] = $request[ 'description' ];
 		}
-		if ( isset( $request['slug'] ) ) {
-			$args['slug'] = $request['slug'];
+		if ( isset( $request[ 'slug' ] ) ) {
+			$args[ 'slug' ] = $request[ 'slug' ];
 		}
-		if ( isset( $request['parent'] ) ) {
+		if ( isset( $request[ 'parent' ] ) ) {
 			if ( ! is_taxonomy_hierarchical( $taxonomy ) ) {
 				return new \WP_Error( 'woocommerce_rest_taxonomy_not_hierarchical', __( 'Can not set resource parent, taxonomy is not hierarchical.', 'woocommerce' ), array( 'status' => 400 ) );
 			}
-			$args['parent'] = $request['parent'];
+			$args[ 'parent' ] = $request[ 'parent' ];
 		}
 
 		$term = wp_insert_term( $name, $taxonomy, $args );
@@ -405,13 +405,13 @@ 
 			// give them the identifier they can actually use.
 			$term_id = $term->get_error_data( 'term_exists' );
 			if ( $term_id ) {
-				$error_data['resource_id'] = $term_id;
+				$error_data[ 'resource_id' ] = $term_id;
 			}
 
 			return new \WP_Error( $term->get_error_code(), $term->get_error_message(), $error_data );
 		}
 
-		$term = get_term( $term['term_id'], $taxonomy );
+		$term = get_term( $term[ 'term_id' ], $taxonomy );
 
 		$this->update_additional_fields_for_object( $term, $request );
 
@@ -438,8 +438,8 @@ 
 		$response->set_status( 201 );
 
 		$base = '/' . $this->namespace . '/' . $this->rest_base;
-		if ( ! empty( $request['attribute_id'] ) ) {
-			$base = str_replace( '(?P<attribute_id>[\d]+)', (int) $request['attribute_id'], $base );
+		if ( ! empty( $request[ 'attribute_id' ] ) ) {
+			$base = str_replace( '(?P<attribute_id>[\d]+)', (int) $request[ 'attribute_id' ], $base );
 		}
 
 		$response->header( 'Location', rest_url( $base . '/' . $term->term_id ) );
@@ -455,7 +455,7 @@ 
 	 */
 	public function get_item( $request ) {
 		$taxonomy = $this->get_taxonomy( $request );
-		$term     = get_term( (int) $request['id'], $taxonomy );
+		$term     = get_term( (int) $request[ 'id' ], $taxonomy );
 
 		if ( is_wp_error( $term ) ) {
 			return $term;
@@ -474,24 +474,24 @@ 
 	 */
 	public function update_item( $request ) {
 		$taxonomy      = $this->get_taxonomy( $request );
-		$term          = get_term( (int) $request['id'], $taxonomy );
+		$term          = get_term( (int) $request[ 'id' ], $taxonomy );
 		$schema        = $this->get_item_schema();
 		$prepared_args = array();
 
-		if ( isset( $request['name'] ) ) {
-			$prepared_args['name'] = $request['name'];
+		if ( isset( $request[ 'name' ] ) ) {
+			$prepared_args[ 'name' ] = $request[ 'name' ];
 		}
-		if ( ! empty( $schema['properties']['description'] ) && isset( $request['description'] ) ) {
-			$prepared_args['description'] = $request['description'];
+		if ( ! empty( $schema[ 'properties' ][ 'description' ] ) && isset( $request[ 'description' ] ) ) {
+			$prepared_args[ 'description' ] = $request[ 'description' ];
 		}
-		if ( isset( $request['slug'] ) ) {
-			$prepared_args['slug'] = $request['slug'];
+		if ( isset( $request[ 'slug' ] ) ) {
+			$prepared_args[ 'slug' ] = $request[ 'slug' ];
 		}
-		if ( isset( $request['parent'] ) ) {
+		if ( isset( $request[ 'parent' ] ) ) {
 			if ( ! is_taxonomy_hierarchical( $taxonomy ) ) {
 				return new \WP_Error( 'woocommerce_rest_taxonomy_not_hierarchical', __( 'Can not set resource parent, taxonomy is not hierarchical.', 'woocommerce' ), array( 'status' => 400 ) );
 			}
-			$prepared_args['parent'] = $request['parent'];
+			$prepared_args[ 'parent' ] = $request[ 'parent' ];
 		}
 
 		// Only update the term if we haz something to update.
@@ -502,7 +502,7 @@ 
 			}
 		}
 
-		$term = get_term( (int) $request['id'], $taxonomy );
+		$term = get_term( (int) $request[ 'id' ], $taxonomy );
 
 		$this->update_additional_fields_for_object( $term, $request );
 
@@ -534,19 +534,19 @@ 
 	 */
 	public function delete_item( $request ) {
 		$taxonomy = $this->get_taxonomy( $request );
-		$force    = isset( $request['force'] ) ? (bool) $request['force'] : false;
+		$force    = isset( $request[ 'force' ] ) ? (bool) $request[ 'force' ] : false;
 
 		// We don't support trashing for this type, error out.
 		if ( ! $force ) {
 			return new \WP_Error( 'woocommerce_rest_trash_not_supported', __( 'Resource does not support trashing.', 'woocommerce' ), array( 'status' => 501 ) );
 		}
 
-		$term = get_term( (int) $request['id'], $taxonomy );
+		$term = get_term( (int) $request[ 'id' ], $taxonomy );
 		// Get default category id.
 		$default_category_id = absint( get_option( 'default_product_cat', 0 ) );
 
 		// Prevent deleting the default product category.
-		if ( $default_category_id === (int) $request['id'] ) {
+		if ( $default_category_id === (int) $request[ 'id' ] ) {
 			return new \WP_Error( 'woocommerce_rest_cannot_delete', __( 'Default product category cannot be deleted.', 'woocommerce' ), array( 'status' => 500 ) );
 		}
 
@@ -580,8 +580,8 @@ 
 	protected function prepare_links( $term, $request ) {
 		$base = '/' . $this->namespace . '/' . $this->rest_base;
 
-		if ( ! empty( $request['attribute_id'] ) ) {
-			$base = str_replace( '(?P<attribute_id>[\d]+)', (int) $request['attribute_id'], $base );
+		if ( ! empty( $request[ 'attribute_id' ] ) ) {
+			$base = str_replace( '(?P<attribute_id>[\d]+)', (int) $request[ 'attribute_id' ], $base );
 		}
 
 		$links = array(
@@ -596,7 +596,7 @@ 
 		if ( $term->parent ) {
 			$parent_term = get_term( (int) $term->parent, $term->taxonomy );
 			if ( $parent_term ) {
-				$links['up'] = array(
+				$links[ 'up' ] = array(
 					'href' => rest_url( trailingslashit( $base ) . $parent_term->term_id ),
 				);
 			}
@@ -631,7 +631,7 @@ 
 	protected function get_terms_for_product( $prepared_args, $request ) {
 		$taxonomy = $this->get_taxonomy( $request );
 
-		$query_result = get_the_terms( $prepared_args['product'], $taxonomy );
+		$query_result = get_the_terms( $prepared_args[ 'product' ], $taxonomy );
 		if ( empty( $query_result ) ) {
 			$this->total_terms = 0;
 			return array();
@@ -639,8 +639,8 @@ 
 
 		// get_items() verifies that we don't have `include` set, and default.
 		// ordering is by `name`.
-		if ( ! in_array( $prepared_args['orderby'], array( 'name', 'none', 'include' ), true ) ) {
-			switch ( $prepared_args['orderby'] ) {
+		if ( ! in_array( $prepared_args[ 'orderby' ], array( 'name', 'none', 'include' ), true ) ) {
+			switch ( $prepared_args[ 'orderby' ] ) {
 				case 'id':
 					$this->sort_column = 'term_id';
 					break;
@@ -648,18 +648,18 @@ 
 				case 'term_group':
 				case 'description':
 				case 'count':
-					$this->sort_column = $prepared_args['orderby'];
+					$this->sort_column = $prepared_args[ 'orderby' ];
 					break;
 			}
 			usort( $query_result, array( $this, 'compare_terms' ) );
 		}
-		if ( strtolower( $prepared_args['order'] ) !== 'asc' ) {
+		if ( strtolower( $prepared_args[ 'order' ] ) !== 'asc' ) {
 			$query_result = array_reverse( $query_result );
 		}
 
 		// Pagination.
 		$this->total_terms = count( $query_result );
-		$query_result      = array_slice( $query_result, $prepared_args['offset'], $prepared_args['number'] );
+		$query_result      = array_slice( $query_result, $prepared_args[ 'offset' ], $prepared_args[ 'number' ] );
 
 		return $query_result;
 	}
@@ -700,9 +700,9 @@ 
 			$taxonomy->hierarchical = true;
 		}
 
-		$params['context']['default'] = 'view';
+		$params[ 'context' ][ 'default' ] = 'view';
 
-		$params['exclude'] = array(
+		$params[ 'exclude' ] = array(
 			'description'       => __( 'Ensure result set excludes specific IDs.', 'woocommerce' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -711,7 +711,7 @@ 
 			'default'           => array(),
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to specific ids.', 'woocommerce' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -721,14 +721,14 @@ 
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
 		if ( ! $taxonomy->hierarchical ) {
-			$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(
 			'description'       => __( 'Order sort attribute ascending or descending.', 'woocommerce' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_key',
@@ -739,7 +739,7 @@ 
 			),
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['orderby']    = array(
+		$params[ 'orderby' ] = array(
 			'description'       => __( 'Sort collection by resource attribute.', 'woocommerce' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_key',
@@ -755,27 +755,27 @@ 
 			),
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['hide_empty'] = array(
+		$params[ 'hide_empty' ] = array(
 			'description'       => __( 'Whether to hide resources not assigned to any products.', 'woocommerce' ),
 			'type'              => 'boolean',
 			'default'           => false,
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 		if ( $taxonomy->hierarchical ) {
-			$params['parent'] = array(
+			$params[ 'parent' ] = array(
 				'description'       => __( 'Limit result set to resources assigned to a specific parent.', 'woocommerce' ),
 				'type'              => 'integer',
 				'sanitize_callback' => 'absint',
 				'validate_callback' => 'rest_validate_request_arg',
 			);
 		}
-		$params['product'] = array(
+		$params[ 'product' ] = array(
 			'description'       => __( 'Limit result set to resources assigned to a specific product.', 'woocommerce' ),
 			'type'              => 'integer',
 			'default'           => null,
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['slug']    = array(
+		$params[ 'slug' ] = array(
 			'description'       => __( 'Limit result set to resources with a specific slug.', 'woocommerce' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',
@@ -797,8 +797,8 @@ 
 			return $this->taxonomy;
 		}
 
-		if ( ! empty( $request['attribute_id'] ) ) {
-			$taxonomy = wc_attribute_taxonomy_name_by_id( (int) $request['attribute_id'] );
+		if ( ! empty( $request[ 'attribute_id' ] ) ) {
+			$taxonomy = wc_attribute_taxonomy_name_by_id( (int) $request[ 'attribute_id' ] );
 
 			$this->taxonomy = $taxonomy;
 		}