WordPress.org
Get WordPress

Make WordPress Core

Ticket #32308: 32308.1.patch

File 32308.1.patch, 7.8 KB (added by chriscct7, 10 years ago)

Fixes formatting issue

  • wp-includes/formatting.php

     
    33323332                case 'admin_email' :
    33333333                case 'new_admin_email' :
    33343334                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3335                         $value = sanitize_email( $value );
    3336                         if ( ! is_email( $value ) ) {
     3335                        if ( is_wp_error( $value ) ) {
    33373336                                $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.' ) );
     3337                                add_settings_error( $option, $value->get_error_message() );
     3338                        } else {
     3339                                $value = sanitize_email( $value );
     3340                                if ( ! is_email( $value ) ) {
     3341                                        $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3342                                        if ( function_exists( 'add_settings_error' ) )
     3343                                                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.' ) );
     3344                                }
    33403345                        }
    33413346                        break;
    33423347
    … …  
    33813386                case 'blogdescription':
    33823387                case 'blogname':
    33833388                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3384                         $value = wp_kses_post( $value );
    3385                         $value = esc_html( $value );
     3389                        if ( is_wp_error( $value ) ) {
     3390                                $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3391                                add_settings_error( $option, $value->get_error_message() );
     3392                        } else {                       
     3393                                $value = wp_kses_post( $value );
     3394                                $value = esc_html( $value );
     3395                        }
    33863396                        break;
    33873397
    33883398                case 'blog_charset':
    … …  
    34043414                case 'mailserver_pass':
    34053415                case 'upload_path':
    34063416                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3407                         $value = strip_tags( $value );
    3408                         $value = wp_kses_data( $value );
     3417                        if ( is_wp_error( $value ) ) {
     3418                                $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3419                                add_settings_error( $option, $value->get_error_message() );
     3420                        } else {                       
     3421                                $value = strip_tags( $value );
     3422                                $value = wp_kses_data( $value );
     3423                        }
    34093424                        break;
    34103425
    34113426                case 'ping_sites':
    … …  
    34213436
    34223437                case 'siteurl':
    34233438                        $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);
    3426                         } else {
     3439                        if ( is_wp_error( $value ) ) {
    34273440                                $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.'));
     3441                                add_settings_error( $option, $value->get_error_message() );
     3442                        } else {                       
     3443                                if ( (bool)preg_match( '#http(s?)://(.+)#i', $value) ) {
     3444                                        $value = esc_url_raw($value);
     3445                                } else {
     3446                                        $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3447                                        if ( function_exists('add_settings_error') )
     3448                                                add_settings_error('siteurl', 'invalid_siteurl', __('The WordPress address you entered did not appear to be a valid URL. Please enter a valid URL.'));
     3449                                }
    34303450                        }
    34313451                        break;
    34323452
    34333453                case 'home':
    34343454                        $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);
    3437                         } else {
     3455                        if ( is_wp_error( $value ) ) {
    34383456                                $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.'));
     3457                                add_settings_error( $option, $value->get_error_message() );
     3458                        } else {                       
     3459                                if ( (bool)preg_match( '#http(s?)://(.+)#i', $value) ) {
     3460                                        $value = esc_url_raw($value);
     3461                                } else {
     3462                                        $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3463                                        if ( function_exists('add_settings_error') )
     3464                                                add_settings_error('home', 'invalid_home', __('The Site address you entered did not appear to be a valid URL. Please enter a valid URL.'));
     3465                                }
    34413466                        }
    34423467                        break;
    34433468
    … …  
    34533478
    34543479                case 'illegal_names':
    34553480                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3456                         if ( ! is_array( $value ) )
    3457                                 $value = explode( ' ', $value );
     3481                        if ( is_wp_error( $value ) ) {
     3482                                $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3483                                add_settings_error( $option, $value->get_error_message() );
     3484                        } else {                       
     3485                                if ( ! is_array( $value ) )
     3486                                        $value = explode( ' ', $value );
    34583487
    3459                         $value = array_values( array_filter( array_map( 'trim', $value ) ) );
     3488                                $value = array_values( array_filter( array_map( 'trim', $value ) ) );
    34603489
    3461                         if ( ! $value )
    3462                                 $value = '';
     3490                                if ( ! $value ) {
     3491                                        $value = '';
     3492                                }
     3493                        }
    34633494                        break;
    34643495
    34653496                case 'limited_email_domains':
    34663497                case 'banned_email_domains':
    34673498                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3468                         if ( ! is_array( $value ) )
    3469                                 $value = explode( "\n", $value );
     3499                        if ( is_wp_error( $value ) ) {
     3500                                $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3501                                add_settings_error( $option, $value->get_error_message() );
     3502                        } else {                       
     3503                                if ( ! is_array( $value ) ) {
     3504                                        $value = explode( "\n", $value );
     3505                                }
    34703506
    3471                         $domains = array_values( array_filter( array_map( 'trim', $value ) ) );
    3472                         $value = array();
     3507                                $domains = array_values( array_filter( array_map( 'trim', $value ) ) );
     3508                                $value = array();
    34733509
    3474                         foreach ( $domains as $domain ) {
    3475                                 if ( ! preg_match( '/(--|\.\.)/', $domain ) && preg_match( '|^([a-zA-Z0-9-\.])+$|', $domain ) )
    3476                                         $value[] = $domain;
     3510                                foreach ( $domains as $domain ) {
     3511                                        if ( ! preg_match( '/(--|\.\.)/', $domain ) && preg_match( '|^([a-zA-Z0-9-\.])+$|', $domain ) ) {
     3512                                                $value[] = $domain;
     3513                                        }
     3514                                }
     3515                                if ( ! $value ) {
     3516                                        $value = '';
     3517                                }
    34773518                        }
    3478                         if ( ! $value )
    3479                                 $value = '';
    34803519                        break;
    34813520
    34823521                case 'timezone_string':
    … …  
    34923531                case 'category_base':
    34933532                case 'tag_base':
    34943533                        $value = $wpdb->strip_invalid_text_for_column( $wpdb->options, 'option_value', $value );
    3495                         $value = esc_url_raw( $value );
    3496                         $value = str_replace( 'http://', '', $value );
     3534                        if ( is_wp_error( $value ) ) {
     3535                                $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3536                                add_settings_error( $option, $value->get_error_message() );
     3537                        } else {                       
     3538                                $value = esc_url_raw( $value );
     3539                                $value = str_replace( 'http://', '', $value );
     3540                        }
    34973541                        break;
    34983542
    34993543                case 'default_role' :
    … …  
    35043548                case 'moderation_keys':
    35053549                case 'blacklist_keys':
    35063550                        $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 );
     3551                        if ( is_wp_error( $value ) ) {
     3552                                $value = get_option( $option ); // Resets option to stored value in the case of failed sanitization
     3553                                add_settings_error( $option, $value->get_error_message() );
     3554                        } else {                       
     3555                                $value = explode( "\n", $value );
     3556                                $value = array_filter( array_map( 'trim', $value ) );
     3557                                $value = array_unique( $value );
     3558                                $value = implode( "\n", $value );
     3559                        }
    35113560                        break;
    35123561        }
    35133562