Transparency log: add missing display classes

src/Audit/Display/AuditLogDisplayFactory.php

@@ -34,6 +34,25 @@ public function build(iterable $auditRecords): array
     public function buildSingle(AuditRecord $record): AuditLogDisplayInterface
     {
         return match ($record->type) {
+            AuditRecordType::MaintainerAdded => new MaintainerAddedDisplay(
+                $record->datetime,
+                $record->attributes['name'],
+                $this->buildActor($record->attributes['maintainer']),
+                $this->buildActor($record->attributes['actor']),
+            ),
+            AuditRecordType::MaintainerRemoved => new MaintainerRemovedDisplay(
+                $record->datetime,
+                $record->attributes['name'],
+                $this->buildActor($record->attributes['maintainer']),
+                $this->buildActor($record->attributes['actor']),
+            ),
+            AuditRecordType::PackageTransferred => new PackageTransferredDisplay(
+                $record->datetime,
+                $record->attributes['name'],
+                $record->attributes['previous_maintainers'],
+                $record->attributes['current_maintainers'],
+                $this->buildActor($record->attributes['actor']),
+            ),
             AuditRecordType::PackageCreated => new PackageCreatedDisplay(
                 $record->datetime,
                 $record->attributes['name'],

src/Audit/Display/MaintainerAddedDisplay.php

@@ -0,0 +1,37 @@
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of Packagist.
+ *
+ * (c) Jordi Boggiano <[email protected]>
+ *     Nils Adermann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace App\Audit\Display;
+
+use App\Audit\AuditRecordType;
+
+readonly class MaintainerAddedDisplay extends AbstractAuditLogDisplay
+{
+    public function __construct(
+        \DateTimeImmutable $datetime,
+        public string $packageName,
+        public ActorDisplay $maintainer,
+        ActorDisplay $actor,
+    ) {
+        parent::__construct($datetime, $actor);
+    }
+
+    public function getType(): AuditRecordType
+    {
+        return AuditRecordType::MaintainerAdded;
+    }
+
+    public function getTemplateName(): string
+    {
+        return 'audit_log/display/maintainer_added.html.twig';
+    }
+}

src/Audit/Display/MaintainerRemovedDisplay.php

@@ -0,0 +1,37 @@
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of Packagist.
+ *
+ * (c) Jordi Boggiano <[email protected]>
+ *     Nils Adermann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace App\Audit\Display;
+
+use App\Audit\AuditRecordType;
+
+readonly class MaintainerRemovedDisplay extends AbstractAuditLogDisplay
+{
+    public function __construct(
+        \DateTimeImmutable $datetime,
+        public string $packageName,
+        public ActorDisplay $maintainer,
+        ActorDisplay $actor,
+    ) {
+        parent::__construct($datetime, $actor);
+    }
+
+    public function getType(): AuditRecordType
+    {
+        return AuditRecordType::MaintainerRemoved;
+    }
+
+    public function getTemplateName(): string
+    {
+        return 'audit_log/display/maintainer_removed.html.twig';
+    }
+}

src/Audit/Display/PackageTransferredDisplay.php

@@ -0,0 +1,44 @@
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of Packagist.
+ *
+ * (c) Jordi Boggiano <[email protected]>
+ *     Nils Adermann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace App\Audit\Display;
+
+use App\Audit\AuditRecordType;
+
+readonly class PackageTransferredDisplay extends AbstractAuditLogDisplay
+{
+    /**
+     * @param array<ActorDisplay> $previousMaintainers
+     * @param array<ActorDisplay> $currentMaintainers
+     */
+    public function __construct(
+        \DateTimeImmutable $datetime,
+        public string $packageName,
+        /** @var array<string> $previousMaintainers */
+        public array $previousMaintainers,
+        /** @var array<string> $currentMaintainers */
+        public array $currentMaintainers,
+        ActorDisplay $actor,
+    ) {
+        parent::__construct($datetime, $actor);
+    }
+
+    public function getType(): AuditRecordType
+    {
+        return AuditRecordType::PackageTransferred;
+    }
+
+    public function getTemplateName(): string
+    {
+        return 'audit_log/display/package_transferred.html.twig';
+    }
+}

templates/audit_log/display/canonical_url_changed.html.twig

@@ -1,10 +1,6 @@
-<strong>
-    {%- if display.packageName is existing_package -%}
-        <a href="{{ path('view_package', { 'name': display.packageName }) }}">{{ display.packageName }}</a>
-    {%- else -%}
-        {{ display.packageName }}
-    {%- endif -%}
-</strong><br>
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong><br>
 From: {{ display.repositoryFrom }}<br>
 To: {{ display.repositoryTo }}<br>
 Changed by: {{ display.actor.username }}

templates/audit_log/display/maintainer_added.html.twig

@@ -0,0 +1,5 @@
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong><br>
+Maintainer: {{ display.maintainer.username }}<br>
+Added by: {{ display.actor.username }}

templates/audit_log/display/maintainer_removed.html.twig

@@ -0,0 +1,5 @@
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong><br>
+Maintainer: {{ display.maintainer.username }}<br>
+Removed by: {{ display.actor.username }}

templates/audit_log/display/package_created.html.twig

@@ -1,9 +1,5 @@
-<strong>
-    {%- if display.packageName is existing_package -%}
-        <a href="{{ path('view_package', { 'name': display.packageName }) }}">{{ display.packageName }}</a>
-    {%- else -%}
-        {{ display.packageName }}
-    {%- endif -%}
-</strong><br>
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong><br>
 Repository: {{ display.repository }}<br>
 Created by: {{ display.actor.username }}

templates/audit_log/display/package_deleted.html.twig

@@ -1,9 +1,5 @@
-<strong>
-    {%- if display.packageName is existing_package -%}
-        <a href="{{ path('view_package', { 'name': display.packageName }) }}">{{ display.packageName }}</a>
-    {%- else -%}
-        {{ display.packageName }}
-    {%- endif -%}
-</strong><br>
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong><br>
 Repository: {{ display.repository }}<br>
 Deleted by: {{ display.actor.username }}

templates/audit_log/display/package_transferred.html.twig

@@ -0,0 +1,6 @@
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong><br>
+Previous maintainers: {{ display.previousMaintainers|column('username')|join(', ') }}<br>
+Current maintainers: {{ display.currentMaintainers|column('username')|join(', ') }}<br>
+Transferred by: {{ display.actor.username }}

templates/audit_log/display/version_deleted.html.twig

@@ -1,8 +1,4 @@
-<strong>
-    {%- if display.packageName is existing_package -%}
-        <a href="{{ path('view_package', { 'name': display.packageName }) }}">{{ display.packageName }}</a>
-    {%- else -%}
-        {{ display.packageName }}
-    {%- endif -%}
-</strong> {{ display.version }}<br>
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong> {{ display.version }}<br>
 Deleted by: {{ display.actor.username }}

templates/audit_log/display/version_reference_changed.html.twig

@@ -1,10 +1,6 @@
-<strong>
-    {%- if display.packageName is existing_package -%}
-        <a href="{{ path('view_package', { 'name': display.packageName }) }}">{{ display.packageName }}</a>
-    {%- else -%}
-        {{ display.packageName }}
-    {%- endif -%}
-</strong> {{ display.version }}<br>
+{% import 'audit_log/macros.html.twig' as auditLog %}
+
+<strong>{{ auditLog.packageLink(display.packageName) }}</strong> {{ display.version }}<br>
 {% if display.sourceFrom or display.sourceTo %}
     Source: {{ display.sourceFrom ?? 'none' }} → {{ display.sourceTo ?? 'none' }}<br>
 {% endif %}

templates/audit_log/macros.html.twig

@@ -0,0 +1,7 @@
+{% macro packageLink(packageName) %}
+    {%- if packageName is existing_package -%}
+        <a href="{{ path('view_package', { 'name': packageName }) }}">{{ packageName }}</a>
+    {%- else -%}
+        {{ packageName }}
+    {%- endif -%}
+{% endmacro %}