[wp-trac] [WordPress Trac] #16501: 'Add Existing' button should be dependent on is_multisite

[WordPress Trac]

#16501: 'Add Existing' button should be dependent on is_multisite
------------------------------+-----------------------
 Reporter:  nacin             |       Owner:
     Type:  defect (bug)      |      Status:  reopened
 Priority:  normal            |   Milestone:  3.1
Component:  Role/Capability   |     Version:
 Severity:  major             |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+-----------------------
Changes (by linuxologos):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Replying to [comment:9 nacin]:
 > Flip-side now: Should the remove_user(s) branches in users.php also have
 multisite checks? Doubt there's anything dangerous there, but still.
 In single-site installs, if a user is given the remove_users cap but not
 the delete_users one, if he tries to remove a user through the Bulk
 Actions dropdown (does this really need to be visible?), he is sent to the
 "Remove Users from Site" screen. I don't know if this already is something
 that should not happen, but if he then presses "Confirm Removal" he gets
 an error like:

 {{{
 Fatal error: Call to undefined function remove_user_from_blog() in
 F:\xampplite\htdocs\31rc4\wp-admin\users.php on line 224
 }}}
 This applies to WP 3.0.5 too.

 {{{
 Fatal error: Call to undefined function remove_user_from_blog() in
 F:\xampplite\htdocs\305\wp-admin\users.php on line 238
 }}}

 (If the user has both remove_users and delete_users, the latter
 "overrides" the former and the problem is not obvious).

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16501#comment:14>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software