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-rest-api' ), 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-rest-api' ), 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-rest-api' ), array( 'status' => 400 ) );

src/Controllers/Version4/AbstractTermsContoller.php

Spacing +69 added lines, -69 removed lines

@@ -132,32 +132,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' ];
 				}
 			}
 		}
@@ -176,20 +176,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();
 			}
 
@@ -201,11 +201,11 @@ 
 		$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 );
 		}
 
 		// Store pagination values for headers then unset for count query.
-		$per_page  = (int) $prepared_args['number'];
+		$per_page  = (int) $prepared_args[ 'number' ];
 		$max_pages = ceil( $total_terms / $per_page );
 
 		$response = rest_ensure_response( $response );
@@ -222,21 +222,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-rest-api' ), array( 'status' => 400 ) );
 			}
-			$args['parent'] = $request['parent'];
+			$args[ 'parent' ] = $request[ 'parent' ];
 		}
 
 		$term = wp_insert_term( $name, $taxonomy, $args );
@@ -247,13 +247,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 );
 
@@ -280,8 +280,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 ) );
@@ -297,7 +297,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;
@@ -316,24 +316,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-rest-api' ), array( 'status' => 400 ) );
 			}
-			$prepared_args['parent'] = $request['parent'];
+			$prepared_args[ 'parent' ] = $request[ 'parent' ];
 		}
 
 		// Only update the term if we haz something to update.
@@ -344,7 +344,7 @@ 
 			}
 		}
 
-		$term = get_term( (int) $request['id'], $taxonomy );
+		$term = get_term( (int) $request[ 'id' ], $taxonomy );
 
 		$this->update_additional_fields_for_object( $term, $request );
 
@@ -376,19 +376,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-rest-api' ), 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-rest-api' ), array( 'status' => 500 ) );
 		}
 
@@ -422,8 +422,8 @@ 
 	protected function prepare_links( $item, $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(
@@ -438,7 +438,7 @@ 
 		if ( $item->parent ) {
 			$parent_term = get_term( (int) $item->parent, $item->taxonomy );
 			if ( $parent_term ) {
-				$links['up'] = array(
+				$links[ 'up' ] = array(
 					'href' => rest_url( trailingslashit( $base ) . $parent_term->term_id ),
 				);
 			}
@@ -473,7 +473,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();
@@ -481,8 +481,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;
@@ -490,18 +490,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;
 	}
@@ -542,9 +542,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-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -553,7 +553,7 @@ 
 			'default'           => array(),
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to specific ids.', 'woocommerce-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -563,14 +563,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-rest-api' ),
 				'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-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_key',
@@ -581,7 +581,7 @@ 
 			),
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['orderby']    = array(
+		$params[ 'orderby' ] = array(
 			'description'       => __( 'Sort collection by resource attribute.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_key',
@@ -597,27 +597,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-rest-api' ),
 			'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-rest-api' ),
 				'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-rest-api' ),
 			'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-rest-api' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',
@@ -639,8 +639,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;
 		}

src/Controllers/Version4/Webhooks.php

Spacing +50 added lines, -50 removed lines

@@ -140,18 +140,18 @@ 
 	 */
 	public function get_items( $request ) {
 		$args            = array();
-		$args['order']   = $request['order'];
-		$args['orderby'] = $request['orderby'];
-		$args['status']  = 'all' === $request['status'] ? '' : $request['status'];
-		$args['include'] = implode( ',', $request['include'] );
-		$args['exclude'] = implode( ',', $request['exclude'] );
-		$args['limit']   = $request['per_page'];
-		$args['search']  = $request['search'];
-		$args['before']  = $request['before'];
-		$args['after']   = $request['after'];
-
-		if ( empty( $request['offset'] ) ) {
-			$args['offset'] = 1 < $request['page'] ? ( $request['page'] - 1 ) * $args['limit'] : 0;
+		$args[ 'order' ]   = $request[ 'order' ];
+		$args[ 'orderby' ] = $request[ 'orderby' ];
+		$args[ 'status' ]  = 'all' === $request[ 'status' ] ? '' : $request[ 'status' ];
+		$args[ 'include' ] = implode( ',', $request[ 'include' ] );
+		$args[ 'exclude' ] = implode( ',', $request[ 'exclude' ] );
+		$args[ 'limit' ]   = $request[ 'per_page' ];
+		$args[ 'search' ]  = $request[ 'search' ];
+		$args[ 'before' ]  = $request[ 'before' ];
+		$args[ 'after' ]   = $request[ 'after' ];
+
+		if ( empty( $request[ 'offset' ] ) ) {
+			$args[ 'offset' ] = 1 < $request[ 'page' ] ? ( $request[ 'page' ] - 1 ) * $args[ 'limit' ] : 0;
 		}
 
 		/**
@@ -161,8 +161,8 @@ 
 		 * @param \WP_REST_Request $request The current request.
 		 */
 		$prepared_args = apply_filters( 'woocommerce_rest_webhook_query', $args, $request );
-		unset( $prepared_args['page'] );
-		$prepared_args['paginate'] = true;
+		unset( $prepared_args[ 'page' ] );
+		$prepared_args[ 'paginate' ] = true;
 
 		// Get the webhooks.
 		$webhooks    = array();
@@ -178,7 +178,7 @@ 
 			}
 
 			$data       = $this->prepare_item_for_response( $object, $request );
-			$webhooks[] = $this->prepare_response_for_collection( $data );
+			$webhooks[ ] = $this->prepare_response_for_collection( $data );
 		}
 
 
@@ -213,7 +213,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-rest-api' ), array( 'status' => 404 ) );
@@ -229,18 +229,18 @@ 
 	 * @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-rest-api' ), $this->post_type ), array( 'status' => 400 ) );
 		}
 
 		// Validate topic.
-		if ( empty( $request['topic'] ) || ! wc_is_webhook_valid_topic( strtolower( $request['topic'] ) ) ) {
+		if ( empty( $request[ 'topic' ] ) || ! wc_is_webhook_valid_topic( strtolower( $request[ 'topic' ] ) ) ) {
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_topic", __( 'Webhook topic is required and must be valid.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
 		}
 
 		// Validate delivery URL.
-		if ( empty( $request['delivery_url'] ) || ! wc_is_valid_url( $request['delivery_url'] ) ) {
+		if ( empty( $request[ 'delivery_url' ] ) || ! wc_is_valid_url( $request[ 'delivery_url' ] ) ) {
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_delivery_url", __( 'Webhook delivery URL must be a valid URL starting with http:// or https://.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
 		}
 
@@ -253,9 +253,9 @@ 
 		$webhook->set_name( $post->post_title );
 		$webhook->set_user_id( $post->post_author );
 		$webhook->set_status( 'publish' === $post->post_status ? 'active' : 'disabled' );
-		$webhook->set_topic( $request['topic'] );
-		$webhook->set_delivery_url( $request['delivery_url'] );
-		$webhook->set_secret( ! empty( $request['secret'] ) ? $request['secret'] : wp_generate_password( 50, true, true ) );
+		$webhook->set_topic( $request[ 'topic' ] );
+		$webhook->set_delivery_url( $request[ 'delivery_url' ] );
+		$webhook->set_secret( ! empty( $request[ 'secret' ] ) ? $request[ 'secret' ] : wp_generate_password( 50, true, true ) );
 		$webhook->set_api_version( $this->get_default_api_version() );
 		$webhook->save();
 
@@ -289,7 +289,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function update_item( $request ) {
-		$id      = (int) $request['id'];
+		$id      = (int) $request[ 'id' ];
 		$webhook = wc_get_webhook( $id );
 
 		if ( empty( $webhook ) || is_null( $webhook ) ) {
@@ -297,32 +297,32 @@ 
 		}
 
 		// Update topic.
-		if ( ! empty( $request['topic'] ) ) {
-			if ( wc_is_webhook_valid_topic( strtolower( $request['topic'] ) ) ) {
-				$webhook->set_topic( $request['topic'] );
+		if ( ! empty( $request[ 'topic' ] ) ) {
+			if ( wc_is_webhook_valid_topic( strtolower( $request[ 'topic' ] ) ) ) {
+				$webhook->set_topic( $request[ 'topic' ] );
 			} else {
 				return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_topic", __( 'Webhook topic must be valid.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
 			}
 		}
 
 		// Update delivery URL.
-		if ( ! empty( $request['delivery_url'] ) ) {
-			if ( wc_is_valid_url( $request['delivery_url'] ) ) {
-				$webhook->set_delivery_url( $request['delivery_url'] );
+		if ( ! empty( $request[ 'delivery_url' ] ) ) {
+			if ( wc_is_valid_url( $request[ 'delivery_url' ] ) ) {
+				$webhook->set_delivery_url( $request[ 'delivery_url' ] );
 			} else {
 				return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_delivery_url", __( 'Webhook delivery URL must be a valid URL starting with http:// or https://.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
 			}
 		}
 
 		// Update secret.
-		if ( ! empty( $request['secret'] ) ) {
-			$webhook->set_secret( $request['secret'] );
+		if ( ! empty( $request[ 'secret' ] ) ) {
+			$webhook->set_secret( $request[ 'secret' ] );
 		}
 
 		// Update status.
-		if ( ! empty( $request['status'] ) ) {
-			if ( wc_is_webhook_valid_status( strtolower( $request['status'] ) ) ) {
-				$webhook->set_status( $request['status'] );
+		if ( ! empty( $request[ 'status' ] ) ) {
+			if ( wc_is_webhook_valid_status( strtolower( $request[ 'status' ] ) ) ) {
+				$webhook->set_status( $request[ 'status' ] );
 			} else {
 				return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_status", __( 'Webhook status must be valid.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
 			}
@@ -363,8 +363,8 @@ 
 	 * @return \WP_REST_Response|\WP_Error
 	 */
 	public function delete_item( $request ) {
-		$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 ) {
@@ -414,13 +414,13 @@ 
 		$data = new \stdClass();
 
 		// Post ID.
-		if ( isset( $request['id'] ) ) {
-			$data->ID = absint( $request['id'] );
+		if ( isset( $request[ 'id' ] ) ) {
+			$data->ID = absint( $request[ 'id' ] );
 		}
 
 		// Validate required POST fields.
 		if ( 'POST' === $request->get_method() && empty( $data->ID ) ) {
-			$data->post_title = ! empty( $request['name'] ) ? $request['name'] : sprintf( __( 'Webhook created on %s', 'woocommerce-rest-api' ), strftime( _x( '%b %d, %Y @ %I:%M %p', 'Webhook created on date parsed by strftime', 'woocommerce-rest-api' ) ) ); // @codingStandardsIgnoreLine
+			$data->post_title = ! empty( $request[ 'name' ] ) ? $request[ 'name' ] : sprintf( __( 'Webhook created on %s', 'woocommerce-rest-api' ), strftime( _x( '%b %d, %Y @ %I:%M %p', 'Webhook created on date parsed by strftime', 'woocommerce-rest-api' ) ) ); // @codingStandardsIgnoreLine
 
 			// Post author.
 			$data->post_author = get_current_user_id();
@@ -433,8 +433,8 @@ 
 		} else {
 
 			// Allow edit post title.
-			if ( ! empty( $request['name'] ) ) {
-				$data->post_title = $request['name'];
+			if ( ! empty( $request[ 'name' ] ) ) {
+				$data->post_title = $request[ 'name' ];
 			}
 		}
 
@@ -606,21 +606,21 @@ 
 	public function get_collection_params() {
 		$params = parent::get_collection_params();
 
-		$params['context']['default'] = 'view';
+		$params[ 'context' ][ 'default' ] = 'view';
 
-		$params['after']   = array(
+		$params[ 'after' ] = array(
 			'description'       => __( 'Limit response to resources published after a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'format'            => 'date-time',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['before']  = array(
+		$params[ 'before' ] = array(
 			'description'       => __( 'Limit response to resources published before a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
 			'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-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -629,7 +629,7 @@ 
 			'default'           => array(),
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to specific ids.', 'woocommerce-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -638,20 +638,20 @@ 
 			'default'           => array(),
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
-		$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(
 			'description'       => __( 'Order sort attribute ascending or descending.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'default'           => 'desc',
 			'enum'              => array( 'asc', 'desc' ),
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['orderby'] = array(
+		$params[ 'orderby' ] = array(
 			'description'       => __( 'Sort collection by object attribute.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'default'           => 'date',
@@ -662,7 +662,7 @@ 
 			),
 			'validate_callback' => 'rest_validate_request_arg',
 		);
-		$params['status']  = array(
+		$params[ 'status' ] = array(
 			'default'           => 'all',
 			'description'       => __( 'Limit result set to webhooks assigned a specific status.', 'woocommerce-rest-api' ),
 			'type'              => 'string',

src/Controllers/Version4/AbstractObjectsController.php

Spacing +61 added lines, -61 removed lines

@@ -65,7 +65,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-rest-api' ), array( 'status' => 404 ) );
@@ -110,7 +110,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-rest-api' ), $this->post_type ), array( 'status' => 400 ) );
 		}
@@ -156,7 +156,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-rest-api' ), array( 'status' => 404 ) );
@@ -199,42 +199,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.
@@ -264,7 +264,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;
@@ -273,7 +273,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' ] ),
 		);
 	}
 
@@ -288,17 +288,17 @@ 
 		$query_results = $this->get_objects( $query_args );
 
 		$objects = array();
-		foreach ( $query_results['objects'] as $object ) {
+		foreach ( $query_results[ 'objects' ] as $object ) {
 			if ( ! Permissions::user_can_read( $this->post_type, $object->get_id() ) ) {
 				continue;
 			}
 
 			$data      = $this->prepare_item_for_response( $object, $request );
-			$objects[] = $this->prepare_response_for_collection( $data );
+			$objects[ ] = $this->prepare_response_for_collection( $data );
 		}
 
-		$total     = $query_results['total'];
-		$max_pages = $query_results['pages'];
+		$total     = $query_results[ 'total' ];
+		$max_pages = $query_results[ 'pages' ];
 
 		$response = rest_ensure_response( $objects );
 		$response = Pagination::add_pagination_headers( $response, $request, $total, $max_pages );
@@ -313,8 +313,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() ) {
@@ -424,10 +424,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,
@@ -436,7 +436,7 @@ 
 			'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,
@@ -446,28 +446,28 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['search'] = array(
+		$params[ 'search' ] = array(
 			'description'       => __( 'Limit results to those matching a string.', 'woocommerce-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ),
 			'type'              => 'string',
 			'default'           => 'date',
@@ -480,14 +480,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-rest-api' ),
 			'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-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -497,7 +497,7 @@ 
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
 
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to specific ids.', 'woocommerce-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -507,14 +507,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-rest-api' ),
 			'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-rest-api' ),
 			'type'              => 'string',
 			'default'           => 'desc',
@@ -522,7 +522,7 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['orderby'] = array(
+		$params[ 'orderby' ] = array(
 			'description'       => __( 'Sort collection by object attribute.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'default'           => 'date',
@@ -538,7 +538,7 @@ 
 		);
 
 		if ( $this->hierarchical ) {
-			$params['parent'] = array(
+			$params[ 'parent' ] = array(
 				'description'       => __( 'Limit result set to those of particular parent IDs.', 'woocommerce-rest-api' ),
 				'type'              => 'array',
 				'items'             => array(
@@ -548,7 +548,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-rest-api' ),
 				'type'              => 'array',
 				'items'             => array(
@@ -599,14 +599,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;
@@ -694,13 +694,13 @@ 
 	 * @return array
 	 */
 	protected function add_meta_query( $args, $meta_query ) {
-		if ( empty( $args['meta_query'] ) ) {
-			$args['meta_query'] = []; // phpcs:ignore
+		if ( empty( $args[ 'meta_query' ] ) ) {
+			$args[ 'meta_query' ] = [ ]; // phpcs:ignore
 		}
 
-		$args['meta_query'][] = $meta_query;
+		$args[ 'meta_query' ][ ] = $meta_query;
 
-		return $args['meta_query'];
+		return $args[ 'meta_query' ];
 	}
 
 	/**

src/Controllers/Version4/Customers.php

Spacing +35 added lines, -35 removed lines

@@ -136,16 +136,16 @@ 
 	 */
 	public function get_items( $request ) {
 		$prepared_args = array(
-			'exclude' => $request['exclude'],
-			'include' => $request['include'],
-			'order'   => $request['order'],
-			'number'  => $request['per_page'],
+			'exclude' => $request[ 'exclude' ],
+			'include' => $request[ 'include' ],
+			'order'   => $request[ 'order' ],
+			'number'  => $request[ 'per_page' ],
 		);
 
-		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' ];
 		}
 
 		$orderby_possibles = array(
@@ -154,22 +154,22 @@ 
 			'name'            => 'display_name',
 			'registered_date' => 'registered',
 		);
-		$prepared_args['orderby'] = $orderby_possibles[ $request['orderby'] ];
-		$prepared_args['search']  = $request['search'];
+		$prepared_args[ 'orderby' ] = $orderby_possibles[ $request[ 'orderby' ] ];
+		$prepared_args[ 'search' ]  = $request[ 'search' ];
 
-		if ( '' !== $prepared_args['search'] ) {
-			$prepared_args['search'] = '*' . $prepared_args['search'] . '*';
+		if ( '' !== $prepared_args[ 'search' ] ) {
+			$prepared_args[ 'search' ] = '*' . $prepared_args[ 'search' ] . '*';
 		}
 
 		// Filter by email.
-		if ( ! empty( $request['email'] ) ) {
-			$prepared_args['search']         = $request['email'];
-			$prepared_args['search_columns'] = array( 'user_email' );
+		if ( ! empty( $request[ 'email' ] ) ) {
+			$prepared_args[ 'search' ]         = $request[ 'email' ];
+			$prepared_args[ 'search_columns' ] = array( 'user_email' );
 		}
 
 		// Filter by role.
-		if ( 'all' !== $request['role'] ) {
-			$prepared_args['role'] = $request['role'];
+		if ( 'all' !== $request[ 'role' ] ) {
+			$prepared_args[ 'role' ] = $request[ 'role' ];
 		}
 
 		/**
@@ -188,21 +188,21 @@ 
 		foreach ( $query->results as $user ) {
 			$customer = new \WC_Customer( $user->ID );
 			$data     = $this->prepare_item_for_response( $customer, $request );
-			$users[]  = $this->prepare_response_for_collection( $data );
+			$users[ ]  = $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 );
 
-		$prepared_args['fields'] = 'ID';
+		$prepared_args[ 'fields' ] = 'ID';
 
 		$total_users = $query->get_total();
 
 		if ( $total_users < 1 ) {
 			// Out-of-bounds, run the query again without LIMIT for total count.
-			unset( $prepared_args['number'] );
-			unset( $prepared_args['offset'] );
+			unset( $prepared_args[ 'number' ] );
+			unset( $prepared_args[ 'offset' ] );
 			$count_query = new \ WP_User_Query( $prepared_args );
 			$total_users = $count_query->get_total();
 		}
@@ -222,7 +222,7 @@ 
 	 */
 	public function create_item( $request ) {
 		try {
-			if ( ! empty( $request['id'] ) ) {
+			if ( ! empty( $request[ 'id' ] ) ) {
 				throw new \WC_REST_Exception( 'woocommerce_rest_customer_exists', __( 'Cannot create existing resource.', 'woocommerce-rest-api' ), 400 );
 			}
 
@@ -264,7 +264,7 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function get_item( $request ) {
-		$id       = (int) $request['id'];
+		$id       = (int) $request[ 'id' ];
 		$customer = new \WC_Customer( $id );
 
 		if ( empty( $id ) || ! $customer->get_id() ) {
@@ -322,9 +322,9 @@ 
 	 * @return \WP_Error|\WP_REST_Response
 	 */
 	public function delete_item( $request ) {
-		$id       = (int) $request['id'];
-		$reassign = isset( $request['reassign'] ) ? absint( $request['reassign'] ) : null;
-		$force    = isset( $request['force'] ) ? (bool) $request['force'] : false;
+		$id       = (int) $request[ 'id' ];
+		$reassign = isset( $request[ 'reassign' ] ) ? absint( $request[ 'reassign' ] ) : null;
+		$force    = isset( $request[ 'force' ] ) ? (bool) $request[ 'force' ] : false;
 
 		// We don't support trashing for this type, error out.
 		if ( ! $force ) {
@@ -663,9 +663,9 @@ 
 	public function get_collection_params() {
 		$params = parent::get_collection_params();
 
-		$params['context']['default'] = 'view';
+		$params[ 'context' ][ 'default' ] = 'view';
 
-		$params['exclude'] = array(
+		$params[ 'exclude' ] = array(
 			'description'       => __( 'Ensure result set excludes specific IDs.', 'woocommerce-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -674,7 +674,7 @@ 
 			'default'           => array(),
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
-		$params['include'] = array(
+		$params[ 'include' ] = array(
 			'description'       => __( 'Limit result set to specific IDs.', 'woocommerce-rest-api' ),
 			'type'              => 'array',
 			'items'             => array(
@@ -683,13 +683,13 @@ 
 			'default'           => array(),
 			'sanitize_callback' => 'wp_parse_id_list',
 		);
-		$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' ),
@@ -697,7 +697,7 @@ 
 			'type'               => 'string',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['orderby'] = array(
+		$params[ 'orderby' ] = array(
 			'default'            => 'name',
 			'description'        => __( 'Sort collection by object attribute.', 'woocommerce-rest-api' ),
 			'enum'               => array(
@@ -710,13 +710,13 @@ 
 			'type'               => 'string',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['email'] = array(
+		$params[ 'email' ] = array(
 			'description'        => __( 'Limit result set to resources with a specific email.', 'woocommerce-rest-api' ),
 			'type'               => 'string',
 			'format'             => 'email',
 			'validate_callback'  => 'rest_validate_request_arg',
 		);
-		$params['role'] = array(
+		$params[ 'role' ] = array(
 			'description'        => __( 'Limit result set to resources with a specific role.', 'woocommerce-rest-api' ),
 			'type'               => 'string',
 			'default'            => 'customer',

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-rest-api' ),
 			'type'        => 'integer',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['edit_url']          = array(
+		$schema[ 'properties' ][ 'edit_url' ] = array(
 			'description' => __( 'URL to edit the order', 'woocommerce-rest-api' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['customer'][]        = array(
+		$schema[ 'properties' ][ 'customer' ][ ] = array(
 			'description' => __( 'Name of the customer for the order', 'woocommerce-rest-api' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['status_name'][]     = array(
+		$schema[ 'properties' ][ 'status_name' ][ ] = array(
 			'description' => __( 'Order Status', 'woocommerce-rest-api' ),
 			'type'        => 'string',
 			'context'     => array( 'view' ),
 			'readonly'    => true,
 		);
-		$schema['properties']['formatted_total'][] = array(
+		$schema[ 'properties' ][ 'formatted_total' ][ ] = array(
 			'description' => __( 'Order total formatted for locale', 'woocommerce-rest-api' ),
 			'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-rest-api' ), $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-rest-api' ), $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/Products.php

Spacing +67 added lines, -67 removed lines

@@ -680,13 +680,13 @@ 
 	public function get_collection_params() {
 		$params = parent::get_collection_params();
 
-		$params['slug'] = array(
+		$params[ 'slug' ] = array(
 			'description'       => __( 'Limit result set to products with a specific slug.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['status'] = array(
+		$params[ 'status' ] = array(
 			'default'           => 'any',
 			'description'       => __( 'Limit result set to products assigned a specific status.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
@@ -695,7 +695,7 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['type'] = array(
+		$params[ 'type' ] = array(
 			'description'       => __( 'Limit result set to products assigned a specific type.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'enum'              => array_keys( wc_get_product_types() ),
@@ -703,49 +703,49 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['sku'] = array(
+		$params[ 'sku' ] = array(
 			'description'       => __( 'Limit result set to products with specific SKU(s). Use commas to separate.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_text_field',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['featured'] = array(
+		$params[ 'featured' ] = array(
 			'description'       => __( 'Limit result set to featured products.', 'woocommerce-rest-api' ),
 			'type'              => 'boolean',
 			'sanitize_callback' => 'wc_string_to_bool',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['category'] = array(
+		$params[ 'category' ] = array(
 			'description'       => __( 'Limit result set to products assigned a specific category ID.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'wp_parse_id_list',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['tag'] = array(
+		$params[ 'tag' ] = array(
 			'description'       => __( 'Limit result set to products assigned a specific tag ID.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'wp_parse_id_list',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['shipping_class'] = array(
+		$params[ 'shipping_class' ] = array(
 			'description'       => __( 'Limit result set to products assigned a specific shipping class ID.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'wp_parse_id_list',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['attribute'] = array(
+		$params[ 'attribute' ] = array(
 			'description'       => __( 'Limit result set to products with a specific attribute. Use the taxonomy name/attribute slug.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_text_field',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['attribute_term'] = array(
+		$params[ 'attribute_term' ] = array(
 			'description'       => __( 'Limit result set to products with a specific attribute term ID (required an assigned attribute).', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'wp_parse_id_list',
@@ -753,7 +753,7 @@ 
 		);
 
 		if ( wc_tax_enabled() ) {
-			$params['tax_class'] = array(
+			$params[ 'tax_class' ] = array(
 				'description'       => __( 'Limit result set to products with a specific tax class.', 'woocommerce-rest-api' ),
 				'type'              => 'string',
 				'enum'              => array_merge( array( 'standard' ), \WC_Tax::get_tax_class_slugs() ),
@@ -762,28 +762,28 @@ 
 			);
 		}
 
-		$params['on_sale'] = array(
+		$params[ 'on_sale' ] = array(
 			'description'       => __( 'Limit result set to products on sale.', 'woocommerce-rest-api' ),
 			'type'              => 'boolean',
 			'sanitize_callback' => 'wc_string_to_bool',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['min_price'] = array(
+		$params[ 'min_price' ] = array(
 			'description'       => __( 'Limit result set to products based on a minimum price.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_text_field',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['max_price'] = array(
+		$params[ 'max_price' ] = array(
 			'description'       => __( 'Limit result set to products based on a maximum price.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'sanitize_callback' => 'sanitize_text_field',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['stock_status'] = array(
+		$params[ 'stock_status' ] = array(
 			'description'       => __( 'Limit result set to products with specified stock status.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'enum'              => array_keys( wc_get_product_stock_status_options() ),
@@ -791,20 +791,20 @@ 
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['low_in_stock'] = array(
+		$params[ 'low_in_stock' ] = array(
 			'description'       => __( 'Limit result set to products that are low or out of stock.', 'woocommerce-rest-api' ),
 			'type'              => 'boolean',
 			'default'           => false,
 			'sanitize_callback' => 'wc_string_to_bool',
 		);
 
-		$params['search'] = array(
+		$params[ 'search' ] = array(
 			'description'       => __( 'Search by similar product name or sku.', 'woocommerce-rest-api' ),
 			'type'              => 'string',
 			'validate_callback' => 'rest_validate_request_arg',
 		);
 
-		$params['orderby']['enum'] = array_merge( $params['orderby']['enum'], array( 'price', 'popularity', 'rating' ) );
+		$params[ 'orderby' ][ 'enum' ] = array_merge( $params[ 'orderby' ][ 'enum' ], array( 'price', 'popularity', 'rating' ) );
 
 		return $params;
 	}
@@ -887,40 +887,40 @@ 
 
 		if ( $wp_query->get( 'search' ) ) {
 			$search         = "'%" . $wpdb->esc_like( $wp_query->get( 'search' ) ) . "%'";
-			$args['join']   = $this->append_product_sorting_table_join( $args['join'] );
-			$args['where'] .= " AND ({$wpdb->posts}.post_title LIKE {$search}";
-			$args['where'] .= wc_product_sku_enabled() ? ' OR wc_product_meta_lookup.sku LIKE ' . $search . ')' : ')';
+			$args[ 'join' ]   = $this->append_product_sorting_table_join( $args[ 'join' ] );
+			$args[ 'where' ] .= " AND ({$wpdb->posts}.post_title LIKE {$search}";
+			$args[ 'where' ] .= wc_product_sku_enabled() ? ' OR wc_product_meta_lookup.sku LIKE ' . $search . ')' : ')';
 		}
 
 		if ( $wp_query->get( 'sku' ) ) {
 			$skus = explode( ',', $wp_query->get( 'sku' ) );
 			// Include the current string as a SKU too.
 			if ( 1 < count( $skus ) ) {
-				$skus[] = $wp_query->get( 'sku' );
+				$skus[ ] = $wp_query->get( 'sku' );
 			}
-			$args['join']   = $this->append_product_sorting_table_join( $args['join'] );
-			$args['where'] .= ' AND wc_product_meta_lookup.sku IN ("' . implode( '","', array_map( 'esc_sql', $skus ) ) . '")';
+			$args[ 'join' ]   = $this->append_product_sorting_table_join( $args[ 'join' ] );
+			$args[ 'where' ] .= ' AND wc_product_meta_lookup.sku IN ("' . implode( '","', array_map( 'esc_sql', $skus ) ) . '")';
 		}
 
 		if ( $wp_query->get( 'min_price' ) ) {
-			$args['join']   = $this->append_product_sorting_table_join( $args['join'] );
-			$args['where'] .= $wpdb->prepare( ' AND wc_product_meta_lookup.min_price >= %f ', floatval( $wp_query->get( 'min_price' ) ) );
+			$args[ 'join' ]   = $this->append_product_sorting_table_join( $args[ 'join' ] );
+			$args[ 'where' ] .= $wpdb->prepare( ' AND wc_product_meta_lookup.min_price >= %f ', floatval( $wp_query->get( 'min_price' ) ) );
 		}
 
 		if ( $wp_query->get( 'max_price' ) ) {
-			$args['join']   = $this->append_product_sorting_table_join( $args['join'] );
-			$args['where'] .= $wpdb->prepare( ' AND wc_product_meta_lookup.max_price <= %f ', floatval( $wp_query->get( 'max_price' ) ) );
+			$args[ 'join' ]   = $this->append_product_sorting_table_join( $args[ 'join' ] );
+			$args[ 'where' ] .= $wpdb->prepare( ' AND wc_product_meta_lookup.max_price <= %f ', floatval( $wp_query->get( 'max_price' ) ) );
 		}
 
 		if ( $wp_query->get( 'stock_status' ) ) {
-			$args['join']   = $this->append_product_sorting_table_join( $args['join'] );
-			$args['where'] .= $wpdb->prepare( ' AND wc_product_meta_lookup.stock_status = %s ', $wp_query->get( 'stock_status' ) );
+			$args[ 'join' ]   = $this->append_product_sorting_table_join( $args[ 'join' ] );
+			$args[ 'where' ] .= $wpdb->prepare( ' AND wc_product_meta_lookup.stock_status = %s ', $wp_query->get( 'stock_status' ) );
 		}
 
 		if ( $wp_query->get( 'low_in_stock' ) ) {
 			$low_stock      = absint( max( get_option( 'woocommerce_notify_low_stock_amount' ), 1 ) );
-			$args['join']   = $this->append_product_sorting_table_join( $args['join'] );
-			$args['where'] .= $wpdb->prepare( ' AND wc_product_meta_lookup.stock_quantity <= %d', $low_stock );
+			$args[ 'join' ]   = $this->append_product_sorting_table_join( $args[ 'join' ] );
+			$args[ 'where' ] .= $wpdb->prepare( ' AND wc_product_meta_lookup.stock_quantity <= %d', $low_stock );
 		}
 
 		return $args;
@@ -952,7 +952,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(
@@ -982,7 +982,7 @@ 
 		// Set tax_query for each passed arg.
 		foreach ( $taxonomies as $taxonomy => $key ) {
 			if ( ! empty( $request[ $key ] ) ) {
-				$tax_query[] = array(
+				$tax_query[ ] = array(
 					'taxonomy' => $taxonomy,
 					'field'    => 'term_id',
 					'terms'    => $request[ $key ],
@@ -991,58 +991,58 @@ 
 		}
 
 		// Filter product type by slug.
-		if ( ! empty( $request['type'] ) ) {
-			$tax_query[] = array(
+		if ( ! empty( $request[ 'type' ] ) ) {
+			$tax_query[ ] = array(
 				'taxonomy' => 'product_type',
 				'field'    => 'slug',
-				'terms'    => $request['type'],
+				'terms'    => $request[ 'type' ],
 			);
 		}
 
 		// Filter by attribute and term.
-		if ( ! empty( $request['attribute'] ) && ! empty( $request['attribute_term'] ) ) {
-			if ( in_array( $request['attribute'], wc_get_attribute_taxonomy_names(), true ) ) {
-				$tax_query[] = array(
-					'taxonomy' => $request['attribute'],
+		if ( ! empty( $request[ 'attribute' ] ) && ! empty( $request[ 'attribute_term' ] ) ) {
+			if ( in_array( $request[ 'attribute' ], wc_get_attribute_taxonomy_names(), true ) ) {
+				$tax_query[ ] = array(
+					'taxonomy' => $request[ 'attribute' ],
 					'field'    => 'term_id',
-					'terms'    => $request['attribute_term'],
+					'terms'    => $request[ 'attribute_term' ],
 				);
 			}
 		}
 
 		// Build tax_query if taxonomies are set.
 		if ( ! empty( $tax_query ) ) {
-			if ( ! empty( $args['tax_query'] ) ) {
-				$args['tax_query'] = array_merge( $tax_query, $args['tax_query'] ); // WPCS: slow query ok.
+			if ( ! empty( $args[ 'tax_query' ] ) ) {
+				$args[ 'tax_query' ] = array_merge( $tax_query, $args[ 'tax_query' ] ); // WPCS: slow query ok.
 			} else {
-				$args['tax_query'] = $tax_query; // WPCS: slow query ok.
+				$args[ 'tax_query' ] = $tax_query; // WPCS: slow query ok.
 			}
 		}
 
 		// Filter featured.
-		if ( is_bool( $request['featured'] ) ) {
-			$args['tax_query'][] = array(
+		if ( is_bool( $request[ 'featured' ] ) ) {
+			$args[ 'tax_query' ][ ] = array(
 				'taxonomy' => 'product_visibility',
 				'field'    => 'name',
 				'terms'    => 'featured',
-				'operator' => true === $request['featured'] ? 'IN' : 'NOT IN',
+				'operator' => true === $request[ 'featured' ] ? 'IN' : 'NOT IN',
 			);
 		}
 
 		// 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.
@@ -1052,20 +1052,20 @@ 
 		}
 
 		// 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;
 		}
 
 		$orderby = $request->get_param( 'orderby' );
 		$order   = $request->get_param( 'order' );
 
 		$ordering_args   = WC()->query->get_catalog_ordering_args( $orderby, $order );
-		$args['orderby'] = $ordering_args['orderby'];
-		$args['order']   = $ordering_args['order'];
-		if ( $ordering_args['meta_key'] ) {
-			$args['meta_key'] = $ordering_args['meta_key']; // WPCS: slow query ok.
+		$args[ 'orderby' ] = $ordering_args[ 'orderby' ];
+		$args[ 'order' ]   = $ordering_args[ 'order' ];
+		if ( $ordering_args[ 'meta_key' ] ) {
+			$args[ 'meta_key' ] = $ordering_args[ 'meta_key' ]; // WPCS: slow query ok.
 		}
 
 		return $args;
@@ -1081,16 +1081,16 @@ 
 	protected function prepare_links( $item, $request ) {
 		$links = array(
 			'self'       => array(
-				'href' => rest_url( sprintf( '/%s/%s/%d', $this->namespace, $this->rest_base, $item->get_id() ) ),  // @codingStandardsIgnoreLine.
+				'href' => rest_url( sprintf( '/%s/%s/%d', $this->namespace, $this->rest_base, $item->get_id() ) ), // @codingStandardsIgnoreLine.
 			),
 			'collection' => array(
-				'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $this->rest_base ) ),  // @codingStandardsIgnoreLine.
+				'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $this->rest_base ) ), // @codingStandardsIgnoreLine.
 			),
 		);
 
 		if ( $item->get_parent_id() ) {
-			$links['up'] = array(
-				'href' => rest_url( sprintf( '/%s/products/%d', $this->namespace, $item->get_parent_id() ) ),  // @codingStandardsIgnoreLine.
+			$links[ 'up' ] = array(
+				'href' => rest_url( sprintf( '/%s/products/%d', $this->namespace, $item->get_parent_id() ) ), // @codingStandardsIgnoreLine.
 			);
 		}
 
@@ -1105,9 +1105,9 @@ 
 	 * @return \WP_REST_Response|\WP_Error
 	 */
 	public function delete_item( $request ) {
-		$id     = (int) $request['id'];
-		$force  = (bool) $request['force'];
-		$object = $this->get_object( (int) $request['id'] );
+		$id     = (int) $request[ 'id' ];
+		$force  = (bool) $request[ 'force' ];
+		$object = $this->get_object( (int) $request[ 'id' ] );
 		$result = false;
 
 		if ( ! $object || 0 === $object->get_id() ) {

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-rest-api' ),
 			'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-rest-api' ),
 			'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-rest-api' ), 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/CustomerDownloads.php

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
 	 * @return \WP_Error|boolean
 	 */
 	public function get_items_permissions_check( $request ) {
-		$customer = get_user_by( 'id', (int) $request['customer_id'] );
+		$customer = get_user_by( 'id', (int) $request[ 'customer_id' ] );
 
 		if ( ! $customer ) {
 			return new \WP_Error( 'woocommerce_rest_customer_invalid', __( 'Resource does not exist.', 'woocommerce-rest-api' ), array( 'status' => 404 ) );
@@ -66,13 +66,13 @@ 
 	 * @return array
 	 */
 	public function get_items( $request ) {
-		$downloads = wc_get_customer_available_downloads( (int) $request['customer_id'] );
+		$downloads = wc_get_customer_available_downloads( (int) $request[ 'customer_id' ] );
 
 		$data = array();
 		foreach ( $downloads as $download_data ) {
 			$download = $this->prepare_item_for_response( (object) $download_data, $request );
 			$download = $this->prepare_response_for_collection( $download );
-			$data[]   = $download;
+			$data[ ]   = $download;
 		}
 
 		return rest_ensure_response( $data );
@@ -109,7 +109,7 @@ 
 	 * @return array
 	 */
 	protected function prepare_links( $item, $request ) {
-		$base  = str_replace( '(?P<customer_id>[\d]+)', $request['customer_id'], $this->rest_base );
+		$base  = str_replace( '(?P<customer_id>[\d]+)', $request[ 'customer_id' ], $this->rest_base );
 		$links = array(
 			'collection' => array(
 				'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $base ) ),