WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#25689 closed defect (bug) (fixed)

Notice - undefined variable checksums

Reported by: GaryJ Owned by: nacin
Milestone: 3.7.1 Priority: normal
Severity: normal Version:
Component: Warnings/Notices Keywords: has-patch
Focuses: Cc:

Description

Saw a PHP Notice referencing an undefined variable checksums during the manual update to 3.7. The page redirected to the About page before I caught the file and reference number, but Nacin says that "Line 753 wp-admin/includes/update-core.php needs an isset() check", so my guess is that line 753 wp-admin/includes/update-core.php needs an isset() check...

Attachments (1)

25689.diff (588 bytes) - added by GaryJ 6 years ago.
Add isset() check

Download all attachments as: .zip

Change History (5)

#1 @nacin
6 years ago

  • Milestone changed from Awaiting Review to 3.7.1

@GaryJ
6 years ago

Add isset() check

#2 @GaryJ
6 years ago

  • Keywords has-patch added; needs-patch removed

Well, fancy that. Seems Nacin was right ;-) 25689.diff added.

#3 @nacin
6 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 25915:

Avoid a notice for an undefined $checksums variable when updating from pre-3.7 to post-3.7.

props GaryJ.
fixes #25689 for trunk.

#4 @nacin
6 years ago

In 25916:

Avoid a notice for an undefined $checksums variable when updating from pre-3.7 to post-3.7.

Merges [25915] to the 3.7 branch.

props GaryJ.
fixes #25689.

Note: See TracTickets for help on using tickets.