Context Navigation

← Previous Change
Next Change →

wp-db.php

Timestamp:

11/30/2021 09:00:32 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Options, Meta APIs: Improve error handling in sanitize_option().

To prevent potential false negatives, set $error to null initially, so we can better tell if it was ever changed during the sanitization and be able to better react if an empty string is added to it.

Additionally, and mainly for the sake of the Settings API at this point, add error messages to some WP_Error objects returned from wpdb methods that were previously causing the issues here.

Follow-up to [32791].

Props iCaleb, audrasjb, hellofromTonya, SergeyBiryukov.
Fixes #53986.

File:

: 1 edited

trunk/src/wp-includes/wp-db.php (modified) (2 diffs)

Legend:

: Unmodified
: Added
: Removed

trunk/src/wp-includes/wp-db.php

-              r52218
+              r52294
                 $results     = $this->get_results( "SHOW FULL COLUMNS FROM $table" );
                 if ( ! $results ) {
                         return new WP_Error( 'wpdb_get_table_charset_failure' );
+                        return new WP_Error( 'wpdb_get_table_charset_failure', __( 'Could not retrieve table charset.' ) );
+                }
 …
                         $row                       = $this->get_row( 'SELECT ' . implode( ', ', $sql ), ARRAY_A );
                         if ( ! $row ) {
                                 return new WP_Error( 'wpdb_strip_invalid_text_failure' );
+                                return new WP_Error( 'wpdb_strip_invalid_text_failure', __( 'Could not strip invalid text.' ) );
+                        }

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 52294 for trunk/src/wp-includes/wp-db.php

Legend:

trunk/src/wp-includes/wp-db.php

Download in other formats: