Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 11 years ago

#25622 closed defect (bug) (fixed)

Plugin change log link incorrect on multisite updates screen

Reported by: mindctrl Owned by: nacin
Priority: normal Milestone: 3.7
Component: Multisite Version: 3.7
Severity: normal Keywords:
Cc: Focuses:

Description

In 3.7-beta2-25801 multisite, there's a problem with the link that shows the change log when there's an update available for plugins. I've tested on two clean multisite installs. When visiting the wp-admin/network/update-core.php screen, the "View version x.x details" link is missing the "network" piece of the URL. When the iframe pops, you get the "Install Plugins" screen instead of the plugin change log.

The link is not broken on the main Plugins screen. It correctly displays the change log.

Visit
​http://37beta2/wp-admin/network/update-core.php

The link looks like this (note it's missing "network" after "wp-admin")
​http://37beta2/wp-admin/plugin-install.php?tab=plugin-information&plugin=plugin-name&section=changelog&TB_iframe=true&width=640&height=670

Visit
​http://37beta2/wp-admin/network/plugins.php

The link looks like this (link is correct)
​http://37beta2/wp-admin/network/plugin-install.php?tab=plugin-information&plugin=plugin-name&section=changelog&TB_iframe=true&width=600&height=800

Change History (7)

#1 @mindctrl
13 years ago

  • Cc public@… added

#2 @jeremyfelt
13 years ago

  • Milestone Awaiting Review → 3.7
  • Severity major → normal

Confirmed. The is_admin and is_network properties of the $current_site global are reset by a new set_current_screen( 'tools' ); call when ​requesting file system credentials in the Automatic Upgrader Skin

Introduced in [25228]

Last edited 13 years ago by jeremyfelt (previous) (diff)

#3 follow-up: @nacin
13 years ago

I don't think the set_current_screen() call is required anymore, thanks to [25806].

#4 in reply to: ↑ 3 @dd32
13 years ago

Replying to nacin:

I don't think the set_current_screen() call is required anymore, thanks to [25806].

Off the top of my head, that would be correct. Removing it shouldn't cause any problems.

#5 @nacin
13 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 25836:

set_current_screen() is now unnecessary in Automatic_Upgrader_Skin, which is good because it breaks things in the network admin.

fixes #25622.

#6 @SergeyBiryukov
13 years ago

  • Version trunk → 3.7

This ticket was mentioned in ​Slack in #core by chriscct7. ​View the logs.


11 years ago

Note: See TracTickets for help on using tickets.