fixed optional openedPages

Author: e11sy

src/infrastructure/storage/abstract/persistant.ts

@@ -1,6 +1,10 @@
 /* eslint-disable-next-line boundaries/element-types */
 import { SubscribableStore } from './subscribable';
 
+type StoreDataWithOptionalFields<StoreData> = {
+  [Property in keyof StoreData]: StoreData[Property] | undefined
+};
+
 export class PersistantStore<StoreData extends Record<string, unknown>> extends SubscribableStore<StoreData> {
   /**
    * Keys of the StoreData type
@@ -11,7 +15,7 @@ export class PersistantStore<StoreData extends Record<string, unknown>> extends
   /**
    * Proxy for data stored
    */
-  protected data: StoreData;
+  protected data: StoreDataWithOptionalFields<StoreData>;
 
   /**
    * Storage that would retain information when proxy is cleared
@@ -21,15 +25,15 @@ export class PersistantStore<StoreData extends Record<string, unknown>> extends
   constructor(keysStored: string[]) {
     super();
     this.keysStored = keysStored;
-    this.data = new Proxy<StoreData>({} as StoreData, this._data);
+    this.data = new Proxy<StoreDataWithOptionalFields<StoreData>>({} as StoreDataWithOptionalFields<StoreData>, this._data);
 
     /**
      * Load data from local store to proxy
      */
     this.loadInitialData();
   };
 
-  protected get _data(): ProxyHandler<StoreData> {
+  protected get _data(): ProxyHandler<StoreDataWithOptionalFields<StoreData>> {
     return {
       get: (target, prop) => {
         const item = this.storage.getItem(prop as string);

src/infrastructure/storage/openedPage.ts

@@ -21,11 +21,17 @@ export class OpenedPagesStore extends PersistantStore<OpenedPagesStoreData> {
    * @param page - page that had beed opened by user
    */
   public addOpenedPage(page: OpenedPage): void {
-    const uniquePageUrls = this.data.openedPages.map(currentPage => currentPage.url);
+    if (this.data.openedPages !== undefined) {
+      const uniquePageUrls = this.data.openedPages?.map(currentPage => currentPage.url);
 
-    if (!uniquePageUrls.includes(page.url) && page.url !== '/') {
+      if (!uniquePageUrls.includes(page.url) && page.url !== '/') {
+        this.data.openedPages = [
+          ...this.data.openedPages,
+          page,
+        ];
+      }
+    } else {
       this.data.openedPages = [
-        ...this.data.openedPages,
         page,
       ];
     }
@@ -36,7 +42,7 @@ export class OpenedPagesStore extends PersistantStore<OpenedPagesStoreData> {
    * @param url - url of closed page
    */
   public deleteOpenedPageByUrl(url: OpenedPage['url']): void {
-    this.data.openedPages = this.data.openedPages.filter(currentPage => !(currentPage.url == url));
+    this.data.openedPages = this.data.openedPages?.filter(currentPage => !(currentPage.url == url));
   }
 
   /**
@@ -46,7 +52,7 @@ export class OpenedPagesStore extends PersistantStore<OpenedPagesStoreData> {
    * @param page - new data for opened page with certain url
    */
   public patchOpenedPageByUrl(url: OpenedPage['url'], page: OpenedPage): void {
-    this.data.openedPages = this.data.openedPages.map((currentPage) => {
+    this.data.openedPages = this.data.openedPages?.map((currentPage) => {
       if (currentPage.url == url) {
         currentPage.title = page.title;
       }