Re: Discussion - Search Objects

Yeah, I know. All of the dialogs have a "binding" to some pgObject, because
all of them modifies something. When I started working I had no clear
picture about the class hierarchy and I thought this would be an easier
entry point. I can change the prefix if this is wished.

btw: You could help me with the factory class. Am I starting the dialog
correctly?

wxWindow *searchObjectFactory::StartDialog(frmMain *form, pgObject *obj)
{
frmSearchObject *so = new frmSearchObject(form, (pgDatabase *) obj);
so->Show();

return 0;
}

Thanks.

2011/6/15 Guillaume Lelarge <guillaume(at)lelarge(dot)info>

> On Wed, 2011-06-15 at 08:08 +0200, Guillaume Lelarge wrote:
> > On Wed, 2011-06-15 at 01:03 +0200, Jasmin Dizdarevic wrote:
> > > Hi,
> > >
> > >
> > > please take another look on it. The UI should work correctly now.
> > >
> > >
> > > Linux make files should also be correct now.
> > >
> >
> > It works much better now. You still have one issue with primary key (and
> > I guess, unique constraint and exclusive constraints too). You show them
> > as constraints and indexes, whereas pgAdmin only shows them as
> > constraints. You shouldn't display indexes for constraints with implicit
> > indexes.
> >
> > And I didn't look at the code yet.
> >
>
> Seems mostly good. The only question I have is: why did you choose to
> name it frm* instead of dlg*? it behaves as a dialog, not as a form.
>
>
> --
> Guillaume
> http://blog.guillaume.lelarge.info
> http://www.dalibo.com
>
>