Changeset 54496

Timestamp:

10/11/2022 06:13:34 PM (22 months ago)

Author:

SergeyBiryukov

Message:

Query: Avoid PHP notices when get_queried_object() returns null.

WP_Query methods assume that get_queried_object() would return a non-null value, which is not always the case.

This commit resolves various warnings in WP_Query along the lines of:

Attempt to read property "post_type" on null in wp-includes/class-wp-query.php on line 4338

Follow-up to [1728], [3639], [8807], [49119].

Props dd32, yellyc, boonebgorges, darkskipper, Howdy_McGee, swissspidy, nacin, mikeschroder, mikejolley, sterlo, datainterlock, utsavmadaan823, kanlukasz, woji29911, hellofromTonya, zikubd, deksar, bwbama, noplanman, nouarah, SergeyBiryukov.
Fixes #29660.

Location:

trunk

Files:

• src/wp-includes/class-wp-query.php (modified) (7 diffs)
• tests/phpunit/tests/query.php (modified) (1 diff)

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

@@ -3964 +3964 @@
 
         $post_obj = $this->get_queried_object();
+
+
+
 
         if ( in_array( (string) $post_obj->ID, $attachment, true ) ) {
@@ -3997 +4000 @@
 
         $author_obj = $this->get_queried_object();
+
+
+
 
         $author = array_map( 'strval', (array) $author );
@@ -4033 +4039 @@
 
         $cat_obj = $this->get_queried_object();
+
+
+
 
         $category = array_map( 'strval', (array) $category );
@@ -4069 +4078 @@
 
         $tag_obj = $this->get_queried_object();
+
+
+
 
         $tag = array_map( 'strval', (array) $tag );
@@ -4316 +4328 @@
 
         $page_obj = $this->get_queried_object();
+
+
+
 
         $page = array_map( 'strval', (array) $page );
@@ -4423 +4438 @@
 
         $post_obj = $this->get_queried_object();
+
+
+
 
         $post = array_map( 'strval', (array) $post );
@@ -4470 +4488 @@
 
         $post_obj = $this->get_queried_object();
+
+
+
 
         return in_array( $post_obj->post_type, (array) $post_types, true );

trunk/tests/phpunit/tests/query.php

@@ -789 +789 @@
         $this->assertFalse( is_post_type_archive( $post_type ) );
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }