Re: Server order - Mailing list pgadmin-support

From Dave Page
Subject Re: Server order
Date
Msg-id 465516B0.7070700@postgresql.org
Whole thread Raw
In response to Re: Server order  (Guillaume Lelarge <guillaume@lelarge.info>)
List pgadmin-support
Guillaume Lelarge wrote:
> Dave Page a écrit :
>> Guillaume Lelarge wrote:
>>> Mike.Blackwell@rrd.com a écrit :
>>>> Is there a way to change the order of the servers in the Servers list?
>>>>  It'd be nice if they were alphabetically sorted.
>>> The patch attached sorts them alphabetically. As it changes the
>>> behavior, perhaps an option is needed to enable it. Dave, what do you
>>> think about it ?
>>
>> I'm in a lecture atm, so can't check that thoroughly, but I think you
>> missed an AppendItem() call site further up in
>> pgServerFactory::CreateObjects (you only got the one where pgInstaller
>> installations are auto-added.
>>
> 
> I've got three of them :
>  - start of pgAdmin, CreateObjects adds the already registered servers ;
>  - after that, still in CreateObjets, pgAdmin tries to find PostgreSQL
> services on win32 servers ;
>  - when a user registers a new server, it calls StartDialog.
> 
> I don't find more of browser->AppendItem. There are a few other
> AppendItem but for the properties variable.
> 
> I've seen another browser->AppendItem on dlgProperty.cpp but it does not
> seem to update the server list, doesn't it ?
> 
>> Otherwise, please commit.
>>
> 
> I didn't commit it yet because of the AppendItem missed that I can't find.

Sorry, my fault - I misread the patch. Told you I couldn't check
thoroughly :-p

>> BTW; did you fix that SQL formatting error that Erwin reported?
>>
> 
> No, I'll take care of this tomorrow.


Thanks, Dave


pgadmin-support by date:

Previous
From: "Joris Dobbelsteen"
Date:
Subject: Re: Server order
Next
From: yoursoft
Date:
Subject: View data / view bug