Remove reading the object from the try/catch block. Leave only readin…

…g the query parameters in it.
WoltLab · Nov 26, 2024 · 4e03021 · 4e03021
1 parent 1a410bd
commit 4e03021
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/wcfsetup/install/files/lib/acp/form/CaptchaQuestionEditForm.class.php b/wcfsetup/install/files/lib/acp/form/CaptchaQuestionEditForm.class.php
@@ -40,13 +40,14 @@ public function readParameters()
                     }
                     EOT
             );
-            $this->formObject = new CaptchaQuestion($queryParameters['id']);
-
-            if (!$this->formObject->getObjectID()) {
-                throw new IllegalLinkException();
-            }
         } catch (MappingError) {
             throw new IllegalLinkException();
         }
+
+        $this->formObject = new CaptchaQuestion($queryParameters['id']);
+
+        if (!$this->formObject->getObjectID()) {
+            throw new IllegalLinkException();
+        }
     }
 }