WordPress.org

Make WordPress Core


Ignore:
Timestamp:
07/25/2019 12:47:53 AM (10 months ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Rename $r variable used with wp_parse_args() to $parsed_args for clarity.

Props freewebmentor.
Fixes #45059.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp-http-streams.php

    r45611 r45667  
    3939        );
    4040
    41         $r = wp_parse_args( $args, $defaults );
    42 
    43         if ( isset( $r['headers']['User-Agent'] ) ) {
    44             $r['user-agent'] = $r['headers']['User-Agent'];
    45             unset( $r['headers']['User-Agent'] );
    46         } elseif ( isset( $r['headers']['user-agent'] ) ) {
    47             $r['user-agent'] = $r['headers']['user-agent'];
    48             unset( $r['headers']['user-agent'] );
     41        $parsed_args = wp_parse_args( $args, $defaults );
     42
     43        if ( isset( $parsed_args['headers']['User-Agent'] ) ) {
     44            $parsed_args['user-agent'] = $parsed_args['headers']['User-Agent'];
     45            unset( $parsed_args['headers']['User-Agent'] );
     46        } elseif ( isset( $parsed_args['headers']['user-agent'] ) ) {
     47            $parsed_args['user-agent'] = $parsed_args['headers']['user-agent'];
     48            unset( $parsed_args['headers']['user-agent'] );
    4949        }
    5050
    5151        // Construct Cookie: header if any cookies are set.
    52         WP_Http::buildCookieHeader( $r );
     52        WP_Http::buildCookieHeader( $parsed_args );
    5353
    5454        $arrURL = parse_url( $url );
     
    7171        }
    7272
    73         if ( isset( $r['headers']['Host'] ) || isset( $r['headers']['host'] ) ) {
    74             if ( isset( $r['headers']['Host'] ) ) {
    75                 $arrURL['host'] = $r['headers']['Host'];
     73        if ( isset( $parsed_args['headers']['Host'] ) || isset( $parsed_args['headers']['host'] ) ) {
     74            if ( isset( $parsed_args['headers']['Host'] ) ) {
     75                $arrURL['host'] = $parsed_args['headers']['Host'];
    7676            } else {
    77                 $arrURL['host'] = $r['headers']['host'];
    78             }
    79             unset( $r['headers']['Host'], $r['headers']['host'] );
     77                $arrURL['host'] = $parsed_args['headers']['host'];
     78            }
     79            unset( $parsed_args['headers']['Host'], $parsed_args['headers']['host'] );
    8080        }
    8181
     
    9191        $connect_host = $secure_transport ? 'ssl://' . $connect_host : 'tcp://' . $connect_host;
    9292
    93         $is_local   = isset( $r['local'] ) && $r['local'];
    94         $ssl_verify = isset( $r['sslverify'] ) && $r['sslverify'];
     93        $is_local   = isset( $parsed_args['local'] ) && $parsed_args['local'];
     94        $ssl_verify = isset( $parsed_args['sslverify'] ) && $parsed_args['sslverify'];
    9595        if ( $is_local ) {
    9696            /**
     
    118118                    'capture_peer_cert' => $ssl_verify,
    119119                    'SNI_enabled'       => true,
    120                     'cafile'            => $r['sslcertificates'],
     120                    'cafile'            => $parsed_args['sslcertificates'],
    121121                    'allow_self_signed' => ! $ssl_verify,
    122122                ),
     
    124124        );
    125125
    126         $timeout         = (int) floor( $r['timeout'] );
    127         $utimeout        = $timeout == $r['timeout'] ? 0 : 1000000 * $r['timeout'] % 1000000;
     126        $timeout         = (int) floor( $parsed_args['timeout'] );
     127        $utimeout        = $timeout == $parsed_args['timeout'] ? 0 : 1000000 * $parsed_args['timeout'] % 1000000;
    128128        $connect_timeout = max( $timeout, 1 );
    129129
     
    183183        }
    184184
    185         $strHeaders = strtoupper( $r['method'] ) . ' ' . $requestPath . ' HTTP/' . $r['httpversion'] . "\r\n";
     185        $strHeaders = strtoupper( $parsed_args['method'] ) . ' ' . $requestPath . ' HTTP/' . $parsed_args['httpversion'] . "\r\n";
    186186
    187187        $include_port_in_host_header = (
     
    197197        }
    198198
    199         if ( isset( $r['user-agent'] ) ) {
    200             $strHeaders .= 'User-agent: ' . $r['user-agent'] . "\r\n";
    201         }
    202 
    203         if ( is_array( $r['headers'] ) ) {
    204             foreach ( (array) $r['headers'] as $header => $headerValue ) {
     199        if ( isset( $parsed_args['user-agent'] ) ) {
     200            $strHeaders .= 'User-agent: ' . $parsed_args['user-agent'] . "\r\n";
     201        }
     202
     203        if ( is_array( $parsed_args['headers'] ) ) {
     204            foreach ( (array) $parsed_args['headers'] as $header => $headerValue ) {
    205205                $strHeaders .= $header . ': ' . $headerValue . "\r\n";
    206206            }
    207207        } else {
    208             $strHeaders .= $r['headers'];
     208            $strHeaders .= $parsed_args['headers'];
    209209        }
    210210
     
    215215        $strHeaders .= "\r\n";
    216216
    217         if ( ! is_null( $r['body'] ) ) {
    218             $strHeaders .= $r['body'];
     217        if ( ! is_null( $parsed_args['body'] ) ) {
     218            $strHeaders .= $parsed_args['body'];
    219219        }
    220220
    221221        fwrite( $handle, $strHeaders );
    222222
    223         if ( ! $r['blocking'] ) {
     223        if ( ! $parsed_args['blocking'] ) {
    224224            stream_set_blocking( $handle, 0 );
    225225            fclose( $handle );
     
    239239        $keep_reading = true;
    240240        $block_size   = 4096;
    241         if ( isset( $r['limit_response_size'] ) ) {
    242             $block_size = min( $block_size, $r['limit_response_size'] );
     241        if ( isset( $parsed_args['limit_response_size'] ) ) {
     242            $block_size = min( $block_size, $parsed_args['limit_response_size'] );
    243243        }
    244244
    245245        // If streaming to a file setup the file handle.
    246         if ( $r['stream'] ) {
     246        if ( $parsed_args['stream'] ) {
    247247            if ( ! WP_DEBUG ) {
    248                 $stream_handle = @fopen( $r['filename'], 'w+' );
     248                $stream_handle = @fopen( $parsed_args['filename'], 'w+' );
    249249            } else {
    250                 $stream_handle = fopen( $r['filename'], 'w+' );
     250                $stream_handle = fopen( $parsed_args['filename'], 'w+' );
    251251            }
    252252            if ( ! $stream_handle ) {
     
    257257                        __( 'Could not open handle for %1$s to %2$s.' ),
    258258                        'fopen()',
    259                         $r['filename']
     259                        $parsed_args['filename']
    260260                    )
    261261                );
     
    278278                $this_block_size = strlen( $block );
    279279
    280                 if ( isset( $r['limit_response_size'] ) && ( $bytes_written + $this_block_size ) > $r['limit_response_size'] ) {
    281                     $this_block_size = ( $r['limit_response_size'] - $bytes_written );
     280                if ( isset( $parsed_args['limit_response_size'] ) && ( $bytes_written + $this_block_size ) > $parsed_args['limit_response_size'] ) {
     281                    $this_block_size = ( $parsed_args['limit_response_size'] - $bytes_written );
    282282                    $block           = substr( $block, 0, $this_block_size );
    283283                }
     
    293293                $bytes_written += $bytes_written_to_file;
    294294
    295                 $keep_reading = ! isset( $r['limit_response_size'] ) || $bytes_written < $r['limit_response_size'];
     295                $keep_reading = ! isset( $parsed_args['limit_response_size'] ) || $bytes_written < $parsed_args['limit_response_size'];
    296296            }
    297297
     
    307307                    $bodyStarted   = true;
    308308                }
    309                 $keep_reading = ( ! $bodyStarted || ! isset( $r['limit_response_size'] ) || strlen( $strResponse ) < ( $header_length + $r['limit_response_size'] ) );
     309                $keep_reading = ( ! $bodyStarted || ! isset( $parsed_args['limit_response_size'] ) || strlen( $strResponse ) < ( $header_length + $parsed_args['limit_response_size'] ) );
    310310            }
    311311
     
    325325            'response' => $arrHeaders['response'],
    326326            'cookies'  => $arrHeaders['cookies'],
    327             'filename' => $r['filename'],
     327            'filename' => $parsed_args['filename'],
    328328        );
    329329
    330330        // Handle redirects.
    331         $redirect_response = WP_Http::handle_redirects( $url, $r, $response );
     331        $redirect_response = WP_Http::handle_redirects( $url, $parsed_args, $response );
    332332        if ( false !== $redirect_response ) {
    333333            return $redirect_response;
     
    339339        }
    340340
    341         if ( true === $r['decompress'] && true === WP_Http_Encoding::should_decode( $arrHeaders['headers'] ) ) {
     341        if ( true === $parsed_args['decompress'] && true === WP_Http_Encoding::should_decode( $arrHeaders['headers'] ) ) {
    342342            $process['body'] = WP_Http_Encoding::decompress( $process['body'] );
    343343        }
    344344
    345         if ( isset( $r['limit_response_size'] ) && strlen( $process['body'] ) > $r['limit_response_size'] ) {
    346             $process['body'] = substr( $process['body'], 0, $r['limit_response_size'] );
     345        if ( isset( $parsed_args['limit_response_size'] ) && strlen( $process['body'] ) > $parsed_args['limit_response_size'] ) {
     346            $process['body'] = substr( $process['body'], 0, $parsed_args['limit_response_size'] );
    347347        }
    348348
Note: See TracChangeset for help on using the changeset viewer.