Completed
Pull Request — master (#14)
by Mike
03:57
created
unit-tests/Tests/Version2/product-reviews.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -53,8 +53,8 @@  discard block
 block discarded – undo
53 53
 		$this->assertContains(
54 54
 			array(
55 55
 				'id'                   => $review_id,
56
-				'date_created'         => $product_reviews[0]['date_created'],
57
-				'date_created_gmt'     => $product_reviews[0]['date_created_gmt'],
56
+				'date_created'         => $product_reviews[ 0 ][ 'date_created' ],
57
+				'date_created_gmt'     => $product_reviews[ 0 ][ 'date_created_gmt' ],
58 58
 				'product_id'           => $product->get_id(),
59 59
 				'status'               => 'approved',
60 60
 				'reviewer'             => 'admin',
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 				'review'               => "<p>Review content here</p>\n",
63 63
 				'rating'               => 0,
64 64
 				'verified'             => false,
65
-				'reviewer_avatar_urls' => $product_reviews[0]['reviewer_avatar_urls'],
65
+				'reviewer_avatar_urls' => $product_reviews[ 0 ][ 'reviewer_avatar_urls' ],
66 66
 				'_links'               => array(
67 67
 					'self'       => array(
68 68
 						array(
@@ -125,8 +125,8 @@  discard block
 block discarded – undo
125 125
 		$this->assertEquals(
126 126
 			array(
127 127
 				'id'                   => $product_review_id,
128
-				'date_created'         => $data['date_created'],
129
-				'date_created_gmt'     => $data['date_created_gmt'],
128
+				'date_created'         => $data[ 'date_created' ],
129
+				'date_created_gmt'     => $data[ 'date_created_gmt' ],
130 130
 				'product_id'           => $product->get_id(),
131 131
 				'status'               => 'approved',
132 132
 				'reviewer'             => 'admin',
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 				'review'               => "<p>Review content here</p>\n",
135 135
 				'rating'               => 0,
136 136
 				'verified'             => false,
137
-				'reviewer_avatar_urls' => $data['reviewer_avatar_urls'],
137
+				'reviewer_avatar_urls' => $data[ 'reviewer_avatar_urls' ],
138 138
 			),
139 139
 			$data
140 140
 		);
@@ -189,9 +189,9 @@  discard block
 block discarded – undo
189 189
 		$this->assertEquals( 201, $response->get_status() );
190 190
 		$this->assertEquals(
191 191
 			array(
192
-				'id'                   => $data['id'],
193
-				'date_created'         => $data['date_created'],
194
-				'date_created_gmt'     => $data['date_created_gmt'],
192
+				'id'                   => $data[ 'id' ],
193
+				'date_created'         => $data[ 'date_created' ],
194
+				'date_created_gmt'     => $data[ 'date_created_gmt' ],
195 195
 				'product_id'           => $product->get_id(),
196 196
 				'status'               => 'approved',
197 197
 				'reviewer'             => 'Admin',
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
 				'review'               => 'Hello world.',
200 200
 				'rating'               => 5,
201 201
 				'verified'             => false,
202
-				'reviewer_avatar_urls' => $data['reviewer_avatar_urls'],
202
+				'reviewer_avatar_urls' => $data[ 'reviewer_avatar_urls' ],
203 203
 			),
204 204
 			$data
205 205
 		);
@@ -266,10 +266,10 @@  discard block
 block discarded – undo
266 266
 
267 267
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v3/products/reviews/' . $product_review_id ) );
268 268
 		$data     = $response->get_data();
269
-		$this->assertEquals( "<p>Review content here</p>\n", $data['review'] );
270
-		$this->assertEquals( 'admin', $data['reviewer'] );
271
-		$this->assertEquals( '[email protected]', $data['reviewer_email'] );
272
-		$this->assertEquals( 0, $data['rating'] );
269
+		$this->assertEquals( "<p>Review content here</p>\n", $data[ 'review' ] );
270
+		$this->assertEquals( 'admin', $data[ 'reviewer' ] );
271
+		$this->assertEquals( '[email protected]', $data[ 'reviewer_email' ] );
272
+		$this->assertEquals( 0, $data[ 'rating' ] );
273 273
 
274 274
 		$request = new WP_REST_Request( 'PUT', '/wc/v3/products/reviews/' . $product_review_id );
275 275
 		$request->set_body_params(
@@ -282,10 +282,10 @@  discard block
 block discarded – undo
282 282
 		);
283 283
 		$response = $this->server->dispatch( $request );
284 284
 		$data     = $response->get_data();
285
-		$this->assertEquals( 'Hello world - updated.', $data['review'] );
286
-		$this->assertEquals( 'Justin', $data['reviewer'] );
287
-		$this->assertEquals( '[email protected]', $data['reviewer_email'] );
288
-		$this->assertEquals( 3, $data['rating'] );
285
+		$this->assertEquals( 'Hello world - updated.', $data[ 'review' ] );
286
+		$this->assertEquals( 'Justin', $data[ 'reviewer' ] );
287
+		$this->assertEquals( '[email protected]', $data[ 'reviewer_email' ] );
288
+		$this->assertEquals( 3, $data[ 'rating' ] );
289 289
 	}
290 290
 
291 291
 	/**
@@ -424,10 +424,10 @@  discard block
 block discarded – undo
424 424
 		$response = $this->server->dispatch( $request );
425 425
 		$data     = $response->get_data();
426 426
 
427
-		$this->assertEquals( 'Updated review.', $data['update'][0]['review'] );
428
-		$this->assertEquals( 'New review.', $data['create'][0]['review'] );
429
-		$this->assertEquals( $review_2_id, $data['delete'][0]['previous']['id'] );
430
-		$this->assertEquals( $review_3_id, $data['delete'][1]['previous']['id'] );
427
+		$this->assertEquals( 'Updated review.', $data[ 'update' ][ 0 ][ 'review' ] );
428
+		$this->assertEquals( 'New review.', $data[ 'create' ][ 0 ][ 'review' ] );
429
+		$this->assertEquals( $review_2_id, $data[ 'delete' ][ 0 ][ 'previous' ][ 'id' ] );
430
+		$this->assertEquals( $review_3_id, $data[ 'delete' ][ 1 ][ 'previous' ][ 'id' ] );
431 431
 
432 432
 		$request = new WP_REST_Request( 'GET', '/wc/v3/products/reviews' );
433 433
 		$request->set_param( 'product', $product->get_id() );
@@ -449,7 +449,7 @@  discard block
 block discarded – undo
449 449
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v3/products/reviews' );
450 450
 		$response   = $this->server->dispatch( $request );
451 451
 		$data       = $response->get_data();
452
-		$properties = $data['schema']['properties'];
452
+		$properties = $data[ 'schema' ][ 'properties' ];
453 453
 
454 454
 		$this->assertEquals( 11, count( $properties ) );
455 455
 		$this->assertArrayHasKey( 'id', $properties );
Please login to merge, or discard this patch.
src/Controllers/Version4/ProductReviews.php 1 patch
Spacing   +60 added lines, -60 removed lines patch added patch discarded remove patch
@@ -176,29 +176,29 @@  discard block
 block discarded – undo
176 176
 			}
177 177
 		}
178 178
 
179
-		if ( isset( $registered['orderby'] ) ) {
180
-			$prepared_args['orderby'] = $this->normalize_query_param( $request['orderby'] );
179
+		if ( isset( $registered[ 'orderby' ] ) ) {
180
+			$prepared_args[ 'orderby' ] = $this->normalize_query_param( $request[ 'orderby' ] );
181 181
 		}
182 182
 
183
-		if ( isset( $prepared_args['status'] ) ) {
184
-			$prepared_args['status'] = 'approved' === $prepared_args['status'] ? 'approve' : $prepared_args['status'];
183
+		if ( isset( $prepared_args[ 'status' ] ) ) {
184
+			$prepared_args[ 'status' ] = 'approved' === $prepared_args[ 'status' ] ? 'approve' : $prepared_args[ 'status' ];
185 185
 		}
186 186
 
187
-		$prepared_args['no_found_rows'] = false;
188
-		$prepared_args['date_query']    = array();
187
+		$prepared_args[ 'no_found_rows' ] = false;
188
+		$prepared_args[ 'date_query' ]    = array();
189 189
 
190 190
 		// Set before into date query. Date query must be specified as an array of an array.
191
-		if ( isset( $registered['before'], $request['before'] ) ) {
192
-			$prepared_args['date_query'][0]['before'] = $request['before'];
191
+		if ( isset( $registered[ 'before' ], $request[ 'before' ] ) ) {
192
+			$prepared_args[ 'date_query' ][ 0 ][ 'before' ] = $request[ 'before' ];
193 193
 		}
194 194
 
195 195
 		// Set after into date query. Date query must be specified as an array of an array.
196
-		if ( isset( $registered['after'], $request['after'] ) ) {
197
-			$prepared_args['date_query'][0]['after'] = $request['after'];
196
+		if ( isset( $registered[ 'after' ], $request[ 'after' ] ) ) {
197
+			$prepared_args[ 'date_query' ][ 0 ][ 'after' ] = $request[ 'after' ];
198 198
 		}
199 199
 
200
-		if ( isset( $registered['page'] ) && empty( $request['offset'] ) ) {
201
-			$prepared_args['offset'] = $prepared_args['number'] * ( absint( $request['page'] ) - 1 );
200
+		if ( isset( $registered[ 'page' ] ) && empty( $request[ 'offset' ] ) ) {
201
+			$prepared_args[ 'offset' ] = $prepared_args[ 'number' ] * ( absint( $request[ 'page' ] ) - 1 );
202 202
 		}
203 203
 
204 204
 		/**
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
 		$prepared_args = apply_filters( 'woocommerce_rest_product_review_query', $prepared_args, $request );
213 213
 
214 214
 		// Make sure that returns only reviews.
215
-		$prepared_args['type'] = 'review';
215
+		$prepared_args[ 'type' ] = 'review';
216 216
 
217 217
 		// Query reviews.
218 218
 		$query        = new \WP_Comment_Query();
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 			}
226 226
 
227 227
 			$data      = $this->prepare_item_for_response( $review, $request );
228
-			$reviews[] = $this->prepare_response_for_collection( $data );
228
+			$reviews[ ] = $this->prepare_response_for_collection( $data );
229 229
 		}
230 230
 
231 231
 		$total_reviews = (int) $query->found_comments;
@@ -233,13 +233,13 @@  discard block
 block discarded – undo
233 233
 
234 234
 		if ( $total_reviews < 1 ) {
235 235
 			// Out-of-bounds, run the query again without LIMIT for total count.
236
-			unset( $prepared_args['number'], $prepared_args['offset'] );
236
+			unset( $prepared_args[ 'number' ], $prepared_args[ 'offset' ] );
237 237
 
238 238
 			$query                  = new \WP_Comment_Query();
239
-			$prepared_args['count'] = true;
239
+			$prepared_args[ 'count' ] = true;
240 240
 
241 241
 			$total_reviews = $query->query( $prepared_args );
242
-			$max_pages     = ceil( $total_reviews / $request['per_page'] );
242
+			$max_pages     = ceil( $total_reviews / $request[ 'per_page' ] );
243 243
 		}
244 244
 
245 245
 		$response = rest_ensure_response( $reviews );
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
 	 */
257 257
 	public function create_item( $request ) {
258 258
 		try {
259
-			if ( ! empty( $request['id'] ) ) {
259
+			if ( ! empty( $request[ 'id' ] ) ) {
260 260
 				return new \WP_Error( 'woocommerce_rest_review_exists', __( 'Cannot create existing product review.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
261 261
 			}
262 262
 
@@ -312,11 +312,11 @@  discard block
 block discarded – undo
312 312
 				throw new \WC_REST_Exception( 'woocommerce_rest_review_failed_create', __( 'Creating product review failed.', 'woocommerce-rest-api' ), 500 );
313 313
 			}
314 314
 
315
-			if ( isset( $request['status'] ) ) {
316
-				$this->handle_status_param( $request['status'], $review_id );
315
+			if ( isset( $request[ 'status' ] ) ) {
316
+				$this->handle_status_param( $request[ 'status' ], $review_id );
317 317
 			}
318 318
 
319
-			update_comment_meta( $review_id, 'rating', ! empty( $request['rating'] ) ? $request['rating'] : '0' );
319
+			update_comment_meta( $review_id, 'rating', ! empty( $request[ 'rating' ] ) ? $request[ 'rating' ] : '0' );
320 320
 		} catch ( \WC_REST_Exception $e ) {
321 321
 			return new \WP_Error( $e->getErrorCode(), $e->getMessage(), array( 'status' => $e->getCode() ) );
322 322
 		}
@@ -358,12 +358,12 @@  discard block
 block discarded – undo
358 358
 	 * @return array
359 359
 	 */
360 360
 	protected function validate_review( $prepared_review, $creating = false ) {
361
-		if ( empty( $prepared_review['comment_content'] ) ) {
361
+		if ( empty( $prepared_review[ 'comment_content' ] ) ) {
362 362
 			throw new \WC_REST_Exception( 'woocommerce_rest_review_content_invalid', __( 'Invalid review content.', 'woocommerce-rest-api' ), 400 );
363 363
 		}
364 364
 
365
-		if ( ! empty( $prepared_review['comment_post_ID'] ) ) {
366
-			if ( 'product' !== get_post_type( $prepared_review['comment_post_ID'] ) ) {
365
+		if ( ! empty( $prepared_review[ 'comment_post_ID' ] ) ) {
366
+			if ( 'product' !== get_post_type( $prepared_review[ 'comment_post_ID' ] ) ) {
367 367
 				throw new \WC_REST_Exception( 'woocommerce_rest_product_invalid_id', __( 'Invalid product ID.', 'woocommerce-rest-api' ), 404 );
368 368
 			}
369 369
 		}
@@ -376,7 +376,7 @@  discard block
 block discarded – undo
376 376
 		}
377 377
 
378 378
 		if ( $creating ) {
379
-			$prepared_review['comment_approved'] = $this->get_comment_approved( $prepared_review );
379
+			$prepared_review[ 'comment_approved' ] = $this->get_comment_approved( $prepared_review );
380 380
 		}
381 381
 
382 382
 		return $prepared_review;
@@ -388,8 +388,8 @@  discard block
 block discarded – undo
388 388
 	 * @return string
389 389
 	 */
390 390
 	protected function get_comment_author_ip() {
391
-		if ( ! empty( $_SERVER['REMOTE_ADDR'] ) && rest_is_ip_address( wp_unslash( $_SERVER['REMOTE_ADDR'] ) ) ) { // WPCS: input var ok, sanitization ok.
392
-			return wc_clean( wp_unslash( $_SERVER['REMOTE_ADDR'] ) ); // WPCS: input var ok.
391
+		if ( ! empty( $_SERVER[ 'REMOTE_ADDR' ] ) && rest_is_ip_address( wp_unslash( $_SERVER[ 'REMOTE_ADDR' ] ) ) ) { // WPCS: input var ok, sanitization ok.
392
+			return wc_clean( wp_unslash( $_SERVER[ 'REMOTE_ADDR' ] ) ); // WPCS: input var ok.
393 393
 		} else {
394 394
 			return '127.0.0.1';
395 395
 		}
@@ -402,8 +402,8 @@  discard block
 block discarded – undo
402 402
 	 * @return string
403 403
 	 */
404 404
 	protected function get_comment_agent( $request ) {
405
-		if ( ! empty( $request['author_user_agent'] ) ) {
406
-			return $request['author_user_agent'];
405
+		if ( ! empty( $request[ 'author_user_agent' ] ) ) {
406
+			return $request[ 'author_user_agent' ];
407 407
 		} elseif ( $request->get_header( 'user_agent' ) ) {
408 408
 			return $request->get_header( 'user_agent' );
409 409
 		} else {
@@ -444,7 +444,7 @@  discard block
 block discarded – undo
444 444
 	 * @return \WP_Error|\WP_REST_Response
445 445
 	 */
446 446
 	public function get_item( $request ) {
447
-		$review = $this->get_review( $request['id'] );
447
+		$review = $this->get_review( $request[ 'id' ] );
448 448
 		if ( is_wp_error( $review ) ) {
449 449
 			return $review;
450 450
 		}
@@ -460,7 +460,7 @@  discard block
 block discarded – undo
460 460
 	 */
461 461
 	public function update_item( $request ) {
462 462
 		try {
463
-			$review = $this->get_review( $request['id'] );
463
+			$review = $this->get_review( $request[ 'id' ] );
464 464
 
465 465
 			if ( is_wp_error( $review ) ) {
466 466
 				return $review;
@@ -468,7 +468,7 @@  discard block
 block discarded – undo
468 468
 
469 469
 			$review_id = (int) $review->comment_ID;
470 470
 
471
-			if ( isset( $request['type'] ) && 'review' !== get_comment_type( $review_id ) ) {
471
+			if ( isset( $request[ 'type' ] ) && 'review' !== get_comment_type( $review_id ) ) {
472 472
 				return new \WP_Error( 'woocommerce_rest_review_invalid_type', __( 'Sorry, you are not allowed to change the comment type.', 'woocommerce-rest-api' ), array( 'status' => 404 ) );
473 473
 			}
474 474
 
@@ -493,12 +493,12 @@  discard block
 block discarded – undo
493 493
 
494 494
 			wp_update_comment( wp_slash( $prepared_review ) );
495 495
 
496
-			if ( isset( $request['status'] ) ) {
497
-				$this->handle_status_param( $request['status'], $review_id );
496
+			if ( isset( $request[ 'status' ] ) ) {
497
+				$this->handle_status_param( $request[ 'status' ], $review_id );
498 498
 			}
499 499
 
500
-			if ( ! empty( $request['rating'] ) ) {
501
-				update_comment_meta( $review_id, 'rating', $request['rating'] );
500
+			if ( ! empty( $request[ 'rating' ] ) ) {
501
+				update_comment_meta( $review_id, 'rating', $request[ 'rating' ] );
502 502
 			}
503 503
 		} catch ( \WC_REST_Exception $e ) {
504 504
 			return new \WP_Error( $e->getErrorCode(), $e->getMessage(), array( 'status' => $e->getCode() ) );
@@ -533,12 +533,12 @@  discard block
 block discarded – undo
533 533
 	 * @return \WP_Error|\WP_REST_Response Response object on success, or error object on failure.
534 534
 	 */
535 535
 	public function delete_item( $request ) {
536
-		$review = $this->get_review( $request['id'] );
536
+		$review = $this->get_review( $request[ 'id' ] );
537 537
 		if ( is_wp_error( $review ) ) {
538 538
 			return $review;
539 539
 		}
540 540
 
541
-		$force = isset( $request['force'] ) ? (bool) $request['force'] : false;
541
+		$force = isset( $request[ 'force' ] ) ? (bool) $request[ 'force' ] : false;
542 542
 
543 543
 		/**
544 544
 		 * Filters whether a review can be trashed.
@@ -634,17 +634,17 @@  discard block
 block discarded – undo
634 634
 			$prepared_review[ $key ] = $request[ $value ];
635 635
 		}
636 636
 
637
-		if ( ! empty( $request['date_created'] ) ) {
638
-			$date_data = rest_get_date_with_gmt( $request['date_created'] );
637
+		if ( ! empty( $request[ 'date_created' ] ) ) {
638
+			$date_data = rest_get_date_with_gmt( $request[ 'date_created' ] );
639 639
 
640 640
 			if ( ! empty( $date_data ) ) {
641
-				list( $prepared_review['comment_date'], $prepared_review['comment_date_gmt'] ) = $date_data;
641
+				list( $prepared_review[ 'comment_date' ], $prepared_review[ 'comment_date_gmt' ] ) = $date_data;
642 642
 			}
643
-		} elseif ( ! empty( $request['date_created_gmt'] ) ) {
644
-			$date_data = rest_get_date_with_gmt( $request['date_created_gmt'], true );
643
+		} elseif ( ! empty( $request[ 'date_created_gmt' ] ) ) {
644
+			$date_data = rest_get_date_with_gmt( $request[ 'date_created_gmt' ], true );
645 645
 
646 646
 			if ( ! empty( $date_data ) ) {
647
-				list( $prepared_review['comment_date'], $prepared_review['comment_date_gmt'] ) = $date_data;
647
+				list( $prepared_review[ 'comment_date' ], $prepared_review[ 'comment_date_gmt' ] ) = $date_data;
648 648
 			}
649 649
 		}
650 650
 
@@ -668,13 +668,13 @@  discard block
 block discarded – undo
668 668
 			),
669 669
 		);
670 670
 		if ( 0 !== (int) $item->comment_post_ID ) {
671
-			$links['up'] = array(
671
+			$links[ 'up' ] = array(
672 672
 				'href'       => rest_url( sprintf( '/%s/products/%d', $this->namespace, $item->comment_post_ID ) ),
673 673
 				'embeddable' => true,
674 674
 			);
675 675
 		}
676 676
 		if ( 0 !== (int) $item->user_id ) {
677
-			$links['reviewer'] = array(
677
+			$links[ 'reviewer' ] = array(
678 678
 				'href'       => rest_url( 'wp/v2/users/' . $item->user_id ),
679 679
 				'embeddable' => true,
680 680
 			);
@@ -769,7 +769,7 @@  discard block
 block discarded – undo
769 769
 					'context'     => array( 'embed', 'view', 'edit' ),
770 770
 				);
771 771
 			}
772
-			$schema['properties']['reviewer_avatar_urls'] = array(
772
+			$schema[ 'properties' ][ 'reviewer_avatar_urls' ] = array(
773 773
 				'description' => __( 'Avatar URLs for the object reviewer.', 'woocommerce-rest-api' ),
774 774
 				'type'        => 'object',
775 775
 				'context'     => array( 'view', 'edit' ),
@@ -789,19 +789,19 @@  discard block
 block discarded – undo
789 789
 	public function get_collection_params() {
790 790
 		$params = parent::get_collection_params();
791 791
 
792
-		$params['context']['default'] = 'view';
792
+		$params[ 'context' ][ 'default' ] = 'view';
793 793
 
794
-		$params['after']            = array(
794
+		$params[ 'after' ] = array(
795 795
 			'description' => __( 'Limit response to resources published after a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
796 796
 			'type'        => 'string',
797 797
 			'format'      => 'date-time',
798 798
 		);
799
-		$params['before']           = array(
799
+		$params[ 'before' ] = array(
800 800
 			'description' => __( 'Limit response to reviews published before a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
801 801
 			'type'        => 'string',
802 802
 			'format'      => 'date-time',
803 803
 		);
804
-		$params['exclude']          = array(
804
+		$params[ 'exclude' ] = array(
805 805
 			'description' => __( 'Ensure result set excludes specific IDs.', 'woocommerce-rest-api' ),
806 806
 			'type'        => 'array',
807 807
 			'items'       => array(
@@ -809,7 +809,7 @@  discard block
 block discarded – undo
809 809
 			),
810 810
 			'default'     => array(),
811 811
 		);
812
-		$params['include']          = array(
812
+		$params[ 'include' ] = array(
813 813
 			'description' => __( 'Limit result set to specific IDs.', 'woocommerce-rest-api' ),
814 814
 			'type'        => 'array',
815 815
 			'items'       => array(
@@ -817,11 +817,11 @@  discard block
 block discarded – undo
817 817
 			),
818 818
 			'default'     => array(),
819 819
 		);
820
-		$params['offset']           = array(
820
+		$params[ 'offset' ] = array(
821 821
 			'description' => __( 'Offset the result set by a specific number of items.', 'woocommerce-rest-api' ),
822 822
 			'type'        => 'integer',
823 823
 		);
824
-		$params['order']            = array(
824
+		$params[ 'order' ] = array(
825 825
 			'description' => __( 'Order sort attribute ascending or descending.', 'woocommerce-rest-api' ),
826 826
 			'type'        => 'string',
827 827
 			'default'     => 'desc',
@@ -830,7 +830,7 @@  discard block
 block discarded – undo
830 830
 				'desc',
831 831
 			),
832 832
 		);
833
-		$params['orderby']          = array(
833
+		$params[ 'orderby' ] = array(
834 834
 			'description' => __( 'Sort collection by object attribute.', 'woocommerce-rest-api' ),
835 835
 			'type'        => 'string',
836 836
 			'default'     => 'date_gmt',
@@ -842,27 +842,27 @@  discard block
 block discarded – undo
842 842
 				'product',
843 843
 			),
844 844
 		);
845
-		$params['reviewer']         = array(
845
+		$params[ 'reviewer' ] = array(
846 846
 			'description' => __( 'Limit result set to reviews assigned to specific user IDs.', 'woocommerce-rest-api' ),
847 847
 			'type'        => 'array',
848 848
 			'items'       => array(
849 849
 				'type' => 'integer',
850 850
 			),
851 851
 		);
852
-		$params['reviewer_exclude'] = array(
852
+		$params[ 'reviewer_exclude' ] = array(
853 853
 			'description' => __( 'Ensure result set excludes reviews assigned to specific user IDs.', 'woocommerce-rest-api' ),
854 854
 			'type'        => 'array',
855 855
 			'items'       => array(
856 856
 				'type' => 'integer',
857 857
 			),
858 858
 		);
859
-		$params['reviewer_email']   = array(
859
+		$params[ 'reviewer_email' ] = array(
860 860
 			'default'     => null,
861 861
 			'description' => __( 'Limit result set to that from a specific author email.', 'woocommerce-rest-api' ),
862 862
 			'format'      => 'email',
863 863
 			'type'        => 'string',
864 864
 		);
865
-		$params['product']          = array(
865
+		$params[ 'product' ] = array(
866 866
 			'default'     => array(),
867 867
 			'description' => __( 'Limit result set to reviews assigned to specific product IDs.', 'woocommerce-rest-api' ),
868 868
 			'type'        => 'array',
@@ -870,7 +870,7 @@  discard block
 block discarded – undo
870 870
 				'type' => 'integer',
871 871
 			),
872 872
 		);
873
-		$params['status']           = array(
873
+		$params[ 'status' ] = array(
874 874
 			'default'           => 'approved',
875 875
 			'description'       => __( 'Limit result set to reviews assigned a specific status.', 'woocommerce-rest-api' ),
876 876
 			'sanitize_callback' => 'sanitize_key',
Please login to merge, or discard this patch.
src/Controllers/Version4/TaxClasses.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 		$tax_classes = array();
94 94
 
95 95
 		// Add standard class.
96
-		$tax_classes[] = array(
96
+		$tax_classes[ ] = array(
97 97
 			'slug' => 'standard',
98 98
 			'name' => __( 'Standard rate', 'woocommerce-rest-api' ),
99 99
 		);
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 		$classes = \WC_Tax::get_tax_classes();
102 102
 
103 103
 		foreach ( $classes as $class ) {
104
-			$tax_classes[] = array(
104
+			$tax_classes[ ] = array(
105 105
 				'slug' => sanitize_title( $class ),
106 106
 				'name' => $class,
107 107
 			);
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 		foreach ( $tax_classes as $tax_class ) {
112 112
 			$class  = $this->prepare_item_for_response( $tax_class, $request );
113 113
 			$class  = $this->prepare_response_for_collection( $class );
114
-			$data[] = $class;
114
+			$data[ ] = $class;
115 115
 		}
116 116
 
117 117
 		return rest_ensure_response( $data );
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 	 * @return \WP_Error|\WP_REST_Response
125 125
 	 */
126 126
 	public function create_item( $request ) {
127
-		$tax_class = \WC_Tax::create_tax_class( $request['name'] );
127
+		$tax_class = \WC_Tax::create_tax_class( $request[ 'name' ] );
128 128
 
129 129
 		if ( is_wp_error( $tax_class ) ) {
130 130
 			return new \WP_Error( 'woocommerce_rest_' . $tax_class->get_error_code(), $tax_class->get_error_message(), array( 'status' => 400 ) );
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
 		$response = $this->prepare_item_for_response( $tax_class, $request );
146 146
 		$response = rest_ensure_response( $response );
147 147
 		$response->set_status( 201 );
148
-		$response->header( 'Location', rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $tax_class['slug'] ) ) );
148
+		$response->header( 'Location', rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $tax_class[ 'slug' ] ) ) );
149 149
 
150 150
 		return $response;
151 151
 	}
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
 	public function delete_item( $request ) {
160 160
 		global $wpdb;
161 161
 
162
-		$force = isset( $request['force'] ) ? (bool) $request['force'] : false;
162
+		$force = isset( $request[ 'force' ] ) ? (bool) $request[ 'force' ] : false;
163 163
 
164 164
 		// We don't support trashing for this type, error out.
165 165
 		if ( ! $force ) {
@@ -168,9 +168,9 @@  discard block
 block discarded – undo
168 168
 
169 169
 		$request->set_param( 'context', 'edit' );
170 170
 
171
-		$tax_class = \WC_Tax::get_tax_class_by( 'slug', sanitize_title( $request['slug'] ) );
171
+		$tax_class = \WC_Tax::get_tax_class_by( 'slug', sanitize_title( $request[ 'slug' ] ) );
172 172
 		$previous  = $this->prepare_item_for_response( $tax_class, $request );
173
-		$deleted   = \WC_Tax::delete_tax_class_by( 'slug', sanitize_title( $request['slug'] ) );
173
+		$deleted   = \WC_Tax::delete_tax_class_by( 'slug', sanitize_title( $request[ 'slug' ] ) );
174 174
 
175 175
 		if ( ! $deleted ) {
176 176
 			return new \WP_Error( 'woocommerce_rest_invalid_id', __( 'Invalid resource id.', 'woocommerce-rest-api' ), array( 'status' => 400 ) );
Please login to merge, or discard this patch.
src/Controllers/Version4/Utilities/DatabaseInformation.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -108,8 +108,8 @@
 block discarded – undo
108 108
 					'engine' => $table->engine,
109 109
 				);
110 110
 
111
-				$database_size['data']  += $table->data;
112
-				$database_size['index'] += $table->index;
111
+				$database_size[ 'data' ]  += $table->data;
112
+				$database_size[ 'index' ] += $table->index;
113 113
 			}
114 114
 		}
115 115
 
Please login to merge, or discard this patch.
src/Controllers/Version4/AbstractObjectsController.php 1 patch
Spacing   +61 added lines, -61 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	 * @return \WP_Error|\WP_REST_Response
66 66
 	 */
67 67
 	public function get_item( $request ) {
68
-		$object = $this->get_object( (int) $request['id'] );
68
+		$object = $this->get_object( (int) $request[ 'id' ] );
69 69
 
70 70
 		if ( ! $object || 0 === $object->get_id() ) {
71 71
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_id", __( 'Invalid ID.', 'woocommerce-rest-api' ), array( 'status' => 404 ) );
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 	 * @return \WP_Error|\WP_REST_Response
111 111
 	 */
112 112
 	public function create_item( $request ) {
113
-		if ( ! empty( $request['id'] ) ) {
113
+		if ( ! empty( $request[ 'id' ] ) ) {
114 114
 			/* translators: %s: post type */
115 115
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_exists", sprintf( __( 'Cannot create existing %s.', 'woocommerce-rest-api' ), $this->post_type ), array( 'status' => 400 ) );
116 116
 		}
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
 	 * @return \WP_Error|\WP_REST_Response
157 157
 	 */
158 158
 	public function update_item( $request ) {
159
-		$object = $this->get_object( (int) $request['id'] );
159
+		$object = $this->get_object( (int) $request[ 'id' ] );
160 160
 
161 161
 		if ( ! $object || 0 === $object->get_id() ) {
162 162
 			return new \WP_Error( "woocommerce_rest_{$this->post_type}_invalid_id", __( 'Invalid ID.', 'woocommerce-rest-api' ), array( 'status' => 404 ) );
@@ -199,42 +199,42 @@  discard block
 block discarded – undo
199 199
 	 */
200 200
 	protected function prepare_objects_query( $request ) {
201 201
 		$args                        = array();
202
-		$args['offset']              = $request['offset'];
203
-		$args['order']               = $request['order'];
204
-		$args['orderby']             = $request['orderby'];
205
-		$args['paged']               = $request['page'];
206
-		$args['post__in']            = $request['include'];
207
-		$args['post__not_in']        = $request['exclude'];
208
-		$args['posts_per_page']      = $request['per_page'];
209
-		$args['name']                = $request['slug'];
210
-		$args['post_parent__in']     = $request['parent'];
211
-		$args['post_parent__not_in'] = $request['parent_exclude'];
212
-		$args['s']                   = $request['search'];
213
-		$args['fields']              = 'ids';
214
-
215
-		if ( 'date' === $args['orderby'] ) {
216
-			$args['orderby'] = 'date ID';
202
+		$args[ 'offset' ]              = $request[ 'offset' ];
203
+		$args[ 'order' ]               = $request[ 'order' ];
204
+		$args[ 'orderby' ]             = $request[ 'orderby' ];
205
+		$args[ 'paged' ]               = $request[ 'page' ];
206
+		$args[ 'post__in' ]            = $request[ 'include' ];
207
+		$args[ 'post__not_in' ]        = $request[ 'exclude' ];
208
+		$args[ 'posts_per_page' ]      = $request[ 'per_page' ];
209
+		$args[ 'name' ]                = $request[ 'slug' ];
210
+		$args[ 'post_parent__in' ]     = $request[ 'parent' ];
211
+		$args[ 'post_parent__not_in' ] = $request[ 'parent_exclude' ];
212
+		$args[ 's' ]                   = $request[ 'search' ];
213
+		$args[ 'fields' ]              = 'ids';
214
+
215
+		if ( 'date' === $args[ 'orderby' ] ) {
216
+			$args[ 'orderby' ] = 'date ID';
217 217
 		}
218 218
 
219
-		$args['date_query'] = array();
219
+		$args[ 'date_query' ] = array();
220 220
 
221 221
 		// Set before into date query. Date query must be specified as an array of an array.
222
-		if ( isset( $request['before'] ) ) {
223
-			$args['date_query'][0]['before'] = $request['before'];
222
+		if ( isset( $request[ 'before' ] ) ) {
223
+			$args[ 'date_query' ][ 0 ][ 'before' ] = $request[ 'before' ];
224 224
 		}
225 225
 
226 226
 		// Set after into date query. Date query must be specified as an array of an array.
227
-		if ( isset( $request['after'] ) ) {
228
-			$args['date_query'][0]['after'] = $request['after'];
227
+		if ( isset( $request[ 'after' ] ) ) {
228
+			$args[ 'date_query' ][ 0 ][ 'after' ] = $request[ 'after' ];
229 229
 		}
230 230
 
231 231
 		// Set date query colummn. Defaults to post_date.
232
-		if ( isset( $request['date_column'] ) && ! empty( $args['date_query'][0] ) ) {
233
-			$args['date_query'][0]['column'] = 'post_' . $request['date_column'];
232
+		if ( isset( $request[ 'date_column' ] ) && ! empty( $args[ 'date_query' ][ 0 ] ) ) {
233
+			$args[ 'date_query' ][ 0 ][ 'column' ] = 'post_' . $request[ 'date_column' ];
234 234
 		}
235 235
 
236 236
 		// Force the post_type argument, since it's not a user input variable.
237
-		$args['post_type'] = $this->post_type;
237
+		$args[ 'post_type' ] = $this->post_type;
238 238
 
239 239
 		/**
240 240
 		 * Filter the query arguments for a request.
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
 		$total_posts = $query->found_posts;
265 265
 		if ( $total_posts < 1 ) {
266 266
 			// Out-of-bounds, run the query again without LIMIT for total count.
267
-			unset( $query_args['paged'] );
267
+			unset( $query_args[ 'paged' ] );
268 268
 			$count_query = new \WP_Query();
269 269
 			$count_query->query( $query_args );
270 270
 			$total_posts = $count_query->found_posts;
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
 		return array(
274 274
 			'objects' => array_filter( array_map( array( $this, 'get_object' ), $result ) ),
275 275
 			'total'   => (int) $total_posts,
276
-			'pages'   => (int) ceil( $total_posts / (int) $query->query_vars['posts_per_page'] ),
276
+			'pages'   => (int) ceil( $total_posts / (int) $query->query_vars[ 'posts_per_page' ] ),
277 277
 		);
278 278
 	}
279 279
 
@@ -288,17 +288,17 @@  discard block
 block discarded – undo
288 288
 		$query_results = $this->get_objects( $query_args );
289 289
 
290 290
 		$objects = array();
291
-		foreach ( $query_results['objects'] as $object ) {
291
+		foreach ( $query_results[ 'objects' ] as $object ) {
292 292
 			if ( ! Permissions::user_can_read( $this->post_type, $object->get_id() ) ) {
293 293
 				continue;
294 294
 			}
295 295
 
296 296
 			$data      = $this->prepare_item_for_response( $object, $request );
297
-			$objects[] = $this->prepare_response_for_collection( $data );
297
+			$objects[ ] = $this->prepare_response_for_collection( $data );
298 298
 		}
299 299
 
300
-		$total     = $query_results['total'];
301
-		$max_pages = $query_results['pages'];
300
+		$total     = $query_results[ 'total' ];
301
+		$max_pages = $query_results[ 'pages' ];
302 302
 
303 303
 		$response = rest_ensure_response( $objects );
304 304
 		$response = Pagination::add_pagination_headers( $response, $request, $total, $max_pages );
@@ -313,8 +313,8 @@  discard block
 block discarded – undo
313 313
 	 * @return \WP_REST_Response|\WP_Error
314 314
 	 */
315 315
 	public function delete_item( $request ) {
316
-		$force  = (bool) $request['force'];
317
-		$object = $this->get_object( (int) $request['id'] );
316
+		$force  = (bool) $request[ 'force' ];
317
+		$object = $this->get_object( (int) $request[ 'id' ] );
318 318
 		$result = false;
319 319
 
320 320
 		if ( ! $object || 0 === $object->get_id() ) {
@@ -424,10 +424,10 @@  discard block
 block discarded – undo
424 424
 	 */
425 425
 	public function get_collection_params() {
426 426
 		$params                       = array();
427
-		$params['context']            = $this->get_context_param();
428
-		$params['context']['default'] = 'view';
427
+		$params[ 'context' ]            = $this->get_context_param();
428
+		$params[ 'context' ][ 'default' ] = 'view';
429 429
 
430
-		$params['page'] = array(
430
+		$params[ 'page' ] = array(
431 431
 			'description'       => __( 'Current page of the collection.', 'woocommerce-rest-api' ),
432 432
 			'type'              => 'integer',
433 433
 			'default'           => 1,
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
 			'minimum'           => 1,
437 437
 		);
438 438
 
439
-		$params['per_page'] = array(
439
+		$params[ 'per_page' ] = array(
440 440
 			'description'       => __( 'Maximum number of items to be returned in result set.', 'woocommerce-rest-api' ),
441 441
 			'type'              => 'integer',
442 442
 			'default'           => 10,
@@ -446,28 +446,28 @@  discard block
 block discarded – undo
446 446
 			'validate_callback' => 'rest_validate_request_arg',
447 447
 		);
448 448
 
449
-		$params['search'] = array(
449
+		$params[ 'search' ] = array(
450 450
 			'description'       => __( 'Limit results to those matching a string.', 'woocommerce-rest-api' ),
451 451
 			'type'              => 'string',
452 452
 			'sanitize_callback' => 'sanitize_text_field',
453 453
 			'validate_callback' => 'rest_validate_request_arg',
454 454
 		);
455 455
 
456
-		$params['after'] = array(
456
+		$params[ 'after' ] = array(
457 457
 			'description'       => __( 'Limit response to resources created after a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
458 458
 			'type'              => 'string',
459 459
 			'format'            => 'date-time',
460 460
 			'validate_callback' => 'rest_validate_request_arg',
461 461
 		);
462 462
 
463
-		$params['before'] = array(
463
+		$params[ 'before' ] = array(
464 464
 			'description'       => __( 'Limit response to resources created before a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
465 465
 			'type'              => 'string',
466 466
 			'format'            => 'date-time',
467 467
 			'validate_callback' => 'rest_validate_request_arg',
468 468
 		);
469 469
 
470
-		$params['date_column'] = array(
470
+		$params[ 'date_column' ] = array(
471 471
 			'description'       => __( 'When limiting response using after/before, which date column to compare against.', 'woocommerce-rest-api' ),
472 472
 			'type'              => 'string',
473 473
 			'default'           => 'date',
@@ -480,14 +480,14 @@  discard block
 block discarded – undo
480 480
 			'validate_callback' => 'rest_validate_request_arg',
481 481
 		);
482 482
 
483
-		$params['modified_before'] = array(
483
+		$params[ 'modified_before' ] = array(
484 484
 			'description'       => __( 'Limit response to resources modified before a given ISO8601 compliant date.', 'woocommerce-rest-api' ),
485 485
 			'type'              => 'string',
486 486
 			'format'            => 'date-time',
487 487
 			'validate_callback' => 'rest_validate_request_arg',
488 488
 		);
489 489
 
490
-		$params['exclude'] = array(
490
+		$params[ 'exclude' ] = array(
491 491
 			'description'       => __( 'Ensure result set excludes specific IDs.', 'woocommerce-rest-api' ),
492 492
 			'type'              => 'array',
493 493
 			'items'             => array(
@@ -497,7 +497,7 @@  discard block
 block discarded – undo
497 497
 			'sanitize_callback' => 'wp_parse_id_list',
498 498
 		);
499 499
 
500
-		$params['include'] = array(
500
+		$params[ 'include' ] = array(
501 501
 			'description'       => __( 'Limit result set to specific ids.', 'woocommerce-rest-api' ),
502 502
 			'type'              => 'array',
503 503
 			'items'             => array(
@@ -507,14 +507,14 @@  discard block
 block discarded – undo
507 507
 			'sanitize_callback' => 'wp_parse_id_list',
508 508
 		);
509 509
 
510
-		$params['offset'] = array(
510
+		$params[ 'offset' ] = array(
511 511
 			'description'       => __( 'Offset the result set by a specific number of items.', 'woocommerce-rest-api' ),
512 512
 			'type'              => 'integer',
513 513
 			'sanitize_callback' => 'absint',
514 514
 			'validate_callback' => 'rest_validate_request_arg',
515 515
 		);
516 516
 
517
-		$params['order'] = array(
517
+		$params[ 'order' ] = array(
518 518
 			'description'       => __( 'Order sort attribute ascending or descending.', 'woocommerce-rest-api' ),
519 519
 			'type'              => 'string',
520 520
 			'default'           => 'desc',
@@ -522,7 +522,7 @@  discard block
 block discarded – undo
522 522
 			'validate_callback' => 'rest_validate_request_arg',
523 523
 		);
524 524
 
525
-		$params['orderby'] = array(
525
+		$params[ 'orderby' ] = array(
526 526
 			'description'       => __( 'Sort collection by object attribute.', 'woocommerce-rest-api' ),
527 527
 			'type'              => 'string',
528 528
 			'default'           => 'date',
@@ -538,7 +538,7 @@  discard block
 block discarded – undo
538 538
 		);
539 539
 
540 540
 		if ( $this->hierarchical ) {
541
-			$params['parent'] = array(
541
+			$params[ 'parent' ] = array(
542 542
 				'description'       => __( 'Limit result set to those of particular parent IDs.', 'woocommerce-rest-api' ),
543 543
 				'type'              => 'array',
544 544
 				'items'             => array(
@@ -548,7 +548,7 @@  discard block
 block discarded – undo
548 548
 				'default'           => array(),
549 549
 			);
550 550
 
551
-			$params['parent_exclude'] = array(
551
+			$params[ 'parent_exclude' ] = array(
552 552
 				'description'       => __( 'Limit result set to all items except those of a particular parent ID.', 'woocommerce-rest-api' ),
553 553
 				'type'              => 'array',
554 554
 				'items'             => array(
@@ -599,14 +599,14 @@  discard block
 block discarded – undo
599 599
 			}
600 600
 		}
601 601
 
602
-		$query_args['ignore_sticky_posts'] = true;
602
+		$query_args[ 'ignore_sticky_posts' ] = true;
603 603
 
604
-		if ( 'include' === $query_args['orderby'] ) {
605
-			$query_args['orderby'] = 'post__in';
606
-		} elseif ( 'id' === $query_args['orderby'] ) {
607
-			$query_args['orderby'] = 'ID'; // ID must be capitalized.
608
-		} elseif ( 'slug' === $query_args['orderby'] ) {
609
-			$query_args['orderby'] = 'name';
604
+		if ( 'include' === $query_args[ 'orderby' ] ) {
605
+			$query_args[ 'orderby' ] = 'post__in';
606
+		} elseif ( 'id' === $query_args[ 'orderby' ] ) {
607
+			$query_args[ 'orderby' ] = 'ID'; // ID must be capitalized.
608
+		} elseif ( 'slug' === $query_args[ 'orderby' ] ) {
609
+			$query_args[ 'orderby' ] = 'name';
610 610
 		}
611 611
 
612 612
 		return $query_args;
@@ -694,13 +694,13 @@  discard block
 block discarded – undo
694 694
 	 * @return array
695 695
 	 */
696 696
 	protected function add_meta_query( $args, $meta_query ) {
697
-		if ( empty( $args['meta_query'] ) ) {
698
-			$args['meta_query'] = []; // phpcs:ignore
697
+		if ( empty( $args[ 'meta_query' ] ) ) {
698
+			$args[ 'meta_query' ] = [ ]; // phpcs:ignore
699 699
 		}
700 700
 
701
-		$args['meta_query'][] = $meta_query;
701
+		$args[ 'meta_query' ][ ] = $meta_query;
702 702
 
703
-		return $args['meta_query'];
703
+		return $args[ 'meta_query' ];
704 704
 	}
705 705
 
706 706
 	/**
Please login to merge, or discard this patch.