From: Hamid Dehnavi <hamid.dev.pro@gmail.com>
Date: Wed, 5 Jul 2023 17:55:05 +0000 (+0330)
Subject: refactor lib/private/Notification
X-Git-Tag: v29.0.0beta1~564^2~3
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0ac1480be26105081b71241fa91c1f56a306db9d;p=nextcloud-server.git

refactor lib/private/Notification

Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
---

diff --git a/lib/private/Notification/Action.php b/lib/private/Notification/Action.php
index ff9cf9e38f5..f0e40f6e1d6 100644
--- a/lib/private/Notification/Action.php
+++ b/lib/private/Notification/Action.php
@@ -28,22 +28,22 @@ use OCP\Notification\IAction;
 
 class Action implements IAction {
 	/** @var string */
-	protected $label;
+	protected string $label;
 
 	/** @var string */
-	protected $labelParsed;
+	protected string $labelParsed;
 
 	/** @var string */
-	protected $link;
+	protected string $link;
 
 	/** @var string */
-	protected $requestType;
+	protected string $requestType;
 
 	/** @var string */
-	protected $icon;
+	protected string $icon;
 
 	/** @var bool */
-	protected $primary;
+	protected bool $primary;
 
 	public function __construct() {
 		$this->label = '';
diff --git a/lib/private/Notification/Manager.php b/lib/private/Notification/Manager.php
index e81b6c4fa35..c712d2754e2 100644
--- a/lib/private/Notification/Manager.php
+++ b/lib/private/Notification/Manager.php
@@ -43,48 +43,35 @@ use Psr\Container\ContainerExceptionInterface;
 use Psr\Log\LoggerInterface;
 
 class Manager implements IManager {
-	/** @var IValidator */
-	protected $validator;
-	/** @var IUserManager */
-	private $userManager;
 	/** @var ICache */
-	protected $cache;
-	/** @var IRegistry */
-	protected $subscription;
-	/** @var LoggerInterface */
-	protected $logger;
-	/** @var Coordinator */
-	private $coordinator;
+	protected ICache $cache;
 
 	/** @var IApp[] */
-	protected $apps;
+	protected array $apps;
 	/** @var string[] */
-	protected $appClasses;
+	protected array $appClasses;
 
 	/** @var INotifier[] */
-	protected $notifiers;
+	protected array $notifiers;
 	/** @var string[] */
-	protected $notifierClasses;
+	protected array $notifierClasses;
 
 	/** @var bool */
-	protected $preparingPushNotification;
+	protected bool $preparingPushNotification;
 	/** @var bool */
-	protected $deferPushing;
+	protected bool $deferPushing;
 	/** @var bool */
-	private $parsedRegistrationContext;
+	private bool $parsedRegistrationContext;
 
-	public function __construct(IValidator $validator,
-		IUserManager $userManager,
+	public function __construct(
+		protected IValidator $validator,
+		private IUserManager $userManager,
 		ICacheFactory $cacheFactory,
-		IRegistry $subscription,
-		LoggerInterface $logger,
-		Coordinator $coordinator) {
-		$this->validator = $validator;
-		$this->userManager = $userManager;
+		protected IRegistry $subscription,
+		protected LoggerInterface $logger,
+		private Coordinator $coordinator,
+	) {
 		$this->cache = $cacheFactory->createDistributed('notifications');
-		$this->subscription = $subscription;
-		$this->logger = $logger;
-		$this->coordinator = $coordinator;
 
 		$this->apps = [];
 		$this->notifiers = [];
@@ -111,7 +98,7 @@ class Manager implements IManager {
 	 * @deprecated 17.0.0 use registerNotifierService instead.
 	 * @since 8.2.0 - Parameter $info was added in 9.0.0
 	 */
-	public function registerNotifier(\Closure $service, \Closure $info) {
+	public function registerNotifier(\Closure $service, \Closure $info): void {
 		$infoData = $info();
 		$exception = new \InvalidArgumentException(
 			'Notifier ' . $infoData['name'] . ' (id: ' . $infoData['id'] . ') is not considered because it is using the old way to register.'
diff --git a/lib/private/Notification/Notification.php b/lib/private/Notification/Notification.php
index 2291c4ae34f..fcacc897c53 100644
--- a/lib/private/Notification/Notification.php
+++ b/lib/private/Notification/Notification.php
@@ -32,74 +32,73 @@ use OCP\RichObjectStrings\InvalidObjectExeption;
 use OCP\RichObjectStrings\IValidator;
 
 class Notification implements INotification {
-	/** @var IValidator */
-	protected $richValidator;
 
 	/** @var string */
-	protected $app;
+	protected string $app;
 
 	/** @var string */
-	protected $user;
+	protected string $user;
 
 	/** @var \DateTime */
-	protected $dateTime;
+	protected \DateTime $dateTime;
 
 	/** @var string */
-	protected $objectType;
+	protected string $objectType;
 
 	/** @var string */
-	protected $objectId;
+	protected string $objectId;
 
 	/** @var string */
-	protected $subject;
+	protected string $subject;
 
 	/** @var array */
-	protected $subjectParameters;
+	protected array $subjectParameters;
 
 	/** @var string */
-	protected $subjectParsed;
+	protected string $subjectParsed;
 
 	/** @var string */
-	protected $subjectRich;
+	protected string $subjectRich;
 
 	/** @var array */
-	protected $subjectRichParameters;
+	protected array $subjectRichParameters;
 
 	/** @var string */
-	protected $message;
+	protected string $message;
 
 	/** @var array */
-	protected $messageParameters;
+	protected array $messageParameters;
 
 	/** @var string */
-	protected $messageParsed;
+	protected string $messageParsed;
 
 	/** @var string */
-	protected $messageRich;
+	protected string $messageRich;
 
 	/** @var array */
-	protected $messageRichParameters;
+	protected array $messageRichParameters;
 
 	/** @var string */
-	protected $link;
+	protected string $link;
 
 	/** @var string */
-	protected $icon;
+	protected string $icon;
 
 	/** @var array */
-	protected $actions;
+	protected array $actions;
 
 	/** @var array */
-	protected $actionsParsed;
+	protected array $actionsParsed;
 
 	/** @var bool */
-	protected $hasPrimaryAction;
+	protected bool $hasPrimaryAction;
 
 	/** @var bool */
-	protected $hasPrimaryParsedAction;
+	protected bool $hasPrimaryParsedAction;
 
-	public function __construct(IValidator $richValidator) {
-		$this->richValidator = $richValidator;
+	public function __construct(
+		protected IValidator $richValidator,
+	) {
 		$this->app = '';
 		$this->user = '';
 		$this->dateTime = new \DateTime();