Changeset 8495

Timestamp:

07/29/2008 11:10:12 PM (17 years ago)

Author:

azaozz

Message:

Include only valid plugins. Props guillep2k, fixes #6871

Location:

trunk

Files:

• wp-admin/includes/plugin.php (modified) (2 diffs)
• wp-admin/plugins.php (modified) (1 diff)
• wp-settings.php (modified) (1 diff)

trunk/wp-admin/includes/plugin.php

@@ -305 +305 @@
     }
 
+    //Invalid is any plugin that is deactivated due to error.
+    $invalid = array(); 
+
     // If a plugin file does not exist, remove it from the list of active
     // plugins.
@@ -310 +313 @@
         $result = validate_plugin($check_plugin);
         if ( is_wp_error( $result ) ) {
+            $invalid[$check_plugin] = $result;
             deactivate_plugins( $check_plugin, true);
         }
     }
+    return $invalid;
 }
 

trunk/wp-admin/plugins.php

@@ -166 +166 @@
 require_once('admin-header.php');
 
-validate_active_plugins();
-
+$invalid = validate_active_plugins();
+if( !empty($invalid) )
+    foreach($invalid as $plugin_file => $error)
+        echo '<div id="message" class="error"><p>' . sprintf(__('The plugin <code>%s</code> has been <strong>deactivated</strong> due to <em>"%s"</em>.'), $plugin_file, $error->get_error_message()) . '</p></div>';
 ?>
 

trunk/wp-settings.php

@@ -422 +422 @@
     if ( is_array($current_plugins) ) {
         foreach ($current_plugins as $plugin) {
-            if ('' != $plugin && file_exists(WP_PLUGIN_DIR . '/' . $plugin))
+            if ( '' != $plugin && ! validate_file($plugin) && file_exists(WP_PLUGIN_DIR . '/' . $plugin) )
                 include_once(WP_PLUGIN_DIR . '/' . $plugin);
         }