From 8364d1735380403ad7f5884e0859cca4797a0662 Mon Sep 17 00:00:00 2001
From: Nathan Franklin <nathan.franklin@gmail.com>
Date: Mon, 11 Nov 2024 18:36:22 -0600
Subject: [PATCH] Rename isPublic to isPublicView for clarity

---
 react/src/AppRouter.tsx                       |  2 +-
 .../components/AssetsPanel/AssetsPanel.tsx    | 10 ++++-----
 .../FeatureFileTree/FeatureFileTree.test.tsx  |  6 ++---
 .../FeatureFileTree/FeatureFileTree.tsx       |  6 ++---
 .../ManageMapProjectModal.tsx                 |  6 ++---
 .../MapProjectNavBar.test.tsx                 |  4 ++--
 .../MapProjectNavBar/MapProjectNavBar.tsx     |  6 ++---
 react/src/hooks/features/useFeatures.ts       |  9 ++++----
 react/src/hooks/projects/useProjects.ts       |  6 ++---
 react/src/hooks/tileServers/useTileServers.ts |  8 +++----
 react/src/pages/MapProject/MapProject.tsx     | 22 +++++++++----------
 11 files changed, 43 insertions(+), 42 deletions(-)

diff --git a/react/src/AppRouter.tsx b/react/src/AppRouter.tsx
index dab56191..5fe9e83b 100644
--- a/react/src/AppRouter.tsx
+++ b/react/src/AppRouter.tsx
@@ -65,7 +65,7 @@ function AppRouter() {
             </ProtectedRoute>
           }
         />
-        <Route path={ROUTES.PUBLIC_PROJECT} element={<MapProject isPublic />} />
+        <Route path={ROUTES.PUBLIC_PROJECT} element={<MapProject isPublicView />} />
         <Route path={ROUTES.CALLBACK} element={<Callback />} />
         <Route
           path={ROUTES.STREETVIEW_CALLBACK}
diff --git a/react/src/components/AssetsPanel/AssetsPanel.tsx b/react/src/components/AssetsPanel/AssetsPanel.tsx
index 64989237..333c49f2 100644
--- a/react/src/components/AssetsPanel/AssetsPanel.tsx
+++ b/react/src/components/AssetsPanel/AssetsPanel.tsx
@@ -56,9 +56,9 @@ interface Props {
   featureCollection: FeatureCollection;
 
   /**
-   * Whether or not the map project is public.
+   * Whether or not the map project is a public view.
    */
-  isPublic: boolean;
+  isPublicView: boolean;
 
   /**
    * active project id
@@ -70,7 +70,7 @@ interface Props {
  * A panel component that displays info on feature assets
  */
 const AssetsPanel: React.FC<Props> = ({
-  isPublic,
+  isPublicView,
   featureCollection,
   projectId,
 }) => {
@@ -81,8 +81,8 @@ const AssetsPanel: React.FC<Props> = ({
       </div>
       <div className={styles.middleSection}>
         <FeatureFileTree
-          projectId={projectId}
-          isPublic={isPublic}
+          projectId={project.id}
+          isPublicView={isPublicView}
           featureCollection={featureCollection}
         />
       </div>
diff --git a/react/src/components/FeatureFileTree/FeatureFileTree.test.tsx b/react/src/components/FeatureFileTree/FeatureFileTree.test.tsx
index acf52002..9f9b40f1 100644
--- a/react/src/components/FeatureFileTree/FeatureFileTree.test.tsx
+++ b/react/src/components/FeatureFileTree/FeatureFileTree.test.tsx
@@ -26,7 +26,7 @@ const renderWithRouter = (ui: React.ReactElement, { route = '/' } = {}) => {
 describe('FeatureFileTree', () => {
   const defaultProps = {
     featureCollection: featureCollection,
-    isPublic: false,
+    isPublicView: false,
     projectId: 1,
   };
 
@@ -73,7 +73,7 @@ describe('FeatureFileTree', () => {
 
   it('does not show delete button for public projects', () => {
     const { queryByTestId } = renderWithRouter(
-      <FeatureFileTree {...defaultProps} isPublic={true} />,
+      <FeatureFileTree {...defaultProps} isPublicView={true} />,
       { route: '/?selectedFeature=1' }
     );
 
@@ -84,7 +84,7 @@ describe('FeatureFileTree', () => {
 
   it('does not show delete button when no feature is selected', () => {
     const { queryByTestId } = renderWithRouter(
-      <FeatureFileTree {...defaultProps} isPublic={true} />
+      <FeatureFileTree {...defaultProps} isPublicView={false} />
     );
 
     // Verify delete button is not present
diff --git a/react/src/components/FeatureFileTree/FeatureFileTree.tsx b/react/src/components/FeatureFileTree/FeatureFileTree.tsx
index 1652597f..c41d17a0 100644
--- a/react/src/components/FeatureFileTree/FeatureFileTree.tsx
+++ b/react/src/components/FeatureFileTree/FeatureFileTree.tsx
@@ -31,7 +31,7 @@ interface FeatureFileTreeProps {
   /**
    * Whether or not the map project is public.
    */
-  isPublic: boolean;
+  isPublicView: boolean;
 
   /**
    * active project id
@@ -44,7 +44,7 @@ interface FeatureFileTreeProps {
  */
 const FeatureFileTree: React.FC<FeatureFileTreeProps> = ({
   featureCollection,
-  isPublic,
+  isPublicView,
   projectId,
 }) => {
   const { mutate: deleteFeature, isLoading } = useDeleteFeature();
@@ -169,7 +169,7 @@ const FeatureFileTree: React.FC<FeatureFileTreeProps> = ({
           <FeatureIcon featureType={featureNode.featureType} />
         )}
         <span className={styles.fileName}>{featureNode.name}</span>
-        {!isPublic && isSelected && (
+        {!isPublicView && isSelected && (
           <Button
             size="small"
             type="primary"
diff --git a/react/src/components/ManageMapProjectModal/ManageMapProjectModal.tsx b/react/src/components/ManageMapProjectModal/ManageMapProjectModal.tsx
index 5616709f..507574c0 100644
--- a/react/src/components/ManageMapProjectModal/ManageMapProjectModal.tsx
+++ b/react/src/components/ManageMapProjectModal/ManageMapProjectModal.tsx
@@ -4,11 +4,11 @@ import styles from './ManageMapProjectModal.module.css';
 import { Modal, ModalHeader, ModalBody } from 'reactstrap';
 
 interface ManageMapProjectModalProps {
-  isPublic: boolean;
+  isPublicView: boolean;
 }
 
 const ManageMapProjectModal: React.FC<ManageMapProjectModalProps> = ({
-  isPublic,
+  isPublicView,
 }) => {
   const navigate = useNavigate();
   const location = useLocation();
@@ -24,7 +24,7 @@ const ManageMapProjectModal: React.FC<ManageMapProjectModalProps> = ({
       <ModalHeader toggle={closeModal}>TODO</ModalHeader>
       <ModalBody>
         <div className={styles.root}>
-          Manage Map Project TODO, isPublic: {isPublic}
+          Manage Map Project TODO, isPublicView: {isPublicView}
         </div>
       </ModalBody>
     </Modal>
diff --git a/react/src/components/MapProjectNavBar/MapProjectNavBar.test.tsx b/react/src/components/MapProjectNavBar/MapProjectNavBar.test.tsx
index 633879b7..98d4a4b5 100644
--- a/react/src/components/MapProjectNavBar/MapProjectNavBar.test.tsx
+++ b/react/src/components/MapProjectNavBar/MapProjectNavBar.test.tsx
@@ -7,7 +7,7 @@ describe('MapProjectNavBar', () => {
   it('renders the public nav items for public maps', () => {
     const { getByText, queryByText } = render(
       <BrowserRouter>
-        <MapProjectNavBar isPublic={true} />
+        <MapProjectNavBar isPublicView={true} />
       </BrowserRouter>
     );
     expect(getByText('Assets')).toBeDefined();
@@ -22,7 +22,7 @@ describe('MapProjectNavBar', () => {
   it('renders all nav items for non-public maps', () => {
     const { getByText } = render(
       <BrowserRouter>
-        <MapProjectNavBar isPublic={false} />
+        <MapProjectNavBar isPublicView={false} />
       </BrowserRouter>
     );
 
diff --git a/react/src/components/MapProjectNavBar/MapProjectNavBar.tsx b/react/src/components/MapProjectNavBar/MapProjectNavBar.tsx
index 9898e54a..6d8cdd19 100644
--- a/react/src/components/MapProjectNavBar/MapProjectNavBar.tsx
+++ b/react/src/components/MapProjectNavBar/MapProjectNavBar.tsx
@@ -59,10 +59,10 @@ const navItems: NavItem[] = [
 ];
 
 interface NavBarPanelProps {
-  isPublic?: boolean;
+  isPublicView?: boolean;
 }
 
-const MapProjectNavBar: React.FC<NavBarPanelProps> = ({ isPublic = false }) => {
+const MapProjectNavBar: React.FC<NavBarPanelProps> = ({ isPublicView = false }) => {
   const location = useLocation();
   const queryParams = new URLSearchParams(location.search);
   const activePanel = queryParams.get(queryPanelKey);
@@ -70,7 +70,7 @@ const MapProjectNavBar: React.FC<NavBarPanelProps> = ({ isPublic = false }) => {
   return (
     <div className={styles.root}>
       {navItems
-        .filter((item) => (isPublic ? item.showWhenPublic : true))
+        .filter((item) => (isPublicView ? item.showWhenPublic : true))
         .map((item) => {
           const updatedQueryParams = new URLSearchParams(location.search);
 
diff --git a/react/src/hooks/features/useFeatures.ts b/react/src/hooks/features/useFeatures.ts
index 2d69fc78..7bb0f5a7 100644
--- a/react/src/hooks/features/useFeatures.ts
+++ b/react/src/hooks/features/useFeatures.ts
@@ -4,18 +4,19 @@ import { useGet } from '@hazmapper/requests';
 
 interface UseFeaturesParams {
   projectId: number;
-  isPublic: boolean;
+  isPublicView: boolean;
   assetTypes: string[];
   options?: object;
 }
 
 export const useFeatures = ({
   projectId,
-  isPublic,
+  isPublicView,
   assetTypes,
   options = {},
 }: UseFeaturesParams): UseQueryResult<FeatureCollection> => {
-  const featuresRoute = isPublic ? 'public-projects' : 'projects';
+  // TODO can be reworked as /projects can be used and /public-projects can be removed since we are no longer a WSO2 API
+  const featuresRoute = isPublicView ? 'public-projects' : 'projects';
   let endpoint = `/${featuresRoute}/${projectId}/features/`;
   if (assetTypes?.length) {
     endpoint += `?assetType=${assetTypes.join(',')}`;
@@ -32,7 +33,7 @@ export const useFeatures = ({
 
   const query = useGet<FeatureCollection>({
     endpoint,
-    key: ['activeProjectFeatures', { projectId, isPublic, assetTypes }],
+    key: ['activeProjectFeatures', { projectId, isPublicView, assetTypes }],
     options: { ...defaultQueryOptions, ...options },
   });
   return query;
diff --git a/react/src/hooks/projects/useProjects.ts b/react/src/hooks/projects/useProjects.ts
index 486ced4d..601a0da8 100644
--- a/react/src/hooks/projects/useProjects.ts
+++ b/react/src/hooks/projects/useProjects.ts
@@ -17,16 +17,16 @@ export const useProjects = (): UseQueryResult<Project[]> => {
 
 interface UseProjectParams {
   projectUUID?: string;
-  isPublic: boolean;
+  isPublicView: boolean;
   options: object;
 }
 
 export const useProject = ({
   projectUUID,
-  isPublic,
+  isPublicView,
   options,
 }: UseProjectParams): UseQueryResult<Project> => {
-  const projectRoute = isPublic ? 'public-projects' : 'projects';
+  const projectRoute = isPublicView ? 'public-projects' : 'projects';
   const endpoint = `/${projectRoute}/?uuid=${projectUUID}`;
   const query = useGet<Project>({
     endpoint,
diff --git a/react/src/hooks/tileServers/useTileServers.ts b/react/src/hooks/tileServers/useTileServers.ts
index a5378ef9..7eaccf60 100644
--- a/react/src/hooks/tileServers/useTileServers.ts
+++ b/react/src/hooks/tileServers/useTileServers.ts
@@ -4,21 +4,21 @@ import { TileServerLayer } from '@hazmapper/types';
 
 interface UseTileServerParams {
   projectId?: number;
-  isPublic: boolean;
+  isPublicView: boolean;
   options?: object;
 }
 
 export const useTileServers = ({
   projectId,
-  isPublic,
+  isPublicView,
   options = {},
 }: UseTileServerParams): UseQueryResult<TileServerLayer[]> => {
-  const tileServersRoute = isPublic ? 'public-projects' : 'projects';
+  const tileServersRoute = isPublicView ? 'public-projects' : 'projects';
   const endpoint = `/${tileServersRoute}/${projectId}/tile-servers/`;
 
   const query = useGet<TileServerLayer[]>({
     endpoint,
-    key: ['tile-servers', { projectId, isPublic }],
+    key: ['tile-servers', { projectId, isPublicView }],
     options,
   });
 
diff --git a/react/src/pages/MapProject/MapProject.tsx b/react/src/pages/MapProject/MapProject.tsx
index 8029ba16..8cf18dc3 100644
--- a/react/src/pages/MapProject/MapProject.tsx
+++ b/react/src/pages/MapProject/MapProject.tsx
@@ -19,13 +19,13 @@ interface MapProjectProps {
    * Whether or not the map project is public.
    * @default false
    */
-  isPublic?: boolean;
+  isPublicView?: boolean;
 }
 
 /**
  * A component that displays a map project including initial loading/error components
  */
-const MapProject: React.FC<MapProjectProps> = ({ isPublic = false }) => {
+const MapProject: React.FC<MapProjectProps> = ({ isPublicView = false }) => {
   const { projectUUID } = useParams();
 
   const {
@@ -34,7 +34,7 @@ const MapProject: React.FC<MapProjectProps> = ({ isPublic = false }) => {
     error,
   } = useProject({
     projectUUID,
-    isPublic,
+    isPublicView,
     options: { enabled: !!projectUUID },
   });
 
@@ -61,7 +61,7 @@ const MapProject: React.FC<MapProjectProps> = ({ isPublic = false }) => {
     );
   }
 
-  return <LoadedMapProject isPublic={isPublic} activeProject={activeProject} />;
+  return <LoadedMapProject isPublicView={isPublicView} activeProject={activeProject} />;
 };
 
 interface LoadedMapProject {
@@ -73,7 +73,7 @@ interface LoadedMapProject {
   /**
    * Whether or not the map project is public.
    */
-  isPublic;
+  isPublicView;
 }
 
 /**
@@ -81,7 +81,7 @@ interface LoadedMapProject {
  */
 const LoadedMapProject: React.FC<LoadedMapProject> = ({
   activeProject,
-  isPublic,
+  isPublicView,
 }) => {
   const [selectedAssetTypes, setSelectedAssetTypes] = useState<string[]>(
     Object.keys(assetTypeOptions)
@@ -112,7 +112,7 @@ const LoadedMapProject: React.FC<LoadedMapProject> = ({
     error: featuresError,
   } = useFeatures({
     projectId: activeProject.id,
-    isPublic,
+    isPublicView,
     assetTypes: formattedAssetTypes,
   });
 
@@ -122,7 +122,7 @@ const LoadedMapProject: React.FC<LoadedMapProject> = ({
     error: tileServerLayersError,
   } = useTileServers({
     projectId: activeProject.id,
-    isPublic,
+    isPublicView,
   });
 
   const location = useLocation();
@@ -157,8 +157,8 @@ const LoadedMapProject: React.FC<LoadedMapProject> = ({
           <div className={styles.panelContainer}>
             {activePanel === Panel.Assets && (
               <AssetsPanel
-                projectId={activeProject.id}
-                isPublic={isPublic}
+                project={activeProject}
+                isPublicView={isPublicView}
                 featureCollection={featureCollection}
               />
             )}
@@ -175,7 +175,7 @@ const LoadedMapProject: React.FC<LoadedMapProject> = ({
           </div>
         )}
         {activePanel === Panel.Manage && (
-          <ManageMapProjectModal isPublic={isPublic} />
+          <ManageMapProjectModal isPublicView={isPublicView} />
         )}
         <div className={styles.map}>
           <Map