Improve type of exception for invalid object types in AbstractCategoryAddForm
authorTim Düsterhus <duesterhus@woltlab.com>
Tue, 5 Jul 2022 09:02:19 +0000 (11:02 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Tue, 5 Jul 2022 09:03:09 +0000 (11:03 +0200)
This technically is a BC break, but this exception must not be caught anyway as
it indicates a clear programming error.

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

index ea9178b474300c96fa7c7a73a21720bc33bb5984..effa7cce1d31c05a64829ae0c2f1a70a79f33598 100644 (file)
@@ -11,8 +11,8 @@ use wcf\system\acl\ACLHandler;
 use wcf\system\category\CategoryHandler;
 use wcf\system\category\CategoryPermissionHandler;
 use wcf\system\category\ICategoryType;
+use wcf\system\exception\InvalidObjectTypeException;
 use wcf\system\exception\PermissionDeniedException;
-use wcf\system\exception\SystemException;
 use wcf\system\exception\UserInputException;
 use wcf\system\language\I18nHandler;
 use wcf\system\WCF;
@@ -196,7 +196,7 @@ abstract class AbstractCategoryAddForm extends AbstractForm
     {
         $this->objectType = CategoryHandler::getInstance()->getObjectTypeByName($this->objectTypeName);
         if ($this->objectType === null) {
-            throw new SystemException("Unknown category object type with name '" . $this->objectTypeName . "'");
+            throw new InvalidObjectTypeException($this->objectTypeName, 'com.woltlab.wcf.category');
         }
 
         // check permissions