Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#37073 closed enhancement (wontfix)

Remove deprecated argument from wp_get_http_headers

Reported by: borgesbruno's profile borgesbruno Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch has-unit-tests
Focuses: Cc:

Description

There is a deprecated argument in wp_get_http_headers since the 2.7 WordPress version that isn't used in any place, actually there is only one call for this function in src/wp-includes/functions.php if ( $headers = wp_get_http_headers( $url) ) {, which do not use this argument, so, seems there is no need for this argument, in aditional I've wrote tests to cover this function and improve the coverage.

If there is some utility for this let me know, because the unit tests will be useful

Attachments (1)

37073.diff (1.7 KB) - added by borgesbruno 8 years ago.

Download all attachments as: .zip

Change History (4)

@borgesbruno
8 years ago

#1 @borgesbruno
8 years ago

  • Keywords has-patch has-unit-tests added

#2 @rmccue
8 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

This argument is still on the function for two reasons: a) so that people who were using it get a deprecation warning, and b) so that we don't accidentally introduce a conflicting argument in future changes.

The unit tests might be useful though; can you submit that as a new ticket please?

#3 @borgesbruno
8 years ago

Tks @rmccue, I had asked about that on slack, but nobody has said that for me.
This is the new ticket #37090

Last edited 8 years ago by borgesbruno (previous) (diff)
Note: See TracTickets for help on using tickets.