Fix search tests.

packages/server/src/api/routes/tests/search.spec.ts

@@ -3554,27 +3554,30 @@ if (descriptions.length) {
                   }).toContainExactly([row])
                 })
 
-                it("can filter by the related _id", async () => {
+                isInternal &&
-                  await expectSearch({
+                  describe("search by _id for relations", () => {
-                    query: {
+                    it("can filter by the related _id", async () => {
-                      equal: { "rel._id": row.rel[0]._id },
+                      await expectSearch({
-                    },
+                        query: {
-                  }).toContainExactly([row])
+                          equal: { "rel._id": row.rel[0]._id },
+                        },
+                      }).toContainExactly([row])
 
-                  await expectSearch({
+                      await expectSearch({
-                    query: {
+                        query: {
-                      equal: { "rel._id": row.rel[1]._id },
+                          equal: { "rel._id": row.rel[1]._id },
-                    },
+                        },
-                  }).toContainExactly([row])
+                      }).toContainExactly([row])
-                })
+                    })
 
-                it("can filter by the related _id and find nothing", async () => {
+                    it("can filter by the related _id and find nothing", async () => {
-                  await expectSearch({
+                      await expectSearch({
-                    query: {
+                        query: {
-                      equal: { "rel._id": "rel_none" },
+                          equal: { "rel._id": "rel_none" },
-                    },
+                        },
-                  }).toFindNothing()
+                      }).toFindNothing()
-                })
+                    })
+                  })
               })
 
             !isInternal &&

packages/server/src/sdk/app/rows/queryUtils.ts

@@ -7,6 +7,7 @@ import {
 } from "@budibase/types"
 import { cloneDeep } from "lodash/fp"
 import sdk from "../../../sdk"
+import { isInternal } from "../tables/utils"
 
 export const removeInvalidFilters = (
   filters: SearchFilters,
@@ -69,8 +70,11 @@ export const getQueryableFields = async (
     fromTables: string[],
     opts?: { noRelationships?: boolean }
   ): Promise<string[]> => {
-    // Querying by _id is always allowed, even if it's never part of the schema
+    const result = []
-    const result = ["_id"]
+    if (isInternal({ table })) {
+      result.push("_id")
+    }
+
     for (const field of Object.keys(table.schema).filter(
       f => allowedFields.includes(f) && table.schema[f].visible !== false
     )) {
@@ -114,7 +118,8 @@ export const getQueryableFields = async (
     return result
   }
 
-  const result = []
+  // Querying by _id is always allowed, even if it's never part of the schema
+  const result = ["_id"]
 
   if (fields == null) {
     fields = Object.keys(table.schema)