WordPress.org
Get WordPress

Make WordPress Core

Changeset 52977


Ignore:
Timestamp:
03/22/2022 02:54:45 PM (3 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Use multi-line strings for the $this->request property in wp-includes/class-wp-*-query.php.

This further improves the readability by replacing implode() calls with string interpolation.

Follow-up to [52973].

Props jrf.
See #54728.

Location:
trunk/src/wp-includes
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/class-wp-comment-query.php

    r52974 r52977  
    962962        $this->sql_clauses['limits']  = $limits;
    963963
    964         $this->request = implode(
    965             ' ',
    966             array(
    967                 $this->sql_clauses['select'],
    968                 $this->sql_clauses['from'],
    969                 $where,
    970                 $this->sql_clauses['groupby'],
    971                 $this->sql_clauses['orderby'],
    972                 $this->sql_clauses['limits'],
    973             )
    974         );
     964        $this->request = "
     965            {$this->sql_clauses['select']}
     966            {$this->sql_clauses['from']}
     967            {$where}
     968            {$this->sql_clauses['groupby']}
     969            {$this->sql_clauses['orderby']}
     970            {$this->sql_clauses['limits']}
     971        ";
    975972
    976973        if ( $this->query_vars['count'] ) {

  • trunk/src/wp-includes/class-wp-network-query.php

    r52974 r52977  
    481481        $this->sql_clauses['limits']  = $limits;
    482482
    483         $this->request = implode(
    484             ' ',
    485             array(
    486                 $this->sql_clauses['select'],
    487                 $this->sql_clauses['from'],
    488                 $where,
    489                 $this->sql_clauses['groupby'],
    490                 $this->sql_clauses['orderby'],
    491                 $this->sql_clauses['limits'],
    492             )
    493         );
     483        $this->request = "
     484            {$this->sql_clauses['select']}
     485            {$this->sql_clauses['from']}
     486            {$where}
     487            {$this->sql_clauses['groupby']}
     488            {$this->sql_clauses['orderby']}
     489            {$this->sql_clauses['limits']}
     490        ";
    494491
    495492        if ( $this->query_vars['count'] ) {

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

    r52974 r52977  
    30013001        }
    30023002
    3003         $old_request = implode(
    3004             ' ',
    3005             array(
    3006                 "SELECT $found_rows $distinct $fields",
    3007                 "FROM {$wpdb->posts} $join",
    3008                 "WHERE 1=1 $where",
    3009                 $groupby,
    3010                 $orderby,
    3011                 $limits,
    3012             )
    3013         );
     3003        $old_request = "
     3004            SELECT $found_rows $distinct $fields
     3005            FROM {$wpdb->posts} $join
     3006            WHERE 1=1 $where
     3007            $groupby
     3008            $orderby
     3009            $limits
     3010        ";
    30143011
    30153012        $this->request = $old_request;
    … …  
    30983095                // First get the IDs and then fill in the objects.
    30993096
    3100                 $this->request = implode(
    3101                     ' ',
    3102                     array(
    3103                         "SELECT $found_rows $distinct {$wpdb->posts}.ID",
    3104                         "FROM {$wpdb->posts} $join",
    3105                         "WHERE 1=1 $where",
    3106                         $groupby,
    3107                         $orderby,
    3108                         $limits,
    3109                     )
    3110                 );
     3097                $this->request = "
     3098                    SELECT $found_rows $distinct {$wpdb->posts}.ID
     3099                    FROM {$wpdb->posts} $join
     3100                    WHERE 1=1 $where
     3101                    $groupby
     3102                    $orderby
     3103                    $limits
     3104                ";
    31113105
    31123106                /**

  • trunk/src/wp-includes/class-wp-site-query.php

    r52974 r52977  
    684684        $this->sql_clauses['limits']  = $limits;
    685685
    686         $this->request = implode(
    687             ' ',
    688             array(
    689                 $this->sql_clauses['select'],
    690                 $this->sql_clauses['from'],
    691                 $where,
    692                 $this->sql_clauses['groupby'],
    693                 $this->sql_clauses['orderby'],
    694                 $this->sql_clauses['limits'],
    695             )
    696         );
     686        $this->request = "
     687            {$this->sql_clauses['select']}
     688            {$this->sql_clauses['from']}
     689            {$where}
     690            {$this->sql_clauses['groupby']}
     691            {$this->sql_clauses['orderby']}
     692            {$this->sql_clauses['limits']}
     693        ";
    697694
    698695        if ( $this->query_vars['count'] ) {

  • trunk/src/wp-includes/class-wp-term-query.php

    r52974 r52977  
    723723        $this->sql_clauses['limits']  = $limits;
    724724
    725         $this->request = implode(
    726             ' ',
    727             array(
    728                 $this->sql_clauses['select'],
    729                 $this->sql_clauses['from'],
    730                 $where,
    731                 $this->sql_clauses['orderby'],
    732                 $this->sql_clauses['limits'],
    733             )
    734         );
     725        $this->request = "
     726            {$this->sql_clauses['select']}
     727            {$this->sql_clauses['from']}
     728            {$where}
     729            {$this->sql_clauses['orderby']}
     730            {$this->sql_clauses['limits']}
     731        ";
    735732
    736733        $this->terms = null;

  • trunk/src/wp-includes/class-wp-user-query.php

    r52973 r52977  
    771771
    772772        if ( null === $this->results ) {
    773             $this->request = implode(
    774                 ' ',
    775                 array(
    776                     "SELECT {$this->query_fields}",
    777                     $this->query_from,
    778                     $this->query_where,
    779                     $this->query_orderby,
    780                     $this->query_limit,
    781                 )
    782             );
     773            $this->request = "
     774                SELECT {$this->query_fields}
     775                {$this->query_from}
     776                {$this->query_where}
     777                {$this->query_orderby}
     778                {$this->query_limit}
     779            ";
    783780
    784781            if ( is_array( $qv['fields'] ) || 'all' === $qv['fields'] ) {
Note: See TracChangeset for help on using the changeset viewer.