Merge pull request #37 from aMarcireau/update-all

Update all
aMarcireau · Dec 8, 2017 · 29a8486 · 29a8486
2 parents 5b66280 + f3a4741
commit 29a8486
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/source/package.json b/source/package.json
@@ -1,6 +1,6 @@
 {
     "name": "Origami",
-    "version": "0.7.3",
+    "version": "0.7.4",
     "main": "main.js",
     "private": true,
     "devDependencies": {

diff --git a/source/reducers/publications.js b/source/reducers/publications.js
@@ -231,18 +231,23 @@ export default function publications(state = new Map(), action, appState) {
             }
             const newState = new Map(state);
             for (const doi of updatableDois.values()) {
-                const publication = {
+                newState.set(doi, {
                     ...newState.get(doi),
                     updated: action.timestamp,
                     bibtex: null,
-                };
+                    citers: [],
+                });
+            }
+            const citers = new Set();
+            for (const publication of newState.values()) {
                 for (const citer of publication.citers) {
-                    if (newState.get(citer).status !== PUBLICATION_STATUS_IN_COLLECTION) {
-                        newState.delete(citer);
-                    }
+                    citers.add(citer);
+                }
+            }
+            for (const [doi, publication] of newState.entries()) {
+                if (publication.status === PUBLICATION_STATUS_DEFAULT && !citers.has(doi)) {
+                    newState.delete(doi);
                 }
-                publication.citers = [];
-                newState.set(doi, publication);
             }
             return newState;
         }