Re: Prevent duplicate attributes

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Thom Brown <thom(at)linux(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Prevent duplicate attributes
Date: 2011-07-12 09:53:19
Message-ID: CA+OCxozVEFrgkNU8iXw_tzm_L3nCYjd+jjU9XQd4JwPa_raboA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Sat, Jul 9, 2011 at 9:02 PM, Thom Brown <thom(at)linux(dot)com> wrote:
> Hi (yes, me... again),
>
> Tiny patch attached to prevent adding attributes with a duplicate name
> to a type.  It seems previous versions prevented this, but must have
> been allowed in a commit since the last release.

For consistency with the other checks on the dialogue, this should
cause the "Add" button to be disabled, not make it silently fail. That
should almost certainly be done in dlgType::CheckChange().

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Thom Brown 2011-07-12 09:55:25 Re: Prevent duplicate attributes
Previous Message Thom Brown 2011-07-11 14:29:51 Re: Overhaul of type attributes modification