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

Timestamp:

12/10/2008 07:10:24 AM (17 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 );