WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
07/02/2019 11:41:16 PM (7 years ago)
Author:
pento
Message:

Coding Standards: Fix the Squiz.PHP.DisallowMultipleAssignments violations in wp-includes.

See #47632.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/class-oembed.php

    r45386 r45590  
    441441                // Fetch URL content
    442442                $request = wp_safe_remote_get( $url, $args );
    443                 if ( $html = wp_remote_retrieve_body( $request ) ) {
     443                $html    = wp_remote_retrieve_body( $request );
     444                if ( $html ) {
    444445
    445446                        /**
    … …  
    462463
    463464                        // Strip <body>
    464                         if ( $html_head_end = stripos( $html, '</head>' ) ) {
     465                        $html_head_end = stripos( $html, '</head>' );
     466                        if ( $html_head_end ) {
    465467                                $html = substr( $html, 0, $html_head_end );
    466468                        }
    … …  
    560562                        return new WP_Error( 'not-implemented' );
    561563                }
    562                 if ( ! $body = wp_remote_retrieve_body( $response ) ) {
     564                $body = wp_remote_retrieve_body( $response );
     565                if ( ! $body ) {
    563566                        return false;
    564567                }
Note: See TracChangeset for help on using the changeset viewer.