Changeset 16992 for trunk/wp-admin/includes/class-wp-ms-themes-list-table.php

Timestamp:

12/16/2010 09:18:28 AM (14 years ago)

Author:

nacin

Message:

Replace check_permissions() with ajax_user_can(). New method returns true/false to current_user_can(), which we then handle in admin ajax. see #15326.

File:

• trunk/wp-admin/includes/class-wp-ms-themes-list-table.php (modified) (1 diff)

trunk/wp-admin/includes/class-wp-ms-themes-list-table.php

@@ -37 +37 @@
     }
 
-    function check_permissions() {
+    function ajax_user_can() {
         $menu_perms = get_site_option( 'menu_items', array() );
 
         if ( empty( $menu_perms['themes'] ) && ! is_super_admin() )
-            wp_die( __( 'Cheatin’ uh?' ) );
+            return false;
 
         if ( $this->is_site_themes && !current_user_can('manage_sites') )
-            wp_die( __( 'You do not have sufficient permissions to manage themes for this site.' ) );
+            return false;
         elseif ( !$this->is_site_themes && !current_user_can('manage_network_themes') )
-            wp_die( __( 'You do not have sufficient permissions to manage network themes.' ) );
+            return false;
+        return true;
     }