fix: crash when delete conversation

Author: lambiengcode

.idea/appInsightsSettings.xml

@@ -1,14 +1,9 @@
 package com.chatgptlite.wanted.data.remote
 
-import android.content.ContentValues
-import android.util.Log
 import com.chatgptlite.wanted.constants.conversationCollection
-import com.chatgptlite.wanted.helpers.DataHolder
 import com.chatgptlite.wanted.models.ConversationModel
-import com.google.firebase.firestore.FieldValue
 import com.google.firebase.firestore.FirebaseFirestore
 import com.google.firebase.firestore.Query
-import com.google.firebase.firestore.QuerySnapshot
 import kotlinx.coroutines.tasks.await
 import javax.inject.Inject
 
@@ -35,42 +30,18 @@ class ConversationRepositoryImpl @Inject constructor(
  }
 
  override suspend fun deleteConversation(conversationId: String) {
- var desiredKey: String? = null
-
- getFireBaseSnapShot().documents.map { documentSnapshot ->
- val id = documentSnapshot.getString("id")
- if (id == conversationId) {
- desiredKey = documentSnapshot.id
- } else {
- null
- }
- }
- DataHolder.docPath = desiredKey.toString()
-
- val docRef = fsInstance
+ fsInstance
  .collection("conversations")
- .document(DataHolder.docPath)
-
- // Remove the 'capital' field from the document
- val updates = hashMapOf<String, Any>(
- "id" to FieldValue.delete(),
- "title" to FieldValue.delete(),
- "createdAt" to FieldValue.delete()
- )
- docRef.update(updates)
- .addOnSuccessListener {
- Log.d(
- ContentValues.TAG,
- "DocumentSnapshot successfully deleted from message!"
- )
- }
- .addOnFailureListener { e ->
- Log.w(
- ContentValues.TAG,
- "Error deleting document", e
- )
+ .whereEqualTo("id", conversationId)
+ .get()
+ .addOnSuccessListener { res ->
+ res.documents.forEach { doc ->
+ // Use the collection and document ID to form the correct path
+ fsInstance.collection("conversations").document(doc.id).delete()
+ }
  }
  }
+
  private suspend fun getFireBaseSnapShot() =
  fsInstance.collection(conversationCollection)
  .orderBy("createdAt", Query.Direction.DESCENDING).get().await()

.idea/deploymentTargetDropDown.xml

@@ -14,7 +14,6 @@ import org.json.JSONObject
 import javax.inject.Inject
 
 
-@Suppress("UNREACHABLE_CODE")
 class OpenAIRepositoryImpl @Inject constructor(
  private val openAIApi: OpenAIApi,
 ) : OpenAIRepository {

.idea/gradle.xml

@@ -24,6 +24,8 @@ import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.filled.*
 import androidx.compose.material.icons.outlined.AddComment
+import androidx.compose.material.icons.outlined.Delete
+import androidx.compose.material.icons.outlined.Message
 import androidx.compose.material3.Divider
 import androidx.compose.material3.Icon
 import androidx.compose.material3.IconButton
@@ -158,7 +160,7 @@ private fun ColumnScope.HistoryConversations(
  items(conversations.size) { index ->
  RecycleChatItem(
  text = conversations[index].title,
- Icons.Filled.Message,
+ Icons.Outlined.Message,
  selected = conversations[index].id == conversationId,
  onChatClicked = {
  onChatClicked(conversations[index].id)
@@ -170,7 +172,6 @@ private fun ColumnScope.HistoryConversations(
  onDeleteClicked = {
  scope.launch {
  conversationViewModel.deleteConversation(conversations[index].id)
- conversationViewModel.deleteMessages(conversations[index].id)
  }
  }
  )
@@ -260,7 +261,7 @@ private fun RecycleChatItem(
  modifier = Modifier
  .height(56.dp)
  .fillMaxWidth()
- .padding(horizontal = 34.dp)
+ .padding(horizontal = 30.dp)
  .clip(CircleShape)
  .then(background)
  .clickable(onClick = onChatClicked),
@@ -293,14 +294,16 @@ private fun RecycleChatItem(
  )
  Spacer(Modifier.weight(0.9f, true))
  Icon(
- imageVector = Icons.Filled.Delete,
+ imageVector = Icons.Outlined.Delete,
  contentDescription = "Delete",
  tint = if (selected) {
  MaterialTheme.colorScheme.primary
  } else {
  MaterialTheme.colorScheme.onSurface
  },
- modifier = Modifier.clickable { onDeleteClicked() }
+ modifier = Modifier.padding(
+ end = 12.dp
+ ).clickable { onDeleteClicked() }
  )
  }
 }

.idea/migrations.xml

@@ -181,21 +181,20 @@ class ConversationViewModel @Inject constructor(
  return response.toList()
  }
 
- fun deleteMessages(conversationId: String) {
+ suspend fun deleteConversation(conversationId: String) {
+ // Delete remote
+ conversationRepo.deleteConversation(conversationId)
 
+ // Delete local
  val conversations: MutableList<ConversationModel> = _conversations.value.toMutableList()
  val conversationToRemove = conversations.find { it.id == conversationId }
 
  if (conversationToRemove != null) {
  conversations.remove(conversationToRemove)
  _conversations.value = conversations
  }
- messageRepo.deleteMessage()
  }
 
- suspend fun deleteConversation(conversationId: String) =
- conversationRepo.deleteConversation(conversationId)
-
  private suspend fun fetchMessages() {
  if (_currentConversation.value.isEmpty() ||
  _messages.value[_currentConversation.value] != null) return