WordPress.org

Make WordPress Core

Ticket #10377: 10377.4.diff

File 10377.4.diff, 7.1 KB (added by rachelbaker, 5 years ago)

with new function wp_get_comment_column_max_length() and unit tests

  • src/wp-includes/comment-template.php

     
    20662066        $fields = apply_filters( 'comment_form_default_fields', $fields );
    20672067        $defaults = array(
    20682068                'fields'               => $fields,
    2069                 'comment_field'        => '<p class="comment-form-comment"><label for="comment">' . _x( 'Comment', 'noun' ) . '</label> <textarea id="comment" name="comment" cols="45" rows="8" aria-required="true" required="required"></textarea></p>',
     2069                'comment_field'        => '<p class="comment-form-comment"><label for="comment">' . _x( 'Comment', 'noun' ) . '</label> <textarea id="comment" name="comment" cols="45" rows="8" maxlength="65525" aria-required="true" required="required"></textarea></p>',
    20702070                /** This filter is documented in wp-includes/link-template.php */
    20712071                'must_log_in'          => '<p class="must-log-in">' . sprintf( __( 'You must be <a href="%s">logged in</a> to post a comment.' ), wp_login_url( apply_filters( 'the_permalink', get_permalink( $post_id ) ) ) ) . '</p>',
    20722072                /** This filter is documented in wp-includes/link-template.php */
  • src/wp-includes/comment.php

     
    948948}
    949949
    950950/**
     951 * Calculate the maximum character length of a column from the comments table.
     952 *
     953 * @since 4.5.0
     954 *
     955 * @global wpdb $wpdb WordPress database abstraction object.
     956 *
     957 * @param string $column Name of a column in the comments table.
     958 * @return int Maximum column character length.
     959 */
     960function wp_get_comment_column_max_length( $column ) {
     961        global $wpdb;
     962
     963        $col_length = $wpdb->get_col_length( $wpdb->comments, $column );
     964        if ( ! is_array( $col_length ) && (int) $col_length > 0 ) {
     965                $max_length = (int) $col_length;
     966        } elseif ( is_array( $col_length ) && isset( $col_length['length'] ) && (int) $col_length > 0 ) {
     967                $max_length = (int) $col_length['length'];
     968        } else {
     969                $max_length = 255;
     970        }
     971
     972        if ( ! empty( $col_length['type'] && 'byte' === $col_length['type'] ) ) {
     973                $max_length = $max_length - 10;
     974        }
     975
     976        /**
     977         * Filters the calculated length for a given column of the comments table.
     978         *
     979         * @since 4.5.0
     980         *
     981         * @param int    $max_length Maximum column character length.
     982         * @param string $column     Column name.
     983         */
     984        return apply_filters( 'wp_get_comment_column_max_length', $max_length, $column );
     985}
     986
     987/**
    951988 * Does comment contain blacklisted characters or words.
    952989 *
    953990 * @since 1.5.0
     
    27782815                }
    27792816        }
    27802817
     2818        if ( isset( $comment_author ) && wp_get_comment_column_max_length( 'comment_author' ) < strlen( $comment_author ) ) {
     2819                return new WP_Error( 'comment_author_column_length', __( '<strong>ERROR</strong>: your name is too long.' ), 200 );
     2820        }
     2821
     2822        if ( isset( $comment_author_email ) && wp_get_comment_column_max_length( 'comment_author_email' ) < strlen( $comment_author_email ) ) {
     2823                return new WP_Error( 'comment_author_email_column_length', __( '<strong>ERROR</strong>: your email address is too long.' ), 200 );
     2824        }
     2825
     2826        if ( isset( $comment_author_url ) && wp_get_comment_column_max_length( 'comment_author_url' ) < strlen( $comment_author_url ) ) {
     2827                return new WP_Error( 'comment_author_url_column_length', __( '<strong>ERROR</strong>: your url is too long.' ), 200 );
     2828        }
     2829
    27812830        if ( '' == $comment_content ) {
    27822831                return new WP_Error( 'require_valid_comment', __( '<strong>ERROR</strong>: please type a comment.' ), 200 );
     2832        } elseif ( wp_get_comment_column_max_length( 'comment_content' ) < strlen( $comment_content ) ) {
     2833                return new WP_Error( 'comment_content_column_length', __( '<strong>ERROR</strong>: your comment is too long.' ), 200 );
    27832834        }
    27842835
    27852836        $commentdata = compact(
  • tests/phpunit/includes/utils.php

     
    66        return substr(md5(uniqid(rand())), 0, $len);
    77}
    88
     9function rand_long_str( $length ) {
     10        $chars = 'abcdefghijklmnopqrstuvwxyz';
     11        $string = '';
     12
     13        for ( $i = 0; $i < $length; $i++ ) {
     14                $rand = rand( 0, strlen( $chars ) - 1 );
     15                $string .= substr( $chars, $rand, 1 );
     16        }
     17
     18        return $string;
     19}
     20
    921// strip leading and trailing whitespace from each line in the string
    1022function strip_ws($txt) {
    1123        $lines = explode("\n", $txt);
  • tests/phpunit/tests/comment-submission.php

     
    593593        }
    594594
    595595        /**
     596         * @ticket 10377
     597         */
     598        public function test_submitting_comment_with_content_too_long_returns_error() {
     599                $error = 'comment_content_column_length';
     600
     601                $post = self::factory()->post->create_and_get();
     602
     603                $data = array(
     604                        'comment_post_ID' => $post->ID,
     605                        'comment'         => rand_long_str( 65536 ),
     606                        'author'          => 'Comment Author',
     607                        'email'           => 'comment@example.org',
     608                );
     609                $comment = wp_handle_comment_submission( $data );
     610
     611                $this->assertWPError( $comment );
     612                $this->assertSame( $error, $comment->get_error_code() );
     613        }
     614
     615        /**
     616         * @ticket 10377
     617         */
     618        public function test_submitting_comment_with_author_too_long_returns_error() {
     619                $error = 'comment_author_column_length';
     620
     621                $post = self::factory()->post->create_and_get();
     622
     623                $data = array(
     624                        'comment_post_ID' => $post->ID,
     625                        'comment'         => rand_str(),
     626                        'author'          => rand_long_str( 255 ),
     627                        'email'           => 'comment@example.org',
     628                );
     629                $comment = wp_handle_comment_submission( $data );
     630
     631                $this->assertWPError( $comment );
     632                $this->assertSame( $error, $comment->get_error_code() );
     633        }
     634
     635        /**
     636         * @ticket 10377
     637         */
     638        public function test_submitting_comment_with_email_too_long_returns_error() {
     639                $error = 'comment_author_email_column_length';
     640
     641                $post = self::factory()->post->create_and_get();
     642
     643                $data = array(
     644                        'comment_post_ID' => $post->ID,
     645                        'comment'         => rand_str(),
     646                        'author'          => 'Comment Author',
     647                        'email'           => rand_long_str( 90 ) . '@example.com',
     648                );
     649                $comment = wp_handle_comment_submission( $data );
     650
     651                $this->assertWPError( $comment );
     652                $this->assertSame( $error, $comment->get_error_code() );
     653        }
     654
     655        /**
     656         * @ticket 10377
     657         */
     658        public function test_submitting_comment_with_url_too_long_returns_error() {
     659                $error = 'comment_author_url_column_length';
     660
     661                $post = self::factory()->post->create_and_get();
     662                $data = array(
     663                        'comment_post_ID' => $post->ID,
     664                        'comment'         => rand_str(),
     665                        'author'          => 'Comment Author',
     666                        'email'           => 'comment@example.org',
     667                        'url'             => rand_long_str( 201 ),
     668                );
     669                $comment = wp_handle_comment_submission( $data );
     670
     671                $this->assertWPError( $comment );
     672                $this->assertSame( $error, $comment->get_error_code() );
     673        }
     674
     675        /**
    596676         * @ticket 34997
    597677         */
    598678        public function test_comment_submission_sends_all_expected_parameters_to_preprocess_comment_filter() {