WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#13897 closed defect (bug) (fixed)

Typo in class-http.php

Reported by: filosofo Owned by:
Milestone: 3.0.1 Priority: normal
Severity: normal Version: 3.0
Component: HTTP API Keywords: has-patch dev-feedback
Focuses: Cc:

Description

"scheme" is an array index of $arrURL, not $url (a string).

Attachments (3)

http-request-typo.13897.diff (480 bytes) - added by filosofo 5 years ago.
13897.patch (5.3 KB) - added by hakre 5 years ago.
13897.2.patch (3.4 KB) - added by hakre 5 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 @hakre5 years ago

just saw, in case someone is on it, let some whitespace in 241:

$arrURL = parse_url( $url );

@hakre5 years ago

comment:2 @hakre5 years ago

While we did a review in IRC and I had my fingers in that file fixing whitespaces in that function I just added it here.

Nice find of the problem btw, the faulty code never does notices. I would even consider to see the single problem fixed in 3.0.0.

comment:3 @dd325 years ago

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

(In [15283]) Fix typo in HTTP Class. Props filosofo. Fixes #13897 for 3.0

comment:4 @dd325 years ago

(In [15284]) Fix typo in HTTP Class. Props filosofo. Fixes #13897 for trunk

comment:5 @dd325 years ago

(In [15285]) Some coding standards cleanups for WP_HTTP. Props hakre for initial patch. See #13897

@hakre5 years ago

comment:6 @hakre5 years ago

A little coding update against very current trunk, those whitespace / coding standards cleanups have not been taken in yet.

comment:7 @hakre5 years ago

  • Keywords dev-feedback added

comment:8 @nacin5 years ago

(In [15309]) Some whitespace in WP_HTTP. props hakre, see #13897.

Note: See TracTickets for help on using tickets.