Fix outdated code

Missing return type for `getTitle()` and `getLink()`

Author: BurntimeX

snippets/php/api/user_notifications/FooUserNotificationEvent.class.php

@@ -1,5 +1,7 @@
 <?php
+
 namespace example\system\user\notification\event;
+
 use example\system\cache\runtime\BazRuntimeCache;
 use example\system\user\notification\object\FooUserNotificationObject;
 use wcf\system\email\Email;
@@ -16,7 +18,8 @@
  *
  * @method	FooUserNotificationObject	getUserNotificationObject()
  */
-class FooUserNotificationEvent extends AbstractSharedUserNotificationEvent {
+class FooUserNotificationEvent extends AbstractSharedUserNotificationEvent
+{
     /**
      * @inheritDoc
      */
@@ -26,7 +29,8 @@ class FooUserNotificationEvent extends AbstractSharedUserNotificationEvent {
     /**
      * @inheritDoc
      */
-    public function checkAccess() {
+    public function checkAccess()
+    {
         $this->getUserNotificationObject()->setBaz(BazRuntimeCache::getInstance()->getObject($this->getUserNotificationObject()->bazID));
 
         if (!$this->getUserNotificationObject()->isAccessible()) {
@@ -42,15 +46,16 @@ public function checkAccess() {
     /**
      * @inheritDoc
      */
-    public function getEmailMessage($notificationType = 'instant') {
+    public function getEmailMessage($notificationType = 'instant')
+    {
         $this->getUserNotificationObject()->setBaz(BazRuntimeCache::getInstance()->getObject($this->getUserNotificationObject()->bazID));
 
-        $messageID = '<com.woltlab.example.baz/'.$this->getUserNotificationObject()->bazID.'@'.Email::getHost().'>';
+        $messageID = '<com.woltlab.example.baz/' . $this->getUserNotificationObject()->bazID . '@' . Email::getHost() . '>';
 
         return [
             'application' => 'example',
             'in-reply-to' => [$messageID],
-            'message-id' => 'com.woltlab.example.foo/'.$this->getUserNotificationObject()->fooID,
+            'message-id' => 'com.woltlab.example.foo/' . $this->getUserNotificationObject()->fooID,
             'references' => [$messageID],
             'template' => 'email_notification_foo'
         ];
@@ -60,7 +65,8 @@ public function getEmailMessage($notificationType = 'instant') {
      * @inheritDoc
      * @since	5.0
      */
-    public function getEmailTitle() {
+    public function getEmailTitle()
+    {
         $this->getUserNotificationObject()->setBaz(BazRuntimeCache::getInstance()->getObject($this->getUserNotificationObject()->bazID));
 
         return $this->getLanguage()->getDynamicVariable('example.foo.notification.mail.title', [
@@ -72,14 +78,16 @@ public function getEmailTitle() {
     /**
      * @inheritDoc
      */
-    public function getEventHash() {
+    public function getEventHash()
+    {
         return sha1($this->eventID . '-' . $this->getUserNotificationObject()->bazID);
     }
 
     /**
      * @inheritDoc
      */
-    public function getLink() {
+    public function getLink(): string
+    {
         return LinkHandler::getInstance()->getLink('Foo', [
             'application' => 'example',
             'object' => $this->getUserNotificationObject()->getDecoratedObject()
@@ -89,7 +97,8 @@ public function getLink() {
     /**
      * @inheritDoc
      */
-    public function getMessage() {
+    public function getMessage()
+    {
         $authors = $this->getAuthors();
         $count = count($authors);
 
@@ -118,7 +127,8 @@ public function getMessage() {
     /**
      * @inheritDoc
      */
-    public function getTitle() {
+    public function getTitle(): string
+    {
         $count = count($this->getAuthors());
         if ($count > 1) {
             return $this->getLanguage()->getDynamicVariable('example.foo.notification.title.stacked', [
@@ -133,7 +143,8 @@ public function getTitle() {
     /**
      * @inheritDoc
      */
-    protected function prepare() {
+    protected function prepare()
+    {
         BazRuntimeCache::getInstance()->cacheObjectID($this->getUserNotificationObject()->bazID);
     }
-}
+}

snippets/php/api/user_notifications/FooUserNotificationObject.class.php

@@ -1,5 +1,7 @@
 <?php
+
 namespace example\system\user\notification\object;
+
 use example\data\foo\Foo;
 use wcf\data\DatabaseObjectDecorator;
 use wcf\system\user\notification\object\IUserNotificationObject;
@@ -15,7 +17,8 @@
  * @method	Foo	getDecoratedObject()
  * @mixin	Foo
  */
-class FooUserNotificationObject extends DatabaseObjectDecorator implements IUserNotificationObject {
+class FooUserNotificationObject extends DatabaseObjectDecorator implements IUserNotificationObject
+{
     /**
      * @inheritDoc
      */
@@ -24,21 +27,24 @@ class FooUserNotificationObject extends DatabaseObjectDecorator implements IUser
     /**
      * @inheritDoc
      */
-    public function getTitle() {
+    public function getTitle(): string
+    {
         return $this->getDecoratedObject()->getTitle();
     }
 
     /**
      * @inheritDoc
      */
-    public function getURL() {
+    public function getURL()
+    {
         return $this->getDecoratedObject()->getLink();
     }
 
     /**
      * @inheritDoc
      */
-    public function getAuthorID() {
+    public function getAuthorID()
+    {
         return $this->getDecoratedObject()->userID;
     }
-}
+}

snippets/tutorial/tutorial-series/part-1/files/lib/data/person/Person.class.php

@@ -32,7 +32,7 @@ public function __toString()
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->firstName . ' ' . $this->lastName;
     }

snippets/tutorial/tutorial-series/part-3/files/lib/data/person/Person.class.php

@@ -35,7 +35,7 @@ public function __toString()
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getControllerLink(PersonPage::class, [
             'object' => $this,
@@ -45,7 +45,7 @@ public function getLink()
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->firstName . ' ' . $this->lastName;
     }

snippets/tutorial/tutorial-series/part-4/files/lib/data/person/Person.class.php

@@ -35,7 +35,7 @@ public function __toString()
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getControllerLink(PersonPage::class, [
             'object' => $this,
@@ -45,7 +45,7 @@ public function getLink()
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->firstName . ' ' . $this->lastName;
     }

snippets/tutorial/tutorial-series/part-5/files/lib/data/person/Person.class.php

@@ -44,7 +44,7 @@ public function __toString()
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getControllerLink(PersonPage::class, [
             'object' => $this,
@@ -77,7 +77,7 @@ public function getInformation(): array
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->firstName . ' ' . $this->lastName;
     }

snippets/tutorial/tutorial-series/part-6/files/lib/data/person/Person.class.php

@@ -44,7 +44,7 @@ public function __toString()
     /**
      * @inheritDoc
      */
-    public function getLink()
+    public function getLink(): string
     {
         return LinkHandler::getInstance()->getControllerLink(PersonPage::class, [
             'object' => $this,
@@ -77,7 +77,7 @@ public function getInformation(): array
     /**
      * @inheritDoc
      */
-    public function getTitle()
+    public function getTitle(): string
     {
         return $this->firstName . ' ' . $this->lastName;
     }