Remove consent.deleted_at index

Prior to this change, consent had an unused index that was not present in the production database.

This change removes the unused index from the mapping. If this index is present in the database, it will remove it.

The deleted_at index is never used on its own. The queries are filtered first on hashed_user_id, after that, the index adds nothing.

#1906

Author: johanib

migrations/DoctrineMigrations/Version20260224000000.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+namespace OpenConext\EngineBlock\Doctrine\Migrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * Patch/repair migration - Removes the deleted_at index from the consent table if present.
+ *
+ * On existing databases where the index is already absent this migration is marked as done
+ * without executing any SQL. On databases where the index still exists it will be dropped.
+ */
+final class Version20260224000000 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Patch migration: Removes the deleted_at index from the consent table. Skips if the index does not exist.';
+    }
+
+    public function preUp(Schema $schema): void
+    {
+        $indexes = $this->connection->createSchemaManager()->listTableIndexes('consent');
+        $this->skipIf(
+            !isset($indexes['deleted_at']),
+            'Index deleted_at on consent table does not exist. Skipping.'
+        );
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE `consent` DROP INDEX `deleted_at`');
+    }
+
+}

src/OpenConext/EngineBlockBundle/Authentication/Entity/Consent.php

@@ -31,7 +31,6 @@
 #[ORM\Entity]
 #[ORM\Index(name: 'hashed_user_id', columns: ['hashed_user_id'])]
 #[ORM\Index(name: 'service_id', columns: ['service_id'])]
-#[ORM\Index(name: 'deleted_at', columns: ['deleted_at'])]
 class Consent
 {
     /**