>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3619117. --> Reported by: [mably](https://www.drupal.org/user/3375160) Related to !80 >>> <p>Nine operator-facing strings named a module by its machine name: <em>audit_trail</em> where an operator reads <em>Audit Trail</em> on the Extend page, and <em>pdv</em> where the module calls itself <em>Personal Data Vault</em> everywhere else. They sit on the Extend page, on the audit-trail settings form, on the menu link that leads to it, and on the ECA channel field.</p> <p>Found while preparing the alpha9 release notes, which claimed the audit-trail settings had readable labels. That claim came from <span class="drupalorg-gitlab-issue-link drupalorg-gitlab-link-wrapper"><a href="https://git.drupalcode.org/project/pdv/-/work_items/3619101" class="drupalorg-gitlab-link">https://git.drupalcode.org/project/pdv/-/work_items/3619101</a></span>, which fixed twenty-four schema labels showing an event machine name. The menu link pointing at that very settings page still read "Toggle individual pdv state transitions on or off as audit_trail chain rows", so the bullet was not yet true. That audit grepped config schema for the defect and never looked at menu links or at PHP, which is the gap this fixes.</p> <h3>The sweep, and what it deliberately keeps</h3> <p>Every YAML that produces operator-facing text (40 files: info, links.menu, links.task, links.action, routing, permissions, config schema) plus every translatable string in PHP and Twig. Seven defects in YAML, two in PHP. The point of running it over both is that the class of defect does not live in one file type, which is exactly how the first pass missed it.</p> <p>Twenty-two PHP strings matched the search and thirteen are correct as they stand, so they are kept rather than swept: each one tells the operator which machine name to <em>type</em>, and quoting <code>civil_status</code> or <code>id_card</code> is the whole point of the help text. Nine more are kept for a different reason: the event-group labels read <em>Item lifecycle (pdv.item)</em>, and that parenthetical channel is what lets an operator match a group against the chain rows it writes and against the <em>pdv.@channel.@action</em> keys. Core shows machine names in parentheses in the same way. Removing them would cost the operator the correlation and gain nothing.</p> <h3>Translations move with the strings</h3> <p>All nine were translated, so rewording the English would have orphaned nine French entries silently and left the module shipping an incomplete translation the day after <span class="drupalorg-gitlab-issue-link drupalorg-gitlab-link-wrapper"><a href="https://git.drupalcode.org/project/pdv/-/work_items/3619101" class="drupalorg-gitlab-link">https://git.drupalcode.org/project/pdv/-/work_items/3619101</a></span> reported it complete. Each msgid moves in the same commit, and the French is reworded too rather than merely re-keyed: the machine names were in the French as well.</p> <p>One entry is also refiled. <em>Data exports (pdv.export)</em> is declared in pdv_audit_trail's settings form but its French was sitting in the root translations/fr.po while its eight siblings were in pdv_audit_trail's. It resolved correctly, because locale keys on the source text and the root file is always imported, so this is tidiness rather than a bug. It came in misfiled with <span class="drupalorg-gitlab-issue-link drupalorg-gitlab-link-wrapper"><a href="https://git.drupalcode.org/project/pdv/-/work_items/3619101" class="drupalorg-gitlab-link">https://git.drupalcode.org/project/pdv/-/work_items/3619101</a></span>.</p> <h3>Metrics</h3> <p>docs/metrics.md is regenerated, because it counts Markdown and PO File lines and this change moves both. Leaving it would restale the page that <span class="drupalorg-gitlab-issue-link drupalorg-gitlab-link-wrapper"><a href="https://git.drupalcode.org/project/pdv/-/work_items/3619101" class="drupalorg-gitlab-link">https://git.drupalcode.org/project/pdv/-/work_items/3619101</a></span> had just brought up to date, in the same release, which is the defect that issue was fixing.</p> <h3>Verified</h3> <p>phpcs 0 errors; phpstan unchanged at 7 class.notFound for the absent message module; cspell 0 once the project word list is subtracted, the one raw hit being <em>unticked</em>, already listed and on an untouched part of an edited line; msgfmt clean on all 11 .po files; PdvEventTest 1/1 and AuditTrailSubscriberTest 3/3; and a two-way string check confirming each of the nine old strings is gone from the tree and each new one appears in both its source file and its .po. The metrics invariants all pass, including that production plus test equals code and that no rect exceeds its viewBox.</p> <p>AI-Generated: Yes (Claude Code was used to run the sweep, draft this issue summary, and write the string and translation changes on the merge request. I reviewed and ran the work myself before posting it.)</p>