Merge pull request #763 from techmatters/gian_CHI-3004-reg-1.18-rc-merge

fix: cast term to number before asserting if is int
techmatters · Oct 15, 2024 · 7bcd357 · 7bcd357
2 parents e9fb9f7 + b6956ed
commit 7bcd357
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/hrm-domain/packages/hrm-search-config/generateElasticsearchQuery.ts b/hrm-domain/packages/hrm-search-config/generateElasticsearchQuery.ts
@@ -214,7 +214,7 @@ const generateQueriesFromId = <TDoc extends DocumentType>({
   const queries = terms
     .map(term => {
       // Ignore terms that are not entirely a number, as that breaks term queries against integer fields
-      if (Number.isNaN(Number(term)) || !Number.isInteger(term)) {
+      if (Number.isNaN(Number(term)) || !Number.isInteger(Number(term))) {
         return null;
       }