Skip site navigation (1) Skip section navigation (2)

Re: pgAdmin III: EDB procedures with non void return types not handled properly

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Nikhil S <nixmisc(at)gmail(dot)com>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgAdmin III: EDB procedures with non void return types not handled properly
Date: 2011-01-31 13:23:36
Message-ID: AANLkTi=i1WE9fjoFa036Z6nfwwZ7Z-sjYFCnC4m9Hjjo@mail.gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
On Mon, Jan 31, 2011 at 7:17 AM, Nikhil S <nixmisc(at)gmail(dot)com> wrote:
> Hi,
>
> With the latest refactorings in the EDBAS90 source base, procedures (CREATE
> PROCEDURE objects) can have non void return types in some cases. For
> example:
>
> edb=# create or replace procedure set_Order (
>     v_index        INOUT  integer
> ) AS
>     counter    integer;
> BEGIN
>     counter := 0;
>     v_index := counter;
> END;
> CREATE PROCEDURE
> edb=# select protype, prorettype from pg_proc where proname = 'set_order';
>  protype | prorettype
> ---------+------------
>  1       |         23
>
> Here the prorettype is "int4" and protype is set to 1 to indicate that this
> is a procedure.
>
> The existing pgadmin source currently expects procedures to have "void"
> return types only. Now that the "protype" column (which can differentiate
> between functions and procedures) is available since 8.1, we can use it to
> identify such objects without resorting to "void" type checks. Ofcourse we
> need to retain existing checks for older versions.
>
> PFA, patch which does the same.

Thanks. This seems to be a bug in fact, as it could mis-identify a
function that returns void as a procedure.

Applied for 1.12.3 and master.


-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

pgadmin-hackers by date

Next:From: Nikhil SDate: 2011-01-31 13:32:54
Subject: Re: pgAdmin III: EDB procedures with non void return types not handled properly
Previous:From: Dave PageDate: 2011-01-31 13:21:16
Subject: pgAdmin III commit: Fix a bug in the way that stored procedures andfun

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group