47719.3.diff on Ticket #47719 – Attachment – WordPress Trac

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

                 $exclude_tree = $args['exclude_tree'];
                 $include      = $args['include'];
-                $inclusions = '';
                 if ( ! empty( $include ) ) {
+                        $exclude      = '';
+                        $exclude_tree = '';
+                        $inclusions   = implode( ',', wp_parse_id_list( $include ) );
+                }
+                if ( ! empty( $inclusions ) ) {
+                        $exclude                                  = '';
+                        $exclude_tree                             = '';
+                        $inclusions                               = implode( ',', wp_parse_id_list( $include ) );
                         $this->sql_clauses['where']['inclusions'] = 't.term_id IN ( ' . $inclusions . ' )';
+                }
 …
                 $terms = $wpdb->get_results( $this->request ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
                 if ( empty( $terms ) ) {
+                        $this->terms = array();
                         if ( $args['cache_results'] ) {
                                 wp_cache_add( $cache_key, array(), 'term-queries' );
+                                wp_cache_add( $cache_key, $this->terms, 'term-queries' );
+                        }
+                        return array();
+                        return $this->terms;
+                }
                 $term_ids = wp_list_pluck( $terms, 'term_id' );

tests/phpunit/tests/post/query.php

+        }
         /**
+         * @ticket 47719
+         */
+        public function test_post__in_should_return_no_posts_when_0() {
+                self::factory()->post->create_many( 4 );
+                $query = new WP_Query(
+                        array(
+                                'post_type' => 'post',
+                                'post__in'  => array( 0 ),
+                        )
+                );
+                $this->assertSame( array(), $query->posts );
+                $this->assertSame( 0, $query->found_posts );
+        }
+        /**
          * @ticket 57296
          * @covers WP_Query::get_posts
          */

tests/phpunit/tests/term/query.php

+        }
         /**
+         * @ticket 47719
+         */
+        public function test_include_should_return_no_terms_when_0() {
+                register_taxonomy( 'wptests_tax', 'post' );
+                self::factory()->term->create_many( 3, array( 'taxonomy' => 'wptests_tax' ) );
+                $query = new WP_Term_Query(
+                        array(
+                                'taxonomy' => 'wptests_tax',
+                                'include'  => array( 0 ),
+                        )
+                );
+                $expected = array();
+                $this->assertSame( $expected, $query->terms );
+                $this->assertSame( $expected, $query->get_terms() );
+        }
+        /**
          * Ensure cache keys are generated without WPDB placeholders.
+         *
          * @ticket 57298

tests/phpunit/tests/user/query.php

                 $this->assertSame( 1, $q->total_users );
+        }
+        /**
+         * @ticket 47719
+         */
+        public function test_include_should_return_no_users_when_0() {
+                $query = new WP_User_Query(
+                        array(
+                                'role'    => '',
+                                'include' => array( 0 ),
+                        )
+                );
+                $this->assertSame( array(), $query->get_results() );
+        }
         public static function filter_users_pre_query( $posts, $query ) {
                 $query->total_users = 1;

Make WordPress Core

Context Navigation

Ticket #47719: 47719.3.diff

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

tests/phpunit/tests/post/query.php

tests/phpunit/tests/term/query.php

tests/phpunit/tests/user/query.php

Download in other formats: