From badf464792bedaed08e39a66464a1b4b22607c82 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?David=20Mudr=C3=A1k?= Date: Fri, 19 Apr 2013 11:24:43 +0200 Subject: [PATCH] MDL-39148 Non-installed plugins cannot be uninstalled If the plugin has been only deployed to the disk without installing into the database, do not allow going through the uninstallation procedure. Not only it does not have much sense. But it can also lead to some tricky situation due to dependencies. Better to block it and wait till the plugin is either fully installed or removed from the disk. --- lib/pluginlib.php | 6 ++++++ lib/tests/fixtures/mockplugins/mod/new/version.php | 5 +++++ lib/tests/pluginlib_test.php | 14 ++++++++++++-- 3 files changed, 23 insertions(+), 2 deletions(-) create mode 100644 lib/tests/fixtures/mockplugins/mod/new/version.php diff --git a/lib/pluginlib.php b/lib/pluginlib.php index c426414a2b7..5742ca20c4b 100644 --- a/lib/pluginlib.php +++ b/lib/pluginlib.php @@ -948,6 +948,12 @@ class plugin_manager { return false; } + if ($pluginfo->get_status() === plugin_manager::PLUGIN_STATUS_NEW) { + // The plugin is not installed. It should be either installed or removed from the disk. + // Relying on this temporary state may be tricky. + return false; + } + if (is_null($pluginfo->get_uninstall_url())) { // Backwards compatibility. debugging('plugininfo_base subclasses should use is_uninstall_allowed() instead of returning null in get_uninstall_url()', diff --git a/lib/tests/fixtures/mockplugins/mod/new/version.php b/lib/tests/fixtures/mockplugins/mod/new/version.php new file mode 100644 index 00000000000..976ddf1204b --- /dev/null +++ b/lib/tests/fixtures/mockplugins/mod/new/version.php @@ -0,0 +1,5 @@ +version = 2013041900; +$module->requires = 2012010100; +$module->component = 'mod_new'; diff --git a/lib/tests/pluginlib_test.php b/lib/tests/pluginlib_test.php index 06abf584075..793ee9b1ae7 100644 --- a/lib/tests/pluginlib_test.php +++ b/lib/tests/pluginlib_test.php @@ -53,11 +53,12 @@ class plugin_manager_test extends advanced_testcase { $pluginman = testable_plugin_manager::instance(); $mods = $pluginman->get_plugins_of_type('mod'); $this->assertEquals('array', gettype($mods)); - $this->assertEquals(4, count($mods)); + $this->assertEquals(5, count($mods)); $this->assertTrue($mods['foo'] instanceof testable_plugininfo_mod); $this->assertTrue($mods['bar'] instanceof testable_plugininfo_mod); $this->assertTrue($mods['baz'] instanceof testable_plugininfo_mod); $this->assertTrue($mods['qux'] instanceof testable_plugininfo_mod); + $this->assertTrue($mods['new'] instanceof testable_plugininfo_mod); $foolishes = $pluginman->get_plugins_of_type('foolish'); $this->assertEquals(2, count($foolishes)); $this->assertTrue($foolishes['frog'] instanceof testable_pluginfo_foolish); @@ -79,11 +80,13 @@ class plugin_manager_test extends advanced_testcase { $this->assertTrue(isset($plugins['mod']['foo'])); $this->assertTrue(isset($plugins['mod']['bar'])); $this->assertTrue(isset($plugins['mod']['baz'])); + $this->assertTrue(isset($plugins['mod']['new'])); $this->assertTrue(isset($plugins['foolish']['frog'])); $this->assertTrue(isset($plugins['foolish']['hippo'])); $this->assertTrue($plugins['mod']['foo'] instanceof testable_plugininfo_mod); $this->assertTrue($plugins['mod']['bar'] instanceof testable_plugininfo_mod); $this->assertTrue($plugins['mod']['baz'] instanceof testable_plugininfo_mod); + $this->assertTrue($plugins['mod']['new'] instanceof testable_plugininfo_mod); $this->assertTrue($plugins['foolish']['frog'] instanceof testable_pluginfo_foolish); $this->assertTrue($plugins['foolish']['hippo'] instanceof testable_pluginfo_foolish); $this->assertTrue($plugins['bazmeg']['one'] instanceof testable_pluginfo_bazmeg); @@ -217,6 +220,7 @@ class plugin_manager_test extends advanced_testcase { $pluginman = testable_plugin_manager::instance(); $plugins = $pluginman->get_plugins(); $this->assertEquals(plugin_manager::PLUGIN_STATUS_UPGRADE, $plugins['mod']['foo']->get_status()); + $this->assertEquals(plugin_manager::PLUGIN_STATUS_NEW, $plugins['mod']['new']->get_status()); $this->assertEquals(plugin_manager::PLUGIN_STATUS_NEW, $plugins['bazmeg']['one']->get_status()); $this->assertEquals(plugin_manager::PLUGIN_STATUS_UPTODATE, $plugins['quxcat']['one']->get_status()); } @@ -240,6 +244,7 @@ class plugin_manager_test extends advanced_testcase { $this->assertFalse($pluginman->can_uninstall_plugin('mod_qux')); // Because even if no plugin (not even subplugins) declare // dependency on it, but its subplugin can't be uninstalled. $this->assertFalse($pluginman->can_uninstall_plugin('mod_baz')); // Because it's subplugin bazmeg_one is required by quxcat_one. + $this->assertFalse($pluginman->can_uninstall_plugin('mod_new')); // Because it is not installed. $this->assertFalse($pluginman->can_uninstall_plugin('quxcat_one')); // Because of testable_pluginfo_quxcat::is_uninstall_allowed(). $this->assertFalse($pluginman->can_uninstall_plugin('foolish_frog')); // Because foolish_hippo requires it. } @@ -546,7 +551,9 @@ class testable_plugininfo_mod extends plugininfo_mod { } public function load_db_version() { - $this->versiondb = 2012022900; + if ($this->component !== 'mod_new') { + $this->versiondb = 2012022900; + } } public function is_uninstall_allowed() { @@ -660,6 +667,8 @@ class testable_plugin_manager extends plugin_manager { $dirroot.'/mod/baz', 'testable_plugininfo_mod'), 'qux' => plugininfo_default_factory::make('mod', $dirroot.'/qux', 'qux', $dirroot.'/mod/qux', 'testable_plugininfo_mod'), + 'new' => plugininfo_default_factory::make('mod', $dirroot.'/new', 'new', + $dirroot.'/mod/new', 'testable_plugininfo_mod'), ), 'foolish' => array( 'frog' => plugininfo_default_factory::make('foolish', $dirroot.'/mod/foo/lish', 'frog', @@ -681,6 +690,7 @@ class testable_plugin_manager extends plugin_manager { $this->pluginsinfo['mod']['foo']->check_available_updates($checker); $this->pluginsinfo['mod']['bar']->check_available_updates($checker); $this->pluginsinfo['mod']['baz']->check_available_updates($checker); + $this->pluginsinfo['mod']['new']->check_available_updates($checker); $this->pluginsinfo['bazmeg']['one']->check_available_updates($checker); $this->pluginsinfo['quxcat']['one']->check_available_updates($checker);