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