Re: Proposed Patchs

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: <thomas(at)microdop(dot)org>
Cc: <pgadmin-support(at)postgresql(dot)org>
Subject: Re: Proposed Patchs
Date: 2006-05-30 12:57:57
Message-ID: E7F85A1B5FF8D44C8A1AF6885BC9A0E401388582@ratbert.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-support

> -----Original Message-----
> From: Thomas Sondag [mailto:thomas(dot)sondag(at)gmail(dot)com]
> Sent: 30 May 2006 13:42
> To: Dave Page
> Cc: pgadmin-support(at)postgresql(dot)org
> Subject: Re: [pgadmin-support] Proposed Patchs
>
> I'm not sure this is the better way, because most of the time objects
> are owned by the current user, so showing only group by default can be
> a bit confusing.

There's no reason we cannot use groups + current user. I do think it
would be more confusing to have the behaviour work in precisely the
opposite way to the privileges.

> If you want to see the result with the "Show group ..." option try
> the folowing patch . the main idea for AddRoles function was to remove
> one sql query.
>
> I test this patch only with postgresql 8.1 and it stop
> segfaulting now :)

Think you forgot the patch :-(

/D

Responses

Browse pgadmin-support by date

  From Date Subject
Next Message Thomas Sondag 2006-05-30 13:14:45 Re: Proposed Patchs
Previous Message Thomas Sondag 2006-05-30 12:42:05 Re: Proposed Patchs