[wp-trac] [WordPress Trac] #44176: Un-map Privacy Capabilities

WordPress Trac noreply at wordpress.org
Tue Jan 28 20:02:51 UTC 2020


#44176: Un-map Privacy Capabilities
-------------------------------------------------+-------------------------
 Reporter:  desrosj                              |       Owner:  xkon
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.4
Component:  Privacy                              |     Version:  4.9.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-screenshots early      |     Focuses:
  needs-dev-note needs-testing commit            |
-------------------------------------------------+-------------------------
Changes (by pbiron):

 * keywords:  has-patch has-screenshots early needs-dev-note needs-testing
     => has-patch has-screenshots early needs-dev-note needs-testing commit


Comment:

 Replying to [comment:40 garrett-eclipse]:
 > It would be nice to get some testing on multisite as the change from
 meta caps to primitive ones means we're adding the capabilities to the
 administrator and editor and there's no longer the is_network conditional;
 > ` $caps[] = is_multisite() ? 'manage_network' : 'manage_options';`

 I've given the latest patch a spin in multisite.  It seems to work as
 expected for `super admins`, "regular" `administrator`, `editor`, as well
 as a custom `DPO` role to which I've granted all 3 of the new primitive
 caps via the URE plugin.

 I think this is ready to commit (I've added the commit keyword).  Doing so
 will allow more testing via the Beta Tester plugin!  Reminder to
 committers: the 2 places in `wp-admin/includes/upgrade.php` where the
 `$wp_current_db_version` is compared will need to be updated prior to
 commiting!!

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/44176#comment:41>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list