[*] Record List - Fix record listing on table which reference table with id not string (#272)

Author: VincentMolinie

CHANGELOG.md

@@ -12,6 +12,7 @@
 ### Fixed
 - Technical - Exclude from pre-commit-hook linter files that are not committed.
 - Security - Patch vulnerabilities upgrading the `mixin-deep` dependency.
+- Record List - Fix record listing on table which reference table with id not string.
 
 ## RELEASE 5.4.1 - 2019-12-11
 ### Changed

src/services/filters-parser.js

@@ -22,10 +22,6 @@ function FiltersParser(model, timezone, options) {
     if (ObjectId.isValid(value) && ObjectId(value).toString() === value) {
       return ObjectId(value);
     }
-    if (_.isArray(value)) {
-      return value
-        .map((arrayValue) => (ObjectId.isValid(arrayValue) ? ObjectId(arrayValue) : arrayValue));
-    }
     return value;
   };
   const parseArray = (value) => ({ $size: value });
@@ -98,7 +94,14 @@ function FiltersParser(model, timezone, options) {
 
     if (!field) return (val) => val;
 
-    return this.getParserForType(field.type);
+    const parse = this.getParserForType(field.type);
+
+    return (value) => {
+      if (value && _.isArray(value)) {
+        return value.map(parse);
+      }
+      return parse(value);
+    };
   };
 
   this.formatAggregatorOperator = (aggregatorOperator) => {