From 9aaa159bd094479d0690b93fdbac26c3dd6ca213 Mon Sep 17 00:00:00 2001
From: geeksville <kevinh@geeksville.com>
Date: Mon, 13 Apr 2020 15:42:36 -0700
Subject: [PATCH] We shouldn't ask for myId until _after_ we have the nodedb
 (we need it)

---
 .../main/java/com/geeksville/mesh/MainActivity.kt  | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/app/src/main/java/com/geeksville/mesh/MainActivity.kt b/app/src/main/java/com/geeksville/mesh/MainActivity.kt
index 707e81321..98f7af325 100644
--- a/app/src/main/java/com/geeksville/mesh/MainActivity.kt
+++ b/app/src/main/java/com/geeksville/mesh/MainActivity.kt
@@ -446,9 +446,6 @@ class MainActivity : AppCompatActivity(), Logging,
                 model.radioConfig.value =
                     MeshProtos.RadioConfig.parseFrom(model.meshService!!.radioConfig)
 
-                // Pull down our real node ID
-                model.nodeDB.myId.value = service.myId
-
                 // Update our nodeinfos based on data from the device
                 val nodes = service.nodes.map {
                     it.user?.id!! to it
@@ -456,6 +453,9 @@ class MainActivity : AppCompatActivity(), Logging,
 
                 model.nodeDB.nodes.value = nodes
 
+                // Pull down our real node ID - This must be done AFTER reading the nodedb because we need the DB to find our nodeinof object
+                model.nodeDB.myId.value = service.myId
+
                 // we have a connection to our device now, do the channel change
                 perhapsChangeChannel()
             }
@@ -592,9 +592,11 @@ class MainActivity : AppCompatActivity(), Logging,
         // If we have received the service, and hence registered with
         // it, then now is the time to unregister.
         // if we never connected, do nothing
-        debug("Unbinding from mesh service!")
-        mesh.close()
-        model.meshService = null
+        mesh?.let {
+            debug("Unbinding from mesh service!")
+            it.close()
+            model.meshService = null
+        }
     }
 
     override fun onStop() {