v1.0.11 (#21)

* More fixes to serialization

* Update changelog, bump version

* Remove console.log

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Change Log
 
+## 1.0.11
+- Extra check for controller instance serialization
+
 ## 1.0.10
 - Don't serialize controllers in diagnostic messages.
 - Several fixes in json schema generator

package.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.0.10",
+  "version": "1.0.11",
   "license": "MIT",
   "main": "dist/index.js",
   "homepage": "https://rolandzwaga.github.io/eligius/",

src/test/unit/util/prepare-value-for-serialization.spec.ts

@@ -6,8 +6,14 @@ import { prepareValueForSerialization } from '../../../util/prepare-value-for-se
 
 const PrepareValueForSerializationSuite = suite('prepareValueForSerialization');
 
+function SimpleClass(this: any) {
+  this.i = 0;
+  return this;
+}
+
 PrepareValueForSerializationSuite('should seriaize the given object', () => {
   // given
+
   const object = {
     a: 1,
     b: $('body'),
@@ -19,6 +25,7 @@ PrepareValueForSerializationSuite('should seriaize the given object', () => {
     f: undefined,
     controllerInstance: new LabelController(),
     someObject: { prop: true },
+    simpleClass: new (SimpleClass as any)(),
   };
 
   // test
@@ -31,8 +38,9 @@ PrepareValueForSerializationSuite('should seriaize the given object', () => {
   expect(result.d).to.eql([1, 'a', 'jQuery object', '(i) => i']);
   expect(result.e).to.be.null;
   expect(result.f).to.be.undefined;
-  expect(result.controllerInstance).to.equal('Complex object');
+  expect(result.controllerInstance).to.equal('class LabelController {');
   expect(result.someObject.prop).to.be.true;
+  expect(result.simpleClass).to.equal('function SimpleClass() {');
 });
 
 PrepareValueForSerializationSuite.run();

src/util/prepare-value-for-serialization.ts

@@ -6,8 +6,12 @@ export function prepareValueForSerialization(value: any): any {
   } else if (value instanceof jQuery) {
     return 'jQuery object';
   } else if (value !== null && typeof value === 'object') {
-    if (value.constructor.toString().substring(0, 5) === 'class') {
-      return 'Complex object';
+    if (
+      value.constructor.toString().substring(0, 5) === 'class' ||
+      !value.constructor.toString().startsWith('function Object()')
+    ) {
+      const funcString: string = value.constructor.toString();
+      return funcString.substring(0, funcString.indexOf('{') + 1);
     } 
     return Object.fromEntries(
       Object.entries(value).map(([propName, propValue]) => [