From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Dhiraj Chawla <dhiraj(dot)chawla(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Grants on sequences via Grant Wizard fix patch |
Date: | 2013-05-26 21:31:45 |
Message-ID: | CA+OCxoyoBpWDmyJ65WhucF6JeiROo8eqQ432rYLMTavcCYrNug@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks Dhiraj - patch applied.
I noticed a related display bug on Mac (see the screenshot) where the
list of privileges selected has an unusably small size. It looks fine
on Windows. I suspect this is the same as the wx-originating bug we
sometimes see with that control on the sizeable dialogues, where it's
invisible until you resize it slightly, e.g. frmTable (of course, the
Grant Wizard is a fixed size so that trick doesn't work there).
Can you take a look when you get a minute please?
Thanks!
On Wed, May 22, 2013 at 11:40 PM, Dhiraj Chawla
<dhiraj(dot)chawla(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> Please find attached the updated patch after having removed the RULE grant.
> I have also made minor tweaks to my earlier code, by which when user selects
> a Sequences container and selects the grant wizard option, in the dialog the
> user sees only the grants options relevant to Sequence and not all the other
> options.
>
> Let me know your view on this patch.
>
> Thanks,
>
>
>
> On Thu, May 23, 2013 at 1:32 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>> Hi
>>
>> On Wed, May 22, 2013 at 8:01 PM, Dhiraj Chawla
>> <dhiraj(dot)chawla(at)enterprisedb(dot)com> wrote:
>> > Hi Dave,
>> >
>> > As per the issue reported on pgadmin-support mailing list related to
>> > grant
>> > wizard treating a sequence like a table and giving it grants similar to
>> > that
>> > of the table, attached is the patch that fixes this issue. According to
>> > this
>> > patch now we check for the sequences object separately and provide
>> > grants to
>> > it accordingly. Also according the mail, it was also reported the we
>> > were
>> > missing the USAGE grant in the Grant Wizard which I have added to it as
>> > part
>> > of this patch.
>>
>> Cool, thanks.
>>
>> > One thing I noticed though, as we have the RULE grant still existing in
>> > the
>> > Grant Wizard, which has been removed after PG-8.1. As the minimum
>> > supported
>> > version by the community is 8.4, should we remove the RULE grant from
>> > the
>> > Wizard??
>>
>> Yes please. Can you update the patch with that change?
>>
>> Thanks!
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>
>
>
>
> --
> regards,
>
> Dhiraj Chawla
> Software Engineer
> EnterpriseDB Corporation
> The Enterprise PostgreSQL Company
>
> Phone: +91-20-30589522
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
Screen Shot 2013-05-26 at 17.28.29.png | image/png | 110.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2013-05-26 21:59:56 | pgAdmin III commit: Fix schema prefixes in cast definitions in the brow |
Previous Message | Dave Page | 2013-05-26 21:27:25 | pgAdmin III commit: Treat sequences as first class objects in the Grant |