Re: Proposed Patchs

["Thomas Sondag" <thomas(dot)sondag(at)gmail(dot)com>]

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