Changeset 30168

Timestamp:

11/01/2014 10:53:35 PM (10 years ago)

Author:

wonderboymusic

Message:

In wp_ajax_get_tagcloud(), bail immediately if $_POST['tax'] isn't set so that all of the variable setting can happen in the same nest scope as the rest of the function - wp_die() confuses Scrutinizer.

See #30224.

File:

• trunk/src/wp-admin/includes/ajax-actions.php (modified) (1 diff)

trunk/src/wp-admin/includes/ajax-actions.php

@@ -820 +820 @@
  */
 function wp_ajax_get_tagcloud() {
-    if ( isset( $_POST['tax'] ) ) {
-        $taxonomy = sanitize_key( $_POST['tax'] );
-        $tax = get_taxonomy( $taxonomy );
-        if ( ! $tax )
-            wp_die( 0 );
-        if ( ! current_user_can( $tax->cap->assign_terms ) )
-            wp_die( -1 );
-    } else {
+    if ( ! isset( $_POST['tax'] ) ) {
         wp_die( 0 );
+    }
+
+    $taxonomy = sanitize_key( $_POST['tax'] );
+    $tax = get_taxonomy( $taxonomy );
+    if ( ! $tax ) {
+        wp_die( 0 );
+    }
+    
+    if ( ! current_user_can( $tax->cap->assign_terms ) ) {
+        wp_die( -1 );
     }