Clean up package-related Doctrine entity classes

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

Author: codedmonkey

src/Doctrine/Entity/Metadata.php

@@ -88,7 +88,7 @@ class Metadata extends TrackedEntity implements \Stringable
     #[ORM\Column(nullable: true)]
     private ?\DateTimeImmutable $releasedAt = null;
 
-    #[ORM\ManyToOne(targetEntity: Version::class, inversedBy: 'metadata')]
+    #[ORM\ManyToOne(targetEntity: Version::class)]
     #[ORM\JoinColumn(nullable: false, onDelete: 'CASCADE')]
     private Version $version;
 

src/Doctrine/Entity/Package.php

@@ -85,11 +85,6 @@ class Package extends TrackedEntity
     #[ORM\Column(nullable: true)]
     private ?\DateTimeImmutable $dumpedAt = null;
 
-    /**
-     * @var array<string, Version> lookup table for versions
-     */
-    private array $cachedVersions;
-
     private array $sortedVersions;
 
     public function __construct()
@@ -271,18 +266,6 @@ public function getVersions(): Collection
         return $this->versions;
     }
 
-    public function getVersion(string $normalizedVersion): ?Version
-    {
-        if (!isset($this->cachedVersions)) {
-            $this->cachedVersions = [];
-            foreach ($this->getVersions() as $version) {
-                $this->cachedVersions[strtolower($version->getNormalizedName())] = $version;
-            }
-        }
-
-        return $this->cachedVersions[strtolower($normalizedVersion)] ?? null;
-    }
-
     #[\Override]
     public function getCreatedAt(): \DateTimeImmutable
     {

src/Doctrine/Entity/Version.php

@@ -3,8 +3,6 @@
 namespace CodedMonkey\Dirigent\Doctrine\Entity;
 
 use CodedMonkey\Dirigent\Doctrine\Repository\VersionRepository;
-use Doctrine\Common\Collections\ArrayCollection;
-use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\Entity(repositoryClass: VersionRepository::class)]
@@ -41,15 +39,11 @@ class Version extends TrackedEntity implements \Stringable
     #[ORM\OneToOne(mappedBy: 'version', cascade: ['persist', 'detach', 'remove'])]
     private VersionInstallations $installations;
 
-    #[ORM\OneToMany(targetEntity: Metadata::class, mappedBy: 'version', cascade: ['persist', 'detach', 'remove'])]
-    private Collection $metadata;
-
     public function __construct(Package $package)
     {
         $this->package = $package;
 
         $this->installations = new VersionInstallations($this);
-        $this->metadata = new ArrayCollection();
     }
 
     public function __toString(): string
@@ -134,14 +128,6 @@ public function getInstallations(): VersionInstallations
         return $this->installations;
     }
 
-    /**
-     * @return Collection<int, Metadata>
-     */
-    public function getMetadata(): Collection
-    {
-        return $this->metadata;
-    }
-
     public function getExtendedName(): string
     {
         return $this->name . ($this->getCurrentMetadata()->hasVersionAlias() ? ' / ' . $this->getCurrentMetadata()->getVersionAlias() : '');