From 9e5eb9d258d76969fac2afc8acc72234a45078ea Mon Sep 17 00:00:00 2001
From: Mitchell McCaffrey <mitchemmc@gmail.com>
Date: Wed, 4 Aug 2021 07:42:36 +1000
Subject: [PATCH] Fix locked tokens being selected in tools other than the move
 tool

---
 src/components/konva/Token.tsx | 10 ++++++----
 src/hooks/useMapTokens.tsx     |  5 +++++
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/components/konva/Token.tsx b/src/components/konva/Token.tsx
index e000b0c..9c321dc 100644
--- a/src/components/konva/Token.tsx
+++ b/src/components/konva/Token.tsx
@@ -42,6 +42,7 @@ type MapTokenProps = {
   onTokenDragStart: TokenDragEventHandler;
   onTokenDragEnd: TokenDragEventHandler;
   draggable: boolean;
+  selectable: boolean;
   fadeOnHover: boolean;
   map: Map;
   selected: boolean;
@@ -55,6 +56,7 @@ function Token({
   onTokenDragStart,
   onTokenDragEnd,
   draggable,
+  selectable,
   fadeOnHover,
   map,
   selected,
@@ -153,7 +155,7 @@ function Token({
   }
 
   function handleClick() {
-    if (draggable && tokenRef.current) {
+    if (selectable && tokenRef.current) {
       onTokenMenuOpen(tokenState.id, tokenRef.current);
     }
   }
@@ -165,7 +167,7 @@ function Token({
     if (draggable) {
       setPreventMapInteraction(true);
     }
-    if (tokenState.locked && map.owner === userId) {
+    if (tokenState.locked && selectable) {
       tokenPointerDownTimeRef.current = event.evt.timeStamp;
     }
   }
@@ -174,9 +176,9 @@ function Token({
     if (draggable) {
       setPreventMapInteraction(false);
     }
-    // Check token click when locked and we are the map owner
+    // Check token click when locked and selectable
     // We can't use onClick because that doesn't check pointer distance
-    if (tokenState.locked && map.owner === userId && tokenRef.current) {
+    if (tokenState.locked && selectable && tokenRef.current) {
       // If down and up time is small trigger a click
       const delta = event.evt.timeStamp - tokenPointerDownTimeRef.current;
       if (delta < 300) {
diff --git a/src/hooks/useMapTokens.tsx b/src/hooks/useMapTokens.tsx
index 7968e76..1ce83dd 100644
--- a/src/hooks/useMapTokens.tsx
+++ b/src/hooks/useMapTokens.tsx
@@ -99,6 +99,11 @@ function useMapTokens(
               !(tokenState.id in disabledTokens) &&
               !tokenState.locked
             }
+            selectable={
+              selectedToolId === "move" &&
+              ((!(tokenState.id in disabledTokens) && !tokenState.locked) ||
+                map.owner === userId)
+            }
             fadeOnHover={selectedToolId === "drawing"}
             map={map}
             selected={