Skip to content

Commit

Permalink
(#768) Resolved duplication bug by removing extra "select collection"…
Browse files Browse the repository at this point in the history
… calls
  • Loading branch information
adkinsrs committed Aug 12, 2024
1 parent 47beec9 commit 56a91bc
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions www/js/dataset_explorer.js
Original file line number Diff line number Diff line change
Expand Up @@ -578,11 +578,8 @@ const addModalEventListeners = () => {

createToast("Display added to collection", "is-success");

const curr_share_id = selected_dc_share_id;

// Update the layout arrangement views
await updateDatasetCollections();
selectDatasetCollection(curr_share_id); // performs DatasetCollectionSelectorCallback when label is set

} catch (error) {
logErrorInConsole(error);
Expand Down Expand Up @@ -615,12 +612,8 @@ const addModalEventListeners = () => {

createToast("Display removed from collection", "is-success");

const curr_share_id = selected_dc_share_id;

// Update the layout arrangement views
await updateDatasetCollections();
selectDatasetCollection(curr_share_id); // performs DatasetCollectionSelectorCallback when label is set

} catch (error) {
logErrorInConsole(error);
createToast("Failed to remove dataset from collection");
Expand Down Expand Up @@ -2196,7 +2189,6 @@ const renderDisplaysModalDisplays = async (displays, collection, displayElt, dat
const displayCount = displayElement.querySelector('.js-collection-display-count');

if (collection?.members) {
console.log(collection.members);
const displayCountValue = collection.members.filter((member) => JSON.parse(member).display_id === displayId).length
displayCount.textContent = displayCountValue;
} else {
Expand Down

0 comments on commit 56a91bc

Please sign in to comment.