Thread: PATCH: FTS configuration node

PATCH: FTS configuration node

From
Sanket Mehta
Date:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb
Attachment

Re: PATCH: FTS configuration node

From
Harshal Dhumal
Date:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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


Re: PATCH: FTS configuration node

From
Sanket Mehta
Date:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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



Attachment

Re: PATCH: FTS configuration node

From
Sanket Mehta
Date:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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




Attachment

Re: PATCH: FTS configuration node

From
Akshay Joshi
Date:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
  • Unable to click on down arrow for token select control. 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
  • Tree node is not getting refreshed on name change.
  • Unable to rename FTS Configuration when name contains any capital letter.
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
  • "URL not found" when click on SQL help icon from the properties dialog.
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

Re: PATCH: FTS configuration node

From
Sanket Mehta
Date:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

Attachment

Re: PATCH: FTS configuration node

From
Harshal Dhumal
Date:
Hi Sanket,

Please find my review comments below:

1. In create mode it generates wrong sql.

CREATE TEXT SEARCH CONFIGURATION test.asdf (
    COPY=
);

steps to reproduce:
    a] Fill up any necessary fields in general tab.
    b] On definition tab set Copy config filed and navigate to SQL tab.
    c] Come back again to definition tab clear Copy config field and set Parser field and again navigate to SQL tab

2. Create new FTS configuration with parser throws below error.

provide atleast copy config or parser.

(this is because you have used key name for parser in js as "prsname" and in python you have applied validations on key "parser")

3. When we clear dictionary for any token using cross (x) button in select2; JavaScript code for selecct2 fails. (see attached screenshot)






-- 
Harshal Dhumal
Software Engineer 




On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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


Attachment

Re: PATCH: FTS configuration node

From
Sanket Mehta
Date:
Hi,

PFA the revised patch.
Response is lnline.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, May 17, 2016 at 12:54 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Sanket,

Please find my review comments below:

1. In create mode it generates wrong sql.

CREATE TEXT SEARCH CONFIGURATION test.asdf (
    COPY=
);

steps to reproduce:
    a] Fill up any necessary fields in general tab.
    b] On definition tab set Copy config filed and navigate to SQL tab.
    c] Come back again to definition tab clear Copy config field and set Parser field and again navigate to SQL tab
Fixed 

2. Create new FTS configuration with parser throws below error.

provide atleast copy config or parser.

(this is because you have used key name for parser in js as "prsname" and in python you have applied validations on key "parser")
Fixed 

3. When we clear dictionary for any token using cross (x) button in select2; JavaScript code for selecct2 fails. (see attached screenshot)
This occurs due to issue in select2 control. Its a generalized issue which will be resolved later.






-- 
Harshal Dhumal
Software Engineer 




On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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



Attachment

Re: PATCH: FTS configuration node

From
Ashesh Vashi
Date:

On Wed, May 18, 2016 at 12:41 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:

Hi,

PFA the revised patch.
Response is lnline.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, May 17, 2016 at 12:54 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Sanket,

Please find my review comments below:

1. In create mode it generates wrong sql.

CREATE TEXT SEARCH CONFIGURATION test.asdf (
    COPY=
);

steps to reproduce:
    a] Fill up any necessary fields in general tab.
    b] On definition tab set Copy config filed and navigate to SQL tab.
    c] Come back again to definition tab clear Copy config field and set Parser field and again navigate to SQL tab
Fixed 

2. Create new FTS configuration with parser throws below error.

provide atleast copy config or parser.

(this is because you have used key name for parser in js as "prsname" and in python you have applied validations on key "parser")
Fixed 

3. When we clear dictionary for any token using cross (x) button in select2; JavaScript code for selecct2 fails. (see attached screenshot)
This occurs due to issue in select2 control. Its a generalized issue which will be resolved later.
When you say - it is common issue, can you please point where else you've seen this behaviour?

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company








-- 
Harshal Dhumal
Software Engineer 




On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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





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


Re: PATCH: FTS configuration node

From
Sanket Mehta
Date:

On Wed, May 18, 2016 at 12:42 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Wed, May 18, 2016 at 12:41 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:

Hi,

PFA the revised patch.
Response is lnline.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, May 17, 2016 at 12:54 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Sanket,

Please find my review comments below:

1. In create mode it generates wrong sql.

CREATE TEXT SEARCH CONFIGURATION test.asdf (
    COPY=
);

steps to reproduce:
    a] Fill up any necessary fields in general tab.
    b] On definition tab set Copy config filed and navigate to SQL tab.
    c] Come back again to definition tab clear Copy config field and set Parser field and again navigate to SQL tab
Fixed 

2. Create new FTS configuration with parser throws below error.

provide atleast copy config or parser.

(this is because you have used key name for parser in js as "prsname" and in python you have applied validations on key "parser")
Fixed 

3. When we clear dictionary for any token using cross (x) button in select2; JavaScript code for selecct2 fails. (see attached screenshot)
This occurs due to issue in select2 control. Its a generalized issue which will be resolved later.
When you say - it is common issue, can you please point where else you've seen this behaviour?
The same behaviour occurs in all nodes where multiselect cell is being used like, exclusion control, debugger etc.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company








-- 
Harshal Dhumal
Software Engineer 




On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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





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



Re: PATCH: FTS configuration node

From
Ashesh Vashi
Date:

On Wed, May 18, 2016 at 12:52 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:

On Wed, May 18, 2016 at 12:42 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Wed, May 18, 2016 at 12:41 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:

Hi,

PFA the revised patch.
Response is lnline.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, May 17, 2016 at 12:54 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Sanket,

Please find my review comments below:

1. In create mode it generates wrong sql.

CREATE TEXT SEARCH CONFIGURATION test.asdf (
    COPY=
);

steps to reproduce:
    a] Fill up any necessary fields in general tab.
    b] On definition tab set Copy config filed and navigate to SQL tab.
    c] Come back again to definition tab clear Copy config field and set Parser field and again navigate to SQL tab
Fixed 

2. Create new FTS configuration with parser throws below error.

provide atleast copy config or parser.

(this is because you have used key name for parser in js as "prsname" and in python you have applied validations on key "parser")
Fixed 

3. When we clear dictionary for any token using cross (x) button in select2; JavaScript code for selecct2 fails. (see attached screenshot)
This occurs due to issue in select2 control. Its a generalized issue which will be resolved later.
When you say - it is common issue, can you please point where else you've seen this behaviour?
The same behaviour occurs in all nodes where multiselect cell is being used like, exclusion control, debugger etc.
We do not have MultiSelectCell.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company








-- 
Harshal Dhumal
Software Engineer 




On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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





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




Re: PATCH: FTS configuration node

From
Akshay Joshi
Date:
Thanks - Committed with some modification and fixes.  

On Wed, May 18, 2016 at 12:57 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Wed, May 18, 2016 at 12:52 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:

On Wed, May 18, 2016 at 12:42 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Wed, May 18, 2016 at 12:41 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:

Hi,

PFA the revised patch.
Response is lnline.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, May 17, 2016 at 12:54 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Sanket,

Please find my review comments below:

1. In create mode it generates wrong sql.

CREATE TEXT SEARCH CONFIGURATION test.asdf (
    COPY=
);

steps to reproduce:
    a] Fill up any necessary fields in general tab.
    b] On definition tab set Copy config filed and navigate to SQL tab.
    c] Come back again to definition tab clear Copy config field and set Parser field and again navigate to SQL tab
Fixed 

2. Create new FTS configuration with parser throws below error.

provide atleast copy config or parser.

(this is because you have used key name for parser in js as "prsname" and in python you have applied validations on key "parser")
Fixed 

3. When we clear dictionary for any token using cross (x) button in select2; JavaScript code for selecct2 fails. (see attached screenshot)
This occurs due to issue in select2 control. Its a generalized issue which will be resolved later.
When you say - it is common issue, can you please point where else you've seen this behaviour?
The same behaviour occurs in all nodes where multiselect cell is being used like, exclusion control, debugger etc.
We do not have MultiSelectCell.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company








-- 
Harshal Dhumal
Software Engineer 




On Mon, May 16, 2016 at 7:37 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

Revised patch is attached with this mail.
My response is inline.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Fri, May 13, 2016 at 6:20 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Sanket 

Below are my review comments:

  • Add button should be disabled in Tokens tab while creating.
Done 
  • Unable to click on down arrow for token select control. 
Done 
  • Title should be change from "Create FTS-dictionaries" to "Create FTS-Configuration".
Done 
  • Tree node is not getting refreshed on name change.
Done 
  • Unable to rename FTS Configuration when name contains any capital letter.
Done 
  • If user tries to add already existing token then respective row in the backgrid should be highlighted.
Done 
  • "URL not found" when click on SQL help icon from the properties dialog
Done 
  • SQL should not be generated when remove and add the same token again. Currently it is creating two sql's one for remove and another is for add.
    • Fix for the above issue is: - Add "keys: ['token']," to TokenModel in your js file.
Done
 

On Fri, May 13, 2016 at 2:58 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi All,

Last patch for FTS configuration does not contain node.ui.js file
Kindly ignore it.

Here is the new revised patch attached with this mail.
Please do review it and let me know if any changes required

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 12, 2016 at 4:38 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA the revised patch.
Please do review it and let me know if anything is not proper.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Thu, May 5, 2016 at 8:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

-- 
Harshal Dhumal
Software Engineer 




On Thu, May 5, 2016 at 8:18 PM, Sanket Mehta <sanket.mehta@enterprisedb.com> wrote:
Hi,

PFA first patch for FTS configuration node.

It depends upon backgrid select2cell multi select control, for which Harshal has sent the patch recently.
Please do apply his patch first and then apply this patch.

Please do review it and let me know if any changes are required.


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb


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






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




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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





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







--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246