From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Anthony Emengo <aemengo(at)pivotal(dot)io> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][RM#3324] - Windows user unable to expand "External Tables" navigation item |
Date: | 2018-05-03 14:54:25 |
Message-ID: | CA+OCxoxWDOpHWWcHPvBEoBmpPGH8jVCwqsWmbZDnYYtET1fkBg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, patch applied!
On Wed, May 2, 2018 at 7:25 PM, Anthony Emengo <aemengo(at)pivotal(dot)io> wrote:
> Forgive me but please consider the following patch instead - as it better
> adheres to the python style guide.
>
> On Wed, May 2, 2018 at 12:43 PM Anthony Emengo <aemengo(at)pivotal(dot)io> wrote:
>
>> Hi Hackers,
>>
>> Please find the attached patch to fix the RM #3324 : user cannot load
>> “External Tables” on the navigation pane.
>>
>> The issue was that ultimately paths were not being cleaned after being
>> munged from the “template” input which would result in template_paths that
>> resembled: path//to//dir.sql. These would work on unix based systems but
>> lead to complication on Windows.
>>
>> The following patch resolves this issue, and also included a small file
>> refactor to better convey developer intent.
>>
>> Thanks,
>> Anthony
>>
>>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-05-03 15:05:31 | Re: [pgAdmin4][patch] - Fix CheckForViewDataTest |
Previous Message | Dave Page | 2018-05-03 14:54:19 | pgAdmin 4 commit: Fix the template loader to work reliably under Window |