[wp-trac] [WordPress Trac] #37593: Replace "Super Admin" with "Network Administrator"
WordPress Trac
noreply at wordpress.org
Tue Sep 27 10:52:23 UTC 2016
#37593: Replace "Super Admin" with "Network Administrator"
--------------------------------------------------+------------------------
Reporter: flixos90 | Owner: vizkr
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 4.7
Component: Administration | Version:
Severity: normal | Resolution:
Keywords: good-first-bug has-patch 2nd-opinion | Focuses: multisite
--------------------------------------------------+------------------------
Changes (by flixos90):
* keywords: good-first-bug has-patch => good-first-bug has-patch 2nd-
opinion
Comment:
Thanks for the patch @Mista-Flo!
@vizkr This ticket is only about the user-facing names in the admin, so we
don't need to adjust any actual PHP code here (and we most likely couldn't
because it would break backward compatibility).
Regarding the patch, it looks good for the most part. Only one thing,
could you always use the term "administrator" instead of "admin"? I see
you replaced some of the "super admin" terms with "network admin" instead
of "network administrator". I think we should be consistent here.
Furthermore there is a function `is_network_admin()` in Core that doesn't
actually deal with the user, but the administration panel. My personal
assumption is that "administrator" denotes a user while "admin" denotes an
administration panel, so we should use the first here (these names are far
from perfect, but somehow established themselves). Another opinion on this
would be much appreciated though, what do you think @jeremyfelt?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37593#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list