Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di - Mailing list pgadmin-hackers

From Guillaume Lelarge
Subject Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di
Date
Msg-id 1330359398.2543.38.camel@localhost.localdomain
Whole thread Raw
In response to Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di  (Erwin Brandstetter <brsaweda@gmail.com>)
List pgadmin-hackers
On Mon, 2012-02-27 at 16:54 +0100, Erwin Brandstetter wrote:
> On 19.12.2011 14:07, Guillaume Lelarge wrote:
> > On Mon, 2011-12-19 at 03:42 -0800, Erwin Brandstetter wrote:
> >> On Dec 10, 11:46 am, guilla...@lelarge.info (Guillaume Lelarge) wrote:
> >>> Fix constraints handling on the table properties dialog
> >>>
> >>> The wxListCtrl control doesn't handle strings or more than 512 characters on
> >>> Windows. So we need to put the definition of constraints in a wxArrayString so
> >>> that we won't have issues on the size of the constraint's definition.
> >>>
> >>> Report from Vjacheslav Vjacheslav.
> >>> Some help from Julius Tuskenis.
> >>>
> >>> Branch
> >>> ------
> >>> REL-1_14_0_PATCHES
> >>>
> >>> Details
> >>> -------http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=19a76d...
> >>>
> >>> Modified Files
> >>> --------------
> >>> CHANGELOG                      |    2 ++
> >>> pgadmin/dlg/dlgTable.cpp       |   16 ++++++++++++++--
> >>> pgadmin/include/dlg/dlgTable.h |    2 +-
> >>> 3 files changed, 17 insertions(+), 3 deletions(-)
> >>>
> >>> --
> >>> Sent via pgadmin-hackers mailing list (pgadmin-hack...@postgresql.org)
> >>> To make changes to your subscription:http://www.postgresql.org/mailpref/pgadmin-hackers
> >> This is quite a crippling bug. I ran into this after recreating a
> >> table in PostgreSQL 8.4 by using the reverse engineered SQL in the SQL
> >> pane. I did not realize the nonsensical statements further down at
> >> first. Like:
> >>
> >> CREATE CONSTRAINT TRIGGER "RI_ConstraintTrigger_142372"
> >>    AFTER INSERT
> >>    ON wuchtel.bet
> >>    FOR EACH ROW
> >>    EXECUTE PROCEDURE "RI_FKey_check_ins"();
> >>
> >> The DDL commands worked - the table was created. But later when I
> >> tried to insert data I ran into strange errors:
> >>     No »pg_constraint«-entry for trigger »RI_ConstraintTrigger_142372«
> >> for table »bet2«
> >>
> >> I am afraid this will make quite a few users trip. I would very much
> >> welcome an early release of 1.14.3.
> >>
> > This bug is not fixed yet.
>
> Not sure which bug it was exactly, but with 1.14.2 (and PostgreSQL 9.1
> now) the bug is gone.

Fixed with this patch:
http://git.postgresql.org/gitweb/?p=pgadmin3.git;a=commit;h=e21e69225c4c003c040f29f2b73bb5a25ddb01d9

> Thanks for the good work!
>

You're welcome :)


--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com


pgadmin-hackers by date:

Previous
From: Erwin Brandstetter
Date:
Subject: Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di
Next
From: Dave Page
Date:
Subject: pgAdmin III commit: Additional warning/assert fixes for wx2.9 support.