MDL-39087 Offer deleting for standard plugins if possible, too
authorDavid Mudrák <david@moodle.com>
Thu, 11 Apr 2013 12:39:51 +0000 (14:39 +0200)
committerDavid Mudrák <david@moodle.com>
Thu, 11 Apr 2013 23:42:58 +0000 (01:42 +0200)
The plugin_manager::is_plugin_folder_removable() method should do just
one thing and do it well. Also, as was raised during the peer-review,
there should not be technical differences between standard plugins and
add-ons.

lib/pluginlib.php

index e92f236..716a310 100644 (file)
@@ -465,9 +465,6 @@ class plugin_manager {
     /**
      * Check to see if the given plugin folder can be removed by the web server process.
      *
-     * This is intended to be used for installed add-ons mainly. For standard plugins,
-     * false is always returned for now.
-     *
      * @param string $component full frankenstyle component
      * @return bool
      */
@@ -479,10 +476,6 @@ class plugin_manager {
             return false;
         }
 
-        if ($pluginfo->is_standard()) {
-            return false;
-        }
-
         // To be able to remove the plugin folder, its parent must be writable, too.
         if (!is_writable(dirname($pluginfo->rootdir))) {
             return false;