Merge pull request #5144 from prestamodule/patch-45

[-] BO: fix $module_name not defined in import module
PrestaShop · Mar 8, 2016 · dc54666 · dc54666
2 parents 0d74e0c + 488c594
commit dc54666
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()),