Changeset 10430

Timestamp:

11/05/2020 08:15:30 PM (4 years ago)

Author:

Otto42

Message:

Plugin Directory: Readme Parser: Allow data:text/plain streams to pass through parse_readme() also, to allow them to become plain contents before parsing them. Fixes #5502, corrects [10429].

File:

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

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

@@ -154 +154 @@
      *
      * @param string $string A Filepath, URL, or contents of a readme to parse.
+     *
+     * Note: data:text/plain streams are URLs and need to pass through
+     * the parse_readme() function, not the parse_readme_contents() function, so
+     * that they can be turned from a URL into plain text via the stream.
      */
     public function __construct( $string ) {
-        if ( file_exists( $string ) || preg_match( '!^https?://!i', $string ) ) {
+        if ( file_exists( $string ) 
+            || preg_match( '!^https?://!i', $string ) 
+            || preg_match( '!^data:text/plain!i', $string) ) 
+        {
             $this->parse_readme( $string );
         } elseif ( $string ) {