fixes to folder

Author: jaasen-livefront

libs/angular/src/vault/components/folder-add-edit.component.ts

@@ -58,7 +58,7 @@ export class FolderAddEditComponent implements OnInit {
 
   async submit(): Promise<boolean> {
     this.folder.name = this.formGroup.controls.name.value;
-    if (this.folder.name === "") {
+    if (this.folder.name == null || this.folder.name === "") {
       this.toastService.showToast({
         variant: "error",
         title: this.i18nService.t("errorOccurred"),

libs/common/src/models/export/folder.export.ts

@@ -19,7 +19,7 @@ export class FolderExport {
   }
 
   static toDomain(req: FolderExport, domain = new FolderDomain()) {
-    domain.name = req.name ? new EncString(req.name) : undefined;
+    domain.name = new EncString(req.name);
     return domain;
   }
 

libs/common/src/vault/models/domain/folder.ts

@@ -8,26 +8,16 @@ import { FolderData } from "../data/folder.data";
 import { FolderView } from "../view/folder.view";
 
 export class Folder extends Domain {
-  id?: string;
-  name?: EncString;
-  revisionDate: Date;
+  id: string = "";
+  name: EncString = new EncString("");
+  revisionDate: Date = new Date();
 
   constructor(obj?: FolderData) {
     super();
     if (obj == null) {
-      this.revisionDate = new Date();
       return;
     }
 
-    this.buildDomainModel(
-      this,
-      obj,
-      {
-        id: null,
-        name: null,
-      },
-      ["id"],
-    );
     this.name = new EncString(obj.name);
     this.revisionDate = new Date(obj.revisionDate);
   }
@@ -44,7 +34,7 @@ export class Folder extends Domain {
     folderView.id = this.id ?? "";
     folderView.revisionDate = this.revisionDate;
     try {
-      folderView.name = await encryptService.decryptString(this.name ?? new EncString(""), key);
+      folderView.name = await encryptService.decryptString(this.name, key);
     } catch (e) {
       // Note: This should be replaced by the owning team with appropriate, domain-specific behavior.
       // eslint-disable-next-line no-console
@@ -58,10 +48,11 @@ export class Folder extends Domain {
     if (obj == null) {
       return null;
     }
-    return new Folder({
-      name: obj.name ?? "",
-      revisionDate: obj.revisionDate,
-      id: obj.id ?? "",
-    });
+
+    const folder = new Folder();
+    folder.id = obj.id;
+    folder.name = EncString.fromJSON(obj.name);
+    folder.revisionDate = new Date(obj.revisionDate);
+    return folder;
   }
 }

libs/common/src/vault/models/view/folder.view.ts

@@ -10,22 +10,24 @@ export class FolderView implements View, ITreeNodeObject {
   name: string = "";
   revisionDate: Date;
 
-  constructor(f?: Folder | DecryptedObject<Folder, undefined>) {
+  constructor(f?: Folder | DecryptedObject<Folder, "name">) {
     if (!f) {
       this.revisionDate = new Date();
       return;
     }
 
-    this.id = f.id ?? "";
-    this.name = f.name?.decryptedValue ?? "";
+    this.id = f.id;
+    this.name = f.name.toString();
     this.revisionDate = f.revisionDate;
   }
 
   static fromJSON(obj: Jsonify<FolderView>) {
     const folderView = new FolderView();
     folderView.id = obj.id ?? "";
     folderView.name = obj.name ?? "";
-    folderView.revisionDate = new Date(obj.revisionDate);
+    if (obj.revisionDate != null) {
+      folderView.revisionDate = new Date(obj.revisionDate);
+    }
     return folderView;
   }
 }