Re: RM4069 - Append the file suffix to filenames when needed in theFile Create dialogue.

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: RM4069 - Append the file suffix to filenames when needed in theFile Create dialogue.
Дата
Msg-id CAFOhELeMsm0PQH=mobwaX=bLbxsWSpO3ZjQVCgHokkWgC-atMA@mail.gmail.com
обсуждение исходный текст
Ответ на RM4069 - Append the file suffix to filenames when needed in the FileCreate dialogue.  (Dave Page <dpage@pgadmin.org>)
Ответы Re: RM4069 - Append the file suffix to filenames when needed in theFile Create dialogue.  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Hi Dave,

On Thu, Mar 14, 2019 at 5:01 PM Dave Page <dpage@pgadmin.org> wrote:
Here's a small patch for $SUBJECT. 

I'd normally commit something like this directly but wanted some thoughts on the fact that it introduces a third method for figuring out what path separator to use to the file manager code:

1) We get it from the config file (which is actually a template)
2) We look at what's already in an existing path

and now:

3) We use navigator.platform 

I could use method 1 here, but it would add quite a bit more code (unless there's a simple way of importing the config object from utility.js that is beyond my limited JS-fu).

Method 2 can't be used as there may not be a path to look at.

Any thoughts on this issue, and tidying up 1 & 2 in general?
Importing config object from the utility.js would be easy if we have written the code in such a way.
Here the main purpose is to get the path separator on the basis of the OS and that is easily available through JS, so I would prefer the way you have used it.
We can also write the code which will call the backend and load the config which will be used in the File Manager module if that config will be used extensively but that is not the case.

Thanks,
Khushboo

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

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

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

Предыдущее
От: Khushboo Vashi
Дата:
Сообщение: Re: [pgAdmin4][Patch] - RM #4019 - Update all JS and Pythondependencies to the latest releases
Следующее
От: Dave Page
Дата:
Сообщение: Re: RM4069 - Append the file suffix to filenames when needed in theFile Create dialogue.