Finish data migration

src/state/bookmark/store.ts

@@ -1,45 +1,13 @@
 import { versionedStoreBuilder } from 'utils/storage';
 
-/*
-version 0
-{
-    [{scraperid}]: {
-        movie: {
-            [{movie-id}]: {
-                full: {
-                    currentlyAt: number,
-                    totalDuration: number,
-                    updatedAt: number, // unix timestamp in ms
-                    meta: FullMetaObject, // no idea whats in here
-                }
-            }
-        },
-        show: {
-            [{show-id}]: {
-                [{season}-{episode}]: {
-                    currentlyAt: number,
-                    totalDuration: number,
-                    updatedAt: number, // unix timestamp in ms
-                    show: {
-                        episode: string,
-                        season: string,
-                    },
-                    meta: FullMetaObject, // no idea whats in here
-                }
-            }
-        }
-    }
-}
-*/
-
 export const BookmarkStore = versionedStoreBuilder()
-    .setKey('mw-bookmarks')
+  .setKey('mw-bookmarks')
-    .addVersion({
+  .addVersion({
-      version: 0,
+    version: 0,
-      create() {
+    create() {
-        return {
+      return {
-          bookmarks: []
+        bookmarks: []
-        }
       }
-    })
+    }
-    .build()
+  })
+  .build()

src/state/watched/context.tsx

@@ -14,7 +14,7 @@ interface WatchedStoreItem extends MWMediaMeta {
   percentage: number;
 }
 
-interface WatchedStoreData {
+export interface WatchedStoreData {
   items: WatchedStoreItem[];
 }
 
@@ -38,7 +38,7 @@ export function getWatchedFromPortable(
 }
 
 const WatchedContext = createContext<WatchedStoreDataWrapper>({
-  updateProgress: () => {},
+  updateProgress: () => { },
   getFilteredWatched: () => [],
   watched: {
     items: [],

src/state/watched/store.ts

@@ -1,50 +1,6 @@
+import { MWMediaType } from "providers";
 import { versionedStoreBuilder } from "utils/storage";
-
+import { WatchedStoreData } from "./context";
-/*
-version 0
-{
-    [{scraperid}]: {
-        movie: {
-            [{movie-id}]: {
-                full: {
-                    currentlyAt: number,
-                    totalDuration: number,
-                    updatedAt: number, // unix timestamp in ms
-                    meta: FullMetaObject, // no idea whats in here
-                }
-            }
-        },
-        show: {
-            [{show-id}]: {
-                [{season}-{episode}]: {
-                    currentlyAt: number,
-                    totalDuration: number,
-                    updatedAt: number, // unix timestamp in ms
-                    show: {
-                        episode: string,
-                        season: string,
-                    },
-                    meta: FullMetaObject, // no idea whats in here
-                }
-            }
-        }
-    }
-}
-*/
-
-/*
-item = {
-  mediaId: media.mediaId,
-  mediaType: media.mediaType,
-  providerId: media.providerId,
-  title: media.title,
-  year: media.year,
-  percentage: 0,
-  progress: 0,
-  episodeId: media.episodeId,
-  seasonId: media.seasonId,
-};
-*/
 
 export const VideoProgressStore = versionedStoreBuilder()
   .setKey("video-progress")
@@ -54,46 +10,58 @@ export const VideoProgressStore = versionedStoreBuilder()
   .addVersion({
     version: 1,
     migrate(data: any) {
-      const output: any = { items: [] as any };
+      const output: WatchedStoreData = { items: [] };
+
+      if (!data || data.constructor !== Object)
+        return output;
 
       Object.keys(data).forEach((scraperId) => {
         if (scraperId === "--version") return;
         if (scraperId === "save") return;
 
-        if (data[scraperId].movie) {
+        if (data[scraperId].movie && data[scraperId].movie.constructor === Object) {
           Object.keys(data[scraperId].movie).forEach((movieId) => {
-            output.items.push({
+            try {
-              mediaId: movieId.includes("player.php") ? movieId.split("player.php%3Fimdb%3D")[1] : movieId,
+              output.items.push({
-              mediaType: "movie",
+                mediaId: movieId.includes("player.php") ? movieId.split("player.php%3Fimdb%3D")[1] : movieId,
-              providerId: scraperId,
+                mediaType: MWMediaType.MOVIE,
-              title: data[scraperId].movie[movieId].full.meta.title,
+                providerId: scraperId,
-              year: data[scraperId].movie[movieId].full.meta.year,
+                title: data[scraperId].movie[movieId].full.meta.title,
-              progress: data[scraperId].movie[movieId].full.currentlyAt,
+                year: data[scraperId].movie[movieId].full.meta.year,
-              percentage: Math.round((data[scraperId].movie[movieId].full.currentlyAt / data[scraperId].movie[movieId].full.totalDuration) * 100)
+                progress: data[scraperId].movie[movieId].full.currentlyAt,
-            });
+                percentage: Math.round((data[scraperId].movie[movieId].full.currentlyAt / data[scraperId].movie[movieId].full.totalDuration) * 100)
+              });
+            } catch (err) {
+              console.error(`Failed to migrate movie: ${scraperId}/${movieId}`, data[scraperId].movie[movieId]);
+            }
           });
         }
 
-        if (data[scraperId].show) {
+        if (data[scraperId].show && data[scraperId].show.constructor === Object) {
           Object.keys(data[scraperId].show).forEach((showId) => {
+            if (data[scraperId].show[showId].constructor !== Object)
+              return;
             Object.keys(data[scraperId].show[showId]).forEach((episodeId) => {
-              output.items.push({
+              try {
-                mediaId: showId,
+                output.items.push({
-                mediaType: "series",
+                  mediaId: showId,
-                providerId: scraperId,
+                  mediaType: MWMediaType.SERIES,
-                title: data[scraperId].show[showId][episodeId].meta.title,
+                  providerId: scraperId,
-                year: data[scraperId].show[showId][episodeId].meta.year,
+                  title: data[scraperId].show[showId][episodeId].meta.title,
-                percentage: Math.round((data[scraperId].show[showId][episodeId].currentlyAt / data[scraperId].show[showId][episodeId].totalDuration) * 100),
+                  year: data[scraperId].show[showId][episodeId].meta.year,
-                progress: data[scraperId].show[showId][episodeId].currentlyAt,
+                  percentage: Math.round((data[scraperId].show[showId][episodeId].currentlyAt / data[scraperId].show[showId][episodeId].totalDuration) * 100),
-                episodeId: data[scraperId].show[showId][episodeId].show.episode,
+                  progress: data[scraperId].show[showId][episodeId].currentlyAt,
-                seasonId: data[scraperId].show[showId][episodeId].show.season,
+                  episodeId: data[scraperId].show[showId][episodeId].show.episode,
-              });
+                  seasonId: data[scraperId].show[showId][episodeId].show.season,
+                });
+              } catch (err) {
+                console.error(`Failed to migrate series: ${scraperId}/${showId}/${episodeId}`, data[scraperId].show[showId][episodeId]);
+              }
             });
           });
         }
       });
 
-      console.log(output);
       return output;
     },
     create() {

src/utils/storage.ts

@@ -157,19 +157,19 @@ export function versionedStoreBuilder(): any {
         version, // version number
         update: migrate
           ? (data: any) => {
-              // update function, and increment version
+            // update function, and increment version
-              migrate(data);
+            const newData = migrate(data);
-              data["--version"] = version; // eslint-disable-line no-param-reassign
+            newData["--version"] = version; // eslint-disable-line no-param-reassign
-              return data;
+            return newData;
-            }
+          }
           : undefined,
         init: create
           ? () => {
-              // return an initial object
+            // return an initial object
-              const data = create();
+            const data = create();
-              data["--version"] = version;
+            data["--version"] = version;
-              return data;
+            return data;
-            }
+          }
           : undefined,
       };
       return this;