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

Timestamp:

04/06/2018 05:04:54 AM (6 years ago)

Author:

dd32

Message:

Plugin Directory: Readme: Ignore invalid Requires PHP headers and add a parsing warning clarifying the expected contents.

Fixes #3551.

File:

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

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

@@ -89 +89 @@
 
     /**
-     * Flag to specify that a Contributor was ignored.
-     *
-     * @var bool
-     */
-    public $contributor_ignored = false;
+     * Warning flags which indicate specific parsing failures have occured.
+     *
+     * @var array
+     */
+    public $warnings = array();
 
     /**
@@ -244 +244 @@
         }
         if ( ! empty( $headers['requires_php'] ) ) {
-            $this->requires_php = $headers['requires_php'];
+            $this->requires_php = $this->sanitize_requires_php( $headers['requires_php'] );
         }
         if ( ! empty( $headers['contributors'] ) ) {
@@ -551 +551 @@
             if ( ! $user ) {
                 unset( $users[ $i ] );
-                $this->contributor_ignored = true;
+                $this->warnings['contributor_ignored'] = true;
                 continue;
             }
@@ -580 +580 @@
 
         return $stable_tag;
+    }
+
+    /**
+     * Sanitizes the Requires PHP header to ensure that it's a valid version header.
+     *
+     * @param string $version
+     * @return string The sanitized $version
+     */
+    protected function sanitize_requires_php( $version ) {
+        $version = trim( $version );
+
+        // x.y or x.y.z
+        if ( $version && ! preg_match( '!^\d+(\.\d+){1,2}$!', $version ) ) {
+            $this->warnings['requires_php_ignored'] = true;
+            // Ignore the readme value.
+            $version = '';
+        }
+
+        return $version;
     }