Changeset 53891

Timestamp:

08/13/2022 10:29:19 PM (2 years ago)

Author:

johnbillion

Message:

Query: Be better at forcing data types for query vars.

Several query vars only accept a scalar value and pass the value through functions that assume a scalar value. Adding extra guard conditions to the types of query vars doesn't affect their functionality but does remove PHP notices and warnings that can otherwise be generated when a non-scalar value such as an array is present in a query var.

Props juliobox, xknown, SergeyBiryukov, dave1010, nacin, tellyworth, dd32, audrasjb, johnregan3

Fixes #17737

Location:

trunk

Files:

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

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

@@ -793 +793 @@
         }
 
-        $qv['page_id']  = absint( $qv['page_id'] );
-        $qv['year']     = absint( $qv['year'] );
-        $qv['monthnum'] = absint( $qv['monthnum'] );
-        $qv['day']      = absint( $qv['day'] );
-        $qv['w']        = absint( $qv['w'] );
+        $qv['page_id']  = ;
+        $qv['year']     = ;
+        $qv['monthnum'] = ;
+        $qv['day']      = ;
+        $qv['w']        = ;
         $qv['m']        = is_scalar( $qv['m'] ) ? preg_replace( '|[^0-9]|', '', $qv['m'] ) : '';
-        $qv['paged']    = absint( $qv['paged'] );
-        $qv['cat']      = preg_replace( '|[^0-9,-]|', '', $qv['cat'] );    // Comma-separated list of positive or negative integers.
-        $qv['author']   = preg_replace( '|[^0-9,-]|', '', $qv['author'] ); // Comma-separated list of positive or negative integers.
-        $qv['pagename'] = trim( $qv['pagename'] );
-        $qv['name']     = trim( $qv['name'] );
-        $qv['title']    = trim( $qv['title'] );
-        if ( '' !== $qv['hour'] ) {
+        $qv['paged']    = is_scalar( $qv['paged'] ) ? absint( $qv['paged'] ) : 0;
+        $qv['cat']      = preg_replace( '|[^0-9,-]|', '', $qv['cat'] ); // Array or comma-separated list of positive or negative integers.
+        $qv['author']   = is_scalar( $qv['author'] ) ? preg_replace( '|[^0-9,-]|', '', $qv['author'] ) : ''; // Comma-separated list of positive or negative integers.
+        $qv['pagename'] = is_scalar( $qv['pagename'] ) ? trim( $qv['pagename'] ) : '';
+        $qv['name']     = is_scalar( $qv['name'] ) ? trim( $qv['name'] ) : '';
+        $qv['title']    = is_scalar( $qv['title'] ) ? trim( $qv['title'] ) : '';
+
+        if ( is_scalar( $qv['hour'] ) && '' !== $qv['hour'] ) {
             $qv['hour'] = absint( $qv['hour'] );
-        }
-        if ( '' !== $qv['minute'] ) {
+        } else {
+            $qv['hour'] = '';
+        }
+
+        if ( is_scalar( $qv['minute'] ) && '' !== $qv['minute'] ) {
             $qv['minute'] = absint( $qv['minute'] );
-        }
-        if ( '' !== $qv['second'] ) {
+        } else {
+            $qv['minute'] = '';
+        }
+
+        if ( is_scalar( $qv['second'] ) && '' !== $qv['second'] ) {
             $qv['second'] = absint( $qv['second'] );
-        }
-        if ( '' !== $qv['menu_order'] ) {
+        } else {
+            $qv['second'] = '';
+        }
+
+        if ( is_scalar( $qv['menu_order'] ) && '' !== $qv['menu_order'] ) {
             $qv['menu_order'] = absint( $qv['menu_order'] );
+
+
         }
 
@@ -824 +836 @@
 
         // Compat. Map subpost to attachment.
-        if ( '' != $qv['subpost'] ) {
+        if ( '' != $qv['subpost'] ) {
             $qv['attachment'] = $qv['subpost'];
         }
-        if ( '' != $qv['subpost_id'] ) {
+        if ( '' != $qv['subpost_id'] ) {
             $qv['attachment_id'] = $qv['subpost_id'];
         }
 
-        $qv['attachment_id'] = absint( $qv['attachment_id'] );
+        $qv['attachment_id'] = ;
 
         if ( ( '' !== $qv['attachment'] ) || ! empty( $qv['attachment_id'] ) ) {

trunk/tests/phpunit/tests/query/parseQuery.php

@@ -105 +105 @@
     }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }