@lenne.tech/nest-server 10.0.5 → 10.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lenne.tech/nest-server",
-  "version": "10.0.5",
+  "version": "10.0.7",
   "description": "Modern, fast, powerful Node.js web framework in TypeScript based on Nest with a GraphQL API and a connection to MongoDB (or other databases).",
   "keywords": [
     "node",

package/src/core/common/helpers/filter.helper.ts

@@ -163,7 +163,7 @@ export function generateFilterQuery<T = any>(
       // Check if value is a string ID and automatic ObjectID filtering is activated
     } else if (config.automaticObjectIdFiltering && checkStringIds(value)) {
       if (field === 'id') {
-        // Replace field name id field with _id and convert value to ObjectId
+        // Replace field name id with _id and convert value to ObjectId
         field = '_id';
         value = getObjectIds(value);
       } else {

package/src/core/common/helpers/input.helper.ts

@@ -1,3 +1,5 @@
+import * as inspector from 'inspector';
+import * as util from 'util';
 import { BadRequestException, UnauthorizedException } from '@nestjs/common';
 import { plainToInstance } from 'class-transformer';
 import { validate } from 'class-validator';
@@ -323,9 +325,8 @@ export function checkAndGetDate(input: any): Date {
  * Clone object
  * @param object Any object
  * @param options Finetuning of rfdc cloning
- * @param options.proto Copy prototype properties as well as own properties into the new object.
- *                      It's marginally faster to allow enumerable properties on the prototype to be copied into the
- *                      cloned object (not onto it's prototype, directly onto the object).
+ * @param options.checkResult Whether to compare object and cloned object via JSON.stringify and try alternative cloning
+ *                            methods if they are not equal
  * @param options.circles Keeping track of circular references will slow down performance with an additional 25% overhead.
  *                        Even if an object doesn't have any circular references, the tracking overhead is the cost.
  *                        By default if an object with a circular reference is passed to rfdc, it will throw
@@ -333,26 +334,48 @@ export function checkAndGetDate(input: any): Date {
  *                        circular references in the object. If performance is important, try removing the circular
  *                        reference from the object (set to undefined) and then add it back manually after cloning
  *                        instead of using this option.
+ * @param options.debug Whether to shoe console.debug messages
+ * @param options.proto Copy prototype properties as well as own properties into the new object.
+ *                      It's marginally faster to allow enumerable properties on the prototype to be copied into the
+ *                      cloned object (not onto it's prototype, directly onto the object).
  */
-export function clone(object: any, options?: { proto?: boolean; circles?: boolean }) {
+export function clone(object: any, options?: { checkResult?: boolean; circles?: boolean; proto?: boolean }) {
   const config = {
-    proto: false,
+    checkResult: true,
     circles: true,
+    debug: inspector.url() !== undefined,
+    proto: false,
     ...options,
   };
+
   try {
-    return rfdc(config)(object);
+    const cloned = rfdc(config)(object);
+    if (config.checkResult && !util.isDeepStrictEqual(object, cloned)) {
+      throw new Error('Cloned object differs from original object');
+    }
+    return cloned;
   } catch (e) {
-    console.debug(e, config, object, 'automatic try to use rfdc with circles');
     if (!config.circles) {
+      if (config.debug) {
+        console.debug(e, config, object, 'automatic try to use rfdc with circles');
+      }
       try {
-        return rfdc({ ...config, ...{ circles: true } })(object);
+        const clonedWithCircles = rfdc({ ...config, ...{ circles: true } })(object);
+        if (config.checkResult && !util.isDeepStrictEqual(object, clonedWithCircles)) {
+          throw new Error('Cloned object differs from original object');
+        }
+        return clonedWithCircles;
       } catch (e) {
-        console.debug(e, 'rfcd with circles did not work automatic use of _.clone!');
-        return _.clone(object);
+        if (config.debug) {
+          console.debug(e, 'rfcd with circles did not work => automatic use of _.clone!');
+        }
+        return _.cloneDeep(object);
       }
     } else {
-      return _.clone(object);
+      if (config.debug) {
+        console.debug(e, config, object, 'automatic try to use _.clone instead rfdc');
+      }
+      return _.cloneDeep(object);
     }
   }
 }

package/src/core/common/services/model-doc.service.ts

@@ -116,7 +116,7 @@ export class ModelDocService implements OnApplicationBootstrap {
    * @param yUmlText
    * @protected
    */
-  protected yUmlToSvg(yUmlText: String) {
+  protected yUmlToSvg(yUmlText: string) {
 
     // Create diagrams
     // see https://github.com/jaime-olivares/yuml-diagram
@@ -125,6 +125,13 @@ export class ModelDocService implements OnApplicationBootstrap {
     const svgLightBg = yuml.processYumlDocument(yUmlText, false);
     const svgDarkBg = yuml.processYumlDocument(yUmlText, true);
 
+    // Save yuml document
+    fs.writeFile('model-doc.yuml', yUmlText, (err) => {
+      if (err) {
+        console.error(err);
+      }
+    });
+
     // Save diagrams
     fs.writeFile('model-doc-light.svg', svgLightBg, (err) => {
       if (err) {