Re: PATCH: PGADMIN 4 - FTS templates node

Hi,

PFA the patch for fts template after *replacing collectionNodeModule with
SchemaChildModule* in __init__.py

Apart from that, there was an error while updating the name of fts template
which I have resolved in the same.

Please do review it and let me know in case of any changes.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, Mar 1, 2016 at 4:56 PM, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>
wrote:

> Hi Dave,
>
> For your first comment,
>
> "When creating a new template, errors in the dialogue are not cleared -
> e.g. the "Name must not be empty" doesn't clear, and save is not enabled,
> even when a name is entered."
>
> This issue should not be handled only at FTS template level but the top
> level as it occurs in all nodes.
> I have created a bug node in kanban-chi regarding the same.
> We will take this issue as per priority and send the patch for the same
> later.
>
>
>
> Regards,
> Sanket Mehta
> Sr Software engineer
> Enterprisedb
>
> On Tue, Mar 1, 2016 at 4:34 PM, Sanket Mehta <
> sanket(dot)mehta(at)enterprisedb(dot)com> wrote:
>
>> Hi Dave,
>>
>> All comments are resolved.
>> PFA the revised patch.
>>
>> Regards,
>> Sanket Mehta
>> Sr Software engineer
>> Enterprisedb
>>
>> On Thu, Feb 25, 2016 at 7:37 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Tue, Feb 23, 2016 at 1:44 PM, Sanket Mehta <
>>> sanket(dot)mehta(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> PFA the revised patch for fts template node which includes dependency
>>>> and dependent route handling code.
>>>> Please do review it and if found proper please do commit it.
>>>>
>>>
>>> - When creating a new template, errors in the dialogue are not cleared -
>>> e.g. the "Name must not be empty" doesn't clear, and save is not enabled,
>>> even when a name is entered.
>>>
>>> - "Fts Templates" should be "FTS Templates" on the treeview.
>>>
>>> - The "Init" field should be labelled "Init Function"
>>>
>>> - The "Lexize" field should be labelled "Lexize Function"
>>>
>>> - Selecting a schema should be optional, with the default being the
>>> current schema.
>>>
>>> - 4 character indents are not consistently used in SQL templates
>>>
>>> Thanks.
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>
>