Update datetime properties and implement TrackedEntity class in Package entity

Signed-off-by: Tim Goudriaan <tim@codedmonkey.com>

Author: codedmonkey

migrations/Version20250715195941.php

@@ -19,6 +19,9 @@ public function up(Schema $schema): void
         $this->addSql(<<<'SQL'
             ALTER TABLE access_token ADD last_modified_at TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL
         SQL);
+        $this->addSql(<<<'SQL'
+            ALTER TABLE package ADD last_modified_at TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL
+        SQL);
         $this->addSql(<<<'SQL'
             ALTER TABLE version ADD last_modified_at TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL
         SQL);
@@ -27,10 +30,13 @@ public function up(Schema $schema): void
     public function down(Schema $schema): void
     {
         $this->addSql(<<<'SQL'
-            ALTER TABLE access_token DROP last_modified_at
+            ALTER TABLE version DROP last_modified_at
         SQL);
         $this->addSql(<<<'SQL'
-            ALTER TABLE version DROP last_modified_at
+            ALTER TABLE package DROP last_modified_at
+        SQL);
+        $this->addSql(<<<'SQL'
+            ALTER TABLE access_token DROP last_modified_at
         SQL);
     }
 }

src/Doctrine/Entity/Package.php

@@ -13,7 +13,7 @@
 #[ORM\Entity(repositoryClass: PackageRepository::class)]
 #[ORM\UniqueConstraint(name: 'package_name_idx', columns: ['name'])]
 #[UniquePackage]
-class Package
+class Package extends TrackedEntity
 {
     #[ORM\Id]
     #[ORM\Column]
@@ -74,17 +74,14 @@ class Package
     #[ORM\OneToMany(targetEntity: Version::class, mappedBy: 'package', cascade: ['remove'])]
     private Collection $versions;
 
-    #[ORM\Column(type: Types::DATETIME_MUTABLE)]
-    private \DateTimeInterface $createdAt;
-
-    #[ORM\Column(type: Types::DATETIME_MUTABLE, nullable: true)]
-    private ?\DateTimeInterface $updatedAt = null;
+    #[ORM\Column(nullable: true)]
+    private ?\DateTimeImmutable $updatedAt = null;
 
-    #[ORM\Column(type: Types::DATETIME_MUTABLE, nullable: true)]
-    private ?\DateTimeInterface $updateScheduledAt = null;
+    #[ORM\Column(nullable: true)]
+    private ?\DateTimeImmutable $updateScheduledAt = null;
 
-    #[ORM\Column(type: Types::DATETIME_MUTABLE, nullable: true)]
-    private ?\DateTimeInterface $dumpedAt = null;
+    #[ORM\Column(nullable: true)]
+    private ?\DateTimeImmutable $dumpedAt = null;
 
     /**
      * @var array<string, Version> lookup table for versions
@@ -95,7 +92,6 @@ public function __construct()
     {
         $this->installations = new PackageInstallations($this);
         $this->versions = new ArrayCollection();
-        $this->createdAt = new \DateTime();
     }
 
     public function getId(): ?int
@@ -308,37 +304,37 @@ public function getVersion(string $normalizedVersion): ?Version
         return $this->cachedVersions[strtolower($normalizedVersion)] ?? null;
     }
 
-    public function getCreatedAt(): \DateTimeInterface
+    public function getCreatedAt(): \DateTimeImmutable
     {
         return $this->createdAt;
     }
 
-    public function getUpdatedAt(): ?\DateTimeInterface
+    public function getUpdatedAt(): ?\DateTimeImmutable
     {
         return $this->updatedAt;
     }
 
-    public function setUpdatedAt(\DateTimeInterface $updatedAt): void
+    public function setUpdatedAt(\DateTimeImmutable $updatedAt): void
     {
         $this->updatedAt = $updatedAt;
     }
 
-    public function getUpdateScheduledAt(): ?\DateTimeInterface
+    public function getUpdateScheduledAt(): ?\DateTimeImmutable
     {
         return $this->updateScheduledAt;
     }
 
-    public function setUpdateScheduledAt(?\DateTimeInterface $updateScheduledAt): void
+    public function setUpdateScheduledAt(?\DateTimeImmutable $updateScheduledAt): void
     {
         $this->updateScheduledAt = $updateScheduledAt;
     }
 
-    public function getDumpedAt(): ?\DateTimeInterface
+    public function getDumpedAt(): ?\DateTimeImmutable
     {
         return $this->dumpedAt;
     }
 
-    public function setDumpedAt(?\DateTimeInterface $dumpedAt): void
+    public function setDumpedAt(?\DateTimeImmutable $dumpedAt): void
     {
         $this->dumpedAt = $dumpedAt;
     }

src/Package/PackageMetadataResolver.php

@@ -197,10 +197,9 @@ private function updatePackage(Package $package, array $composerPackages, ?VcsDr
         }
 
         if ($primaryVersionName) {
-            $message = new Envelope(new UpdatePackageLinks($package->getId(), $primaryVersionName), [
-                new DispatchAfterCurrentBusStamp(),
-                new TransportNamesStamp('async'),
-            ]);
+            $message = Envelope::wrap(new UpdatePackageLinks($package->getId(), $primaryVersionName))
+                ->with(new DispatchAfterCurrentBusStamp())
+                ->with(new TransportNamesStamp('async'));
             $this->messenger->dispatch($message);
         }
 
@@ -211,10 +210,7 @@ private function updatePackage(Package $package, array $composerPackages, ?VcsDr
             $this->entityManager->remove($versionEntity);
         }
 
-        $updatedAt = new \DateTime();
-        $package->setUpdatedAt($updatedAt);
-
-        $this->entityManager->persist($package);
+        $package->setUpdatedAt(new \DateTimeImmutable());
     }
 
     private function updateVersion(Package $package, Version $version, CompletePackageInterface $data, ?VcsDriverInterface $driver = null): void

src/Package/PackageProviderManager.php

@@ -41,7 +41,7 @@ public function dump(Package $package): void
         $this->write($package->getName(), $releasePackages);
         $this->write($package->getName(), $devPackages, true);
 
-        $package->setDumpedAt(new \DateTime());
+        $package->setDumpedAt(new \DateTimeImmutable());
     }
 
     public function exists(string $packageName): bool