Re: [pgAdmin4] Patch: Added new Control SqlFieldControl

Поиск
Список
Период
Сортировка
От Ashesh Vashi
Тема Re: [pgAdmin4] Patch: Added new Control SqlFieldControl
Дата
Msg-id CAG7mmoxDrnzqgzN9P44g+d1-s49CWV2ZZvNYxK0q=sCabSpvcw@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4] Patch: Added new Control SqlFieldControl  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
Список pgadmin-hackers
Hi Surinder,

On Tue, Feb 2, 2016 at 1:05 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi,


Please find patch for SqlFieldControl.
Thanks for the patch. 

This control allows the user to write/edit SQL queries.

To use SqlFieldControl, Set control property of field to 'sql-field' in model's schema like:

{
          id: 'definition', label:'{{ _('Definition') }}', cell: 'string', 
          control: 'sql-field'
}
K.


Please extend the SqlFieldContorl from the Backform.TextareaControl instead of Backform.Control.

* Do not need to set events, TextareaControl has handled those events.
* Extend 'getValueFromDom' function in order to get the value from the sql, and set it to the model.
* Don't listen to the 'pg-property-tab-changed' event here. (It is redundant here).
  In your implementation, you're already refreshing lazely using setTimeout function, which should refresh issue of CodeMirror any way.

As per our personal discussion, please update the 'updateInvalid' function for the Control to look for only the 'input' tag with 'name' attribute.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


http://www.linkedin.com/in/asheshvashi

 



Regards,
Surinder Kumar


--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers


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

Предыдущее
От: Sergey Busel
Дата:
Сообщение: Re: pgAdmin III: Muliple SQL tabs
Следующее
От: Neel Patel
Дата:
Сообщение: Re: pgAdmin 4 commit: Update the runtime to properly support Windows.