Changeset 53503 for trunk/src/wp-includes/class-wp-term-query.php

Timestamp:

06/14/2022 04:15:52 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Docs: Add a comment for stripping the leading AND from SQL clauses in some query classes.

Follow-up to [30084], [34529], [34542], [36598], [37477], [37572], [43010].

See #55646.

File:

• trunk/src/wp-includes/class-wp-term-query.php (modified) (8 diffs)

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

@@ -485 +485 @@
                         $exclude_tree      = wp_parse_id_list( $exclude_tree );
                         $excluded_children = $exclude_tree;
+
                         foreach ( $exclude_tree as $extrunk ) {
                                 $excluded_children = array_merge(
@@ -498 +499 @@
                                 );
                         }
+
                         $exclusions = array_merge( $excluded_children, $exclusions );
                 }
@@ -532 +534 @@
 
                 if ( ! empty( $exclusions ) ) {
-                        // Must do string manipulation here for backward compatibility with filter.
+                        // Strip leading 'AND'. Must do string manipulation here for backward compatibility with filter.
                         $this->sql_clauses['where']['exclusions'] = preg_replace( '/^\s*AND\s*/', '', $exclusions );
                 }
@@ -544 +546 @@
                 if ( ! empty( $args['name'] ) ) {
                         $names = $args['name'];
+
                         foreach ( $names as &$_name ) {
                                 // `sanitize_term_field()` returns slashed data.
@@ -651 +654 @@
 
                 if ( ! empty( $meta_clauses ) ) {
-                        $join                                    .= $mq_sql['join'];
+                        $join .= $mq_sql['join'];
+
+                        // Strip leading 'AND'.
                         $this->sql_clauses['where']['meta_query'] = preg_replace( '/^\s*AND\s*/', '', $mq_sql['where'] );
-                        $distinct                                .= 'DISTINCT';
+
+                        $distinct .= 'DISTINCT';
 
                 }
@@ -786 +792 @@
                                 $cache = array_map( 'intval', $cache );
                         } elseif ( 'count' !== $_fields ) {
-                                if ( ( 'all_with_object_id' === $_fields && ! empty( $args['object_ids'] ) ) || ( 'all' === $_fields && $args['pad_counts'] ) ) {
+                                if ( ( 'all_with_object_id' === $_fields && ! empty( $args['object_ids'] ) )
+                                        || ( 'all' === $_fields && $args['pad_counts'] )
+                                ) {
                                         $term_ids = wp_list_pluck( $cache, 'term_id' );
                                 } else {
                                         $term_ids = array_map( 'intval', $cache );
                                 }
+
                                 _prime_term_caches( $term_ids, $args['update_term_meta_cache'] );
+
                                 $term_objects = $this->populate_terms( $cache );
                                 $cache        = $this->format_terms( $term_objects, $_fields );
@@ -838 +848 @@
                                 if ( ! $term->count ) {
                                         $children = get_term_children( $term->term_id, $term->taxonomy );
+
                                         if ( is_array( $children ) ) {
                                                 foreach ( $children as $child_id ) {
@@ -887 +898 @@
                         $term_cache = wp_list_pluck( $term_objects, 'term_id' );
                 }
+
                 wp_cache_add( $cache_key, $term_cache, 'terms' );
                 $this->terms = $this->format_terms( $term_objects, $_fields );