Add explicit `return null;` statements
authorMatthias Schmidt <gravatronics@live.com>
Mon, 10 May 2021 14:07:35 +0000 (16:07 +0200)
committerMatthias Schmidt <gravatronics@live.com>
Mon, 10 May 2021 14:07:35 +0000 (16:07 +0200)
28 files changed:
wcfsetup/install/files/lib/data/DatabaseObjectEditor.class.php
wcfsetup/install/files/lib/data/DatabaseObjectList.class.php
wcfsetup/install/files/lib/data/article/content/ViewableArticleContent.class.php
wcfsetup/install/files/lib/data/bbcode/media/provider/BBCodeMediaProvider.class.php
wcfsetup/install/files/lib/data/box/Box.class.php
wcfsetup/install/files/lib/data/comment/response/StructuredCommentResponse.class.php
wcfsetup/install/files/lib/data/devtools/missing/language/item/DevtoolsMissingLanguageItem.class.php
wcfsetup/install/files/lib/data/package/PackageCache.class.php
wcfsetup/install/files/lib/data/smiley/category/SmileyCategory.class.php
wcfsetup/install/files/lib/data/user/User.class.php
wcfsetup/install/files/lib/data/user/object/watch/UserObjectWatch.class.php
wcfsetup/install/files/lib/data/user/online/UserOnline.class.php
wcfsetup/install/files/lib/data/user/option/category/UserOptionCategory.class.php
wcfsetup/install/files/lib/system/WCF.class.php
wcfsetup/install/files/lib/system/clipboard/ClipboardHandler.class.php
wcfsetup/install/files/lib/system/devtools/DevtoolsSetup.class.php
wcfsetup/install/files/lib/system/form/builder/field/DateFormField.class.php
wcfsetup/install/files/lib/system/importer/ImportHandler.class.php
wcfsetup/install/files/lib/system/language/LanguageFactory.class.php
wcfsetup/install/files/lib/system/menu/user/profile/UserProfileMenu.class.php
wcfsetup/install/files/lib/system/package/PackageInstallationDispatcher.class.php
wcfsetup/install/files/lib/system/package/PackageInstallationNodeBuilder.class.php
wcfsetup/install/files/lib/system/package/PackageInstallationSQLParser.class.php
wcfsetup/install/files/lib/system/request/ControllerMap.class.php
wcfsetup/install/files/lib/system/tagging/TagEngine.class.php
wcfsetup/install/files/lib/system/template/TemplateEngine.class.php
wcfsetup/install/files/lib/system/user/notification/UserNotificationHandler.class.php
wcfsetup/install/files/lib/util/DOMUtil.class.php

index c4b209aef0e5c76cca3042380e75ac41db9384a8..80c86ab036d582f2c88984e49264bc8a190b0b96 100644 (file)
@@ -132,7 +132,7 @@ abstract class DatabaseObjectEditor extends DatabaseObjectDecorator implements I
         } catch (DatabaseQueryExecutionException $e) {
             // Error code 23000 = duplicate key
             if ($e->getCode() == '23000' && $e->getDriverCode() == '1062') {
-                return;
+                return null;
             }
 
             throw $e;
index e8359db1d8be2401601fbcdb1277c54324576c9e..4762ea07c6e3d5a9f334d951bf07200dcac70fae 100644 (file)
@@ -410,7 +410,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
         }
 
         if (empty($this->objects)) {
-            return;
+            return null;
         }
 
         return \reset($this->objects);
index 076edbf7eac8c1a6ef4668661a7263d0607cbc2b..2f2961ba33bb6eeee845780033db0f3f17ede4dc 100644 (file)
@@ -81,7 +81,7 @@ class ViewableArticleContent extends DatabaseObjectDecorator
         }
 
         if ($this->image === null || !$this->image->isAccessible()) {
-            return;
+            return null;
         }
 
         return $this->image;
@@ -113,7 +113,7 @@ class ViewableArticleContent extends DatabaseObjectDecorator
         }
 
         if ($this->teaserImage === null || !$this->teaserImage->isAccessible()) {
-            return;
+            return null;
         }
 
         return $this->teaserImage;
index 7f376e4e3dcebfe55df778e140ceb9ba145a949b..90231203192eb6ecce1c5cb76861cb6929c59d5e 100644 (file)
@@ -136,12 +136,12 @@ class BBCodeMediaProvider extends DatabaseObject implements IRouteController
     /**
      * Returns media provider callback instance.
      *
-     * @return      IBBCodeMediaProvider
+     * @return      IBBCodeMediaProvider|null
      */
     public function getCallback()
     {
         if (!$this->className) {
-            return;
+            return null;
         }
 
         if ($this->callback === null) {
index 25877a1909455f07ae021ba0be693cfd054789c4..2e00f9522c4150a2bf6a34d87b009c081d618432 100644 (file)
@@ -386,7 +386,7 @@ class Box extends DatabaseObject
     public function getImage()
     {
         if ($this->boxType === 'menu') {
-            return;
+            return null;
         }
 
         if ($this->image === null) {
@@ -405,7 +405,7 @@ class Box extends DatabaseObject
         }
 
         if ($this->image === null || !$this->image->isAccessible()) {
-            return;
+            return null;
         }
 
         return $this->image;
index 4639dd1d50db4a27b0db84fdd9863e5f8a5e84ea..249091c8f68e357170c039dce8bd7edd94da9627 100644 (file)
@@ -81,7 +81,7 @@ class StructuredCommentResponse extends DatabaseObjectDecorator
     {
         $response = new CommentResponse($responseID);
         if (!$response->responseID) {
-            return;
+            return null;
         }
 
         // prepare structured response
index a9467a6f9883934188786f56cf4791e61b900364..07f9da7cfc88c4237324e5ca6fa61077cb7a2325 100644 (file)
@@ -34,7 +34,7 @@ class DevtoolsMissingLanguageItem extends DatabaseObject
     public function getLanguage()
     {
         if ($this->languageID === null) {
-            return;
+            return null;
         }
 
         return LanguageFactory::getInstance()->getLanguage($this->languageID);
index b2652c877bf722f5a3e8116379fa3c5c3b53b2fb..3ac224b65bd5e0dbd24507b85e539975ee0f1ea8 100644 (file)
@@ -71,7 +71,7 @@ class PackageCache extends SingletonFactory
     {
         $packageID = $this->getPackageID($package);
         if ($packageID === null) {
-            return;
+            return null;
         }
 
         return $this->getPackage($packageID);
index ec4e85dec517fddd0d0fa65408c6b5a60fb66f10..9aa89c22c61f7a676c211d631d6e89d05a475ed5 100644 (file)
@@ -143,7 +143,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
 
             return $this->current();
         } catch (SystemException $e) {
-            return;
+            return null;
         }
     }
 
index 315298427814d5013c373121b57b951d13f2d675..d890a73a7580f6448f699bfeb4a00ed94098cc8b 100644 (file)
@@ -321,7 +321,7 @@ final class User extends DatabaseObject implements IPopoverObject, IRouteControl
      * Returns the id of a user option.
      *
      * @param string $name
-     * @return  int     id
+     * @return  int|null
      */
     public static function getUserOptionID($name)
     {
@@ -331,7 +331,7 @@ final class User extends DatabaseObject implements IPopoverObject, IRouteControl
         }
 
         if (!isset(self::$userOptions[$name])) {
-            return;
+            return null;
         }
 
         return self::$userOptions[$name]->optionID;
index cb1815d9cb229a81b1353c39fe31380032a91bd8..2239ec8c2e0e03500b4a49d659a6b0242ea6e343 100644 (file)
@@ -28,7 +28,7 @@ class UserObjectWatch extends DatabaseObject
      * @param int $objectTypeID
      * @param int $userID
      * @param int $objectID
-     * @return  UserObjectWatch
+     * @return  UserObjectWatch|null
      */
     public static function getUserObjectWatch($objectTypeID, $userID, $objectID)
     {
@@ -41,7 +41,7 @@ class UserObjectWatch extends DatabaseObject
         $statement->execute([$objectTypeID, $userID, $objectID]);
         $row = $statement->fetch();
         if (!$row) {
-            return;
+            return null;
         }
 
         return new self(null, $row);
index 8ddb434c247243fd3cbbb123c91265da79584510..877b6a847295111b4fa3529d6fc45bd59da39c1b 100644 (file)
@@ -154,12 +154,12 @@ class UserOnline extends UserProfile
     /**
      * Returns the spider object
      *
-     * @return  Spider
+     * @return  Spider|null
      */
     public function getSpider()
     {
         if (!$this->spiderID) {
-            return;
+            return null;
         }
 
         if ($this->spider === null) {
index a5ca6c66dca740b3260c6b64385d5d1dedb9112f..b559012ba5c317d365aa629853f081a456bb1875 100644 (file)
@@ -46,7 +46,7 @@ class UserOptionCategory extends DatabaseObject implements ITitledObject
      * Returns an instance of UserOptionCategory by name.
      *
      * @param string $categoryName
-     * @return  UserOptionCategory
+     * @return  UserOptionCategory|null
      */
     public static function getCategoryByName($categoryName)
     {
@@ -57,7 +57,7 @@ class UserOptionCategory extends DatabaseObject implements ITitledObject
         $statement->execute([$categoryName]);
         $row = $statement->fetchArray();
         if ($row === false) {
-            return;
+            return null;
         }
 
         return new self(null, $row);
index c8bbf4a9c6e8889adaba82e2e6d984af27758818..3e249e2f3a55def6f7c5309c3ad8efba7b90ac2d 100644 (file)
@@ -1045,7 +1045,7 @@ class WCF
     public static function getActivePage()
     {
         if (self::getActiveRequest() === null) {
-            return;
+            return null;
         }
 
         if (self::getActiveRequest()->getClassName() === CmsPage::class) {
@@ -1054,7 +1054,7 @@ class WCF
                 return PageCache::getInstance()->getPage($metaData['cms']['pageID']);
             }
 
-            return;
+            return null;
         }
 
         return PageCache::getInstance()->getPageByController(self::getActiveRequest()->getClassName());
index d6485d62e81e55a8392a41b5006950eca856cd7a..f3f8e50852a76d64ba57a12b47e4759629a388c2 100644 (file)
@@ -340,7 +340,7 @@ class ClipboardHandler extends SingletonFactory
         // get objects
         $this->loadMarkedItems();
         if (empty($this->markedItems)) {
-            return;
+            return null;
         }
 
         $this->pageClasses = $pages;
index fb946929af1fc1bca181243ababbc704e908d482..c7ac51af62608c833ac21a8613d686954b3adcc4 100644 (file)
@@ -58,7 +58,7 @@ class DevtoolsSetup extends SingletonFactory
     public function getDatabaseConfig()
     {
         if (!isset($this->configuration['setup']) || !isset($this->configuration['setup']['database'])) {
-            return;
+            return null;
         }
 
         // dirname return a single backslash on Windows if there are no parent directories
index d1ddc58c1c9c5e8ddb4f7f57b212b666d2e75b54..02303db33c1d6a3be67f959dae0ade3413b06316 100644 (file)
@@ -215,7 +215,7 @@ class DateFormField extends AbstractFormField implements
         }
 
         if ($dateTime === false) {
-            return;
+            return null;
         }
 
         return $dateTime;
index ae6aac8b2c563c29c4db9713daa2508814d2490f..7ee4604d48f90b579379d3c68920f6b8c5daa283 100644 (file)
@@ -90,12 +90,12 @@ class ImportHandler extends SingletonFactory implements IAJAXInvokeAction
      *
      * @param string $type
      * @param mixed $oldID
-     * @return  int     $newID
+     * @return  int|null
      */
     public function getNewID($type, $oldID)
     {
         if (!$oldID) {
-            return;
+            return null;
         }
         $objectTypeID = $this->objectTypes[$type]->objectTypeID;
 
index e9613aaabf38170ea88f1e14c6a5401da60c13b5..5f382bacdbdbdea8e139380a5bce538c062b0209 100644 (file)
@@ -50,13 +50,13 @@ class LanguageFactory extends SingletonFactory
      * Returns a Language object for the language with the given id.
      *
      * @param int $languageID
-     * @return  Language
+     * @return  Language|null
      */
     public function getLanguage($languageID)
     {
         if (!isset($this->languages[$languageID])) {
             if (!isset($this->cache['languages'][$languageID])) {
-                return;
+                return null;
             }
 
             $this->languages[$languageID] = $this->cache['languages'][$languageID];
index cef1acb074ae7fea06a3d10bbdf71ca631fbbc18..b85086104cd613d9263715b7b35ec621f856f8f9 100644 (file)
@@ -112,12 +112,12 @@ class UserProfileMenu extends SingletonFactory
      * Returns the first visible menu item.
      *
      * @param int $userID
-     * @return  UserProfileMenuItem
+     * @return  UserProfileMenuItem|null
      */
     public function getActiveMenuItem($userID = 0)
     {
         if (empty($this->menuItems)) {
-            return;
+            return null;
         }
 
         if ($this->activeMenuItem === null) {
index edb123972f7de14d59e2788cf0713e5c9d3ae752..ee98ad09be42fdd1ef27de02a9ab36700c4efc10 100644 (file)
@@ -896,7 +896,7 @@ class PackageInstallationDispatcher
      * Prompts for a text input for package directory (applies for applications only)
      *
      * @param string $applicationDirectory
-     * @return  FormDocument
+     * @return  FormDocument|null
      */
     protected function promptPackageDir($applicationDirectory)
     {
@@ -1031,7 +1031,7 @@ class PackageInstallationDispatcher
                 FileUtil::makeWritable($packageDir);
             }
 
-            return;
+            return null;
         }
     }
 
index 833326405bc37fc2a55aa58aecd1f87386ca200f..91a9f306334fdf26a24efb5b7172b7021138906a 100644 (file)
@@ -813,7 +813,7 @@ class PackageInstallationNodeBuilder
         if ($row === false) {
             // PHP <7.4 _silently_ returns `null` when attempting to read an array index
             // when the source value equals `false`.
-            return;
+            return null;
         }
 
         return $row['queueID'];
index 1d53f96ebfd119c63c88aa1da4eab29a96d54962..469aa244033af47303f33a1ee6cbdc67d10bb7cc 100644 (file)
@@ -214,7 +214,7 @@ class PackageInstallationSQLParser extends SQLParser
      *
      * @param string $tableName
      * @param string $columnName
-     * @return  int     package id
+     * @return  int|null     package id
      */
     protected function getColumnOwnerID($tableName, $columnName)
     {
@@ -233,7 +233,7 @@ class PackageInstallationSQLParser extends SQLParser
         } elseif (isset($this->knownTables[$tableName])) {
             return $this->knownTables[$tableName];
         } else {
-            return;
+            return null;
         }
     }
 
index 8c87c36cc3f202a6561da09fedf9533c3b33f7b3..d2284c1e886f1716809edd07111853394e43bd88 100644 (file)
@@ -265,7 +265,7 @@ class ControllerMap extends SingletonFactory
         $controller = $data[1];
 
         if ($application === 'wcf' && empty($controller)) {
-            return;
+            return null;
         } elseif (\preg_match('~^__WCF_CMS__(?P<pageID>\d+)$~', $controller, $matches)) {
             $cmsPageData = $this->lookupCmsPage($matches['pageID'], 0);
             if ($cmsPageData === null) {
@@ -442,17 +442,17 @@ class ControllerMap extends SingletonFactory
             if ($pageType === 'action' && $application !== 'wcf') {
                 $className = 'wcf\\' . ($isAcpRequest ? 'acp\\' : '') . $pageType . '\\' . $controller . \ucfirst($pageType);
                 if (!\class_exists($className)) {
-                    return;
+                    return null;
                 }
             } else {
-                return;
+                return null;
             }
         }
 
         // check for abstract classes
         $reflectionClass = new \ReflectionClass($className);
         if ($reflectionClass->isAbstract()) {
-            return;
+            return null;
         }
 
         return [
index 1c2d04d89e75b72c79d3b9814fa4995b6467c997..e43de5a70cf882b46e4965c966edbb476106348b 100644 (file)
@@ -257,7 +257,7 @@ class TagEngine extends SingletonFactory
     {
         $existingTags = $this->getObjectTags($objectType, $objectID);
         if (empty($existingTags)) {
-            return;
+            return null;
         }
 
         $languageIDs = [];
index a910ef33e7835f4840dcabd5219309044b1ea868..cbf84b053771d61bd78c460a820b48b50227bc3e 100755 (executable)
@@ -801,12 +801,12 @@ class TemplateEngine extends SingletonFactory
      *
      * @param string $templateName
      * @param string $filename
-     * @return  array
+     * @return  array|null
      */
     protected function getMetaData($templateName, $filename)
     {
         if (!\file_exists($filename) || !\is_readable($filename)) {
-            return;
+            return null;
         }
 
         // get file contents
@@ -815,7 +815,7 @@ class TemplateEngine extends SingletonFactory
         // find first newline
         $position = \strpos($contents, "\n");
         if ($position === false) {
-            return;
+            return null;
         }
 
         // cut contents
@@ -824,7 +824,7 @@ class TemplateEngine extends SingletonFactory
         // read serializes data
         $data = @\unserialize($contents);
         if ($data === false || !\is_array($data)) {
-            return;
+            return null;
         }
 
         return $data;
index 48154678921a26ef3862fcfadc28687b2d2b4efe..9213f606bbd823b808f4c249fde65b652b859f62 100644 (file)
@@ -596,12 +596,12 @@ class UserNotificationHandler extends SingletonFactory
      *
      * @param string $objectType
      * @param string $eventName
-     * @return  IUserNotificationEvent
+     * @return  IUserNotificationEvent|null
      */
     public function getEvent($objectType, $eventName)
     {
         if (!isset($this->availableEvents[$objectType][$eventName])) {
-            return;
+            return null;
         }
 
         return $this->availableEvents[$objectType][$eventName];
index 13f2fa88041b0d85c8373c08e81da6fbd017119a..5dd8dc31d26f5d59f0c7d060e9d90e127c34486f 100644 (file)
@@ -139,7 +139,7 @@ final class DOMUtil
     public static function getParentBefore(\DOMNode $node, \DOMElement $ancestor)
     {
         if ($node->parentNode === $ancestor) {
-            return;
+            return null;
         }
 
         $parents = self::getParents($node);