WordPress.org

Make WordPress Core

Ticket #15292: 16573.2.diff

File 16573.2.diff, 1.9 KB (added by scribu, 7 years ago)

Don't use array_filter()

  • wp-includes/meta.php

     
    387387        $i = 0;
    388388        foreach ( $meta_query as $q ) {
    389389                $meta_key = isset( $q['key'] ) ? trim( $q['key'] ) : '';
    390                 $meta_value = isset( $q['value'] ) ? $q['value'] : '';
    391390                $meta_compare = isset( $q['compare'] ) ? strtoupper( $q['compare'] ) : '=';
    392391                $meta_type = isset( $q['type'] ) ? strtoupper( $q['type'] ) : 'CHAR';
    393392
     
    413412                if ( !empty( $meta_key ) )
    414413                        $where .= $wpdb->prepare( " AND $alias.meta_key = %s", $meta_key );
    415414
     415                if ( !isset( $q['value'] ) )
     416                        continue;
     417                $meta_value = $q['value'];
     418
    416419                if ( in_array( $meta_compare, array( 'IN', 'NOT IN', 'BETWEEN', 'NOT BETWEEN' ) ) ) {
    417420                        if ( ! is_array( $meta_value ) )
    418421                                $meta_value = preg_split( '/[,\s]+/', $meta_value );
     422
     423                        if ( empty( $meta_value ) )
     424                                continue;
    419425                } else {
    420426                        $meta_value = trim( $meta_value );
    421427                }
    422428
    423                 if ( empty( $meta_value ) )
    424                         continue;
    425 
    426429                if ( 'IN' == substr( $meta_compare, -2) ) {
    427430                        $meta_compare_string = '(' . substr( str_repeat( ',%s', count( $meta_value ) ), 1 ) . ')';
    428431                } elseif ( 'BETWEEN' == substr( $meta_compare, -7) ) {
     
    457460        $meta_query = array();
    458461
    459462        // Simple query needs to be first for orderby=meta_value to work correctly
    460         foreach ( array( 'key', 'value', 'compare', 'type' ) as $key ) {
     463        foreach ( array( 'key', 'compare', 'type' ) as $key ) {
    461464                if ( !empty( $qv[ "meta_$key" ] ) )
    462465                        $meta_query[0][ $key ] = $qv[ "meta_$key" ];
    463466        }
    464467
     468        // WP_Query sets 'meta_value' = '' by default
     469        if ( isset( $qv[ 'meta_value' ] ) && '' !== $qv[ 'meta_value' ] )
     470                $meta_query[0]['value'] = $qv[ 'meta_value' ];
     471
    465472        if ( !empty( $qv['meta_query'] ) && is_array( $qv['meta_query'] ) ) {
    466473                $meta_query = array_merge( $meta_query, $qv['meta_query'] );
    467474        }