feat: Stop SELECT * queries

Author: thrownullexception

jest.config.ts

@@ -114,7 +114,6 @@ const customJestConfig = {
   setupFiles: [
     // "<rootDir>/__tests__/setupDotEnv.ts",
     "<rootDir>/src/__tests__/setupGlobals.ts",
-    "<rootDir>/src/__tests__/consoleOverrides.ts",
   ],
 
   // A list of paths to modules that run some code to configure or set up the testing framework before each test
@@ -143,18 +142,6 @@ const customJestConfig = {
   //   "/node_modules/"
   // ],
 
-  // The regexp pattern or array of patterns that Jest uses to detect test files
-  // testRegex: [],
-
-  // This option allows the use of a custom results processor
-  // testResultsProcessor: undefined,
-
-  // This option allows use of a custom test runner
-  // testRunner: "jest-circus/runner",
-
-  // This option sets the URL for the jsdom environment. It is reflected in properties such as location.href
-  // testURL: "http://localhost",
-
   // Setting this value to "fake" allows the use of fake timers for functions such as "setTimeout"
   // timers: "real",
 
@@ -164,18 +151,6 @@ const customJestConfig = {
   // An array of regexp pattern strings that are matched against all source file paths, matched files will skip transformation
   transformIgnorePatterns: [],
 
-  // An array of regexp pattern strings that are matched against all modules before the module loader will automatically return a mock for them
-  // unmockedModulePathPatterns: undefined,
-
-  // Indicates whether each individual test should be reported during the run
-  // verbose: undefined,
-
-  // An array of regexp patterns that are matched against all source file paths before re-running tests in watch mode
-  // watchPathIgnorePatterns: [],
-
-  // Whether to use watchman for file crawling
-  // watchman: true,
-
   testTimeout: 10000,
 };
 

next-env.d.ts

@@ -1,5 +1,2 @@
 /// <reference types="next" />
 /// <reference types="next/image-types/global" />
-
-// NOTE: This file should not be edited
-// see https://nextjs.org/docs/basic-features/typescript for more information.

src/__tests__/consoleOverrides.ts

@@ -115,11 +115,39 @@ export class DataController {
     return TemplateService.compile(relationshipSettings.format, data);
   }
 
+  private async getAllowedCrudsFieldsToShow(
+    entity: string,
+    crudKey: "hidden_entity_details_columns"
+  ): Promise<string[]> {
+    const [hiddenFields, entityFields] = await Promise.all([
+      this._configurationService.show<string[]>(crudKey, entity),
+      this._entitiesService.getEntityFields(entity),
+    ]);
+
+    if (hiddenFields.length === 0) {
+      return entityFields.map(({ name }) => name);
+    }
+
+    const hiddenFieldsMap = Object.fromEntries(
+      hiddenFields.map((field) => [field, 1])
+    );
+
+    return entityFields
+      .filter((entityField) => !hiddenFieldsMap[entityField.name])
+      .map(({ name }) => name);
+  }
+
   async showData(entity: string, id: string) {
-    // Send in the show fields
-    return await this._dataService.show(entity, [], {
-      [await this._entitiesService.getEntityPrimaryField(entity)]: id,
-    });
+    return await this._dataService.show(
+      entity,
+      await this.getAllowedCrudsFieldsToShow(
+        entity,
+        "hidden_entity_details_columns"
+      ),
+      {
+        [await this._entitiesService.getEntityPrimaryField(entity)]: id,
+      }
+    );
   }
 
   async createData(entity: string, data: Record<string, unknown>) {

src/backend/data/data.controller.ts

@@ -121,6 +121,11 @@ export class DataService {
     select: string[],
     query: Record<string, unknown>
   ): Promise<T> {
+    if (select.length === 0) {
+      throw new Error(
+        "We dont do that here, Please define the fields you want to select"
+      );
+    }
     return await (await DataService.getInstance())
       .table(entity)
       .select(select)