Re: [pgadmin-hackers] patch for RM2243 and RM2244 [pgAdmin4]

Поиск
Список
Период
Сортировка
От Harshal Dhumal
Тема Re: [pgadmin-hackers] patch for RM2243 and RM2244 [pgAdmin4]
Дата
Msg-id CAFiP3vzxzVSiZOuLNsrzTzKV-phV2yNqQv9u3A4FAd8fiOF1WQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgadmin-hackers] patch for RM2243 and RM2244 [pgAdmin4]  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgadmin-hackers] patch for RM2243 and RM2244 [pgAdmin4]  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Hi,

-- 
Harshal Dhumal
Software Engineer

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

On Fri, Mar 17, 2017 at 4:04 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Fri, Mar 17, 2017 at 7:02 AM, Harshal Dhumal
<harshal.dhumal@enterprisedb.com> wrote:
> Hi,
>
> Here is patch for RM2243 and RM2244
>
> Changes: 1. Pickup correct encoding in qtLiteral to decode.
> 2. While releasing database connection only decode database name if it's not
> unicode.

This fixes 2243, but 2244 is an SQL error that occurs when querying
tables with no columns in the Query Tool. Although Fahar found it
whilst testing with an Arabic name, it's otherwise unrelated.

Patch committed to fix 2243 only.


Yes right, I missed that. I thought 2244 was also related to encoding.

Please find attached patch for RM2244.

 
Thanks.

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

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

Вложения

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

Предыдущее
От: Jonas Thelemann
Дата:
Сообщение: [pgadmin-hackers] Translations Fix #1
Следующее
От: Atira Odhner
Дата:
Сообщение: Re: [pgadmin-hackers] Patch submissions