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@lelarge.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@lelarge.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