Fix for issue #3740

[tcchhabra]

No description provided.

[maxceem]

@tcchhabra the proposed solution has several issues.

I've created 2 projects, one with name "XYZ" and another one with name "ABC/D". Note, that we can search using a normal string abc/d or string in quotes "abc/d".

How it works before your changes

• ✅ xyz:

  

• ✅ "xyz":

  

• ❌ abc/d:

  

• ✅ "abc/d":

  

How it works after your changes

• ✅ xyz:

  

• ❌ "xyz":

  

• ✅ abc/d:

  

• ❌ "abc/d":

  

Sum up

1. Now the case when we are searching using quotes is broken. To not broke this case, I think we only have to apply the escape method in case of wildcards search, here https://github.com/topcoder-platform/tc-project-service/blob/dev/src/routes/projects/list.js#L387 or something like this. I guess we shouldn't apply it in all the cases.

2. We already have a method to encode query to ES, can we reuse it instead of creating a new method?

   

3. To validate that fix works properly, please create 2 unit tests:

   • should find a project by keyword with a special symbol in the name - find project abc/d using keyword=abc/d
   • should find a project by quoted keyword with a special symbol in the name - find project abc/d using keyword="abc/d"

[tcchhabra]

checking

[tcchhabra]

@maxceem I cannot use the other function, I have spent a good chunk of time try to reuse it but in vain so sticking with my function for handling this, which takes care of both the scenarios. Both test cases added.

[maxceem]

Works good.

PS existent method escapeEsKeyword can be fixed by removing extra \\.