On Thu, Nov 20, 2025 at 8:28 PM Álvaro Herrera <[email protected]> wrote:

> Hi,
>
> > -                appendStringInfo(&buffer, _("text search configuration
> %s"),
> > -                                 quote_qualified_identifier(nspname,
> > -
> NameStr(cfgForm->cfgname)));
> > +                appendStringInfoQualifiedIdentifier(&buffer,
> > +                                                    _("text search
> configuration "),
> > +                                                    nspname,
> NameStr(cfgForm->cfgname), NULL);
> >                  ReleaseSysCache(tup);
> >                  break;
> >              }
>
> This doesn't work from a i18n point of view.  In the original
> formulation, the translator is free to place the %s wherever it suits
> the language.  In the new one there's no such freedom: the name will be
> appended at the end.  Some existing translations:
>
> ko.po:msgid "text search configuration %s"
> ko.po-msgstr "%s 전문 검색 구성"
>
> tr.po:msgid "text search configuration %s"
> tr.po-msgstr "%s metin arama yapılandırması"
>

Thanks for the feedback. I reverted that piece of change in v3.

Best regards,
Chao Li (Evan)
---------------------
HighGo Software Co., Ltd.
https://www.highgo.com/

Attachment: v3-0001-Add-appendStringInfoIdentifier-to-avoid-intermedi.patch
Description: Binary data

Attachment: v3-0003-Remove-quoteOneName-and-related-buffer-sizing-mac.patch
Description: Binary data

Attachment: v3-0004-Use-appendStringInfoIdentifier-in-more-places.patch
Description: Binary data

Attachment: v3-0002-Use-appendStringInfoIdentifier-throughout-ri_trig.patch
Description: Binary data

Reply via email to