Only call resource provider if type matches

Signed-off-by: Julius Härtl <jus@bitgrid.net>

2 files changed, 36 insertions, 10 deletions

diff --git a/lib/private/Collaboration/Resources/Manager.php b/lib/private/Collaboration/Resources/Manager.php
index 071fa3c9436..1214145bd03 100644
--- a/lib/private/Collaboration/Resources/Manager.php
+++ b/lib/private/Collaboration/Resources/Manager.php
@@ -110,9 +110,11 @@ class Manager implements IManager {
 	 */
 	public function getName(IResource $resource): string {
 		foreach ($this->getProviders() as $provider) {
-			try {
-				return $provider->getName($resource);
-			} catch (ResourceException $e) {
+			if ($provider->getType() === $resource->getType()) {
+				try {
+					return $provider->getName($resource);
+				} catch (ResourceException $e) {
+				}
 			}
 		}
 
@@ -126,9 +128,11 @@ class Manager implements IManager {
 	 */
 	public function getIconClass(IResource $resource): string {
 		foreach ($this->getProviders() as $provider) {
-			try {
-				return $provider->getIconClass($resource);
-			} catch (ResourceException $e) {
+			if ($provider->getType() === $resource->getType()) {
+				try {
+					return $provider->getIconClass($resource);
+				} catch (ResourceException $e) {
+				}
 			}
 		}
 
@@ -145,11 +149,13 @@ class Manager implements IManager {
 	 */
 	public function canAccess(IResource $resource, IUser $user = null): bool {
 		foreach ($this->getProviders() as $provider) {
-			try {
-				if ($provider->canAccess($resource, $user)) {
-					return true;
+			if ($provider->getType() === $resource->getType()) {
+				try {
+					if ($provider->canAccess($resource, $user)) {
+						return true;
+					}
+				} catch (ResourceException $e) {
 				}
-			} catch (ResourceException $e) {
 			}
 		}
 
@@ -162,4 +168,15 @@ class Manager implements IManager {
 	public function registerResourceProvider(IProvider $provider): void {
 		$this->providers[] = $provider;
 	}
+
+	/**
+	 * Get the type of a resource
+	 *
+	 * @param IResource $resource
+	 * @return string
+	 * @since 15.0.0
+	 */
+	public function getType(): string {
+		return '';
+	}
 }
diff --git a/lib/public/Collaboration/Resources/IProvider.php b/lib/public/Collaboration/Resources/IProvider.php
index c5f6f7ed560..0299bfd0ddf 100644
--- a/lib/public/Collaboration/Resources/IProvider.php
+++ b/lib/public/Collaboration/Resources/IProvider.php
@@ -27,6 +27,15 @@ use OCP\IUser;
 interface IProvider {
 
 	/**
+	 * Get the type of a resource
+	 *
+	 * @param IResource $resource
+	 * @return string
+	 * @since 15.0.0
+	 */
+	public function getType(): string;
+
+	/**
 	 * Get the display name of a resource
 	 *
 	 * @param IResource $resource