cleanup

Author: deer-wmde

app/Http/Controllers/Backend/WikiController.php

@@ -7,28 +7,23 @@
 use Illuminate\Http\Request;
 
 class WikiController extends Controller {
-    private static $with = ['wikiDb', 'wikiQueryserviceNamespace', 'settings'];
-
     public function getWikiForDomain(Request $request): \Illuminate\Http\JsonResponse {
-        $domain = $request->input('domain');
+        $validated = $request->validate([
+            'domain' => 'required|string',
+        ]);
 
+        $domain = $validated['domain'];
         // XXX: this same logic is in quickstatements.php and platform api WikiController backend
         try {
-            if ($domain === 'localhost' || $domain === 'mediawiki') {
-                // If just using localhost then just get the first undeleted wiki
-                $result = Wiki::with(self::$with)->first();
-            } else {
-                // TODO don't select the timestamps and redundant info for the settings?
-                $result = Wiki::where('domain', $domain)->with(self::$with)->first();
-            }
-        } catch (\Exception $ex) {
-            return response()->json($ex->getMessage(), 500);
+            $wiki = Wiki::with(['wikiDb', 'wikiQueryserviceNamespace', 'settings'])->firstWhere('domain', $domain);
+        } catch (\Exception $e) {
+            return response()->json($e->getMessage(), 500);
         }
 
-        if (!$result) {
+        if (!$wiki) {
             return response()->json(['error' => 'Not found'], 404);
         }
 
-        return response()->json(['data' => $result], 200);
+        return response()->json(['data' => $wiki], 200);
     }
 }