Merge pull request #11242 from nextcloud/refactor/remove-unused-properties

refactor: remove unused properties

Author: ChristophWurst

lib/Folder.php

@@ -12,8 +12,6 @@
 use Horde_Imap_Client_Mailbox;
 
 class Folder {
-	/** @var int */
-	private $accountId;
 
 	/** @var Horde_Imap_Client_Mailbox */
 	private $mailbox;
@@ -32,12 +30,10 @@ class Folder {
 
 	private ?string $myAcls;
 
-	public function __construct(int $accountId,
-		Horde_Imap_Client_Mailbox $mailbox,
+	public function __construct(Horde_Imap_Client_Mailbox $mailbox,
 		array $attributes,
 		?string $delimiter,
 		?array $status) {
-		$this->accountId = $accountId;
 		$this->mailbox = $mailbox;
 		$this->attributes = $attributes;
 		$this->delimiter = $delimiter;

lib/Http/Middleware/ProvisioningMiddleware.php

@@ -15,7 +15,6 @@
 use OCP\Authentication\Exceptions\PasswordUnavailableException;
 use OCP\Authentication\LoginCredentials\IStore as ICredentialStore;
 use OCP\IUserSession;
-use Psr\Log\LoggerInterface;
 
 class ProvisioningMiddleware extends Middleware {
 	/** @var IUserSession */
@@ -27,17 +26,12 @@ class ProvisioningMiddleware extends Middleware {
 	/** @var ProvisioningManager */
 	private $provisioningManager;
 
-	/** @var LoggerInterface */
-	private $logger;
-
 	public function __construct(IUserSession $userSession,
 		ICredentialStore $credentialStore,
-		ProvisioningManager $provisioningManager,
-		LoggerInterface $logger) {
+		ProvisioningManager $provisioningManager) {
 		$this->userSession = $userSession;
 		$this->credentialStore = $credentialStore;
 		$this->provisioningManager = $provisioningManager;
-		$this->logger = $logger;
 	}
 
 	#[\Override]

lib/IMAP/FolderMapper.php

@@ -65,7 +65,6 @@ public function getFolders(Account $account, Horde_Imap_Client_Socket $client,
 		});
 		return array_map(static function (array $mailbox) use ($account) {
 			return new Folder(
-				$account->getId(),
 				$mailbox['mailbox'],
 				$mailbox['attributes'],
 				$mailbox['delimiter'],
@@ -75,7 +74,6 @@ public function getFolders(Account $account, Horde_Imap_Client_Socket $client,
 	}
 
 	public function createFolder(Horde_Imap_Client_Socket $client,
-		Account $account,
 		string $name): Folder {
 		$client->createMailbox($name);
 
@@ -92,7 +90,6 @@ public function createFolder(Horde_Imap_Client_Socket $client,
 		}
 
 		return new Folder(
-			$account->getId(),
 			$mb['mailbox'],
 			$mb['attributes'],
 			$mb['delimiter'],

lib/Listener/NewMessagesSummarizeListener.php

@@ -9,10 +9,8 @@
 
 namespace OCA\Mail\Listener;
 
-use OCA\Mail\Contracts\IMailManager;
 use OCA\Mail\Events\NewMessagesSynchronized;
 use OCA\Mail\Exception\ServiceException;
-use OCA\Mail\IMAP\IMAPClientFactory;
 use OCA\Mail\Service\AiIntegrations\AiIntegrationsService;
 use OCP\AppFramework\Services\IAppConfig;
 use OCP\EventDispatcher\Event;
@@ -26,9 +24,7 @@ class NewMessagesSummarizeListener implements IEventListener {
 
 	public function __construct(
 		private LoggerInterface $logger,
-		private IMAPClientFactory $imapFactory,
 		private AiIntegrationsService $aiService,
-		private IMailManager $mailManager,
 		private IAppConfig $appConfig,
 	) {
 	}

lib/Migration/MigrateImportantFromImapAndDb.php

@@ -16,13 +16,11 @@
 use OCA\Mail\Db\MailboxMapper;
 use OCA\Mail\Db\Tag;
 use OCA\Mail\Exception\ServiceException;
-use OCA\Mail\IMAP\IMAPClientFactory;
 use OCA\Mail\IMAP\MessageMapper;
 use Psr\Log\LoggerInterface;
 
 class MigrateImportantFromImapAndDb {
-	/** @var IMAPClientFactory */
-	private $clientFactory;
+
 
 	/** @var MessageMapper */
 	private $messageMapper;

lib/PhishingDetectionList.php

@@ -17,8 +17,6 @@ class PhishingDetectionList implements JsonSerializable {
 	/** @var PhishingDetectionResult[] */
 	private array $checks;
 
-	private bool $warning = false;
-
 	/**
 	 * @param PhishingDetectionResult[] $checks
 	 */

lib/Service/AvatarService.php

@@ -16,7 +16,6 @@
 use OCA\Mail\Service\Avatar\Cache as AvatarCache;
 use OCA\Mail\Service\Avatar\CompositeAvatarSource;
 use OCA\Mail\Service\Avatar\Downloader;
-use OCP\IURLGenerator;
 
 class AvatarService implements IAvatarService {
 	/** @var AvatarCache */
@@ -28,9 +27,6 @@ class AvatarService implements IAvatarService {
 	/** @var CompositeAvatarSource */
 	private $source;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
 	/** @var AvatarFactory */
 	private $avatarFactory;
 
@@ -41,19 +37,16 @@ class AvatarService implements IAvatarService {
 	 * @param CompositeAvatarSource $source
 	 * @param Downloader $downloader
 	 * @param AvatarCache $cache
-	 * @param IURLGenerator $urlGenerator
 	 * @param AvatarFactory $avatarFactory
 	 * @param IUserPreferences $preferences
 	 */
 	public function __construct(CompositeAvatarSource $source,
 		Downloader $downloader,
 		AvatarCache $cache,
-		IURLGenerator $urlGenerator,
 		AvatarFactory $avatarFactory,
 		IUserPreferences $preferences) {
 		$this->source = $source;
 		$this->cache = $cache;
-		$this->urlGenerator = $urlGenerator;
 		$this->downloader = $downloader;
 		$this->avatarFactory = $avatarFactory;
 		$this->preferences = $preferences;

lib/Service/MailManager.php

@@ -152,7 +152,7 @@ public function getMailboxes(Account $account): array {
 	public function createMailbox(Account $account, string $name): Mailbox {
 		$client = $this->imapClientFactory->getClient($account);
 		try {
-			$folder = $this->folderMapper->createFolder($client, $account, $name);
+			$folder = $this->folderMapper->createFolder($client, $name);
 			$this->folderMapper->fetchFolderAcls([$folder], $client);
 		} catch (Horde_Imap_Client_Exception $e) {
 			throw new ServiceException(
@@ -809,7 +809,7 @@ public function filterFlags(Horde_Imap_Client_Socket $client, Account $account,
 		if (in_array("\*", $capabilities['permflags']) || in_array($flag, $capabilities['permflags'])) {
 			return [$flag];
 		}
-		
+
 		return [];
 	}
 

lib/Service/OutboxService.php

@@ -11,7 +11,6 @@
 
 use OCA\Mail\Account;
 use OCA\Mail\Contracts\IMailManager;
-use OCA\Mail\Contracts\IMailTransmission;
 use OCA\Mail\Db\LocalMessage;
 use OCA\Mail\Db\LocalMessageMapper;
 use OCA\Mail\Db\Recipient;
@@ -29,8 +28,7 @@
 use Throwable;
 
 class OutboxService {
-	/** @var IMailTransmission */
-	private $transmission;
+
 
 	/** @var LocalMessageMapper */
 	private $mapper;
@@ -57,7 +55,6 @@ class OutboxService {
 	private $logger;
 
 	public function __construct(
-		IMailTransmission $transmission,
 		LocalMessageMapper $mapper,
 		AttachmentService $attachmentService,
 		IEventDispatcher $eventDispatcher,
@@ -68,7 +65,6 @@ public function __construct(
 		LoggerInterface $logger,
 		private Chain $sendChain,
 	) {
-		$this->transmission = $transmission;
 		$this->mapper = $mapper;
 		$this->attachmentService = $attachmentService;
 		$this->eventDispatcher = $eventDispatcher;

lib/Service/Search/MailSearch.php

@@ -13,7 +13,6 @@
 use OCA\Mail\Account;
 use OCA\Mail\Contracts\IMailSearch;
 use OCA\Mail\Db\Mailbox;
-use OCA\Mail\Db\MailboxMapper;
 use OCA\Mail\Db\Message;
 use OCA\Mail\Db\MessageMapper;
 use OCA\Mail\Exception\ClientException;
@@ -30,9 +29,6 @@ class MailSearch implements IMailSearch {
 	/** @var FilterStringParser */
 	private $filterStringParser;
 
-	/** @var MailboxMapper */
-	private $mailboxMapper;
-
 	/** @var ImapSearchProvider */
 	private $imapSearchProvider;
 
@@ -46,13 +42,11 @@ class MailSearch implements IMailSearch {
 	private $timeFactory;
 
 	public function __construct(FilterStringParser $filterStringParser,
-		MailboxMapper $mailboxMapper,
 		ImapSearchProvider $imapSearchProvider,
 		MessageMapper $messageMapper,
 		PreviewEnhancer $previewEnhancer,
 		ITimeFactory $timeFactory) {
 		$this->filterStringParser = $filterStringParser;
-		$this->mailboxMapper = $mailboxMapper;
 		$this->imapSearchProvider = $imapSearchProvider;
 		$this->messageMapper = $messageMapper;
 		$this->previewEnhancer = $previewEnhancer;