nextcloud / server

apps/files_sharing/tests/Controller/ShareControllerTest.php

Indentation +770 added lines, -770 removed lines

@@ -56,774 +56,774 @@
  */
 class ShareControllerTest extends \Test\TestCase {
 
-	private string $user;
-	private string $oldUser;
-	private string $appName = 'files_sharing';
-	private ShareController $shareController;
-
-	private IL10N&MockObject $l10n;
-	private IConfig&MockObject $config;
-	private ISession&MockObject $session;
-	private Defaults&MockObject $defaults;
-	private IAppConfig&MockObject $appConfig;
-	private Manager&MockObject $shareManager;
-	private IPreview&MockObject $previewManager;
-	private IUserManager&MockObject $userManager;
-	private ITemplateManager&MockObject $templateManager;
-	private IInitialState&MockObject $initialState;
-	private IURLGenerator&MockObject $urlGenerator;
-	private ISecureRandom&MockObject $secureRandom;
-	private IAccountManager&MockObject $accountManager;
-	private IEventDispatcher&MockObject $eventDispatcher;
-	private FederatedShareProvider&MockObject $federatedShareProvider;
-	private IPublicShareTemplateFactory&MockObject $publicShareTemplateFactory;
-
-	protected function setUp(): void {
-		parent::setUp();
-		$this->appName = 'files_sharing';
-
-		$this->shareManager = $this->createMock(Manager::class);
-		$this->urlGenerator = $this->createMock(IURLGenerator::class);
-		$this->session = $this->createMock(ISession::class);
-		$this->previewManager = $this->createMock(IPreview::class);
-		$this->config = $this->createMock(IConfig::class);
-		$this->appConfig = $this->createMock(IAppConfig::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->templateManager = $this->createMock(ITemplateManager::class);
-		$this->initialState = $this->createMock(IInitialState::class);
-		$this->federatedShareProvider = $this->createMock(FederatedShareProvider::class);
-		$this->federatedShareProvider->expects($this->any())
-			->method('isOutgoingServer2serverShareEnabled')->willReturn(true);
-		$this->federatedShareProvider->expects($this->any())
-			->method('isIncomingServer2serverShareEnabled')->willReturn(true);
-		$this->accountManager = $this->createMock(IAccountManager::class);
-		$this->eventDispatcher = $this->createMock(IEventDispatcher::class);
-		$this->l10n = $this->createMock(IL10N::class);
-		$this->secureRandom = $this->createMock(ISecureRandom::class);
-		$this->defaults = $this->createMock(Defaults::class);
-		$this->publicShareTemplateFactory = $this->createMock(IPublicShareTemplateFactory::class);
-		$this->publicShareTemplateFactory
-			->expects($this->any())
-			->method('getProvider')
-			->willReturn(
-				new DefaultPublicShareTemplateProvider(
-					$this->userManager,
-					$this->accountManager,
-					$this->previewManager,
-					$this->federatedShareProvider,
-					$this->urlGenerator,
-					$this->eventDispatcher,
-					$this->l10n,
-					$this->defaults,
-					$this->config,
-					$this->createMock(IRequest::class),
-					$this->templateManager,
-					$this->initialState,
-					$this->appConfig,
-				)
-			);
-
-		$this->shareController = new ShareController(
-			$this->appName,
-			$this->createMock(IRequest::class),
-			$this->config,
-			$this->urlGenerator,
-			$this->userManager,
-			$this->createMock(IManager::class),
-			$this->shareManager,
-			$this->session,
-			$this->previewManager,
-			$this->createMock(IRootFolder::class),
-			$this->federatedShareProvider,
-			$this->accountManager,
-			$this->eventDispatcher,
-			$this->l10n,
-			$this->secureRandom,
-			$this->defaults,
-			$this->publicShareTemplateFactory,
-		);
-
-
-		// Store current user
-		$this->oldUser = \OC_User::getUser();
-
-		// Create a dummy user
-		$this->user = Server::get(ISecureRandom::class)->generate(12, ISecureRandom::CHAR_LOWER);
-
-		Server::get(IUserManager::class)->createUser($this->user, $this->user);
-		\OC_Util::tearDownFS();
-		$this->loginAsUser($this->user);
-	}
-
-	protected function tearDown(): void {
-		\OC_Util::tearDownFS();
-		\OC_User::setUserId('');
-		Filesystem::tearDown();
-		$user = Server::get(IUserManager::class)->get($this->user);
-		if ($user !== null) {
-			$user->delete();
-		}
-		\OC_User::setIncognitoMode(false);
-
-		Server::get(ISession::class)->set('public_link_authenticated', '');
-
-		// Set old user
-		\OC_User::setUserId($this->oldUser);
-		\OC_Util::setupFS($this->oldUser);
-		parent::tearDown();
-	}
-
-	public function testShowShareInvalidToken(): void {
-		$this->shareController->setToken('invalidtoken');
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('invalidtoken')
-			->willThrowException(new ShareNotFound());
-
-		$this->expectException(NotFoundException::class);
-
-		// Test without a not existing token
-		$this->shareController->showShare();
-	}
-
-	public function testShowShareNotAuthenticated(): void {
-		$this->shareController->setToken('validtoken');
-
-		$share = Server::get(\OCP\Share\IManager::class)->newShare();
-		$share->setPassword('password');
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('validtoken')
-			->willReturn($share);
-
-		$this->expectException(NotFoundException::class);
-
-		// Test without a not existing token
-		$this->shareController->showShare();
-	}
-
-
-	public function testShowShare(): void {
-		$note = 'personal note';
-		$filename = 'file1.txt';
-
-		$this->shareController->setToken('token');
-
-		$owner = $this->createMock(IUser::class);
-		$owner->method('getDisplayName')->willReturn('ownerDisplay');
-		$owner->method('getUID')->willReturn('ownerUID');
-		$owner->method('isEnabled')->willReturn(true);
-
-		$initiator = $this->createMock(IUser::class);
-		$initiator->method('getDisplayName')->willReturn('initiatorDisplay');
-		$initiator->method('getUID')->willReturn('initiatorUID');
-		$initiator->method('isEnabled')->willReturn(true);
-
-		$file = $this->createMock(File::class);
-		$file->method('getName')->willReturn($filename);
-		$file->method('getMimetype')->willReturn('text/plain');
-		$file->method('getSize')->willReturn(33);
-		$file->method('isReadable')->willReturn(true);
-		$file->method('isShareable')->willReturn(true);
-		$file->method('getId')->willReturn(111);
-
-		$accountName = $this->createMock(IAccountProperty::class);
-		$accountName->method('getScope')
-			->willReturn(IAccountManager::SCOPE_PUBLISHED);
-		$account = $this->createMock(IAccount::class);
-		$account->method('getProperty')
-			->with(IAccountManager::PROPERTY_DISPLAYNAME)
-			->willReturn($accountName);
-		$this->accountManager->expects($this->once())
-			->method('getAccount')
-			->with($owner)
-			->willReturn($account);
-
-		/** @var Manager */
-		$manager = Server::get(Manager::class);
-		$share = $manager->newShare();
-		$share->setId(42)
-			->setPermissions(Constants::PERMISSION_READ | Constants::PERMISSION_UPDATE)
-			->setPassword('password')
-			->setShareOwner('ownerUID')
-			->setSharedBy('initiatorUID')
-			->setNode($file)
-			->setNote($note)
-			->setTarget("/$filename")
-			->setToken('token');
-
-		$this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
-		$this->session->method('get')->with('public_link_authenticated')->willReturn('42');
-
-		$this->urlGenerator->expects(self::atLeast(2))
-			->method('linkToRouteAbsolute')
-			->willReturnMap([
-				// every file has the show show share url in the opengraph url prop
-				['files_sharing.sharecontroller.showShare', ['token' => 'token'], 'shareUrl'],
-				// this share is not an image to the default preview is used
-				['files_sharing.PublicPreview.getPreview', ['x' => 256, 'y' => 256, 'file' => $share->getTarget(), 'token' => 'token'], 'previewUrl'],
-			]);
-
-		$this->urlGenerator->expects($this->once())
-			->method('getAbsoluteURL')
-			->willReturnMap([
-				['/public.php/dav/files/token/?accept=zip', 'downloadUrl'],
-			]);
-
-		$this->previewManager->method('isMimeSupported')->with('text/plain')->willReturn(true);
-
-		$this->config->method('getSystemValue')
-			->willReturnMap(
-				[
-					['max_filesize_animated_gifs_public_sharing', 10, 10],
-					['enable_previews', true, true],
-					['preview_max_x', 1024, 1024],
-					['preview_max_y', 1024, 1024],
-				]
-			);
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('token')
-			->willReturn($share);
-
-		$this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
-			if ($uid === 'ownerUID') {
-				return $owner;
-			}
-			if ($uid === 'initiatorUID') {
-				return $initiator;
-			}
-			return null;
-		});
-
-		$this->eventDispatcher->method('dispatchTyped')->with(
-			$this->callback(function ($event) use ($share) {
-				if ($event instanceof BeforeTemplateRenderedEvent) {
-					return $event->getShare() === $share;
-				} else {
-					return true;
-				}
-			})
-		);
-
-		$this->l10n->expects($this->any())
-			->method('t')
-			->willReturnCallback(function ($text, $parameters) {
-				return vsprintf($text, $parameters);
-			});
-
-		$this->defaults->expects(self::any())
-			->method('getProductName')
-			->willReturn('Nextcloud');
-
-		// Ensure the correct initial state is setup
-		// Shared node is a file so this is a single file share:
-		$view = 'public-file-share';
-		// Set up initial state
-		$initialState = [];
-		$this->initialState->expects(self::any())
-			->method('provideInitialState')
-			->willReturnCallback(function ($key, $value) use (&$initialState): void {
-				$initialState[$key] = $value;
-			});
-		$expectedInitialState = [
-			'isPublic' => true,
-			'sharingToken' => 'token',
-			'sharePermissions' => (Constants::PERMISSION_READ | Constants::PERMISSION_UPDATE),
-			'filename' => $filename,
-			'view' => $view,
-			'fileId' => 111,
-			'owner' => 'ownerUID',
-			'ownerDisplayName' => 'ownerDisplay',
-			'isFileRequest' => false,
-			'templates' => [],
-		];
-
-		$response = $this->shareController->showShare();
-
-		$this->assertEquals($expectedInitialState, $initialState);
-
-		$csp = new ContentSecurityPolicy();
-		$csp->addAllowedFrameDomain('\'self\'');
-		$expectedResponse = new PublicTemplateResponse('files', 'index');
-		$expectedResponse->setParams(['pageTitle' => $filename]);
-		$expectedResponse->setContentSecurityPolicy($csp);
-		$expectedResponse->setHeaderTitle($filename);
-		$expectedResponse->setHeaderDetails('shared by ownerDisplay');
-		$expectedResponse->setHeaderActions([
-			new SimpleMenuAction('download', $this->l10n->t('Download'), 'icon-download', 'downloadUrl', 0, '33'),
-			new ExternalShareMenuAction($this->l10n->t('Add to your Nextcloud'), 'icon-external', 'owner', 'ownerDisplay', $filename),
-			new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', 'downloadUrl'),
-		]);
-
-		$this->assertEquals($expectedResponse, $response);
-	}
-
-	public function testShowFileDropShare(): void {
-		$filename = 'folder1';
-
-		$this->shareController->setToken('token');
-
-		$owner = $this->createMock(IUser::class);
-		$owner->method('getDisplayName')->willReturn('ownerDisplay');
-		$owner->method('getUID')->willReturn('ownerUID');
-		$owner->method('isEnabled')->willReturn(true);
-
-		$initiator = $this->createMock(IUser::class);
-		$initiator->method('getDisplayName')->willReturn('initiatorDisplay');
-		$initiator->method('getUID')->willReturn('initiatorUID');
-		$initiator->method('isEnabled')->willReturn(true);
-
-		$file = $this->createMock(Folder::class);
-		$file->method('isReadable')->willReturn(true);
-		$file->method('isShareable')->willReturn(true);
-		$file->method('getId')->willReturn(1234);
-		$file->method('getMimetype')->willReturn('text/plain');
-		$file->method('getName')->willReturn($filename);
-
-		$accountName = $this->createMock(IAccountProperty::class);
-		$accountName->method('getScope')
-			->willReturn(IAccountManager::SCOPE_PUBLISHED);
-		$account = $this->createMock(IAccount::class);
-		$account->method('getProperty')
-			->with(IAccountManager::PROPERTY_DISPLAYNAME)
-			->willReturn($accountName);
-		$this->accountManager->expects($this->once())
-			->method('getAccount')
-			->with($owner)
-			->willReturn($account);
-
-		/** @var Manager */
-		$manager = Server::get(Manager::class);
-		$share = $manager->newShare();
-		$share->setId(42)
-			->setPermissions(Constants::PERMISSION_CREATE)
-			->setPassword('password')
-			->setShareOwner('ownerUID')
-			->setSharedBy('initiatorUID')
-			->setNote('The note')
-			->setLabel('A label')
-			->setNode($file)
-			->setTarget("/$filename")
-			->setToken('token');
-
-		$this->appConfig
-			->expects($this->once())
-			->method('getValueString')
-			->with('core', 'shareapi_public_link_disclaimertext', '')
-			->willReturn('My disclaimer text');
-
-		$this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
-		$this->session->method('get')->with('public_link_authenticated')->willReturn('42');
-
-		$this->urlGenerator->expects(self::atLeastOnce())
-			->method('linkToRouteAbsolute')
-			->willReturnMap([
-				// every file has the show show share url in the opengraph url prop
-				['files_sharing.sharecontroller.showShare', ['token' => 'token'], 'shareUrl'],
-				// there is no preview or folders so no other link for opengraph
-			]);
-
-		$this->config->method('getSystemValue')
-			->willReturnMap(
-				[
-					['max_filesize_animated_gifs_public_sharing', 10, 10],
-					['enable_previews', true, true],
-					['preview_max_x', 1024, 1024],
-					['preview_max_y', 1024, 1024],
-				]
-			);
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('token')
-			->willReturn($share);
-
-		$this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
-			if ($uid === 'ownerUID') {
-				return $owner;
-			}
-			if ($uid === 'initiatorUID') {
-				return $initiator;
-			}
-			return null;
-		});
-
-		$this->eventDispatcher->method('dispatchTyped')->with(
-			$this->callback(function ($event) use ($share) {
-				if ($event instanceof BeforeTemplateRenderedEvent) {
-					return $event->getShare() === $share;
-				} else {
-					return true;
-				}
-			})
-		);
-
-		$this->l10n->expects($this->any())
-			->method('t')
-			->willReturnCallback(function ($text, $parameters) {
-				return vsprintf($text, $parameters);
-			});
-
-		// Set up initial state
-		$initialState = [];
-		$this->initialState->expects(self::any())
-			->method('provideInitialState')
-			->willReturnCallback(function ($key, $value) use (&$initialState): void {
-				$initialState[$key] = $value;
-			});
-		$expectedInitialState = [
-			'isPublic' => true,
-			'sharingToken' => 'token',
-			'sharePermissions' => Constants::PERMISSION_CREATE,
-			'filename' => $filename,
-			'view' => 'public-file-drop',
-			'disclaimer' => 'My disclaimer text',
-			'owner' => 'ownerUID',
-			'ownerDisplayName' => 'ownerDisplay',
-			'isFileRequest' => false,
-			'note' => 'The note',
-			'label' => 'A label',
-			'templates' => [],
-		];
-
-		$response = $this->shareController->showShare();
-
-		$this->assertEquals($expectedInitialState, $initialState);
-
-		$csp = new ContentSecurityPolicy();
-		$csp->addAllowedFrameDomain('\'self\'');
-		$expectedResponse = new PublicTemplateResponse('files', 'index');
-		$expectedResponse->setParams(['pageTitle' => 'A label']);
-		$expectedResponse->setContentSecurityPolicy($csp);
-		$expectedResponse->setHeaderTitle('A label');
-		$expectedResponse->setHeaderDetails('shared by ownerDisplay');
-		$expectedResponse->setHeaderActions([
-			new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', 'shareUrl'),
-		]);
-
-		$this->assertEquals($expectedResponse, $response);
-	}
-
-	public function testShowShareWithPrivateName(): void {
-		$note = 'personal note';
-		$filename = 'file1.txt';
-
-		$this->shareController->setToken('token');
-
-		$owner = $this->createMock(IUser::class);
-		$owner->method('getDisplayName')->willReturn('ownerDisplay');
-		$owner->method('getUID')->willReturn('ownerUID');
-		$owner->method('isEnabled')->willReturn(true);
-
-		$initiator = $this->createMock(IUser::class);
-		$initiator->method('getDisplayName')->willReturn('initiatorDisplay');
-		$initiator->method('getUID')->willReturn('initiatorUID');
-		$initiator->method('isEnabled')->willReturn(true);
-
-		$file = $this->createMock(File::class);
-		$file->method('getName')->willReturn($filename);
-		$file->method('getMimetype')->willReturn('text/plain');
-		$file->method('getSize')->willReturn(33);
-		$file->method('isReadable')->willReturn(true);
-		$file->method('isShareable')->willReturn(true);
-		$file->method('getId')->willReturn(111);
-
-		$accountName = $this->createMock(IAccountProperty::class);
-		$accountName->method('getScope')
-			->willReturn(IAccountManager::SCOPE_LOCAL);
-		$account = $this->createMock(IAccount::class);
-		$account->method('getProperty')
-			->with(IAccountManager::PROPERTY_DISPLAYNAME)
-			->willReturn($accountName);
-		$this->accountManager->expects($this->once())
-			->method('getAccount')
-			->with($owner)
-			->willReturn($account);
-
-		/** @var IShare */
-		$share = Server::get(Manager::class)->newShare();
-		$share->setId(42);
-		$share->setPassword('password')
-			->setShareOwner('ownerUID')
-			->setSharedBy('initiatorUID')
-			->setNode($file)
-			->setNote($note)
-			->setToken('token')
-			->setPermissions(Constants::PERMISSION_ALL & ~Constants::PERMISSION_SHARE)
-			->setTarget("/$filename");
-
-		$this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
-		$this->session->method('get')->with('public_link_authenticated')->willReturn('42');
-
-		$this->urlGenerator->expects(self::atLeast(2))
-			->method('linkToRouteAbsolute')
-			->willReturnMap([
-				// every file has the show show share url in the opengraph url prop
-				['files_sharing.sharecontroller.showShare', ['token' => 'token'], 'shareUrl'],
-				// this share is not an image to the default preview is used
-				['files_sharing.PublicPreview.getPreview', ['x' => 256, 'y' => 256, 'file' => $share->getTarget(), 'token' => 'token'], 'previewUrl'],
-			]);
-
-		$this->urlGenerator->expects($this->once())
-			->method('getAbsoluteURL')
-			->willReturnMap([
-				['/public.php/dav/files/token/?accept=zip', 'downloadUrl'],
-			]);
-
-		$this->previewManager->method('isMimeSupported')->with('text/plain')->willReturn(true);
-
-		$this->config->method('getSystemValue')
-			->willReturnMap(
-				[
-					['max_filesize_animated_gifs_public_sharing', 10, 10],
-					['enable_previews', true, true],
-					['preview_max_x', 1024, 1024],
-					['preview_max_y', 1024, 1024],
-				]
-			);
-		$shareTmpl['maxSizeAnimateGif'] = $this->config->getSystemValue('max_filesize_animated_gifs_public_sharing', 10);
-		$shareTmpl['previewEnabled'] = $this->config->getSystemValue('enable_previews', true);
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('token')
-			->willReturn($share);
-
-		$this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
-			if ($uid === 'ownerUID') {
-				return $owner;
-			}
-			if ($uid === 'initiatorUID') {
-				return $initiator;
-			}
-			return null;
-		});
-
-		$this->eventDispatcher->method('dispatchTyped')->with(
-			$this->callback(function ($event) use ($share) {
-				if ($event instanceof BeforeTemplateRenderedEvent) {
-					return $event->getShare() === $share;
-				} else {
-					return true;
-				}
-			})
-		);
-
-		$this->l10n->expects($this->any())
-			->method('t')
-			->willReturnCallback(function ($text, $parameters) {
-				return vsprintf($text, $parameters);
-			});
-
-		$this->defaults->expects(self::any())
-			->method('getProductName')
-			->willReturn('Nextcloud');
-
-		$response = $this->shareController->showShare();
-
-		$csp = new ContentSecurityPolicy();
-		$csp->addAllowedFrameDomain('\'self\'');
-		$expectedResponse = new PublicTemplateResponse('files', 'index');
-		$expectedResponse->setParams(['pageTitle' => $filename]);
-		$expectedResponse->setContentSecurityPolicy($csp);
-		$expectedResponse->setHeaderTitle($filename);
-		$expectedResponse->setHeaderDetails('');
-		$expectedResponse->setHeaderActions([
-			new SimpleMenuAction('download', $this->l10n->t('Download'), 'icon-download', 'downloadUrl', 0, '33'),
-			new ExternalShareMenuAction($this->l10n->t('Add to your Nextcloud'), 'icon-external', 'owner', 'ownerDisplay', $filename),
-			new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', 'downloadUrl'),
-		]);
-
-		$this->assertEquals($expectedResponse, $response);
-	}
-
-
-	public function testShowShareInvalid(): void {
-		$this->expectException(NotFoundException::class);
-
-		$filename = 'file1.txt';
-		$this->shareController->setToken('token');
-
-		$owner = $this->getMockBuilder(IUser::class)->getMock();
-		$owner->method('getDisplayName')->willReturn('ownerDisplay');
-		$owner->method('getUID')->willReturn('ownerUID');
-
-		$file = $this->getMockBuilder('OCP\Files\File')->getMock();
-		$file->method('getName')->willReturn($filename);
-		$file->method('getMimetype')->willReturn('text/plain');
-		$file->method('getSize')->willReturn(33);
-		$file->method('isShareable')->willReturn(false);
-		$file->method('isReadable')->willReturn(true);
-
-		$share = Server::get(\OCP\Share\IManager::class)->newShare();
-		$share->setId(42);
-		$share->setPassword('password')
-			->setShareOwner('ownerUID')
-			->setNode($file)
-			->setTarget("/$filename");
-
-		$this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
-		$this->session->method('get')->with('public_link_authenticated')->willReturn('42');
-
-		$this->previewManager->method('isMimeSupported')->with('text/plain')->willReturn(true);
-
-		$this->config->method('getSystemValue')
-			->willReturnMap(
-				[
-					['max_filesize_animated_gifs_public_sharing', 10, 10],
-					['enable_previews', true, true],
-				]
-			);
-		$shareTmpl['maxSizeAnimateGif'] = $this->config->getSystemValue('max_filesize_animated_gifs_public_sharing', 10);
-		$shareTmpl['previewEnabled'] = $this->config->getSystemValue('enable_previews', true);
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('token')
-			->willReturn($share);
-
-		$this->userManager->method('get')->with('ownerUID')->willReturn($owner);
-
-		$this->shareController->showShare();
-	}
-
-	public function testDownloadShareWithCreateOnlyShare(): void {
-		$share = $this->getMockBuilder(IShare::class)->getMock();
-		$share->method('getPassword')->willReturn('password');
-		$share
-			->expects($this->once())
-			->method('getPermissions')
-			->willReturn(Constants::PERMISSION_CREATE);
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('validtoken')
-			->willReturn($share);
-
-		// Test with a password protected share and no authentication
-		$response = $this->shareController->downloadShare('validtoken');
-		$expectedResponse = new DataResponse('Share has no read permission');
-		$this->assertEquals($expectedResponse, $response);
-	}
-
-	public function testDownloadShareWithoutDownloadPermission(): void {
-		$attributes = $this->createMock(IAttributes::class);
-		$attributes->expects(self::once())
-			->method('getAttribute')
-			->with('permissions', 'download')
-			->willReturn(false);
-
-		$share = $this->createMock(IShare::class);
-		$share->method('getPassword')->willReturn('password');
-		$share->expects(self::once())
-			->method('getPermissions')
-			->willReturn(Constants::PERMISSION_READ);
-		$share->expects(self::once())
-			->method('getAttributes')
-			->willReturn($attributes);
-
-		$this->shareManager
-			->expects(self::once())
-			->method('getShareByToken')
-			->with('validtoken')
-			->willReturn($share);
-
-		// Test with a password protected share and no authentication
-		$response = $this->shareController->downloadShare('validtoken');
-		$expectedResponse = new DataResponse('Share has no download permission');
-		$this->assertEquals($expectedResponse, $response);
-	}
-
-	public function testDisabledOwner(): void {
-		$this->shareController->setToken('token');
-
-		$owner = $this->getMockBuilder(IUser::class)->getMock();
-		$owner->method('isEnabled')->willReturn(false);
-
-		$initiator = $this->createMock(IUser::class);
-		$initiator->method('isEnabled')->willReturn(false);
-
-		/* @var MockObject|Folder $folder */
-		$folder = $this->createMock(Folder::class);
-
-		$share = Server::get(\OCP\Share\IManager::class)->newShare();
-		$share->setId(42);
-		$share->setPermissions(Constants::PERMISSION_CREATE)
-			->setShareOwner('ownerUID')
-			->setSharedBy('initiatorUID')
-			->setNode($folder)
-			->setTarget('/share');
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('token')
-			->willReturn($share);
-
-		$this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
-			if ($uid === 'ownerUID') {
-				return $owner;
-			}
-			if ($uid === 'initiatorUID') {
-				return $initiator;
-			}
-			return null;
-		});
-
-		$this->expectException(NotFoundException::class);
-
-		$this->shareController->showShare();
-	}
-
-	public function testDisabledInitiator(): void {
-		$this->shareController->setToken('token');
-
-		$owner = $this->getMockBuilder(IUser::class)->getMock();
-		$owner->method('isEnabled')->willReturn(false);
-
-		$initiator = $this->createMock(IUser::class);
-		$initiator->method('isEnabled')->willReturn(true);
-
-		/* @var MockObject|Folder $folder */
-		$folder = $this->createMock(Folder::class);
-
-		$share = Server::get(\OCP\Share\IManager::class)->newShare();
-		$share->setId(42);
-		$share->setPermissions(Constants::PERMISSION_CREATE)
-			->setShareOwner('ownerUID')
-			->setSharedBy('initiatorUID')
-			->setNode($folder)
-			->setTarget('/share');
-
-		$this->shareManager
-			->expects($this->once())
-			->method('getShareByToken')
-			->with('token')
-			->willReturn($share);
-
-		$this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
-			if ($uid === 'ownerUID') {
-				return $owner;
-			}
-			if ($uid === 'initiatorUID') {
-				return $initiator;
-			}
-			return null;
-		});
-
-		$this->expectException(NotFoundException::class);
-
-		$this->shareController->showShare();
-	}
+    private string $user;
+    private string $oldUser;
+    private string $appName = 'files_sharing';
+    private ShareController $shareController;
+
+    private IL10N&MockObject $l10n;
+    private IConfig&MockObject $config;
+    private ISession&MockObject $session;
+    private Defaults&MockObject $defaults;
+    private IAppConfig&MockObject $appConfig;
+    private Manager&MockObject $shareManager;
+    private IPreview&MockObject $previewManager;
+    private IUserManager&MockObject $userManager;
+    private ITemplateManager&MockObject $templateManager;
+    private IInitialState&MockObject $initialState;
+    private IURLGenerator&MockObject $urlGenerator;
+    private ISecureRandom&MockObject $secureRandom;
+    private IAccountManager&MockObject $accountManager;
+    private IEventDispatcher&MockObject $eventDispatcher;
+    private FederatedShareProvider&MockObject $federatedShareProvider;
+    private IPublicShareTemplateFactory&MockObject $publicShareTemplateFactory;
+
+    protected function setUp(): void {
+        parent::setUp();
+        $this->appName = 'files_sharing';
+
+        $this->shareManager = $this->createMock(Manager::class);
+        $this->urlGenerator = $this->createMock(IURLGenerator::class);
+        $this->session = $this->createMock(ISession::class);
+        $this->previewManager = $this->createMock(IPreview::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->appConfig = $this->createMock(IAppConfig::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->templateManager = $this->createMock(ITemplateManager::class);
+        $this->initialState = $this->createMock(IInitialState::class);
+        $this->federatedShareProvider = $this->createMock(FederatedShareProvider::class);
+        $this->federatedShareProvider->expects($this->any())
+            ->method('isOutgoingServer2serverShareEnabled')->willReturn(true);
+        $this->federatedShareProvider->expects($this->any())
+            ->method('isIncomingServer2serverShareEnabled')->willReturn(true);
+        $this->accountManager = $this->createMock(IAccountManager::class);
+        $this->eventDispatcher = $this->createMock(IEventDispatcher::class);
+        $this->l10n = $this->createMock(IL10N::class);
+        $this->secureRandom = $this->createMock(ISecureRandom::class);
+        $this->defaults = $this->createMock(Defaults::class);
+        $this->publicShareTemplateFactory = $this->createMock(IPublicShareTemplateFactory::class);
+        $this->publicShareTemplateFactory
+            ->expects($this->any())
+            ->method('getProvider')
+            ->willReturn(
+                new DefaultPublicShareTemplateProvider(
+                    $this->userManager,
+                    $this->accountManager,
+                    $this->previewManager,
+                    $this->federatedShareProvider,
+                    $this->urlGenerator,
+                    $this->eventDispatcher,
+                    $this->l10n,
+                    $this->defaults,
+                    $this->config,
+                    $this->createMock(IRequest::class),
+                    $this->templateManager,
+                    $this->initialState,
+                    $this->appConfig,
+                )
+            );
+
+        $this->shareController = new ShareController(
+            $this->appName,
+            $this->createMock(IRequest::class),
+            $this->config,
+            $this->urlGenerator,
+            $this->userManager,
+            $this->createMock(IManager::class),
+            $this->shareManager,
+            $this->session,
+            $this->previewManager,
+            $this->createMock(IRootFolder::class),
+            $this->federatedShareProvider,
+            $this->accountManager,
+            $this->eventDispatcher,
+            $this->l10n,
+            $this->secureRandom,
+            $this->defaults,
+            $this->publicShareTemplateFactory,
+        );
+
+
+        // Store current user
+        $this->oldUser = \OC_User::getUser();
+
+        // Create a dummy user
+        $this->user = Server::get(ISecureRandom::class)->generate(12, ISecureRandom::CHAR_LOWER);
+
+        Server::get(IUserManager::class)->createUser($this->user, $this->user);
+        \OC_Util::tearDownFS();
+        $this->loginAsUser($this->user);
+    }
+
+    protected function tearDown(): void {
+        \OC_Util::tearDownFS();
+        \OC_User::setUserId('');
+        Filesystem::tearDown();
+        $user = Server::get(IUserManager::class)->get($this->user);
+        if ($user !== null) {
+            $user->delete();
+        }
+        \OC_User::setIncognitoMode(false);
+
+        Server::get(ISession::class)->set('public_link_authenticated', '');
+
+        // Set old user
+        \OC_User::setUserId($this->oldUser);
+        \OC_Util::setupFS($this->oldUser);
+        parent::tearDown();
+    }
+
+    public function testShowShareInvalidToken(): void {
+        $this->shareController->setToken('invalidtoken');
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('invalidtoken')
+            ->willThrowException(new ShareNotFound());
+
+        $this->expectException(NotFoundException::class);
+
+        // Test without a not existing token
+        $this->shareController->showShare();
+    }
+
+    public function testShowShareNotAuthenticated(): void {
+        $this->shareController->setToken('validtoken');
+
+        $share = Server::get(\OCP\Share\IManager::class)->newShare();
+        $share->setPassword('password');
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('validtoken')
+            ->willReturn($share);
+
+        $this->expectException(NotFoundException::class);
+
+        // Test without a not existing token
+        $this->shareController->showShare();
+    }
+
+
+    public function testShowShare(): void {
+        $note = 'personal note';
+        $filename = 'file1.txt';
+
+        $this->shareController->setToken('token');
+
+        $owner = $this->createMock(IUser::class);
+        $owner->method('getDisplayName')->willReturn('ownerDisplay');
+        $owner->method('getUID')->willReturn('ownerUID');
+        $owner->method('isEnabled')->willReturn(true);
+
+        $initiator = $this->createMock(IUser::class);
+        $initiator->method('getDisplayName')->willReturn('initiatorDisplay');
+        $initiator->method('getUID')->willReturn('initiatorUID');
+        $initiator->method('isEnabled')->willReturn(true);
+
+        $file = $this->createMock(File::class);
+        $file->method('getName')->willReturn($filename);
+        $file->method('getMimetype')->willReturn('text/plain');
+        $file->method('getSize')->willReturn(33);
+        $file->method('isReadable')->willReturn(true);
+        $file->method('isShareable')->willReturn(true);
+        $file->method('getId')->willReturn(111);
+
+        $accountName = $this->createMock(IAccountProperty::class);
+        $accountName->method('getScope')
+            ->willReturn(IAccountManager::SCOPE_PUBLISHED);
+        $account = $this->createMock(IAccount::class);
+        $account->method('getProperty')
+            ->with(IAccountManager::PROPERTY_DISPLAYNAME)
+            ->willReturn($accountName);
+        $this->accountManager->expects($this->once())
+            ->method('getAccount')
+            ->with($owner)
+            ->willReturn($account);
+
+        /** @var Manager */
+        $manager = Server::get(Manager::class);
+        $share = $manager->newShare();
+        $share->setId(42)
+            ->setPermissions(Constants::PERMISSION_READ | Constants::PERMISSION_UPDATE)
+            ->setPassword('password')
+            ->setShareOwner('ownerUID')
+            ->setSharedBy('initiatorUID')
+            ->setNode($file)
+            ->setNote($note)
+            ->setTarget("/$filename")
+            ->setToken('token');
+
+        $this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
+        $this->session->method('get')->with('public_link_authenticated')->willReturn('42');
+
+        $this->urlGenerator->expects(self::atLeast(2))
+            ->method('linkToRouteAbsolute')
+            ->willReturnMap([
+                // every file has the show show share url in the opengraph url prop
+                ['files_sharing.sharecontroller.showShare', ['token' => 'token'], 'shareUrl'],
+                // this share is not an image to the default preview is used
+                ['files_sharing.PublicPreview.getPreview', ['x' => 256, 'y' => 256, 'file' => $share->getTarget(), 'token' => 'token'], 'previewUrl'],
+            ]);
+
+        $this->urlGenerator->expects($this->once())
+            ->method('getAbsoluteURL')
+            ->willReturnMap([
+                ['/public.php/dav/files/token/?accept=zip', 'downloadUrl'],
+            ]);
+
+        $this->previewManager->method('isMimeSupported')->with('text/plain')->willReturn(true);
+
+        $this->config->method('getSystemValue')
+            ->willReturnMap(
+                [
+                    ['max_filesize_animated_gifs_public_sharing', 10, 10],
+                    ['enable_previews', true, true],
+                    ['preview_max_x', 1024, 1024],
+                    ['preview_max_y', 1024, 1024],
+                ]
+            );
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('token')
+            ->willReturn($share);
+
+        $this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
+            if ($uid === 'ownerUID') {
+                return $owner;
+            }
+            if ($uid === 'initiatorUID') {
+                return $initiator;
+            }
+            return null;
+        });
+
+        $this->eventDispatcher->method('dispatchTyped')->with(
+            $this->callback(function ($event) use ($share) {
+                if ($event instanceof BeforeTemplateRenderedEvent) {
+                    return $event->getShare() === $share;
+                } else {
+                    return true;
+                }
+            })
+        );
+
+        $this->l10n->expects($this->any())
+            ->method('t')
+            ->willReturnCallback(function ($text, $parameters) {
+                return vsprintf($text, $parameters);
+            });
+
+        $this->defaults->expects(self::any())
+            ->method('getProductName')
+            ->willReturn('Nextcloud');
+
+        // Ensure the correct initial state is setup
+        // Shared node is a file so this is a single file share:
+        $view = 'public-file-share';
+        // Set up initial state
+        $initialState = [];
+        $this->initialState->expects(self::any())
+            ->method('provideInitialState')
+            ->willReturnCallback(function ($key, $value) use (&$initialState): void {
+                $initialState[$key] = $value;
+            });
+        $expectedInitialState = [
+            'isPublic' => true,
+            'sharingToken' => 'token',
+            'sharePermissions' => (Constants::PERMISSION_READ | Constants::PERMISSION_UPDATE),
+            'filename' => $filename,
+            'view' => $view,
+            'fileId' => 111,
+            'owner' => 'ownerUID',
+            'ownerDisplayName' => 'ownerDisplay',
+            'isFileRequest' => false,
+            'templates' => [],
+        ];
+
+        $response = $this->shareController->showShare();
+
+        $this->assertEquals($expectedInitialState, $initialState);
+
+        $csp = new ContentSecurityPolicy();
+        $csp->addAllowedFrameDomain('\'self\'');
+        $expectedResponse = new PublicTemplateResponse('files', 'index');
+        $expectedResponse->setParams(['pageTitle' => $filename]);
+        $expectedResponse->setContentSecurityPolicy($csp);
+        $expectedResponse->setHeaderTitle($filename);
+        $expectedResponse->setHeaderDetails('shared by ownerDisplay');
+        $expectedResponse->setHeaderActions([
+            new SimpleMenuAction('download', $this->l10n->t('Download'), 'icon-download', 'downloadUrl', 0, '33'),
+            new ExternalShareMenuAction($this->l10n->t('Add to your Nextcloud'), 'icon-external', 'owner', 'ownerDisplay', $filename),
+            new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', 'downloadUrl'),
+        ]);
+
+        $this->assertEquals($expectedResponse, $response);
+    }
+
+    public function testShowFileDropShare(): void {
+        $filename = 'folder1';
+
+        $this->shareController->setToken('token');
+
+        $owner = $this->createMock(IUser::class);
+        $owner->method('getDisplayName')->willReturn('ownerDisplay');
+        $owner->method('getUID')->willReturn('ownerUID');
+        $owner->method('isEnabled')->willReturn(true);
+
+        $initiator = $this->createMock(IUser::class);
+        $initiator->method('getDisplayName')->willReturn('initiatorDisplay');
+        $initiator->method('getUID')->willReturn('initiatorUID');
+        $initiator->method('isEnabled')->willReturn(true);
+
+        $file = $this->createMock(Folder::class);
+        $file->method('isReadable')->willReturn(true);
+        $file->method('isShareable')->willReturn(true);
+        $file->method('getId')->willReturn(1234);
+        $file->method('getMimetype')->willReturn('text/plain');
+        $file->method('getName')->willReturn($filename);
+
+        $accountName = $this->createMock(IAccountProperty::class);
+        $accountName->method('getScope')
+            ->willReturn(IAccountManager::SCOPE_PUBLISHED);
+        $account = $this->createMock(IAccount::class);
+        $account->method('getProperty')
+            ->with(IAccountManager::PROPERTY_DISPLAYNAME)
+            ->willReturn($accountName);
+        $this->accountManager->expects($this->once())
+            ->method('getAccount')
+            ->with($owner)
+            ->willReturn($account);
+
+        /** @var Manager */
+        $manager = Server::get(Manager::class);
+        $share = $manager->newShare();
+        $share->setId(42)
+            ->setPermissions(Constants::PERMISSION_CREATE)
+            ->setPassword('password')
+            ->setShareOwner('ownerUID')
+            ->setSharedBy('initiatorUID')
+            ->setNote('The note')
+            ->setLabel('A label')
+            ->setNode($file)
+            ->setTarget("/$filename")
+            ->setToken('token');
+
+        $this->appConfig
+            ->expects($this->once())
+            ->method('getValueString')
+            ->with('core', 'shareapi_public_link_disclaimertext', '')
+            ->willReturn('My disclaimer text');
+
+        $this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
+        $this->session->method('get')->with('public_link_authenticated')->willReturn('42');
+
+        $this->urlGenerator->expects(self::atLeastOnce())
+            ->method('linkToRouteAbsolute')
+            ->willReturnMap([
+                // every file has the show show share url in the opengraph url prop
+                ['files_sharing.sharecontroller.showShare', ['token' => 'token'], 'shareUrl'],
+                // there is no preview or folders so no other link for opengraph
+            ]);
+
+        $this->config->method('getSystemValue')
+            ->willReturnMap(
+                [
+                    ['max_filesize_animated_gifs_public_sharing', 10, 10],
+                    ['enable_previews', true, true],
+                    ['preview_max_x', 1024, 1024],
+                    ['preview_max_y', 1024, 1024],
+                ]
+            );
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('token')
+            ->willReturn($share);
+
+        $this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
+            if ($uid === 'ownerUID') {
+                return $owner;
+            }
+            if ($uid === 'initiatorUID') {
+                return $initiator;
+            }
+            return null;
+        });
+
+        $this->eventDispatcher->method('dispatchTyped')->with(
+            $this->callback(function ($event) use ($share) {
+                if ($event instanceof BeforeTemplateRenderedEvent) {
+                    return $event->getShare() === $share;
+                } else {
+                    return true;
+                }
+            })
+        );
+
+        $this->l10n->expects($this->any())
+            ->method('t')
+            ->willReturnCallback(function ($text, $parameters) {
+                return vsprintf($text, $parameters);
+            });
+
+        // Set up initial state
+        $initialState = [];
+        $this->initialState->expects(self::any())
+            ->method('provideInitialState')
+            ->willReturnCallback(function ($key, $value) use (&$initialState): void {
+                $initialState[$key] = $value;
+            });
+        $expectedInitialState = [
+            'isPublic' => true,
+            'sharingToken' => 'token',
+            'sharePermissions' => Constants::PERMISSION_CREATE,
+            'filename' => $filename,
+            'view' => 'public-file-drop',
+            'disclaimer' => 'My disclaimer text',
+            'owner' => 'ownerUID',
+            'ownerDisplayName' => 'ownerDisplay',
+            'isFileRequest' => false,
+            'note' => 'The note',
+            'label' => 'A label',
+            'templates' => [],
+        ];
+
+        $response = $this->shareController->showShare();
+
+        $this->assertEquals($expectedInitialState, $initialState);
+
+        $csp = new ContentSecurityPolicy();
+        $csp->addAllowedFrameDomain('\'self\'');
+        $expectedResponse = new PublicTemplateResponse('files', 'index');
+        $expectedResponse->setParams(['pageTitle' => 'A label']);
+        $expectedResponse->setContentSecurityPolicy($csp);
+        $expectedResponse->setHeaderTitle('A label');
+        $expectedResponse->setHeaderDetails('shared by ownerDisplay');
+        $expectedResponse->setHeaderActions([
+            new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', 'shareUrl'),
+        ]);
+
+        $this->assertEquals($expectedResponse, $response);
+    }
+
+    public function testShowShareWithPrivateName(): void {
+        $note = 'personal note';
+        $filename = 'file1.txt';
+
+        $this->shareController->setToken('token');
+
+        $owner = $this->createMock(IUser::class);
+        $owner->method('getDisplayName')->willReturn('ownerDisplay');
+        $owner->method('getUID')->willReturn('ownerUID');
+        $owner->method('isEnabled')->willReturn(true);
+
+        $initiator = $this->createMock(IUser::class);
+        $initiator->method('getDisplayName')->willReturn('initiatorDisplay');
+        $initiator->method('getUID')->willReturn('initiatorUID');
+        $initiator->method('isEnabled')->willReturn(true);
+
+        $file = $this->createMock(File::class);
+        $file->method('getName')->willReturn($filename);
+        $file->method('getMimetype')->willReturn('text/plain');
+        $file->method('getSize')->willReturn(33);
+        $file->method('isReadable')->willReturn(true);
+        $file->method('isShareable')->willReturn(true);
+        $file->method('getId')->willReturn(111);
+
+        $accountName = $this->createMock(IAccountProperty::class);
+        $accountName->method('getScope')
+            ->willReturn(IAccountManager::SCOPE_LOCAL);
+        $account = $this->createMock(IAccount::class);
+        $account->method('getProperty')
+            ->with(IAccountManager::PROPERTY_DISPLAYNAME)
+            ->willReturn($accountName);
+        $this->accountManager->expects($this->once())
+            ->method('getAccount')
+            ->with($owner)
+            ->willReturn($account);
+
+        /** @var IShare */
+        $share = Server::get(Manager::class)->newShare();
+        $share->setId(42);
+        $share->setPassword('password')
+            ->setShareOwner('ownerUID')
+            ->setSharedBy('initiatorUID')
+            ->setNode($file)
+            ->setNote($note)
+            ->setToken('token')
+            ->setPermissions(Constants::PERMISSION_ALL & ~Constants::PERMISSION_SHARE)
+            ->setTarget("/$filename");
+
+        $this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
+        $this->session->method('get')->with('public_link_authenticated')->willReturn('42');
+
+        $this->urlGenerator->expects(self::atLeast(2))
+            ->method('linkToRouteAbsolute')
+            ->willReturnMap([
+                // every file has the show show share url in the opengraph url prop
+                ['files_sharing.sharecontroller.showShare', ['token' => 'token'], 'shareUrl'],
+                // this share is not an image to the default preview is used
+                ['files_sharing.PublicPreview.getPreview', ['x' => 256, 'y' => 256, 'file' => $share->getTarget(), 'token' => 'token'], 'previewUrl'],
+            ]);
+
+        $this->urlGenerator->expects($this->once())
+            ->method('getAbsoluteURL')
+            ->willReturnMap([
+                ['/public.php/dav/files/token/?accept=zip', 'downloadUrl'],
+            ]);
+
+        $this->previewManager->method('isMimeSupported')->with('text/plain')->willReturn(true);
+
+        $this->config->method('getSystemValue')
+            ->willReturnMap(
+                [
+                    ['max_filesize_animated_gifs_public_sharing', 10, 10],
+                    ['enable_previews', true, true],
+                    ['preview_max_x', 1024, 1024],
+                    ['preview_max_y', 1024, 1024],
+                ]
+            );
+        $shareTmpl['maxSizeAnimateGif'] = $this->config->getSystemValue('max_filesize_animated_gifs_public_sharing', 10);
+        $shareTmpl['previewEnabled'] = $this->config->getSystemValue('enable_previews', true);
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('token')
+            ->willReturn($share);
+
+        $this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
+            if ($uid === 'ownerUID') {
+                return $owner;
+            }
+            if ($uid === 'initiatorUID') {
+                return $initiator;
+            }
+            return null;
+        });
+
+        $this->eventDispatcher->method('dispatchTyped')->with(
+            $this->callback(function ($event) use ($share) {
+                if ($event instanceof BeforeTemplateRenderedEvent) {
+                    return $event->getShare() === $share;
+                } else {
+                    return true;
+                }
+            })
+        );
+
+        $this->l10n->expects($this->any())
+            ->method('t')
+            ->willReturnCallback(function ($text, $parameters) {
+                return vsprintf($text, $parameters);
+            });
+
+        $this->defaults->expects(self::any())
+            ->method('getProductName')
+            ->willReturn('Nextcloud');
+
+        $response = $this->shareController->showShare();
+
+        $csp = new ContentSecurityPolicy();
+        $csp->addAllowedFrameDomain('\'self\'');
+        $expectedResponse = new PublicTemplateResponse('files', 'index');
+        $expectedResponse->setParams(['pageTitle' => $filename]);
+        $expectedResponse->setContentSecurityPolicy($csp);
+        $expectedResponse->setHeaderTitle($filename);
+        $expectedResponse->setHeaderDetails('');
+        $expectedResponse->setHeaderActions([
+            new SimpleMenuAction('download', $this->l10n->t('Download'), 'icon-download', 'downloadUrl', 0, '33'),
+            new ExternalShareMenuAction($this->l10n->t('Add to your Nextcloud'), 'icon-external', 'owner', 'ownerDisplay', $filename),
+            new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', 'downloadUrl'),
+        ]);
+
+        $this->assertEquals($expectedResponse, $response);
+    }
+
+
+    public function testShowShareInvalid(): void {
+        $this->expectException(NotFoundException::class);
+
+        $filename = 'file1.txt';
+        $this->shareController->setToken('token');
+
+        $owner = $this->getMockBuilder(IUser::class)->getMock();
+        $owner->method('getDisplayName')->willReturn('ownerDisplay');
+        $owner->method('getUID')->willReturn('ownerUID');
+
+        $file = $this->getMockBuilder('OCP\Files\File')->getMock();
+        $file->method('getName')->willReturn($filename);
+        $file->method('getMimetype')->willReturn('text/plain');
+        $file->method('getSize')->willReturn(33);
+        $file->method('isShareable')->willReturn(false);
+        $file->method('isReadable')->willReturn(true);
+
+        $share = Server::get(\OCP\Share\IManager::class)->newShare();
+        $share->setId(42);
+        $share->setPassword('password')
+            ->setShareOwner('ownerUID')
+            ->setNode($file)
+            ->setTarget("/$filename");
+
+        $this->session->method('exists')->with('public_link_authenticated')->willReturn(true);
+        $this->session->method('get')->with('public_link_authenticated')->willReturn('42');
+
+        $this->previewManager->method('isMimeSupported')->with('text/plain')->willReturn(true);
+
+        $this->config->method('getSystemValue')
+            ->willReturnMap(
+                [
+                    ['max_filesize_animated_gifs_public_sharing', 10, 10],
+                    ['enable_previews', true, true],
+                ]
+            );
+        $shareTmpl['maxSizeAnimateGif'] = $this->config->getSystemValue('max_filesize_animated_gifs_public_sharing', 10);
+        $shareTmpl['previewEnabled'] = $this->config->getSystemValue('enable_previews', true);
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('token')
+            ->willReturn($share);
+
+        $this->userManager->method('get')->with('ownerUID')->willReturn($owner);
+
+        $this->shareController->showShare();
+    }
+
+    public function testDownloadShareWithCreateOnlyShare(): void {
+        $share = $this->getMockBuilder(IShare::class)->getMock();
+        $share->method('getPassword')->willReturn('password');
+        $share
+            ->expects($this->once())
+            ->method('getPermissions')
+            ->willReturn(Constants::PERMISSION_CREATE);
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('validtoken')
+            ->willReturn($share);
+
+        // Test with a password protected share and no authentication
+        $response = $this->shareController->downloadShare('validtoken');
+        $expectedResponse = new DataResponse('Share has no read permission');
+        $this->assertEquals($expectedResponse, $response);
+    }
+
+    public function testDownloadShareWithoutDownloadPermission(): void {
+        $attributes = $this->createMock(IAttributes::class);
+        $attributes->expects(self::once())
+            ->method('getAttribute')
+            ->with('permissions', 'download')
+            ->willReturn(false);
+
+        $share = $this->createMock(IShare::class);
+        $share->method('getPassword')->willReturn('password');
+        $share->expects(self::once())
+            ->method('getPermissions')
+            ->willReturn(Constants::PERMISSION_READ);
+        $share->expects(self::once())
+            ->method('getAttributes')
+            ->willReturn($attributes);
+
+        $this->shareManager
+            ->expects(self::once())
+            ->method('getShareByToken')
+            ->with('validtoken')
+            ->willReturn($share);
+
+        // Test with a password protected share and no authentication
+        $response = $this->shareController->downloadShare('validtoken');
+        $expectedResponse = new DataResponse('Share has no download permission');
+        $this->assertEquals($expectedResponse, $response);
+    }
+
+    public function testDisabledOwner(): void {
+        $this->shareController->setToken('token');
+
+        $owner = $this->getMockBuilder(IUser::class)->getMock();
+        $owner->method('isEnabled')->willReturn(false);
+
+        $initiator = $this->createMock(IUser::class);
+        $initiator->method('isEnabled')->willReturn(false);
+
+        /* @var MockObject|Folder $folder */
+        $folder = $this->createMock(Folder::class);
+
+        $share = Server::get(\OCP\Share\IManager::class)->newShare();
+        $share->setId(42);
+        $share->setPermissions(Constants::PERMISSION_CREATE)
+            ->setShareOwner('ownerUID')
+            ->setSharedBy('initiatorUID')
+            ->setNode($folder)
+            ->setTarget('/share');
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('token')
+            ->willReturn($share);
+
+        $this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
+            if ($uid === 'ownerUID') {
+                return $owner;
+            }
+            if ($uid === 'initiatorUID') {
+                return $initiator;
+            }
+            return null;
+        });
+
+        $this->expectException(NotFoundException::class);
+
+        $this->shareController->showShare();
+    }
+
+    public function testDisabledInitiator(): void {
+        $this->shareController->setToken('token');
+
+        $owner = $this->getMockBuilder(IUser::class)->getMock();
+        $owner->method('isEnabled')->willReturn(false);
+
+        $initiator = $this->createMock(IUser::class);
+        $initiator->method('isEnabled')->willReturn(true);
+
+        /* @var MockObject|Folder $folder */
+        $folder = $this->createMock(Folder::class);
+
+        $share = Server::get(\OCP\Share\IManager::class)->newShare();
+        $share->setId(42);
+        $share->setPermissions(Constants::PERMISSION_CREATE)
+            ->setShareOwner('ownerUID')
+            ->setSharedBy('initiatorUID')
+            ->setNode($folder)
+            ->setTarget('/share');
+
+        $this->shareManager
+            ->expects($this->once())
+            ->method('getShareByToken')
+            ->with('token')
+            ->willReturn($share);
+
+        $this->userManager->method('get')->willReturnCallback(function (string $uid) use ($owner, $initiator) {
+            if ($uid === 'ownerUID') {
+                return $owner;
+            }
+            if ($uid === 'initiatorUID') {
+                return $initiator;
+            }
+            return null;
+        });
+
+        $this->expectException(NotFoundException::class);
+
+        $this->shareController->showShare();
+    }
 }

lib/private/Preview/GeneratorHelper.php

Indentation +17 added lines, -17 removed lines

@@ -18,23 +18,23 @@
  * @psalm-import-type ProviderClosure from IPreview
  */
 class GeneratorHelper {
-	public function getThumbnail(IProviderV2 $provider, File $file, int $maxWidth, int $maxHeight, bool $crop = false): IImage|false {
-		if ($provider instanceof Imaginary) {
-			return $provider->getCroppedThumbnail($file, $maxWidth, $maxHeight, $crop) ?? false;
-		}
-		return $provider->getThumbnail($file, $maxWidth, $maxHeight) ?? false;
-	}
+    public function getThumbnail(IProviderV2 $provider, File $file, int $maxWidth, int $maxHeight, bool $crop = false): IImage|false {
+        if ($provider instanceof Imaginary) {
+            return $provider->getCroppedThumbnail($file, $maxWidth, $maxHeight, $crop) ?? false;
+        }
+        return $provider->getThumbnail($file, $maxWidth, $maxHeight) ?? false;
+    }
 
-	public function getImage(ISimpleFile $maxPreview): IImage {
-		$image = new OCPImage();
-		$image->loadFromData($maxPreview->getContent());
-		return $image;
-	}
+    public function getImage(ISimpleFile $maxPreview): IImage {
+        $image = new OCPImage();
+        $image->loadFromData($maxPreview->getContent());
+        return $image;
+    }
 
-	/**
-	 * @param \Closure|string $providerClosure (string is only authorized in unit tests)
-	 */
-	public function getProvider(\Closure|string $providerClosure): IProviderV2|false {
-		return $providerClosure();
-	}
+    /**
+     * @param \Closure|string $providerClosure (string is only authorized in unit tests)
+     */
+    public function getProvider(\Closure|string $providerClosure): IProviderV2|false {
+        return $providerClosure();
+    }
 }

lib/private/Preview/Generator.php

Indentation +512 added lines, -512 removed lines

@@ -26,530 +26,530 @@
 use Psr\Log\LoggerInterface;
 
 class Generator {
-	public const SEMAPHORE_ID_ALL = 0x0a11;
-	public const SEMAPHORE_ID_NEW = 0x07ea;
-
-	public function __construct(
-		private IConfig $config,
-		private IPreview $previewManager,
-		private GeneratorHelper $helper,
-		private IEventDispatcher $eventDispatcher,
-		private LoggerInterface $logger,
-		private PreviewMapper $previewMapper,
-		private StorageFactory $storageFactory,
-	) {
-	}
-
-	/**
-	 * Returns a preview of a file
-	 *
-	 * The cache is searched first and if nothing usable was found then a preview is
-	 * generated by one of the providers
-	 *
-	 * @return ISimpleFile
-	 * @throws NotFoundException
-	 * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
-	 */
-	public function getPreview(
-		File $file,
-		int $width = -1,
-		int $height = -1,
-		bool $crop = false,
-		string $mode = IPreview::MODE_FILL,
-		?string $mimeType = null,
-		bool $cacheResult = true,
-	): ISimpleFile {
-		$specification = [
-			'width' => $width,
-			'height' => $height,
-			'crop' => $crop,
-			'mode' => $mode,
-		];
-
-		$this->eventDispatcher->dispatchTyped(new BeforePreviewFetchedEvent(
-			$file,
-			$width,
-			$height,
-			$crop,
-			$mode,
-			$mimeType,
-		));
-
-		$this->logger->debug('Requesting preview for {path} with width={width}, height={height}, crop={crop}, mode={mode}, mimeType={mimeType}', [
-			'path' => $file->getPath(),
-			'width' => $width,
-			'height' => $height,
-			'crop' => $crop,
-			'mode' => $mode,
-			'mimeType' => $mimeType,
-		]);
-
-
-		// since we only ask for one preview, and the generate method return the last one it created, it returns the one we want
-		return $this->generatePreviews($file, [$specification], $mimeType, $cacheResult);
-	}
-
-	/**
-	 * Generates previews of a file
-	 *
-	 * @throws NotFoundException
-	 * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
-	 */
-	public function generatePreviews(File $file, array $specifications, ?string $mimeType = null, bool $cacheResult = true): ISimpleFile {
-		//Make sure that we can read the file
-		if (!$file->isReadable()) {
-			$this->logger->warning('Cannot read file: {path}, skipping preview generation.', ['path' => $file->getPath()]);
-			throw new NotFoundException('Cannot read file');
-		}
-
-		if ($mimeType === null) {
-			$mimeType = $file->getMimeType();
-		}
-
-		[$file->getId() => $previews] = $this->previewMapper->getAvailablePreviews([$file->getId()]);
-
-		$previewVersion = null;
-		if ($file instanceof IVersionedPreviewFile) {
-			$previewVersion = $file->getPreviewVersion();
-		}
-
-		// Get the max preview and infer the max preview sizes from that
-		$maxPreview = $this->getMaxPreview($previews, $file, $mimeType, $previewVersion);
-		$maxPreviewImage = null; // only load the image when we need it
-		if ($maxPreview->getSize() === 0) {
-			$this->storageFactory->deletePreview($maxPreview);
-			$this->previewMapper->delete($maxPreview);
-			$this->logger->error('Max preview generated for file {path} has size 0, deleting and throwing exception.', ['path' => $file->getPath()]);
-			throw new NotFoundException('Max preview size 0, invalid!');
-		}
-
-		$maxWidth = $maxPreview->getWidth();
-		$maxHeight = $maxPreview->getHeight();
-
-		if ($maxWidth <= 0 || $maxHeight <= 0) {
-			throw new NotFoundException('The maximum preview sizes are zero or less pixels');
-		}
-
-		$previewFile = null;
-		foreach ($specifications as $specification) {
-			$width = $specification['width'] ?? -1;
-			$height = $specification['height'] ?? -1;
-			$crop = $specification['crop'] ?? false;
-			$mode = $specification['mode'] ?? IPreview::MODE_FILL;
-
-			// If both width and height are -1 we just want the max preview
-			if ($width === -1 && $height === -1) {
-				$width = $maxWidth;
-				$height = $maxHeight;
-			}
-
-			// Calculate the preview size
-			[$width, $height] = $this->calculateSize($width, $height, $crop, $mode, $maxWidth, $maxHeight);
-
-			// No need to generate a preview that is just the max preview
-			if ($width === $maxWidth && $height === $maxHeight) {
-				// ensure correct return value if this was the last one
-				$previewFile = new PreviewFile($maxPreview, $this->storageFactory, $this->previewMapper);
-				continue;
-			}
-
-			// Try to get a cached preview. Else generate (and store) one
-			try {
-				$preview = array_find($previews, fn (Preview $preview): bool => $preview->getWidth() === $width
-					&& $preview->getHeight() === $height && $preview->getMimetype() === $maxPreview->getMimetype()
-					&& $preview->getVersion() === $previewVersion && $preview->isCropped() === $crop);
-
-				if ($preview) {
-					$previewFile = new PreviewFile($preview, $this->storageFactory, $this->previewMapper);
-				} else {
-					if (!$this->previewManager->isMimeSupported($mimeType)) {
-						throw new NotFoundException();
-					}
-
-					if ($maxPreviewImage === null) {
-						$maxPreviewImage = $this->helper->getImage(new PreviewFile($maxPreview, $this->storageFactory, $this->previewMapper));
-					}
-
-					$this->logger->debug('Cached preview not found for file {path}, generating a new preview.', ['path' => $file->getPath()]);
-					$previewFile = $this->generatePreview($file, $maxPreviewImage, $width, $height, $crop, $maxWidth, $maxHeight, $previewVersion, $cacheResult);
-				}
-			} catch (\InvalidArgumentException $e) {
-				throw new NotFoundException('', 0, $e);
-			}
-
-			if ($previewFile->getSize() === 0) {
-				$previewFile->delete();
-				throw new NotFoundException('Cached preview size 0, invalid!');
-			}
-		}
-		assert($previewFile !== null);
-
-		// Free memory being used by the embedded image resource.  Without this the image is kept in memory indefinitely.
-		// Garbage Collection does NOT free this memory.  We have to do it ourselves.
-		if ($maxPreviewImage instanceof \OCP\Image) {
-			$maxPreviewImage->destroy();
-		}
-
-		return $previewFile;
-	}
-
-	/**
-	 * Acquire a semaphore of the specified id and concurrency, blocking if necessary.
-	 * Return an identifier of the semaphore on success, which can be used to release it via
-	 * {@see Generator::unguardWithSemaphore()}.
-	 *
-	 * @param int $semId
-	 * @param int $concurrency
-	 * @return false|\SysvSemaphore the semaphore on success or false on failure
-	 */
-	public static function guardWithSemaphore(int $semId, int $concurrency) {
-		if (!extension_loaded('sysvsem')) {
-			return false;
-		}
-		$sem = sem_get($semId, $concurrency);
-		if ($sem === false) {
-			return false;
-		}
-		if (!sem_acquire($sem)) {
-			return false;
-		}
-		return $sem;
-	}
-
-	/**
-	 * Releases the semaphore acquired from {@see Generator::guardWithSemaphore()}.
-	 *
-	 * @param false|\SysvSemaphore $semId the semaphore identifier returned by guardWithSemaphore
-	 * @return bool
-	 */
-	public static function unguardWithSemaphore(false|\SysvSemaphore $semId): bool {
-		if ($semId === false || !($semId instanceof \SysvSemaphore)) {
-			return false;
-		}
-		return sem_release($semId);
-	}
-
-	/**
-	 * Get the number of concurrent threads supported by the host.
-	 *
-	 * @return int number of concurrent threads, or 0 if it cannot be determined
-	 */
-	public static function getHardwareConcurrency(): int {
-		static $width;
-
-		if (!isset($width)) {
-			if (function_exists('ini_get')) {
-				$openBasedir = ini_get('open_basedir');
-				if (empty($openBasedir) || strpos($openBasedir, '/proc/cpuinfo') !== false) {
-					$width = is_readable('/proc/cpuinfo') ? substr_count(file_get_contents('/proc/cpuinfo'), 'processor') : 0;
-				} else {
-					$width = 0;
-				}
-			} else {
-				$width = 0;
-			}
-		}
-		return $width;
-	}
-
-	/**
-	 * Get number of concurrent preview generations from system config
-	 *
-	 * Two config entries, `preview_concurrency_new` and `preview_concurrency_all`,
-	 * are available. If not set, the default values are determined with the hardware concurrency
-	 * of the host. In case the hardware concurrency cannot be determined, or the user sets an
-	 * invalid value, fallback values are:
-	 * For new images whose previews do not exist and need to be generated, 4;
-	 * For all preview generation requests, 8.
-	 * Value of `preview_concurrency_all` should be greater than or equal to that of
-	 * `preview_concurrency_new`, otherwise, the latter is returned.
-	 *
-	 * @param string $type either `preview_concurrency_new` or `preview_concurrency_all`
-	 * @return int number of concurrent preview generations, or -1 if $type is invalid
-	 */
-	public function getNumConcurrentPreviews(string $type): int {
-		static $cached = [];
-		if (array_key_exists($type, $cached)) {
-			return $cached[$type];
-		}
-
-		$hardwareConcurrency = self::getHardwareConcurrency();
-		switch ($type) {
-			case 'preview_concurrency_all':
-				$fallback = $hardwareConcurrency > 0 ? $hardwareConcurrency * 2 : 8;
-				$concurrency_all = $this->config->getSystemValueInt($type, $fallback);
-				$concurrency_new = $this->getNumConcurrentPreviews('preview_concurrency_new');
-				$cached[$type] = max($concurrency_all, $concurrency_new);
-				break;
-			case 'preview_concurrency_new':
-				$fallback = $hardwareConcurrency > 0 ? $hardwareConcurrency : 4;
-				$cached[$type] = $this->config->getSystemValueInt($type, $fallback);
-				break;
-			default:
-				return -1;
-		}
-		return $cached[$type];
-	}
-
-	/**
-	 * @param Preview[] $previews
-	 * @throws NotFoundException
-	 */
-	private function getMaxPreview(array $previews, File $file, string $mimeType, ?string $version): Preview {
-		// We don't know the max preview size, so we can't use getCachedPreview.
-		// It might have been generated with a higher resolution than the current value.
-		foreach ($previews as $preview) {
-			if ($preview->isMax() && ($version === $preview->getVersion())) {
-				return $preview;
-			}
-		}
-
-		$maxWidth = $this->config->getSystemValueInt('preview_max_x', 4096);
-		$maxHeight = $this->config->getSystemValueInt('preview_max_y', 4096);
-
-		return $this->generateProviderPreview($file, $maxWidth, $maxHeight, false, true, $mimeType, $version);
-	}
-
-	private function generateProviderPreview(File $file, int $width, int $height, bool $crop, bool $max, string $mimeType, ?string $version): Preview {
-		$previewProviders = $this->previewManager->getProviders();
-		foreach ($previewProviders as $supportedMimeType => $providers) {
-			// Filter out providers that does not support this mime
-			if (!preg_match($supportedMimeType, $mimeType)) {
-				continue;
-			}
-
-			foreach ($providers as $providerClosure) {
-
-				$provider = $this->helper->getProvider($providerClosure);
-				if (!$provider) {
-					continue;
-				}
-
-				if (!$provider->isAvailable($file)) {
-					continue;
-				}
-
-				$previewConcurrency = $this->getNumConcurrentPreviews('preview_concurrency_new');
-				$sem = self::guardWithSemaphore(self::SEMAPHORE_ID_NEW, $previewConcurrency);
-				try {
-					$this->logger->debug('Calling preview provider for {mimeType} with width={width}, height={height}', [
-						'mimeType' => $mimeType,
-						'width' => $width,
-						'height' => $height,
-					]);
-					$preview = $this->helper->getThumbnail($provider, $file, $width, $height);
-				} finally {
-					self::unguardWithSemaphore($sem);
-				}
-
-				if (!($preview instanceof IImage)) {
-					continue;
-				}
-
-				try {
-					$previewEntry = new Preview();
-					$previewEntry->setFileId($file->getId());
-					$previewEntry->setStorageId($file->getMountPoint()->getNumericStorageId());
-					$previewEntry->setSourceMimeType($file->getMimeType());
-					$previewEntry->setWidth($preview->width());
-					$previewEntry->setHeight($preview->height());
-					$previewEntry->setVersion($version);
-					$previewEntry->setMax($max);
-					$previewEntry->setCropped($crop);
-					$previewEntry->setEncrypted(false);
-					$previewEntry->setMimetype($preview->dataMimeType());
-					$previewEntry->setEtag($file->getEtag());
-					$previewEntry->setMtime((new \DateTime())->getTimestamp());
-					$previewEntry->setSize(0);
-					return $this->savePreview($previewEntry, $preview);
-				} catch (NotPermittedException) {
-					throw new NotFoundException();
-				}
-			}
-		}
-
-		throw new NotFoundException('No provider successfully handled the preview generation');
-	}
-
-	/**
-	 * @psalm-param IPreview::MODE_* $mode
-	 * @return int[]
-	 */
-	private function calculateSize(int $width, int $height, bool $crop, string $mode, int $maxWidth, int $maxHeight): array {
-		/*
+    public const SEMAPHORE_ID_ALL = 0x0a11;
+    public const SEMAPHORE_ID_NEW = 0x07ea;
+
+    public function __construct(
+        private IConfig $config,
+        private IPreview $previewManager,
+        private GeneratorHelper $helper,
+        private IEventDispatcher $eventDispatcher,
+        private LoggerInterface $logger,
+        private PreviewMapper $previewMapper,
+        private StorageFactory $storageFactory,
+    ) {
+    }
+
+    /**
+     * Returns a preview of a file
+     *
+     * The cache is searched first and if nothing usable was found then a preview is
+     * generated by one of the providers
+     *
+     * @return ISimpleFile
+     * @throws NotFoundException
+     * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
+     */
+    public function getPreview(
+        File $file,
+        int $width = -1,
+        int $height = -1,
+        bool $crop = false,
+        string $mode = IPreview::MODE_FILL,
+        ?string $mimeType = null,
+        bool $cacheResult = true,
+    ): ISimpleFile {
+        $specification = [
+            'width' => $width,
+            'height' => $height,
+            'crop' => $crop,
+            'mode' => $mode,
+        ];
+
+        $this->eventDispatcher->dispatchTyped(new BeforePreviewFetchedEvent(
+            $file,
+            $width,
+            $height,
+            $crop,
+            $mode,
+            $mimeType,
+        ));
+
+        $this->logger->debug('Requesting preview for {path} with width={width}, height={height}, crop={crop}, mode={mode}, mimeType={mimeType}', [
+            'path' => $file->getPath(),
+            'width' => $width,
+            'height' => $height,
+            'crop' => $crop,
+            'mode' => $mode,
+            'mimeType' => $mimeType,
+        ]);
+
+
+        // since we only ask for one preview, and the generate method return the last one it created, it returns the one we want
+        return $this->generatePreviews($file, [$specification], $mimeType, $cacheResult);
+    }
+
+    /**
+     * Generates previews of a file
+     *
+     * @throws NotFoundException
+     * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
+     */
+    public function generatePreviews(File $file, array $specifications, ?string $mimeType = null, bool $cacheResult = true): ISimpleFile {
+        //Make sure that we can read the file
+        if (!$file->isReadable()) {
+            $this->logger->warning('Cannot read file: {path}, skipping preview generation.', ['path' => $file->getPath()]);
+            throw new NotFoundException('Cannot read file');
+        }
+
+        if ($mimeType === null) {
+            $mimeType = $file->getMimeType();
+        }
+
+        [$file->getId() => $previews] = $this->previewMapper->getAvailablePreviews([$file->getId()]);
+
+        $previewVersion = null;
+        if ($file instanceof IVersionedPreviewFile) {
+            $previewVersion = $file->getPreviewVersion();
+        }
+
+        // Get the max preview and infer the max preview sizes from that
+        $maxPreview = $this->getMaxPreview($previews, $file, $mimeType, $previewVersion);
+        $maxPreviewImage = null; // only load the image when we need it
+        if ($maxPreview->getSize() === 0) {
+            $this->storageFactory->deletePreview($maxPreview);
+            $this->previewMapper->delete($maxPreview);
+            $this->logger->error('Max preview generated for file {path} has size 0, deleting and throwing exception.', ['path' => $file->getPath()]);
+            throw new NotFoundException('Max preview size 0, invalid!');
+        }
+
+        $maxWidth = $maxPreview->getWidth();
+        $maxHeight = $maxPreview->getHeight();
+
+        if ($maxWidth <= 0 || $maxHeight <= 0) {
+            throw new NotFoundException('The maximum preview sizes are zero or less pixels');
+        }
+
+        $previewFile = null;
+        foreach ($specifications as $specification) {
+            $width = $specification['width'] ?? -1;
+            $height = $specification['height'] ?? -1;
+            $crop = $specification['crop'] ?? false;
+            $mode = $specification['mode'] ?? IPreview::MODE_FILL;
+
+            // If both width and height are -1 we just want the max preview
+            if ($width === -1 && $height === -1) {
+                $width = $maxWidth;
+                $height = $maxHeight;
+            }
+
+            // Calculate the preview size
+            [$width, $height] = $this->calculateSize($width, $height, $crop, $mode, $maxWidth, $maxHeight);
+
+            // No need to generate a preview that is just the max preview
+            if ($width === $maxWidth && $height === $maxHeight) {
+                // ensure correct return value if this was the last one
+                $previewFile = new PreviewFile($maxPreview, $this->storageFactory, $this->previewMapper);
+                continue;
+            }
+
+            // Try to get a cached preview. Else generate (and store) one
+            try {
+                $preview = array_find($previews, fn (Preview $preview): bool => $preview->getWidth() === $width
+                    && $preview->getHeight() === $height && $preview->getMimetype() === $maxPreview->getMimetype()
+                    && $preview->getVersion() === $previewVersion && $preview->isCropped() === $crop);
+
+                if ($preview) {
+                    $previewFile = new PreviewFile($preview, $this->storageFactory, $this->previewMapper);
+                } else {
+                    if (!$this->previewManager->isMimeSupported($mimeType)) {
+                        throw new NotFoundException();
+                    }
+
+                    if ($maxPreviewImage === null) {
+                        $maxPreviewImage = $this->helper->getImage(new PreviewFile($maxPreview, $this->storageFactory, $this->previewMapper));
+                    }
+
+                    $this->logger->debug('Cached preview not found for file {path}, generating a new preview.', ['path' => $file->getPath()]);
+                    $previewFile = $this->generatePreview($file, $maxPreviewImage, $width, $height, $crop, $maxWidth, $maxHeight, $previewVersion, $cacheResult);
+                }
+            } catch (\InvalidArgumentException $e) {
+                throw new NotFoundException('', 0, $e);
+            }
+
+            if ($previewFile->getSize() === 0) {
+                $previewFile->delete();
+                throw new NotFoundException('Cached preview size 0, invalid!');
+            }
+        }
+        assert($previewFile !== null);
+
+        // Free memory being used by the embedded image resource.  Without this the image is kept in memory indefinitely.
+        // Garbage Collection does NOT free this memory.  We have to do it ourselves.
+        if ($maxPreviewImage instanceof \OCP\Image) {
+            $maxPreviewImage->destroy();
+        }
+
+        return $previewFile;
+    }
+
+    /**
+     * Acquire a semaphore of the specified id and concurrency, blocking if necessary.
+     * Return an identifier of the semaphore on success, which can be used to release it via
+     * {@see Generator::unguardWithSemaphore()}.
+     *
+     * @param int $semId
+     * @param int $concurrency
+     * @return false|\SysvSemaphore the semaphore on success or false on failure
+     */
+    public static function guardWithSemaphore(int $semId, int $concurrency) {
+        if (!extension_loaded('sysvsem')) {
+            return false;
+        }
+        $sem = sem_get($semId, $concurrency);
+        if ($sem === false) {
+            return false;
+        }
+        if (!sem_acquire($sem)) {
+            return false;
+        }
+        return $sem;
+    }
+
+    /**
+     * Releases the semaphore acquired from {@see Generator::guardWithSemaphore()}.
+     *
+     * @param false|\SysvSemaphore $semId the semaphore identifier returned by guardWithSemaphore
+     * @return bool
+     */
+    public static function unguardWithSemaphore(false|\SysvSemaphore $semId): bool {
+        if ($semId === false || !($semId instanceof \SysvSemaphore)) {
+            return false;
+        }
+        return sem_release($semId);
+    }
+
+    /**
+     * Get the number of concurrent threads supported by the host.
+     *
+     * @return int number of concurrent threads, or 0 if it cannot be determined
+     */
+    public static function getHardwareConcurrency(): int {
+        static $width;
+
+        if (!isset($width)) {
+            if (function_exists('ini_get')) {
+                $openBasedir = ini_get('open_basedir');
+                if (empty($openBasedir) || strpos($openBasedir, '/proc/cpuinfo') !== false) {
+                    $width = is_readable('/proc/cpuinfo') ? substr_count(file_get_contents('/proc/cpuinfo'), 'processor') : 0;
+                } else {
+                    $width = 0;
+                }
+            } else {
+                $width = 0;
+            }
+        }
+        return $width;
+    }
+
+    /**
+     * Get number of concurrent preview generations from system config
+     *
+     * Two config entries, `preview_concurrency_new` and `preview_concurrency_all`,
+     * are available. If not set, the default values are determined with the hardware concurrency
+     * of the host. In case the hardware concurrency cannot be determined, or the user sets an
+     * invalid value, fallback values are:
+     * For new images whose previews do not exist and need to be generated, 4;
+     * For all preview generation requests, 8.
+     * Value of `preview_concurrency_all` should be greater than or equal to that of
+     * `preview_concurrency_new`, otherwise, the latter is returned.
+     *
+     * @param string $type either `preview_concurrency_new` or `preview_concurrency_all`
+     * @return int number of concurrent preview generations, or -1 if $type is invalid
+     */
+    public function getNumConcurrentPreviews(string $type): int {
+        static $cached = [];
+        if (array_key_exists($type, $cached)) {
+            return $cached[$type];
+        }
+
+        $hardwareConcurrency = self::getHardwareConcurrency();
+        switch ($type) {
+            case 'preview_concurrency_all':
+                $fallback = $hardwareConcurrency > 0 ? $hardwareConcurrency * 2 : 8;
+                $concurrency_all = $this->config->getSystemValueInt($type, $fallback);
+                $concurrency_new = $this->getNumConcurrentPreviews('preview_concurrency_new');
+                $cached[$type] = max($concurrency_all, $concurrency_new);
+                break;
+            case 'preview_concurrency_new':
+                $fallback = $hardwareConcurrency > 0 ? $hardwareConcurrency : 4;
+                $cached[$type] = $this->config->getSystemValueInt($type, $fallback);
+                break;
+            default:
+                return -1;
+        }
+        return $cached[$type];
+    }
+
+    /**
+     * @param Preview[] $previews
+     * @throws NotFoundException
+     */
+    private function getMaxPreview(array $previews, File $file, string $mimeType, ?string $version): Preview {
+        // We don't know the max preview size, so we can't use getCachedPreview.
+        // It might have been generated with a higher resolution than the current value.
+        foreach ($previews as $preview) {
+            if ($preview->isMax() && ($version === $preview->getVersion())) {
+                return $preview;
+            }
+        }
+
+        $maxWidth = $this->config->getSystemValueInt('preview_max_x', 4096);
+        $maxHeight = $this->config->getSystemValueInt('preview_max_y', 4096);
+
+        return $this->generateProviderPreview($file, $maxWidth, $maxHeight, false, true, $mimeType, $version);
+    }
+
+    private function generateProviderPreview(File $file, int $width, int $height, bool $crop, bool $max, string $mimeType, ?string $version): Preview {
+        $previewProviders = $this->previewManager->getProviders();
+        foreach ($previewProviders as $supportedMimeType => $providers) {
+            // Filter out providers that does not support this mime
+            if (!preg_match($supportedMimeType, $mimeType)) {
+                continue;
+            }
+
+            foreach ($providers as $providerClosure) {
+
+                $provider = $this->helper->getProvider($providerClosure);
+                if (!$provider) {
+                    continue;
+                }
+
+                if (!$provider->isAvailable($file)) {
+                    continue;
+                }
+
+                $previewConcurrency = $this->getNumConcurrentPreviews('preview_concurrency_new');
+                $sem = self::guardWithSemaphore(self::SEMAPHORE_ID_NEW, $previewConcurrency);
+                try {
+                    $this->logger->debug('Calling preview provider for {mimeType} with width={width}, height={height}', [
+                        'mimeType' => $mimeType,
+                        'width' => $width,
+                        'height' => $height,
+                    ]);
+                    $preview = $this->helper->getThumbnail($provider, $file, $width, $height);
+                } finally {
+                    self::unguardWithSemaphore($sem);
+                }
+
+                if (!($preview instanceof IImage)) {
+                    continue;
+                }
+
+                try {
+                    $previewEntry = new Preview();
+                    $previewEntry->setFileId($file->getId());
+                    $previewEntry->setStorageId($file->getMountPoint()->getNumericStorageId());
+                    $previewEntry->setSourceMimeType($file->getMimeType());
+                    $previewEntry->setWidth($preview->width());
+                    $previewEntry->setHeight($preview->height());
+                    $previewEntry->setVersion($version);
+                    $previewEntry->setMax($max);
+                    $previewEntry->setCropped($crop);
+                    $previewEntry->setEncrypted(false);
+                    $previewEntry->setMimetype($preview->dataMimeType());
+                    $previewEntry->setEtag($file->getEtag());
+                    $previewEntry->setMtime((new \DateTime())->getTimestamp());
+                    $previewEntry->setSize(0);
+                    return $this->savePreview($previewEntry, $preview);
+                } catch (NotPermittedException) {
+                    throw new NotFoundException();
+                }
+            }
+        }
+
+        throw new NotFoundException('No provider successfully handled the preview generation');
+    }
+
+    /**
+     * @psalm-param IPreview::MODE_* $mode
+     * @return int[]
+     */
+    private function calculateSize(int $width, int $height, bool $crop, string $mode, int $maxWidth, int $maxHeight): array {
+        /*
 		 * If we are not cropping we have to make sure the requested image
 		 * respects the aspect ratio of the original.
 		 */
-		if (!$crop) {
-			$ratio = $maxHeight / $maxWidth;
+        if (!$crop) {
+            $ratio = $maxHeight / $maxWidth;
 
-			if ($width === -1) {
-				$width = $height / $ratio;
-			}
-			if ($height === -1) {
-				$height = $width * $ratio;
-			}
+            if ($width === -1) {
+                $width = $height / $ratio;
+            }
+            if ($height === -1) {
+                $height = $width * $ratio;
+            }
 
-			$ratioH = $height / $maxHeight;
-			$ratioW = $width / $maxWidth;
+            $ratioH = $height / $maxHeight;
+            $ratioW = $width / $maxWidth;
 
-			/*
+            /*
 			 * Fill means that the $height and $width are the max
 			 * Cover means min.
 			 */
-			if ($mode === IPreview::MODE_FILL) {
-				if ($ratioH > $ratioW) {
-					$height = $width * $ratio;
-				} else {
-					$width = $height / $ratio;
-				}
-			} elseif ($mode === IPreview::MODE_COVER) {
-				if ($ratioH > $ratioW) {
-					$width = $height / $ratio;
-				} else {
-					$height = $width * $ratio;
-				}
-			}
-		}
-
-		if ($height !== $maxHeight && $width !== $maxWidth) {
-			/*
+            if ($mode === IPreview::MODE_FILL) {
+                if ($ratioH > $ratioW) {
+                    $height = $width * $ratio;
+                } else {
+                    $width = $height / $ratio;
+                }
+            } elseif ($mode === IPreview::MODE_COVER) {
+                if ($ratioH > $ratioW) {
+                    $width = $height / $ratio;
+                } else {
+                    $height = $width * $ratio;
+                }
+            }
+        }
+
+        if ($height !== $maxHeight && $width !== $maxWidth) {
+            /*
 			 * Scale to the nearest power of four
 			 */
-			$pow4height = 4 ** ceil(log($height) / log(4));
-			$pow4width = 4 ** ceil(log($width) / log(4));
-
-			// Minimum size is 64
-			$pow4height = max($pow4height, 64);
-			$pow4width = max($pow4width, 64);
-
-			$ratioH = $height / $pow4height;
-			$ratioW = $width / $pow4width;
-
-			if ($ratioH < $ratioW) {
-				$width = $pow4width;
-				$height /= $ratioW;
-			} else {
-				$height = $pow4height;
-				$width /= $ratioH;
-			}
-		}
-
-		/*
+            $pow4height = 4 ** ceil(log($height) / log(4));
+            $pow4width = 4 ** ceil(log($width) / log(4));
+
+            // Minimum size is 64
+            $pow4height = max($pow4height, 64);
+            $pow4width = max($pow4width, 64);
+
+            $ratioH = $height / $pow4height;
+            $ratioW = $width / $pow4width;
+
+            if ($ratioH < $ratioW) {
+                $width = $pow4width;
+                $height /= $ratioW;
+            } else {
+                $height = $pow4height;
+                $width /= $ratioH;
+            }
+        }
+
+        /*
 		 * Make sure the requested height and width fall within the max
 		 * of the preview.
 		 */
-		if ($height > $maxHeight) {
-			$ratio = $height / $maxHeight;
-			$height = $maxHeight;
-			$width /= $ratio;
-		}
-		if ($width > $maxWidth) {
-			$ratio = $width / $maxWidth;
-			$width = $maxWidth;
-			$height /= $ratio;
-		}
-
-		return [(int)round($width), (int)round($height)];
-	}
-
-	/**
-	 * @throws NotFoundException
-	 * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
-	 */
-	private function generatePreview(
-		File $file,
-		IImage $maxPreview,
-		int $width,
-		int $height,
-		bool $crop,
-		int $maxWidth,
-		int $maxHeight,
-		?string $version,
-		bool $cacheResult,
-	): ISimpleFile {
-		$preview = $maxPreview;
-		if (!$preview->valid()) {
-			throw new \InvalidArgumentException('Failed to generate preview, failed to load image');
-		}
-
-		$previewConcurrency = $this->getNumConcurrentPreviews('preview_concurrency_new');
-		$sem = self::guardWithSemaphore(self::SEMAPHORE_ID_NEW, $previewConcurrency);
-		try {
-			if ($crop) {
-				if ($height !== $preview->height() && $width !== $preview->width()) {
-					//Resize
-					$widthR = $preview->width() / $width;
-					$heightR = $preview->height() / $height;
-
-					if ($widthR > $heightR) {
-						$scaleH = $height;
-						$scaleW = $maxWidth / $heightR;
-					} else {
-						$scaleH = $maxHeight / $widthR;
-						$scaleW = $width;
-					}
-					$preview = $preview->preciseResizeCopy((int)round($scaleW), (int)round($scaleH));
-				}
-				$cropX = (int)floor(abs($width - $preview->width()) * 0.5);
-				$cropY = (int)floor(abs($height - $preview->height()) * 0.5);
-				$preview = $preview->cropCopy($cropX, $cropY, $width, $height);
-			} else {
-				$preview = $maxPreview->resizeCopy(max($width, $height));
-			}
-		} finally {
-			self::unguardWithSemaphore($sem);
-		}
-
-		$previewEntry = new Preview();
-		$previewEntry->setFileId($file->getId());
-		$previewEntry->setStorageId($file->getMountPoint()->getNumericStorageId());
-		$previewEntry->setWidth($width);
-		$previewEntry->setSourceMimeType($file->getMimeType());
-		$previewEntry->setHeight($height);
-		$previewEntry->setVersion($version);
-		$previewEntry->setMax(false);
-		$previewEntry->setCropped($crop);
-		$previewEntry->setEncrypted(false);
-		$previewEntry->setMimeType($preview->dataMimeType());
-		$previewEntry->setEtag($file->getEtag());
-		$previewEntry->setMtime((new \DateTime())->getTimestamp());
-		$previewEntry->setSize(0);
-		if ($cacheResult) {
-			$previewEntry = $this->savePreview($previewEntry, $preview);
-			return new PreviewFile($previewEntry, $this->storageFactory, $this->previewMapper);
-		} else {
-			return new InMemoryFile($previewEntry->getName(), $preview->data());
-		}
-	}
-
-	/**
-	 * @throws InvalidPathException
-	 * @throws NotFoundException
-	 * @throws NotPermittedException
-	 * @throws \OCP\DB\Exception
-	 */
-	public function savePreview(Preview $previewEntry, IImage $preview): Preview {
-		$previewEntry = $this->previewMapper->insert($previewEntry);
-
-		// we need to save to DB first
-		try {
-			if ($preview instanceof IStreamImage) {
-				$size = $this->storageFactory->writePreview($previewEntry, $preview->resource());
-			} else {
-				$stream = fopen('php://temp', 'w+');
-				fwrite($stream, $preview->data());
-				rewind($stream);
-				$size = $this->storageFactory->writePreview($previewEntry, $stream);
-			}
-			if (!$size) {
-				throw new \RuntimeException('Unable to write preview file');
-			}
-		} catch (\Exception $e) {
-			$this->previewMapper->delete($previewEntry);
-			throw $e;
-		}
-		$previewEntry->setSize($size);
-		return $this->previewMapper->update($previewEntry);
-	}
+        if ($height > $maxHeight) {
+            $ratio = $height / $maxHeight;
+            $height = $maxHeight;
+            $width /= $ratio;
+        }
+        if ($width > $maxWidth) {
+            $ratio = $width / $maxWidth;
+            $width = $maxWidth;
+            $height /= $ratio;
+        }
+
+        return [(int)round($width), (int)round($height)];
+    }
+
+    /**
+     * @throws NotFoundException
+     * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
+     */
+    private function generatePreview(
+        File $file,
+        IImage $maxPreview,
+        int $width,
+        int $height,
+        bool $crop,
+        int $maxWidth,
+        int $maxHeight,
+        ?string $version,
+        bool $cacheResult,
+    ): ISimpleFile {
+        $preview = $maxPreview;
+        if (!$preview->valid()) {
+            throw new \InvalidArgumentException('Failed to generate preview, failed to load image');
+        }
+
+        $previewConcurrency = $this->getNumConcurrentPreviews('preview_concurrency_new');
+        $sem = self::guardWithSemaphore(self::SEMAPHORE_ID_NEW, $previewConcurrency);
+        try {
+            if ($crop) {
+                if ($height !== $preview->height() && $width !== $preview->width()) {
+                    //Resize
+                    $widthR = $preview->width() / $width;
+                    $heightR = $preview->height() / $height;
+
+                    if ($widthR > $heightR) {
+                        $scaleH = $height;
+                        $scaleW = $maxWidth / $heightR;
+                    } else {
+                        $scaleH = $maxHeight / $widthR;
+                        $scaleW = $width;
+                    }
+                    $preview = $preview->preciseResizeCopy((int)round($scaleW), (int)round($scaleH));
+                }
+                $cropX = (int)floor(abs($width - $preview->width()) * 0.5);
+                $cropY = (int)floor(abs($height - $preview->height()) * 0.5);
+                $preview = $preview->cropCopy($cropX, $cropY, $width, $height);
+            } else {
+                $preview = $maxPreview->resizeCopy(max($width, $height));
+            }
+        } finally {
+            self::unguardWithSemaphore($sem);
+        }
+
+        $previewEntry = new Preview();
+        $previewEntry->setFileId($file->getId());
+        $previewEntry->setStorageId($file->getMountPoint()->getNumericStorageId());
+        $previewEntry->setWidth($width);
+        $previewEntry->setSourceMimeType($file->getMimeType());
+        $previewEntry->setHeight($height);
+        $previewEntry->setVersion($version);
+        $previewEntry->setMax(false);
+        $previewEntry->setCropped($crop);
+        $previewEntry->setEncrypted(false);
+        $previewEntry->setMimeType($preview->dataMimeType());
+        $previewEntry->setEtag($file->getEtag());
+        $previewEntry->setMtime((new \DateTime())->getTimestamp());
+        $previewEntry->setSize(0);
+        if ($cacheResult) {
+            $previewEntry = $this->savePreview($previewEntry, $preview);
+            return new PreviewFile($previewEntry, $this->storageFactory, $this->previewMapper);
+        } else {
+            return new InMemoryFile($previewEntry->getName(), $preview->data());
+        }
+    }
+
+    /**
+     * @throws InvalidPathException
+     * @throws NotFoundException
+     * @throws NotPermittedException
+     * @throws \OCP\DB\Exception
+     */
+    public function savePreview(Preview $previewEntry, IImage $preview): Preview {
+        $previewEntry = $this->previewMapper->insert($previewEntry);
+
+        // we need to save to DB first
+        try {
+            if ($preview instanceof IStreamImage) {
+                $size = $this->storageFactory->writePreview($previewEntry, $preview->resource());
+            } else {
+                $stream = fopen('php://temp', 'w+');
+                fwrite($stream, $preview->data());
+                rewind($stream);
+                $size = $this->storageFactory->writePreview($previewEntry, $stream);
+            }
+            if (!$size) {
+                throw new \RuntimeException('Unable to write preview file');
+            }
+        } catch (\Exception $e) {
+            $this->previewMapper->delete($previewEntry);
+            throw $e;
+        }
+        $previewEntry->setSize($size);
+        return $this->previewMapper->update($previewEntry);
+    }
 }

lib/private/PreviewManager.php

Indentation +414 added lines, -414 removed lines

@@ -33,418 +33,418 @@
  * @psalm-import-type ProviderClosure from IPreview
  */
 class PreviewManager implements IPreview {
-	protected IConfig $config;
-	protected IRootFolder $rootFolder;
-	protected IEventDispatcher $eventDispatcher;
-	private ?Generator $generator = null;
-	private GeneratorHelper $helper;
-	protected bool $providerListDirty = false;
-	protected bool $registeredCoreProviders = false;
-	/**
-	 * @var array<string, list<ProviderClosure>> $providers
-	 */
-	protected array $providers = [];
-
-	/** @var array mime type => support status */
-	protected array $mimeTypeSupportMap = [];
-	/** @var ?list<class-string<IProviderV2>> $defaultProviders */
-	protected ?array $defaultProviders = null;
-	protected ?string $userId;
-	private Coordinator $bootstrapCoordinator;
-
-	/**
-	 * Hash map (without value) of loaded bootstrap providers
-	 * @psalm-var array<string, null>
-	 */
-	private array $loadedBootstrapProviders = [];
-	private ContainerInterface $container;
-	private IBinaryFinder $binaryFinder;
-	private IMagickSupport $imagickSupport;
-	private bool $enablePreviews;
-
-	public function __construct(
-		IConfig $config,
-		IRootFolder $rootFolder,
-		IEventDispatcher $eventDispatcher,
-		GeneratorHelper $helper,
-		?string $userId,
-		Coordinator $bootstrapCoordinator,
-		ContainerInterface $container,
-		IBinaryFinder $binaryFinder,
-		IMagickSupport $imagickSupport,
-	) {
-		$this->config = $config;
-		$this->rootFolder = $rootFolder;
-		$this->eventDispatcher = $eventDispatcher;
-		$this->helper = $helper;
-		$this->userId = $userId;
-		$this->bootstrapCoordinator = $bootstrapCoordinator;
-		$this->container = $container;
-		$this->binaryFinder = $binaryFinder;
-		$this->imagickSupport = $imagickSupport;
-		$this->enablePreviews = $config->getSystemValueBool('enable_previews', true);
-	}
-
-	/**
-	 * In order to improve lazy loading a closure can be registered which will be
-	 * called in case preview providers are actually requested
-	 *
-	 * @param string $mimeTypeRegex Regex with the mime types that are supported by this provider
-	 * @param ProviderClosure $callable
-	 */
-	public function registerProvider(string $mimeTypeRegex, Closure $callable): void {
-		if (!$this->enablePreviews) {
-			return;
-		}
-
-		if (!isset($this->providers[$mimeTypeRegex])) {
-			$this->providers[$mimeTypeRegex] = [];
-		}
-		$this->providers[$mimeTypeRegex][] = $callable;
-		$this->providerListDirty = true;
-	}
-
-	/**
-	 * Get all providers
-	 */
-	public function getProviders(): array {
-		if (!$this->enablePreviews) {
-			return [];
-		}
-
-		$this->registerCoreProviders();
-		$this->registerBootstrapProviders();
-		if ($this->providerListDirty) {
-			$keys = array_map('strlen', array_keys($this->providers));
-			array_multisort($keys, SORT_DESC, $this->providers);
-			$this->providerListDirty = false;
-		}
-
-		return $this->providers;
-	}
-
-	/**
-	 * Does the manager have any providers
-	 */
-	public function hasProviders(): bool {
-		$this->registerCoreProviders();
-		return !empty($this->providers);
-	}
-
-	private function getGenerator(): Generator {
-		if ($this->generator === null) {
-			$this->generator = new Generator(
-				$this->config,
-				$this,
-				new GeneratorHelper(),
-				$this->eventDispatcher,
-				$this->container->get(LoggerInterface::class),
-				$this->container->get(PreviewMapper::class),
-				$this->container->get(StorageFactory::class),
-			);
-		}
-		return $this->generator;
-	}
-
-	public function getPreview(
-		File $file,
-		int $width = -1,
-		int $height = -1,
-		bool $crop = false,
-		string $mode = IPreview::MODE_FILL,
-		?string $mimeType = null,
-		bool $cacheResult = true,
-	): ISimpleFile {
-		$this->throwIfPreviewsDisabled($file, $mimeType);
-		$previewConcurrency = $this->getGenerator()->getNumConcurrentPreviews('preview_concurrency_all');
-		$sem = Generator::guardWithSemaphore(Generator::SEMAPHORE_ID_ALL, $previewConcurrency);
-		try {
-			$preview = $this->getGenerator()->getPreview($file, $width, $height, $crop, $mode, $mimeType, $cacheResult);
-		} finally {
-			Generator::unguardWithSemaphore($sem);
-		}
-
-		return $preview;
-	}
-
-	/**
-	 * Generates previews of a file
-	 *
-	 * @param array $specifications
-	 * @return ISimpleFile the last preview that was generated
-	 * @throws NotFoundException
-	 * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
-	 * @since 19.0.0
-	 */
-	public function generatePreviews(File $file, array $specifications, ?string $mimeType = null): ISimpleFile {
-		$this->throwIfPreviewsDisabled($file, $mimeType);
-		return $this->getGenerator()->generatePreviews($file, $specifications, $mimeType);
-	}
-
-	public function isMimeSupported(string $mimeType = '*'): bool {
-		if (!$this->enablePreviews) {
-			return false;
-		}
-
-		if (isset($this->mimeTypeSupportMap[$mimeType])) {
-			return $this->mimeTypeSupportMap[$mimeType];
-		}
-
-		$this->registerCoreProviders();
-		$this->registerBootstrapProviders();
-		$providerMimeTypes = array_keys($this->providers);
-		foreach ($providerMimeTypes as $supportedMimeType) {
-			if (preg_match($supportedMimeType, $mimeType)) {
-				$this->mimeTypeSupportMap[$mimeType] = true;
-				return true;
-			}
-		}
-		$this->mimeTypeSupportMap[$mimeType] = false;
-		return false;
-	}
-
-	public function isAvailable(\OCP\Files\FileInfo $file, ?string $mimeType = null): bool {
-		if (!$this->enablePreviews) {
-			return false;
-		}
-
-		$fileMimeType = $mimeType ?? $file->getMimeType();
-
-		$this->registerCoreProviders();
-		if (!$this->isMimeSupported($fileMimeType)) {
-			return false;
-		}
-
-		$mount = $file->getMountPoint();
-		if ($mount && !$mount->getOption('previews', true)) {
-			return false;
-		}
-
-		foreach ($this->providers as $supportedMimeType => $providers) {
-			if (preg_match($supportedMimeType, $fileMimeType)) {
-				foreach ($providers as $providerClosure) {
-					$provider = $this->helper->getProvider($providerClosure);
-					if (!$provider) {
-						continue;
-					}
-					if ($provider->isAvailable($file)) {
-						return true;
-					}
-				}
-			}
-		}
-		return false;
-	}
-
-	/**
-	 * List of enabled default providers
-	 *
-	 * The following providers are enabled by default:
-	 *  - OC\Preview\PNG
-	 *  - OC\Preview\JPEG
-	 *  - OC\Preview\GIF
-	 *  - OC\Preview\BMP
-	 *  - OC\Preview\XBitmap
-	 *  - OC\Preview\MarkDown
-	 *  - OC\Preview\MP3
-	 *  - OC\Preview\TXT
-	 *
-	 * The following providers are disabled by default due to performance or privacy concerns:
-	 *  - OC\Preview\Font
-	 *  - OC\Preview\HEIC
-	 *  - OC\Preview\Illustrator
-	 *  - OC\Preview\Movie
-	 *  - OC\Preview\MSOfficeDoc
-	 *  - OC\Preview\MSOffice2003
-	 *  - OC\Preview\MSOffice2007
-	 *  - OC\Preview\OpenDocument
-	 *  - OC\Preview\PDF
-	 *  - OC\Preview\Photoshop
-	 *  - OC\Preview\Postscript
-	 *  - OC\Preview\StarOffice
-	 *  - OC\Preview\SVG
-	 *  - OC\Preview\TIFF
-	 *
-	 * @return list<class-string<IProviderV2>>
-	 */
-	protected function getEnabledDefaultProvider(): array {
-		if ($this->defaultProviders !== null) {
-			return $this->defaultProviders;
-		}
-
-		$imageProviders = [
-			Preview\PNG::class,
-			Preview\JPEG::class,
-			Preview\GIF::class,
-			Preview\BMP::class,
-			Preview\XBitmap::class,
-			Preview\Krita::class,
-			Preview\WebP::class,
-		];
-
-		$this->defaultProviders = $this->config->getSystemValue('enabledPreviewProviders', array_merge([
-			Preview\MarkDown::class,
-			Preview\MP3::class,
-			Preview\TXT::class,
-			Preview\OpenDocument::class,
-		], $imageProviders));
-
-		if (in_array(Preview\Image::class, $this->defaultProviders)) {
-			$this->defaultProviders = array_merge($this->defaultProviders, $imageProviders);
-		}
-		$this->defaultProviders = array_values(array_unique($this->defaultProviders));
-		/** @var list<class-string<IProviderV2>> $providers */
-		$providers = $this->defaultProviders;
-		return $providers;
-	}
-
-	/**
-	 * Register the default providers (if enabled)
-	 */
-	protected function registerCoreProvider(string $class, string $mimeType, array $options = []): void {
-		if (in_array(trim($class, '\\'), $this->getEnabledDefaultProvider())) {
-			$this->registerProvider($mimeType, function () use ($class, $options) {
-				return new $class($options);
-			});
-		}
-	}
-
-	/**
-	 * Register the default providers (if enabled)
-	 */
-	protected function registerCoreProviders(): void {
-		if ($this->registeredCoreProviders) {
-			return;
-		}
-		$this->registeredCoreProviders = true;
-
-		$this->registerCoreProvider(Preview\TXT::class, '/text\/plain/');
-		$this->registerCoreProvider(Preview\MarkDown::class, '/text\/(x-)?markdown/');
-		$this->registerCoreProvider(Preview\PNG::class, '/image\/png/');
-		$this->registerCoreProvider(Preview\JPEG::class, '/image\/jpeg/');
-		$this->registerCoreProvider(Preview\GIF::class, '/image\/gif/');
-		$this->registerCoreProvider(Preview\BMP::class, '/image\/bmp/');
-		$this->registerCoreProvider(Preview\XBitmap::class, '/image\/x-xbitmap/');
-		$this->registerCoreProvider(Preview\WebP::class, '/image\/webp/');
-		$this->registerCoreProvider(Preview\Krita::class, '/application\/x-krita/');
-		$this->registerCoreProvider(Preview\MP3::class, '/audio\/mpeg$/');
-		$this->registerCoreProvider(Preview\OpenDocument::class, '/application\/vnd.oasis.opendocument.*/');
-		$this->registerCoreProvider(Preview\Imaginary::class, Preview\Imaginary::supportedMimeTypes());
-		$this->registerCoreProvider(Preview\ImaginaryPDF::class, Preview\ImaginaryPDF::supportedMimeTypes());
-
-		// SVG and Bitmap require imagick
-		if ($this->imagickSupport->hasExtension()) {
-			$imagickProviders = [
-				'SVG' => ['mimetype' => '/image\/svg\+xml/', 'class' => Preview\SVG::class],
-				'TIFF' => ['mimetype' => '/image\/tiff/', 'class' => Preview\TIFF::class],
-				'PDF' => ['mimetype' => '/application\/pdf/', 'class' => Preview\PDF::class],
-				'AI' => ['mimetype' => '/application\/illustrator/', 'class' => Preview\Illustrator::class],
-				'PSD' => ['mimetype' => '/application\/x-photoshop/', 'class' => Preview\Photoshop::class],
-				'EPS' => ['mimetype' => '/application\/postscript/', 'class' => Preview\Postscript::class],
-				'TTF' => ['mimetype' => '/application\/(?:font-sfnt|x-font$)/', 'class' => Preview\Font::class],
-				'HEIC' => ['mimetype' => '/image\/(x-)?hei(f|c)/', 'class' => Preview\HEIC::class],
-				'TGA' => ['mimetype' => '/image\/(x-)?t(ar)?ga/', 'class' => Preview\TGA::class],
-				'SGI' => ['mimetype' => '/image\/(x-)?sgi/', 'class' => Preview\SGI::class],
-			];
-
-			foreach ($imagickProviders as $queryFormat => $provider) {
-				$class = $provider['class'];
-				if (!in_array(trim($class, '\\'), $this->getEnabledDefaultProvider())) {
-					continue;
-				}
-
-				if ($this->imagickSupport->supportsFormat($queryFormat)) {
-					$this->registerCoreProvider($class, $provider['mimetype']);
-				}
-			}
-		}
-
-		$this->registerCoreProvidersOffice();
-
-		// Video requires ffmpeg
-		if (in_array(Preview\Movie::class, $this->getEnabledDefaultProvider())) {
-			$movieBinary = $this->config->getSystemValue('preview_ffmpeg_path', null);
-			if (!is_string($movieBinary)) {
-				$movieBinary = $this->binaryFinder->findBinaryPath('ffmpeg');
-			}
-
-
-			if (is_string($movieBinary)) {
-				$this->registerCoreProvider(Preview\Movie::class, '/video\/.*/', ['movieBinary' => $movieBinary]);
-			}
-		}
-	}
-
-	private function registerCoreProvidersOffice(): void {
-		$officeProviders = [
-			['mimetype' => '/application\/msword/', 'class' => Preview\MSOfficeDoc::class],
-			['mimetype' => '/application\/vnd.ms-.*/', 'class' => Preview\MSOffice2003::class],
-			['mimetype' => '/application\/vnd.openxmlformats-officedocument.*/', 'class' => Preview\MSOffice2007::class],
-			['mimetype' => '/application\/vnd.oasis.opendocument.*/', 'class' => Preview\OpenDocument::class],
-			['mimetype' => '/application\/vnd.sun.xml.*/', 'class' => Preview\StarOffice::class],
-			['mimetype' => '/image\/emf/', 'class' => Preview\EMF::class],
-		];
-
-		$findBinary = true;
-		$officeBinary = false;
-
-		foreach ($officeProviders as $provider) {
-			$class = $provider['class'];
-			if (!in_array(trim($class, '\\'), $this->getEnabledDefaultProvider())) {
-				continue;
-			}
-
-			if ($findBinary) {
-				// Office requires openoffice or libreoffice
-				$officeBinary = $this->config->getSystemValue('preview_libreoffice_path', false);
-				if ($officeBinary === false) {
-					$officeBinary = $this->binaryFinder->findBinaryPath('libreoffice');
-				}
-				if ($officeBinary === false) {
-					$officeBinary = $this->binaryFinder->findBinaryPath('openoffice');
-				}
-				$findBinary = false;
-			}
-
-			if ($officeBinary) {
-				$this->registerCoreProvider($class, $provider['mimetype'], ['officeBinary' => $officeBinary]);
-			}
-		}
-	}
-
-	private function registerBootstrapProviders(): void {
-		$context = $this->bootstrapCoordinator->getRegistrationContext();
-
-		if ($context === null) {
-			// Just ignore for now
-			return;
-		}
-
-		$providers = $context->getPreviewProviders();
-		foreach ($providers as $provider) {
-			$key = $provider->getMimeTypeRegex() . '-' . $provider->getService();
-			if (array_key_exists($key, $this->loadedBootstrapProviders)) {
-				// Do not load the provider more than once
-				continue;
-			}
-			$this->loadedBootstrapProviders[$key] = null;
-
-			$this->registerProvider($provider->getMimeTypeRegex(), function () use ($provider): IProviderV2|false {
-				try {
-					return $this->container->get($provider->getService());
-				} catch (NotFoundExceptionInterface) {
-					return false;
-				}
-			});
-		}
-	}
-
-	/**
-	 * @throws NotFoundException if preview generation is disabled
-	 */
-	private function throwIfPreviewsDisabled(File $file, ?string $mimeType = null): void {
-		if (!$this->isAvailable($file, $mimeType)) {
-			throw new NotFoundException('Previews disabled');
-		}
-	}
+    protected IConfig $config;
+    protected IRootFolder $rootFolder;
+    protected IEventDispatcher $eventDispatcher;
+    private ?Generator $generator = null;
+    private GeneratorHelper $helper;
+    protected bool $providerListDirty = false;
+    protected bool $registeredCoreProviders = false;
+    /**
+     * @var array<string, list<ProviderClosure>> $providers
+     */
+    protected array $providers = [];
+
+    /** @var array mime type => support status */
+    protected array $mimeTypeSupportMap = [];
+    /** @var ?list<class-string<IProviderV2>> $defaultProviders */
+    protected ?array $defaultProviders = null;
+    protected ?string $userId;
+    private Coordinator $bootstrapCoordinator;
+
+    /**
+     * Hash map (without value) of loaded bootstrap providers
+     * @psalm-var array<string, null>
+     */
+    private array $loadedBootstrapProviders = [];
+    private ContainerInterface $container;
+    private IBinaryFinder $binaryFinder;
+    private IMagickSupport $imagickSupport;
+    private bool $enablePreviews;
+
+    public function __construct(
+        IConfig $config,
+        IRootFolder $rootFolder,
+        IEventDispatcher $eventDispatcher,
+        GeneratorHelper $helper,
+        ?string $userId,
+        Coordinator $bootstrapCoordinator,
+        ContainerInterface $container,
+        IBinaryFinder $binaryFinder,
+        IMagickSupport $imagickSupport,
+    ) {
+        $this->config = $config;
+        $this->rootFolder = $rootFolder;
+        $this->eventDispatcher = $eventDispatcher;
+        $this->helper = $helper;
+        $this->userId = $userId;
+        $this->bootstrapCoordinator = $bootstrapCoordinator;
+        $this->container = $container;
+        $this->binaryFinder = $binaryFinder;
+        $this->imagickSupport = $imagickSupport;
+        $this->enablePreviews = $config->getSystemValueBool('enable_previews', true);
+    }
+
+    /**
+     * In order to improve lazy loading a closure can be registered which will be
+     * called in case preview providers are actually requested
+     *
+     * @param string $mimeTypeRegex Regex with the mime types that are supported by this provider
+     * @param ProviderClosure $callable
+     */
+    public function registerProvider(string $mimeTypeRegex, Closure $callable): void {
+        if (!$this->enablePreviews) {
+            return;
+        }
+
+        if (!isset($this->providers[$mimeTypeRegex])) {
+            $this->providers[$mimeTypeRegex] = [];
+        }
+        $this->providers[$mimeTypeRegex][] = $callable;
+        $this->providerListDirty = true;
+    }
+
+    /**
+     * Get all providers
+     */
+    public function getProviders(): array {
+        if (!$this->enablePreviews) {
+            return [];
+        }
+
+        $this->registerCoreProviders();
+        $this->registerBootstrapProviders();
+        if ($this->providerListDirty) {
+            $keys = array_map('strlen', array_keys($this->providers));
+            array_multisort($keys, SORT_DESC, $this->providers);
+            $this->providerListDirty = false;
+        }
+
+        return $this->providers;
+    }
+
+    /**
+     * Does the manager have any providers
+     */
+    public function hasProviders(): bool {
+        $this->registerCoreProviders();
+        return !empty($this->providers);
+    }
+
+    private function getGenerator(): Generator {
+        if ($this->generator === null) {
+            $this->generator = new Generator(
+                $this->config,
+                $this,
+                new GeneratorHelper(),
+                $this->eventDispatcher,
+                $this->container->get(LoggerInterface::class),
+                $this->container->get(PreviewMapper::class),
+                $this->container->get(StorageFactory::class),
+            );
+        }
+        return $this->generator;
+    }
+
+    public function getPreview(
+        File $file,
+        int $width = -1,
+        int $height = -1,
+        bool $crop = false,
+        string $mode = IPreview::MODE_FILL,
+        ?string $mimeType = null,
+        bool $cacheResult = true,
+    ): ISimpleFile {
+        $this->throwIfPreviewsDisabled($file, $mimeType);
+        $previewConcurrency = $this->getGenerator()->getNumConcurrentPreviews('preview_concurrency_all');
+        $sem = Generator::guardWithSemaphore(Generator::SEMAPHORE_ID_ALL, $previewConcurrency);
+        try {
+            $preview = $this->getGenerator()->getPreview($file, $width, $height, $crop, $mode, $mimeType, $cacheResult);
+        } finally {
+            Generator::unguardWithSemaphore($sem);
+        }
+
+        return $preview;
+    }
+
+    /**
+     * Generates previews of a file
+     *
+     * @param array $specifications
+     * @return ISimpleFile the last preview that was generated
+     * @throws NotFoundException
+     * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
+     * @since 19.0.0
+     */
+    public function generatePreviews(File $file, array $specifications, ?string $mimeType = null): ISimpleFile {
+        $this->throwIfPreviewsDisabled($file, $mimeType);
+        return $this->getGenerator()->generatePreviews($file, $specifications, $mimeType);
+    }
+
+    public function isMimeSupported(string $mimeType = '*'): bool {
+        if (!$this->enablePreviews) {
+            return false;
+        }
+
+        if (isset($this->mimeTypeSupportMap[$mimeType])) {
+            return $this->mimeTypeSupportMap[$mimeType];
+        }
+
+        $this->registerCoreProviders();
+        $this->registerBootstrapProviders();
+        $providerMimeTypes = array_keys($this->providers);
+        foreach ($providerMimeTypes as $supportedMimeType) {
+            if (preg_match($supportedMimeType, $mimeType)) {
+                $this->mimeTypeSupportMap[$mimeType] = true;
+                return true;
+            }
+        }
+        $this->mimeTypeSupportMap[$mimeType] = false;
+        return false;
+    }
+
+    public function isAvailable(\OCP\Files\FileInfo $file, ?string $mimeType = null): bool {
+        if (!$this->enablePreviews) {
+            return false;
+        }
+
+        $fileMimeType = $mimeType ?? $file->getMimeType();
+
+        $this->registerCoreProviders();
+        if (!$this->isMimeSupported($fileMimeType)) {
+            return false;
+        }
+
+        $mount = $file->getMountPoint();
+        if ($mount && !$mount->getOption('previews', true)) {
+            return false;
+        }
+
+        foreach ($this->providers as $supportedMimeType => $providers) {
+            if (preg_match($supportedMimeType, $fileMimeType)) {
+                foreach ($providers as $providerClosure) {
+                    $provider = $this->helper->getProvider($providerClosure);
+                    if (!$provider) {
+                        continue;
+                    }
+                    if ($provider->isAvailable($file)) {
+                        return true;
+                    }
+                }
+            }
+        }
+        return false;
+    }
+
+    /**
+     * List of enabled default providers
+     *
+     * The following providers are enabled by default:
+     *  - OC\Preview\PNG
+     *  - OC\Preview\JPEG
+     *  - OC\Preview\GIF
+     *  - OC\Preview\BMP
+     *  - OC\Preview\XBitmap
+     *  - OC\Preview\MarkDown
+     *  - OC\Preview\MP3
+     *  - OC\Preview\TXT
+     *
+     * The following providers are disabled by default due to performance or privacy concerns:
+     *  - OC\Preview\Font
+     *  - OC\Preview\HEIC
+     *  - OC\Preview\Illustrator
+     *  - OC\Preview\Movie
+     *  - OC\Preview\MSOfficeDoc
+     *  - OC\Preview\MSOffice2003
+     *  - OC\Preview\MSOffice2007
+     *  - OC\Preview\OpenDocument
+     *  - OC\Preview\PDF
+     *  - OC\Preview\Photoshop
+     *  - OC\Preview\Postscript
+     *  - OC\Preview\StarOffice
+     *  - OC\Preview\SVG
+     *  - OC\Preview\TIFF
+     *
+     * @return list<class-string<IProviderV2>>
+     */
+    protected function getEnabledDefaultProvider(): array {
+        if ($this->defaultProviders !== null) {
+            return $this->defaultProviders;
+        }
+
+        $imageProviders = [
+            Preview\PNG::class,
+            Preview\JPEG::class,
+            Preview\GIF::class,
+            Preview\BMP::class,
+            Preview\XBitmap::class,
+            Preview\Krita::class,
+            Preview\WebP::class,
+        ];
+
+        $this->defaultProviders = $this->config->getSystemValue('enabledPreviewProviders', array_merge([
+            Preview\MarkDown::class,
+            Preview\MP3::class,
+            Preview\TXT::class,
+            Preview\OpenDocument::class,
+        ], $imageProviders));
+
+        if (in_array(Preview\Image::class, $this->defaultProviders)) {
+            $this->defaultProviders = array_merge($this->defaultProviders, $imageProviders);
+        }
+        $this->defaultProviders = array_values(array_unique($this->defaultProviders));
+        /** @var list<class-string<IProviderV2>> $providers */
+        $providers = $this->defaultProviders;
+        return $providers;
+    }
+
+    /**
+     * Register the default providers (if enabled)
+     */
+    protected function registerCoreProvider(string $class, string $mimeType, array $options = []): void {
+        if (in_array(trim($class, '\\'), $this->getEnabledDefaultProvider())) {
+            $this->registerProvider($mimeType, function () use ($class, $options) {
+                return new $class($options);
+            });
+        }
+    }
+
+    /**
+     * Register the default providers (if enabled)
+     */
+    protected function registerCoreProviders(): void {
+        if ($this->registeredCoreProviders) {
+            return;
+        }
+        $this->registeredCoreProviders = true;
+
+        $this->registerCoreProvider(Preview\TXT::class, '/text\/plain/');
+        $this->registerCoreProvider(Preview\MarkDown::class, '/text\/(x-)?markdown/');
+        $this->registerCoreProvider(Preview\PNG::class, '/image\/png/');
+        $this->registerCoreProvider(Preview\JPEG::class, '/image\/jpeg/');
+        $this->registerCoreProvider(Preview\GIF::class, '/image\/gif/');
+        $this->registerCoreProvider(Preview\BMP::class, '/image\/bmp/');
+        $this->registerCoreProvider(Preview\XBitmap::class, '/image\/x-xbitmap/');
+        $this->registerCoreProvider(Preview\WebP::class, '/image\/webp/');
+        $this->registerCoreProvider(Preview\Krita::class, '/application\/x-krita/');
+        $this->registerCoreProvider(Preview\MP3::class, '/audio\/mpeg$/');
+        $this->registerCoreProvider(Preview\OpenDocument::class, '/application\/vnd.oasis.opendocument.*/');
+        $this->registerCoreProvider(Preview\Imaginary::class, Preview\Imaginary::supportedMimeTypes());
+        $this->registerCoreProvider(Preview\ImaginaryPDF::class, Preview\ImaginaryPDF::supportedMimeTypes());
+
+        // SVG and Bitmap require imagick
+        if ($this->imagickSupport->hasExtension()) {
+            $imagickProviders = [
+                'SVG' => ['mimetype' => '/image\/svg\+xml/', 'class' => Preview\SVG::class],
+                'TIFF' => ['mimetype' => '/image\/tiff/', 'class' => Preview\TIFF::class],
+                'PDF' => ['mimetype' => '/application\/pdf/', 'class' => Preview\PDF::class],
+                'AI' => ['mimetype' => '/application\/illustrator/', 'class' => Preview\Illustrator::class],
+                'PSD' => ['mimetype' => '/application\/x-photoshop/', 'class' => Preview\Photoshop::class],
+                'EPS' => ['mimetype' => '/application\/postscript/', 'class' => Preview\Postscript::class],
+                'TTF' => ['mimetype' => '/application\/(?:font-sfnt|x-font$)/', 'class' => Preview\Font::class],
+                'HEIC' => ['mimetype' => '/image\/(x-)?hei(f|c)/', 'class' => Preview\HEIC::class],
+                'TGA' => ['mimetype' => '/image\/(x-)?t(ar)?ga/', 'class' => Preview\TGA::class],
+                'SGI' => ['mimetype' => '/image\/(x-)?sgi/', 'class' => Preview\SGI::class],
+            ];
+
+            foreach ($imagickProviders as $queryFormat => $provider) {
+                $class = $provider['class'];
+                if (!in_array(trim($class, '\\'), $this->getEnabledDefaultProvider())) {
+                    continue;
+                }
+
+                if ($this->imagickSupport->supportsFormat($queryFormat)) {
+                    $this->registerCoreProvider($class, $provider['mimetype']);
+                }
+            }
+        }
+
+        $this->registerCoreProvidersOffice();
+
+        // Video requires ffmpeg
+        if (in_array(Preview\Movie::class, $this->getEnabledDefaultProvider())) {
+            $movieBinary = $this->config->getSystemValue('preview_ffmpeg_path', null);
+            if (!is_string($movieBinary)) {
+                $movieBinary = $this->binaryFinder->findBinaryPath('ffmpeg');
+            }
+
+
+            if (is_string($movieBinary)) {
+                $this->registerCoreProvider(Preview\Movie::class, '/video\/.*/', ['movieBinary' => $movieBinary]);
+            }
+        }
+    }
+
+    private function registerCoreProvidersOffice(): void {
+        $officeProviders = [
+            ['mimetype' => '/application\/msword/', 'class' => Preview\MSOfficeDoc::class],
+            ['mimetype' => '/application\/vnd.ms-.*/', 'class' => Preview\MSOffice2003::class],
+            ['mimetype' => '/application\/vnd.openxmlformats-officedocument.*/', 'class' => Preview\MSOffice2007::class],
+            ['mimetype' => '/application\/vnd.oasis.opendocument.*/', 'class' => Preview\OpenDocument::class],
+            ['mimetype' => '/application\/vnd.sun.xml.*/', 'class' => Preview\StarOffice::class],
+            ['mimetype' => '/image\/emf/', 'class' => Preview\EMF::class],
+        ];
+
+        $findBinary = true;
+        $officeBinary = false;
+
+        foreach ($officeProviders as $provider) {
+            $class = $provider['class'];
+            if (!in_array(trim($class, '\\'), $this->getEnabledDefaultProvider())) {
+                continue;
+            }
+
+            if ($findBinary) {
+                // Office requires openoffice or libreoffice
+                $officeBinary = $this->config->getSystemValue('preview_libreoffice_path', false);
+                if ($officeBinary === false) {
+                    $officeBinary = $this->binaryFinder->findBinaryPath('libreoffice');
+                }
+                if ($officeBinary === false) {
+                    $officeBinary = $this->binaryFinder->findBinaryPath('openoffice');
+                }
+                $findBinary = false;
+            }
+
+            if ($officeBinary) {
+                $this->registerCoreProvider($class, $provider['mimetype'], ['officeBinary' => $officeBinary]);
+            }
+        }
+    }
+
+    private function registerBootstrapProviders(): void {
+        $context = $this->bootstrapCoordinator->getRegistrationContext();
+
+        if ($context === null) {
+            // Just ignore for now
+            return;
+        }
+
+        $providers = $context->getPreviewProviders();
+        foreach ($providers as $provider) {
+            $key = $provider->getMimeTypeRegex() . '-' . $provider->getService();
+            if (array_key_exists($key, $this->loadedBootstrapProviders)) {
+                // Do not load the provider more than once
+                continue;
+            }
+            $this->loadedBootstrapProviders[$key] = null;
+
+            $this->registerProvider($provider->getMimeTypeRegex(), function () use ($provider): IProviderV2|false {
+                try {
+                    return $this->container->get($provider->getService());
+                } catch (NotFoundExceptionInterface) {
+                    return false;
+                }
+            });
+        }
+    }
+
+    /**
+     * @throws NotFoundException if preview generation is disabled
+     */
+    private function throwIfPreviewsDisabled(File $file, ?string $mimeType = null): void {
+        if (!$this->isAvailable($file, $mimeType)) {
+            throw new NotFoundException('Previews disabled');
+        }
+    }
 }

lib/public/IPreview.php

Indentation +72 added lines, -72 removed lines

@@ -25,84 +25,84 @@
  */
 #[Consumable(since: '6.0.0')]
 interface IPreview {
-	/**
-	 * @since 11.0.0
-	 */
-	public const MODE_FILL = 'fill';
+    /**
+     * @since 11.0.0
+     */
+    public const MODE_FILL = 'fill';
 
-	/**
-	 * @since 11.0.0
-	 */
-	public const MODE_COVER = 'cover';
+    /**
+     * @since 11.0.0
+     */
+    public const MODE_COVER = 'cover';
 
-	/**
-	 * In order to improve lazy loading a closure can be registered which will be
-	 * called in case preview providers are actually requested
-	 *
-	 * @param string $mimeTypeRegex Regex with the mime types that are supported by this provider
-	 * @param ProviderClosure $callable
-	 * @since 8.1.0
-	 * @see \OCP\AppFramework\Bootstrap\IRegistrationContext::registerPreviewProvider
-	 *
-	 * @deprecated 23.0.0 Register your provider via the IRegistrationContext when booting the app
-	 */
-	public function registerProvider(string $mimeTypeRegex, Closure $callable): void;
+    /**
+     * In order to improve lazy loading a closure can be registered which will be
+     * called in case preview providers are actually requested
+     *
+     * @param string $mimeTypeRegex Regex with the mime types that are supported by this provider
+     * @param ProviderClosure $callable
+     * @since 8.1.0
+     * @see \OCP\AppFramework\Bootstrap\IRegistrationContext::registerPreviewProvider
+     *
+     * @deprecated 23.0.0 Register your provider via the IRegistrationContext when booting the app
+     */
+    public function registerProvider(string $mimeTypeRegex, Closure $callable): void;
 
-	/**
-	 * Get all providers
-	 * @return array<string, list<ProviderClosure>>
-	 * @since 8.1.0
-	 */
-	public function getProviders(): array;
+    /**
+     * Get all providers
+     * @return array<string, list<ProviderClosure>>
+     * @since 8.1.0
+     */
+    public function getProviders(): array;
 
-	/**
-	 * Does the manager have any providers
-	 * @since 8.1.0
-	 */
-	public function hasProviders(): bool;
+    /**
+     * Does the manager have any providers
+     * @since 8.1.0
+     */
+    public function hasProviders(): bool;
 
-	/**
-	 * Returns a preview of a file
-	 *
-	 * The cache is searched first and if nothing usable was found then a preview is
-	 * generated by one of the providers
-	 *
-	 * @param IPreview::MODE_* $mode
-	 * @param string $mimeType To force a given mimetype for the file (files_versions needs this)
-	 * @param bool $cacheResult Whether to cache the preview on the filesystem. Default to true. Can be useful to set to false to limit the amount of stored previews.
-	 * @return ISimpleFile
-	 * @throws NotFoundException
-	 * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
-	 * @since 11.0.0 - \InvalidArgumentException was added in 12.0.0
-	 * @since 32.0.0 - getPreview($cacheResult) added the $cacheResult argument to the signature
-	 */
-	public function getPreview(File $file, int $width = -1, int $height = -1, bool $crop = false, string $mode = IPreview::MODE_FILL, ?string $mimeType = null, bool $cacheResult = true): ISimpleFile;
+    /**
+     * Returns a preview of a file
+     *
+     * The cache is searched first and if nothing usable was found then a preview is
+     * generated by one of the providers
+     *
+     * @param IPreview::MODE_* $mode
+     * @param string $mimeType To force a given mimetype for the file (files_versions needs this)
+     * @param bool $cacheResult Whether to cache the preview on the filesystem. Default to true. Can be useful to set to false to limit the amount of stored previews.
+     * @return ISimpleFile
+     * @throws NotFoundException
+     * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
+     * @since 11.0.0 - \InvalidArgumentException was added in 12.0.0
+     * @since 32.0.0 - getPreview($cacheResult) added the $cacheResult argument to the signature
+     */
+    public function getPreview(File $file, int $width = -1, int $height = -1, bool $crop = false, string $mode = IPreview::MODE_FILL, ?string $mimeType = null, bool $cacheResult = true): ISimpleFile;
 
-	/**
-	 * Returns true if the passed mime type is supported
-	 * @param string $mimeType A glob
-	 * @since 6.0.0
-	 */
-	public function isMimeSupported(string $mimeType = '*'): bool;
+    /**
+     * Returns true if the passed mime type is supported
+     * @param string $mimeType A glob
+     * @since 6.0.0
+     */
+    public function isMimeSupported(string $mimeType = '*'): bool;
 
-	/**
-	 * Check if a preview can be generated for a file
-	 *
-	 * @param FileInfo $file
-	 * @param string|null $mimeType To force a given mimetype for the file
-	 * @since 8.0.0
-	 * @since 32.0.0 - isAvailable($mimeType) added the $mimeType argument to the signature
-	 */
-	public function isAvailable(FileInfo $file, ?string $mimeType = null): bool;
+    /**
+     * Check if a preview can be generated for a file
+     *
+     * @param FileInfo $file
+     * @param string|null $mimeType To force a given mimetype for the file
+     * @since 8.0.0
+     * @since 32.0.0 - isAvailable($mimeType) added the $mimeType argument to the signature
+     */
+    public function isAvailable(FileInfo $file, ?string $mimeType = null): bool;
 
-	/**
-	 * Generates previews of a file
-	 *
-	 * @param array $specifications
-	 * @return ISimpleFile the last preview that was generated
-	 * @throws NotFoundException
-	 * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
-	 * @since 19.0.0
-	 */
-	public function generatePreviews(File $file, array $specifications, ?string $mimeType = null): ISimpleFile;
+    /**
+     * Generates previews of a file
+     *
+     * @param array $specifications
+     * @return ISimpleFile the last preview that was generated
+     * @throws NotFoundException
+     * @throws \InvalidArgumentException if the preview would be invalid (in case the original image is invalid)
+     * @since 19.0.0
+     */
+    public function generatePreviews(File $file, array $specifications, ?string $mimeType = null): ISimpleFile;
 }