Merge pull request #7 from codedmonkey/package-metadata

Refactor package metadata

Author: codedmonkey

migrations/Version20260325114538.php

@@ -11,8 +11,6 @@ parameters:
         - tests/bootstrap.php
     ignoreErrors:
         - '#^Call to an undefined method Symfony\\Component\\Config\\Definition\\Builder\\NodeDefinition\:\:children\(\)\.$#'
-        - '#CodedMonkey\\Dirigent\\Doctrine\\Entity\\AbstractVersionLink given\.#'
-        - '#^PHPDoc tag @var with type CodedMonkey\\Dirigent\\Doctrine\\Entity\\AbstractVersionLink is not subtype of native type#'
         - '#^Property CodedMonkey\\Dirigent\\Doctrine\\Entity\\[a-zA-Z]+\:\:\$id \(int\|null\) is never assigned int so it can be removed from the property type\.$#'
         -
             message: '#^Class CodedMonkey\\Dirigent\\Doctrine\\Entity\\TrackedEntity has an uninitialized readonly property \$createdAt\. Assign it in the constructor\.$#'

phpstan.dist.neon

@@ -128,7 +128,7 @@ public function packageDistribution(Request $request, string $reference, string
                 throw $this->createNotFoundException();
             }
 
-            if (null === $version = $this->versionRepository->findOneByNormalizedVersion($package, $versionName)) {
+            if (null === $version = $this->versionRepository->findOneByNormalizedName($package, $versionName)) {
                 throw $this->createNotFoundException();
             }
 

src/Controller/ApiController.php

@@ -52,6 +52,7 @@ public function versionInfo(#[MapPackage] Package $package, #[MapPackage] Versio
         return $this->render('dashboard/packages/package_info.html.twig', [
             'package' => $package,
             'version' => $version,
+            'metadata' => $version->getCurrentMetadata(),
 
             'dependentCount' => $dependentCount,
             'implementationCount' => $implementationCount,

src/Controller/Dashboard/DashboardPackagesInfoController.php

@@ -2,70 +2,71 @@
 
 namespace CodedMonkey\Dirigent\Doctrine\Entity;
 
+use CodedMonkey\Dirigent\Entity\MetadataLinkType;
 use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\MappedSuperclass]
-abstract class AbstractVersionLink
+abstract class AbstractMetadataLink
 {
     #[ORM\Id]
     #[ORM\Column]
     #[ORM\GeneratedValue]
     private ?int $id = null;
 
-    protected Version $version;
-
-    #[ORM\Column]
-    private int $index;
+    protected Metadata $metadata;
 
     #[ORM\Column(length: 191)]
     private string $linkedPackageName;
 
     #[ORM\Column(type: Types::TEXT)]
     private string $linkedVersionConstraint;
 
-    public function getId(): ?int
-    {
-        return $this->id;
-    }
-
-    public function getIndex(): int
-    {
-        return $this->index;
-    }
+    #[ORM\Column]
+    private int $index;
 
-    public function setIndex(int $index): void
-    {
+    public function __construct(
+        Metadata $metadata,
+        string $linkedPackageName,
+        string $linkedVersionConstraint,
+        int $index,
+    ) {
+        $this->metadata = $metadata;
+        $this->linkedPackageName = $linkedPackageName;
+        $this->linkedVersionConstraint = $linkedVersionConstraint;
         $this->index = $index;
+
+        $this->addToCollection();
     }
 
-    public function getVersion(): Version
+    public function getId(): ?int
     {
-        return $this->version;
+        return $this->id;
     }
 
-    public function setVersion(Version $version): void
+    public function getMetadata(): Metadata
     {
-        $this->version = $version;
+        return $this->metadata;
     }
 
     public function getLinkedPackageName(): string
     {
         return $this->linkedPackageName;
     }
 
-    public function setLinkedPackageName(string $packageName): void
+    public function getLinkedVersionConstraint(): string
     {
-        $this->linkedPackageName = $packageName;
+        return $this->linkedVersionConstraint;
     }
 
-    public function getLinkedVersionConstraint(): string
+    public function getIndex(): int
     {
-        return $this->linkedVersionConstraint;
+        return $this->index;
     }
 
-    public function setLinkedVersionConstraint(string $packageVersion): void
+    private function addToCollection(): void
     {
-        $this->linkedVersionConstraint = $packageVersion;
+        $linkType = MetadataLinkType::fromClass(static::class);
+        $linkType->getMetadataLinks($this->metadata)->add($this);
     }
 }