Use repair step instead of migration to conditionally drop table.

Signed-off-by: Antoon P. <antoon.prins@surf.nl>

Author: redblom

apps/cloud_federation_api/appinfo/info.xml

@@ -21,4 +21,9 @@
 	<dependencies>
 		<nextcloud min-version="34" max-version="34"/>
 	</dependencies>
+	<repair-steps>
+		<post-migration>
+			<step>OCA\CloudFederationAPI\Migration\DropFederatedInvitesTable</step>
+		</post-migration>
+	</repair-steps>
 </info>

apps/cloud_federation_api/composer/composer/autoload_classmap.php

@@ -12,6 +12,6 @@
     'OCA\\CloudFederationAPI\\Controller\\OCMRequestController' => $baseDir . '/../lib/Controller/OCMRequestController.php',
     'OCA\\CloudFederationAPI\\Controller\\RequestHandlerController' => $baseDir . '/../lib/Controller/RequestHandlerController.php',
     'OCA\\CloudFederationAPI\\Migration\\Version1016Date202502262004' => $baseDir . '/../lib/Migration/Version1016Date202502262004.php',
-    'OCA\\CloudFederationAPI\\Migration\\Version1018Date20260202110547' => $baseDir . '/../lib/Migration/Version1018Date20260202110547.php',
+    'OCA\\CloudFederationAPI\\Migration\\DropFederatedInvitesTable' => $baseDir . '/../lib/Migration/DropFederatedInvitesTable.php',
     'OCA\\CloudFederationAPI\\ResponseDefinitions' => $baseDir . '/../lib/ResponseDefinitions.php',
 );

apps/cloud_federation_api/composer/composer/autoload_static.php

@@ -27,7 +27,7 @@ class ComposerStaticInitCloudFederationAPI
         'OCA\\CloudFederationAPI\\Controller\\OCMRequestController' => __DIR__ . '/..' . '/../lib/Controller/OCMRequestController.php',
         'OCA\\CloudFederationAPI\\Controller\\RequestHandlerController' => __DIR__ . '/..' . '/../lib/Controller/RequestHandlerController.php',
         'OCA\\CloudFederationAPI\\Migration\\Version1016Date202502262004' => __DIR__ . '/..' . '/../lib/Migration/Version1016Date202502262004.php',
-        'OCA\\CloudFederationAPI\\Migration\\Version1018Date20260202110547' => __DIR__ . '/..' . '/../lib/Migration/Version1018Date20260202110547.php',
+        'OCA\\CloudFederationAPI\\Migration\\DropFederatedInvitesTable' => __DIR__ . '/..' . '/../lib/Migration/DropFederatedInvitesTable.php',
         'OCA\\CloudFederationAPI\\ResponseDefinitions' => __DIR__ . '/..' . '/../lib/ResponseDefinitions.php',
     );
 

apps/cloud_federation_api/lib/Migration/DropFederatedInvitesTable.php

@@ -0,0 +1,62 @@
+<?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 Closure;
+use OC\DB\Connection;
+use OC\DB\SchemaWrapper;
+use OCP\AppFramework\Services\IAppConfig;
+use OCP\DB\ISchemaWrapper;
+use OCP\IDBConnection;
+use OCP\Migration\Attributes\DropTable;
+use OCP\Migration\IOutput;
+use OCP\Migration\IRepairStep;
+use OCP\Migration\SimpleMigrationStep;
+use Override;
+
+class DropFederatedInvitesTable implements IRepairStep {
+
+	public function __construct(
+		protected Connection $db,
+	) {
+	}
+
+	public function getName(): string {
+		return 'Conditionally drop the federated_invites table';
+	}
+
+	/**
+	 * Drops the federated_invites table unless it contains data.
+	 * 
+	 * @param IOutput $output
+	 * @return void
+	 */
+	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.');
+		}
+	}
+}