Re: Fixed help information issue for "Server Instrumentation" guru hint

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: Fixed help information issue for "Server Instrumentation" guru hint
Дата
Msg-id CANxoLDc7D1EFs3DQJfWp9Q8iRoioRhFfAXmYCNbzeBP8OP4aJQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Fixed help information issue for "Server Instrumentation" guru hint  (Dave Page <dpage@pgadmin.org>)
Ответы Re: Fixed help information issue for "Server Instrumentation" guru hint  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers



On Mon, Aug 5, 2013 at 6:50 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Aug 5, 2013 at 7:54 AM, Akshay Joshi
<akshay.joshi@enterprisedb.com> wrote:
> Hi Dave
>
> I have fixed on minor issue where there is no help information (.rst)
> available for "Server Instrumentation" guru hint. Bug is reproducible when
> user click on "Help" button on guru hint dialog for "Server
> Instrumentation".
>
> I have fixed it by disabling the "Help" button for "Server Instrumentation"
> guru hint. Attached is the patch file. Please review it and if it look good
> to you then can you please commit it.

This doesn't seem like the right approach. We should remove the
invalid helpfile names from the hintArray (set them to NULL/0 I
guess), and then if necessary, modify the dialogue to enable or
disable the button based on that value.

   I have tried by replacing the invalid helpfile names with "wxEmptyString" from the hintArray. It works. With this implementation no action will be perform when user click on "Help" button. Is this behavior acceptable or do I need to disable the "Help" button?   

That will be more future-proof, and follows the existing design much
more closely.

Thanks.

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

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



--
Akshay Joshi
Senior Software Engineer 
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
Phone: +91 20-3058-9522
Mobile: +91 976-788-8246

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: Fixed "Slony" related issues
Следующее
От: Dave Page
Дата:
Сообщение: Re: Fixed help information issue for "Server Instrumentation" guru hint