WordPress.org

Make WordPress Core

Ticket #38708: 38708.2.diff

File 38708.2.diff, 1.9 KB (added by rachelbaker, 4 years ago)

Adds tests and removes extra handling in rest-api

  • src/wp-includes/formatting.php

     
    28722872                _deprecated_argument( __FUNCTION__, '3.0.0' );
    28732873
    28742874        // Test for the minimum length the email can be
    2875         if ( strlen( $email ) < 3 ) {
     2875        if ( strlen( $email ) < 6 ) {
    28762876                /**
    28772877                 * Filters whether an email address is valid.
    28782878                 *
  • src/wp-includes/rest-api.php

     
    10771077                                break;
    10781078
    10791079                        case 'email' :
    1080                                 // is_email() checks for 3 characters (a@b), but
    1081                                 // wp_handle_comment_submission() requires 6 characters (a@b.co)
    1082                                 //
    1083                                 // https://core.trac.wordpress.org/ticket/38506
    1084                                 if ( ! is_email( $value ) || strlen( $value ) < 6 ) {
     1080                                if ( ! is_email( $value ) ) {
    10851081                                        return new WP_Error( 'rest_invalid_email', __( 'Invalid email address.' ) );
    10861082                                }
    10871083                                break;
  • tests/phpunit/tests/formatting/IsEmail.php

     
    99                        "bob@example.com",
    1010                        "phil@example.info",
    1111                        "ace@204.32.222.14",
    12                         "kevin@many.subdomains.make.a.happy.man.edu"
     12                        "kevin@many.subdomains.make.a.happy.man.edu",
     13                        "a@b.co",
    1314                        );
    1415                foreach ( $data as $datum ) {
    1516                        $this->assertEquals( $datum, is_email( $datum ), $datum );
     
    2223                        'http://bob.example.com/',
    2324                        "sif i'd give u it, spamer!1",
    2425                        "com.exampleNOSPAMbob",
    25                         "bob@your mom"
     26                        "bob@your mom",
     27                        "a@b.c",
    2628                        );
    2729                foreach ($data as $datum) {
    2830                        $this->assertFalse(is_email($datum), $datum);