Re: quoteOneName() inconsistency with quote_all_identifiers — replacement API proposed - Mailing list pgsql-hackers

From Chao Li
Subject Re: quoteOneName() inconsistency with quote_all_identifiers — replacement API proposed
Date
Msg-id CAEoWx2nPbikvjp0JBsHzbpE=u7aMkiVy-WCmPim2iX5hDF9fBw@mail.gmail.com
Whole thread Raw
In response to Re: quoteOneName() inconsistency with quote_all_identifiers — replacement API proposed  (Álvaro Herrera <alvherre@kurilemu.de>)
List pgsql-hackers

On Thu, Nov 20, 2025 at 8:28 PM Álvaro Herrera <alvherre@kurilemu.de> 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.
Attachment

pgsql-hackers by date:

Previous
From: Yuefei Shi
Date:
Subject: Re: Extended Statistics set/restore/clear functions.
Next
From: Tatsuo Ishii
Date:
Subject: Re: Row pattern recognition