Fixed problems with instancesByJsName, IdeSettings state is now saved…

… properly

src/v1/utils/Compilation/JsImporter.ts

@@ -46,6 +46,9 @@ export class JsImporter implements IYamlImporter {
 
         const ksyModel = YamlParser.parseIKsyFile(<string>ksyContent, fn);
         Object.assign(this.ksyTypes, SchemaUtils.collectKsyTypes(ksyModel));
-        return ksyModel;
+
+        // we have to modify the schema (add typesByJsName for example) before sending into the compiler, so we need a copy
+        const compilerSchema = YamlParser.parseIKsyFile(<string>ksyContent, fn);
+        return compilerSchema;
     }
 }

src/v1/utils/Compilation/SchemaUtils.ts

@@ -27,14 +27,11 @@ export class SchemaUtils {
     }
 
     static collectKsyTypes(schema: KsySchema.IKsyFile): IKsyTypes {
-        // Function was modifying input and the reason was:
-        // we have to modify the schema (add typesByJsName for example) before sending into the compiler, so we need a copy
-        const schemaTemp = {...schema};
         var types: IKsyTypes = {};
-        SchemaUtils.collectTypes(types, schemaTemp);
+        SchemaUtils.collectTypes(types, schema);
 
-        var typeName = SchemaUtils.ksyNameToJsName(schemaTemp.meta.id, false);
-        types[typeName] = schemaTemp;
+        var typeName = SchemaUtils.ksyNameToJsName(schema.meta.id, false);
+        types[typeName] = schema;
 
         return types;
     }

src/v1/utils/LocalStorageApi.ts

@@ -29,7 +29,10 @@ export class LocalStorageApi {
     }
 
     public static storeIdeSettings = (store: IdeSettings): void => {
-        localStorage.setItem("ideSettings", JSON.stringify(store));
+        const state: IdeSettings = {
+            eagerMode: store.eagerMode
+        };
+        localStorage.setItem("ideSettings", JSON.stringify(state));
     }
 
     public static getDoNotShowWelcomeFlag = (): boolean => {