Re-add persistence

web/src/app/Repository.js

@@ -1,42 +1,41 @@
-import {topicUrl} from "./utils";
 import Subscription from "./Subscription";
+import Subscriptions from "./Subscriptions";
 
 export class Repository {
     loadSubscriptions() {
         console.log(`[Repository] Loading subscriptions from localStorage`);
 
-        const subscriptions = {};
+        const subscriptions = new Subscriptions();
-        const rawSubscriptions = localStorage.getItem('subscriptions');
+        const serialized = localStorage.getItem('subscriptions');
-        if (rawSubscriptions === null) {
+        if (serialized === null) return subscriptions;
-            return {};
+
-        }
         try {
-            const serializedSubscriptions = JSON.parse(rawSubscriptions);
+            const serializedSubscriptions = JSON.parse(serialized);
             serializedSubscriptions.forEach(s => {
                 const subscription = new Subscription(s.baseUrl, s.topic);
-                subscription.notifications = s.notifications;
+                subscription.addNotifications(s.notifications);
-                subscriptions[topicUrl(s.baseUrl, s.topic)] = subscription;
+                subscriptions.add(subscription);
             });
+            console.log(`[Repository] Loaded ${subscriptions.size()} subscription(s) from localStorage`);
             return subscriptions;
         } catch (e) {
-            console.log("LocalStorage", `Unable to deserialize subscriptions: ${e.message}`)
+            console.log(`[Repository] Unable to deserialize subscriptions: ${e.message}`);
-            return {};
+            return subscriptions;
         }
     }
 
     saveSubscriptions(subscriptions) {
-        return;
+        console.log(`[Repository] Saving ${subscriptions.size()} subscription(s) to localStorage`);
-        console.log(`[Repository] Saving subscriptions ${subscriptions} to localStorage`);
 
-        const serializedSubscriptions = Object.keys(subscriptions).map(k => {
+        const serialized = JSON.stringify(subscriptions.map( (id, subscription) => {
-            const subscription = subscriptions[k];
             return {
                 baseUrl: subscription.baseUrl,
                 topic: subscription.topic,
-                notifications: subscription.notifications
+                notifications: subscription.getNotifications(),
+                last: subscription.last
             }
-        });
+        }));
-        localStorage.setItem('subscriptions', JSON.stringify(serializedSubscriptions));
+        localStorage.setItem('subscriptions', serialized);
     }
 }
 

web/src/app/Subscriptions.js

@@ -42,6 +42,10 @@ class Subscriptions {
         return first;
     }
 
+    size() {
+        return this.subscriptions.size;
+    }
+
     clone() {
         const c = new Subscriptions();
         c.subscriptions = new Map(this.subscriptions);

web/src/components/App.js

@@ -153,6 +153,9 @@ const App = () => {
     const toggleDrawer = () => {
         setDrawerOpen(!drawerOpen);
     };
+    useEffect(() => {
+        setSubscriptions(repository.loadSubscriptions());
+    }, [/* initial render only */]);
     useEffect(() => {
         connectionManager.refresh(subscriptions, handleNotification);
         repository.saveSubscriptions(subscriptions);