[wp-trac] [WordPress Trac] #14786: It's WP_Http_ExtHTTP not WP_Http_ExtHttp

[WordPress Trac]

#14786: It's WP_Http_ExtHTTP not WP_Http_ExtHttp
--------------------------+-------------------------------------------------
 Reporter:  hakre         |       Owner:                        
     Type:  defect (bug)  |      Status:  new                   
 Priority:  normal        |   Milestone:  Awaiting Review       
Component:  HTTP          |     Version:  3.0.1                 
 Severity:  normal        |    Keywords:  has-patch dev-feedback
--------------------------+-------------------------------------------------
Changes (by dd32):

  * component:  General => HTTP


Comment:

 {{{
 is_scalar( $named ) && ( $named = array( (string) $named ) );
 foreach ( $named as $context ) {
 }}}

 Can be simplified to:

 {{{
 foreach ( (array)$named as $context ) {
 }}}

 However, $context is undefined outside of the loop in the return
 statements of the code.

 I think originally, The priority orders for Get/Post were different, which
 is why there are 2 methods?

 I'd rather see WP_HTTP_Fopen::test() return false for POST requests to
 limit the locations which detects which transport is available

 Related Tickets: #8622 #11613

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/14786#comment:8>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software