Re: Issue 598

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: Issue 598
Дата
Msg-id CA+OCxowVp7OAjmiY=x_w_tj_rNNiFy8ecc6mqYaWL+BSoWUOfw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Issue 598  (Dieter Van de Walle <Dieter.VandeWalle@ebit.be>)
Список pgadmin-hackers
Hi

On Wed, Sep 4, 2013 at 2:59 PM, Dieter Van de Walle
<Dieter.VandeWalle@ebit.be> wrote:
> I have attached a patch to the issue, see: https://redmine.postgresql.org/issues/598 .
> Not sure about the preferred way to submit patches? I generated a patch file using 'git format-patch' .

The format is fine. Why does the patch remove the check to stop you
dropping a system table though? That is unrelated and seems
ill-advised.

> It works, but I'd love some feedback on this...
> I'm presuming the selected object will always be a collection when doing truncate, however not sure if this is
true...
> Would be nice to be enlightened on this ...

No, I doubt it will always be a collection. Have a look at the code to
drop objects - Iirc, that does already handle multiple objects and
should get this right. However, if memory serves it does it in a
different way; you select multiple objects in the listview (NOT the
treeview) and drop them all at once. That gives more flexibility than
dropping the entire collection of course.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


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

Предыдущее
От: Dieter Van de Walle
Дата:
Сообщение: Re: Issue 598
Следующее
От: Rūdolfs Mazurs
Дата:
Сообщение: