pg.ddx.io  pgsql-sql@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Richard Hayward <richard@tortoise.demon.co.uk>
To: pgsql-sql@postgresql.org
Subject: function parameters : bug?
Date: Sun, 10 Jul 2005 11:52:10 +0100
Message-ID: <ugu1d19614tuteki2jgm0496d58kcggdhn@4ax.com> (raw)

pg 8.0.3
This behaviour seems odd to me:

CREATE TABLE mytable (
  inta INTEGER
);

CREATE OR REPLACE FUNCTION myfunction (inta integer) RETURNS integer 
AS 
$body$ 
begin
  insert into mytable(inta) values (inta);
  return(0);
end;
$body$
LANGUAGE 'plpgsql';

select myfunction(123);

ERROR:  syntax error at or near "$1" at character 22
QUERY:  insert into myTable( $1 ) values ( $2 )
CONTEXT:  PL/pgSQL function "myfunction" line 2 at SQL statement

The problem is with the line in my myfunction:
  insert into mytable(inta) values (inta);

The server doesn't like the field name being the same as the name of
the function parameter. I don't see why, as I can see no opportunity
for ambiguity.

Problem vanishes with a different function parameter name:

CREATE OR REPLACE FUNCTION myfunction (my_inta integer) RETURNS
integer 
AS $body$ begin
  insert into myTable(inta) values (my_inta);
  return(0);
end;
$body$
LANGUAGE 'plpgsql'


But, this means that if I change the design of my tables, I have to
possibly change my function headers rather than merely the code inside
them.

regards
Richard







view thread (2+ messages)  latest in thread

Message-ID: <ugu1d19614tuteki2jgm0496d58kcggdhn@4ax.com>
Permalink:  ../ugu1d19614tuteki2jgm0496d58kcggdhn@4ax.com/
Also on:    postgresql.org/message-id/ugu1d19614tuteki2jgm0496d58kcggdhn@4ax.com

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-sql@postgresql.org
  Cc: richard@tortoise.demon.co.uk
  Subject: Re: function parameters : bug?
  In-Reply-To: <ugu1d19614tuteki2jgm0496d58kcggdhn@4ax.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox