Re: Patch: Config editors - Mailing list pgadmin-hackers

From Magnus Hagander
Subject Re: Patch: Config editors
Date
Msg-id 6BCB9D8A16AC4241919521715F4D8BCE6C7F18@algol.sollentuna.se
Whole thread Raw
In response to Patch: Config editors  ("Magnus Hagander" <mha@sollentuna.net>)
List pgadmin-hackers
Updated version attached:

* Include xrc file that was missing last time ;-) Haven't included the
appropriate .cpp update because it was huge ;-)
* Fix help button in all editors (didn't work in hba editor either..)
* Add delete button
* Disable the hint button in the hba and pgpass editors, because we
don't hint anyway
* "Fix" the undo by disabling the undo button when you delete something,
so you don't think it works (see below)


Now, as for the undo functionality. The undo as it is now in these
dialogs don't work like a standard undo which is "undo the latest edit".
It is "undo the latest edit on the current line". So this patch actually
follows that (since there is no way to select a deleted line). Changing
this to a "normal undo" is a different patch alltogether ;-)

For example:
1) Edit line #1
2) Edit line #2
3) Select line #1
4) Click undo
5) Watch how the change from (1) gets reverted, not the one from (2)


//Magnus


> -----Original Message-----
> From: Dave Page [mailto:dpage@vale-housing.co.uk]
> Sent: Monday, January 23, 2006 4:00 PM
> To: Magnus Hagander; pgadmin-hackers@postgresql.org
> Subject: RE: [pgadmin-hackers] Patch: Config editors
>
> Patch discussed with Magnus on IM. To be reposted with fixes for the
> following:
>
> - Undo button doesn't undo a delete.
> - There is no delete button on the dialogue
> - Help button on the main editor window doesn't work.
>
> Regards, Dave.
>
> > -----Original Message-----
> > From: pgadmin-hackers-owner@postgresql.org
> > [mailto:pgadmin-hackers-owner@postgresql.org] On Behalf Of Magnus
> > Hagander
> > Sent: 23 January 2006 13:18
> > To: pgadmin-hackers@postgresql.org
> > Subject: [pgadmin-hackers] Patch: Config editors
> >
> > Attached patch implements an editor for pgpass.conf/.pgpass
> modeled on
> > the HBA editor. In doing so it also:
> >
> > * Adds the ability to delete a row to the pg_hba editor
> > * Moves the "get the name of the password file" to pgSettings so it
> > can be shared. Sets up a tiny bit of infrastructure for
> when there may
> > be more config files.
> >
> > Trying a .zip this time so it doesn't get eaten by the list filter.
> >
> > //Ma <<Config_editors.zip>> gnus
> >
> >
> >
> >
>

Attachment

pgadmin-hackers by date:

Previous
From: "Dave Page"
Date:
Subject: Re: Patch: Config editors
Next
From: Andreas Pflug
Date:
Subject: Re: bugfix: Encoding of config files