Opened 8 years ago
Closed 8 years ago
#39209 closed enhancement (fixed)
replace 1 check with current_user_can( 'manage_network' ) in wp-signup.php
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Role/Capability | Keywords: | has-patch |
Focuses: | multisite | Cc: |
Description (last modified by )
replace 1 check with current_user_can( 'manage_network' )
in wp-signup.php
;
in addition, change the message string to Greetings Network Administrator! The network currently allows “%s” registrations. To change or disable registration go to your <a href="%s">Options page</a>.
and the $i18n['blog']
string to site to match current naming conventions
Attachments (3)
Change History (11)
#2
@
8 years ago
- Focuses multisite added
- Keywords needs-refresh added
- Milestone changed from Awaiting Review to 4.8
- Owner set to flixos90
- Status changed from new to assigned
Thanks for the patch @jignesh.nakrani! Can you update the capability used there? In the patch you currently have manage_options
, but it should be manage_network
. The rest of it looks good. :)
#3
@
8 years ago
- Description modified (diff)
- Keywords needs-patch added; has-patch needs-refresh removed
#5
@
8 years ago
- Keywords needs-refresh added
The capability change here looks good. I think we should take care of changing the text in another ticket so that it's more obvious from looking at the changeset what's actually being changed here.
Replaced is_super_admin() check with current_user_can( 'manage_options' )