Hi,
On Tue, Sep 7, 2010 at 6:16 AM, Alvaro Herrera
<alvherre@commandprompt.com> wrote:
> Excerpts from Jim Nasby's message of jue jun 10 17:54:43 -0400 2010:
>> test_us@workbook=# select has_table_privilege( 'public', 'test', 'SELECT' );
>> ERROR: role "public" does not exist
>
> Here's a patch implementing this idea.
It specially treats only "public" in all lower cases, right?
The pseudo-role name is described as "PUBLIC" (upper) in docs,
but we accept only "public" (lower) as the pseudo-name.
BTW, does the patch need to be back-patched to older versions?
Since they use get_roleid_checked() instead of get_role_oid(), the fix
cannot be applied cleanly to them, though it will be similar codes.
--
Itagaki Takahiro
