Re: Proposed Patchs

2006/5/29, Dave Page <dpage(at)vale-housing(dot)co(dot)uk>:
>
>
> > -----Original Message-----
> > From: Thomas Sondag [mailto:thomas(dot)sondag(at)gmail(dot)com]
> > Sent: 29 May 2006 10:02
> > To: Dave Page
> > Cc: pgadmin-support(at)postgresql(dot)org
> > Subject: Re: [pgadmin-support] Proposed Patchs
> >
> > Hum, I miss this option ... sorry, but the main difference with the
> > current behaviour is for object owning. The main idea was to set
> > object owner to a group like that :
> > database foo -> group foo
> > schema bar -> group bar
> > schema bar read user -> user toto
> >
> > I don't know if that's a good policy, but this case may exist, we may
> > add an option like "Show Group for object owning" ?
>
> Yes, that seems like a much better option - though I wonder if it should
> default to just groups and then use the existing setting as per the
> permissions. That would seem more consistent.
>

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.

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

Thomas