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

WordPress Trac wp-trac at lists.automattic.com
Wed Feb 9 16:27:34 UTC 2011


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

 * keywords:   => has-patch commit


Comment:

 I gave MarkJaquith a walkthrough in IRC (posted below). Want ryan and
 westi to as well.

 wp-admin/user.php: First piece, add an additional promote_users cap check
 before the promote_user meta cap checks in the post handler for bulk
 action promotions.

 Second piece, don't show the 'Add Existing' button in single-site mode
 when you can't create new users. (All users are shown even if they don't
 have a blog, such as shared user tables.)

 wp-users-list-table: Don't show the bulk actions if you can't
 promote_users. It doesn't work post-side.

 wp-admin/user-new.php: Check my logic, but — deny if you don't have
 create_users. Exception is that on multisite, you can be missing
 create_users but have promote_users.

 wp-admin/menu.php: We need user-new.php accessible for those who have
 create_users but not promote_users. Trick is to do a cap check then pass
 the right cap to $submenu.

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


More information about the wp-trac mailing list