fix(federation): Remove duplicate method isFederatedRemoteRoom() vs isFederatedConversation()

Signed-off-by: Joas Schilling <coding@schilljs.com>

2 files changed, 3 insertions, 10 deletions

diff --git a/lib/Federation/CloudFederationProviderTalk.php b/lib/Federation/CloudFederationProviderTalk.php
index cb161b119..a17cb402e 100644
--- a/lib/Federation/CloudFederationProviderTalk.php
+++ b/lib/Federation/CloudFederationProviderTalk.php
@@ -270,7 +270,7 @@ class CloudFederationProviderTalk implements ICloudFederationProvider {
 		}
 
 		// Sanity check to make sure the room is a remote room
-		if (!$room->isFederatedRemoteRoom()) {
+		if (!$room->isFederatedConversation()) {
 			throw new ShareNotFound(FederationManager::OCM_RESOURCE_NOT_FOUND);
 		}
 
@@ -303,7 +303,7 @@ class CloudFederationProviderTalk implements ICloudFederationProvider {
 		}
 
 		// Sanity check to make sure the room is a remote room
-		if (!$room->isFederatedRemoteRoom()) {
+		if (!$room->isFederatedConversation()) {
 			throw new ShareNotFound(FederationManager::OCM_RESOURCE_NOT_FOUND);
 		}
 
@@ -341,7 +341,7 @@ class CloudFederationProviderTalk implements ICloudFederationProvider {
 		}
 
 		// Sanity check to make sure the room is a remote room
-		if (!$room->isFederatedRemoteRoom()) {
+		if (!$room->isFederatedConversation()) {
 			throw new ShareNotFound(FederationManager::OCM_RESOURCE_NOT_FOUND);
 		}
 
diff --git a/lib/Room.php b/lib/Room.php
index e00625fda..4d2d2a3ab 100644
--- a/lib/Room.php
+++ b/lib/Room.php
@@ -415,13 +415,6 @@ class Room {
 		return $this->remoteToken;
 	}
 
-	/**
-	 * @deprecated
-	 */
-	public function isFederatedRemoteRoom(): bool {
-		return $this->remoteServer !== '';
-	}
-
 	public function setParticipant(?string $userId, Participant $participant): void {
 		// FIXME Also used with cloudId, need actorType checking?
 		$this->currentUser = $userId;