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 |
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) |