![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 10 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: | jignesh.nakrani | Owned by: | flixos90 |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.8 |
| Component: | Role/Capability | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | multisite |
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
@
10 years ago
- Focuses multisite added
- Keywords needs-refresh added
- Milestone Awaiting Review → 4.8
- Owner set to
- Status new → 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
@
10 years ago
- Description modified (diff)
- Keywords needs-patch added; has-patch needs-refresh removed
#5
@
10 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' )