coderabbitai changes

canopas · Nov 22, 2024 · ab5e252 · ab5e252
1 parent 4f9567c
commit ab5e252
Showing 1 changed file with 41 additions and 11 deletions.
diff --git a/app/src/main/java/com/canopas/yourspace/ui/flow/home/map/component/SelectedUserDetail.kt b/app/src/main/java/com/canopas/yourspace/ui/flow/home/map/component/SelectedUserDetail.kt
@@ -206,15 +206,16 @@ private fun MemberInfoView(
                 iconSize = 24.dp,
                 icon = if (isCurrentUser) R.drawable.ic_share else R.drawable.ic_navigation,
                 onClick = {
+                    if (location == null) return@ActionIconButton
                     if (isCurrentUser) {
                         shareLocation(
                             context,
-                            LatLng(location?.latitude ?: 0.0, location?.longitude ?: 0.0)
+                            LatLng(location.latitude, location.longitude)
                         )
                     } else {
                         openNavigation(
                             context,
-                            LatLng(location?.latitude ?: 0.0, location?.longitude ?: 0.0)
+                            LatLng(location.latitude, location.longitude)
                         )
                     }
                 }
@@ -246,23 +247,52 @@ private fun MemberInfoView(
     }
 }
 
-fun shareLocation(context: Context, location: LatLng) {
-    val mapsLink = "https://www.google.com/maps/dir/?api=1&destination=${location.latitude},${location.longitude}"
+fun shareLocation(context: Context, location: LatLng): Boolean {
+    if (location.latitude < -90 || location.latitude > 90 ||
+        location.longitude < -180 || location.longitude > 180
+    ) {
+        return false
+    }
+
+    val mapsLink =
+        "https://www.google.com/maps/dir/?api=1&destination=${location.latitude},${location.longitude}"
     val intent = Intent(Intent.ACTION_SEND).apply {
         type = "text/plain"
-        putExtra(
-            Intent.EXTRA_TEXT,
-            "Check out my location: $mapsLink"
-        )
+        putExtra(Intent.EXTRA_TEXT, "Check out my location: $mapsLink")
+    }
+    return try {
+        context.startActivity(Intent.createChooser(intent, "Share Location"))
+        true
+    } catch (e: Exception) {
+        e.printStackTrace()
+        false
     }
-    context.startActivity(Intent.createChooser(intent, "Share Location"))
 }
 
-fun openNavigation(context: Context, destination: LatLng) {
+fun openNavigation(context: Context, destination: LatLng): Boolean {
+    if (destination.latitude < -90 || destination.latitude > 90 ||
+        destination.longitude < -180 || destination.longitude > 180
+    ) {
+        return false
+    }
+
     val gmmIntentUri =
         Uri.parse("https://maps.google.com/maps?daddr=${destination.latitude},${destination.longitude}")
     val mapIntent = Intent(Intent.ACTION_VIEW, gmmIntentUri).apply {
         setPackage("com.google.android.apps.maps")
     }
-    context.startActivity(mapIntent)
+    return try {
+        if (mapIntent.resolveActivity(context.packageManager) != null) {
+            context.startActivity(mapIntent)
+            true
+        } else {
+            // If Google Maps is not installed, open in browser
+            val browserIntent = Intent(Intent.ACTION_VIEW, gmmIntentUri)
+            context.startActivity(browserIntent)
+            true
+        }
+    } catch (e: Exception) {
+        e.printStackTrace()
+        false
+    }
 }