Fix DELETE /api/extensions/* returning 500 (#1580)

* Use extension string as parameter for ::disable & ::uninstall * Remove repeated 'ExtensionManager::disable' call * Fix StyleCI
flarum · Dec 12, 2018 · c761802 · c761802
1 parent 16eb1fa
commit c761802
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/Api/Controller/UninstallExtensionController.php b/src/Api/Controller/UninstallExtensionController.php
@@ -38,9 +38,10 @@ protected function delete(ServerRequestInterface $request)
 
         $name = array_get($request->getQueryParams(), 'name');
 
-        $extension = $this->extensions->getExtension($name);
+        if ($this->extensions->getExtension($name) == null) {
+            return;
+        }
 
-        $this->extensions->disable($extension);
-        $this->extensions->uninstall($extension);
+        $this->extensions->uninstall($name);
     }
 }