Thread: pgAdmin III commit: Fix constraints handling on the table properties di
pgAdmin III commit: Fix constraints handling on the table properties di
From
Guillaume Lelarge
Date:
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=19a76d642f7db215c0d9ad432d13c02da8663833 Modified Files -------------- CHANGELOG | 2 ++ pgadmin/dlg/dlgTable.cpp | 16 ++++++++++++++-- pgadmin/include/dlg/dlgTable.h | 2 +- 3 files changed, 17 insertions(+), 3 deletions(-)
Re: pgAdmin III commit: Fix constraints handling on the table properties di
From
Erwin Brandstetter
Date:
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. Regards Erwin
Re: pgAdmin III commit: Fix constraints handling on the table properties di
From
Erwin Brandstetter
Date:
On Dec 19, 12:42 pm, Erwin Brandstetter <brsaw...@gmail.com> 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. > > Regards > Erwin Oops. s/1.14.3/1.14.2
Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di
From
Guillaume Lelarge
Date:
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. -- Guillaume http://blog.guillaume.lelarge.info http://www.dalibo.com PostgreSQL Sessions #3: http://www.postgresql-sessions.org
Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di
From
Erwin Brandstetter
Date:
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. Thanks for the good work! Regards Erwiin
Re: Re: pgAdmin III commit: Fix constraints handling on the table properties di
From
Guillaume Lelarge
Date:
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