WordPress.org
Get WordPress

Make WordPress Core

Changeset 33326


Ignore:
Timestamp:
07/19/2015 11:25:19 PM (11 years ago)
Author:
pento
Message:

sanitize_option() needs to handle WP_Error. DRY the conditional calls to add_settings_error().

Merge of [32791] to the 4.2 branch.

Props wonderboymusic, chriscct7 for an initial patch.

Fixes #32350.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • branches/4.2/src/wp-includes/formatting.php

    r33315 r33326  
    33283328function sanitize_option($option, $value) {
    33293329        global $wpdb;
     3330        $error = '';
    33303331
    33313332        switch ( $option ) {
    … …  
    33333334                case 'new_admin_email' :
    33343335                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3335                         $value = sanitize_email( $value );
    3336                         if ( ! is_email( $value ) ) {
    3337                                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3338                                 if ( function_exists( 'add_settings_error' ) )
    3339                                         add_settings_error( $option, 'invalid_admin_email', __( 'The email address entered did not appear to be a valid email address. Please enter a valid email address.' ) );
     3336                        if ( is_wp_error( $value ) ) {
     3337                                $error = $value->get_error_message();
     3338                        } else {
     3339                                $value = sanitize_email( $value );
     3340                                if ( ! is_email( $value ) ) {
     3341                                        $error = __( 'The email address entered did not appear to be a valid email address. Please enter a valid email address.' );
     3342                                }
    33403343                        }
    33413344                        break;
    … …  
    33823385                case 'blogname':
    33833386                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3384                         $value = wp_kses_post( $value );
    3385                         $value = esc_html( $value );
     3387                        if ( is_wp_error( $value ) ) {
     3388                                $error = $value->get_error_message();
     3389                        } else {
     3390                                $value = wp_kses_post( $value );
     3391                                $value = esc_html( $value );
     3392                        }
    33863393                        break;
    33873394
    … …  
    34053412                case 'upload_path':
    34063413                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3407                         $value = strip_tags( $value );
    3408                         $value = wp_kses_data( $value );
     3414                        if ( is_wp_error( $value ) ) {
     3415                                $error = $value->get_error_message();
     3416                        } else {
     3417                                $value = strip_tags( $value );
     3418                                $value = wp_kses_data( $value );
     3419                        }
    34093420                        break;
    34103421
    … …  
    34223433                case 'siteurl':
    34233434                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3424                         if ( (bool)preg_match( '#http(s?)://(.+)#i', $value) ) {
    3425                                 $value = esc_url_raw($value);
     3435                        if ( is_wp_error( $value ) ) {
     3436                                $error = $value->get_error_message();
    34263437                        } else {
    3427                                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3428                                 if ( function_exists('add_settings_error') )
    3429                                         add_settings_error('siteurl', 'invalid_siteurl', __('The WordPress address you entered did not appear to be a valid URL. Please enter a valid URL.'));
     3438                                if ( preg_match( '#http(s?)://(.+)#i', $value ) ) {
     3439                                        $value = esc_url_raw( $value );
     3440                                } else {
     3441                                        $error = __( 'The WordPress address you entered did not appear to be a valid URL. Please enter a valid URL.' );
     3442                                }
    34303443                        }
    34313444                        break;
    … …  
    34333446                case 'home':
    34343447                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3435                         if ( (bool)preg_match( '#http(s?)://(.+)#i', $value) ) {
    3436                                 $value = esc_url_raw($value);
     3448                        if ( is_wp_error( $value ) ) {
     3449                                $error = $value->get_error_message();
    34373450                        } else {
    3438                                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3439                                 if ( function_exists('add_settings_error') )
    3440                                         add_settings_error('home', 'invalid_home', __('The Site address you entered did not appear to be a valid URL. Please enter a valid URL.'));
     3451                                if ( preg_match( '#http(s?)://(.+)#i', $value ) ) {
     3452                                        $value = esc_url_raw( $value );
     3453                                } else {
     3454                                        $error = __( 'The Site address you entered did not appear to be a valid URL. Please enter a valid URL.' );
     3455                                }
    34413456                        }
    34423457                        break;
    … …  
    34543469                case 'illegal_names':
    34553470                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3456                         if ( ! is_array( $value ) )
    3457                                 $value = explode( ' ', $value );
    3458 
    3459                         $value = array_values( array_filter( array_map( 'trim', $value ) ) );
    3460 
    3461                         if ( ! $value )
    3462                                 $value = '';
     3471                        if ( is_wp_error( $value ) ) {
     3472                                $error = $value->get_error_message();
     3473                        } else {
     3474                                if ( ! is_array( $value ) )
     3475                                        $value = explode( ' ', $value );
     3476
     3477                                $value = array_values( array_filter( array_map( 'trim', $value ) ) );
     3478
     3479                                if ( ! $value )
     3480                                        $value = '';
     3481                        }
    34633482                        break;
    34643483
    … …  
    34663485                case 'banned_email_domains':
    34673486                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3468                         if ( ! is_array( $value ) )
    3469                                 $value = explode( "\n", $value );
    3470 
    3471                         $domains = array_values( array_filter( array_map( 'trim', $value ) ) );
    3472                         $value = array();
    3473 
    3474                         foreach ( $domains as $domain ) {
    3475                                 if ( ! preg_match( '/(--|\.\.)/', $domain ) && preg_match( '|^([a-zA-Z0-9-\.])+$|', $domain ) )
    3476                                         $value[] = $domain;
     3487                        if ( is_wp_error( $value ) ) {
     3488                                $error = $value->get_error_message();
     3489                        } else {
     3490                                if ( ! is_array( $value ) )
     3491                                        $value = explode( "\n", $value );
     3492
     3493                                $domains = array_values( array_filter( array_map( 'trim', $value ) ) );
     3494                                $value = array();
     3495
     3496                                foreach ( $domains as $domain ) {
     3497                                        if ( ! preg_match( '/(--|\.\.)/', $domain ) && preg_match( '|^([a-zA-Z0-9-\.])+$|', $domain ) ) {
     3498                                                $value[] = $domain;
     3499                                        }
     3500                                }
     3501                                if ( ! $value )
     3502                                        $value = '';
    34773503                        }
    3478                         if ( ! $value )
    3479                                 $value = '';
    34803504                        break;
    34813505
    … …  
    34833507                        $allowed_zones = timezone_identifiers_list();
    34843508                        if ( ! in_array( $value, $allowed_zones ) && ! empty( $value ) ) {
    3485                                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3486                                 if ( function_exists('add_settings_error') )
    3487                                         add_settings_error('timezone_string', 'invalid_timezone_string', __('The timezone you have entered is not valid. Please select a valid timezone.') );
     3509                                $error = __( 'The timezone you have entered is not valid. Please select a valid timezone.' );
    34883510                        }
    34893511                        break;
    … …  
    34933515                case 'tag_base':
    34943516                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3495                         $value = esc_url_raw( $value );
    3496                         $value = str_replace( 'http://', '', $value );
     3517                        if ( is_wp_error( $value ) ) {
     3518                                $error = $value->get_error_message();
     3519                        } else {
     3520                                $value = esc_url_raw( $value );
     3521                                $value = str_replace( 'http://', '', $value );
     3522                        }
    34973523                        break;
    34983524
    … …  
    35053531                case 'blacklist_keys':
    35063532                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3507                         $value = explode( "\n", $value );
    3508                         $value = array_filter( array_map( 'trim', $value ) );
    3509                         $value = array_unique( $value );
    3510                         $value = implode( "\n", $value );
     3533                        if ( is_wp_error( $value ) ) {
     3534                                $error = $value->get_error_message();
     3535                        } else {
     3536                                $value = explode( "\n", $value );
     3537                                $value = array_filter( array_map( 'trim', $value ) );
     3538                                $value = array_unique( $value );
     3539                                $value = implode( "\n", $value );
     3540                        }
    35113541                        break;
     3542        }
     3543
     3544        if ( ! empty( $error ) ) {
     3545                $value = get_option( $option );
     3546                if ( function_exists( 'add_settings_error' ) ) {
     3547                        add_settings_error( $option, "invalid_{$option}", $error );
     3548                }
    35123549        }
    35133550
Note: See TracChangeset for help on using the changeset viewer.