Bug in CHECK constraints statement reverse engineering.

Hello,

PgAdmin 1.3.0 (Apr 24 2005)
Wrong CHECK reverse engineering.

In PostgreSQL documentation i found:

----------------------------------------------------------------------
41.10. pg_constraint
Note
consrc is not updated when referenced objects change; for example,
it won't track renaming of columns. Rather than relying on this field,
it's best to use pg_get_constraintdef() to extract the definition
of a check constraint.
----------------------------------------------------------------------

Here the script to demonstraint this behaviour:

-- First create function and constraints on table field and domain
CREATE OR REPLACE FUNCTION "Check_IntegerGreaterZero"(int4)
RETURNS bool AS
$BODY$
select $1 > 0;
$BODY$
LANGUAGE 'sql' VOLATILE SECURITY DEFINER;

CREATE TABLE "TestTable"
(
test int4,
CONSTRAINT "CHK_TestTable_Test" CHECK ("Check_IntegerGreaterZero"(test))
)
WITHOUT OIDS;

CREATE DOMAIN "TestDomain"
AS int4
CONSTRAINT TestDomain_check CHECK "Check_IntegerGreaterZero"(VALUE);

-- Then just rename function
ALTER FUNCTION "Check_IntegerGreaterZero"(int4) RENAME TO "Check_IGZ";

--Watch pg_constraint and result of pg_get_constraintdef()
select conname, consrc, pg_get_constraintdef(pg_constraint.oid) from pg_constraint

conname |consrc |pg_get_constraintdef
--------------------------------------------------------------------------------------------------------------------------------
cardinal_number_domain_check |(VALUE >= 0) |CHECK ((VALUE>= 0))
CHK_TestTable_Test |"Check_IntegerGreaterZero"(test) |CHECK("Check_IGZ"(test))
TestDomain_check |"Check_IntegerGreaterZero"(VALUE) |CHECK("Check_IGZ"(VALUE))

I suppose that PgAdmin shows values from consrc.

Another feature request. The function described above ("Check_IntegerGreaterZero") in PgAdmin
right bottom pane shows as
-------------------------------------------------------------------------------------

-- Function: Check_IntegerGreaterZero(in_Value int4)

-- DROP FUNCTION "Check_IntegerGreaterZero"(int4);

CREATE OR REPLACE FUNCTION "Check_IntegerGreaterZero"(int4)
RETURNS bool AS
$BODY$
select $1 > 0;
$BODY$
LANGUAGE 'sql' VOLATILE SECURITY DEFINER;
ALTER FUNCTION "Check_IntegerGreaterZero"(int4) OWNER TO postgres;

-------------------------------------------------------------------------------------

It will be convenient for me if the first line will be

-- Function: "Check_IntegerGreaterZero"("in_Value" int4)

- quoted function name and argument names. It is useful for copy / paste purposes :)

Thank you for support.

--
Best regards,
Ivan mailto:Ivan-Sun1(at)mail(dot)ru