refactor: mark classes final

lib/Account.php

@@ -14,7 +14,7 @@
 use OCA\Mail\Service\Quota;
 use ReturnTypeWillChange;
 
-class Account implements JsonSerializable {
+final class Account implements JsonSerializable {
 	public function __construct(
 		private MailAccount $account,
 	) {

lib/Address.php

@@ -16,7 +16,7 @@
 /**
  * @psalm-immutable
  */
-class Address implements JsonSerializable {
+final class Address implements JsonSerializable {
 	public const TYPE_FROM = 0;
 	public const TYPE_TO = 1;
 	public const TYPE_CC = 2;

lib/AddressList.php

@@ -19,7 +19,7 @@
 /**
  * @psalm-immutable
  */
-class AddressList implements Countable, JsonSerializable {
+final class AddressList implements Countable, JsonSerializable {
 	/** @var Address[] */
 	private $addresses;
 

lib/AppInfo/Application.php

@@ -87,7 +87,7 @@
 
 include_once __DIR__ . '/../../vendor/autoload.php';
 
-class Application extends App implements IBootstrap {
+final class Application extends App implements IBootstrap {
 	public const APP_ID = 'mail';
 
 	public function __construct(array $urlParams = []) {

lib/Attachment.php

@@ -10,7 +10,7 @@
 
 use Horde_Mime_Part;
 
-class Attachment {
+final class Attachment {
 	private ?string $id;
 	private ?string $name;
 	private string $type;

lib/BackgroundJob/CleanupJob.php

@@ -14,7 +14,7 @@
 use OCP\BackgroundJob\TimedJob;
 use Psr\Log\LoggerInterface;
 
-class CleanupJob extends TimedJob {
+final class CleanupJob extends TimedJob {
 	private CleanupService $cleanupService;
 	private LoggerInterface $logger;
 

lib/BackgroundJob/DeleteDuplicatedUidsJob.php

@@ -13,7 +13,7 @@
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\BackgroundJob\QueuedJob;
 
-class DeleteDuplicatedUidsJob extends QueuedJob {
+final class DeleteDuplicatedUidsJob extends QueuedJob {
 	public function __construct(
 		ITimeFactory $time,
 		private MessageMapper $messageMapper,

lib/BackgroundJob/DraftsJob.php

@@ -13,7 +13,7 @@
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\BackgroundJob\TimedJob;
 
-class DraftsJob extends TimedJob {
+final class DraftsJob extends TimedJob {
 	private DraftsService $draftsService;
 
 	public function __construct(ITimeFactory $time,

lib/BackgroundJob/FollowUpClassifierJob.php

@@ -20,7 +20,7 @@
 use OCP\DB\Exception;
 use Psr\Log\LoggerInterface;
 
-class FollowUpClassifierJob extends QueuedJob {
+final class FollowUpClassifierJob extends QueuedJob {
 
 	public const PARAM_MESSAGE_ID = 'messageId';
 	public const PARAM_MAILBOX_ID = 'mailboxId';

lib/BackgroundJob/IMipMessageJob.php

@@ -13,7 +13,7 @@
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\BackgroundJob\TimedJob;
 
-class IMipMessageJob extends TimedJob {
+final class IMipMessageJob extends TimedJob {
 	private IMipService $iMipService;
 
 	public function __construct(ITimeFactory $time,

lib/BackgroundJob/MigrateImportantJob.php

@@ -22,7 +22,7 @@
 use OCP\BackgroundJob\QueuedJob;
 use Psr\Log\LoggerInterface;
 
-class MigrateImportantJob extends QueuedJob {
+final class MigrateImportantJob extends QueuedJob {
 	private MailboxMapper $mailboxMapper;
 	private MailAccountMapper $mailAccountMapper;
 	private MailManager $mailManager;

lib/BackgroundJob/OutboxWorkerJob.php

@@ -13,7 +13,7 @@
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\BackgroundJob\TimedJob;
 
-class OutboxWorkerJob extends TimedJob {
+final class OutboxWorkerJob extends TimedJob {
 	private OutboxService $outboxService;
 
 	public function __construct(ITimeFactory $time,

lib/BackgroundJob/PreviewEnhancementProcessingJob.php

@@ -18,7 +18,7 @@
 use Psr\Log\LoggerInterface;
 use function sprintf;
 
-class PreviewEnhancementProcessingJob extends TimedJob {
+final class PreviewEnhancementProcessingJob extends TimedJob {
 	private IUserManager $userManager;
 	private AccountService $accountService;
 	private LoggerInterface $logger;

lib/BackgroundJob/QuotaJob.php

@@ -19,7 +19,7 @@
 use Psr\Log\LoggerInterface;
 use function sprintf;
 
-class QuotaJob extends TimedJob {
+final class QuotaJob extends TimedJob {
 	private IUserManager $userManager;
 	private AccountService $accountService;
 	private IMailManager $mailManager;

lib/BackgroundJob/RepairSyncJob.php

@@ -21,7 +21,7 @@
 use OCP\IUserManager;
 use Psr\Log\LoggerInterface;
 
-class RepairSyncJob extends TimedJob {
+final class RepairSyncJob extends TimedJob {
 	public function __construct(
 		ITimeFactory $time,
 		private SyncService $syncService,

lib/BackgroundJob/SyncJob.php

@@ -25,7 +25,7 @@
 use function max;
 use function sprintf;
 
-class SyncJob extends TimedJob {
+final class SyncJob extends TimedJob {
 	private IUserManager $userManager;
 	private AccountService $accountService;
 	private ImapToDbSynchronizer $syncService;

lib/BackgroundJob/TrainImportanceClassifierJob.php

@@ -19,7 +19,7 @@
 use Psr\Log\LoggerInterface;
 use Throwable;
 
-class TrainImportanceClassifierJob extends TimedJob {
+final class TrainImportanceClassifierJob extends TimedJob {
 	private AccountService $accountService;
 	private ImportanceClassifier $classifier;
 	private IJobList $jobList;

lib/BackgroundJob/TrashRetentionJob.php

@@ -23,7 +23,7 @@
 use OCP\BackgroundJob\TimedJob;
 use Psr\Log\LoggerInterface;
 
-class TrashRetentionJob extends TimedJob {
+final class TrashRetentionJob extends TimedJob {
 	public function __construct(
 		ITimeFactory $time,
 		private LoggerInterface $logger,

lib/BackgroundJob/WakeJob.php

@@ -13,7 +13,7 @@
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\BackgroundJob\TimedJob;
 
-class WakeJob extends TimedJob {
+final class WakeJob extends TimedJob {
 	public function __construct(
 		ITimeFactory $time,
 		private SnoozeService $snoozeService,

lib/Cache/Cache.php

@@ -19,7 +19,7 @@
  * This class passes the minimum amount of data from the db cache to Horde to make QRESYNC work
  * reliably.
  */
-class Cache extends Horde_Imap_Client_Cache_Backend {
+final class Cache extends Horde_Imap_Client_Cache_Backend {
 	/** @var CachedMailbox[] */
 	private array $cachedMailboxes = [];
 

lib/Cache/CachedMailbox.php

@@ -9,7 +9,7 @@
 
 namespace OCA\Mail\Cache;
 
-class CachedMailbox {
+final class CachedMailbox {
 	/** @var int[]|null */
 	private ?array $uids = null;
 

lib/Cache/HordeCacheFactory.php

@@ -13,7 +13,7 @@
 use OCA\Mail\Db\MailboxMapper;
 use OCA\Mail\Db\MessageMapper;
 
-class HordeCacheFactory {
+final class HordeCacheFactory {
 	public function __construct(
 		private MailboxMapper $mailboxMapper,
 		private MessageMapper $messageMapper,

lib/Cache/HordeSyncToken.php

@@ -9,7 +9,7 @@
 
 namespace OCA\Mail\Cache;
 
-class HordeSyncToken {
+final class HordeSyncToken {
 	public function __construct(
 		private ?int $nextUid,
 		private ?int $uidValidity,

lib/Cache/HordeSyncTokenParser.php

@@ -9,7 +9,7 @@
 
 namespace OCA\Mail\Cache;
 
-class HordeSyncTokenParser {
+final class HordeSyncTokenParser {
 	public function parseSyncToken(string $token): HordeSyncToken {
 		$decodedToken = base64_decode($token, true);
 		$parts = explode(',', $decodedToken);

lib/Command/AddMissingTags.php

@@ -19,7 +19,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class AddMissingTags extends Command {
+final class AddMissingTags extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 
 	private LoggerInterface $logger;

lib/Command/CleanUp.php

@@ -16,7 +16,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class CleanUp extends Command {
+final class CleanUp extends Command {
 	private CleanupService $cleanupService;
 	private LoggerInterface $logger;
 

lib/Command/CreateAccount.php

@@ -19,7 +19,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class CreateAccount extends Command {
+final class CreateAccount extends Command {
 	public const ARGUMENT_USER_ID = 'user-id';
 	public const ARGUMENT_NAME = 'name';
 	public const ARGUMENT_EMAIL = 'email';

lib/Command/CreateTagMigrationJobEntry.php

@@ -16,7 +16,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class CreateTagMigrationJobEntry extends Command {
+final class CreateTagMigrationJobEntry extends Command {
 	private JobList $jobList;
 	private MailboxMapper $mailboxMapper;
 

lib/Command/DebugAccount.php

@@ -18,7 +18,7 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class DebugAccount extends Command {
+final class DebugAccount extends Command {
 	protected const ARGUMENT_ACCOUNT_ID = 'account-id';
 	protected const OPTION_DEBUG_ON = 'on';
 	protected const OPTION_DEBUG_OFF = 'off';

lib/Command/DeleteAccount.php

@@ -19,7 +19,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class DeleteAccount extends Command {
+final class DeleteAccount extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 
 	private AccountService $accountService;

lib/Command/DiagnoseAccount.php

@@ -25,7 +25,7 @@
 use function json_decode;
 use function sort;
 
-class DiagnoseAccount extends Command {
+final class DiagnoseAccount extends Command {
 	private const ARGUMENT_ACCOUNT_ID = 'account-id';
 
 	private AccountService $accountService;

lib/Command/ExportAccount.php

@@ -15,7 +15,7 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class ExportAccount extends Command {
+final class ExportAccount extends Command {
 	public const ARGUMENT_USER_ID = 'user-id';
 	public const ARGUMENT_OUTPUT_FORMAT = 'output';
 

lib/Command/ExportAccountThreads.php

@@ -22,7 +22,7 @@
 use Symfony\Component\Console\Output\OutputInterface;
 use function json_encode;
 
-class ExportAccountThreads extends Command {
+final class ExportAccountThreads extends Command {
 	private const ARGUMENT_ACCOUNT_ID = 'account-id';
 	private const OPTION_REDACT = 'redact';
 

lib/Command/PredictImportance.php

@@ -23,7 +23,7 @@
 use Symfony\Component\Console\Output\OutputInterface;
 use function memory_get_peak_usage;
 
-class PredictImportance extends Command {
+final class PredictImportance extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 	public const ARGUMENT_SENDER = 'sender';
 	public const ARGUMENT_SUBJECT = 'subject';

lib/Command/RunMetaEstimator.php

@@ -28,7 +28,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class RunMetaEstimator extends Command {
+final class RunMetaEstimator extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 	public const ARGUMENT_SHUFFLE = 'shuffle';
 

lib/Command/SyncAccount.php

@@ -26,7 +26,7 @@
 use function memory_get_peak_usage;
 use function memory_get_usage;
 
-class SyncAccount extends Command {
+final class SyncAccount extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 	public const OPTION_FORCE = 'force';
 

lib/Command/Thread.php

@@ -23,7 +23,7 @@
 use function json_decode;
 use function memory_get_peak_usage;
 
-class Thread extends Command {
+final class Thread extends Command {
 	public const ARGUMENT_INPUT_FILE = 'thread-file';
 
 	private ThreadBuilder $builder;

lib/Command/TrainAccount.php

@@ -21,7 +21,7 @@
 use Symfony\Component\Console\Output\OutputInterface;
 use function memory_get_peak_usage;
 
-class TrainAccount extends Command {
+final class TrainAccount extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 	public const ARGUMENT_SHUFFLE = 'shuffle';
 	public const ARGUMENT_DRY_RUN = 'dry-run';

lib/Command/UpdateAccount.php

@@ -16,7 +16,7 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class UpdateAccount extends Command {
+final class UpdateAccount extends Command {
 	public const ARGUMENT_ACCOUNT_ID = 'account-id';
 	public const ARGUMENT_NAME = 'name';
 	public const ARGUMENT_EMAIL = 'email';

lib/Command/UpdateSystemAutoresponders.php

@@ -16,7 +16,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class UpdateSystemAutoresponders extends Command {
+final class UpdateSystemAutoresponders extends Command {
 	public function __construct(
 		private MailAccountMapper $mailAccountMapper,
 		private IUserManager $userManager,

lib/Controller/AccountApiController.php

@@ -25,7 +25,7 @@
 /**
  * @psalm-import-type MailAccountListResponse from ResponseDefinitions
  */
-class AccountApiController extends OCSController {
+final class AccountApiController extends OCSController {
 	public function __construct(
 		string $appName,
 		IRequest $request,

lib/Controller/AccountsController.php

@@ -37,7 +37,7 @@
 use OCP\Security\IRemoteHostValidator;
 use Psr\Log\LoggerInterface;
 
-#[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
+final #[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
 class AccountsController extends Controller {
 	private AccountService $accountService;
 	private string $currentUserId;

lib/Controller/AliasesController.php

@@ -19,7 +19,7 @@
 use OCP\AppFramework\Http\JSONResponse;
 use OCP\IRequest;
 
-#[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
+final #[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
 class AliasesController extends Controller {
 	private AliasesService $aliasService;
 	private string $currentUserId;

lib/Controller/AutoCompleteController.php

@@ -17,7 +17,7 @@
 use OCP\AppFramework\Http\JSONResponse;
 use OCP\IRequest;
 
-#[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
+final #[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
 class AutoCompleteController extends Controller {
 	private AutoCompleteService $service;
 	private ?string $userId;