Merge pull request #6 from elevati/development

Salil Kothadia · web-flow · commit 62a03f2d8737 · 2017-03-06T10:11:17.000+05:30
API Params Validation to return proper status instead of 500
diff --git a/lib/endpoints/class-wp-rest-multiple-posttype-controller.php b/lib/endpoints/class-wp-rest-multiple-posttype-controller.php
@@ -69,7 +69,7 @@ public function get_items($request) {
 
         # Param Validation
         if (empty($args['post_type']) || !is_array($args['post_type'])) {
-            return new WP_Error('type_need_to_be_array', 'Type Param is a required parameter and needs to be an array: &type[]=post&type[]=page&type[]=slideshow');
+            return new WP_Error('type_need_to_be_array', 'Type Param is a required parameter and needs to be an array: &type[]=post&type[]=page&type[]=slideshow', ['status' => 422]);
         }
 
         $args['date_query'] = array();

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ public function get_items($request) {`
`69`	`69`
`70`	`70`	`# Param Validation`
`71`	`71`	`if (empty($args['post_type']) \|\| !is_array($args['post_type'])) {`
`72`		`- return new WP_Error('type_need_to_be_array', 'Type Param is a required parameter and needs to be an array: &type[]=post&type[]=page&type[]=slideshow');`
	`72`	`+ return new WP_Error('type_need_to_be_array', 'Type Param is a required parameter and needs to be an array: &type[]=post&type[]=page&type[]=slideshow', ['status' => 422]);`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`$args['date_query'] = array();`