Re: [FEATURE] Add schema option to all relevant objects

On Fri, 2011-07-08 at 20:53 +0100, Thom Brown wrote:
> On 8 July 2011 20:52, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
> > On Fri, 2011-07-08 at 20:46 +0200, Guillaume Lelarge wrote:
> >> On Thu, 2011-07-07 at 23:24 +0100, Thom Brown wrote:
> >> > On 7 July 2011 23:20, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
> >> > > On Wed, 2011-07-06 at 11:59 +0100, Thom Brown wrote:
> >> > >> add_schemas_to_all_items_filtered.patch - same patch as before, but
> >> > >> with the above patch contents removed or adjusted to assume the fixes
> >> > >> have been applied.
> >> > >>
> >> > >
> >> > > Not yet done. I'll try to take care of this tomorrow.
> >> >
> >> > Awesomes. Thanks Guillaume. Please ensure I haven't sneaked anything
> >> > in I shouldn't have, and that I don't break anything for previous
> >> > versions.
> >> >
> >>
> >> OK, just a few things that I fixed on your patch:
> >> * pgadmin/schema/pgIndex.cpp contained reversed change of one of your
> >> previous patches, I don't keep that.
> >> * all extension changes are wrong according to me because they aren't
> >> schema objects, but database objects. I don't keep them.
> >> * you added possibility to change a type, and a sequence, but forgot to
> >> add the code to detect the changes in CheckChange. I fixed that
> >> * in dlgForeignTable.cpp, you declare a direction variable, but don't
> >> use it. Fixed that too.
> >> * still in dlgForeignTable.cpp, you enable the schema widget only when
> >> the user is connected in 9.1 but this guy can have this window only
> >> when he's on 9.1. Fixed that.
> >>
> >
> > I forgot one thing:
> >
> > * extract the SQL changes to commit them now in 1.14 and master.
> >
> >> And that should be all. The fixed patch is attached. There is one
> >> remaining issue: how to refresh the object's parent node in the new
> >> schema?
>
> Yes, I think I need to be more disciplined in what I include in
> patches. I got a bit carried away with this one in particular.
>

It's not easy to do, I understand you got carried away. Happened
sometimes to me too. Still happens actually.

--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com