From ed15b3fbba49c2f0769b68f5ce6305fa29e3c42f Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Sat, 19 Sep 2026 19:09:06 +0200
Subject: [PATCH] chore: drop DropFederatedInvitesTable

the table name is now used by contacts, so the migration would end up removing the table that it added

Signed-off-by: Robin Appelman <robin@icewind.nl>
---
 apps/cloud_federation_api/appinfo/info.xml    |  5 --
 .../composer/composer/autoload_classmap.php   |  1 -
 .../composer/composer/autoload_static.php     |  1 -
 .../Migration/DropFederatedInvitesTable.php   | 54 -------------------
 4 files changed, 61 deletions(-)
 delete mode 100644 apps/cloud_federation_api/lib/Migration/DropFederatedInvitesTable.php

diff --git a/apps/cloud_federation_api/appinfo/info.xml b/apps/cloud_federation_api/appinfo/info.xml
index b0ce0c71d445c..44ffd15d7a72a 100644
--- a/apps/cloud_federation_api/appinfo/info.xml
+++ b/apps/cloud_federation_api/appinfo/info.xml
@@ -25,9 +25,4 @@
 	<background-jobs>
 		<job>OCA\CloudFederationAPI\BackgroundJob\CleanupExpiredOcmTokensJob</job>
 	</background-jobs>
-	<repair-steps>
-		<post-migration>
-			<step>OCA\CloudFederationAPI\Migration\DropFederatedInvitesTable</step>
-		</post-migration>
-	</repair-steps>
 </info>
diff --git a/apps/cloud_federation_api/composer/composer/autoload_classmap.php b/apps/cloud_federation_api/composer/composer/autoload_classmap.php
index f5a74754b281e..e7473bd6632d9 100644
--- a/apps/cloud_federation_api/composer/composer/autoload_classmap.php
+++ b/apps/cloud_federation_api/composer/composer/autoload_classmap.php
@@ -16,7 +16,6 @@
     'OCA\\CloudFederationAPI\\Db\\OcmTokenMap' => $baseDir . '/../lib/Db/OcmTokenMap.php',
     'OCA\\CloudFederationAPI\\Db\\OcmTokenMapMapper' => $baseDir . '/../lib/Db/OcmTokenMapMapper.php',
     'OCA\\CloudFederationAPI\\Listener\\ShareDeletedListener' => $baseDir . '/../lib/Listener/ShareDeletedListener.php',
-    'OCA\\CloudFederationAPI\\Migration\\DropFederatedInvitesTable' => $baseDir . '/../lib/Migration/DropFederatedInvitesTable.php',
     'OCA\\CloudFederationAPI\\Migration\\Version1016Date202502262004' => $baseDir . '/../lib/Migration/Version1016Date202502262004.php',
     'OCA\\CloudFederationAPI\\Migration\\Version1017Date20260306120000' => $baseDir . '/../lib/Migration/Version1017Date20260306120000.php',
     'OCA\\CloudFederationAPI\\ResponseDefinitions' => $baseDir . '/../lib/ResponseDefinitions.php',
diff --git a/apps/cloud_federation_api/composer/composer/autoload_static.php b/apps/cloud_federation_api/composer/composer/autoload_static.php
index 87d1dac996bce..92da8638f1853 100644
--- a/apps/cloud_federation_api/composer/composer/autoload_static.php
+++ b/apps/cloud_federation_api/composer/composer/autoload_static.php
@@ -31,7 +31,6 @@ class ComposerStaticInitCloudFederationAPI
         'OCA\\CloudFederationAPI\\Db\\OcmTokenMap' => __DIR__ . '/..' . '/../lib/Db/OcmTokenMap.php',
         'OCA\\CloudFederationAPI\\Db\\OcmTokenMapMapper' => __DIR__ . '/..' . '/../lib/Db/OcmTokenMapMapper.php',
         'OCA\\CloudFederationAPI\\Listener\\ShareDeletedListener' => __DIR__ . '/..' . '/../lib/Listener/ShareDeletedListener.php',
-        'OCA\\CloudFederationAPI\\Migration\\DropFederatedInvitesTable' => __DIR__ . '/..' . '/../lib/Migration/DropFederatedInvitesTable.php',
         'OCA\\CloudFederationAPI\\Migration\\Version1016Date202502262004' => __DIR__ . '/..' . '/../lib/Migration/Version1016Date202502262004.php',
         'OCA\\CloudFederationAPI\\Migration\\Version1017Date20260306120000' => __DIR__ . '/..' . '/../lib/Migration/Version1017Date20260306120000.php',
         'OCA\\CloudFederationAPI\\ResponseDefinitions' => __DIR__ . '/..' . '/../lib/ResponseDefinitions.php',
diff --git a/apps/cloud_federation_api/lib/Migration/DropFederatedInvitesTable.php b/apps/cloud_federation_api/lib/Migration/DropFederatedInvitesTable.php
deleted file mode 100644
index 988876ef21f88..0000000000000
--- a/apps/cloud_federation_api/lib/Migration/DropFederatedInvitesTable.php
+++ /dev/null
@@ -1,54 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-/**
- * SPDX-FileCopyrightText: 2026 Nextcloud GmbH and Nextcloud contributors
- * SPDX-License-Identifier: AGPL-3.0-or-later
- */
-
-namespace OCA\CloudFederationAPI\Migration;
-
-use OC\DB\Connection;
-use OC\DB\SchemaWrapper;
-use OCP\Migration\IOutput;
-use OCP\Migration\IRepairStep;
-
-/**
- * Drops the federated_invites table unless it contains data.
- */
-class DropFederatedInvitesTable implements IRepairStep {
-
-	public function __construct(
-		protected Connection $db,
-	) {
-	}
-
-	#[\Override]
-	public function getName(): string {
-		return 'Conditionally drop the federated_invites table';
-	}
-
-	#[\Override]
-	public function run(IOutput $output): void {
-		$table_name = 'federated_invites';
-		$schema = new SchemaWrapper($this->db);
-		if (!$schema->hasTable($table_name)) {
-			echo("$table_name does not exist");
-			return;
-		}
-		$qb = $this->db->getQueryBuilder();
-		$qb->select('*')
-			->from($table_name)
-			->setMaxResults(1);
-		$result = $qb->executeQuery();
-		$hasRows = $result->fetchOne();
-		if (!$hasRows) {
-			$schema->dropTable($table_name);
-			$schema->performDropTableCalls();
-			$output->info('Table federated_invites dropped');
-		} else {
-			$output->info('Table federated_invites contains data. Table will be kept.');
-		}
-	}
-}