After review 0

Author: tatevikg1

resources/Database/Schema.sql

@@ -32,8 +32,8 @@ class Administrator implements DomainModel, Identity, CreationDate, Modification
     #[ORM\GeneratedValue]
     private ?int $id = null;
 
-    #[ORM\Column(name: 'created', type: 'datetime', nullable: true)]
-    protected DateTime $createdAt;
+    #[ORM\Column(name: 'created', type: 'datetime', nullable: false)]
+    protected ?DateTime $createdAt = null;
 
     #[ORM\Column(name: 'modified', type: 'datetime', nullable: false)]
     private DateTime $updatedAt;
@@ -77,7 +77,7 @@ public function getId(): ?int
         return $this->id;
     }
 
-    public function getCreatedAt(): DateTime
+    public function getCreatedAt(): ?DateTime
     {
         return $this->createdAt;
     }

src/Domain/Identity/Model/Administrator.php

@@ -30,7 +30,7 @@ class Bounce implements DomainModel, Identity
     #[ORM\Column(type: 'blob', nullable: true)]
     private ?string $data;
 
-    #[ORM\Column(type: 'string', length: 20, nullable: true)]
+    #[ORM\Column(type: 'string', length: 255, nullable: true)]
     private ?string $status;
 
     #[ORM\Column(type: 'text', nullable: true)]

src/Domain/Messaging/Model/Bounce.php

@@ -17,19 +17,19 @@ class MessageFormat implements EmbeddableInterface
     #[ORM\Column(name: 'sendformat', type: 'string', length: 20, nullable: true)]
     private ?string $sendFormat = null;
 
-    #[ORM\Column(name: 'astext', type: 'boolean', options: ['default' => false])]
+    #[ORM\Column(name: 'astext', type: 'boolean')]
     private bool $asText = false;
 
-    #[ORM\Column(name: 'ashtml', type: 'boolean', options: ['default' => false])]
+    #[ORM\Column(name: 'ashtml', type: 'boolean')]
     private bool $asHtml = false;
 
-    #[ORM\Column(name: 'aspdf', type: 'boolean', options: ['default' => false])]
+    #[ORM\Column(name: 'aspdf', type: 'boolean')]
     private bool $asPdf = false;
 
-    #[ORM\Column(name: 'astextandhtml', type: 'boolean', options: ['default' => false])]
+    #[ORM\Column(name: 'astextandhtml', type: 'boolean')]
     private bool $asTextAndHtml = false;
 
-    #[ORM\Column(name: 'astextandpdf', type: 'boolean', options: ['default' => false])]
+    #[ORM\Column(name: 'astextandpdf', type: 'boolean')]
     private bool $asTextAndPdf = false;
 
     public const FORMAT_TEXT = 'text';

src/Domain/Messaging/Model/Message/MessageFormat.php

@@ -192,7 +192,6 @@ public function down(Schema $schema): void
             )
         );
         // this down() migration is auto-generated, please modify it to your needs
-        $this->addSql('CREATE SCHEMA public');
         $this->addSql('ALTER TABLE phplist_admin_attribute DROP CONSTRAINT FK_58E07690D3B10C48');
         $this->addSql('ALTER TABLE phplist_admin_attribute DROP CONSTRAINT FK_58E07690B8ED4D93');
         $this->addSql('ALTER TABLE phplist_admin_login DROP CONSTRAINT FK_5FCE0842B8ED4D93');

src/Migrations/Version20251030081120PostgreSqlPlatform.php

@@ -4,6 +4,7 @@
 
 namespace PhpList\Core\Migrations;
 
+use Doctrine\DBAL\Platforms\MySQLPlatform;
 use Doctrine\DBAL\Schema\Schema;
 use Doctrine\Migrations\AbstractMigration;
 
@@ -19,6 +20,15 @@ public function getDescription(): string
 
     public function up(Schema $schema): void
     {
+        $platform = $this->connection->getDatabasePlatform();
+        $this->skipIf(
+            !$platform instanceof MySQLPlatform,
+            sprintf(
+                'This migration is only applicable for MySQL. Current platform: %s',
+                get_class($platform)
+            )
+        );
+
         // this up() migration is auto-generated, please modify it to your needs
         $this->addSql('ALTER TABLE phplist_admin RENAME INDEX loginnameidx TO phplist_admin_loginnameidx');
         $this->addSql('ALTER TABLE phplist_bounce RENAME INDEX dateindex TO phplist_bounce_dateindex');
@@ -103,6 +113,15 @@ public function up(Schema $schema): void
 
     public function down(Schema $schema): void
     {
+        $platform = $this->connection->getDatabasePlatform();
+        $this->skipIf(
+            !$platform instanceof MySQLPlatform,
+            sprintf(
+                'This migration is only applicable for MySQL. Current platform: %s',
+                get_class($platform)
+            )
+        );
+
         // this down() migration is auto-generated, please modify it to your needs
         $this->addSql('ALTER TABLE phplist_admin RENAME INDEX phplist_admin_loginnameidx TO loginnameidx');
         $this->addSql('ALTER TABLE phplist_bounce RENAME INDEX phplist_bounce_dateindex TO dateindex');