Opened 9 years ago
Closed 9 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
@
9 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
@
9 years ago
- Description modified (diff)
- Keywords needs-patch added; has-patch needs-refresh removed
#5
@
9 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' )