From 26055c26e7bba1f6aa332bed2ecce10452fae5f4 Mon Sep 17 00:00:00 2001 From: vAugagneur Date: Wed, 9 Jan 2013 11:58:48 +0100 Subject: [PATCH] //small fix --- controllers/admin/AdminModulesController.php | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/controllers/admin/AdminModulesController.php b/controllers/admin/AdminModulesController.php index 5e1c141c3..af6cf3348 100644 --- a/controllers/admin/AdminModulesController.php +++ b/controllers/admin/AdminModulesController.php @@ -529,13 +529,16 @@ class AdminModulesControllerCore extends AdminController foreach ($modules as $name) { $full_report = null; - if ($key == 'update' && !ConfigurationTest::test_dir('modules/'.$name, true, $full_report)) + if ($key == 'update') { - $module = Module::getInstanceByName(urldecode($name)); - $this->errors[] = $this->l(sprintf("Module %s can't be upgraded : ", $module->displayName)).$full_report; + if (ConfigurationTest::test_dir('modules/'.$name, true, $full_report)) + Tools::deleteDirectory('../modules/'.$name.'/'); + else + { + $module = Module::getInstanceByName(urldecode($name)); + $this->errors[] = $this->l(sprintf("Module %s can't be upgraded : ", $module->displayName)).$full_report; + } } - else - Tools::deleteDirectory('../modules/'.$name.'/'); // If Addons module, download and unzip it before installing it if (!is_null($full_report) && !is_dir('../modules/'.$name.'/'))