Re: [pgAdmin4][Patch] - RM #3309 - Backup does not work with --format=d

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: [pgAdmin4][Patch] - RM #3309 - Backup does not work with --format=d
Дата
Msg-id CAFOhELdWjX3czSKSPEq_8ynq8e-Sx+wPhTtCESLvq-iC0OoFKg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch] - RM #3309 - Backup does not work with --format=d  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin4][Patch] - RM #3309 - Backup does not work with --format=d  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Дерево обсуждения


On Mon, 25 Jun 2018, 19:47 Dave Page, <dpage@pgadmin.org> wrote:
Hi

On Mon, Jun 25, 2018 at 1:23 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch for RM #3309 - Backup does not work with --format=d.

Fixes in the patch :

1. Format = directory fix for Backup and Restore utilities
2. Test cases for format = directory
2. File manager fix - The selected directory was not properly populated in the text box

The patch disables the file picker button if you select Directory format. This seems like poor behaviour; I think we need to modify the dialogue to allow a directory to be selected.
 
In the directory format, which ever the folder name given, that will be created by pg_dump, so  I have disabled it.
3. Refactoring of File manager JS file

That's somewhat more readable. Can we add tests? 
Will look into it.

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

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

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch] - RM #3309 - Backup does not work with --format=d
Следующее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch] - RM #3309 - Backup does not work with --format=d