Page MenuHomePhabricator

Correct a mangled translation string in "bin/phd log --id X"
ClosedPublic

Authored by epriestley on Oct 24 2018, 4:41 PM.
Tags
None
Referenced Files
F15462951: D19755.id47206.diff
Tue, Apr 1, 7:13 PM
F15461346: D19755.id47206.diff
Tue, Apr 1, 7:42 AM
F15450630: D19755.id47188.diff
Fri, Mar 28, 4:48 PM
F15445314: D19755.id.diff
Thu, Mar 27, 1:25 PM
F15440756: D19755.diff
Wed, Mar 26, 2:23 PM
F15429413: D19755.id47188.diff
Mon, Mar 24, 2:06 AM
F15369475: D19755.diff
Mar 12 2025, 7:02 AM
F15341325: D19755.diff
Mar 9 2025, 7:43 PM
Subscribers
None

Details

Summary

Ref T13210. See PHI930. This translation is wrong: the parameter is a comma-separated list as a string, but the USEnglish translation provides alternatives. We can't select among alternatives based on a random string (it isn't a plurality value to let us select "chair" vs "chairs", and isn't a gender value to let us select "his profile" vs "her profile") so we get an error.

But the string itself is also misleading, since "bin/phd log --id A --id B --id C" will say "none of these are valid" if any of them are invalid.

Instead, just tell the user explicitly about the first problem.

Test Plan
  • Ran bin/phd log --id with good (got logs) and bad IDs (got sensible error).
  • Ran bin/phd log with any logs (got logs) and (simluated) without any logs (got error).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable