WordPress.org
Get WordPress

Make WordPress Core

Changeset 32791


Ignore:
Timestamp:
06/16/2015 01:28:47 AM (9 years ago)
Author:
wonderboymusic
Message:

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

Props chriscct7 for an initial patch.
Fixes #32350.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/formatting.php

    r32790 r32791  
    33663366
    33673367    $original_value = $value;
     3368    $error = '';
    33683369
    33693370    switch ( $option ) {
    … …  
    33713372        case 'new_admin_email' :
    33723373            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3373             $value = sanitize_email( $value );
    3374             if ( ! is_email( $value ) ) {
    3375                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3376                 if ( function_exists( 'add_settings_error' ) )
    3377                     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.' ) );
     3374            if ( is_wp_error( $value ) ) {
     3375                $error = $value->get_error_message();
     3376            } else {
     3377                $value = sanitize_email( $value );
     3378                if ( ! is_email( $value ) ) {
     3379                    $error = __( 'The email address entered did not appear to be a valid email address. Please enter a valid email address.' );
     3380                }
    33783381            }
    33793382            break;
    … …  
    34203423        case 'blogname':
    34213424            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3422             $value = wp_kses_post( $value );
    3423             $value = esc_html( $value );
     3425            if ( is_wp_error( $value ) ) {
     3426                $error = $value->get_error_message();
     3427            } else {
     3428                $value = wp_kses_post( $value );
     3429                $value = esc_html( $value );
     3430            }
    34243431            break;
    34253432
    … …  
    34433450        case 'upload_path':
    34443451            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3445             $value = strip_tags( $value );
    3446             $value = wp_kses_data( $value );
     3452            if ( is_wp_error( $value ) ) {
     3453                $error = $value->get_error_message();
     3454            } else {
     3455                $value = strip_tags( $value );
     3456                $value = wp_kses_data( $value );
     3457            }
    34473458            break;
    34483459
    … …  
    34603471        case 'siteurl':
    34613472            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3462             if ( (bool)preg_match( '#http(s?)://(.+)#i', $value) ) {
    3463                 $value = esc_url_raw($value);
     3473            if ( is_wp_error( $value ) ) {
     3474                $error = $value->get_error_message();
    34643475            } else {
    3465                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3466                 if ( function_exists('add_settings_error') )
    3467                     add_settings_error('siteurl', 'invalid_siteurl', __('The WordPress address you entered did not appear to be a valid URL. Please enter a valid URL.'));
     3476                if ( preg_match( '#http(s?)://(.+)#i', $value ) ) {
     3477                    $value = esc_url_raw( $value );
     3478                } else {
     3479                    $error = __( 'The WordPress address you entered did not appear to be a valid URL. Please enter a valid URL.' );
     3480                }
    34683481            }
    34693482            break;
    … …  
    34713484        case 'home':
    34723485            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3473             if ( (bool)preg_match( '#http(s?)://(.+)#i', $value) ) {
    3474                 $value = esc_url_raw($value);
     3486            if ( is_wp_error( $value ) ) {
     3487                $error = $value->get_error_message();
    34753488            } else {
    3476                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3477                 if ( function_exists('add_settings_error') )
    3478                     add_settings_error('home', 'invalid_home', __('The Site address you entered did not appear to be a valid URL. Please enter a valid URL.'));
     3489                if ( preg_match( '#http(s?)://(.+)#i', $value ) ) {
     3490                    $value = esc_url_raw( $value );
     3491                } else {
     3492                    $error = __( 'The Site address you entered did not appear to be a valid URL. Please enter a valid URL.' );
     3493                }
    34793494            }
    34803495            break;
    … …  
    34923507        case 'illegal_names':
    34933508            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3494             if ( ! is_array( $value ) )
    3495                 $value = explode( ' ', $value );
    3496 
    3497             $value = array_values( array_filter( array_map( 'trim', $value ) ) );
    3498 
    3499             if ( ! $value )
    3500                 $value = '';
     3509            if ( is_wp_error( $value ) ) {
     3510                $error = $value->get_error_message();
     3511            } else {
     3512                if ( ! is_array( $value ) )
     3513                    $value = explode( ' ', $value );
     3514
     3515                $value = array_values( array_filter( array_map( 'trim', $value ) ) );
     3516
     3517                if ( ! $value )
     3518                    $value = '';
     3519            }
    35013520            break;
    35023521
    … …  
    35043523        case 'banned_email_domains':
    35053524            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3506             if ( ! is_array( $value ) )
    3507                 $value = explode( "\n", $value );
    3508 
    3509             $domains = array_values( array_filter( array_map( 'trim', $value ) ) );
    3510             $value = array();
    3511 
    3512             foreach ( $domains as $domain ) {
    3513                 if ( ! preg_match( '/(--|\.\.)/', $domain ) && preg_match( '|^([a-zA-Z0-9-\.])+$|', $domain ) )
    3514                     $value[] = $domain;
     3525            if ( is_wp_error( $value ) ) {
     3526                $error = $value->get_error_message();
     3527            } else {
     3528                if ( ! is_array( $value ) )
     3529                    $value = explode( "\n", $value );
     3530
     3531                $domains = array_values( array_filter( array_map( 'trim', $value ) ) );
     3532                $value = array();
     3533
     3534                foreach ( $domains as $domain ) {
     3535                    if ( ! preg_match( '/(--|\.\.)/', $domain ) && preg_match( '|^([a-zA-Z0-9-\.])+$|', $domain ) ) {
     3536                        $value[] = $domain;
     3537                    }
     3538                }
     3539                if ( ! $value )
     3540                    $value = '';
    35153541            }
    3516             if ( ! $value )
    3517                 $value = '';
    35183542            break;
    35193543
    … …  
    35213545            $allowed_zones = timezone_identifiers_list();
    35223546            if ( ! in_array( $value, $allowed_zones ) && ! empty( $value ) ) {
    3523                 $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
    3524                 if ( function_exists('add_settings_error') )
    3525                     add_settings_error('timezone_string', 'invalid_timezone_string', __('The timezone you have entered is not valid. Please select a valid timezone.') );
     3547                $error = __( 'The timezone you have entered is not valid. Please select a valid timezone.' );
    35263548            }
    35273549            break;
    … …  
    35313553        case 'tag_base':
    35323554            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3533             $value = esc_url_raw( $value );
    3534             $value = str_replace( 'http://', '', $value );
     3555            if ( is_wp_error( $value ) ) {
     3556                $error = $value->get_error_message();
     3557            } else {
     3558                $value = esc_url_raw( $value );
     3559                $value = str_replace( 'http://', '', $value );
     3560            }
    35353561            break;
    35363562
    … …  
    35433569        case 'blacklist_keys':
    35443570            $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3545             $value = explode( "\n", $value );
    3546             $value = array_filter( array_map( 'trim', $value ) );
    3547             $value = array_unique( $value );
    3548             $value = implode( "\n", $value );
     3571            if ( is_wp_error( $value ) ) {
     3572                $error = $value->get_error_message();
     3573            } else {
     3574                $value = explode( "\n", $value );
     3575                $value = array_filter( array_map( 'trim', $value ) );
     3576                $value = array_unique( $value );
     3577                $value = implode( "\n", $value );
     3578            }
    35493579            break;
     3580    }
     3581
     3582    if ( ! empty( $error ) ) {
     3583        $value = get_option( $option );
     3584        if ( function_exists( 'add_settings_error' ) ) {
     3585            add_settings_error( $option, "invalid_{$option}", $error );
     3586        }
    35503587    }
    35513588
Note: See TracChangeset for help on using the changeset viewer.