Thanks - applied.
On Mon, Jul 12, 2010 at 1:21 PM, Ashesh Vashi
> Hi Dave,
> PFA the updated patch.
> [Please check my inline answers.]
> On Mon, Jul 12, 2010 at 4:37 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> On Mon, Jul 12, 2010 at 11:39 AM, Ashesh Vashi
>> <ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>> > Hi Dave,
>> > Please find the patch for the same.
>> Thanks. This looks good to me following some quick testing, but I
>> uncovered some other issues - could you look at them please?
>> - We seem to stop populating the server combo box as soon as we find
>> the one we're currently connected to. This happens at ~ line 286 in
>> dlgSelectConnection.cpp, where we set foundServer = true. I can't see
>> why we do this, but it seems fairly deliberate. Any guesses? Should we
>> just remove that?
> My mistake - I introduced that variable in the previous patch.
>> - It seems like we ignore sslmode when we create new connections. We
>> need to honour the server config.
>> Note that I haven't tested to ensure this doesn't break anything in
>> command line startup mode (eg. with -q). Can you please ensure this
>> mode is unaffected?
> Thanks & Regards,
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise Postgres Company
>> Dave Page
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise Postgres Company
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise Postgres Company
In response to
pgadmin-hackers by date
|Next:||From: pgAdmin Trac||Date: 2010-07-13 13:56:42|
|Subject: [pgAdmin III] #212: Changes to functions via properties dialog not
|Previous:||From: svn||Date: 2010-07-13 08:31:22|
|Subject: SVN Commit by dpage: r8467 - in trunk/pgadmin3/pgadmin: . dlg frm include/dlg|