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

Re: Patch: Query favourites

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: "Magnus Hagander" <mha(at)sollentuna(dot)net>
Cc: <pgadmin-hackers(at)postgresql(dot)org>,"Andreas Pflug" <pgadmin(at)pse-consulting(dot)de>
Subject: Re: Patch: Query favourites
Date: 2006-02-08 15:39:00
Message-ID: E7F85A1B5FF8D44C8A1AF6885BC9A0E40103DFA7@ratbert.vale-housing.co.uk (view raw or flat)
Thread:
Lists: pgadmin-hackers
Patch applied with the following changes:

- Moved all deps to $PGASRC/../pgadmin3-deps/ on Windows.
- Replaced the wxTreeCtrl with a ctlTree (which draws lines properly on
GTK), and added the appropriate XRC resource handlers.

There is a slightly odd effect with the text box on dlgAddFavourite and
the wxTextEntry dialogue on Windows in which the cursor keeps returning
to immediately before char 4 (even if the textbox is actually empty),
but it does actually work as expected.

Andreas; can you look at that please? I'm somewhat baffled...

Regards, Dave

> -----Original Message-----
> From: Magnus Hagander [mailto:mha(at)sollentuna(dot)net] 
> Sent: 07 February 2006 20:05
> To: Dave Page
> Cc: pgadmin-hackers(at)postgresql(dot)org
> Subject: RE: [pgadmin-hackers] Patch: Query favourites
> 
> > > > > As for the libxml2/msxml - I think going with *just*
> > > libxml2 is the
> > > > > way to go. The APIs are so completely different that it
> > > > would be two
> > > > > completely different implementations. And AFAIK, there are
> > > > no problems
> > > > > with libxml on Win32 in general.
> > > 
> > > Here's a version that uses libxml2 natively, including 
> updated unix 
> > > build scripts. No longer any dependency on wxxml2.
> > 
> > Finally got around to looking at this :-). Couple of things 
> I noticed:
> > 
> > - If the ctlSQLBox is empty, the 'Add Favourites' option is 
> > still enabled, but does nothing. SVN trunk calls updateMenu 
> > (or whatever it's
> > called) on every change now, so this should be a 30 second fix.
> 
> Indeed. Tried that before, had the problem that it didn't activate,
> couldn't figure out why. Seems you've fixed that one, so yes, 
> 30 second
> fix.
> 
> 
> > - On Linux, with no favourites at all, the root node in the 
> > Manage Favourites tree has no icon. Add a favourite, and the 
> > root node gains a folder icon, but the favourites themselves 
> > remain without an icon. Works properly on Windows, haven't 
> tried Mac.
> 
> Ah. Turns out there is a default image for a tree node when running on
> Windows, but not when running on Linux. Cross-platform? Hmm... Fixed.
> 
> 
> > - You can only create one new folder on the manage dialogue. 
> > To create mode, you need to close/re-open the dialogue.
> 
> Major logic bomb. Actually, could create more than one subfolder - as
> long as the new one was added to a *different* one. It overwrote the
> treeid of the parent folder instead of the child...
> Along this, also fixed a bug with empty folders.
> 
> 
> > - You can create folders with duplicate names in the same 
> > folder. Is this a bug? Dunno...
> 
> Dunno either, but it seems stupid. So now it refuses it.
> 
> 
> > - The buttons on the manage dialogue should left-align with 
> > the side of the tree control.
> 
> Yeah. Fixed.
> 
> 
> > - There is no doc update. An update to the build instructions 
> > on the website would also be nice :-)
> 
> Hmm. It was included in the first one, missed it in the 
> libxml version.
> Included again in this one.
> 
> Build instructions attached as a separate patch. Don't have an env to
> test it in, but it should be simple 'nuf.
> 
> 
> > On a related note, this introduces dependencies on libxml2 
> and iconv.
> > These are both available from www.xmlsoft.org, precompiled 
> > for Windows, and are both on most unixes already, however, on 
> > Windows there is no standard place for them to live. There 
> > are two sensible options I can see for these, and wxWidgets:
> > 
> > C:\wxWidgets-2.6\
> > C:\libxml2\
> > C:\iconv\
> > 
> > Or
> > 
> > C:\pgadmin-prereqs (or some other name)
> >       \wxWidgets-2.6\
> >       \libxml2\
> >       \iconv\
> > 
> > Thoughts/preferences?
> 
> I like the latter. Even better if it could be made a relative 
> directory
> wrt to the pgadmin directory. Say I have c:\src, then I could have
> c:\src\pgadmin3 and c:\src\pgamdin-preqreqs. Or so.
> 
> > Ooops, forgot one - I get:
> > 
> > precomp.cpp
> > c:\documents and settings\dpage\my
> > documents\svn\pgadmin3\src\include\favourites.h(43) : warning C4284:
> > return type for 
> > 'queryFavouriteArray::reverse_iterator::operator ->' is 
> > 'class queryFavouriteItem ** ' (ie; not a UDT or reference to a UDT.
> > Wil
> > l produce errors if applied using infix notation) 
> > c:\documents and settings\dpage\my
> > documents\svn\pgadmin3\src\include\favourites.h(43) : warning C4284:
> > return type for 
> 'queryFavouriteArray::const_reverse_iterator::operator
> > ->' is 'class queryFavouriteItem *const * ' (ie; not a UDT or 
> > reference
> > to
> >  a UDT.  Will produce errors if applied using infix notation)
> > 
> > Compiling in VC++ 6.0
> 
> Doesn't show up in VC7 or GCC. Google told me to use
> WX_DEFINE_ARRAY_PTR() instead of WX_DEFINE_ARRAY(). Can't test it
> though. (Google also says the warning is harmless, but if 
> there is a way
> to make it go away it should be done)
> 
> 
> Unix build system patch unchanged and thus not included.
> 
> //Magnus
> 

Responses

pgadmin-hackers by date

Next:From: svnDate: 2006-02-08 15:43:52
Subject: SVN Commit by dpage: r4993 - trunk/pgadmin3
Previous:From: svnDate: 2006-02-08 15:34:59
Subject: SVN Commit by dpage: r4992 - trunk/www/download

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