handle and return touchProvider errors

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

2 files changed, 5 insertions, 3 deletions

diff --git a/lib/private/Collaboration/Reference/ReferenceManager.php b/lib/private/Collaboration/Reference/ReferenceManager.php
index 381f3aea8f4..5db30a50ea6 100644
--- a/lib/private/Collaboration/Reference/ReferenceManager.php
+++ b/lib/private/Collaboration/Reference/ReferenceManager.php
@@ -242,7 +242,7 @@ class ReferenceManager implements IReferenceManager {
 	/**
 	 * @inheritDoc
 	 */
-	public function touchProvider(string $userId, string $providerId, ?int $timestamp = null): void {
+	public function touchProvider(string $userId, string $providerId, ?int $timestamp = null): bool {
 		$providers = $this->getDiscoverableProviders();
 		$providerIds = array_map(static function (IDiscoverableReferenceProvider $provider) {
 			return $provider->getId();
@@ -254,7 +254,9 @@ class ReferenceManager implements IReferenceManager {
 			}
 
 			$this->config->setUserValue($userId, 'references', $configKey, (string) $timestamp);
+			return true;
 		}
+		return false;
 	}
 
 	/**
diff --git a/lib/public/Collaboration/Reference/IReferenceManager.php b/lib/public/Collaboration/Reference/IReferenceManager.php
index 8e6dee4aa2d..d757d35834a 100644
--- a/lib/public/Collaboration/Reference/IReferenceManager.php
+++ b/lib/public/Collaboration/Reference/IReferenceManager.php
@@ -83,10 +83,10 @@ interface IReferenceManager {
 	 * @param string $userId
 	 * @param string $providerId
 	 * @param int|null $timestamp use current timestamp if null
-	 * @return void
+	 * @return bool
 	 * @since 26.0.0
 	 */
-	public function touchProvider(string $userId, string $providerId, ?int $timestamp = null): void;
+	public function touchProvider(string $userId, string $providerId, ?int $timestamp = null): bool;
 
 	/**
 	 * Get all known last used timestamps for reference providers