Hi,
On 2023-Apr-03, Andres Freund wrote:
> Hm? That's what the _'s do. We build strings in parts in other places too.
No, what _() does is mark each piece for translation separately. But a
translation cannot be done on string pieces, and later have all the
pieces appended together to form a full sentence. Let me show the
"!terminating" case as example and grab some translations for it from
src/backend/po/de.po:
"invalidating" -> "... wird ungültig gemacht" (?)
(if logical) " obsolete replication" -> " obsolete Replikation"
" slot \"%s\" because it conflicts with recovery" -> " Slot \"%s\", weil sie in Konflikt mit Wiederherstellung steht"
If you just concatenate all the translated phrases together, the
resulting string will make no sense; keep in mind the "obsolete
replication" part may or not may not be there. And there's no way to
make that work: even if you found an ordering of the English parts that
allows you to translate each piece separately and have it make sense for
German, the same won't work for Spanish or Japanese.
You have to give the translator a complete phrase and let them turn into
a complete translated phrases. Building from parts doesn't work. We're
very good at avoiding string building; we have a couple of cases, but
they are *very* minor.
string 1 "invalidating slot \"%s\" because it conflicts with recovery"
string 2 "invalidating obsolete replication slot \"%s\" because it conflicts with recovery"
(I'm not clear on why did Bertrand omitted the word "replication" in the
case where the slot is not logical)
I think the errdetail() are okay, it's the errmsg() bits that are bogus.
And yes, well caught on having to use errmsg_internal and
errdetail_internal() to avoid double translation.
Cheers
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/