From 4ebb4341b7ad05e909f87ed7606449db702ac094 Mon Sep 17 00:00:00 2001
From: John Lyon-Smith <john@lyon-smith.org>
Date: Mon, 4 Jun 2018 09:28:03 -0700
Subject: [PATCH] Was allowing existing work item location to be updated

---
 mobile/src/WorkItem/WorkItem.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/mobile/src/WorkItem/WorkItem.js b/mobile/src/WorkItem/WorkItem.js
index eb35329..8f1be6d 100644
--- a/mobile/src/WorkItem/WorkItem.js
+++ b/mobile/src/WorkItem/WorkItem.js
@@ -221,8 +221,9 @@ export class WorkItem extends React.Component {
   }
 
   handleRegionChange(region) {
-    if (!this.isMapReady) {
+    if (this.state.binder._id || !this.isMapReady) {
       // On iOS we get this after setting the initial region, before the map declared as ready!
+      // Also, if we are only viewing the work item, then don't allow region changes
       return
     }
 
@@ -428,7 +429,9 @@ export class WorkItem extends React.Component {
             )}
           {isIphoneX ? <View style={{ height: 30, width: "100%" }} /> : null}
         </ScrollView>
-        <Geolocation onUpdate={this.handlePositionUpdate} watch={false} />
+        {!this.state.binder._id && (
+          <Geolocation onUpdate={this.handlePositionUpdate} watch={false} />
+        )}
         <ProgressModal
           open={!!progressModal}
           message={progressModal ? progressModal.message : ""}