Make skill matching case-insensitive in getRoleBySkills

Author: mbaghel

src/services/TeamService.js

@@ -13,7 +13,7 @@ const errors = require('../common/errors')
 const JobService = require('./JobService')
 const ResourceBookingService = require('./ResourceBookingService')
 const HttpStatus = require('http-status-codes')
-const { Op } = require('sequelize')
+const { Op, where, fn, col } = require('sequelize')
 const models = require('../models')
 const stopWords = require('../../data/stopWords.json')
 const { getAuditM2Muser } = require('../common/helper')
@@ -799,17 +799,30 @@ roleSearchRequest.schema = Joi.object()
  * @returns {Role} the best matching Role
  */
 async function getRoleBySkills (skills) {
+  // Case-insensitive search for roles matching any of the given skills
   const lowerCaseSkills = skills.map(skill => skill.toLowerCase())
-  // find all roles which includes any of the given skills
   const queryCriteria = {
-    where: { listOfSkills: { [Op.overlap]: lowerCaseSkills } },
+    where: where(
+      fn(
+        'string_to_array', 
+        fn(
+          'lower', 
+          fn(
+            'array_to_string', 
+            col('list_of_skills'),
+            ','
+          )
+        ),
+        ','
+      ),
+      {[Op.overlap]: lowerCaseSkills }),
     raw: true
   }
   const roles = await Role.findAll(queryCriteria)
   if (roles.length > 0) {
     let result = _.each(roles, role => {
-      // calculate each found roles matching rate
-      role.skillsMatch = _.intersection(role.listOfSkills, lowerCaseSkills).length / skills.length
+      // calculate each found roles matching rate (must again be made case-insensitive)
+      role.skillsMatch = _.intersection(role.listOfSkills.map(skill => skill.toLowerCase()), lowerCaseSkills).length / skills.length
       // each role can have multiple rates, get the maximum of global rates
       role.maxGlobal = _.maxBy(role.rates, 'global').global
     })