[wp-trac] [WordPress Trac] #37836: Cannot disable 'Add existing user'
WordPress Trac
noreply at wordpress.org
Tue Oct 11 17:50:34 UTC 2016
#37836: Cannot disable 'Add existing user'
------------------------------+--------------------------------------------
Reporter: bseddon | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Users | Version: 3.1
Severity: normal | Resolution:
Keywords: needs-testing | Focuses: ui, administration, multisite
has-patch |
------------------------------+--------------------------------------------
Comment (by Mista-Flo):
Here's a second patch. Thanks to @fperdaan for the review.
Since the conditional test of $do_both is already made line 13 with the
is_multisite, and user cap test, it does not necessary to test again with
$both var, we only need to test if it's multisite.
By the way the code of this file is really hard to read, a refactoring
would be a nice thing to do to avoid DRY code.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37836#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list