From e6a10cbde26092e7aa2b089fa66c07b9db39ffbc Mon Sep 17 00:00:00 2001
From: Jennifer Chen <shinyi.chen@gmail.com>
Date: Wed, 23 Feb 2022 16:46:55 -0500
Subject: [PATCH] Fix the public library getting into my libraries collection
 bug

---
 src/js/apps/discovery/navigator.js          |  2 +-
 src/js/components/library_controller.js     | 10 ++++++----
 src/js/widgets/library_individual/widget.js |  2 +-
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/js/apps/discovery/navigator.js b/src/js/apps/discovery/navigator.js
index 14e24c4b3..c4d3f25b4 100644
--- a/src/js/apps/discovery/navigator.js
+++ b/src/js/apps/discovery/navigator.js
@@ -417,7 +417,7 @@ define([
           .then(function() {
             app
               .getObject('LibraryController')
-              .getLibraryMetadata(data.id)
+              .getLibraryMetadata(data.id, !data.publicView)
               .done(function(metadata) {
                 data.editRecords =
                   _.contains(
diff --git a/src/js/components/library_controller.js b/src/js/components/library_controller.js
index f69627c77..1e7c46c29 100644
--- a/src/js/components/library_controller.js
+++ b/src/js/components/library_controller.js
@@ -245,11 +245,11 @@ define([
      * about the lib with that id
      * */
 
-    getLibraryMetadata: function(id) {
+    getLibraryMetadata: function(id, merge = true) {
       // check to see if the id is even in the collection,
       // if not return fetchLibraryMetadata;
       if (id && !this.collection.get(id)) {
-        return this.fetchLibraryMetadata(id);
+        return this.fetchLibraryMetadata(id, merge);
       }
       var deferred = $.Deferred();
       var that = this;
@@ -278,7 +278,7 @@ define([
      * in case the new data hasn't been added to the collection yet
      * */
 
-    fetchLibraryMetadata: function(id) {
+    fetchLibraryMetadata: function(id, merge = true) {
       var that = this;
       if (!id) throw new Error('need to provide a library id');
       var deferred = $.Deferred();
@@ -287,7 +287,9 @@ define([
         .done(function(data) {
           deferred.resolve(data.metadata);
           // set into collection
-          that.collection.add(data.metadata, { merge: true });
+          if (merge) {
+            that.collection.add(data.metadata, { merge: true });
+          }
         })
         .fail(function(xhr) {
           deferred.reject(xhr);
diff --git a/src/js/widgets/library_individual/widget.js b/src/js/widgets/library_individual/widget.js
index 617082102..331b7ad9e 100644
--- a/src/js/widgets/library_individual/widget.js
+++ b/src/js/widgets/library_individual/widget.js
@@ -85,7 +85,7 @@ define([
 
       this.getBeeHive()
         .getObject('LibraryController')
-        .getLibraryMetadata(this.model.get('id'))
+        .getLibraryMetadata(this.model.get('id'), !this.model.get('publicView'))
         .done(done);
     },