Changeset 33884

Timestamp:

09/03/2015 08:44:37 AM (9 years ago)

Author:

SergeyBiryukov

Message:

Provide more helpful feedback than just "Cheatin' uh?" for permission errors in wp-admin/user-new.php.

props ericlewis, kraftbj, lukecarbis, mrmist.
fixes #33678. see #14530.

File:

• trunk/src/wp-admin/user-new.php (modified) (3 diffs)

trunk/src/wp-admin/user-new.php

@@ -11 +11 @@
 
 if ( is_multisite() ) {
-    if ( ! current_user_can( 'create_users' ) && ! current_user_can( 'promote_users' ) )
-        wp_die( __( 'Cheatin’ uh?' ), 403 );
+    if ( ! current_user_can( 'create_users' ) && ! current_user_can( 'promote_users' ) ) {
+        wp_die(
+            '<h1>' . __( 'Cheatin&#8217; uh?' ) . '</h1>' .
+            '<p>' . __( 'You do not have sufficient permissions to add users to this network.' ) . '</p>',
+            403
+        );
+    }
 } elseif ( ! current_user_can( 'create_users' ) ) {
-    wp_die( __( 'Cheatin&#8217; uh?' ), 403 );
+    wp_die(
+        '<h1>' . __( 'Cheatin&#8217; uh?' ) . '</h1>' .
+        '<p>' . __( 'You are not allowed to create users.' ) . '</p>',
+        403
+    );
 }
 
@@ -60 +69 @@
     }
 
-    if ( ! current_user_can('promote_user', $user_details->ID) )
-        wp_die( __( 'Cheatin&#8217; uh?' ), 403 );
+    if ( ! current_user_can( 'promote_user', $user_details->ID ) ) {
+        wp_die(
+            '<h1>' . __( 'Cheatin&#8217; uh?' ) . '</h1>' .
+            '<p>' . __( 'You do not have sufficient permissions to add users to this network.' ) . '</p>',
+            403
+        );
+    }
 
     // Adding an existing user to this blog
@@ -109 +123 @@
     check_admin_referer( 'create-user', '_wpnonce_create-user' );
 
-    if ( ! current_user_can('create_users') )
-        wp_die( __( 'Cheatin&#8217; uh?' ), 403 );
+    if ( ! current_user_can( 'create_users' ) ) {
+        wp_die(
+            '<h1>' . __( 'Cheatin&#8217; uh?' ) . '</h1>' .
+            '<p>' . __( 'You are not allowed to create users.' ) . '</p>',
+            403
+        );
+    }
 
     if ( ! is_multisite() ) {