Merge pull request #577 from psteinroe/fix/nested-objs

fix: has paths on nested objects

Author: psteinroe

.changeset/eighty-turtles-remain.md

@@ -0,0 +1,5 @@
+---
+"@supabase-cache-helpers/postgrest-core": patch
+---
+
+fix: hasPaths for nested objects

packages/postgrest-core/src/postgrest-filter.ts

@@ -113,14 +113,14 @@ export class PostgrestFilter<Result extends Record<string, unknown>> {
   }
 
   private hasPathRecursive(obj: unknown, path: string): boolean {
+    // obj is valid if v is null, because the foreign key relation can be null
+    if (obj === null) return true;
+
     // normalise json operators "->" and "->>" to "."
     const pathElements = path.replace(/->>|->/g, '.').split('.');
 
     // we are at the deepest level
     if (pathElements.length === 1) {
-      // obj is valid if v is null, because the foreign key relation can be null
-      if (obj === null) return true;
-
       // else check if the path exists
       return typeof get(obj, pathElements[0]) !== 'undefined';
     }

packages/postgrest-core/tests/postgrest-filter.spec.ts

@@ -520,6 +520,20 @@ describe('PostgrestFilter', () => {
       ).toEqual(true);
     });
 
+    it('with null value for nested object', () => {
+      expect(
+        new PostgrestFilter({
+          filters: [],
+          paths: [
+            {
+              path: 'null_value.value.nested',
+              declaration: 'null_value.value.nested',
+            },
+          ],
+        }).hasPaths(MOCK),
+      ).toEqual(true);
+    });
+
     it('with empty array', () => {
       expect(
         new PostgrestFilter({