Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/date.php

    r41162 r42343  
    264264     */
    265265    public function get_compare( $query ) {
    266         if ( ! empty( $query['compare'] ) && in_array( $query['compare'], array( '=', '!=', '>', '>=', '<', '<=', 'IN', 'NOT IN', 'BETWEEN', 'NOT BETWEEN' ) ) )
     266        if ( ! empty( $query['compare'] ) && in_array( $query['compare'], array( '=', '!=', '>', '>=', '<', '<=', 'IN', 'NOT IN', 'BETWEEN', 'NOT BETWEEN' ) ) ) {
    267267            return strtoupper( $query['compare'] );
     268        }
    268269
    269270        return $this->compare;
     
    294295         * values generate errors too.
    295296         */
    296         if ( array_key_exists( 'before', $date_query ) && is_array( $date_query['before'] ) ){
     297        if ( array_key_exists( 'before', $date_query ) && is_array( $date_query['before'] ) ) {
    297298            $valid = $this->validate_date_values( $date_query['before'] );
    298299        }
    299300
    300         if ( array_key_exists( 'after', $date_query ) && is_array( $date_query['after'] ) ){
     301        if ( array_key_exists( 'after', $date_query ) && is_array( $date_query['after'] ) ) {
    301302            $valid = $this->validate_date_values( $date_query['after'] );
    302303        }
     
    325326        $min_max_checks['dayofyear'] = array(
    326327            'min' => 1,
    327             'max' => $max_days_of_year
     328            'max' => $max_days_of_year,
    328329        );
    329330
     
    331332        $min_max_checks['dayofweek'] = array(
    332333            'min' => 1,
    333             'max' => 7
     334            'max' => 7,
    334335        );
    335336
     
    337338        $min_max_checks['dayofweek_iso'] = array(
    338339            'min' => 1,
    339             'max' => 7
     340            'max' => 7,
    340341        );
    341342
     
    343344        $min_max_checks['month'] = array(
    344345            'min' => 1,
    345             'max' => 12
     346            'max' => 12,
    346347        );
    347348
     
    361362        $min_max_checks['week'] = array(
    362363            'min' => 1,
    363             'max' => $week_count
     364            'max' => $week_count,
    364365        );
    365366
     
    367368        $min_max_checks['day'] = array(
    368369            'min' => 1,
    369             'max' => 31
     370            'max' => 31,
    370371        );
    371372
     
    373374        $min_max_checks['hour'] = array(
    374375            'min' => 0,
    375             'max' => 23
     376            'max' => 23,
    376377        );
    377378
     
    379380        $min_max_checks['minute'] = array(
    380381            'min' => 0,
    381             'max' => 59
     382            'max' => 59,
    382383        );
    383384
     
    385386        $min_max_checks['second'] = array(
    386387            'min' => 0,
    387             'max' => 59
     388            'max' => 59,
    388389        );
    389390
     
    439440                $valid = false;
    440441            }
    441 
    442442        } elseif ( $day_exists && $month_exists ) {
    443443            /*
     
    481481
    482482        $valid_columns = array(
    483             'post_date', 'post_date_gmt', 'post_modified',
    484             'post_modified_gmt', 'comment_date', 'comment_date_gmt',
    485             'user_registered', 'registered', 'last_updated',
     483            'post_date',
     484            'post_date_gmt',
     485            'post_modified',
     486            'post_modified_gmt',
     487            'comment_date',
     488            'comment_date_gmt',
     489            'user_registered',
     490            'registered',
     491            'last_updated',
    486492        );
    487493
     
    497503             *                             are 'post_date', 'post_date_gmt', 'post_modified',
    498504             *                             'post_modified_gmt', 'comment_date', 'comment_date_gmt',
    499              *                             'user_registered'
     505             *                             'user_registered'
    500506             */
    501507            if ( ! in_array( $column, apply_filters( 'date_query_valid_columns', $valid_columns ) ) ) {
     
    504510
    505511            $known_columns = array(
    506                 $wpdb->posts => array(
     512                $wpdb->posts    => array(
    507513                    'post_date',
    508514                    'post_date_gmt',
     
    514520                    'comment_date_gmt',
    515521                ),
    516                 $wpdb->users => array(
     522                $wpdb->users    => array(
    517523                    'user_registered',
    518524                ),
    519                 $wpdb->blogs => array(
     525                $wpdb->blogs    => array(
    520526                    'registered',
    521527                    'last_updated',
     
    530536                }
    531537            }
    532 
    533538        }
    534539
     
    616621        $indent = '';
    617622        for ( $i = 0; $i < $depth; $i++ ) {
    618             $indent .= "  ";
     623            $indent .= '  ';
    619624        }
    620625
     
    638643
    639644                    $sql_chunks['join'] = array_merge( $sql_chunks['join'], $clause_sql['join'] );
    640                 // This is a subquery, so we recurse.
     645                    // This is a subquery, so we recurse.
    641646                } else {
    642647                    $clause_sql = $this->get_sql_for_query( $clause, $depth + 1 );
     
    735740        // Specific value queries.
    736741
    737         if ( isset( $query['year'] ) && $value = $this->build_value( $compare, $query['year'] ) )
     742        if ( isset( $query['year'] ) && $value = $this->build_value( $compare, $query['year'] ) ) {
    738743            $where_parts[] = "YEAR( $column ) $compare $value";
     744        }
    739745
    740746        if ( isset( $query['month'] ) && $value = $this->build_value( $compare, $query['month'] ) ) {
     
    748754            $where_parts[] = _wp_mysql_week( $column ) . " $compare $value";
    749755        }
    750         if ( isset( $query['dayofyear'] ) && $value = $this->build_value( $compare, $query['dayofyear'] ) )
     756        if ( isset( $query['dayofyear'] ) && $value = $this->build_value( $compare, $query['dayofyear'] ) ) {
    751757            $where_parts[] = "DAYOFYEAR( $column ) $compare $value";
    752 
    753         if ( isset( $query['day'] ) && $value = $this->build_value( $compare, $query['day'] ) )
     758        }
     759
     760        if ( isset( $query['day'] ) && $value = $this->build_value( $compare, $query['day'] ) ) {
    754761            $where_parts[] = "DAYOFMONTH( $column ) $compare $value";
    755 
    756         if ( isset( $query['dayofweek'] ) && $value = $this->build_value( $compare, $query['dayofweek'] ) )
     762        }
     763
     764        if ( isset( $query['dayofweek'] ) && $value = $this->build_value( $compare, $query['dayofweek'] ) ) {
    757765            $where_parts[] = "DAYOFWEEK( $column ) $compare $value";
    758 
    759         if ( isset( $query['dayofweek_iso'] ) && $value = $this->build_value( $compare, $query['dayofweek_iso'] ) )
     766        }
     767
     768        if ( isset( $query['dayofweek_iso'] ) && $value = $this->build_value( $compare, $query['dayofweek_iso'] ) ) {
    760769            $where_parts[] = "WEEKDAY( $column ) + 1 $compare $value";
     770        }
    761771
    762772        if ( isset( $query['hour'] ) || isset( $query['minute'] ) || isset( $query['second'] ) ) {
     
    793803     */
    794804    public function build_value( $compare, $value ) {
    795         if ( ! isset( $value ) )
     805        if ( ! isset( $value ) ) {
    796806            return false;
     807        }
    797808
    798809        switch ( $compare ) {
     
    904915        $datetime = array_map( 'absint', $datetime );
    905916
    906         if ( ! isset( $datetime['year'] ) )
     917        if ( ! isset( $datetime['year'] ) ) {
    907918            $datetime['year'] = gmdate( 'Y', $now );
    908 
    909         if ( ! isset( $datetime['month'] ) )
     919        }
     920
     921        if ( ! isset( $datetime['month'] ) ) {
    910922            $datetime['month'] = ( $default_to_max ) ? 12 : 1;
    911 
    912         if ( ! isset( $datetime['day'] ) )
     923        }
     924
     925        if ( ! isset( $datetime['day'] ) ) {
    913926            $datetime['day'] = ( $default_to_max ) ? (int) date( 't', mktime( 0, 0, 0, $datetime['month'], 1, $datetime['year'] ) ) : 1;
    914 
    915         if ( ! isset( $datetime['hour'] ) )
     927        }
     928
     929        if ( ! isset( $datetime['hour'] ) ) {
    916930            $datetime['hour'] = ( $default_to_max ) ? 23 : 0;
    917 
    918         if ( ! isset( $datetime['minute'] ) )
     931        }
     932
     933        if ( ! isset( $datetime['minute'] ) ) {
    919934            $datetime['minute'] = ( $default_to_max ) ? 59 : 0;
    920 
    921         if ( ! isset( $datetime['second'] ) )
     935        }
     936
     937        if ( ! isset( $datetime['second'] ) ) {
    922938            $datetime['second'] = ( $default_to_max ) ? 59 : 0;
     939        }
    923940
    924941        return sprintf( '%04d-%02d-%02d %02d:%02d:%02d', $datetime['year'], $datetime['month'], $datetime['day'], $datetime['hour'], $datetime['minute'], $datetime['second'] );
     
    945962
    946963        // Have to have at least one
    947         if ( ! isset( $hour ) && ! isset( $minute ) && ! isset( $second ) )
     964        if ( ! isset( $hour ) && ! isset( $minute ) && ! isset( $second ) ) {
    948965            return false;
     966        }
    949967
    950968        // Complex combined queries aren't supported for multi-value queries
     
    952970            $return = array();
    953971
    954             if ( isset( $hour ) && false !== ( $value = $this->build_value( $compare, $hour ) ) )
     972            if ( isset( $hour ) && false !== ( $value = $this->build_value( $compare, $hour ) ) ) {
    955973                $return[] = "HOUR( $column ) $compare $value";
    956 
    957             if ( isset( $minute ) && false !== ( $value = $this->build_value( $compare, $minute ) ) )
     974            }
     975
     976            if ( isset( $minute ) && false !== ( $value = $this->build_value( $compare, $minute ) ) ) {
    958977                $return[] = "MINUTE( $column ) $compare $value";
    959 
    960             if ( isset( $second ) && false !== ( $value = $this->build_value( $compare, $second ) ) )
     978            }
     979
     980            if ( isset( $second ) && false !== ( $value = $this->build_value( $compare, $second ) ) ) {
    961981                $return[] = "SECOND( $column ) $compare $value";
     982            }
    962983
    963984            return implode( ' AND ', $return );
     
    974995
    975996        // Single units were already handled. Since hour & second isn't allowed, minute must to be set.
    976         if ( ! isset( $minute ) )
     997        if ( ! isset( $minute ) ) {
    977998            return false;
     999        }
    9781000
    9791001        $format = $time = '';
Note: See TracChangeset for help on using the changeset viewer.