From 874e1ceea29301b25d39476232f59f80b47eb217 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20V=C3=A1clav=C3=ADk?= <jvaclavik@gmail.com>
Date: Wed, 17 Apr 2024 19:51:09 +0200
Subject: [PATCH] climbing: Tick fix (#315)

---
 .../Climbing/RouteList/ExpandedRow.tsx        | 27 ++++++++---------
 .../Climbing/contexts/osmToClimbingRoutes.ts  |  4 +--
 src/services/__tests__/ticks.test.ts          | 19 ++++++++++++
 src/services/ticks.ts                         | 29 +++++++++++++++++--
 4 files changed, 60 insertions(+), 19 deletions(-)
 create mode 100644 src/services/__tests__/ticks.test.ts

diff --git a/src/components/FeaturePanel/Climbing/RouteList/ExpandedRow.tsx b/src/components/FeaturePanel/Climbing/RouteList/ExpandedRow.tsx
index feb32b0c..c331cc5f 100644
--- a/src/components/FeaturePanel/Climbing/RouteList/ExpandedRow.tsx
+++ b/src/components/FeaturePanel/Climbing/RouteList/ExpandedRow.tsx
@@ -15,7 +15,6 @@ import {
 import DeleteIcon from '@material-ui/icons/Delete';
 import ArrowForwardIcon from '@material-ui/icons/ArrowForward';
 import AddIcon from '@material-ui/icons/Add';
-import Router from 'next/router';
 import { RouteDifficultySelect } from '../RouteDifficultySelect';
 import { useClimbingContext } from '../contexts/ClimbingContext';
 import { RouteInDifferentPhotos } from './RouteInDifferentPhotos';
@@ -68,9 +67,6 @@ export const ExpandedRow = ({
     machine.execute('deleteRoute', { routeNumber });
     hideDeleteDialog();
   };
-  const onNodeDetailClick = () => {
-    Router.push(`${getOsmappLink(tempRoute.feature)}${window.location.hash}`);
-  };
 
   return (
     <>
@@ -175,15 +171,20 @@ export const ExpandedRow = ({
                 </ListItem>
               )}
               <ListItem>
-                <Button
-                  onClick={onNodeDetailClick}
-                  color="secondary"
-                  size="small"
-                  variant="text"
-                  endIcon={<ArrowForwardIcon />}
-                >
-                  Show route detail
-                </Button>
+                {tempRoute.feature ? (
+                  <Button
+                    color="secondary"
+                    size="small"
+                    variant="text"
+                    endIcon={<ArrowForwardIcon />}
+                    href={`${getOsmappLink(tempRoute.feature)}${
+                      window.location.hash
+                    }`}
+                    component="a"
+                  >
+                    Show route detail
+                  </Button>
+                ) : null}
               </ListItem>
               <ListItem>
                 <Button
diff --git a/src/components/FeaturePanel/Climbing/contexts/osmToClimbingRoutes.ts b/src/components/FeaturePanel/Climbing/contexts/osmToClimbingRoutes.ts
index 5c1829da..e90567e1 100644
--- a/src/components/FeaturePanel/Climbing/contexts/osmToClimbingRoutes.ts
+++ b/src/components/FeaturePanel/Climbing/contexts/osmToClimbingRoutes.ts
@@ -63,9 +63,7 @@ export const osmToClimbingRoutes = (feature: Feature): Array<ClimbingRoute> => {
     return [];
   }
 
-  const routes = feature.memberFeatures.filter(({ tags }) =>
-    ['route', 'route_bottom'].includes(tags.climbing),
-  );
+  const routes = feature.memberFeatures;
 
   return routes.map((route) => {
     const { paths, photoToKeyMap } = getPathsByImage(route.tags);
diff --git a/src/services/__tests__/ticks.test.ts b/src/services/__tests__/ticks.test.ts
new file mode 100644
index 00000000..761adc25
--- /dev/null
+++ b/src/services/__tests__/ticks.test.ts
@@ -0,0 +1,19 @@
+import { getStorageIndex } from '../ticks';
+
+const ticks = [
+  {
+    osmId: 'n3462908984',
+    date: '2024-04-17T15:59:42.163Z',
+    style: 'PP' as const,
+  },
+  {
+    osmId: 'n3462910811',
+    date: '2024-04-17T16:00:16.911Z',
+    style: 'FS' as const,
+  },
+];
+
+const osmId = 'n3462908984';
+test('getStorageIndex', () => {
+  expect(getStorageIndex(ticks, osmId, 0)).toBe(0);
+});
diff --git a/src/services/ticks.ts b/src/services/ticks.ts
index 411f5b2b..e0a1dd93 100644
--- a/src/services/ticks.ts
+++ b/src/services/ticks.ts
@@ -75,6 +75,28 @@ export const onTickAdd = ({ osmId }) => {
   ]);
 };
 
+export const getStorageIndex = (
+  ticks: Array<Tick>,
+  osmId: string,
+  routeIndex: number,
+): number | null => {
+  const found = ticks?.reduce(
+    (acc, tick, storageIndex) => {
+      if (osmId === tick.osmId) {
+        const newRouteIndex = acc.routeIndex + 1;
+        return {
+          storageIndex:
+            newRouteIndex === routeIndex ? storageIndex : acc.storageIndex,
+          routeIndex: newRouteIndex,
+        };
+      }
+      return acc;
+    },
+    { routeIndex: -1, storageIndex: null },
+  );
+  return found.storageIndex;
+};
+
 export const findTicks = (osmId: string): Array<Tick> => {
   const ticks = getLocalStorageItem(KEY);
 
@@ -90,10 +112,11 @@ export const onTickUpdate = ({
   index: number;
   updatedObject: Partial<Tick>;
 }) => {
-  const routeTicks = findTicks(osmId);
+  const ticks = getLocalStorageItem(KEY);
+  const storageIndex = getStorageIndex(ticks, osmId, index);
   const updatedArray = updateElementOnIndex<Tick>(
-    routeTicks,
-    index,
+    ticks,
+    storageIndex,
     (item) => ({ ...item, ...updatedObject }),
   );
   setLocalStorageItem(KEY, updatedArray);