Re: [pgAdmin4][patch][FileManager]: RM-2110 - Invalid path error displayed

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][patch][FileManager]: RM-2110 - Invalid path error displayed
Date: 2017-02-03 14:20:14
Message-ID: CA+OCxoxC1Wou1_UWMoS5LzXHSeRfjZrT2yZQ49M2Fyf6bhP9=w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks - applied!

On Fri, Feb 3, 2017 at 12:33 PM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> Please find attached updated patch with the fix of mentioned issue.
>
> Do review it and let us know for comments.
>
> Thanks,
> Neel Patel
>
> On Mon, Jan 30, 2017 at 5:44 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>> Hi
>>
>> On Fri, Jan 27, 2017 at 10:30 AM, Neel Patel
>> <neel(dot)patel(at)enterprisedb(dot)com> wrote:
>> > Hi,
>> >
>> > Please find attached patch file for the fix of RM#2110.
>> >
>> > When user try to save the file through fileManager then it gives error
>> > saying "Invalid Path" while getting folder details from python server.
>> > This
>> > issue is only reproducible with desktop runtime application with windows
>> > OS.
>> >
>> > I have tested with windows, Mac - web and desktop mode.
>> > Do review it and let us know for comments.
>>
>> I can still reproduce this, on Mac, running in Desktop mode (but in a
>> browser). The problem I see is not that it's failing to set the root
>> directory properly (as you've fixed in the patch), but that it's
>> trying to navigate to a previously-saved directory that no longer
>> exists - in my case, /Users/dpage/foo. Per the bug report, it should
>> travel back up the tree and navigate to the first directory in the
>> path that does still exist (so in my case, it would go to
>> /Users/dpage).
>>
>> --
>> 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

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-02-03 14:23:45 Re: [pgAdmin4][Patch]: RM 2089 - PARALLEL SAFE support for functions missing
Previous Message Dave Page 2017-02-03 14:19:25 pgAdmin 4 commit: Handle saved directories that no longer exist gracefu