| From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
|---|---|
| To: | Dave Page <dpage(at)postgresql(dot)org> |
| Cc: | Mike(dot)Blackwell(at)rrd(dot)com, pgadmin-support(at)postgresql(dot)org |
| Subject: | Re: Server order |
| Date: | 2007-05-23 22:01:58 |
| Message-ID: | 4654B9D6.80102@lelarge.info |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-support |
Dave Page a écrit :
> Guillaume Lelarge wrote:
>> Mike(dot)Blackwell(at)rrd(dot)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.
> BTW; did you fix that SQL formatting error that Erwin reported?
>
No, I'll take care of this tomorrow.
Regards.
--
Guillaume.
<!-- http://abs.traduc.org/
http://lfs.traduc.org/
http://docs.postgresqlfr.org/ -->
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Guillaume Lelarge | 2007-05-23 22:07:48 | Re: List to Father Christmas |
| Previous Message | Joe Moyle | 2007-05-23 19:29:28 | Re: Bug Report - PGAdmin3 windows pgpass.conf passwords stored in plain text |