Fix Mapbox reverse geocoding when address is null

app/src/main/java/org/tasks/extensions/JsonObject.kt

@@ -0,0 +1,9 @@
+package org.tasks.extensions
+
+import com.google.gson.JsonObject
+
+object JsonObject {
+    fun JsonObject.getStringOrNull(key: String): String? = getOrNull(key)?.asString
+
+    fun JsonObject.getOrNull(key: String) = if (has(key)) get(key) else null
+}

app/src/main/java/org/tasks/location/GeocoderMapbox.kt

@@ -13,6 +13,7 @@ import org.tasks.DebugNetworkInterceptor
 import org.tasks.R
 import org.tasks.data.Place
 import org.tasks.data.Place.Companion.newPlace
+import org.tasks.extensions.JsonObject.getStringOrNull
 import org.tasks.preferences.Preferences
 import java.io.IOException
 import javax.inject.Inject
@@ -52,7 +53,10 @@ class GeocoderMapbox @Inject constructor(
                     val types = feature.get("place_type").asStringList
                     val text = feature.get("text").asString
                     name = if (types.contains("address")) {
-                        "${feature.get("address").asString} $text"
+                        feature
+                            .getStringOrNull("address")
+                            ?.let { "$it $text" }
+                            ?: text
                     } else {
                         text
                     }

app/src/main/java/org/tasks/location/GeocoderNominatim.kt

@@ -2,7 +2,6 @@ package org.tasks.location
 
 import android.content.Context
 import com.google.gson.JsonElement
-import com.google.gson.JsonObject
 import com.google.gson.JsonParser
 import dagger.hilt.android.qualifiers.ApplicationContext
 import kotlinx.coroutines.Dispatchers
@@ -12,6 +11,8 @@ import org.tasks.BuildConfig
 import org.tasks.R
 import org.tasks.data.Place
 import org.tasks.data.Place.Companion.newPlace
+import org.tasks.extensions.JsonObject.getOrNull
+import org.tasks.extensions.JsonObject.getStringOrNull
 import org.tasks.http.HttpClientFactory
 import org.tasks.http.HttpException
 import javax.inject.Inject
@@ -63,9 +64,5 @@ class GeocoderNominatim @Inject constructor(
 
         private val JsonElement.asCoordinates: Pair<Double, Double>
             get() = asJsonArray.let { Pair(it[0].asDouble, it[1].asDouble) }
-
-        private fun JsonObject.getStringOrNull(key: String): String? = getOrNull(key)?.asString
-
-        private fun JsonObject.getOrNull(key: String) = if (has(key)) get(key) else null
     }
 }