From 367ee27f565f6930f60bf7e0482fbc7c1950c37e 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    |  6 ---
 .../composer/composer/autoload_classmap.php   |  1 -
 .../composer/composer/autoload_static.php     |  1 -
 .../Migration/DropFederatedInvitesTable.php   | 54 -------------------
 4 files changed, 62 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 ee414032a1b52..84fc243a63fff 100644
--- a/apps/cloud_federation_api/appinfo/info.xml
+++ b/apps/cloud_federation_api/appinfo/info.xml
@@ -21,10 +21,4 @@
 	<dependencies>
 		<nextcloud min-version="34" max-version="34"/>
 	</dependencies>
-
-	<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 c338ba5f627b2..a2190d1656832 100644
--- a/apps/cloud_federation_api/composer/composer/autoload_classmap.php
+++ b/apps/cloud_federation_api/composer/composer/autoload_classmap.php
@@ -11,7 +11,6 @@
     'OCA\\CloudFederationAPI\\Config' => $baseDir . '/../lib/Config.php',
     'OCA\\CloudFederationAPI\\Controller\\OCMRequestController' => $baseDir . '/../lib/Controller/OCMRequestController.php',
     'OCA\\CloudFederationAPI\\Controller\\RequestHandlerController' => $baseDir . '/../lib/Controller/RequestHandlerController.php',
-    'OCA\\CloudFederationAPI\\Migration\\DropFederatedInvitesTable' => $baseDir . '/../lib/Migration/DropFederatedInvitesTable.php',
     'OCA\\CloudFederationAPI\\Migration\\Version1016Date202502262004' => $baseDir . '/../lib/Migration/Version1016Date202502262004.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 bb848b2157052..822e03000e488 100644
--- a/apps/cloud_federation_api/composer/composer/autoload_static.php
+++ b/apps/cloud_federation_api/composer/composer/autoload_static.php
@@ -26,7 +26,6 @@ class ComposerStaticInitCloudFederationAPI
         'OCA\\CloudFederationAPI\\Config' => __DIR__ . '/..' . '/../lib/Config.php',
         'OCA\\CloudFederationAPI\\Controller\\OCMRequestController' => __DIR__ . '/..' . '/../lib/Controller/OCMRequestController.php',
         'OCA\\CloudFederationAPI\\Controller\\RequestHandlerController' => __DIR__ . '/..' . '/../lib/Controller/RequestHandlerController.php',
-        'OCA\\CloudFederationAPI\\Migration\\DropFederatedInvitesTable' => __DIR__ . '/..' . '/../lib/Migration/DropFederatedInvitesTable.php',
         'OCA\\CloudFederationAPI\\Migration\\Version1016Date202502262004' => __DIR__ . '/..' . '/../lib/Migration/Version1016Date202502262004.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.');
-		}
-	}
-}