Make WordPress Core

Changeset 52475 for branches/4.8


Ignore:
Timestamp:
01/06/2022 06:14:56 PM (2 years ago)
Author:
desrosj
Message:

Grouped backports to the 4.8 branch.

  • Query: Improve sanitization within WP_Tax_Query.
  • Query: Improve sanitization within WP_Meta_Query.
  • Upgrade/Install: Avoid using unserialize() unnecessarily.
  • Formatting: Correctly encode ASCII characters in post slugs.

Merges [52454-52457] to the 4.8 branch.
Props vortfu, dd32, ehtis, zieladam, whyisjake, xknown, peterwilsoncc, desrosj, iandunn.

Location:
branches/4.8
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • branches/4.8

  • branches/4.8/src/wp-admin/includes/upgrade.php

    r40864 r52475  
    12491249        while( $rows = $wpdb->get_results( "SELECT option_name, option_value FROM $wpdb->options ORDER BY option_id LIMIT $start, 20" ) ) {
    12501250            foreach ( $rows as $row ) {
    1251                 $value = $row->option_value;
    1252                 if ( !@unserialize( $value ) )
     1251                $value = maybe_unserialize( $row->option_value );
     1252                if ( $value === $row->option_value )
    12531253                    $value = stripslashes( $value );
    12541254                if ( $value !== $row->option_value ) {
  • branches/4.8/src/wp-includes/class-wp-meta-query.php

    r38768 r52475  
    718718            $sibling_compare = strtoupper( $sibling['compare'] );
    719719            if ( in_array( $clause_compare, $compatible_compares ) && in_array( $sibling_compare, $compatible_compares ) ) {
    720                 $alias = $sibling['alias'];
     720                $alias = preg_replace( '/\W/', '_', $sibling['alias'] );
    721721                break;
    722722            }
  • branches/4.8/src/wp-includes/class-wp-tax-query.php

    r39662 r52475  
    544544            // The sibling must both have compatible operator to share its alias.
    545545            if ( in_array( strtoupper( $sibling['operator'] ), $compatible_operators ) ) {
    546                 $alias = $sibling['alias'];
     546                $alias = preg_replace( '/\W/', '_', $sibling['alias'] );
    547547                break;
    548548            }
     
    574574        }
    575575
    576         $query['terms'] = array_unique( (array) $query['terms'] );
     576        if ( 'slug' === $query['field'] || 'name' === $query['field'] ) {
     577            $query['terms'] = array_unique( (array) $query['terms'] );
     578        } else {
     579            $query['terms'] = wp_parse_id_list( $query['terms'] );
     580        }
    577581
    578582        if ( is_taxonomy_hierarchical( $query['taxonomy'] ) && $query['include_children'] ) {
  • branches/4.8/src/wp-includes/formatting.php

    r49398 r52475  
    10711071 *
    10721072 * @since 1.5.0
    1073  *
    1074  * @param string $utf8_string
    1075  * @param int    $length Max  length of the string
     1073 * @since 5.8.3 Added the `encode_ascii_characters` parameter.
     1074 *
     1075 * @param string $utf8_string             String to encode.
     1076 * @param int    $length                  Max length of the string
     1077 * @param bool   $encode_ascii_characters Whether to encode ascii characters such as < " '
    10761078 * @return string String with Unicode encoded for URI.
    10771079 */
    1078 function utf8_uri_encode( $utf8_string, $length = 0 ) {
     1080function utf8_uri_encode( $utf8_string, $length = 0, $encode_ascii_characters = false ) {
    10791081    $unicode        = '';
    10801082    $values         = array();
     
    10911093
    10921094        if ( $value < 128 ) {
    1093             if ( $length && ( $unicode_length >= $length ) ) {
     1095            $char                = chr( $value );
     1096            $encoded_char        = $encode_ascii_characters ? rawurlencode( $char ) : $char;
     1097            $encoded_char_length = strlen( $encoded_char );
     1098            if ( $length && ( $unicode_length + $encoded_char_length ) > $length ) {
    10941099                break;
    10951100            }
    1096             $unicode .= chr( $value );
    1097             $unicode_length++;
     1101            $unicode        .= $encoded_char;
     1102            $unicode_length += $encoded_char_length;
    10981103        } else {
    10991104            if ( count( $values ) == 0 ) {
  • branches/4.8/src/wp-includes/post.php

    r47649 r52475  
    38233823            $slug = substr( $slug, 0, $length );
    38243824        else
    3825             $slug = utf8_uri_encode( $decoded_slug, $length );
     3825            $slug = utf8_uri_encode( $decoded_slug, $length, true );
    38263826    }
    38273827
Note: See TracChangeset for help on using the changeset viewer.