#27869 closed defect (bug) (fixed)
Theme install return wrong install_url on network admin
| Reported by: | hirizh | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9.1 |
| Component: | Themes | Version: | 3.9 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | administration, multisite |
Description
When installing a theme on network admin, the install url is pointing to wp-admin/update.php?action=install-theme instead of wp-admin/network/update.php?action=install-theme. From here, we'll be redirected to the first site dashboard view after the theme installed successfully.
Its not happening with plugins. So I thought its because the ajax call by the themes query. Maybe it relates to #22589.
Attachments (2)
Change History (8)
#2
@
12 years ago
- Component Networks and Sites → Themes
- Keywords has-patch added; needs-patch removed
#3
@
12 years ago
- Component Themes → Networks and Sites
Well, that could have broken in far worse ways. :-)
I think there is a simpler patch for this. 27869.diff.
network_admin_url() should be used in theme-install.php too (where it is changed in 27869.patch. self_admin_url() is appropriate for pages that work both inside and outside the network admin even when in multisite (like the plugins list table), while network_admin_url() is good for a page that is network-admin-only in multisite. We probably use self_admin_url() in a few places where network_admin_url() is more appropriate, but you only really notice it in odd situations like this, where admin-ajax.php doesn't have context.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Confirmed. Introduced by [28126].