From 313382fb82e439b039a9ba50020c15b122d0bf8a Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Wed, 10 Mar 2021 14:40:14 +0200
Subject: [PATCH] Fixed library subscription issues

---
 app/imports/ui/library/LibraryAndNode.vue | 18 +++++++++------
 app/imports/ui/library/LibraryBrowser.vue | 28 +++++++++++++----------
 app/imports/ui/pages/Library.vue          |  4 +++-
 3 files changed, 30 insertions(+), 20 deletions(-)

diff --git a/app/imports/ui/library/LibraryAndNode.vue b/app/imports/ui/library/LibraryAndNode.vue
index 8f8cc1e8..ce3da940 100644
--- a/app/imports/ui/library/LibraryAndNode.vue
+++ b/app/imports/ui/library/LibraryAndNode.vue
@@ -19,7 +19,7 @@
       >
         <v-spacer />
         <v-switch
-          v-if="!$route.params.id || canEditLibrary"
+          v-if="!libraryId || canEditLibrary"
           v-model="organize"
           label="Organize"
           class="mx-3"
@@ -27,11 +27,11 @@
         />
       </v-toolbar>
       <div
-        v-if="$route.params.id"
+        v-if="libraryId"
         style="width: 100%; height: 100%; overflow: auto;"
       >
         <library-contents-container
-          :library-id="$route.params.id"
+          :library-id="libraryId"
           :organize-mode="organize"
           :selected-node-id="selected"
           should-subscribe
@@ -81,6 +81,10 @@ export default {
   },
   props: {
     selection: Boolean,
+    libraryId: {
+      type: String,
+      default: undefined,
+    },
   },
   data(){ return {
     organize: false,
@@ -131,8 +135,8 @@ export default {
   meteor: {
     $subscribe: {
       'library'(){
-        if (this.$route.params.id){
-          return [this.$route.params.id];
+        if (this.libraryId){
+          return [this.libraryId]
         } else {
           return [];
         }
@@ -144,12 +148,12 @@ export default {
       }).fetch();
     },
     library(){
-      let libraryId = this.$route.params.id;
+      let libraryId = this.libraryId;
       if (!libraryId) return;
       return Libraries.findOne(libraryId);
     },
     canEditLibrary(){
-      if (!this.$route.params.id) return;
+      if (!this.libraryId) return;
       try {
         assertEditPermission(this.library, Meteor.userId());
         return true;
diff --git a/app/imports/ui/library/LibraryBrowser.vue b/app/imports/ui/library/LibraryBrowser.vue
index d10beb65..5d5bf7ea 100644
--- a/app/imports/ui/library/LibraryBrowser.vue
+++ b/app/imports/ui/library/LibraryBrowser.vue
@@ -6,13 +6,13 @@
     "
   >
     <v-expansion-panel
-      v-model="expandedLibrary"
       style="box-shadow: none;"
       expand
     >
       <v-expansion-panel-content
         v-for="(library, index) in libraries"
         :key="library._id"
+        v-model="expandedLibrary[index]"
         lazy
         :data-id="library._id"
       >
@@ -87,11 +87,25 @@ export default {
   props: {
     organizeMode: Boolean,
     editMode: Boolean,
-    selectedNodeId: String,
+    selectedNodeId: {
+      type: String,
+      default: undefined,
+    },
   },
   data(){ return {
     expandedLibrary: [],
+    expandedLibraryContent: [],
   };},
+  computed: {
+    noLibrariesExpanded(){
+      if (!this.expandedLibrary) return true;
+      let noneExpanded = true;
+      this.expandedLibrary.forEach(lib => {
+        if(lib) noneExpanded = false;
+      });
+      return noneExpanded;
+    },
+  },
   meteor: {
     $subscribe: {
       'libraries': [],
@@ -106,16 +120,6 @@ export default {
       return tier && tier.paidBenefits;
     },
   },
-  computed: {
-    noLibrariesExpanded(){
-      if (!this.expandedLibrary) return true;
-      let noneExpanded = true;
-      this.expandedLibrary.forEach(lib => {
-        if(lib) noneExpanded = false;
-      });
-      return noneExpanded;
-    },
-  },
   methods: {
     log: console.log,
     insertLibrary(){
diff --git a/app/imports/ui/pages/Library.vue b/app/imports/ui/pages/Library.vue
index 312c7f1c..01a68985 100644
--- a/app/imports/ui/pages/Library.vue
+++ b/app/imports/ui/pages/Library.vue
@@ -1,6 +1,8 @@
 <template lang="html">
   <single-card-layout>
-    <library-and-node />
+    <library-and-node
+      :library-id="$route.params.id"
+    />
   </single-card-layout>
 </template>