From bf0256857b5e1f53c754e9fe904d56ca196458c6 Mon Sep 17 00:00:00 2001
From: Colin Slater <ctslater@uw.edu>
Date: Thu, 5 Dec 2024 13:44:40 -0300
Subject: [PATCH] Add band selection to visit and tract plots.

Also make the band selector not appear when band is not in the
dimensions.
---
 app/_components/plotPager.js                  | 6 +++++-
 app/tract/[repo]/[collection]/[tract]/page.js | 2 +-
 app/visit/[repo]/[collection]/[visit]/page.js | 2 +-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/_components/plotPager.js b/app/_components/plotPager.js
index 752be38..f4c0bb4 100644
--- a/app/_components/plotPager.js
+++ b/app/_components/plotPager.js
@@ -37,6 +37,10 @@ export default function PlotPager({plotEntries, plotsPerPage = 10}) {
         }
     }, [selectedBands])
 
+    const displayBandSelector = () => {
+        const dimensions = new Set(plotEntries.map((entry) => Object.keys(entry.dataId)).flat())
+        return (dimensions.has('band'))
+    }
 
     const previousPage = () => {
         if(currentPage > 1) {
@@ -99,7 +103,7 @@ export default function PlotPager({plotEntries, plotsPerPage = 10}) {
                     </div>
                 </div>
                 <div className="m-2">
-                    <BandSelector selectedBands={selectedBands} onBandUpdated={onBandUpdated} />
+                    { displayBandSelector() ? <BandSelector selectedBands={selectedBands} onBandUpdated={onBandUpdated} /> : "" }
                 </div>
             </div>
             <div className="flex flex-row flex-wrap justify-center">
diff --git a/app/tract/[repo]/[collection]/[tract]/page.js b/app/tract/[repo]/[collection]/[tract]/page.js
index a1648d1..f045085 100644
--- a/app/tract/[repo]/[collection]/[tract]/page.js
+++ b/app/tract/[repo]/[collection]/[tract]/page.js
@@ -58,7 +58,7 @@ export default async function Collection({params, searchParams}) {
                     <div key={n}>
                         <div className="m-8 text-xl font-medium border-b-2 border-black">{plotGroup}_*</div>
                     <PlotPager plotsPerPage={6} plotEntries={findMatchingPlots(plotEntries, plotGroup).map((entry, n) =>
-                            ({dataId: {}, plot: <PlotDisplay key={n} plotEntry={ ({...entry, repo: repo}) } showDataId={false} showDatasetType={true} /> }))}  />
+                            ({dataId: JSON.parse(entry.dataId), plot: <PlotDisplay key={n} plotEntry={ ({...entry, repo: repo}) } showDataId={false} showDatasetType={true} /> }))}  />
                         <div className="clear-both"></div>
                     </div>
                 )}
diff --git a/app/visit/[repo]/[collection]/[visit]/page.js b/app/visit/[repo]/[collection]/[visit]/page.js
index 3fe1be8..0b3d9dd 100644
--- a/app/visit/[repo]/[collection]/[visit]/page.js
+++ b/app/visit/[repo]/[collection]/[visit]/page.js
@@ -58,7 +58,7 @@ export default async function Collection({params, searchParams}) {
                     <div key={n}>
                         <div className="m-8 text-xl font-medium border-b-2 border-black">{plotGroup}_*</div>
                     <PlotPager plotsPerPage={6} plotEntries={findMatchingPlots(plotEntries, plotGroup).map((entry, n) =>
-                        ({dataId: {}, plot: <PlotDisplay key={n} plotEntry={ ({...entry, repo: repo}) } showDataId={false} showDatasetType={true} />})
+                        ({dataId: JSON.parse(entry.dataId), plot: <PlotDisplay key={n} plotEntry={ ({...entry, repo: repo}) } showDataId={false} showDatasetType={true} />})
                     )} />
                         <div className="clear-both"></div>
                     </div>