Remove reading the object from the try/catch block. Leave only reading the query parameters in it.

Author: Cyperghost

wcfsetup/install/files/lib/acp/form/UserOptionEditForm.class.php

@@ -43,14 +43,15 @@ public function readParameters()
                     }
                     EOT
             );
-            $this->formObject = new UserOption($queryParameters['id']);
-
-            if (!$this->formObject->getObjectID()) {
-                throw new IllegalLinkException();
-            }
         } catch (MappingError) {
             throw new IllegalLinkException();
         }
+
+        $this->formObject = new UserOption($queryParameters['id']);
+
+        if (!$this->formObject->getObjectID()) {
+            throw new IllegalLinkException();
+        }
     }
 
     #[\Override]