[wp-trac] [WordPress Trac] #39220: Remove 1 `is_super_admin()` check in wp-admin/user-new.php

WordPress Trac noreply at wordpress.org
Tue Dec 13 17:27:41 UTC 2016


#39220: Remove 1 `is_super_admin()` check in wp-admin/user-new.php
-----------------------------+------------------------
 Reporter:  supercoder       |       Owner:  flixos90
     Type:  enhancement      |      Status:  reviewing
 Priority:  normal           |   Milestone:  4.8
Component:  Role/Capability  |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  needs-patch      |     Focuses:  multisite
-----------------------------+------------------------

Comment (by supercoder):

 Replying to [comment:2 flixos90]:
 > Hi @supercoder, and thanks for the ticket and patch! :)
 >
 > Can you maybe have another look at the patch and review "Ticket 11" on
 https://core.trac.wordpress.org/ticket/37616#comment:26 in detail? In your
 patch you removed the `$username != null` check, but instead the
 `!is_super_admin( $user_id )` check should be removed. You could also
 adjust the spacing in `( array_key_exists($blog_id,
 get_blogs_of_user($user_id)) )` (same line) to follow the WordPress coding
 standards.

 Thanks for the update. I have updated the patch. Please check.

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


More information about the wp-trac mailing list