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

Timestamp:

10/11/2022 06:13:34 PM (4 years 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.

File:

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

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

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