Register default share template provider

+ Properly return the first provider in the share template factory Signed-off-by: Louis Chemineau <[email protected]>
nextcloud · Dec 22, 2022 · 8df671e · 8df671e
1 parent 1d1f16e
commit 8df671e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/apps/files_sharing/lib/AppInfo/Application.php b/apps/files_sharing/lib/AppInfo/Application.php
@@ -53,6 +53,7 @@
 use OCA\Files_Sharing\Notification\Notifier;
 use OCA\Files\Event\LoadAdditionalScriptsEvent;
 use OCA\Files\Event\LoadSidebar;
+use OCA\Files_Sharing\DefaultShareDisplayTemplateProvider;
 use OCP\Files\Event\BeforeDirectGetEvent;
 use OCA\Files_Sharing\ShareBackend\File;
 use OCA\Files_Sharing\ShareBackend\Folder;
@@ -131,6 +132,10 @@ public function boot(IBootContext $context): void {
 		 * Always add main sharing script
 		 */
 		Util::addScript(self::APP_ID, 'main');
+
+		$container = $context->getServerContainer();
+		$factory = $container->get(\OCP\Share\IShareDisplayTemplateFactory::class);
+		$factory->registerDisplayShareTemplate(DefaultShareDisplayTemplateProvider::class);
 	}
 
 

diff --git a/lib/private/Share20/ShareDisplayTemplateFactory.php b/lib/private/Share20/ShareDisplayTemplateFactory.php
@@ -27,6 +27,6 @@ public function getTemplateProvider(IShare $share): IShareDisplayTemplateProvide
 		);
 		usort($providers, fn (IShareDisplayTemplateProvider $a, IShareDisplayTemplateProvider $b) => $b->getPriority() - $a->getPriority());
 		$filteredProviders = array_filter($providers, fn (IShareDisplayTemplateProvider $provider) => $provider->shouldRespond($share));
-		return $filteredProviders[0];
+		return array_shift($filteredProviders);
 	}
 }