Skip to content

Commit ca527e7

Browse files
Use a proper translation string for audit record types
1 parent a6c579c commit ca527e7

File tree

3 files changed

+15
-7
lines changed

3 files changed

+15
-7
lines changed

templates/audit_log/view_audit_logs.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
{% for log in auditLogs %}
2121
<tr>
2222
<td>{{ log.datetime|date('Y-m-d H:i:s') }} UTC</td>
23-
<td data-test="audit-log-type">{{ log.type.value }}</td>
23+
<td data-test="audit-log-type">{{ ('audit_log.type.'~log.type.value)|trans }}</td>
2424
<td>{{ log.vendor ?? '-' }}</td>
2525
</tr>
2626
{% endfor %}

tests/Controller/AuditLogControllerTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,10 @@ public function testViewAuditLogs(): void
2424

2525
$this->store($user, $package);
2626

27-
$auditLog1 = AuditRecord::canonicalUrlChange($package, $user, 'https://github.com/vendor1/package1-new');
28-
$auditLog2 = AuditRecord::packageDeleted($package, $user);
27+
$auditRecord1 = AuditRecord::canonicalUrlChange($package, $user, 'https://github.com/vendor1/package1-new');
28+
$auditRecord = AuditRecord::packageDeleted($package, $user);
2929

30-
$this->store($auditLog1, $auditLog2);
30+
$this->store($auditRecord1, $auditRecord);
3131

3232
$this->client->loginUser($user);
3333
$crawler = $this->client->request('GET', '/audit-log');
@@ -36,9 +36,9 @@ public function testViewAuditLogs(): void
3636
$rows = $crawler->filter('[data-test=audit-log-type]');
3737
static::assertGreaterThanOrEqual(3, $rows->count(), 'Should have at least 3 audit log entries');
3838
static::assertSame([
39-
'package_deleted',
40-
'canonical_url_change',
41-
'package_created',
39+
'Package deleted',
40+
'Canonical URL changed',
41+
'Package created',
4242
], $rows->each(fn ($element) => trim($element->text())));
4343
}
4444
}

translations/messages.en.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,3 +168,11 @@ freezing_reasons:
168168
spam: This package was marked as spam and has been frozen as a result.
169169
remote_id: This package's canonical repository id has changed and the package has been frozen as a result. Email us for help if needed.
170170
gone: This package's canonical repository appears to be gone and the package has been frozen as a result. Email us for help if needed.
171+
172+
audit_log:
173+
type:
174+
package_created: Package created
175+
package_deleted: Package deleted
176+
version_deleted: Version deleted
177+
version_reference_change: Version reference changed
178+
canonical_url_change: Canonical URL changed

0 commit comments

Comments
 (0)