fix: allow filtering on date on smart field (#1080)

* fix: allow filtering on date on smart field

* docs: either nested or virtual

* test: add unit test

---------

Co-authored-by: Nicolas Moreau <[email protected]>

Author: realSpok

src/services/filters-parser.js

@@ -96,7 +96,11 @@ function FiltersParser(model, timezone, options) {
     }
 
     const fieldPath = subfieldName ? `${fieldName}.${subfieldName}` : fieldName;
-    const fieldType = utils.getNestedFieldType(model.schema, fieldPath);
+
+    // NOTICE: either nested or virtual, not both
+    const fieldType = field.isVirtual
+      ? field.type
+      : utils.getNestedFieldType(model.schema, fieldPath);
 
     if (!fieldType) return (val) => val;
 

test/tests/services/filters-parser.test.js

@@ -346,6 +346,23 @@ describe('service > filters-parser', () => {
         expect(fakeParser).toHaveBeenCalledTimes(1);
         expect(fakeParser).toHaveBeenCalledWith('myValue');
 
+        spy.mockRestore();
+      });
+    });
+    describe('with a virtual field', () => {
+      it('should return the field type', async () => {
+        expect.hasAssertions();
+        const fakeParser = jest.fn().mockReturnValue('parsedValue');
+        const spy = jest.spyOn(defaultParser, 'getParserForType').mockReturnValue(fakeParser);
+        const type = Symbol('type');
+        jest.spyOn(Interface.SchemaUtils, 'getField').mockReturnValue({ isVirtual: true, type });
+
+
+        await defaultParser.getParserForField('mySmartField');
+
+        expect(defaultParser.getParserForType).toHaveBeenCalledTimes(1);
+        expect(defaultParser.getParserForType).toHaveBeenCalledWith(type);
+
         spy.mockRestore();
       });
     });