Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#8772 closed defect (bug) (wontfix)

replace the filter 'use_http_extension_transport' with 'use_exthttp_transport'

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


The filter to disable the PHP HTTP Extension differs in naming from the rest of the filters, This makes it slightly awkward for plugins to manage the enabling/disabling due to the filters name differing from the class name, Ignoring the fact that plugins can work around it, It seems worthy that such a change should be made to at least bring consistency.

The attached patch depreciates the current filter name, and introduces the new. The current filter continues to work via a depreciated function hook, I dont think removing it entirely would be appreciated, And whilst it could just use 2 apply_filter() calls in the HTTP class, This seemed a better option.

Note: Theres no function to mark a filter as depreciated, i guess they've generally just been removed..

As much as i'd like this in 2.7, i'm leaving it open to consideration if its even going to make it into 2.8, I'll work around it in my plugin for now..

Attachments (1)

8772.diff (2.2 KB) - added by DD32 7 years ago.

Download all attachments as: .zip

Change History (4)

7 years ago

#1 @DD32
7 years ago

  • Keywords dev-feedback added

Description = s/depreciated/deprecated/g

#2 @jacobsantos
7 years ago

That the other filters match the class name is a coincidence and nothing more.

#3 @ryan
7 years ago

  • Milestone 2.8 deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.