From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Euler Taveira de Oliveira <euler(at)timbira(dot)com> |
Cc: | "Decibel!" <decibel(at)decibel(dot)org>, pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: public.pg_class isn't a catalog table |
Date: | 2007-09-06 06:23:46 |
Message-ID: | 46DF9CF2.9060603@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers pgsql-bugs |
Euler Taveira de Oliveira a écrit :
> Decibel! wrote:
>
>> Version 1.6.3; if you have a table in public called pg_class and try to
>> drop it, pgAdmin gives an error:
>>
>> ERROR: permission denied: "pg_class" is a system catalog
>>
> That's because pgAdmin is issuing "DROP TABLE pg_class" instead of "DROP
> TABLE public.pg_class". By the way, is there any reason for not using
> the namespace?
> I don't know if it's the best way but this little proof of concept fix
> might be applied to all objects that can be created inside a schema. I
> could provide I patch if you want it.
>
Thanks for your patch. I took a look at it and I wonder why you check if
the object's schema is the default one. I mean, we just need to add the
object's schema and we're done, aren't we ?
And do we have a GetQuotedName method for the schema ? if yes, it would
be better to use it.
Regards.
--
Guillaume.
<!-- http://abs.traduc.org/
http://lfs.traduc.org/
http://docs.postgresqlfr.org/ -->
From | Date | Subject | |
---|---|---|---|
Next Message | Dhanaraj M | 2007-09-06 07:05:30 | Re: pgAdmin3 - Language support on Solaris |
Previous Message | Euler Taveira de Oliveira | 2007-09-06 04:35:58 | Re: public.pg_class isn't a catalog table |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2007-09-06 07:54:37 | Re: public.pg_class isn't a catalog table |
Previous Message | Euler Taveira de Oliveira | 2007-09-06 04:35:58 | Re: public.pg_class isn't a catalog table |