Changeset 13235 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/readme/class-parser.php

Timestamp:

02/22/2024 05:46:59 AM (2 years ago)

Author:

dd32

Message:

Plugin Directory: Introduce a maximum length to the readme sections.

See #7477.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/readme/class-parser.php (modified) (8 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/readme/class-parser.php

@@ -100 +100 @@
      * @var array
      */
-    private $expected_sections = array(
+    public $expected_sections = array(
         'description',
         'installation',
@@ -115 +115 @@
      * @var array
      */
-    private $alias_sections = array(
+    public $alias_sections = array(
         'frequently_asked_questions' => 'faq',
         'change_log'                 => 'changelog',
@@ -126 +126 @@
      * @var array
      */
-    private $valid_headers = array(
+    public $valid_headers = array(
         'tested'            => 'tested',
         'tested up to'      => 'tested',
@@ -149 +149 @@
         'wordpress',
     );
+
+    /**
+     * The maximum field lengths for the readme.
+     *
+     * @var array
+     */
+    public $maximum_field_lengths = array(
+        'short_description' => 150,
+        'section'           => 1500,
+    );
+
+    /**
+     * The raw contents of the readme file.
+     *
+     * @var string
+     */
+    public $raw_contents = '';
 
     /**
@@ -197 +214 @@
      */
     protected function parse_readme_contents( $contents ) {
+        $this->raw_contents = $contents;
+
         if ( preg_match( '!!u', $contents ) ) {
             $contents = preg_split( '!\R!u', $contents );
@@ -411 +430 @@
         }
 
+        foreach ( $this->sections as $section => $content ) {
+            $this->sections[ $section ] = $this->trim_length( $content, 'section', 'words' );
+
+            if ( $content !== $this->sections[ $section ] ) {
+                $this->warnings["trimmed_section_{$section}"] = true;
+            }
+        }
+
         // Display FAQs as a definition list.
         if ( isset( $this->sections['faq'] ) ) {
@@ -432 +459 @@
         $this->short_description = $this->parse_markdown( $this->short_description );
         $this->short_description = wp_strip_all_tags( $this->short_description );
-        $short_description       = $this->trim_length( $this->short_description, 150 );
+        $short_description       = $this->trim_length( $this->short_description, 'short_description' );
         if ( $short_description !== $this->short_description ) {
             if ( empty( $this->warnings['no_short_description_present'] ) ) {
@@ -506 +533 @@
      * @param string $desc
      * @param int    $length
+     * @param string $type   The type of the length, 'char' or 'words'.
      * @return string
      */
-    protected function trim_length( $desc, $length = 150 ) {
+    protected function trim_length( $desc, $length = 150, $type = 'char' ) {
+        if ( is_string( $length ) ) {
+            $length = $this->maximum_field_lengths[ $length ] ?? $length;
+        }
+
+        if ( 'words' === $type ) {
+            return wp_trim_words( $desc, $length, ' …' );
+        }
+
         // Apply the length restriction without counting html entities.
         $str_length = mb_strlen( html_entity_decode( $desc ) ?: $desc );