Changeset 1842 for branches/0.9/bb-admin/admin-functions.php

Timestamp:

12/10/2008 07:10:24 AM (18 years ago)

Author:

sambauers

Message:

Only grab the first comment block when reading data from plugins and themes - speeds things up and avoids rogue plugins turning up in the plugins list. branches/0.9

File:

• branches/0.9/bb-admin/admin-functions.php (modified) (2 diffs)

branches/0.9/bb-admin/admin-functions.php

@@ -848 +848 @@
                 );
         }
-        $plugin_data = implode('', file($plugin_file));
+        $plugin_code = implode('', file($plugin_file));
+        // Grab just the first commented area from the file
+        if ( !preg_match( '|/\*(.*)\*/|msU', $plugin_code, $plugin_block ) )
+                return false;
+        $plugin_data = trim( $plugin_block[1] );
         if ( !preg_match("|Plugin Name:(.*)|i", $plugin_data, $plugin_name) )
                 return false;
@@ -917 +921 @@
         if ( strpos($theme_file, '#') !== false )
                 $theme_file = bb_get_theme_directory( $theme_file ) . 'style.css';
-        $theme_data = implode( '', file( $theme_file ) );
-        $theme_data = str_replace ( '\r', '\n', $theme_data ); 
+        $theme_code = implode( '', file( $theme_file ) );
+        $theme_code = str_replace ( '\r', '\n', $theme_code );
+        // Grab just the first commented area from the file
+        preg_match( '|/\*(.*)\*/|msU', $theme_code, $theme_block );
+        $theme_data = trim( $theme_block[1] );
         preg_match( '|Theme Name:(.*)|i', $theme_data, $theme_name );
         preg_match( '|Theme URI:(.*)|i', $theme_data, $theme_uri );