1 files changed, 13 insertions, 30 deletions

diff --git a/apps/files_sharing/lib/Notification/Notifier.php b/apps/files_sharing/lib/Notification/Notifier.php
index 3947e5aa5f2..e4434ef0b37 100644
--- a/apps/files_sharing/lib/Notification/Notifier.php
+++ b/apps/files_sharing/lib/Notification/Notifier.php
@@ -28,31 +28,14 @@ class Notifier implements INotifier {
 	public const INCOMING_USER_SHARE = 'incoming_user_share';
 	public const INCOMING_GROUP_SHARE = 'incoming_group_share';
 
-	/** @var IFactory */
-	protected $l10nFactory;
-	/** @var IManager */
-	private $shareManager;
-	/** @var IRootFolder */
-	private $rootFolder;
-	/** @var IGroupManager  */
-	protected $groupManager;
-	/** @var IUserManager  */
-	protected $userManager;
-	/** @var IURLGenerator */
-	protected $url;
-
-	public function __construct(IFactory $l10nFactory,
-		IManager $shareManager,
-		IRootFolder $rootFolder,
-		IGroupManager $groupManager,
-		IUserManager $userManager,
-		IURLGenerator $url) {
-		$this->l10nFactory = $l10nFactory;
-		$this->shareManager = $shareManager;
-		$this->rootFolder = $rootFolder;
-		$this->groupManager = $groupManager;
-		$this->userManager = $userManager;
-		$this->url = $url;
+	public function __construct(
+		protected IFactory $l10nFactory,
+		private IManager $shareManager,
+		private IRootFolder $rootFolder,
+		protected IGroupManager $groupManager,
+		protected IUserManager $userManager,
+		protected IURLGenerator $url,
+	) {
 	}
 
 	/**
@@ -84,9 +67,9 @@ class Notifier implements INotifier {
 	 * @since 9.0.0
 	 */
 	public function prepare(INotification $notification, string $languageCode): INotification {
-		if ($notification->getApp() !== 'files_sharing' ||
-			($notification->getSubject() !== 'expiresTomorrow' &&
-				$notification->getObjectType() !== 'share')) {
+		if ($notification->getApp() !== 'files_sharing'
+			|| ($notification->getSubject() !== 'expiresTomorrow'
+				&& $notification->getObjectType() !== 'share')) {
 			throw new UnknownNotificationException('Unhandled app or subject');
 		}
 
@@ -126,9 +109,9 @@ class Notifier implements INotifier {
 				[
 					'node' => [
 						'type' => 'file',
-						'id' => $node->getId(),
+						'id' => (string)$node->getId(),
 						'name' => $node->getName(),
-						'path' => $path,
+						'path' => (string)$path,
 					],
 				]
 			);