Changeset 38478

Timestamp:

08/31/2016 07:00:59 PM (8 years ago)

Author:

westonruter

Message:

Customize: Use new $status_code parameter for wp_send_json_error() instead of calling status_header() separately.

Props johnbillion.
See #35666.
Fixes #37897.

Location:

trunk/src/wp-includes

Files:

• class-wp-customize-nav-menus.php (modified) (1 diff)
• customize/class-wp-customize-selective-refresh.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-customize-nav-menus.php

@@ -750 +750 @@
     public function ajax_insert_auto_draft_post() {
         if ( ! check_ajax_referer( 'customize-menus', 'customize-menus-nonce', false ) ) {
-            status_header( 400 );
-            wp_send_json_error( 'bad_nonce' );
+            wp_send_json_error( 'bad_nonce', 400 );
         }
 
         if ( ! current_user_can( 'customize' ) ) {
-            status_header( 403 );
-            wp_send_json_error( 'customize_not_allowed' );
+            wp_send_json_error( 'customize_not_allowed', 403 );
         }
 
         if ( empty( $_POST['params'] ) || ! is_array( $_POST['params'] ) ) {
-            status_header( 400 );
-            wp_send_json_error( 'missing_params' );
+            wp_send_json_error( 'missing_params', 400 );
         }
 

trunk/src/wp-includes/customize/class-wp-customize-selective-refresh.php

@@ -316 +316 @@
          */
         if ( ! is_customize_preview() ) {
-            status_header( 403 );
-            wp_send_json_error( 'expected_customize_preview' );
+            wp_send_json_error( 'expected_customize_preview', 403 );
         } else if ( ! isset( $_POST['partials'] ) ) {
-            status_header( 400 );
-            wp_send_json_error( 'missing_partials' );
+            wp_send_json_error( 'missing_partials', 400 );
         }