From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [patch][pgAdmin] Fix for pgadmin4-linux-qa #1651 failure |
Date: | 2021-06-29 08:26:07 |
Message-ID: | CA+OCxozh_DV2m7vcEAaD0rsY31OntJF5GdRBXmqj0PCWULStew@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Tue, Jun 29, 2021 at 4:38 AM Rahul Shirsat <
rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
> I feel gettext sometimes won't escape the characters as it should be.
>
> I now tried to escape those using some utils.
>
That won't work either. The string being passed to gettext() *must* be in
the gettext call.
When gettext extracts strings to create/update the catalogs, it will search
the code for all gettext calls, and then extract a string constant from the
first argument. You cannot have variables, function calls or expressions in
there. It *must* be a string constant.
Keep in mind that msgextract is scanning the source code; it's not
executing it. There are many examples in the code, e.g. (from node.js):
title = gettext('Drop %s?', obj.label);
I don't see anything obviously wrong with the existing code. Are you sure
there are no weird unprintable characters in there?
>
> Please find the updated patch.
>
> On Mon, Jun 28, 2021 at 9:33 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Mon, Jun 28, 2021 at 4:57 PM Rahul Shirsat <
>> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Please find the attached patch for fixation of jenkins failure.
>>>
>>
>> That won't work - you can't include variables (or string building
>> operations) in the first argument to gettext calls, as there won't be any
>> way to extract a complete message into the catalogs. The way it's being
>> done at the moment is correct (I don't know why it's failing, but it's the
>> correct way to structure the gettext calls).
>>
>> --
>> Dave Page
>> Blog: https://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EDB: https://www.enterprisedb.com
>>
>>
>
> --
> *Rahul Shirsat*
> Senior Software Engineer | EnterpriseDB Corporation.
>
--
Dave Page
Blog: https://pgsnake.blogspot.com
Twitter: @pgsnake
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2021-06-29 09:09:06 | Re: [pgAdmin][RM6130] React based framework for properties dialog and port Server Group, Server and Database dialogs |
Previous Message | Rahul Shirsat | 2021-06-29 03:38:13 | Re: [patch][pgAdmin] Fix for pgadmin4-linux-qa #1651 failure |