WordPress.org

Make WordPress Core

Ticket #14786: 14786.10.patch

File 14786.10.patch, 5.6 KB (added by hakre, 4 years ago)

Strictly there is no need of an iterator inside the function.

  • wp-includes/class-http.php

     
    5959         * @return WP_Http 
    6060         */ 
    6161        function __construct() { 
    62                 WP_Http::_getTransport(); 
    63                 WP_Http::_postTransport(); 
     62                foreach ( array( 'get', 'post' ) as $context ) 
     63                        WP_Http::_namedTransport( $context ); 
    6464        } 
    6565 
    6666        /** 
     67         * Tests through all WP_Http objects and stores the first match  
     68         * and returns. 
     69         *  
     70         * Test Order: 
     71         *  
     72         *   1. exthttp  : WP_Http_ExtHTTP 
     73         *   2. curl     : WP_Http_Curl 
     74         *   3. streams  : WP_Http_Streams 
     75         *   4. fopen    : WP_Http_Fopen 
     76         *   5. fsockopen: WP_Http_Fsockopen 
     77         *  
     78         * @since 3.1.0 
     79         * @access private 
     80         * @param string $context named context 
     81         * @param array $args (optional) Request arguments 
     82         * @return object|null Concrete HTTP Transport Object; Null if no transport was found. 
     83         */ 
     84        function &_namedTransport( $context, $args = array() ) { 
     85                static $working, $blocking, $nonblocking; 
     86 
     87                if ( !isset( $working[$context] ) ) { 
     88                        $transports = array( 
     89                                'exthttp'   => 'ExtHTTP', 
     90                                'curl'      => 'Curl', 
     91                                'streams'   => 'Streams', 
     92                                'fopen'     => 'Fopen', 
     93                                'fsockopen' => 'Fsockopen', 
     94                        ); 
     95 
     96                        foreach ( $transports as $name => $spec ) { 
     97                                $class = 'WP_Http_' . $spec; 
     98                                if ( !call_user_func( array( $class, 'test' ), $args ) ) 
     99                                        continue; 
     100 
     101                                $wp_http = new $class(); 
     102                                 
     103                                $working[$context][$name] = &$wp_http;  
     104                                $blocking[$context][]     = &$wp_http; 
     105                                $nonblocking[$context][]  = &$wp_http; 
     106 
     107                                break; 
     108                        } 
     109                } 
     110 
     111                do_action( 'http_transport_get_debug', $working[$context], $blocking[$context], $nonblocking[$context] ); 
     112 
     113                if ( isset( $args['blocking'] ) && !$args['blocking'] ) 
     114                        return $nonblocking[$context]; 
     115                else 
     116                        return $blocking[$context]; 
     117                 
     118        } 
     119 
     120        /** 
    67121         * Tests the WordPress HTTP objects for an object to use and returns it. 
    68122         * 
    69123         * Tests all of the objects and returns the object that passes. Also caches that object to be 
     
    86140         * @return object|null Null if no transports are available, HTTP transport object. 
    87141         */ 
    88142        function &_getTransport( $args = array() ) { 
    89                 static $working_transport, $blocking_transport, $nonblocking_transport; 
    90  
    91                 if ( is_null($working_transport) ) { 
    92                         if ( true === WP_Http_ExtHttp::test($args) ) { 
    93                                 $working_transport['exthttp'] = new WP_Http_ExtHttp(); 
    94                                 $blocking_transport[] = &$working_transport['exthttp']; 
    95                         } else if ( true === WP_Http_Curl::test($args) ) { 
    96                                 $working_transport['curl'] = new WP_Http_Curl(); 
    97                                 $blocking_transport[] = &$working_transport['curl']; 
    98                         } else if ( true === WP_Http_Streams::test($args) ) { 
    99                                 $working_transport['streams'] = new WP_Http_Streams(); 
    100                                 $blocking_transport[] = &$working_transport['streams']; 
    101                         } else if ( true === WP_Http_Fopen::test($args) ) { 
    102                                 $working_transport['fopen'] = new WP_Http_Fopen(); 
    103                                 $blocking_transport[] = &$working_transport['fopen']; 
    104                         } else if ( true === WP_Http_Fsockopen::test($args) ) { 
    105                                 $working_transport['fsockopen'] = new WP_Http_Fsockopen(); 
    106                                 $blocking_transport[] = &$working_transport['fsockopen']; 
    107                         } 
    108  
    109                         foreach ( array('curl', 'streams', 'fopen', 'fsockopen', 'exthttp') as $transport ) { 
    110                                 if ( isset($working_transport[$transport]) ) 
    111                                         $nonblocking_transport[] = &$working_transport[$transport]; 
    112                         } 
    113                 } 
    114  
    115                 do_action( 'http_transport_get_debug', $working_transport, $blocking_transport, $nonblocking_transport ); 
    116  
    117                 if ( isset($args['blocking']) && !$args['blocking'] ) 
    118                         return $nonblocking_transport; 
    119                 else 
    120                         return $blocking_transport; 
     143                return $this->_namedTransport( 'get', $args ); 
    121144        } 
    122145 
    123146        /** 
     
    136159         * @return object|null Null if no transports are available, HTTP transport object. 
    137160         */ 
    138161        function &_postTransport( $args = array() ) { 
    139                 static $working_transport, $blocking_transport, $nonblocking_transport; 
    140  
    141                 if ( is_null($working_transport) ) { 
    142                         if ( true === WP_Http_ExtHttp::test($args) ) { 
    143                                 $working_transport['exthttp'] = new WP_Http_ExtHttp(); 
    144                                 $blocking_transport[] = &$working_transport['exthttp']; 
    145                         } else if ( true === WP_Http_Curl::test($args) ) { 
    146                                 $working_transport['curl'] = new WP_Http_Curl(); 
    147                                 $blocking_transport[] = &$working_transport['curl']; 
    148                         } else if ( true === WP_Http_Streams::test($args) ) { 
    149                                 $working_transport['streams'] = new WP_Http_Streams(); 
    150                                 $blocking_transport[] = &$working_transport['streams']; 
    151                         } else if ( true === WP_Http_Fsockopen::test($args) ) { 
    152                                 $working_transport['fsockopen'] = new WP_Http_Fsockopen(); 
    153                                 $blocking_transport[] = &$working_transport['fsockopen']; 
    154                         } 
    155  
    156                         foreach ( array('curl', 'streams', 'fsockopen', 'exthttp') as $transport ) { 
    157                                 if ( isset($working_transport[$transport]) ) 
    158                                         $nonblocking_transport[] = &$working_transport[$transport]; 
    159                         } 
    160                 } 
    161  
    162                 do_action( 'http_transport_post_debug', $working_transport, $blocking_transport, $nonblocking_transport ); 
    163  
    164                 if ( isset($args['blocking']) && !$args['blocking'] ) 
    165                         return $nonblocking_transport; 
    166                 else 
    167                         return $blocking_transport; 
     162                return $this->_namedTransport( 'post', $args ); 
    168163        } 
    169164 
    170165        /** 
     
    914909                if ( ! function_exists('fopen') || (function_exists('ini_get') && true != ini_get('allow_url_fopen')) ) 
    915910                        return false; 
    916911 
    917                 if ( isset($args['method']) && 'HEAD' == $args['method'] ) //This transport cannot make a HEAD request 
     912                if ( isset( $args['method'] ) && 'GET' !== $args['method'] ) //This transport cannot make anything else then a GET request 
    918913                        return false; 
    919914 
    920915                $use = true;