WordPress.org

Make WordPress Core

Ticket #14786: 14786.10.patch

File 14786.10.patch, 5.6 KB (added by hakre, 11 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;