Re: public.pg_class isn't a catalog table

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/ -->

In response to

Browse pgadmin-hackers by date

  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

Browse pgsql-bugs by date

  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