chore(deps): bump nextcloud/coding-standard from 1.3.1 to ^1.3.2 (mai…

…n) (#10283)

chore(deps): bump nextcloud/coding-standard from 1.3.1 to ^1.3.2

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

lib/Controller/AliasesController.php

@@ -60,10 +60,10 @@ public function show() {
 	 * @NoAdminRequired
 	 */
 	#[TrapError]
-	public function update(int    $id,
+	public function update(int $id,
 		string $alias,
 		string $aliasName,
-		?int   $smimeCertificateId = null): JSONResponse {
+		?int $smimeCertificateId = null): JSONResponse {
 		return new JSONResponse(
 			$this->aliasService->update(
 				$this->currentUserId,

lib/Controller/DraftsController.php

@@ -74,18 +74,18 @@ public function __construct(string $appName,
 	 */
 	#[TrapError]
 	public function create(
-		int     $accountId,
-		string  $subject,
-		string  $body,
-		string  $editorBody,
-		bool    $isHtml,
+		int $accountId,
+		string $subject,
+		string $body,
+		string $editorBody,
+		bool $isHtml,
 		?bool $smimeSign,
 		?bool $smimeEncrypt,
-		array   $to = [],
-		array   $cc = [],
-		array   $bcc = [],
-		array   $attachments = [],
-		?int    $aliasId = null,
+		array $to = [],
+		array $cc = [],
+		array $bcc = [],
+		array $attachments = [],
+		?int $aliasId = null,
 		?string $inReplyToMessageId = null,
 		?int $smimeCertificateId = null,
 		?int $sendAt = null,
@@ -144,19 +144,19 @@ public function create(
 	 */
 	#[TrapError]
 	public function update(int $id,
-		int     $accountId,
-		string  $subject,
-		string  $body,
-		string  $editorBody,
-		bool    $isHtml,
+		int $accountId,
+		string $subject,
+		string $body,
+		string $editorBody,
+		bool $isHtml,
 		?bool $smimeSign,
 		?bool $smimeEncrypt,
-		bool    $failed = false,
-		array   $to = [],
-		array   $cc = [],
-		array   $bcc = [],
-		array   $attachments = [],
-		?int    $aliasId = null,
+		bool $failed = false,
+		array $to = [],
+		array $cc = [],
+		array $bcc = [],
+		array $attachments = [],
+		?int $aliasId = null,
 		?string $inReplyToMessageId = null,
 		?int $smimeCertificateId = null,
 		?int $sendAt = null,

lib/Controller/InternalAddressController.php

@@ -22,8 +22,8 @@ class InternalAddressController extends Controller {
 	private ?string $uid;
 
 	public function __construct(
-		IRequest        $request,
-		?string                                 $userId,
+		IRequest $request,
+		?string $userId,
 		private InternalAddressService $internalAddressService,
 	) {
 		parent::__construct(Application::APP_ID, $request);

lib/Controller/PageController.php

@@ -79,17 +79,17 @@ public function __construct(string $appName,
 		IConfig $config,
 		AccountService $accountService,
 		AliasesService $aliasesService,
-		?string          $UserId,
-		IUserSession     $userSession,
+		?string $UserId,
+		IUserSession $userSession,
 		IUserPreferences $preferences,
-		IMailManager     $mailManager,
-		TagMapper        $tagMapper,
-		IInitialState    $initialStateService,
-		LoggerInterface  $logger,
-		OutboxService    $outboxService,
+		IMailManager $mailManager,
+		TagMapper $tagMapper,
+		IInitialState $initialStateService,
+		LoggerInterface $logger,
+		OutboxService $outboxService,
 		IEventDispatcher $dispatcher,
 		ICredentialStore $credentialStore,
-		SmimeService     $smimeService,
+		SmimeService $smimeService,
 		AiIntegrationsService $aiIntegrationsService,
 		IUserManager $userManager,
 		ContainerInterface $container,

lib/Controller/SmimeCertificatesController.php

@@ -27,9 +27,9 @@ class SmimeCertificatesController extends Controller {
 	private ?string $userId;
 	private SmimeService $certificateService;
 
-	public function __construct(string       $appName,
-		IRequest     $request,
-		?string      $userId,
+	public function __construct(string $appName,
+		IRequest $request,
+		?string $userId,
 		SmimeService $certificateService) {
 		parent::__construct($appName, $request);
 		$this->userId = $userId;

lib/IMAP/ImapMessageFetcherFactory.php

@@ -21,7 +21,7 @@ class ImapMessageFetcherFactory {
 	private Converter $charsetConverter;
 	private PhishingDetectionService $phishingDetectionService;
 
-	public function __construct(Html         $htmlService,
+	public function __construct(Html $htmlService,
 		SmimeService $smimeService,
 		Converter $charsetConverter,
 		PhishingDetectionService $phishingDetectionService) {

lib/IMAP/MessageMapper.php

@@ -50,8 +50,8 @@ class MessageMapper {
 	private SMimeService $smimeService;
 	private ImapMessageFetcherFactory $imapMessageFactory;
 
-	public function __construct(LoggerInterface           $logger,
-		SmimeService              $smimeService,
+	public function __construct(LoggerInterface $logger,
+		SmimeService $smimeService,
 		ImapMessageFetcherFactory $imapMessageFactory, ) {
 		$this->logger = $logger;
 		$this->smimeService = $smimeService;
@@ -512,9 +512,9 @@ public function getFullText(Horde_Imap_Client_Socket $client,
 	 * @throws ServiceException
 	 */
 	public function getHtmlBody(Horde_Imap_Client_Socket $client,
-		string                   $mailbox,
-		int                      $uid,
-		string                   $userId): ?string {
+		string $mailbox,
+		int $uid,
+		string $userId): ?string {
 		$messageQuery = new Horde_Imap_Client_Fetch_Query();
 		$messageQuery->envelope();
 		$messageQuery->structure();

lib/Model/RepliedMessageData.php

@@ -24,7 +24,7 @@ class RepliedMessageData {
 	/** @var Message */
 	private $message;
 
-	public function __construct(Account $account, Message  $message) {
+	public function __construct(Account $account, Message $message) {
 		$this->account = $account;
 		$this->message = $message;
 	}

lib/Service/DraftsService.php

@@ -37,10 +37,10 @@ class DraftsService {
 	private AccountService $accountService;
 	private ITimeFactory $time;
 
-	public function __construct(IMailTransmission  $transmission,
+	public function __construct(IMailTransmission $transmission,
 		LocalMessageMapper $mapper,
-		AttachmentService  $attachmentService,
-		IEventDispatcher    $eventDispatcher,
+		AttachmentService $attachmentService,
+		IEventDispatcher $eventDispatcher,
 		IMAPClientFactory $clientFactory,
 		IMailManager $mailManager,
 		LoggerInterface $logger,

lib/Service/OutboxService.php

@@ -57,10 +57,10 @@ class OutboxService {
 	private $logger;
 
 	public function __construct(
-		IMailTransmission  $transmission,
+		IMailTransmission $transmission,
 		LocalMessageMapper $mapper,
-		AttachmentService  $attachmentService,
-		IEventDispatcher    $eventDispatcher,
+		AttachmentService $attachmentService,
+		IEventDispatcher $eventDispatcher,
 		IMAPClientFactory $clientFactory,
 		IMailManager $mailManager,
 		AccountService $accountService,

lib/Service/SmimeService.php

@@ -43,11 +43,11 @@ class SmimeService {
 	private SmimeCertificateMapper $certificateMapper;
 	private ITimeFactory $timeFactory;
 
-	public function __construct(ITempManager           $tempManager,
-		ICertificateManager    $certificateManager,
-		ICrypto                $crypto,
+	public function __construct(ITempManager $tempManager,
+		ICertificateManager $certificateManager,
+		ICrypto $crypto,
 		SmimeCertificateMapper $certificateMapper,
-		ITimeFactory           $timeFactory) {
+		ITimeFactory $timeFactory) {
 		$this->tempManager = $tempManager;
 		$this->certificateManager = $certificateManager;
 		$this->crypto = $crypto;
@@ -346,7 +346,7 @@ public function createCertificate(string $userId,
 	 * @throws SmimeSignException If signing the message fails
 	 * @throws ServiceException If decrypting the certificate or private key fails or the private key is missing
 	 */
-	public function signMimePart(Horde_Mime_Part  $part,
+	public function signMimePart(Horde_Mime_Part $part,
 		SmimeCertificate $certificate): Horde_Mime_Part {
 		if ($certificate->getPrivateKey() === null) {
 			throw new ServiceException('Certificate does not have a private key');
@@ -572,7 +572,7 @@ public function isEncrypted(Horde_Imap_Client_Data_Fetch $message): bool {
 	 * @throws ServiceException If decrypting the certificates fails
 	 * @throws SmimeEncryptException If encrypting the message fails
 	 */
-	public function encryptMimePart(Horde_Mime_Part  $part, array $certificates): Horde_Mime_Part {
+	public function encryptMimePart(Horde_Mime_Part $part, array $certificates): Horde_Mime_Part {
 		try {
 			$decryptedCertificates = array_map(function (SmimeCertificate $certificate) {
 				return $this->crypto->decrypt($certificate->getCertificate());

lib/Service/Sync/ImapToDbSynchronizer.php

@@ -304,7 +304,7 @@ private function runInitialSync(
 		Horde_Imap_Client_Base $client,
 		Account $account,
 		Mailbox $mailbox,
-		LoggerInterface  $logger): void {
+		LoggerInterface $logger): void {
 		$perf = $this->performanceLogger->startWithLogger(
 			'Initial sync ' . $account->getId() . ':' . $mailbox->getName(),
 			$logger
@@ -507,7 +507,7 @@ private function runPartialSync(
 	public function repairSync(
 		Account $account,
 		Mailbox $mailbox,
-		LoggerInterface  $logger,
+		LoggerInterface $logger,
 	): void {
 		$this->mailboxMapper->lockForVanishedSync($mailbox);
 

vendor-bin/cs-fixer/composer.json

@@ -6,6 +6,6 @@
 		"sort-packages": true
 	},
     "require-dev": {
-        "nextcloud/coding-standard": "^1.3.1"
+        "nextcloud/coding-standard": "^1.3.2"
     }
 }