Ticket #8964: 8964.diff

wp-admin/includes/plugin.php

@@ -75 +75 @@
         // PHP will close file handle, but we are good citizens.
         fclose($fp);
 
-        preg_match( '|Plugin Name:(.*)$|mi', $plugin_data, $name );
-        preg_match( '|Plugin URI:(.*)$|mi', $plugin_data, $uri );
-        preg_match( '|Version:(.*)|i', $plugin_data, $version );
-        preg_match( '|Description:(.*)$|mi', $plugin_data, $description );
-        preg_match( '|Author:(.*)$|mi', $plugin_data, $author_name );
-        preg_match( '|Author URI:(.*)$|mi', $plugin_data, $author_uri );
-        preg_match( '|Text Domain:(.*)$|mi', $plugin_data, $text_domain );
-        preg_match( '|Domain Path:(.*)$|mi', $plugin_data, $domain_path );
-
-        foreach ( array( 'name', 'uri', 'version', 'description', 'author_name', 'author_uri', 'text_domain', 'domain_path' ) as $field ) {
+        $default_headers = array( 
+                'Name'=>'Plugin Name', 
+                'PluginURI'=>'Plugin URI', 
+                'Version'=>'Version', 
+                'Description'=>'Description', 
+                'Author'=>'Author', 
+                'AuthorURI'=>'Author URI', 
+                'TextDomain'=>'Text Domain', 
+                'DomainPath'=>'Domain Path' 
+                );
+        $extra_headers = apply_filters( 'plugin_headers', array());
+        $all_headers = array_merge($extra_headers, $default_headers);
+        
+        foreach ( $all_headers as $field => $regex ) {
+                preg_match('/' . preg_quote($regex, '/') . ':(.*)$/mi', $plugin_data, ${$field});
+                
                 if ( !empty( ${$field} ) )
                         ${$field} = _cleanup_header_comment(${$field}[1]);
                 else
                         ${$field} = '';
         }
 
-        $plugin_data = array(
-                                'Name' => $name, 'Title' => $name, 'PluginURI' => $uri, 'Description' => $description,
-                                'Author' => $author_name, 'AuthorURI' => $author_uri, 'Version' => $version,
-                                'TextDomain' => $text_domain, 'DomainPath' => $domain_path
-                                );
+        $plugin_data = compact(array_keys($all_headers));
+        
         if ( $markup || $translate )
                 $plugin_data = _get_plugin_data_markup_translate($plugin_file, $plugin_data, $markup, $translate);