Ticket #44541: 44541.patch

src/wp-admin/includes/dashboard.php

@@ -587 +587 @@
         }
         echo '<h2 class="hide-if-no-js">' . __( 'Your Recent Drafts' ) . "</h2>\n<ul>";
 
+        /* translators: This sets the text length for the draft. */
+        $draft_length = intval( _x( '10', 'draft_length' ) );
+
         $drafts = array_slice( $drafts, 0, 3 );
         foreach ( $drafts as $draft ) {
                 $url   = get_edit_post_link( $draft->ID );
@@ -595 +598 @@
                 /* translators: %s: post title */
                 echo '<div class="draft-title"><a href="' . esc_url( $url ) . '" aria-label="' . esc_attr( sprintf( __( 'Edit &#8220;%s&#8221;' ), $title ) ) . '">' . esc_html( $title ) . '</a>';
                 echo '<time datetime="' . get_the_time( 'c', $draft ) . '">' . get_the_time( __( 'F j, Y' ), $draft ) . '</time></div>';
-                if ( $the_content = wp_trim_words( $draft->post_content, 10 ) ) {
+                if ( $the_content = wp_trim_words( $draft->post_content, $draft_length ) ) {
                         echo '<p>' . $the_content . '</p>';
                 }
                 echo "</li>\n";

src/wp-includes/comment-template.php

@@ -593 +593 @@
 function get_comment_excerpt( $comment_ID = 0 ) {
         $comment      = get_comment( $comment_ID );
         $comment_text = strip_tags( str_replace( array( "\n", "\r" ), ' ', $comment->comment_content ) );
-        $words        = explode( ' ', $comment_text );
+
+        /* translators: This sets the text length for the comment excerpt. */
+        $comment_excerpt_length = intval( _x( '20', 'comment_excerpt_length' ) );
 
         /**
          * Filters the amount of words used in the comment excerpt.
@@ -602 +604 @@
          *
          * @param int $comment_excerpt_length The amount of words you want to display in the comment excerpt.
          */
-        $comment_excerpt_length = apply_filters( 'comment_excerpt_length', 20 );
+        $comment_excerpt_length = apply_filters( 'comment_excerpt_length', $comment_excerpt_length );
 
-        $use_ellipsis = count( $words ) > $comment_excerpt_length;
-        if ( $use_ellipsis ) {
-                $words = array_slice( $words, 0, $comment_excerpt_length );
-        }
+        $excerpt = wp_trim_words( $comment_text, $comment_excerpt_length, '…' );
 
-        $excerpt = trim( join( ' ', $words ) );
-        if ( $use_ellipsis ) {
-                $excerpt .= '…';
-        }
         /**
          * Filters the retrieved comment excerpt.
          *

src/wp-includes/formatting.php

@@ -3636 +3636 @@
                 $text = apply_filters( 'the_content', $text );
                 $text = str_replace( ']]>', ']]>', $text );
 
+                /* translators: This sets the text length for the excerpt. */
+                $excerpt_length = intval( _x( '55', 'excerpt_length' ) );
+
                 /**
                  * Filters the number of words in an excerpt.
                  *
@@ -3643 +3646 @@
                  *
                  * @param int $number The number of words. Default 55.
                  */
-                $excerpt_length = apply_filters( 'excerpt_length', 55 );
+                $excerpt_length = apply_filters( 'excerpt_length', $excerpt_length );
                 /**
                  * Filters the string in the "more" link displayed after a trimmed excerpt.
                  *

tests/phpunit/data/languages/ja_JP.po

@@ -40 +40 @@
 #: wp-includes/script-loader.php:620
 msgid "Update %s now"
 msgstr "今すぐ %s を更新"
+
+#. translators: If your word count is based on single characters (e.g. East
+#. Asian characters), enter 'characters_excluding_spaces' or
+#. 'characters_including_spaces'. Otherwise, enter 'words'. Do not translate
+#. into your own language.
+#: wp-includes/formatting.php:3372 wp-includes/script-loader.php:1100
+msgctxt "Word count type. Do not translate!"
+msgid "words"
+msgstr "characters_including_spaces"
+
+#. translators: This sets the text length for the excerpt.
+#: wp-includes/formatting.php:3640
+msgctxt "excerpt_length"
+msgid "55"
+msgstr "110"
+
+#. translators: This sets the text length for the comment excerpt.
+#: src/wp-includes/comment-template.ph:599
+msgctxt "comment_excerpt_length"
+msgid "20"
+msgstr "40"
+
+#. translators: This sets the text length for the comment excerpt.
+#: src/wp-admin/includes/dashboard.php:591
+msgctxt "draft_length"
+msgid "10"
+msgstr "40"

tests/phpunit/tests/formatting/WPTrimWords.php

@@ -42 +42 @@
                 $text = 'This is some short text.';
                 $this->assertEquals( $text, wp_trim_words( $text ) );
         }
+
+        function test_trims_to_20_counted_by_chars() {
+                switch_to_locale( 'ja_JP' );
+                $expected = substr( $this->long_text, 0, 20 ) . '…';
+                $this->assertEquals( $expected, wp_trim_words( $this->long_text, 20 ) );
+                restore_previous_locale();
+        }
+
+        function test_trims_to_20_counted_by_chars_with_double_width_chars() {
+                switch_to_locale( 'ja_JP' );
+                $text = str_repeat( 'あ', 100 );
+                $expected = str_repeat( 'あ', 19 ) . '…';
+                $this->assertEquals( $expected, wp_trim_words( $text, 19 ) );
+                restore_previous_locale();
+        }
 }

tests/phpunit/tests/l10n.php

@@ -5 +5 @@
  * @group i18n
  */
 class Tests_L10n extends WP_UnitTestCase {
-
         /**
          * @ticket 35961
          */
@@ -242 +241 @@
                 $this->assertNotEmpty( $array['Project-Id-Version'] );
                 $this->assertNotEmpty( $array['X-Generator'] );
         }
+
+        /**
+         * The text length of excerpt should be counted by words.
+         */
+        function test_length_of_excerpt_should_be_counted_by_words() {
+                global $post;
+
+                switch_to_locale( 'en_US' );
+
+                $args = array(
+                        'post_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                        'post_excerpt' => '',
+                );
+
+                $post = $this->factory()->post->create_and_get( $args );
+                setup_postdata( $post );
+
+                $expect = "<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat [&hellip;]</p>\n";
+                $this->expectOutputString( $expect );
+                the_excerpt();
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of excerpt should be counted by chars.
+         */
+        function test_length_of_excerpt_should_be_counted_by_chars() {
+                global $post;
+
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'post_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                        'post_excerpt' => '',
+                );
+
+                $post = $this->factory()->post->create_and_get( $args );
+                setup_postdata( $post );
+
+                $expect = "<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore  [&hellip;]</p>\n";
+                $this->expectOutputString( $expect );
+                the_excerpt();
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of excerpt should be counted by chars with Japanese.
+         */
+        function test_length_of_excerpt_should_be_counted_by_chars_in_japanese() {
+                global $post;
+
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'post_content' => str_repeat( 'あ', 200 ),
+                        'post_excerpt' => '',
+                );
+
+                $post = $this->factory()->post->create_and_get( $args );
+                setup_postdata( $post );
+
+                $expect = "<p>" . str_repeat( 'あ', 110 ) . " [&hellip;]</p>\n";
+                $this->expectOutputString( $expect );
+                the_excerpt();
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of excerpt should be counted by words.
+         */
+        function test_length_of_excerpt_rss_should_be_counted_by_words() {
+                global $post;
+
+                switch_to_locale( 'en_US' );
+
+                $args = array(
+                        'post_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                        'post_excerpt' => '',
+                );
+
+                $post = $this->factory()->post->create_and_get( $args );
+                setup_postdata( $post );
+
+                $expect = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat [&#8230;]";
+                $this->expectOutputString( $expect );
+                the_excerpt_rss();
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of excerpt should be counted by chars.
+         */
+        function test_length_of_excerpt_rss_should_be_counted_by_chars() {
+                global $post;
+
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'post_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                        'post_excerpt' => '',
+                );
+
+                $post = $this->factory()->post->create_and_get( $args );
+                setup_postdata( $post );
+
+                $expect = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore  [&#8230;]";
+                $this->expectOutputString( $expect );
+                the_excerpt_rss();
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of draft in dashboard should be counted by chars.
+         */
+        function test_length_of_draft_should_be_counted_by_words() {
+                require_once dirname( dirname( dirname( dirname( __FILE__ ) ) ) ) . '/src/wp-admin/includes/dashboard.php';
+
+                switch_to_locale( 'en_US' );
+
+                $args = array(
+                        'post_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                        'post_excerpt' => '',
+                        'post_status'  => 'draft',
+                );
+
+                $post = $this->factory()->post->create( $args );
+
+                $expect = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do&hellip;";
+
+                ob_start();
+                wp_dashboard_recent_drafts();
+                $result = ob_get_clean();
+
+                $this->assertTrue( !! strpos( $result, $expect ) );
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of draft in dashboard should be counted by chars.
+         */
+        function test_length_of_draft_should_be_counted_by_chars() {
+                require_once dirname( dirname( dirname( dirname( __FILE__ ) ) ) ) . '/src/wp-admin/includes/dashboard.php';
+
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'post_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                        'post_excerpt' => '',
+                        'post_status'  => 'draft',
+                );
+
+                $post = $this->factory()->post->create( $args );
+
+                $expect = "Lorem ipsum dolor sit amet, consectetur &hellip;";
+
+                ob_start();
+                wp_dashboard_recent_drafts();
+                $result = ob_get_clean();
+
+                $this->assertTrue( !! strpos( $result, $expect ) );
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of draft in dashboard should be counted by chars in Japanese.
+         */
+        function test_length_of_draft_should_be_counted_by_chars_in_japanese() {
+                require_once dirname( dirname( dirname( dirname( __FILE__ ) ) ) ) . '/src/wp-admin/includes/dashboard.php';
+
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'post_content' => str_repeat( 'あ', 200 ),
+                        'post_excerpt' => '',
+                        'post_status'  => 'draft',
+                );
+
+                $post = $this->factory()->post->create( $args );
+
+                $expect = str_repeat( 'あ', 40 ) . "&hellip;";
+
+                ob_start();
+                wp_dashboard_recent_drafts();
+                $result = ob_get_clean();
+
+                $this->assertTrue( !! strpos( $result, $expect ) );
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of draft in dashboard should be counted by chars.
+         */
+        function test_length_of_comment_excerpt_should_be_counted_by_words() {
+                switch_to_locale( 'en_US' );
+
+                $args = array(
+                        'comment_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                );
+
+                $comment_id = $this->factory()->comment->create( $args );
+
+                $expect = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut&hellip;";
+
+                $comment_excerpt = get_comment_excerpt( $comment_id );
+
+                $this->assertSame( $expect, $comment_excerpt );
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of draft in dashboard should be counted by chars.
+         */
+        function test_length_of_comment_excerpt_should_be_counted_by_chars() {
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'comment_content' => 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
+                );
+
+                $comment_id = $this->factory()->comment->create( $args );
+
+                $expect = "Lorem ipsum dolor sit amet, consectetur &hellip;";
+
+                $comment_excerpt = get_comment_excerpt( $comment_id );
+
+                $this->assertSame( $expect, $comment_excerpt );
+
+                restore_previous_locale();
+        }
+
+        /**
+         * The text length of draft in dashboard should be counted by chars.
+         */
+        function test_length_of_comment_excerpt_should_be_counted_by_chars_in_Japanese() {
+                switch_to_locale( 'ja_JP' );
+
+                $args = array(
+                        'comment_content' => str_repeat( 'あ', 200 ),
+                );
+
+                $comment_id = $this->factory()->comment->create( $args );
+
+                $expect = str_repeat( 'あ', 40 ) . "&hellip;";
+
+                $comment_excerpt = get_comment_excerpt( $comment_id );
+
+                $this->assertSame( $expect, $comment_excerpt );
+
+                restore_previous_locale();
+        }
 }