[-] BO: fix $module_name not defined when you reach the catch in impo…

…rtModuleAction
spyroslangkos · Mar 7, 2016 · 488c594 · 488c594
1 parent 93870a1
commit 488c594
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/PrestaShopBundle/Controller/Admin/ModuleController.php b/src/PrestaShopBundle/Controller/Admin/ModuleController.php
@@ -231,8 +231,10 @@ public function importModuleAction(Request $request)
                 array( 'Content-Type' => 'application/json' )
             );
         } catch (Exception $e) {
-            $modulesProvider = $this->container->get('prestashop.core.admin.data_provider.module_interface');
-            $modulesProvider->removeModuleFromDisk($module_name);
+            if (isset($module_name)) {
+                $modulesProvider = $this->container->get('prestashop.core.admin.data_provider.module_interface');
+                $modulesProvider->removeModuleFromDisk($module_name);
+            }
             return new JsonResponse(array(
                 'status' => false,
                 'msg' => $e->getMessage()),