Re: pgAdmin3: check_option translated in View editing dialog

From: Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
To: Sebastian Albert <albert(at)math(dot)uni-goettingen(dot)de>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin3: check_option translated in View editing dialog
Date: 2015-12-28 10:43:41
Message-ID: CACCA4P3ZAzSrht8O70NocLN6wCUeytGs-bB+CtnSz36zg5YdPw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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-----
>>
>
>

Attachment Content-Type Size
View_Fix.patch application/octet-stream 3.0 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Martin Andersson 2015-12-29 10:13:47 ]GTK 3
Previous Message Ashesh Vashi 2015-12-28 04:47:23 pgAdmin 4 commit: Keep all the information in the tree node data in the