Re: pgAdmin3: check_option translated in View editing dialog

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: Sebastian Albert <albert(at)math(dot)uni-goettingen(dot)de>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin3: check_option translated in View editing dialog
Date: 2015-12-30 11:05:26
Message-ID: 28D414BF-856A-4D21-97BE-33EC3AAF6DF7@pgadmin.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Ashesh, can you review/commit please? Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK:http://www.enterprisedb.com
The Enterprise PostgreSQL Company

> On 28 Dec 2015, at 10:43, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> wrote:
>
> Hi Dave,
>
> I have fixed the issue. Can you please review the attached patch file ? Let me know for any comments.
>
> @Sebastian - Can you please apply and test the attached patch file ? Let us know for any issue.
>
> Thanks,
> Neel Patel
>
>> On Thu, Dec 24, 2015 at 5:44 PM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> wrote:
>> Hi Sebastian,
>>
>> Thank you for the information. Issue is reproduced with the given steps.
>> I will submit the patch after the fix.
>>
>> Thanks,
>> Neel Patel
>>
>>> On Thu, Dec 24, 2015 at 5:28 PM, Sebastian Albert <albert(at)math(dot)uni-goettingen(dot)de> wrote:
>>> -----BEGIN PGP SIGNED MESSAGE-----
>>> Hash: SHA256
>>>
>>> Hi Neel
>>>
>>> Thanks for looking into it. Dave is right, it's about the SQL
>>> statement not working (because of translation appearing for a sort of
>>> keyword) *unless* edited manually. You should not *have to* edit the
>>> SQL manually, but due to the issue, you do, no matter what property of
>>> the view you are trying to change.
>>>
>>> So, to reproduce:
>>> 1. Create a view
>>> 2. Set language of pgAdmin to German
>>> 3. Try to change anything on the view, e.g., try to rename it, using
>>> the properties dialog.
>>>
>>> Unless I edit the SQL statement, I am presented an SQL error, which
>>> comes from the check_option setting being translated.
>>>
>>> Happy holidays
>>> Sebastian
>>>
>>> On 24.12.2015 12:20, Dave Page wrote:
>>> > On Thu, Dec 24, 2015 at 11:18 AM, Neel Patel
>>> > <neel(dot)patel(at)enterprisedb(dot)com> wrote:
>>> >> Hi Sebastian,
>>> >>
>>> >> We have just tried to reproduce the issue but not able to
>>> >> understand the issue from the description.
>>> >>
>>> >> We followed below steps.
>>> >>
>>> >> 1. Change the language to German. 2. Open the existing View's
>>> >> properties and try to change "With check option". 3. Open SQL
>>> >> tab - According to change in option, we are able to see and edit
>>> >> the query.
>>> >>
>>> >> Let me know if anything missing in above steps or Can you please
>>> >> give steps to understand the issue ?
>>> >
>>> > Hi Neel,
>>> >
>>> > I believe the issue is that "no" is being translated to "nein" in
>>> > the SQL:
>>> >
>>> > ALTER VIEW myview SET (check_option=nein);
>>> >
>>> -----BEGIN PGP SIGNATURE-----
>>> Version: GnuPG v2
>>>
>>> iQIcBAEBCAAGBQJWe93qAAoJEKZN7hpT3kUKTrgP/1wmrrScAtfvNc269K9+t1sa
>>> UKyBfyMFvSgcMgydLC6bjsV5ECN2EVeV/qtNy6tGIlpWppzTtXccQxbKvQiwkyT1
>>> F3MHM6tp869kegB0N2ZU5mzE+8ko/PaFRAISQomSlpxpDsATZ+5nR3VnFnh5DMde
>>> 4XYbPvdZWH4Kos9yrWDKTxi5FVHbCvj4m86Qxg8hMlOGUixRDlVx77t1dccchEjq
>>> OJRenlQWO9x65iRQbobXJtJddyQLplulc7enBgSJapQahQ4QTqe4RTx7f4Qya2B8
>>> 9bIndvgIvbI08cnss7BQ8iJvK737gMoHc9CscwEAwadaI69/2QAlB4wtwS5bwIZp
>>> DIeW+ulNJC40ZX6Ol/QMNZEvm1pBdwncLAUJBaiHht6M9rJcXld99uEPJDfP5wji
>>> 2M9WAa3fjyLJFV3d1lA2GYujPgUBAr7KCRcAxgQSXopCj8KICYJzCOkP5skHIPlL
>>> zYEMDML3uATmMPbabhy65/mf44WgzjebnQwM86iwnPRbWzfzSrWyK26fm+9CZ6aF
>>> kgZvS9bWtQDViS/+6vTMuR3YQK4H+iU88RCIDbrHa8Kn53S8ryglg1f2CKtqrV9z
>>> 2bNlK+iCNFxHoz1ApDw0CE8cJTKU31fL/B17+gB/bL6cCtnpzyCjNosRtO7vr89b
>>> 7N9Y5AOnyVwbQD42/3EN
>>> =KwP3
>>> -----END PGP SIGNATURE-----
>
> <View_Fix.patch>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dmitriy Olshevskiy 2015-12-30 18:46:00 Re: patch: fix to use ``pg_get_functiondef()``
Previous Message Martin Andersson 2015-12-29 10:13:47 ]GTK 3