Skip site navigation (1) Skip section navigation (2)

Re: pgAdmin III commit: Fix selection of objects in backup/restore dialogs

From: Thom Brown <thom(at)linux(dot)com>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgAdmin III commit: Fix selection of objects in backup/restore dialogs
Date: 2011-03-14 21:17:56
Message-ID: AANLkTinnbiAhSVxu7w95vZdrgidDw2-USk+pnFCKJxwZ@mail.gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
On 12 March 2011 09:27, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
> Fix selection of objects in backup/restore dialogs
>
> When a user selects a table, the schema and the database should be selected as
> well. This patch handles this automatically.
>
> Report from Melvin Davidson.
>
> Branch
> ------
> REL-1_12_0_PATCHES
>
> Details
> -------
> http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=cbf2e0765ebf18da6256852bf53021d26287630a
>
> Modified Files
> --------------
> CHANGELOG                              |    2 ++
> pgadmin/ctl/ctlCheckTreeView.cpp       |   15 ++++++++++++---
> pgadmin/include/ctl/ctlCheckTreeView.h |    1 +
> 3 files changed, 15 insertions(+), 3 deletions(-)

Sorry for the following bit of noise, but...

Is there any need for this line?:

if (newimage == 0 || newimage == 1)

newimage is initialised as 0, and can only otherwise be set to 1, so
I'm not sure where this logic comes in.

Also, I'm not clear as to why if, when newimage = 1, the parent image
is seemingly set twice.  I'm referring to the function being called
suggesting both the parent and child image is being set, followed by
the parent image.  Shouldn't there at least be an "else if" in there,
or is one of them misnamed?

Cheers

-- 
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935

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

In response to

Responses

pgadmin-hackers by date

Next:From: Guillaume LelargeDate: 2011-03-14 22:02:49
Subject: Re: pgAdmin III commit: Fix selection of objects in backup/restore dialogs
Previous:From: Dave PageDate: 2011-03-14 20:46:43
Subject: Re: [pgadmin-hackers] Re: [pgadmin-hackers] Re: [pga?==?windows-1252?Q?dmin-hackers] error: expected initializer before ‘!’ tok?==?windows-1252?Q?en

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group