Fix mapping issue with consent.deleted_at

Prior to this change, the production schema would differ from the Doctrine mapping.

On prod, deleted_at was `NOT NULL, default=0000-00-00 00:00:00`.
This is correct, as it is part of the primary key, and cannot be null.

This change corrects the mapping to align with production.

See #1906

Author: johanib

CHANGELOG.md

@@ -18,6 +18,12 @@ Upgrade to `doctrine/dbal` 4
 Bugfixes:
 * Metadata push will now reject all metadata if any service contains invalid PHP syntax in its attribute manipulations (#1778)
 
+Changes:
+* The `consent.deleted_at` should be not nullable, and have a default value of `0000-00-00 00:00:00`.
+  * Because `deleted_at` is part of the PK, no migration is provided. The database engine should not allow this to be null in the first place, so it is probably not nullable already on your db.
+  * The `0000-00-00 00:00:00` is added for clarity/consistency, as this is probably the default behaviour of your database already.
+* Removed unused index `consent.deleted_at`. Delete this from your production database if it's there.
+
 ## 7.1.0
 SRAM integration
 

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

@@ -70,6 +70,6 @@ class Consent
      * @var DateTime
      */
     #[ORM\Id]
-    #[ORM\Column(name: 'deleted_at', type: Types::DATETIME_MUTABLE, nullable: true, options: ['default' => null])]
+    #[ORM\Column(name: 'deleted_at', type: Types::DATETIME_MUTABLE, nullable: false, options: ['default' => '0000-00-00 00:00:00'])]
     public ?DateTimeInterface $deletedAt = null;
 }