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

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: [FEATURE] Add schema option to all relevant objects
Дата
Msg-id 1310150812.2902.43.camel@laptop
обсуждение исходный текст
Ответ на Re: [FEATURE] Add schema option to all relevant objects  (Thom Brown <thom@linux.com>)
Ответы Re: [FEATURE] Add schema option to all relevant objects
Re: [FEATURE] Add schema option to all relevant objects
Список pgadmin-hackers
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.

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?


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

Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Thom Brown
Дата:
Сообщение: Re: Overhaul of type attributes modification
Следующее
От: Thom Brown
Дата:
Сообщение: Re: [FEATURE] Add schema option to all relevant objects