Re: help with function

> Hello,
>
> I have 2 tables where each table has a column named "comments" and the
> tables are related as a one to many.  I want to concatenate all the
> comments of the many side to the one side so I wrote the following
> plpgsql function to do so.
>
> <pre>
> CREATE OR REPLACE FUNCTION fixcomments()
>    RETURNS int4 AS
> $BODY$
> DECLARE
>      mviews RECORD;
>      i int4;
> BEGIN
>
>      FOR mviews IN SELECT * FROM saleorder WHERE comments is not null
> and comments <> '' LOOP
>
>          -- Now "mviews" has one record from saleorder
>
>          EXECUTE 'UPDATE sale SET comments = ' ||
> quote_ident(sale.comments) || quote_ident(mviews.comments)
>         || ' WHERE sale.id = ' || quote_ident(mviews.sale_id);

EXECUTE 'UPDATE sale SET comments = ''' || quote_ident(sale.comments ||
mviews.comment) || ''' WHERE sale.id = ''' ||
quote_ident(mviews.sale_id) || '''';

Does that help?


>      i := i + 1;
>      END LOOP;
>
>      RETURN i;
> END;
> $BODY$
>    LANGUAGE 'plpgsql' VOLATILE;
>
> </pre>
>
> I have the following error when I run this code:
>
> <pre>
> ERROR:  missing FROM-clause entry for table "sale"
> CONTEXT:  SQL statement "SELECT  'UPDATE sale SET comments = ' ||
> quote_ident(sale.comments) || quote_ident( $1 ) || ' WHERE sale.id = '
> || quote_ident( $2 )"
> PL/pgSQL function "fixcomments" line 11 at execute statement
> </pre>
>
> Doesn anybody know what I am doing wrong here ?
>
> Lacou.
>
>
> ---------------------------(end of
broadcast)---------------------------
> TIP 6: explain analyze is your friend