WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#28648 closed defect (bug) (fixed)

$homeURL['host'] missing null check on site install: Notice: Undefined index: host in /wp-includes/class-http.php on line 186

Reported by: meekyhwang's profile meekyhwang Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.0 Priority: normal
Severity: minor Version: 4.0
Component: HTTP API Keywords: has-patch
Focuses: Cc:

Description

Notice: Undefined index: host in /wp-includes/class-http.php on line 186
It is an warning which does not break the site, only shows up with debugger on.

Attachments (1)

28648.patch​ (633 bytes) - added by meekyhwang 10 years ago.
empty check to fix the issue.

Download all attachments as: .zip

Change History (9)

#1 @meekyhwang
10 years ago

  • Severity changed from normal to minor

@meekyhwang
10 years ago

empty check to fix the issue.

#2 @meekyhwang
10 years ago

  • Keywords has-patch needs-testing added

#3 @SergeyBiryukov
10 years ago

  • Component changed from General to HTTP API
  • Keywords reporter-feedback added

Could you provide the steps to reproduce the issue? Why would get_bloginfo( 'url' ) return a URL without a host name?

#4 @meekyhwang
10 years ago

It happened when I tried to install the site for the first time when it redirected to /wp-admin/install.php. (the warning appeared on the screen where it asks you to set up the language before setting up the site admin account.) get_bloginfo( 'url' ) is not set at this point.

Last edited 10 years ago by SergeyBiryukov (previous) (diff)

#5 @meekyhwang
10 years ago

  • Keywords reporter-feedback removed

#6 @SergeyBiryukov
10 years ago

  • Milestone changed from Awaiting Review to 4.0

#7 @SergeyBiryukov
10 years ago

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

In 28865:

Avoid a PHP notice and warning on language selection screen.

props meekyhwang.
fixes #28648.

#8 @SergeyBiryukov
10 years ago

  • Keywords needs-testing removed

Introduced in [28774].

Note: See TracTickets for help on using tickets.