build(csfixer): Bump CS Fixer to 3.82

composer.json

@@ -24,6 +24,7 @@
 		"test:integration": "cd tests/Integration && ./run.sh"
 	},
 	"require-dev": {
+		"friendsofphp/php-cs-fixer": "^3.82.0",
 		"nextcloud/coding-standard": "^1.2.1",
 		"nextcloud/ocp": "dev-master",
 		"phpunit/phpunit": "^10.5.28",

lib/AppInfo/Application.php

@@ -54,11 +54,13 @@ public function addStorageWrapperCallback($mountPoint, IStorage $storage) {
 		return $storage;
 	}
 
+	#[\Override]
 	public function register(IRegistrationContext $context): void {
 		Util::connectHook('OC_Filesystem', 'preSetup', $this, 'addStorageWrapper');
 		$context->registerEventListener(RegisterOperationsEvent::class, FlowRegisterOperationListener::class);
 	}
 
+	#[\Override]
 	public function boot(IBootContext $context): void {
 	}
 }

lib/CacheWrapper.php

@@ -30,6 +30,7 @@ public function __construct(
 			& ~Constants::PERMISSION_DELETE;
 	}
 
+	#[\Override]
 	protected function formatCacheEntry($entry) {
 		if (isset($entry['path']) && isset($entry['permissions'])) {
 			try {

lib/Listener/FlowRegisterOperationListener.php

@@ -23,6 +23,7 @@ public function __construct(
 	) {
 	}
 
+	#[\Override]
 	public function handle(Event $event): void {
 		if (!$event instanceof RegisterOperationsEvent) {
 			return;

lib/Operation.php

@@ -174,6 +174,7 @@ protected function isCreatingSkeletonFiles(): bool {
 	 * @param string $operation
 	 * @throws UnexpectedValueException
 	 */
+	#[\Override]
 	public function validateOperation(string $name, array $checks, string $operation): void {
 		if (empty($checks)) {
 			throw new UnexpectedValueException($this->l->t('No rule given'));
@@ -187,6 +188,7 @@ public function validateOperation(string $name, array $checks, string $operation
 	 *
 	 * @since 18.0.0
 	 */
+	#[\Override]
 	public function getDisplayName(): string {
 		return $this->l->t('Block access to a file');
 	}
@@ -200,6 +202,7 @@ public function getDisplayName(): string {
 	 *
 	 * @since 18.0.0
 	 */
+	#[\Override]
 	public function getDescription(): string {
 		return '';
 	}
@@ -214,6 +217,7 @@ public function getDescription(): string {
 	 *
 	 * @since 18.0.0
 	 */
+	#[\Override]
 	public function getIcon(): string {
 		return $this->urlGenerator->imagePath('files_accesscontrol', 'app.svg');
 	}
@@ -230,6 +234,7 @@ public function getIcon(): string {
 	 *
 	 * @since 18.0.0
 	 */
+	#[\Override]
 	public function isAvailableForScope(int $scope): bool {
 		return $scope === IManager::SCOPE_ADMIN;
 	}
@@ -241,6 +246,7 @@ public function isAvailableForScope(int $scope): bool {
 	 *
 	 * @since 18.0.0
 	 */
+	#[\Override]
 	public function getEntityId(): string {
 		return File::class;
 	}
@@ -254,10 +260,12 @@ public function getEntityId(): string {
 	 *
 	 * @since 18.0.0
 	 */
+	#[\Override]
 	public function getTriggerHint(): string {
 		return $this->l->t('File is accessed');
 	}
 
+	#[\Override]
 	public function onEvent(string $eventName, Event $event, IRuleMatcher $ruleMatcher): void {
 		// Noop
 	}

lib/StorageWrapper.php

@@ -55,6 +55,7 @@ protected function checkFileAccess(string $path, ?bool $isDir = null): void {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function mkdir($path): bool {
 		$this->checkFileAccess($path, true);
 		return $this->storage->mkdir($path);
@@ -67,6 +68,7 @@ public function mkdir($path): bool {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function rmdir($path): bool {
 		$this->checkFileAccess($path, true);
 		return $this->storage->rmdir($path);
@@ -78,6 +80,7 @@ public function rmdir($path): bool {
 	 * @param string $path
 	 * @return bool
 	 */
+	#[\Override]
 	public function isCreatable($path): bool {
 		try {
 			$this->checkFileAccess($path);
@@ -93,6 +96,7 @@ public function isCreatable($path): bool {
 	 * @param string $path
 	 * @return bool
 	 */
+	#[\Override]
 	public function isReadable($path): bool {
 		try {
 			$this->checkFileAccess($path);
@@ -108,6 +112,7 @@ public function isReadable($path): bool {
 	 * @param string $path
 	 * @return bool
 	 */
+	#[\Override]
 	public function isUpdatable($path): bool {
 		try {
 			$this->checkFileAccess($path);
@@ -123,6 +128,7 @@ public function isUpdatable($path): bool {
 	 * @param string $path
 	 * @return bool
 	 */
+	#[\Override]
 	public function isDeletable($path): bool {
 		try {
 			$this->checkFileAccess($path);
@@ -132,6 +138,7 @@ public function isDeletable($path): bool {
 		return $this->storage->isDeletable($path);
 	}
 
+	#[\Override]
 	public function getPermissions($path): int {
 		try {
 			$this->checkFileAccess($path);
@@ -148,6 +155,7 @@ public function getPermissions($path): int {
 	 * @return string
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function file_get_contents($path): string|false {
 		$this->checkFileAccess($path, false);
 		return $this->storage->file_get_contents($path);
@@ -161,6 +169,7 @@ public function file_get_contents($path): string|false {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function file_put_contents($path, $data): int|float|false {
 		$this->checkFileAccess($path, false);
 		return $this->storage->file_put_contents($path, $data);
@@ -173,6 +182,7 @@ public function file_put_contents($path, $data): int|float|false {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function unlink($path): bool {
 		$this->checkFileAccess($path, false);
 		return $this->storage->unlink($path);
@@ -186,6 +196,7 @@ public function unlink($path): bool {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function rename($source, $target): bool {
 		$isDir = $this->is_dir($source);
 		$this->checkFileAccess($source, $isDir);
@@ -201,6 +212,7 @@ public function rename($source, $target): bool {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function copy($source, $target): bool {
 		$isDir = $this->is_dir($source);
 		$this->checkFileAccess($source, $isDir);
@@ -216,6 +228,7 @@ public function copy($source, $target): bool {
 	 * @return resource
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function fopen($path, $mode) {
 		$this->checkFileAccess($path, false);
 		return $this->storage->fopen($path, $mode);
@@ -230,6 +243,7 @@ public function fopen($path, $mode) {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function touch($path, $mtime = null): bool {
 		$this->checkFileAccess($path, false);
 		return $this->storage->touch($path, $mtime);
@@ -242,6 +256,7 @@ public function touch($path, $mtime = null): bool {
 	 * @param Storage (optional) the storage to pass to the cache
 	 * @return Cache
 	 */
+	#[\Override]
 	public function getCache($path = '', $storage = null): ICache {
 		if (!$storage) {
 			$storage = $this;
@@ -259,6 +274,7 @@ public function getCache($path = '', $storage = null): ICache {
 	 * @return array
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function getDirectDownload($path): array|false {
 		$this->checkFileAccess($path, false);
 		return $this->storage->getDirectDownload($path);
@@ -271,6 +287,7 @@ public function getDirectDownload($path): array|false {
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function copyFromStorage(IStorage $sourceStorage, $sourceInternalPath, $targetInternalPath): bool {
 		if ($sourceStorage === $this) {
 			return $this->copy($sourceInternalPath, $targetInternalPath);
@@ -287,6 +304,7 @@ public function copyFromStorage(IStorage $sourceStorage, $sourceInternalPath, $t
 	 * @return bool
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function moveFromStorage(IStorage $sourceStorage, $sourceInternalPath, $targetInternalPath): bool {
 		if ($sourceStorage === $this) {
 			return $this->rename($sourceInternalPath, $targetInternalPath);
@@ -299,6 +317,7 @@ public function moveFromStorage(IStorage $sourceStorage, $sourceInternalPath, $t
 	/**
 	 * @throws ForbiddenException
 	 */
+	#[\Override]
 	public function writeStream(string $path, $stream, ?int $size = null): int {
 		if (!$this->isPartFile($path)) {
 			$this->checkFileAccess($path, false);