From 93f110158fd0861eb5f659a4653db78995c1bda9 Mon Sep 17 00:00:00 2001
From: Kevin Hester <kevinh@geeksville.com>
Date: Fri, 19 Mar 2021 23:14:46 +0800
Subject: [PATCH] preserve even old 1.0 style region names during firmware
 update

---
 .../geeksville/mesh/service/MeshService.kt    | 48 +++++++++++--------
 1 file changed, 27 insertions(+), 21 deletions(-)

diff --git a/app/src/main/java/com/geeksville/mesh/service/MeshService.kt b/app/src/main/java/com/geeksville/mesh/service/MeshService.kt
index 15536e970..b58f90d56 100644
--- a/app/src/main/java/com/geeksville/mesh/service/MeshService.kt
+++ b/app/src/main/java/com/geeksville/mesh/service/MeshService.kt
@@ -171,7 +171,7 @@ class MeshService : Service(), Logging {
         // FIXME - currently we don't support location reading without google play
         if (fusedLocationClient == null && isGooglePlayAvailable(this)) {
             GeeksvilleApplication.analytics.track("location_start") // Figure out how many users needed to use the phone GPS
-            
+
             val request = LocationRequest.create().apply {
                 interval = requestInterval
                 priority = LocationRequest.PRIORITY_HIGH_ACCURACY
@@ -949,13 +949,13 @@ class MeshService : Service(), Logging {
     }
 
     private var locationRequestInterval: Long = 0;
-    private fun setupLocationRequest () {
+    private fun setupLocationRequest() {
         val desiredInterval: Long = if (myNodeInfo?.hasGPS == true) {
             0L  // no requests when device has GPS
-        } else if (numOnlineNodes < 2)  {
+        } else if (numOnlineNodes < 2) {
             5 * 60 * 1000L  // send infrequently, device needs these requests to set its clock
         } else {
-            radioConfig?.preferences?.positionBroadcastSecs?.times( 1000L) ?: 5 * 60 * 1000L
+            radioConfig?.preferences?.positionBroadcastSecs?.times(1000L) ?: 5 * 60 * 1000L
         }
 
         debug("desired location request $desiredInterval, current $locationRequestInterval")
@@ -1288,6 +1288,9 @@ class MeshService : Service(), Logging {
         }
     }
 
+    /// If found, the old region string of the form 1.0-EU865 etc...
+    private var legacyRegion: String? = null
+
     /**
      * Update the nodeinfo (called from either new API version or the old one)
      */
@@ -1301,6 +1304,7 @@ class MeshService : Service(), Logging {
         insertPacket(packetToSave)
 
         rawMyNodeInfo = myInfo
+        legacyRegion = myInfo.region
         regenMyNodeInfo()
 
         // We'll need to get a new set of channels and settings now
@@ -1334,34 +1338,37 @@ class MeshService : Service(), Logging {
             }
 
             if (curRegionValue == RadioConfigProtos.RegionCode.Unset_VALUE) {
-                TODO("Need gui for setting region")
-                /* // look for a legacy region
+                // look for a legacy region
                 val legacyRegex = Regex(".+-(.+)")
-                myNodeInfo?.region?.let { legacyRegion ->
-                    val matches = legacyRegex.find(legacyRegion)
+                legacyRegion?.let { lr ->
+                    val matches = legacyRegex.find(lr)
                     if (matches != null) {
                         val (region) = matches.destructured
                         val newRegion = RadioConfigProtos.RegionCode.valueOf(region)
                         info("Upgrading legacy region $newRegion (code ${newRegion.number})")
                         curRegionValue = newRegion.number
                     }
-                } */
+                }
             }
 
             // If nothing was set in our (new style radio preferences, but we now have a valid setting - slam it in)
             if (curConfigRegion == RadioConfigProtos.RegionCode.Unset && curRegionValue != RadioConfigProtos.RegionCode.Unset_VALUE) {
-                info("Telling device to upgrade region")
+                if (deviceVersion >= minFirmwareVersion) {
+                    info("Telling device to upgrade region")
 
-                // Tell the device to set the new region field (old devices will simply ignore this)
-                radioConfig?.let { currentConfig ->
-                    val newConfig = currentConfig.toBuilder()
+                    // Tell the device to set the new region field (old devices will simply ignore this)
+                    radioConfig?.let { currentConfig ->
+                        val newConfig = currentConfig.toBuilder()
 
-                    val newPrefs = currentConfig.preferences.toBuilder()
-                    newPrefs.regionValue = curRegionValue
-                    newConfig.preferences = newPrefs.build()
+                        val newPrefs = currentConfig.preferences.toBuilder()
+                        newPrefs.regionValue = curRegionValue
+                        newConfig.preferences = newPrefs.build()
 
-                    sendRadioConfig(newConfig.build())
+                        sendRadioConfig(newConfig.build())
+                    }
                 }
+                else
+                    warn("Device is too old to understand region changes")
             }
         }
     }
@@ -1405,14 +1412,13 @@ class MeshService : Service(), Logging {
 
                 regenMyNodeInfo() // we have a node db now, so can possibly find a better hwmodel
                 myNodeInfo = newMyNodeInfo // we might have just updated myNodeInfo
-                
+
                 sendAnalytics()
 
-                if(deviceVersion < minFirmwareVersion) {
+                if (deviceVersion < minFirmwareVersion) {
                     info("Device firmware is too old, faking config so firmware update can occur")
                     onHasSettings()
-                }
-                else
+                } else
                     requestRadioConfig()
             }
         } else