Re: Patch for RM1911 Direct file navigation [pgAdmin4] [Feature]

From: Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Patch for RM1911 Direct file navigation [pgAdmin4] [Feature]
Date: 2017-01-14 14:27:09
Message-ID: CAFiP3vxvxV7PpjEiepJHW1Rgs0TpD_eDM0WSjv3eLdKc8vJ5nA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

Pls updated patch for RM1911.

1. This includes fix for issue index out of range when user enters path of
folder without trailing slash (showed by Dave).
2. To make this functionality compatible with save last used directory
feature.

--
*Harshal Dhumal*
*Software Engineer*

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

On Mon, Jan 9, 2017 at 12:40 PM, Harshal Dhumal <
harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:

> Hi Dave,
>
>
> Please find attached rebased patch.
>
>
> --
> *Harshal Dhumal*
> *Software Engineer*
>
> EnterpriseDB India: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On Sun, Jan 8, 2017 at 4:07 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> Can you rebase this please? It no longer applies :-(
>>
>>
>> On Wednesday, December 28, 2016, Harshal Dhumal <
>> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Dave,
>>> Here is updated (V4) patch.
>>>
>>> Changes: 1] Now can enter both type of slashes ( / and \ ) and all will
>>> get replaced with the platform standard.
>>> 2] Added unicode support.
>>>
>>> --
>>> *Harshal Dhumal*
>>> *Software Engineer*
>>>
>>> EnterpriseDB India: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Mon, Dec 19, 2016 at 5:16 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>> Hi
>>>>
>>>> On Fri, Dec 16, 2016 at 6:46 PM, Harshal Dhumal
>>>> <harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>>>> > Hi Dave,
>>>> >
>>>> > Please find updated patch below for direct file navigation.
>>>> > I have covered all of above mentioned case.
>>>>
>>>> Still not there I'm afraid:
>>>>
>>>> - On Mac, if I type \Users\dpage, it changes it to /\Users\dpage and
>>>> then tells me it doesn't exist. Per the RM, either forward or back
>>>> slashes should be acceptable ("The path should accept either / or \ as
>>>> separators. Upon successful navigation to the path (after pressing
>>>> Return), the slashes should be replaced with the platform standard if
>>>> needed.")
>>>>
>>>> - Unicode handling seems to be completely broken - see the attached
>>>> screenshot.
>>>>
>>>> I haven't tested on Windows yet, and only in Desktop mode on Mac.
>>>> Please test on Windows and Mac or Linux with both Python 2 and 3, in
>>>> both Server and Desktop modes to ensure that the behaviour meets the
>>>> requirements of the ticket with Unicode and non-Unicode paths and
>>>> files before resubmitting.
>>>>
>>>> Thanks.
>>>>
>>>> --
>>>> Dave Page
>>>> Blog: http://pgsnake.blogspot.com
>>>> Twitter: @pgsnake
>>>>
>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>
>>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>>
>
>

Attachment Content-Type Size
RM1911_V6_14_Jan.patch application/octet-stream 61.5 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Jonas Thelemann 2017-01-15 04:00:50 .gitignore Improvement Suggestion
Previous Message Murtuza Zabuawala 2017-01-13 12:29:23 PATCH: To fix the issue in Variable control (pgAdmin4)