Add various explicit return types
authorTim Düsterhus <duesterhus@woltlab.com>
Wed, 1 Feb 2023 08:56:28 +0000 (09:56 +0100)
committerTim Düsterhus <duesterhus@woltlab.com>
Wed, 1 Feb 2023 08:56:28 +0000 (09:56 +0100)
see WoltLab/WCF#5157

files/lib/data/conversation/Conversation.class.php
files/lib/data/conversation/FeedConversation.class.php
files/lib/data/conversation/message/ConversationMessage.class.php
files/lib/system/user/notification/event/ConversationMessageUserNotificationEvent.class.php
files/lib/system/user/notification/event/ConversationUserNotificationEvent.class.php
files/lib/system/user/notification/object/ConversationMessageUserNotificationObject.class.php
files/lib/system/user/notification/object/ConversationUserNotificationObject.class.php

index 23389b855ac361d4966996b921ce2d7d4ed614ae..d0c8afc9193aede0bec34ed4f9a54c97045787bd 100644 (file)
@@ -91,7 +91,7 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->subject;
     }
@@ -99,7 +99,7 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getLink('Conversation', [
             'object' => $this,
@@ -109,10 +109,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if this conversation is new for the active user.
-     *
-     * @return  bool
      */
-    public function isNew()
+    public function isNew(): bool
     {
         if (!$this->isDraft && $this->lastPostTime > $this->lastVisitTime) {
             return true;
@@ -123,11 +121,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if the active user doesn't have read the given message.
-     *
-     * @param ConversationMessage $message
-     * @return  bool
      */
-    public function isNewMessage(ConversationMessage $message)
+    public function isNewMessage(ConversationMessage $message): bool
     {
         if (!$this->isDraft && $message->time > $this->lastVisitTime) {
             return true;
@@ -138,10 +133,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if the conversation is not closed or the user was not removed.
-     *
-     * @return      bool
      */
-    public function canReply()
+    public function canReply(): bool
     {
         if (!$this->canRead()) {
             return false;
@@ -242,10 +235,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if the active user has the permission to read this conversation.
-     *
-     * @return  bool
      */
-    public function canRead()
+    public function canRead(): bool
     {
         if (!WCF::getUser()->userID) {
             return false;
@@ -264,10 +255,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if the current user can add new participants to this conversation.
-     *
-     * @return  bool
      */
-    public function canAddParticipants()
+    public function canAddParticipants(): bool
     {
         if ($this->isDraft) {
             return false;
@@ -298,10 +287,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if the current user can add participants without limitations.
-     *
-     * @return      bool
      */
-    public function canAddParticipantsUnrestricted()
+    public function canAddParticipantsUnrestricted(): bool
     {
         if ($this->canAddUnrestricted === null) {
             $this->canAddUnrestricted = false;
@@ -405,10 +392,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns false if the active user is the last participant of this conversation.
-     *
-     * @return  bool
      */
-    public function hasOtherParticipants()
+    public function hasOtherParticipants(): bool
     {
         if ($this->userID == WCF::getUser()->userID) {
             // author
@@ -447,10 +432,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
 
     /**
      * Returns true if the current user is an active participant of this conversation.
-     *
-     * @return      bool
      */
-    public function isActiveParticipant()
+    public function isActiveParticipant(): bool
     {
         if ($this->isActiveParticipant === null) {
             $sql = "SELECT  leftAt
@@ -473,7 +456,7 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
     /**
      * @inheritDoc
      */
-    public function getPopoverLinkClass()
+    public function getPopoverLinkClass(): string
     {
         return 'conversationLink';
     }
@@ -484,9 +467,8 @@ class Conversation extends DatabaseObject implements IPopoverObject, IRouteContr
      *
      * @param int[] $conversationIDs
      * @param int $userID
-     * @return  bool
      */
-    public static function isParticipant(array $conversationIDs, $userID = null)
+    public static function isParticipant(array $conversationIDs, $userID = null): bool
     {
         if ($userID === null) {
             $userID = WCF::getUser()->userID;
index c97ec93ec7002c3da810d1cddc663ebd18907f15..3789a4ec4ebb88abaa47dac13f90d6892df224fa 100644 (file)
@@ -26,7 +26,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getLink('Conversation', [
             'object' => $this->getDecoratedObject(),
@@ -37,7 +37,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->getDecoratedObject()->getTitle();
     }
@@ -45,7 +45,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function getFormattedMessage()
+    public function getFormattedMessage(): string
     {
         return '';
     }
@@ -53,7 +53,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function getMessage()
+    public function getMessage(): string
     {
         return '';
     }
@@ -61,7 +61,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function getExcerpt($maxLength = 255)
+    public function getExcerpt($maxLength = 255): string
     {
         return '';
     }
@@ -77,7 +77,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function getUsername()
+    public function getUsername(): string
     {
         return $this->getDecoratedObject()->lastPoster;
     }
@@ -93,7 +93,7 @@ class FeedConversation extends DatabaseObjectDecorator implements IFeedEntry
     /**
      * @inheritDoc
      */
-    public function __toString()
+    public function __toString(): string
     {
         return $this->getFormattedMessage();
     }
index 12cc57f0974daa122175935c840fd3264e2871c2..4dec86e4159d673979d1b415016a99ade75840f9 100644 (file)
@@ -45,7 +45,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function getFormattedMessage()
+    public function getFormattedMessage(): string
     {
         $processor = new HtmlOutputProcessor();
         $processor->process($this->message, 'com.woltlab.wcf.conversation.message', $this->messageID);
@@ -55,10 +55,8 @@ class ConversationMessage extends DatabaseObject implements IMessage
 
     /**
      * Returns a simplified version of the formatted message.
-     *
-     * @return  string
      */
-    public function getSimplifiedFormattedMessage()
+    public function getSimplifiedFormattedMessage(): string
     {
         $processor = new HtmlOutputProcessor();
         $processor->setOutputType('text/simplified-html');
@@ -97,7 +95,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function getExcerpt($maxLength = 255)
+    public function getExcerpt($maxLength = 255): string
     {
         return StringUtil::truncateHTML($this->getSimplifiedFormattedMessage(), $maxLength);
     }
@@ -106,9 +104,8 @@ class ConversationMessage extends DatabaseObject implements IMessage
      * Returns a version of this message optimized for use in emails.
      *
      * @param string $mimeType Either 'text/plain' or 'text/html'
-     * @return  string
      */
-    public function getMailText($mimeType = 'text/plain')
+    public function getMailText($mimeType = 'text/plain'): string
     {
         switch ($mimeType) {
             case 'text/plain':
@@ -152,10 +149,8 @@ class ConversationMessage extends DatabaseObject implements IMessage
 
     /**
      * Returns true if current user may edit this message.
-     *
-     * @return  bool
      */
-    public function canEdit()
+    public function canEdit(): bool
     {
         return WCF::getUser()->userID == $this->userID
             && (
@@ -168,7 +163,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function getMessage()
+    public function getMessage(): string
     {
         return $this->message;
     }
@@ -176,7 +171,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getLink('Conversation', [
             'object' => $this->getConversation(),
@@ -188,7 +183,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         if ($this->messageID == $this->getConversation()->firstMessageID) {
             return $this->getConversation()->subject;
@@ -200,7 +195,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function isVisible()
+    public function isVisible(): bool
     {
         return true;
     }
@@ -208,7 +203,7 @@ class ConversationMessage extends DatabaseObject implements IMessage
     /**
      * @inheritDoc
      */
-    public function __toString()
+    public function __toString(): string
     {
         return $this->getFormattedMessage();
     }
index 86216017323ce7c9bcd4c42138c3d8c57454a4f0..7ae93b4cd275965d25efeecfcded06001f1be0ac 100644 (file)
@@ -29,7 +29,7 @@ class ConversationMessageUserNotificationEvent extends AbstractUserNotificationE
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         $count = \count($this->getAuthors());
         if ($count > 1) {
@@ -45,7 +45,7 @@ class ConversationMessageUserNotificationEvent extends AbstractUserNotificationE
     /**
      * @inheritDoc
      */
-    public function getMessage()
+    public function getMessage(): string
     {
         $authors = \array_values($this->getAuthors());
         $count = \count($authors);
@@ -93,7 +93,7 @@ class ConversationMessageUserNotificationEvent extends AbstractUserNotificationE
      * @inheritDoc
      * @since   5.2
      */
-    public function getEmailTitle()
+    public function getEmailTitle(): string
     {
         if (\count($this->getAuthors()) > 1) {
             return parent::getEmailTitle();
@@ -109,7 +109,7 @@ class ConversationMessageUserNotificationEvent extends AbstractUserNotificationE
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return $this->getUserNotificationObject()->getLink();
     }
@@ -117,7 +117,7 @@ class ConversationMessageUserNotificationEvent extends AbstractUserNotificationE
     /**
      * @inheritDoc
      */
-    public function getEventHash()
+    public function getEventHash(): string
     {
         return \sha1($this->eventID . '-' . $this->getUserNotificationObject()->conversationID);
     }
@@ -125,7 +125,7 @@ class ConversationMessageUserNotificationEvent extends AbstractUserNotificationE
     /**
      * @inheritDoc
      */
-    public function checkAccess()
+    public function checkAccess(): bool
     {
         return $this->getUserNotificationObject()->getConversation()->canRead();
     }
index 2536e946e538b826d7c0cff3fe54d336d8bdaa83..fb9f8bdf4360db95fb1a9451b96b5a879fb82ff7 100644 (file)
@@ -22,7 +22,7 @@ class ConversationUserNotificationEvent extends AbstractUserNotificationEvent im
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->getLanguage()->get('wcf.user.notification.conversation.title');
     }
@@ -30,7 +30,7 @@ class ConversationUserNotificationEvent extends AbstractUserNotificationEvent im
     /**
      * @inheritDoc
      */
-    public function getMessage()
+    public function getMessage(): string
     {
         return $this->getLanguage()->getDynamicVariable('wcf.user.notification.conversation.message', [
             'author' => $this->author,
@@ -58,7 +58,7 @@ class ConversationUserNotificationEvent extends AbstractUserNotificationEvent im
      * @inheritDoc
      * @since   5.2
      */
-    public function getEmailTitle()
+    public function getEmailTitle(): string
     {
         return $this->getLanguage()->getDynamicVariable('wcf.user.notification.conversation.mail.title', [
             'author' => $this->author,
@@ -69,7 +69,7 @@ class ConversationUserNotificationEvent extends AbstractUserNotificationEvent im
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return $this->getUserNotificationObject()->getLink();
     }
@@ -77,7 +77,7 @@ class ConversationUserNotificationEvent extends AbstractUserNotificationEvent im
     /**
      * @inheritDoc
      */
-    public function checkAccess()
+    public function checkAccess(): bool
     {
         return $this->getUserNotificationObject()->canRead();
     }
index a69c660c00cf5a305d5ed0a3c72decbe1deeb193..3ca8b3613b53107ca0ad7cb3286f1e31fe535776 100644 (file)
@@ -25,7 +25,7 @@ class ConversationMessageUserNotificationObject extends DatabaseObjectDecorator
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->getConversation()->subject;
     }
@@ -33,7 +33,7 @@ class ConversationMessageUserNotificationObject extends DatabaseObjectDecorator
     /**
      * @inheritDoc
      */
-    public function getURL()
+    public function getURL(): string
     {
         return $this->getLink();
     }
index fe6164d211c843893bae179abd87272f0bd6e9b8..5bedbae6d1e0661a6638e28f3bb33a5cab146bc2 100644 (file)
@@ -25,7 +25,7 @@ class ConversationUserNotificationObject extends DatabaseObjectDecorator impleme
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->subject;
     }
@@ -33,7 +33,7 @@ class ConversationUserNotificationObject extends DatabaseObjectDecorator impleme
     /**
      * @inheritDoc
      */
-    public function getURL()
+    public function getURL(): string
     {
         return $this->getLink();
     }